[Kmymoney-devel] MyMoneyGncReader::convertSplit

[Alvaro Soliverez]

On Tue, May 25, 2010 at 11:30 PM, Jack <ostroffjh at users.sourceforge.net> wrote:
> 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.
>

Indeed, it seems to be out of place, and Qt Creator parses it as out
of place. However, if you remove it, all subsequents methods error
out. So, there is an opening bracket hidden somewhere, but I couldn't
find it.