dalek arVM/no-manual-symlinks: 4bf289a | (Rob Hoelz)++ | src/io/fileops.c:
Don't resolve symlinks by hand

Let the operating system resolve them for us, because it knows best.
03:30
arVM/no-manual-symlinks: d3559f9 | (Rob Hoelz)++ | src/io/fileops. (2 files):
Remove MVM_file_stat_follow_symlink

It's no longer used
hoelzro o/ #moarvm 15:15
brrt \o hoelzro 15:29
hoelzro o/ brrt
brrt off-topic: recall the statement that 'luajit useth no pointers' in their IR?
i recall vaguely writing a html-recompiler once (in perl) that used a very similar strategy 15:30
to representing the tree
the entire tree was contained in a single array, and a 'open' tag contained the offset of it's corresponding 'close' tag
of course, the close tag wasn't strictly necessary, but they were used as triggers 15:31
jnthn Sounds neat. I'm sure I used a similar trick somewhere once too...forget where exactly. 15:33
brrt not all roses and sunshine. cutting out a part of the tree was... tricky 15:47
but yeah, it can be done :-) 15:48
hoelzro has anyone had a chance to look at my removal of MVM_file_stat_follow_symlink changes? 16:05
JimmyZ_ hoelzro: looks good 16:09
hoelzro I'm just hoping I'm not missing anything with the removal of that manual resolution logic
I think FROGGS wrote it?
JimmyZ_: thanks for looking! 16:10
JimmyZ_ yeah
jnthn hoelzro: Does it change the current behavior (in so far as, things work the same aside from the potential infinite loop bug that should now be gone)?
hoelzro jnthn: it doesn't change the current behavior
jnthn OK 16:12
JimmyZ_ hoelzro: +1 to merge, as long as it doesn't break roast :) 16:13
hoelzro I roasted last night; no beans were burnt =)
jnthn: regarding that exception thrown from callbacks issue I reported earlier this week, what would it take to catch the exception from the nested interpreter call and re-throw it up to the nesting interpreter? 16:30
dalek_p6c arVM/no-manual-symlinks: 4bf289a | hoelzro++ | src/io/fileops.c:
Don't resolve symlinks by hand

Let the operating system resolve them for us, because it knows best.
20:24
arVM/no-manual-symlinks: d3559f9 | hoelzro++ | src/io/fileops. (2 files):
Remove MVM_file_stat_follow_symlink

It's no longer used
jnthn That's a re-deliver of an earlier one, just to check I got it right :)
hoelzro ok, I was confused =) 20:25
jnthn :)
New dalek :)
TimToady is it going to stay named that? 'cuz my highlighter picks the p6 over the dalek... :/ 20:26
moritz TimToady: nope, I plan to rename it back to dalek once the feather3 instance has been shut off 20:27
TimToady kay
rurban who about lstat in addition to stat? 22:42