Skip to content

[RFC] Adding tables without constants - #184

Draft
Suzanna-Linn wants to merge 2 commits into
secondlife:mainfrom
Suzanna-Linn:Suzanna-Linn-tables-texts
Draft

Suzanna-Linn wants to merge 2 commits into
secondlife:mainfrom
Suzanna-Linn:Suzanna-Linn-tables-texts

Conversation

@Suzanna-Linn

@Suzanna-Linn Suzanna-Linn commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

Adding enums of tables that haven't got constants defined.

SimulatorDataRating and SimulatorDataStatus are used in #185.

Adding enums of tables that haven't got constants defined.
@Suzanna-Linn
Suzanna-Linn marked this pull request as draft July 21, 2026 14:43
@Suzanna-Linn Suzanna-Linn changed the title Adding tables without constants [RFC] Adding tables without constants Jul 21, 2026
Comment thread lsl_definitions.yaml Outdated
type: enum
tooltip: Names of the animation states
value-type: string
other-values: true

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no other values

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"ALL" should be another documented value. then you won't need other values

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The main reason to add "other-values" is the llGetAnimation() caveat in Wiki:

  • "New return values could conceivably be added at any time and this list may not in-fact be complete. Scripts should be written under the assumption that they may receive a value they won't recognize."

Also llGetAnimation() has two more tables with possible values.

"ALL" is only used in llResetAnimationOverride().

Comment thread lsl_definitions.yaml Outdated
Comment thread lsl_definitions.yaml Outdated
Comment on lines +678 to +679
tooltip: Get the time the region was last (re)started, in llGetUnixTime format
details: "Typecast to: integer"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What's the purpose of having both "tooltip" and "details"? I feel like they could be combined, unless you have specific purposes in mind for each one

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • "tooltip" is the tooltip itself.
  • "details" has the other columns in the Wiki tables, formatted as "column name: cell content, ...", also used in OBJECT_* and PRIM_*.

Using two fields to preserve the Wiki structure and because they have different format.

Comment thread lsl_definitions.yaml Outdated
type: enum
tooltip: Names of the animation states
value-type: string
other-values: true

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"ALL" should be another documented value. then you won't need other values

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants