page: fix reading from stdin

[russcox_gmail.com, 2009-08-12 22:15:58]

thanks for doing this.  a few small nits to clean up.
resubmit with -i 105097 so that it updates
the existing issue insted of creating a new one

http://codereview.appspot.com/105097/diff/1/2
File src/cmd/page/page.c (right):

http://codereview.appspot.com/105097/diff/1/2#newcode87
Line 87: void cleanup()
void
cleanup(void)

http://codereview.appspot.com/105097/diff/1/2#newcode183
Line 183: e = atexit(cleanup);
no need for this paranoia

atexit(cleanup);

is fine

[russcox_gmail.com, 2009-08-12 23:28:42]

http://codereview.appspot.com/105097/diff/8/9
File src/cmd/page/page.c (right):

http://codereview.appspot.com/105097/diff/8/9#newcode87
Line 87: void cleanup()
void
cleanup(void)

you missed this one

[russcox_gmail.com, 2009-08-18 06:16:19]

http://codereview.appspot.com/105097/diff/1005/12
File src/cmd/page/util.c (right):

http://codereview.appspot.com/105097/diff/1005/12#newcode53
Line 53: extern char tempfile[40];
sorry, i missed this before.
please put the definition in page.h
instead of using an extern declaration here.
having it in a central location means that
the compiler ensures it matches the 
real definition.

[russcox_gmail.com, 2009-08-20 16:43:04]

so close...

http://codereview.appspot.com/105097/diff/3001/3004
File src/cmd/page/util.c (right):

http://codereview.appspot.com/105097/diff/3001/3004#newcode53
Line 53: extern char tempfile[40];
now this one can be deleted.

[russcox_gmail.com, 2009-08-20 18:37:42]

codebot: apply

page: remove temporary file at program exit; fixes reading from stdin

[old.codebot, 2009-08-20 18:38:50]

# codebot: apply

applied.