Re: xine-plugin

Hi,

Siggi Langauf wrote:

> On Fri, 1 Aug 2003, Olivier Tache wrote:
>
>  
>
> Well, if you really _have_ to embed it, you'd better review the whole xine
> plugin code. I'm sure it could use some facelifting/updates/bugfixes...
> We're always happily accepting patches ;-)
>
> The solution to your DVD playback issue is probably adding support for an
> MRL parameter (replacing the mozilla-managed URL) that is handed directly
> to the plugin. However, (you guessed it) that involves actually working on
> the plugin...
>  
I tried this approach and it works. I attach a small patch to 
/xine-plugin/src/plugin.c if someone wants to commit it. It adds the "mrl"
parameter wich sould replace the classic "src" parameter of the <EMBED>
tag. This is a very quick hack and it's the first time i try and
contribute to xine so this patch should be refused ;-)
Anyway i thank Siggi for his help.

Cheers,
Olivier

101a102
>   int                 skip_NewStream;     /* true if the mrl parameter was 
> given in the EMBED tag */
662a664
>     this->skip_NewStream = 0;
707a710,712
>     } else if (!strcasecmp (argn[i], "mrl")) {
>     snprintf(this->url,MAX_URL,argv[i]);
>     this->skip_NewStream = 1;
797c802
<   /* printf("*** NPP_SetWindow: ");  fflush(stdout); */
---
>   /* printf("*** NPP_SetWindow: ");  fflush(stdout); */ 
836a842
> 
850c856,857
<   
---
>   /* if the mrl parameter was given in the EMBED tag, pass it directly to 
> xine-lib */
>   if (!this->skip_NewStream){
903a911,912
>   
>   }