Ros-dev

ros-dev@reactos.org

5791 discussions

Apologies
by Olaf Siejka 07 May '11

07 May '11

I am sorry to Vicmarcal for outbursting on ros-dev. The reason for that was yet another pointless bugreport, after my requestes, passed several times, regarding bug reports quality. Sorry, Vicmarcal, as well as those who had to witness it. Regards

3 2

Vicmarcal - are you mad?
by Olaf Siejka 06 May '11

06 May '11

Bugzilla is crowded with untouched bugs for so long, and you HAVE to add more crud to it? Bugs like no.6214, very usefull as we can see below: "1)Download ClickToDesktop 2)Run in ReactOS 3)Click on the desktop to hide. It doesn't work. Apps just lost the focus. Nothing appears in the DebugLog." Now that is informative... Why dont you get back doing something useful for a change?

8 13

Re: [ros-dev] [EVENTVWR/EVENTLOG] Fix a double off-by-one bug
by Love Nystrom 05 May '11

05 May '11

Hi Gabriel, Eric, Isn't the real issue why LogFile->Header.CurrentRecordNumber is off by one ? If that is found and fixed, this patch will make the error re-emerge as an off by -1. And if CurrentRecordNumber is actually used as the next *pending* record number, the identifier is somewhat mis-leading .. ? Just my penny to the pot Best Regards L. ---------- Forwarded message ---------- > From: Gabriel ilardi <gabrielilardi(a)hotmail.it> > To: ros-dev <ros-dev(a)reactos.org> > Date: Mon, 2 May 2011 21:50:03 +0200 > Subject: Re: [ros-dev] [ros-diffs] [ekohl] 51558: [EVENTVWR/EVENTLOG] Fix a > double off-by-one bug: - The eventlog service was reporting one event more > than was available (+1). - The event viewer did not display the latest event > from... > Great work on event service and event viewer... keep it up Eric :) > > > > Date: Mon, 2 May 2011 19:38:24 +0000 > > To: ros-diffs(a)reactos.org > > From: ekohl(a)svn.reactos.org > > Subject: [ros-diffs] [ekohl] 51558: [EVENTVWR/EVENTLOG] Fix a double > off-by-one bug: - The eventlog service was reporting one event more than was > available (+1). - The event viewer did not display the latest event from... > > > > Author: ekohl > > Date: Mon May 2 19:38:23 2011 > > New Revision: 51558 > > > > URL: http://svn.reactos.org/svn/reactos?rev=51558&view=rev > > Log: > > [EVENTVWR/EVENTLOG] > > Fix a double off-by-one bug: > > - The eventlog service was reporting one event more than was available > (+1). > > - The event viewer did not display the latest event from the eventlog > service (-1). > > > > See issue #6182 for more details. > > > > Modified: > > trunk/reactos/base/applications/mscutils/eventvwr/eventvwr.c > > trunk/reactos/base/services/eventlog/rpc.c > > > > Modified: trunk/reactos/base/applications/mscutils/eventvwr/eventvwr.c > > URL: > http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/mscutils… > > > ============================================================================== > > --- trunk/reactos/base/applications/mscutils/eventvwr/eventvwr.c > [iso-8859-1] (original) > > +++ trunk/reactos/base/applications/mscutils/eventvwr/eventvwr.c > [iso-8859-1] Mon May 2 19:38:23 2011 > > @@ -503,7 +503,7 @@ > > HWND hwndDlg; > > HANDLE hEventLog; > > EVENTLOGRECORD *pevlr; > > - DWORD dwRead, dwNeeded, dwThisRecord, dwTotalRecords = 0, > dwCurrentRecord = 1, dwRecordsToRead = 0, dwFlags, dwMaxLength; > > + DWORD dwRead, dwNeeded, dwThisRecord, dwTotalRecords = 0, > dwCurrentRecord = 0, dwRecordsToRead = 0, dwFlags, dwMaxLength; > > LPWSTR lpSourceName; > > LPWSTR lpComputerName; > > LPSTR lpData; > > > > Modified: trunk/reactos/base/services/eventlog/rpc.c > > URL: > http://svn.reactos.org/svn/reactos/trunk/reactos/base/services/eventlog/rpc… > > > ============================================================================== > > --- trunk/reactos/base/services/eventlog/rpc.c [iso-8859-1] (original) > > +++ trunk/reactos/base/services/eventlog/rpc.c [iso-8859-1] Mon May 2 > 19:38:23 2011 > > @@ -199,6 +199,7 @@ > > DWORD *NumberOfRecords) > > { > > PLOGHANDLE lpLogHandle; > > + DWORD dwRecords; > > > > lpLogHandle = ElfGetLogHandleEntryByHandle(LogHandle); > > if (!lpLogHandle) > > @@ -206,7 +207,9 @@ > > return STATUS_INVALID_HANDLE; > > } > > > > - *NumberOfRecords = lpLogHandle->LogFile->Header.CurrentRecordNumber; > > + dwRecords = lpLogHandle->LogFile->Header.CurrentRecordNumber; > > + > > + *NumberOfRecords = (dwRecords > 0) ? (dwRecords - 1) : 0; > > > > return STATUS_SUCCESS; > > } > > > > >

2 1

Re: [ros-dev] [freeldr] Add PXE "filesystem" (Ged Murphy)
by Love Nystrom 05 May '11

05 May '11

Hi Ged, Yes of course I read Hervé's mail. I did not suggest that he should go about porting existing code, I just expressed my opinion that we shouldn't use AT&T for new code. As for porting the existing freeldr AT&T code, I guess I could, eventually, when I get time, if I can just get a firm enough grip on the AT&T directive voodoo, which is obscure. But in the short run, no, sorry, I'm really bogged down by RL :( (It's even hard to try and keep up with the mailing list) You wouldn't happen to have a good pointer for that obscure AT&T string voodoo? Best Regards L. On Tue, May 3, 2011 at 11:12 PM, <ros-dev-request(a)reactos.org> wrote: > Send Ros-dev mailing list submissions to > ros-dev(a)reactos.org > > To subscribe or unsubscribe via the World Wide Web, visit > http://www.reactos.org/mailman/listinfo/ros-dev > or, via email, send a message with subject or body 'help' to > ros-dev-request(a)reactos.org > > You can reach the person managing the list at > ros-dev-owner(a)reactos.org > > When replying, please edit your Subject line so it is more specific > than "Re: Contents of Ros-dev digest..." > > Today's Topics: > > 1. Re: [EVENTVWR/EVENTLOG] Fix a double off-by-one bug (Love Nystrom) > 2. Re: [freeldr] Add PXE "filesystem" (Love Nystrom) > 3. Re: [freeldr] Add PXE "filesystem" (Ged Murphy) > > > ---------- Forwarded message ---------- > From: Love Nystrom <love.nystrom(a)gmail.com> > To: ros-dev(a)reactos.org > Date: Tue, 3 May 2011 21:52:40 +0700 > Subject: Re: [ros-dev] [EVENTVWR/EVENTLOG] Fix a double off-by-one bug > > Hi Gabriel, Eric, > > Isn't the real issue why LogFile->Header.CurrentRecordNumber is off by one > ? > If that is found and fixed, this patch will make the error re-emerge as an > off by -1. > And if CurrentRecordNumber is actually used as the next *pending* record > number, > the identifier is somewhat mis-leading .. ? > > Just my penny to the pot > Best Regards > L. > > ---------- Forwarded message ---------- >> From: Gabriel ilardi <gabrielilardi(a)hotmail.it> >> To: ros-dev <ros-dev(a)reactos.org> >> Date: Mon, 2 May 2011 21:50:03 +0200 >> Subject: Re: [ros-dev] [ros-diffs] [ekohl] 51558: [EVENTVWR/EVENTLOG] Fix >> a double off-by-one bug: - The eventlog service was reporting one event more >> than was available (+1). - The event viewer did not display the latest event >> from... >> Great work on event service and event viewer... keep it up Eric :) >> >> >> > Date: Mon, 2 May 2011 19:38:24 +0000 >> > To: ros-diffs(a)reactos.org >> > From: ekohl(a)svn.reactos.org >> > Subject: [ros-diffs] [ekohl] 51558: [EVENTVWR/EVENTLOG] Fix a double >> off-by-one bug: - The eventlog service was reporting one event more than was >> available (+1). - The event viewer did not display the latest event from... >> > >> > Author: ekohl >> > Date: Mon May 2 19:38:23 2011 >> > New Revision: 51558 >> > >> > URL: http://svn.reactos.org/svn/reactos?rev=51558&view=rev >> > Log: >> > [EVENTVWR/EVENTLOG] >> > Fix a double off-by-one bug: >> > - The eventlog service was reporting one event more than was available >> (+1). >> > - The event viewer did not display the latest event from the eventlog >> service (-1). >> > >> > See issue #6182 for more details. >> > >> > Modified: >> > trunk/reactos/base/applications/mscutils/eventvwr/eventvwr.c >> > trunk/reactos/base/services/eventlog/rpc.c >> > >> > Modified: trunk/reactos/base/applications/mscutils/eventvwr/eventvwr.c >> > URL: >> http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/mscutils… >> > >> ============================================================================== >> > --- trunk/reactos/base/applications/mscutils/eventvwr/eventvwr.c >> [iso-8859-1] (original) >> > +++ trunk/reactos/base/applications/mscutils/eventvwr/eventvwr.c >> [iso-8859-1] Mon May 2 19:38:23 2011 >> > @@ -503,7 +503,7 @@ >> > HWND hwndDlg; >> > HANDLE hEventLog; >> > EVENTLOGRECORD *pevlr; >> > - DWORD dwRead, dwNeeded, dwThisRecord, dwTotalRecords = 0, >> dwCurrentRecord = 1, dwRecordsToRead = 0, dwFlags, dwMaxLength; >> > + DWORD dwRead, dwNeeded, dwThisRecord, dwTotalRecords = 0, >> dwCurrentRecord = 0, dwRecordsToRead = 0, dwFlags, dwMaxLength; >> > LPWSTR lpSourceName; >> > LPWSTR lpComputerName; >> > LPSTR lpData; >> > >> > Modified: trunk/reactos/base/services/eventlog/rpc.c >> > URL: >> http://svn.reactos.org/svn/reactos/trunk/reactos/base/services/eventlog/rpc… >> > >> ============================================================================== >> > --- trunk/reactos/base/services/eventlog/rpc.c [iso-8859-1] (original) >> > +++ trunk/reactos/base/services/eventlog/rpc.c [iso-8859-1] Mon May 2 >> 19:38:23 2011 >> > @@ -199,6 +199,7 @@ >> > DWORD *NumberOfRecords) >> > { >> > PLOGHANDLE lpLogHandle; >> > + DWORD dwRecords; >> > >> > lpLogHandle = ElfGetLogHandleEntryByHandle(LogHandle); >> > if (!lpLogHandle) >> > @@ -206,7 +207,9 @@ >> > return STATUS_INVALID_HANDLE; >> > } >> > >> > - *NumberOfRecords = lpLogHandle->LogFile->Header.CurrentRecordNumber; >> > + dwRecords = lpLogHandle->LogFile->Header.CurrentRecordNumber; >> > + >> > + *NumberOfRecords = (dwRecords > 0) ? (dwRecords - 1) : 0; >> > >> > return STATUS_SUCCESS; >> > } >> > >> > >> > > > > ---------- Forwarded message ---------- > From: Love Nystrom <love.nystrom(a)gmail.com> > To: ros-dev(a)reactos.org > Date: Tue, 3 May 2011 22:35:30 +0700 > Subject: Re: [ros-dev] [freeldr] Add PXE "filesystem" > > Hi Hervé, > > No disrespect for your work, but I'm inclined to concur with Timo regrading > AT&T vs. Intel. > > Even though I usually preach "don't fix it if it isn't broken', I really > think that AT&T syntax > is broken by design, and we should use only Intel assembly, which is *much* > clearer. > After all, we're programming Intel CPU's, not AT&T telephone switchboards > ;) > > 1) AT&T does not conform with Intel instruction reference. > Intel syntax conforms with the instruction reference. > > 2) AT&T has a lot of quaint obscure directive voodoo. > What little directives exist in Intel is clear and concise. > > Eventually, I hope we'll get rid of all backwards & muddy AT&T assembly, > but until then I think we should at least not use it for new code. > > Best Regards > L. > > > ---------- Forwarded message ---------- >> From: "Hervé Poussineau" <hpoussin(a)reactos.org> >> To: ReactOS Development List <ros-dev(a)reactos.org> >> Date: Mon, 02 May 2011 19:33:19 +0200 >> Subject: Re: [ros-dev] [ros-diffs] [hpoussin] 51517: [freeldr] Add PXE >> "filesystem" >> Hi, >> >> The file is in AT&T syntax, as all asm files in >> reactos/boot/freeldr/freeldr/arch/i386/ , except fathelp.asm >> Sorry, I won't "fix" it, so you'll have to find someone to do it. >> >> Regards, >> >> Hervé >> >> Timo Kreuzer a écrit : >> >>> >>> Why did you write the asm code in AT&T syntax? Its both ugly and breaks >>> compilation with MSVC. >>> I've spent quite some time converting all asm code to MSVC friendly intel >>> syntax. >>> Could you please fix that? >>> >>> Regards, >>> Timo >>> >>> Am 01.05.2011 10:11, schrieb hpoussin(a)svn.reactos.org: >>> >>>> Author: hpoussin >>>> Date: Sun May 1 08:11:43 2011 >>>> New Revision: 51517 >>>> >>>> URL: http://svn.reactos.org/svn/reactos?rev=51517&view=rev >>>> Log: >>>> [freeldr] Add PXE "filesystem" >>>> >>>> Added: >>>> trunk/reactos/boot/freeldr/freeldr/arch/i386/i386pxe.S (with >>>> props) >>>> trunk/reactos/boot/freeldr/freeldr/fs/pxe.c (with props) >>>> trunk/reactos/boot/freeldr/freeldr/include/arch/i386/pxe.h (with >>>> props) >>>> trunk/reactos/boot/freeldr/freeldr/include/fs/pxe.h (with props) >>>> Modified: >>>> trunk/reactos/boot/freeldr/freeldr/arch/i386/machpc.c >>>> trunk/reactos/boot/freeldr/freeldr/arch/i386/pcdisk.c >>>> trunk/reactos/boot/freeldr/freeldr/freeldr_base.rbuild >>>> trunk/reactos/boot/freeldr/freeldr/freeldr_base64k.rbuild >>>> trunk/reactos/boot/freeldr/freeldr/fs/fs.c >>>> trunk/reactos/boot/freeldr/freeldr/include/arch/i386/hardware.h >>>> trunk/reactos/boot/freeldr/freeldr/include/arch/i386/machpc.h >>>> trunk/reactos/boot/freeldr/freeldr/include/freeldr.h >>>> >>>> Added: trunk/reactos/boot/freeldr/freeldr/arch/i386/i386pxe.S >>>> URL: >>>> http://svn.reactos.org/svn/reactos/trunk/reactos/boot/freeldr/freeldr/arch/… >>>> ============================================================================== >>>> >>>> --- trunk/reactos/boot/freeldr/freeldr/arch/i386/i386pxe.S (added) >>>> +++ trunk/reactos/boot/freeldr/freeldr/arch/i386/i386pxe.S [iso-8859-1] >>>> Sun May 1 08:11:43 2011 >>>> @@ -1,0 +1,97 @@ >>>> +/* >>>> + * FreeLoader >>>> + * Copyright (C) 2011 Hervé Poussineau >>>> + * >>>> + * This program is free software; you can redistribute it and/or >>>> modify >>>> + * it under the terms of the GNU General Public License as published >>>> by >>>> + * the Free Software Foundation; either version 2 of the License, or >>>> + * (at your option) any later version. >>>> + * >>>> + * This program is distributed in the hope that it will be useful, >>>> + * but WITHOUT ANY WARRANTY; without even the implied warranty of >>>> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the >>>> + * GNU General Public License for more details. >>>> + * >>>> + * You should have received a copy of the GNU General Public License >>>> along >>>> + * with this program; if not, write to the Free Software Foundation, >>>> Inc., >>>> + * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA. >>>> + */ >>>> + >>>> + .text >>>> + .code16 >>>> + >>>> +#define ASM >>>> + >>>> +#include<arch.h> >>>> + >>>> +/* >>>> + * U16 PxeCallApi(U16 Segment, U16 Offset, U16 Service, VOID >>>> *Parameter); >>>> + * >>>> + * RETURNS: >>>> + */ >>>> +_pxe_function: >>>> + .word 0 >>>> +_pxe_entry_point: >>>> + .long 0 >>>> +_pxe_buffer_segment: >>>> + .word0 >>>> +_pxe_buffer_offset: >>>> + .word0 >>>> +_pxe_result: >>>> + .word 0 >>>> + >>>> +EXTERN(_PxeCallApi) >>>> + .code32 >>>> + pushl %ebp >>>> + movl %esp,%ebp >>>> + >>>> + >>>> >>> > > > ---------- Forwarded message ---------- > From: "Ged Murphy" <gedmurphy(a)gmail.com> > To: "'ReactOS Development List'" <ros-dev(a)reactos.org> > Date: Tue, 3 May 2011 17:12:03 +0100 > Subject: Re: [ros-dev] [freeldr] Add PXE "filesystem" > > Did you read Herve’s email?? > > Are you volunteering to convert all the files he mentioned into intel > syntax? > > > > > > *From:* ros-dev-bounces(a)reactos.org [mailto:ros-dev-bounces@reactos.org] *On > Behalf Of *Love Nystrom > *Sent:* 03 May 2011 16:36 > *To:* ros-dev(a)reactos.org > *Subject:* Re: [ros-dev] [freeldr] Add PXE "filesystem" > > > > > > Hi Hervé, > > > > No disrespect for your work, but I'm inclined to concur with Timo regrading > AT&T vs. Intel. > > > > Even though I usually preach "don't fix it if it isn't broken', I really > think that AT&T syntax > > is broken by design, and we should use only Intel assembly, which is *much* > clearer. > > After all, we're programming Intel CPU's, not AT&T telephone switchboards > ;) > > > > 1) AT&T does not conform with Intel instruction reference. > > Intel syntax conforms with the instruction reference. > > > > 2) AT&T has a lot of quaint obscure directive voodoo. > > What little directives exist in Intel is clear and concise. > > > > Eventually, I hope we'll get rid of all backwards & muddy AT&T assembly, > > but until then I think we should at least not use it for new code. > > > > Best Regards > > L. > > > > > > ---------- Forwarded message ---------- > From: "Hervé Poussineau" <hpoussin(a)reactos.org> > To: ReactOS Development List <ros-dev(a)reactos.org> > Date: Mon, 02 May 2011 19:33:19 +0200 > Subject: Re: [ros-dev] [ros-diffs] [hpoussin] 51517: [freeldr] Add PXE > "filesystem" > Hi, > > The file is in AT&T syntax, as all asm files in > reactos/boot/freeldr/freeldr/arch/i386/ , except fathelp.asm > Sorry, I won't "fix" it, so you'll have to find someone to do it. > > Regards, > > Hervé > > Timo Kreuzer a écrit : > > > Why did you write the asm code in AT&T syntax? Its both ugly and breaks > compilation with MSVC. > I've spent quite some time converting all asm code to MSVC friendly intel > syntax. > Could you please fix that? > > Regards, > Timo > > Am 01.05.2011 10:11, schrieb hpoussin(a)svn.reactos.org: > > Author: hpoussin > Date: Sun May 1 08:11:43 2011 > New Revision: 51517 > > URL: http://svn.reactos.org/svn/reactos?rev=51517&view=rev > Log: > [freeldr] Add PXE "filesystem" > > Added: > trunk/reactos/boot/freeldr/freeldr/arch/i386/i386pxe.S (with props) > trunk/reactos/boot/freeldr/freeldr/fs/pxe.c (with props) > trunk/reactos/boot/freeldr/freeldr/include/arch/i386/pxe.h (with > props) > trunk/reactos/boot/freeldr/freeldr/include/fs/pxe.h (with props) > Modified: > trunk/reactos/boot/freeldr/freeldr/arch/i386/machpc.c > trunk/reactos/boot/freeldr/freeldr/arch/i386/pcdisk.c > trunk/reactos/boot/freeldr/freeldr/freeldr_base.rbuild > trunk/reactos/boot/freeldr/freeldr/freeldr_base64k.rbuild > trunk/reactos/boot/freeldr/freeldr/fs/fs.c > trunk/reactos/boot/freeldr/freeldr/include/arch/i386/hardware.h > trunk/reactos/boot/freeldr/freeldr/include/arch/i386/machpc.h > trunk/reactos/boot/freeldr/freeldr/include/freeldr.h > > Added: trunk/reactos/boot/freeldr/freeldr/arch/i386/i386pxe.S > URL: > http://svn.reactos.org/svn/reactos/trunk/reactos/boot/freeldr/freeldr/arch/… > ============================================================================== > > --- trunk/reactos/boot/freeldr/freeldr/arch/i386/i386pxe.S (added) > +++ trunk/reactos/boot/freeldr/freeldr/arch/i386/i386pxe.S [iso-8859-1] Sun > May 1 08:11:43 2011 > @@ -1,0 +1,97 @@ > +/* > + * FreeLoader > + * Copyright (C) 2011 Hervé Poussineau > + * > + * This program is free software; you can redistribute it and/or modify > + * it under the terms of the GNU General Public License as published by > + * the Free Software Foundation; either version 2 of the License, or > + * (at your option) any later version. > + * > + * This program is distributed in the hope that it will be useful, > + * but WITHOUT ANY WARRANTY; without even the implied warranty of > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the > + * GNU General Public License for more details. > + * > + * You should have received a copy of the GNU General Public License > along > + * with this program; if not, write to the Free Software Foundation, > Inc., > + * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA. > + */ > + > + .text > + .code16 > + > +#define ASM > + > +#include<arch.h> > + > +/* > + * U16 PxeCallApi(U16 Segment, U16 Offset, U16 Service, VOID *Parameter); > + * > + * RETURNS: > + */ > +_pxe_function: > + .word 0 > +_pxe_entry_point: > + .long 0 > +_pxe_buffer_segment: > + .word0 > +_pxe_buffer_offset: > + .word0 > +_pxe_result: > + .word 0 > + > +EXTERN(_PxeCallApi) > + .code32 > + pushl %ebp > + movl %esp,%ebp > + > + > > > > _______________________________________________ > Ros-dev mailing list > Ros-dev(a)reactos.org > http://www.reactos.org/mailman/listinfo/ros-dev >

