abhishekkr/rubygem_xml_motor — comment on commit 087aa06fc56ba3d269d6bb7ae06c55764abf107e
File:  line 0
By jacaetevha:

This is a very naive implementation. I found two things right away, and I won't even begin to talk about performance.

1. It can't find nodes by an attribute value if the attributes are separated by a linefeed (it hard-codes a separator regex of the space and tab characters).
2. It can't find nodes if you don't quote the attribute value exactly the same as it is in the XML.

## Simple test cases:
```ruby
xml = %Q{i=1,j=2\ni=1,j=3}
XMLMotor.get_node_from_content xml, 'a', "j='2'" # => []
XMLMotor.get_node_from_content xml, 'a', 'j="2"' # => []
XMLMotor.get_node_from_content xml, 'a', 'i=1' # => ["i=1,j=3"]
XMLMotor.get_node_from_content xml, 'a', 'i="1"' # => []
XMLMotor.get_node_from_content xml, 'a', "i='1'" # => []
```
Normally, I wouldn't comment like this. But, if you're going to release a gem and then link to it like you did on LinkedIn, you open up yourself for comments and criticism. The proliferation of barely usable gems in the universe is getting annoying.