delete unnecessary #include lines

558ab49f · rsc · 99b11b6c · 558ab49f · 558ab49f · 558ab49f
--- a/8253pit.c
+++ b/8253pit.c
 #include "types.h"
-#include "x86.h"
 #include "defs.h"
 #include "traps.h"
+#include "x86.h"
 // Register definitions for the Intel
 // 8253/8254/82C54 Programmable Interval Timer (PIT).

--- a/bio.c
+++ b/bio.c
@@ -22,11 +22,8 @@
 //     and needs to be written to disk.
 #include "types.h"
-#include "param.h"
-#include "x86.h"
-#include "mmu.h"
-#include "proc.h"
 #include "defs.h"
+#include "param.h"
 #include "spinlock.h"
 #include "buf.h"

--- a/console.c
+++ b/console.c
 #include "types.h"
-#include "x86.h"
-#include "traps.h"
 #include "defs.h"
+#include "param.h"
+#include "traps.h"
 #include "spinlock.h"
 #include "dev.h"
-#include "param.h"
 #include "mmu.h"
 #include "proc.h"
+#include "x86.h"
 #include "kbd.h"
 #define CRTPORT 0x3d4

--- a/cuth
+++ b/cuth
+#!/usr/bin/perl
+$| = 1;
+sub writefile($@){
+	my ($file, @lines) = @_;
+	sleep(1);
+	open(F, ">$file") || die "open >$file: $!";
+	print F @lines;
+	close(F);
+}
+# Cut out #include lines that don't contribute anything.
+for($i=0; $i<@ARGV; $i++){
+	$file = $ARGV[$i];
+	if(!open(F, $file)){
+		print STDERR "open $file: $!\n";
+		next;
+	}
+	@lines = <F>;
+	close(F);
+	$obj = "$file.o";
+	$obj =~ s/\.c\.o$/.o/;
+	system("touch $file");
+	if(system("make CC='gcc -Werror' $obj >/dev/null 2>\&1") != 0){
+		print STDERR "make $obj failed: $rv\n";
+		next;
+	}
+	system("cp $file =$file");
+	for($j=@lines-1; $j>=0; $j--){
+		if($lines[$j] =~ /^#include/){
+			$old = $lines[$j];
+			$lines[$j] = "/* CUT-H */\n";
+			writefile($file, @lines);
+			if(system("make CC='gcc -Werror' $obj >/dev/null 2>\&1") != 0){
+				$lines[$j] = $old;
+			}else{
+				print STDERR "$file $old";
+			}
+		}
+	}
+	writefile($file, grep {!/CUT-H/} @lines);
+	system("rm =$file");
+}
--- a/exec.c
+++ b/exec.c
@@ -4,7 +4,6 @@
 #include "proc.h"
 #include "defs.h"
 #include "x86.h"
-#include "fs.h"
 #include "elf.h"
 int

--- a/file.c
+++ b/file.c
 #include "types.h"
-#include "stat.h"
-#include "param.h"
-#include "x86.h"
-#include "mmu.h"
-#include "proc.h"
 #include "defs.h"
+#include "param.h"
 #include "file.h"
 #include "spinlock.h"
 #include "dev.h"
-#include "fs.h"
-#include "fsvar.h"
 struct devsw devsw[NDEV];
 struct spinlock file_table_lock;

--- a/fs.c
+++ b/fs.c
@@ -11,12 +11,11 @@
 // are in sysfile.c.
 #include "types.h"
-#include "stat.h"
+#include "defs.h"
 #include "param.h"
-#include "x86.h"
+#include "stat.h"
 #include "mmu.h"
 #include "proc.h"
-#include "defs.h"
 #include "spinlock.h"
 #include "buf.h"
 #include "fs.h"

--- a/ide.c
+++ b/ide.c
 // Simple PIO-based (non-DMA) IDE driver code.
 #include "types.h"
+#include "defs.h"
 #include "param.h"
 #include "mmu.h"
 #include "proc.h"
