| Subject: | Might be author don't know this... |  
| Summary: | Package rating comment |  
| Messages: | 2 |  
| Author: | Oleg Butuzov |  
| Date: | 2008-05-30 18:23:41 |  
| Update: | 2008-05-30 22:25:12 |  
|   |  
 
 | 
 | 
Oleg Butuzov rated this package as follows:
| Utility:  | Bad | 
| Consistency:  | Bad | 
| Examples:  | Not sure | 
| 
 | 
  Oleg Butuzov - 2008-05-30 18:23:41  
Might be author don't know this function but... 
php.net/parse_ini_file
--- 
btw,  i think its to stupid run preg regularexpressions twice. 
if (preg_match('/(\[)(.*)(\])/i', $l)){ 
    preg_match('/(\[)(.*)(\])/i', $l, $ini_key);  
  
  Roy Cunningham - 2008-05-30 22:25:12 -  In reply to message 1 from Oleg Butuzov 
Thanks for your comment - I've fixed the offending code.  One of the reasons I'm going to start posting more of my stuff is to have it reviewed so I can learn.  I've never had peer review of my code before, so thanks for the feedback. 
 
Roy 
 
  
   |