add comment about misuse of enum set function.
authorCampbell Barton <ideasman42@gmail.com>
Sat, 20 Oct 2012 14:16:56 +0000 (14:16 +0000)
committerCampbell Barton <ideasman42@gmail.com>
Sat, 20 Oct 2012 14:16:56 +0000 (14:16 +0000)
source/blender/makesrna/intern/rna_wm.c

index 2462733915b63afa9c0300d752cce43ed83169fe..28b034fc164399f26ad3ea3d3ec00f942eace2dd 100644 (file)
@@ -635,11 +635,18 @@ static void rna_wmKeyMapItem_keymodifier_set(PointerRNA *ptr, int value)
 {
        wmKeyMapItem *kmi = ptr->data;
        
-       if (value == ESCKEY);
-       else if (value >= AKEY)
+       /* XXX, this should really be managed in an _itemf function,
+        * giving a list of valid enums, then silently changing them when they are set is not
+        * a good precedent, don't do this unless you have a good reason! */
+       if (value == ESCKEY) {
+               /* pass */
+       }
+       else if (value >= AKEY) {
                kmi->keymodifier = value;
-       else
+       }
+       else {
                kmi->keymodifier = 0;
+       }
 }