[M3commit] CVS Update: cm3

Jay K jay.krell at cornell.edu
Mon Jul 5 22:45:57 CEST 2010


I will maybe see about doing better here.
Initializing locals doesn't seem like such a bad change though.
I like the code to be "obviously correct" by a quick read by a human.

 - Jay

----------------------------------------
> From: hosking at cs.purdue.edu
> Date: Mon, 5 Jul 2010 14:31:01 -0400
> To: jkrell at elego.de
> CC: m3commit at elegosoft.com
> Subject: Re: [M3commit] CVS Update: cm3
>
> Jay, I am perplexed that we are adding things to Modula-3 source because of compiler backend brokenness just to avoid back-end warnings. Better to fix the backend so that RAISE is noreturn. This should not be difficult to do.
>
> On 5 Jul 2010, at 15:22, Jay Krell wrote:
>
> > CVSROOT: /usr/cvs
> > Changes by: jkrell at birch. 10/07/05 15:22:28
> >
> > Modified files:
> > cm3/m3-tools/cvsup/suplib/src/text_cm3/: SupMiscText.m3
> >
> > Log message:
> > ../src/text_cm3/SupMiscText.m3: In function 'SupMisc__DecodeWS':
> > ../src/text_cm3/SupMiscText.m3:211: warning: 'M3_Bkn9rd_ch' may be used uninitialized in this function
> > ../src/text_cm3/SupMiscText.m3:211: note: 'M3_Bkn9rd_ch' was declared here
> >
> > because raising an exception isn't currently "noreturn"
> > so initialize it to 0
>
 		 	   		  


More information about the M3commit mailing list