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)