Remove explicit Properties.File#getContent() method

While indeed the LST should not be modified by modifying a `List` returned by a getter, it is also not right for a getter to return a new list every time (e.g. by wrapping an internal field using `Collections.unmodifiableList()`.

This commit removes the explicit getter and wither and instead has these generated by Lombok. Further, the initial contents, as populated by the parser, are now wrapped using `Collections.unmodifiableList()` to avoid any situations, where the contents are directly modified.

rewrite-properties/src/main/java/org/openrewrite/properties/PropertiesParser.java

@@ -27,6 +27,7 @@
 
 import java.nio.file.Path;
 import java.util.ArrayList;
+import java.util.Collections;
 import java.util.List;
 import java.util.stream.Stream;
 
@@ -103,7 +104,7 @@ private Properties.File parseFromInput(Path sourceFile, EncodingDetectingInputSt
                 "",
                 Markers.EMPTY,
                 sourceFile,
-                contents,
+                Collections.unmodifiableList(contents),
                 prefix.toString(),
                 source.getCharset().name(),
                 source.isCharsetBomMarked(),

rewrite-properties/src/main/java/org/openrewrite/properties/tree/Properties.java

@@ -28,8 +28,6 @@
 import java.nio.charset.Charset;
 import java.nio.charset.StandardCharsets;
 import java.nio.file.Path;
-import java.util.ArrayList;
-import java.util.Collections;
 import java.util.List;
 import java.util.UUID;
 import java.util.regex.Pattern;
@@ -70,6 +68,7 @@ class File implements Properties, SourceFile {
         @With
         Path sourcePath;
 
+        @With
         List<Content> content;
         @With
         String eof;
@@ -100,28 +99,6 @@ public SourceFile withCharset(Charset charset) {
             return withCharsetName(charset.name());
         }
 
-        public List<Content> getContent() {
-            return Collections.unmodifiableList(content);
-        }
-
-        public File withContent(List<Content> content) {
-            int size = content.size();
-            if (size == this.content.size()) {
-                boolean allIdentical = true;
-                for (int i = 0; i < size; i++) {
-                    if (content.get(i) != this.content.get(i)) {
-                        allIdentical = false;
-                        break;
-                    }
-                }
-                if (allIdentical) {
-                    return this;
-                }
-            }
-
-            return new File(id, prefix, markers, sourcePath, new ArrayList<>(content), eof, charsetName, charsetBomMarked, fileAttributes, checksum);
-        }
-
         @Override
         public <P> Properties acceptProperties(PropertiesVisitor<P> v, P p) {
             return v.visitFile(this, p);