3 2

Re: [ros-dev] [freeldr] Add PXE "filesystem"
by Love Nystrom 04 May '11

04 May '11

Hi Hervé, No disrespect for your work, but I'm inclined to concur with Timo regrading AT&T vs. Intel. Even though I usually preach "don't fix it if it isn't broken', I really think that AT&T syntax is broken by design, and we should use only Intel assembly, which is *much* clearer. After all, we're programming Intel CPU's, not AT&T telephone switchboards ;) 1) AT&T does not conform with Intel instruction reference. Intel syntax conforms with the instruction reference. 2) AT&T has a lot of quaint obscure directive voodoo. What little directives exist in Intel is clear and concise. Eventually, I hope we'll get rid of all backwards & muddy AT&T assembly, but until then I think we should at least not use it for new code. Best Regards L. ---------- Forwarded message ---------- > From: "Hervé Poussineau" <hpoussin(a)reactos.org> > To: ReactOS Development List <ros-dev(a)reactos.org> > Date: Mon, 02 May 2011 19:33:19 +0200 > Subject: Re: [ros-dev] [ros-diffs] [hpoussin] 51517: [freeldr] Add PXE > "filesystem" > Hi, > > The file is in AT&T syntax, as all asm files in > reactos/boot/freeldr/freeldr/arch/i386/ , except fathelp.asm > Sorry, I won't "fix" it, so you'll have to find someone to do it. > > Regards, > > Hervé > > Timo Kreuzer a écrit : > >> >> Why did you write the asm code in AT&T syntax? Its both ugly and breaks >> compilation with MSVC. >> I've spent quite some time converting all asm code to MSVC friendly intel >> syntax. >> Could you please fix that? >> >> Regards, >> Timo >> >> Am 01.05.2011 10:11, schrieb hpoussin(a)svn.reactos.org: >> >>> Author: hpoussin >>> Date: Sun May 1 08:11:43 2011 >>> New Revision: 51517 >>> >>> URL: http://svn.reactos.org/svn/reactos?rev=51517&view=rev >>> Log: >>> [freeldr] Add PXE "filesystem" >>> >>> Added: >>> trunk/reactos/boot/freeldr/freeldr/arch/i386/i386pxe.S (with props) >>> trunk/reactos/boot/freeldr/freeldr/fs/pxe.c (with props) >>> trunk/reactos/boot/freeldr/freeldr/include/arch/i386/pxe.h (with >>> props) >>> trunk/reactos/boot/freeldr/freeldr/include/fs/pxe.h (with props) >>> Modified: >>> trunk/reactos/boot/freeldr/freeldr/arch/i386/machpc.c >>> trunk/reactos/boot/freeldr/freeldr/arch/i386/pcdisk.c >>> trunk/reactos/boot/freeldr/freeldr/freeldr_base.rbuild >>> trunk/reactos/boot/freeldr/freeldr/freeldr_base64k.rbuild >>> trunk/reactos/boot/freeldr/freeldr/fs/fs.c >>> trunk/reactos/boot/freeldr/freeldr/include/arch/i386/hardware.h >>> trunk/reactos/boot/freeldr/freeldr/include/arch/i386/machpc.h >>> trunk/reactos/boot/freeldr/freeldr/include/freeldr.h >>> >>> Added: trunk/reactos/boot/freeldr/freeldr/arch/i386/i386pxe.S >>> URL: >>> http://svn.reactos.org/svn/reactos/trunk/reactos/boot/freeldr/freeldr/arch/… >>> ============================================================================== >>> >>> --- trunk/reactos/boot/freeldr/freeldr/arch/i386/i386pxe.S (added) >>> +++ trunk/reactos/boot/freeldr/freeldr/arch/i386/i386pxe.S [iso-8859-1] >>> Sun May 1 08:11:43 2011 >>> @@ -1,0 +1,97 @@ >>> +/* >>> + * FreeLoader >>> + * Copyright (C) 2011 Hervé Poussineau >>> + * >>> + * This program is free software; you can redistribute it and/or modify >>> + * it under the terms of the GNU General Public License as published by >>> + * the Free Software Foundation; either version 2 of the License, or >>> + * (at your option) any later version. >>> + * >>> + * This program is distributed in the hope that it will be useful, >>> + * but WITHOUT ANY WARRANTY; without even the implied warranty of >>> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the >>> + * GNU General Public License for more details. >>> + * >>> + * You should have received a copy of the GNU General Public License >>> along >>> + * with this program; if not, write to the Free Software Foundation, >>> Inc., >>> + * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA. >>> + */ >>> + >>> + .text >>> + .code16 >>> + >>> +#define ASM >>> + >>> +#include<arch.h> >>> + >>> +/* >>> + * U16 PxeCallApi(U16 Segment, U16 Offset, U16 Service, VOID >>> *Parameter); >>> + * >>> + * RETURNS: >>> + */ >>> +_pxe_function: >>> + .word 0 >>> +_pxe_entry_point: >>> + .long 0 >>> +_pxe_buffer_segment: >>> + .word0 >>> +_pxe_buffer_offset: >>> + .word0 >>> +_pxe_result: >>> + .word 0 >>> + >>> +EXTERN(_PxeCallApi) >>> + .code32 >>> + pushl %ebp >>> + movl %esp,%ebp >>> + >>> + >>> >>

