-----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-----
