Skip to content

Problem 12 project Euler solution#380

Closed
abdelhak-ihadjadene wants to merge 39 commits intoTheAlgorithms:masterfrom
abdelhak-ihadjadene:problem12_project_euler
Closed

Problem 12 project Euler solution#380
abdelhak-ihadjadene wants to merge 39 commits intoTheAlgorithms:masterfrom
abdelhak-ihadjadene:problem12_project_euler

Conversation

@abdelhak-ihadjadene
Copy link

No description provided.

SanyuktaSaha and others added 30 commits October 1, 2019 10:40
Added RC4 stream cipher algorithm
7.c :reverse of a number with overflow check added
The correct format specifier for `size_t` is `%zu`. If C89 support is needed, `%lu` along with a cast to `unsigned long` will suffice.

Also added error checking for calloc.
Cleaned up the code and used meaningful variable names
Fixed a critical flaw where `strcpy` would write the final NUL-terminator in an invalid location invoking Undefined Behavior. Allocated one extra byte to fix it.

Cleaned up the code as well in the process
Improved code Asthetics
Cleaned up, meaningful variable names
+1 byte for the NUL-terminator
Format specifier for size_t, calloc check
@github-actions
Copy link
Contributor

github-actions bot commented Jan 1, 2022

This pull request has been automatically marked as abandoned because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@github-actions github-actions bot added the Stale label Jan 1, 2022
@github-actions
Copy link
Contributor

github-actions bot commented Jan 9, 2022

Please ping one of the maintainers once you commit the changes requested or make improvements on the code. If this is not the case and you need some help, feel free to ask for help in our Gitter channel or our Discord server. Thank you for your contributions!

@github-actions github-actions bot closed this Jan 9, 2022
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.

9 participants