Ah, that is a bug indeed. On 12/09/2014 12:04 PM, Rostislav Khlebnikov wrote: > Hi Sascha, > > thanks for answer! Actually, LocalFile was the first thing I tried - but > it didn't link, so I just thought that something is missing/forgotten. > Adding MITK_CORE_EXPORT to the LocalFile inner class solves the problem > (MSVS 2013). > > All best, > Rostislav. > > On 09/12/2014 10:56, Sascha Zelzer wrote: >> Hi Rostislav, >> >> the documentation for the new I/O concept still needs improvement and >> unfortunately it is actually wrong in the AbstractFileWriter class >> about the GetLocalFile() method. For writers, there is a nested helper >> class called "LocalFile" which can be instantiated and which provides >> a local file name to write to. The instance of LocalFile must be kept >> alive during writing and on destruction, it copies the file contents >> to the output stream. See for example >> >> http://mitk.org/git/?p=MITK.git;a=blob;f=Core/Code/Internal/mitkSurfaceStlIO.cpp;h=942d2792771d7440fecb93ecb2db12f39684c373;hb=HEAD#l139 >> >> >> I agree that in the AbstractFileIO case, it might be confusing that >> for reading GetLocalFileName() but for writing LocalFile(this) must be >> used. >> >> Best, >> Sascha >> >> On 12/08/2014 07:37 PM, Rostislav Khlebnikov wrote: >>> Hi guys, >>> >>> I'm moving to the new save/load concept (changing from 10 files [reader, >>> writer, source, readerfactory, writerfactory headers and sources] to 2 >>> is very tempting) and it seems that some functionality was forgotten. >>> While for AbstractFileReader the GetLocalFileName function is >>> implemented for writers not supporting input from stream, for >>> AbstractFileWriter, its counterpart which is mentions in the doc string >>> - GetLocalFile() - is not. >>> I also believe, that AbstractFileIO class should have a couple of >>> functions for "forwarding" such calls, e.g. GetReaderLocalFileName() and >>> GetWriterLocalFileName(). >>> >>> I guess for now I can just use GetOutputLocation(), but something tells >>> me that this functionality was just forgotten. Would you like me to file >>> a bug? >>> >>> Thanks, >>> Rostislav. >>> >>> ------------------------------------------------------------------------------ >>> >>> Download BIRT iHub F-Type - The Free Enterprise-Grade BIRT Server >>> from Actuate! Instantly Supercharge Your Business Reports and Dashboards >>> with Interactivity, Sharing, Native Excel Exports, App Integration & >>> more >>> Get technology previously reserved for billion-dollar corporations, FREE >>> http://pubads.g.doubleclick.net/gampad/clk?id=164703151&iu=/4140/ostg.clktrk >>> >>> _______________________________________________ >>> mitk-users mailing list >>> [email protected] >>> https://lists.sourceforge.net/lists/listinfo/mitk-users
------------------------------------------------------------------------------ Download BIRT iHub F-Type - The Free Enterprise-Grade BIRT Server from Actuate! Instantly Supercharge Your Business Reports and Dashboards with Interactivity, Sharing, Native Excel Exports, App Integration & more Get technology previously reserved for billion-dollar corporations, FREE http://pubads.g.doubleclick.net/gampad/clk?id=164703151&iu=/4140/ostg.clktrk _______________________________________________ mitk-users mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/mitk-users
