From a5c1a7f6c154ba70dce5881d3689f5e39259d604 Mon Sep 17 00:00:00 2001 From: Bruno Haible Date: Fri, 12 Dec 2025 11:24:14 +0100 Subject: [PATCH] Modernize coding style: Avoid 'continue;' statements where possible. * gnulib-local/lib/backupfile.c (max_backup_version): Use if/else instead of 'continue;'. * gnulib-local/lib/markup.c (emit_start_element): Likewise. --- gnulib-local/lib/backupfile.c | 14 +++++++------- gnulib-local/lib/markup.c | 14 +++++++------- 2 files changed, 14 insertions(+), 14 deletions(-) diff --git a/gnulib-local/lib/backupfile.c b/gnulib-local/lib/backupfile.c index 15595c16a..17fe77536 100644 --- a/gnulib-local/lib/backupfile.c +++ b/gnulib-local/lib/backupfile.c @@ -1,5 +1,5 @@ /* backupfile.c -- make Emacs style backup file names - Copyright (C) 1990-2003, 2005-2006, 2012, 2020 Free Software Foundation, Inc. + Copyright (C) 1990-2003, 2005-2006, 2012, 2020, 2025 Free Software Foundation, Inc. This program is free software: you can redistribute it and/or modify it under the terms of the GNU General Public License as published by @@ -149,12 +149,12 @@ max_backup_version (const char *file, const char *dir) while ((dp = readdir (dirp)) != 0) { - if (!REAL_DIR_ENTRY (dp) || strlen (dp->d_name) < file_name_length + 4) - continue; - - this_version = version_number (file, dp->d_name, file_name_length); - if (this_version > highest_version) - highest_version = this_version; + if (REAL_DIR_ENTRY (dp) && strlen (dp->d_name) >= file_name_length + 4) + { + this_version = version_number (file, dp->d_name, file_name_length); + if (this_version > highest_version) + highest_version = this_version; + } } if (closedir (dirp)) return 0; diff --git a/gnulib-local/lib/markup.c b/gnulib-local/lib/markup.c index f182ab665..8479ef7cc 100644 --- a/gnulib-local/lib/markup.c +++ b/gnulib-local/lib/markup.c @@ -833,13 +833,13 @@ emit_start_element (markup_parse_context_ty *context) for (i = 0; i < context->cur_attr + 1; i++) { /* Possibly omit qualified attribute names from the list */ - if ((context->flags & MARKUP_IGNORE_QUALIFIED) - && strchr (context->attr_names[i], ':')) - continue; - - attr_names[j] = context->attr_names[i]; - attr_values[j] = context->attr_values[i]; - j++; + if (!((context->flags & MARKUP_IGNORE_QUALIFIED) + && strchr (context->attr_names[i], ':'))) + { + attr_names[j] = context->attr_names[i]; + attr_values[j] = context->attr_values[i]; + j++; + } } attr_names[j] = NULL; attr_values[j] = NULL;