Matthew Russell
Matthew Russell
Hi @goncaloalmeida, PR is rebased, and all the tests pass.
Tweaked the uid/gid deserialization. @goncaloalmeida
This PR does not address all the warnings, but it does address a good number of them
There's no strictly compelling reason. However it does provide advantages such as `constexpr`. We use AOSP (C++20) and QNX 7.1 (C++17), this commit was first created for 3.1.20 which actually...
> @kheaactua with force push it is not easy for the reviews and this will take more time. Oh, sorry about that. It's SOP at work so I suppose just...
> Hi @kheaactua, can you have a look at the failing windows build? We are interested in merging this PR. Thanks Will do! I thought I had but I suppose...
> Hi @kheaactua, can you have a look at the failing windows build? We are interested in merging this PR. Thanks @fcmonteiro All tests pass!
> > @kheaactua I tried to create a draft PR with all the changes as per this PR and looks like there is compatibility issue with boost-1.65.0 and the build...
> @kheaactua I tried to create a draft PR with all the changes as per this PR and looks like there is compatibility issue with boost-1.65.0 and the build fails...
> > Clang 17.0.6 on Linux > > CXX compiler identification is GNU 11.4.0 Great, I'll spin up a container and try the build again