New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Moar bug flattening Arg in Baggy with slurpy_positional #3528
Comments
From grtodd@gmail.comUsing masak's "p6-golfed" word counter: perl6 -e `.say for (bag slurp.words).pairs.sort(*.value).reverse[^10]' on this file http://www.gutenberg.org/files/24007/24007-8.txt which I have saved as UTF-8 text in La_psychologie_des_foules.txt iconv -f ISO-8859-1 -t UTF-8 24007-8.txt > La_psychologie_des_foules.txt I get this error: Arg has not been flattened in slurpy_positional src/gen/m-CORE.setting:17249 comes from "slurpy_positional" shows up under nqp/MoarVM/: if (arg_info.flags & MVM_CALLSITE_ARG_FLAT) {, so my guess is that this is a Baggy/Arg <--> moar bug or a not yet Neither perl6-j and perl6-p have this error. Other data points: The file is 6294 lines long and has many words to count: If I reduce the size of the file this the code works without the error head -n 5655 > short_La_psychologie_des_foules.txt This seems to predictably occur when the file is > 5654 lines long. for i in *txt; do echo ""; wc $i ; p6wordcount $i; done gives only faint glimmers of light ... it seems where files have Hope that helps someone track this down. |
From grtodd@gmail.comThis commit appears to fix the issue: A test is needed to confirm |
grtodd@gmail.com - Status changed from 'new' to 'open' |
From grtodd@gmail.comOn Sun Sep 21 07:28:51 2014, grtodd@gmail.com wrote:
Bug in bug report! Wrong commit mentioned!! This is the commit that has the fix: |
From grtodd@gmail.comWith some help from #perl6 I seem to have found a reliable way to trip this bug that could comprise a test. Adding this note here so I don't forget. Building @commit e630782 Passes: Fails:
Builds from after the fix committed by jnth (noted above) pass with very large arrays (e.g. my @arr = ('a'...*)[^2000000] ; ) so a test could possibly be written for this bug using some appropriate value. |
From grtodd@gmail.comw/ a test: this fails on older build and passes with the fix noted above: ok bag(my @big_arr = ("a"...*)[^50000]), "big array goes into a bag"; pushed as perl6/roast/commit/d720ab5 to S03-operators/bag.t |
grtodd@gmail.com - Status changed from 'open' to 'resolved' |
Migrated from rt.perl.org#122810 (status was 'resolved')
Searchable as RT122810$
The text was updated successfully, but these errors were encountered: