search

DISTRHO / DISTRHO-Ports

Linux audio plugins and LV2 ports
http://distrho.sourceforge.net/ports
252 stars 44 forks source link

Patches to fix compile warnings #35

Open gitterdude opened 5 years ago

gitterdude commented 5 years ago

When I compiled the latest git version of DISTRHO-Ports (commit 500ac85e3d10d) on Ubuntu 18.04 (gcc 7.3.0) it spewed out quite a few warnings and I've attached the fixes I used to shut them up.

I'm not that familiar with C++ , nor with the DISTRHO code, so please check if my solutions are correct before applying.

Another warning was ports/juce-opl/source/DROMultiplexer.cpp:292:21: warning: self-comparison always evaluates to false : for (int i = 0; i < i; i++) but I have no idea what "i" actually should be compared to. DISTRHO-Port-patches.tar.gz

falkTX commented 5 years ago

hello there, thanks for your efforts. do you mind creating a pull request? that way I can provide comments directly in the code

gitterdude commented 5 years ago

I've posted 5 PRs.

I didn't do any for the ports/juce-opl/source/DROMultiplexer.cpp:292:21: warning: self-comparison always evaluates to false : for (int i = 0; i < i; i++) since I don't know what it actually should be.

falkTX commented 5 years ago

all is good, thanks. I will have a look at them sometime soon.

some of them could be sent upstream to the official projects I guess.

falkTX commented 4 years ago

these are applied now right? I think I merged all PRs but forgot about this ticket