Skip to content

ROASTER-72 Adds a new API to parse the multiple declarations of type in a file.#34

Closed
kuzukami wants to merge 4 commits intoforge:masterfrom
kuzukami:master
Closed

ROASTER-72 Adds a new API to parse the multiple declarations of type in a file.#34
kuzukami wants to merge 4 commits intoforge:masterfrom
kuzukami:master

Conversation

@kuzukami
Copy link
Copy Markdown

@kuzukami kuzukami commented Jul 5, 2015

ROASTER-72: Adds a new API and data structure to parse the multiple declarations of
classes in a java file.
An instance of the 'JavaSourceUnit' represents a java source file which
contains multiple declarations of type.
An API named Roaster.parse2(String encoding, InputStream data) is
tentatively created to extract the JavaSourceUnit result.
Some tests for the new function are created at
'JavaClassMultiDeclsInAFile.java'.

classes in a java file.
An instance of the 'JavaSourceUnit' represents a java source file which
contains multiple declarations of type.
A API named Roaster.parse2(String encoding, InputStream data) is
tentatively created to extract the JavaSourceUnit result.
Some tests for the new function are created at
'JavaClassMultiDeclsInAFile.java'.
@forge-bot
Copy link
Copy Markdown
Contributor

Build 57 is now running using a merge of 1e5b593

@forge-bot
Copy link
Copy Markdown
Contributor

Build 57 is now running using a merge of 1e5b593

@forge-bot
Copy link
Copy Markdown
Contributor

Build 57 outcome was SUCCESS using a merge of 1e5b593
Summary: Tests passed: 752, ignored: 52 Build time: 0:00:27

@forge-bot
Copy link
Copy Markdown
Contributor

Build 57 outcome was SUCCESS using a merge of 1e5b593
Summary: Tests passed: 752, ignored: 52 Build time: 0:00:28

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why add this method to Roaster?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If you don't need this method, please remove the next release time.

@gastaldi
Copy link
Copy Markdown
Member

gastaldi commented Jul 5, 2015

Hi kuzukami. Thank you for the PR and your effort on this. Can you create a JIRA in https://issues.jboss.org/browse/ROASTER?

@kuzukami kuzukami changed the title Adds a new API to parse the multiple declarations of type in a file. ROASTER-72 Adds a new API to parse the multiple declarations of type in a file. Jul 5, 2015
@forge-bot
Copy link
Copy Markdown
Contributor

Build 58 is now running using a merge of babb76a

@forge-bot
Copy link
Copy Markdown
Contributor

Build 58 is now running using a merge of babb76a

@forge-bot
Copy link
Copy Markdown
Contributor

Build 58 outcome was SUCCESS using a merge of babb76a
Summary: Tests passed: 752, ignored: 52 Build time: 0:00:25

@forge-bot
Copy link
Copy Markdown
Contributor

Build 58 outcome was SUCCESS using a merge of babb76a
Summary: Tests passed: 752, ignored: 52 Build time: 0:00:27

kuzukami.sh added 2 commits July 7, 2015 13:42
code nearer to the original code as well as it can instead of such
formatted code as in #toString().
@kuzukami
Copy link
Copy Markdown
Author

kuzukami commented Jul 8, 2015

There is a bug in this pull request. I'll pull-req again.

@kuzukami kuzukami closed this Jul 8, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants