bug#69402: 28.3; checkdoc does not check for docstrings wider than 80 characters

bug#69402: 28.3; checkdoc does not check for docstrings wider than 80 characters

[Tim Landscheidt]

(I am not sure whether this is a bug or a missing feature;
the reference to byte-compile-docstring-max-column in
lisp/emacs-lisp/checkdoc.el made me assume this should
work.)

With Emacs 28.3 and a test-checkdoc.el:

| ;;; test-checkdoc.el --- Tests

| ;;; Commentary:
| ;;

| ;;; Code:

| (defcustom my-test-configuration
|   nil
|   "This docstring is longer than xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx characters."
|   :group 'does-not-matter
|   :type 'string)

| (defvar my-test-variable
|   nil
|   "This docstring is longer than xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx characters.")

| (defun my-test-function
|   nil
|   "This docstring is longer than xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx characters.")

| (provide 'test-checkdoc)

| ;;; test-checkdoc.el ends here

M-x checkdoc RET reports:

| Buffer comments and tags:  Ok
| Documentation style:       Ok
| Message/Query text style:  Ok
| Unwanted Spaces:           Ok

However, byte-compile-file complains:

| ^L
| Compiling file /tmp/test-checkdoc.el at Mon Feb 26 08:10:04 2024
| test-checkdoc.el:8:1: Warning: custom-declare-variable `my-test-configuration'
|     docstring wider than 80 characters
| test-checkdoc.el:14:1: Warning: defvar `my-test-variable' docstring wider than
|     80 characters

| In my-test-function:
| test-checkdoc.el:18:8: Warning: docstring wider than 80 characters

IMHO, checkdoc should check that docstrings are no longer
than 80 (or byte-compile-docstring-max-column) characters.

bug#69402: 28.3; checkdoc does not check for docstrings wider than 80 characters

[Eli Zaretskii]

> From: Tim Landscheidt <tim@tim-landscheidt.de>
> Date: Mon, 26 Feb 2024 08:12:17 +0000
> 
> IMHO, checkdoc should check that docstrings are no longer
> than 80 (or byte-compile-docstring-max-column) characters.

It does.  Add one more long line to the doc string, and you will see
it.

bug#69402: 28.3; checkdoc does not check for docstrings wider than 80 characters

[Stefan Kangas]

severity 69402 wishlist
tags 69402 wontfix
close 69402
thanks

Eli Zaretskii <eliz@gnu.org> writes:

>> From: Tim Landscheidt <tim@tim-landscheidt.de>
>> Date: Mon, 26 Feb 2024 08:12:17 +0000
>>
>> IMHO, checkdoc should check that docstrings are no longer
>> than 80 (or byte-compile-docstring-max-column) characters.
>
> It does.  Add one more long line to the doc string, and you will see
> it.

I think we would take a patch to fix this minor inconsistency, but it's
not something that we will invest any time in fixing.

I'm therefore closing this bug report now.  Sorry.