home *** CD-ROM | disk | FTP | other *** search
/ Programmer's ROM - The Computer Language Library / programmersrom.iso / ada / style / styleins.rpt < prev    next >
Encoding:
Text File  |  1988-05-03  |  4.1 KB  |  96 lines
  1. -- Editorial comments by Rick Conn are included, and they take the
  2. -- form of Ada comments to distinguish them for the author's original
  3. -- text
  4.  
  5.  
  6. Hello.  I have transferred STYLE_CHECKER 1.0 (17 Apr 85) from SIMTEL20's
  7. Ada Software Repository and gotten it to run on my system (a VAX 750 running
  8. VMS with Dec's Ada compiler).  This is my report:
  9.  
  10. First, to be perfectly honest, I did this more as an exercise in getting
  11. software from the repository to run then for real usage.  We (Alsys Inc. and
  12. Alsys S.A.) have about half a million lines of Ada code (+/- 150000) and its
  13. 'styled' to our satisfication (largely) so I don't think we'll be running
  14. much of our code through this style checker.  I think its use will mainly
  15. be for organizations converting programmers from some other language, and
  16. the selection of statistics bears this out.
  17.  
  18. Yet, I am going to look carefully at the source for several reasons,
  19. including the possibility of reuse of modules (I LIKE the help facility,
  20. which is quite similar to the VMS facility) and for Ada hints and techniques
  21. in standard file I/O (none of our work is done with TEXT_IO).
  22.  
  23. I had the following problems creating STYLE_CHECKER on my system:
  24.  
  25. 1)  The file MAINTMAN.MEM in STYLEDOC.SRC is trashed.  Please check the
  26. copy on SIMTEL20 and see if it is what you thought it was.
  27.  
  28. -- I checked this file and found no problem other than it
  29. -- is double-spaced due to the formatter that was used to generate it.
  30.  
  31.  
  32. 2)  When STYLESRC.SRC is unpacked with the PAGER utility none of the
  33. filenames match the filenames in STYLE_CMP.CLI.  I had to tediously
  34. rename all the files (or I could have regenerated STYLE_CMP.CLI, either
  35. way it was a bore).
  36.  
  37. -- Yes, there is a file name conversion problem.  This is caused by
  38. -- very long files names being used on the DG, and these names had
  39. -- to be truncated in order to be delivered on a VAX VMS 2.x system.
  40.  
  41.  
  42. 3)  STYLE_CMP.CLI is missing compilations of HELP_INFO_SUPPORT_SPEC.ADA
  43. and HELP_INFO_SUPPORT_BODY.ADA.
  44.  
  45. 4)  The package CURRENT_EXCEPTION which is apparently provided in Rolm's
  46. Ada environment is non-standard and not available in Dec Ada.  I replaced
  47. it with:
  48.  
  49.    package current_exception is
  50.       name: constant := "<unknown exception>";
  51.    end current_exception;
  52.  
  53. and a file like this should be included in the source package with ample
  54. notes on what it is SUPPOSED to do.  (This package is a good idea!  I'm
  55. going to suggest we supply one with our compiler, and I'm going to suggest
  56. that Dec do the same, unless of course they have some method for doing it
  57. and I just don't know it.  Furthermore, I'd like to know what the full
  58. definition of the package in the Rolm system really is .. that is, what
  59. other facilties does it provide?)
  60.  
  61. -- NAME is the only function provided by CURRENT_EXCEPTION
  62.  
  63.  
  64. 5)  The main procedure STYLE_CHECKER.ADA still has a type declaration
  65. for UNIVERSAL_DATA_TYPE which isn't used, for the obvious reason that it
  66. would completely hide the predefined types from the rest of the program
  67. and the subunits of STYLE_CHECKER.  The declaration should be removed.
  68.  
  69. -- Such is not the case with ROLM Ada
  70.  
  71.  
  72. 6)  The file COMMENT_TOKEN is missing the token "in" at the procedure
  73. heading's parameter, leading to a conformance error vis-a-vis the procedure's
  74. specification in the parent unit.
  75.  
  76. -- This does not pose a problem with ROLM Ada
  77.  
  78.  
  79. 7)  Same as 6) for the file EXITING_BLOCK.
  80.  
  81. 8)  On VAX/VMS, Dec Ada, and a VT100 terminal, the Help package seems to
  82. fill all lines to 80 characters, leading to wrap around thus double spacing
  83. the lines on the screen, and a very slow help facility on a 1200 baud line.
  84. I'll have to investigate further, but I would guess lines aren't trimmed of
  85. trailing blanks before being send to text_io.put_line.
  86.  
  87. Anyway, with the above points noted and fixed (where applicable) the
  88. program compiles and runs (though I haven't run the test suite through
  89. it, just a few of its own source files).  I hope the sources on SIMTEL20
  90. are fixed, or perhaps this note can be added to the directory, in order
  91. to help anyone else who is going to try to use this program.
  92.  
  93. Dave Bakin
  94.  
  95. (Bakin -at mit-multics)
  96.