diff options
author | Pedro Alves <palves@redhat.com> | 2013-04-19 15:36:52 +0000 |
---|---|---|
committer | Pedro Alves <palves@redhat.com> | 2013-04-19 15:36:52 +0000 |
commit | 295123e116bd75f28ca1a6c310471668960f2fc0 (patch) | |
tree | 255c4529bb48d4fef59742c56fb363bc1c14374f /gdb/ctf.c | |
parent | bf63885a12b4ff83174a400335f6258536e0ff2a (diff) | |
download | gdb-295123e116bd75f28ca1a6c310471668960f2fc0.tar.gz |
-Wpointer-sign: ctf.c.
ctf_save_write's second parameter is gdb_byte *, and all these
arguments are 'char *'. Since this function is ultimately just
writing host bytes to a local file with fwrite, an alternative would
be to change ctf_save_write to take a 'void *' instead of 'gdb_byte
*', thus removing the need for any cast (we have more calls with casts
than without).
gdb/
2013-04-19 Pedro Alves <palves@redhat.com>
* ctf.c (ctf_write_uploaded_tsv, ctf_write_uploaded_tp): Add casts
to 'gdb_byte *'.
Diffstat (limited to 'gdb/ctf.c')
-rw-r--r-- | gdb/ctf.c | 15 |
1 files changed, 8 insertions, 7 deletions
diff --git a/gdb/ctf.c b/gdb/ctf.c index b8252e40fc5..240e909aa11 100644 --- a/gdb/ctf.c +++ b/gdb/ctf.c @@ -529,7 +529,8 @@ ctf_write_uploaded_tsv (struct trace_file_writer *self, /* name */ if (tsv->name != NULL) - ctf_save_write (&writer->tcs, tsv->name, strlen (tsv->name)); + ctf_save_write (&writer->tcs, (gdb_byte *) tsv->name, + strlen (tsv->name)); ctf_save_write (&writer->tcs, &zero, 1); } @@ -581,30 +582,30 @@ ctf_write_uploaded_tp (struct trace_file_writer *self, /* condition */ if (tp->cond != NULL) - ctf_save_write (&writer->tcs, tp->cond, strlen (tp->cond)); + ctf_save_write (&writer->tcs, (gdb_byte *) tp->cond, strlen (tp->cond)); ctf_save_write (&writer->tcs, &zero, 1); /* actions */ u32 = VEC_length (char_ptr, tp->actions); ctf_save_align_write (&writer->tcs, (gdb_byte *) &u32, 4, 4); for (a = 0; VEC_iterate (char_ptr, tp->actions, a, act); ++a) - ctf_save_write (&writer->tcs, act, strlen (act) + 1); + ctf_save_write (&writer->tcs, (gdb_byte *) act, strlen (act) + 1); /* step_actions */ u32 = VEC_length (char_ptr, tp->step_actions); ctf_save_align_write (&writer->tcs, (gdb_byte *) &u32, 4, 4); for (a = 0; VEC_iterate (char_ptr, tp->step_actions, a, act); ++a) - ctf_save_write (&writer->tcs, act, strlen (act) + 1); + ctf_save_write (&writer->tcs, (gdb_byte *) act, strlen (act) + 1); /* at_string */ if (tp->at_string != NULL) - ctf_save_write (&writer->tcs, tp->at_string, + ctf_save_write (&writer->tcs, (gdb_byte *) tp->at_string, strlen (tp->at_string)); ctf_save_write (&writer->tcs, &zero, 1); /* cond_string */ if (tp->cond_string != NULL) - ctf_save_write (&writer->tcs, tp->cond_string, + ctf_save_write (&writer->tcs, (gdb_byte *) tp->cond_string, strlen (tp->cond_string)); ctf_save_write (&writer->tcs, &zero, 1); @@ -612,7 +613,7 @@ ctf_write_uploaded_tp (struct trace_file_writer *self, u32 = VEC_length (char_ptr, tp->cmd_strings); ctf_save_align_write (&writer->tcs, (gdb_byte *) &u32, 4, 4); for (a = 0; VEC_iterate (char_ptr, tp->cmd_strings, a, act); ++a) - ctf_save_write (&writer->tcs, act, strlen (act) + 1); + ctf_save_write (&writer->tcs, (gdb_byte *) act, strlen (act) + 1); } |