Hello,

I've tried to see some TODOs in my code, but it seems the TODO list windows isn't working, however I don't know if it is happening only with my Lazarus copy.

Could anybody confirm if it is working properly?

Thank you!

Environment: Lazarus 1.7 r53323 FPC 3.1.1 x86_64-linux-gtk 2

Oops,

On Mon, Nov 21, 2016 at 3:14 PM, silvioprog wrote: [...]

... it seems the TODO list windows isn't working, however I don't know ...

I meant: "... it seems the TODO list window isn't working, it always return an empty list, however I don't know ..."

It works here.

http://wiki.lazarus.freepascal.org/IDE_Window:_ToDo_List

Mattias

Mattias Gaertner wrote

http://wiki.lazarus.freepascal.org/IDE_Window:_ToDo_List

Most of my ToDo's are in //ToDo: format, which I see is not handled.

I must admit I never tried that package.

Bart

On Tue, Nov 22, 2016 at 6:34 PM, Bart via Lazarus <

I can see three valid TODOs (please see attached picture), but only when I add the unit to the lpi. The problem happen when I have the unit declared only in the lpr uses clause.

So, for Delphi compatibility it should look TODOs in the opened units at the source editor tabs too.

(Lazarus is flexible, so it would be nice to have an extra option: search TODOs at project's folder too :-) )

Oops,

On Tue, Nov 22, 2016 at 11:46 PM, silvioprog wrote: [...]

Can you do more two tests?:

I meant "could you do more two tests?". ^^'

+1

Perhaps search in all path that project uses.
I have many projects with no unit included. I just point paths.

—
Marcos Douglas

[...] So, for Delphi compatibility it should look TODOs in the opened units at the source editor tabs too.

Feel free to add an option to search in source editor units too.

Mattias

I updated the wiki page. It supports //, {} and (**).

Mattias

On Wed, 23 Nov 2016 16:15:39 -0200
"Marcos Douglas B. Santos via Lazarus"

[...] Perhaps search in all path that project uses. I have many projects with no unit included. I just point paths.

It now searches in all used project units, excluding used packages. Formerly it only searched in the units listed in the project inspector.

Mattias

Mattias Gaertner wrote

It now searches in all used project units, excluding used packages. Formerly it only searched in the units listed in the project inspector

Hmm, I didn't know... Thanks for that.

On Wed, Nov 23, 2016 at 9:46 PM, Mattias Gaertner via Lazarus <

wrote:

[...]

Formerly it only searched in the units listed in the project inspector. Mattias

I sent a patch (issue #31006 <http://bugs.freepascal.org/view.php?id=31006>) extending IDEIntf and allowing the ToDo window to find units opened as tabs at source editor. It's easy to test:

1. open Lazarus keeping its default new project opened;
2. choose some unit (declaring some ToDo) outside project and drop that to the project, now it is just a tab at source editor;
3. open the ToDo window and finally you can see that unit's ToDo (or any opened).

It works in same way on Delphi, and it would be nice if Lazarus could provide this same functionality. :-)

What do you mean with "drop that to the project"?

Mattias

On Fri, Nov 25, 2016 at 9:57 AM, Mattias Gaertner via Lazarus <

wrote:

[...]

What do you mean with "drop that to the project"?

Oops, sorry, I meant when a user drag some unit (even units from other projects) from his system file manager and drop to the source editor.

So I reopened the issue sending a new patch applying a big upgrade in the ToDo list window.

(ps. I don't know why but I'm not receiving all e-mails from the Lazarus list, I was lucky I've opened the archives link and saw your message there :-/ - http://lists.lazarus-ide.org/pipermail/lazarus/2016-November /thread.html)