On Tue, Jun 22, 2021 at 01:29:59PM +0000, Klemens Nanni wrote:
> On Tue, Jun 22, 2021 at 06:35:44AM +0100, Jason McIntyre wrote:
> > > -sets the files from which the public certificate, and private key will 
> > > be read.
> > > +loads two files from which the public certificate, and private key will 
> > > be read.
> > 
> > this is a weird place for a comma. i would remove it.
> > jmc
> 
> Agreed, but all the other sentences below have it as well so I left it
> to keep my diff simple.  I'm sure this manual can be polished further
> afterwards.
> 

that is because they are lists of more than two items. putting a comma
in a list with two items is wrong (well, that's a simplification, but
it's wrong in your diff).

jmc

> > >  .Pp
> > >  .Fn tls_config_set_keypair_mem
> > >  directly sets the public certificate, and private key from memory.
> > >  .Pp
> > >  .Fn tls_config_set_keypair_ocsp_file
> > > -sets the files from which the public certificate, private key, and 
> > > DER-encoded
> > > -OCSP staple will be read.
> > > +loads three files containing the public certificate, private key, and 
> > > DER-encoded
> > > +OCSP staple.
> > >  .Pp
> > >  .Fn tls_config_set_keypair_ocsp_mem
> > >  directly sets the public certificate, private key, and DER-encoded OCSP 
> > > staple
> > > 
> > 
> 

Reply via email to