Submitted by Richard Kelly on Thu, 29 Apr 1999
17:27:14 -0700
Thank you for producing such a useful tool! We have
started using
PERCEPS on our project and are very pleased with it.
Before doing so,
however, I had to make some minor changes to your
code. Some are bug
fixes, and some are just "nice to have". Please
consider
incorporating some or all of these changes in a
future version of
PERCEPS.
A diff of your version 3.4.1 vs. our modified version
follows this
summary of changes. To find each change, search for
RMK1, RMK2, etc.
RMK1. Added ".hpp" to default suffixes list.
RMK2. Made {if brief} and {if detail} return true is
documentation
is available from a merged (-m option function
definition.
RMK3. Extended -u option to work for all comments,
not just comments
before a class definition.
RMK4. Function-name pattern matching doesn't permit
code like this:
//: brief comment
// detail comment
int
Foo::
bar() {
// code
}
That's because the whitespace after the :: messes up
the pattern
match. I made fixes in three places, but they're not
very elegant.
RMK5. Some legal C++ function names contain regexp
meta-characters,
which must be quoted in subsequent matches. I fixed
one such match,
but there may be more that should be fixed. Without
the fix, names
like operator+, operator*, etc. get messed up.
Diff to version 3.4.1