[Mesquitelist] CategoricalData.setStateName bug?

[Wayne Maddison]

Sam,
Yes, a bug -- thanks for noticing it!  This is not in any release 
version, and so won't have affected users.  Since MesquiteList is 
intended primarily for biologists using Mesquite, it's best to put 
developer discussion of bugs on the MesquiteDev mailing list.
Thanks
Wayne

At 1:06 PM -0500 5.11.2009, Sam Donnelly wrote:
>Hi, 
>
>It looks to me like there's a bug 
>in mesquite.categ.lib.CategoricalData.setStateName(...) in at least 
>revision 6240 of the trunk: 
>
>Here's the first couple of lines 
>	/** sets state name of state "state" of character ic to name */
>	void setStateName(int ic, int state, String name) { //todo: 
>check if in bounds
>		if (ic< 1 || ic >= numChars)
>			return;
>
>
>Should that be "ic < 0" instead of "ic < 1"? Otherwise, it doesn't 
>seem to allow setting state names for the first column of the 
>matrix. 
>
>thanks!
>
>Sam
>
>_______________________________________________
>Mesquitelist mailing list
>Mesquitelist at mesquiteproject.org
>http://mesquiteproject.org/mailman/listinfo/mesquitelist


-- 
--------------------------------------------------------
Wayne Maddison
  Professor and Canada Research Chair
          Depts. of Zoology and Botany and
          Biodiversity Research Centre
  & Director
          Beaty Biodiversity Museum
  6270 University Boulevard
  University of British Columbia
  Vancouver, BC  V6T 1Z4   Canada

email: wmaddisn at interchange.ubc.ca   FAX: +1 604 822-2416

Mesquite: http://mesquiteproject.org
MacClade: http://macclade.org
Salticidae:  http://salticidae.org
Tree of Life: http://tolweb.org
Beaty Biodiversity Museum: http://beatymuseum.ubc.ca
Home page: http://salticidae.org/wpm/home.html