Description

youknowone

opened on Dec 28, 2023

Issue body actions

Summary

sys.flags.int_max_str_digits now returns sys.get_int_max_str_digits().

Introduced from https://github.com/RustPython/RustPython/pull/5127/files#diff-128fff16ea63b541718887f5c714e0c1a357414fe1146b4893ef699e8853e3b0R953

Related RustPython code is vm/src/stdlib.sys.rs Flags::int_max_str_digits.

Not sure yet if we need to change the value of Settings or sys::Flags.

Activity

1. youknowone

   added

   C-bugSomething isn't workingSomething isn't working

   on Dec 28, 2023
2. itsankitkp commented on Jan 1, 2024

   

   itsankitkp

   on Jan 1, 2024

   Contributor

   More actions

   Is this fixed now?
   in rustpython (317f449)
   sys.flags.int_max_str_digits returns -1
   sys.get_int_max_str_digits() returns 4300
   in cpython 3.11.6
   same results as above.

   Also, worked fine with -X int_max_str_digits flag
3. fanninpm commented on Jan 3, 2024

   

   fanninpm

   on Jan 3, 2024

   Contributor

   More actions

   in cpython 3.11.6

   Have you tried CPython 3.12?
4. itsankitkp commented on Jan 5, 2024

   

   itsankitkp

   on Jan 5, 2024

   Contributor

   More actions

   in cpython 3.11.6

   Have you tried CPython 3.12?

   Tried latest cpython (3.13 dev version)
   both sys.flags.int_max_str_digits and sys.get_int_max_str_digits() returns 4300
   its coming from here: python/cpython#96499
5. youknowone

   added

   good first issueGood for newcomersGood for newcomers

   on May 20, 2024
6. JamesClarke7283

   added a commit that references this issue on Jun 24, 2024

   changed int_max_str_digits from -1 to 4300 to be more cpython complie…

   This issue will close once commit d3db469 is merged into the 'main' branch.

   d3db469
7. youknowone

   closed this as completedin #5343on Jun 27, 2024
8. youknowone

   closed this as completedin e142d65on Jun 27, 2024
9. pull

   added a commit that references this issue on Jun 27, 2024

   changed int_max_str_digits from -1 to 4300 to be more cpython complie…

   This issue will close once commit e142d65 is merged into the 'main' branch.

   e142d65

Issue actions