R: [PATCH]: [vdr] VDR Multiple frontends, another try

2007-02-10 Thread Eddi
Hi, I've made another patch to vdr 1.4.5.

Now I have used your suggestion, and the patch seems to be stable.

http://tux.dpeddi.com/lr319sta/downloads/vdr_1.4.5_eddi-multiple-frontend_v2
.patch

At the moment I've tested it only with live view.

Klaus, please look at this patch again and tell me what do you think.

However I'm sure that what I'm done can be optimized.

Best regards 

Eddi



 -Messaggio originale-
 Da: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] Per conto di
 Eddi
 Inviato: venerdì 9 febbraio 2007 14.28
 A: 'VDR Mailing List'
 Oggetto: R: [PATCH]: [vdr] VDR  Multiple frontends
 
 Ok, I hope that what I have done, may describe the algorithm that I think
 may be necessary to get multiple frontend working.
 I know that the solution I choose are not the best and right, I'm not a
 c++
 programmer!
 
 The only solution I found to close all file descriptors and
 /dev/dvb/adapter0/frontend0 was to add the close fd_frontend to the dvb
 device destructor.
 
 Eddi
 
  -Messaggio originale-
  Da: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] Per conto
 di
  Klaus Schmidinger
  Inviato: venerdì 9 febbraio 2007 13.43
  A: vdr@linuxtv.org
  Oggetto: Re: [PATCH]: [vdr] VDR  Multiple frontends
 
  Eddi wrote:
   So what do you think should be the right way?
 
  I haven't decided yet how to do that.
 
  I just wanted to let you know that deleting the cDvbDevice
  is not the way to go - especially if you delete it from within
  one of its own member functions ;-) It's no wonder your patched
  version fails it you saw off the branch you're sitting on.
 
  Klaus
 
   -Messaggio originale-
   Da: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] Per
 conto
  di
   Klaus Schmidinger
   Inviato: venerdì 9 febbraio 2007 8.27
   A: vdr@linuxtv.org
   Oggetto: Re: R: [PATCH] R: R: [vdr] VDR  Multiple frontends
  
   Eddi wrote:
   I need help
  
   This is why sometimes hangs after my patch...
  
   On ProvidesChannel I delete the cDvbDevice and i make it again.
   Deleting the cDvbDevice is definitely the wrong way to do this.
  
   Klaus
 
  ___
  vdr mailing list
  vdr@linuxtv.org
  http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr
 
 
 ___
 vdr mailing list
 vdr@linuxtv.org
 http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr


vdr_1.4.5_eddi-multiple-frontend_v2.patch
Description: Binary data
___
vdr mailing list
vdr@linuxtv.org
http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr


R: [PATCH]: [vdr] VDR Multiple frontends

2007-02-09 Thread Eddi
Ok, I hope that what I have done, may describe the algorithm that I think
may be necessary to get multiple frontend working.
I know that the solution I choose are not the best and right, I'm not a c++
programmer!

The only solution I found to close all file descriptors and
/dev/dvb/adapter0/frontend0 was to add the close fd_frontend to the dvb
device destructor.

Eddi

 -Messaggio originale-
 Da: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] Per conto di
 Klaus Schmidinger
 Inviato: venerdì 9 febbraio 2007 13.43
 A: vdr@linuxtv.org
 Oggetto: Re: [PATCH]: [vdr] VDR  Multiple frontends
 
 Eddi wrote:
  So what do you think should be the right way?
 
 I haven't decided yet how to do that.
 
 I just wanted to let you know that deleting the cDvbDevice
 is not the way to go - especially if you delete it from within
 one of its own member functions ;-) It's no wonder your patched
 version fails it you saw off the branch you're sitting on.
 
 Klaus
 
  -Messaggio originale-
  Da: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] Per conto
 di
  Klaus Schmidinger
  Inviato: venerdì 9 febbraio 2007 8.27
  A: vdr@linuxtv.org
  Oggetto: Re: R: [PATCH] R: R: [vdr] VDR  Multiple frontends
 
  Eddi wrote:
  I need help
 
  This is why sometimes hangs after my patch...
 
  On ProvidesChannel I delete the cDvbDevice and i make it again.
  Deleting the cDvbDevice is definitely the wrong way to do this.
 
  Klaus
 
 ___
 vdr mailing list
 vdr@linuxtv.org
 http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr


___
vdr mailing list
vdr@linuxtv.org
http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr