Originally posted by: Evadman
Dim m as lotusMemo
Do until ForumExists("Programming") = true
I believe that you can optimize this - the "= true" part should be redundant. Unless of course your corporate coding style guide dictates it.
Originally posted by: Evadman
m.SendTo(1) = "anand@Anandtech.com"
m.SendTo(2) = "Moderator@anandtech.com"
Couldn't that be implemented as "X,Y", on the same line? Is m.SendTo a dynamic array? Is that safe from exploitation due to buffer-overflows?
Originally posted by: Evadman
m.cc(1) = "fs5@wherever.com"
m.Subject = "New Forum Request"
m.Body = "Please give us a programming forum!" & vbcrlf & vbcrlf & "Evadman"
m.Replyto = "Evadman"
Doesn't the m.ReplyTo need to have an "@domain" after it, for it to be properly recieved, if someone chooses to reply?
Originally posted by: Evadman
m.Attachment = "ROFLcopter.gif"
What about the path? Isn't that exploitable, if someone changes the current drive/directory in a batch file, and then executes this code? What if they rename something to .gif.exe, and the recipient has "show extensions" disabled. Evil!
Originally posted by: Evadman
Call SendLotusEmail(m)
loop
Call msgbox ("OMG! the programming forum exists!",vbcritical,"Horray!")
Again, unless corporate style guidelines dictate it, aren't both of those "call" statements unnecessary and superfluous?
(Disclaimer, my code-review should be considered tongue-in-cheek, and really, it was just a devious excuse to bump this thread.

But it's been ages since I've done any BASIC programming, so I'm not sure if my "corrections" are right or not. Port the code to C++ and then we'll talk.)