ffAudio / ff_meters

Plug and play component to display LED meters for JUCE audio buffers
BSD 3-Clause "New" or "Revised" License
113 stars 31 forks source link

Unable to build ff_meters with latest version of Juce #12

Open mediawizrd opened 4 years ago

mediawizrd commented 4 years ago

I get errors on the following header files when I try to build ff_meters using the latest version of the Juce library:

include <juce_audio_basics/juce_audio_basics.h>

include <juce_gui_basics/juce_gui_basics.h>

include <juce_events/juce_events.h>

ffAudio commented 4 years ago

Thanks for using ff_meters.

Do you mind sharing a little more detail?

I must admit, that currently I don't have an active project using the meters, but I'm still happy to fix it, if we can get to the bottom of it.

ffAudio commented 4 years ago

I tried and didn't have a problem. If it was related to issue #13 this should be fixed now. Can you please try again? (you will need to resave with Projucer or use whatever method to update your project files)

mediawizrd commented 4 years ago

Hi Daniel

I am attaching my build log.  These sorts of errors seem to occur every time I try to include the  ffAudio/ff_meters module in my build.  I am having trouble building the code with both VS2017 and VS2019, when I use the latest (production) release of Juce (Version 5.4.7)..

Russ B

On 6/21/2020 2:58:38 PM, Daniel Walz notifications@github.com wrote: I tried and didn't have a problem. If it was related to issue #13 [https://github.com/ffAudio/ff_meters/issues/13] this should be fixed now. Can you please try again? (you will need to resave with Projucer or use whatever method to update your project files) — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub [https://github.com/ffAudio/ff_meters/issues/12#issuecomment-647187053], or unsubscribe [https://github.com/notifications/unsubscribe-auth/AQAPFESIW7JFQURKLP6UDEDRXZ7A3ANCNFSM4ODBYC4Q].

ffAudio commented 4 years ago

Yes, a buildlog would help massively. Unfortunately your attachment didn't go through, I guess github doesn't support attachments. Just copy it in the post...

Thanks!

mediawizrd commented 4 years ago

