Re: [PATCH 1/2] Provide a common malloc wrappers and palloc et al. emulation for frontend'ish environs

Am I the only one who finds this way of posting patches really annoying?

Here is a patch with no description other than a list of changed
files. And discussion happens in a completely different email.

What's wrong with just posting the patch as a regular attachment(s) to
a regular thread, like other people do?

Yes, I'm well aware that some mailers thread them in right. Our
archives code does. But many MUAs don't. But even when using a MUA
that threads it correctly, I find it quite annoying that you have to
open one email to read the comments and a different email toview the
patch.

It may be just me. But it may be others as well, so I figured I should
raise the issue :)

//Magnus


On Wed, Jan 9, 2013 at 12:27 PM, Andres Freund  wrote:
> ---
>  contrib/oid2name/oid2name.c            |  52 +------------
>  contrib/pg_upgrade/pg_upgrade.h        |   5 +-
>  contrib/pg_upgrade/util.c              |  49 -------------
>  contrib/pgbench/pgbench.c              |  54 +-------------
>  src/backend/utils/mmgr/mcxt.c          |  78 +++++++++++---------
>  src/bin/initdb/initdb.c                |  40 +---------
>  src/bin/pg_basebackup/pg_basebackup.c  |   2 +-
>  src/bin/pg_basebackup/pg_receivexlog.c |   1 +
>  src/bin/pg_basebackup/receivelog.c     |   1 +
>  src/bin/pg_basebackup/streamutil.c     |  38 +---------
>  src/bin/pg_basebackup/streamutil.h     |   4 -
>  src/bin/pg_ctl/pg_ctl.c                |  39 +---------
>  src/bin/pg_dump/Makefile               |   6 +-
>  src/bin/pg_dump/common.c               |   1 -
>  src/bin/pg_dump/compress_io.c          |   1 -
>  src/bin/pg_dump/dumpmem.c              |  76 -------------------
>  src/bin/pg_dump/dumpmem.h              |  22 ------
>  src/bin/pg_dump/dumputils.h            |   1 +
>  src/bin/pg_dump/pg_backup_archiver.c   |   1 -
>  src/bin/pg_dump/pg_backup_custom.c     |   2 +-
>  src/bin/pg_dump/pg_backup_db.c         |   1 -
>  src/bin/pg_dump/pg_backup_directory.c  |   1 -
>  src/bin/pg_dump/pg_backup_null.c       |   1 -
>  src/bin/pg_dump/pg_backup_tar.c        |   1 -
>  src/bin/pg_dump/pg_dump.c              |   1 -
>  src/bin/pg_dump/pg_dump_sort.c         |   1 -
>  src/bin/pg_dump/pg_dumpall.c           |   1 -
>  src/bin/pg_dump/pg_restore.c           |   1 -
>  src/bin/psql/common.c                  |  50 -------------
>  src/bin/psql/common.h                  |  10 +--
>  src/bin/scripts/common.c               |  49 -------------
>  src/bin/scripts/common.h               |   5 +-
>  src/include/port/palloc.h              |  19 +++++
>  src/include/utils/palloc.h             |  12 +--
>  src/port/Makefile                      |   8 +-
>  src/port/dirmod.c                      |  75 +------------------
>  src/port/palloc.c                      | 130 +++++++++++++++++++++++++++++++++
>  37 files changed, 218 insertions(+), 621 deletions(-)
>  delete mode 100644 src/bin/pg_dump/dumpmem.c
>  delete mode 100644 src/bin/pg_dump/dumpmem.h
>  create mode 100644 src/include/port/palloc.h
>  create mode 100644 src/port/palloc.c
>
>
>
> --
> Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
> To make changes to your subscription:
> http://www.postgresql.org/mailpref/pgsql-hackers
>



-- Magnus HaganderMe: http://www.hagander.net/Work: http://www.redpill-linpro.com/