The following review has been posted through the commitfest application: make installcheck-world: tested, passed Implements feature: tested, passed Spec compliant: not tested Documentation: tested, passed
The v2 patch is IMO ready to commit. The documentation in the form of source comments is updated, and having Catalog::ParseData() do the expansion instead of genbki.pl is more in line with what bki.sgml says: If you want to add a new method of making the data representation smaller, you must implement it in <filename>reformat_dat_file.pl</filename> and also teach <function>Catalog::ParseData()</function> how to expand the data back into the full representation. - ilmari The new status of this patch is: Ready for Committer