clean up

Author: samukweku

src/core/expr/fexpr.cc

@@ -171,13 +171,6 @@ oobj PyFExpr::m__getitem__(py::robj item) {
 
 
 //----- Basic arithmetics ------------------------------------------------------
-
-// static oobj make_unexpr(dt::expr::Op op, const PyObject* self) {
-//   return robj(Expr_Type).call({
-//                   oint(static_cast<int>(op)),
-//                   otuple{oobj(self)}});
-// }
-
 static oobj make_binexpr(dt::expr::Op op, robj lhs, robj rhs) {
   return robj(Expr_Type).call({
                   oint(static_cast<int>(op)),
@@ -215,21 +208,6 @@ bool PyFExpr::nb__bool__() {
       "        f.B != 0\n";
 }
 
-// oobj PyFExpr::nb__invert__() {
-//   return make_unexpr(dt::expr::Op::UINVERT, this);
-// }
-
-// oobj PyFExpr::nb__neg__() {
-//   return make_unexpr(dt::expr::Op::UMINUS, this);
-// }
-
-// oobj PyFExpr::nb__pos__() {
-//   return make_unexpr(dt::expr::Op::UPLUS, this);
-// }
-
-
-
-
 //----- Other methods ----------------------------------------------------------
 
 oobj PyFExpr::extend(const XArgs& args) {

src/core/expr/fexpr.h

@@ -175,9 +175,6 @@ class PyFExpr : public py::XObject<PyFExpr> {
     static py::oobj nb__lshift__  (py::robj, py::robj);
     static py::oobj nb__rshift__  (py::robj, py::robj);
     bool nb__bool__();
-    //py::oobj nb__invert__();
-    //py::oobj nb__neg__();
-    //py::oobj nb__pos__();
 
     py::oobj len();                        // [DEPRECATED]
     py::oobj re_match(const py::XArgs&);   // [DEPRECATED]

src/core/expr/funary/umaker.cc

@@ -60,11 +60,6 @@ static const umaker_ptr& get_umaker(Op opcode, SType stype) {
 umaker_ptr resolve_op(Op opcode, SType stype)
 {
   switch (opcode) {
-    // Basic
-    case Op::UPLUS:     return resolve_op_uplus(stype);
-    //case Op::UMINUS:    return resolve_op_uminus(stype);
-    //case Op::UINVERT:   return resolve_op_uinvert(stype);
-
     // Math: trigonometric
     case Op::SIN:       return resolve_op_sin(stype);
     case Op::COS:       return resolve_op_cos(stype);

src/core/expr/funary/umaker.h

@@ -81,11 +81,6 @@ using umaker_ptr = std::unique_ptr<umaker>;
 // Main resolver, calls individual-op resolvers below
 umaker_ptr resolve_op(Op, SType);
 
-// Basic
-umaker_ptr resolve_op_uplus(SType);
-//umaker_ptr resolve_op_uminus(SType);
-//umaker_ptr resolve_op_uinvert(SType);
-
 // Trigonometric
 umaker_ptr resolve_op_sin(SType);
 umaker_ptr resolve_op_cos(SType);