Angus Leeming <[EMAIL PROTECTED]> writes:

| On Thursday 28 November 2002 12:38 pm, Angus Leeming wrote:
| > Should these functions not be re-written as
| > string const scex(string const & str)
| > {
| >         return split(str, '|');
| > }
| >
| > string const idex(string const & str)
| > {
| >         string identifier;
| >         split(str, identifier, '|');
| >         return identifier;
| > }
| 
| Here's the patch. It works fine.
| 
| I still can't believe that this was really a memory leak (simply because I'm 
| amazed it hasn't been picked up before), so would like confirmation that I've 
| done the right thing.
| 
| Ok to apply or just plain unnecessary?

apply it.

It seems somewhat cleaner.

-- 
        Lgb

Reply via email to