<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
</head>
<body text="#000000" bgcolor="#FFFFFF">
<p>Hi Aman<br>
</p>
<p>The original patch is correct. Sorry. I need a better process for
this. The patch from MythTV does not apply in FFmpeg because of
the different path, so I changed it manually, wrongly. I will fix
that. Can I resubmit the patch once fixed?</p>
<p>Peter<br>
</p>
<br>
<div class="moz-cite-prefix">On 07/19/2018 12:11 PM, Aman Gupta
wrote:<br>
</div>
<blockquote type="cite"
cite="mid:CAK=uwuyM2d6=3gL+K2XvSjw-sCG6pMMggaCymqvXppmpWdKwsw@mail.gmail.com">
<meta http-equiv="content-type" content="text/html; charset=utf-8">
<div dir="ltr">Hi Peter,
<div><br>
</div>
<div>The patch you submitted to the list changes <span
style="font-size:small;background-color:rgb(255,255,255);text-decoration-style:initial;text-decoration-color:initial;float:none;display:inline">ff_mediacodec_sw_buffer_copy_y</span><span
style="font-size:small;background-color:rgb(255,255,255);text-decoration-style:initial;text-decoration-color:initial;float:none;display:inline">uv420_planar,
but the patch you sent me above
changes ff_mediacodec_sw_buffer_copy_yuv420_semi_planar</span></div>
<div><span
style="font-size:small;background-color:rgb(255,255,255);text-decoration-style:initial;text-decoration-color:initial;float:none;display:inline"><br>
</span></div>
<div><span
style="font-size:small;background-color:rgb(255,255,255);text-decoration-style:initial;text-decoration-color:initial;float:none;display:inline">Which
one is correct?</span></div>
<div><span
style="font-size:small;background-color:rgb(255,255,255);text-decoration-style:initial;text-decoration-color:initial;float:none;display:inline"><br>
</span></div>
<div><span
style="font-size:small;background-color:rgb(255,255,255);text-decoration-style:initial;text-decoration-color:initial;float:none;display:inline">Aman</span></div>
</div>
<br>
<div class="gmail_quote">
<div dir="ltr">On Wed, Jul 18, 2018 at 12:06 PM Aman Gupta <<a
href="mailto:aman@tmm1.net" moz-do-not-send="true">aman@tmm1.net</a>>
wrote:<br>
</div>
<blockquote class="gmail_quote" style="margin:0 0 0
.8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr">Please submit to ffmpeg-devel so others can
review. I will commit if there are no objections there.
<div><br>
</div>
<div>Aman</div>
</div>
<br>
<div class="gmail_quote">
<div dir="ltr">On Wed, Jul 18, 2018 at 12:03 PM Peter
Bennett <<a href="mailto:pb.mythtv@gmail.com"
target="_blank" moz-do-not-send="true">pb.mythtv@gmail.com</a>>
wrote:<br>
</div>
<blockquote class="gmail_quote" style="margin:0 0 0
.8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Aman<br>
<br>
This patch fixes the seg fault in mediacodec with 1080i
playback on <br>
android oreo. Also I have confirmation from others who
have tried it <br>
that it does not cause any issue with earlier android
versions. Please <br>
can you get it committed to FFmpeg. See attached file.<br>
<br>
Peter<br>
<br>
</blockquote>
</div>
</blockquote>
</div>
</blockquote>
<br>
</body>
</html>