added comments
diff --git a/plugins/org.eclipse.mylyn.docs.intent.compare/src/org/eclipse/mylyn/docs/intent/compare/match/IntentSimilarityChecker.java b/plugins/org.eclipse.mylyn.docs.intent.compare/src/org/eclipse/mylyn/docs/intent/compare/match/IntentSimilarityChecker.java
index 4a55e59..96c14cf 100644
--- a/plugins/org.eclipse.mylyn.docs.intent.compare/src/org/eclipse/mylyn/docs/intent/compare/match/IntentSimilarityChecker.java
+++ b/plugins/org.eclipse.mylyn.docs.intent.compare/src/org/eclipse/mylyn/docs/intent/compare/match/IntentSimilarityChecker.java
@@ -137,13 +137,13 @@
* @throws FactoryException
* - on error accessing features.
*/
- protected boolean areSimilarSessions(IntentSection session1, IntentSection session2)
+ protected boolean areSimilarSessions(IntentSection session1, IntentSection session2) //TODO FIXME [naming] section should be better than session
throws FactoryException {
// 2 session are equals if :
// they have the same title
Block title1 = session1.getTitle();
Block title2 = session2.getTitle();
-
+ // TODO FIXME if section don't have any title yet, NPE
return isSimilar(title1, title2);
}
diff --git a/plugins/org.eclipse.mylyn.docs.intent.parser/src/org/eclipse/mylyn/docs/intent/parser/internal/IntentBuilder.java b/plugins/org.eclipse.mylyn.docs.intent.parser/src/org/eclipse/mylyn/docs/intent/parser/internal/IntentBuilder.java
index 83e06d8..764ce98 100644
--- a/plugins/org.eclipse.mylyn.docs.intent.parser/src/org/eclipse/mylyn/docs/intent/parser/internal/IntentBuilder.java
+++ b/plugins/org.eclipse.mylyn.docs.intent.parser/src/org/eclipse/mylyn/docs/intent/parser/internal/IntentBuilder.java
@@ -257,6 +257,7 @@
* if the description unit parser detect any parse error.
*/
public void descriptionUnitContent(int offset, String descriptionUnitContent) throws ParseException {
+ // TODO FIXME sometimes currentState can be null
currentState = currentState.descriptionUnitContent(offset, descriptionUnitContent.length(),
formatUsingImbricationLevel(descriptionUnitContent, false));
}
