Perl 6 - the future is here, just unevenly distributed

IRC log for #perl6-dev, 2016-05-23

| Channels | #perl6-dev index | Today | | Search | Google Search | Plain-Text | summary

All times shown according to UTC.

Time Nick Message
01:06 stmuk joined #perl6-dev
03:05 skids joined #perl6-dev
04:20 lizmat joined #perl6-dev
06:10 sno joined #perl6-dev
06:11 [Tux] This is Rakudo version 2016.05-6-g2c45068 built on MoarVM version 2016.05
06:11 [Tux] test            20.440
06:11 [Tux] test-t          13.533
06:11 [Tux] csv-parser      35.139
06:21 nine AlexDaniel: please don't
06:42 raydiak joined #perl6-dev
07:14 brrt joined #perl6-dev
07:28 RabidGravy joined #perl6-dev
07:37 masak joined #perl6-dev
09:04 AlexDaniel joined #perl6-dev
09:05 AlexDaniel nine: ok, but what's your reasoning?
09:21 llfourn I think it's not worth it because git blame etc is less clear if there's a bunch of lines changed with "whitespace removal" as the commit
09:25 brrt also, very difficult to check all these files by hand if there are no sudden and unexpected changes of semantics
09:25 brrt which would be bad
09:33 AlexDaniel llfourn: git blame -w
09:34 llfourn AlexDaniel: cool didn't know about that thx!
09:35 AlexDaniel brrt: you mean when whitespace is part of the test? I've checked the diffs.
09:35 brrt i know. but i haven't
09:35 brrt and neither has anyone else
09:36 * llfourn wonders if he can do git blame -w with magit
09:36 brrt probably an emacs variable you can define
09:39 llfourn actually it's built into magit magit-blame-popup lets you enable it :)
09:39 AlexDaniel llfourn: also, if you're using github, then just add ?w=1 to the url
09:40 llfourn nice.
09:45 brrt i'm no authority AlexDaniel, it may be worth it
09:45 brrt but i wouldn't be able to tell you
10:13 AlexDaniel my problem with it is that a lot of people have their editors set up in such a way that trailing whitespace is shown in red. When you open a file that has a lot of that, it shines up like a christmas tree and makes you cringe… It looks a bit unprofessional, IMO
10:13 AlexDaniel another point is that people tend to delete trailing whitespace anyway, e.g. see https://github.com/perl6/roast/commit/25c40d05f70ba3fabb83cb35a714344bc34c449f#diff-f857cacc3f7bbbfe0b3d614b036a9e96R52
10:14 AlexDaniel so it pollutes the diffs anyway, it is just spread across many commits
10:14 AlexDaniel .oO( bikeshedding again… )
10:30 stmuk I don't understand why anyone would care about trailing whitespace
10:35 RabidGravy I've worked with people who get quite agitated about it
10:36 jnthn Some quite widely used editors will also put whitespace characters on an "empty" line that correspond to the indent of code lines above/below.
10:37 stmuk I've worked with people who wouldn't eat different coloured vegetables which touched as well :)
10:37 jnthn And that is often counted as "trailing whitespace" too. :/
10:38 stmuk I'd be annoyed by trailing whitespace at the end of a file (more common with emacs users than vim)
10:39 RabidGravy stmuk, I think I recall the person in question
10:42 AlexDaniel stmuk: why?
10:42 AlexDaniel stmuk: I mean, why is it worse than 5 extra spaces at the end of some line
10:43 AlexDaniel no newline at the end of a file – I get that, some tools freak out because of that
10:58 RabidGravy I keep meaning to go through the moar source to fix that, there's quite a few files that a C compiler of a certain vintage moans about
10:59 brrt joined #perl6-dev
11:03 brrt the same c compiler that complains about %ld not being the right thing for $lld
11:03 brrt long long decimal
11:16 stmuk AlexDaniel: I think you have to go out of your way with config to see trailing whitespace at the end of the line whereas trailing lines at eol are visible in some default editor configs
11:16 brrt whitespace-mode shows it directly in emacs
11:16 brrt i think whitespace-mode is very useful
11:18 AlexDaniel it is, yes
11:18 stmuk but its not a default mode in emacs whereas vim does show trailing lines by default
11:18 AlexDaniel but I think that spacemacs displays trailing whitespace even when whitespace-mode is off
12:27 [Coke] it's possible to write a script to do blame that skips whitespace if we need it.
12:28 [Coke] I find that non-conforming code still bugs me these days, but not enough to keep it fixed like I've had on other projects.
12:28 [Coke] (esp. when I'm not the one who decides what "conform" means.)
12:31 [Coke] oh. git blame -w, even better.
12:58 skids joined #perl6-dev
13:03 hoelzro o/ #perl6
13:10 [Coke] hio
13:10 AlexDaniel OK so I'll submit it as a pull request tomorrow. Those who are against are welcome to downvote it :)
13:15 hoelzro I think I may have found a rakudo bug, but it may be a misunderstanding of how nextsame works
13:16 hoelzro multi sub foo { bar }; sub bar { nextsame }; foo()
13:16 hoelzro should that nextsame dispatch to other &foo candidates (if there were any)?
13:18 jnthn No
13:18 jnthn If it does it's a bug
13:18 jnthn (Does it? :))
13:18 jnthn The nextsame has to appear lexically within the multi, though
13:19 hoelzro jnthn: it seems to; at the very least, it doesn't throw a "no dispatcher" error
13:20 jnthn Are you sure?
13:20 jnthn nextsame is designed to be quiet if there's nowhere for it to dispatch to
13:20 jnthn m: sub foo() { nextsame() }; say foo()
13:20 camelia rakudo-moar 2c4506: OUTPUT«nextsame is not in the dynamic scope of a dispatcher␤  in sub foo at /tmp/ewT5vCAlHr line 1␤  in block <unit> at /tmp/ewT5vCAlHr line 1␤␤»
13:21 jnthn Oh...
13:21 jnthn Hmm
13:21 jnthn I thought an only would clobber the dispatcher. Hmmmm.
13:21 hoelzro m: multi sub outer(Any $) { say 'any way you want it' }; multi sub outer(Str $) { inner() }; sub inner() { nextsame }; outer('foo')
13:21 camelia rakudo-moar 2c4506: OUTPUT«any way you want it␤»
13:21 jnthn Curious.
13:21 hoelzro my thoughts too
13:21 hoelzro jnthn: should I RT that?
13:21 jnthn Yeah, I mean it may be OK but I'm not really sure it is :)
13:23 hoelzro well, I dug into this because of a problem we had with Test::subtest just before the release
13:24 hoelzro it was changed to a multi, which interestingly enough broke a test using nextsame
13:33 nine AlexDaniel: yes, there's probably a solution for every way one can do a git blame. But every user will have to know and use them.
13:34 dalek nqp: d0b3b6e | brrt++ | t/moar/50-jit-register-alloc.t:
13:34 dalek nqp: Add test for even-moar-jit register allocator
13:34 dalek nqp:
13:34 dalek nqp: The current register allocator can't spill registers, so it may
13:34 dalek nqp: run out of them and die.
13:34 dalek nqp: review: https://github.com/perl6/nqp/commit/d0b3b6eca7
13:34 nine AlexDaniel: for example, I've no idea how to tell vim to ignore whitespace for :Gblame
13:43 AlexDaniel nine: what about :Gblame w
13:43 [Coke] nine - you ever run the perl6 doc site builder, htmlify?
13:44 [Coke] getting warnings on "        $handle = $precomp.load($id);", where precomp isa my $precomp = CompUnit::PrecompilationRepository::Default.new(store => $precomp-store);
13:44 AlexDaniel nine: “You can give any of ltfnsewMC as flags and they will be passed along to git-blame.”
13:45 [Coke] (issue might have nothing to do with precomp, digging more...)
13:46 AlexDaniel or maybe it is :Gblame -w
13:57 [Coke] nine: looks like the uninitialize value in str contet warning is coming from inside CompUnit::PrecompilationRepository's precompile method.
14:09 lizmat joined #perl6-dev
14:09 [Coke] is there a way to get ll-exception to work with warnings?
14:10 [Coke] if my diagnosis is right (that the warning is coming from the compunit code), then --ll-exception isn't changing the reporting.
14:32 JimmyZ joined #perl6-dev
15:45 nine [Coke]: no, haven't tried it.
16:30 [Coke] nine: doing a special run with a locally modified perl6 to see if I can figure out where it's coming from.
16:32 [Coke] tadzik++ for making GIT_PROTOCOL work everywhere.
16:36 [Coke] nine: do you think it's OK for a non core script to be using the CompUnit::* stuff directly?
16:37 nine [Coke]: it's slowly becoming ok. There's still one open PR that will change interfaces to be much closer to S22.
16:38 nine [Coke]: which parts would you like to use?
16:39 [Coke] ok. Takes a few dozen seconds for me to add debug prints here and narrow it down, but I'm guessing the problem is the run() of perl6 in the precompile().
16:39 [Coke] nine: I'm just looking at https://github.com/perl6/doc/blob/master/htmlify.p6#L160
16:40 nine [Coke]: maybe $source-name?
16:40 nine Ah yes, there's no $source-name: https://github.com/perl6/doc/blob/master/htmlify.p6#L171
16:41 [Coke] yup, https://github.com/rakudo/rakudo/blob/nom/src/core/CompUnit/PrecompilationRepository.pm#L204 is what's warning.
16:41 [Coke] is source-name required?
16:41 [Coke] (but not marked as such?)
16:41 nine Without it backtraces of the precompiled files will be useless but should otherwise work
16:42 [Coke] should it default to "", perhaps? or something like "<unspecified>" ?
16:42 [Coke] I mean, I can just update htmlify.pl to pass it in for now, obv.
16:43 nine $path.Str should be a good default
16:43 nine source-name is really a workaround for the unreadable sha-1 file names. It's basically "$path ($module's-short-name)"
16:47 [Coke] trying a patch with that.
16:48 [Coke] nine++ for finding it in 3 seconds. :)
16:49 nine Well there weren't that many candidates :)
16:50 [Coke] IWBNI this worked in rakudo: GIT_PROTOCOL=ssh make stresstest
16:51 [Coke] (but it's fine that it doesn't)
16:52 [Coke] nine: ok. that fixes htmlify (adding the default to rakudo) - doing a test run to make sure nothing else explodes.
17:01 * [Coke] goes for lunch/walks
18:04 sno joined #perl6-dev
18:42 AlexDani` joined #perl6-dev
18:47 dalek rakudo/nom: 7a98505 | coke++ | / (2 files):
18:47 dalek rakudo/nom: Provide default for precompile()'s 'source-name'
18:47 dalek rakudo/nom:
18:47 dalek rakudo/nom: We use this value in a Str context, so warnings will be emitted. Pick
18:47 dalek rakudo/nom: a sane default (nine++).
18:47 dalek rakudo/nom:
18:47 dalek rakudo/nom: Avoids many warnings for perl6/doc's htmlify
18:47 dalek rakudo/nom: review: https://github.com/rakudo/rakudo/commit/7a98505208
18:47 [Coke] crap. s/will be/would otherwise be/
18:47 [Coke] ah well
18:48 nine [Coke]++ # thanks for fixing my mishaps :)
18:51 kXmjyqYw joined #perl6-dev
18:51 kXmjyqYw left #perl6-dev
19:19 [Coke] is anyone hacking on the html profiler? I'd like to improve the responsiveness.
19:19 [Coke] a 65928890 byte file is really show to load (chrome is actually OH, SNAP dying on it)
19:19 [Coke] just didn't want to step on anyone's toes.
19:22 timotimo the only thing i know about is someone working on making the json dump code faster
19:25 nine It's amazing how modern software stacks can make sheer unlimited computing power unbearingly slow
19:26 [Coke] spare run of htmlify is still unusably slow in the profiler in chrome. ah well.
19:26 [Coke] *sparse
19:27 timotimo yeah, you're right
19:27 timotimo the best way to make it work is still to try to build limiting the depth of the emitted call graph in moar, IMO
19:27 timotimo i mean, i suppose you could do the work in nqp, too. as a post-processing step
19:28 timotimo or even build a program that does that to the already-written .json file
19:48 jdv79 i think its the json parse that's the, at least initial issue.
19:49 jdv79 i almost feel like its november again when i was complaining about the profiler perf.  or around november at least.
20:12 * nine thinks the profiler's UI should be a locally running client/server application with profile data deserialized on-demand and the browser being fed what's necessary for display
20:17 dalek rakudo/nom: b348f2f | coke++ | .travis.yml:
20:17 dalek rakudo/nom: Track channel rename
20:17 dalek rakudo/nom: review: https://github.com/rakudo/rakudo/commit/b348f2f021
20:18 [Coke] maybe that'll get the travis comments to come here.
20:20 [Coke] I see that rakudo-jvm is failing. that a result of my commit, or a holdover?
20:29 Ven joined #perl6-dev
20:57 AlexDani` joined #perl6-dev
21:20 travis-ci joined #perl6-dev
21:20 travis-ci Rakudo build failed. Will "Coke" Coleda 'Track channel rename'
21:20 travis-ci https://travis-ci.org/rakudo/rakudo/builds/132380962 https://github.com/rakudo/rakudo/compare/7a98505208fb...b348f2f02147
21:20 travis-ci left #perl6-dev
22:44 perlpilot_ joined #perl6-dev
22:48 jdv79_ joined #perl6-dev
22:53 eviltwin_b joined #perl6-dev

| Channels | #perl6-dev index | Today | | Search | Google Search | Plain-Text | summary