From: Jim Procter <jprocter@issues.jalview.org>
Date: Thu, 26 May 2016 15:01:55 +0000 (+0100)
Subject: JAL-2121 refactor todo
X-Git-Tag: Release_2_10_0~212^2~2
X-Git-Url: http://source.jalview.org/gitweb/?a=commitdiff_plain;h=b92809e042da3d7445392119c159ef8730b8a136;p=jalview.git

JAL-2121 refactor todo
---

diff --git a/src/jalview/api/DBRefEntryI.java b/src/jalview/api/DBRefEntryI.java
index 9415745..490b21a 100644
--- a/src/jalview/api/DBRefEntryI.java
+++ b/src/jalview/api/DBRefEntryI.java
@@ -2,6 +2,7 @@ package jalview.api;
 
 import jalview.datamodel.Mapping;
 
+//JBPComment: this is a datamodel API - so it should be in datamodel (it's a peer of SequenceI)
 
 public interface DBRefEntryI
 {
diff --git a/src/jalview/api/SiftsClientI.java b/src/jalview/api/SiftsClientI.java
index e2fac14..527ae17 100644
--- a/src/jalview/api/SiftsClientI.java
+++ b/src/jalview/api/SiftsClientI.java
@@ -29,6 +29,8 @@ import jalview.xml.binding.sifts.Entry.Entity;
 import java.util.HashMap;
 import java.util.HashSet;
 
+// JBPComment: this isn't a top-level Jalview API - should be in its own package api
+
 public interface SiftsClientI
 {
   /**