https://bugs.dragonflybsd.org/https://bugs.dragonflybsd.org/favicon.ico?16293952082010-06-24T10:55:51ZDragonFlyBSD bugtrackerDragonFlyBSD - Bug #1787: Hangman: case-sensitive problem?https://bugs.dragonflybsd.org/issues/1787?journal_id=86862010-06-24T10:55:51Zdylan
<ul></ul><p>The code makes all your answers lower case.</p>
<p>This patch will also compare the uppercase version of your letter.</p>
<p>diff --git a/games/hangman/getguess.c b/games/hangman/getguess.c<br />index b5b60c0..8a40c7c 100644<br />--- a/games/hangman/getguess.c<br />+<ins>+ b/games/hangman/getguess.c<br /><code>@ -71,8 +71,8 </code>@ getguess(void)<br /> Guessed[ch - 'a'] = TRUE;<br /> correct = FALSE;<br /> for (i = 0; Word[i] != '\0'; i</ins>+) {<br />- if (Word[i] == ch) {<br />- Known[i] = ch;<br />+ if (Word[i] ch || Word[i] toupper(ch) ) {<br />+ Known[i] = Word[i];<br /> correct = TRUE;<br /> }<br /> }</p>
<p>Dylan</p>
<p>On 06/23/2010 02:33 PM, Tim Darby wrote:</p>
<blockquote>
<p>While playing hangman, the word offered was 'Theocritan', but when I <br />guessed 't', it only filled in the second 't'. I haven't looked at <br />the code, but I'm guessing that it's not recognizing the uppercase T.</p>
<p>Tim</p>
</blockquote> DragonFlyBSD - Bug #1787: Hangman: case-sensitive problem?https://bugs.dragonflybsd.org/issues/1787?journal_id=86872010-06-24T16:00:49Zswildner
<ul></ul><p>On 6/24/2010 5:53, Dylan Reinhold wrote:</p>
<blockquote>
<p>The code makes all your answers lower case.</p>
<p>This patch will also compare the uppercase version of your letter.</p>
</blockquote>
<p>I've fixed it. However I took OpenBSD's code (oops, should have noted <br />that in the commit message) which calls toupper() only once.</p>
<p>Regards,<br />Sascha</p> DragonFlyBSD - Bug #1787: Hangman: case-sensitive problem?https://bugs.dragonflybsd.org/issues/1787?journal_id=86882010-06-24T16:04:47Zswildner
<ul></ul><p>Committed -> 09871f9d92dfbc060605c073f2615ebca6c12c94</p>
<p>Thanks, guys</p>