Another version of an infinite timer loop in Developer Help

ID:139290
 
Mar 20 2011, 1:13 pm (Edited on Mar 20 2011, 2:17 pm)
Code:
            spawn()
            {
                while(1)
                {
                    var/time = Proc_GetTime();
                    winset(src, "default.ClockLabel","text=\"Server Time: [time]\"");
                    sleep(600); // Update once a minute.
                }
            }


Problem description:
May not be a problem but I'm trying to track down a potential problem. Is this infinite loop following best practices or is it problematic?
#1 Mar 20 2011, 1:40 pm
It looks fine. However, naturally, the result can depend on where it's called from and how often (i.e. inside what block you placed it) and the specifics of the Proc_GetTime() proc.
#2 Mar 20 2011, 2:28 pm
In response to Kaioken (#1)
It's called once at world startup and Proc_GetTime() simply builds and returns a string.