Bug 334326

Summary:	API documentation for DisplayTwo::turn(int cardNumber) has a flaw
Product:	[Applications] lskat	Reporter:	Martin Walch <walch.martin>
Component:	general	Assignee:	martin
Status:	RESOLVED FIXED
Severity:	minor	CC:	kde-games-bugs-null
Priority:	NOR
Version First Reported In:	unspecified
Target Milestone:	---
Platform:	unspecified
OS:	Unspecified
Latest Commit:	commits.kde.org/lskat/349959b88dcbaf7f5cf0d6629ff8ca3dfe441cd0	Version Fixed/Implemented In:	4.14.0
Sentry Crash Report:
Attachments:	fix API doc, typos in code comments and clean up whitespace same patch as above, but with additional typo fixed

Description Martin Walch 2014-05-04 12:56:39 UTC

src/display_two.h says for DisplayTwo::turn(int cardNumber):

* @param cardNumber The card number (0-7)

This should probably read

* @param cardNumber The card number (0-31)

Also, there are some typos like "Recive" and "esxisting".

Reproducible: Always

Comment 1 Martin Walch 2014-05-04 12:58:46 UTC

Created attachment 86451 [details]
fix API doc, typos in code comments and clean up whitespace

Fix the documentation flaw, some typos and clean up whitespace when touching the files anyway.

Comment 2 Martin Walch 2014-05-04 13:25:14 UTC

Created attachment 86452 [details]
same patch as above, but with additional typo fixed

numner -> number

Comment 3 Martin Walch 2014-07-23 23:19:44 UTC

I just added the patch to the review board:

https://git.reviewboard.kde.org/r/119431/

Comment 4 Martin Walch 2014-07-31 10:53:38 UTC

Submitted by Albert Astals Cid (Thank you)

http://commits.kde.org/lskat/349959b88dcbaf7f5cf0d6629ff8ca3dfe441cd0