Don't wrap comments that are part of a table (#4214)

* Don't wrap comments that are part of a table

Closes #4210

* fixup! Don't wrap comments that are part of a table

Author: topecongiro

Diff for: rustfmt-core/rustfmt-lib/src/formatting/comment.rs

@@ -738,6 +738,7 @@ impl<'a> CommentRewrite<'a> {
         if self.fmt.config.wrap_comments()
             && unicode_str_width(line) > self.fmt.shape.width
             && !has_url(line)
+            && !is_table_item(line)
         {
             match rewrite_string(line, &self.fmt, self.max_width) {
                 Some(ref s) => {
@@ -867,6 +868,18 @@ fn has_url(s: &str) -> bool {
     s.contains("https://") || s.contains("http://") || s.contains("ftp://") || s.contains("file://")
 }
 
+/// Returns true if the given string may be part of a Markdown talble.
+fn is_table_item(mut s: &str) -> bool {
+    // This function may return false positive, but should get its job done in most cases (i.e.
+    // markdown tables with two column delimiters).
+    s = s.trim_start();
+    return s.starts_with('|')
+        && match s.rfind('|') {
+            Some(0) | None => false,
+            _ => true,
+        };
+}
+
 /// Given the span, rewrite the missing comment inside it if available.
 /// Note that the given span must only include comments (or leading/trailing whitespaces).
 pub(crate) fn rewrite_missing_comment(

Diff for: rustfmt-core/rustfmt-lib/tests/target/issue-4210.rs

@@ -0,0 +1,15 @@
+// rustfmt-wrap_comments: true
+
+/// Table that is > 80 symbols:
+///
+/// | table | xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx |
+/// |-------|-----------------------------------------------------------------------------|
+/// | val   | x                                                                           |
+pub struct Item;
+
+/// Table that is > 80 symbols:
+///
+/// | table | xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx
+/// |-------|-----------------------------------------------------------------------------
+/// | val   | x                                                                          
+pub struct Item;