From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org [79.124.17.100]) by master.gitmailbox.com (Postfix) with ESMTP id D039246895 for ; Thu, 22 Jun 2023 22:00:26 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id C38CE68C21E; Fri, 23 Jun 2023 01:00:24 +0300 (EEST) Received: from relay3-d.mail.gandi.net (relay3-d.mail.gandi.net [217.70.183.195]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 02B1168C026 for ; Fri, 23 Jun 2023 01:00:17 +0300 (EEST) X-GND-Sasl: michael@niedermayer.cc Received: by mail.gandi.net (Postfix) with ESMTPSA id 4C3EA60003 for ; Thu, 22 Jun 2023 22:00:17 +0000 (UTC) Date: Fri, 23 Jun 2023 00:00:16 +0200 From: Michael Niedermayer To: FFmpeg development discussions and patches Message-ID: <20230622220016.GG3250409@pb2> References: <20230618222805.4054410-1-michael@niedermayer.cc> <20230622134328.GA3250409@pb2> <6f1c5b3a-99cb-c2e2-41fd-da0a1273b9fe@gmail.com> <20230622150527.GC3250409@pb2> <8e1965cc-9479-5fd1-0b0b-dbbe6a26cf36@gmail.com> <20230622162625.GD3250409@pb2> <020f7a32-60c4-22c4-19e9-9c25fef3b108@gmail.com> MIME-Version: 1.0 In-Reply-To: <020f7a32-60c4-22c4-19e9-9c25fef3b108@gmail.com> X-Spam-Flag: yes X-Spam-Level: ************ X-GND-Spam-Score: 180 X-GND-Status: SPAM Subject: Re: [FFmpeg-devel] [PATCH v2] avformat: add Software Defined Radio support X-BeenThere: ffmpeg-devel@ffmpeg.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: FFmpeg development discussions and patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: FFmpeg development discussions and patches Content-Type: multipart/mixed; boundary="===============8838393985864138660==" Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" Archived-At: List-Archive: List-Post: --===============8838393985864138660== Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="yQbNiKLmgenwUfTN" Content-Disposition: inline --yQbNiKLmgenwUfTN Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Thu, Jun 22, 2023 at 01:42:39PM -0300, James Almer wrote: > On 6/22/2023 1:26 PM, Michael Niedermayer wrote: > > On Thu, Jun 22, 2023 at 12:10:06PM -0300, James Almer wrote: > > > On 6/22/2023 12:05 PM, Michael Niedermayer wrote: > > > > On Thu, Jun 22, 2023 at 10:55:44AM -0300, James Almer wrote: > > > > > On 6/22/2023 10:43 AM, Michael Niedermayer wrote: > > > > > > On Mon, Jun 19, 2023 at 12:28:05AM +0200, Michael Niedermayer w= rote: > > > > > > > Signed-off-by: Michael Niedermayer > > > > > > > --- > > > > > > > configure | 4 + > > > > > > > doc/demuxers.texi | 71 ++ > > > > > > > libavformat/Makefile | 2 + > > > > > > > libavformat/allformats.c | 2 + > > > > > > > libavformat/sdrdemux.c | 1739 +++++++++++++++++++++++++= +++++++++++++ > > > > > > > 5 files changed, 1818 insertions(+) > > > > > > > create mode 100644 libavformat/sdrdemux.c > > > > > >=20 > > > > > > Ill post a v3 later today or tomorrow that makes this work with= the RTL-SDR Blog V3 > > > > >=20 > > > > > Shouldn't the SDR "demuxer" be in libavdevice? Being AVFMT_NOFILE= and pretty > > > > > much a capture device, it seems to me that's the proper place. > > > > > I guess the problem arises with the sdrfile demuxer, which shares= code with > > > > > the other one. > > > >=20 > > > > I have no oppinon on this. I can move it to libavdevice if people p= refer. > > > > do people prefer libavdevice for this ? > > >=20 > > > I do. It's a capture device and depends on an external library to int= erface > > > with hardware. And like i said, you can keep the file demuxer in lavf= , where > > > it does fit. > >=20 > > anyone objects if i move sdrfile then too into libavdevice ? because it= s the > > same code basically. Otherwise it would have to be some sort of #includ= e of c > > file accross libs >=20 > It does not belong in lavd. The doxy in avdevice.h states "the (de)muxers= in > libavdevice are of the AVFMT_NOFILE type". And you can make the common co= de > avpriv_ since both libraries are basically version locked. There's no need > to duplicate it, but if you prefer that, you can indeed do an #include c > file using the SHLIBOBJS list in Makefile which will trigger on shared > builds. it seemed to me wiser to not put effort in seperating the sdrfile and sdr cases when really device and format will be merged at some future point. Its work now and then work to undo it. but ill look into it thx [...] --=20 Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB If a bugfix only changes things apparently unrelated to the bug with no further explanation, that is a good sign that the bugfix is wrong. --yQbNiKLmgenwUfTN Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iF0EABEIAB0WIQSf8hKLFH72cwut8TNhHseHBAsPqwUCZJTEawAKCRBhHseHBAsP q89DAJ9/ki47DQPPWhdE/HvdYK+gX7viJgCfXSGRz0VUBCassBOJDdMDsveh61w= =GrPN -----END PGP SIGNATURE----- --yQbNiKLmgenwUfTN-- --===============8838393985864138660== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe". --===============8838393985864138660==--