tbrowder AlexDaniel: I looked at one of the diffs that shows misalignment on github but i don’t see it on my local host. Let me get the exact line number of the file src/NQP/Actions.nqp... 01:09
line 257... 01:11
AlexDaniel ok that one has two tabs 01:12
where I can see the file after your changes?
ah it's still there on github
yea, you replaced two tabs with 8 spaces 01:13
tbrowder but the diff on github doesn’t look right to me...
AlexDaniel when it needed 16 spaces
it's indeed not right
it's not just the diff, it's the unmodified patch also: patch-diff.githubusercontent.com/r.../403.patch 01:14
how many spaces do you see?
(not just the visual html diff I meant, surely) 01:15
tbrowder i count 16 spaces
AlexDaniel how is that possible? :)
weird
geekosaur old is two tabs 01:16
new is 8 spaces
at a guess, the original came from an editor that uses 4-space tabs 01:17
tbrowder beats me...maybe here in niceville it’s lala land!
AlexDaniel
.oO( how many people does it take to replace one tab with spaces? )
geekosaur or possibly passed through one
(this is why tabs are a bad idea in programming: there's no guarantee what you will get. editors, compilers, etc. all have different ideas of what a tab is)
AlexDaniel geekosaur: why 4…
it needs 16 spaces, the editor was assuming 8 per tab 01:18
tbrowder i think i’m wasting my time here...i’ll keep the pr there, there, though, just in case someone can explain what’s going on.
geekosaur because the editor doesn't have smart indentation and expects the user to use tabs, and 8-space tabs lead to too much indentation
AlexDaniel but the editor was emacs 01:19
which by default does mixed tabs and spaces…
where each tab is 8 spaces…
so it goes
tbrowder btw, the last time i looked at the particular file i was using vi which clearly shows tabs if they exist.
AlexDaniel 4 spaces, 1 tab, 1 tab 4 spaces, 2 tabs…
tbrowder no, you may remember i found my .emacs file was in error, ani fixed it so i reliably get 4 spaces per tab key. 01:21
geekosaur notices the most recnet comment explicitly says convert tabs to *4* spaces
tbrowder anyhoo, vi foes show 16 spaces
AlexDaniel I was talking about that line before your changes :)
geekosaur but browsers render them as 8
tbrowder *shows
geekosaur unix browsers at least
tbrowder oh... 01:22
geekosaur so anyway this looks self-inflicted.
8 is generally most common on unixlikes
tbrowder i get 12 spaces there 01:23
duh, you mean *before* my changes...it’s almost my bedtime... 01:24
i put my money on a github diff problem 01:27
geekosaur like I said, the raw diff was two tabs before, 8 spaces after. unix browsers render the two tabs as 16 spaces
AlexDaniel and the correct indentation on that line is 16 spaces 01:38
not sure what the rest of the discussion was for when I said this straight away: 01:40
<AlexDaniel> yea, you replaced two tabs with 8 spaces
<AlexDaniel> when it needed 16 spaces
squashable6: next 02:09
squashable6 AlexDaniel, ⚠🍕 Next SQUASHathon in 2 days and ≈7 hours (2018-02-03 UTC-12⌁UTC+14). See github.com/rakudo/rakudo/wiki/Mont...Squash-Day
geekosaur and that was more or less why I jumped in, because there's lots of misunderstandings and differences in what various people *and* programs mean by tab 02:11
which is why they're a bad idea in general for programming
tbrowder AlexDaniel: how should i proceed on the detabify? 02:21
AlexDaniel tbrowder: given the issues, maybe not? 02:22
not a big deal anyway
POD stuff is definitely more important than tabs in the source code :)
tbrowder i agree, putting it to bed...g’night all
Geth rakudo: 671c3d6016 | (Stefan Seifert)++ | 5 files
Initialize $*PERL during setting compilation to avoid runtime cost

All information provided by $*PERL (most notably rakudo and Perl 6 version) can be determined when compiling the setting. By creating the Perl object at BEGIN time, we can safe the runtime overhead (most notably when loading the first module). However at BEGIN time we cannot use most of the string processing code like regex matching, so we need to parse the versions using specialized nqp code.		 08:15
nine This was....so much harder than expected
Geth rakudo: c820e4be17 | (Stefan Seifert)++ | src/core/Backtrace.pm
Avoid error when throwing error during setting compilation

