<div dir="ltr"><div>Hi Thomas,</div><div> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div>Why did you rename Switzerland to Swiss? I don't see a necessity to do so in<br>
the first place. After all, it is still Netherlands and not Dutch. The country's<br>
name is Switzerland and they are swiss people. It's like India and Indian.</div></blockquote><div> </div><div>It was a misunderstanding on my part, I think. I saw this <a href="https://mail.kde.org/pipermail/kde-finance-apps/2020-July/000919.html">https://mail.kde.org/pipermail/kde-finance-apps/2020-July/000919.html</a></div><div>and didn't realise that this correction was intended for the email and thought it was for the actual database. I will change it back 🤦‍♂️.</div><div><br></div><div>Thanks.</div><div>Prasun<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, 14 Aug 2020 at 13:37, Thomas Baumgart <<a href="mailto:thb@net-bembel.de">thb@net-bembel.de</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Prasun,<br>
<br>
On Samstag, 8. August 2020 05:28:37 CEST Prasun Kumar wrote:<br>
<br>
> Hi mentors,<br>
> <br>
> The rest of the work from the second phase has been completed. The databases<br>
> for the additional countries have been incorporated in the library. Please<br>
> take a look<br>
> and suggest any necessary changes.<br>
<br>
Sorry for being a bit late, but I was swamped with work during recent weeks.<br>
<br>
I took a look at the changes you made after July 24th. They look good in general.<br>
I have a few comments, which I will relate to the commits, though:<br>
<br>
396718222dd063f52bb2c7de461fd6bd03e45b45:<br>
2c2e6a5e8856997c94cf80969be503d43fc0ed52:<br>
<br>
Why did you rename Switzerland to Swiss? I don't see a necessity to do so in<br>
the first place. After all, it is still Netherlands and not Dutch. The country's<br>
name is Switzerland and they are swiss people. It's like India and Indian.<br>
<br>
<br>
<br>
0bdd7fb302190697d2fc7170f0f455c34b639b1d:<br>
<br>
        if (accountId.size() > 12 || bankId.size() != 5) {<br>
            return ERROR;<br>
        }<br>
<br>
Where does 12 and 5 come from? Please use symbolic constants as in<br>
<br>
#define NL_MIN_ACCOUNT_ID_SIZE (12) and<br>
#define NL_BANK_ID_SIZE        (5)<br>
<br>
Note: I guessed the names and have no idea if they are correct. They<br>
should simply serve as example.<br>
<br>
<br>
0d473847b3966c4b321cbf52360ea4451d3eeab1:<br>
<br>
        char *bankId = new char[6];<br>
        char *name = new char[61];<br>
<br>
Same here: please use symbolic constants so that one knows where the size comes from.<br>
If you need an extra byte for the trailing zero it is a good practice to do it like<br>
this (example):<br>
<br>
#define NL_BANK_ID_SIZE        (5)<br>
<br>
        char *bankId = new char[NL_BANK_ID_SIZE + 1];<br>
<br>
        if (bankId.size() != NL_BANK_ID_SIZE) {<br>
<br>
<br>
<br>
So much from my end. Keep going.<br>
<br>
Regards<br>
<br>
Thomas<br>
<br>
<br>
<br>
-- <br>
<br>
Regards<br>
<br>
Thomas Baumgart<br>
<br>
<a href="https://www.signal.org/" rel="noreferrer" target="_blank">https://www.signal.org/</a>       Signal, the better WhatsApp<br>
-------------------------------------------------------------<br>
As soon as we wish to be happier, we are no longer happy. -- Walter Landor<br>
-------------------------------------------------------------<br>
</blockquote></div>