You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@groovy.apache.org by su...@apache.org on 2026/06/13 14:35:54 UTC

(groovy) branch master updated: Trivial refactor: use switch expression instead

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

asf-gitbox-commits pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/groovy.git


The following commit(s) were added to refs/heads/master by this push:
     new 73ff60ca61 Trivial refactor: use switch expression instead
73ff60ca61 is described below

commit 73ff60ca61edfa1490d2a788ef285f695066d587
Author: Daniel Sun <su...@apache.org>
AuthorDate: Sat Jun 13 23:35:23 2026 +0900

    Trivial refactor: use switch expression instead
---
 .../groovy/runtime/powerassert/AssertionRenderer.java        | 12 ++++--------
 1 file changed, 4 insertions(+), 8 deletions(-)

diff --git a/src/main/java/org/codehaus/groovy/runtime/powerassert/AssertionRenderer.java b/src/main/java/org/codehaus/groovy/runtime/powerassert/AssertionRenderer.java
index 360c2de82b..759e747f5d 100644
--- a/src/main/java/org/codehaus/groovy/runtime/powerassert/AssertionRenderer.java
+++ b/src/main/java/org/codehaus/groovy/runtime/powerassert/AssertionRenderer.java
@@ -167,14 +167,10 @@ public final class AssertionRenderer {
      * emoji, and 1 otherwise. See {@link #displayColumn} for the caveats.
      */
     private static int displayWidth(int codePoint) {
-        switch (Character.getType(codePoint)) {
-            case Character.NON_SPACING_MARK:
-            case Character.ENCLOSING_MARK:
-            case Character.FORMAT:
-                return 0;
-            default:
-                return isWide(codePoint) ? 2 : 1;
-        }
+        return switch (Character.getType(codePoint)) {
+            case Character.NON_SPACING_MARK, Character.ENCLOSING_MARK, Character.FORMAT -> 0;
+            default -> isWide(codePoint) ? 2 : 1;
+        };
     }
 
     private static boolean isWide(int cp) {