Re: xlive: stub some functions (2/3)

2011-08-17 Thread Austin English
On Wed, Aug 17, 2011 at 12:33, Giovanni Ongaro wrote: > I am working oon a xlive patch can you tell me how to finf dll ordinal > functions by using winedump i am tryiing specifically to find how much > parameters xlive 5282 has > > Can you help me? >From the patch: +/* This DLL is completely undo

Re: mshtml: Use the last colon in proxy url as port separator

2011-08-17 Thread Michael Stefaniuc
On 08/17/2011 10:31 PM, André Hentschel wrote: > Am 17.08.2011 22:18, schrieb Juan Lang: >> Hi André, >> >>> http://domain\user:passw...@server.com:8080 makes it visible that it's >>> intended to get the last colon >> >> What about http://domain\user:passw...@server.com? password isn't the >> por

Re: mshtml: Use the last colon in proxy url as port separator

2011-08-17 Thread André Hentschel
Am 17.08.2011 22:18, schrieb Juan Lang: > Hi André, > >> http://domain\user:passw...@server.com:8080 makes it visible that it's >> intended to get the last colon > > What about http://domain\user:passw...@server.com? password isn't the > port number. > --Juan in case password is not a number a

mshtml: Use the last colon in proxy url as port separator

2011-08-17 Thread Juan Lang
Hi André, > http://domain\user:passw...@server.com:8080 makes it visible that it's > intended to get the last colon What about http://domain\user:passw...@server.com? password isn't the port number. --Juan

buildbot status

2011-08-17 Thread Dan Kegel
I brought up the proof-of-concept buildbot-based patchwatcher on two old PCs here; its status page is http://buildbot.kegel.com/ It has caught one real problem so far this morning. to-do list: - make cleaner script and doc for adding a new slave - make it start automatically after reboot - sandbox

Re: regedit: adding confirmation message before importing from file

2011-08-17 Thread Nowres Rafid
On 17/08/2011 15:19, Dan Kegel wrote: On Wed, Aug 17, 2011 at 8:17 AM, Dan Kegel wrote: Hi Nowres, did you run the regedit tests, cd programs/regedit/tests make test after that change? Here it fails with regedit.c:384: Test failed: regedit not available, skipping regedit tests ( log at

Re: regedit: adding confirmation message before importing from file

2011-08-17 Thread Dan Kegel
On Wed, Aug 17, 2011 at 8:17 AM, Dan Kegel wrote: > Hi Nowres, > did you run the regedit tests, >   cd programs/regedit/tests >   make test > after that change?  Here it fails with >  regedit.c:384: Test failed: regedit not available, skipping regedit tests > ( log at http://buildbot.kegel.com/bui

re: regedit: adding confirmation message before importing from file

2011-08-17 Thread Dan Kegel
Hi Nowres, did you run the regedit tests, cd programs/regedit/tests make test after that change? Here it fails with regedit.c:384: Test failed: regedit not available, skipping regedit tests ( log at http://buildbot.kegel.com/builders/runtests/builds/2 ) You might need to fully implement th

Re: cmd: fixing an issue with extra whitespaces after cd parameter (retry)

2011-08-17 Thread Nowres Rafid
On 17/08/2011 14:42, Dan Kegel wrote: Frédéric Delanoy wote: Nowres Rafid wrote: how can I generate the expected file (.exp)? It already exists in the source tree I think Nowres meant, how can he update the .exp file to match the expected output of his new test. The answer is: run the new te

re: cmd: fixing an issue with extra whitespaces after cd parameter (retry)

2011-08-17 Thread Dan Kegel
Frédéric Delanoy wote: > Nowres Rafid wrote: >> how can I generate the expected file (.exp)? > It already exists in the source tree I think Nowres meant, how can he update the .exp file to match the expected output of his new test. The answer is: run the new test, save its output, and add it to t

Re: cmd: fixing an issue with extra whitespaces after cd parameter (retry)

2011-08-17 Thread Frédéric Delanoy
2011/8/17 Nowres Rafid : > On 17/08/2011 14:02, Frédéric Delanoy wrote: >> >> On Fri, Aug 12, 2011 at 19:14, Nowres Rafid >>  wrote: >>> >>> fixing cd ..@space@ >>> >>> please ignore the previous patch. >> >> You didn't include any test in your patch. >> All patches including a fix must have tests

Re: cmd: fixing an issue with extra whitespaces after cd parameter (retry)

2011-08-17 Thread Frédéric Delanoy
On Fri, Aug 12, 2011 at 19:14, Nowres Rafid wrote: > fixing cd ..@space@ > > please ignore the previous patch. You didn't include any test in your patch. All patches including a fix must have tests confirming the bug is fixed

Vista/w2k8/w7 users, please test mmdevapi capture

2011-08-17 Thread Joerg-Cyril . Hoehle
Brian Vincent wrote: >It looks like I got one "test failed" followed by an ok one on the second run. >capture.c:548: Test failed: Master volume wasn't 1: 0.769175 I'm not familiar with the volume test, but past winmm experience is that native may have remembered your previous volume setting (winmm

Re: regedit: adding confirmation message before importing from file

2011-08-17 Thread Andrew Eikum
On Wed, Aug 17, 2011 at 12:24:18PM +, Nowres Rafid wrote: > -reg_file = fopen(realname, "r"); > + > + reg_file = fopen(realname, "r"); The file uses spaces for indenting. Please be consistent and don't use tabs. This is true elsewhere, to

Re: Listed package maintainers

2011-08-17 Thread Ben Klein
On 17 August 2011 22:21, Scott Ritchie wrote: > On 07/30/2011 08:27 AM, Ben Klein wrote: >> I also notice the pages have been renamed (so instead of deb = ubuntu, >> ubuntu and debian are named correctly and unambiguously). I would like >> to voice my approval as former maintainer of the Debian pa

Re: Listed package maintainers

2011-08-17 Thread Scott Ritchie
On 07/30/2011 08:27 AM, Ben Klein wrote: > I also notice the pages have been renamed (so instead of deb = ubuntu, > ubuntu and debian are named correctly and unambiguously). I would like > to voice my approval as former maintainer of the Debian packages, as > token as that may seem at this stage. >

Re: dpnet: add missing IDirectPlay8Peer methods

2011-08-17 Thread Michael Stefaniuc
Luis, Louis Lenders wrote: > This is part of bug http://bugs.winehq.org/show_bug.cgi?id=28008 a line length of 400+ chars kinda stretches what is acceptable. Please try to stick to a length of 100 in dpnet; that's the Wine preferred line length and also what I used during the COM cleanup in dpnet.

Re: d3dx9: Implemented D3DXWeldVertices and test.

2011-08-17 Thread Michael Mc Donnell
On Tue, Aug 16, 2011 at 9:14 PM, Stefan Dösinger wrote: > On Monday 15 August 2011 12:16:55 Michael Mc Donnell wrote: >> Stefan thought a previous version of this looked ok, and I haven't had >> any negative feedback on this version either. > The 104 kb / 3000 line patch is rather big, you could p