[asterisk-dev] [svn-commits] lathama: branch 1.8 r308393 - /branches/1.8/main/http.c

[Andrew Latham]

On Mon, Feb 21, 2011 at 6:36 PM, Tilghman Lesher <tilghman at meg.abyt.es> wrote:
> On Monday 21 February 2011 08:36:17 Paul Belanger wrote:
>> On 11-02-21 09:24 AM, SVN commits to the Digium repositories wrote:
>> > @@ -804,7 +807,7 @@
>> >
>> >             }
>> >
>> >             if (option_debug) {
>> >
>> > -                   ast_log(LOG_DEBUG, "mmm ... cookie!  Name: '%s'  Value:
> '%s'\n",
>> > name, val); +                       ast_log(LOG_DEBUG, "HTTP Cookie, Name: '%s'
> Value:
>> > '%s'\n", name, val);
>>
>> "We value a sense of humor in the Asterisk code base." :-P /russellb
>
> and should be replaced with ast_debug(1, ...
>
> --
> Tilghman

Saddly ast_debug is not working in http.c.....

my patch [1] is not working, http.c is getting utils.h which is
including logger.h but it is not working...  And all of this is being
noticed after I messed up a merge and am reinstalling my development
environment.


[1] Index: main/http.c
===================================================================
--- main/http.c (revision 308525)
+++ main/http.c (working copy)
@@ -661,8 +661,7 @@
        struct ast_variable *get_vars = NULL, *v, *prev = NULL;
        struct http_uri_redirect *redirect;

-       if (option_debug > 4)
-               ast_verbose("HTTP Request URI is %s \n", uri);
+       ast_debug(2, "HTTP Request URI is %s \n", uri);

        strsep(&params, "?");
        /* Extract arguments from the request and store them in variables. */
@@ -806,9 +805,7 @@
                        continue;
                }

-               if (option_debug) {
-                       ast_log(LOG_DEBUG, "HTTP Cookie, Name: '%s'
Value: '%s'\n", name, val);
-               }
+               ast_debug(1, "HTTP Cookie, Name: '%s'  Value: '%s'\n",
name, val);

                var = ast_variable_new(name, val, __FILE__);
                var->next = vars;

~~~ Andrew "lathama" Latham lathama at gmail.com ~~~