You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by mi...@apache.org on 2022/12/03 23:30:39 UTC

[maven-doxia] branch DOXIA-679 created (now 4f83ce50)

This is an automated email from the ASF dual-hosted git repository.

michaelo pushed a change to branch DOXIA-679
in repository https://gitbox.apache.org/repos/asf/maven-doxia.git


      at 4f83ce50 [DOXIA-679] Introduce Sink#tableRows(void) method

This branch includes the following new commits:

     new 4f83ce50 [DOXIA-679] Introduce Sink#tableRows(void) method

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.



[maven-doxia] 01/01: [DOXIA-679] Introduce Sink#tableRows(void) method

Posted by mi...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

michaelo pushed a commit to branch DOXIA-679
in repository https://gitbox.apache.org/repos/asf/maven-doxia.git

commit 4f83ce500e67c5e58ceac97cff5e811c00224836
Author: Michael Osipov <mi...@apache.org>
AuthorDate: Sun Dec 4 00:29:09 2022 +0100

    [DOXIA-679] Introduce Sink#tableRows(void) method
    
    This closes #127
---
 .../apache/maven/doxia/sink/impl/RandomAccessSink.java    |  7 +++++++
 .../org/apache/maven/doxia/sink/impl/SinkAdapter.java     |  9 ++++++++-
 .../org/apache/maven/doxia/sink/impl/Xhtml5BaseSink.java  |  6 ++++++
 .../org/apache/maven/doxia/sink/impl/SinkAdapterTest.java |  4 +---
 .../maven/doxia/sink/impl/SinkEventTestingSink.java       |  6 ++++++
 .../java/org/apache/maven/doxia/sink/impl/TextSink.java   |  8 +++++++-
 .../maven/doxia/sink/impl/WellformednessCheckingSink.java |  8 +++++++-
 .../apache/maven/doxia/sink/impl/Xhtml5BaseSinkTest.java  | 15 ++++++---------
 .../java/org/apache/maven/doxia/module/apt/AptSink.java   |  6 ++++++
 .../src/main/java/org/apache/maven/doxia/sink/Sink.java   |  7 +++++++
 10 files changed, 61 insertions(+), 15 deletions(-)

diff --git a/doxia-core/src/main/java/org/apache/maven/doxia/sink/impl/RandomAccessSink.java b/doxia-core/src/main/java/org/apache/maven/doxia/sink/impl/RandomAccessSink.java
index 87950fd2..4663af25 100644
--- a/doxia-core/src/main/java/org/apache/maven/doxia/sink/impl/RandomAccessSink.java
+++ b/doxia-core/src/main/java/org/apache/maven/doxia/sink/impl/RandomAccessSink.java
@@ -1196,6 +1196,13 @@ public class RandomAccessSink
         currentSink.tableRow_();
     }
 
+    /** {@inheritDoc} */
+    @Override
+    public void tableRows()
+    {
+        currentSink.tableRows();
+    }
+
     /** {@inheritDoc} */
     @Override
     public void tableRows( int[] justification, boolean grid )
diff --git a/doxia-core/src/main/java/org/apache/maven/doxia/sink/impl/SinkAdapter.java b/doxia-core/src/main/java/org/apache/maven/doxia/sink/impl/SinkAdapter.java
index 307735b7..f0cc09ae 100644
--- a/doxia-core/src/main/java/org/apache/maven/doxia/sink/impl/SinkAdapter.java
+++ b/doxia-core/src/main/java/org/apache/maven/doxia/sink/impl/SinkAdapter.java
@@ -299,7 +299,7 @@ public class SinkAdapter
 
     /** {@inheritDoc} */
     @Override
-    public void tableRows( int[] justification, boolean grid )
+    public void tableRows()
     {
         // nop
     }
@@ -1079,6 +1079,13 @@ public class SinkAdapter
         table();
     }
 
+    /** {@inheritDoc} */
+    @Override
+    public void tableRows( int[] justification, boolean grid )
+    {
+        tableRows();
+    }
+
     /** {@inheritDoc} */
     @Override
     public void tableRow( SinkEventAttributes attributes )
