[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: scintila patch adds NotifyParent

From: Stefan Küng <tortoisesvn_at_gmail.com>
Date: Tue, 8 Jan 2013 10:51:53 +0100

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.


  oo  // \\      "De Chelonian Mobile"
 (_,\/ \_/ \     TortoiseSVN
   \ \_/_\_/>    The coolest Interface to (Sub)Version Control
   /_/   \_\     http://tortoisesvn.net
To unsubscribe from this discussion, e-mail: [dev-unsubscribe_at_tortoisesvn.tigris.org].
Received on 2013-01-08 10:52:19 CET

This is an archived mail posted to the TortoiseSVN Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.