[vlc-devel] [PATCH 2/2] stream_filter: remove redundant tests

Rémi Denis-Courmont remi at remlab.net
Fri Sep 25 08:41:09 CEST 2020


Le perjantaina 25. syyskuuta 2020, 9.33.53 EEST Steve Lhomme a écrit :
> On 2020-09-25 7:01, Rémi Denis-Courmont wrote:
> > Le perjantaina 25. syyskuuta 2020, 7.56.14 EEST Steve Lhomme a écrit :
> >> On 2020-09-24 19:07, Rémi Denis-Courmont wrote:
> >>> Those checks are redundant as of the previous patch.
> >>> ---
> >>> 
> >>>    modules/demux/playlist/asx.c      | 3 ---
> >>>    modules/demux/playlist/b4s.c      | 1 -
> >>>    modules/demux/playlist/bdmv.c     | 2 --
> >>>    modules/demux/playlist/dvb.c      | 1 -
> >>>    modules/demux/playlist/ifo.c      | 2 --
> >>>    modules/demux/playlist/itml.c     | 2 +-
> >>>    modules/demux/playlist/m3u.c      | 1 -
> >>>    modules/demux/playlist/playlist.h | 6 ------
> >>>    modules/demux/playlist/pls.c      | 2 --
> >>>    modules/demux/playlist/podcast.c  | 1 -
> >>>    modules/demux/playlist/qtl.c      | 1 -
> >>>    modules/demux/playlist/ram.c      | 1 -
> >>>    modules/demux/playlist/sgimb.c    | 1 -
> >>>    modules/demux/playlist/wms.c      | 2 --
> >>>    modules/demux/playlist/wpl.c      | 1 -
> >>>    modules/demux/playlist/xspf.c     | 2 --
> >>>    modules/lua/stream_filter.c       | 6 ++----
> >>>    modules/stream_filter/record.c    | 3 ---
> >>>    modules/stream_filter/ytdl.c      | 2 +-
> >> 
> >> The patch doesn't apply because of this extra file.
> > 
> > No, the patchset depends on the previous patchset.
> 
> I didn't see that mentioned. So it cannot be merged until youtube-dl is
> merged ?

I'm not interested in reviewers playing stupid.

-- 
レミ・デニ-クールモン
http://www.remlab.net/





More information about the vlc-devel mailing list