I don't see any big problems at all. Here are some minor suggestions for improvements.

I believe you have a typo in your warn message: change 'characted' to 'character'.

You should indent the code in the 'Code Sample' section of the SYNOPSIS in your POD. When I use perldoc, it renders poorly for me. You should also remove the tab characters in that code example because it still renders poorly even after I indented it.

You should add more data validation for your input. If I do something that you probably don't expect, I do get lots of warnings, but they could be more helpful. I tried this (probably) illegal thing, and I got about 20 warnings:

my $foo = $gsm->decode_7bit_wlen(5);
Of course, that means you should add tests to your test suite. The Perl Best Practices book has a nice, succinct checklist of general-purpose things to test.

As a user, I absolutely love it when a CPAN author includes a script I can run out-of-the-box, preferably in the bin directory, or at least in the examples directory. This could be nothing more than the code from the SYNOPSIS.

There are plenty more style suggestions if you run the code through perlcritic.

Update: Do this before you upload your tar file: Fix CPAN uploads for world writable files

You should delete the INSTALLATION section from the POD. There is no need to tell someone how to install something which has already been installed.

Another typo: change 'costructor' to 'constructor'. I guess you should run your code through a spell-checker.


In reply to Re: RFC: Module - GSM::Nbit by toolic
in thread RFC: Module - GSM::Nbit by techcode

Title:
Use:  <p> text here (a paragraph) </p>
and:  <code> code here </code>
to format your post, it's "PerlMonks-approved HTML":



  • Posts are HTML formatted. Put <p> </p> tags around your paragraphs. Put <code> </code> tags around your code and data!
  • Titles consisting of a single word are discouraged, and in most cases are disallowed outright.
  • Read Where should I post X? if you're not absolutely sure you're posting in the right place.
  • Please read these before you post! —
  • Posts may use any of the Perl Monks Approved HTML tags:
    a, abbr, b, big, blockquote, br, caption, center, col, colgroup, dd, del, details, div, dl, dt, em, font, h1, h2, h3, h4, h5, h6, hr, i, ins, li, ol, p, pre, readmore, small, span, spoiler, strike, strong, sub, summary, sup, table, tbody, td, tfoot, th, thead, tr, tt, u, ul, wbr
  • You may need to use entities for some characters, as follows. (Exception: Within code tags, you can put the characters literally.)
            For:     Use:
    & &amp;
    < &lt;
    > &gt;
    [ &#91;
    ] &#93;
  • Link using PerlMonks shortcuts! What shortcuts can I use for linking?
  • See Writeup Formatting Tips and other pages linked from there for more info.