# Warning reports

**URL:** <https://forum.juce.com/t/warning-reports/9270>\
**Category:** General JUCE discussion\
**Created:** [September 20, 2012, 2:56pm UTC](https://forum.juce.com/t/warning-reports/9270 "2012-09-20T14:56:08Z")\
**Posts on this page:** 14\
**Page:** 2

<div class="post-metadata">

**Author:** ![yfede](https://sea2.discourse-cdn.com/flex026/user_avatar/forum.juce.com/yfede/32/506_2.png) [@yfede](https://forum.juce.com/u/yfede)\
**Post date:** [September 24, 2012, 10:46am UTC](https://forum.juce.com/t/warning-reports/9270/21 "2012-09-24T10:46:47Z")

</div>

juce\_GZIPDecompressorInputStream.cpp, line 73:

```auto

```

noWrap is already a field of GZIPDecompressorInputStream

---

<div class="post-metadata">

**Author:** ![yfede](https://sea2.discourse-cdn.com/flex026/user_avatar/forum.juce.com/yfede/32/506_2.png) [@yfede](https://forum.juce.com/u/yfede)\
**Post date:** [September 24, 2012, 10:47am UTC](https://forum.juce.com/t/warning-reports/9270/22 "2012-09-24T10:47:59Z")

</div>

juce\_GZIPCompressorOutputStream.cpp

all the methods taking an argument named “destStream” trigger a warning because it is also a field of the class

---

<div class="post-metadata">

**Author:** ![yfede](https://sea2.discourse-cdn.com/flex026/user_avatar/forum.juce.com/yfede/32/506_2.png) [@yfede](https://forum.juce.com/u/yfede)\
**Post date:** [September 24, 2012, 10:50am UTC](https://forum.juce.com/t/warning-reports/9270/23 "2012-09-24T10:50:24Z")

</div>

juce\_posix\_SharedCode.h

all the “= { 0 };” initialisations trigger a warning, plus line 704:

```auto

```

name shadows a field of InterProcessLock class

---

<div class="post-metadata">

**Author:** ![yfede](https://sea2.discourse-cdn.com/flex026/user_avatar/forum.juce.com/yfede/32/506_2.png) [@yfede](https://forum.juce.com/u/yfede)\
**Post date:** [September 24, 2012, 10:56am UTC](https://forum.juce.com/t/warning-reports/9270/24 "2012-09-24T10:56:27Z")

</div>

juce\_mac\_Files.mm,

line 354:

```auto

```

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

---

<div class="post-metadata">

**Author:** ![yfede](https://sea2.discourse-cdn.com/flex026/user_avatar/forum.juce.com/yfede/32/506_2.png) [@yfede](https://forum.juce.com/u/yfede)\
**Post date:** [September 24, 2012, 11:07am UTC](https://forum.juce.com/t/warning-reports/9270/25 "2012-09-24T11:07:58Z")

</div>

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:

```auto

```

“key” is unused here

line 866:

```auto

```

inControllerType is unused

line 884:

```auto

```

“value” is unused

---

<div class="post-metadata">

**Author:** ![jules](https://avatars.discourse-cdn.com/v4/letter/j/41988e/32.png) [@jules](https://forum.juce.com/u/jules)\
**Post date:** [September 24, 2012, 11:09am UTC](https://forum.juce.com/t/warning-reports/9270/26 "2012-09-24T11:09:53Z")

</div>

Ok, stop, stop!! Much easier if I just enable those flags myself and work through them!

---

<div class="post-metadata">

**Author:** ![yfede](https://sea2.discourse-cdn.com/flex026/user_avatar/forum.juce.com/yfede/32/506_2.png) [@yfede](https://forum.juce.com/u/yfede)\
**Post date:** [September 24, 2012, 1:08pm UTC](https://forum.juce.com/t/warning-reports/9270/27 "2012-09-24T13:08:01Z")

</div>

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

---

<div class="post-metadata">

**Author:** ![yfede](https://sea2.discourse-cdn.com/flex026/user_avatar/forum.juce.com/yfede/32/506_2.png) [@yfede](https://forum.juce.com/u/yfede)\
**Post date:** [September 25, 2012, 10:37am UTC](https://forum.juce.com/t/warning-reports/9270/28 "2012-09-25T10:37:37Z")

</div>

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?

---

<div class="post-metadata">

**Author:** ![jules](https://avatars.discourse-cdn.com/v4/letter/j/41988e/32.png) [@jules](https://forum.juce.com/u/jules)\
**Post date:** [September 25, 2012, 10:48am UTC](https://forum.juce.com/t/warning-reports/9270/29 "2012-09-25T10:48:12Z")

</div>

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!

---

<div class="post-metadata">

**Author:** ![yfede](https://sea2.discourse-cdn.com/flex026/user_avatar/forum.juce.com/yfede/32/506_2.png) [@yfede](https://forum.juce.com/u/yfede)\
**Post date:** [September 25, 2012, 10:57am UTC](https://forum.juce.com/t/warning-reports/9270/30 "2012-09-25T10:57:00Z")

</div>

How many beers does it take? I may afford that after all 😃

---

<div class="post-metadata">

**Author:** ![jrlanglois](https://sea2.discourse-cdn.com/flex026/user_avatar/forum.juce.com/jrlanglois/32/21372_2.png) [@jrlanglois](https://forum.juce.com/u/jrlanglois)\
**Post date:** [September 25, 2012, 3:06pm UTC](https://forum.juce.com/t/warning-reports/9270/31 "2012-09-25T15:06:54Z")

</div>

This is the only warning I get in VS - been around for a while;

> [@](#):
>
> warning C4244: ‘=’ : conversion from ‘juce::int64’ to ‘double’, possible loss of data […]juce\modules\juce\_audio\_processors\format\_types\juce\_vstpluginformat.cpp Line 1012

---

<div class="post-metadata">

**Author:** ![yfede](https://sea2.discourse-cdn.com/flex026/user_avatar/forum.juce.com/yfede/32/506_2.png) [@yfede](https://forum.juce.com/u/yfede)\
**Post date:** [October 2, 2012, 9:35am UTC](https://forum.juce.com/t/warning-reports/9270/32 "2012-10-02T09:35:49Z")

</div>

`#ifdef __clang__
#pragma clang diagnostic push
#pragma clang diagnostic ignored "-Wshorten-64-to-32"
#endif
`  
and

```auto
#ifdef __clang__
#pragma clang diagnostic pop
#endif
```

should 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.

---

<div class="post-metadata">

**Author:** ![jules](https://avatars.discourse-cdn.com/v4/letter/j/41988e/32.png) [@jules](https://forum.juce.com/u/jules)\
**Post date:** [October 2, 2012, 10:01am UTC](https://forum.juce.com/t/warning-reports/9270/33 "2012-10-02T10:01:36Z")

</div>

Nice one, thanks!

---

<div class="post-metadata">

**Author:** ![yfede](https://sea2.discourse-cdn.com/flex026/user_avatar/forum.juce.com/yfede/32/506_2.png) [@yfede](https://forum.juce.com/u/yfede)\
**Post date:** [October 2, 2012, 10:20am UTC](https://forum.juce.com/t/warning-reports/9270/34 "2012-10-02T10:20:24Z")

</div>

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.

[Previous page](https://forum.juce.com/t/warning-reports/9270.md?page=1)
