Hi, On Sun, Jan 20, 2019 at 01:19:50PM +0000, Simon Rozman wrote: > Now, I could leave that semicolon in place to keep IntelliSense false > warning away, but looking at the code I asked myself a question: "If my code > is so convoluted it produces a false warning to a bot, how convoluted it > must be for a human to understand?" False warnings are usually sign of > overcomplex or atypical code design. I have refactored the code.
Thanks for the explanation, I wasn't aware that there is a special case
where a semicolon is required after a label. Even if only MSVC :-)
But second half is even better ;-)
> > Something else might not be obvious to the reader:
> >
> > + if (pQsc->dwStartType <= SERVICE_AUTO_START)
> > + {
> >
> > so what does "lesser than AUTO_START" mean? manual start? no start at
> > all? Comparing for "lesser or equal" with something enum-like might
> > warrant a comment /* BOOT_START = 0, SYSTEM_START = 1, AUTO_START = 2 */
> > or so... (yes, I can google this, but still).
>
> This is a very generic test "Is driver or service set to start automatically
> in any phase of the system launch?" on the Windows.
Ah. So "anyone who has any clue about windows services would know and
understand" - good enough. We need more clueful windows people to review
windows patches :-)
(Selva is hiding somewhere)
> I've added a comment with a brief explanation.
>
> A patch will follow.
Thanks!
gert
--
"If was one thing all people took for granted, was conviction that if you
feed honest figures into a computer, honest figures come out. Never doubted
it myself till I met a computer with a sense of humor."
Robert A. Heinlein, The Moon is a Harsh Mistress
Gert Doering - Munich, Germany [email protected]
smime.p7s
Description: S/MIME cryptographic signature
_______________________________________________ Openvpn-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/openvpn-devel
