Browse Source

Remove reduntant globals

No need for declaring these variables as globals in these functions.
Values are only read in there, not modified.
pull/4/head
George Vlahavas 9 years ago
parent
commit
de13b106f2
  1. 12
      src/xdgmenumaker

12
src/xdgmenumaker

@ -145,7 +145,6 @@ class MenuEntry:
self.path)) self.path))
def icon_full_path(icon): def icon_full_path(icon):
global iconsize
# If the icon path is absolute and exists, leave it alone. # If the icon path is absolute and exists, leave it alone.
# This takes care of software that has its own icons stored # This takes care of software that has its own icons stored
# in non-standard directories # in non-standard directories
@ -166,8 +165,6 @@ def icon_full_path(icon):
return icon return icon
def get_entry_info(desktopfile): def get_entry_info(desktopfile):
global desktop
global seticon
show = True show = True
de = dentry.DesktopEntry(filename = desktopfile) de = dentry.DesktopEntry(filename = desktopfile)
name = de.getName().encode('utf-8') name = de.getName().encode('utf-8')
@ -321,8 +318,6 @@ def category_icon(category):
return icon return icon
def fluxboxmenu(): def fluxboxmenu():
global seticon
global submenu
if submenu: if submenu:
spacing = ' ' spacing = ' '
if seticon: if seticon:
@ -376,8 +371,6 @@ def windowmakermenu():
print '"'+applications+'" END' print '"'+applications+'" END'
def icewmmenu(): def icewmmenu():
global seticon
global submenu
if submenu: if submenu:
spacing = ' ' spacing = ' '
if seticon: if seticon:
@ -410,9 +403,6 @@ def icewmmenu():
print '}' print '}'
def pekwmmenu(): def pekwmmenu():
global seticon
global submenu
global pekwmdynamic
if pekwmdynamic: if pekwmdynamic:
print "Dynamic {" print "Dynamic {"
dspacing = ' ' dspacing = ' '
@ -460,8 +450,6 @@ def pekwmmenu():
print "}" print "}"
def jwmmenu(): def jwmmenu():
global seticon
global submenu
print '<?xml version="1.0"?>' print '<?xml version="1.0"?>'
print '<JWM>' print '<JWM>'
if submenu: if submenu:

Loading…
Cancel
Save