pg.ddx.io  pgsql-committers@postgresql.org mailing list archive  
help / color / mirror / Atom feed
From: Alvaro Herrera <alvherre@alvh.no-ip.org>
Subject: [PATCH] Fix flag tests in src/test/modules/test_oat_hooks
Date: Tue, 12 Jul 2022 17:18:23 +0200

In what must have been a copy'n paste mistake, all the flag tests use
the same flag rather than a different flag each.  The bug is not
suprising, considering that it's dead code.
---
 src/test/modules/test_oat_hooks/test_oat_hooks.c | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

inline patch 1: src/test/modules/test_oat_hooks/test_oat_hooks.c (+5/-5 lines) download inline-1.patchdiff --git a/src/test/modules/test_oat_hooks/test_oat_hooks.c b/src/test/modules/test_oat_hooks/test_oat_hooks.c index 1f40d632e0..900d597f5d 100644 --- a/src/test/modules/test_oat_hooks/test_oat_hooks.c +++ b/src/test/modules/test_oat_hooks/test_oat_hooks.c @@ -1795,15 +1795,15 @@ accesstype_arg_to_string(ObjectAccessType access, void *arg) return psprintf("%s%s%s%s%s%s", ((drop_arg->dropflags & PERFORM_DELETION_INTERNAL) ? "internal action," : ""), - ((drop_arg->dropflags & PERFORM_DELETION_INTERNAL) + ((drop_arg->dropflags & PERFORM_DELETION_CONCURRENTLY) ? "concurrent drop," : ""), - ((drop_arg->dropflags & PERFORM_DELETION_INTERNAL) + ((drop_arg->dropflags & PERFORM_DELETION_QUIETLY) ? "suppress notices," : ""), - ((drop_arg->dropflags & PERFORM_DELETION_INTERNAL) + ((drop_arg->dropflags & PERFORM_DELETION_SKIP_ORIGINAL) ? "keep original object," : ""), - ((drop_arg->dropflags & PERFORM_DELETION_INTERNAL) + ((drop_arg->dropflags & PERFORM_DELETION_SKIP_EXTENSIONS) ? "keep extensions," : ""), - ((drop_arg->dropflags & PERFORM_DELETION_INTERNAL) + ((drop_arg->dropflags & PERFORM_DELETION_CONCURRENT_LOCK) ? "normal concurrent drop," : "")); } break; -- 2.30.2
--r2xqiotarqgqvhp7--

Message-ID: <no-message-id-3488@localhost>
Permalink:  ../no-message-id-3488@localhost/
Also on:    postgresql.org/message-id/no-message-id-3488@localhost

 ·  · 

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: pgsql-committers@postgresql.org
  Cc: alvherre@alvh.no-ip.org
  Subject: Re: [PATCH] Fix flag tests in src/test/modules/test_oat_hooks
  In-Reply-To: <no-message-id-3488@localhost>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox