You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@geode.apache.org by "Ernest Burghardt (JIRA)" <ji...@apache.org> on 2018/10/11 18:57:00 UTC

[jira] [Closed] (GEODE-5567) Replace usage of std::atomic

     [ https://issues.apache.org/jira/browse/GEODE-5567?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Ernest Burghardt closed GEODE-5567.
-----------------------------------

> Replace usage of std::atomic<std::chrono::time_span>
> ----------------------------------------------------
>
>                 Key: GEODE-5567
>                 URL: https://issues.apache.org/jira/browse/GEODE-5567
>             Project: Geode
>          Issue Type: Improvement
>          Components: native client
>            Reporter: Blake Bender
>            Priority: Major
>              Labels: pull-request-available
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
> std::atomic template is only compatible with types which are "trivially copyable".  std::chrono::*::time_span is _not guaranteed_ trivially copyable, and will probably change in future versions of the library, so this code needs to be refactored to use std::atomic with a template parameter that's an integral type like a long long.  The "rep" field of std:: duration should work fine for this.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)