Dan Bailey
Dan Bailey
@Idclip - are you able to shed any light on this question?
This looks pretty good to me. I've approved running our CI on this PR so we can see if there are any issues uncovered through doing this on different platforms....
Thanks @gaida-exe, this looks pretty close to being ready to merge. Should we move this out of being a draft PR now?
Looks good to me. If you just make the switch from openvdb::util to openvdb::string and maybe add a line of two of comments to each of these new methods as...
@Idclip - can this one be merged now?
Thanks Kuba! :) I actually have a new convenience class coming (AttributeSet::Info) that I hope will help to alleviate some of the extra work you've had to do here. It's...
This looks fine to me, though there is perhaps a little ambiguity on what whether `prune=false` will override `prunecancelledtiles=true`. There's a missing explicit template instantiation parameter that is causing the...
Yes, this was written pre-C++11 as it used boost::scoped_ptr instead of std::unique_ptr. We can actually deprecate alias declarations, so we should probably mark ScopedPtr as deprecated and add UniquePtr. That...
> It isn't clear from the documentation of the flag that this results in a very different ABI? > > Someone might enable this just to make interprocess/etc optional, and...
> Hey Dan, I'm taking another look at the list of members of the AttributeSet::Array struct and was wondering if adding following would require much work? > > * type...