-#include "defs.h"
 #include "x86.h"
 #include "traps.h"
 #include "spinlock.h"

--- a/init.c
+++ b/init.c
 #include "types.h"
 #include "stat.h"
 #include "user.h"
-#include "fs.h"
 #include "fcntl.h"
 // init: The initial user-level program

--- a/ioapic.c
+++ b/ioapic.c
@@ -2,9 +2,7 @@
 // http://www.intel.com/design/chipsets/datashts/29056601.pdf
 #include "types.h"
-#include "mp.h"
 #include "defs.h"
-#include "x86.h"
 #include "traps.h"
 #define IOAPIC  0xFEC00000   // Default physical address of IO APIC

--- a/kalloc.c
+++ b/kalloc.c
@@ -5,12 +5,9 @@
 // One reason the page size is 4k is that the x86 segment size
 // granularity is 4k.
-#include "param.h"
 #include "types.h"
 #include "defs.h"
 #include "param.h"
-#include "mmu.h"
-#include "proc.h"
 #include "spinlock.h"
 struct spinlock kalloc_lock;

--- a/lapic.c
+++ b/lapic.c
@@ -2,12 +2,7 @@
 // See Chapter 8 & Appendix C of Intel processor manual volume 3.
 #include "types.h"
-#include "defs.h"
-#include "param.h"
-#include "x86.h"
 #include "traps.h"
-#include "mmu.h"
-#include "proc.h"
 // Local APIC registers, divided by 4 for use as uint[] indices.
 #define ID      (0x0020/4)   // ID

--- a/main.c
+++ b/main.c
 #include "types.h"
+#include "defs.h"
 #include "param.h"
 #include "mmu.h"
 #include "proc.h"
-#include "defs.h"
 #include "x86.h"
-#include "traps.h"
-#include "syscall.h"
-#include "elf.h"
-#include "param.h"
-#include "spinlock.h"
 extern char edata[], end[];
 void bootothers(void);
 // Bootstrap processor starts running C code here.
-// This is called main0 not main so that it can have
-// a void return type.  Gcc can't handle functions named
-// main that don't return int.  Really.
 void
 main0(void)
 {
@@ -39,7 +31,7 @@ main0(void)
  asm volatile("movl %0, %%ebp" : : "r" (cpus[bcpu].mpstack+MPSTACK));
  lapic_init(bcpu);
-  cprintf("\\ncpu%d: starting xv6\\n\\n", cpu());
+  cprintf("\ncpu%d: starting xv6\n\n", cpu());
  pinit();         // process table
  binit();         // buffer cache
@@ -69,7 +61,7 @@ main0(void)
 void
 mpmain(void)
 {
-  cprintf("cpu%d: starting\\n", cpu());
+  cprintf("cpu%d: starting\n", cpu());
  idtinit();
  lapic_init(cpu());
  setupsegs(0);

--- a/mkfs.c
+++ b/mkfs.c
@@ -5,7 +5,6 @@
 #include <fcntl.h>
 #include <assert.h>
 #include "types.h"
-#include "param.h"
 #include "fs.h"
 int nblocks = 995;
@@ -55,7 +54,7 @@ int
 main(int argc, char *argv[])
 {
  int i, cc, fd;
-  uint bn, rootino, inum, off;
+  uint rootino, inum, off;
  struct dirent de;
  char buf[512];
  struct dinode din;

--- a/mp.c
+++ b/mp.c
 // http://developer.intel.com/design/pentium/datashts/24201606.pdf
 #include "types.h"
-#include "mp.h"
 #include "defs.h"
 #include "param.h"
+#include "mp.h"
 #include "x86.h"
-#include "traps.h"
 #include "mmu.h"
 #include "proc.h"

--- a/picirq.c
+++ b/picirq.c
 #include "types.h"
 #include "x86.h"
 #include "traps.h"
-#include "defs.h"
 // I/O Addresses of the two 8259A programmable interrupt controllers
 #define IO_PIC1         0x20    // Master (IRQs 0-7)

--- a/pipe.c
+++ b/pipe.c
 #include "types.h"
+#include "defs.h"
 #include "param.h"
-#include "x86.h"
 #include "mmu.h"
 #include "proc.h"
-#include "defs.h"
 #include "file.h"
 #include "spinlock.h"

--- a/proc.c
+++ b/proc.c
 #include "types.h"
+#include "defs.h"
+#include "param.h"
 #include "mmu.h"
 #include "x86.h"
-#include "param.h"
-#include "file.h"
 #include "proc.h"
-#include "defs.h"
 #include "spinlock.h"
 struct spinlock proc_table_lock;

--- a/sh.c
+++ b/sh.c
 #include "types.h"
 #include "stat.h"
 #include "user.h"
-#include "fs.h"
 #include "fcntl.h"
 #define BUFSIZ  512

--- a/spinlock.c
+++ b/spinlock.c
@@ -2,9 +2,9 @@
 #include "types.h"
 #include "defs.h"
+#include "param.h"
 #include "x86.h"
 #include "mmu.h"
-#include "param.h"
 #include "proc.h"
 #include "spinlock.h"

--- a/string.c
+++ b/string.c
 #include "types.h"
-#include "defs.h"
 void*
 memset(void *dst, int c, uint n)

--- a/syscall.c
+++ b/syscall.c
 #include "types.h"
-#include "stat.h"
+#include "defs.h"
 #include "param.h"
 #include "mmu.h"
 #include "proc.h"
-#include "defs.h"
 #include "x86.h"
-#include "traps.h"
 #include "syscall.h"
-#include "spinlock.h"
-#include "buf.h"
-#include "fs.h"
-#include "fsvar.h"
-#include "elf.h"
-#include "file.h"
-#include "fcntl.h"
 // User code makes a system call with INT T_SYSCALL.
 // System call number in %eax.

--- a/sysfile.c
+++ b/sysfile.c
 #include "types.h"
-#include "stat.h"
+#include "defs.h"
 #include "param.h"
+#include "stat.h"
 #include "mmu.h"
 #include "proc.h"
-#include "defs.h"
-#include "x86.h"
-#include "traps.h"
-#include "syscall.h"
-#include "spinlock.h"
-#include "buf.h"
 #include "fs.h"
 #include "fsvar.h"
 #include "file.h"

--- a/sysproc.c
+++ b/sysproc.c
 #include "types.h"
-#include "stat.h"
+#include "defs.h"
 #include "param.h"
 #include "mmu.h"
 #include "proc.h"
-#include "defs.h"
-#include "x86.h"
-#include "traps.h"
-#include "syscall.h"
-#include "spinlock.h"
-#include "buf.h"
-#include "fs.h"
-#include "fsvar.h"
-#include "elf.h"
-#include "file.h"
-#include "fcntl.h"
 int
 sys_fork(void)

--- a/trap.c
+++ b/trap.c
 #include "types.h"
+#include "defs.h"
 #include "param.h"
 #include "mmu.h"
 #include "proc.h"
-#include "defs.h"
 #include "x86.h"
 #include "traps.h"
-#include "syscall.h"
 #include "spinlock.h"
 // Interrupt descriptor table (shared by all CPUs).

--- a/umalloc.c
+++ b/umalloc.c
@@ -3,8 +3,8 @@
 #include "user.h"
 #include "param.h"
-// Memory allocator by Kernighan and Ritchie, The C programming Language,
+// Memory allocator by Kernighan and Ritchie,
-// 2nd ed.  Section 8.7.
+// The C programming Language, 2nd ed.  Section 8.7.
 typedef long Align;

--- a/zombie.c
+++ b/zombie.c
@@ -8,8 +8,6 @@
 int
 main(void)
 {
-  int i;
  if(fork() > 0)
    sleep(5);  // Let child exit before parent.
  exit();