fftools/resman: remove redundant includes, use inflate loop #90
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: softworkz <softworkz@hotmail.com>
Collaborator
Author
|
/submit |
|
Submitted as pull.90.ffstaging.FFmpeg.1748656229.ffmpegagent@gmail.com To fetch this version into To fetch this version to local tag |
Signed-off-by: softworkz <softworkz@hotmail.com>
48fc3a6 to
32cd61e
Compare
Collaborator
Author
|
/submit |
|
Submitted as pull.90.v2.ffstaging.FFmpeg.1748663925.ffmpegagent@gmail.com To fetch this version into To fetch this version to local tag |
| #include <zlib.h> | ||
| #endif | ||
|
|
||
| #include "resman.h" |
There was a problem hiding this comment.
On the FFmpeg mailing list, Michael Niedermayer wrote (reply to this):
--===============8464981479943917701==
Content-Type: multipart/signed; micalg=pgp-sha512;
protocol="application/pgp-signature"; boundary="lNK9RXEfBTwZsqA6"
Content-Disposition: inline
--lNK9RXEfBTwZsqA6
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable
Hi softworkz
On Sat, May 31, 2025 at 03:58:44AM +0000, softworkz wrote:
> From: softworkz <softworkz@hotmail.com>
>=20
> Signed-off-by: softworkz <softworkz@hotmail.com>
> ---
> fftools/resources/resman.c | 7 +++----
> fftools/resources/resman.h | 10 +---------
> 2 files changed, 4 insertions(+), 13 deletions(-)
seems to breaks build
fftools/resources/resman.c:60:8: error: unknown type name =E2=80=98AVMutex=
=E2=80=99
60 | static AVMutex mutex =3D AV_MUTEX_INITIALIZER;
| ^~~~~~~
fftools/resources/resman.c:60:24: error: =E2=80=98AV_MUTEX_INITIALIZER=E2=
=80=99 undeclared here (not in a function)
60 | static AVMutex mutex =3D AV_MUTEX_INITIALIZER;
| ^~~~~~~~~~~~~~~~~~~~
fftools/resources/resman.c: In function =E2=80=98get_resman_context=E2=80=
=99:
fftools/resources/resman.c:123:5: error: implicit declaration of function =
=E2=80=98ff_mutex_lock=E2=80=99 [-Werror=3Dimplicit-function-declaration]
123 | ff_mutex_lock(&mutex);
| ^~~~~~~~~~~~~
fftools/resources/resman.c:138:5: error: implicit declaration of function =
=E2=80=98ff_mutex_unlock=E2=80=99 [-Werror=3Dimplicit-function-declaration]
138 | ff_mutex_unlock(&mutex);
| ^~~~~~~~~~~~~~~
cc1: some warnings being treated as errors
make: *** [ffbuild/common.mak:81: fftools/resources/resman.o] Error 1
make: *** Waiting for unfinished jobs....
[...]
--=20
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
No great genius has ever existed without some touch of madness. -- Aristotle
--lNK9RXEfBTwZsqA6
Content-Type: application/pgp-signature; name="signature.asc"
-----BEGIN PGP SIGNATURE-----
iF0EABEKAB0WIQSf8hKLFH72cwut8TNhHseHBAsPqwUCaDz+MwAKCRBhHseHBAsP
q3gSAJ9YHbZSYTJ2o7ORJRS6yBl/S5m5HQCbBbAJqgspS0O0gquGV/fyJwLkjrE=
=V/c5
-----END PGP SIGNATURE-----
--lNK9RXEfBTwZsqA6--
--===============8464981479943917701==
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".
--===============8464981479943917701==--
There was a problem hiding this comment.
On the FFmpeg mailing list, "softworkz ." wrote (reply to this):
> -----Original Message-----
> From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On Behalf Of Michael
> Niedermayer
> Sent: Montag, 2. Juni 2025 03:28
> To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org>
> Subject: Re: [FFmpeg-devel] [PATCH v2 1/2] fftools/resman: remove unused
> includes, fix declaration
>
> Hi softworkz
>
> On Sat, May 31, 2025 at 03:58:44AM +0000, softworkz wrote:
> > From: softworkz <softworkz@hotmail.com>
> >
> > Signed-off-by: softworkz <softworkz@hotmail.com>
> > ---
> > fftools/resources/resman.c | 7 +++----
> > fftools/resources/resman.h | 10 +---------
> > 2 files changed, 4 insertions(+), 13 deletions(-)
>
> seems to breaks build
>
> fftools/resources/resman.c:60:8: error: unknown type name ‘AVMutex’
> 60 | static AVMutex mutex = AV_MUTEX_INITIALIZER;
> | ^~~~~~~
> fftools/resources/resman.c:60:24: error: ‘AV_MUTEX_INITIALIZER’ undeclared
> here (not in a function)
> 60 | static AVMutex mutex = AV_MUTEX_INITIALIZER;
> | ^~~~~~~~~~~~~~~~~~~~
> fftools/resources/resman.c: In function ‘get_resman_context’:
> fftools/resources/resman.c:123:5: error: implicit declaration of function
> ‘ff_mutex_lock’ [-Werror=implicit-function-declaration]
> 123 | ff_mutex_lock(&mutex);
> | ^~~~~~~~~~~~~
> fftools/resources/resman.c:138:5: error: implicit declaration of function
> ‘ff_mutex_unlock’ [-Werror=implicit-function-declaration]
> 138 | ff_mutex_unlock(&mutex);
> | ^~~~~~~~~~~~~~~
> cc1: some warnings being treated as errors
> make: *** [ffbuild/common.mak:81: fftools/resources/resman.o] Error 1
> make: *** Waiting for unfinished jobs....
>
>
> [...]
> --
Thanks!
Seems it got broken due to changes in master. On submission,
all builds were successful:
https://patchwork.ffmpeg.org/project/ffmpeg/patch/4d7af186ce9f8a8d3b12cdc2c979ea54320eed96.1748663925.git.ffmpegagent@gmail.com/
Will post an update.
Thanks
sw
_______________________________________________
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".
There was a problem hiding this comment.
On the FFmpeg mailing list, "softworkz ." wrote (reply to this):
> -----Original Message-----
> From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On Behalf Of softworkz .
> Sent: Montag, 2. Juni 2025 03:43
> To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org>
> Subject: Re: [FFmpeg-devel] [PATCH v2 1/2] fftools/resman: remove unused
> includes, fix declaration
>
>
>
> > -----Original Message-----
> > From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On Behalf Of Michael
> > Niedermayer
> > Sent: Montag, 2. Juni 2025 03:28
> > To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org>
> > Subject: Re: [FFmpeg-devel] [PATCH v2 1/2] fftools/resman: remove unused
> > includes, fix declaration
> >
> > Hi softworkz
> >
> > On Sat, May 31, 2025 at 03:58:44AM +0000, softworkz wrote:
> > > From: softworkz <softworkz@hotmail.com>
> > >
> > > Signed-off-by: softworkz <softworkz@hotmail.com>
> > > ---
> > > fftools/resources/resman.c | 7 +++----
> > > fftools/resources/resman.h | 10 +---------
> > > 2 files changed, 4 insertions(+), 13 deletions(-)
> >
> > seems to breaks build
> >
> > fftools/resources/resman.c:60:8: error: unknown type name ‘AVMutex’
> > 60 | static AVMutex mutex = AV_MUTEX_INITIALIZER;
> > | ^~~~~~~
> > fftools/resources/resman.c:60:24: error: ‘AV_MUTEX_INITIALIZER’ undeclared
> > here (not in a function)
> > 60 | static AVMutex mutex = AV_MUTEX_INITIALIZER;
> > | ^~~~~~~~~~~~~~~~~~~~
> > fftools/resources/resman.c: In function ‘get_resman_context’:
> > fftools/resources/resman.c:123:5: error: implicit declaration of function
> > ‘ff_mutex_lock’ [-Werror=implicit-function-declaration]
> > 123 | ff_mutex_lock(&mutex);
> > | ^~~~~~~~~~~~~
> > fftools/resources/resman.c:138:5: error: implicit declaration of function
> > ‘ff_mutex_unlock’ [-Werror=implicit-function-declaration]
> > 138 | ff_mutex_unlock(&mutex);
> > | ^~~~~~~~~~~~~~~
> > cc1: some warnings being treated as errors
> > make: *** [ffbuild/common.mak:81: fftools/resources/resman.o] Error 1
> > make: *** Waiting for unfinished jobs....
> >
> >
> > [...]
> > --
>
> Thanks!
>
> Seems it got broken due to changes in master. On submission,
> all builds were successful:
> https://patchwork.ffmpeg.org/project/ffmpeg/patch/4d7af186ce9f8a8d3b12cdc2c979
> ea54320eed96.1748663925.git.ffmpegagent@gmail.com/
>
> Will post an update.
> _______________________________________________
No I won't. This set is obsolete and superseded by
https://patchwork.ffmpeg.org/project/ffmpeg/list/?series=14663
sw
_______________________________________________
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".
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
and other cosmetic fixes.
As per review by Ramiro Polla in message:
"Re: [FFmpeg-devel] [FFmpeg-cvslog] fftools/resources: Add resource manager files with build-time compression"
V2
(in response to comment by Kieran)
.