Skip to content

Conversation

gregbo
Copy link

@gregbo gregbo commented Apr 4, 2021

  • Declared int main() to suppress compiler warning
  • Refactored main() so check for negative input needs to be made only once
  • Updated comment to indicate the program uses an iterative method

@gregbo
Copy link
Author

gregbo commented Apr 5, 2021

Recreated Sort Bubble.c with the following changes:

  • Declared int main() to suppress compiler warning
  • Added bounds checking on input array

No need to merge the deletes and adds between 12e16bb and dcdffb2

@gregbo
Copy link
Author

gregbo commented Apr 5, 2021

  • Declared int main() to suppress compiler warning
  • Added check for input to be a positive integer

@gregbo
Copy link
Author

gregbo commented Apr 7, 2021

  • Declared int main() to suppress compiler warning
  • Determined array size of initial call to sumEven() at runtime

@gregbo
Copy link
Author

gregbo commented Apr 8, 2021

Refactored to use an array of const char* instead of individual case statements for ease in maintainability

@gregbo
Copy link
Author

gregbo commented Apr 8, 2021

s/nums/words

@gregbo
Copy link
Author

gregbo commented Apr 18, 2021

  • Declared int main() to suppress compiler warning
  • Added code to handle input reaching EOF before a newline

@gregbo
Copy link
Author

gregbo commented Apr 29, 2021

  • Declared int main() to suppress compiler warning
  • Added code to handle zero and negative numbers
  • Refactored to use labs() to compute absolute values of long ints

@gregbo
Copy link
Author

gregbo commented May 5, 2021

Modified to exit with status 0 since quitting from the menu is normal termination

@gregbo
Copy link
Author

gregbo commented May 6, 2021

Modified to exit with status 0 since quitting from the menu is normal termination

add test data, refactor some printf() statements, and check for malloc() errors
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.

1 participant