[vlc-devel] [vlc-commits] posix: rely on F_DUPFD_CLOEXEC

Rémi Denis-Courmont remi at remlab.net
Mon Sep 23 18:17:10 CEST 2019


The technical committee, or the general assembly, have yet to decide on any kind of review mandate  Don't decide things in your own. Pretty rich from somebody that accused me of deciding alone stuff that had actually been discussed.

Le 23 septembre 2019 13:36:30 GMT+03:00, Thomas Guillem <thomas at gllm.fr> a écrit :
>
>
>On Mon, Sep 23, 2019, at 11:28, Rémi Denis-Courmont wrote:
>> I am sad to see that you are still attacking me after other people
>have done the same thing, trivial changes to the core, and then claim
>the contrary.
>
>I told the same thing to everyone, privately or publicly. I will
>continue to ask that same thing until we move to gitlab and are all
>forced to ho through MR.
>
>These changes don't look that trivial to me. 
>
>> 
>> I don't have advice on code review from somebody that knowingly
>merged a broken UI rework in spite of objections, and still is
>inhibiting the development process 6 months later
>> 
>> Le 23 septembre 2019 11:23:43 GMT+03:00, Thomas Guillem
><thomas at gllm.fr> a écrit :
>>> Hello,
>>> 
>>> I'm sad to see that this didn't went through the mailing list. I
>think that every one now is doing this extra effort.
>>> 
>>> Nevertheless, I'm OK with this set.
>>> 
>>> On Sat, Sep 21, 2019, at 11:49, Rémi Denis-Courmont wrote:
>>>> vlc | branch: master | Rémi Denis-Courmont <remi at remlab.net> | Sat
>Sep 
>>>> 21 11:57:19 2019 +0300| [95041019b09a5ecd8c5343721a136b349590be92]
>| 
>>>> committer: Rémi Denis-Courmont
>>>> 
>>>> posix: rely on F_DUPFD_CLOEXEC
>>>> 
>>>> This was added in POSIX.2008 and thus missing on MacOS < 10.7.
>>>> But those old versions are no longer supported since VLC 3.0.
>>>> 
>>>>>
>http://git.videolan.org/gitweb.cgi/vlc.git/?a=commit;h=95041019b09a5ecd8c5343721a136b349590be92
>>>>   src/posix/filesystem.c | 7 -------
>>>>   1 file changed, 7 deletions(-)
>>>>  
>>>>  diff --git a/src/posix/filesystem.c b/src/posix/filesystem.c
>>>>  index 2ac9c1187e..0ba1a37957 100644
>>>>  --- a/src/posix/filesystem.c
>>>>  +++ b/src/posix/filesystem.c
>>>>  @@ -185,14 +185,7 @@ char *vlc_getcwd (void)
>>>>   
>>>>   int vlc_dup (int oldfd)
>>>>   {
>>>>  -#ifdef F_DUPFD_CLOEXEC
>>>>       return fcntl (oldfd, F_DUPFD_CLOEXEC, 0);
>>>>  -#else
>>>>  -    int newfd = dup (oldfd);
>>>>  -    if (newfd != -1)
>>>>  -        vlc_cloexec(oldfd);
>>>>  -    return newfd;
>>>>  -#endif
>>>>   }
>>>>   
>>>>   int vlc_pipe (int fds[2]) vlc-commits mailing list
>>>>  vlc-commits at videolan.org
>>>>  https://mailman.videolan.org/listinfo/vlc-commits
>>>> 
>>> vlc-devel mailing list
>>> To unsubscribe or modify your subscription options:
>>> https://mailman.videolan.org/listinfo/vlc-devel
>> 
>> -- 
>> Envoyé de mon appareil Android avec Courriel K-9 Mail. Veuillez
>excuser ma brièveté. 
>> _______________________________________________
>> vlc-devel mailing list
>> To unsubscribe or modify your subscription options:
>> https://mailman.videolan.org/listinfo/vlc-devel

-- 
Envoyé de mon appareil Android avec Courriel K-9 Mail. Veuillez excuser ma brièveté.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20190923/b3100716/attachment-0001.html>


More information about the vlc-devel mailing list