Review Request 129003: Separate investment page

Thomas Baumgart tbaumgart at kde.org
Sat Sep 24 08:43:09 UTC 2016


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




kmymoney/plugins/csvimport/investmentwizardpage.cpp (line 437)
<https://git.reviewboard.kde.org/r/129003/#comment66930>

    Yes, sorry for the confusion.



kmymoney/plugins/csvimport/investmentwizardpage.cpp (line 918)
<https://git.reviewboard.kde.org/r/129003/#comment66931>

    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.


- Thomas Baumgart


On Sept. 23, 2016, 5:09 nachm., Łukasz Wojniłowicz wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/129003/
> -----------------------------------------------------------
> 
> (Updated Sept. 23, 2016, 5:09 nachm.)
> 
> 
> 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/20160924/fbcf994d/attachment.html>


More information about the KMyMoney-devel mailing list