Making count return a correct long or BigInt value in such cases would be a
significant change, given that Java's int is used for the return value for
many count() and countFrom() methods sprinkled around Clojure's source code.

I've created a ticket CLJ-1229 and attached a patch that causes count to
throw an ArithmeticException if the sequence contains more than
Integer/MAX_VALUE elements.  I don't know if such a patch is acceptable to
the Clojure/core team or not, but we may find out.

    http://dev.clojure.org/jira/browse/CLJ-1229

Andy


On Sun, Jul 7, 2013 at 5:13 PM, John Jacobsen <eigenhom...@gmail.com> wrote:

> Was unsure whether to post to clojure or clojure-dev, but here goes.
>
> I was surprised to learn today that count silently overflows to negative
> numbers:
>
> jenome.core> (time (count (range (*' 1000 1000 1000 3))))
> "Elapsed time: 375225.663 msecs"
> -1294967296
> jenome.core>
>
> I can easily get around this with my own (non-overflowing) count function,
> but would it make sense for count throw an ArithmeticException on overflow,
> with *unchecked-math* set to false (as it is in my case)?
>
> Curiously,
> John
>
> --
> --
> You received this message because you are subscribed to the Google
> Groups "Clojure" group.
> To post to this group, send email to clojure@googlegroups.com
> Note that posts from new members are moderated - please be patient with
> your first post.
> To unsubscribe from this group, send email to
> clojure+unsubscr...@googlegroups.com
> For more options, visit this group at
> http://groups.google.com/group/clojure?hl=en
> ---
> You received this message because you are subscribed to the Google Groups
> "Clojure" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to clojure+unsubscr...@googlegroups.com.
> For more options, visit https://groups.google.com/groups/opt_out.
>
>
>

-- 
-- 
You received this message because you are subscribed to the Google
Groups "Clojure" group.
To post to this group, send email to clojure@googlegroups.com
Note that posts from new members are moderated - please be patient with your 
first post.
To unsubscribe from this group, send email to
clojure+unsubscr...@googlegroups.com
For more options, visit this group at
http://groups.google.com/group/clojure?hl=en
--- 
You received this message because you are subscribed to the Google Groups 
"Clojure" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to clojure+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.


Reply via email to