Ros-dev

ros-dev@reactos.org

5789 discussions

Re: [ros-dev] [ros-diffs] [akhaldi] 67974: [USER32] Sync edit.c with Wine Staging 1.7.37. CORE-9246

by Michael Fritscher

Interesting that the wine code doesn't check for NULL at HeapAlloc at all - could these be submitted to wine? Additionally, at least one HeapAlloc isn't checked also in our build: piDx = HeapAlloc(GetProcessHeap(),0,sizeof(INT) * (*count)); Perhaps a #ifdef __REACTOS__ if (piDx == NULL) { return FALSE; //Not sure about this, just looked at the diff ;) } #endif would be usefull there. Best regards, Michael Fritscher > Author: akhaldi > Date: Sat May 30 17:14:16 2015 > New Revision: 67974 > > URL: http://svn.reactos.org/svn/reactos?rev=67974&view=rev > Log: > [USER32] Sync edit.c with Wine Staging 1.7.37. CORE-9246 > > Modified: > trunk/reactos/media/doc/README.WINE > trunk/reactos/win32ss/user/user32/controls/edit.c >

10 years, 1 month

Re: [ros-dev] [ros-diffs] [jimtabor] 67937: [NtUser] - De-reference global cursor. See CORE-8305.

by Thomas Faber

That's a bit** of a hack. Either the code holds a reference and needs to release it or not. cLockObj is a private field and code outside of UserReference* should not be allowed to access it. Thanks! -Thomas ** actually it's a pretty big hack On May 28, 2015 12:13:03 AM CEST, jimtabor(a)svn.reactos.org wrote: >Author: jimtabor >Date: Wed May 27 22:13:03 2015 >New Revision: 67937 > >URL: http://svn.reactos.org/svn/reactos?rev=67937&view=rev >Log: >[NtUser] >- De-reference global cursor. See CORE-8305. > >Modified: > trunk/reactos/win32ss/user/ntuser/cursoricon.c > >Modified: trunk/reactos/win32ss/user/ntuser/cursoricon.c >URL: >http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/user/ntuser/cursor… >============================================================================== >--- trunk/reactos/win32ss/user/ntuser/cursoricon.c [iso-8859-1] >(original) >+++ trunk/reactos/win32ss/user/ntuser/cursoricon.c [iso-8859-1] Wed May >27 22:13:03 2015 >@@ -1077,6 +1077,12 @@ > if (pcurOld->CURSORF_flags & CURSORF_GLOBAL) > { > TRACE("Returning Global Cursor hcur %p\n",hOldCursor); >+ >+ if (pcurOld->head.cLockObj > 2) // Throttle down to 2. >+ { >+ UserDereferenceObject(pcurOld); >+ } >+ > goto leave; > } > -- Sent from my Android phone with K-9 Mail. Please excuse my brevity.

10 years, 1 month

Status Meeting (May 2015)

by Aleksey Bragin

Hello, Let me invite you to the monthly status meeting taking place 28th of May, 19:00 UTC, as always. IRC service will only be started shortly before the meeting. Your participation passwords and server address will be emailed to you shortly before the meeting starts, and they are going to be different once again as they are not stored in any database. Hopefully it's not much of inconvenience. Please send agenda proposals to me before the meeting, so that we can start with a proposed agenda. Regards, Aleksey Bragin

10 years, 1 month

Re: [ros-dev] [ros-diffs] [akhaldi] 67940: [TCPIP] Identify TCP/IP driver as version 4. By Lee Schroeder. CORE-9744

by Thomas Faber

> --- trunk/reactos/drivers/network/tcpip/tcpip.rc [iso-8859-1] (original) > +++ trunk/reactos/drivers/network/tcpip/tcpip.rc [iso-8859-1] Thu May 28 10:31:38 2015 > @@ -1,5 +1,5 @@ > #define REACTOS_VERSION_DLL > -#define REACTOS_STR_FILE_DESCRIPTION "TCP/IP protocol driver" > +#define REACTOS_STR_FILE_DESCRIPTION "TCP/IPv4 protocol driver" > #define REACTOS_STR_INTERNAL_NAME "tcpip" > #define REACTOS_STR_ORIGINAL_FILENAME "tcpip.sys" > #include <reactos/version.rc> This part makes no sense. tcpip.sys does both v4 and v6

10 years, 1 month

ReactOS Updates flowing to our Social Channels.

by victor martinez

