Wed Sep 22 13:26:17 2010: Request 61484 was acted upon.
Transaction: Correspondence added by DOLMEN
       Queue: Win32-EventLog
     Subject: [patch] missing results with Win32::EventLog
   Broken in: (no value)
    Severity: (no value)
       Owner: Nobody
  Requestors: p...@sennovation.com
      Status: open
 Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=61484 >


Le Mar 21 Sep 2010 10:04:50, p...@sennovation.com a écrit :
> Good catching the bug.

Well... My proposed fix did not work. I've been trapped by the variable
names. Anyway I finally found the problem.

> Worth noting in the module documentation,
> since others may miss it.

A real fix is always better than documentation.
So here is the full patch (really tested this time), with a test case.

Jan: can I have a commit bit on the libwin32 repository?

> Thanks!  I was working off the example in the O'Reilly perl sysadm
> book http://oreilly.com/catalog/perlsysadm/chapter/ch09.html

Thanks for this reference. It helped me to fix the bug while preserving
compatibility with this book sample.


-- 
Olivier Mengué - http://search.cpan.org/~dolmen/ http://github.com/dolmen/

Attachment: RT61484.patch
Description: Binary data

Reply via email to