Re: Remove useless casts to (void *)

Aleksander Alekseev <aleksander@tigerdata.com>

From: Aleksander Alekseev <aleksander@tigerdata.com>

To: pgsql-hackers@lists.postgresql.org

Cc: Bertrand Drouvot <bertranddrouvot.pg@gmail.com>, Peter Eisentraut <peter@eisentraut.org>

Date: 2025-11-20T15:07:35Z

Lists: pgsql-hackers

Commits

Same data as JSON: GET /api/v1/messages/:b64id/commits the thread's linked commits as JSON, with link sources. API reference →

Remove useless casts to (void *)
- ef8fe693606a 19 (unreleased) landed

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