<p dir="ltr"><br>
2015-05-12 14:39 GMT+03:00 Vishesh Handa <<a href="mailto:me@vhanda.in">me@vhanda.in</a>>:<br>
><br>
> On Mon, May 11, 2015 at 6:37 PM, Alexander Potashev <<a href="mailto:aspotashev@gmail.com">aspotashev@gmail.com</a>><br>
> wrote:<br>
>><br>
>> 3. Make module names case insensitive. This can be implemented in<br>
>> Kross framework by listing and filtering directory contents instead of<br>
>> checking for a specific file. It will be slower and this looks like a<br>
>> hack to me.<br>
><br>
> I would go for this one. It's trivial to implement and the speed difference<br>
> would be minuscule.</p>
<p dir="ltr">Hi Vishesh and others,</p>
<p dir="ltr">You are right it's easy to implement and will not be slow, but<br>
 a. This puts us in a ridiculous position where we are slaves of naming conventions. Easy-going users will shoot us down in flames.<br>
 b. I don't like case insensitive names in programming for the same reason identifiers are case sensitive in most programming languages: as soon as there's no rule, anyone is free to spell it "forms", then "FORMS" and finally "fOrMs".<br></p>
<p dir="ltr">It seemed like all three solutions are equally bad, but I finally found an excuse to ignore the library naming convention: Kross modules are not C++ libraries, they are _plugins_.</p>
<p dir="ltr">Notice that the rule "Library names are in CamelCase" goes in section "Frameworks buildsystem is consistent", thus it is meant to beautify CMakeLists.txt files where you may want to link to KF5 libraries. But nobody will ever link to a Kross plugin bypassing the standard loader for Kross modules.</p>
<p dir="ltr">That said, I think it should be absolutely fine if we rename all Kross modules back to lowercase spelling and be happy again.</p>
<p dir="ltr">What do you think?</p>
<p dir="ltr">-- <br>
Alexander Potashev</p>