You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kudu.apache.org by al...@apache.org on 2023/04/21 21:25:14 UTC

[kudu] branch master updated: [client] fix clang warning

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

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


The following commit(s) were added to refs/heads/master by this push:
     new 6064d5ecf [client] fix clang warning
6064d5ecf is described below

commit 6064d5ecf234c2b0e3e28291ab39d6171de397e8
Author: Alexey Serbin <al...@apache.org>
AuthorDate: Fri Apr 21 12:35:15 2023 -0700

    [client] fix clang warning
    
    A contemporary CLANG compiler issues the following warning when
    given a header (.h/.hpp) file as input:
    
      treating 'c-header' input as 'c++-header' when in C++ mode, this behavior is deprecated [-Wdeprecated]
    
    There are no functional changes in this patch; it just addresses
    the compiler's warning.
    
    Change-Id: Iae586b683e06a6af1e44c9859769309d3b6a1a94
    Reviewed-on: http://gerrit.cloudera.org:8080/19781
    Tested-by: Kudu Jenkins
    Reviewed-by: Abhishek Chennaka <ac...@cloudera.com>
---
 src/kudu/client/client_examples-test.sh | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/src/kudu/client/client_examples-test.sh b/src/kudu/client/client_examples-test.sh
index 226701fe2..8bf08ddbc 100755
--- a/src/kudu/client/client_examples-test.sh
+++ b/src/kudu/client/client_examples-test.sh
@@ -117,10 +117,12 @@ popd
 for include_file in $(find "$LIBRARY_DIR" -name \*.h) ; do
   echo Checking standalone compilation of "$include_file"...
   if ! ${CXX:-g++} \
+       -c \
        -o /dev/null \
+       -x c++ \
        -std=c++98 \
-       -I"$LIBRARY_DIR/usr/local/include" \
-       "$include_file" ; then
+       -I"$LIBRARY_DIR/usr/local/include" - \
+       < "$include_file" ; then
     set +x
     echo
     echo -----------------------------------------