Permalink
Browse files

Fix a typo and silly crash case

Would crash instead of gracefully exiting if a tileset dir wasn't provided. Might also change how the memory limit option works - currently has to be provided, even though that's not mentioned.
  • Loading branch information...
1 parent 924d396 commit 52651c034ceca3a6e3273ef7cc40d1cf9fe29b0b @eanmclaughlin eanmclaughlin committed Aug 31, 2014
Showing with 3 additions and 2 deletions.
  1. +3 −2 contrib/png-it.py
View
@@ -34,14 +34,15 @@ def main():
metavar = '<output>', type = str, dest = 'output', default = "output.png")
(options, args) = parser.parse_args()
- tileset = args[0]
+
# arg is overviewer tile set folder
if len(args) > 1:
parser.error("Error! Only one overviewer tile set accepted as input. Use --help for a complete list of options.")
if not args:
parser.error("Error! Need an overviewer tile set folder. Use --help for a complete list of options.")
+ tileset = args[0] #set the tileset dir after ensuring it's been provided, not before.
if not options.zoom_level:
parser.error("Error! The option zoom-level is mandatory.")
@@ -126,7 +127,7 @@ def main():
print "The image size will be {0}x{1}".format(final_cropped_img_size[0],final_cropped_img_size[1])
print "A total of {0} MB of memory will be used.".format(mem/1024**2)
if mem/1024.**2. > options.memory_limit:
- print "Warning! The expected RAM usage exceeds the spicifyed limit. Exiting."
+ print "Warning! The expected RAM usage exceeds the specified limit. Exiting."
sys.exit(1)
# Create a new huge image

1 comment on commit 52651c0

@kirajjxd

b

Please sign in to comment.