Hi,

this one remove a dead assignment and reduce the scope of a variable.

 mod_proxy_balancer.c |    3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

Best regards,
CJ


On 09/14/2012 08:10 PM, Igor Galić wrote:

Hey folks,

over at trafficserver we started looking into running
clang's checkers over our code. Using the momentum, Daniel
and myself rand the checkers over httpd and apr (both from
trunk) as well.

Daniel's checks are ran under FreeBSD/clang 3.0:

    http://inhouse.humbedooh.com/clang/2012-09-14-3/

Mine are ran under Fedora and clang 3.2 (trunk):

    http://people.apache.org/~igalic/checks/httpd/latest/


So long,

i


Index: modules/proxy/mod_proxy_balancer.c
===================================================================
--- modules/proxy/mod_proxy_balancer.c	(revision 1384907)
+++ modules/proxy/mod_proxy_balancer.c	(working copy)
@@ -702,7 +702,6 @@
                          apr_pool_t *ptemp, server_rec *s)
 {
     apr_status_t rv;
-    void *sconf = s->module_config;
     proxy_server_conf *conf;
     ap_slotmem_instance_t *new = NULL;
     apr_time_t tstamp;
@@ -744,7 +743,7 @@
     while (s) {
         int i,j;
         proxy_balancer *balancer;
-        sconf = s->module_config;
+        void *sconf = s->module_config;
         conf = (proxy_server_conf *)ap_get_module_config(sconf, &proxy_module);
 
         if (conf->bslot) {

Reply via email to