search

kasparsklavins / bigint

A lightweight big integer library for c++
MIT License
220 stars 55 forks source link

operator* and operator*= #23

Open huyuguang opened 6 years ago

huyuguang commented 6 years ago

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 6 years ago

Should be safe to change to long long.

g19fanatic commented 4 years ago

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