Skip to content

fixed compile error for gcc 11.4#7

Closed
pletzer wants to merge 2 commits into
PacificCommunity:dev-skjmodelfrom
pletzer:dev-skjmodel
Closed

fixed compile error for gcc 11.4#7
pletzer wants to merge 2 commits into
PacificCommunity:dev-skjmodelfrom
pletzer:dev-skjmodel

Conversation

@pletzer

@pletzer pletzer commented May 11, 2025

Copy link
Copy Markdown
Collaborator

With gcc 11.4 I get the following error:

src/seapodym_coupled.cpp: In function ‘void verify_identifier_string2(char*)’:
src/seapodym_coupled.cpp:621:29: error: cannot call member function ‘DF_FILE* gradient_structure::get_fp()’ without object
621 | gradient_structure::get_fp()->fread(str,num_bytes);
| ~~~~~~~~~~~~~~~~~~~~~~~~~~^~
src/seapodym_coupled.cpp: In function ‘int save_identifier_string2(char*)’:
src/seapodym_coupled.cpp:646:29: error: cannot call member function ‘DF_FILE* gradient_structure::get_fp()’ without object
646 | gradient_structure::get_fp()->fwrite(str,length);
| ~~~~~~~~~~~~~~~~~~~~~~~~~~^~
src/seapodym_coupled.cpp: In function ‘void save_long_int_value(long unsigned int)’:
src/seapodym_coupled.cpp:654:29: error: cannot call member function ‘DF_FILE* gradient_structure::get_fp()’ without object
654 | gradient_structure::get_fp()->fwrite(y,num_bytes);
| ~~~~~~~~~~~~~~~~~~~~~~~~~~^~
src/seapodym_coupled.cpp: In function ‘long unsigned int restore_long_int_value()’:
src/seapodym_coupled.cpp:661:29: error: cannot call member function ‘DF_FILE* gradient_structure::get_fp()’ without object
661 | gradient_structure::get_fp()->fread(&tmpout,num_bytes);
| ~~~~~~~~~~~~~~~~~~~~~~~~~~^~
make: *** [Makefile:151: objs/seapodym_coupled.o] Error 1
(

I think gradient_structure::get_fp() should be replace with gradient_structure().get_fp()

@pletzer

pletzer commented Jul 21, 2025

Copy link
Copy Markdown
Collaborator Author

The above fix causes a seg fault. It is necessary to move back to version 13.1 of admb.

@pletzer pletzer closed this Jul 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant