FordLawnmower commented on a Page, Twitter Bot  -  Jan 09, 2010

What on earth is all of this for?

  ; connection
  sockwrite -n $sockname GET %twitter-connection HTTP/1.1

  ; Write Connection Details
  sockwrite -n $sockname Host: twitter.com
  sockwrite -n $sockname User-Agent: IRCSpider/mIRC $version
  sockwrite -n $sockname Accept: *.*, */*
  sockwrite -n $sockname Referer: $server
  sockwrite -n $sockname Connection: Keep-Alive
  sockwrite -n $sockname Content-Type: text/html
  sockwrite -n $sockname $crlf 

On occasion you may need User Agent:, Referer: is never needed, Content-Type: is for POST not GET as is Accept:, and Connection: is not needed here.
All that is needed here is Host: and GET the rest is just slowing down the script.

  ; connection
  sockwrite -n $sockname GET %twitter-connection HTTP/1.1
  sockwrite -n $sockname Host: twitter.com
  sockwrite -n $sockname $crlf 

Also, did you realize that you pointed an if at empty brackets { }

if ((http://twitter.com/ isin %mirc.temp) || (tweet-url isin %mirc.temp) || (%t = $iif(%twitter.reads == $null,3,%twitter.reads))) { }

What is the point of this?

on *:SOCKCLOSE:twitter:{ /noop }

You didn't unset your variables either.

Sorry if it seems like I'm ranting on your script, I'm sure you worked hard on it, please forgive me :((

I'll stop now because I see several other things.

 Respond  
Are you sure you want to unfollow this person?
Are you sure you want to delete this?
Click "Unsubscribe" to stop receiving notices pertaining to this post.
Click "Subscribe" to resume notices pertaining to this post.