Remove redundant if-else in EXPLAIN by using ExplainPropertyText

2025-04-05 Thread Ilia Evdokimov
2ae2c73f4ed8ddce11d2a18474cb2e9e4fe75c6d Mon Sep 17 00:00:00 2001 From: Evdokimov Ilia Date: Thu, 20 Mar 2025 23:18:38 +0300 Subject: [PATCH v1] Remove redundant if-else in EXPLAIN by using ExplainPropertyText --- src/backend/commands/explain.c | 15 +++ 1 file changed, 3 insertions

Re: Remove redundant if-else in EXPLAIN by using ExplainPropertyText

2025-03-22 Thread David Rowley
On Fri, 21 Mar 2025 at 09:24, Ilia Evdokimov wrote: > 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 simil

Re: Remove redundant if-else in EXPLAIN by using ExplainPropertyText

2025-03-20 Thread David Rowley
On Fri, 21 Mar 2025 at 10:12, David Rowley wrote: > The patch looks good to me and seems worth applying to master. Ok. Pushed. David