[chirp_devel] [PATCH 1 of 1] [FT817-857] reduce pylint to silence
Dan Smith
Sat Apr 28 05:47:32 PDT 2012
> # HG changeset patch
> # User Filippi Marco <iz3gme.marco at gmail.com>
> # Date 1335607418 -7200
> # Node ID 5f6c45e053c37213c388b3a2908793e66672e4cf
> # Parent f805c420592f8a8b1b0dd89107920c6bbed7fdf7
> [FT817-857] reduce pylint to silence
Yikes! You're too quick! :)
I should have said something about this, but I didn't expect anyone to
pay it much attention. I think that some of the stuff that pylint
complains about are things that perhaps don't make much sense and that
we might want to change the config just a bit.
However, thanks very much for doing this and being so proactive :)
> -def _ft8x7_read(pipe, block, blocknum):
I was going to leave these types of functions at the module level
instead of pushing them into the class. Doing the latter makes the
classes much larger and potentially harder to understand. However, it
does make it easier to access pieces of the radio, such as
_block_lengths in your case.
What do people think? Should they go in or out of the class?
--
Dan Smith
www.danplanet.com
KK7DS
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 490 bytes
Desc: OpenPGP digital signature
Url : http://intrepid.danplanet.com/pipermail/chirp_devel/attachments/20120428/98d3fe2a/attachment-0001.bin
More information about the chirp_devel
mailing list