Playing video without audio about androidffmpeg

[jacek-marchwicki]

I see that this file does not have Audio content is it?
now my player do not support video files without audio stream.

[Rrod8607]

Can the player be extended to handle files without audio streams?
would it be a modification of player.c ?
Right now I'm not building your project from source, just using the JAR & .so's

[jacek-marchwicki]

Yes, you have to modify synchronization in player.c.
Afaik there is some guy who working on this, but im not sure. If you want I
can try send him address to you?
On Nov 21, 2012 3:27 PM, "Rrod8607" [email protected] wrote:

Can the player be extended to handle files without audio streams?
would it be a modification of player.c ?
Right now I'm not building your project from source, just using the JAR &
.so's

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/11#issuecomment-10598775.

[sweetLou42]

I'd like the address of the guy working on modifying player.c to support no audio stream also please

[Rrod8607]

Jacek is this a modification you have on your to do list ?
Or should I start looking into doing it myself ?

[jacek-marchwicki]

@Rrod8607 No i do not have this in my working plan - is not needed for my propose.
I contacted with Simon Robinson and he is not working on solution either.

It would be glad if you can do it by yourself or maybe with @sweetLou42 .

[jacek-marchwicki]

It would be much simpler to do this when using version: https://review.appunite.com/#/c/3807/

[myrmidon-media]

@jacek-marchwicki

I have been looking into this, take a look at this patch:

struct Player {
.....
+ int no_audio;
....
}

int player_set_data_source(struct State *state, const char *file_path,
        AVDictionary *dictionary, int video_stream_no, int audio_stream_no,
        int subtitle_stream_no) {

...

-if ((player->audio_stream_no = player_find_stream(player,
-           AVMEDIA_TYPE_AUDIO, audio_stream_no)) < 0) {
-       err = player->audio_stream_no;
-       goto error;
-}


+if ((player->audio_stream_no = player_find_stream(player,
+           AVMEDIA_TYPE_AUDIO, audio_stream_no)) < 0) {
+       player->no_audio = TRUE;
    }
+   else{
+       player->no_audio = FALSE;
+   }


-if ((err = player_create_audio_track(player, state)) < 0)
-       goto error;

+ if (player->no_audio == FALSE && ((err = player_create_audio_track(player, state)) < 0))
+       goto error;

...
}


void jni_player_resume(JNIEnv *env, jobject thiz) {
....

-(*env)->CallVoidMethod(env, player->audio_track,
-           player->audio_track_play_method);

+if(player->no_audio == FALSE){
+     (*env)->CallVoidMethod(env, player->audio_track,
+           player->audio_track_play_method);
+   // just leave exception
+   }
....
}

Also substitute all:

-player_create_audio_track_free(player, state);

+if(player->no_audio == FALSE)
+       player_create_audio_track_free(player, state);

[bhollosi]

This addon has saved my life, thanks! Why not patch it into the main branch? Nice job!

[jacek-marchwicki]

Sorry... But I did not have time to watch on that. You can do a pull request.

[jacek-marchwicki]

Fixed