I am pasting the build log errors in below.

  include_ff_meters_LevelMeter.cpp c:\dev\x-platform\modules\ff_meters\ff_meters_soundfieldlookandfeelmethods.h(107): error C2429: attribute 'maybe_unused' requires compiler flag '/std:c++17' (compiling source file ....\Source\MonoInputChannelCls.cpp) c:\dev\x-platform\modules\ff_meters\ff_meters_soundfieldlookandfeelmethods.h(108): error C2429: attribute 'maybe_unused' requires compiler flag '/std:c++17' (compiling source file ....\Source\MonoInputChannelCls.cpp) c:\dev\x-platform\modules\ff_meters\ff_meters_soundfieldlookandfeelmethods.h(109): error C2429: attribute 'maybe_unused' requires compiler flag '/std:c++17' (compiling source file ....\Source\MonoInputChannelCls.cpp) c:\dev\x-platform\modules\ff_meters\ff_meters_soundfieldlookandfeelmethods.h(110): error C2429: attribute 'maybe_unused' requires compiler flag '/std:c++17' (compiling source file ....\Source\MonoInputChannelCls.cpp) c:\dev\x-platform\modules\ff_meters\ff_meters_soundfieldlookandfeelmethods.h(111): error C2429: attribute 'maybe_unused' requires compiler flag '/std:c++17' (compiling source file ....\Source\MonoInputChannelCls.cpp) c:\dev\x-platform\modules\ff_meters\ff_meters_lookandfeelmethods.h(412): warning C4456: declaration of 'numChannels' hides previous local declaration (compiling source file ....\Source\MonoInputChannelCls.cpp)   c:\dev\x-platform\modules\ff_meters\ff_meters_lookandfeelmethods.h(334): note: see declaration of 'numChannels' (compiling source file ....\Source\MonoInputChannelCls.cpp) c:\dev\x-platform\modules\ff_meters\ff_meters_soundfieldlookandfeelmethods.h(107): error C2429: attribute 'maybe_unused' requires compiler flag '/std:c++17' (compiling source file ....\Source\Main.cpp) c:\dev\x-platform\modules\ff_meters\ff_meters_soundfieldlookandfeelmethods.h(108): error C2429: attribute 'maybe_unused' requires compiler flag '/std:c++17' (compiling source file ....\Source\Main.cpp) c:\dev\x-platform\modules\ff_meters\ff_meters_soundfieldlookandfeelmethods.h(109): error C2429: attribute 'maybe_unused' requires compiler flag '/std:c++17' (compiling source file ....\Source\Main.cpp) c:\dev\x-platform\modules\ff_meters\ff_meters_soundfieldlookandfeelmethods.h(110): error C2429: attribute 'maybe_unused' requires compiler flag '/std:c++17' (compiling source file ....\Source\Main.cpp) c:\dev\x-platform\modules\ff_meters\ff_meters_soundfieldlookandfeelmethods.h(111): error C2429: attribute 'maybe_unused' requires compiler flag '/std:c++17' (compiling source file ....\Source\Main.cpp) c:\dev\x-platform\modules\ff_meters\ff_meters_soundfieldlookandfeelmethods.h(107): error C2429: attribute 'maybe_unused' requires compiler flag '/std:c++17' (compiling source file ....\Source\MainComponent.cpp) c:\dev\x-platform\modules\ff_meters\ff_meters_soundfieldlookandfeelmethods.h(108): error C2429: attribute 'maybe_unused' requires compiler flag '/std:c++17' (compiling source file ....\Source\MainComponent.cpp) c:\dev\x-platform\modules\ff_meters\ff_meters_soundfieldlookandfeelmethods.h(109): error C2429: attribute 'maybe_unused' requires compiler flag '/std:c++17' (compiling source file ....\Source\MainComponent.cpp) c:\dev\x-platform\modules\ff_meters\ff_meters_soundfieldlookandfeelmethods.h(110): error C2429: attribute 'maybe_unused' requires compiler flag '/std:c++17' (compiling source file ....\Source\MainComponent.cpp) c:\dev\x-platform\modules\ff_meters\ff_meters_soundfieldlookandfeelmethods.h(111): error C2429: attribute 'maybe_unused' requires compiler flag '/std:c++17' (compiling source file ....\Source\MainComponent.cpp) c:\dev\x-platform\modules\ff_meters\ff_meters_lookandfeelmethods.h(412): warning C4456: declaration of 'numChannels' hides previous local declaration (compiling source file ....\Source\Main.cpp)   c:\dev\x-platform\modules\ff_meters\ff_meters_lookandfeelmethods.h(334): note: see declaration of 'numChannels' (compiling source file ....\Source\Main.cpp) c:\dev\x-platform\modules\ff_meters\ff_meters_lookandfeelmethods.h(412): warning C4456: declaration of 'numChannels' hides previous local declaration (compiling source file ....\Source\MainComponent.cpp)   c:\dev\x-platform\modules\ff_meters\ff_meters_lookandfeelmethods.h(334): note: see declaration of 'numChannels' (compiling source file ....\Source\MainComponent.cpp) c:\dev\x-platform\modules\ff_meters\ff_meters_soundfieldlookandfeelmethods.h(107): error C2429: attribute 'maybe_unused' requires compiler flag '/std:c++17' (compiling source file ....\JuceLibraryCode\include_ff_meters_LevelMeter.cpp) c:\dev\x-platform\modules\ff_meters\ff_meters_soundfieldlookandfeelmethods.h(108): error C2429: attribute 'maybe_unused' requires compiler flag '/std:c++17' (compiling source file ....\JuceLibraryCode\include_ff_meters_LevelMeter.cpp) c:\dev\x-platform\modules\ff_meters\ff_meters_soundfieldlookandfeelmethods.h(109): error C2429: attribute 'maybe_unused' requires compiler flag '/std:c++17' (compiling source file ....\JuceLibraryCode\include_ff_meters_LevelMeter.cpp) c:\dev\x-platform\modules\ff_meters\ff_meters_soundfieldlookandfeelmethods.h(110): error C2429: attribute 'maybe_unused' requires compiler flag '/std:c++17' (compiling source file ....\JuceLibraryCode\include_ff_meters_LevelMeter.cpp) c:\dev\x-platform\modules\ff_meters\ff_meters_soundfieldlookandfeelmethods.h(111): error C2429: attribute 'maybe_unused' requires compiler flag '/std:c++17' (compiling source file ....\JuceLibraryCode\include_ff_meters_LevelMeter.cpp) c:\dev\x-platform\modules\ff_meters\ff_meters_lookandfeelmethods.h(412): warning C4456: declaration of 'numChannels' hides previous local declaration (compiling source file ....\JuceLibraryCode\include_ff_meters_LevelMeter.cpp)   c:\dev\x-platform\modules\ff_meters\ff_meters_lookandfeelmethods.h(334): note: see declaration of 'numChannels' (compiling source file ....\JuceLibraryCode\include_ff_meters_LevelMeter.cpp) c:\dev\x-platform\modules\ff_meters\ff_meters_levelmeter.cpp(44): error C2429: attribute 'maybe_unused' requires compiler flag '/std:c++17' (compiling source file ....\JuceLibraryCode\include_ff_meters_LevelMeter.cpp) c:\dev\x-platform\modules\ff_meters\ff_meters_levelmeter.cpp(51): error C2429: attribute 'maybe_unused' requires compiler flag '/std:c++17' (compiling source file ....\JuceLibraryCode\include_ff_meters_LevelMeter.cpp)

On 6/22/2020 12:05:06 PM, Daniel Walz notifications@github.com wrote:

Yes, a buildlog would help massively. Unfortunately your attachment didn't go through, I guess github doesn't support attachments. Just copy it in the post...

Thanks!

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub [https://github.com/ffAudio/ff_meters/issues/12#issuecomment-647717124], or unsubscribe [https://github.com/notifications/unsubscribe-auth/AQAPFER2LVPUSPXN4I6DHZTRX6TOBANCNFSM4ODBYC4Q].

ffAudio commented 4 years ago

Oh, I see. I wasn't aware that maybe_unused is a C++17 feature. You can easily fix that by switching your project to C++17, but I will move back to use ignoreUnused() so you don't have to switch just for that feature...

Thanks for the report