Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

when there is only one project matching the mainclass, projectName is not required for evaluation. #156

Merged
merged 5 commits into from
Mar 14, 2018

Conversation

andxu
Copy link
Contributor

@andxu andxu commented Mar 14, 2018

No description provided.

@@ -14,4 +14,5 @@
public final class Constants {
public static final String PROJECTNAME = "projectName";
public static final String DEBUGGEE_ENCODING = "debuggeeEncoding";
public static final String MAINCLASS = "mainClass";
Copy link
Contributor

Choose a reason for hiding this comment

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

MAIN_CLASS

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done with the same issue fixed for project name

*
* @param mainclass the main class specified by launch.json for finding possible projects
*/
private void initializePossibleProjects(String mainclass) {
Copy link
Contributor

Choose a reason for hiding this comment

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

initializeProjectCandidates

// ignore
}

logger.severe("Cannot evaluate when project is not specified.");
Copy link
Contributor

Choose a reason for hiding this comment

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

"Cannot evaluate when the project is not specified

Copy link
Contributor

@yaohaizh yaohaizh left a comment

Choose a reason for hiding this comment

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

Minor issues

@andxu andxu merged commit 6d6b930 into andy_cond3 Mar 14, 2018
@andxu andxu deleted the andy_missing_projectname branch March 14, 2018 08:18
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.

2 participants