search

rougier / mu4e-dashboard

A dashboard for mu4e (mu for emacs)
GNU General Public License v3.0
467 stars 43 forks source link

convert limit to number when reading arguments. fixes issue #8 #9

Closed Sk4zz closed 3 years ago

Sk4zz commented 3 years ago

Converting the limit argument into a numerical variable count earlier fixes issue #8.

Sk4zz commented 3 years ago

The fix broke something else. Now I replaced (> count 0) by (> (lenght count) 0) and that fixes the problem.

rougier commented 3 years ago

Your fix makes sense but now I wonder why I never had the same problem. count is still a string in the test and I should not be able to compare it to a number. Can you confirm that any link with a limit leads to the same error ?

Sk4zz commented 3 years ago

At least all the three links (100-500-9999) for the one entry led to the same error. I could try more, but looking at the code, I too wonder how it ever worked :)