[Insight-users] typo in recently merged code caught by wrapping

Alexandre GOUAILLARD agouaillard at gmail.com
Thu Dec 1 12:57:32 EST 2011


reviewed ;)

On Fri, Dec 2, 2011 at 1:51 AM, Johnson, Hans J <hans-johnson at uiowa.edu> wrote:
> Alex,
>
> Sorry about that.  This also exposed that these member functions were not
> part of our test suite.
>
> I've submitted a gerrit patch that addresses both the bug typo and the
> test coverage.  http://review.source.kitware.com/3417
>
> I'll merge as soon as dashboards pass and someone reviews.
>
>
> Hans
>
> --
> Hans J. Johnson, Ph.D.
> hans-johnson at uiowa.edu
> Assistant Professor of Psychiatry
> University of Iowa Carver College of Medicine
> W278 GH, 200 Hawkins Drive
>
> Iowa City, Iowa 52242
> Phone:  319-353-8587
>
>
>
>
>
>
>
> -----Original Message-----
> From: Alexandre GOUAILLARD <agouaillard at gmail.com>
> Date: Fri, 2 Dec 2011 00:19:37 +0800
> To: Hans Johnson <hans-johnson at uiowa.edu>, ITK <insight-users at itk.org>
> Subject: typo in recently merged code caught by wrapping
>
> dear hans,
>
> your commit on  itkMattesMutualInformationImageToImageMetric.h
> introduced an error in the code line 230 (repeated m_) that was caught
> by "Durian" build:
> http://www.cdash.org/CDash/viewBuildError.php?buildid=1778539
>
> Unfortunatly I will not have time to work on this this week. Could you
> take a look?
>
> alex.
>
>
>
> ________________________________
> Notice: This UI Health Care e-mail (including attachments) is covered by the Electronic Communications Privacy Act, 18 U.S.C. 2510-2521, is confidential and may be legally privileged.  If you are not the intended recipient, you are hereby notified that any retention, dissemination, distribution, or copying of this communication is strictly prohibited.  Please reply to the sender that you have received the message in error, then delete it.  Thank you.
> ________________________________


More information about the Insight-users mailing list