Re: scintila patch adds NotifyParent

On Sun, Jan 6, 2013 at 10:33 PM, Oto BREZINA <otik_at_printflow.eu> wrote:

> It seems it not get thru first time.
>
> TSVn scintila patch introduces
> virtual void NotifyParent(SCNotification * scn)
> is this neccesary ?
>
> It seems be used only in other part of patch where NotifyParent(&scn)
> can be replaced by NotifyParent(scn).
>
>
Nope: in TBlame we use NotifyParent(&scn) so that we can return the changed
background color. Using the (now) existing NotifyParent(scn) function can't
do that since we get a copy of the data, not access to the original data.

Stefan

-- 
       ___
  oo  // \\      "De Chelonian Mobile"
 (_,\/ \_/ \     TortoiseSVN
   \ \_/_\_/>    The coolest Interface to (Sub)Version Control
   /_/   \_\     http://tortoisesvn.net
------------------------------------------------------
http://tortoisesvn.tigris.org/ds/viewMessage.do?dsForumId=757&dsMessageId=3041774
To unsubscribe from this discussion, e-mail: [dev-unsubscribe_at_tortoisesvn.tigris.org].