|
From: | Kevin Atkinson |
Subject: | Re: [aspell-devel] Problems building 0.60.2 on HP-UX 11.x |
Date: | Sat, 1 Jan 2005 21:22:04 -0500 (EST) |
On Wed, 22 Dec 2004, Gary Setter wrote:
<snip>const char empty_str[1]; ^^^^^^^^^<snip> My cheap fix was to change the definition to of empty_str to const char *empty_str; and initialize it to empty string in the constructors. StringMap() : empty_str("") {} StringMap(const StringMap & other) : empty_str("") {copy(other);} But the real question is, why do we need this?
It was a memory optimization that most likely isn't worth it. The attached patch removes empty_str and replaces it with "".
cvs.diff
Description: Text document
[Prev in Thread] | Current Thread | [Next in Thread] |