[vlc-devel] [PATCH]Change read to fstat

Kaarlo Räihä kaarlo.raiha at gmail.com
Sun Jan 29 19:25:13 CET 2012


2012/1/29 Rémi Denis-Courmont <remi at remlab.net>

> Le dimanche 29 janvier 2012 18:13:00 Kaarlo Räihä, vous avez écrit :
> > I noticed modules/access/file.c has two parts which have msg_Err
> (p_access,
> > "failed to read (%m)"); (lines 187 and 297). In yearly years this wasn't
> > issue since it also printed the function name (access_file error:
> > fstat(3):) but now it is just filesystem error:
>
> fstat is not a verb. I think this patch makes the error harder rather than
> easier to understand.
>

I was under impression that read in this case refers to the function name
that is called earlier (line 286) not the verb. But I can resend as fstat
-> open if you prefer it.


>
> --
> Rémi Denis-Courmont
> http://www.remlab.net/
> http://fi.linkedin.com/in/remidenis
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> http://mailman.videolan.org/listinfo/vlc-devel
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20120129/d99a656f/attachment.html>


More information about the vlc-devel mailing list