Bug 475 - Use of uninitialized value in repeat (x) at Slim/Display/Graphics.pm line 56.
: Use of uninitialized value in repeat (x) at Slim/Display/Graphics.pm line 56.
Status: RESOLVED WORKSFORME
Product: Logitech Media Server
Classification: Unclassified
Component: Tools
: 5.x or older
: Other RedHat Linux
: P2 trivial (vote)
: ---
Assigned To: Blackketter Dean
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2004-08-04 19:09 UTC by Jim Knepley
Modified: 2008-12-18 11:51 UTC (History)
1 user (show)

See Also:
Category: ---


Attachments
initialise $cursorend to avoid a warnign msg (181 bytes, patch)
2004-08-17 15:21 UTC, KDF
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jim Knepley 2004-08-04 19:09:44 UTC
Use of uninitialized value in repeat (x) at Slim/Display/Graphics.pm line 56.

It appears that $cursorend is not always set to a defined value and generates a
warning (only verified under --d_plugins).
Comment 1 KDF 2004-08-05 21:48:50 UTC
I saw this too, while working on INPUT:Text.  This might be fixed becuase I
haven't seen it since cleaning it up.  If you have a repeatable case where this
would occur, please test it again after trying the Aug 6 build.  Let us now if
this issue has been resolved.
Comment 2 Blackketter Dean 2004-08-12 10:02:47 UTC
Appears to be fixed.  Reopen if it's still a problem.
Comment 3 KDF 2004-08-12 11:11:21 UTC
still occurs in certain cases then exiting INPUT.Text using SavePlaylist or
Live365 Plugins. The string is split on null so it seems odd that $char would be
undefined. However, forcing $cursorend to be simply 0 if not previously defined.
Comment 4 KDF 2004-08-17 15:21:24 UTC
Created attachment 105 [details]
initialise $cursorend to avoid a warnign msg

This defines $cursorend so that the value is never undefined.  The later test
can still use !$cursorend instead of !defined with the same results.
Comment 5 KDF 2004-08-20 13:44:12 UTC
looks like Dean commited this fix a day or two ago.  Jim, if you can confirm
that this also works for you, please feel free to mark this one as closed.
Comment 6 Chris Owens 2008-12-18 11:51:25 UTC
Routine bug db maintenance; removing old versions which cause confusion.  I apologize for the inconvenience.