>I'm sure you fellows have a valid point here, since one of my major weak points is API, but just for the record, I've been using that code for years, with probably over 10,000 logins, on different OS, and all types of machines, without any trouble...a pretty solid record. I am going to move to SYS(0) now that we've established it's working fine, in an upgrade eventually, however.

Hi Bruce,

Speaking for myself (but I suspect that Fred would say the same), the messages were simply a warning about passing an integer by value when it's supposed to be by reference. It may work, and may work without a problem. But if it ever blows up, it can be very difficult to track down.

I don't claim to be a C/C++ programmer. I can read it a bit (sometimes< g >). However, one of the rules of thumb that I have is that when I see in the declaration comments either "address of" or "pointer to", I always pass by reference. Usually, the parameter names use the "lp" prefix. It helps keep me out of trouble (except when I do something stupid like with ExtractAssociatedIcon()< g >).