kasparsklavins/bigint

operator* and operator*=

Opened this issue · 2 comments

huyuguang commented

the current code is:

    //Multiplication
    Bigint operator*(Bigint const &);
    Bigint &operator*=(Bigint const &);
    Bigint operator*(long long const &);
    Bigint &operator*=(int const &);

why not be "Bigint &operator*=(long long const &);" ?

kasparsklavins commented

Should be safe to change to long long.

g19fanatic commented

@kasparsklavins do you want pull requests with these types of changes or will you eventually add them yourself?