From dd3afd4c4e66d822d984a07de26b03083309fb9d Mon Sep 17 00:00:00 2001
From: Thomas Geymayer <tomgey@gmail.com>
Date: Tue, 14 Feb 2012 14:41:53 +0100
Subject: [PATCH] Use simgear functions for clip and wrap around.

---
 docs-mini/README.protocol |  2 +-
 src/Network/generic.hxx   | 13 ++-----------
 2 files changed, 3 insertions(+), 12 deletions(-)

diff --git a/docs-mini/README.protocol b/docs-mini/README.protocol
index e106f5f80..80a7163c6 100644
--- a/docs-mini/README.protocol
+++ b/docs-mini/README.protocol
@@ -144,7 +144,7 @@ For input chunks there exist some more options:
   <max>         an optional upper limit for the input value to be clamped to. If
                 <min> equals <max> no limit is applied. (default: 0)
   <wrap>        instead of clamping to minimum and maximum limits, wrap values
-                around. (default: false)
+                around. Values will be in [min, max[ (default: false)
                 (Usefull for eg. heading selector to start again with 1 for
                 values higher than 360)
 
diff --git a/src/Network/generic.hxx b/src/Network/generic.hxx
index 0b27248f5..55f0ca8a5 100644
--- a/src/Network/generic.hxx
+++ b/src/Network/generic.hxx
@@ -113,18 +113,9 @@ private:
       if( prot.max > prot.min )
       {
         if( prot.wrap )
-        {
-          T range = prot.max - prot.min + 1;
-          if( range > 0 )
-          {
-            while( new_val < prot.min )
-              new_val += range;
-            while( new_val > prot.max )
-              new_val -= range;
-          }
-        }
+          new_val = SGMisc<double>::normalizePeriodic(prot.min, prot.max, new_val);
         else
-          new_val = std::min<T>(prot.max, std::max<T>(prot.min, new_val));
+          new_val = SGMisc<T>::clip(new_val, prot.min, prot.max);
       }
 
       setValue(prot.prop, new_val);