3 2

Re: [ros-dev] [ros-diffs] [ekohl] 51558: [EVENTVWR/EVENTLOG] Fix a double off-by-one bug: - The eventlog service was reporting one event more than was available (+1). - The event viewer did not display the latest event from...
by Gabriel ilardi 02 May '11

02 May '11

Great work on event service and event viewer... keep it up Eric :) > Date: Mon, 2 May 2011 19:38:24 +0000 > To: ros-diffs(a)reactos.org > From: ekohl(a)svn.reactos.org > Subject: [ros-diffs] [ekohl] 51558: [EVENTVWR/EVENTLOG] Fix a double off-by-one bug: - The eventlog service was reporting one event more than was available (+1). - The event viewer did not display the latest event from... > > Author: ekohl > Date: Mon May 2 19:38:23 2011 > New Revision: 51558 > > URL: http://svn.reactos.org/svn/reactos?rev=51558&view=rev > Log: > [EVENTVWR/EVENTLOG] > Fix a double off-by-one bug: > - The eventlog service was reporting one event more than was available (+1). > - The event viewer did not display the latest event from the eventlog service (-1). > > See issue #6182 for more details. > > Modified: > trunk/reactos/base/applications/mscutils/eventvwr/eventvwr.c > trunk/reactos/base/services/eventlog/rpc.c > > Modified: trunk/reactos/base/applications/mscutils/eventvwr/eventvwr.c > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/mscutils… > ============================================================================== > --- trunk/reactos/base/applications/mscutils/eventvwr/eventvwr.c [iso-8859-1] (original) > +++ trunk/reactos/base/applications/mscutils/eventvwr/eventvwr.c [iso-8859-1] Mon May 2 19:38:23 2011 > @@ -503,7 +503,7 @@ > HWND hwndDlg; > HANDLE hEventLog; > EVENTLOGRECORD *pevlr; > - DWORD dwRead, dwNeeded, dwThisRecord, dwTotalRecords = 0, dwCurrentRecord = 1, dwRecordsToRead = 0, dwFlags, dwMaxLength; > + DWORD dwRead, dwNeeded, dwThisRecord, dwTotalRecords = 0, dwCurrentRecord = 0, dwRecordsToRead = 0, dwFlags, dwMaxLength; > LPWSTR lpSourceName; > LPWSTR lpComputerName; > LPSTR lpData; > > Modified: trunk/reactos/base/services/eventlog/rpc.c > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/services/eventlog/rpc… > ============================================================================== > --- trunk/reactos/base/services/eventlog/rpc.c [iso-8859-1] (original) > +++ trunk/reactos/base/services/eventlog/rpc.c [iso-8859-1] Mon May 2 19:38:23 2011 > @@ -199,6 +199,7 @@ > DWORD *NumberOfRecords) > { > PLOGHANDLE lpLogHandle; > + DWORD dwRecords; > > lpLogHandle = ElfGetLogHandleEntryByHandle(LogHandle); > if (!lpLogHandle) > @@ -206,7 +207,9 @@ > return STATUS_INVALID_HANDLE; > } > > - *NumberOfRecords = lpLogHandle->LogFile->Header.CurrentRecordNumber; > + dwRecords = lpLogHandle->LogFile->Header.CurrentRecordNumber; > + > + *NumberOfRecords = (dwRecords > 0) ? (dwRecords - 1) : 0; > > return STATUS_SUCCESS; > } > >

