Skip to content

enhance doxygen comments for CQE handler#68

Merged
wokron merged 1 commit into
masterfrom
cqe-handler-comment
Mar 12, 2026
Merged

enhance doxygen comments for CQE handler#68
wokron merged 1 commit into
masterfrom
cqe-handler-comment

Conversation

@wokron
Copy link
Copy Markdown
Member

@wokron wokron commented Mar 11, 2026

No description provided.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR improves the generated documentation for CQE handlers by expanding Doxygen comments around handler return values and result structs.

Changes:

  • Adds return-value descriptions to several CQE handler Doxygen blocks.
  • Replaces inline field comments in NVMeResult and TxTimestampResult with per-member Doxygen docs.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

Comment thread include/condy/cqe_handler.hpp
Comment thread include/condy/cqe_handler.hpp
Comment thread include/condy/cqe_handler.hpp
Comment thread include/condy/cqe_handler.hpp
@wokron wokron merged commit aeaf4cf into master Mar 12, 2026
15 checks passed
@wokron wokron deleted the cqe-handler-comment branch March 12, 2026 05:48
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.

2 participants