gas pointer to int and vice versa

[Sam James]

"Maciej W. Rozycki" <macro@orcam.me.uk> writes:

> On Wed, 9 Jul 2025, Sam James wrote:
>
>> >  Hmm, given that updates were made to the affected lines this piece and 
>> > the rest of the changes applied to this file could have benefitted from 
>> > fixing formatting to follow the GNU Coding Standards, i.e. adding the 
>> > missing space after the first cast:
>> >
>> > 		    (char *) (intptr_t) fixups[0].opindex);
>> >
>> > but given that you've pushed it already I think the file will best remain 
>> > as it is so as not to clutter `git blame' with syntactic noise.  Sigh.
>> 
>> I think the "Sigh." can come across as passive aggressive and is
>> unhelpful.
>
>  Apologies to give such impression, not meant as such, not at all.  Just a 
> reflection on a missed opportunity; a minor one admittedly, I wish we only 
> had such issues in life.

No worries, and I didn't mean to imply you were -- more just I've been
non the receiving end of it used that way and it drove me mad ;)

>
>> Anyway, there's nothing stopping us maintaining a .git-blame-ignore-revs
>> file at the root of the repo which contains commits to be ignored for
>> the purposes of `git blame`. git has to be configured to use it (the
>> filename isn't special / automatically recognised) but that's not a big
>> deal.
>
>  Good to know, thanks.  I'm not sure if it scales well though.
>

Yes, it's hard to justify it so much given it's a lot of effort *and*
git doesn't recognise it by default. OTOH, if we have people git-blaming
regularly who find it valuable, maybe it is. Dunno. We'll see if it
comes up in future.

>   Maciej

sam