Skip to content

Conversation

Martin-Pitt
Copy link
Contributor

No description provided.

@HaroldCindy
Copy link
Contributor

@Rider-Linden @bennettgoble thoughts?

Copy link

@Rider-Linden Rider-Linden left a comment

Choose a reason for hiding this comment

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

llSetLinkPrimitiveParamsFast does everything that both those functions do, and in fact all three go to the same code under the hood.

If we pair this with a documentation update. Something along the lines of:

Use llSetLinkPrimitiveParamsFast rather than this function.

Then I think it's a good idea. (Does the lua compiler issue a warning for deprecated functions?)

@HaroldCindy
Copy link
Contributor

@Rider-Linden It doesn't, but there's no reason it couldn't, so long as there's no aliasing (i.e. you do ll.SomeFunc() rather than local foo = ll; foo.SomeFunc()

…llSetLinkPrimitiveParams`

Replaces the tooltips on `llSetPrimitiveParams` and `llSetLinkPrimitiveParams` to mention that they are deprecated and to use `llSetLinkPrimitiveParamsFast` instead. This tooltip matches the same deprecation message as used on llSetPrimURL
@Martin-Pitt
Copy link
Contributor Author

@Rider-Linden I've now replaced the tooltips as

Deprecated: Use llSetLinkPrimitiveParamsFast instead.

Which is similar to the warning used on the tooltip of the deprecated llSetPrimURL (which asks to use llSetPrimMediaParams instead)

@Rider-Linden
Copy link

@Rider-Linden It doesn't, but there's no reason it couldn't, so long as there's no aliasing (i.e. you do ll.SomeFunc() rather than local foo = ll; foo.SomeFunc()

It seems like if you do that you deserve what you get.

Copy link

@Rider-Linden Rider-Linden left a comment

Choose a reason for hiding this comment

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

This should encourage newer scripters to use the more modern function, and they will live happier lives because if it. Approved.

@HaroldCindy HaroldCindy merged commit 6fd87c0 into secondlife:main Sep 8, 2025
3 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Sep 8, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants