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

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