Re: TreeModelFilter modify func

muppet wrote:

Hrm.  Yeah, that looks suspect.  We pass in the value, but don't  
marshal it back.  So, while changing the behavior at this point could  
be considered an API break, nobody could really be using this  
successfully at present.

The patch looks okay, and it's only the API stability issues that  
worry me.

I don't think there's a way someone could have used this successfully,

We could put the patch into an unstable release first.


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]