<div dir="ltr">I believe Cristian's comments were directed to the author of the commit, looped-in.<div><br></div><div>cheers,</div><div>Łukasz (the other one ;))</div></div><div class="gmail_extra"><br><div class="gmail_quote">2016-06-18 20:24 GMT+02:00 Cristian Oneț <span dir="ltr"><<a href="mailto:onet.cristian@gmail.com" target="_blank">onet.cristian@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Is the start line always greater than zero? Isn't it 0 based like all<br>
sane programming structures which use indexes?<br>
<br>
On a side note, when committing bugfixes to thr 4.8 branch make sure<br>
that the following conditions are met:<br>
1. Use the FIXED-IN tag to describe the version in which the fix will<br>
be available 4.8.1 (the next release) [1], [2]<br>
2. Follow that your commits get cherry-picked in master by someone if<br>
you don't do that yourself<br>
3. Only commit essential stuff like bugfixes and try to avoid changing<br>
too much code because 4.8 is a dead end branch and merging to master<br>
could be painful because of the possible conflicts (the more lines are<br>
changed the higher the probability of a conflict)<br>
<br>
A general note about the CSV importer in master. A bunch of files<br>
making up the CSV importer are just copy pasted and the code is way<br>
too verbose and contains too much deprecated API. During the removal<br>
of kdelibs4 support a lot of code was simply disabled (search for<br>
"TODO: port to kf5"). I will not invest any of my time into porting<br>
that unmaintainable code so someone must need to step up and do this<br>
job otherwise the CSV importer will either be removed or rewritten<br>
before a 5.0 release.<br>
<br>
Regards,<br>
Cristian<br>
<br>
[1] <a href="https://techbase.kde.org/Development/Git/Configuration#Commit_Template" rel="noreferrer" target="_blank">https://techbase.kde.org/Development/Git/Configuration#Commit_Template</a><br>
[2] <a href="https://community.kde.org/20110213_GitWorkflowAgenda#Commit_Template" rel="noreferrer" target="_blank">https://community.kde.org/20110213_GitWorkflowAgenda#Commit_Template</a><br>
<br>
2016-06-18 19:23 GMT+02:00 Łukasz Wojniłowicz <<a href="mailto:lukasz.wojnilowicz@gmail.com">lukasz.wojnilowicz@gmail.com</a>>:<br>
> Git commit 28fbbe52fb98bc1e6add434670b5fcd359c357c4 by Łukasz Wojniłowicz.<br>
> Committed on 18/06/2016 at 17:14.<br>
> Pushed by wojnilowicz into branch '4.8'.<br>
><br>
> Set start line to one during CSV import<br>
><br>
> Initial value of m_startLine should always be greater than zero. Shall<br>
> user have start line greater than one, it will be set by appropriate<br>
> slot.<br>
><br>
> M  +2    -2    kmymoney/plugins/csvimport/investprocessing.cpp<br>
><br>
> <a href="http://commits.kde.org/kmymoney/28fbbe52fb98bc1e6add434670b5fcd359c357c4" rel="noreferrer" target="_blank">http://commits.kde.org/kmymoney/28fbbe52fb98bc1e6add434670b5fcd359c357c4</a><br>
><br>
> diff --git a/kmymoney/plugins/csvimport/investprocessing.cpp b/kmymoney/plugins/csvimport/investprocessing.cpp<br>
> index 34e53c8..7562c73 100644<br>
> --- a/kmymoney/plugins/csvimport/investprocessing.cpp<br>
> +++ b/kmymoney/plugins/csvimport/investprocessing.cpp<br>
> @@ -104,7 +104,7 @@ InvestProcessing::InvestProcessing()<br>
>    m_detailColumn = -1;<br>
>    m_endLine = 0;<br>
>    m_fileEndLine = 0;<br>
> -  m_startLine = 0;<br>
> +  m_startLine = 1;<br>
>    m_topLine = 0;<br>
>    m_row = 0;<br>
>    m_endColumn = 0;<br>
> @@ -2038,7 +2038,7 @@ void InvestProcessing::readSettings()<br>
>      m_csvDialog->m_wiz->m_pageInvestment->ui->comboBoxInv_securityName->setCurrentIndex(tmp);<br>
><br>
>      tmp = m_startLine;<br>
> -    m_startLine = profilesGroup.readEntry("StartLine", -1) + 1;<br>
> +    m_startLine = profilesGroup.readEntry("StartLine", 0) + 1;<br>
>      if (m_startLine > m_endLine) {<br>
>        m_startLine = tmp;<br>
>      }<br>
><br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature">pozdrawiam,<br>Łukasz<br><br></div>
</div>