From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 22566 invoked by alias); 22 Jan 2015 18:09:36 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org Received: (qmail 22160 invoked by uid 89); 22 Jan 2015 18:09:09 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-2.7 required=5.0 tests=AWL,BAYES_00,RCVD_IN_DNSWL_LOW,SPF_PASS,T_RP_MATCHES_RCVD autolearn=ham version=3.3.2 X-HELO: mail-ob0-f175.google.com Received: from mail-ob0-f175.google.com (HELO mail-ob0-f175.google.com) (209.85.214.175) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES128-GCM-SHA256 encrypted) ESMTPS; Thu, 22 Jan 2015 18:07:41 +0000 Received: by mail-ob0-f175.google.com with SMTP id wp4so797986obc.6 for ; Thu, 22 Jan 2015 10:07:38 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:mime-version:in-reply-to:references:date :message-id:subject:from:to:cc:content-type; bh=+sPHqErrQSpPH7oGH510VgWPdGivq7oP3t+p8OGhBpw=; b=OC/0viC3cc6/pOTuxses5lPl0i4uP/aPPlo6RJe1T147P7+hBJarvbbLgWDOH8Q1Ul F57B6puPEirzjFIAzRvb06vMBk3si9kyGQMAyhOOMGBgfV/swARZed/JMsh8m+vqgrd+ GIxlh/RiI8bNp8LnVRu9R6NskYa7ctiLGbs0EyvAB0Z80RKloKThBlciml9eDalfRa5E uGzCaYCuDOfVGSApgUdZ8RXZvOCBPiXmDg5M+uJmtJjJ9OcYuw5vi3BlZzHqAV6cASFS h5mNxC311yGPCFBjV8mKwoMVHgLDawiREL/cLG3ZmpvqP2/YxMoBwa0OrCIMxnA0qrLQ X/gg== X-Gm-Message-State: ALoCoQk5P91kLauynf19FlO7WMAXRaDvftHAA06c1C9upPo6oYaz58QWGN7ZIm0luwhJEsjfrY2V MIME-Version: 1.0 X-Received: by 10.202.61.9 with SMTP id k9mr1668926oia.116.1421950058093; Thu, 22 Jan 2015 10:07:38 -0800 (PST) Received: by 10.182.222.98 with HTTP; Thu, 22 Jan 2015 10:07:37 -0800 (PST) In-Reply-To: <20150122171757.GA12383@host2.jankratochvil.net> References: <20150115183316.GA16405@host2.jankratochvil.net> <20150116233802.GA8732@host2.jankratochvil.net> <21696.17350.451535.337528@ruffy2.mtv.corp.google.com> <54C0DC83.20401@redhat.com> <20150122164301.GA11240@host2.jankratochvil.net> <54C12E55.1040907@redhat.com> <20150122171757.GA12383@host2.jankratochvil.net> Date: Thu, 22 Jan 2015 18:09:00 -0000 Message-ID: Subject: Re: [patchv2] Sort threads for thread apply all (bt) From: Doug Evans To: Jan Kratochvil Cc: Pedro Alves , gdb-patches Content-Type: text/plain; charset=UTF-8 X-IsSubscribed: yes X-SW-Source: 2015-01/txt/msg00606.txt.bz2 On Thu, Jan 22, 2015 at 9:17 AM, Jan Kratochvil wrote: > On Thu, 22 Jan 2015 18:07:33 +0100, Pedro Alves wrote: >> https://sourceware.org/bugzilla/show_bug.cgi?id=17539 > > While we're getting to the classical bikeshed before coding in the '-desc' > option I would like to reassure we should not go the global setting way > instead which I would find more appropriate myself in such situation, contrary > to what Doug said: > > On Thu, 15 Jan 2015 20:29:07 +0100, Doug Evans wrote: > # Do we want an option to choose the sort direction? > # [I wouldn't make it a global parameter, just an option to > # thread apply.] I'd rather not add to the global state until there's a justified reason for it. We're not there yet IMO. And we can add -desc later. [Though -asc vs -desc is a bit awkward. -a and -d? -ascending and -descending ?] So I'm happy with the patch as is. [Which isn't to say I wouldn't be happy with something different though.]