While I agree that people need to review any and all code they get from the internet, I have to tell you that if you drop someone's version of sp_help_revlogin, then you've done a very bad thing. At least give people some warning in your write up on the script so that neophytes that may not be so careful aren't screwed by your script. A "Principal Consultant" should know these things and provide much better documentation, especially within the code itself.
--Jeff Moden
RBAR is pronounced "
ree-bar" and is a "
Modenism" for
Row-
By-
Agonizing-
Row.
First step towards the paradigm shift of writing Set Based code:
________Stop thinking about what you want to do to a ROW... think, instead, of what you want to do to a COLUMN.
"Change is inevitable... change for the better is not".
"Dear Lord... I'm a DBA so please give me patience because, if you give me strength, I'm going to need bail money too!"
Helpful Links:
How to post code problems
How to Post Performance Problems
Create a Tally Function (fnTally)