bug-gnu-emacs
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

bug#48228: json-serialize should signal error when dll is not found [MS


From: Dmitry Gutov
Subject: bug#48228: json-serialize should signal error when dll is not found [MS Windows]
Date: Tue, 4 May 2021 18:25:05 +0300
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.7.1

Tags: patch

As discussed on Emacs Help.

Another commenter suggested signaling a specific error. Please advise what to name it and where to put it.

diff --git a/src/json.c b/src/json.c
index 3f1d27ad7f..ece057ae41 100644
--- a/src/json.c
+++ b/src/json.c
@@ -596,8 +596,7 @@ DEFUN ("json-serialize", Fjson_serialize, Sjson_serialize, 1, MANY,
     }
   if (!json_initialized)
     {
-      message1 ("jansson library not found");
-      return Qnil;
+ Fsignal (Qerror, list1 (build_unibyte_string ("jansson library not found")));
     }
 #endif

@@ -707,8 +706,7 @@ DEFUN ("json-insert", Fjson_insert, Sjson_insert, 1, MANY,
     }
   if (!json_initialized)
     {
-      message1 ("jansson library not found");
-      return Qnil;
+ Fsignal (Qerror, list1 (build_unibyte_string ("jansson library not found")));
     }
 #endif

@@ -966,8 +964,7 @@ DEFUN ("json-parse-string", Fjson_parse_string, Sjson_parse_string, 1, MANY,
     }
   if (!json_initialized)
     {
-      message1 ("jansson library not found");
-      return Qnil;
+ Fsignal (Qerror, list1 (build_unibyte_string ("jansson library not found")));
     }
 #endif

@@ -1065,8 +1062,7 @@ DEFUN ("json-parse-buffer", Fjson_parse_buffer, Sjson_parse_buffer,
     }
   if (!json_initialized)
     {
-      message1 ("jansson library not found");
-      return Qnil;
+ Fsignal (Qerror, list1 (build_unibyte_string ("jansson library not found")));
     }
 #endif





reply via email to

[Prev in Thread] Current Thread [Next in Thread]