From 62188f6151c513a9e62883d98b7d10aaf811f802 Mon Sep 17 00:00:00 2001 From: Julian Oes Date: Tue, 20 Nov 2018 14:34:05 +0100 Subject: [PATCH] px_romfs_pruner.py: PEP8 and comments fixes --- Tools/px_romfs_pruner.py | 16 +++++++++------- 1 file changed, 9 insertions(+), 7 deletions(-) diff --git a/Tools/px_romfs_pruner.py b/Tools/px_romfs_pruner.py index 7fc45fb212..82e7c45281 100644 --- a/Tools/px_romfs_pruner.py +++ b/Tools/px_romfs_pruner.py @@ -1,7 +1,7 @@ #!/usr/bin/env python ############################################################################ # -# Copyright (C) 2014-2016 PX4 Development Team. All rights reserved. +# Copyright (C) 2014-2018 PX4 Development Team. All rights reserved. # Redistribution and use in source and binary forms, with or without # modification, are permitted provided that the following conditions @@ -35,10 +35,10 @@ """ px_romfs_pruner.py: -Try to keep size of ROMFS minimal. +Try to keep size of the ROMFS minimal. This script goes through the temporarily copied ROMFS data and deletes all -comments, empty lines and leading whitespace. +comments, empty lines and unnecessary whitespace. It also deletes hidden files such as auto-saved backups that a text editor might have left in the tree. @@ -91,12 +91,14 @@ def main(): board_excluded = False with io.open(file_path, "r", newline=None) as f: for line in f: - if re.search(r'\b{0} exclude\b'.format(args.board),line): - board_excluded = True; + if re.search(r'\b{0} exclude\b'.format(args.board), line): + board_excluded = True # handle mixer files differently than startup files if file_path.endswith(".mix"): - if line.startswith(("Z:", "M:", "R: ", "O:", "S:", "H:", "T:", "P:")): - # reduce multiple consecutive spaces into a single space + if line.startswith(("Z:", "M:", "R: ", "O:", "S:", + "H:", "T:", "P:")): + # reduce multiple consecutive spaces into a + # single space line_reduced = re.sub(' +', ' ', line) pruned_content += line_reduced else: