Nope. Looks good to me. Have you constructed a test situation and tried it? If you say it works, I'll take your word for it. (Soon we'll have genuine JS tests.)