| From: | Neil Conway <neilc(at)samurai(dot)com> |
|---|---|
| To: | pgsql-patches <pgsql-patches(at)postgresql(dot)org> |
| Subject: | void * cast cleanup |
| Date: | 2004-09-26 23:52:45 |
| Message-ID: | 1096242764.25688.689.camel@localhost.localdomain |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-patches |
This patch removes a few instances where a pointer type was being
explicitly cast to void * when pfree() was called. Per ANSI89, this is
redundant: any pointer type can be assigned to void *, and vice versa,
without a cast. IMHO adding the cast in this case serves only to make
the code harder to read.
Would anyone object to me removing some more void * casts, e.g. the
parameters to the various _walker functions?
-Neil
| Attachment | Content-Type | Size |
|---|---|---|
| void-pointer-cleanup-1.patch | text/x-patch | 3.9 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Neil Conway | 2004-09-26 23:56:00 | minor list cleanup |
| Previous Message | Koju Iijima | 2004-09-26 23:39:06 | patch for temporary view from TODO list |