[ https://issues.apache.org/jira/browse/TS-4532?focusedWorklogId=28120&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-28120 ]
ASF GitHub Bot logged work on TS-4532: -------------------------------------- Author: ASF GitHub Bot Created on: 05/Sep/16 16:07 Start Date: 05/Sep/16 16:07 Worklog Time Spent: 10m Work Description: Github user SolidWallOfCode commented on a diff in the pull request: https://github.com/apache/trafficserver/pull/965#discussion_r77540451 --- Diff: iocore/eventsystem/I_Lock.h --- @@ -241,7 +241,7 @@ class ProxyMutex : public RefCountObj thread_holding = NULL; nthread_holding = 0; #ifdef DEBUG - hold_time = 0; + hold_time = hold_time.min(); --- End diff -- SHould be ``hold_time = TS_HRTICK_ZERO;``. Issue Time Tracking ------------------- Worklog Id: (was: 28120) Time Spent: 50m (was: 40m) > Static type checking for time units > ----------------------------------- > > Key: TS-4532 > URL: https://issues.apache.org/jira/browse/TS-4532 > Project: Traffic Server > Issue Type: Improvement > Components: Core > Reporter: James Peach > Assignee: Alan M. Carroll > Fix For: 7.0.0 > > Time Spent: 50m > Remaining Estimate: 0h > > Since the various time units {{ink_time_t}}, {{time_t}}, {{ink_hrtime}} are > all typedefs of C integral types, it is very hard to ensure that units are > being converted and compared correctly. > Consider wrapping these in trivial classes as part of making the time APIs > more straightforward. Alternatively, if we move to {{C++11}} investigate > {{std::chrono}} for this. -- This message was sent by Atlassian JIRA (v6.3.4#6332)