View Single Post
Old 05-11-2009, 11:30 AM   #4 (permalink)
Kalle
The Frequenter
Zend Certified 
 
Join Date: Sep 2007
Location: Denmark
Posts: 352
Thanks: 8
Kalle is on a distinguished road
Default

That getInstance method seems odd to me, the new self(), should be self::$instance = new self; plus its odd to have this static call just for this.

But else it looks good by skimming over it, I wrote a similar thing in C as a PHP extension, however I didn't use curl but rather the streams api, and a notifier for grabbing the HTTP code which is a quite useful feature that no many seems to know about :)
__________________
Send a message via MSN to Kalle Send a message via Skype™ to Kalle
Kalle is offline  
Reply With Quote