atuttle / atom-language-cfml

:space_invader: A CFML Language for the Atom Editor
38 stars 24 forks source link

Starting Atom: Uncaught RangeError: Maximum call stack size exceeded #85

Closed icimasa closed 6 years ago

icimasa commented 8 years ago

I get this error everytime I start Atom if there is a tab open with cfm files (CFML/HTML) language in the list that Atom wants to reopen, if there is not a tab open when starting Atom, everything is fine..

This issue has been since v. 1.7.0

I run Windows 10

C:\Users\masa\AppData\Local\atom\app-1.7.1\resources\app.asar\node_modules\underscore-plus\node_modules\underscore\underscore.js:219

RangeError: Maximum call stack size exceeded at Array.indexOf (native) at Object..contains..include (C:\Users\masa\AppData\Local\atom\app-1.7.1\resources\app.asar\node_modules\underscore-plus\node_modules\underscore\underscore.js:219:68) at Grammar.module.exports.Grammar.grammarUpdated (C:\Users\masa\AppData\Local\atom\app-1.7.1\resources\app.asar\node_modules\first-mate\lib\grammar.js:275:14) at GrammarRegistry.module.exports.GrammarRegistry.grammarUpdated (C:\Users\masa\AppData\Local\atom\app-1.7.1\resources\app.asar\node_modules\first-mate\lib\grammar-registry.js:180:23) at Grammar.module.exports.Grammar.grammarUpdated (C:\Users\masa\AppData\Local\atom\app-1.7.1\resources\app.asar\node_modules\first-mate\lib\grammar.js:279:21) at GrammarRegistry.module.exports.GrammarRegistry.grammarUpdated (C:\Users\masa\AppData\Local\atom\app-1.7.1\resources\app.asar\node_modules\first-mate\lib\grammar-registry.js:180:23) at Grammar.module.exports.Grammar.grammarUpdated (C:\Users\masa\AppData\Local\atom\app-1.7.1\resources\app.asar\node_modules\first-mate\lib\grammar.js:279:21) at GrammarRegistry.module.exports.GrammarRegistry.grammarUpdated (C:\Users\masa\AppData\Local\atom\app-1.7.1\resources\app.asar\node_modules\first-mate\lib\grammar-registry.js:180:23) at Grammar.module.exports.Grammar.grammarUpdated (C:\Users\masa\AppData\Local\atom\app-1.7.1\resources\app.asar\node_modules\first-mate\lib\grammar.js:279:21) at GrammarRegistry.module.exports.GrammarRegistry.grammarUpdated (C:\Users\masa\AppData\Local\atom\app-1.7.1\resources\app.asar\node_modules\first-mate\lib\grammar-registry.js:180:23) at Grammar.module.exports.Grammar.grammarUpdated (C:\Users\masa\AppData\Local\atom\app-1.7.1\resources\app.asar\node_modules\first-mate\lib\grammar.js:279:21) at GrammarRegistry.module.exports.GrammarRegistry.grammarUpdated (C:\Users\masa\AppData\Local\atom\app-1.7.1\resources\app.asar\node_modules\first-mate\lib\grammar-registry.js:180:23) at Grammar.module.exports.Grammar.grammarUpdated (C:\Users\masa\AppData\Local\atom\app-1.7.1\resources\app.asar\node_modules\first-mate\lib\grammar.js:279:21) at GrammarRegistry.module.exports.GrammarRegistry.grammarUpdated (C:\Users\masa\AppData\Local\atom\app-1.7.1\resources\app.asar\node_modules\first-mate\lib\grammar-registry.js:180:23) at Grammar.module.exports.Grammar.grammarUpdated (C:\Users\masa\AppData\Local\atom\app-1.7.1\resources\app.asar\node_modules\first-mate\lib\grammar.js:279:21) at GrammarRegistry.module.exports.GrammarRegistry.grammarUpdated (C:\Users\masa\AppData\Local\atom\app-1.7.1\resources\app.asar\node_modules\first-mate\lib\grammar-registry.js:180:23) at Grammar.module.exports.Grammar.grammarUpdated (C:\Users\masa\AppData\Local\atom\app-1.7.1\resources\app.asar\node_modules\first-mate\lib\grammar.js:279:21) at GrammarRegistry.module.exports.GrammarRegistry.grammarUpdated (C:\Users\masa\AppData\Local\atom\app-1.7.1\resources\app.asar\node_modules\first-mate\lib\grammar-registry.js:180:23) at Grammar.module.exports.Grammar.grammarUpdated (C:\Users\masa\AppData\Local\atom\app-1.7.1\resources\app.asar\node_modules\first-mate\lib\grammar.js:279:21) at GrammarRegistry.module.exports.GrammarRegistry.grammarUpdated (C:\Users\masa\AppData\Local\atom\app-1.7.1\resources\app.asar\node_modules\first-mate\lib\grammar-registry.js:180:23) at Grammar.module.exports.Grammar.grammarUpdated (C:\Users\masa\AppData\Local\atom\app-1.7.1\resources\app.asar\node_modules\first-mate\lib\grammar.js:279:21) at GrammarRegistry.module.exports.GrammarRegistry.grammarUpdated (C:\Users\masa\AppData\Local\atom\app-1.7.1\resources\app.asar\node_modules\first-mate\lib\grammar-registry.js:180:23) at Grammar.module.exports.Grammar.grammarUpdated (C:\Users\masa\AppData\Local\atom\app-1.7.1\resources\app.asar\node_modules\first-mate\lib\grammar.js:279:21) at GrammarRegistry.module.exports.GrammarRegistry.grammarUpdated (C:\Users\masa\AppData\Local\atom\app-1.7.1\resources\app.asar\node_modules\first-mate\lib\grammar-registry.js:180:23) at Grammar.module.exports.Grammar.grammarUpdated (C:\Users\masa\AppData\Local\atom\app-1.7.1\resources\app.asar\node_modules\first-mate\lib\grammar.js:279:21) at GrammarRegistry.module.exports.GrammarRegistry.grammarUpdated (C:\Users\masa\AppData\Local\atom\app-1.7.1\resources\app.asar\node_modules\first-mate\lib\grammar-registry.js:180:23) at Grammar.module.exports.Grammar.grammarUpdated (C:\Users\masa\AppData\Local\atom\app-1.7.1\resources\app.asar\node_modules\first-mate\lib\grammar.js:279:21) at GrammarRegistry.module.exports.GrammarRegistry.grammarUpdated (C:\Users\masa\AppData\Local\atom\app-1.7.1\resources\app.asar\node_modules\first-mate\lib\grammar-registry.js:180:23) at Grammar.module.exports.Grammar.grammarUpdated (C:\Users\masa\AppData\Local\atom\app-1.7.1\resources\app.asar\node_modules\first-mate\lib\grammar.js:279:21) at GrammarRegistry.module.exports.GrammarRegistry.grammarUpdated (C:\Users\masa\AppData\Local\atom\app-1.7.1\resources\app.asar\node_modules\first-mate\lib\grammar-registry.js:180:23) at Grammar.module.exports.Grammar.grammarUpdated (C:\Users\masa\AppData\Local\atom\app-1.7.1\resources\app.asar\node_modules\first-mate\lib\grammar.js:279:21)

