This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: mallopt indentation bug?
- From: Andreas Jaeger <aj at suse dot com>
- To: libc-alpha at sourceware dot org
- Date: Wed, 05 Sep 2012 10:20:38 +0200
- Subject: Re: mallopt indentation bug?
- References: <orobll4bb9.fsf@livre.localdomain>
On Tuesday, September 04, 2012 20:28:58 Alexandre Oliva wrote:
> The assignment to no_dyn_threshold is indented as if it was meant to
> be run only in the else branch, but since C is not Python, it's run
> unconditionally. That's not necessarily a bug, but since indentation
> is misleading here, I figured I'd better fix it one way or the other,
> and I decided to add braces to match the intent implied by
> indentation, rather than reindent.
>
> Patch pushed to lxoliva/malloc-probes-bz742038. Ok for master?
Yes, this looks fine,
Thanks,
Andreas
--
Andreas Jaeger aj@{suse.com,opensuse.org} Twitter/Identica: jaegerandi
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn,Jennifer Guild,Felix Imendörffer,HRB16746 (AG Nürnberg)
GPG fingerprint = 93A3 365E CE47 B889 DF7F FED1 389A 563C C272 A126