[openssl-dev] [openssl.org #4343] master: EC_KEY_priv2buf (): check parameter sanity
Kurt Roeckx
kurt at roeckx.be
Fri Feb 26 19:49:38 UTC 2016
On Fri, Feb 26, 2016 at 05:34:14PM +0000, Viktor Dukhovni wrote:
> On Fri, Feb 26, 2016 at 05:29:26PM +0000, Salz, Rich wrote:
>
> > As just about the only team member who trolls through RT and closes things
> > with any quantity, I am not sure that I agree that fixing a bug requires
> > documentation if the API isn't already documented.
>
> Focus on fixing bugs in documented interfaces first, and even then review
> the documentation, to make sure the fix comports with the documentation and
> that the latter is not stale.
>
> Once all the bugs in documented interfaces are fixed, I'm afraid
> we really must start documenting the code we're updating. Otherwise,
> there's no way to know we're doing it right, and we're digging the
> hole deeper. New code needs to be documented as it is written,
> old code also, but at a slower pace.
I have to agree with Viktor. I've also been requesting minimal
documentation for the new non-public functions. If I'm changing
something in any non-documented function I try documenting it,
but sometimes I might forget that, and I hope someone would remind
me in that case.
Kurt
More information about the openssl-dev
mailing list