OSSP CVS Repository

ossp - Check-in [3250]
Not logged in
[Honeypot]  [Browse]  [Home]  [Login]  [Reports
[Search]  [Ticket]  [Timeline
  [Patchset]  [Tagging/Branching

Check-in Number: 3250
Date: 2003-Feb-13 16:37:28 (local)
2003-Feb-13 15:37:28 (UTC)
User:rse
Branch:
Comment: Remove NULL checks for va_list based variables because it is not portable to assume that va_list behaves like a pointer or other scalar type. Indeed it is a full structure on some platforms like FreeBSD/alpha.
Tickets:
Inspections:
Files:
ossp-pkg/l2/l2_channel.c      1.31 -> 1.32     1 inserted, 1 deleted
ossp-pkg/l2/l2_ut_param.c      1.10 -> 1.11     1 inserted, 1 deleted

ossp-pkg/l2/l2_channel.c 1.31 -> 1.32

--- l2_channel.c 2003/01/27 16:01:35     1.31
+++ l2_channel.c 2003/02/13 15:37:28     1.32
@@ -560,7 +560,7 @@
     l2_env_t *env;
 
     /* argument sanity check */
-    if (ch == NULL || level == 0 || fmt == NULL || ap == NULL)
+    if (ch == NULL || level == 0 || fmt == NULL)
         return L2_ERR_ARG;
 
     /* make sure only a single level is specified */


ossp-pkg/l2/l2_ut_param.c 1.10 -> 1.11

--- l2_ut_param.c        2003/01/06 11:41:52     1.10
+++ l2_ut_param.c        2003/02/13 15:37:28     1.11
@@ -46,7 +46,7 @@
     int n;
 
     /* argument sanity check */
-    if (env == NULL || pa == NULL || fmt == NULL || ap == NULL)
+    if (env == NULL || pa == NULL || fmt == NULL)
         return L2_ERR_ARG;
 
     /* on-the-fly create or just take over parameter specification string */

CVSTrac 2.0.1