From mboxrd@z Thu Jan 1 00:00:00 1970 X-Spam-Checker-Version: SpamAssassin 3.4.4 (2020-01-24) on inbox.vuxu.org X-Spam-Level: X-Spam-Status: No, score=0.8 required=5.0 tests=DATE_IN_PAST_12_24 autolearn=no autolearn_force=no version=3.4.4 Received: (qmail 1869 invoked from network); 14 Feb 2021 14:45:48 -0000 Received: from 1ess.inri.net (216.126.196.35) by inbox.vuxu.org with ESMTPUTF8; 14 Feb 2021 14:45:48 -0000 Received: from 5ess.inri.net ([107.191.111.177]) by 1ess; Sat Feb 13 15:40:02 -0500 2021 Received: from [127.0.0.1] ([104.59.85.219]) by 5ess; Sat Feb 13 15:22:00 -0500 2021 Date: Sat, 13 Feb 2021 15:21:59 -0500 From: Stanley Lieber To: 9front@9front.org In-Reply-To: <359D5A90DCD0DE4120049A0064989632@eigenstate.org> References: <359D5A90DCD0DE4120049A0064989632@eigenstate.org> Message-ID: <20CFE940-F68E-49A7-9E80-1A3548063BC7@stanleylieber.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable List-ID: <9front.9front.org> List-Help: X-Glyph: ➈ X-Bullshit: extensible scale-out realtime interface Subject: Re: [9front] sam input line length limits Reply-To: 9front@9front.org Precedence: bulk On February 13, 2021 3:06:22 PM EST, ori@eigenstate=2Eorg wrote: >Quoth Steve Simon : >> Normally I would be very wary of changing sam it has served >> me well for many years, but I am with you on this one, >> it has bitten me in the past and its annoying=2E >>=20 >> Re the patch: >>=20 >> I am being pedantic, I know, but for completeness=2E=2E=2E >> free(c) should come before the test for s =3D=3D nil, >> as it could lead to a memory leak=2E reordering the code >> fractionally fixes it=2E > >Yep, it definitely should=2E Thanks for looking=2E >I'll actually *run* the code today, and see if >I missed anything=2E > > installed it yesterday, no reports of ill effects so far=2E sl