Re: [pgsql-fr-generale] Re: [pgsql-fr-generale] 9.4 en dev : ALTER SYSTEM SET => généré avec ou sans cotes ?

Merci pour ton analyse,

Par curiosité, comment es-tu arrivé la fonction en cause ?

Bonne journée.
AG.

Le 9 juillet 2014 09:27, Guillaume Lelarge <guillaume(at)lelarge(dot)info> a écrit
:

> Le 9 juillet 2014 08:52, Michael Paquier <michael(dot)paquier(at)gmail(dot)com> a
> écrit :
>
>> TODO
>>
>> 2014-07-02 23:48 GMT+09:00 Guillaume Lelarge <guillaume(at)lelarge(dot)info>:
>> > Le 2 juillet 2014 16:40, Arnaud G <arnaud(dot)gambonnet(at)gmail(dot)com> a écrit
>> :
>> >> Est-ce que c'est une anomalie à remonter ?
>> >>
>> >
>> > Ça peut être remonté. En soit, il n'y a pas d'erreurs, les chaînes
>> doivent
>> > être entre guillemets simples. Ça ne veut pas dire que les autres ne
>> peuvent
>> > pas l'être :)
>>
>> En regardant le code, celà vient de write_auto_conf_file(at)guc(dot)c où des
>> guillemets sont utilisés pour toutes les variables:
>> appendStringInfoString(&buf, "\'");
>> escaped = escape_single_quotes_ascii(item->value);
>> appendStringInfoString(&buf, escaped);
>> free(escaped);
>> appendStringInfoString(&buf, "\'");
>> Eliminer ces guillemets est purement cosmétique comme Guillaume l'a
>> déjà mentionné, et engendrerait des changements plus compliqué qu'au
>> premier abord comme ConfigVariable ne joue pas du tout avec les
>> concepts de type de variable GUC... Juste pour cette raison je ne vois
>> pas que de tels changements vaudraient le coup, et la fonctionnalité
>> n'est pas impactée.
>>
>
> Merci d'avoir regardé dans le code, j'avoue que j'avais la flemme :)
>
> Très intéressant en tout cas.
>
>
> --
> Guillaume.
> http://blog.guillaume.lelarge.info
> http://www.dalibo.com
>