wpleonardo commented on code in PR #1375:
URL: https://github.com/apache/orc/pull/1375#discussion_r1148180787
##########
c++/src/RLEv2.hh:
##########
@@ -189,23 +203,10 @@ namespace orc {
resetReadLongs();
}
- unsigned char readByte();
-
int64_t readLongBE(uint64_t bsz);
int64_t readVslong();
uint64_t readVulong();
- void readLongs(int64_t* data, uint64_t offset, uint64_t len, uint64_t fbs);
- void plainUnpackLongs(int64_t* data, uint64_t offset, uint64_t len,
uint64_t fbs);
-
- void unrolledUnpack4(int64_t* data, uint64_t offset, uint64_t len);
- void unrolledUnpack8(int64_t* data, uint64_t offset, uint64_t len);
- void unrolledUnpack16(int64_t* data, uint64_t offset, uint64_t len);
- void unrolledUnpack24(int64_t* data, uint64_t offset, uint64_t len);
- void unrolledUnpack32(int64_t* data, uint64_t offset, uint64_t len);
- void unrolledUnpack40(int64_t* data, uint64_t offset, uint64_t len);
- void unrolledUnpack48(int64_t* data, uint64_t offset, uint64_t len);
- void unrolledUnpack56(int64_t* data, uint64_t offset, uint64_t len);
- void unrolledUnpack64(int64_t* data, uint64_t offset, uint64_t len);
+ int readLongs(int64_t* data, uint64_t offset, uint64_t len, uint64_t fbs);
Review Comment:
Changed the RleDecoderV2::readLongs return type back to void.
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
