Add explaination as to why we do clearing inside effect set* functions

Author: FelixWolf

indra/newview/llhudeffectlookat.cpp

@@ -401,6 +401,7 @@ bool LLHUDEffectLookAt::setLookAt(ELookAtType target_type, LLViewerObject *objec
     static LLCachedControl<bool> enable_lookat_hints(gSavedSettings, "EnableLookAtTarget", true);
     if (!enable_lookat_hints)
     {
+        // Clear the effect so it doesn't linger around if it gets disabled
         if (mTargetType != LOOKAT_TARGET_IDLE)
         {
             mTargetObject = gAgentAvatarp;

indra/newview/llhudeffectpointat.cpp

@@ -230,6 +230,7 @@ bool LLHUDEffectPointAt::setPointAt(EPointAtType target_type, LLViewerObject *ob
     static LLCachedControl<bool> enable_selection_hints(gSavedSettings, "EnableSelectionHints", true);
     if (!enable_selection_hints)
     {
+        // Clear the effect so it doesn't linger around if it gets disabled
         if (mTargetType != POINTAT_TARGET_NONE)
         {
             clearPointAtTarget();