[BioSQL-l] seqfeature.display_id

[Hilmar Lapp]

I just realized that display_id() in SeqFeature::Generic is just an 
alias for seqname(). I thought it's a genuine feature attribute.

This is really bad IMO, because seqname() clearly refers to the 
attached sequence, whereas display_id is easily mistaken for being a 
genuine property of the feature (in fact, it may even be useful to 
have a 'human-readable' name of a feature - but 'display_name' or 
something similar may be better suited names).

I vote for removing display_id() from SeqFeature::Generic. Who needs 
to have it named display_id, and cannot call it as seqname()?

	-hilmar
--
-------------------------------------------------------------
Hilmar Lapp                            email: lapp at gnf.org
GNF, San Diego, Ca. 92121              phone: +1-858-812-1757
-------------------------------------------------------------