Skip to content

Correct type of bgl_bignum_to_string 2nd parameter#153

Open
jamesjer wants to merge 1 commit into
manuel-serrano:masterfrom
jamesjer:master
Open

Correct type of bgl_bignum_to_string 2nd parameter#153
jamesjer wants to merge 1 commit into
manuel-serrano:masterfrom
jamesjer:master

Conversation

@jamesjer

Copy link
Copy Markdown
Contributor

Fixes warnings such as this when building with LTO:

Clib/cwriter.c:20:14: warning: type of ‘bgl_bignum_to_string’ does not match original declaration [-Wlto-type-mismatch]
   20 | extern obj_t bgl_bignum_to_string(obj_t x, int radix);
      |              ^
objs/obj_u/Unsafe/bignumber.c:10609:32: note: type mismatch in parameter 2
10609 |         BGL_EXPORTED_DEF obj_t bgl_bignum_to_string(obj_t BgL_xz00_101,
      |                                ^
objs/obj_u/Unsafe/bignumber.c:10609:32: note: type ‘long int’ should match type ‘int’
objs/obj_u/Unsafe/bignumber.c:10609:32: note: ‘bgl_bignum_to_string’ was previously declared here
objs/obj_u/Unsafe/bignumber.c:10609:32: note: code may be misoptimized unless ‘-fno-strict-aliasing’ is used

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