From 5506642b505b4a12bb025ca9dd94f8acd4c0fa8b Mon Sep 17 00:00:00 2001 From: Dane Springmeyer Date: Wed, 12 Dec 2012 16:05:35 -0800 Subject: [PATCH] fix a few argument checking odd syntax --- src/mapnik_fonts.hpp | 2 +- src/mapnik_image.cpp | 2 +- src/mapnik_image_view.cpp | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/src/mapnik_fonts.hpp b/src/mapnik_fonts.hpp index 89e1641904..bf82ee5e0b 100644 --- a/src/mapnik_fonts.hpp +++ b/src/mapnik_fonts.hpp @@ -28,7 +28,7 @@ static inline Handle register_fonts(const Arguments& args) try { - if (!args.Length() >= 1 || !args[0]->IsString()) + if (args.Length() == 0 || !args[0]->IsString()) return ThrowException(Exception::TypeError( String::New("first argument must be a path to a directory of fonts"))); diff --git a/src/mapnik_image.cpp b/src/mapnik_image.cpp index a56a7e424b..8c58bdaa8b 100644 --- a/src/mapnik_image.cpp +++ b/src/mapnik_image.cpp @@ -571,7 +571,7 @@ Handle Image::save(const Arguments& args) { HandleScope scope; - if (!args.Length() >= 1 || !args[0]->IsString()){ + if (args.Length() == 0 || !args[0]->IsString()){ return ThrowException(Exception::TypeError( String::New("filename required"))); } diff --git a/src/mapnik_image_view.cpp b/src/mapnik_image_view.cpp index 68f5a54fa4..be7194c703 100644 --- a/src/mapnik_image_view.cpp +++ b/src/mapnik_image_view.cpp @@ -466,7 +466,7 @@ Handle ImageView::save(const Arguments& args) { HandleScope scope; - if (!args.Length() >= 1 || !args[0]->IsString()){ + if (args.Length() == 0 || !args[0]->IsString()){ return ThrowException(Exception::TypeError( String::New("filename required"))); }