Don't throw exceptions on warnings

lib/js-yaml/loader.js

@@ -167,12 +167,8 @@ function throwError(state, message) {
 }
 
 function throwWarning(state, message) {
-  var error = generateError(state, message);
-
   if (state.onWarning) {
-    state.onWarning.call(null, error);
-  } else {
-    throw error;
+    state.onWarning.call(null, generateError(state, message));
   }
 }
 

test/11-load-errors.js

@@ -19,7 +19,15 @@ suite('Load errors', function () {
       var yamlSource = fs.readFileSync(yamlFile, { encoding: 'utf8' });
 
       assert.throws(function () {
-        yaml.loadAll(yamlSource, function () {}, { filename: yamlFile, schema: TEST_SCHEMA });
+        yaml.loadAll(
+          yamlSource,
+          function () {},
+          {
+            filename: yamlFile,
+            schema: TEST_SCHEMA,
+            onWarning: function (e) { throw e; }
+          }
+        );
       }, yaml.YAMLException, yamlFile);
     });
   });

test/issues/0194.js

@@ -0,0 +1,21 @@
+'use strict';
+
+
+var assert = require('assert');
+var yaml = require('../../');
+var readFileSync = require('fs').readFileSync;
+
+
+test('Don\'t throw on warning', function () {
+  var src = readFileSync(require('path').join(__dirname, '/0194.yml'), 'utf8'),
+      warnings = [],
+      data;
+
+  data = yaml.safeLoad(src);
+
+  assert.deepEqual(data, { foo: { bar: true } });
+
+  yaml.safeLoad(src, { onWarning: function (e) { warnings.push(e); } });
+
+  assert.equal(warnings.length, 1);
+});

test/issues/0194.yml

@@ -0,0 +1,3 @@
+foo: {
+    bar: true
+}