From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.133.124]) by sourceware.org (Postfix) with ESMTPS id 3E4B7383E82E for ; Fri, 20 May 2022 09:22:14 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 3E4B7383E82E Received: from mail-yb1-f198.google.com (mail-yb1-f198.google.com [209.85.219.198]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id us-mta-518-f0ZfRjm8O5yv1tIdbOzojA-1; Fri, 20 May 2022 05:22:12 -0400 X-MC-Unique: f0ZfRjm8O5yv1tIdbOzojA-1 Received: by mail-yb1-f198.google.com with SMTP id x3-20020a25b3c3000000b0064e03a85ccbso6663967ybf.5 for ; Fri, 20 May 2022 02:22:12 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=FbCuWyg4gZd0pmPQDNkect9SZuFKEA2wU+MOErqTOb4=; b=g6EIEDS9/TKTURKqrtSzoticZAW3ayphVt3hvN8L6hF7701cLMWZw81sUxiewk6tfy CSmxMTXiMNZn57+vc/5rigludRwZtSCaSsNQwFtgl4sHqWwUkR288fA4+n9z6sMZhTOz KQptmat8zQu6pWzup0SH6XSJ5Xmp7NvNhHNndIGYJshZSn1jBk6tCAMoxKQiIkE0wz7n c/mw2fEaENq/UFMnBKcN5D0+aoiyf42ZTG+vtQa7bVf/+v5wAg7f5u89K3BFXg3cwdU/ GifU8O1s1rJdLCsRvp3dyjsf6PDxtEpSqkPAb5j4Oqq+Nu1AvQV46T5Duzp8s6Vzg/wa Ln/A== X-Gm-Message-State: AOAM530w+WdHQeryspQ51q5h5giLrqR3vwGHI6Z0NdvVnQExJUYF3MzO +nGk7V1+loA9gb8Ogj90m+QSeqw774NT5sDS97bC5MGKUoeY0F4N3EIoi/TnmHo7/onaGRzKDJ5 bmooRYTpGChgLpgm+bAiZI5fuGsX2INY= X-Received: by 2002:a81:1b4b:0:b0:2ef:5ab8:71f9 with SMTP id b72-20020a811b4b000000b002ef5ab871f9mr8577543ywb.313.1653038532175; Fri, 20 May 2022 02:22:12 -0700 (PDT) X-Google-Smtp-Source: ABdhPJwLTJIjahOylDfA7+leRpTx5eJo6TjajE/fcGUpUs/pqHpeBg4P4dOmJV+PdOVU7BglxP00oohb2RQjQvDfMa8= X-Received: by 2002:a81:1b4b:0:b0:2ef:5ab8:71f9 with SMTP id b72-20020a811b4b000000b002ef5ab871f9mr8577531ywb.313.1653038532019; Fri, 20 May 2022 02:22:12 -0700 (PDT) MIME-Version: 1.0 References: <20220520021933.7CE782042B@pchp3.se.axis.com> In-Reply-To: From: Jonathan Wakely Date: Fri, 20 May 2022 10:22:00 +0100 Message-ID: Subject: Re: [PATCH] libstdc++-v3: Set 26_numerics/random/n.b.d./op./values.cc timeout-factor to 3 To: Hans-Peter Nilsson Cc: gcc Patches , "libstdc++" X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset="UTF-8" X-Spam-Status: No, score=-6.7 required=5.0 tests=BAYES_00, DKIMWL_WL_HIGH, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, DKIM_VALID_EF, KAM_NUMSUBJECT, KAM_SHORT, RCVD_IN_DNSWL_LOW, SPF_HELO_NONE, SPF_NONE, TXREP, T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org X-BeenThere: libstdc++@gcc.gnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Libstdc++ mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 20 May 2022 09:22:15 -0000 On Fri, 20 May 2022 at 10:03, Jonathan Wakely wrote: > > On Fri, 20 May 2022 at 03:20, Hans-Peter Nilsson via Libstdc++ > wrote: > > > > Also, how about shortening those test-suite file-paths? > > They're path sort-of overlong for any git (and ChangeLog) > > commit-line limit. 1/2 :-) > > Yes, they're silly. I like what libc++ does, i.e. name the testsuite > directories after the [stable.name] tags in the standard. They have > std/rand/rand.dis/rand.dist.bern/rand.dist.bern.negbin (for some > reason they use "rand.dis" instead of "rand.dist" for the third > component there, which is not what the standard has, > https://eel.is/c++draft/rand.dist.bern.negbin shows the subclause > hierarchy). > > That's still quite unwieldy, so we could remove the repetition and use > 26_numerics/rand/dist/bernoulli/negbin/. The problem now is that it > gets quite cryptic, so finding the tests for the > negative_binomial_distribution is a bit harder if you don't know where > to look (but git grep solves that in less than a second). Or we could forget about mirroring the nesting in the standard and just use 26_numerics/random/dist/negative_binomial That's still too long for ChangeLogs though.