attached is version 8, fixing remaining issues, adding docs and tests as requested/agreed.
Marius & Arne
--- Marius Timmer Zentrum für Informationsverarbeitung Westfälische Wilhelms-Universität Münster Einsteinstraße 60
mtimm_01@uni-muenster.de
Am 14.01.2015 um 17:42 schrieb Arne Scheffer <scheffa@uni-muenster.de>:
> Hi, > > we will also remove the following is lc_collate hint in the next version, showing only mandatory info as suggested. > > /* for those who use COLLATE although their default is already the wanted */ > if (strcmp(collname, localeptr) == 0) > { > appendStringInfo(sortorderInformation, " (%s is LC_COLLATE)", collname); > } > > Anybody insisting on that? > > Arne > > Note: I see, at the moment we use the wrong default for DESC. We'll fix that. > > On Wed, 14 Jan 2015, Heikki Linnakangas wrote: > >> On 01/14/2015 05:26 PM, Timmer, Marius wrote: >>> Hello Heikki, >>> abbreviated version: >>> Sorry, the problem is only the unhandy patch text format, not different opinions how to proceed. >>> Long version: >>> The v7 patch file already addressed your suggestions, >>> but the file contained serveral (old) local commits, >>> the new ones at the end of the patch text/file. >> >> Ah, missed that. I stopped reading when I saw the old stuff there :-). >> >>> v7.1 is attached and addresses this issue providing a clean patch file. >> >> Ok, thanks, will take a look. >> >>> V8 will - as mentioned - add missing docs and regression tests, >> >> Great! >> >> - Heikki >> >>