Skip to content

Conversation

@levitte
Copy link
Member

@levitte levitte commented Sep 27, 2019

It's all in the details, from man-pages(7):

Formatting conventions for manual pages describing functions

    ...
    Variable names should, like argument names, be specified in italics.
    ...

Formatting conventions (general)

    ...
    Special macros, which are usually in uppercase, are in bold.
    Exception: don't boldface NULL.
    ...
Checklist
  • documentation is added or updated
  • tests are added or updated

It's all in the details, from man-pages(7):

    Formatting conventions for manual pages describing functions

        ...
        Variable names should, like argument names, be specified in italics.
        ...

    Formatting conventions (general)

        ...
        Special macros, which are usually in uppercase, are in bold.
        Exception: don't boldface NULL.
        ...
@levitte levitte added the branch: master Applies to master branch label Sep 27, 2019
Copy link
Contributor

@paulidale paulidale left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Consistency is good!
A couple of mostly harmless comments.

@richsalz
Copy link
Contributor

Oh man. This is good work, but it's sure going to be painful on all the doc-nits changes I have waiting review :(

@levitte
Copy link
Member Author

levitte commented Sep 27, 2019

How should I say this... I aim to please 😉

@richsalz
Copy link
Contributor

I took a look, it won't be too bad. Most of my pain will be self-inflicted, with #9952 and #9953 and even that's minor.

@levitte
Copy link
Member Author

levitte commented Sep 27, 2019

See it on the bright side: you'll get to learn real regexps 😉

We have a few pages where part of function names can be considered
variable.  There are no normative guidelines for such a case, but if
we draw from the formatting convention of variable and argument names,
we can draw the conclusion that this variable part should be italized,
within already given conventions.  In other words, we need to help the
POD processor along in cases like these:

    SPARSE_ARRAY_OF(TYPE)
    ossl_sa_TYPE_num()

These need explicit formatting:

    B<SPARSE_ARRAY_OF>(I<TYPE>)
    B<ossl_sa_I<TYPE>_num>()
@paulidale
Copy link
Contributor

See it on the bright side: you'll get to learn real regexps

Nah, learn something strictly more powerful. I'm a fan of PEGs but there are others.

Copy link
Contributor

@paulidale paulidale left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Safestack and lhash could also do with the same treatment I suspect.

Copy link
Contributor

@paulidale paulidale left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm happy with the changes here, my prior comment can be a new PR or an extra commit here.

@levitte
Copy link
Member Author

levitte commented Sep 28, 2019

Safestack and lhash could also do with the same treatment I suspect.

Yes. Do notice that it's not within the scope of this PR. I'll make another for doc/man3/, probably piece meal (i.e. a few other) as they are easier to digest.

levitte added a commit that referenced this pull request Sep 28, 2019
It's all in the details, from man-pages(7):

    Formatting conventions for manual pages describing functions

        ...
        Variable names should, like argument names, be specified in italics.
        ...

    Formatting conventions (general)

        ...
        Special macros, which are usually in uppercase, are in bold.
        Exception: don't boldface NULL.
        ...

Reviewed-by: Paul Dale <paul.dale@oracle.com>
(Merged from #10034)
levitte added a commit that referenced this pull request Sep 28, 2019
We have a few pages where part of function names can be considered
variable.  There are no normative guidelines for such a case, but if
we draw from the formatting convention of variable and argument names,
we can draw the conclusion that this variable part should be italized,
within already given conventions.  In other words, we need to help the
POD processor along in cases like these:

    SPARSE_ARRAY_OF(TYPE)
    ossl_sa_TYPE_num()

These need explicit formatting:

    B<SPARSE_ARRAY_OF>(I<TYPE>)
    B<ossl_sa_I<TYPE>_num>()

Reviewed-by: Paul Dale <paul.dale@oracle.com>
(Merged from #10034)
@levitte
Copy link
Member Author

levitte commented Sep 28, 2019

Merged.

dfabee8 Make doc/man7/ and doc/internal/man3/ conform with man-pages(7)
c18d2d9 Funtion name with variable part in doc/man7/ and doc/internal/man3/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

branch: master Applies to master branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants