> Yes, it's nice. The "context" however in the IResourceChangeListener can > be confused with a RenderingContext which is often passed as "context" > as well. Maybe changecontext? or changeinfo? Interesting that Olemis pointed out the same issue on bloodhound-dev mailing list. I hope he will join this discussion also.
I agree about possible confusion with parameter named "context". I'm thinking towards "event_context" name but I'm aslo fine with suggested "changecontext" or "changeinfo". > Very much, thanks for thinking about contributing this one back. > You should open a ticket with that patch (against 1.1.2), so that we can > follow the progress of the integration, and discuss this further if > needed (discussion here on trac-dev is welcome as well). Sure, I'll do it. Cheers, Andrej -- You received this message because you are subscribed to the Google Groups "Trac Development" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To post to this group, send email to [email protected]. Visit this group at http://groups.google.com/group/trac-dev?hl=en. For more options, visit https://groups.google.com/groups/opt_out.
