A minor update: running flex on escan.l does return lex.yy.c which is compatible (essentially identical - except for improvements from being generated by flex 3.5 rather than flex 3.3) with escan.cpp, except for the win32 changes that Peter made.<div>
<br></div><div>It still has the memory leak.<br><br><div class="gmail_quote">On Sat, Jan 15, 2011 at 3:16 PM, Barth Netterfield <span dir="ltr"><<a href="mailto:netterfield@astro.utoronto.ca">netterfield@astro.utoronto.ca</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">Hi,<div><br></div><div>There is a memory leak in escan.cpp, and I'm looking for some advice:</div><div><br></div><div>
-It looks like escan.cpp was at some time generated from escan.l </div><div><div>
-It looks like eparse.cpp was at some time generated from eparse.y </div></div><div>-These cpp files were committed by fidler in commit 726751. </div><div>-changes to the .l and .y files were committed after this by fidler (731128), but it is possible the .cpp and .h files were generated from these. Or maybe not. Most of the changes are much older than this.</div>
<div>-I don't think that lex/yacc have been part of the build system for a very long time, if ever.</div><div>-kuemmel made some changes to the cpp files recently to allow it to build under windows.</div><div><br></div>
<div>We are now unable to re-generate the cpp files from the .l/.y files without hand editing the output.</div><div><br></div><div>1) Is this OK and so should we just continue to modify the cpp files for bug fixes as they come along? If not... ?</div>
<div><br></div><div>2) eparse.cpp and escan.cpp are is listed in the cmake files as kst_dont_merge. What does kst_dont_merge do/mean?</div><div><br></div><div>-- </div><div><span style="font-family:arial, sans-serif;font-size:13px;border-collapse:collapse;color:rgb(136, 136, 136)">C. Barth Netterfield<br>
University of Toronto<br>416-845-0946</span><div><span style="font-family:arial, sans-serif;font-size:13px;border-collapse:collapse;color:rgb(136, 136, 136)"><br>
</span></div><br>
</div>
</blockquote></div><br><br clear="all"><br>-- <br><span style="font-family:arial, sans-serif;font-size:13px;border-collapse:collapse;color:rgb(136, 136, 136)">C. Barth Netterfield<br>University of Toronto<br>416-845-0946</span><div>
<span style="font-family:arial, sans-serif;font-size:13px;border-collapse:collapse;color:rgb(136, 136, 136)"><br></span></div><br>
</div>