9fans - fans of the OS Plan 9 from Bell Labs
 help / color / mirror / Atom feed
* [9fans] APE isinf()
@ 2013-02-11 21:46 Jeff Sickel
  2013-02-11 21:50 ` erik quanstrom
  2013-02-11 22:40 ` Federico G. Benavento
  0 siblings, 2 replies; 3+ messages in thread
From: Jeff Sickel @ 2013-02-11 21:46 UTC (permalink / raw)
  To: 9fans

Here's a little issue I found w/ the new Python 2.7.3 port:

acme# cd /n/sources/plan9/
acme# grep isinf */include/ape/math.h
arm/include/ape/math.h:#define isinf(x) isInf(x)
mips/include/ape/math.h:#define isinf(x) isInf(x)
power/include/ape/math.h:#define isinf(x) isInf(x)
sparc/include/ape/math.h:#define isinf(x) isInf(x)

isinf() is missing from 386 & amd64 APE headers.

Of course, having only one argument to isInf(x) will give
you the following if you actually try to use it:

/usr/jas/src/cmd/cpython/Plan9/../Python/bltinmodule.c:2173[stdin:16039] not enough function arguments: isInf

So it's probably safe to just have

#define isinf(x) isInf(x,0)

in /*/include/ape/math.h





^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [9fans] APE isinf()
  2013-02-11 21:46 [9fans] APE isinf() Jeff Sickel
@ 2013-02-11 21:50 ` erik quanstrom
  2013-02-11 22:40 ` Federico G. Benavento
  1 sibling, 0 replies; 3+ messages in thread
From: erik quanstrom @ 2013-02-11 21:50 UTC (permalink / raw)
  To: 9fans

> So it's probably safe to just have
>
> #define isinf(x) isInf(x,0)
>
> in /*/include/ape/math.h

yes, let's do that.

i'd like to move all the function definitions in /$objtype/include/math.h
to /sys/include/ape/math.h and move the system dependent goo to
/$objtype/include/u.h.  this would require doing the same thing to
/sys/include/ape/u.h.

- erik



^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [9fans] APE isinf()
  2013-02-11 21:46 [9fans] APE isinf() Jeff Sickel
  2013-02-11 21:50 ` erik quanstrom
@ 2013-02-11 22:40 ` Federico G. Benavento
  1 sibling, 0 replies; 3+ messages in thread
From: Federico G. Benavento @ 2013-02-11 22:40 UTC (permalink / raw)
  To: Fans of the OS Plan 9 from Bell Labs


On Feb 11, 2013, at 6:46 PM, Jeff Sickel <jas@corpus-callosum.com> wrote:

> So it's probably safe to just have
>
> #define isinf(x) isInf(x,0)

yes, I've used that in the past.

---
Federico G. Benavento
benavento@gmail.com






^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2013-02-11 22:40 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-02-11 21:46 [9fans] APE isinf() Jeff Sickel
2013-02-11 21:50 ` erik quanstrom
2013-02-11 22:40 ` Federico G. Benavento

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).