On Wed, Jan 01, 2014 at 07:49:16PM -0800, Mike Stump wrote: > On Nov 23, 2013, at 12:16 PM, Steve Kargl <s...@troutmask.apl.washington.edu> > wrote: >> On Sat, Nov 23, 2013 at 11:21:21AM -0800, Mike Stump wrote: >>> Richi has asked the we break the wide-int patch so that the >>> individual port and front end maintainers can review their >>> parts without have to go through the entire patch. This >>> patch covers the fortran front end. >>> >>> Ok? >>> >>> + *logical = wi::eq_p (t, 0) ? 0 : 1; >> >> I can't find the meaning of :: in n1256.pdf. What does this do? >> >> Also, given the complete lack of a description of what this >> patch does and no pointer to a discussion of what this >> patch does, and no description of its benefit to gfortran, >> I vote "no". > > I don't like the notion that one person says yes, and one says no, > and then we ignore the no, and use the yes to approve a patch. Can > the fortran come up with a final unified answer? Thanks.
My original comment had nothing to do with the technical merit of the patch. My comment, as shown above, objected to dropping a patch on (gfortran) developers with *no* explanation of the patch and *no* pointer to where a discussion may have occurred. -- Steve