On Tue, Dec 16, 2014 at 7:25 AM, David Crayford <[email protected]> wrote:
> Sorry to be critical but you really do need to check the return value from > malloc() and realloc(). I would also replace the ascending goto with a > while loop. Other than that it looks good. > > Not critical, helpful! I put in a check on the malloc(). It never occurred to me that the realloc() could fail because it is only used to shrink the existing allocation. I guess that I thought that shrinking was always an "in place" operation. I'll fix that too. -- While a transcendent vocabulary is laudable, one must be eternally careful so that the calculated objective of communication does not become ensconced in obscurity. In other words, eschew obfuscation. Maranatha! <>< John McKown ---------------------------------------------------------------------- For IBM-MAIN subscribe / signoff / archive access instructions, send email to [email protected] with the message: INFO IBM-MAIN
