9fans - fans of the OS Plan 9 from Bell Labs
 help / color / mirror / Atom feed
* [9fans] Documentation errors
@ 2007-11-12 21:26 Erik Dalén
  2007-11-12 22:14 ` geoff
  0 siblings, 1 reply; 6+ messages in thread
From: Erik Dalén @ 2007-11-12 21:26 UTC (permalink / raw)
  To: 9fans

Hi

Another bug report, hopefully more correct this time.

The function prototypes for a number of functions differ in the source
and the man pages, so far I've found removefid(), removereq(),
readstr() & readbuf() in the man pages 9p(2) and 9pfid(2).

-- 
Erik Dalén
dalen@socialisterna.org

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

* Re: [9fans] Documentation errors
  2007-11-12 21:26 [9fans] Documentation errors Erik Dalén
@ 2007-11-12 22:14 ` geoff
  2007-11-13  0:09   ` Erik Dalén
  2007-11-13  1:10   ` Erik Dalén
  0 siblings, 2 replies; 6+ messages in thread
From: geoff @ 2007-11-12 22:14 UTC (permalink / raw)
  To: 9fans

I've fixed 9pfid(2)'s declarations and descriptions of removefid and
removereq.

The declarations of readstr and readbuf in /sys/include/9p.h and
/sys/man/2/9p match each other and the definitions in
/sys/src/lib9p/util.c.


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

* Re: [9fans] Documentation errors
  2007-11-12 22:14 ` geoff
@ 2007-11-13  0:09   ` Erik Dalén
  2007-11-13  0:18     ` Charles Forsyth
  2007-11-13  1:10   ` Erik Dalén
  1 sibling, 1 reply; 6+ messages in thread
From: Erik Dalén @ 2007-11-13  0:09 UTC (permalink / raw)
  To: Fans of the OS Plan 9 from Bell Labs

On 11/12/07, geoff@plan9.bell-labs.com <geoff@plan9.bell-labs.com> wrote:
> I've fixed 9pfid(2)'s declarations and descriptions of removefid and
> removereq.
>
thanks

> The declarations of readstr and readbuf in /sys/include/9p.h and
> /sys/man/2/9p match each other and the definitions in
> /sys/src/lib9p/util.c.

I must have looked at the printed manual on those. It doesn't match
the source there, it says they return ulong instead of void.

-- 
Erik Dalén
erik.dalen@socialisterna.org

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

* Re: [9fans] Documentation errors
  2007-11-13  0:09   ` Erik Dalén
@ 2007-11-13  0:18     ` Charles Forsyth
  2007-11-13  0:19       ` Charles Forsyth
  0 siblings, 1 reply; 6+ messages in thread
From: Charles Forsyth @ 2007-11-13  0:18 UTC (permalink / raw)
  To: 9fans

> I must have looked at the printed manual on those. It doesn't match
> the source there, it says they return ulong instead of void.

the trouble with printing a manual is that the source changes
but the printed page does not.  yet.


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

* Re: [9fans] Documentation errors
  2007-11-13  0:18     ` Charles Forsyth
@ 2007-11-13  0:19       ` Charles Forsyth
  0 siblings, 0 replies; 6+ messages in thread
From: Charles Forsyth @ 2007-11-13  0:19 UTC (permalink / raw)
  To: 9fans

> the trouble with printing a manual is that the source changes
> but the printed page does not.  yet.

fortunately, most of it has been fairly stable in the external interfaces
since fourth edition.


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

* Re: [9fans] Documentation errors
  2007-11-12 22:14 ` geoff
  2007-11-13  0:09   ` Erik Dalén
@ 2007-11-13  1:10   ` Erik Dalén
  1 sibling, 0 replies; 6+ messages in thread
From: Erik Dalén @ 2007-11-13  1:10 UTC (permalink / raw)
  To: Fans of the OS Plan 9 from Bell Labs

Discovered another quite small one, on readdirfile() the second
argument is type uchar * in 9p.h but char * in the man page 9pfile(2).

On 11/12/07, geoff@plan9.bell-labs.com <geoff@plan9.bell-labs.com> wrote:
> I've fixed 9pfid(2)'s declarations and descriptions of removefid and
> removereq.
>
> The declarations of readstr and readbuf in /sys/include/9p.h and
> /sys/man/2/9p match each other and the definitions in
> /sys/src/lib9p/util.c.
>
>


-- 
Erik Dalén
erik.dalen@socialisterna.org

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

end of thread, other threads:[~2007-11-13  1:10 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2007-11-12 21:26 [9fans] Documentation errors Erik Dalén
2007-11-12 22:14 ` geoff
2007-11-13  0:09   ` Erik Dalén
2007-11-13  0:18     ` Charles Forsyth
2007-11-13  0:19       ` Charles Forsyth
2007-11-13  1:10   ` Erik Dalén

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).