Patrick Steinhardt <ps@xxxxxx> writes: > Let me use this to give a quick status update regarding my upstream > quest to address the feedback I got during reviews on the clar itself: > > - There is a .editorconfig file now. > > - All the cross-platform compatibility fixes have been merged. > > - We have Win32 wired up in CI. Doing so via Makefiles was too much of > a hassle, so I converted the project to use CMake for easier cross > platform testability. The fact that the project uses CMake does not > impact us though, as we wire it up ourselves anyway. > > - All memory allocation errors are now handled consistently. > > Currently in review is: > > - Self-tests for the clar, where we use clar to assert that clar > works. > > - A small memory leak fix, as well as wiring up leak sanitizers in CI. > > I've also got a patch series sitting locally that introduces type-safe > wrappers for the assertions that I'll move into review once self-tests > have landed. That would then address the last bit of feedback I got, if > I remember correctly. > > Just to let you folks know that I didn't just do nothing after this has > landed in Git. ;-) Nice to see that the code is improved not just for us but for other consumers. Thanks.
