DDMsgReader: As a loadable module, now make use of user# parameter for reading personal email (for the sysop when deleting a user account)
1 unresolved thread
DDMsgReader v1.64: As a loadable module, now make use of user# parameter for personal email (for the sysop when deleting a user). In this case, it makes use of the user number specified on the command line as a loadable module.
Merge request reports
Activity
17184 17220 switch (whichMailbox) 17185 17221 { 17186 17222 case MAIL_YOUR: // Mail sent to you 17223 /* 17224 // Temporary 17225 if (user.is_sysop) 17226 { 17227 console.attributes = "N"; 17228 console.print("Reading your email - userNum: " + userNum + "\r\n"); 17229 console.pause(); 17230 } 17231 // End Temporary 17232 */ 17187 17233 argVals.personalemail = true; 17188 argVals.usernum = argv[1]; 17234 argVals.usernum = +(argv[1]); 
It's just a different style of ensuring it's a number, I suppose. I saw this style in an example years ago and have been doing it this way for a while. I don't know if there's any real advantage either way.
Edited by Eric Oulashin-
Yep. For this block of code, I did a regex test to make sure argv[1] is all digits, so that I'd be able to expect it could be cast to a number successfully.
Edited by Eric Oulashin
added 1 commit
- 4a968540 - Removed "temporary" code blocks per review comments
enabled an automatic merge when the pipeline for 2d99db83 succeeds
mentioned in commit d61a3589
Please register or sign in to reply