Introduce enum ERezzedStatus replacing number values for avatar loading status

secondlife / viewer

🖥️ Second Life's official client

GNU Lesser General Public License v2.1

209 stars 54 forks source link

Introduce enum ERezzedStatus replacing number values for avatar loading status #1359

Open LLGuru opened 5 months ago

LLGuru commented 5 months ago

The class LLVOAvatar uses values [-1, 0, 1, 2, 3, 4] to describe the following statuses: initial state, "cloud", "gray", "downloading baked", "loading attachments" and "full"

These values could and should be replaced with an enumeration

Test Plan

start the viewer
wait unril all avaars are fully loaded
ensure that nothing is changed in this workflow after this change

LLGuru commented 5 months ago

Implemented in https://github.com/secondlife/viewer/pull/1361

sergiiiproductengine commented 3 months ago

Passed QA.

Verified on the Second Life Release 7.1.9.9469671545 (64bit) on Win10/OSX in the scope of https://github.com/secondlife/iqa/issues/258.