lowered logging level for swallowed exception

Author: retailcoder

Rubberduck.VBEditor.VB6/SafeComWrappers/VB/CodeModule.cs

@@ -128,7 +128,10 @@ public void InsertLines(int line, string content)
             }
             catch (Exception e)
             {
-                _logger.Error(e);
+                // "too many line continuations" is one possible cause for a COMException here.
+                // deleting the only line in a module is another.
+                // we can log the exception, but really we're just intentionally swallowing it.
+                _logger.Warn(e, $"{nameof(InsertLines)} failed.");
             }
         }
 
@@ -142,7 +145,10 @@ public void DeleteLines(int startLine, int count = 1)
             }
             catch (Exception e)
             {
-                _logger.Error(e);
+                // "too many line continuations" is one possible cause for a COMException here.
+                // deleting the only line in a module is another.
+                // we can log the exception, but really we're just intentionally swallowing it.
+                _logger.Warn(e, $"{nameof(DeleteLines)} failed.");
             }
         }
 
@@ -160,7 +166,10 @@ public void ReplaceLine(int line, string content)
             }
             catch (Exception e)
             {
-                _logger.Error(e);
+                // "too many line continuations" is one possible cause for a COMException here.
+                // deleting the only line in a module is another.
+                // we can log the exception, but really we're just intentionally swallowing it.
+                _logger.Warn(e, $"{nameof(ReplaceLine)} failed.");
             }
         }
 

Rubberduck.VBEditor.VBA/SafeComWrappers/VB/CodeModule.cs

@@ -129,8 +129,9 @@ public void InsertLines(int line, string content)
             catch (Exception e)
             {
                 // "too many line continuations" is one possible cause for a COMException here.
-                _logger.Error(e);
-                throw;
+                // deleting the only line in a module is another.
+                // we can log the exception, but really we're just intentionally swallowing it.
+                _logger.Warn(e, $"{nameof(InsertLines)} failed.");
             }
         }
 
@@ -148,8 +149,9 @@ public void DeleteLines(int startLine, int count = 1)
                 catch (Exception e)
                 {
                     // "too many line continuations" is one possible cause for a COMException here.
-                    _logger.Error(e);
-                    throw;
+                    // deleting the only line in a module is another.
+                    // we can log the exception, but really we're just intentionally swallowing it.
+                    _logger.Warn(e, $"{nameof(DeleteLines)} failed.");
                 }
             }
         }
@@ -176,8 +178,9 @@ public void ReplaceLine(int line, string content)
             catch (Exception e)
             {
                 // "too many line continuations" is one possible cause for a COMException here.
-                _logger.Error(e);
-                throw;
+                // deleting the only line in a module is another.
+                // we can log the exception, but really we're just intentionally swallowing it.
+                _logger.Warn(e, $"{nameof(ReplaceLine)} failed.");
             }
         }