Hi Grant,<br><br>I fixed GradientImage in r2956. If you notice any others, please fix and commit.<br><br>The problem is that I frequently use the command "Clean up" in Eclipse which formats source code according to our conventions. One of the things it does is add final where possible, which normally is very nice, but in this one case causes problems. A workaround might be to add methods such as "setWidth(int)" and "setHeight(int)" to GradientImage so that the parameters cannot legally be final.<br>
<br>-Curtis<br><br><div class="gmail_quote">On Tue, May 24, 2011 at 5:07 PM, Grant B. Harris <span dir="ltr"><<a href="mailto:gharris@mbl.edu">gharris@mbl.edu</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
I discovered the hard way - don't declare @Parameter fields final.<br>
<br>
In the GradientImage plugin, the width and height are declared thus:<br>
@Parameter(min = "1")<br>
private final int width = 512;<br>
<br>
@Parameter(min = "1")<br>
private final int height = 512;<br>
<br>
The input dialog gives you two spinners to change these values. It even remembers the last values input. But then the plugin uses the initial final values set when declared... which makes sense. But the input dialog does not recognize, and there are not warnings that the inputs will be ignored.<br>
<br>
So, if you really want the user to be able to change a parameter, best to not final !<br>
<br>
-- Grant<br>
<br>
_______________________________________________<br>
ImageJ-devel mailing list<br>
<a href="mailto:ImageJ-devel@imagejdev.org" target="_blank">ImageJ-devel@imagejdev.org</a><br>
<a href="http://imagejdev.org/mailman/listinfo/imagej-devel" target="_blank">http://imagejdev.org/mailman/listinfo/imagej-devel</a><br>
</blockquote></div><br>