Project

General

Profile

Actions

Bug #5099

closed

Code Cleanup patch

Added by Ryan J about 7 years ago. Updated over 4 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Category:
-
Target version:
Start date:
08/24/2017
Due date:
% Done:

0%

Estimated time:
Chirp Version:
daily
Model affected:
(All models)
Platform:
Windows
Debug Log:
I read the instructions above:

Description

Hey there,

I just inherited a pile of radios and started playing with CHIRP. I noticed that some of the code style is showing it's age and I went ahead and did a general cleanup. They only thing preventing this from running on Python3 is the use of unicode() in repeaterbook.py, platform.py and pyPEG.py. I decided to take a break and show the work I've done to make sure this is desired.

I am attaching a patch file here but did the work on a Github mirror of the code (https://github.com/Cabalist/chirp) I know you all use mercury and not Github but I couldn't find a way to open a pull request here. Let me know if you have any questions or even if this is wanted.

Thanks! I really appreciate that this exists. :)


Files

modernize_python.patch (203 KB) modernize_python.patch Python2+3 changes Ryan J, 08/24/2017 11:44 PM
Actions

Also available in: Atom PDF