{"comment":{"user":{"avatar_url":"https://secure.gravatar.com/avatar/81390a4163558ff14d455a3dc124abce?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","gravatar_id":"81390a4163558ff14d455a3dc124abce","url":"https://api.github.com/users/brettp","login":"brettp","id":43375},"position":null,"updated_at":"2011-11-01T19:21:52Z","commit_id":"518083ff5d13b6bec789bbdac03f4004b457c0ed","url":"https://api.github.com/repos/Elgg/Elgg/comments/688471","line":null,"body":"Looked into this more. The DOM isn't as robust as the current solution because it doesn't support user and password. I'm looking into a hybrid approach because there are still some issues with the regex version.\r\n\r\nHere's a good discussion on this, as well as some benchmarking:\r\nhttp://james.padolsey.com/javascript/parsing-urls-with-the-dom/\r\n\r\nBasically, regex is more versatile, though less reliable. DOM is slower, but no one will notice.","html_url":"https://github.com/Elgg/Elgg/commit/518083ff5d#commitcomment-688471","id":688471,"path":null,"created_at":"2011-11-01T19:21:52Z"}}