August 2011 - Wine-Devel - list.winehq.org

Re: Assorted spelling fixes.
by Marvin Aug. 24, 2011

Aug. 24, 2011

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=13664 Your paranoid android. === WNT4WSSP6 (32 bit) === No test summary line found === W2KPROSP4 (32 bit) === No test summary line found === WXPPROSP3 (32 bit) === No test summary line found === W2K3R2SESP2 (32 bit) === No test summary line found === WVISTAADM (32 bit) === No test summary line found === W2K8SE (32 bit) === No test summary line found === W7PRO (32 bit) === No test summary line found === W7PROX64 (32 bit) === No test summary line found === W7PROX64 (64 bit) === No test summary line found

1 0

Re: [2/3] msi/tests: Skip a test if the process is limited.
by Austin English Aug. 24, 2011

Aug. 24, 2011

On Wed, Aug 24, 2011 at 03:22, Hans Leidekker <hans(a)codeweavers.com> wrote: > --- > dlls/msi/tests/action.c | 6 ++++++ > 1 files changed, 6 insertions(+), 0 deletions(-) > > diff --git a/dlls/msi/tests/action.c b/dlls/msi/tests/action.c > index aac601f..90832af 100644 > --- a/dlls/msi/tests/action.c > +++ b/dlls/msi/tests/action.c > @@ -4721,6 +4721,12 @@ static void test_create_remove_folder(void) > { > UINT r; > > + if (is_process_limited()) > + { > + skip("process is limited\n"); > + return; > + } > + > CreateDirectoryA("msitest", NULL); > CreateDirectoryA("msitest\\first", NULL); > CreateDirectoryA("msitest\\second", NULL); > -- > 1.7.4.1 Shouldn't that be a win_skip()? Wine always runs as administrator, and if the process is running as a limited user, something may be broken? Unless you're preparing for when wine supports non-admin mode.. :) -- -Austin

4 6

Re: [PATCH 1/8] msi: use an array instead of a hashtable for WHEREVIEW
by Alexandre Julliard Aug. 24, 2011

Aug. 24, 2011

