Remove unused functions with a bug

The `std::remove` only reorders the range, the lines are never erased.

Author: kontura

libdnf/utils/smartcols/Table.cpp

@@ -99,24 +99,12 @@ void Table::addLine(const std::shared_ptr<Line> &line)
     lines.push_back(line);
 }
 
-void Table::removeColumn(const std::shared_ptr<Column> &column)
-{
-    std::remove(std::begin(columns), std::end(columns), column);
-    scols_table_remove_column(table, column->column);
-}
-
 void Table::removeColumns()
 {
     columns.clear();
     scols_table_remove_columns(table);
 }
 
-void Table::removeLine(const std::shared_ptr<Line> &line)
-{
-    std::remove(std::begin(lines), std::end(lines), line);
-    scols_table_remove_line(table, line->line);
-}
-
 void Table::removeLines()
 {
     lines.clear();

libdnf/utils/smartcols/Table.hpp

@@ -92,14 +92,12 @@ class Table
     void enableNolinesep(bool enable) { scols_table_enable_nolinesep(table, enable); }
 
     void addColumn(const std::shared_ptr<Column> &column);
-    void removeColumn(const std::shared_ptr<Column> &column);
     void removeColumns();
     void moveColumn(const std::shared_ptr<Column> &before, const std::shared_ptr<Column> &toMove);
     std::shared_ptr<Column> newColumn(const std::string &name, double widthHint = 0, int flags = 0);
     std::shared_ptr<Column> nextColumn(std::vector<std::shared_ptr<Column>>::iterator &iterator) { return *(iterator++); }
 
     void addLine(const std::shared_ptr<Line> &line);
-    void removeLine(const std::shared_ptr<Line> &line);
     void removeLines();
     std::shared_ptr<Line> newLine();
     std::shared_ptr<Line> newLine(const std::shared_ptr<Line> &parent);