The unary + operator may not yet be set up correctly, causing a further error when trying to report an error.		 08:16
rakudo: 00cde17573 | (Stefan Seifert)++ | src/core/CompUnit/PrecompilationUnit.pm
Micro optimize CompUnit::PrecompilationUnit::deserialize

This avoids costly Str.split and List.STORE. I hope the code is still somewhat readable despite losing the variable names. The 4 parts are used for initializing named arguments right after anyway.		 08:19
TimToady jnthn: an update, the leak appears to be from $buf.subbuf(0,$bytes).decode or so; at least, when I decode the buf directly to latin-1 and use substr on that that, it seems to be much more stable so far, but I'm gonna run it overnight and see how it is by morning... 09:13
Geth rakudo: ed9b287c99 | (Stefan Seifert)++ | src/core/Stash.pm
Make Stash.merge-symbols a bit faster and the code more clear
AlexDaniel squashable6: next 09:30
squashable6 AlexDaniel, ⚠🍕 Next SQUASHathon in 2 days and ≈0 hours (2018-02-03 UTC-12⌁UTC+14). See github.com/rakudo/rakudo/wiki/Mont...Squash-Day
jnthn morning o/ 10:02
travis-ci Rakudo build passed. Stefan Seifert 'Make Stash.merge-symbols a bit faster and the code more clear' 10:07
travis-ci.org/rakudo/rakudo/builds/335531694 github.com/rakudo/rakudo/compare/0...9b287c9910
AlexDaniel greppable6: I have no idea 11:26
greppable6 AlexDaniel, 8 lines, 8 modules: gist.github.com/8e313c5e424c24b6a2...49702d3d6c
AlexDaniel timotimo: ↑ :) 11:27
no inline summary for treegrep though 11:34
because, well… I don't see anybody using that feature :)
treegrep: awesome
greppable6 AlexDaniel, gist.github.com/abf0fce55a3821af58...29c436ac9c
AlexDaniel well… that was less than awesome :)
greppable6: oenthunteohunteou
greppable6 AlexDaniel, Found nothing!
AlexDaniel treegrep: bad 11:35
greppable6 AlexDaniel, gist.github.com/dc50016c0b559050c4...e767295532
lizmat flying& 11:56
|Tux| Rakudo version 2018.01-65-ged9b287c9 - MoarVM version 2018.01-49-g783a4f07c
csv-ip5xs0.937 - 0.944
csv-ip5xs-2010.297 - 10.542
csv-parser11.053 - 11.311
csv-test-xs-200.454 - 0.527
test9.039 - 9.098
test-t2.565 - 2.576
test-t --race1.039 - 1.098
test-t-2044.682 - 45.381
test-t-20 --race15.316 - 15.662
 13:43
Geth nqp: 5f7fc969ce | pmurias++ | 3 files
[js] Decoding as utf8-c8
 13:49
nqp: 4bffb46af3 | pmurias++ | src/vm/js/nqp-runtime/codecs.js
[js] Implement encoding as utf8-c8
nqp: 8aea6d7d68 | pmurias++ | t/nqp/082-decode.t
Test encoding and decoding as utf8-c8
travis-ci NQP build failed. pmurias 'Test encoding and decoding as utf8-c8' 14:07
travis-ci.org/perl6/nqp/builds/335628781 github.com/perl6/nqp/compare/c1420...ea6d7d687f
stmuk pl6anet.org/drop/rakudo-star-2018.01.RC0.dmg 14:14
pmurias (the nqp build fails seems to be because of a concurrency test not a result of js work) 14:36
Geth rakudo: 72948e8477 | (Zoffix Znet)++ (committed using GitHub Web editor) | src/Perl6/Actions.nqp
Fix typo in comment
 16:09
rpg: 8c46d32edc | (Zoffix Znet)++ (committed using GitHub Web editor) | README.md
Stub repo description

since people are asking...		 18:54
AlexDaniel haha 18:57
Zoffix++
DrForr We support military hardware now? 19:50
sjn Rakudo Propelled Grenades! 19:52
AlexDaniel FWIW I tweaked evalable6 a little bit, there's a chance that it will misbehave 20:03
if so let me know
but it should be better 20:04
squashable6: next 20:39
squashable6 AlexDaniel, ⚠🍕 Next SQUASHathon in 1 day and ≈13 hours (2018-02-03 UTC-12⌁UTC+14). See github.com/rakudo/rakudo/wiki/Mont...Squash-Day

Powered by Cro and the Raku® Programming Language.

Please report any issues / comments / feature requests as an issue on App::Raku::Log.

Thank you!