Camelia, the Perl 6 bug

IRC log for #parrot, 2011-10-31

Parrot | source cross referenced

| Channels | #parrot index | Today | | Search | Google Search | Plain-Text | summary

All times shown according to UTC.

Time Nick Message
01:07 nbezzala joined #parrot
01:12 japhb joined #parrot
04:59 schmooster joined #parrot
05:11 jsut joined #parrot
05:30 nbezzala joined #parrot
05:38 moritz what about giving nine a commit bit? that would make thing easier :-)
07:26 schmooster joined #parrot
08:10 contingencyplan joined #parrot
08:51 mj41 joined #parrot
09:05 nbrown joined #parrot
09:05 lucian joined #parrot
09:10 lucian joined #parrot
09:36 he joined #parrot
09:50 dalek nqp/bigint: f8beb9d | moritz++ | src/6model/reprs/P6bigint.c:
09:50 dalek nqp/bigint: avoid allocations and deallocations in bigints {s,g}eg_int
09:50 dalek nqp/bigint:
09:50 dalek nqp/bigint: jaeckel++ in #libtom told me that this works reliably, and a look into the
09:50 dalek nqp/bigint: source confirms it
09:50 dalek nqp/bigint: review: https://github.com/perl6/nqp/commit/f8beb9d627
10:01 dalek parrot: c4e4578 | mls++ | t/pmc/hash.t:
10:01 dalek parrot: add testcase for lexed key miscompilation
10:01 dalek parrot: review: https://github.com/parrot/parrot/commit/c4e45784d2
10:58 jsut_ joined #parrot
11:11 GeJ joined #parrot
11:12 dalek parrot: 94f418e | mls++ | src/hash.c:
11:12 dalek parrot: fix overlong line
11:12 dalek parrot: review: https://github.com/parrot/parrot/commit/94f418e555
11:12 dalek parrot: 1c18b96 | mls++ | / (116 files):
11:12 dalek parrot: Merge branch 'master' into mls/directaccess
11:12 dalek parrot:
11:12 dalek parrot: Conflicts:
11:12 dalek parrot: src/hash.c
11:12 dalek parrot: review: https://github.com/parrot/parrot/commit/1c18b9696b
11:12 dalek parrot: 305b704 | mls++ | compilers/pct/src/PAST/Compiler.pir:
11:12 dalek parrot: simplily PAST::Control, fix bug in block handler code when addreturn is not used
11:12 dalek parrot: review: https://github.com/parrot/parrot/commit/305b704a6b
11:12 dalek Heuristic branch merge: pushed 49 commits to parrot/mls/directaccess by mlschroe
11:22 SHODAN joined #parrot
11:38 Psyche^ joined #parrot
11:40 lucian joined #parrot
12:02 dalek nqp: 91838e0 | mls++ | src/how/NQPClassHOW.pm:
12:02 dalek nqp: use reverse() method instead of manually reversing the mro array
12:02 dalek nqp: review: https://github.com/perl6/nqp/commit/91838e07cd
12:02 atrodo joined #parrot
12:21 lucian joined #parrot
12:39 bluescreen joined #parrot
12:41 whiteknight joined #parrot
12:42 whiteknight good morning, #parrot
12:42 nine good morning, whiteknight
12:44 tadzik good morning #parrot
12:44 whiteknight hello nine, tadzik
12:48 dalek rakudo/nom: 53dd308 | moritz++ | src/core/Hash.pm:
12:48 dalek rakudo/nom: fix Hash.push; fix proposed by coto++
12:48 dalek rakudo/nom: review: https://github.com/rakudo/rakudo/commit/53dd308687
12:59 bluescreen joined #parrot
13:03 mls hi parrot!
13:05 whiteknight good morning, mls
13:05 tadzik hello mls
13:05 mls I see that the branch op now calls Parrot_cx_check_scheduler(), which looks a bit arbitrary
13:06 nine mls: well I guess that's because it is ;)
13:06 whiteknight mls: is that a problem? The branch op is infrequently used, I think
13:06 whiteknight unless it isn't
13:06 mls is that to emulate preemption but still keep the overhead low
13:07 nine mls: yes
13:07 whiteknight we don't exactly get a lot of usage statistics in the feedback
13:07 mls i.e. with the hope that branch gets called often enough?
13:07 mls ok, makes sense
13:07 whiteknight mls: We used to have several ops tagged with a :check_events flag. Ops with that flag would automatically check events to simulate preemption
13:08 whiteknight but the number of ops decreased, and eventually we sort of stopped using the flag, or forgot it, or whatever
13:08 whiteknight so the branch op was probably the only one left, for reasons nobody can remember
13:08 mls ;)
13:08 whiteknight also, we were drinking
13:08 mls ah, that explains a lot ;)
13:08 whiteknight :)
13:09 pbaker joined #parrot
13:09 whiteknight actually, I don't really drink, so sometimes I just make random stupid decisions to feel alive again
13:09 mls so you emulate drinking?
13:09 whiteknight the bad decisions and feeling of regret, yes :) Especially when it comes to software
13:11 mls .oO( warning: don't drink and code... )
13:12 nine mls: why not? That's what we have VCS and test suits for ;)
13:12 mls I have bad memories of coding while I was drunk: the next day, most of the code didn't make sense anymore
13:14 whiteknight other coders are calling you up, "dude, do you have any idea what kind of code you wrote last night?"
13:15 mls yes, that too. No sparc of ingenuity, I'm afraid ;)
13:15 nine drink, code, commit....just never push on the same day. Lest your embarrassment becomes public :)
13:19 tadzik mls: I did that once during GSoC, and not only the code worked the next day, but it also passed all tests and I was unable to figure out any tests that it will fail. I just commited and pushed that :)
13:20 mls wow, I need to switch to that type of alcohol ;)
13:22 whiteknight Seriously. I'm high on life, and I break tests all the time
13:24 whiteknight one weekend I need to get really loaded and write a blog post about IMCC
13:25 mls imcc is not so bad, I think. It gets the job done and is reasonable fast.
13:25 mls the code is just a bit hard to understand
13:27 whiteknight that's the problem with it: the code grew very organically with no long-term plan
13:27 whiteknight it's far too hard to maintain
13:28 mls it has its dark corners, yes
13:28 whiteknight it might get better with continued cleaning, but there are a lot of extremely bad design decisions to overcome
13:28 mls I had fun with that miscompilation on Friday
13:28 whiteknight "let me just jam this in here to get finished faster"
13:28 whiteknight "I'll come back and fix this later"
13:29 mls there should be some interface so that pct and other generators can generate ops without needing to convert them to a string that later on gets parsed again
13:29 whiteknight yes, that would be key
13:30 whiteknight I think that's going to happen through either the packfile API or the Packfile PMCs
13:30 whiteknight we don't have a good interface figured out yet, and suggestions are welcome
13:31 tadzik newPOST was supposed to do that, ney?
13:32 whiteknight tadzik: well, it's two different problems. newPOST is going to require a good interface for generating ops
13:32 whiteknight so we need a good way to make ops, then newPOSt can use that
13:33 whiteknight but any tool for dynamic code generation is going to need the same kind of interface
13:33 tadzik mhm
13:33 dalek nqp: ae3236c | mls++ | / (4 files):
13:33 dalek nqp: bump parrot revision to get directaccess support, use directaccess for most var lookups
13:33 dalek nqp: review: https://github.com/perl6/nqp/commit/ae3236c822
13:33 tadzik oh, direcaccess got merged?
13:33 whiteknight directyaccess support?
13:33 tadzik whiteknight: no find_lex when we remember the register number, OSLT, iirc
13:34 whiteknight oh, okay
13:34 mls yes, dalek didn't report it for some reason
13:35 mls would it make sense to update nqp-rx as well?
13:35 mls (I think it also doesn't have the c3 mro fix)
13:35 whiteknight mls: sure, if you want to get involved in that. I don't know how much effort it will take
13:35 whiteknight github makes it look reversed. Github says master was merged into directaccess, not the other way around
13:36 moritz and it was, at first
13:36 tadzik oh, I know why
13:36 tadzik fast-forward took action
13:36 moritz and then mls merged directaccess into master
13:36 moritz without --no-ff
13:36 tadzik aye
13:36 mls ah, sorry, my mistake
13:36 mls Last time I used --no-ff
13:36 moritz and since the branches haven't diverged, no separate merge commit was made
13:36 tadzik I wonder if that's hard to get right, or does git have unsane defaults (unsane intended)
13:37 whiteknight it doesn't really matter, the important thing is that the code is in the repo
13:37 tadzik maybe "subsane" is more appropriate
13:37 moritz tadzik: well, if 'git merge' defaulted to --no-ff, at least 'git pull' shouldn't default to 'git merge' :-)
13:37 tadzik moritz: I have a feeling we ought to --rebase when we pull, and --no-ff when we merge, I wonder how bad other things will get if that was the default
13:51 JimmyZ joined #parrot
13:52 mls oh no, rakudo doesn't build anymore :(
13:53 whiteknight oh noes!
13:53 tadzik oh, just noticed
13:53 tadzik Method 'pirop' not found for invocant of class 'POST;Ops'
13:53 mls ...last minute code cleanups...
13:54 mls all the 6model stuff is not really used in nqp, thus not checked with the testsuite
13:56 mls eek, wrong label
13:56 mls sorry
13:57 mls testing...
14:04 dalek nqp: ca12593 | mls++ | src/PAST/SixModelPASTExtensions.pir:
14:04 dalek nqp: fix label name so that rakudo builds again
14:04 dalek nqp: review: https://github.com/perl6/nqp/commit/ca12593779
14:07 benabik joined #parrot
14:08 benabik o/ #parrot
14:40 whiteknight good morning, benabik
14:40 benabik o/ whiteknight.  How's things?
14:40 whiteknight benabik: good, thanks. We're moving forward on yet another house so we are cautiously optimistic
14:41 benabik The cycle begins again.  Sending good vibes in your direction and hope for a lack of problems.
14:41 whiteknight we had the inspection over the weekend and things are mostly looking good, so that has us hopeful
15:04 PacoLinux joined #parrot
15:05 particle joined #parrot
15:38 dalek winxed: 6b9c3c8 | NotFound++ | winxedst1.winxed:
15:38 dalek winxed: fix operators && and ||, issue 14
15:38 dalek winxed: review: https://github.com/NotFoun​d/winxed/commit/6b9c3c8213
15:40 dalek rakudo/nom: ee1a1c5 | moritz++ | tools/build/NQP_REVISION:
15:40 dalek rakudo/nom: bump NQP revision, just because we can
15:40 dalek rakudo/nom: review: https://github.com/rakudo/rakudo/commit/ee1a1c5d2e
15:50 bluescreen joined #parrot
15:56 dalek winxed: afd7c7a | NotFound++ | t/intarray.t:
15:56 dalek winxed: improve int array tests
15:56 dalek winxed: review: https://github.com/NotFoun​d/winxed/commit/afd7c7ab22
15:56 dalek winxed: 24b8d40 | NotFound++ | t/ (2 files):
15:56 dalek winxed: move int array tests to the appropiate directory
15:56 dalek winxed: review: https://github.com/NotFoun​d/winxed/commit/24b8d40540
16:13 dalek winxed: 375d25b | NotFound++ | t/basic/04try.t:
16:13 dalek winxed: basic tests for try/catch
16:13 dalek winxed: review: https://github.com/NotFoun​d/winxed/commit/375d25b161
16:50 dalek parrot: f7b8d31 | mls++ | compilers/pct/src/PAST/Compiler.pir:
16:50 dalek parrot: fix another bug in the return-less wrap_handlers case
16:50 dalek parrot: review: https://github.com/parrot/parrot/commit/f7b8d31f85
17:02 dalek nqp/reprapi2: a743c97 | jnthn++ | src/ (9 files):
17:02 dalek nqp/reprapi2: Toss clone from the REPR API; it's just implementable in terms of the more primitive allocate and copy_to.
17:02 dalek nqp/reprapi2: review: https://github.com/perl6/nqp/commit/a743c97fab
17:10 cotto ~~
17:11 dalek nqp/reprapi2: e4e4bb4 | jnthn++ | src/6model/ (8 files):
17:11 dalek nqp/reprapi2: Add gc_cleanup to REPR API; none of the current reprs need it, but bigint will. Not yet called.
17:11 dalek nqp/reprapi2: review: https://github.com/perl6/nqp/commit/e4e4bb465f
17:35 dalek nqp/reprapi2: 6dc6dc9 | jnthn++ | src/6model/reprs/P6opaque. (2 files):
17:35 dalek nqp/reprapi2: Teach P6opaque to delegate to other reprs where it should. We now longer special case handling of various things in P6opaque that we probably never should have, have the hooks for getting P6bigint correctly flattenable into P6opaque, etc.
17:35 dalek nqp/reprapi2: review: https://github.com/perl6/nqp/commit/6dc6dc92b1
17:36 aloha joined #parrot
17:37 bacek_at_work joined #parrot
17:42 dukeleto msg bubablub would love your help with parrot-libgit2. just gave you a commit bit. currently trying to fix https://github.com/letolab​s/parrot-libgit2/issues/8
17:42 aloha OK. I'll deliver the message.
17:53 fperrad joined #parrot
17:53 dalek rakudo/reprapi2: 5711fcf | jnthn++ | src/binder/sixmodelobject.h:
17:53 dalek rakudo/reprapi2: Update sixmodelobject.h. Unsurprisingly, build epicly busted.
17:53 dalek rakudo/reprapi2: review: https://github.com/rakudo/rakudo/commit/5711fcf213
17:53 dalek rakudo/reprapi2: 63a6131 | jnthn++ | src/pmc/perl6lexpad.pmc:
17:53 dalek rakudo/reprapi2: Update Perl6Lexpad for new REPR API.
17:53 dalek rakudo/reprapi2: review: https://github.com/rakudo/rakudo/commit/63a613104a
18:07 dalek rakudo/reprapi2: 71cf8b8 | jnthn++ | src/ops/perl6.ops:
18:07 dalek rakudo/reprapi2: Update ops for REPR API changes.
18:07 dalek rakudo/reprapi2: review: https://github.com/rakudo/rakudo/commit/71cf8b8ecb
18:07 dalek rakudo/reprapi2: 69b450a | jnthn++ | src/binder/bind.c:
18:07 dalek rakudo/reprapi2: Update the binder for the new REPR API.
18:08 dalek rakudo/reprapi2: review: https://github.com/rakudo/rakudo/commit/69b450a6a0
18:08 mj41 joined #parrot
18:19 dalek rakudo/reprapi2: cf864ab | jnthn++ | src/binder/multidispatch.c:
18:19 dalek rakudo/reprapi2: Update multi-dispatcher for new REPR API.
18:19 dalek rakudo/reprapi2: review: https://github.com/rakudo/rakudo/commit/cf864abc2a
18:19 dalek rakudo/reprapi2: 55293a2 | jnthn++ | src/binder/ (2 files):
18:19 dalek rakudo/reprapi2: Get remaining C parts updated, or at least to a first approximation. We now get to trying to build the setting...BOOM SEGFAULT!
18:19 dalek rakudo/reprapi2: review: https://github.com/rakudo/rakudo/commit/55293a2a75
18:32 benabik joined #parrot
18:35 whiteknight BOOM HEADSHOT
18:36 benabik Ahhhh!!!
18:36 * benabik ducks.
18:43 dukeleto ~~
18:51 contingencyplan joined #parrot
19:01 ambs joined #parrot
19:08 mls uh, wtf! Why is PARROT_ERRORS_RESULT_COUNT_FLAG off by default?
19:10 whiteknight is it?
19:10 whiteknight you can toggle it if you want it
19:10 mls seems to be the case. another two hours lost in debugging rakudo
19:12 benabik Likely it's off because it didn't exist and whoever added it didn't want to break code that ignored extra return values.
19:13 benabik Arguably underflow and overflow are different kinds of error.  Wanting two but getting three is somewhat different than wanting two and getting one.
19:13 mls uh undefined behavior is never a good idea...
19:13 mls overflow is never an error AFAICT
19:14 mls this is about underflow, i.e. using an return value of a function that returned nothing
19:14 benabik PARROT_ERRORS_RESULT_COUNT_FLAG seems to consider over or underflow an error.
19:15 mls really? checking the code...
19:15 benabik Well, it's documented that it checks for both.
19:15 mls oh, then it must be correct ;)
19:15 benabik I'm not sure where it's actually implemented.
19:16 mls fill_params
19:17 mls Hmm, there is an "too many positional arguments: %d passed, %d expected" error message
19:17 mls that would be bad for return value
19:17 mls values
19:17 benabik Yes.  Experimentally I get error on over and underflow.
19:19 mls we should make it possible to only flag underflow for return values, I think
19:19 mls but I want underflow errors to be on. Those are really hard to find errors
19:20 benabik Makes sense.
19:20 mls you change some code and suddenly things stop working because some register value is different and the register got reused as return value
19:20 mls I just had that with rakudo
19:20 mls cost me two hours
19:21 mj41 joined #parrot
19:37 mls afk -> home
19:41 davidfetter joined #parrot
19:43 dukeleto davidfetter: fancy seeing you here
19:43 * whiteknight hates sql a little bit more every day
19:43 whiteknight especialaly SQL Server Management Studio
19:43 davidfetter dukeleto, yeah, it's all about the defaults for chatzilla ;)
20:18 dukeleto https://github.com/blog/977-access-your-even​t-timelines-and-oauth-tokens-through-the-api
20:32 dalek winxed: 414187f | NotFound++ | winxedst1.winxed:
20:32 dalek winxed: fix for volatile and indent emited store_lex
20:32 dalek winxed: review: https://github.com/NotFoun​d/winxed/commit/414187f019
20:33 dalek winxed: 9ed54d6 | NotFound++ | t/advanced/10volatile.t:
20:33 dalek winxed: some tests for volatile
20:33 dalek winxed: review: https://github.com/NotFoun​d/winxed/commit/9ed54d6be1
20:51 dalek winxed: 39a16dc | NotFound++ | t/b (3 files):
20:51 dalek winxed: move tests for & and | to t/basic/
20:51 dalek winxed: review: https://github.com/NotFoun​d/winxed/commit/39a16dc602
20:51 plobsing joined #parrot
21:56 dalek winxed: 687124e | NotFound++ | t/b (2 files):
21:56 dalek winxed: move to t/basic and improve tests for &&, add tests for ||
21:56 dalek winxed: review: https://github.com/NotFoun​d/winxed/commit/687124e5a8
22:01 whiteknight joined #parrot
22:03 AzureSto_ joined #parrot
22:04 dalek winxed: 102d4c8 | NotFound++ | t/basic/02boolops.t:
22:04 dalek winxed: simplify workaround for stage 0
22:04 dalek winxed: review: https://github.com/NotFoun​d/winxed/commit/102d4c8556
22:10 dalek winxed: 988cf03 | NotFound++ | t/ (2 files):
22:10 dalek winxed: move tests for operator ! to t/basic/
22:10 dalek winxed: review: https://github.com/NotFoun​d/winxed/commit/988cf03c0b
22:18 rfw joined #parrot
22:45 dalek nqp/reprapi2: a375e18 | jnthn++ | src/ops/nqp.ops:
22:45 dalek nqp/reprapi2: Don't look up something we already have.
22:45 dalek nqp/reprapi2: review: https://github.com/perl6/nqp/commit/a375e18532
22:45 dalek nqp/reprapi2: c0c5bb0 | jnthn++ | src/NQP/SymbolTable.pm:
22:45 dalek nqp/reprapi2: Fix a nasty NQP bug where the REPL and code run immediately rather than pre-compiled didn't properly see the setting.
22:45 dalek nqp/reprapi2: review: https://github.com/perl6/nqp/commit/c0c5bb0703
22:45 dalek nqp/reprapi2: 516c219 | jnthn++ | src/6model/reprs/P6opaque.c:
22:45 dalek nqp/reprapi2: Add a missing sanity check to P6opaque that catches some nasty bugs.
22:45 dalek nqp/reprapi2: review: https://github.com/perl6/nqp/commit/516c219f3a
22:45 whiteknight happy halloween, for those of you who observe
22:59 dalek rakudo/reprapi2: caf5040 | jnthn++ | src/Perl6/Metamodel/ContainerDescriptor.pm:
22:59 dalek rakudo/reprapi2: Avoid the NQP default constructor for now, which doesn't handle native types properly. This gets us a bit further into the build.
22:59 dalek rakudo/reprapi2: review: https://github.com/rakudo/rakudo/commit/caf50408f8
23:17 dalek nqp/reprapi2: fd65ca0 | jnthn++ | src/ops/nqp.ops:
23:17 dalek nqp/reprapi2: Fix thinko in repr_clone op.
23:17 dalek nqp/reprapi2: review: https://github.com/perl6/nqp/commit/fd65ca0b4f
23:23 benabik joined #parrot
23:27 bubaflub joined #parrot
23:29 dalek winxed: 175ad9b | NotFound++ | winxedst0.cpp:
23:29 dalek winxed: quick fix to assignment in stage 0 to allow some tests
23:29 dalek winxed: review: https://github.com/NotFoun​d/winxed/commit/175ad9b4c4

| Channels | #parrot index | Today | | Search | Google Search | Plain-Text | summary

Parrot | source cross referenced