Skip Menu |

This queue is for tickets about the Imager-Screenshot CPAN distribution.

Report information
The Basics
Id: 72631
Status: resolved
Priority: 0/
Queue: Imager-Screenshot

People
Owner: Nobody in particular
Requestors: tech [...] vedurumudi.com
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)



Subject: Fwd: Minor correction for Imager::Screenshot code in CPAN to fix compilation error
Date: Tue, 22 Nov 2011 17:52:58 -0500
To: bug-Imager-Screenshot [...] rt.cpan.org
From: Sri Vedurumudi <tech [...] vedurumudi.com>
Hi Tony, Resending the issue, just to make sure the mail reaches you. Sorry if you got this twice. Also, here is some additional data: Perl version I'm using (but it doesn't matter): 5.14 Imager-Screenshot version: 0.010 Platform: Windows 32/64. Regards, Sri Show quoted text
---------- Forwarded message ---------- From: Sri Vedurumudi <tech@vedurumudi.com> Date: Mon, Nov 21, 2011 at 6:34 PM Subject: Minor correction for Imager::Screenshot code in CPAN to fix compilation error To: tony@develop-help.com Hi Tony, In the file scwin32.c, line 153, the following code exists: else { DISPLAY_DEVICE dd; dd.cb = sizeof(dd); int work_display = 0; int primary_display = -1; int real_display = -1; This was causing a compiler error when I tried to compile it using VC compiler on my Windows 7 machine. Changing it to: else { DISPLAY_DEVICE dd; int work_display = 0; int primary_display = -1; int real_display = -1; dd.cb = sizeof(dd); Fixed the issue. (just moving the dd.cb = sizeof(dd); statement to the end of variable declarations for that block). Please submit the code with this change for the compilation to work, so others can benefit from the cool library of yours. Best regards, Sri Vedurmudi
Subject: Re: [rt.cpan.org #72631] Fwd: Minor correction for Imager::Screenshot code in CPAN to fix compilation error
Date: Wed, 23 Nov 2011 10:48:58 +1100
To: Sri Vedurumudi via RT <bug-Imager-Screenshot [...] rt.cpan.org>
From: Tony Cook <tony [...] develop-help.com>
On Tue, Nov 22, 2011 at 05:53:09PM -0500, Sri Vedurumudi via RT wrote: Show quoted text
> Hi Tony, > Resending the issue, just to make sure the mail reaches you. Sorry if > you got this twice. > Also, here is some additional data:
Thanks. This is fixed for the next release of Imager-Screenshot. Sorry for not responding sooner. I'm planning on testing on various platforms tonight before doing a release. Tony
On Tue Nov 22 17:53:09 2011, tech@vedurumudi.com wrote: Show quoted text
> Hi Tony, > In the file scwin32.c, line 153, the following code exists: > > else { > DISPLAY_DEVICE dd; > dd.cb = sizeof(dd); > int work_display = 0; > int primary_display = -1; > int real_display = -1;
This is fixed in Imager-Screenshot 0.011 which I've just released to CPAN. Thank you. Tony