[Kss-devel] Default compression method fail (patch included)
Godefroid Chapelle
gotcha at bubblenet.be
Sat Mar 8 08:56:36 CET 2008
Jeroen Vloothuis wrote:
> Balazs Ree wrote:
>>> Do you think it's a good idea to commit it in the trunk ? If you don't
>>> have time, I can commit it for you if you give me an access to the
>>> repository.
>>>
>> Before thinking of a fix, please provide more details to convince us that
>> we really need this change. We absolutely need to understand and
>> reproduce the problem before taking any more action about the issue.
>>
> The change just seems to fix the number of parameters of a lambda which
> is used as a callback. Afaik we should just create a test to make sure
> it does not break again (and is supposed to have two arguments).
> Discussing the need for this behavior seems irrelevant since the code is
> already there, its just broken.
I think you are right.
--
Godefroid Chapelle (aka __gotcha) http://bubblenet.be
More information about the Kss-devel
mailing list