🚀 go-pugleaf

Gambit

1)
  if(Reg->OpenKey("\\Software\\Microsoft\\DirectX", false))
  {
    if(Reg->KeyExists("\\Software\\Microsoft\\DirectX"))
    {
    }
  }
 Reg->CloseKey();



2)
  if(Reg->OpenKey("\\Software\\Microsoft\\DirectX", false))
  {
    if(Reg->KeyExists("\\Software\\Microsoft\\DirectX"))
    {
    }
    Reg->CloseKey();
  }




2 is the answer?:


TY
Edward








 "Remy Lebeau (TeamB)" <gambit47.no.spam@no.spam.yahoo.com> escreveu na
mensagem news:40eb8d7b$1@newsgroups.borland.com...
>
> "Edward" <Registrados2004@aol.com> wrote in message
> news:40eb41eb$1@newsgroups.borland.com...
>
> > What I wound up doing, was something much simpler, but just as
effective.
>
> That won't always work, either.
>
> >   while(i < sizeof(biosnro))
> >   {
> >     biosnro[i] = ' ';
> >     i++;
> >   }
>
> You could use memset() instead of looping manually.
>
> >   Reg->OpenKey("\\HARDWARE\\DESCRIPTION\\System", false);
> >   if(Reg->KeyExists("\\HARDWARE\\DESCRIPTION\\System"))
> >   {
>
> You should be checking the return value of OpenKey() instead of
KeyExists():
>
>     if( Reg->OpenKey("\\HARDWARE\\DESCRIPTION\\System", false) )
>     {
>
> >      Reg->ReadBinaryData("SystemBiosVersion",&biosnro,sizeof(biosnro));
>
> I just checked on my Win98 machine, and although the key you are using
does
> exist, the particular value you are querying does not.  Which means that
> your code would fail on my machine, for instance.
>
> >     It doenst work because some times the datavaleu os
> > "SystemBiosVersion" are REG_MULTI_SZ other times
> > REG_SZ and other times are Unknow type
>
> And other times it simply does not exist at all.
>
> > I dont know what more i can do.
>
> If all you want is a unique ID, then why not just generate a new GUID via
> the CoCreateGuid() function?  It is already guaranteed to produce a unique
> identifier every time it is called.
>
>
> Gambit
>
>