<div dir="auto"><div><div class="gmail_quote"><div dir="ltr" class="gmail_attr">J. Gareth Moreton <<a href="mailto:gareth@moreton-family.com">gareth@moreton-family.com</a>> schrieb am So., 19. Mai 2019, 05:17:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Yes, I'm jumpy, especially as you always like to open your responses <br>
with a direct insult of some kind - yes, I checked.  You want to kill me <br>
or ban me from the mailing list, go right ahead. I'm sure Free Pascal <br>
will survive without me.<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">First of Ralf Quint is not part of the FPC development team, thus he has no power over the mailing list or whether *we* (the development team) accept your contributions. </div><div dir="auto"><br></div><div dir="auto">Second the term "lazy programmer" is a known term in development: </div><div dir="auto"><a href="https://wiki.c2.com/?LazyProgrammer">https://wiki.c2.com/?LazyProgrammer</a><br></div><div dir="auto">So he's not calling you "lazy" personally, but he's pointing out that your argumentation is that of a "lazy programmer". </div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I just feel that in this case, it should be a note, not a warning, <br>
unless you're compiling under a dialect of Pascal (e.g. Extended) that <br>
is stricter in its interpretation of the whitesheet and demands an error <br>
be returned.<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">No, I don't think so. The compiler might generate quite some hints and notes some of which might be false positives (take the "uninitialized variable" in context of "var" parameters) and thus this note/hint would be lost among the noise. </div><div dir="auto"><br></div><div dir="auto">Also you need to keep in mind that enums are not necessarily static over the lifetime of the software. E.g. (using the compiler as an example) a new def type could be added. If you now haven't added empty else-clauses to the case-statements you'll now get a nice warning for every such case statement so that you can be reasonably sure you caught all. </div><div dir="auto"><br></div><div dir="auto">At my work (where we use MSVC) we enabled the compiler's warning for this reason. </div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">P.S. And while I agree that one should not merely look towards C and C++ <br>
for inspiration (I see the practice of lowercasing everything and not <br>
having a space between operators and operands as 'colonisation', for <br>
lack of a better term), it seems that we are a little inconsistent when <br>
it comes to that, given that Free Pascal supports the C-style assignment <br>
operators now.<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">The C-style operators are merely syntactic sugar and don't do any optimization of the operation. Also they can't be used on properties. </div><div dir="auto">Not to mention that not everyone likes their use (I for example am pondering whether I should add a "$COperators Off" to FPC fpcdefs.inc so that contributors won't be tempted to use them). </div><div dir="auto"><br></div><div dir="auto">Regards, </div><div dir="auto">Sven </div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
</blockquote></div></div></div>