Re: [PATCH] Add pretty-printed XML output option

Поиск
Список
Период
Сортировка
От Andrey Borodin
Тема Re: [PATCH] Add pretty-printed XML output option
Дата
Msg-id CAAhFRxjM4XZeQhKTfSKeQSyMAK6xssOu6UCT2rmVkBfi4NtmJQ@mail.gmail.com
обсуждение исходный текст
Ответ на Re: [PATCH] Add pretty-printed XML output option  (Jim Jones <jim.jones@uni-muenster.de>)
Ответы Re: [PATCH] Add pretty-printed XML output option  (Jim Jones <jim.jones@uni-muenster.de>)
Список pgsql-hackers
On Thu, Feb 16, 2023 at 2:12 PM Jim Jones <jim.jones@uni-muenster.de> wrote:
>
> I'm squashing v12-0001 and v12-0002 (v13 attached).
I've looked into the patch. The code looks to conform to usual expectations.
One nit: this comment should have just one asterisk.
+ /**
And I have a dumb question: is this function protected from using
external XML namespaces? What if the user passes some xmlns that will
force it to read namespace data from the server filesystem? Or is it
not possible? I see there are a lot of calls to xml_parse() anyway,
but still...

Best regards, Andrey Borodin.



В списке pgsql-hackers по дате отправления:

Предыдущее
От: Andrey Borodin
Дата:
Сообщение: psql \watch 2nd argument: iteration count
Следующее
От: Andres Freund
Дата:
Сообщение: Re: Introduce list_reverse() to make lcons() usage less inefficient