<div dir="ltr">Thanks, merged with the fix.</div><br><div class="gmail_quote"><div dir="ltr">On Wed, Jul 12, 2017 at 11:03 AM Jean-Marc Valin <<a href="mailto:jmvalin@jmvalin.ca">jmvalin@jmvalin.ca</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">r+ with James' fix.<br>
<br>
On 12/07/17 02:01 PM, James Zern wrote:<br>
> On Wed, Jul 12, 2017 at 10:58 AM, Felicia Lim <<a href="mailto:flim@google.com" target="_blank">flim@google.com</a>> wrote:<br>
>> |frame_size_enum| in tests/<a href="http://test_opus_encode.cl:117" rel="noreferrer" target="_blank">test_opus_encode.cl:117</a> is flagged as potentially<br>
>> uninitialized but get_frame_size_enum() will fail anyway if a valid value is<br>
>> not found.<br>
>> ---<br>
>>  tests/test_opus_common.h | 2 ++<br>
>>  1 file changed, 2 insertions(+)<br>
>><br>
>> diff --git a/tests/test_opus_common.h b/tests/test_opus_common.h<br>
>> index ff7f0142..8b878607 100644<br>
>> --- a/tests/test_opus_common.h<br>
>> +++ b/tests/test_opus_common.h<br>
>> @@ -64,6 +64,8 @@ static opus_uint32 iseed;<br>
>><br>
>>  #ifdef __GNUC__<br>
>>  __attribute__((noreturn))<br>
>> +#elif _MSC_VER<br>
><br>
> defined(_MSC_VER) would avoid any new warnings from -Wundef.<br>
><br>
>> +__declspec(noreturn)<br>
>>  #endif<br>
>>  static OPUS_INLINE void _test_failed(const char *file, int line)<br>
>>  {<br>
>> --<br>
>> 2.13.2.932.g7449e964c-goog<br>
>><br>
>> _______________________________________________<br>
>> opus mailing list<br>
>> <a href="mailto:opus@xiph.org" target="_blank">opus@xiph.org</a><br>
>> <a href="http://lists.xiph.org/mailman/listinfo/opus" rel="noreferrer" target="_blank">http://lists.xiph.org/mailman/listinfo/opus</a><br>
> _______________________________________________<br>
> opus mailing list<br>
> <a href="mailto:opus@xiph.org" target="_blank">opus@xiph.org</a><br>
> <a href="http://lists.xiph.org/mailman/listinfo/opus" rel="noreferrer" target="_blank">http://lists.xiph.org/mailman/listinfo/opus</a><br>
><br>
</blockquote></div>