Re: svn commit: r1933532 - subversion/trunk/subversion/svnbrowse

[Branko Čibej]

On 29. 4. 26 21:37, rin...@apache.org wrote:
Author: rinrab
Date: Wed Apr 29 19:37:32 2026
New Revision: 1933532
Log:
Format code.

* subversion/svnbrowse/svnbrowse.c
   (get_item_style): Properly indent a nested switch.

Modified:
    subversion/trunk/subversion/svnbrowse/svnbrowse.c

Modified: subversion/trunk/subversion/svnbrowse/svnbrowse.c
==============================================================================
--- subversion/trunk/subversion/svnbrowse/svnbrowse.c   Wed Apr 29 19:36:17 
2026        (r1933531)
+++ subversion/trunk/subversion/svnbrowse/svnbrowse.c   Wed Apr 29 19:37:32 
2026        (r1933532)
@@ -403,14 +403,14 @@ get_item_style(const svn_browse__style_t
                 svn_boolean_t selected)
  {
    switch (kind)
-  {
-    case svn_node_dir:
-      return (selected) ? style->dir_selected : style->dir;
-    case svn_node_file:
-      return (selected) ? style->file_selected : style->file;
-    default:
-      abort();
-  }
+    {
+      case svn_node_dir:
+        return (selected) ? style->dir_selected : style->dir;

Just curious, why are there parentheses around 'selected'? They're not 
needed.

+      case svn_node_file:
+        return (selected) ? style->file_selected : style->file;
+      default:
+        abort();
+    }
  }
  
  static void


-- Brane