Branch: refs/heads/master Home: https://github.com/Starlink/starjava Commit: 6c4a3beed49a4d81fe98adb4e0c9e4eb85966204 https://github.com/Starlink/starjava/commit/6c4a3beed49a4d81fe98adb4e0c9e4eb85966204 Author: Mark Taylor <[log in to unmask]> Date: 2015-02-26 (Thu, 26 Feb 2015) Changed paths: M topcat/src/main/uk/ac/starlink/topcat/plot2/SkyPlotWindow.java M ttools/src/main/uk/ac/starlink/ttools/plot2/geom/SkySys.java Log Message: ----------- ttools: remove duplicated code Commit: 000359fe8156c4cd02586f0e91d1b9428305cd85 https://github.com/Starlink/starjava/commit/000359fe8156c4cd02586f0e91d1b9428305cd85 Author: Mark Taylor <[log in to unmask]> Date: 2015-02-26 (Thu, 26 Feb 2015) Changed paths: M topcat/src/main/uk/ac/starlink/topcat/plot2/BasicCoordLayerControl.java A topcat/src/main/uk/ac/starlink/topcat/plot2/ConfigStyler.java M topcat/src/main/uk/ac/starlink/topcat/plot2/FormControl.java M topcat/src/main/uk/ac/starlink/topcat/plot2/FormLayerControl.java M topcat/src/main/uk/ac/starlink/topcat/plot2/FormStylePanel.java M topcat/src/main/uk/ac/starlink/topcat/plot2/PlotPositionSpecifier.java M ttools/src/main/uk/ac/starlink/ttools/plot2/SurfaceFactory.java M ttools/src/main/uk/ac/starlink/ttools/plot2/config/BooleanConfigKey.java M ttools/src/main/uk/ac/starlink/ttools/plot2/config/ChoiceConfigKey.java M ttools/src/main/uk/ac/starlink/ttools/plot2/config/ConfigException.java M ttools/src/main/uk/ac/starlink/ttools/plot2/config/DoubleConfigKey.java M ttools/src/main/uk/ac/starlink/ttools/plot2/config/OptionConfigKey.java M ttools/src/main/uk/ac/starlink/ttools/plot2/config/SkySysConfigKey.java M ttools/src/main/uk/ac/starlink/ttools/plot2/config/SubrangeConfigKey.java M ttools/src/main/uk/ac/starlink/ttools/plot2/config/TimeConfigKey.java M ttools/src/main/uk/ac/starlink/ttools/plot2/geom/CubeSurfaceFactory.java M ttools/src/main/uk/ac/starlink/ttools/plot2/geom/PlaneSurfaceFactory.java M ttools/src/main/uk/ac/starlink/ttools/plot2/geom/SkySurfaceFactory.java M ttools/src/main/uk/ac/starlink/ttools/plot2/layer/BinSizer.java M ttools/src/main/uk/ac/starlink/ttools/plot2/layer/FunctionPlotter.java M ttools/src/main/uk/ac/starlink/ttools/plot2/layer/LabelPlotter.java M ttools/src/main/uk/ac/starlink/ttools/plot2/layer/ShapePlotter.java M ttools/src/main/uk/ac/starlink/ttools/plot2/task/AbstractPlot2Task.java Log Message: ----------- ttools: ConfigException is now checked, not unchecked Reparent ConfigException to inherit from Exception not RuntimeException. It's really necessary to have it as a checked exception to make it clear when you have to handle it. It was only unchecked before because I was in a tearing hurry when I put did it in the first place. Added some code in topcat to behave sensibly when such an exception is caught, but this is not much tested so far; currently these are not often thrown. Commit: 25ae21c0e5fd0dc1f0f890d76671dfdd72929d07 https://github.com/Starlink/starjava/commit/25ae21c0e5fd0dc1f0f890d76671dfdd72929d07 Author: Mark Taylor <[log in to unmask]> Date: 2015-02-26 (Thu, 26 Feb 2015) Changed paths: M topcat/src/main/uk/ac/starlink/topcat/plot2/ConfigSpecifier.java Log Message: ----------- topcat: provide ConfigMap check hook in ConfigSpecifier ConfigSpecifier subclasses can now override the new checkConfig method and throw a ConfigException if something is wrong with the map. The class will then do something about it to ensure that the eventual returned config maps are safe to use. Commit: 8977b52fe72cd30768e8de386ef62acb6721821d https://github.com/Starlink/starjava/commit/8977b52fe72cd30768e8de386ef62acb6721821d Author: Mark Taylor <[log in to unmask]> Date: 2015-02-26 (Thu, 26 Feb 2015) Changed paths: M topcat/src/docs/sun253.xml M topcat/src/main/uk/ac/starlink/topcat/plot2/ConfigSpecifier.java M topcat/src/main/uk/ac/starlink/topcat/plot2/CubeAxisController.java M topcat/src/main/uk/ac/starlink/topcat/plot2/HistogramAxisController.java M topcat/src/main/uk/ac/starlink/topcat/plot2/PlaneAxisController.java M topcat/src/main/uk/ac/starlink/topcat/plot2/ShaderControl.java M topcat/src/main/uk/ac/starlink/topcat/plot2/TimeAxisController.java M ttools/src/main/uk/ac/starlink/ttools/plot2/PlotUtil.java Log Message: ----------- topcat: check and fix axis ranges before it's too late Axis range values manually entered by users in Min/Max fields are now checked in a consistent way before use. Previously if the user entered values with the wrong sense (min>=max) it could lead to uncaught exceptions for the geometric axes, though the Aux axis checked these and dealt with them using custom code. Compare: https://github.com/Starlink/starjava/compare/e9a6ff56fbcd...8977b52fe72c