Clean up handling of empty argument description

Author: govert

Source/ExcelDna.IntelliSense/FormattedText.cs

@@ -18,24 +18,19 @@ public FormattedText()
             _lines = new List<TextLine>();
         }
 
-        public void Add(TextLine line) { _lines.Add(line); }
+        public void Add(TextLine line) => _lines.Add(line);
 
-        public void Add(IEnumerable<TextLine> lines) { if (lines != null) _lines.AddRange(lines); }
-
-        public IEnumerator<TextLine> GetEnumerator()
+        public void Add(IEnumerable<TextLine> lines)
         {
-            return _lines.GetEnumerator();
+            if (lines != null)
+                _lines.AddRange(lines);
         }
 
-        IEnumerator IEnumerable.GetEnumerator()
-        {
-            return GetEnumerator();
-        }
+        public IEnumerator<TextLine> GetEnumerator() => _lines.GetEnumerator();
 
-        public override string ToString()
-        {
-            return string.Join("\r\n", _lines.Select(l => l.ToString()));
-        }
+        IEnumerator IEnumerable.GetEnumerator() => GetEnumerator();
+
+        public override string ToString() => string.Join("\r\n", _lines.Select(l => l.ToString()));
     }
 
     class TextLine : IEnumerable<TextRun>
@@ -47,31 +42,21 @@ public TextLine()
             _runs = new List<TextRun>();
         }
 
-        public void Add(TextRun run) { _runs.Add(run); }
+        public void Add(TextRun run) => _runs.Add(run);
 
+        public IEnumerator<TextRun> GetEnumerator() => _runs.GetEnumerator();
 
-        public IEnumerator<TextRun> GetEnumerator()
-        {
-            return _runs.GetEnumerator();
-        }
-
-        IEnumerator IEnumerable.GetEnumerator()
-        {
-            return GetEnumerator();
-        }
+        IEnumerator IEnumerable.GetEnumerator() => GetEnumerator();
 
-        public override string ToString()
-        {
-            return string.Concat(_runs.Select(r => r.Text));
-        }
+        public override string ToString() => string.Concat(_runs.Select(r => r.Text));
     }
 
     class TextRun
     {
         public string Text { get; set; }
         public FontStyle Style { get; set; }    
         public string LinkAddress { get; set; }
-        public bool IsLink { get { return !string.IsNullOrEmpty(LinkAddress); } }
+        public bool IsLink => !string.IsNullOrEmpty(LinkAddress);
     }
 
 }

Source/ExcelDna.IntelliSense/IntelliSenseDisplay.cs

@@ -466,15 +466,19 @@ FormattedText GetFunctionIntelliSense(FunctionInfo functionInfo, int currentArgI
             var formattedText = new FormattedText { nameLine, descriptionLines };
             if (functionInfo.ArgumentList.Count > currentArgIndex)
             {
-                var description = GetArgumentDescription(functionInfo.ArgumentList[currentArgIndex]);
-                formattedText.Add(description);
+                var description = GetArgumentDescriptionOrNull(functionInfo.ArgumentList[currentArgIndex]);
+                if (description != null)
+                    formattedText.Add(description);
             }
 
             return formattedText;
         }
 
-        TextLine GetArgumentDescription(FunctionInfo.ArgumentInfo argumentInfo)
+        TextLine GetArgumentDescriptionOrNull(FunctionInfo.ArgumentInfo argumentInfo)
         {
+            if (string.IsNullOrEmpty(argumentInfo.Description))
+                return null;
+
             return new TextLine { 
                     new TextRun
                     {