[ 
https://issues.apache.org/jira/browse/TS-130?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12831075#action_12831075
 ] 

Leif Hedstrom commented on TS-130:
----------------------------------

Alright, I'll do another review with the new diffs :).

The reason I asked about DDebug() was because it was used in the v2 patch, but 
I couldn't see it defined anywhere.

Finally, +1 on the proposal to Diag(). I'd like to add to that that it'd be 
really great if we came up with a better, well organized "tree" of tags for the 
diagnostics messages. Right now, it's a crapshoot what you will get when 
tracing on say '.*http.*'.

> we should generate a config.h file instead of putting all the -Ds in the 
> CFLAGS
> -------------------------------------------------------------------------------
>
>                 Key: TS-130
>                 URL: https://issues.apache.org/jira/browse/TS-130
>             Project: Traffic Server
>          Issue Type: Improvement
>            Reporter: John Plevyak
>            Assignee: John Plevyak
>         Attachments: config.h-jp-v1.patch, ts-config.h-jp-v2.patch
>
>
> We should use a config.h instead of a huge pile of -Ds

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to