Re: SDL::Cursor failure, plus patch

-----BEGIN PGP SIGNED MESSAGE-----

Moin Andy,

On Saturday 26 November 2005 05:23, Andy wrote:
> It seems that SDL/Cursor.pm needs a "use SDL;" in it in order to pull
> the definition of 'verify' into the scope.  Otherwise, the ->new
> function fails.
>
> Also, I find the fact that you need to create a cursor object in order
> to hide or show the cursor to be counter-intuitive.  SDL::Cursor::show
> should be callable as function, not just as a method.  Alternatively,
> the new method could be changed to not require making a new cursor just
> to access these functions (it could just return a blessed reference,
> rather than actually calling SDL::NewCursor).
>
> Also, and the SDL docs are confusing in this regard, the argument to
> show is labeled "$toggle", which implies there is some state kept.  I
> think a toggle_visiblity function would be nice, to make this obvious.
>
> See the attached patch which implements these changes, and fixes some
> minor issues with the POD section.

Cool. :)

Not to discourage you but what version of SDL are you using/patching 
against?

I lost the overview on what is current and where.

Best wishes,

Tels

- -- 
 Signed on Sat Nov 26 10:26:51 2005 with key 0x93B84C15.
 Visit my photo gallery at http://bloodgate.com/photos/
 PGP key on http://bloodgate.com/tels.asc or per email.

 "I'm not a vegetarian, but I eat animals who are" -- Groucho Marx

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.4 (GNU/Linux)

iQEVAwUBQ4gqnHcLPEOTuEwVAQFi+Af/QsgaqV/0IopeYmqqaZSzfoCmqEHRLe2D
xog6IR81X2/AMJT3DVQoupu/HalXPw8pCuUWgiQEsna0kpoP1woFCsweHbejnxqW
VFLLtGl6a/vB6zp1a+33B0pM8DjVGmO/LBXI9odLQET0KlmvnSSDVGqTSakmX0HH
nenMcm6JZMnGn7PnuYY8/0Wp+edFU5s9fir8HEARJZ/N4e0tnDkrbsFXBOzvQHre
Tl/UPM0D4UYMnQo2WE804vAWGUx7IgtFSDN8y8RqMrxQUfZr8BaFqLNbIkXeAfhl
u9trL+rtwQFb2hxRUkwulPzlW3fpIatJ00Xab2fb7xXL9xZhePgvXg==
=K9Mx
-----END PGP SIGNATURE-----