|
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
|
|||