irclog.perlgeek.de/perl6book/today | source: github.com/perl6/book/
Set by moderator on 24 March 2010.
01:08 sorear joined 02:03 sorear joined 08:25 eternaleye joined 09:50 hicx174 joined 09:56 smash_ joined
smash_ hello 09:57
moritz_ hi
smash_ i was reading chapter 5 in the book
and there's this example that it's working quite well for me 09:58
(i'm using master trunk)
moritz_ which one is chapter 5? regexes?
smash_ the one in the Constrains subsection
sorear ponders porting Pod::PseudoPod::LaTeX to POD6 09:59
10:00 masak joined
smash_ moritz_: gist.github.com/376404 10:00
moritz_ smash_: what's your problem? that it says 2 instead 1? 10:02
smash_ correct 10:03
innocently i was expecting 1, but i could be wrong
moritz_ well, it's not really wrong, but it points to a possible rakudo regression 10:04
smash_ it's more a philosofical thing i guess 10:05
why he is checking the constrain when there's a "better" match 10:06
moritz_ huh?
he isn't
that's why you get a 0 in the first place
smash_ ah,, true.. sorry, my bad
then i will expect it to say 1 yes
moritz_ it was down to 1 at some point 10:08
the multi dispatcher checked the constraint
and then told the binder not to re-check
smash_ ok, i saw your message in #perl6 for jnthn
moritz_ but the spec doesn't say anywhere that the minimum number of checks has to performed
smash_ hmm, then i guess that should be indicated in that section of the book 10:09
moritz_ aye
smash_ otherwise people might get confused, it's ver easy to expect that the constrains will only run once 10:10
ok, i'll add something there poiting that issue
using that exact example
moritz_ not necessary, I'll add an explanation right now (or after lunch)
smash_ i can add it and you can review it, it's fine with me :) 10:11
or you can add it if you prefer 10:13
either way is fine by fine
s/fine by fine/fine by me/ 10:14
moritz_ pushed.
dalek ok: bfe7d2a | moritz++ | src/multi-dispatch.pod:
[mmd] clarify number of executions of a where-block, smash++
10:15
smash_ moritz_++ 10:17
pod2latex conversion missing the '=begin/=end screen' ? 11:38
smash_ afk, lunch 11:42
moritz_ smash_: might well be :( 11:44
12:02 sundar joined
dalek ok: 6fabc2f | masak++ | src/preface.pod:
[preface] fixed grammaro
12:26
ok: b41705f | masak++ | src/preface.pod:
[preface] slight rephrasings
12:32
ok: 23285cb | masak++ | src/preface.pod:
[preface] various further small file

  - removed trailing slash in perl.org/
  - 'the compiler' sounds strange after explaining that there are several
   Perl 6 implementations. changed to 'Perl 6 compilers'
12:44
smash_ is back 12:51
13:02 cognominal joined
moritz_ PerlJam, masak: if you haven't done already, please sign the book contract and send it back to chromatic++ 13:09
masak did that ages ago :)
moritz_ masak++ 13:10
dalek ok: e325b80 | masak++ | src/basics.pod:
[basics] rw review
13:25
smash_ where can i send a patch for perl6/book ? 14:30
moritz_ smash_: uhm, commit it directly
smash_: or is it something you'd like to discuss first? 14:31
smash_ although it's some simple stuff i'm not sure i feel confortable just push'ing it
masak please do feel comfortable. :)
moritz_ smash_: we'll have a close look and discuss and/or revert the parts that are not OK - that's normal development strategy here 14:32
"better to ask for forgiveness than permission"
smash_ nods. 14:33
smash_ push'ed, let me know it you don't agree with anything and i will gladly revert it 14:40
dalek ok: da67c46 | smash++ | src/ (4 files):
Several small fixes.
moritz_ smash_++ 14:41
there's one thing I don't like, and one that I'm not sure - the rest is great 14:42
- say $counter;
+ say $counter; # says B<2>
the output is implementation specific
and can actually differ from version to version
so that should be made clear from the comment 14:43
smash_ yeap, i agree on that one.. something should mention there that that output is implementation specific
moritz_ second thing is: before you commit markup fixes, please check if they acctually improve parsing 14:44
Pod::PseudoPod has some weird charactersitics 14:45
in particular I've been wondering about 14:46
smash_ i tested every markup change
moritz_ smash_++
in particular I wondered about the closing of the N<...>
smash_ unless the module behaves diferently across different environments 14:47
moritz_ don't think it does 14:48
smash_ nice
moritz_ actually it's fine here... I just thought I'd fixed that one a while back, and it was a very counter-intuitive fix
but I mis-remembed
*remembered
dalek ok: 24ca22d | moritz++ | src/multi-dispatch.pod:
[mmd] now that we have Real, there is no need for separate Num and Int candidates
14:52
smash_ moritz_: (regarding you first comment) gist.github.com/376638 15:14
moritz_ smash_: I like it... feel free to revert the commit I just pushed, because they collide 15:16
smash_: but I like yours better
dalek ok: c8aefbb | moritz++ | src/operators.pod:
[ops] working code, screen output
ok: 39769b6 | moritz++ | src/multi-dispatch.pod:
[mmd] number of type checks is implementation dependent
smash_ ok, let me do that 15:19
moritz_: push'ed, see what you think of it now, and if it still needs more tweaks 15:26
moritz_ pulls 15:27
smash_++ # nice 15:28
dalek ok: 62090c3 | smash++ | src/multi-dispatch.pod:
[mmd] subtype checks blocks number of executed times is implementation specific
smash_ moritz_: anything more from the previous patch ? 15:38
(i mean the original one)
moritz_ I'm fine with the current state of affairs 15:39
smash_ ok, good 15:40
and thks
moritz_ thank you for the contributions
dalek ok: 470ed50 | masak++ | src/operators.pod:
[operators] rw review
16:57
17:38 japhb joined