Skip to content

Commit 9426919

Browse files
committed
COMMENT: Added two TODOs.
1 parent 9ce49b3 commit 9426919

3 files changed

Lines changed: 5 additions & 5 deletions

File tree

src/main/java/jce/codemanipulation/ImportOrganizer.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ public ImportOrganizer(EcorificationProperties properties) {
3737
@Override
3838
protected void manipulate(ICompilationUnit unit) throws JavaModelException {
3939
unit.becomeWorkingCopy(monitor); // changes unit handle to working copy
40-
CompilationUnit reconciledUnit = unit.reconcile(AST.JLS8, false, null, monitor); // Don't ask me.
40+
CompilationUnit reconciledUnit = unit.reconcile(AST.JLS8, false, null, monitor); // TODO (MEDIUM) replace with AST.JLS10 eventually
4141
try {
4242
OrganizeImportsOperation operation = new OrganizeImportsOperation(unit, reconciledUnit, true, true, true, null);
4343
TextEdit edit = operation.createTextEdit(monitor);

src/main/java/jce/generators/WrapperGenerator.xtend

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -49,12 +49,12 @@ final class WrapperGenerator extends ClassGenerator {
4949
* @param path is the current file path of the {@link EPackage}. Should be initially an empty string.
5050
*/
5151
def private void buildWrappers(EPackage ePackage, String path) {
52-
if(containsEClass(ePackage)) { // avoids empty folders
52+
if (containsEClass(ePackage)) { // avoids empty folders
5353
createFolder(append(wrapperFolder, path), javaProject.project)
5454
}
5555
for (eClassifier : ePackage.EClassifiers) { // for every classifier
56-
if(eClassifier instanceof EClass) { // if is EClass
57-
if(!eClassifier.interface && !isRootContainer(eClassifier, path)) { // if is not interface or root
56+
if (eClassifier instanceof EClass) { // if is EClass
57+
if (!eClassifier.interface && !isRootContainer(eClassifier, path)) { // if is not interface or root
5858
createXtendWrapper(eClassifier, path) // create wrapper class
5959
}
6060
}

src/main/java/jce/util/jdt/ASTUtil.xtend

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -95,7 +95,7 @@ final class ASTUtil {
9595
*/
9696
def static CompilationUnit parse(ICompilationUnit unit, IProgressMonitor monitor) throws JavaModelException {
9797
unit.becomeWorkingCopy(monitor)
98-
var ASTParser parser = ASTParser.newParser(AST.JLS8)
98+
var ASTParser parser = ASTParser.newParser(AST.JLS8) // TODO (MEDIUM) replace with AST.JLS10 eventually
9999
var Map<String, String> options = JavaCore.options
100100
JavaCore.setComplianceOptions(JavaCore.VERSION_1_8, options)
101101
parser.setCompilerOptions(options)

0 commit comments

Comments
 (0)