1 0

Re: [ros-dev] [ros-diffs] [hpoussin] 51517: [freeldr] Add PXE "filesystem"
by Timo Kreuzer 02 May '11

02 May '11

Why did you write the asm code in AT&T syntax? Its both ugly and breaks compilation with MSVC. I've spent quite some time converting all asm code to MSVC friendly intel syntax. Could you please fix that? Regards, Timo Am 01.05.2011 10:11, schrieb hpoussin(a)svn.reactos.org: > Author: hpoussin > Date: Sun May 1 08:11:43 2011 > New Revision: 51517 > > URL: http://svn.reactos.org/svn/reactos?rev=51517&view=rev > Log: > [freeldr] Add PXE "filesystem" > > Added: > trunk/reactos/boot/freeldr/freeldr/arch/i386/i386pxe.S (with props) > trunk/reactos/boot/freeldr/freeldr/fs/pxe.c (with props) > trunk/reactos/boot/freeldr/freeldr/include/arch/i386/pxe.h (with props) > trunk/reactos/boot/freeldr/freeldr/include/fs/pxe.h (with props) > Modified: > trunk/reactos/boot/freeldr/freeldr/arch/i386/machpc.c > trunk/reactos/boot/freeldr/freeldr/arch/i386/pcdisk.c > trunk/reactos/boot/freeldr/freeldr/freeldr_base.rbuild > trunk/reactos/boot/freeldr/freeldr/freeldr_base64k.rbuild > trunk/reactos/boot/freeldr/freeldr/fs/fs.c > trunk/reactos/boot/freeldr/freeldr/include/arch/i386/hardware.h > trunk/reactos/boot/freeldr/freeldr/include/arch/i386/machpc.h > trunk/reactos/boot/freeldr/freeldr/include/freeldr.h > > Added: trunk/reactos/boot/freeldr/freeldr/arch/i386/i386pxe.S > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/boot/freeldr/freeldr/arch/… > ============================================================================== > --- trunk/reactos/boot/freeldr/freeldr/arch/i386/i386pxe.S (added) > +++ trunk/reactos/boot/freeldr/freeldr/arch/i386/i386pxe.S [iso-8859-1] Sun May 1 08:11:43 2011 > @@ -1,0 +1,97 @@ > +/* > + * FreeLoader > + * Copyright (C) 2011 Hervé Poussineau > + * > + * This program is free software; you can redistribute it and/or modify > + * it under the terms of the GNU General Public License as published by > + * the Free Software Foundation; either version 2 of the License, or > + * (at your option) any later version. > + * > + * This program is distributed in the hope that it will be useful, > + * but WITHOUT ANY WARRANTY; without even the implied warranty of > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the > + * GNU General Public License for more details. > + * > + * You should have received a copy of the GNU General Public License along > + * with this program; if not, write to the Free Software Foundation, Inc., > + * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA. > + */ > + > + .text > + .code16 > + > +#define ASM > + > +#include<arch.h> > + > +/* > + * U16 PxeCallApi(U16 Segment, U16 Offset, U16 Service, VOID *Parameter); > + * > + * RETURNS: > + */ > +_pxe_function: > + .word 0 > +_pxe_entry_point: > + .long 0 > +_pxe_buffer_segment: > + .word0 > +_pxe_buffer_offset: > + .word0 > +_pxe_result: > + .word 0 > + > +EXTERN(_PxeCallApi) > + .code32 > + pushl %ebp > + movl %esp,%ebp > + > +

