[PATCH] Minor code cleanup in profilewidget2.cpp

Joakim Bygdell j.bygdell at gmail.com
Fri Jan 9 11:04:10 PST 2015


> On 09 Jan 2015, at 19:28, Willem Ferguson <willemferguson at zoology.up.ac.za> wrote:
> 
> On 09/01/2015 20:06, Joakim Bygdell wrote:
>> 
>> That code needs to be in those two locations, one is executed when you interact with the profile.
>> The other when a dive is selected and everything is initiated.
>> 
>> /Jocke
>> 
>> __
> 
> Joakim,
> 
> Attached is an image of the relevant code snippet in order to try and avoid the indenting being scrambled by the email.
> Assuming the code in this particular if-then segment is executed from top to bottom, how does your explanation above
> apply? Please explain this to me in human-understandable terms because I admit I am even less than a beginner in this game.
> I would have thought the topmost yellow and blue instructions are superfluous??
> Kind regards,
> willem
> 

Ah there seems to have been a copy-paste error.
You where indeed correct.

When I said both locations I where referring to the duplications at lines around 680 and 960 since that is what I saw when I looked at your patch.


/Jocke



More information about the subsurface mailing list