[docutils:patches] #214 Give better messages on malformed tables

Günter Milde via Docutils-develop <[email protected]> Tue, 19 Aug 2025 21:30:52 -0000
Newsgroups gmane.text.docutils.devel
Message-ID </p/docutils/patches/214/c94ae5ef120b58f9ffccd195e14ac8c1de9d0ada.patches@docutils.p.sourceforge.net>
This is a multi-part message in MIME format.
--===============4485964883302952372==
Content-Type: multipart/related; boundary="===============8458957580448641025=="

This is a multi-part message in MIME format.
--===============8458957580448641025==
Content-Type: multipart/alternative; boundary="===============4867195941627503904=="
MIME-Version: 1.0

--===============4867195941627503904==
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit

- **status**: open --> open-fixed
- **Comment**:

Fixed in [r10208]. Thank you for report and patch.



---

**[patches:#214] Give better messages on malformed tables**

**Status:** open-fixed
**Group:** None
**Created:** Sun Jun 08, 2025 05:58 PM UTC by Jynn Nelson
**Last Updated:** Thu Jul 31, 2025 01:06 PM UTC
**Owner:** nobody
**Attachments:**

- [tables.diff](https://sourceforge.net/p/docutils/patches/214/attachment/tables.diff) (5.2 kB; application/octet-stream)


This does several things:
- Specifies `Misaligned right border` for that error, instead of just "malformed table".
- Shows the line where each error happened, not the line where the table starts.
  - This had a complication that line numbers appear to be wrong when `include` directives are present (they include the lines in the source document, instead of being relative to the included document). Just disabled the new smarter logic in that case.
- Changes `malformed_table` to require both detail and an offset, so poor errors like this can't happen in the future.

Fixes https://sourceforge.net/p/docutils/bugs/504/.


---

Sent from sourceforge.net because [email protected] is subscribed to https://sourceforge.net/p/docutils/patches/

To unsubscribe from further messages, a project admin can change settings at https://sourceforge.net/p/docutils/admin/patches/options.  Or, if this is a mailing list, you can unsubscribe from the mailing list.
--===============4867195941627503904==
MIME-Version: 1.0
Content-Type: text/html; charset="us-ascii"
Content-Transfer-Encoding: 7bit

<div class="markdown_content"><ul>
<li><strong>status</strong>: open --&gt; open-fixed</li>
<li><strong>Comment</strong>:</li>
</ul>
<p>Fixed in <a class="alink" href="https://sourceforge.net/p/docutils/code/10208/">[r10208]</a>. Thank you for report and patch.</p>
<hr/>
<p><strong><a class="alink" href="https://sourceforge.net/p/docutils/patches/214/">[patches:#214]</a> Give better messages on malformed tables</strong></p>
<p><strong>Status:</strong> open-fixed<br/>
<strong>Group:</strong> None<br/>
<strong>Created:</strong> Sun Jun 08, 2025 05:58 PM UTC by Jynn Nelson<br/>
<strong>Last Updated:</strong> Thu Jul 31, 2025 01:06 PM UTC<br/>
<strong>Owner:</strong> nobody<br/>
<strong>Attachments:</strong></p>
<ul>
<li><a class="" href="https://sourceforge.net/p/docutils/patches/214/attachment/tables.diff">tables.diff</a> (5.2 kB; application/octet-stream)</li>
</ul>
<p>This does several things:<br/>
- Specifies <code>Misaligned right border</code> for that error, instead of just "malformed table".<br/>
- Shows the line where each error happened, not the line where the table starts.<br/>
  - This had a complication that line numbers appear to be wrong when <code>include</code> directives are present (they include the lines in the source document, instead of being relative to the included document). Just disabled the new smarter logic in that case.<br/>
- Changes <code>malformed_table</code> to require both detail and an offset, so poor errors like this can't happen in the future.</p>
<p>Fixes <a href="https://sourceforge.net/p/docutils/bugs/504/.">https://sourceforge.net/p/docutils/bugs/504/.</a></p>
<hr/>
<p>Sent from sourceforge.net because [email protected] is subscribed to <a href="https://sourceforge.net/p/docutils/patches/">https://sourceforge.net/p/docutils/patches/</a></p>
<p>To unsubscribe from further messages, a project admin can change settings at <a href="https://sourceforge.net/p/docutils/admin/patches/options.">https://sourceforge.net/p/docutils/admin/patches/options.</a>  Or, if this is a mailing list, you can unsubscribe from the mailing list.</p></div>
--===============4867195941627503904==--

--===============8458957580448641025==--


--===============4485964883302952372==
Content-Type: text/plain; charset="us-ascii"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Content-Disposition: inline


--===============4485964883302952372==
Content-Type: text/plain; charset="us-ascii"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Content-Disposition: inline