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>.
>
---
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 [email protected] or file a JIRA ticket
with INFRA.
---
