From 8603576c5593ab900192d29bf136c615a0f962db Mon Sep 17 00:00:00 2001
From: Vladimir Krivosheev <vladimir.krivosheev@jetbrains.com>
Date: Wed, 18 Jan 2017 08:40:54 +0100
Subject: [PATCH] IDEA-CR-17324 improve comment

---
 .../openapi/roots/impl/JavaModuleExternalPathsImpl.java         | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/platform/projectModel-impl/src/com/intellij/openapi/roots/impl/JavaModuleExternalPathsImpl.java b/platform/projectModel-impl/src/com/intellij/openapi/roots/impl/JavaModuleExternalPathsImpl.java
index 7f5b62c7b6eb..dc01925c852a 100644
--- a/platform/projectModel-impl/src/com/intellij/openapi/roots/impl/JavaModuleExternalPathsImpl.java
+++ b/platform/projectModel-impl/src/com/intellij/openapi/roots/impl/JavaModuleExternalPathsImpl.java
@@ -93,6 +93,8 @@ public class JavaModuleExternalPathsImpl extends JavaModuleExternalPaths {
     if (container == null) {
       if (urls.length == 0) {
         // do not store if no container and nothing to store
+        // otherwise our module extension model will be changed and it can leads to unnecessary model commit
+        // (that in turn can mask issues like https://youtrack.jetbrains.com/issue/IDEA-166461)
         return;
       }