Remove the only use of boost::thread_specific_ptr

fpsunflower commented 2 months ago

Description

This had already been marked as deprecated, and there is concensus that the overhead does not outweight the slight inconvenience of creating one object per thread

Tests

Testsuite passes on my machine.

Checklist:

[x] I have read the contribution guidelines.
[x] I have updated the documentation, if applicable.
[x] I have ensured that the change is tested somewhere in the testsuite (adding new test cases if necessary).
[x] My code follows the prevailing code style of this project. If I haven't already run clang-format v17 before submitting, I definitely will look at the CI test that runs clang-format and fix anything that it highlights as being nonconforming.

lgritz commented 2 months ago

The Mac CI failures appear to be because Homebrew just upgraded to llvm 18 and we apparently have some fixups to do. They aren't related to your PR.

fpsunflower commented 2 months ago

Any notes on this one? I have the remaining boost removal changes queued up after this.

AcademySoftwareFoundation / OpenShadingLanguage