[thrift] branch master updated: THRIFT-5350 char is unsigned on non-x86 arches, use signed char to avoid compiler warning about always true comparisons Patch: Orion Poplawski

[jensg]

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

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


The following commit(s) were added to refs/heads/master by this push:
     new 36bd59f  THRIFT-5350 char is unsigned on non-x86 arches, use signed 
char to avoid compiler warning about always true comparisons Patch: Orion 
Poplawski
36bd59f is described below

commit 36bd59ff8260b2d5bd5825d3361121423022948e
Author: Orion Poplawski <or...@nwra.com>
AuthorDate: Wed Feb 17 07:52:49 2021 -0700

    THRIFT-5350 char is unsigned on non-x86 arches, use signed char to avoid 
compiler warning about always true comparisons
    Patch: Orion Poplawski
    
    This closes #2331
---
 compiler/cpp/src/thrift/generate/t_delphi_generator.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/compiler/cpp/src/thrift/generate/t_delphi_generator.cc 
b/compiler/cpp/src/thrift/generate/t_delphi_generator.cc
index d3ad76a..eac46a6 100644
--- a/compiler/cpp/src/thrift/generate/t_delphi_generator.cc
+++ b/compiler/cpp/src/thrift/generate/t_delphi_generator.cc
@@ -1062,7 +1062,7 @@ std::string 
t_delphi_generator::make_pascal_string_literal(std::string value) {
   }
 
   result << "'";
-  for (char const &c: value) {
+  for (signed char const c: value) {
     if( (c >= 0) && (c < 32)) {  // convert ctrl chars, but leave UTF-8 alone
       result << "#" << (int)c;  
     } else if (c == '\'') {