On Sat, Aug 13, 2011 at 10:08:08PM +0100, Connor Lane Smith wrote: > On 13 August 2011 21:41, Eckehard Berns <ecki-suckl...@ecki.to> wrote: > > I do like your patch, something paranoid in me just wanted to point out > > that your patch relies on the implementation of XGetClassHint not > > tempering with ch when returning failure. > > So does yours: it calls XFree on the two ch strings whether or not > XGetClassHint succeeded.
Good point :) -- Eckehard Berns