[Masterview-devel] Code review request

Deb Lewis
Sun, 08 Oct 2006 18:41:30 -0000

Jeff - please code review the changes I''m proposing to make to
analyzer.rb
before I submit that; I ended up doing more than just cosmetic name-changing
and I''m not sure how good how test coverage is in this area, so
I''d like you
to check before I commit.

Objective: getting rid of the MasterView::NamespacePrefix constant(s), I
don''t think a module-level constant is the right way to expose that,
esp.
since we now have 2 flavors of namespace defaults (mv: and mvx:).

rubyforge.org/var/svn/masterview/branches/djl-analyzer-changes-preview/analy
zer.rb - I put out a copy from the trunk and then stored changes so you
should be able to diff the two versions there in the branch.

Thanks.

~ Deb