[
https://issues.apache.org/jira/browse/GEODE-2478?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15878969#comment-15878969
]
ASF GitHub Bot commented on GEODE-2478:
---------------------------------------
Github user PivotalSarge commented on the issue:
https://github.com/apache/geode-native/pull/23
Good idea. I think you're right about redundancies. I'll look into that.
Sarge
> On 22 Feb, 2017, at 10:48, Jacob Barrett <[email protected]> wrote:
>
> @pivotal-jbarrett approved this pull request.
>
> I would suggest opening a ticket with any cleanup ideas you may have seen
when going through all these headers. Seems like geode_defs and geode_base and
and whatever generically named includes we have might be fairly redundant and
ripe for cleanup or reorganization.
>
> Otherwise this looks good.
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
<https://github.com/apache/geode-native/pull/23#pullrequestreview-23299880>, or
mute the thread
<https://github.com/notifications/unsubscribe-auth/ATlz4uDEUOa4K1RgRpgEMG-urzHriHQGks5rfILmgaJpZM4MI5mz>.
>
> Replace gf with geode
> ---------------------
>
> Key: GEODE-2478
> URL: https://issues.apache.org/jira/browse/GEODE-2478
> Project: Geode
> Issue Type: Improvement
> Components: native client
> Reporter: Michael Dodge
>
> The substring "gf" still occurs in some places in the native client codebase.
> It ought to be replaced with "geode" or "geode-native", whichever makes more
> sense on a case-by-case basis.
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)
