[ https://issues.apache.org/jira/browse/GEODE-9078?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17337660#comment-17337660 ]
ASF GitHub Bot commented on GEODE-9078: --------------------------------------- gaussianrecurrence commented on pull request #776: URL: https://github.com/apache/geode-native/pull/776#issuecomment-830434263 > @gaussianrecurrence The sub group of people working on geode-native has historically been a VERY small group and we did a very poor job communicating outward and recording some of these variations. I wonder if for the sake of not holding up this PR if you should just blackout that variable renaming commit, sorry, and we start an open discussion on the dev@geode list on the C++ style guide. We should discuss deviations we want and how far we should go to try to refactor code to fit it. Then we should record the consensus in the CONTRIBUTING.md. Yes, I also think that would be the fastest approach for this PR. Will create a new PR with that change and solving Windows compilation -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Remove ACE mutexes > ------------------ > > Key: GEODE-9078 > URL: https://issues.apache.org/jira/browse/GEODE-9078 > Project: Geode > Issue Type: Task > Components: native client > Reporter: Mario Salazar de Torres > Assignee: Mario Salazar de Torres > Priority: Major > Labels: obliterate-ace, pull-request-available > > *AS AN* geode-native contributor > *I WANT TO* remove all occurrences of ACE mutexes > *SO THAT* we can get rid of ACE for good -- This message was sent by Atlassian Jira (v8.3.4#803005)