Mapping @HOMESHR is mostly unnecessary - it its defined in the user profile, it should map auto-magically.

You SHOULD remove the reference to SetTime - this was for Wintendo (WFW and Win-9x) platforms that did not know how to sync with a DC. All of the domain-aware platforms since Server 2000 and XP auto-sync time and you should not force time-sync during logon.

Minor thing - "?" is a substitute for @CRLF, which is usually placed at the end of a line. It's not a "print" command (like the old BASIC interpreters). If you put it first, you will leave the cursor hanging at the end of a line when the script ends, which might make a difference if other commands that follow (or the bat file running the Kix script) outputs anything.

Glenn

PS - If you are thinking about OU-based mapping, look at the login script on my site. We provide mapping by group, subnet, OU, Computer-OU, AD Site, user and more, and extend most of those options with the ability to dynamically change the UNC - for example, a single drive mapping can dynamically switch the UNC Path based on site, ou, subnet and more via a lookup table. The full user guide is available, and the script is free for small organizations (under 5 domain controllers) and "damn cheap" for everyone else.
_________________________
Actually I am a Rocket Scientist! \:D