diff options
Diffstat (limited to 'docs/sandbox/inoculated/readme.internal.txt')
-rw-r--r-- | docs/sandbox/inoculated/readme.internal.txt | 54 |
1 files changed, 54 insertions, 0 deletions
diff --git a/docs/sandbox/inoculated/readme.internal.txt b/docs/sandbox/inoculated/readme.internal.txt new file mode 100644 index 000000000..61d69591a --- /dev/null +++ b/docs/sandbox/inoculated/readme.internal.txt @@ -0,0 +1,54 @@ +// XXX do not distribute + +------ contents +05sd.Isberg40-43,76.pdf # not for distribution +BufferTest.java +CompileTime.java +Injection.java +MainFailure.java +RecordingInput.java +RoundTrip.java +RunTime.java +RuntimeWrites.java +StubReplace.java +buildRun.sh +readme.internal.txt # not for distribution +readme.txt + +------ summary of todo's +- consider moving to packages, combining PrinterStream, etc. +- use DOS linefeeds - check throughout (also line length) +- see XXX + - assess handling of one style mistake + - see if second mistake was actually in article - corrected in code + +------ fyi +- standard of care: show language, not problem +- formatting: lineation, line width, DOS linefeeds, etc. +- organization: + - code currently compiles/runs one at a time + and does not compile all at once b/c of + common fixtures (PrinterStream...) + - currently packages (com.xerox.printing) in base dir + +- Copyright/license: examples, ,but PARC Inc. +- article code unit flagged with "article page #" + +------ style fyi +- flagging style mistake in StubReplace.java: + + // XXX style mistake in article code + //pointcut printerStreamTestCalls() : call(* PrinterStream+.write()); + +- leaving CompileTime.java use of + in call for factory pointcut: + + call(Point+ SubPoint+.create(..)) + + - for static methods where the method name specification + involves no * but does reflect a factory naming convention + (and not polymorphism) + (though not restricting factory methods to being static) + + - for referring to the return value when I want to pick out + the type and all subtypes + |