summaryrefslogtreecommitdiff
path: root/android
diff options
context:
space:
mode:
authorLukas <lukasmolleman@gmail.com>2022-04-20 19:54:39 +0200
committerIlmari Lauhakangas <ilmari.lauhakangas@libreoffice.org>2022-06-22 14:07:19 +0200
commit157298bb808a943616991927f9370a86c1f2ca48 (patch)
treea0ea376a0ce971e917df43af8e6e676038797c1c /android
parent4e9b23cb2356cf7019a6ed81e08abf1664a72051 (diff)
android and bin/update: make pythonic
Change-Id: Iaf791bfa8d9822843b26f2a2f2c3d94c55a60a0b Reviewed-on: https://gerrit.libreoffice.org/c/core/+/133358 Reviewed-by: Arkadiy Illarionov <qarkai@gmail.com> Tested-by: Ilmari Lauhakangas <ilmari.lauhakangas@libreoffice.org> Reviewed-by: Ilmari Lauhakangas <ilmari.lauhakangas@libreoffice.org>
Diffstat (limited to 'android')
-rwxr-xr-xandroid/mobile-config.py37
1 files changed, 23 insertions, 14 deletions
diff --git a/android/mobile-config.py b/android/mobile-config.py
index 5998e5d1f9b7..596f27f107a8 100755
--- a/android/mobile-config.py
+++ b/android/mobile-config.py
@@ -14,13 +14,13 @@
# already over-complicated by rampant conditionals.
import sys
-import xml.etree.ElementTree as ET
+import xml.etree.ElementTree as ElementTree
main_xcd_discard = [
- 'org.openoffice.Office/TableWizard', # huge
+ 'org.openoffice.Office/TableWizard', # huge
- 'org.openoffice.Office.DataAccess/Drivers', # no database
- 'org.openoffice.Office/Addons', # no addons
+ 'org.openoffice.Office.DataAccess/Drivers', # no database
+ 'org.openoffice.Office/Addons', # no addons
# no conventional UI; reverse sorted by size
'org.openoffice.Office.UI/GenericCommands',
@@ -46,15 +46,15 @@ main_xcd_discard = [
'org.openoffice.Office.UI/GlobalSettings',
'org.openoffice.Office.UI/BibliographyWindowState',
'org.openoffice.Office.UI/Category',
- ]
+]
if __name__ == '__main__':
- tree = ET.parse(sys.argv[1])
+ tree = ElementTree.parse(sys.argv[1])
root = tree.getroot()
total = 0
for child in root:
- total += len(ET.tostring(child))
+ total += len(ElementTree.tostring(child))
saved = 0
to_remove = []
@@ -62,7 +62,7 @@ if __name__ == '__main__':
for child in root:
section = child.attrib['{http://openoffice.org/2001/registry}name']
package = child.attrib['{http://openoffice.org/2001/registry}package']
- size = len(ET.tostring(child));
+ size = len(ElementTree.tostring(child))
key = '%s/%s' % (package, section)
if key in main_xcd_discard:
print('removed %s - saving %d' % (key, size))
@@ -72,7 +72,7 @@ if __name__ == '__main__':
for child in to_remove:
root.remove(child)
- print("saved %d of %d bytes: %2.f%%" % (saved, total, saved*100.0/total))
+ print("saved %d of %d bytes: %2.f%%" % (saved, total, saved * 100.0 / total))
# Don't do pointless Word -> Writer and similar conversions when we have no UI.
nsDict = {
@@ -80,28 +80,37 @@ if __name__ == '__main__':
"component-data": "{http://openoffice.org/2001/registry}component-data",
"name": "{http://openoffice.org/2001/registry}name",
}
- microsoftImport = '%(component-schema)s[@%(name)s="Common"]/component/group[@%(name)s="Filter"]/group[@%(name)s="Microsoft"]/group[@%(name)s="Import"]/prop' % nsDict
+ microsoftImport = '%(component-schema)s[@%(name)s="Common"]/component/group[@%(name)s="Filter"]/group[@%(' +\
+ 'name)s="Microsoft"]/group[@%(name)s="Import"]/prop' % nsDict
props = root.findall(microsoftImport)
for prop in props:
prop.findall("value")[0].text = "false"
# Disable View -> Text Boundaries
- for prop in root.findall('%(component-schema)s[@%(name)s="UI"]/templates/group[@%(name)s="ColorScheme"]/group[@%(name)s="DocBoundaries"]/prop' % nsDict):
+ for prop in root.findall(
+ '%(component-schema)s[@%(name)s="UI"]/templates/group[@%(name)s="ColorScheme"]/group[@%(' +
+ 'name)s="DocBoundaries"]/prop' % nsDict):
for value in prop.findall("value"):
value.text = "false"
# Disable Table -> Table Boundaries
- for prop in root.findall('%(component-schema)s[@%(name)s="UI"]/templates/group[@%(name)s="ColorScheme"]/group[@%(name)s="TableBoundaries"]/prop' % nsDict):
+ for prop in root.findall(
+ '%(component-schema)s[@%(name)s="UI"]/templates/group[@%(name)s="ColorScheme"]/group[@%(' +
+ 'name)s="TableBoundaries"]/prop' % nsDict):
for value in prop.findall("value"):
value.text = "false"
# Disable follow link with Ctrl+Click, use Click only for mobile app.
- for prop in root.findall('%(component-schema)s[@%(name)s="Common"]/component/group[@%(name)s="Security"]/group[@%(name)s="Scripting"]/prop[@%(name)s="HyperlinksWithCtrlClick"]' % nsDict):
+ for prop in root.findall(
+ '%(component-schema)s[@%(name)s="Common"]/component/group[@%(name)s="Security"]/group[@%(' +
+ 'name)s="Scripting"]/prop[@%(name)s="HyperlinksWithCtrlClick"]' % nsDict):
for value in prop.findall("value"):
value.text = "false"
# Disable Impress View -> Slide Pane
- for prop in root.findall('%(component-data)s[@%(name)s="Impress"]/node[@%(name)s="MultiPaneGUI"]/node[@%(name)s="SlideSorterBar"]/node[@%(name)s="Visible"]/prop[@%(name)s="ImpressView"]' % nsDict):
+ for prop in root.findall(
+ '%(component-data)s[@%(name)s="Impress"]/node[@%(name)s="MultiPaneGUI"]/node[@%(' +
+ 'name)s="SlideSorterBar"]/node[@%(name)s="Visible"]/prop[@%(name)s="ImpressView"]' % nsDict):
for value in prop.findall("value"):
value.text = "false"