On Sat, Feb 10, 2018 at 12:50 AM, Robert Abel <[email protected]> wrote: > Using '\0' instead of plain 0 makes the intent clearer that this is indeed a > string and not a series of integers. > > Signed-off-by: Robert Abel <[email protected]> > --- > drivers/auxdisplay/charlcd.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/auxdisplay/charlcd.c b/drivers/auxdisplay/charlcd.c > index 324d02f9f1c5..92549c8344a4 100644 > --- a/drivers/auxdisplay/charlcd.c > +++ b/drivers/auxdisplay/charlcd.c > @@ -527,7 +527,7 @@ static void charlcd_write_char(struct charlcd *lcd, char > c) > if ((c != '\n') && priv->esc_seq.len >= 0) { > /* yes, let's add this char to the buffer */ > priv->esc_seq.buf[priv->esc_seq.len++] = c; > - priv->esc_seq.buf[priv->esc_seq.len] = 0; > + priv->esc_seq.buf[priv->esc_seq.len] = '\0'; > } else { > /* aborts any previous escape sequence */ > priv->esc_seq.len = -1; > @@ -536,7 +536,7 @@ static void charlcd_write_char(struct charlcd *lcd, char > c) > case LCD_ESCAPE_CHAR: > /* start of an escape sequence */ > priv->esc_seq.len = 0; > - priv->esc_seq.buf[priv->esc_seq.len] = 0; > + priv->esc_seq.buf[priv->esc_seq.len] = '\0';
Trivial. Compile-tested. Signed-off-by: Miguel Ojeda <[email protected]> > break; > case '\b': > /* go back one char and clear it */ > -- > 2.11.0 >
