Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Pe build breaks due to inconsistent type definitions in lex-based language addons #1373

Closed
anevilyak opened this issue May 29, 2017 · 2 comments
Labels

Comments

@anevilyak
Copy link
Contributor

Owing to changes in the locale kit, the x86 Pe package no longer launches. When attempting to rebuild it via HaikuPorter though, I see errors such as:

Languages/Sources/Lout_Popup.l:71:62: error: invalid initialization of non-const reference of type 'yy_size_t& {aka long unsigned int&}' from an rvalue of type 'yy_size_t {aka long unsigned int}'
#define YY_INPUT(tok,result,max) GetNextChar(tok, result, max)
^
generated/objects-nodebug/Languages/Sources/Lout_Popup.cpp:1303:3: note: in expansion of macro 'YY_INPUT'
YY_INPUT( (&YY_CURRENT_BUFFER_LVALUE->yy_ch_buf[number_to_move]),

This appears to be due to the yy_n_chars variable being declared as a simple static int, while GetNextChar expects it to be a yy_size_t ; however, said variable is, as best I can tell, automatically generated by flex/bison themselves, at least I find no explicit declaration of it anywhere, so I'm uncertain as to how to fix it. This is on hrev51190, with Flex 2.6.1 and Bison 3.0.4.

@diversys
Copy link
Member

Same here on x86_64

@korli
Copy link
Contributor

korli commented Jun 7, 2017

Should be fixed in ffc1cd9.

@korli korli closed this as completed Jun 7, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

No branches or pull requests

4 participants