Changeset 2288705 for set.c

Timestamp:

2009-12-07T21:54:19Z (14 years ago)

Author:

Wilmer van der Gaast <wilmer@…>

Branches:

master

Children:

1c3008a

Parents:

aac4017 (diff), 36cf9fd (diff)
Note: this is a merge changeset, the changes displayed below correspond to the merge itself.
Use the (diff) links above to see all the changes relative to each parent.

Message:

Merging head.

File:

• set.c (modified) (7 diffs)

set.c

@@ -25 +25 @@
 #define BITLBEE_CORE
 #include "bitlbee.h"
+
+/* Used to use NULL for this, but NULL is actually a "valid" value. */
+char *SET_INVALID = "nee";
 
 set_t *set_add( set_t **head, char *key, char *def, set_eval eval, void *data )
@@ -114 +117 @@
         
         if( !s )
+                /*
+                Used to do this, but it never really made sense.
                 s = set_add( head, key, NULL, NULL, NULL );
-        
-        if( s->eval && !( nv = s->eval( s, value ) ) )
+                */
+                return 0;
+        
+        if( value == NULL && ( s->flags & SET_NULL_OK ) == 0 )
+                return 0;
+        
+        /* Call the evaluator. For invalid values, evaluators should now
+           return SET_INVALID, but previously this was NULL. Try to handle
+           that too if NULL is not an allowed value for this setting. */
+        if( s->eval && ( ( nv = s->eval( s, value ) ) == SET_INVALID ||
+                         ( ( s->flags & SET_NULL_OK ) == 0 && nv == NULL ) ) )
                 return 0;
         
@@ -168 +182 @@
 }
 
-void set_reset( set_t **head, char *key )
+int set_reset( set_t **head, char *key )
 {
         set_t *s;
@@ -174 +188 @@
         s = set_find( head, key );
         if( s )
-                set_setstr( head, key, s->def );
+                return set_setstr( head, key, s->def );
+        
+        return 0;
 }
 
@@ -187 +203 @@
         for( ; *s; s ++ )
                 if( !isdigit( *s ) )
-                        return NULL;
+                        return SET_INVALID;
         
         return value;
@@ -194 +210 @@
 char *set_eval_bool( set_t *set, char *value )
 {
-        return is_bool( value ) ? value : NULL;
+        return is_bool( value ) ? value : SET_INVALID;
 }
 
@@ -226 +242 @@
                                                               irc->channel, "-oo", irc->nick, irc->mynick );
         else
-                return NULL;
+                return SET_INVALID;
         
         return value;