[Insight-developers] PhilipsRecImageIO

[Bill Lorensen]

We should definitely turn on modules like this for the factory builds.
The modules that depend on VTK are turned on for some nightlies I
believe.

Bill

On Tue, Dec 18, 2012 at 8:04 AM,  <M.Staring at lumc.nl> wrote:
> Hi Bill,
>
> Thanks for the confirmation. What is worrisome is that this could never have compiled, also not on the system this patch was developed. Should the three gerrit build systems be modified, so that as many as possible switches are ON?
>
> Regards, Marius
>
> -----Original Message-----
> From: Bill Lorensen [mailto:bill.lorensen at gmail.com]
> Sent: dinsdag 18 december 2012 14:01
> To: Staring, M. (LKEB)
> Cc: insight-developers at itk.org
> Subject: Re: [Insight-developers] PhilipsRecImageIO
>
> Marius,
>
> I also just discovered this on a system I have not build on for a while. Must be the only system I have with Philips IO turned on.
>
> I'll submit a gerrit patch. Actually, thisDic->Clear() should be thisDic.Clear().
>
> Bill
>
>
> On Tue, Dec 18, 2012 at 5:08 AM,  <M.Staring at lumc.nl> wrote:
>> Hi Kent,
>>
>>
>>
>> I noticed that the change 966fcb62c859229f9b421403c4faf3e6ef5e2b81
>> included the line:
>>
>>
>>
>> metaDict->Clear();
>>
>> to the function ReadImageInformation from itkPhilipsRECImageIO.cxx.
>>
>>
>>
>> This gives a compile error:
>>
>>
>>
>> /usr/local/toolkits/ITK/latest_release/src/Modules/IO/PhilipsREC/src/itkPhilipsRECImageIO.cxx:
>> In member function âvirtual void
>> itk::PhilipsRECImageIO::ReadImageInformation()â:
>>
>> /usr/local/toolkits/ITK/latest_release/src/Modules/IO/PhilipsREC/src/itkPhilipsRECImageIO.cxx:744:3:
>> error: âmetaDictâ was not declared in this scope
>>
>>
>>
>> Did you not mean:
>>
>> thisDic->Clear();
>>
>> ?
>>
>>
>>
>> If this was not discovered at the dashboard, then probably the option
>> Module_ITKIOPhilipsREC was not enabled. By default it is OFF.
>>
>>
>>
>> Regards, Marius
>>
>>
>>
>>
>>
>> Marius Staring, PhD
>>
>> Division of Image Processing (LKEB)
>>
>> Department of Radiology
>>
>> Leiden University Medical Center
>>
>> PO Box 9600, 2300 RC Leiden, The Netherlands
>>
>> phone: +31 (0)71 526 2137, fax: +31 (0)71 524 8256
>>
>> m.staring at lumc.nl
>>
>>
>>
>>
>> _______________________________________________
>> Powered by www.kitware.com
>>
>> Visit other Kitware open-source projects at
>> http://www.kitware.com/opensource/opensource.html
>>
>> Kitware offers ITK Training Courses, for more information visit:
>> http://kitware.com/products/protraining.php
>>
>> Please keep messages on-topic and check the ITK FAQ at:
>> http://www.itk.org/Wiki/ITK_FAQ
>>
>> Follow this link to subscribe/unsubscribe:
>> http://www.itk.org/mailman/listinfo/insight-developers
>>
>
>
>
> --
> Unpaid intern in BillsBasement at noware dot com



-- 
Unpaid intern in BillsBasement at noware dot com