Is it considered bad practice to put an infinite loop inside of TThread.Execute?

Question

i wrote a Thread.descendent class and in the execute method i put a a infinite loop to listen a com event, is considered a bad Threading practice use a infinite loop to do this ? the applications works fine , does not freeze and is allways responsive, i just answer because i want use the best method to threading.

procedure TMyThread.Execute;
begin
    while True and not Terminated do
    begin
     AResult:= FListener.GetResult(Param1,Param2,5000);
      if not VarIsNull(AResult) then
        Synchronize(Process);
    end;
end;

Answer 1

The compiler transforms this into:

while not Terminated do

When written out this way I'm sure you'll agree it looks perfectly natural. This is a very common idiom.