Skip to content

docs: improve double factorial documentation - #3246

Open
samaira-core wants to merge 2 commits into
TheAlgorithms:masterfrom
samaira-core:master
Open

docs: improve double factorial documentation#3246
samaira-core wants to merge 2 commits into
TheAlgorithms:masterfrom
samaira-core:master

Conversation

@samaira-core

@samaira-core samaira-core commented Oct 11, 2026 •
edited
Loading

Copy link
Copy Markdown

Changes

  • Added @PARAM and @return comments to the iterative function.
  • Updated the comments for the recursive function.
  • Fixed a spelling mistake in the word "recursive".

Testing

  • Compiled and ran the program.
  • All three tests passed.

realstealthninja
realstealthninja requested changes Oct 12, 2026
View reviewed changes
Comment thread math/double_factorial.cpp Outdated
* @param n The number whose double factorial is calculated.
* @return The calculated double factorial.
*
* Recursion can be costly for large numbers.

@realstealthninja realstealthninja Oct 12, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Should be a note

This branch has not been deployed

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

Reviewers

@realstealthninja realstealthninja realstealthninja requested changes
@Panquesito7 Panquesito7 Awaiting requested review from Panquesito7 Panquesito7 is a code owner

Requested changes must be addressed to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@samaira-core @realstealthninja