Good catch, i was lazy on the alpha value :p<br>i'll update that when i have a moment, thanks !!<br><br><div class="gmail_quote">On Mon, Sep 26, 2011 at 11:35 AM, Martin Storsjö <span dir="ltr"><<a href="mailto:martin@martin.st">martin@martin.st</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">This fixes displaying the RGBA data on HTC Sensation.<br>
---<br>
The code this patches is only in the vlc android port repo as a<br>
patch, and thus not intended for the main repo.<br>
<br>
This only fixes the reference (commented out) C implementation,<br>
the arm neon version needs to be updated accordingly, too.<br>
<br>
 modules/arm_neon/i420_rgb.c |    2 +-<br>
 1 files changed, 1 insertions(+), 1 deletions(-)<br>
<br>
diff --git a/modules/arm_neon/i420_rgb.c b/modules/arm_neon/i420_rgb.c<br>
index 3524022..96b3fa8 100644<br>
--- a/modules/arm_neon/i420_rgb.c<br>
+++ b/modules/arm_neon/i420_rgb.c<br>
@@ -84,7 +84,7 @@ static void I420_RGBA_C (filter_t *filter, picture_t *src, picture_t *dst)<br>
             if (unlikely(G > 255)) G = 255;<br>
             if (unlikely(B > 255)) B = 255;<br>
<br>
-            ((uint32_t*)out)[d + i] = R | (G<<8) | (B<<16);<br>
+            ((uint32_t*)out)[d + i] = R | (G<<8) | (B<<16) | (0xff<<24);<br>
         }<br>
     }<br>
 }<br>
<font color="#888888">--<br>
1.7.2.5<br>
<br>
_______________________________________________<br>
vlc-devel mailing list<br>
To unsubscribe or modify your subscription options:<br>
<a href="http://mailman.videolan.org/listinfo/vlc-devel" target="_blank">http://mailman.videolan.org/listinfo/vlc-devel</a><br>
</font></blockquote></div><br>