commit f3019aff2421d9c3d16498f22cd245e3f8948d4c
author Stephen Hemminger <stephen.hemminger@vyatta.com> Sun Aug 17 17:51:47 2008 +0100
committer Paul Jakma <paul@quagga.net> Fri Aug 22 19:56:26 2008 +0100
tree e5630448e6bf0697cbdb35e42054ef84763bda21
parent ef9b113e2d4c0d49bd7d61458fa9acc113658d2c

[bgpd] remove useless check and return in bgp_config_write_damp

2008-08-17 Stephen Hemminger <stephen.hemminger@vyatta.com>

	* bgp_damp.?: (bgp_config_write_damp) remove useless check
	  of statically allocated config storage, and useless return
	  value

Signed-off-by: Paul Jakma <paul@quagga.net>

bgpd/bgp_damp.c

diff --git a/bgpd/bgp_damp.c b/bgpd/bgp_damp.c
index 5a7c9aa..ba25f4d 100644
--- a/bgpd/bgp_damp.c
+++ b/bgpd/bgp_damp.c
@@ -517,33 +517,28 @@
   return 0;
 }
 
-int
+void
 bgp_config_write_damp (struct vty *vty)
 {
-  if (&bgp_damp_cfg)
-    {
-      if (bgp_damp_cfg.half_life == DEFAULT_HALF_LIFE*60
-	  && bgp_damp_cfg.reuse_limit == DEFAULT_REUSE
-	  && bgp_damp_cfg.suppress_value == DEFAULT_SUPPRESS
-	  && bgp_damp_cfg.max_suppress_time == bgp_damp_cfg.half_life*4)
-	vty_out (vty, " bgp dampening%s", VTY_NEWLINE);
-      else if (bgp_damp_cfg.half_life != DEFAULT_HALF_LIFE*60
-	       && bgp_damp_cfg.reuse_limit == DEFAULT_REUSE
-	       && bgp_damp_cfg.suppress_value == DEFAULT_SUPPRESS
-	       && bgp_damp_cfg.max_suppress_time == bgp_damp_cfg.half_life*4)
-	vty_out (vty, " bgp dampening %ld%s",
-		 bgp_damp_cfg.half_life/60,
-		 VTY_NEWLINE);
-      else
-	vty_out (vty, " bgp dampening %ld %d %d %ld%s",
-		 bgp_damp_cfg.half_life/60,
-		 bgp_damp_cfg.reuse_limit,
-		 bgp_damp_cfg.suppress_value,
-		 bgp_damp_cfg.max_suppress_time/60,
-		 VTY_NEWLINE);
-      return 1;
-    }
-  return 0;
+  if (bgp_damp_cfg.half_life == DEFAULT_HALF_LIFE*60
+      && bgp_damp_cfg.reuse_limit == DEFAULT_REUSE
+      && bgp_damp_cfg.suppress_value == DEFAULT_SUPPRESS
+      && bgp_damp_cfg.max_suppress_time == bgp_damp_cfg.half_life*4)
+    vty_out (vty, " bgp dampening%s", VTY_NEWLINE);
+  else if (bgp_damp_cfg.half_life != DEFAULT_HALF_LIFE*60
+	   && bgp_damp_cfg.reuse_limit == DEFAULT_REUSE
+	   && bgp_damp_cfg.suppress_value == DEFAULT_SUPPRESS
+	   && bgp_damp_cfg.max_suppress_time == bgp_damp_cfg.half_life*4)
+    vty_out (vty, " bgp dampening %ld%s",
+	     bgp_damp_cfg.half_life/60,
+	     VTY_NEWLINE);
+  else
+    vty_out (vty, " bgp dampening %ld %d %d %ld%s",
+	     bgp_damp_cfg.half_life/60,
+	     bgp_damp_cfg.reuse_limit,
+	     bgp_damp_cfg.suppress_value,
+	     bgp_damp_cfg.max_suppress_time/60,
+	     VTY_NEWLINE);
 }
 
 #define BGP_UPTIME_LEN 25