Message ID | 4D7BEF04.4080607@netcologne.de |
---|---|
State | New |
Headers | show |
On 03/12/2011 02:09 PM, Thomas Koenig wrote: > Hello world, > > the attached patch fixes PR 48066, a regression introduced way back when. It is > pretty self-explanatory. Regression-tested. > > OK for trunk? OK and thanks for patch! Jerry
Hi Jerry, > On 03/12/2011 02:09 PM, Thomas Koenig wrote: >> Hello world, >> >> the attached patch fixes PR 48066, a regression introduced way back >> when. It is >> pretty self-explanatory. Regression-tested. >> >> OK for trunk? > > OK and thanks for patch! Übertrage Daten ...................................................................................................................... Revision 170908 übertragen. Thanks for the ultra-quick review! Thomas
On Saturday 12 March 2011 23:40:32 Thomas Koenig wrote: > Hi Jerry, > > > On 03/12/2011 02:09 PM, Thomas Koenig wrote: > >> Hello world, > >> > >> the attached patch fixes PR 48066, a regression introduced way back > >> when. It is > >> pretty self-explanatory. Regression-tested. > >> > >> OK for trunk? > > > > OK and thanks for patch! > > Übertrage Daten > ........................................................................... > ........................................... Revision 170908 übertragen. > > Thanks for the ultra-quick review! > > Thomas There was a typo in the PR number (both in the testcases and in the changelogs).
Am 13.03.2011 00:04, schrieb Mikael Morin: > On Saturday 12 March 2011 23:40:32 Thomas Koenig wrote: > > There was a typo in the PR number (both in the testcases and in the > changelogs) Corrected. Thanks! Thomas
On 03/12/2011 03:04 PM, Mikael Morin wrote: > On Saturday 12 March 2011 23:40:32 Thomas Koenig wrote: >> Hi Jerry, >> >>> On 03/12/2011 02:09 PM, Thomas Koenig wrote: >>>> Hello world, >>>> >>>> the attached patch fixes PR 48066, a regression introduced way back >>>> when. It is >>>> pretty self-explanatory. Regression-tested. >>>> >>>> OK for trunk? >>> >>> OK and thanks for patch! >> >> Übertrage Daten >> ........................................................................... >> ........................................... Revision 170908 übertragen. >> >> Thanks for the ultra-quick review! >> >> Thomas > There was a typo in the PR number (both in the testcases and in the > changelogs). > > Sometimes our brains "see" what they want to see! ;) Jerry
Index: m4/ifunction_logical.m4 =================================================================== --- m4/ifunction_logical.m4 (Revision 170897) +++ m4/ifunction_logical.m4 (Arbeitskopie) @@ -130,7 +130,7 @@ count[n] = 0; dstride[n] = GFC_DESCRIPTOR_STRIDE(retarray,n); if (extent[n] <= 0) - len = 0; + return; } base = array->data; Index: m4/ifunction.m4 =================================================================== --- m4/ifunction.m4 (Revision 170897) +++ m4/ifunction.m4 (Arbeitskopie) @@ -116,7 +116,7 @@ count[n] = 0; dstride[n] = GFC_DESCRIPTOR_STRIDE(retarray,n); if (extent[n] <= 0) - len = 0; + return; } base = array->data;