<meta http-equiv="Content-Type" content="text/html; charset=us-ascii"><div id="geary-body" dir="auto"><div>I've mostly finished going over the build-system, but I had a few last</div><div>questions. The mailing-list seemed more appropriate since I don't think</div><div>they're necessarily issues with Poppler:</div><div><br></div><div>1. If I try to build with clang & sanitizers enabled, GObject-</div><div>introspection (all sanitizers) fails to link. There's already a known</div><div>asan error with GObject-introspection that's tracked upstream at:</div><div><a href="https://gitlab.gnome.org/GNOME/gobject-introspection/-/issues/176">https://gitlab.gnome.org/GNOME/gobject-introspection/-/issues/176</a></div><div><br></div><div>However, if I configure to build with clang, I don't even reach that</div><div>point. My logs show g-ir-scanner trying to link with gcc anyways, then</div><div>failing to find the sanitizer libraries.</div><div><br></div><div>* I'm leaning towards g-ir-scanner being the cause, but could it be a</div><div>Poppler issue? I didn't notice any obvious causes while skimming the</div><div>CMake files. Google didn't turn up clear reports of this override</div><div>happening anywhere else though.</div><div><br></div><div>2. The GLib tests also fail at linking if ubsan is enabled. The cause is</div><div>that some of the triggered ubsan checks are only provided in the C++</div><div>sanitizer library, while CMake defaults to linking the C library once it</div><div>sees the tests are .c files.</div><div><br></div><div>It only takes one line setting `LINKER_LANGUAGE` to "CXX" for those</div><div>targets to build successfully, but that seems too brittle to count as a</div><div>solution, unless it's applied to the entire build tree.</div><div><br></div><div>* At that point, I'm thinking it's more of an issue with extra-cmake-</div><div>modules for not checking & handling this behavior.</div><div><br></div><div>3. For the project generally, how much appetite is there for minor code</div><div>refactoring? Do you all prefer "if it ain't broke, don't fix it", or do</div><div>you typically accept minor changes to streamline the code?</div><div><br></div><div>Kyle</div></div>