Skip to content
This repository has been archived by the owner on Jan 20, 2024. It is now read-only.

It dies on malformed %YAML directive #4

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

menduz
Copy link
Contributor

@menduz menduz commented Sep 5, 2016

What is the FIXME on line 186?
Why is line 216 commented?

Example of breaking code:

const y = require('yaml-ast-parser');

function printResult(){ console.log(arguments) }

y.loadAll(
`%YAML 12

---
a: 123`, printResult);
// TypeError: Cannot read property '1' of null

What is the FIXME on line 186?
Why is line 216 commented?

Example of breaking code:
```javascript
const y = require('yaml-ast-parser');

y.loadAll(
`%YAML 12
---
a: 123`,
function(){ console.log(require('util').inspect(arguments, false, 10, true)) })
TypeError: Cannot read property '1' of null
```
@menduz
Copy link
Contributor Author

menduz commented Sep 5, 2016

@sichvoge @ddenisenko

@menduz menduz mentioned this pull request Oct 4, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant