Ru-MaNGOS - return-by-reference for Object field access

Ru-MaNGOS (http://mangos.ytdb.ru/index.php)

- Development (http://mangos.ytdb.ru/forumdisplay.php?f=64)

- - return-by-reference for Object field access (http://mangos.ytdb.ru/showthread.php?t=6030)

return-by-reference for Object field access

Can somebody explain why we use return-by-reference to access object-fields?

Код:

        const int32& GetInt32Value(uint16 index) const

        {

            MANGOS_ASSERT(index < m_valuesCount || PrintIndexError(index , false));

            return m_int32Values[ index ];

        }



        const uint32& GetUInt32Value(uint16 index) const

        {

            MANGOS_ASSERT(index < m_valuesCount || PrintIndexError(index , false));

            return m_uint32Values[ index ];

        }



        const uint64& GetUInt64Value(uint16 index) const

        {

            MANGOS_ASSERT(index + 1 < m_valuesCount || PrintIndexError(index , false));

            return *((uint64*) & (m_uint32Values[ index ]));

        }



        const float& GetFloatValue(uint16 index) const

        {

            MANGOS_ASSERT(index < m_valuesCount || PrintIndexError(index , false));

            return m_floatValues[ index ];

        }



        uint8 GetByteValue(uint16 index, uint8 offset) const

        {

            MANGOS_ASSERT(index < m_valuesCount || PrintIndexError(index , false));

            MANGOS_ASSERT(offset < 4);

            return *(((uint8*)&m_uint32Values[ index ]) + offset);

        }



        uint16 GetUInt16Value(uint16 index, uint8 offset) const

        {

            MANGOS_ASSERT(index < m_valuesCount || PrintIndexError(index , false));

            MANGOS_ASSERT(offset < 2);

            return *(((uint16*)&m_uint32Values[ index ]) + offset);

        }



        ObjectGuid const& GetGuidValue(uint16 index) const { return *reinterpret_cast<ObjectGuid const*>(&GetUInt64Value(index)); }

instead of normal return by value?

(Historical note: we do this since long forgotten times)

A discussion of a different yet related matter:
https://github.com/mangos/server/com...omment-1673031

Code inlined and in result this is array access. For small size types i think no difference, it's just consistent with way how this used for large structures access for avoid copy constructor call.

I always pass POD types by value if they fit into 64-bit register.
As some people here said in theory compiler can optimize out access by reference but no one proved that average compiler can into such stuff.
Same with 'const'. I heard it can lead to better optimization by compiler but no one have real examples.

Maybe you right, i not have personal strong points against use return values instead refs for types fit to 64-bits.