2 1

Re: [ros-dev] [ros-diffs] [rharabien] 51510: [COMCTL32] * Properly support WM_SYSCOLORCHANGE in ListView controls with default background. Should be sent to Wine (tm)
by Ged Murphy 01 May '11

01 May '11

There's little point in (tm)ing these wine changes, if they don't get sent to wine or properly documented, then will definitely be overwritten by the next wine sync. On 30 April 2011 22:41, <rharabien(a)svn.reactos.org> wrote: > Author: rharabien > Date: Sat Apr 30 21:41:15 2011 > New Revision: 51510 > > URL: http://svn.reactos.org/svn/reactos?rev=51510&view=rev > Log: > [COMCTL32] > * Properly support WM_SYSCOLORCHANGE in ListView controls with default background. Should be sent to Wine (tm) > > Modified: > trunk/reactos/dll/win32/comctl32/listview.c > > Modified: trunk/reactos/dll/win32/comctl32/listview.c > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/comctl32/listvie… > ============================================================================== > --- trunk/reactos/dll/win32/comctl32/listview.c [iso-8859-1] (original) > +++ trunk/reactos/dll/win32/comctl32/listview.c [iso-8859-1] Sat Apr 30 21:41:15 2011 > @@ -315,6 +315,7 @@ > COLORREF clrBk; > COLORREF clrText; > COLORREF clrTextBk; > + BOOL bDefaultBkColor; > > /* font */ > HFONT hDefaultFont; > @@ -7870,7 +7871,8 @@ > static BOOL LISTVIEW_SetBkColor(LISTVIEW_INFO *infoPtr, COLORREF clrBk) > { > TRACE("(clrBk=%x)\n", clrBk); > - > + > + infoPtr->bDefaultBkColor = FALSE; > if(infoPtr->clrBk != clrBk) { > if (infoPtr->clrBk != CLR_NONE) DeleteObject(infoPtr->hBkBrush); > infoPtr->clrBk = clrBk; > @@ -9263,6 +9265,7 @@ > infoPtr->clrText = CLR_DEFAULT; > infoPtr->clrTextBk = CLR_DEFAULT; > LISTVIEW_SetBkColor(infoPtr, comctl32_color.clrWindow); > + infoPtr->bDefaultBkColor = TRUE; > > /* set default values */ > infoPtr->nFocusedItem = -1; > @@ -11525,6 +11528,11 @@ > > case WM_SYSCOLORCHANGE: > COMCTL32_RefreshSysColors(); > + if (infoPtr->bDefaultBkColor) > + { > + LISTVIEW_SetBkColor(infoPtr, comctl32_color.clrWindow); > + infoPtr->bDefaultBkColor = TRUE; > + } > return 0; > > /* case WM_TIMER: */ > > >

