{"comment":{"created_at":"2011-12-14T02:27:06Z","position":null,"commit_id":"8e1193ae60e0c589631f3c0f168773d99cde4b12","updated_at":"2011-12-14T02:27:06Z","user":{"gravatar_id":"7e79a6eb407de124c5cb7ca320bb4ad0","id":46139,"url":"https://api.github.com/users/peller","login":"peller","avatar_url":"https://secure.gravatar.com/avatar/7e79a6eb407de124c5cb7ca320bb4ad0?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png"},"body":"dojo.clone is also susceptible to circular references, and unlike JSON.stringify, it doesn't fail gracefully, it results in a stack overflow.  Under certain circumstances, I believe the data object had cycles to begin with (e.g. a Node object)\r\n\r\nIt sounds like 'deep' cloning isn't necessary.  Perhaps we only have to do a shallow copy of some object (data.properties?)  dojo.clone is both dangerous and expensive.","id":792513,"path":null,"line":null,"html_url":"https://github.com/maqetta/maqetta/commit/8e1193ae60#commitcomment-792513","url":"https://api.github.com/repos/maqetta/maqetta/comments/792513"}}