[MCOL-3824] CLONE - Some compiler warnings need fixing Created: 2020-02-20  Updated: 2020-03-16  Resolved: 2020-03-16

Status: Closed
Project: MariaDB ColumnStore
Component/s: ?
Affects Version/s: None
Fix Version/s: 1.2.6

Type: Bug Priority: Minor
Reporter: Patrick LeBlanc (Inactive) Assignee: Daniel Lee (Inactive)
Resolution: Fixed Votes: 0
Labels: None

Issue Links:
Problem/Incident
causes MCOL-3415 we_splitterapp has a setuid error in ... Closed

 Description   

Some of the following are definitely bugs, we should look at fixing them all:

/home/linuxjedi/Programming/Git/mariadb-columnstore-server/mariadb-columnstore-engine/utils/funcexp/func_div.cpp: In member function ‘virtual int64_t funcexp::Func_div::getIntVal(rowgroup::Row&, funcexp::FunctionParm&, bool&, execplan::CalpontSystemCatalog::ColType&)’:
/home/linuxjedi/Programming/Git/mariadb-columnstore-server/mariadb-columnstore-engine/utils/funcexp/func_div.cpp:58:16: warning: converting to non-pointer type ‘long int’ from NULL [-Wconversion-null]
         return NULL;
                ^~~~
 
/home/linuxjedi/Programming/Git/mariadb-columnstore-server/mariadb-columnstore-engine/utils/funcexp/func_div.cpp: In member function ‘virtual uint64_t funcexp::Func_div::getUintVal(rowgroup::Row&, funcexp::FunctionParm&, bool&, execplan::CalpontSystemCatalog::ColType&)’:
/home/linuxjedi/Programming/Git/mariadb-columnstore-server/mariadb-columnstore-engine/utils/funcexp/func_div.cpp:81:16: warning: converting to non-pointer type ‘uint64_t {aka long unsigned int}’ from NULL [-Wconversion-null]
         return NULL;
                ^~~~
 
/home/linuxjedi/Programming/Git/mariadb-columnstore-server/mariadb-columnstore-engine/utils/funcexp/func_lpad.cpp: In member function ‘virtual std::__cxx11::string funcexp::Func_lpad::getStrVal(rowgroup::Row&, funcexp::FunctionParm&, bool&, execplan::CalpontSystemCatalog::ColType&)’:
/home/linuxjedi/Programming/Git/mariadb-columnstore-server/mariadb-columnstore-engine/utils/funcexp/func_lpad.cpp:133:14: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
  if(strwclen > len)
     ~~~~~~~~~^~~~~
/home/linuxjedi/Programming/Git/mariadb-columnstore-server/mariadb-columnstore-engine/utils/funcexp/func_lpad.cpp:150:17: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
     if (strSize == len)
         ~~~~~~~~^~~~~~
/home/linuxjedi/Programming/Git/mariadb-columnstore-server/mariadb-columnstore-engine/utils/funcexp/func_lpad.cpp:157:17: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
     if (strSize > len)
         ~~~~~~~~^~~~~
 
 In member function ‘virtual std::__cxx11::string funcexp::Func_rpad::getStrVal(rowgroup::Row&, funcexp::FunctionParm&, bool&, execplan::CalpontSystemCatalog::ColType&)’:
/home/linuxjedi/Programming/Git/mariadb-columnstore-server/mariadb-columnstore-engine/utils/funcexp/func_rpad.cpp:133:14: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
  if(strwclen > len)
     ~~~~~~~~~^~~~~
/home/linuxjedi/Programming/Git/mariadb-columnstore-server/mariadb-columnstore-engine/utils/funcexp/func_rpad.cpp:150:17: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
     if (strSize == len)
         ~~~~~~~~^~~~~~
/home/linuxjedi/Programming/Git/mariadb-columnstore-server/mariadb-columnstore-engine/utils/funcexp/func_rpad.cpp:157:17: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
     if (strSize > len)
         ~~~~~~~~^~~~~
 
/home/linuxjedi/Programming/Git/mariadb-columnstore-server/mariadb-columnstore-engine/oam/oamcpp/liboamcpp.cpp: In member function ‘void oam::Oam::sendStatusUpdate(messageqcpp::ByteStream, messageqcpp::ByteStream::byte)’:
/home/linuxjedi/Programming/Git/mariadb-columnstore-server/mariadb-columnstore-engine/oam/oamcpp/liboamcpp.cpp:9321:28: warning: self-comparison always evaluates to true [-Wtautological-compare]
     if ( returnRequestType == returnRequestType ) {
          ~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~
 
/home/linuxjedi/Programming/Git/mariadb-columnstore-server/mariadb-columnstore-engine/primitives/blockcache/filebuffer.cpp: In copy constructor ‘dbbc::FileBuffer::FileBuffer(const dbbc::FileBuffer&)’:
/home/linuxjedi/Programming/Git/mariadb-columnstore-server/mariadb-columnstore-engine/primitives/blockcache/filebuffer.cpp:41:2: warning: nonnull argument ‘this’ compared to NULL [-Wnonnull-compare]
  if (this==NULL || this==&rhs)
  ^~
 
/home/linuxjedi/Programming/Git/mariadb-columnstore-server/mariadb-columnstore-engine/dbcon/mysql/is_columnstore_files.cpp: In function ‘bool get_file_sizes(int, const char*, off_t*, off_t*)’:
/home/linuxjedi/Programming/Git/mariadb-columnstore-server/mariadb-columnstore-engine/dbcon/mysql/is_columnstore_files.cpp:70:16: warning: ‘msgQueueClient’ may be used uninitialized in this function [-Wmaybe-uninitialized]
         delete msgQueueClient;
                ^~~~~~~~~~~~~~
 
/home/linuxjedi/Programming/Git/mariadb-columnstore-server/mariadb-columnstore-engine/dbcon/mysql/ha_calpont_impl.cpp: In function ‘int ha_calpont_impl_rnd_init(TABLE*)’:
/home/linuxjedi/Programming/Git/mariadb-columnstore-server/mariadb-columnstore-engine/dbcon/mysql/ha_calpont_impl.cpp:2432:24: warning: comparison of constant ‘0’ with boolean expression is always false [-Wbool-compare]
  if (bSystemQueryReady < 0)
      ~~~~~~~~~~~~~~~~~~^~~
 
/home/linuxjedi/Programming/Git/mariadb-columnstore-server/mariadb-columnstore-engine/oamapps/postConfigure/postConfigure.cpp: In function ‘int main(int, char**)’:
/home/linuxjedi/Programming/Git/mariadb-columnstore-server/mariadb-columnstore-engine/oamapps/postConfigure/postConfigure.cpp:2808:19: warning: comparison with string literal results in unspecified behavior [-Waddress]
     if ( pass1 == "exit")
                   ^~~~~~



 Comments   
Comment by Daniel Lee (Inactive) [ 2020-03-16 ]

Build verified: 1.2.6-1 source

/root/ColumnStore/buildColumnstoreFromGithubSource/mariadb-columnstore-server/mariadb-columnstore-engine
commit a94f8ca6ddde6377fcbf0f34ffeee6e5a24067b4
Merge: 62b0667 2e15054
Author: David.Hall <david.hall@mariadb.com>
Date: Fri Mar 6 17:17:42 2020 -0600

Merge pull request #1093 from jmrojas2332/develop-1.2

MCOL-3760 Fix rand func in columnstore not matching mdb

Made a build and checked terminal output.

Generated at Thu Feb 08 02:45:43 UTC 2024 using Jira 8.20.16#820016-sha1:9d11dbea5f4be3d4cc21f03a88dd11d8c8687422.