mikoMK commented 8 years ago

Can confirm this for Windows 8.1 and Atom 1.7.3, but only when Atom has to reopen files on a network drive. Local cfm files reopen without a problem.

icimasa commented 8 years ago

Hi, did confirm that too, all of my work is on network drive, and the error is still in 1.7.3! I wonder if this is a bug in the cfml/html language pack or atom itself!

atuttle commented 8 years ago

That part of easy enough to test: disable the plugin and try again. On May 2, 2016 2:03 AM, "icimasa" notifications@github.com wrote:

Hi, did confirm that too, all of my work is on network drive, and the error is still in 1.7.3! I wonder if this is a bug in the cfml/html language pack or atom itself!

— You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub https://github.com/atuttle/atom-language-cfml/issues/85#issuecomment-216112007

icimasa commented 8 years ago

@atuttle well sort of already confirmed that the error must be in the plugin, I just set the language on the files to regular text file and restarted atom without the error and then switched back to cfml/html and restarted and the error came back! So yea must be the cfml/html plugin! :)

mikoMK commented 8 years ago

Sorry, I forgot to mention that above: The error doesn't occure when "language-cfml" is disabled. The error seems to cause no further problems. The cfm/cfc Files are completely loaded and you can still edit and save them and open other files without getting an error again.

icimasa commented 8 years ago

Yes correct everything works fine except that every time you start Atom, you get this error popup! So any new on fixing this?

jcush commented 8 years ago

I'm also getting this error (Win 8.1). Quite frustrating as it completely locks Atom up for a good 10 seconds. Any news on a fix?

dannyrich commented 7 years ago

It seems that this is related to the decisions in https://github.com/atuttle/atom-language-cfml/pull/35. If you force it to use cfml.cson rather than html-cfml.cson this error goes away and it loads quicker, but you lose html syntax highlighting if you do this.

icimasa commented 7 years ago

Ohh okay,,

Seems like relly sad if you cannot have cfml and html at the same time. So for me it is not an option, I solved it by forcing atom to allways open empty and I have to open up app pages from scratch.. ☹

Hope that this will be solved sometimes..

//Martin

From: Danny Richelieu [mailto:notifications@github.com] Sent: Thursday, September 14, 2017 20:03 To: atuttle/atom-language-cfml atom-language-cfml@noreply.github.com Cc: Martin Sagerstrand masa@hms.se; Author author@noreply.github.com Subject: Re: [atuttle/atom-language-cfml] Starting Atom: Uncaught RangeError: Maximum call stack size exceeded (#85)

It seems that this is related to the decisions in #35https://github.com/atuttle/atom-language-cfml/pull/35. If you force it to use cfml.cson rather than html-cfml.cson this error goes away and it loads quicker, but you lose html syntax highlighting if you do this.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/atuttle/atom-language-cfml/issues/85#issuecomment-329562405, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AOiX8qiPRtq0tQS-c3bu7hrJtRjzWFnqks5siWpcgaJpZM4IQC2e.

David-Polehonski commented 6 years ago

Is this still an Issue?

jcush commented 6 years ago

For one I use VSCode now so I can't comment on if this is still an issue, sorry.

I don't know how Atom plugins are built, but looking at the above if there's some way to 'import' syntax highlighting rules then maybe that could be done in some way so you stick on cfml.cson for the quick, bug-free loading but still inherit html highlighting rules?

David-Polehonski commented 6 years ago

I don't experience this issue when loading atom with a file already loaded. So unless someone else thinks otherwise it may be worth closing this down for now?

icimasa commented 6 years ago

I haven't seen this error for a while now, but could also be that I changed how I work, when I open atom I always open it empty cause I work in so many different projects it was no point in opening up the last project with all tabs open..