Thanks for the information !
And I’ve checked where we introduced those try_return(S) without nothing in S, and guess what I’ve found:
http://svn.reactos.org/svn/reactos?view=revision&revision=38302
AAAArrrrggghhhh !!!!
Regards,
Hermès
De : ros-dev-bounces@reactos.org [mailto:ros-dev-bounces@reactos.org] De la part de Sylvain Petreolle
Envoyé : mercredi 26 mars 2014 01:00
À : ReactOS Development List
Objet : Re: [ros-dev] [ros-diffs] [hbelusca] 62562: [EXT2] try_return() == try_return(S) with nothing in S . The code sometimes use it. Shut up MSVC warning C4003: not enough actual parameters for macro 'try_return'.
I checked the original source from http://sourceforge.net/projects/winext2fsd/.
try_return(S) with nothing in S are ours exclusively. those are to be fixed instead.
Kind regards,
Sylvain Petreolle
De : Thomas Faber <thomas.faber@reactos.org>
À : ReactOS Development List <ros-dev@reactos.org>
Envoyé le : Mardi 25 mars 2014 10h44
Objet : Re: [ros-dev] [ros-diffs] [hbelusca] 62562: [EXT2] try_return() == try_return(S) with nothing in S . The code sometimes use it. Shut up MSVC warning C4003: not enough actual parameters for macro 'try_return'.
I took the warning form the GCC builder.
Looks like it should be C4005 on MSVC though.
Shouldn't you have seen that when you built it? ;)
Hmm, also, ext2 has allow_warnings for some reason (which is why this
even builds). We should really remove them all, only ntdll_winetest
should need one at this point.
On 2014-03-25 02:20, Hermès BÉLUSCA - MAÏTO wrote:
> Hmmm also, is there any Cxxxx error code associated with that, on MSVC ?
>
> -----Message d'origine-----
> De : ros-dev-bounces@reactos.org [mailto:ros-dev-bounces@reactos.org] De la
> part de Thomas Faber
> Envoyé : lundi 24 mars 2014 23:29
> À : ros-dev@reactos.org
> Objet : Re: [ros-dev] [ros-diffs] [hbelusca] 62562: [EXT2] try_return() ==
> try_return(S) with nothing in S . The code sometimes use it. Shut up MSVC
> warning C4003: not enough actual parameters for macro 'try_return'.
>
> That's illegal, macros can't be overloaded.
>
> drivers/filesystems/ext2/inc/ext2fsd.h:64:0: warning: "try_return" redefined
> [enabled by default]
>
> You could try
> #define try_return(...) { __VA_ARGS__; goto try_exit; }
>
> If that doesn't do it, it might simply need a separate macro (I'm not sure
> to what extent this is third party code though)
>
> (also if these were the last instances of that -- and I think so --, we
> might want to make 4003 an error)
>
>
> On 2014-03-24 23:20, hbelusca@svn.reactos.org wrote:
>> Author: hbelusca
>> Date: Mon Mar 24 22:20:52 2014
>> New Revision: 62562
>>
>> URL: http://svn.reactos.org/svn/reactos?rev=62562&view=rev
>> Log:
>> [EXT2]
>> try_return() == try_return(S) with nothing in S . The code sometimes use
> it.
>> Shut up MSVC warning C4003: not enough actual parameters for macro
> 'try_return'.
>>
>> Modified:
>> trunk/reactos/drivers/filesystems/ext2/inc/ext2fsd.h
>>
>> Modified: trunk/reactos/drivers/filesystems/ext2/inc/ext2fsd.h
>> URL:
>> http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/filesystems/e
>> xt2/inc/ext2fsd.h?rev=62562&r1=62561&r2=62562&view=diff
>>
> ============================================================================
> ==
>> --- trunk/reactos/drivers/filesystems/ext2/inc/ext2fsd.h [iso-8859-1]
> (original)
>> +++ trunk/reactos/drivers/filesystems/ext2/inc/ext2fsd.h [iso-8859-1]
> Mon Mar 24 22:20:52 2014
>> @@ -60,6 +60,7 @@
>> extern Ext2Data Ext2GlobalData;
>>
>> // try-finally simulation
>> +#define try_return() { goto try_exit; }
>> #define try_return(S) { S; goto try_exit; }
>> #define try_return1(S) { S; goto try_exit1; }
>> #define try_return2(S) { S; goto try_exit2; }
>>
>>
>
>
> _______________________________________________
> Ros-dev mailing list
> Ros-dev@reactos.org
> http://www.reactos.org/mailman/listinfo/ros-dev
>
> _______________________________________________
> Ros-dev mailing list
> Ros-dev@reactos.org
> http://www.reactos.org/mailman/listinfo/ros-dev
>
_______________________________________________
Ros-dev mailing list
Ros-dev@reactos.org
http://www.reactos.org/mailman/listinfo/ros-dev