Re: [PATCH RFC 0/4] Fix PDF build errors

From	Jani Nikula <jani.nikula@linux.intel.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH RFC 0/4] Fix PDF build errors
Date	2016-11-03 10:20 +0100
Message-ID	<szmlI-1Wi-13@gated-at.bofh.it> (permalink)
References	<syQ2t-6iF-5@gated-at.bofh.it> <sz1Kh-5v9-1@gated-at.bofh.it> <sz6gV-8vT-7@gated-at.bofh.it> <sz6TE-jL-17@gated-at.bofh.it> <sz7cZ-FB-11@gated-at.bofh.it>
Organization	Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo

Show all headers | View raw

On Wed, 02 Nov 2016, Markus Heiser <markus.heiser@darmarit.de> wrote:
> Am 02.11.2016 um 17:47 schrieb Mauro Carvalho Chehab <mchehab@infradead.org>:
>> I'm staring to think that we should just redefine the default for ::
>> to be "none", and use the "C" handling **only** when explicitly
>> requested.
>> 
>> I remember that Jon did such suggestion sometime ago.
>
> *shrug* ... I think about kernel-doc comments in the sources,
> mostly you will have small examples and won't those verbose 
> ".. code-block::" markup / using "::" keeps the comment compact.

I sent the patch [1] to default to "none". It's never wrong, not even
for code. But getting the highlighting wrong, OTOH, is pretty bad. We
have that.

And really, if you look at the kernel-doc comments and the rst
documentation, we don't have all that many literal blocks that would
benefit from syntax highlighting in the first place.

BR,
Jani.

[1] http://lkml.kernel.org/r/1478164053-4562-1-git-send-email-jani.nikula@intel.com

-- 
Jani Nikula, Intel Open Source Technology Center

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

[PATCH RFC 0/4] Fix PDF build errors Mauro Carvalho Chehab <mchehab@osg.samsung.com> - 2016-11-01 23:50 +0100
  Re: [PATCH RFC 0/4] Fix PDF build errors Mauro Carvalho Chehab <mchehab@infradead.org> - 2016-11-02 00:00 +0100
  [PATCH RFC 2/4] [media] subdev-formats.rst: we need a pdf picture for bayer Mauro Carvalho Chehab <mchehab@osg.samsung.com> - 2016-11-02 03:40 +0100
  [PATCH RFC 3/4] [media] subdev-formats.rst: don't use adjustbox on a longtable Mauro Carvalho Chehab <mchehab@osg.samsung.com> - 2016-11-02 03:40 +0100
  Re: [PATCH RFC 0/4] Fix PDF build errors Jani Nikula <jani.nikula@linux.intel.com> - 2016-11-02 12:20 +0100
    Re: [PATCH RFC 0/4] Fix PDF build errors Mauro Carvalho Chehab <mchehab@infradead.org> - 2016-11-02 13:30 +0100
      Re: [PATCH RFC 0/4] Fix PDF build errors Jani Nikula <jani.nikula@linux.intel.com> - 2016-11-02 14:30 +0100
        Re: [PATCH RFC 0/4] Fix PDF build errors Mauro Carvalho Chehab <mchehab@infradead.org> - 2016-11-02 17:40 +0100
        Re: [PATCH RFC 0/4] Fix PDF build errors Fengguang Wu <fengguang.wu@intel.com> - 2016-11-02 22:50 +0100
    Re: [PATCH RFC 0/4] Fix PDF build errors Markus Heiser <markus.heiser@darmarit.de> - 2016-11-02 17:10 +0100
      Re: [PATCH RFC 0/4] Fix PDF build errors Mauro Carvalho Chehab <mchehab@infradead.org> - 2016-11-02 17:50 +0100
        Re: [PATCH RFC 0/4] Fix PDF build errors Markus Heiser <markus.heiser@darmarit.de> - 2016-11-02 18:10 +0100
          Re: [PATCH RFC 0/4] Fix PDF build errors Jani Nikula <jani.nikula@linux.intel.com> - 2016-11-03 10:20 +0100

csiph-web