Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 320522 - JSP discoverer : java heap space with some misc characters
Summary: JSP discoverer : java heap space with some misc characters
Status: CLOSED FIXED
Alias: None
Product: MoDisco
Classification: Modeling
Component: Technologies (show other bugs)
Version: 0.9.0   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 0.9.0 M1   Edit
Assignee: Fabien Giquel CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-07-21 11:42 EDT by Fabien Giquel CLA
Modified: 2010-10-18 08:34 EDT (History)
3 users (show)

See Also:
fabien.giquel: indigo+

Attachments
Jsp sample with misc characters (78 bytes, application/octet-stream)
2010-07-21 11:42 EDT, Fabien Giquel CLA 		no flags Details
Patch to add LATIN-1 SUPPLEMENT character to the Lexer (737 bytes, patch)
2010-07-23 06:37 EDT, Nicolas Guyomar CLA 		fabien.giquel: iplog+
Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Fabien Giquel CLA 2010-07-21 11:42:13 EDT 
Created attachment 174870 [details]
Jsp sample with misc characters

Here is an example of JSP containing specific characters which are not supported by the JSP discoverer.
Comment 1 Nicolas Guyomar CLA 2010-07-23 06:36:25 EDT 
Hi Fabien,

I added the UNICODE CHARACTER TABLE: LATIN-1 SUPPLEMENT  to the list of token in the attached patch. Those character were missing


1. I, Nicolas Guyomar, and others contributors listed in this bug, authored
100% of the content
2. I have the rights to donate the content to Eclipse under the EPL
3. I have already provided the filled Employer Consent Forms to the Foundation


Regards

Nicolas
Comment 2 Nicolas Guyomar CLA 2010-07-23 06:37:07 EDT 
Created attachment 175049 [details]
Patch to add LATIN-1 SUPPLEMENT character to the Lexer
Comment 3 Fabien Giquel CLA 2010-07-23 08:43:16 EDT 
The fix has been commited into SVN.
Comment 4 Hugo Bruneliere CLA 2010-10-05 04:23:32 EDT 
The patch has been successfully applied.