Subject: bug#30423: separate flymake into latex-flymake.el

From: Arash Esbati <[email-masked]>
Date: Sun, 18 Feb 2018 21:14:58 +0100

Alex Branham <[email-masked]> writes:

> Here's a different version of the patch I sent with flymake support in
> its own file. I also removed the lexical-binding from latex.el.

Hi Alex,

many thanks for your patch.  Some comments:

> +;;; latex-flymake.el --- Flymake integration  -*- lexical-binding: t; -*-

Could you please add the standard header with Copyright etc. here?

> +;;; Commentary:
> +;; This file provides flymake integration for latex documents using
> +;; "chktex" as a backend.  You must be running Emacs 26 or newer.
> +;; Enable it by adding the following to your init file:
> +
> +;; (add-hook 'LaTeX-mode-hook flymake-mode)
                               _/^\_
There is a quote missing here.

Is there a reason why you didn't add this file to Makefile.in in order
to get byte-compiled?  And could you please add something to the manual?

Best, Arash