[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] TTY Support for ECMA-48 strike-through graphic rendition
From: |
Mike Hamrick |
Subject: |
Re: [PATCH] TTY Support for ECMA-48 strike-through graphic rendition |
Date: |
Mon, 14 Sep 2020 15:25:24 -0700 |
User-agent: |
mu4e 1.0; emacs 25.3.1 |
Eli Zaretskii writes:
>> From: Mike Hamrick <mikeh@muppetlabs.com>
>> Cc: emacs-devel@gnu.org
>> Date: Fri, 26 Jun 2020 12:23:48 -0700
>>
>> I've updated my patch to include a NEWS item. I grepped through the
>> manual looking for areas that might need updating, and did not find
>> any.
>
> Thanks, a couple of really minor nits below:
>
>> @@ -1971,7 +1974,8 @@ turn_off_face (struct frame *f, int face_id)
>> if (face->tty_bold_p
>> || face->tty_italic_p
>> || face->tty_reverse_p
>> - || face->tty_underline_p)
>> + || face->tty_underline_p
>> + || face->tty_strike_through_p)
>
> Please use our style of indentation in C source files (basically, make
> sure indent-tabs-mode is non-nil).
>
> Also, please make sure the patch still applies to the current master,
> and if not, please update it.
It occurs to me that last time I responded to this, I didn't attach the
patch to this email, but instead tacked it on to the end of my
message. Sending the patch as an attachment this time.
Mike
0001-TTY-Support-for-ECMA-48-strike-through-graphic-rendi.patch
Description: Text Data