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 subversion and git services (JIRA)" <ji...@apache.org> on 2019/05/23 22:16:00 UTC

[jira] [Commented] (GEODE-3287) Remove Assert.hpp

    [ https://issues.apache.org/jira/browse/GEODE-3287?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16847103#comment-16847103 ] 

ASF subversion and git services commented on GEODE-3287:
--------------------------------------------------------

Commit f23a7fa78fa13627aae85c3951a4322366a408de in geode-native's branch refs/heads/develop from Blake Bender
[ https://gitbox.apache.org/repos/asf?p=geode-native.git;h=f23a7fa ]

GEODE-3287: remove Assert.hpp (#487)

* GEODE-3287: Remove Assert.hpp
- The only asserts that were ever enabled were GF_R_ASSERT
- There were only 3 occurrences of GF_R_ASSERT
- Replaced these macros with the code they'd expand to
- Removed all other GF_*_ASSERT* macros
- Deleted Assert.hpp and Assert.cpp
- Going forward just use <cassert> and the assert call from std

Co-authored-by: Matthew Reddington <mr...@pivotal.io>

> Remove Assert.hpp
> -----------------
>
>                 Key: GEODE-3287
>                 URL: https://issues.apache.org/jira/browse/GEODE-3287
>             Project: Geode
>          Issue Type: Improvement
>          Components: native client
>            Reporter: Ernest Burghardt
>            Assignee: Blake Bender
>            Priority: Major
>          Time Spent: 1h
>  Remaining Estimate: 0h
>
> replace with std::assert



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