diff mbox series

[FFmpeg-devel] doc/fate: advise on --assert-level=2

Message ID 20240312120920.300369-1-nicolas.gaullier@cji.paris
State Accepted
Commit b96571df3a96c65c9cd258a6d066df30d9d57c3e
Headers show
Series [FFmpeg-devel] doc/fate: advise on --assert-level=2 | expand

Checks

Context Check Description
yinshiyou/make_loongarch64 success Make finished
yinshiyou/make_fate_loongarch64 success Make fate finished
andriy/make_x86 success Make finished
andriy/make_fate_x86 success Make fate finished

Commit Message

Nicolas Gaullier March 12, 2024, 12:09 p.m. UTC
Signed-off-by: Nicolas Gaullier <nicolas.gaullier@cji.paris>
---
 doc/fate.texi | 8 ++++++++
 1 file changed, 8 insertions(+)

Comments

Nicolas Gaullier March 18, 2024, 3:05 p.m. UTC | #1
>De : Nicolas Gaullier <nicolas.gaullier@cji.paris> 
>Envoyé : mardi 12 mars 2024 13:09
>Objet : [PATCH] doc/fate: advise on --assert-level=2
>
>diff --git a/doc/fate.texi b/doc/fate.texi index 2fa8c34c2d..17644ce65a 100644
>--- a/doc/fate.texi
>+++ b/doc/fate.texi
>@@ -79,6 +79,14 @@ Do not put a '~' character in the samples path to indicate a home  directory. Because of shell nuances, this will cause FATE to fail.
> @end float
> 
>+Beware that some assertions are disabled by default, so mind setting 
>+@option{--assert-level=<level>} at configuration time, e.g. when 
>+seeking the highest possible test coverage:
>+@example
>+./configure --assert-level=2
>+@end example
>+Note that raising the assert level could have a performance impact.

Ping ? (I forgot to raise the version to v2, but it actually includes a notice on performance impact following Michael's comment)
Feel free to amend/reauthor if my english is not good enough.

Nicolas
Michael Niedermayer March 19, 2024, 2:36 a.m. UTC | #2
On Tue, Mar 12, 2024 at 01:09:20PM +0100, Nicolas Gaullier wrote:
> Signed-off-by: Nicolas Gaullier <nicolas.gaullier@cji.paris>
> ---
>  doc/fate.texi | 8 ++++++++
>  1 file changed, 8 insertions(+)

will apply

thx

[...]
diff mbox series

Patch

diff --git a/doc/fate.texi b/doc/fate.texi
index 2fa8c34c2d..17644ce65a 100644
--- a/doc/fate.texi
+++ b/doc/fate.texi
@@ -79,6 +79,14 @@  Do not put a '~' character in the samples path to indicate a home
 directory. Because of shell nuances, this will cause FATE to fail.
 @end float
 
+Beware that some assertions are disabled by default, so mind setting
+@option{--assert-level=<level>} at configuration time, e.g. when seeking
+the highest possible test coverage:
+@example
+./configure --assert-level=2
+@end example
+Note that raising the assert level could have a performance impact.
+
 To get the complete list of tests, run the command:
 @example
 make fate-list