Hi guys! After a week dealing with several scripts and options, we're finally integrating the most important info from Reactos.org and Community.reactos.org directly to our Social channels to keep a constant flow of updates to our followers. Since now our ReactOS Facebook and ReactOS Twitter will be automatically updated when... ... A Jira bugreport is resolved with a 'Fixed' and 'Cannot reproduce' status. ... A blog post is published in "ReactOS.org" ... A blog post is published in "Community.ReactOS.org" Some of the features: - Updates are queued and published with a time gap between them, so in case, eg, a Tester/Dev runs a Spring Cleanup, we won't be publishing 10 Jira reports in the same minute. - Facebook and Twitter updates include pictures in case the blog post or the Jira report has one. So please add a picture in case you're blogging even if it is just a screenshot of you running "whatever" in ReactOS. - Includes #hashtags related to the current Jira/Blog report. - Uses bit.ly shortener when needed. - And several hidden Easter Eggs in Twitter...(I was bored in the train) If you've any suggestions about other potential updates to be sent to our Social Channels, just tell me and I'd try to integrate them in an automated way. I'm planning a way to ease and have community contributions filtered and joining the automated flow, but it'll have to wait until I master Drupal. Also I'm working in some "crossposting" among our Social Channels, so a direct post in Facebook not coming from our PR Update System is replicated in Twitter and the other way around. We can be really proud of the current PR Update system implemented, since probably it even beats several bigger opensource projects and/or companies out there. If you didn't join our Twitter or Facebook yet, please feel free to join: - Facebook: https://www.facebook.com/pages/ReactOS/19143619259 - Twitter: http://www.twitter.com/reactos Brought to you by the ReactOS PR Team.

10 years, 1 month

Re: [ros-dev] [ros-diffs] [tkreuzer] 67718: [CRT] Add simple C implementations for acosf, ceilf, floorf, fmodf and fix sqrtf

by Thomas Faber

On 2015-05-14 15:44, tkreuzer(a)svn.reactos.org wrote: > +_Check_return_ > +float > +__cdecl > +ceilf( > +_In_ float x) > +{ > + return (float)ceilf((double)x); > +} > +_Check_return_ > +float > +__cdecl > +floorf( > +_In_ float x) > +{ > + return (float)floorf((double)x); > +} Did you mean to call ceil/floor here?

10 years, 1 month

Re: [ros-dev] [ros-diffs] [tkreuzer] 67709: [D3DX9_36/WINED3D] - Remove useless copysignf invocation and the wrong defines for it - BrokenHacksOnTopOfBrokenCode--; - Should be sent upstream

by Thomas Faber

On 2015-05-14 06:00, tkreuzer(a)svn.reactos.org wrote: > - int sign = (copysignf(1, in) < 0); > + int sign = (in < 0); > - if (copysignf(1.0f, value) < 0.0f) > + if (value < 0.0f) > ++idx; I believe the behavior would be different here for negative zero: copysignf(1.0f, -0.0f) should be < 0.0f -0.0f should be == 0.0f Maybe that's the reason for having these calls?

10 years, 1 month

Re: [ros-dev] [ros-diffs] [akhaldi] 67689: [CMAKE] The /SUBSYSTEM flag takes uppercase parameters.

by Timo Kreuzer

Can you please also add an approproate value for ARM (I guess win8)? :) Am 12.05.2015 um 14:18 schrieb akhaldi(a)svn.reactos.org: > Author: akhaldi > Date: Tue May 12 12:18:02 2015 > New Revision: 67689 > > URL: http://svn.reactos.org/svn/reactos?rev=67689&view=rev > Log: > [CMAKE] The /SUBSYSTEM flag takes uppercase parameters. > > Modified: > trunk/reactos/cmake/msvc.cmake > > Modified: trunk/reactos/cmake/msvc.cmake > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/cmake/msvc.cmake?rev=67689… > ============================================================================== > --- trunk/reactos/cmake/msvc.cmake [iso-8859-1] (original) > +++ trunk/reactos/cmake/msvc.cmake [iso-8859-1] Tue May 12 12:18:02 2015 > @@ -210,10 +210,11 @@ > endfunction() > > function(set_subsystem MODULE SUBSYSTEM) > + string(TOUPPER ${SUBSYSTEM} _subsystem) > if(ARCH STREQUAL "amd64") > - add_target_link_flags(${MODULE} "/SUBSYSTEM:${SUBSYSTEM},5.02") > - else() > - add_target_link_flags(${MODULE} "/SUBSYSTEM:${SUBSYSTEM},5.01") > + add_target_link_flags(${MODULE} "/SUBSYSTEM:${_subsystem},5.02") > + else() > + add_target_link_flags(${MODULE} "/SUBSYSTEM:${_subsystem},5.01") > endif() > endfunction() > > > >

10 years, 1 month

Re: [ros-dev] [ros-diffs] [akhaldi] 67683: [SHELL32] Don't provide the format option when the volume is read only. By Barrett Karish. CORE-9530

by Thomas Faber

On 2015-05-12 06:57, akhaldi(a)svn.reactos.org wrote: > +/* FIXME: ntifs.h */ > +#define FILE_READ_ONLY_VOLUME 0x00080000 This should go in ndk/iotypes.h inside an NTOS_MODE_USER ifdef.

10 years, 1 month

Fezile motherboard has finally failed

by Colin Finck

Hi all, Today, the motherboard in our Fezile server has finally failed after many hick-ups over the year. This causes an outage for the following services: * doxygen.reactos.org * cppcheck.reactos.org * VMware Player Test slave * VMware Player Patchbot As this is not the first time we struggle with such problems, iso.reactos.org is routed to a different server right now and not affected. We have already ordered replacement hardware for this system and hope to be able to get it back to a working state soon. Even if this machine is from 2007, we have decided to revive it with replacement parts instead of going for a brand-new one as this is much cheaper and quicker to realize. Also there is a huge pile of replacement hardware available. Sorry for the inconveniences! Cheers, Colin

10 years, 1 month

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

Ros-dev