[PR #1133] [MERGED] Update tic_tac_toe.c #1701

Open
opened 2026-01-29 15:23:42 +00:00 by claunia · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/TheAlgorithms/C/pull/1133
Author: @ManogjnaSinguluri
Created: 11/8/2022
Status: Merged
Merged: 11/14/2022
Merged by: @Panquesito7

Base: masterHead: patch-3


📝 Commits (1)

📊 Changes

1 file changed (+1 additions, -1 deletions)

View changed files

📝 games/tic_tac_toe.c (+1 -1)

📄 Description

owing to issue #960,this change though small will remove one of the bug

Description of Change

e<=8 is not needed as remainder is always less than remainder so e<9(e>=8) is obvious in every case

References

Checklist

  • Added description of change
  • Added file name matches File name guidelines
  • Added tests and example, test must pass
  • Relevant documentation/comments is changed or added
  • PR title follows semantic commit guidelines
  • Search previous suggestions before making a new one, as yours may be a duplicate.
  • I acknowledge that all my contributions will be made under the project's license.

Notes:


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/TheAlgorithms/C/pull/1133 **Author:** [@ManogjnaSinguluri](https://github.com/ManogjnaSinguluri) **Created:** 11/8/2022 **Status:** ✅ Merged **Merged:** 11/14/2022 **Merged by:** [@Panquesito7](https://github.com/Panquesito7) **Base:** `master` ← **Head:** `patch-3` --- ### 📝 Commits (1) - [`9d5e1cc`](https://github.com/TheAlgorithms/C/commit/9d5e1cc1844493615b6793563bcb28f246d98bff) Update tic_tac_toe.c ### 📊 Changes **1 file changed** (+1 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `games/tic_tac_toe.c` (+1 -1) </details> ### 📄 Description owing to issue #960,this change though small will remove one of the bug #### Description of Change e<=8 is not needed as remainder is always less than remainder so e<9(e>=8) is obvious in every case <!-- Thank you for your Pull Request. Please provide a description above and review the requirements below. Contributors guide: https://github.com/TheAlgorithms/C/blob/master/CONTRIBUTING.md --> #### References <!-- Add any reference to previous pull-request or issue --> #### Checklist <!-- Remove items that do not apply. For completed items, change [ ] to [x]. --> - [x] Added description of change - [x] Added file name matches [File name guidelines](https://github.com/TheAlgorithms/C/blob/master/CONTRIBUTING.md#File-Name-guidelines) - [x] Added tests and example, test must pass - [x] Relevant documentation/comments is changed or added - [x] PR title follows semantic [commit guidelines](https://github.com/TheAlgorithms/C/blob/master/CONTRIBUTING.md#Commit-Guidelines) - [x] Search previous suggestions before making a new one, as yours may be a duplicate. - [x] I acknowledge that all my contributions will be made under the project's license. Notes: <!-- Please add a one-line description for developers or pull request viewers --> --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
claunia added the pull-request label 2026-01-29 15:23:43 +00:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/C#1701