Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@abetomo
Copy link
Member

@abetomo abetomo commented Oct 7, 2024

GitHub: GH-1817

Prepare to switch to documents automatically generated by Doxygen.

GitHub: groongaGH-1817

Prepare to switch to documents automatically generated by Doxygen.
* \brief Checks the passed database and recovers it if it is broken and it
* can be recovered.
*
* This API uses lock existence for checking whether the database is
Copy link
Member Author

Choose a reason for hiding this comment

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

\details は書かなくても良いっぽいので、省略してみたのだけれど、明記したほうが良いだろうか。

https://www.doxygen.nl/manual/commands.html#cmddetails

Copy link
Member

Choose a reason for hiding this comment

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

なくていいならなくていいんじゃない?

Comment on lines 64 to 65
* Table and data column can be recovered by removing an existence
* lock and re-add data.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
* Table and data column can be recovered by removing an existence
* lock and re-add data.
* Table and data column can be recovered manually by removing an existence
* lock and re-add data.

GRN_API void
grn_db_touch(grn_ctx *ctx, grn_obj *db);
/**
* \brief Checks the passed database and recovers it if it is broken and it
Copy link
Contributor

Choose a reason for hiding this comment

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

[nits] 三単現のsがないほうが多いので合わせてあげられると良さそうです。

Suggested change
* \brief Checks the passed database and recovers it if it is broken and it
* \brief Check the passed database and recover it if it is broken and it

@kou kou merged commit c70e7bb into groonga:main Oct 7, 2024
18 of 19 checks passed
@abetomo abetomo deleted the docs-move-to-header-grn_db_recover branch October 7, 2024 01:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants