[Lazarus] Verdict of Lazarus 0.9.29

[Mattias Gärtner]

Zitat von Graeme Geldenhuys <graemeg.lists at gmail.com>:

> I have also concluded that the editor settings "smart tabs" should be
> renamed to "idiot/stupid tabs" instead, because that is exactly what it does!
>
>
> | symbol denotes cursor pos, and "stupid tabs" enabled, and "auto indent"
> disabled and tab indentation set to 2, tabs set to "spaces".

Have you seen the link on the editor options to the codetools option?


> eg:
>   if Accept then
>   begin|           // <--- cursor pos
>     ADropAction := daCopy;
>     if AMimeChoice <> s then
>       AMimeChoice := s;
>     ShowMimeList(AMimeList);
>   end;
>
> .... I press ENTER...
>
>
>   if Accept then
>   begin
> |           // <--- cursor pos
>     ADropAction := daCopy;
>     if AMimeChoice <> s then
>       AMimeChoice := s;
>     ShowMimeList(AMimeList);
>   end;

Please report this bug.
Probably the tool spotted some unclosed structure in front or some  
misindented lines.
I'm currently busy working on another feature you et al requested.


> .... I then press tab once...
>
>   if Accept then
>   begin
>   |           // <--- cursor pos
>     ADropAction := daCopy;
>     if AMimeChoice <> s then
>       AMimeChoice := s;
>     ShowMimeList(AMimeList);
>   end;
>
> .... and then press tab once more...
>
>   if Accept then
>   begin
>       |           // <--- cursor pos (jumped 4 spaces now!)
>     ADropAction := daCopy;
>     if AMimeChoice <> s then
>       AMimeChoice := s;
>     ShowMimeList(AMimeList);
>   end;

If this is a bug depends on your editor settings. Probably Martin knows.


>
> WTF?!

+1


> And that's with "smart tabs" enabled! Hence the reason it should be
> renamed to "stupid tabs" instead. It better describes what it does.
>
>
> PS:
> There are many more such examples.

Please give them.

Mattias