Re: "loco" after ottava

[Thomas Morley]

Am Sa., 3. Dez. 2022 um 13:53 Uhr schrieb Jean Abou Samra <jean@abou-samra.fr>:

> Le 03/12/2022 à 13:26, Thomas Morley a écrit :

> > \version "2.23.0"
>
>
> 2.23.80, I suppose? horizontal-line-spanner is new in 2.23.6.

Ofcourse, that 8 slipped away somehow...

> > Btw, I can't find -dcheck-internal-types in the 2.23.-docs any more
> > and was not able to find out why.
> >
> > Anyone with more insight?
>
>
>
> In such cases, use
>
> $ git log -S "check-internal-types"

I didn't know about -S
Thanks.

Alas, this command extended as
$ git log -S "check-internal-types" --oneline --author="Han-Wen"
prints for me:
$ git log -S "check-internal-types" --oneline --author="Han-Wen"
264570e68e Documentation: remove documentation for internal options
e435a88122 Remove en/contributor/build-notes.texi
8ad3efa86f output-distance: make self test pass again:
51117d43f1 Don't Graham's compile logs.
cf7f4f614b (tag: release/2.11.33-1) Make test generation generic.
9291834601 replace SCM_ASSERT_TYPE with LY_ASSERT_TYPE and friends
acc8482ad0 set LILYPOND_BOOK_LILYPOND_FLAGS explicitly for test.
Removes PDF for tests.
b7da29a7aa * scm/framework-ps.scm (output-preview-framework): use new name.
29ebcd1876 * scm/framework-ps.scm (output-preview-framework): use new name.

I.e. this one is not found:

> commit a1dead391ed016a8aefd1be7fb62946e094b4535
> Author: Han-Wen Nienhuys <hanwenn@gmail.com>
> Date:   Sat Feb 26 12:19:32 2022 +0100
>
>      Add internal/external distinction to -d flags

whereas

> commit 264570e68ea13b06c1b9dd88c491948979926aa1
> Author: Han-Wen Nienhuys <hanwenn@gmail.com>
> Date:   Sun Apr 3 12:45:51 2022 +0200
>
>      Documentation: remove documentation for internal options

is indeed found.
Anyway, "Add internal/external distinction to -d flags" is
https://gitlab.com/lilypond/lilypond/-/merge_requests/1231


That said, I researched a bit more about "check-internal-types".
and found we have:

lilypond --help
lilypond -dhelp
and actually
lilypond -dhelp-internal

The latter is undocumented as intended by one of the mentioned
patches, but shows "check-internal-types"

I agree with separating internal/external options and not explicitley
documenting internal ones.
But not to document at all how to reach the internal ones is too much, imho.
I'd vote for documenting help-internal in "Application Usage".  With a
short hint, something at the lines of:

help-internal bool
  If bool is #t, display options for internal usage. Default: #f.

Opinions?

Thanks,
  Harm