Merge pull request #91 from node-xmpp/no-json-2

remove Element.toJSON

benchmarks/index.js

@@ -13,7 +13,7 @@ readdir(__dirname).forEach(function (file) {
     console.log(event.target.toString())
   })
   .on('complete', function () {
-    console.log('Fastest is ' + this.filter('fastest').map('name'))
+    console.log('Fastest is "' + this.filter('fastest').map('name') + '"')
   })
   .run({'async': false})
   console.log('\n')

benchmarks/parsers.js

@@ -1,5 +1,10 @@
 'use strict'
 
+/*
+ * benchmark the parsing speed of the supported backends
+ * difference with parse.js benchmark is that this doesn't use ltx at all
+*/
+
 var benchmark = require('benchmark')
 var node_xml = require('node-xml')
 var libxml = require('libxmljs')

lib/Element.js

@@ -318,16 +318,6 @@ Element.prototype.toString = function () {
   return s
 }
 
-Element.prototype.toJSON = function () {
-  return {
-    name: this.name,
-    attrs: this.attrs,
-    children: this.children.map(function (child) {
-      return child && child.toJSON ? child.toJSON() : child
-    })
-  }
-}
-
 Element.prototype._addChildren = function (writer) {
   writer('>')
   for (var i = 0; i < this.children.length; i++) {

lib/parsers/sax-js.js

@@ -24,7 +24,6 @@ var SaxSaxjs = module.exports = function SaxSaxjs () {
   this.parser.onerror = function (e) {
     that.emit('error', e)
   }
-// TODO: other events, esp. entityDecl (billion laughs!)
 }
 
 inherits(SaxSaxjs, EventEmitter)

test/element-test.js

@@ -82,16 +82,6 @@ vows.describe('Element').addBatch({
     'serialize with integer text': function () {
       var e = new Element('e').t(1000)
       assert.equal(e.getText(), 1000)
-    },
-    'serialize to json': function () {
-      var e = new Element('e', { foo: 23, bar: 0, nil: null }).c('f').t(1000).up()
-      assert.deepEqual(e.toJSON(), {
-        name: 'e',
-        attrs: { foo: 23, bar: 0, nil: null },
-        children: [
-          { name: 'f', attrs: {}, children: [1000] }
-        ]
-      })
     }
   },
   'remove': {