Github user rip-nsk commented on a diff in the pull request:

    https://github.com/apache/orc/pull/212#discussion_r162753893
  
    --- Diff: c++/src/RLEv1.cc ---
    @@ -148,7 +148,7 @@ void RleEncoderV1::write(int64_t value) {
             numLiterals += 1;
           } else {
             numLiterals -= static_cast<int>(MINIMUM_REPEAT - 1);
    -        long base = literals[numLiterals];
    +        int64_t base = literals[numLiterals];
    --- End diff --
    
    'auto' is more suitable, but it seems its usage is avoided here


---

Reply via email to