It *IS* really bad code, because it is not documentated. You used a undocumented feature and undocumented feature can change *every* version without noticing it.
When you want a secure and stable code, DONT USE THIS KIND OF THINGS!
btw. you could "macro" the openwindow, when you really want this feature. In this case it will work allways:
Code: Select all
procedure _openwindow(id,...)
x=openwindow(id,...)
if id = #pb_any
procedurereturn x
elseif x
procedurereturn windowid(id)
endif
procedurereturn 0
endprocedure
macro openwindow(id,...)
_openwindow(id,...)
endmacro