Merge branch 'refs/heads/refactor/YamlOutputStreamWriter' into refact…

…or/StreamReader
krzema12 · May 11, 2024 · a8149be · a8149be
2 parents d7d6fca + ef584aa
commit a8149be
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 13 deletions.
diff --git a/src/jvmMain/java/it/krzeminski/snakeyaml/engine/kmp/api/YamlOutputStreamWriter.kt b/src/jvmMain/java/it/krzeminski/snakeyaml/engine/kmp/api/YamlOutputStreamWriter.kt
@@ -24,16 +24,15 @@ import java.nio.charset.Charset
  * @param out the output
  * @param cs encoding to use to translate String to bytes
  */
-abstract class YamlOutputStreamWriter(
+open class YamlOutputStreamWriter(
     out: OutputStream,
     cs: Charset,
 ) : OutputStreamWriter(out, cs), StreamDataWriter {
-    /**
-     * to be implemented
-     *
-     * @param e - the reason
-     */
-    abstract fun processIOException(e: IOException?)
+
+    /** Handle [IOException]s thrown while writing. */
+    open fun processIOException(e: IOException) {
+        throw e
+    }
 
     override fun flush() {
         try {

diff --git a/src/jvmTest/java/org/snakeyaml/engine/v2/api/DumpTest.java b/src/jvmTest/java/org/snakeyaml/engine/v2/api/DumpTest.java
@@ -120,12 +120,7 @@ void dumpToFile() throws IOException {
     assertFalse(file.exists());
     file.getParentFile().mkdirs();
     file.createNewFile();
-    StreamDataWriter writer = new YamlOutputStreamWriter(new FileOutputStream(file), StandardCharsets.UTF_8) {
-      @Override
-      public void processIOException(IOException e) {
-        throw new RuntimeException(e);
-      }
-    };
+    StreamDataWriter writer = new YamlOutputStreamWriter(new FileOutputStream(file), StandardCharsets.UTF_8);
     dump.dump(ImmutableMap.of("x", 1, "y", 2, "z", 3), writer);
     assertTrue(file.exists());
     file.delete();// on Windows the file is not deleted