BulletSoftBody\btSoftBody.cpp \line: 527: Problem with eAeroModel enumerator.

[AxMedia]

The line 527 in file btSoftBody.cpp looks kindof suspicious to me:
if (btSoftBody::eAeroModel::V_TwoSided)
This produces compiler warning C4127 as it is a constant conditional expression.
Usally I'd assume this is so that the following line could be disabled at compile time, but that is usually done using a #define, right?
The definition of btSoftBody::eAeroModel then lets me continue doubting that this is actually correct in the context.

struct eAeroModel {
    enum _ {
	V_Point,			///Vertex normals are oriented toward velocity
	V_TwoSided,			///Vertex normals are flipped to match velocity	
	V_TwoSidedLiftDrag,             ///Vertex normals are flipped to match velocity and lift and drag forces are applied
	V_OneSided,			///Vertex normals are taken as it is	
	F_TwoSided,			///Face normals are flipped to match velocity
	F_TwoSidedLiftDrag,	        ///Face normals are flipped to match velocity and lift and drag forces are applied 
	F_OneSided,			///Face normals are taken as it is		
	END
    };
};

To disable line 528 you would have to switch V_Point and V_TwoSided, right?

I have changed it in my copy to this:

else if (m_cfg.aeromodel == btSoftBody::eAeroModel::V_Point || m_cfg.aeromodel == 
        btSoftBody::eAeroModel::V_OneSided || m_cfg.aeromodel == btSoftBody::eAeroModel::V_TwoSided)
{
    if (m_cfg.aeromodel == btSoftBody::eAeroModel::V_TwoSided)
        nrm *= (btScalar)( (btDot(nrm,rel_v) < 0) ? -1 : +1);

This makes a lot more sense to me.
If any issues arrise from this change I will for sure comment again.

EDIT: Same thing in line 622