TODOs -> references to #168

src/fmt_md.rs

@@ -156,7 +156,11 @@ impl<'s, 'md> MdWriterState<'s, 'md> {
             MdElemRef::Paragraph(para) => self.write_paragraph(out, para),
             MdElemRef::BlockQuote(block) => self.write_block_quote(out, block),
             MdElemRef::List(list) => self.write_list(out, list),
-            MdElemRef::Table(table) => self.write_table(out, table.into()), // TODO maybe have a generic table trait, so I don't need to do the copying?
+            MdElemRef::Table(table) => {
+                // GH #168 maybe have a trait for tables, so we can parameterize write_table instead of calling into()?
+                // That would let us avoid copying various vecs.
+                self.write_table(out, table.into())
+            }
             MdElemRef::TableSlice(table) => self.write_table(out, table),
             MdElemRef::Inline(inline) => {
                 self.inlines_writer.write_inline_element(out, inline);

src/select/api.rs

@@ -199,8 +199,8 @@ impl MdqRefSelector {
 
     fn build_output<'md>(&self, out: &mut Vec<MdElemRef<'md>>, node: MdElemRef<'md>) {
         // try_select_node is defined in macro_helpers::selectors!
+        // GH #168 can we remove the clone()? Maybe by having try_select_node take a reference.
         match self.try_select_node(node.clone()) {
-            // TODO can we remove this? I don't think so, but let's follow up
             Some(found) => out.push(found),
             None => {
                 for child in Self::find_children(node) {

src/select/sel_table.rs

@@ -35,7 +35,7 @@ impl TableSliceSelector {
 
 impl<'md> Selector<'md, TableSlice<'md>> for TableSliceSelector {
     fn try_select(&self, slice: TableSlice<'md>) -> Option<MdElemRef<'md>> {
-        let mut slice = slice.clone(); // TODO is there any way to avoid this? There may not be.
+        let mut slice = slice.clone(); // GH #168 is there any way to avoid this? There may not be.
         slice.normalize();
 
         slice.retain_columns_by_header(|line| self.headers_matcher.matches_inlines(line));