Re: [FFmpeg-devel] Revert "avformat/tls_openssl: properly get new BIO index"

Re: [FFmpeg-devel] Revert "avformat/tls_openssl: properly get new BIO index"

[Nicolas George]

Kacper Michajłow (HE12025-07-30):
> > Note that BIO_get_new_index() can only be used 127 times before it
> > returns an error.
> 
> We cannot call it repeatedly, because it will fail eventually.
> 
> To my understanding the index is not needed in our use and we could
> safely use BIO_TYPE_NONE. Documentation states:
> 
> > type can be set to either BIO_TYPE_NONE or via BIO_get_new_index() if
> > a unique type is required for searching (See BIO_find_type(3))
> 
> We don't use any search related functions.

This should never have reached the commit message.

-- 
  Nicolas George
_______________________________________________
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".

Re: [FFmpeg-devel] Revert "avformat/tls_openssl: properly get new BIO index"

[Kacper Michajlow]

On Wed, 30 Jul 2025 at 19:53, Nicolas George <george@nsup.org> wrote:
>
> Kacper Michajłow (HE12025-07-30):
> > > Note that BIO_get_new_index() can only be used 127 times before it
> > > returns an error.
> >
> > We cannot call it repeatedly, because it will fail eventually.
> >
> > To my understanding the index is not needed in our use and we could
> > safely use BIO_TYPE_NONE. Documentation states:
> >
> > > type can be set to either BIO_TYPE_NONE or via BIO_get_new_index() if
> > > a unique type is required for searching (See BIO_find_type(3))
> >
> > We don't use any search related functions.
>
> This should never have reached the commit message.

Sorry, if the commit description is not formal enough. I'm not native
English speaker, I tend to word things in uncommon ways. I gave
commentary there to excerpts from the OpenSSL documentation.

As for BIO_TYPE_NONE vs BIO_TYPE_SOURCE_SINK, there is no difference
in current or previous versions of OpenSSL, BIO_TYPE_SOURCE_SINK is
not actually used in any meaningful way. I didn't include the 2nd
commit to change to BIO_TYPE_NONE. As the main objective was to fix
regression only and restore the previous status quo is good for now.

- Kacper
_______________________________________________
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".

Re: [FFmpeg-devel] Revert "avformat/tls_openssl: properly get new BIO index"

[Niklas Haas]

On Wed, 30 Jul 2025 19:52:59 +0200 Nicolas George <george@nsup.org> wrote:
> Kacper Michajłow (HE12025-07-30):
> > > Note that BIO_get_new_index() can only be used 127 times before it
> > > returns an error.
> >
> > We cannot call it repeatedly, because it will fail eventually.
> >
> > To my understanding the index is not needed in our use and we could
> > safely use BIO_TYPE_NONE. Documentation states:
> >
> > > type can be set to either BIO_TYPE_NONE or via BIO_get_new_index() if
> > > a unique type is required for searching (See BIO_find_type(3))
> >
> > We don't use any search related functions.
>
> This should never have reached the commit message.

I disagree. Without this important context, the motivation behind the
change is much less clear.

>
> --
>   Nicolas George
> _______________________________________________
> 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".
_______________________________________________
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".

Re: [FFmpeg-devel] Revert "avformat/tls_openssl: properly get new BIO index"

[Nicolas George]

Niklas Haas (HE12025-07-30):
> I disagree. Without this important context, the motivation behind the
> change is much less clear.

It is not context, it is a dump of an exchange of mails that led to the
patch. It should have been rewritten, “To my understanding” is not
something that should appear in a commit message.

And it is definitely not a matter of language, it is matter of
attention.

Regards,

-- 
  Nicolas George
_______________________________________________
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".

Re: [FFmpeg-devel] Revert "avformat/tls_openssl: properly get new BIO index"

[Kacper Michajlow]

On Wed, 30 Jul 2025 at 21:53, Nicolas George <george@nsup.org> wrote:
>
> Niklas Haas (HE12025-07-30):
> > I disagree. Without this important context, the motivation behind the
> > change is much less clear.
>
> It is not context, it is a dump of an exchange of mails that led to the
> patch. It should have been rewritten, “To my understanding” is not
> something that should appear in a commit message.
>
> And it is definitely not a matter of language, it is matter of
> attention.
>
> Regards,

Can we please find something more relevant to be upset about?

I've written this commit description, and I disagree that it's as bad
as you're making it out to be.

It's not a dump of any email exchange. It's commentary to OpenSSL documentation.

That said, I’ve taken your feedback seriously and will continue to
improve going forward.

- Kacper
_______________________________________________
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".

Re: [FFmpeg-devel] Revert "avformat/tls_openssl: properly get new BIO index"

[Nicolas George]

Kacper Michajlow (HE12025-07-30):
> Can we please find something more relevant to be upset about?

Can you please stop behaving with such arrogance when somebody who has
30 times as much experience as you in the project points you you did
something wrong?

-- 
  Nicolas George
_______________________________________________
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".

Re: [FFmpeg-devel] Revert "avformat/tls_openssl: properly get new BIO index"

[Niklas Haas]

On Wed, 30 Jul 2025 23:39:18 +0200 Nicolas George <george@nsup.org> wrote:
> Kacper Michajlow (HE12025-07-30):
> > Can we please find something more relevant to be upset about?
>
> Can you please stop behaving with such arrogance when somebody who has
> 30 times as much experience as you in the project points you you did
> something wrong?

What you pointed out is pointess nitpicking, and I could point you to
plenty of other comments by "experienced" FFmpeg developers who have done
the same in the past, presumably without having sat through diatribes.

>
> --
>   Nicolas George
> _______________________________________________
> 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".
_______________________________________________
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".

Re: [FFmpeg-devel] Revert "avformat/tls_openssl: properly get new BIO index"

[Nicolas George]

Niklas Haas (HE12025-07-30):
> What you pointed out is pointess nitpicking, and I could point you to
> plenty of other comments by "experienced" FFmpeg developers who have done
> the same in the past, presumably without having sat through diatribes.

Where they argued that they were right instead of admitting “my bad”? I
doubt it, but you can try.

-- 
  Nicolas George
_______________________________________________
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".

Re: [FFmpeg-devel] Revert "avformat/tls_openssl: properly get new BIO index"

[Michael Niedermayer]

[-- Attachment #1.1: Type: text/plain, Size: 808 bytes --]

Hi

On Wed, Jul 30, 2025 at 09:53:26PM +0200, Nicolas George wrote:
> Niklas Haas (HE12025-07-30):
> > I disagree. Without this important context, the motivation behind the
> > change is much less clear.
> 
> It is not context, it is a dump of an exchange of mails that led to the

It seems the commit message contains quotes from openssl documentation
not emails

I did misread this commit the same way thinking its mails

thx

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Awnsering whenever a program halts or runs forever is
On a turing machine, in general impossible (turings halting problem).
On any real computer, always possible as a real computer has a finite number
of states N, and will either halt in less than N cycles or never halt.

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 195 bytes --]

[-- Attachment #2: Type: text/plain, Size: 251 bytes --]

_______________________________________________
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".