On Wed, Mar 09, 2016 at 10:03:51PM +0100, Markus Armbruster wrote: > Kevin Wolf <kw...@redhat.com> writes: > > I think it's unnecessary, but fine with me. I'm just trying to say that > > making it a fixed 128 byte array on the stack certainly doesn't improve > > anything. > > It trades a few bytes of stack for a fixed stack frame. A fixed stack > frame is a bit more efficient (not that it matters here), and lets us > scratch the function permanently from the list of stack fram size false > positives. I think that's a reasonable trade.
Yes, that's what I want to do. I did fix nothing, but tried to avoid the warning. Sorry that I made it a wrong title (also in the following splitted patch). I should say: "Fix unbounded stack warning for qdict_array_entries" Rather than: "Fix unbounded stack for qdict_array_entries" Thanks. Peter