nitpicks

c54c7926 · rsc · 564f787e · c54c7926 · c54c7926 · c54c7926
--- a/console.c
+++ b/console.c
@@ -4,7 +4,7 @@
 #include "spinlock.h"
 struct spinlock console_lock;
-int paniced = 0;
+int panicked = 0;
 int use_console_lock = 0;
 /*
@@ -31,9 +31,9 @@ real_cons_putc(int c)
  unsigned short *crt = (unsigned short *) 0xB8000; // base of CGA memory
  int ind;
-  if(paniced){
+  if(panicked){
    cli();
-    while(1)
+    for(;;)
      ;
  }
@@ -161,7 +161,7 @@ panic(char *s)
  cprintf("panic: ");
  cprintf(s, 0);
  cprintf("\n", 0);
-  paniced = 1; // freeze other CPU
+  panicked = 1; // freeze other CPU
-  while(1)
+  for(;;)
    ;
 }
--- a/defs.h
+++ b/defs.h
@@ -79,8 +79,8 @@ int pipe_write(struct pipe *p, char *addr, int n);
 int pipe_read(struct pipe *p, char *addr, int n);
 // fd.c
-int fd_ualloc();
+int fd_ualloc(void);
-struct fd * fd_alloc();
+struct fd * fd_alloc(void);
 void fd_close(struct fd *);
 int fd_read(struct fd *fd, char *addr, int n);
 int fd_write(struct fd *fd, char *addr, int n);

--- a/fd.c
+++ b/fd.c
@@ -15,7 +15,7 @@ struct fd fds[NFD];
 * allocate a file descriptor number for curproc.
 */
 int
-fd_ualloc()
+fd_ualloc(void)
 {
  int fd;
  struct proc *p = curproc[cpu()];
@@ -29,7 +29,7 @@ fd_ualloc()
 * allocate a file descriptor structure
 */
 struct fd *
-fd_alloc()
+fd_alloc(void)
 {
  int i;

--- a/kalloc.c
+++ b/kalloc.c
@@ -23,7 +23,7 @@ struct run {
 };
 struct run *freelist;
-void ktest();
+void ktest(void);
 /*
 * initialize free list of physical pages. this code

--- a/lapic.c
+++ b/lapic.c
@@ -108,7 +108,7 @@ lapic_write(int r, int data)
 void
-lapic_timerinit()
+lapic_timerinit(void)
 {
  cprintf("%d: init timer\n", cpu());
  lapic_write(LAPIC_TDCR, LAPIC_X1);
@@ -118,7 +118,7 @@ lapic_timerinit()
 }
 void
-lapic_timerintr()
+lapic_timerintr(void)
 {
  cprintf("%d: timer interrupt!\n", cpu());
  lapic_write (LAPIC_EOI, 0);

--- a/mp.c
+++ b/mp.c
@@ -117,7 +117,7 @@ mp_detect(void)
 }
 void
-mp_init()
+mp_init(void)
 { 
  int r;
  uint8_t *p, *e;
@@ -193,7 +193,7 @@ mp_bcpu(void)
 extern void mpmain(void);
 void
-mp_startthem()
+mp_startthem(void)
 {
  extern uint8_t _binary_bootother_start[], _binary_bootother_size[];
  extern int main();

--- a/proc.c
+++ b/proc.c
@@ -311,7 +311,7 @@ proc_kill(int pid)
 // Exited processes remain in the zombie state 
 // until their parent calls wait() to find out they exited.
 void
-proc_exit()
+proc_exit(void)
 {
  struct proc *p;
  struct proc *cp = curproc[cpu()];

--- a/trap.c
+++ b/trap.c
@@ -10,11 +10,11 @@
 struct Gatedesc idt[256];
 extern unsigned vectors[]; /* vectors.S, array of 256 entry point addresses */
-extern void trapenter();
+extern void trapenter(void);
-extern void trapenter1();
+extern void trapenter1(void);
 void
-tvinit()
+tvinit(void)
 {
  int i;
@@ -25,7 +25,7 @@ tvinit()
 }
 void
-idtinit()
+idtinit(void)
 {
  lidt(idt, sizeof idt);
 }

--- a/user1.c
+++ b/user1.c
@@ -3,7 +3,7 @@
 char buf[32];
 int
-main()
+main(void)
 {
  int pid, fds[2], n;

--- a/userfs.c
+++ b/userfs.c
@@ -5,7 +5,7 @@
 char buf[1024];
 int
-main()
+main(void)
 {
  puts("userfs running\n");
  block();

--- a/usertests.c
+++ b/usertests.c
@@ -5,7 +5,7 @@ char buf[2048];
 // simple fork and pipe read/write
 void
-pipe1()
+pipe1(void)
 {
  int fds[2], pid;
  int seq = 0, i, n, cc, total;
@@ -27,10 +27,7 @@ pipe1()
    close(fds[1]);
    total = 0;
    cc = 1;
-    while(1){
+    while((n = read(fds[0], buf, cc)) > 0){
-      n = read(fds[0], buf, cc);
-      if(n < 1)
-        break;
      for(i = 0; i < n; i++){
        if((buf[i] & 0xff) != (seq++ & 0xff)){
          panic("pipe1 oops 2\n");
@@ -52,19 +49,19 @@ pipe1()
 // meant to be run w/ at most two CPUs
 void
-preempt()
+preempt(void)
 {
  int pid1, pid2, pid3;
  int pfds[2];
  pid1 = fork();
  if(pid1 == 0)
-    while(1)
+    for(;;)
      ;
  pid2 = fork();
  if(pid2 == 0)
-    while(1)
+    for(;;)
      ;
  pipe(pfds);
@@ -74,7 +71,7 @@ preempt()
    if(write(pfds[1], "x", 1) != 1)
      panic("preempt write error");
    close(pfds[1]);
-    while(1)
+    for(;;)
      ;
  }
@@ -95,7 +92,7 @@ preempt()
 // try to find any races between exit and wait
 void
-exitwait()
+exitwait(void)
 {
  int i, pid;
@@ -118,7 +115,7 @@ exitwait()
 }
 int
-main()
+main(void)
 {
  puts("usertests starting\n");