Fix opened_position intent and add "?startupindex=XX" to URI's

Examples of usage to start index 5:

intent:

 int vlcRequestCode = 42;
 Uri uri = Uri.parse("https://www.domain.com/playlist.m3u8");
 Intent vlcIntent = new Intent(Intent.ACTION_VIEW);
 vlcIntent.setPackage("org.videolan.vlc");
 vlcIntent.setDataAndTypeAndNormalize(uri, "audio/*");
 vlcIntent.putExtra("title", "Test");
 vlcIntent.putExtra("from_start", false);
 vlcIntent.putExtra("opened_position", 5);
 startActivityForResult(vlcIntent, vlcRequestCode);

URL: vlc://https://www.domain.com/playlist.m3u8?startupindex=5

added 72 commits

• 1447932a...3e3a5913 - 71 commits from branch videolan:master
• 25bd59e9 - Fix opened_position intent and add "?startupindex=XX" to uri's.

Compare with previous version

added 4 commits

• 25bd59e9...ee9b362f - 3 commits from branch videolan:master
• 325ae901 - Fix opened_position intent and add "?startupindex=XX" to uri's.

Compare with previous version

To be honest, your Merge request is modifying code that will (most probably) have a lot of side effects. You should at least split it in two commits to separate the fix from the feature before we can discuss it.

Also, can you please explain what opened_position is as it's not documented in https://wiki.videolan.org/Android_Player_Intents/

I'll close this MR. Feel free to reopen when all the questions above are answered.

closed

mentioned in merge request !1720 (merged)