From: Jean Privat <jean@pryen.org>
Date: Mon, 6 Apr 2015 15:08:52 +0000 (+0700)
Subject: lib/file: String::files does not exist on error
X-Git-Tag: v0.7.4~32^2~6
X-Git-Url: http://nitlanguage.org

lib/file: String::files does not exist on error

Signed-off-by: Jean Privat <jean@pryen.org>
---

diff --git a/lib/standard/file.nit b/lib/standard/file.nit
index 956d72f..545f59b 100644
--- a/lib/standard/file.nit
+++ b/lib/standard/file.nit
@@ -964,7 +964,17 @@ redef class String
 		end
 	end
 
-	# returns files contained within the directory represented by self
+	# Returns entries contained within the directory represented by self.
+	#
+	#     var files = "/etc".files
+	#     assert files.has("issue")
+	#
+	# Returns an empty array in case of error
+	#
+	#     files = "/etc/issue".files
+	#     assert files.is_empty
+	#
+	# TODO find a better way to handle errors and to give them back to the user.
 	fun files: Array[String] is extern import Array[String], Array[String].add, NativeString.to_s, String.to_cstring `{
 		char *dir_path;
 		DIR *dir;
@@ -972,8 +982,11 @@ redef class String
 		dir_path = String_to_cstring( recv );
 		if ((dir = opendir(dir_path)) == NULL)
 		{
-			perror( dir_path );
-			exit( 1 );
+			//perror( dir_path );
+			//exit( 1 );
+			Array_of_String results;
+			results = new_Array_of_String();
+			return results;
 		}
 		else
 		{