[ 
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)

Reply via email to