From: SourceForge.net <no...@so...> - 2005-07-02 12:03:01
|
Bugs item #1207885, was opened at 2005-05-24 15:20 Message generated for change (Settings changed) made by helly You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=616200&aid=1207885&group_id=96864 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: None Group: None >Status: Closed Resolution: None Priority: 5 Submitted By: Marc Noirot (noirotm) Assigned to: Marcus Börger (helly) Summary: Invalid code with -b option in Visual .NET Initial Comment: When compiling the last example from the re2c manual generated with the -b option in Visual .NET 2003 ( with option /TC - compile as C code) , I get several errors : simple.c(222) : error C2065: 'yybm' : undeclared identifier simple.c(222) : error C2109: subscript requires array or pointer type simple.c(248) : error C2109: subscript requires array or pointer type simple.c(440) : error C2109: subscript requires array or pointer type simple.c(570) : error C2109: subscript requires array or pointer type simple.c(586) : error C2109: subscript requires array or pointer type simple.c(638) : error C2109: subscript requires array or pointer type etc... it seems that this is caused by fact the bit vectors array is declared outside the scanner's main block in the scan function. ---------------------------------------------------------------------- Comment By: Marc Noirot (noirotm) Date: 2005-06-26 22:02 Message: Logged In: YES user_id=772613 Yes, I was going to reply this. My patch was meant not to break anything if the 0.9.7 was generating the code like it did for a reason i didn't get. Now that the official new release works fine, my patch isn't useful anymore. Regards, Marc ---------------------------------------------------------------------- Comment By: Marcus Börger (helly) Date: 2005-06-26 21:41 Message: Logged In: YES user_id=271023 Btw, if it compiles and works, the patch obviously is no longer needed right? ---------------------------------------------------------------------- Comment By: Marcus Börger (helly) Date: 2005-06-26 21:31 Message: Logged In: YES user_id=271023 Could you please send an updated patch against 0.9..8? Your current patch doesn't apply anymore, sorry for the additional work. ---------------------------------------------------------------------- Comment By: Marc Noirot (noirotm) Date: 2005-06-26 20:15 Message: Logged In: YES user_id=772613 It compiles and works ok in Visual .NET with re2c 0.9.8. Keep up the good work ! ---------------------------------------------------------------------- Comment By: Marcus Börger (helly) Date: 2005-06-26 13:58 Message: Logged In: YES user_id=271023 Can you recheck please with re2c 0.9.8? ---------------------------------------------------------------------- Comment By: Marc Noirot (noirotm) Date: 2005-05-28 16:10 Message: Logged In: YES user_id=772613 Of course, I forgot to check the checkbox... ---------------------------------------------------------------------- Comment By: Marc Noirot (noirotm) Date: 2005-05-28 16:09 Message: Logged In: YES user_id=772613 Here is a quick fix I've made. I've added a -n switch to have the bit vectors array declared inside the generated scanner block. The code generated with -n compiles fine in VC++ and GCC 3.2. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=616200&aid=1207885&group_id=96864 |