Review Request 129003: Separate investment page

[Łukasz Wojniłowicz]

> On Wrz 24, 2016, 10:43 rano, Thomas Baumgart wrote:
> > kmymoney/plugins/csvimport/investmentwizardpage.cpp, line 918
> > <https://git.reviewboard.kde.org/r/129003/diff/1/?file=477386#file477386line918>
> >
> >     It will work but uses more runtime. Since the "Something" is a const char * with encoding UTF-8 which gets converted to a UTF-16 QString and then added. Also a temporary QString object is allocated and destroyed after the operation. Using
> >     
> >       strEType += QStringLiteral("Something");
> >       
> >     is much faster, as the QStringLiteral is already UTF-16 encoded. The drawback is that it uses twice as much memory, but that does not seem to be a problem here.

Done.
I also used QChar with QLatin1Char where appropriate, because it's probably the same kind of problem.
Is it OK now?

BTW. Thanks form making me aware of UTF8->UTF16 thing :)


- Łukasz


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/129003/#review99495
-----------------------------------------------------------


On Wrz 23, 2016, 5:09 po południu, Łukasz Wojniłowicz wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/129003/
> -----------------------------------------------------------
> 
> (Updated Wrz 23, 2016, 5:09 po południu)
> 
> 
> Review request for KMymoney.
> 
> 
> Repository: kmymoney
> 
> 
> Description
> -------
> 
> csvwizard.cpp has many lines of code, so it's good to separate
> investment page from it. The outcome is:
> 
> 1) m_pageInvestment is initialized only when it's needed,
> 2) combo boxes on m_pageInvestment are initialized only when needed,
> 3) removed registered fields, because there is no need to use them on
> other pages,
> 4) merged investprocessing with investmentwizardpage, so there is no
> ping-pong communication between them
> 5) used only one slot for every combo box,
> 6) removed name and security filter lineedits, because they aren't
> essential/usefull to import,
> 7) removed redundant includes,
> 8) clear button causes fees also to be cleared,
> 9) clear fee column right before user goes back to rows page,
> 10) changed operation lists initialization to facilitate translation,
> 11) don't update window size if m_pageInvestment isn't visible, because
> it won't work correctly,
> 12) tidied up names and structure of investmentwizardpage.
> 
> 
> Diffs
> -----
> 
>   kmymoney/plugins/csvimport/CMakeLists.txt 04664f3 
>   kmymoney/plugins/csvimport/csvimporterplugin.h 847d5b8 
>   kmymoney/plugins/csvimport/csvimporterplugin.cpp c4478f4 
>   kmymoney/plugins/csvimport/csvwizard.h 1c75de6 
>   kmymoney/plugins/csvimport/csvwizard.cpp db6e571 
>   kmymoney/plugins/csvimport/investmentwizardpage.h PRE-CREATION 
>   kmymoney/plugins/csvimport/investmentwizardpage.cpp PRE-CREATION 
>   kmymoney/plugins/csvimport/investmentwizardpage.ui 3b4d6a2 
>   kmymoney/plugins/csvimport/investprocessing.h 87b5512 
>   kmymoney/plugins/csvimport/investprocessing.cpp 860ca77 
>   kmymoney/plugins/csvimport/redefinedlg.h 688725e 
>   kmymoney/plugins/csvimport/redefinedlg.cpp 58fa625 
> 
> Diff: https://git.reviewboard.kde.org/r/129003/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Łukasz Wojniłowicz
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kmymoney-devel/attachments/20160925/cd4151e8/attachment.html>