[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