mallopt indentation bug?

Andreas Jaeger aj@suse.com
Wed Sep 5 08:20:00 GMT 2012


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



More information about the Libc-alpha mailing list