Re: [11 PATCH] libiberty, Darwin: Fix a build warning. [PR112823]
> Thanks. I can't push it myself - could you do that for me? Pushed. FX
Re: [11 PATCH] libiberty, Darwin: Fix a build warning. [PR112823]
Iain Sandoe writes: > HI Sam, Hi Iain, > > I think this qualifies as obvious (it’s on my list, but I did not get to it > yet, > so go ahead). Thanks. I can't push it myself - could you do that for me? thanks again, sam > > Iain > >> On 2 Dec 2023, at 05:30, Sam James wrote: >> >> From: Iain Sandoe >> >> r12-3005-g220c410162ebece4f missed a cast for the set_32 call. >> Fixed thus. >> >> Signed-off-by: Iain Sandoe >> Signed-off-by: Sam James >> >> libiberty/ChangeLog: >> PR other/112823 >> * simple-object-mach-o.c (simple_object_mach_o_write_segment): >> Cast the first argument to set_32 as needed. >> >> (cherry picked from commit 38757aa88735ab2e511bc428e2407a5a5e9fa0be) >> --- >> libiberty/simple-object-mach-o.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/libiberty/simple-object-mach-o.c >> b/libiberty/simple-object-mach-o.c >> index 72b69d19c216..a8869e7c6395 100644 >> --- a/libiberty/simple-object-mach-o.c >> +++ b/libiberty/simple-object-mach-o.c >> @@ -1228,7 +1228,7 @@ simple_object_mach_o_write_segment >> (simple_object_write *sobj, int descriptor, >> /* Swap the indices, if required. */ >> >> for (i = 0; i < (nsects_in * 4); ++i) >> -set_32 (&index[i], index[i]); >> +set_32 ((unsigned char *) &index[i], index[i]); >> >> sechdr_offset += sechdrsize; >> >> -- >> 2.43.0 >>
Re: [11 PATCH] libiberty, Darwin: Fix a build warning. [PR112823]
HI Sam, I think this qualifies as obvious (it’s on my list, but I did not get to it yet, so go ahead). Iain > On 2 Dec 2023, at 05:30, Sam James wrote: > > From: Iain Sandoe > > r12-3005-g220c410162ebece4f missed a cast for the set_32 call. > Fixed thus. > > Signed-off-by: Iain Sandoe > Signed-off-by: Sam James > > libiberty/ChangeLog: > PR other/112823 > * simple-object-mach-o.c (simple_object_mach_o_write_segment): > Cast the first argument to set_32 as needed. > > (cherry picked from commit 38757aa88735ab2e511bc428e2407a5a5e9fa0be) > --- > libiberty/simple-object-mach-o.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libiberty/simple-object-mach-o.c > b/libiberty/simple-object-mach-o.c > index 72b69d19c216..a8869e7c6395 100644 > --- a/libiberty/simple-object-mach-o.c > +++ b/libiberty/simple-object-mach-o.c > @@ -1228,7 +1228,7 @@ simple_object_mach_o_write_segment (simple_object_write > *sobj, int descriptor, > /* Swap the indices, if required. */ > > for (i = 0; i < (nsects_in * 4); ++i) > - set_32 (&index[i], index[i]); > + set_32 ((unsigned char *) &index[i], index[i]); > > sechdr_offset += sechdrsize; > > -- > 2.43.0 >
[11 PATCH] libiberty, Darwin: Fix a build warning. [PR112823]
From: Iain Sandoe r12-3005-g220c410162ebece4f missed a cast for the set_32 call. Fixed thus. Signed-off-by: Iain Sandoe Signed-off-by: Sam James libiberty/ChangeLog: PR other/112823 * simple-object-mach-o.c (simple_object_mach_o_write_segment): Cast the first argument to set_32 as needed. (cherry picked from commit 38757aa88735ab2e511bc428e2407a5a5e9fa0be) --- libiberty/simple-object-mach-o.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libiberty/simple-object-mach-o.c b/libiberty/simple-object-mach-o.c index 72b69d19c216..a8869e7c6395 100644 --- a/libiberty/simple-object-mach-o.c +++ b/libiberty/simple-object-mach-o.c @@ -1228,7 +1228,7 @@ simple_object_mach_o_write_segment (simple_object_write *sobj, int descriptor, /* Swap the indices, if required. */ for (i = 0; i < (nsects_in * 4); ++i) - set_32 (&index[i], index[i]); + set_32 ((unsigned char *) &index[i], index[i]); sechdr_offset += sechdrsize; -- 2.43.0