| From: | Aleksander Alekseev <aleksander(at)tigerdata(dot)com> |
|---|---|
| To: | pgsql-hackers(at)lists(dot)postgresql(dot)org |
| Cc: | Bertrand Drouvot <bertranddrouvot(dot)pg(at)gmail(dot)com>, Peter Eisentraut <peter(at)eisentraut(dot)org> |
| Subject: | Re: Remove useless casts to (void *) |
| Date: | 2025-11-20 15:07:35 |
| Message-ID: | CAJ7c6TNGRQDhY476XrbNaC1=JJk1YokrKaRd7vzX-eiHuM5DdQ@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Hi,
> > Personally I don't think this is a good change.
>
> Only this one (because maybe databuf is used twice) or the whole idea of 7f798aca1d5
> and this patch?
The whole idea to be honest. This being said, this is just my personal
opinion as of today. The majority of the community may disagree and/or
have good arguments to do this I'm not aware of or failed to
understand and/or just have other sense of beauty. And that's OK. IMO
what actually is important is for the code to be consistent. That's as
I understand what your patch is trying to accomplish.
Let's see what other people think.
--
Best regards,
Aleksander Alekseev
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Ashutosh Bapat | 2025-11-20 15:13:16 | Re: SQL Property Graph Queries (SQL/PGQ) |
| Previous Message | Ashutosh Bapat | 2025-11-20 15:00:53 | Re: SQL Property Graph Queries (SQL/PGQ) |