[GRASS5] PATCH: i.gensig

Brad Douglas rez at touchofmadness.com
Fri Jul 22 16:53:48 EDT 2005


On Fri, 2005-07-22 at 15:36 -0500, Daniel Calvelo Aros wrote:
> From: Markus Neteler <neteler at itc.it>
> Sent: Fri, 22 Jul 2005 09:44:30 +0200
> >
> > Brad,
> > 
> > I did a quick inspection - looks good to me.
> > 
> > Markus
> > 
> > On Thu, Jul 21, 2005 at 02:23:05PM -0700, Brad Douglas wrote:
> > > I need to clean up some old cruft I had not already committed to
> > > generate a clean, uncluttered patch for the tempfile issues.
> > > 
> > > Any objections to the attached patch?  Just some benign updates to
> > > i.gensig to do with message output and localization.
> 
> [...]
> > > -    fprintf (stderr, "Calculating class covariance matri%s ...",
> S->nsigs==1?"x":"ces");
> > > +    G_message(_("Calculating class covariance matri%s ..."), 
> > > +              S->nsigs==1 ? "x" : "ces");
> 
> Please correct these to have two distinct strings in singluar and plural, e.g.
> 
> if( S->nsigs == 1 ){
>   G_message(_("Calculating class covariance matrix ..."));
> }else{
>   G_message(_("Calculating class covariance matrices ..."));
> }

Ack!  Thanks for catching this.  I must have been in "drone mode".  I'll
commit fixes today.


-- 
Brad Douglas <rez at touchofmadness.com>




More information about the grass-dev mailing list