<html><head><meta http-equiv="Content-Type" content="text/html charset=windows-1252"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;">Yes, that is a bug, and your patch is all that is needed to fix it. Thank you very much for pointing this out!&nbsp;<div>—Jennifer</div><div><br><div><div>On Mar 30, 2015, at 5:06 AM, KODAMA Chihiro &lt;<a href="mailto:kodamac@jamstec.go.jp">kodamac@jamstec.go.jp</a>&gt; wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite">Dear GrADS developers,<br><br>I encountered an trouble of gsfpath() function, and may found a bug in<br>the (nearly) latest GrADS.<br><br>GrADS script which includes gsfpath(), say,<br> &nbsp;...<br> &nbsp;rc = gsfallow('on')<br> &nbsp;rc = gsfpath('dir1 dir2')<br> &nbsp;...<br>never stops (I found it with GrADS ver. 2.1.a2).<br><br>It seems that the following line around line 3065 in gscrpt.c,<br> &nbsp;while (j&lt;i) *(pcmn-&gt;ppath+j) = *(pvar-&gt;strng+j);<br>, should be replaced with<br> &nbsp;while (j&lt;i) { *(pcmn-&gt;ppath+j) = *(pvar-&gt;strng+j); j++; }<br>to avoid infinite loop. I did not carefully read the whole source code,<br>and my understanding may be wrong. (if so, please teach me!)<br><br>I hope it may help improve GrADS.<br><br>Sincerely,<br><br>Chihiro Kodama<br><br>-- <br>=====================================================<br>KODAMA Chihiro, Ph.D.<br>Dept. Seamless Environmental Prediction Research<br>Japan Agency for Marine-Earth Science and Technology<br>PHONE: +81-45-778-5623 (direct)<br>URL: &nbsp;&nbsp;<a href="http://kodama.fubuki.info/">http://kodama.fubuki.info/</a><br>=====================================================<br><br>_______________________________________________<br>gradsusr mailing list<br><a href="mailto:gradsusr@gradsusr.org">gradsusr@gradsusr.org</a><br>http://gradsusr.org/mailman/listinfo/gradsusr<br></blockquote></div><br><div apple-content-edited="true">
<span class="Apple-style-span" style="border-collapse: separate; color: rgb(0, 0, 0); font-family: Helvetica; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-align: -webkit-auto; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-border-horizontal-spacing: 0px; -webkit-border-vertical-spacing: 0px; -webkit-text-decorations-in-effect: none; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px;  "><span class="Apple-style-span" style="border-collapse: separate; -webkit-border-horizontal-spacing: 0px; -webkit-border-vertical-spacing: 0px; color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; -webkit-text-decorations-in-effect: none; text-indent: 0px; -webkit-text-size-adjust: auto; text-transform: none; orphans: 2; white-space: normal; widows: 2; word-spacing: 0px; "><div>--</div><div>Jennifer M. Adams<br>Center for Ocean-Land-Atmosphere Studies (COLA)<br>111 Research Hall, Mail Stop 2B3<br>George Mason University<br>4400 University Drive<br>Fairfax, VA 22030&nbsp;<br><br></div><div><br class="khtml-block-placeholder"></div><br class="Apple-interchange-newline"></span></span><br class="Apple-interchange-newline">
</div>
<br></div></body></html>