Skip to content

Fix building RelWithDebInfo due to undeclared identifier isnan#230

Open
DarlCat wants to merge 1 commit into
FirestormViewer:masterfrom
DarlCat:fix/xenhat-RelWithDebInfo-fix
Open

Fix building RelWithDebInfo due to undeclared identifier isnan#230
DarlCat wants to merge 1 commit into
FirestormViewer:masterfrom
DarlCat:fix/xenhat-RelWithDebInfo-fix

Conversation

@DarlCat
Copy link
Copy Markdown
Contributor

@DarlCat DarlCat commented May 12, 2026

Simple swap of isnan to llisnan to fix building RelWithDebinfo on Linux. Specifically with Clang 22.1.3

Contributed with permission and blessing from @XenHat

indra/llinventory/llsettingsbase.cpp:767:15: error: use of undeclared identifier 'isnan'
  767 |     llassert(!isnan(blendf));
      |               ^~~~~

Signed-off-by: XenHat <me@xenh.at>
@DarlCat
Copy link
Copy Markdown
Contributor Author

DarlCat commented May 12, 2026

Added note: this is going to conflict with linden's develop-linux branch anyway, since that has a std::isnan() pass which is currently missing in FS. Felt a code tag may just be more disruptive to that than the conflict itself.

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants