[X2Go-Commits] [x2goserver] 01/99: x2goserver/lib/x2goupdateoptionsstring: simplify a for loop a bit.

[git-admin at x2go.org]

This is an automated email from the git hooks/post-receive script.

x2go pushed a commit to branch master
in repository x2goserver.

commit 53293ac0cc0030b3aebd09e39f683eeaf0afab51
Author: Mihai Moldovan <ionic at ionic.de>
Date:   Thu Nov 5 09:46:10 2020 +0100

    x2goserver/lib/x2goupdateoptionsstring: simplify a for loop a bit.
---
 debian/changelog                       | 1 +
 x2goserver/bin/x2goupdateoptionsstring | 3 +--
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/debian/changelog b/debian/changelog
index 3e27a07a..c7f71624 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -184,6 +184,7 @@ x2goserver (4.1.0.4-0x2go1.2) UNRELEASED; urgency=medium
       created, but we already have such a check in another place. Instead,
       treat a missing PID, which can only happen during session startup, as a
       success.
+    - x2goserver/lib/x2goupdateoptionsstring: simplify a for loop a bit.
   * debian/control:
     + Build-depend upon lsb-release for distro version detection.
   * debian/x2goserver.manpages:
diff --git a/x2goserver/bin/x2goupdateoptionsstring b/x2goserver/bin/x2goupdateoptionsstring
index 561fd3f5..22d0f34e 100755
--- a/x2goserver/bin/x2goupdateoptionsstring
+++ b/x2goserver/bin/x2goupdateoptionsstring
@@ -703,8 +703,7 @@ sub sanitize_program_options {
   if (!($error_detected)) {
     $args = dclone ($args);
 
-    my $opt = 1;
-    for (my $cur_arg = shift (@{$args}); ((defined ($cur_arg)) && ($opt)); $cur_arg = shift (@{$args})) {
+    foreach my $cur_arg (@{$args}) {
       if (q{-} eq substr ($cur_arg, 0, 1)) {
         # Looks like an option so far. Let's continue scanning.
 

--
Alioth's /home/x2go-admin/maintenancescripts/git/hooks/post-receive-email on /srv/git/code.x2go.org/x2goserver.git