Skip to content

Commit

Permalink
Merge pull request #2 from epics-motor/190920_send_mess_uses_const_char
Browse files Browse the repository at this point in the history
The 3rd parameter in send_mess(), "name" can and should
be a 'const char *' instead of just 'char *'.
Modern compilers complain here, so that the signature now
gets the const.
  • Loading branch information
kmpeters authored Sep 25, 2019
2 parents 1ec6839 + 80d425f commit 9b5dc61
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions newFocusApp/src/drvPMNC87xx.cc
Original file line number Diff line number Diff line change
Expand Up @@ -158,8 +158,8 @@ static socketStruct socketStructs[MAX_SOCKETS];

/*----------------functions-----------------*/
STATIC int recv_mess(int, char *, int);
STATIC RTN_STATUS send_mess(int, char const *, char *);
STATIC int send_recv_mess(int, char const *, char *, char const *);
STATIC RTN_STATUS send_mess(int, const char *, const char *);
STATIC int send_recv_mess(int, const char *, char *, const char *);
STATIC int set_status(int, int);
// STATIC void start_status(int);
static long report(int);
Expand Down Expand Up @@ -550,7 +550,7 @@ STATIC int set_status(int card, int signal)
/* with the return of the prompt '>' character */
/* send_recv_mess() */
/*****************************************************/
STATIC int send_recv_mess(int card, char const *send_com, char *recv_com, char const *eos)
STATIC int send_recv_mess(int card, const char *send_com, char *recv_com, const char *eos)
{
struct PMNCcontroller *cntrl;
int size;
Expand Down Expand Up @@ -610,7 +610,7 @@ STATIC int send_recv_mess(int card, char const *send_com, char *recv_com, char c
/* send a message to the PMNC87xx board */
/* send_mess() */
/*****************************************************/
STATIC RTN_STATUS send_mess(int card, char const *com, char *name)
STATIC RTN_STATUS send_mess(int card, const char *com, const char *name)
{
struct PMNCcontroller *cntrl;
int size;
Expand Down

0 comments on commit 9b5dc61

Please sign in to comment.