Perl 6 - the future is here, just unevenly distributed

IRC log for #moarvm, 2015-08-29

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

All times shown according to UTC.

Time Nick Message
03:20 camelia joined #moarvm
07:25 colomon joined #moarvm
07:27 lizmat joined #moarvm
07:42 FROGGS joined #moarvm
07:42 Ven joined #moarvm
07:58 dalek MoarVM: 6fec043 | paultcochrane++ | src/io/procops.c:
07:58 dalek MoarVM: Fix copy-and-paste error
07:58 dalek MoarVM:
07:58 dalek MoarVM: The attribute si->ds_stderr is actually being destroyed in this part of the
07:58 dalek MoarVM: code (similarly to si->ds_stdout in th block above this change), thus the
07:58 dalek MoarVM: call to `MVM_string_decodestream_destory` should really use si->ds_stderr
07:58 dalek MoarVM: instead of si->ds_stdout.
07:58 dalek MoarVM: review: https://github.com/MoarVM/MoarVM/commit/6fec043a20
07:58 dalek MoarVM: 7fdde0b | FROGGS++ | src/io/procops.c:
07:58 dalek MoarVM: Merge pull request #238 from paultcochrane/pr/fix-copy-paste-error
07:58 dalek MoarVM:
07:58 dalek MoarVM: Fix copy-and-paste error
07:58 dalek MoarVM: review: https://github.com/MoarVM/MoarVM/commit/7fdde0b901
07:58 Ven joined #moarvm
08:15 Ven joined #moarvm
08:19 _itz_ joined #moarvm
08:24 camelia joined #moarvm
08:39 Ven joined #moarvm
09:07 virtualsue joined #moarvm
09:09 colomon joined #moarvm
09:09 * virtualsue can haz PR: https://github.com/MoarVM/MoarVM/pull/239
09:17 dalek MoarVM: 48f5a84 | (Sue Spence)++ | src/mast/driver.c:
09:17 dalek MoarVM: Replace use of comma op inside an init of an array of pointers as this wasn't doing what was intended. Also gets rid of the last warning in this codebase (as of now).
09:17 dalek MoarVM: review: https://github.com/MoarVM/MoarVM/commit/48f5a8419a
09:17 dalek MoarVM: e187d89 | (Jimmy Zhuo)++ | src/mast/driver.c:
09:17 dalek MoarVM: Merge pull request #239 from virtualsue/master
09:17 dalek MoarVM:
09:17 dalek MoarVM: Replace use of comma op inside an init of an array of pointers as thi…
09:17 dalek MoarVM: review: https://github.com/MoarVM/MoarVM/commit/e187d8922c
09:17 * [Coke] is just waiting for the travisci to finish.
09:17 JimmyZ virtualsue: thank you :)
09:17 [Coke] ... ah, JimmyZ++ got it. virtualsue++
09:17 JimmyZ well, clang is slow ...
09:18 JimmyZ from the travis ci ..
09:24 _itz_ clang36 is a lot faster compiling moar than earlier versions
09:24 JimmyZ yeah
09:29 lizmat joined #moarvm
09:38 Ven joined #moarvm
09:41 hoelzro m: https://gist.github.com/hoelzro/73404929704270e26891
09:41 camelia rakudo-moar 5fb81f: OUTPUT«before waithere␤␤Internal error: Unwound entire stack and missed handler␤»
09:41 hoelzro weeeeee
09:42 FROGGS lol
09:42 FROGGS (actually not so loud to not to disturb people)
09:44 hoelzro hehe
09:44 hoelzro we probably shouldn't even compile that program
09:48 JimmyZ p:  https://gist.github.com/hoelzro/73404929704270e26891
09:48 JimmyZ r:  https://gist.github.com/hoelzro/73404929704270e26891
09:49 camelia rakudo-jvm 5fb81f: OUTPUT«before wait␤here␤Exception in thread "Thread-4" org.perl6.nqp.runtime.LexoticException␤       at org.perl6.nqp.runtime.ThreadContext.<init>(ThreadContext.java:119)␤  at org.perl6.nqp.runtime.GlobalContext.getCurrentThreadContext(GlobalContext.java:340)␤ …»
09:49 camelia ..rakudo-moar 5fb81f: OUTPUT«before wait␤here␤Internal error: Unwound entire stack and missed handler␤»
09:51 JimmyZ looks like  return cause it
09:51 timotimo yeah
09:52 hoelzro yeah, it's return
09:54 JimmyZ without start, it is also wrong
09:55 JimmyZ so, it is a rakudo bug, not moarvm
10:09 [Coke] ugh. I feel like we've closed more tickets than this. only at 1,039
10:10 [Coke] ww.
10:10 hoelzro JimmyZ: I agree; I don't think we should be compiling that
10:22 Ven joined #moarvm
10:34 dalek MoarVM: 601561e | FROGGS++ | src/io/procops.c:
10:34 dalek MoarVM: move declaration before code
10:34 dalek MoarVM: review: https://github.com/MoarVM/MoarVM/commit/601561e947
11:20 brrt joined #moarvm
11:29 virtualsue joined #moarvm
11:40 brrt good *
11:42 dalek MoarVM: 4c2c3cb | paultcochrane++ | src/core/dll.c:
11:42 dalek MoarVM: Unlock mutex before returning
11:42 dalek MoarVM:
11:42 dalek MoarVM: The locked mutex could return without being unlocked beforehand.  This
11:42 dalek MoarVM: change adds the missing unlock and avoids a potential thread blockage.
11:42 dalek MoarVM: review: https://github.com/MoarVM/MoarVM/commit/4c2c3cb6c7
11:42 dalek MoarVM: f69883d | lizmat++ | src/core/dll.c:
11:42 dalek MoarVM: Merge pull request #240 from paultcochrane/pr/unlock-mutex
11:42 dalek MoarVM:
11:43 dalek MoarVM: Unlock mutex before returning
11:43 dalek MoarVM: review: https://github.com/MoarVM/MoarVM/commit/f69883d6bd
11:44 lizmat brrt o/
12:10 Ven joined #moarvm
12:15 virtualsue joined #moarvm
12:23 rarara joined #moarvm
12:40 virtualsue joined #moarvm
12:44 dalek MoarVM: 45cae9b | paultcochrane++ | src/6model/reprs/MultiDimArray.c:
12:44 dalek MoarVM: Initialise all spec attributes to default values
12:44 dalek MoarVM:
12:44 dalek MoarVM: Not all of the `spec` struct attributes are assigned within
12:44 dalek MoarVM: `get_elem_storage_spec()`.  This change sets all attributes so that none
12:44 dalek MoarVM: will be potentially used uninitialised.
12:44 dalek MoarVM: review: https://github.com/MoarVM/MoarVM/commit/45cae9b0ae
12:44 dalek MoarVM: 10dce90 | (Jimmy Zhuo)++ | src/6model/reprs/MultiDimArray.c:
12:44 dalek MoarVM: Merge pull request #237 from paultcochrane/pr/initialise-storage-spec
12:44 dalek MoarVM:
12:44 dalek MoarVM: Initialise all spec attributes to default values
12:44 dalek MoarVM: review: https://github.com/MoarVM/MoarVM/commit/10dce907e4
12:45 dalek MoarVM: 1a4e7d8 | paultcochrane++ | src/core/exceptions.c:
12:45 dalek MoarVM: Plug resource leaks in exceptions.c
12:45 dalek MoarVM:
12:45 dalek MoarVM: Since `MVM_string_utf8_encode_C_string()` was being called inside another
12:45 dalek MoarVM: expression/function, it was implicitly allocating memory which wasn't being
12:45 dalek MoarVM: released.  This change refactors the code allocate the memory explicitly so
12:45 dalek MoarVM: that it can be freed, while also avoiding potentially dereferencing NULL.
12:45 dalek MoarVM: review: https://github.com/MoarVM/MoarVM/commit/1a4e7d8ed9
12:45 dalek MoarVM: b8ee3c5 | (Jimmy Zhuo)++ | src/core/exceptions.c:
12:45 dalek MoarVM: Merge pull request #241 from paultcochrane/pr/fix-exceptions-resource-leak
12:45 dalek MoarVM:
12:45 dalek MoarVM: Plug resource leaks in exceptions.c
12:47 dalek MoarVM: 10e8783 | (Jimmy Zhuo)++ | src/6model/reprs/MultiDimArray.c:
12:47 dalek MoarVM: cleans up
12:47 dalek MoarVM: review: https://github.com/MoarVM/MoarVM/commit/10e8783c65
12:48 Ven joined #moarvm
12:54 dalek MoarVM: e87e8cc | hoelzro++ | build/setup.pm:
12:54 dalek MoarVM: Make mixed declarations and statements fatal on GCC
12:54 dalek MoarVM:
12:54 dalek MoarVM: Prevent hoelzro (me) from breaking the Windows build!
12:54 dalek MoarVM: review: https://github.com/MoarVM/MoarVM/commit/e87e8cce4e
12:57 dalek MoarVM: 409d30e | timotimo++ | src/core/exceptions.c:
12:57 dalek MoarVM: move declaration to beginning of function
12:57 dalek MoarVM: review: https://github.com/MoarVM/MoarVM/commit/409d30e973
12:57 FROGGS hoelzro++
12:58 hoelzro timotimo beat me to the fix =/
13:02 timotimo :-)
13:24 mtj_ joined #moarvm
13:25 nebuchadnezzar joined #moarvm
13:25 _itz joined #moarvm
13:25 ShimmerFairy joined #moarvm
13:25 tadzik joined #moarvm
13:26 camelia joined #moarvm
13:27 virtualsue joined #moarvm
13:40 travis-ci joined #moarvm
13:40 travis-ci MoarVM build failed. Rob Hoelz 'Make mixed declarations and statements fatal on GCC
13:40 travis-ci https://travis-ci.org/MoarVM/MoarVM/builds/77812544 https://github.com/MoarVM/MoarVM/compare/10e8783c65f5...e87e8cce4e70
13:40 travis-ci left #moarvm
13:40 virtualsue joined #moarvm
13:44 Ven joined #moarvm
13:44 colomon joined #moarvm
13:49 dalek MoarVM: b5db76e | FROGGS++ | src/core/nativecall_libffi.c:
13:49 dalek MoarVM: move declaration before code
13:49 dalek MoarVM: review: https://github.com/MoarVM/MoarVM/commit/b5db76e5c7
13:49 timotimo thank ci for running libffi, too
13:51 travis-ci joined #moarvm
13:51 travis-ci MoarVM build failed. Timo Paulssen 'move declaration to beginning of function'
13:51 travis-ci https://travis-ci.org/MoarVM/MoarVM/builds/77812865 https://github.com/MoarVM/MoarVM/compare/e87e8cce4e70...409d30e97393
13:51 travis-ci left #moarvm
13:53 FROGGS my commit should pass the thang
13:55 brrt joined #moarvm
13:55 dalek MoarVM: be09cbe | paultcochrane++ | src/spesh/graph.c:
13:55 dalek MoarVM: Remove resource leaks in spesh/graph.c
13:55 dalek MoarVM:
13:55 dalek MoarVM: `MVM_spesh_dump` was being used inside a `printf`, thus the implicitly
13:55 dalek MoarVM: allocated memory can't be released.  This refactors the code to make the
13:55 dalek MoarVM: allocation explicit so that the memory can be freed.
13:55 dalek MoarVM: review: https://github.com/MoarVM/MoarVM/commit/be09cbef9d
13:55 dalek MoarVM: 4b284dd | FROGGS++ | src/spesh/graph.c:
13:55 dalek MoarVM: Merge pull request #242 from paultcochrane/pr/plug-resource-leak-in-graph
13:55 dalek MoarVM:
13:55 dalek MoarVM: Remove resource leaks in spesh/graph.c
13:55 dalek MoarVM: review: https://github.com/MoarVM/MoarVM/commit/4b284dd9bd
13:58 colomon joined #moarvm
14:07 brrt lizmat o/
14:07 * brrt has been picking low-hanging fruit today
14:08 dalek joined #moarvm
14:10 brrt what :-o i stop at enter-conditional?
14:11 brrt awesome
14:13 travis-ci joined #moarvm
14:13 travis-ci MoarVM build passed. Tobias Leich 'move declaration before code'
14:13 travis-ci https://travis-ci.org/MoarVM/MoarVM/builds/77816654 https://github.com/MoarVM/MoarVM/compare/409d30e97393...b5db76e5c7a3
14:13 travis-ci left #moarvm
14:13 timotimo brrt: does that mean we're going even further forward towards running jitted code?
14:14 brrt yes
14:14 brrt onward, comrades
14:24 travis-ci joined #moarvm
14:24 travis-ci MoarVM build passed. Tobias Leich 'Merge pull request #242 from paultcochrane/pr/plug-resource-leak-in-graph
14:24 travis-ci https://travis-ci.org/MoarVM/MoarVM/builds/77817076 https://github.com/MoarVM/MoarVM/compare/b5db76e5c7a3...4b284dd9bd3e
14:24 travis-ci left #moarvm
14:56 Ven joined #moarvm
15:02 dalek MoarVM: d4385c9 | hoelzro++ | src/ (3 files):
15:02 dalek MoarVM: Add ready callback for async procs
15:02 dalek MoarVM:
15:02 dalek MoarVM: This way the HLL can find out when a process is ready
15:02 dalek MoarVM: review: https://github.com/MoarVM/MoarVM/commit/d4385c98dd
15:03 timotimo brrt: looking forward to seeing the next commit :)
15:03 brrt next commit removes the 'mem' thingie
15:04 timotimo ah
15:05 dalek joined #moarvm
15:16 travis-ci joined #moarvm
15:16 travis-ci MoarVM build failed. Rob Hoelz 'Add ready callback for async procs
15:16 travis-ci https://travis-ci.org/MoarVM/MoarVM/builds/77822270 https://github.com/MoarVM/MoarVM/compare/4b284dd9bd3e...d4385c98dd81
15:16 travis-ci left #moarvm
15:19 dalek MoarVM: 80db16c | hoelzro++ | src/io/procops.c:
15:19 dalek MoarVM: Fix mixed declaration failure
15:19 dalek MoarVM: review: https://github.com/MoarVM/MoarVM/commit/80db16ce91
15:26 timotimo do you never compile before you push? :)
15:27 JimmyZ I guess he use clang
15:28 timotimo poor he
15:28 timotimo he has a thinkpad, why would he use clang?
15:28 hoelzro timotimo: no, I forgot to re Configure.pl, so my Makefile was out of date =/
15:29 timotimo that makes sense
15:29 hoelzro hmm...should Makefile check that its mtime > mtime(Configure.pl)?
15:39 travis-ci joined #moarvm
15:39 travis-ci MoarVM build passed. Rob Hoelz 'Fix mixed declaration failure'
15:39 travis-ci https://travis-ci.org/MoarVM/MoarVM/builds/77823656 https://github.com/MoarVM/MoarVM/compare/d4385c98dd81...80db16ce910b
15:39 travis-ci left #moarvm
16:11 colomon joined #moarvm
16:22 virtualsue joined #moarvm
16:40 colomon joined #moarvm
16:47 lizmat joined #moarvm
16:48 FROGGS joined #moarvm
16:52 dalek MoarVM: 37300a9 | paultcochrane++ | src/spesh/candidate.c:
16:52 dalek MoarVM: Fix resource leak in spesh/candidate.c
16:52 dalek MoarVM:
16:52 dalek MoarVM: The `before` and `after` variables were not being freed before returning
16:52 dalek MoarVM: from the function and were thus leaking resources.  This change plugs the
16:52 dalek MoarVM: leak.
16:52 dalek MoarVM: review: https://github.com/MoarVM/MoarVM/commit/37300a9da6
16:52 dalek MoarVM: 1ee1e91 | FROGGS++ | src/spesh/candidate.c:
16:52 dalek MoarVM: Merge pull request #243 from paultcochrane/pr/fix-resource-leak-candidate
16:52 dalek MoarVM:
16:52 dalek MoarVM: Fix resource leak in spesh/candidate.c
16:52 dalek MoarVM: review: https://github.com/MoarVM/MoarVM/commit/1ee1e91dc6
17:56 ShimmerFairy joined #moarvm
18:42 colomon joined #moarvm
18:47 lizmat joined #moarvm
18:50 lizmat_ joined #moarvm
19:36 colomon joined #moarvm
19:57 colomon joined #moarvm
20:04 colomon joined #moarvm
20:19 colomon joined #moarvm
20:23 FROGGS joined #moarvm
20:25 lizmat joined #moarvm
20:51 dalek MoarVM: bd65022 | paultcochrane++ | src/core/dll.c:
20:51 dalek MoarVM: Add missing curlies to if statement
20:51 dalek MoarVM:
20:51 dalek MoarVM: In my previous fix of the mutex unlock issue, the code flow had changed.  To
20:51 dalek MoarVM: maintain the old behaviour curlies are needed in the if statement.
20:51 dalek MoarVM: review: https://github.com/MoarVM/MoarVM/commit/bd650220d8
20:51 dalek MoarVM: e8f7c01 | lizmat++ | src/core/dll.c:
20:51 dalek MoarVM: Merge pull request #244 from paultcochrane/pr/add-missing-curlies
20:51 dalek MoarVM:
20:51 dalek MoarVM: Add missing curlies to if statement
20:51 dalek MoarVM: review: https://github.com/MoarVM/MoarVM/commit/e8f7c0149a
20:56 colomon joined #moarvm
21:03 vendethiel joined #moarvm
21:05 virtualsue joined #moarvm
22:32 ggoebel joined #moarvm
22:32 _itz_ joined #moarvm
22:40 _itz joined #moarvm
22:48 _itz joined #moarvm
23:03 _itz_ joined #moarvm

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