Re: WINEBROWSER: also handle mailto: urls.

[David Lee Lambert]

Hans Leidekker wrote:
> As suggested by Alexandre, I factored out some common code
> in winebrowser and extended it to handle mailto: urls as well.

This version looks much better.  Just one small question:
> +    length = sizeof(mailers);
> +    /* @@ Wine registry key: HKCU\Software\Wine\WineMailer */
> +    if (RegCreateKeyEx( HKEY_CURRENT_USER, "Software\\Wine\\WineMailer", 0, NULL,
> +                        REG_OPTION_NON_VOLATILE, KEY_ALL_ACCESS, NULL, &key, NULL ))
> +    {
> +        fprintf( stderr, "winebrowser: cannot create config key\n" );
> +        return 1;
> +    }
> +
> +    r = RegQueryValueExA( key, "Mailers", 0, &type, (LPBYTE)mailers, &length );
> +    if (r != ERROR_SUCCESS)
> +    {
> +        /* set value to the default */
> +        RegSetValueExA( key, "Mailers", 0, REG_SZ, (LPBYTE)defaultmailers,
> +                        lstrlen( defaultmailers ) + 1 );
> +        strcpy( mailers, defaultmailers );
> +    }
> +    RegCloseKey( key );
> +

Why create another key in the registry?  Wouldn't 
HKCU\Software\Wine\WineBrowser , Mailers make it easier for a user to 
find the value in the registry?

--
DLL