Github user pivotal-jbarrett commented on a diff in the pull request:

    https://github.com/apache/geode-native/pull/3#discussion_r100912867
  
    --- Diff: src/cppcache/src/TcrMessage.hpp ---
    @@ -1170,7 +1170,7 @@ class TcrMessageHelper {
         }
         if (compId != expectedPartType) {
           char exMsg[256];
    -      ACE_OS::snprintf(exMsg, 255,
    +      std::snprintf(exMsg, 255,
    --- End diff --
    
    We don't have a good command to execute. I suggest looking for integration 
into what ever editor you have with clang-format. There appears to be some 
differences between clang-format on mac and windows so limit the formatting to 
files you are editing. 
    
    For example, in Eclipse, I have reformat on save configured. 
    
    The following lines used to be aligned with the changed line's open paren. 
The guide defines specific rules for when to align to the paren or indent.


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to