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
|