<p dir="ltr"><br>
2015/03/12 0:40 "Ryo Munakata" <<a href="mailto:ryomnktml@gmail.com">ryomnktml@gmail.com</a>>:<br>
><br>
> On Wed, 11 Mar 2015 16:05:55 +0200<br>
> Pekka Paalanen <<a href="mailto:ppaalanen@gmail.com">ppaalanen@gmail.com</a>> wrote:<br>
><br>
> > On Tue, 10 Mar 2015 11:34:45 +0900<br>
> > Ryo Munakata <<a href="mailto:ryomnktml@gmail.com">ryomnktml@gmail.com</a>> wrote:<br>
> > > +<br>
> > >  struct fbdev_compositor {<br>
> > >     struct weston_compositor base;<br>
> > >     uint32_t prev_state;<br>
> > > @@ -690,6 +692,9 @@ fbdev_output_destroy(struct weston_output *base)<br>
> > >             gl_renderer->output_destroy(base);<br>
> > >     }<br>
> > ><br>
> > > +   if (output->device)<br>
> > > +           free((char *)output->device);<br>
> ><br>
> > No need to cast. Also no need for 'if' because free(NULL) is perfectly<br>
> > legal.<br>
><br>
> This cast causes a warning like:<br>
>  src/compositor-fbdev.c:696:7: warning: passing argument 1 of ‘free’ discards ‘const’ qualifier from pointer target type<br>
> I think we need it for silencing the warning.<br>
> What do you think?</p>
<p dir="ltr">Typo.<br>
Correctly, free(output->device), which is without the cast, causes the waring above.</p>
<p dir="ltr">Thanks.</p>