From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: mail-a.sr.ht; dkim=pass header.d=johnnyrichard.com header.i=@johnnyrichard.com Received: from out-171.mta0.migadu.com (out-171.mta0.migadu.com [91.218.175.171]) by mail-a.sr.ht (Postfix) with ESMTPS id 4312B20345 for <~johnnyrichard/olang-devel@lists.sr.ht>; Wed, 28 Feb 2024 11:40:59 +0000 (UTC) Date: Wed, 28 Feb 2024 13:40:59 +0100 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=johnnyrichard.com; s=key1; t=1709120458; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=XpVxFy76S8OEoa5S7sIPUUdRpqSgst4jVsybQ1BPQj4=; b=T+cEWXC+fwy8RA3EgefHw+R3ayA2VvmdFUg9irlLyL0Iq5nq6o2cZCuHnCbjVg72ZYOwT5 Yo43Md+If6qq29c9Jr1Jw8Xy0ORtX9hrd+j1VaZo2gopnuZgg4TmPb1cRfDN8PZd27vfXL DUYBr70k5E7vKJNvZeIE0QyTIU/cr9VdKy9WgvlFlxsr2C9VdBjF/m+dOlpWdBMhSGcvef 4php91nQgGWRJjknmE9HlaJRP5cHLUN1GepO2XdC8UdE1qItZTZmSvOD7nV2mIrODfQQOV DzmlSFqn/raH70Omup9e9aCVnG92BhutyJgKKzvFhySx7aZ0raj6/ZvSZr8wvQ== X-Report-Abuse: Please report any abuse attempt to abuse@migadu.com and include these headers. From: Johnny Richard To: Carlos Maniero Cc: ~johnnyrichard/olang-devel@lists.sr.ht Subject: Re: [PATCH olang] cli: replace memory allocation malloc -> arena Message-ID: X-Sourcehut-Patchset-Update: SUPERSEDED References: <20240222191007.78350-1-johnny@johnnyrichard.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: X-Migadu-Flow: FLOW_OUT X-TUID: DKN9QEZ+bVKb On Tue, Feb 27, 2024 at 08:22:32PM -0300, Carlos Maniero wrote: > > Do you think the following fixup is good enough? > It's perfect! Sorry to hold your patch just for a comment, but since we > don't use any tracking tool for TODOs, I think it is a good idea to add > TODOs comments on the code to keep tracking on them. No worries. This patch has been SUPERSEDED by V2[1] [1]: https://lists.sr.ht/~johnnyrichard/olang-devel/%3C20240228123810.44195-1-johnny%40johnnyrichard.com%3E