<div dir="ltr"><div><div>Hi Tobias,<br><br>The <a href="https://github.com/scifio/scifio/blob/master/scifio/src/main/java/io/scif/img/cell/SCIFIOCell.java">SCIFIOCell</a> class needs to be serializable to allow the library we're using to persist opened cells to disk, and thus its superclass(es) needs to be serializable with non-final fields. <br>

<br>The only potential alternative I can think of would be to fully duplicate AbstractCell's fields and logic in SCIFIOCell... but that would still require the ArrayDataAccess to be serializable (do you see that as a potential problem, or only AbstractCell?).<br>

<br></div>In the <a href="https://github.com/imagej/imglib/commit/7bc7dde9b095d6e3544d5f781716c0dc06de439b">merge commit</a> I mentioned we ran the PerformanceBenchmark with/without implementing Serializable and saw no difference. We thought that was sufficient for merging the changes, but I'm sorry if we circumvented discussion.<br>

<br></div>If there are other tests you'd like to see run, or would like me to investigate duplicating AbstractCell's code, let me know.<br><br>Thanks,<br>Mark<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">

On Wed, Jul 3, 2013 at 10:40 AM, Tobias Pietzsch <span dir="ltr"><<a href="mailto:pietzsch@mpi-cbg.de" target="_blank">pietzsch@mpi-cbg.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Hi Mark,<br>
<br>
we just noticed the merge of the cell-serialization branch, that makes AbstractCell Serializable and therefore final field non-final.<br>
Actually Stephan Saalfeld noticed, I would have missed it probably. It would be nice if such changes could be discussed before they are merged to master.<br>
Did you do any tests to evaluate the performance impact of un-finalizing the fields.<br>
What is this needed for and are there other alternatives to achieve it instead of making AbstractCell Serializable.<br>
<br>
best regards,<br>
Tobias</blockquote></div><br></div>