Thank you for the fix, just tried it on fresh trunk, works!

Michael

Am 03.11.15 um 17:34 schrieb Mattias Gaertner:
On Tue, 3 Nov 2015 17:24:02 +0100
Ondrej Pokorny <laza...@kluug.net> wrote:

On 03.11.2015 17:08, Mattias Gaertner wrote:
Fixed.
How did you find it from the description?
I didn't. I tested on OS X.

You are a wizard :)
Sorry for introducing the bug in my code. But I know how it happened: it
was introduced by CodeTools variable completion:

procedure TIDECommands.PostponeUpdateEvents;
begin
    FDontExecuteUpdateEventsUntil := GetTickCount64 + 500;
end;
codetools use only a simple heuristic to calculate the type of a
binary operator.
The function TFindDeclarationTool.CalculateBinaryOperator needs to be
extended for Alias types (e.g. 'integer' instead of longint) and for
ranges (e.g. if the operands do not fit into an integer).


Should I report it in the bug tracker?
Yes.

Mattias

--
_______________________________________________
Lazarus mailing list
Lazarus@lists.lazarus.freepascal.org
http://lists.lazarus.freepascal.org/mailman/listinfo/lazarus


--
_______________________________________________
Lazarus mailing list
Lazarus@lists.lazarus.freepascal.org
http://lists.lazarus.freepascal.org/mailman/listinfo/lazarus

Reply via email to