From: Paul Eggert <eggert@cs.ucla.edu>
Date: Mon, 21 Mar 2011 18:09:45 +0000 (-0700)
Subject: * frame.c (Fmodify_frame_parameters): Simplify loop counter.
X-Git-Tag: emacs-pretest-24.0.90~104^2~275^2~513^2~12
X-Git-Url: http://git.eshelyaron.com/gitweb/?a=commitdiff_plain;h=7f650bb9a878f4ce87828bf068edd9d2b5ca032e;p=emacs.git

* frame.c (Fmodify_frame_parameters): Simplify loop counter.

This also avoids a warning with gcc -Wstrict-overflow.
---

diff --git a/src/ChangeLog b/src/ChangeLog
index f8ccbfd1f3c..d833648667d 100644
--- a/src/ChangeLog
+++ b/src/ChangeLog
@@ -1,5 +1,8 @@
 2011-03-21  Paul Eggert  <eggert@cs.ucla.edu>
 
+	* frame.c (Fmodify_frame_parameters): Simplify loop counter.
+	This also avoids a warning with gcc -Wstrict-overflow.
+
 	* fileio.c (Fcopy_file): Report error if fchown or fchmod
 	fail (Bug#8306).
 
diff --git a/src/frame.c b/src/frame.c
index 05938f3e1f0..5c676527909 100644
--- a/src/frame.c
+++ b/src/frame.c
@@ -2529,8 +2529,9 @@ use is not recommended.  Explicitly check for a frame-parameter instead.  */)
 	}
 
       /* Now process them in reverse of specified order.  */
-      for (i--; i >= 0; i--)
+      while (i != 0)
 	{
+	  i--;
 	  prop = parms[i];
 	  val = values[i];
 	  store_frame_param (f, prop, val);