1 0

Fwd: [NOTEPAD] Patch by Edijs Kolesnikovics: Write default font, weight and point...
by Jérôme Gardou 29 Apr '11

29 Apr '11

SaveDword(hKey, _T("iPointSize"), Globals.lfFont.lfHeight); shouldn't it be SaveDword(hKey, _T("iPointSize"), PointSizeFromHeight(Globals.lfFont.lfHeight)); Also, this would permit to simplify code a bit :-) http://cia.vc/stats/project/ReactOS/.message/3522f92

1 0

Regular meeting
by Aleksey Bragin 29 Apr '11

29 Apr '11

Hello, I want to remind you of the regular meeting, Thursday the 28th at 20:00 UTC at freenode #reactos-meeting. The suggested meeting agenda is: - Status of our GSoC participation - Current ReactOS work. Developers saying what they are working on - Action List proposal - Status of the upcoming LinuxTag event preparations - Website revamp. Current status, who is working, who is willing to help If nobody else volunteers, Amine would be ready to take the minute taker position again in this meeting and I would lead the meeting. In case I am late, Amine will open the up meeting and I will join as soon as I get online. The current list of ReactOS members is as follows. Only these people will be able to participate in the discussions and votings, please tell us if we have forgotten anybody or if you want to be added to this list: - Giannis Adamopoulos (smiley1_) - Johannes Anderwald (janderwald) - Javier Agustìn Fernàndez Arroyo (elhoir) - Maciej Bialas (niski) - Aleksey Bragin (abragin) - Colin Finck (Colin_Finck) - Danny Götte (dangerground) - Cameron Gutman (aicom) - Ziliang Guo (ZWabbit) - Rafal Harabien (rafalh) - Kamil Hornicek (Pigglesworth) - Amine Khaldi (AmineKhaldi) - Timo Kreuzer (tkreuzer) - Matthias Kupfer (Collibri) - Michael Martin (mjmartin) - Victor Martinez (vicmarcal) - Roel Messiant (Mephisto) - Andrew Munger (WaxDragon) - Ged Murphy (GedMurphy) - Sylvain Petreolle (Usurp) - Hervé Poussineau (hpoussin) - Daniel Reimer (dreimer) - Pierre Schweitzer (HeisSpiter) - Samuel Serapion (encoded) - Olaf Siejka (Caemyr) - James Tabor (jimtabor) - Art Yerkes (arty) Everybody may join the meeting channel as a non-participating observer though. See you at the meeting today, Aleksey Bragin.

6 6

Jump to page:

2026

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

Ros-dev