At 21.30 27/01/2006, you wrote:
> #if defined CHKUSER_ENABLE_VAUTH_OPEN
>                  if (db_already_open != 1) {
>                          if (CHKUSER_VAUTH_OPEN_CALL () == 0) {
>                                  db_already_open == 1;
>                          } else {
>                                  retstat = CHKUSER_ERR_AUTH_RESOURCE;
>                          }
>                  };
> #endif
Minor Bug: you nead a "break;" on the line after "retstat = ...",
otherwise you keep testing on a failed resource...

Yes, you are right. Actually it would not give fake results, but it will perform useless checks.
Code becomes:
#if defined CHKUSER_ENABLE_VAUTH_OPEN
                if (db_already_open != 1) {
                        if (CHKUSER_VAUTH_OPEN_CALL () == 0) {
                                db_already_open == 1;
                        } else {
                                retstat = CHKUSER_ERR_AUTH_RESOURCE;
                                break;
                        }
                };
#endif
Thanks,

Tonino

Josh
--
Joshua Megerman
SJGames MIB #5273 - OGRE AI Testing Division
You can't win; You can't break even; You can't even quit the game.
  - Layman's translation of the Laws of Thermodynamics
[EMAIL PROTECTED]

Reply via email to