[jira] [Work logged] (TS-4532) Static type checking for time units

["ASF GitHub Bot (JIRA)" <ji...@apache.org>]

     [ 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 <am...@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)