From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) 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.0 Received: from localhost (dcvr.yhbt.net [127.0.0.1]) by dcvr.yhbt.net (Postfix) with ESMTP id B4DEC1FF40 for ; Wed, 21 Dec 2016 07:36:08 +0000 (UTC) From: Eric Wong To: meta@public-inbox.org Subject: [PATCH 0/2] searchthread: minor cleanups and API change Date: Wed, 21 Dec 2016 07:36:05 +0000 Message-Id: <20161221073607.32236-1-e@80x24.org> List-Id: Couldn't find more ways to speed things up, but it's looking pretty good. walk_thread could probably be moved into SearchThread.pm (from View.pm), though I also read enough of the Perl5 source code to learn "scalar keys %hash" and "scalar values %hash" follow the same code path and is optimized to not generate an intermediate array :) Eric Wong (2): searchthread: update comment about loop prevention searchthread: simplify API and remove needless OO lib/PublicInbox/SearchThread.pm | 39 ++++++++++++++------------------------- lib/PublicInbox/SearchView.pm | 25 ++++++++++++------------- lib/PublicInbox/View.pm | 19 ++++++++----------- t/thread-cycle.t | 8 ++++---- 4 files changed, 38 insertions(+), 53 deletions(-)