Skip to content

Fix compilation warnings from Solaris Developer Studio#267

Merged
michael-grunder merged 1 commit intovalkey-io:mainfrom
michael-grunder:allow-werror-on-solaris
Dec 8, 2025
Merged

Fix compilation warnings from Solaris Developer Studio#267
michael-grunder merged 1 commit intovalkey-io:mainfrom
michael-grunder:allow-werror-on-solaris

Conversation

@michael-grunder
Copy link
Copy Markdown
Collaborator

This lets us build on solaris without disabling -Werror

  • Switches from UL to ULL for very large constants in sds.c
  • Removes break statements when they immediately follow an unconditional goto.

This lets us build on solaris without disabling -Werror

* Switches from `UL` to `ULL` for very large constants in sds.c
* Removes `break` statements when they immediately follow an
  unconditional goto.

Signed-off-by: michael-grunder <michael.grunder@gmail.com>
@michael-grunder
Copy link
Copy Markdown
Collaborator Author

I've got one more after this that adds DeveloperStudio CI coverage and fixes the USE_TLS=1 build for that compiler.

I acquired an oracle cert and key which we can add to secrets to get it running here. It wasn't too bad:
https://github.com/michael-grunder/libvalkey/actions/runs/19952241836

Copy link
Copy Markdown
Collaborator

@bjosv bjosv left a comment

Choose a reason for hiding this comment

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

Nice! LGTM

@michael-grunder michael-grunder merged commit a87338f into valkey-io:main Dec 8, 2025
93 checks passed
@michael-grunder michael-grunder deleted the allow-werror-on-solaris branch December 8, 2025 18:21
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