X-Git-Url: http://source.jalview.org/gitweb/?a=blobdiff_plain;f=src%2Fjalview%2Fgui%2FJalview2XML.java;h=6d55c3299db85af1c1e3a7724e22f55715b0bca1;hb=6bd9630607bd38aaf9b42738a0ae90de8fe5c6e8;hp=5a13723499a52162dfb028846069bbf27a98cc54;hpb=f8efe615d1c7d5c23fd838c1abcd77be542b154c;p=jalview.git diff --git a/src/jalview/gui/Jalview2XML.java b/src/jalview/gui/Jalview2XML.java old mode 100755 new mode 100644 index 5a13723..6d55c32 --- a/src/jalview/gui/Jalview2XML.java +++ b/src/jalview/gui/Jalview2XML.java @@ -1,6 +1,6 @@ /* - * Jalview - A Sequence Alignment Editor and Viewer (Version 2.6) - * Copyright (C) 2010 J Procter, AM Waterhouse, G Barton, M Clamp, S Searle + * Jalview - A Sequence Alignment Editor and Viewer (Version 2.7) + * Copyright (C) 2011 J Procter, AM Waterhouse, J Engelhardt, LM Lui, G Barton, M Clamp, S Searle * * This file is part of Jalview. * @@ -38,6 +38,7 @@ import jalview.datamodel.SequenceI; import jalview.schemabinding.version2.*; import jalview.schemes.*; import jalview.structure.StructureSelectionManager; +import jalview.util.Platform; import jalview.util.jarInputStreamProvider; /** @@ -48,7 +49,7 @@ import jalview.util.jarInputStreamProvider; * will be :) * * @author $author$ - * @version $Revision$ + * @version $Revision: 1.134 $ */ public class Jalview2XML { @@ -426,9 +427,9 @@ public class Jalview2XML object.setCreationDate(new java.util.Date(System.currentTimeMillis())); object.setVersion(jalview.bin.Cache.getProperty("VERSION")); - jalview.datamodel.AlignmentI jal = av.alignment; + jalview.datamodel.AlignmentI jal = av.getAlignment(); - if (av.hasHiddenRows) + if (av.hasHiddenRows()) { jal = jal.getHiddenSequences().getFullAlignment(); } @@ -497,15 +498,13 @@ public class Jalview2XML jseq.setId(id); // jseq id should be a string not a number - if (av.hasHiddenRows) + if (av.hasHiddenRows()) { - jseq.setHidden(av.alignment.getHiddenSequences().isHidden(jds)); + jseq.setHidden(av.getAlignment().getHiddenSequences().isHidden(jds)); - if (av.hiddenRepSequences != null - && av.hiddenRepSequences.containsKey(jal.getSequenceAt(i))) + if (av.isHiddenRepSequence(jal.getSequenceAt(i))) { - jalview.datamodel.SequenceI[] reps = ((jalview.datamodel.SequenceGroup) av.hiddenRepSequences - .get(jal.getSequenceAt(i))).getSequencesInOrder(jal); + jalview.datamodel.SequenceI[] reps = av.getRepresentedSequences(jal.getSequenceAt(i)).getSequencesInOrder(jal); for (int h = 0; h < reps.length; h++) { @@ -612,7 +611,8 @@ public class Jalview2XML for (int smap = 0; smap < jmol.jmb.sequence[peid].length; smap++) { - if (jal.findIndex(jmol.jmb.sequence[peid][smap]) > -1) +// if (jal.findIndex(jmol.jmb.sequence[peid][smap]) > -1) + if (jds==jmol.jmb.sequence[peid][smap]) { StructureState state = new StructureState(); state.setVisible(true); @@ -706,9 +706,9 @@ public class Jalview2XML jms.addJSeq(jseq); } - if (av.hasHiddenRows) + if (av.hasHiddenRows()) { - jal = av.alignment; + jal = av.getAlignment(); } // SAVE MAPPINGS if (jal.getCodonFrames() != null && jal.getCodonFrames().length > 0) @@ -764,7 +764,7 @@ public class Jalview2XML { TreePanel tp = (TreePanel) frames[t]; - if (tp.treeCanvas.av.alignment == jal) + if (tp.treeCanvas.av.getAlignment() == jal) { Tree tree = new Tree(); tree.setTitle(tp.getTitle()); @@ -842,7 +842,7 @@ public class Jalview2XML an.setCentreColLabels(aa[i].centreColLabels); an.setScaleColLabels(aa[i].scaleColLabel); an.setShowAllColLabels(aa[i].showAllColLabels); - + if (aa[i].graph > 0) { an.setGraph(true); @@ -864,8 +864,8 @@ public class Jalview2XML an.setLabel(aa[i].label); - if (aa[i] == av.quality || aa[i] == av.conservation - || aa[i] == av.consensus || aa[i].autoCalculated) + if (aa[i] == av.getAlignmentQualityAnnot() || aa[i] == av.getAlignmentConservationAnnotation() + || aa[i] == av.getAlignmentConsensusAnnotation() || aa[i].autoCalculated) { // new way of indicating autocalculated annotation - an.setAutoCalculated(aa[i].autoCalculated); @@ -909,7 +909,9 @@ public class Jalview2XML an.addAnnotationElement(ae); if (aa[i].autoCalculated) { - // only write one non-null entry into the annotation row - sufficient to get the visualization attributes necessary to display data + // only write one non-null entry into the annotation row - + // sufficient to get the visualization attributes necessary to + // display data continue; } } @@ -1117,40 +1119,43 @@ public class Jalview2XML Vector settingsAdded = new Vector(); Object gstyle = null; GraduatedColor gcol = null; - for (int ro = 0; ro < renderOrder.length; ro++) + if (renderOrder != null) { - gstyle = ap.seqPanel.seqCanvas.getFeatureRenderer() - .getFeatureStyle(renderOrder[ro]); - Setting setting = new Setting(); - setting.setType(renderOrder[ro]); - if (gstyle instanceof GraduatedColor) - { - gcol = (GraduatedColor) gstyle; - setting.setColour(gcol.getMaxColor().getRGB()); - setting.setMincolour(gcol.getMinColor().getRGB()); - setting.setMin(gcol.getMin()); - setting.setMax(gcol.getMax()); - setting.setColourByLabel(gcol.isColourByLabel()); - setting.setAutoScale(gcol.isAutoScale()); - setting.setThreshold(gcol.getThresh()); - setting.setThreshstate(gcol.getThreshType()); - } - else + for (int ro = 0; ro < renderOrder.length; ro++) { - setting.setColour(ap.seqPanel.seqCanvas.getFeatureRenderer() - .getColour(renderOrder[ro]).getRGB()); - } + gstyle = ap.seqPanel.seqCanvas.getFeatureRenderer() + .getFeatureStyle(renderOrder[ro]); + Setting setting = new Setting(); + setting.setType(renderOrder[ro]); + if (gstyle instanceof GraduatedColor) + { + gcol = (GraduatedColor) gstyle; + setting.setColour(gcol.getMaxColor().getRGB()); + setting.setMincolour(gcol.getMinColor().getRGB()); + setting.setMin(gcol.getMin()); + setting.setMax(gcol.getMax()); + setting.setColourByLabel(gcol.isColourByLabel()); + setting.setAutoScale(gcol.isAutoScale()); + setting.setThreshold(gcol.getThresh()); + setting.setThreshstate(gcol.getThreshType()); + } + else + { + setting.setColour(ap.seqPanel.seqCanvas.getFeatureRenderer() + .getColour(renderOrder[ro]).getRGB()); + } - setting.setDisplay(av.featuresDisplayed - .containsKey(renderOrder[ro])); - float rorder = ap.seqPanel.seqCanvas.getFeatureRenderer().getOrder( - renderOrder[ro]); - if (rorder > -1) - { - setting.setOrder(rorder); + setting.setDisplay(av.featuresDisplayed + .containsKey(renderOrder[ro])); + float rorder = ap.seqPanel.seqCanvas.getFeatureRenderer() + .getOrder(renderOrder[ro]); + if (rorder > -1) + { + setting.setOrder(rorder); + } + fs.addSetting(setting); + settingsAdded.addElement(renderOrder[ro]); } - fs.addSetting(setting); - settingsAdded.addElement(renderOrder[ro]); } // Make sure we save none displayed feature settings @@ -1199,7 +1204,7 @@ public class Jalview2XML } - if (av.hasHiddenColumns) + if (av.hasHiddenColumns()) { if (av.getColumnSelection() == null || av.getColumnSelection().getHiddenColumns() == null) @@ -1817,9 +1822,9 @@ public class Jalview2XML } ; out.close(); - - alreadyLoadedPDB.put(pdbId, outFile.getAbsolutePath()); - return outFile.getAbsolutePath(); + String t=outFile.getAbsolutePath(); + alreadyLoadedPDB.put(pdbId, t); + return t; } else { @@ -2088,7 +2093,7 @@ public class Jalview2XML /** * store any annotations which forward reference a group's ID */ - Hashtable> groupAnnotRefs = new Hashtable>(); + Hashtable> groupAnnotRefs = new Hashtable>(); if (vamsasSet.getAnnotationCount() > 0) { @@ -2111,7 +2116,9 @@ public class Jalview2XML an[i].setAutoCalculated(true); } } - if (autoForView || (an[i].hasAutoCalculated() && an[i].isAutoCalculated())) { + if (autoForView + || (an[i].hasAutoCalculated() && an[i].isAutoCalculated())) + { // remove ID - we don't recover annotation from other views for // view-specific annotation an[i].setId(null); @@ -2168,12 +2175,13 @@ public class Jalview2XML if (an[i].getGraph()) { - float llim=0,hlim=0; - // if (autoForView || an[i].isAutoCalculated()) { - // hlim=11f; - // } + float llim = 0, hlim = 0; + // if (autoForView || an[i].isAutoCalculated()) { + // hlim=11f; + // } jaa = new jalview.datamodel.AlignmentAnnotation(an[i].getLabel(), - an[i].getDescription(), anot, llim, hlim, an[i].getGraphType()); + an[i].getDescription(), anot, llim, hlim, + an[i].getGraphType()); jaa.graphGroup = an[i].getGraphGroup(); @@ -2185,9 +2193,11 @@ public class Jalview2XML an[i].getThresholdLine().getColour()))); } - if (autoForView || an[i].isAutoCalculated()) { - // Hardwire the symbol display line to ensure that labels for histograms are displayed - jaa.hasText=true; + if (autoForView || an[i].isAutoCalculated()) + { + // Hardwire the symbol display line to ensure that labels for + // histograms are displayed + jaa.hasText = true; } } else @@ -2218,10 +2228,12 @@ public class Jalview2XML // and make a note of any group association if (an[i].getGroupRef() != null && an[i].getGroupRef().length() > 0) { - ArrayList aal=groupAnnotRefs.get(an[i].getGroupRef()); - if (aal==null) { + ArrayList aal = groupAnnotRefs + .get(an[i].getGroupRef()); + if (aal == null) + { aal = new ArrayList(); - groupAnnotRefs.put(an[i].getGroupRef(),aal); + groupAnnotRefs.put(an[i].getGroupRef(), aal); } aal.add(jaa); } @@ -2254,10 +2266,12 @@ public class Jalview2XML if (jaa.autoCalculated) { autoAlan.add(new JvAnnotRow(i, jaa)); - } else + } + else // if (!autoForView) { - // add autocalculated group annotation and any user created annotation for the view + // add autocalculated group annotation and any user created annotation + // for the view al.addAnnotation(jaa); } } @@ -2352,19 +2366,24 @@ public class Jalview2XML .get(groups[i].getId()); if (jaal != null) { - for (jalview.datamodel.AlignmentAnnotation jaa:jaal) { + for (jalview.datamodel.AlignmentAnnotation jaa : jaal) + { jaa.groupRef = sg; if (jaa.autoCalculated) { - // match up and try to set group autocalc alignment row for this annotation - if (jaa.label.startsWith("Consensus for ")) { + // match up and try to set group autocalc alignment row for this + // annotation + if (jaa.label.startsWith("Consensus for ")) + { sg.setConsensus(jaa); } - // match up and try to set group autocalc alignment row for this annotation - if (jaa.label.startsWith("Conservation for ")) { + // match up and try to set group autocalc alignment row for this + // annotation + if (jaa.label.startsWith("Conservation for ")) + { sg.setConservationRow(jaa); - } } + } } } } @@ -2575,10 +2594,11 @@ public class Jalview2XML { x, y, width, height }, "", new Hashtable(), new boolean[] { false, false, true } }); - // Legacy pre-2.7 conversion JAL-823 : - // do not assume any view has to be linked for colour by sequence + // Legacy pre-2.7 conversion JAL-823 : + // do not assume any view has to be linked for colour by + // sequence } - + // assemble String[] { pdb files }, String[] { id for each // file }, orig_fileloc, SequenceI[][] {{ seqs_file 1 }, { // seqs_file 2}, boolean[] { @@ -2593,7 +2613,9 @@ public class Jalview2XML .getStructureState(s).getColourwithAlignPanel() : false; // default for pre-2.7 projects is that Jmol colouring is enabled - ((boolean[])jmoldat[3])[2] &=ids[p].getStructureState(s).hasColourByJmol() ? ids[p].getStructureState(s).getColourByJmol() : true; + ((boolean[]) jmoldat[3])[2] &= ids[p].getStructureState(s) + .hasColourByJmol() ? ids[p].getStructureState(s) + .getColourByJmol() : true; if (((String) jmoldat[1]).length() < ids[p] .getStructureState(s).getContent().length()) @@ -2602,22 +2624,33 @@ public class Jalview2XML jmoldat[1] = ids[p].getStructureState(s).getContent(); } } - Object[] seqstrmaps = (Object[]) ((Hashtable) jmoldat[2]) - .get(ids[p].getFile()); - if (seqstrmaps == null) + if (ids[p].getFile() != null) { - ((Hashtable) jmoldat[2]).put( - new File(ids[p].getFile()).toString(), - seqstrmaps = new Object[] - { pdbFile, ids[p].getId(), new Vector(), - new Vector() }); + File mapkey=new File(ids[p].getFile()); + Object[] seqstrmaps = (Object[]) ((Hashtable) jmoldat[2]) + .get(mapkey); + if (seqstrmaps == null) + { + ((Hashtable) jmoldat[2]).put( + mapkey, + seqstrmaps = new Object[] + { pdbFile, ids[p].getId(), new Vector(), + new Vector() }); + } + if (!((Vector) seqstrmaps[2]).contains(seq)) + { + ((Vector) seqstrmaps[2]).addElement(seq); + // ((Vector)seqstrmaps[3]).addElement(n) : + // in principle, chains + // should be stored here : do we need to + // TODO: store and recover seq/pdb_id : + // chain mappings + } } - if (!((Vector) seqstrmaps[2]).contains(seq)) + else { - ((Vector) seqstrmaps[2]).addElement(seq); - // ((Vector)seqstrmaps[3]).addElement(n) : in principle, chains - // should be stored here : do we need to - // TODO: store and recover seq/pdb_id : chain mappings + errorMessage = ("The Jmol views in this project were imported\nfrom an older version of Jalview.\nPlease review the sequence colour associations\nin the Colour by section of the Jmol View menu.\n\nIn the case of problems, see note at\nhttp://issues.jalview.org/browse/JAL-747"); + warn(errorMessage); } } } @@ -2632,8 +2665,8 @@ public class Jalview2XML Object[] svattrib = entry.getValue(); int[] geom = (int[]) svattrib[0]; String state = (String) svattrib[1]; - Hashtable oldFiles = (Hashtable) svattrib[2]; - final boolean useinJmolsuperpos = ((boolean[]) svattrib[3])[0], usetoColourbyseq = ((boolean[]) svattrib[3])[1], jmolColouring=((boolean[])svattrib[3])[2]; + Hashtable oldFiles = (Hashtable) svattrib[2]; + final boolean useinJmolsuperpos = ((boolean[]) svattrib[3])[0], usetoColourbyseq = ((boolean[]) svattrib[3])[1], jmolColouring = ((boolean[]) svattrib[3])[2]; int x = geom[0], y = geom[1], width = geom[2], height = geom[3]; // collate the pdbfile -> sequence mappings from this view Vector pdbfilenames = new Vector(); @@ -2696,6 +2729,8 @@ public class Jalview2XML { newFileLoc = new StringBuffer(); } + do { + // look for next filename in load statement newFileLoc.append(state.substring(cp, ncp = (state.indexOf("\"", ncp + 1) + 1))); String oldfilenam = state.substring(ncp, @@ -2703,9 +2738,8 @@ public class Jalview2XML // recover the new mapping data for this old filename // have to normalize filename - since Jmol and jalview do filename // translation differently. - Object[] filedat = oldFiles.get(new File(oldfilenam) - .toString()); - newFileLoc.append(((String) filedat[0])); + Object[] filedat = oldFiles.get(new File(oldfilenam)); + newFileLoc.append(Platform.escapeString((String) filedat[0])); pdbfilenames.addElement((String) filedat[0]); pdbids.addElement((String) filedat[1]); seqmaps.addElement((SequenceI[]) ((Vector) filedat[2]) @@ -2713,6 +2747,7 @@ public class Jalview2XML newFileLoc.append("\""); cp = ecp + 1; // advance beyond last \" and set cursor so we can // look for next file statement. + } while ((ncp=state.indexOf("/*file*/",cp))>-1); } if (cp > 0) { @@ -2725,7 +2760,7 @@ public class Jalview2XML .print("Ignoring incomplete Jmol state for PDB ids: "); newFileLoc = new StringBuffer(state); newFileLoc.append("; load append "); - for (String id : oldFiles.keySet()) + for (File id : oldFiles.keySet()) { // add this and any other pdb files that should be present in // the viewer @@ -2789,8 +2824,8 @@ public class Jalview2XML try { sview = new AppJmol(pdbf, id, sq, alf.alignPanel, - useinJmolsuperpos, usetoColourbyseq, jmolColouring, fileloc, - rect, vid); + useinJmolsuperpos, usetoColourbyseq, + jmolColouring, fileloc, rect, vid); } catch (OutOfMemoryError ex) { new OOMWarning("restoring structure view for PDB id " @@ -2823,7 +2858,7 @@ public class Jalview2XML // add mapping for sequences in this view to an already open Jmol // instance - for (String id : oldFiles.keySet()) + for (File id : oldFiles.keySet()) { // add this and any other pdb files that should be present in the // viewer @@ -2831,9 +2866,8 @@ public class Jalview2XML String pdbFile = (String) filedat[0]; SequenceI[] seq = (SequenceI[]) ((Vector) filedat[2]) .toArray(new SequenceI[0]); - StructureSelectionManager.getStructureSelectionManager() - .setMapping(seq, null, pdbFile, - jalview.io.AppletFormatAdapter.FILE); + ((AppJmol) comp).jmb.ssm.setMapping(seq, null, pdbFile, + jalview.io.AppletFormatAdapter.FILE); ((AppJmol) comp).jmb.addSequenceForStructFile(pdbFile, seq); } // and add the AlignmentPanel's reference to the Jmol view @@ -2848,7 +2882,8 @@ public class Jalview2XML } if (usetoColourbyseq) { - ((AppJmol) comp).useAlignmentPanelForColourbyseq(ap, !jmolColouring); + ((AppJmol) comp).useAlignmentPanelForColourbyseq(ap, + !jmolColouring); } else { @@ -2875,7 +2910,7 @@ public class Jalview2XML for (int i = 0; i < JSEQ.length; i++) { - af.viewport.setSequenceColour(af.viewport.alignment.getSequenceAt(i), + af.viewport.setSequenceColour(af.viewport.getAlignment().getSequenceAt(i), new java.awt.Color(JSEQ[i].getColour())); } @@ -2886,7 +2921,7 @@ public class Jalview2XML jalview.gui.AlignViewport av = (jalview.gui.AlignViewport) viewportsAdded .get(uniqueSeqSetId); - af.viewport.sequenceSetID = uniqueSeqSetId; + af.viewport.setSequenceSetId(uniqueSeqSetId); if (av != null) { // propagate shared settings to this new view @@ -2976,18 +3011,18 @@ public class Jalview2XML else if (view.getBgColour().startsWith("Annotation")) { // int find annotation - if (af.viewport.alignment.getAlignmentAnnotation() != null) + if (af.viewport.getAlignment().getAlignmentAnnotation() != null) { - for (int i = 0; i < af.viewport.alignment + for (int i = 0; i < af.viewport.getAlignment() .getAlignmentAnnotation().length; i++) { - if (af.viewport.alignment.getAlignmentAnnotation()[i].label + if (af.viewport.getAlignment().getAlignmentAnnotation()[i].label .equals(view.getAnnotationColours().getAnnotation())) { - if (af.viewport.alignment.getAlignmentAnnotation()[i] + if (af.viewport.getAlignment().getAlignmentAnnotation()[i] .getThreshold() == null) { - af.viewport.alignment.getAlignmentAnnotation()[i] + af.viewport.getAlignment().getAlignmentAnnotation()[i] .setThreshold(new jalview.datamodel.GraphLine(view .getAnnotationColours().getThreshold(), "Threshold", java.awt.Color.black) @@ -2999,7 +3034,7 @@ public class Jalview2XML .equals("None")) { cs = new AnnotationColourGradient( - af.viewport.alignment.getAlignmentAnnotation()[i], + af.viewport.getAlignment().getAlignmentAnnotation()[i], new java.awt.Color(view.getAnnotationColours() .getMinColour()), new java.awt.Color(view .getAnnotationColours().getMaxColour()), @@ -3009,7 +3044,7 @@ public class Jalview2XML .startsWith("ucs")) { cs = new AnnotationColourGradient( - af.viewport.alignment.getAlignmentAnnotation()[i], + af.viewport.getAlignment().getAlignmentAnnotation()[i], GetUserColourScheme(jms, view .getAnnotationColours().getColourScheme()), view.getAnnotationColours().getAboveThreshold()); @@ -3017,7 +3052,7 @@ public class Jalview2XML else { cs = new AnnotationColourGradient( - af.viewport.alignment.getAlignmentAnnotation()[i], + af.viewport.getAlignment().getAlignmentAnnotation()[i], ColourSchemeProperty.getColour(al, view .getAnnotationColours().getColourScheme()), view.getAnnotationColours().getAboveThreshold()); @@ -3040,7 +3075,7 @@ public class Jalview2XML * if * (view.getAnnotationColours().getColourScheme().equals("None" * )) { sg.cs = new AnnotationColourGradient( - * af.viewport.alignment.getAlignmentAnnotation()[i], new + * af.viewport.getAlignment().getAlignmentAnnotation()[i], new * java.awt.Color(view.getAnnotationColours(). * getMinColour()), new * java.awt.Color(view.getAnnotationColours(). @@ -3049,7 +3084,7 @@ public class Jalview2XML */ { sg.cs = new AnnotationColourGradient( - af.viewport.alignment.getAlignmentAnnotation()[i], + af.viewport.getAlignment().getAlignmentAnnotation()[i], sg.cs, view.getAnnotationColours() .getAboveThreshold()); } @@ -3071,7 +3106,7 @@ public class Jalview2XML if (cs != null) { cs.setThreshold(view.getPidThreshold(), true); - cs.setConsensus(af.viewport.hconsensus); + cs.setConsensus(af.viewport.getSequenceConsensusHash()); } } @@ -3097,8 +3132,8 @@ public class Jalview2XML } if (view.hasIgnoreGapsinConsensus()) { - af.viewport.ignoreGapsInConsensusCalculation = view - .getIgnoreGapsinConsensus(); + af.viewport.setIgnoreGapsConsensus(view + .getIgnoreGapsinConsensus(), null); } if (view.hasFollowHighlight()) { @@ -3119,11 +3154,11 @@ public class Jalview2XML } if (view.hasShowSequenceLogo()) { - af.viewport.showSequenceLogo = view.getShowSequenceLogo(); + af.viewport.setShowSequenceLogo(view.getShowSequenceLogo()); } else { - af.viewport.showSequenceLogo = false; + af.viewport.setShowSequenceLogo(false); } if (view.hasShowDbRefTooltip()) { @@ -3230,14 +3265,14 @@ public class Jalview2XML Desktop.addInternalFrame(af, view.getTitle(), view.getWidth(), view.getHeight()); af.alignPanel.updateAnnotation(false); // recompute any autoannotation - reorderAutoannotation(af,al,autoAlan); + reorderAutoannotation(af, al, autoAlan); return af; } private void reorderAutoannotation(AlignFrame af, Alignment al, ArrayList autoAlan) { - // copy over visualization settings for autocalculated annotation in the + // copy over visualization settings for autocalculated annotation in the // view if (al.getAlignmentAnnotation() != null) { @@ -3268,32 +3303,34 @@ public class Jalview2XML if (valan != null) { // delete the auto calculated row from the alignment - al.deleteAnnotation(al.getAlignmentAnnotation()[h],false); + al.deleteAnnotation(al.getAlignmentAnnotation()[h], false); hSize--; h--; if (valan != nullAnnot) { - if (jalan!=valan.template) { + if (jalan != valan.template) + { // newly created autoannotation row instance // so keep a reference to the visible annotation row // and copy over all relevant attributes - if (valan.template.graphHeight >= 0) - - { - jalan.graphHeight = valan.template.graphHeight; - } - jalan.visible = valan.template.visible; + if (valan.template.graphHeight >= 0) + + { + jalan.graphHeight = valan.template.graphHeight; + } + jalan.visible = valan.template.visible; } reorder.add(new JvAnnotRow(valan.order, jalan)); } } } } - int s=0,srt[] = new int[reorder.size()]; + int s = 0, srt[] = new int[reorder.size()]; JvAnnotRow[] rws = new JvAnnotRow[reorder.size()]; - for (JvAnnotRow jvar:reorder) { + for (JvAnnotRow jvar : reorder) + { rws[s] = jvar; - srt[s++]=jvar.order; + srt[s++] = jvar.order; } reorder.clear(); jalview.util.QuickSort.sort(srt, rws); @@ -3685,11 +3722,11 @@ public class Jalview2XML af.closeMenuItem_actionPerformed(true); /* - * if(ap.av.alignment.getAlignmentAnnotation()!=null) { for(int i=0; - * i