k, I\'d have to agree with mountaindew (not sure his code would work though, I\'d rather use isin). Easier for users to learn and check how to script things better. It would really make the code look neater, as thats really the only messy part about it. I really think you should make an option to select nicks from a channel, it\'d be much easier to click and go vs. typing /whois everytime. Other then that, looks good.
I\'m not going to start a flame war, but you should take criticism seriously. Without an open mind, you can\'t really improve. I hope I didn\'t come off to harsh there, my comments are usually as so. I am not trying to put you down or anything, and I do believe that you tried hard for this, but it needs work (as any project surely does). My scripts are NEVER perfect, I tend to make those mistakes too. And snippets shouldn\'t have colors in them, as sometimes they don\'t match the tastes of the user. What I meant by very sloppy is that the code is slightly disorganized, and the /\'s make my head spin.
Just a point of view from a harsh, maybe not so harsh as others I have seen, critic. Also, the advertising should be an option in a menu or something. Just hate to see it every time you go away.
.. VERY sloppy. Dislike intensive use of colors, and you practically advertise every time you\'re away. Could replace %away and %reason with $away and $awaymsg since you are using the /away command. It\'s okay, but the dialog could be a bit reworked; as it\'s a bit larger than it needs to be.
alias awayness {
if ($dialog(as) != $null) { dialog -v as | halt }
; First of all, this is very, very poor. No need of a /halt since you use the else event.
; != $null is pointless. You could just use: if ($dialog(as))
else { dialog -m as as }
}
The whole alias could really just be alias awayness if (!$dialog(as)) dialog -m as as
Good luck.
Agreed. A list would be excellent. Just by looking at the code, you obviously supplement a % to the input, thus if a user types %etc it would return %%etc. Probably the reason mountaindew received no feedback without selecting the wildmatch option. Because it would search %etc out of %%etc when selecting wildmatch. A misfit in the documentation of this script.
Overall, to users of this script, you should not include a % when typing in the variable. And to the author, please back that up with some error checking ($iif($left($did($dname,1).text != %,$+(%,$did($dname,1).text),$did($dname,1).text)) or include it in the documentation. kthx.
http://www.mircscripts.org/comments.php?cid=2261 Try harder :)