home *** CD-ROM | disk | FTP | other *** search
/ NetNews Usenet Archive 1992 #20 / NN_1992_20.iso / spool / comp / lang / perl / 5811 < prev    next >
Encoding:
Text File  |  1992-09-09  |  1.8 KB  |  58 lines
  1. Newsgroups: comp.lang.perl
  2. Path: sparky!uunet!fmrco!fmrco!asherman
  3. From: asherman@laser.fmrco.com (Aaron Sherman)
  4. Subject: Re: processing of "diff -c" for code reviews
  5. In-Reply-To: utashiro@sran230.sra.co.jp's message of Mon, 7 Sep 1992 08:07:38 GMT
  6. Message-ID: <ASHERMAN.92Sep9161406@laser.fmrco.com>
  7. Sender: news@fmrco.uucp
  8. Reply-To: asherman@fmrco.COM
  9. Organization: I-Kinetics, 19 Bishop-Allen Dr., Cambridge, MA
  10. References: <17580@autodesk.COM> <Bu77ws.B7K@sran230.sra.co.jp>
  11. Date: Wed, 9 Sep 1992 21:14:06 GMT
  12. Lines: 44
  13.  
  14.  
  15. >>>>> utashiro@sran230.sra.co.jp (Kazumasa Utashiro) said:
  16.  
  17. utashiro> dansmith@Autodesk.COM (Daniel Smith) writes:
  18.  
  19. >>     Looking for something that can take a context diff (like diff -c)
  20. >> and process it so that it can be useful in code reviews.  Any pointers
  21. >> to scripts already written out there?
  22.  
  23. utashiro> I hope that you'll like my cdif and sdif.  Cdif is
  24. utashiro> post-processor of diff which hilights modified part of text
  25. utashiro> by changing text attributes.  Sdif is a clone of System V
  26. utashiro> sdiff command but more powerfull except lack of -o option.
  27. utashiro> Sdif can handle cdif output.
  28.  
  29. If the original poster wanted something to look at at a conference
  30. table with others around, this may not work, since getting a terminal
  31. to the meeting might not be trivial.
  32.  
  33. I've actually found that GNU's diff -u (unified diff) is quite
  34. readable. Here is a sample:
  35.  
  36. --- c.c.old    Wed Sep  9 16:11:36 1992
  37. +++ c.c        Wed Sep  9 16:12:26 1992
  38. @@ -1,8 +1,9 @@
  39.  #include <stdio.h>
  40.  
  41. -main(argc,argv)
  42. -    int argc;
  43. -    char *argv[];
  44. +void
  45. +main(int argc,char *argv[])
  46.  {
  47.      printf("Hello, world!\n");
  48. +
  49. +    exit(0);
  50.  }
  51.  
  52.  
  53. --
  54. --------
  55. Disclaimer: I am solely responsible for the content of this message.
  56. The views expressed here may not be the views of I-Kinetics, Fidelity,
  57. any of the Fidelity-owned corporations or my mother.
  58.