minor inconsequential stuff.

SPID/include/OutfitManager.h

@@ -9,13 +9,26 @@ namespace Outfits
 	public:
 		static void Register();
 
+		/// <summary>
+		/// Checks whether the actor can technically wear a given outfit.
+		/// Actor can wear an outfit when all of its components are compatible with actor's race.
+		/// 
+		/// This method doesn't validate any other logic.
+		/// </summary>
+		/// <param name="Actor">Target Actor to be tested</param>
+		/// <param name="Outfit">An outfit that needs to be equipped</param>
+		/// <returns>True if the actor can wear the outfit, false otherwise</returns>
+		bool CanEquipOutfit(const RE::Actor*, RE::BGSOutfit*);
+
 		/// <summary>
 		/// Sets given outfit as default outfit for the actor.
 		///
 		/// This method also makes sure to properly remove previously distributed outfit.
 		/// </summary>
 		/// <param name="Actor">Target Actor for whom the outfit will be set.</param>
 		/// <param name="Outfit">A new outfit to set as the default.</param>
+		/// <param name="allowOverwrites">If true, the outfit will be set even if the actor already has a distributed outfit.</param>
+		/// <returns>True if the outfit was successfully set, false otherwise.</returns>
 		bool SetDefaultOutfit(RE::Actor*, RE::BGSOutfit*, bool allowOverwrites);
 
 		/// <summary>

SPID/src/OutfitManager.cpp

@@ -129,7 +129,7 @@ namespace Outfits
 		return RE::BSEventNotifyControl::kContinue;
 	}
 
-	bool CanEquipOutfit(const RE::Actor* actor, RE::BGSOutfit* outfit)
+	bool Manager::CanEquipOutfit(const RE::Actor* actor, RE::BGSOutfit* outfit)
 	{
 		const auto race = actor->GetRace();
 		for (const auto& item : outfit->outfitItems) {
@@ -155,7 +155,7 @@ namespace Outfits
 		auto  defaultOutfit = npc->defaultOutfit;
 
 		if (!allowOverwrites && replacements.find(actor->formID) != replacements.end()) {
-			return true;
+			return true; // return true to indicate that some outfit was already set for this actor, and with overwrite disabled we won't be able to set any outfit.
 		}
 
 		if (!CanEquipOutfit(actor, outfit)) {
@@ -165,7 +165,7 @@ namespace Outfits
 			return false;
 		}
 
-		actor->SetDefaultOutfit(outfit, false);  // Having true here causes infinite loading. It seems that it works either way.
+		actor->SetDefaultOutfit(outfit, false);  // Having true here causes infinite loading. It seems that equipping works either way, so we are good :)
 
 		if (auto previous = replacements.find(actor->formID); previous != replacements.end()) {
 			previous->second.distributed = outfit;