Index: TODO =================================================================== diff -u -r6c7bb9ba109a65cf8b80fbe947268522dc528ebe -raccdf4beee7067c76291e80b85d8336c69075510 --- TODO (.../TODO) (revision 6c7bb9ba109a65cf8b80fbe947268522dc528ebe) +++ TODO (.../TODO) (revision accdf4beee7067c76291e80b85d8336c69075510) @@ -5556,6 +5556,8 @@ - write body-blocks of if on separate lines - change variable name "validCscPtr" to "isValidCsc", since it is a boolean and not a ptr (tcl coding guidelines) +- prefer explicit comparisons +- remove test, since it is covered already by assertions ======================================================================== TODO: Index: generic/nsf.c =================================================================== diff -u -r6c7bb9ba109a65cf8b80fbe947268522dc528ebe -raccdf4beee7067c76291e80b85d8336c69075510 --- generic/nsf.c (.../nsf.c) (revision 6c7bb9ba109a65cf8b80fbe947268522dc528ebe) +++ generic/nsf.c (.../nsf.c) (revision accdf4beee7067c76291e80b85d8336c69075510) @@ -30554,7 +30554,8 @@ NsfClass *cl = entry->clorobj; assert(cl != NULL); - if (cl == NULL || !NsfObjectIsClass(&cl->object)) { + + if (!NsfObjectIsClass(&cl->object)) { continue; }