[GitHub] geode-native issue #23: GEODE-2478: Replace gf with geode.

[PivotalSarge <gi...@git.apache.org>]

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 <no...@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.
    > 
    > \u2014
    > 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>.
    > 
    



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---