domschrei / krunner-symbols

A lightweight KRunner plugin (Plasma 5) to retrieve unicode symbols, or any other string, based on a corresponding keyword.
GNU General Public License v3.0
118 stars 12 forks source link

Fix compiler warnings and optimize build #16

Closed alex1701c closed 4 years ago

alex1701c commented 4 years ago

Hello,

I have added the -DCMAKE_BUILD_TYPE=Release flag to the cmake command in the installer, with this the compiler optimizes the code significantly better (without it a debug build is created).

Additionally the warning: ‘virtual void Symbols::match(Plasma::RunnerContext&)’ can be marked override [-Wsuggest-override] compiler warnings have been fixed.

domschrei commented 4 years ago

Thanks a lot! That's a lot of small fixes I should've integrated for a long time :)