Skip Menu |

This queue is for tickets about the Net-SMPP CPAN distribution.

Report information
The Basics
Id: 49370
Status: open
Priority: 0/
Queue: Net-SMPP

People
Owner: Nobody in particular
Requestors: hlein [...] korelogic.com
Cc:
AdminCc:

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



Subject: Typoes in Net::SNMP examples / documentation
Some of the examples have misspellings or typos. It may be that the syntax of some things changed since an early version, and the examples are not up to date. For instance, from Net::SMPP 1.12, file SMPP.pm v1.31 line 3289: $smpp = Net::SMPP->new_transciever('smsc.foo.net', Port=>2552) or die; new_tranciever should be new_tranceiver Port should be port. I don't know if there are other bugs in the examples yet, I have not run them all. There are other cases of s/transciever/transceiver/ in comments and pod text, but none are as important as the code examples ;) (This probably comes from English not being Sampo's first language, and it having utterly redikulus spelling rules.)
CC: sampo [...] symlabs.com
Subject: Re: [rt.cpan.org #49370] Typoes in Net::SNMP examples / documentation
Date: Thu, 3 Sep 2009 13:52:38 +0200 (CEST)
To: bug-Net-SMPP [...] rt.cpan.org
From: sampo [...] symlabs.com
Hank Leininger via RT wrote: Show quoted text
> Wed Sep 02 12:33:19 2009: Request 49370 was acted upon. > Transaction: Ticket created by hlein > Queue: Net-SMPP > Subject: Typoes in Net::SNMP examples / documentation
Very misliading Subject line. Show quoted text
> Broken in: 1.12 > Severity: Important > Owner: Nobody > Requestors: hlein@korelogic.com > Status: new > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=49370 > > > > Some of the examples have misspellings or typos. It may be that the > syntax of some things changed since an early version, and the examples > are not up to date. For instance, from Net::SMPP 1.12, file SMPP.pm > v1.31 line 3289: > > $smpp = Net::SMPP->new_transciever('smsc.foo.net', Port=>2552) or die; > > new_tranciever should be new_tranceiver
Right. Someone once sent me a patch correcting the spelling in code, but as often happens, the patch did not correct spelling in documentation. THanks. Show quoted text
> Port should be port.
Clear typo as rest of the documentation is right. Show quoted text
> I don't know if there are other bugs in the examples yet, I have not run > them all. There are other cases of s/transciever/transceiver/ in > comments and pod text, but none are as important as the code examples ;) > (This probably comes from English not being Sampo's first language, and > it having utterly redikulus spelling rules.)
Right. Thanks for the fixes so far. Let me know how it goes. I'll hold off a release until you have had chance to report other bugs, if any. Net::SMPP was developed originally in perl 5.005 days so it is possible that newer versions of perl need adjustment. But to its merit, it is used in various production systems so at least the subset of functionality they use must be right. Cheers, --Sampo