Skip to main content

Please review: fix for issue #60

1 reply [Last post]
Anonymous

Folks,

Fix for Issue #60: https://cqme.dev.java.net/issues/show_bug.cgi?id=60
is ready for review.

Link for subversion to branch with fix:
https://cqme.dev.java.net/svn/cqme/branches/users/dtrounine/testExport/i...

Link to changeset:
http://fisheye4.cenqua.com/changelog/cqme/branches/users/dtrounine/testE...

Thanks,
Dmitri.

---------------------------------------------------------------------
To unsubscribe, e-mail: meframework-unsubscribe@cqme.dev.java.net
For additional commands, e-mail: meframework-help@cqme.dev.java.net

Reply viewing options

Select your preferred way to display the comments and click "Save settings" to activate your changes.
Vladimir Sizikov

Hi Dmitri,

I've reviewed the fix, and have no further comments, the changes are
good.
Please, commit.

Thanks,
--Vladimir

P.S. One minor comment - In the future, please include a bug synopsis in the email
subject and the body. It's easier to figure out which changes are for
what and which were already reviewed (otherwise, one has to remember
the bug numbers...) :)

On Tue, Mar 13, 2007 at 03:52:28PM +0300, Dmitri Trounine wrote:
> Folks,
>
> Fix for Issue #60: https://cqme.dev.java.net/issues/show_bug.cgi?id=60
> is ready for review.
>
> Link for subversion to branch with fix:
> https://cqme.dev.java.net/svn/cqme/branches/users/dtrounine/testExport/i...
>
> Link to changeset:
> http://fisheye4.cenqua.com/changelog/cqme/branches/users/dtrounine/testE...
>
> Thanks,
> Dmitri.
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: meframework-unsubscribe@cqme.dev.java.net
> For additional commands, e-mail: meframework-help@cqme.dev.java.net
>

---------------------------------------------------------------------
To unsubscribe, e-mail: meframework-unsubscribe@cqme.dev.java.net
For additional commands, e-mail: meframework-help@cqme.dev.java.net