diff mbox

[ANNOUNCE] VDR developer version 1.7.28

Message ID 20120603125456.220750@gmx.net
State Accepted
Headers

Commit Message

Udo Richter June 3, 2012, 12:54 p.m. UTC 
  > > timers.c: In constructor ‘cSortedTimers::cSortedTimers()’:
> > timers.c:832: error: class ‘cSortedTimers’ does not have any field
> named
> > ‘cVector’
> 
> I don't get such an error message here.


May be a compiler dependent problem. The attached patch makes it work for me.


Cheers,

Udo

Comments

Wolfgang Rohdewald June 3, 2012, 1:01 p.m. UTC | #1 
Am Sonntag, 3. Juni 2012, 14:54:56 schrieb Udo Richter:
> > > timers.c: In constructor ‘cSortedTimers::cSortedTimers()’:
> > > timers.c:832: error: class ‘cSortedTimers’ does not have any field
> > 
> > named
> > 
> > > ‘cVector’
> > 
> > I don't get such an error message here.
> 
> May be a compiler dependent problem. The attached patch makes it work for
> me.

compiles with your patch, thanks!

server:/usr/local/src/vdr-1.7.28# gcc --version
gcc (Debian 4.4.5-8) 4.4.5
Klaus Schmidinger June 3, 2012, 1:03 p.m. UTC | #2 
On 03.06.2012 14:54, Udo Richter wrote:
>>> timers.c: In constructor ‘cSortedTimers::cSortedTimers()’:
>>> timers.c:832: error: class ‘cSortedTimers’ does not have any field
>> named
>>> ‘cVector’
>>
>> I don't get such an error message here.
>
>
> May be a compiler dependent problem. The attached patch makes it work for me.

> --- a/timers.c	2012-06-03 14:48:59.000000000 +0200
> +++ b/timers.c	2012-06-03 14:49:35.000000000 +0200
> @@ -829,7 +829,7 @@
>  }
>
>  cSortedTimers::cSortedTimers(void)
> -:cVector(Timers.Count())
> +:cVector<const cTimer *>(Timers.Count())
>  {
>    for (const cTimer *Timer = Timers.First(); Timer; Timer = Timers.Next(Timer))
>        Append(Timer);

Makes sense - I wonder why gcc version 4.5.1 20101208 didn't catch this.

@Wolfgang & Stefan: can you confirm that this fixes your problem?

Klaus
Stefan Lucke June 3, 2012, 3:12 p.m. UTC | #3 
On 03.06.2012 14:54, Udo Richter wrote:
>>> timers.c: In constructor ‘cSortedTimers::cSortedTimers()’:
>>> timers.c:832: error: class ‘cSortedTimers’ does not have any field
>> named
>>> ‘cVector’
>> I don't get such an error message here.
>
> May be a compiler dependent problem. The attached patch makes it work for me.

Patch works for me too, thanks.

Stefan Lucke
diff mbox

Patch

--- a/timers.c	2012-06-03 14:48:59.000000000 +0200
+++ b/timers.c	2012-06-03 14:49:35.000000000 +0200
@@ -829,7 +829,7 @@ 
 }
 
 cSortedTimers::cSortedTimers(void)
-:cVector(Timers.Count())
+:cVector<const cTimer *>(Timers.Count())
 {
   for (const cTimer *Timer = Timers.First(); Timer; Timer = Timers.Next(Timer))
       Append(Timer);