Skip to content
This repository has been archived by the owner on Nov 16, 2023. It is now read-only.

Support CRLF #11

Closed
octref opened this issue Jun 26, 2018 · 1 comment
Closed

Support CRLF #11

octref opened this issue Jun 26, 2018 · 1 comment

Comments

@octref
Copy link
Contributor

octref commented Jun 26, 2018

From @tboby on June 25, 2018 15:36

Hi, just saw this go live. Hope this is the right place for feedback.

It seems that this tool doesn't support CRLF line endings. It throws a JSON parsing error.

VM2150:7 Uncaught SyntaxError: Unexpected token [ in JSON at position 65
    at JSON.parse (<anonymous>)
    at be (app.695a9541.js:formatted:26621)
    at app.695a9541.js:formatted:26603
    at Array.forEach (<anonymous>)
    at pe (app.695a9541.js:formatted:26602)
    at h.updateLog (app.695a9541.js:formatted:26692)
    at chunk-vendors.a8885d66.js:12
    at chunk-vendors.a8885d66.js:12
    at Array.forEach (<anonymous>)
    at chunk-vendors.a8885d66.js:12

Same file, different line endings:
crlf.log
lf.log

Copied from original issue: octref/lsp-inspector#11

@octref
Copy link
Contributor Author

octref commented Jun 26, 2018

Thanks, addressed and will deploy soon once I fix #10.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant