From: | Michael Paquier <michael(dot)paquier(at)gmail(dot)com> |
---|---|
To: | Guillaume Lelarge <guillaume(at)lelarge(dot)info> |
Cc: | Arnaud G <arnaud(dot)gambonnet(at)gmail(dot)com>, pgsql-fr-generale <pgsql-fr-generale(at)postgresql(dot)org> |
Subject: | Re: [pgsql-fr-generale] Re: [pgsql-fr-generale] 9.4 en dev : ALTER SYSTEM SET => généré avec ou sans cotes ? |
Date: | 2014-07-09 06:52:31 |
Message-ID: | CAB7nPqTemzEXRkrqiy+2WsUufjcYxHwNLaURjDBZW7EebF6Deg@mail.gmail.com |
Views: | Whole Thread | Raw Message | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-fr-generale |
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.
--
Michael
--
Envoi via la liste pgsql-fr-generale (pgsql-fr-generale(at)postgresql(dot)org)
From | Date | Subject | |
---|---|---|---|
Next Message | Guillaume Lelarge | 2014-07-09 07:27:09 | Re: [pgsql-fr-generale] Re: [pgsql-fr-generale] 9.4 en dev : ALTER SYSTEM SET => généré avec ou sans cotes ? |
Previous Message | Rodolphe Quiédeville | 2014-07-04 07:04:21 | Re: Update et effet sur les INDEX |