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

Posted to issues@geode.apache.org by "ASF GitHub Bot (JIRA)" <ji...@apache.org> on 2018/08/10 22:51:00 UTC

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

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

ASF GitHub Bot updated GEODE-5567:
----------------------------------
    Labels: pull-request-available  (was: )

> 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
>
> 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)