refactor: changed how comment blocks are handled

We now consider all consecutive comment lines to be
part of the same block, regardless of the comment
prefix used for each line. (So no longer do we take
a block to end when the comment prefix changes)

Author: quintesse

src/main/java/org/codejive/properties/Properties.java

@@ -269,15 +269,8 @@ private List<Integer> findCommentLines(int idx) {
             idx--;
         }
         // Now find the first line of the comment block
-        int commentSym = -1;
         PropertiesParser.Token token;
         while (idx >= 0 && (token = tokens.get(idx)).getType() == PropertiesParser.Type.COMMENT) {
-            if (commentSym != -1 && commentSym != token.raw.charAt(0)) {
-                // Comment doesn't start with the same comment symbol, so the block ends here
-                break;
-            } else {
-                commentSym = token.raw.charAt(0);
-            }
             result.add(0, idx);
             // Skip any preceding whitespace making sure to stop at EOL
             while (--idx >= 0 && !tokens.get(idx).isEol()) {}

src/test/java/org/codejive/properties/TestProperties.java

@@ -68,7 +68,7 @@ void testGetComment() throws IOException, URISyntaxException {
         Properties p = Properties.loadProperties(getResource("/test.properties"));
         assertThat(p.getComment("one"), contains("! comment3"));
         assertThat(p.getComment("two"), empty());
-        assertThat(p.getComment("three"), contains("! and a comment", "! block"));
+        assertThat(p.getComment("three"), contains("# another comment", "! and a comment", "! block"));
     }
 
     @Test

src/test/resources/test-comment.properties

@@ -7,7 +7,6 @@ one=simple
 # line
 # comment
 two=value containing spaces
-# another comment
 three=and escapes\n\t\r\f
 \ with\ spaces   =    everywhere  
 altsep:value