Skip to content

Fixed compile errors#4

Open
garak92 wants to merge 1 commit intofnky:masterfrom
garak92:fix-compiler-errors
Open

Fixed compile errors#4
garak92 wants to merge 1 commit intofnky:masterfrom
garak92:fix-compiler-errors

Conversation

@garak92
Copy link
Copy Markdown

@garak92 garak92 commented Feb 12, 2024

Hello

I tried to compile this on Arch Linux, but I've got compilation errors. To be more precise: the object files were generated just fine, these errors occurred at the linker phase.
I did some changes, and now it works.

The errors were caused by:

  • Use of memmove without including string.h
  • Multiple definitions of variables (fixed by declaring them with extern on the header file and initializing them on studiomdl.c...I am not a C magician, so tell me if this isn't the right way to do it)
  • Use of deprecated and unsafe getwc function, which I replaced with getcwc from unistd.h

In case this isn't merged, I include the patch here anyways to download in case anyone else is having the same issue.

Cheers,

Elim Garak (just a tailor...)

Patch file (I had to upload it as a .txt instead of .patch because of a dumb GitHub bug, sorry):
0001-Fixed-compile-errors-caused-by-Use-of-deprecated-and.txt

…function * Multiple definitions of variables * Use of memmove without including string.h
@hossine7
Copy link
Copy Markdown

thank you 😭

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.

2 participants