Hi, On 1 November 2011 22:42, Alan Coopersmith <alan.coopersm...@oracle.com> wrote: > When we want to print a string, it's okay to just print it. > We don't need to first allocate a buffer 2 bytes bigger than the > string, copy the entire string unmodified to the buffer, print the > buffer, and then leak the buffer (though we AbortDDX 8 lines later, > and then just in case we survived that, call exit as well, so the > leak is short lived, just oh so pointless). > > Oh, and for good measure, put the "r" in String, no matter how > much it stings.
Bonus points if you felt like just removing optionTypeToSting, now that it's unused. > Signed-off-by: Alan Coopersmith <alan.coopersm...@oracle.com> Reviewed-by: Daniel Stone <dan...@fooishbar.org> Cheers, Daniel _______________________________________________ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel