Hi Yuao,

Yuao Ma wrote:
This patch is a follow-up to commit r16-938-ge8fdd55ec90749. In this patch, we
add intrinsic documentation for the newly added trig functions with
half-revolutions. We also reorder the documentation for `atand` to place it in
correct alphabetical order.

When I try to "make pdf" your patch, I get the following error:

intrinsic.texi:1707: Use of @atan
doesn't match its definition.
<argument> -@pi /2 @leq @Re @atan (
                                   x) @leq @pi /2
@finishmath #1->#1
                  $@endgroup
l.1707 @math{-\pi/2 \leq \Re \atan(x) \leq \pi/2}


Additionally, I think all "half-revolutions" should be "half revolutions"
(i.e. no hyphen) in your patch as they stand alone as in
"The result is in half-revolutions"

[The hyphen makes sense if a noun follows as in "half-revolution function".]

Otherwise, it looks okay to me.

Tobias

Reply via email to