Re: git commit: TS-1572: Plugin response status change can trigger ATS assertion

[Leif Hedstrom]

On 11/16/12 2:47 AM, Igor Galić wrote:
--- a/proxy/http/HttpTransact.cc
+++ b/proxy/http/HttpTransact.cc
@@ -8622,10 +8622,8 @@ HttpTransact::client_result_stat(State* s,
ink_hrtime total_time, ink_hrtime req
      break;
    default:
      HTTP_SUM_TRANS_STAT(http_ua_msecs_counts_other_unclassified_stat,
      total_msec);
-    if (is_debug_tag_set("http")) {
Can someone explain to me what is_debug_tag_set() is good for,
in those cases where it's used used with f/printf()?

Same "functionality" as the DEBUG statements. The reason to use them is if 
the code that has to be executed during debug is complicated (more than just 
a printf()), you use is_debug_tag_set() instead. That way, you avoid 
evaluating code that is only needed for the DEBUG state.

-- Leif