[compiz] Just another ini patch

Patrick Niklaus 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.

Regards,
Patrick
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Fixed-bug-with-action-parsing.patch
Type: text/x-patch
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 mailing list