|
From: | John W. Eaton |
Subject: | Re: Rename all_strings() to str_vec_value() ? |
Date: | Thu, 24 Dec 2015 11:12:31 -0500 |
User-agent: | Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Icedove/31.7.0 |
On 12/24/2015 12:49 AM, Rik wrote:
I think it would be clearer to rename the function all_strings in ov.h to str_vec_value. This would follow the naming convention of the other value extractor routines in ov.h such as array_value() or string_value(). Any objections?
How about string_vector_value instead? That's consistent with the name of the class instead of str_vec. Otherwise, yes, I think this is a good change to make. Please preserve the old name for backward compatibility. Maybe deprecate it now as well?
jwe
[Prev in Thread] | Current Thread | [Next in Thread] |