GNU bug report logs - #75755
feature/igc: Missing IGC_CHECK_RES?

Previous Next

Package: emacs;

Reported by: Stefan Kangas <stefankangas <at> gmail.com>

Date: Wed, 22 Jan 2025 10:28:02 UTC

Severity: normal

Done: Pip Cet <pipcet <at> protonmail.com>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: Stefan Kangas <stefankangas <at> gmail.com>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: pipcet <at> protonmail.com, eller.helmut <at> gmail.com, gerd <at> gnu.org, gerd.moellmann <at> gmail.com, 75755 <at> debbugs.gnu.org, acorallo <at> gnu.org
Subject: bug#75755: feature/igc: Missing IGC_CHECK_RES?
Date: Fri, 24 Jan 2025 18:02:11 -0600
Eli Zaretskii <eliz <at> gnu.org> writes:

>> From: Stefan Kangas <stefankangas <at> gmail.com>
>> Date: Thu, 23 Jan 2025 15:32:16 -0600
>> Cc: pipcet <at> protonmail.com, gerd.moellmann <at> gmail.com, 75755 <at> debbugs.gnu.org,
>> 	gerd <at> gnu.org, eller.helmut <at> gmail.com, acorallo <at> gnu.org
>>
>> >> Please let me know what you think of my most recent patch.  If you want
>> >> something different, may I ask that you send a patch?  I don't think
>> >> I understand if your idea is different from what I posted.  Thanks.
>> >
>> > Your patch still uses _mps_RES_ENUM, which is a macro that's not
>> > supposed to be used outside of MPS.  Isn't that so?  I thought all
>> > this discussion was how to avoid using that macro, wasn't it?
>>
>> AFAIU, the patch I proposed removes that macro.  I just now grepped for
>> _mps_RES_ENUM after applying it, and couldn't find any matches in my
>> tree.  Are you seeing something different?
>
> Yes, I was looking at a stale igc.c, so I didn't see that
> mps_res_to_string was modified exactly as I suggested to do.

No problem, and thanks.  Installed on feature/igc as commit 2ce718a5274.




This bug report was last modified 164 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.