[Kmymoney-devel] MyMoneyGncReader::convertSplit

[Jack]

On 2010.05.25 19:00, Ian Neal wrote:
> I was going through the various files that the astyle-kmymoney.sh  
> script changes and noticed that the above function seems to have an  
> extra bracket/brace where it shouldn't - just before the line  
> starting: "else { // not stock"
> 
> Could someone who knows this code (or at least a second pair of eyes)  
> confirm I'm not imagining things?

I certainly don't know the code, but just using emacs it looks like the  
} immediately above that line matches the { on the switch line, the one  
above that matches the one in "if (splitAccount.accountType() ==  
MyMoneyAccount::Stock) {" and the one above that matches the one in "if  
(!newPrice.isZero()) {".  (I just delete the "}" and when I type it  
back in, emacs bounces to the matching one.  (the emacs function is  
blink-matching-open)

However - if the else (the one you quote) follows the if (immediately  
after the case) then you are right - there does seem to be one extra }  
in that set of three.


Jack