Remove redundant if-else in EXPLAIN by using ExplainPropertyText

Поиск
Список
Период
Сортировка
От Ilia Evdokimov
Тема Remove redundant if-else in EXPLAIN by using ExplainPropertyText
Дата
Msg-id 88a71bcd-0b5c-4d0b-8107-757e96f402d5@tantorlabs.com
обсуждение исходный текст
Ответы Re: Remove redundant if-else in EXPLAIN by using ExplainPropertyText
Список pgsql-hackers
Hi hackers,

Thanks to David [0], we found that the if and else branches contained 
equivalent code. Since ExplainPropertyText already handles non-text 
formats, the extra condition is unnecessary.

I reviewed other files related to EXPLAIN where similar patterns might 
exist, but this was the only instance I found.

I’m attaching a patch with the fix. If anyone spots a similar case 
elsewhere, please let me know.

[0]: 
https://www.postgresql.org/message-id/CAApHDvpq2gOt3gnzkd4Ud%3DwrT7YRGrF3zb29jgWzDsYEhETrOA%40mail.gmail.com

--
Best regards,
Ilia Evdokimov,
Tantor Labs LLC.

Вложения

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