Skip to content

Remove unused doc argument from DEFINE_ macros.#857

Merged
dmah42 merged 1 commit intogoogle:masterfrom
aerosayan:addressing_#856
Aug 21, 2019
Merged

Remove unused doc argument from DEFINE_ macros.#857
dmah42 merged 1 commit intogoogle:masterfrom
aerosayan:addressing_#856

Conversation

@aerosayan
Copy link
Copy Markdown
Contributor

  - Adresses : google#856

  - The unused `doc` argument was removed from the `DEFINE_` macros in
    `commandlineflags.h`

  - Converted all the previous `doc` strings passed to the `DEFINE_`
    macros to multiline comments.
@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 91.753% when pulling bbe4d3d on aerosayan:addressing_#856 into ffadb65 on google:master.

@dmah42
Copy link
Copy Markdown
Member

dmah42 commented Aug 21, 2019

Thanks!

@dmah42 dmah42 merged commit 7ee7286 into google:master Aug 21, 2019
@aerosayan aerosayan deleted the addressing_#856 branch August 22, 2019 06:31
JBakamovic pushed a commit to JBakamovic/benchmark that referenced this pull request Sep 11, 2020
- Adresses : google#856

  - The unused `doc` argument was removed from the `DEFINE_` macros in
    `commandlineflags.h`

  - Converted all the previous `doc` strings passed to the `DEFINE_`
    macros to multiline comments.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants