<div dir="ltr"><div dir="ltr"><div dir="ltr"><div>Hello,</div><div><br></div><div>Could somebody take a look at this old patch?</div><div><br></div><div>Thanks,</div><div>Andrii.<br></div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Nov 13, 2018 at 6:41 PM Eric Engestrom <<a href="mailto:eric.engestrom@intel.com">eric.engestrom@intel.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Tuesday, 2018-11-13 14:19:31 +0200, <a href="mailto:asimiklit.work@gmail.com" target="_blank">asimiklit.work@gmail.com</a> wrote:<br>
> From: Andrii Simiklit <<a href="mailto:andrii.simiklit@globallogic.com" target="_blank">andrii.simiklit@globallogic.com</a>><br>
> <br>
> 1. main/texcompress_etc.c:1314:12:<br>
>     warning: ‘*((void *)&dst+2)’ may be used uninitialized in this function<br>
> 2. main/texcompress_etc.c:1354:12:<br>
>     warning: ‘*((void *)&dst+2)’ may be used uninitialized in this function<br>
> 3. main/texcompress_etc.c:1293:12:<br>
>     warning: ‘dst’ may be used uninitialized in this function<br>
> 4. main/texcompress_etc.c:1335:12:<br>
>     warning: ‘dst’ may be used uninitialized in this function<br>
> 5. main/texcompress_etc.c:1460:12:<br>
>     warning: ‘*((void *)&dst+1)’ may be used uninitialized in this function<br>
> <br>
> v2: Fixed by adding the unreachable case to the etc2_rgb8_fetch_texel<br>
>        ( Eric Engestrom <<a href="mailto:eric.engestrom@intel.com" target="_blank">eric.engestrom@intel.com</a>> )<br>
>     Changes for warning 'pixerrorcolorbest' were removed.<br>
> <br>
> Signed-off-by: Andrii Simiklit <<a href="mailto:andrii.simiklit@globallogic.com" target="_blank">andrii.simiklit@globallogic.com</a>><br>
<br>
This is the right way of fixing this code-wise, but I'm not 100% sure we<br>
can actually guarantee this logic-wise, so I'll let someone else review<br>
(and push) this patch.<br>
<br>
Acked-by: Eric Engestrom <<a href="mailto:eric.engestrom@intel.com" target="_blank">eric.engestrom@intel.com</a>><br>
<br>
> ---<br>
>  src/mesa/main/texcompress_etc.c | 1 +<br>
>  1 file changed, 1 insertion(+)<br>
> <br>
> diff --git a/src/mesa/main/texcompress_etc.c b/src/mesa/main/texcompress_etc.c<br>
> index b39ab33d36..f1da4d0f11 100644<br>
> --- a/src/mesa/main/texcompress_etc.c<br>
> +++ b/src/mesa/main/texcompress_etc.c<br>
> @@ -548,6 +548,7 @@ etc2_rgb8_fetch_texel(const struct etc2_block *block,<br>
>        if (punchthrough_alpha)<br>
>           dst[3] = 255;<br>
>     }<br>
> +   else unreachable("unhandled block mode");<br>
>  }<br>
>  <br>
>  static void<br>
> -- <br>
> 2.17.1<br>
> <br>
</blockquote></div></div>