Bernhard Loos <bernhardloos(a)googlemail.com> writes: > +static UINT add_row(MSIWHEREVIEW *wv, UINT val) > { > - MSIHASHENTRY *new = msi_alloc(sizeof(MSIHASHENTRY)); > - MSIHASHENTRY *prev; > + MSIROWENTRY *new; > + > + if (wv->reorder_size <= wv->row_count) > + { > + MSIROWENTRY **new_reorder; > + UINT newsize = wv->reorder_size * 2; > + > + new_reorder = msi_alloc_zero(sizeof(MSIROWENTRY *) * newsize); > + > + if (!new_reorder) > + return ERROR_OUTOFMEMORY; > + > + memcpy(new_reorder, wv->reorder, wv->row_count * sizeof(MSIROWENTRY *)); > + msi_free(wv->reorder); Any reason for not doing a realloc instead? -- Alexandre Julliard julliard(a)winehq.org

1 0

mmdevapi: Do not mix frames and bytes.
by Joerg-Cyril.Hoehle＠t-systems.com Aug. 24, 2011

Aug. 24, 2011

Hi, what happened to that patch? Perhaps there is too much understatement in the subject line. If you look at the code, not the patch, you'll see that alsa_wrap_buffer is called twice, once with bytes, once with frames. That is obviously wrong, so my patch puts frames everywhere. Regards, Jörg Höhle

2 1

Re: [2/8] cmd: Fix FOR tab handling
by Alexandre Julliard Aug. 24, 2011

Aug. 24, 2011

Frédéric Delanoy <frederic.delanoy(a)gmail.com> writes: > @@ -902,8 +902,10 @@ void WCMD_for (WCHAR *p, CMD_LIST **cmdList) { > WIN32_FIND_DATAW fd; > HANDLE hff; > int i; > - const WCHAR inW[] = {'i', 'n', ' ', '\0'}; > - const WCHAR doW[] = {'d', 'o', ' ', '\0'}; > + const WCHAR inW[] = {'i', 'n', ' ', '\0'}; > + const WCHAR inTabW[] = {'i', 'n', '\t', '\0'}; > + const WCHAR doW[] = {'d', 'o', ' ', '\0'}; > + const WCHAR doTabW[] = {'d', 'o', '\t', '\0'}; You don't want to duplicate every string this way. This needs some sort of helper function. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: [PATCH] ntdll: provide a timestamp debug channel, which prints a millisecond resolution timestamp in front of debug output, very useful to debug deadlocks
by Dmitry Timoshkov Aug. 24, 2011

Aug. 24, 2011

Bernhard Loos <bernhardloos(a)googlemail.com> wrote: > ret = wine_dbg_printf( "%7u:", (unsigned int) (now.QuadPart - server_start_time) / 10000 ); Probably you want to use wine_dbgstr_longlong() here. -- Dmitry.

1 0

Re: ntdll: provide a timestamp debug channel, which prints a millisecond resolution timestamp in front of debug output, very useful to debug deadlocks
by Alexandre Julliard Aug. 24, 2011

Aug. 24, 2011

Bernhard Loos <bernhardloos(a)googlemail.com> writes: > @@ -164,8 +165,16 @@ static int NTDLL_dbg_vlog( enum __wine_debug_class cls, struct __wine_debug_chan > /* only print header if we are at the beginning of the line */ > if (info->out_pos == info->output || info->out_pos[-1] == '\n') > { > + if (TRACE_ON(timestamp)) > + { > + LARGE_INTEGER now; > + extern timeout_t server_start_time; > + > + NtQuerySystemTime( &now ); > + ret = wine_dbg_printf( "% 7lld:", (now.QuadPart - server_start_time) / 10000 ); Don't use %ll formats. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: gdi32: Add a buch of CreateScalableFontResource() tests. Resend.
by Alexandre Julliard Aug. 24, 2011

Aug. 24, 2011

Dmitry Timoshkov <dmitry(a)baikal.ru> writes: > + > +/* @makedep: wine_test.ttf */ > +wine_test.ttf RCDATA wine_test.ttf How was this file created? -- Alexandre Julliard julliard(a)winehq.org

3 10

Re: [4/4] urlmon: Added basic MapUrlToZoneEx2 implementation
by Jacek Caban Aug. 24, 2011

Aug. 24, 2011

Hi Thomas, On 08/24/11 01:58, Thomas Mullaly wrote: > +static HRESULT map_uri_to_zone(IUri *uri, DWORD *zone) > +{ > + HRESULT hres; > + BOOL has_scheme = FALSE; > + > + *zone = URLZONE_INVALID; > + > + hres = IUri_HasProperty(uri, Uri_PROPERTY_SCHEME_NAME,&has_scheme); > + if(hres != S_OK) > + return hres; > + > + if(has_scheme) { > + BSTR display_uri; > + > + hres = IUri_GetDisplayUri(uri,&display_uri); > + if(FAILED(hres)) > + return hres; > + > + hres = map_url_to_zone(display_uri, zone, NULL); I think map_url_to_zone should be integrated to map_uri_to_zone and operate on IUri instead of URL string (ideally we should get rid of map_url_to_zone once transition to IUri-based API will be done), not the other way around. Jacek

1 0

Re: winhttp: Support setting WinHttpRequestOption_EnableRedirects.
by Jacek Caban Aug. 24, 2011

Aug. 24, 2011

Hi Hans, On 08/24/11 10:23, Hans Leidekker wrote: > + EnterCriticalSection(&request->cs ); > + switch (option) > + { > + case WinHttpRequestOption_EnableRedirects: > + { > + if (V_BOOL(&value ) == VARIANT_TRUE) You shouldn't compare bools like that, it should be if(V_BOOL(&value)). > + request->disable_feature&= !WINHTTP_DISABLE_REDIRECTS; Here is a typo, it should be ~WINHTTP_DISABLE_REDIRECTS. Jacek

1 0

mmdevapi: IsFormatSupported fills closest match iff it returns S_FALSE.
by Joerg-Cyril.Hoehle＠t-systems.com Aug. 24, 2011

Aug. 24, 2011

Hi, what about this patch of mine? #77305 is about to fall off the queue. 3 weeks later, it's still ok AFAICT. >This will need another sweep in some future after somebody tests what happens >in EXCLUSIVE mode about which MSDN says the pointer is set to null. >I only tested SHARED mode with standard formats. I've 2 patches in my queue that depend on this one: - indeed, no format suggestion in exclusive mode; - if all else fails, suggest GetFormatMix in shared mode BTW, #77096 was superseded by something long committed. Regards, Jörg Höhle

1 0

Re: [PATCH] attrib: Move implementation from cmd.exe to the standalone command
by Frédéric Delanoy Aug. 24, 2011

Aug. 24, 2011

On Wed, Aug 24, 2011 at 08:20, Christian Costa <titan.costa(a)wanadoo.fr> wrote: > -- > > I added a stub last year to make an installer work but didn't know the command existed as a builtin in cmd.exe until Frédéric Delanoy told me. So here is a patch that takes what's in cmd.exe and move it to the standalone command. Some few code was taken from xcopy. > --- > programs/attrib/Makefile.in | 4 + > programs/attrib/attrib.c | 190 +++++++++++++++++++++++++++++++++++++++++-- > programs/attrib/attrib.h | 28 ++++++ > programs/attrib/attrib.rc | 49 +++++++++++ > programs/cmd/builtins.c | 83 ------------------- > programs/cmd/cmd.rc | 2 > programs/cmd/wcmd.h | 94 ++++++++++----------- > programs/cmd/wcmdmain.c | 4 - > 8 files changed, 309 insertions(+), 145 deletions(-) > create mode 100644 programs/attrib/attrib.h > create mode 100644 programs/attrib/attrib.rc Most translatable strings (except those in dialogs) have been moved to/adapted for use in po files. You have to regenerate the po files using "make depend && make" in maintainer mode (./configure --enable-maintainer-mode). Frédéric

1 1

Re: André Hentschel : po: German translations added and updated.
by Francois Gouget Aug. 23, 2011

Aug. 23, 2011

On Mon, 22 Aug 2011, Alexandre Julliard wrote: > Module: wine > Branch: master > Commit: aeb9f621292dec7a6366209106468da8d0371684 > URL: http://source.winehq.org/git/wine.git/?a=commit;h=aeb9f621292dec7a636620910… I think the string below should really be translated. It's displayed in winecfg -> Desktop Integration tab, in the 'Item' list more precisely. This is where one can decide what color or font to use for each of the GUI elements. And actually all the items around "Message Box Text" are translated. > msgid "Message Box Text" > -msgstr "" > +msgstr "Message Box Text" There's also another translation that I think is wrong: > #: wordpad.rc:88 > -#, fuzzy > msgid "&Pale yellow\tCtrl+2" > msgstr "&PostIt-Notiz\tStrg+2" This used to be 'PostThat Yellow' which was a word play on PostIt. However 'Pale Yellow' is much clearer and the German translation does not look like a color name at all currently. So I think it's best not to continue with the bad pun (besides otherwise you would probably need to change it to 'PostIt[tm]...' anyway). Everything else looks good (to a non-German speaking guy). -- Francois Gouget <fgouget(a)free.fr> http://fgouget.free.fr/ Nouvelle version : les anciens bogues ont été remplacés par de nouveaux.

2 1

Summer is ending, you can't procrastinate any more, come to Wineconf!
by Jeremy White Aug. 23, 2011

Aug. 23, 2011

So summer holidays are rapidly ending, and before you know it, Wineconf will be upon us - just 5 more weeks until Wineconf! The full details, including the hotel information, is available here: http://wiki.winehq.org/WineConf2011 For those of you that have asked for sponsorship, I'm trying to come to a 'final' set of requests so I can figure out how best to divvy up the funds we have available. I hope to have that info soon. If you're interested in coming, but can't afford it, please write to me privately to see if we can help. Hope to see you there! Cheers, Jeremy p.s. I promise to try to keep most discussion on the http://www.winehq.org/mailman/listinfo/wineconf mailing list, which you should join now if you're at all interested in Wineconf.

1 0

Re: [5/5] ddraw: Emulate a 24 bit depth format without stencil or padding
by Henri Verbeet Aug. 23, 2011

Aug. 23, 2011

On 22 August 2011 21:16, Stefan Dösinger <stefandoesinger(a)gmx.at> wrote: > + {WINED3DFMT_D24_UNORM, 0x0, 0x0, 0x0, 0x0, 4, 24, 0}, > {WINED3DFMT_X8D24_UNORM, 0x0, 0x0, 0x0, 0x0, 4, 24, 0}, How is WINED3DFMT_D24_UNORM different from WINED3DFMT_X8D24_UNORM?

2 4

Re: [3/7] msi: Don't create a temporary copy of the package. (try 2)
by Octavian Voicu Aug. 23, 2011

Aug. 23, 2011

On Tue, Aug 23, 2011 at 3:13 PM, Joris Huizer <joris_huizer(a)yahoo.com>wrote: > On 08/23/2011 11:45 AM, Hans Leidekker wrote: > >> + if (!strcmpiW( package, unsquashed )) >> > > I'm confused about the last part of the function. > I think if the comparison between 'package' and 'unsquashed' fails, you > want to return an error, > but the return value 'r' will be ERROR_SUCCESS at that point, and isn't > updated. > Hello, The function strcmpiW returns 0 when the strings are equal, and non-zero when they are different (< 0 if first is smaller, > 0 if first is larger, lexicographically speaking). The condition checks if the strings match, not if the function failed (string comparison cannot "fail"). Cheers, Octavian

1 0

Re: [3/7] msi: Don't create a temporary copy of the package. (try 2)
by Joris Huizer Aug. 23, 2011

Aug. 23, 2011

Just an implementation detail I noticed: On 08/23/2011 11:45 AM, Hans Leidekker wrote: > + > +static UINT get_registered_local_package( const WCHAR *product, const WCHAR *package, WCHAR *localfile ) > +{ > + MSIINSTALLCONTEXT context; > + HKEY product_key, props_key; > + WCHAR *registered_package = NULL, unsquashed[GUID_SIZE]; > + UINT r = ERROR_FUNCTION_FAILED; > + > + r = msi_locate_product( product,&context ); > + if (r != ERROR_SUCCESS) > + return r; > + > + r = MSIREG_OpenProductKey( product, NULL, context,&product_key, FALSE ); > + if (r != ERROR_SUCCESS) > + return r; > + > + r = MSIREG_OpenInstallProps( product, context, NULL,&props_key, FALSE ); > + if (r != ERROR_SUCCESS) > + { > + RegCloseKey( product_key ); > + return r; > + } > + registered_package = msi_reg_get_val_str( product_key, INSTALLPROPERTY_PACKAGECODEW ); > + if (!registered_package) > + goto done; > + > + unsquash_guid( registered_package, unsquashed ); > + if (!strcmpiW( package, unsquashed )) > + { > + WCHAR *filename = msi_reg_get_val_str( props_key, INSTALLPROPERTY_LOCALPACKAGEW ); > + strcpyW( localfile, filename ); > + msi_free( filename ); > + r = ERROR_SUCCESS; > + } > +done: > + msi_free( registered_package ); > + RegCloseKey( props_key ); > + RegCloseKey( product_key ); > + return r; > +} > + > I'm confused about the last part of the function. I think if the comparison between 'package' and 'unsquashed' fails, you want to return an error, but the return value 'r' will be ERROR_SUCCESS at that point, and isn't updated. HTH, Joris

1 0

Re: [7/7] msi/tests: Clean up local packages.
by Marvin Aug. 23, 2011

Aug. 23, 2011

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=13580 Your paranoid android. === WNT4WSSP6 (32 bit install) === install.c:5997: Test failed: Per-user icon file isn't where it's expected (C:\WINNT\Profiles\Default User\Application Data\Microsoft\Installer\{7DF88A49-996F-4EC8-A022-BF956F9B2CBB}\testicon)

1 0

Re: [6/7] msi: Remove icons when the product is uninstalled.
by Marvin Aug. 23, 2011

Aug. 23, 2011

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=13579 Your paranoid android. === WNT4WSSP6 (32 bit install) === install.c:5979: Test failed: Per-user icon file isn't where it's expected (C:\WINNT\Profiles\Default User\Application Data\Microsoft\Installer\{7DF88A49-996F-4EC8-A022-BF956F9B2CBB}\testicon)

1 0

Re: tools/winegcc: support a trailing / in paths to winebuild
by Alexandre Julliard Aug. 23, 2011

Aug. 23, 2011

Bernhard Loos <bernhardloos(a)googlemail.com> writes: > diff --git a/tools/winegcc/winegcc.c b/tools/winegcc/winegcc.c > index 284223e..3a7e93a 100644 > --- a/tools/winegcc/winegcc.c > +++ b/tools/winegcc/winegcc.c > @@ -1326,6 +1326,7 @@ int main(int argc, char **argv) > { > case 'B': > str = strdup(option_arg); > + if (strendswith(str, "/")) str[strlen(str) - 1] = 0; Why would you want to append a slash to winebuild? -- Alexandre Julliard julliard(a)winehq.org

4 5

Re: [7/7] msi/tests: Clean up local packages.
by Marvin Aug. 23, 2011

Aug. 23, 2011

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=13576 Your paranoid android. === WNT4WSSP6 (32 bit install) === install.c:5997: Test failed: Per-user icon file isn't where it's expected (C:\WINNT\Profiles\Default User\Application Data\Microsoft\Installer\{7DF88A49-996F-4EC8-A022-BF956F9B2CBB}\testicon) === WXPPROSP3 (32 bit install) === Timeout

1 0

Re: [6/7] msi: Remove icons when the product is uninstalled.
by Marvin Aug. 23, 2011

Aug. 23, 2011

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=13575 Your paranoid android. === WNT4WSSP6 (32 bit install) === install.c:5979: Test failed: Per-user icon file isn't where it's expected (C:\WINNT\Profiles\Default User\Application Data\Microsoft\Installer\{7DF88A49-996F-4EC8-A022-BF956F9B2CBB}\testicon)

1 0

Re: [PATCH 2/4] shell32: Register standard known folders on initialization.
by Alexandre Julliard Aug. 23, 2011

Aug. 23, 2011

Mariusz Pluciński <vshader(a)gmail.com> writes: > + for(i=0; i < sizeof(CSIDL_Data)/sizeof(CSIDL_Data[0]); ++i) > + { > + if(!CSIDL_Data[i].szValueName) > + { > + FIXME("known folder %d has no value name, ignoring\n", i); > + continue; > + } > + > + /* FIXME: remove this check after fixing all known folders definitions */ > + if(is_knownfolder_registered(CSIDL_Data[i].id)) > + { > + FIXME("known folder %d already registered, ignoring\n", i); > + continue; > + } It's pretty annoying to see these fixmes on every prefix update. You don't need them anyway, what needs to be done is clear from the code. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: [PATCH] GNU/kFreeBSD detection
by Alexandre Julliard Aug. 23, 2011

Aug. 23, 2011

Robert Millan <rmh(a)debian.org> writes: > diff --git a/tools/winebuild/build.h b/tools/winebuild/build.h > index 05adf31..cf389fb 100644 > --- a/tools/winebuild/build.h > +++ b/tools/winebuild/build.h > @@ -148,6 +148,7 @@ enum target_platform > PLATFORM_UNSPECIFIED, > PLATFORM_APPLE, > PLATFORM_FREEBSD, > + PLATFORM_GNUKFREEBSD, > PLATFORM_SOLARIS, > PLATFORM_WINDOWS > }; If it doesn't need a different behavior there's no need for a new platform, just use PLATFORM_FREEBSD. -- Alexandre Julliard julliard(a)winehq.org

3 2

Re: mshtml: Use wininet to get and parse the proxy (try 2)
by André Hentschel Aug. 23, 2011

Aug. 23, 2011

please ignore also that one. APPINFO_QueryOption needs to be fixed first Am 20.08.2011 17:19, schrieb André Hentschel: > > fixed the leak > --- > dlls/mshtml/nsembed.c | 62 +++++++++++++++++++++++------------------------- > 1 files changed, 30 insertions(+), 32 deletions(-) > > diff --git a/dlls/mshtml/nsembed.c b/dlls/mshtml/nsembed.c > index 2bebf8b..c2581e6 100644 > --- a/dlls/mshtml/nsembed.c > +++ b/dlls/mshtml/nsembed.c > @@ -27,6 +27,7 @@ > #include "winbase.h" > #include "winuser.h" > #include "winreg.h" > +#include "wininet.h" > #include "ole2.h" > #include "shlobj.h" > > @@ -488,51 +489,48 @@ static void set_lang(nsIPrefBranch *pref) > > static void set_proxy(nsIPrefBranch *pref) > { > - char proxy[512]; > - char * proxy_port; > - int proxy_port_num; > - DWORD enabled = 0, res, size, type; > - HKEY hkey; > + CHAR hostname[INTERNET_MAX_HOST_NAME_LENGTH]; > + URL_COMPONENTSA UrlComponents; > + INTERNET_PROXY_INFOA *pi; > + DWORD size; > > - static const WCHAR proxy_keyW[] = > - {'S','o','f','t','w','a','r','e', > - '\\','M','i','c','r','o','s','o','f','t', > - '\\','W','i','n','d','o','w','s', > - '\\','C','u','r','r','e','n','t','V','e','r','s','i','o','n', > - '\\','I','n','t','e','r','n','e','t',' ','S','e','t','t','i','n','g','s',0}; > + InternetQueryOptionA(NULL, INTERNET_OPTION_PROXY, NULL, &size); > + if (!size) return; > > - res = RegOpenKeyW(HKEY_CURRENT_USER, proxy_keyW, &hkey); > - if(res != ERROR_SUCCESS) > - return; > + pi = heap_alloc(size); > + if (!pi) return; > > - size = sizeof(enabled); > - res = RegQueryValueExA(hkey, "ProxyEnable", 0, &type, (LPBYTE)&enabled, &size); > - if(res != ERROR_SUCCESS || type != REG_DWORD || enabled == 0) > + if(!InternetQueryOptionA(NULL, INTERNET_OPTION_PROXY, pi, &size)) > { > - RegCloseKey(hkey); > + heap_free(pi); > return; > } > > - size = sizeof(proxy); > - res = RegQueryValueExA(hkey, "ProxyServer", 0, &type, (LPBYTE)proxy, &size); > - RegCloseKey(hkey); > - if(res != ERROR_SUCCESS || type != REG_SZ) > - return; > + if (pi->dwAccessType != INTERNET_OPEN_TYPE_PROXY) return; > > - proxy_port = strchr(proxy, ':'); > - if (!proxy_port) > + memset( &UrlComponents, 0, sizeof(UrlComponents)); > + UrlComponents.dwStructSize = sizeof(UrlComponents); > + UrlComponents.lpszHostName = hostname; > + UrlComponents.dwHostNameLength = INTERNET_MAX_HOST_NAME_LENGTH; > + if(!InternetCrackUrlA(pi->lpszProxy, 0, 0, &UrlComponents)) > + { > + heap_free(pi); > return; > + } > + > + if(UrlComponents.nPort == INTERNET_INVALID_PORT_NUMBER) > + UrlComponents.nPort = INTERNET_DEFAULT_HTTP_PORT; > > - *proxy_port = 0; > - proxy_port_num = atoi(proxy_port + 1); > - TRACE("Setting proxy to %s, port %d\n", debugstr_a(proxy), proxy_port_num); > + TRACE("Setting proxy to %s, port %d\n", UrlComponents.lpszHostName, UrlComponents.nPort); > > - set_string_pref(pref, "network.proxy.http", proxy); > - set_string_pref(pref, "network.proxy.ssl", proxy); > + set_string_pref(pref, "network.proxy.http", UrlComponents.lpszHostName); > + set_string_pref(pref, "network.proxy.ssl", UrlComponents.lpszHostName); > > set_int_pref(pref, "network.proxy.type", 1); > - set_int_pref(pref, "network.proxy.http_port", proxy_port_num); > - set_int_pref(pref, "network.proxy.ssl_port", proxy_port_num); > + set_int_pref(pref, "network.proxy.http_port", UrlComponents.nPort); > + set_int_pref(pref, "network.proxy.ssl_port", UrlComponents.nPort); > + > + heap_free(pi); > } > > static void set_preferences(void) -- Best Regards, André Hentschel

2 1