Skip to content

bool-semantics for function parameters#33

Merged
HaroldCindy merged 1 commit intosecondlife:mainfrom
WolfGangS:feature/parameter-bool-semantics
Dec 18, 2025
Merged

bool-semantics for function parameters#33
HaroldCindy merged 1 commit intosecondlife:mainfrom
WolfGangS:feature/parameter-bool-semantics

Conversation

@WolfGangS
Copy link
Contributor

@WolfGangS WolfGangS commented Dec 9, 2025

This adds the bool-semantics property for function parameters if they are integer values that operate with a 0 - off else on mechanic.

Resolves #32

I have added the needed yamale changes for schema validation. If #30 is completed before or after this adjustments will be needed in either feature.

@buildersbrewery

This comment was marked as resolved.

@WolfGangS
Copy link
Contributor Author

If #30 is merged first, you can see that:

* `bool-semantics` there is optional

* but if it is added as key, it's value has to be `true`

That looks like its for function return type.

This pr is for function parameters and would require bool-semantics to be added in here.

buildersbrewery added a commit to buildersbrewery/lsl-definitions that referenced this pull request Dec 12, 2025
address pull request comment on secondlife#33
@HaroldCindy HaroldCindy reopened this Dec 18, 2025
Copy link
Collaborator

@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.

Looks good to me, just has to be rebased on top of main

Signed-off-by: WolfGangS <flamin2k8@gmail.com>
@WolfGangS WolfGangS force-pushed the feature/parameter-bool-semantics branch from 0539f08 to 0c6e7b0 Compare December 18, 2025 15:54
@HaroldCindy HaroldCindy merged commit 8b13b96 into secondlife:main Dec 18, 2025
4 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Dec 18, 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.

bool-semantics for function integer parameters

3 participants