[ https://issues.apache.org/jira/browse/TS-4532?focusedWorklogId=28116&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-28116 ]
ASF GitHub Bot logged work on TS-4532: -------------------------------------- Author: ASF GitHub Bot Created on: 05/Sep/16 16:00 Start Date: 05/Sep/16 16:00 Worklog Time Spent: 10m Work Description: GitHub user SolidWallOfCode opened a pull request: https://github.com/apache/trafficserver/pull/965 TS-4532: Static type checking for time. Rough first pass just for iocore/eventsystem to get a feel for the changes. THIS WILL NOT RUN - it's just an experiment to be verified before further work is done. You can merge this pull request into a Git repository by running: $ git pull https://github.com/SolidWallOfCode/trafficserver ts-4532 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/trafficserver/pull/965.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #965 ---- commit c96fdecad810ee680475377faf9fca2bdbf817b4 Author: Alan M. Carroll <a...@apache.org> Date: 2016-09-05T15:58:14Z TS-4532: Static type checking for time. Rough first pass just for iocore/eventsystem to get a feel for the changes. ---- Issue Time Tracking ------------------- Worklog Id: (was: 28116) Time Spent: 10m Remaining Estimate: 0h > 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: 10m > 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)