[
https://issues.apache.org/jira/browse/GEODE-2411?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15852508#comment-15852508
]
ASF GitHub Bot commented on GEODE-2411:
---------------------------------------
Github user pivotal-jbarrett commented on the issue:
https://github.com/apache/geode/pull/387
The convention in this pull request is not consistent with the Google C++
Style Guide.
https://google.github.io/styleguide/cppguide.html#The__define_Guard
> Remove references to Gemfire from include guards
> ------------------------------------------------
>
> Key: GEODE-2411
> URL: https://issues.apache.org/jira/browse/GEODE-2411
> Project: Geode
> Issue Type: Bug
> Components: native client
> Reporter: Michael Dodge
> Assignee: Michael Dodge
>
> Many of the macro definitions used as include guards in the C++ header files
> refer to Gemfire. These references should be replaced at least with
> references to Geode but preferably using pragmas instead of defines.
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)
