search

lexxmark / winflexbison

Main winflexbision repository
GNU General Public License v3.0
414 stars 120 forks source link

register keyword used in the generated lexer #3

Closed khaledmohammed closed 7 years ago

khaledmohammed commented 7 years ago

register keyword used in the generated lexer such as the following: register yy_size_t number_to_move = (yy_n_chars) + 2;

c++11 has deprecated the "register" keyword. register keyword is a hint to the compiler.

It would be nice if we can remove the "register" keyword from the generated scanner for at least c++, if not for c.

Some compilers have started printing a warning message if c++11x option is choosen to compile the scanner, and causes issue in the build system

Thanks, -Khaled

lexxmark commented 7 years ago

Ok, will look into it

lexxmark commented 7 years ago

Sorry for delay, I was sick a little. I will try to upgrade win_flex to version 2.6.3 Seems there is no any register words in the generated code. Will it works for you? Or do you depend on 2.5.37 version?

khaledmohammed commented 7 years ago

Hi Alex,

Thank you so much. Upgrading to 2.6.3 will be wonderful.

Thanks again, -khaled

On Jan 17, 2017 8:56 PM, "Alex Zhondin" notifications@github.com wrote:

Sorry for delay, I was sick a little. I will try to upgrade win_flex to version 2.6.3 Seems there is no any register words in the generated code. Will it works for you? Or do you depend on 2.5.37 version?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/lexxmark/winflexbison/issues/3#issuecomment-273382871, or mute the thread https://github.com/notifications/unsubscribe-auth/AI5-B9i9cXGn_RsugnVLiDzCIbgbc2loks5rTZtfgaJpZM4LdYRf .

lexxmark commented 7 years ago

Hi Khaled,

just finished upgrading win_flex, now testing it. Could you please take a new version from releses section here and try it. And let me know in case any issues.

Regards, Alex

khaledmohammed commented 7 years ago

Thank you. I did not get a chance to try it, but I will. Thanks :)

On Fri, Jan 27, 2017 at 12:06 AM, Alex Zhondin notifications@github.com wrote:

Closed #3 https://github.com/lexxmark/winflexbison/issues/3.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/lexxmark/winflexbison/issues/3#event-938946981, or mute the thread https://github.com/notifications/unsubscribe-auth/AI5-B8ycVAGWd67rFSQaRB5xi3gyCSxbks5rWaVtgaJpZM4LdYRf .

lexxmark commented 7 years ago

You are welcome. I've released flex 2.6.3 so you can get it from http://sourceforge.net/projects/winflexbison/