nix::Value: Use more descriptive names

This commit is contained in:
José Luis Lafuente 2024-04-20 14:15:05 +02:00
parent 5cc4af5231
commit 9d7dee4a8f
No known key found for this signature in database
GPG Key ID: 8A3455EBE455489A
2 changed files with 12 additions and 7 deletions

View File

@ -23,7 +23,7 @@ class BindingsBuilder;
typedef enum {
tUnset,
tUninitialized = 0,
tInt = 1,
tBool,
tString,
@ -167,7 +167,7 @@ public:
struct Value
{
private:
InternalType internalType = tUnset;
InternalType internalType = tUninitialized;
friend std::string showType(const Value & v);
@ -271,7 +271,7 @@ public:
inline ValueType type(bool invalidIsThunk = false) const
{
switch (internalType) {
case tUnset: break;
case tUninitialized: break;
case tInt: return nInt;
case tBool: return nBool;
case tString: return nString;
@ -296,9 +296,14 @@ public:
internalType = newType;
}
inline bool isInitialized()
/**
* A value becomes valid when it is initialized. We don't use this
* in the evaluator; only in the bindings, where the slight extra
* cost is warranted because of inexperienced callers.
*/
inline bool isValid() const
{
return internalType != tUnset;
return internalType != tUninitialized;
}
inline void mkInt(NixInt n)

View File

@ -10,7 +10,7 @@ class ValueTest : public LibStoreTest
TEST_F(ValueTest, unsetValue)
{
Value unsetValue;
ASSERT_EQ(false, unsetValue.isInitialized());
ASSERT_EQ(false, unsetValue.isValid());
ASSERT_EQ(nThunk, unsetValue.type(true));
ASSERT_DEATH(unsetValue.type(), "");
}
@ -19,7 +19,7 @@ TEST_F(ValueTest, vInt)
{
Value vInt;
vInt.mkInt(42);
ASSERT_EQ(true, vInt.isInitialized());
ASSERT_EQ(true, vInt.isValid());
}
} // namespace nix