Skip to content

Conversation

@Martin-Pitt
Copy link
Contributor

The function llSetGroundTexture does not actually return anything meaningful in the implementation.

According to @HaroldCindy and Monty Linden (per SL discord #scripting) checking the code there were no return values.

If that's the case then we could tweak the definition to match with implementation.

Alternatively, this PR could be rejected and the implementation updated to match with estate management functions e.g. llManageEstateAccess:

Returns a boolean (an integer) TRUE if the call was successful; FALSE if throttled, invalid action, invalid [...] or object owner is not allowed to manage the estate.

Copy link
Contributor

@HaroldCindy HaroldCindy left a comment

Choose a reason for hiding this comment

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

This can't be changed to void because that would break API compatibility, a note should be added to explain that the retval has no meaning for now. We can update the note if / when that changes.

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