Comments by Wolfgang_Baron (Top 11 by date)

No dude, you should not use iexplore. First, it is a silly, slow and retarded program. Second, it will only start itself and not the user's preferred browser. Third, it is not in the default search path and will therefore fail in many situations (the command line in the windows start menu has special handling to make it work).

Surely anyone can program whatever he wants for himself and I am not insulted (though maybe a bit frustrated), if you continue to produce code with multiple returns. I was only pointing out, that if you want to be nice to other people by keeping your code readable, you will certainly refrain from using multiple returns.

As an example, look at the function QueryLocalGroupSIDs() in CoreCode.cpp of the codeproject article "GUI-Based RunAsEx". Clearly, a nice program and a good job. However, the cruel things the author does to the lpData2 and bSuccess variables results in two return statements inside a __finally section (which is undefined behavior during termination handling btw). Most of the code does nothing at all, because bSuccess can only be TRUE if lpData2 is not NULL. There is no point in using two return statements, because just calculating a result value and returning that at the end is always easier to read. You need 5 to 10 times more time to understand what this function actually does. I even go as far as to say, that the author did not understand his own code here, or he would not have produced so much non functional code.

Again, my point is only meant as a help for anyone, who is interested in the well being of his readers and of course, you can do with your life whatever you want :)

The first person wanting to add a debug statement at the end will prove you wrong.

Sorry, I assumed you were going to use exceptions for your regular control flow, which is unnecessarily slow, produces extra code and makes your debugging experience a nightmare. Maybe you could show a little example to clarify, what you mean?