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 2019/05/23 15:55:05 UTC

[arrow] branch master updated: ARROW-5404: [C++] force usage of nonstd::sv_lite::string_view instead of std::string_view

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 9fcc12f  ARROW-5404: [C++] force usage of nonstd::sv_lite::string_view instead of std::string_view
9fcc12f is described below

commit 9fcc12fc094b85ec2e3e9798bae5c8151d14df5e
Author: Benjamin Kietzman <be...@gmail.com>
AuthorDate: Thu May 23 10:54:55 2019 -0500

    ARROW-5404: [C++] force usage of nonstd::sv_lite::string_view instead of std::string_view
    
    Otherwise libraries built with C++11 enabled can have ABI conflicts when built against applications built with C++ >= 17
    
    Author: Benjamin Kietzman <be...@gmail.com>
    
    Closes #4378 from bkietz/5404-nonstdstring-view-conflicts-with-stdstri and squashes the following commits:
    
    4ff84802b <Benjamin Kietzman> force usage of nonstd::sv_lite::string_view instead of std::string_view
---
 cpp/src/arrow/util/string_view.h | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/cpp/src/arrow/util/string_view.h b/cpp/src/arrow/util/string_view.h
index a1a8137..8874842 100644
--- a/cpp/src/arrow/util/string_view.h
+++ b/cpp/src/arrow/util/string_view.h
@@ -18,6 +18,8 @@
 #ifndef ARROW_UTIL_STRING_VIEW_H
 #define ARROW_UTIL_STRING_VIEW_H
 
+#define nssv_CONFIG_SELECT_STRING_VIEW nssv_STRING_VIEW_NONSTD
+
 #include "arrow/vendored/string_view.hpp"  // IWYU pragma: export
 
 namespace arrow {