From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from mailout-webserver.scc.kit.edu (mailout-webmail.scc.kit.edu [129.13.185.232])
	by krisdoz.my.domain (8.14.5/8.14.5) with ESMTP id s7E1QSIw005003
	for <tech@mdocml.bsd.lv>; Wed, 13 Aug 2014 21:26:29 -0400 (EDT)
Received: from hekate.usta.de (asta-nat.asta.uni-karlsruhe.de [172.22.63.82])
	by scc-mailout-02.scc.kit.edu with esmtp (Exim 4.72 #1)
	id 1XHjoB-0006EW-SH; Thu, 14 Aug 2014 03:26:27 +0200
Received: from donnerwolke.usta.de ([172.24.96.3])
	by hekate.usta.de with esmtp (Exim 4.77)
	(envelope-from <schwarze@usta.de>)
	id 1XHjoB-0005t9-Qd; Thu, 14 Aug 2014 03:26:27 +0200
Received: from iris.usta.de ([172.24.96.5] helo=usta.de)
	by donnerwolke.usta.de with esmtp (Exim 4.72)
	(envelope-from <schwarze@usta.de>)
	id 1XHjoB-0002mB-P7; Thu, 14 Aug 2014 03:26:27 +0200
Received: from schwarze by usta.de with local (Exim 4.77)
	(envelope-from <schwarze@usta.de>)
	id 1XHjnQ-0007iq-Vv; Thu, 14 Aug 2014 03:25:41 +0200
Date: Thu, 14 Aug 2014 03:25:40 +0200
From: Ingo Schwarze <schwarze@usta.de>
To: Kristaps Dzonsons <kristaps@bsd.lv>
Cc: tech@mdocml.bsd.lv
Subject: Re: HTML5 redux
Message-ID: <20140814012540.GI26534@iris.usta.de>
References: <53EBE549.2000100@bsd.lv>
X-Mailinglist: mdocml-tech
Reply-To: tech@mdocml.bsd.lv
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <53EBE549.2000100@bsd.lv>
User-Agent: Mutt/1.5.21 (2010-09-15)

Hi Kristaps,

Kristaps Dzonsons wrote on Thu, Aug 14, 2014 at 12:23:05AM +0200:

> Enclosed is another patch for HTML5.  This one is designed to *add*
> HTML5.  So it creates an extra shim.  Subsequent patches would then
> remove the existing functionality.  I think this is a good step:
> first we add it, then we can start to decrease the complexity.

I don't object to that route.

I'd probably not merge to the VERSION_1_12 branch until the release
is done, and i'd probably merge to OpenBSD after the cleanup is done.

> It consists of the following:
> 
>  - a single <meta charset="utf-8" />
>  - removed summary, align, and width attributes from header/footer
>  - remove <col /> from header/footer
>  - add a <style /> shim *before* the <link /> to provide a default
> layout of the header/footer (instead of the hard-coded crap)
> 
> (The <style /> shim would work with HTML4 as well, but eh.)
> 
> With this patch, both -Thtml and -Txhtml will produce HTML5.  This
> can be disabled in html.c by replacing the HTML_HTML5 allocation
> with the respective type.  The resulting HTML5 validates just fine,
> as do the existing modes (if replaced in html.c).

Not for me, there is one regression.

But if you remove the chunk cited below, the regression goes
away, so i'd say, go ahead - in particular if you plan to do the
cleanup right afterwards.

> Index: html.c
> ===================================================================
> RCS file: /usr/vhosts/mdocml.bsd.lv/cvs/mdocml/html.c,v
> retrieving revision 1.162
> diff -u -p -r1.162 html.c
> --- html.c	13 Aug 2014 20:34:29 -0000	1.162
> +++ html.c	13 Aug 2014 21:29:43 -0000
> @@ -68,13 +68,14 @@ static	const struct htmldata htmltags[TA
>  	{"dt",		HTML_CLRLINE}, /* TAG_DT */
>  	{"dd",		HTML_CLRLINE}, /* TAG_DD */
>  	{"blockquote",	HTML_CLRLINE}, /* TAG_BLOCKQUOTE */
> -	{"p",		HTML_CLRLINE | HTML_NOSTACK | HTML_AUTOCLOSE}, /* TAG_P */
> +	{"p",		HTML_CLRLINE}, /* TAG_P */

This change causes regressions for me.
If i remove that chunk, it works fine.

Sample output:

<div class="section">
<h1 id="x4445534352495054494f4e">DESCRIPTION</h1> normal line of text second nor
mal line<br>
 line with a leading space <span class="unix">UNIX</span> normal line after a ma
cro line <span class="unix">UNIX</span><br>
 leading space after a macro line<p>   ### look here
<pre style="margin-left: 0.00ex;" class="lit display">
normal line in a literal display 
 leading space in a literal display 
another normal line</pre>
</p>   ### and here
<p>
<div style="margin-left: 0.00ex;" class="display">
normal line in a filled display<br>
 leading space in a filled display another normal line</div>
</p>   ### and here
</div>

Sample input:

/usr/src/regress/usr.bin/mandoc/char/space/leading-mdoc.in

Yours,
  Ingo
--
 To unsubscribe send an email to tech+unsubscribe@mdocml.bsd.lv