I implemented a small RSS feed into my application in less than fifteen minutes. This is the module I wish I had previously when I was banging my head against the other modules I tried.
4 out of 4 found this review helpful. Was this review helpful to you? Yes No
There is a huge memory leak in ->parse_string that makes this impossible to run as a forked process.
Great idea, poor implementation. I suppose it is a fine module if you just running a short lived process but not for serious work.
2 out of 3 found this review helpful. Was this review helpful to you? Yes No
i like it, it saves a lot of time and work.
until now i did not got any problems.
1 out of 3 found this review helpful. Was this review helpful to you? Yes No
Good lord. This is such a good idea, but it's implemented so badly. Iterating over a 13k RSS feed from the BBC chewed up 135M of memory before I killed it. Aside from that, it's really quite nice.
1 out of 1 found this review helpful. Was this review helpful to you? Yes No
The wantarray() nonsense of all autoloaded methods on XML::RAI::Item objects is a SERIOUS inconvenience, and is NOT documented as such. BAD BAD BAD. My recommendation: get rid of the wantarray nonsense. Another suggestion: MENTION that $item->title, $item->link, etc need to be called in scalar context if you want the actual content. I shouldn't have to go to your source code to figure out why your module's interface is not working as expected. It cost much more time than I would have liked.
Further, using AUTOLOAD like you do is wasting resources (autoloading is costly) and questionable here. The concept of RAI is very sexy, but work on the interface huh?
Was this review helpful to you? Yes No