From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on dcvr.yhbt.net X-Spam-Level: X-Spam-ASN: X-Spam-Status: No, score=-4.0 required=3.0 tests=ALL_TRUSTED,BAYES_00 shortcircuit=no autolearn=ham autolearn_force=no version=3.4.2 Received: from localhost (dcvr.yhbt.net [127.0.0.1]) by dcvr.yhbt.net (Postfix) with ESMTP id 4A6A71F4B9 for ; Sun, 26 Jan 2020 01:17:44 +0000 (UTC) From: Eric Wong To: meta@public-inbox.org Subject: [PATCH 0/3] writing/admin code cleanups lock fix Date: Sun, 26 Jan 2020 01:17:41 +0000 Message-Id: <20200126011744.6278-1-e@yhbt.net> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit List-Id: I accidentally did some parallel -compact runs and noticed some confusing (though harmless) errors. It could be harmful if -xcpdb is running reshard, though, but I don't do that much. There's also some cleanups leading up to it since I got tired of looking at '||' and '//' all over the code. Eric Wong (3): inbox: add ->version method search: {version} => {ibx_ver} xapcmd: increase scope of lock lib/PublicInbox/Admin.pm | 5 +-- lib/PublicInbox/AltId.pm | 2 +- lib/PublicInbox/Feed.pm | 2 +- lib/PublicInbox/Inbox.pm | 11 +++--- lib/PublicInbox/InboxWritable.pm | 9 ++--- lib/PublicInbox/Search.pm | 6 +-- lib/PublicInbox/SearchIdx.pm | 6 +-- lib/PublicInbox/Xapcmd.pm | 64 +++++++++++++++++--------------- script/public-inbox-convert | 2 +- scripts/import_slrnspool | 2 +- 10 files changed, 56 insertions(+), 53 deletions(-)