Menu

#205 "spell %s" should now read "spell %f"

xfig
open
nobody
None
2026-07-31
2026-07-30
Markus Kuhn
No

xfig 3.2.9 changed the placeholder for the file to be spell checked from %s to %f, but the documentation and default still say %s:

$ rg "spell %s"
doc/xfig.man.in
979:Default is ``spell %s''.
1270:spellcheckcommand  string  spell %s    \-spellcheckcommand

doc/xfig_man.html.in
1674:&lsquo;&lsquo;spell %s&rsquo;&rsquo;.</p>
3170:<p>spell %s</p></td>

src/resources.h
194:                       "spell %s" or  "ispell -l < %s | sort -u" */

In the above lines, all instances of %s should be corrected to %f to match the implementation.

Discussion

  • Markus Kuhn

    Markus Kuhn - 2026-07-30

    Full patch attached.

    I'd also suggest considering

    aspell list < %f | sort -u
    

    or

    enchant-2 -l %f
    

    as a modern default for the spell check command on Linux.

     
  • tkl

    tkl - 2026-07-31

    I consulted with debian popularity contest and noticed that, indeed, the enchant2 and aspell packages are quite frequently installed and used. Enchant and ispell or spell seem obsolete or even unmaintained. I will happily apply your patch.

    Is it OK to name you as the author in the commit message?

     

Log in to post a comment.