Skip Menu |

This queue is for tickets about the Event-RPC CPAN distribution.

Report information
The Basics
Id: 107405
Status: open
Priority: 0/
Queue: Event-RPC
People
Owner: JRED [...] cpan.org
Requestors: ppisar [...] redhat.com
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: 1.08
Fixed in: (no value)

History Show all quoted text
  Tue Sep 29 06:38:00 2015 ppisar [...] redhat.com - Ticket created
Subject: Useless code in /Event/RPC/Message.pm
1.08's lib/Event/RPC/Message.pm defines %DECODERS and %ENCODERS hashes like: my %DECODERS = ( STOR => sub { require Storable; Storable::thaw($_[0]) }, ... ); I cannot find where the two hashes are used. I believe these are some remnants superseded by %MESSAGE_FORMATS in lib/Event/RPC/Message/Negotiate.pm. If it is so, I recomend to remove the dead code from the lib/Event/RPC/Message.pm.
  Tue Sep 29 14:16:59 2015 JRED [...] cpan.org - Correspondence added
Thanks for the report, you are right. I will remove this code for the next release. I think there is no need to push a new release quickly just for this, since this code is harmless and just wastes a few bytes for a hash with a few code refs :)
  Tue Sep 29 14:17:00 2015 The RT System itself - Status changed from 'new' to 'open'
  Tue Sep 29 14:17:00 2015 JRED [...] cpan.org - Taken