diff --git a/doxia-core/src/main/java/org/apache/maven/doxia/sink/impl/Xhtml5BaseSink.java b/doxia-core/src/main/java/org/apache/maven/doxia/sink/impl/Xhtml5BaseSink.java
index 817156bc..e35cd901 100644
--- a/doxia-core/src/main/java/org/apache/maven/doxia/sink/impl/Xhtml5BaseSink.java
+++ b/doxia-core/src/main/java/org/apache/maven/doxia/sink/impl/Xhtml5BaseSink.java
@@ -1436,6 +1436,12 @@ public class Xhtml5BaseSink
         }
     }
 
+    @Override
+    public void tableRows()
+    {
+        tableRows( null, false );
+    }
+
     /**
      * The default class style is <code>bodyTable</code>.
      *
diff --git a/doxia-core/src/test/java/org/apache/maven/doxia/sink/impl/SinkAdapterTest.java b/doxia-core/src/test/java/org/apache/maven/doxia/sink/impl/SinkAdapterTest.java
index e29bcc82..d280bbcf 100644
--- a/doxia-core/src/test/java/org/apache/maven/doxia/sink/impl/SinkAdapterTest.java
+++ b/doxia-core/src/test/java/org/apache/maven/doxia/sink/impl/SinkAdapterTest.java
@@ -258,9 +258,7 @@ public class SinkAdapterTest
     @Test
     public void testTableRows()
     {
-        final int[] justification = null;
-        final boolean grid = false;
-        instance.tableRows( justification, grid );
+        instance.tableRows();
         instance.tableRows_();
     }
 
diff --git a/doxia-core/src/test/java/org/apache/maven/doxia/sink/impl/SinkEventTestingSink.java b/doxia-core/src/test/java/org/apache/maven/doxia/sink/impl/SinkEventTestingSink.java
index 5b302b8f..a4a4e5c8 100644
--- a/doxia-core/src/test/java/org/apache/maven/doxia/sink/impl/SinkEventTestingSink.java
+++ b/doxia-core/src/test/java/org/apache/maven/doxia/sink/impl/SinkEventTestingSink.java
@@ -286,6 +286,12 @@ public class SinkEventTestingSink
         addEvent( "table_" );
     }
 
+    @Override
+    public void tableRows()
+    {
+        addEvent( "tableRows", new Object[] { null, false } );
+    }
+
     @Override
     public void tableRows( int[] justification, boolean grid )
     {
diff --git a/doxia-core/src/test/java/org/apache/maven/doxia/sink/impl/TextSink.java b/doxia-core/src/test/java/org/apache/maven/doxia/sink/impl/TextSink.java
index b5a34f76..d00b7b46 100644
--- a/doxia-core/src/test/java/org/apache/maven/doxia/sink/impl/TextSink.java
+++ b/doxia-core/src/test/java/org/apache/maven/doxia/sink/impl/TextSink.java
@@ -278,7 +278,7 @@ public class TextSink
     }
 
     @Override
-    public void tableRows( int[] justification, boolean grid )
+    public void tableRows()
     {
         writeln( "begin:tableRows" );
     }
@@ -897,6 +897,12 @@ public class TextSink
         table();
     }
 
+    @Override
+    public void tableRows( int[] justification, boolean grid )
+    {
+        tableRows();
+    }
+
     @Override
     public void tableRow( SinkEventAttributes attributes )
     {
diff --git a/doxia-core/src/test/java/org/apache/maven/doxia/sink/impl/WellformednessCheckingSink.java b/doxia-core/src/test/java/org/apache/maven/doxia/sink/impl/WellformednessCheckingSink.java
index 301b2968..ebd864a9 100644
--- a/doxia-core/src/test/java/org/apache/maven/doxia/sink/impl/WellformednessCheckingSink.java
+++ b/doxia-core/src/test/java/org/apache/maven/doxia/sink/impl/WellformednessCheckingSink.java
@@ -305,7 +305,7 @@ public class WellformednessCheckingSink
     }
 
     @Override
-    public void tableRows( int[] justification, boolean grid )
+    public void tableRows()
     {
         startElement( "tableRows" );
     }
@@ -940,6 +940,12 @@ public class WellformednessCheckingSink
         table();
     }
 
+    @Override
+    public void tableRows( int[] justification, boolean grid )
+    {
+        tableRows();
+    }
+
     @Override
     public void tableRow( SinkEventAttributes attributes )
     {
diff --git a/doxia-core/src/test/java/org/apache/maven/doxia/sink/impl/Xhtml5BaseSinkTest.java b/doxia-core/src/test/java/org/apache/maven/doxia/sink/impl/Xhtml5BaseSinkTest.java
index 61838c51..79bf6fe9 100644
--- a/doxia-core/src/test/java/org/apache/maven/doxia/sink/impl/Xhtml5BaseSinkTest.java
+++ b/doxia-core/src/test/java/org/apache/maven/doxia/sink/impl/Xhtml5BaseSinkTest.java
@@ -1057,15 +1057,12 @@ public class Xhtml5BaseSinkTest
     @Test
     public void testTableRows()
     {
-        final int[] justification = null;
-        final boolean grid = false;
-
         try
         {
             sink = new Xhtml5BaseSink( writer );
 
             sink.table();
-            sink.tableRows( justification, grid );
+            sink.tableRows();
             sink.tableRows_();
             sink.table_();
         }
@@ -1088,7 +1085,7 @@ public class Xhtml5BaseSinkTest
             sink = new Xhtml5BaseSink( writer );
 
             sink.table();
-            sink.tableRows( null, false );
+            sink.tableRows();
             sink.tableRow( attributes );
             sink.tableRow_();
             sink.tableRow();
@@ -1121,7 +1118,7 @@ public class Xhtml5BaseSinkTest
             sink = new Xhtml5BaseSink( writer );
 
             sink.table();
-            sink.tableRows( null, false );
+            sink.tableRows();
             sink.tableRow();
             sink.tableRow_();
             sink.tableRow( attributes );
@@ -1225,7 +1222,7 @@ public class Xhtml5BaseSinkTest
             sink = new Xhtml5BaseSink( writer );
 
             sink.table();
-            sink.tableRows( null, false );
+            sink.tableRows();
             sink.tableCaption( attributes );
             sink.text( "caption" );
             sink.tableCaption_();
@@ -1626,8 +1623,8 @@ public class Xhtml5BaseSinkTest
         try
         {
             sink = new Xhtml5BaseSink( writer );
-            sink.table( null );
-            sink.tableRows( null, true );
+            sink.table();
+            sink.tableRows();
             sink.tableRow( null );
             sink.tableCell();
             sink.text( "\u2713", null );
diff --git a/doxia-modules/doxia-module-apt/src/main/java/org/apache/maven/doxia/module/apt/AptSink.java b/doxia-modules/doxia-module-apt/src/main/java/org/apache/maven/doxia/module/apt/AptSink.java
index c263fb9a..6273e16b 100644
--- a/doxia-modules/doxia-module-apt/src/main/java/org/apache/maven/doxia/module/apt/AptSink.java
+++ b/doxia-modules/doxia-module-apt/src/main/java/org/apache/maven/doxia/module/apt/AptSink.java
@@ -703,6 +703,12 @@ public class AptSink
         resetTableCaptionBuffer();
     }
 
+    @Override
+    public void tableRows()
+    {
+        tableRows( null, false );
+    }
+
     /** {@inheritDoc} */
     public void tableRows( int[] justification, boolean grid )
     {
diff --git a/doxia-sink-api/src/main/java/org/apache/maven/doxia/sink/Sink.java b/doxia-sink-api/src/main/java/org/apache/maven/doxia/sink/Sink.java
index 6f90503c..6eb57d50 100644
--- a/doxia-sink-api/src/main/java/org/apache/maven/doxia/sink/Sink.java
+++ b/doxia-sink-api/src/main/java/org/apache/maven/doxia/sink/Sink.java
@@ -1027,6 +1027,13 @@ public interface Sink
      */
     void table_();
 
+    /**
+     * Starts an element that contains rows of table data.
+     *
+     * @See {@link #tableRows(int[], boolean)}
+     */
+    void tableRows();
+
     /**
      * Starts an element that contains rows of table data.
      *