I used the Edward's repo now. <br><br><div class="gmail_quote">On Tue, Mar 31, 2009 at 4:11 AM, Felipe Contreras <span dir="ltr"><<a href="mailto:felipe.contreras@gmail.com">felipe.contreras@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><div><div></div><div class="h5">On Fri, Mar 27, 2009 at 11:12 AM, Prajnashi S <<a href="mailto:prajnashi@gmail.com">prajnashi@gmail.com</a>> wrote:<br>
><br>
><br>
> On Fri, Mar 27, 2009 at 5:43 AM, Felipe Contreras<br>
> <<a href="mailto:felipe.contreras@gmail.com">felipe.contreras@gmail.com</a>> wrote:<br>
>><br>
>> Hi,<br>
>><br>
>> I've taken your patch and I've put it in a new branch in github. I<br>
>> took a quick look and it seems it will require quite a bit of effort<br>
>> before this can be merged, bit doesn't look bad :)<br>
>><br>
>> First I want to start minimizing the changes and I already started<br>
>> with the aacdec, can you take a look and see if you can do the same<br>
>> for the other components?<br>
><br>
> OK, but I can do it after completing video flinger sink and java glue code.<br>
><br>
>><br>
>><br>
>> A few comments:<br>
>><br>
>> Remove BUILD_WITH_ANDROID, for now this branch should be only for<br>
>> Android. Once it's clean we can start merging the stuff that can be<br>
>> shared by all the omx implementations and then rebase on top of the<br>
>> latest master.<br>
><br>
> OK, actually, most of them is not android specific, they shall be moved<br>
> outside BUILD_WITH_ANDROID. But, we need analyze them case by case, because<br>
> I'm not sure if PV OpenMax follow spec strictly.<br>
><br>
>><br>
>> Can we remove the extra debugging stuff like "Enter" and "Leave"? Once<br>
>> the android branch is merged we can think on adding them back.<br>
>><br>
>> Also, is it possible for you to use the branch I just setup? Or a fork<br>
>> of it? That way we can verify that there are no regressions in the<br>
>> process.<br>
>><br>
>> <a href="http://github.com/felipec/gst-openmax/commits/android" target="_blank">http://github.com/felipec/gst-openmax/commits/android</a><br>
><br>
> Of cause, I can use your branch.<br>
<br>
</div></div>In which git repo are you working on right now?<br>
<br>
--<br>
<font color="#888888">Felipe Contreras<br>
</font></blockquote></div><br><br clear="all"><br>-- <br>-- Prajnashi S<br>