groff_mm(7) manpage

[Ulrich Lauther]

In the groff_mm manual page I see

PIC [-L] [-C] [-R] [-I n] filename [width [height]]
Include a PostScript file in the document.

I think, it should be PSPIC, not PIC
--
-ulrich lauther

[Werner LEMBERG]

> In the groff_mm manual page I see
>
> PIC [-L] [-C] [-R] [-I n] filename [width [height]]
> Include a PostScript file in the document.
>
> I think, it should be PSPIC, not PIC

Have you actually tried it? `PIC' is correct.


Werner

[Ulrich Lauther]

Yes, I did.
With PSPIC I get the drawing as expected, with PIC I get nothing.

ulrich

[Ulrich Lauther]

On Wed, Mar 06, 2013 at 04:46:38PM +0100, Werner LEMBERG wrote:
> Well, for me it works out of the box. Attached you can find an input
> file which includes a small EPS file, and its PS output. Process it
> with
>
> mmroff testxxx.troff > testxxx.ps
>
>
> Werner

> .INITR testxxx-refs
> .P
> Before the image.
> .P
> .PIC test.eps
> .P
> After the image.

You are right: my test did not have the .INITR line.

But there seems to be another difference:

.PSPIC centers the image by default, whereas for .PIC the default is left-aligned.

Thanks for your help,

ulrich




--
-lauther

[Werner LEMBERG]

> But there seems to be another difference:
>
> .PSPIC centers the image by default, whereas for .PIC the default is
> left-aligned.

Interestingly, the groff_mm(7) man page doesn't say which of the
parameters is the default. Maybe the left-aligned default has
historical reasons, I don't know.

While looking into the source code I see that option -B (to produce a
box around the image) of the PIC macro isn't documented either...

Sigh. No time to work on this. If someone provides a patch (*with* a
ChangeLog entry), I can commit it.


Werner

[Deri James]

> _______________________________________________

> bug-groff mailing list

> bug-...@gnu.org

> https://lists.gnu.org/mailman/listinfo/bug-groff

 

Hi Werner,

 

Attached patch includes a patch to the mm/changelog as well as the man page.

 

Cheers

 

Deri

 

[Werner LEMBERG]

> Attached patch includes a patch to the mm/changelog as well as the
> man page.

Applied, thanks (with a small fix).


Werner