[compiz] Just another ini patch
patrick.niklaus at googlemail.com
Fri Apr 13 09:41:58 PDT 2007
2007/4/13, Mike Dransfield <mike at blueroot.co.uk>:
> I think its easier to keep the original in the end otherwise you will
> end up with lots of special cases. I thought that solution would cover
> 99% weirdly formatted filenames.
> I am also interested in .blah-blo.screen0.conf and --blah-screen0.conf
> as well as .-.-.-.-.-.-.-.-.-.-.-.-.conf. These files could also
> contain binary
> data so we should check by copying random files into the options dir
> with valid (and invalid) names (I haven't checked that yet, but it should
> reject everything as early as possible).
> The rest seems fine (although I don't personally see the advantage of
> fnLen over len :))
I'm ok with keeping your method if it suits the problem best.
But I have here another patch for you, which actually should have been
included in the last patches but I forgot to commit them. So here it
is as single patch.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 2001 bytes
Desc: not available
Url : http://lists.freedesktop.org/archives/compiz/attachments/20070413/be3af75e/0001-Fixed-bug-with-action-parsing-0001.bin
More information about the compiz