<html><head><meta http-equiv="Content-Type" content="text/html; charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class=""><br class=""><div><br class=""><blockquote type="cite" class=""><div class="">On Jun 25, 2019, at 2:14 AM, Sandro Santilli <<a href="mailto:strk@kbt.io" class="">strk@kbt.io</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div class="">On Sat, Jun 22, 2019 at 08:37:48AM -0700, Paul Ramsey wrote:<br class=""><blockquote type="cite" class="">On Sat, Jun 22, 2019 at 1:05 AM Sandro Santilli <<a href="mailto:strk@kbt.io" class="">strk@kbt.io</a>> wrote:<br class=""><blockquote type="cite" class=""><br class="">Where's the serialized version info, btw ?<br class=""></blockquote><br class="">In the end of the gflags. Check the contents of gserialized.h and gserialized2.h<br class=""></blockquote><br class=""><br class="">I understand this is being done a bit in rush, but I think we would<br class="">benefit from a clear document about the new format. </div></div></blockquote><div><br class=""></div><div>Please see if the updated <a href="https://github.com/pramsey/postgis/blob/svn-trunk-gserialized2/liblwgeom/gserialized.txt" class="">https://github.com/pramsey/postgis/blob/svn-trunk-gserialized2/liblwgeom/gserialized.txt</a> addresses your questions now?</div><div><br class=""></div><blockquote type="cite" class=""><div class=""><div class=""><br class="">I love the "G2FLAG_UNUSED", for this matter (maybe it could be renamed<br class="">"G2FLAG_RESERVED" to clarify that we plan to use it in the future but<br class="">current code should ensure that bit is always clear now).<br class=""></div></div></blockquote><div><br class=""></div><div>Unfortunately there is only one spare, even after version change, so how to use it is a one-time (until v3) choice. LightPoint is tempting, and a feature that requires use of a bit in the primary flags slot, but I’m not sure if the 25% space savings is such a killer feature or not.</div><br class=""><blockquote type="cite" class=""><div class=""><div class="">I'm not sure I understand the use of 2 bits for the version, since we<br class="">currently only have 2 versions (so a single bit should do).<br class=""></div></div></blockquote><div><br class=""></div><div>If we take away the second version bit, this is the only version update we can do without starting to eat space just because the version is “newer”. Keeping the second bit gives us room to have two more versions without having to commit to a larger header. Given our 10-year revision cycle and the amount of room in the extended flags space, this may be a false economy, but we can always choose to steal that second version bit in the future if we so desire.</div><div><br class=""></div><div>P.</div></div><br class=""></body></html>