juce_GZIPDecompressorInputStream.cpp, line 73:
noWrap is already a field of GZIPDecompressorInputStream
juce_GZIPCompressorOutputStream.cpp
all the methods taking an argument named “destStream” trigger a warning because it is also a field of the class
juce_posix_SharedCode.h
all the “= { 0 };” initialisations trigger a warning, plus line 704:
name shadows a field of InterProcessLock class
juce_mac_Files.mm,
line 354:
path is a member of DirectoryIterator already
line 371:
DirectoryIterator::NativeIterator::NativeIterator (const File& directory, const String& wildCard)
wildCard is a member of DirectoryIterator already
juce_RTAS_Wrapper.cpp,
line 328:
EditorCompWrapper (void* const hostWindow_,
Component* const editorComp,
JuceCustomUIView* const owner_)
editorComp is a field of JucePlugInProcess::JuceCustomUIView and is shadowed by the constructor argument with the same name
line 864:
“key” is unused here
line 866:
inControllerType is unused
line 884:
“value” is unused
Ok, stop, stop!! Much easier if I just enable those flags myself and work through them!
Well thanks! I was getting tired of all that work 
I was going for “obsessively persuasive :D”
anyway, if you then need to lower the warning level again, I will keep an eye on possible warnings that slip in from time to time and report them here
Hi, I have noticed the great improvement!
There are still some lingering, though, which make me suspect that you didn’t enable the warnings for:
“Implicit Signedness Conversions” (-Wsign-conversion, CLANG_WARN_IMPLICIT_SIGN_CONVERSION),
“Implicit Conversion to 32 Bit Type” (-Wshorten-64-to-32, GCC_WARN_64_TO_32_BIT_CONVERSION)
“Suspicious Implicit Conversions” (-Wconversion, CLANG_WARN_SUSPICIOUS_IMPLICIT_CONVERSION)
“Unused Parameters” (-Wunused-parameter, GCC_WARN_UNUSED_PARAMETER)
will you take care of these as well?
Yeah, I couldn’t spend any more time on it. I generally do this kind of minor tidying-up when I’ve got a hangover and can’t concentrate on anything more challenging!
How many beers does it take? I may afford that after all
This is the only warning I get in VS - been around for a while;
#ifdef __clang__
#pragma clang diagnostic push
#pragma clang diagnostic ignored "-Wshorten-64-to-32"
#endif
and
#ifdef __clang__
#pragma clang diagnostic pop
#endifshould be added around the inclusions of AU source files in juce_AU_wrapper.mm (roughly between line 40 and 80) in order to avoid some warnings there.
Nice one, thanks!
For consistency with the other wrappers, I’d put the “push” section after the inclusion of juce_IncludeSystemHeaders.h, just to keep the area where they are inhibited to a minimum, and there are no warnings coming from that inclusion that need to be inhibited here.