[ 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 <notificati...@github.com> 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)