From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.io!.POSTED.blaine.gmane.org!not-for-mail From: Eli Zaretskii Newsgroups: gmane.emacs.devel Subject: Re: [PATCH] Flymake Support Indicator Errors in Margin Date: Sun, 07 Apr 2024 08:47:58 +0300 Message-ID: <86msq5206p.fsf@gnu.org> References: <87r0gfajlh.fsf@posteo.net> <868r2mtjdk.fsf@gnu.org> <86msr19ipu.fsf@gnu.org> <864jd677qh.fsf@gnu.org> <87jzm0iza9.fsf@catern.com> <87cyrsitqw.fsf@catern.com> <871q7w4jn7.fsf@catern.com> <86edbi3eqw.fsf@gnu.org> Injection-Info: ciao.gmane.io; posting-host="blaine.gmane.org:116.202.254.214"; logging-data="9537"; mail-complaints-to="usenet@ciao.gmane.io" Cc: sbaugh@catern.com, sbaugh@janestreet.com, philipk@posteo.net, emacs-devel@gnu.org, luangruo@yahoo.com To: Elijah G Original-X-From: emacs-devel-bounces+ged-emacs-devel=m.gmane-mx.org@gnu.org Sun Apr 07 07:48:53 2024 Return-path: Envelope-to: ged-emacs-devel@m.gmane-mx.org Original-Received: from lists.gnu.org ([209.51.188.17]) by ciao.gmane.io with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1rtLOf-0002GS-PG for ged-emacs-devel@m.gmane-mx.org; Sun, 07 Apr 2024 07:48:53 +0200 Original-Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1rtLNt-0003NK-9E; Sun, 07 Apr 2024 01:48:05 -0400 Original-Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1rtLNr-0003NA-9v for emacs-devel@gnu.org; Sun, 07 Apr 2024 01:48:03 -0400 Original-Received: from fencepost.gnu.org ([2001:470:142:3::e]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1rtLNp-0002CF-CM; Sun, 07 Apr 2024 01:48:01 -0400 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=gnu.org; s=fencepost-gnu-org; h=References:Subject:In-Reply-To:To:From:Date: mime-version; bh=YII/aw/GexrTNz5rSeewJ+DVhTPooHA9yF+6p+Pg2EE=; b=a8gHbXiWE53q RjVh6b9vNf0aTaZ3v1FX/23i9CVcqcJr0E9LUBGXND5Jb9RFB/BklqJ4unf+8kWh5EiIR9trNg+oT ig+0CBKpv0BJe4l0rcsmzC5sRzj9HckI4zq3k6nznIhLpoMtpnKbcaPs7ch6ix5g5YkNZ/NGWCJI/ blySLKXa5ItnT6m2ZTvJggRq2+pUqGH4CKm6aVP5qlbtMSZJnDlLSBCX4iCrH6nllbOsmxMUE3Xxy e/8hI+VQDlcl7p4PyomDs+iNZkzxLAAbCWX3+76el9XTu5meR0t7A9+0co7Z1Gi6A44SvnHWufF1T HPERF4cK4Fn1hDcyH50rUQ==; In-Reply-To: (message from Elijah G on Sat, 6 Apr 2024 14:14:34 -0600) X-BeenThere: emacs-devel@gnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: "Emacs development discussions." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: emacs-devel-bounces+ged-emacs-devel=m.gmane-mx.org@gnu.org Original-Sender: emacs-devel-bounces+ged-emacs-devel=m.gmane-mx.org@gnu.org Xref: news.gmane.io gmane.emacs.devel:317581 Archived-At: > From: Elijah G > Date: Sat, 6 Apr 2024 14:14:34 -0600 > Cc: sbaugh@catern.com, sbaugh@janestreet.com, philipk@posteo.net, > emacs-devel@gnu.org, luangruo@yahoo.com > > > Last, but not least: please mention this change (and the new user > > options) in NEWS and in the Flymake manual. > > Thank you, I've now fixed the patch and documented the changes done. See below for some additional comments. > Also I have a question, Do I need to assign copyright each time > that I want to contribute to Flymake? No, you assign the copyright for past and future changes, and you do it only once. Did you start the paperwork already? I don't see your assignment on file, so if you haven't started the paperwork, now is the time to do it. Let me know if you need me to send you the form to fill and the instructions, to start the paperwork. > I have a lot of ideas that I would like to contribute. Thanks, looking forward to your contributions. > --- a/doc/misc/flymake.texi > +++ b/doc/misc/flymake.texi > @@ -309,6 +309,11 @@ Customizable variables > A custom face for highlighting regions for which a note has been > reported. > > +@item flymake-indicator-type > +Which indicator type should Flymake use. Suggest to reword: The indicator type which Flymake should use to indicate lines with errors or warnings. > +@item flymake-margin-indicators-string > +A List used for specify string, face and error types to > +use in the margin indicators. Suggest to reword: Specifies the string and face to use for the margin indicators, for each error type. > +@item flymake-autoresize-margins > +If non-@code{nil}, will resize margins either @code{flymake-mode} is > +non-@code{nil} or @code{nil}. Something is wrong with this sentence. You probably mean If non-@code{nil}, Flymake will resize the margins when @code{flymake-mode} is turned on or off. > +@item > +@cindex margin of diagnostic > +@code{flymake-margin-string}, an string displayed in the margin ^^^^^^^^^ "a string" > +The value actually follows the syntax of @code{flymake-margin-indicators-string} > +(@pxref{Customizable variables}). It is overridden by any ^^ Two spaces between sentences, please. > ++++ > +*** New user option 'flymake-indicator-type'. > +This user option controls which error indicator type Flymake should use > +in current buffer. Depending on your preference, this can either use ^^ Likewise. > +*** New user option 'flymake-margin-indicators-string' All heading lines in NEWS should end in a period. > +It controls the string, their face and type error to display in > +margin indicators. It controls, for each error type, the string and its face to display as the margin indicator. > +*** New user option 'flymake-autoresize-margins' Missing period at the end. > +When non-nil, Flymake will resize margins either `flymake-mode` is > +non-nil or nil. See above for correction in the manual, I think the same wording should be used here. Also, please quote 'like this', not `like this` (here and elsewhere in the patch for NEWS). > +*** New user option 'flymake-margin-indicator-position' Period missing at the end. > +It controls which position should Flymake show margin indicators. It controls which margin (left or right) is used for margin indicators. > +(defvar-local flymake--original-margin-width nil > + "Store original margin width. > +Used by `flymake--resize-margins' for restore original margin width ^^^^^^^^^^^ "for restoring"