PDA

View Full Version : Large number of warnings in Info.pm



Jason Holtzapple
2003-11-30, 14:36
In the current nightly release (30 Nov) I get a very large number of
warnings from Info.pm:

Use of uninitialized value in addition (+) at Slim/Music/Info.pm line 1653.
....

I assume that it is safe for this value to be zero if undefined, so I
cleared these warnings by changing line 1649 to:

my $header = $tempCacheEntry->{'OFFSET'} || 0;

Thanks
--Jason

__________________________________
Do you Yahoo!?
Free Pop-Up Blocker - Get it now
http://companion.yahoo.com/

dean
2003-12-01, 13:49
Jason,

I'm not sure that this is the right solution, I'd rather find out why
the offsets are zero to begin with.

Do you have an idea of what kind of song files you might have in your
library that could be triggering these warnings? (Turn on --d_info and
you should see some context around the warnings...)

-dean

On Nov 30, 2003, at 1:36 PM, Jason Holtzapple wrote:

> In the current nightly release (30 Nov) I get a very large number of
> warnings from Info.pm:
>
> Use of uninitialized value in addition (+) at Slim/Music/Info.pm line
> 1653.
> ...
>
> I assume that it is safe for this value to be zero if undefined, so I
> cleared these warnings by changing line 1649 to:
>
> my $header = $tempCacheEntry->{'OFFSET'} || 0;
>
> Thanks
> --Jason
>
> __________________________________
> Do you Yahoo!?
> Free Pop-Up Blocker - Get it now
> http://companion.yahoo.com/
>

Dan Sully
2003-12-02, 08:37
Fixed.

* Jason Holtzapple <jasonholtzapple (AT) yahoo (DOT) com> shaped the electrons to say...

> Music/Info.pm needs a
>
> use Slim::Formats::MP3;
>
> at the top.
>
> thanks,
> --Jason
>
> __________________________________
> Do you Yahoo!?
> Free Pop-Up Blocker - Get it now
> http://companion.yahoo.com/
>