|
From: <sv...@va...> - 2014-10-28 20:28:40
|
Author: florian
Date: Tue Oct 28 20:28:32 2014
New Revision: 14671
Log:
When traversing VG_(args_for_client) there is no need to check
the array element for NULL. Function VG_(split_up_argv), which
sets up that array, ensures that no element is NULL.
Modified:
trunk/cachegrind/cg_main.c
trunk/massif/ms_main.c
Modified: trunk/cachegrind/cg_main.c
==============================================================================
--- trunk/cachegrind/cg_main.c (original)
+++ trunk/cachegrind/cg_main.c Tue Oct 28 20:28:32 2014
@@ -1420,10 +1420,8 @@
VG_(strlen)( VG_(args_the_exename) ));
for (i = 0; i < VG_(sizeXA)( VG_(args_for_client) ); i++) {
HChar* arg = * (HChar**) VG_(indexXA)( VG_(args_for_client), i );
- if (arg) {
- VG_(write)(fd, " ", 1);
- VG_(write)(fd, arg, VG_(strlen)( arg ));
- }
+ VG_(write)(fd, " ", 1);
+ VG_(write)(fd, arg, VG_(strlen)( arg ));
}
// "events:" line
if (clo_cache_sim && clo_branch_sim) {
Modified: trunk/massif/ms_main.c
==============================================================================
--- trunk/massif/ms_main.c (original)
+++ trunk/massif/ms_main.c Tue Oct 28 20:28:32 2014
@@ -2334,8 +2334,7 @@
FP("%s", VG_(args_the_exename));
for (i = 0; i < VG_(sizeXA)( VG_(args_for_client) ); i++) {
HChar* arg = * (HChar**) VG_(indexXA)( VG_(args_for_client), i );
- if (arg)
- FP(" %s", arg);
+ FP(" %s", arg);
}
FP("\n");
|