Educational-MySQL / UDFs-written-in-Python

MySQL UDFs written in Python
GNU General Public License v3.0
2 stars 5 forks source link

Fixed #8 added check for big numbers #10

Closed Komport closed 7 years ago

Komport commented 7 years ago

if (((longlong)args->args[0]) > 20) { my_stpcpy(message,"Factorial function can't calculate numbers more than 20"); return 1; }

ShahriyarR commented 7 years ago

Well, we need check for this not in udf initialization function but inside implementation function. Exactly saying where we convert string to longlong using std::stol() if this string can not be converted, there should be some message for this. Could you try to implement this?