> On April 5, 2012, 10:24 a.m., Cristian Oneț wrote: > > kmymoney/converter/mymoneyqifwriter.cpp, line 43 > > <http://git.reviewboard.kde.org/r/104059/diff/2/?file=53128#file53128line43> > > > > Why is this include needed if a QVarLengthArray is not used from what I > > can see.
I'm utterly clueless as to why that is in. Some of this update work goes back several months and I have no recollection of this. In fact, I don't even know what would require that. I wondered if it was an auto-completion gone wrong, but can't see anything nearby that I might have wanted. It's out now. I will do another diff and double check changes. - Allan ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/104059/#review12184 ----------------------------------------------------------- On March 14, 2012, 6:03 p.m., Allan Anderson wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/104059/ > ----------------------------------------------------------- > > (Updated March 14, 2012, 6:03 p.m.) > > > Review request for KMymoney. > > > Description > ------- > > This was never implemented. > > Remove unused parameter. Ran astyle. > > > This addresses bug 213284. > http://bugs.kde.org/show_bug.cgi?id=213284 > > > Diffs > ----- > > kmymoney/converter/mymoneyqifwriter.h 2f66f9b > kmymoney/converter/mymoneyqifwriter.cpp 3f76484 > > Diff: http://git.reviewboard.kde.org/r/104059/diff/ > > > Testing > ------- > > Numerous qif files imported, exported and re-imported successfully. > Astyle run. > > > Thanks, > > Allan Anderson > >
_______________________________________________ KMyMoney-devel mailing list KMyMoney-devel@kde.org https://mail.kde.org/mailman/listinfo/kmymoney-devel
