<HTML>
<style> BODY { font-family:Arial, Helvetica, sans-serif;font-size:12px; }</style>Something tells me that we should write our own patch.exe at some point to alleviate these shortcomings! Thanks for the patch again.<br>
<div><br>
</div><div>Any word on what I've submitted so far? I ask because I found some new peephole optimisations that can make some good speed and size savings, but one of them requires a new Pass 1 function and will either have to be merged into the binary search list, or the large case block, so I can't submit it yet until I know which way the source tree will go.</div><div><br>
</div><div>Gareth aka. Kit<br>
</div> <br>
<br>
<span style="font-weight: bold;">On Fri 15/06/18 20:03 , Florian Klämpfl florian@freepascal.org sent:<br>
</span><blockquote style="BORDER-LEFT: #F5F5F5 2px solid; MARGIN-LEFT: 5px; MARGIN-RIGHT:0px; PADDING-LEFT: 5px; PADDING-RIGHT: 0px">Am 15.06.2018 um 18:17 schrieb J. Gareth Moreton:
<br>
<span style="color: rgb(102, 102, 102);">> Not much luck for me - the file won't patch without options or modifications, and using -p 1 to remove the "a/" and "b/"
</span><br>
<span style="color: rgb(102, 102, 102);">> from the starts of the files causes an assertion in patch.exe.
</span><br>
<br>
Sorry, my bad. The patch has unix line feeds, this crashes patch.exe for windows. Try again with the attached one or
<br>
convert the line endings of the first one to window ones.
<br>
<br>
<br>
_______________________________________________<br>
fpc-devel maillist - <a href="mailto:fpc-devel@lists.freepascal.org">fpc-devel@lists.freepascal.org</a><br>
<a target="_blank" href="<a href="http://lists.freepascal.org/cgi-bin/mailman/listinfo/fpc-devel">http://lists.freepascal.org/cgi-bin/mailman/listinfo/fpc-devel</a>"><span style="color: red;">http://lists.freepascal.org/cgi-bin/mailman/listinfo/fpc-devel</span></a><br>
<br>
</blockquote></HTML>