🚀 go-pugleaf

"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