winterstein / Eclipse-Markdown-Editor-Plugin

Adds Markdown text editing support to Eclipse
Eclipse Public License 1.0
297 stars 73 forks source link

When opening a file ran into StackOverflow and Eclipse asking to exit (on Eclipse 4.3.0 only?) #33

Open paulvi opened 10 years ago

paulvi commented 10 years ago

When opening a file below ran into StackOverflow and Eclipse asking to exit

https://github.com/felixge/node-mysql/blob/master/Readme.md

(Eclipse failed to start, only renaming project folder (thus making Readme.md unavailable) )

There is definitely circular calls, with no visible class to blame (stack too long).

 Unhandled event loop exception

eclipse.buildId=4.3.0.I20130605-2000 java.version=1.7.0_40

java.lang.StackOverflowError
    at java.util.regex.Pattern$Slice.match(Pattern.java:3867)
    at java.util.regex.Pattern$LazyLoop.match(Pattern.java:4741)
    at java.util.regex.Pattern$GroupTail.match(Pattern.java:4615)
    at java.util.regex.Pattern$BmpCharProperty.match(Pattern.java:3715)
    at java.util.regex.Pattern$Curly.match0(Pattern.java:4170)
    at java.util.regex.Pattern$Curly.match(Pattern.java:4132)
    at java.util.regex.Pattern$GroupHead.match(Pattern.java:4556)
    at java.util.regex.Pattern$LazyLoop.match(Pattern.java:4745)
    at java.util.regex.Pattern$GroupTail.match(Pattern.java:4615)
    at java.util.regex.Pattern$BmpCharProperty.match(Pattern.java:3715)
    at java.util.regex.Pattern$Curly.match0(Pattern.java:4170)
    at java.util.regex.Pattern$Curly.match(Pattern.java:4132)
    at java.util.regex.Pattern$GroupHead.match(Pattern.java:4556)
    at java.util.regex.Pattern$LazyLoop.match(Pattern.java:4745)
    at java.util.regex.Pattern$GroupTail.match(Pattern.java:4615)
    at java.util.regex.Pattern$BmpCharProperty.match(Pattern.java:3715)
    at java.util.regex.Pattern$Curly.match0(Pattern.java:4170)
    at java.util.regex.Pattern$Curly.match(Pattern.java:4132)
    at java.util.regex.Pattern$GroupHead.match(Pattern.java:4556)
    at java.util.regex.Pattern$LazyLoop.match(Pattern.java:4745)
    at java.util.regex.Pattern$GroupTail.match(Pattern.java:4615)
    at java.util.regex.Pattern$BmpCharProperty.match(Pattern.java:3715)
    at java.util.regex.Pattern$Curly.match0(Pattern.java:4170)
    at java.util.regex.Pattern$Curly.match(Pattern.java:4132)
    at java.util.regex.Pattern$GroupHead.match(Pattern.java:4556)
    at java.util.regex.Pattern$LazyLoop.match(Pattern.java:4745)
    at java.util.regex.Pattern$GroupTail.match(Pattern.java:4615)
    at java.util.regex.Pattern$BmpCharProperty.match(Pattern.java:3715)
    at java.util.regex.Pattern$Curly.match0(Pattern.java:4177)
    at java.util.regex.Pattern$Curly.match(Pattern.java:4132)
    at java.util.regex.Pattern$GroupHead.match(Pattern.java:4556)
    at java.util.regex.Pattern$LazyLoop.match(Pattern.java:4745)
    at java.util.regex.Pattern$GroupTail.match(Pattern.java:4615)
    at java.util.regex.Pattern$BmpCharProperty.match(Pattern.java:3715)
    at java.util.regex.Pattern$Curly.match0(Pattern.java:4170)
    at java.util.regex.Pattern$Curly.match(Pattern.java:4132)
    at java.util.regex.Pattern$GroupHead.match(Pattern.java:4556)
    at java.util.regex.Pattern$LazyLoop.match(Pattern.java:4745)
    at java.util.regex.Pattern$GroupTail.match(Pattern.java:4615)
    at java.util.regex.Pattern$BmpCharProperty.match(Pattern.java:3715)
    at java.util.regex.Pattern$Curly.match0(Pattern.java:4170)
    at java.util.regex.Pattern$Curly.match(Pattern.java:4132)
    at java.util.regex.Pattern$GroupHead.match(Pattern.java:4556)
    at java.util.regex.Pattern$LazyLoop.match(Pattern.java:4745)
    at java.util.regex.Pattern$GroupTail.match(Pattern.java:4615)
    at java.util.regex.Pattern$BmpCharProperty.match(Pattern.java:3715)
    at java.util.regex.Pattern$Curly.match0(Pattern.java:4170)
    at java.util.regex.Pattern$Curly.match(Pattern.java:4132)

and so on

winterstein commented 10 years ago

Thanks for the bug report Paul, and sorry about the bug. I think I have a lead on this one -- will investigate today...

paulvi commented 10 years ago

Has baby grown up already ? :-)

Will be great if you can devote some time again.

winterstein commented 10 years ago

Hi Paul,

Sorry, I have to report drawing a blank on this bug.
When I try the example file, it works fine for me.

I have seen stack-overflows myself occasionally -- thought I had some
likely test cases -- but no.

There's only one use of Pattern in the plugin code itself
(MarkdownPage.multiMarkdownTag).
I don't think that could cause a stackoverflow.

What version of Eclipse and what OS are you running?
Do you get this error every time you open the file, or is it intermittent?

Thank you & Best regards,

paulvi commented 10 years ago

At work and at home Windows 7 with Eclipse 4.3.x (above version is reported as 4.3.0)

I have just tried again with 4.3.1 and it was OK.

winterstein commented 10 years ago

On 8 January 2014 09:24, Paul Verest notifications@github.com wrote:

Has baby grown up already ? :-)

Not quite -- but only a decade or so to go.

At work and at home Windows 7 with Eclipse 4.3.x (above version is reported as 4.3.0)

I have just tried again with 4.3.1 and it was OK If the bug goes away on upgrade -- I guess we mark this one as a yeti; something slightly broken in one version of Eclipse and fixed in the next.

Best regards,

paulvi commented 10 years ago

I think this issue should be opened with wontfix tag, so that users know they may need to update Eclipse.

Otherwise README should be updated to say that there is bug on 4.3.0

winterstein commented 10 years ago

Good point. - Dan

On 23 January 2014 06:58, Paul Verest notifications@github.com wrote:

I think this issue should be opened with wontfix tag, so that users know they may need to update Eclipse.

Otherwise README should be updated to say that there is bug on 4.3.0

— Reply to this email directly or view it on GitHubhttps://github.com/winterstein/Eclipse-Markdown-Editor-Plugin/issues/33#issuecomment-33101355 .

Dr Daniel Winterstein Director

A: TechCube, Edinburgh, EH9 1PL M: +44 (0)772 5172 612 http://winterwell.com http://sodash.com

eremmel commented 10 years ago

I run in comparable issue (at least same stack according jvisualvm. Stil no StackOverflow due to 64bit JVM. I'm at eclipse 4.3.2v20140221... It happened today two times during Java comment source editing. I found on internet a few bug reports on the regexp in Java (JDK-6882582). Not sure how to find the misbehaving regex.

paulvi commented 9 years ago

@eremmel Can you reproduce now as of 2015 with Luna ?

paulvi commented 9 years ago

I don't have problem with https://raw.githubusercontent.com/felixge/node-mysql/master/Readme.md on Luna

jiangxincode commented 8 years ago

Maybe the file is too large. I guess.