[Groff] www.tmac: apparent error and comments re man page

[Dean Allen Provins, P. Geoph.]

Hello:

I've been experimenting with the -Thtml macro package which employs
www.tmac.

In trying to get "IMG" and "PIMG" to function, I noted that the macro
definitions seemed to have the arguments confused when interpreting the
width and height arguments.  As well, the IMG macro appears to be looking
for the location argument in the wrong place.

Specifically, IMG is looking for the location (i.e. -R, -L etc.)
as argument 2, not 1 as the documentation indicates (lines 542 and 545
of the 1.20.1 version of groff).

The IMG macro definition has comments that suggest the location argument
is optional which conflicts with the man page as it indicates 4 arguments
are required.

The PIMG macro doesn't indicate that 4 arguments are mandatory, and
inspection of the definition indicates that the location arguments are
optional.  When they are interpreted, they are shifted out of the way.
This unfortunately causes the remaining arguments to be misaligned with
the code.

Lines 600 - 604 test arguments 2 and 3, but pick up values from 3 and
4 respectively.

With respect to the man page, I noted that the IMG macro will "include a
picture", which I discovered had to be a PNG file, not an EPS - which is
what I had on hand.  I think that I understand now that the PIMG macro
will convert an image to an EPS if -Tps is the chosen output device,
while IMG will not.  Perhaps only the PIMG macro is required?

My interpretation of the IMG and PIMG macros suggest that IMG
expects pixel counts for iamge width and height, while PIMG
expects inches.  This isn't clear on the man page.

With respect to including EPS files, by studying the supplied example,
I found that the .HTML-IMAGE, .HTML-IMAGE-LEFT, .HTML-IMAGE-RIGHT,
.HTML-IMAGE-INLINE and .HTML-IMAGE-END, combined with PSPIC would handle
that for me.  This is missing from the man page, and might be included.

I noted also found when inspecting the www.tmac package that macros .OLS,
.OLE, .DLS, .DLE, .CDS and .CDE were not mentioned in the man page -
probably an oversight.  There may be others, but their use wasn't
clear to me.

I hope that I have not misinterpreted the code.  If I can be of
any assistance - perhaps with the man page?, I'd be pleased to do
so.

Regards,

Dean

-- 
                           Dean Provins, P. Geoph.
                         address@hidden
                  KeyID at at pgpkeys.mit.edu:11371: 0x9643AE65
          Fingerprint: 9B79 75FB 5C2B 22D0 6C8C 5A87 D579 9BE5 9643 AE65