ToDoList.pas comments and code inconsistencies
Original Reporter info from Mantis: KevinOfOz @kevinofoz
-
Reporter name: Kevin Jesshope
Original Reporter info from Mantis: KevinOfOz @kevinofoz
- Reporter name: Kevin Jesshope
Description:
in Components/todolist/ToDoList.pas
The main comments at the top of the unit:
1 - do not reference the Priority option
2 - refer to a semicolon when it is actually a colon in the code.
3 - indicate that the {TODO Text WIthout Colon} is valid - it is not (see comment at line ToDoList.pas:569 // Alternative keyword requires a semicolon to prevent false positives.)
4 - indicate that the {Done Text WIthout Colon} is valid - it is not (see comment at line ToDoList.pas:569 // Alternative keyword requires a semicolon to prevent false positives.)
The code has a constant cSemicolon at line 487 when it is actually a colon in TIDETodoWindow.CreateToDoItem
It also has a boolean var HasSemiColon that should be HasColon
Mantis conversion info:
- Mantis ID: 36721
- OS: Win64
- OS Build: Win 10 1909
- Build: FPC 3.3.1 svn
- Platform: Lazarus IDE v2.1.0
- Version: 2.0.7 (SVN)
- Fixed in revision: r62657 (#bfe50dae)