Sorry, only fixed Win32... Now MacOS and Linux should be fixed as well.

/home/lance/upp.src/uppsrc/MySql/MySql.h (9): fatal error: 'mysql.h' file not found

/home/lance/upp.src/uppsrc/plugin/dbf/dbf.cpp (198): error: no viable conversion from returned value of type 'StringBuffer' to function return type 'String'

coolman wrote on Thu, 19 September 2024 11:09

Hi Mirek,

Would it be possible to clean up these annoying warnings?

uppsrc/RichText/ParaPaint.cpp (468): warning: unused variable 'pe' [-Wunused-variable]
uppsrc/ide/IconDes/Event.cpp (102): warning: unused variable 'c' [-Wunused-variable]
uppsrc/ide/IconDes/IconDes.cpp (130): warning: unused variable 'isz' [-Wunused-variable]
uppsrc/ide/IconDes/ImlFile.cpp (272): warning: unused function 'PutOctalString' [-Wunused-function]
uppsrc/ide/SelectPkg.cpp:661:3: warning: misleading indentation; statement is not part of the previous 'if' [-Wmisleading-indentation]
  661 |                 for(int i = 0; i < dlg.list.GetCount(); i++)
      |                 ^
uppsrc/ide/SelectPkg.cpp:658:2: note: previous statement is here
  658 |         if(n)
      |         ^
1 warning generated.

Thank you, Radek

Is this GCC?

mirek wrote on Thu, 19 September 2024 11:54

coolman wrote on Thu, 19 September 2024 11:09

Hi Mirek,

Would it be possible to clean up these annoying warnings?

uppsrc/RichText/ParaPaint.cpp (468): warning: unused variable 'pe' [-Wunused-variable]
uppsrc/ide/IconDes/Event.cpp (102): warning: unused variable 'c' [-Wunused-variable]
uppsrc/ide/IconDes/IconDes.cpp (130): warning: unused variable 'isz' [-Wunused-variable]
uppsrc/ide/IconDes/ImlFile.cpp (272): warning: unused function 'PutOctalString' [-Wunused-function]
uppsrc/ide/SelectPkg.cpp:661:3: warning: misleading indentation; statement is not part of the previous 'if' [-Wmisleading-indentation]
  661 |                 for(int i = 0; i < dlg.list.GetCount(); i++)
      |                 ^
uppsrc/ide/SelectPkg.cpp:658:2: note: previous statement is here
  658 |         if(n)
      |         ^
1 warning generated.

Thank you, Radek

Is this GCC?

No, it is clang --version
Ubuntu clang version 17.0.6 (++20231208085846+6009708b4367-1~exp1~20231208085949.74)
Target: x86_64-pc-linux-gnu
Thread model: posix
InstalledDir: /usr/bin

coolman wrote on Thu, 19 September 2024 15:05

mirek wrote on Thu, 19 September 2024 11:54

coolman wrote on Thu, 19 September 2024 11:09

Hi Mirek,

Would it be possible to clean up these annoying warnings?

uppsrc/RichText/ParaPaint.cpp (468): warning: unused variable 'pe' [-Wunused-variable]
uppsrc/ide/IconDes/Event.cpp (102): warning: unused variable 'c' [-Wunused-variable]
uppsrc/ide/IconDes/IconDes.cpp (130): warning: unused variable 'isz' [-Wunused-variable]
uppsrc/ide/IconDes/ImlFile.cpp (272): warning: unused function 'PutOctalString' [-Wunused-function]
uppsrc/ide/SelectPkg.cpp:661:3: warning: misleading indentation; statement is not part of the previous 'if' [-Wmisleading-indentation]
  661 |                 for(int i = 0; i < dlg.list.GetCount(); i++)
      |                 ^
uppsrc/ide/SelectPkg.cpp:658:2: note: previous statement is here
  658 |         if(n)
      |         ^
1 warning generated.

Thank you, Radek

Is this GCC?

No, it is clang --version
Ubuntu clang version 17.0.6 (++20231208085846+6009708b4367-1~exp1~20231208085949.74)
Target: x86_64-pc-linux-gnu
Thread model: posix
InstalledDir: /usr/bin

Fixed. BTW, how is your build method CLANG settings? I have found that I have these warnings off by default... (looking for possibility to setup build method with these automatically)

mirek wrote on Thu, 19 September 2024 03:53

Sorry, only fixed Win32... Now MacOS and Linux should be fixed as well.

The above problem is fixed. Now we are almost sure that U++ will not hindering people who wants to use the most recent language features.

When try to build UPP/AllForI18N, with CLANG+c++23, I got the following errors:

Quote:

/home/lance/upp.src/uppsrc/MySql/MySql.h (9): fatal error: 'mysql.h' file not found

And
Quote:

/home/lance/upp.src/uppsrc/plugin/dbf/dbf.cpp (198): error: no viable conversion from returned value of type 'StringBuffer' to function return type 'String'

/home/lance/upp/uppsrc/MySql/MySql.h (9): fatal error: 'mysql.h' file not found

Thank you, Mirek!

I am not sure if it's just me but, even with std=c++17 (or otherwise), the following problem persists
Quote:

/home/lance/upp/uppsrc/MySql/MySql.h (9): fatal error: 'mysql.h' file not found

It's not bothering me at all though.