[v4,05/13] sim/erc32: Fix a few compiler warnings
Commit Message
* func.c: Minor edits to remove compiler warnings.
* func.c, sis.h (exec_cmd): make second argument const char.
* func.c, sis.h (bfd_load): make argument const char.
---
sim/erc32/func.c | 16 ++++++++--------
sim/erc32/sis.h | 4 ++--
2 files changed, 10 insertions(+), 10 deletions(-)
Comments
On 17 Mar 2015 22:02, Jiri Gaisler wrote:
> * func.c: Minor edits to remove compiler warnings.
> * func.c, sis.h (exec_cmd): make second argument const char.
> * func.c, sis.h (bfd_load): make argument const char.
pushed, but with the ChangeLog:
2015-03-28 Jiri Gaisler <jiri@gaisler.se>
* func.c (exec_cmd): Fix old style prototype. Mark cmd const. Save
the command in a new cmdsave2 to avoid modifying.
(bfd_load): Fix old style prototype. Mark fname const.
* sis.h (exec_cmd): Mark cmd const.
(bfd_load): Mark fname const.
all changes to a single file have to be grouped
-mike
@@ -378,21 +378,20 @@ limcalc (freq)
}
return (lim);
}
-
+
int
-exec_cmd(sregs, cmd)
- char *cmd;
- struct pstate *sregs;
+exec_cmd(struct pstate *sregs, const char *cmd)
{
char *cmd1, *cmd2;
int32 stat;
uint32 len, i, clen, j;
static uint32 daddr = 0;
- char *cmdsave;
+ char *cmdsave, *cmdsave2 = NULL;
stat = OK;
cmdsave = strdup(cmd);
- if ((cmd1 = strtok(cmd, " \t")) != NULL) {
+ cmdsave2 = strdup (cmd);
+ if ((cmd1 = strtok (cmdsave2, " \t")) != NULL) {
clen = strlen(cmd1);
if (strncmp(cmd1, "bp", clen) == 0) {
for (i = 0; i < sregs->bptnum; i++) {
@@ -609,6 +608,8 @@ exec_cmd(sregs, cmd)
} else
printf("syntax error\n");
}
+ if (cmdsave2 != NULL)
+ free(cmdsave2);
if (cmdsave != NULL)
free(cmdsave);
return (stat);
@@ -1048,8 +1049,7 @@ sys_halt()
#define LOAD_ADDRESS 0
int
-bfd_load(fname)
- char *fname;
+bfd_load (const char *fname)
{
asection *section;
bfd *pbfd;
@@ -184,7 +184,7 @@ extern struct pstate sregs;
extern void set_regi (struct pstate *sregs, int32 reg,
uint32 rval);
extern void get_regi (struct pstate *sregs, int32 reg, char *buf);
-extern int exec_cmd (struct pstate *sregs, char *cmd);
+extern int exec_cmd (struct pstate *sregs, const char *cmd);
extern void reset_stat (struct pstate *sregs);
extern void show_stat (struct pstate *sregs);
extern void init_bpt (struct pstate *sregs);
@@ -202,7 +202,7 @@ extern int check_bpt (struct pstate *sregs);
extern void reset_all (void);
extern void sys_reset (void);
extern void sys_halt (void);
-extern int bfd_load (char *fname);
+extern int bfd_load (const char *fname);
extern double get_time (void);
/* exec.c */