You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@arrow.apache.org by we...@apache.org on 2018/08/30 17:40:29 UTC

[arrow] branch master updated: ARROW-3148: [C++] Remove needless U+00A0 NO-BREAK SPACE (#2500)

This is an automated email from the ASF dual-hosted git repository.

wesm pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/arrow.git


The following commit(s) were added to refs/heads/master by this push:
     new b79b869  ARROW-3148: [C++] Remove needless U+00A0 NO-BREAK SPACE (#2500)
b79b869 is described below

commit b79b8698a9de0d4d860861a128e78341bb8ab3bd
Author: Kouhei Sutou <ko...@cozmixng.org>
AuthorDate: Fri Aug 31 02:40:26 2018 +0900

    ARROW-3148: [C++] Remove needless U+00A0 NO-BREAK SPACE (#2500)
    
    If we have NO-BREAK SPACE, we can't process this file as ASCII only
    file. MSVC shows C4819 warning on code page 932.
    
    C4819: https://docs.microsoft.com/en-us/cpp/error-messages/compiler-warnings/compiler-warning-level-1-c4819
    
    > The file contains a character that cannot be represented in the
    > current code page (number).
---
 cpp/src/arrow/util/rle-encoding.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/cpp/src/arrow/util/rle-encoding.h b/cpp/src/arrow/util/rle-encoding.h
index 8d9ac1d..418788a 100644
--- a/cpp/src/arrow/util/rle-encoding.h
+++ b/cpp/src/arrow/util/rle-encoding.h
@@ -71,7 +71,7 @@ namespace arrow {
 /// Examples with bit-width 1 (eg encoding booleans):
 /// ----------------------------------------
 /// 100 1s followed by 100 0s:
-/// <varint(100 << 1)> <1, padded to 1 byte>  <varint(100 << 1)> <0, padded to 1 byte>
+/// <varint(100 << 1)> <1, padded to 1 byte> <varint(100 << 1)> <0, padded to 1 byte>
 ///  - (total 4 bytes)
 //
 /// alternating 1s and 0s (200 total):