DMARC-Filter: OpenDMARC Filter v1.3.2 sourceware.org EB44E386102D
Subject: Re: Sv: Sv: Limit for number of child processes
To: cygwin AT cygwin DOT com
References: <000401d67ba0$8b1f33b0$a15d9b10$@gmail.com>
 <20200826175724 DOT GQ3272 AT calimero DOT vinschen DOT de>
 <000701d67c6c$10bcf720$3236e560$@gmail.com>
 <6d698a32-06bb-a47b-58e6-ceeecca722c9 AT cornell DOT edu>
 <001101d67d16$aa5db9f0$ff192dd0$@gmail.com>
 <6242991f-e5ec-150b-bd6c-15a8c348c7cf AT cornell DOT edu>
 <20200828133643 DOT GK3272 AT calimero DOT vinschen DOT de>
 <395e2d18-802f-9ac3-012d-eb0a12f6ae02 AT cornell DOT edu>
 <20200828173950 DOT GM3272 AT calimero DOT vinschen DOT de>
Message-ID: <26f151fd-7be0-c9d4-e471-e0987476f9a9@cornell.edu>
Date: Sat, 29 Aug 2020 06:01:30 -0400
User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:68.0) Gecko/20100101
 Thunderbird/68.12.0
In-Reply-To: <20200828173950.GM3272@calimero.vinschen.de>
Content-Language: en-US
MIME-Version: 1.0
X-MS-Exchange-MessageSentRepresentingType: 1
Received: from 255.255.255.255 (255.255.255.255) by
 MN2PR15CA0046.namprd15.prod.outlook.com (2603:10b6:208:237::15) with
 Microsoft SMTP Server (version=TLS1_2,
 cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3326.20 via Frontend
 Transport; Sat, 29 Aug 2020 10:01:30 +0000
X-Originating-IP: [68.175.129.7]
X-MS-PublicTrafficType: Email
X-MS-Office365-Filtering-Correlation-Id: 6aadd570-a8ac-430a-610d-08d84c028103
X-MS-TrafficTypeDiagnostic: MN2PR04MB5631:
X-Microsoft-Antispam-PRVS: <MN2PR04MB56318EF84871F5EF9821351FD8530 AT MN2PR04MB5631 DOT namprd04 DOT prod DOT outlook DOT com>
X-MS-Oob-TLC-OOBClassifiers: OLM:7219;
X-MS-Exchange-SenderADCheck: 1
X-Microsoft-Antispam: BCL:0;
X-Microsoft-Antispam-Message-Info: gIyG9gUdSBDGVsyOrYloPCl8E7cWRrTNj/oaPZKF9rLClghDslhRkef1RiG44IZ0eiTzJd+AQnQHNdrIotkvDwsStOM98xqH9FwiQrYMZddEm9y7ypChFlafLcrtYhLFoZXrI+LoUg1WudvL8yxm0c3MOEuVgos3lmDXOdc1V3iye9nujt0kFUw3BSdtATOlIoUJAbVsbcR2+c7O5FlXlpa3iv9/5g/LjsDUv6EnIHOd3ltmAyBqQq8QJItM7AsCxQIxhe5kFuWiP5ZA3KPwYXzRodUCDAH5Lri/1zD81PGX5dF7AyYU2zGAaPrS0I2rA+tpPzjBX3zxhmxYFU1h0YHBlZatgUn39D/Aho8CJDv1fJccUr3sYvBlprfZACSR
X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:;
 IPV:NLI; SFV:NSPM; H:MN2PR04MB6176.namprd04.prod.outlook.com; PTR:; CAT:NONE;
 SFS:(4636009)(136003)(396003)(376002)(366004)(39860400002)(346002)(86362001)(6916009)(2906002)(75432002)(2616005)(956004)(8676002)(16576012)(5660300002)(36756003)(8936002)(4744005)(31696002)(52116002)(6486002)(26005)(53546011)(478600001)(316002)(31686004)(66476007)(786003)(186003)(66556008)(66946007)(43740500002);
 DIR:OUT; SFP:1102; 
X-MS-Exchange-AntiSpam-MessageData: szfrjdOiYNrlQ4ulatV+RDnnNVDbxi1vYmnN2vEPgjXjhIVGGPYF3sAO9fYct9JFSNSrb+GWrALR81WJTMjUZKXJbKdOs0n5P5k2iMBwgPdoRyqvVaVHhFQWu+wI5wsow6ZsIFxalpiOTueL767GnHvPnQPUEzSHHWK+Fr1kuv8librh6IGIoabz3w7f7V02Bk57SygQgo/jo/zZCkta+usJmIIs/jhchvC9BRo3erFAUVN3tEfsEHgKpEYE8FReZd1MvBWBj1Hjo52A5qQc8EzNkPcEHmLu08PLmyClkpFGWaWxxSMF8eTUGAV4CdRKH9nPIwUIqrUCL2NdZcaZElhqk9WVBbWMw3aHhQjiIVPN6k2AqAuQNwpH3qeqCdH5R5rF9JdJU54In8L7+pLiUIDMM/DlKxs6cUmz1UYBft2RDMFc0PNFZQg9yQ4NE/p9Vcj0u0jWSnwPAChaPNb39N5QU2WpXR1YbxFIJq4DlXaOp/DJ/wO6JPb77/gYEJtLDEWozWvVqT8/rIddxi0RPXHDdfRJ9dOZBN3lNJvIdIWb/pUTef0821xwG7603OWbjA2D462x09vdP16nTOBbOX14j7NkNrCtV43BL8GsCrUcNSMluYwR+6pmdlokLv7xm5sX04MN0XlrDZ3AblDwcg==
X-OriginatorOrg: cornell.edu
X-MS-Exchange-CrossTenant-Network-Message-Id: 6aadd570-a8ac-430a-610d-08d84c028103
X-MS-Exchange-CrossTenant-AuthSource: MN2PR04MB6176.namprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 29 Aug 2020 10:01:31.3021 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 5d7e4366-1b9b-45cf-8e79-b14b27df46e1
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: fnOCUbhTsiOAJa03Cdy5QSVXlqPPweH9WPROf+/bwyTxGTIKfzxTP3ugb/eNyWLQ4yioDqU2nomzhdbH4FWomA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MN2PR04MB5631
X-Spam-Status: No, score=-4.4 required=5.0 tests=BAYES_00, DKIM_SIGNED,
 DKIM_VALID, DKIM_VALID_AU, DKIM_VALID_EF, MSGID_FROM_MTA_HEADER, NICE_REPLY_A,
 RCVD_ILLEGAL_IP, RCVD_IN_DNSWL_NONE, RCVD_IN_MSPIKE_H2, SPF_HELO_PASS,
 SPF_PASS, TXREP autolearn=no autolearn_force=no version=3.4.2
X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on
 server2.sourceware.org
X-BeenThere: cygwin AT cygwin DOT com
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: General Cygwin discussions and problem reports <cygwin.cygwin.com>
List-Unsubscribe: <https://cygwin.com/mailman/options/cygwin>,
 <mailto:cygwin-request AT cygwin DOT com?subject=unsubscribe>
List-Archive: <https://cygwin.com/pipermail/cygwin/>
List-Post: <mailto:cygwin AT cygwin DOT com>
List-Help: <mailto:cygwin-request AT cygwin DOT com?subject=help>
List-Subscribe: <https://cygwin.com/mailman/listinfo/cygwin>,
 <mailto:cygwin-request AT cygwin DOT com?subject=subscribe>
From: Ken Brown via Cygwin <cygwin AT cygwin DOT com>
Reply-To: Ken Brown <kbrown AT cornell DOT edu>
Content-Transfer-Encoding: 7bit
Content-Type: text/plain; charset="us-ascii"; Format="flowed"
Errors-To: cygwin-bounces AT cygwin DOT com
Sender: "Cygwin" <cygwin-bounces AT cygwin DOT com>

On 8/28/2020 1:39 PM, Corinna Vinschen wrote:
> On Aug 28 12:02, Ken Brown via Cygwin wrote:
>> On 8/28/2020 9:36 AM, Corinna Vinschen wrote:
>>> As POC, I added a bit of code to maintain a second array, which is only
>>> allocated (using HeapAlloc so as not to spill into the child processes)
>>
>> Should there be a call to HeapFree somewhere, or is there some reason this
>> isn't needed?
> 
> It's local memory in a single process, and it's allocated only once.
> There's no reason to shrink or free this ever again.

It took a night's sleep for this to fully sink in, but I get it now.  A class 
that allocates memory would normally have a destructor that frees the memory. 
But only one instance of this class is created per process, so there's no need 
for the destructor.

I'll send a patch with a comment in case you think it might be helpful to future 
readers of the code.

Ken
--
Problem reports:      https://cygwin.com/problems.html
FAQ:                  https://cygwin.com/faq/
Documentation:        https://cygwin.com/docs.html
Unsubscribe info:     https://cygwin.com/ml/#unsubscribe-simple