These are chat archives for opal/opal

3rd
Jul 2015
Jikku Jose
@JikkuJose
Jul 03 2015 02:45
In case someone is interested: I was able to solve the problem of not being able to assign class through initialize (http://tinyurl.com/opal-box-class) by creating a class method to call initialize: http://git.io/vqJBs
Rick Carlino
@RickCarlino
Jul 03 2015 02:48
@JikkuJose Neat find. I wonder why it doesn't work with initialize? Did you try calling super?
eh, actually, Im not sure
Jikku Jose
@JikkuJose
Jul 03 2015 05:23
@RickCarlino yes tried that. See the first example, I was assuming it will work just like in Ruby but somehow the class didn't get added.
Ryan Stout
@ryanstout
Jul 03 2015 22:17
if any opal core people are around, I think I found a regression in 0.8 (in process of upgrading volt): opal/opal#973
I'm wondering if there's an easy fix, it used to work
I'm working on moving volt over to 0.8 finally :-)
Elia Schito
@elia
Jul 03 2015 22:21
\o/
Ryan Stout
@ryanstout
Jul 03 2015 22:27
@elia have you been following sprockets stuff at all? I'm curious what their ETA is for 4.0
Elia Schito
@elia
Jul 03 2015 22:28
I asked a couple of times but the answer was either not in the near future or no answer at all
maybe it's time to ask @josh or @rafaelfranca again
Ryan Stout
@ryanstout
Jul 03 2015 22:34
yea
I'm not sure it will ever happen :-)
Elia Schito
@elia
Jul 03 2015 22:34
eheheh
Ryan Stout
@ryanstout
Jul 03 2015 22:34
if I had more time I would build a replacement :-)
@vais you around by chance?
Elia Schito
@elia
Jul 03 2015 22:35
I think he's asleep
Ryan Stout
@ryanstout
Jul 03 2015 22:36
ok, no rush
trying to understand the split method.
on string
Ryan Stout
@ryanstout
Jul 03 2015 22:46
anyway, other than the string split issue, it looks like everything with the 0.8 upgrade works :-)
really excited to have sourcemaps back
Elia Schito
@elia
Jul 03 2015 22:46
yeah, it's a bliss
Ryan Stout
@ryanstout
Jul 03 2015 22:47
it adds a few seconds to my page load time, so I think I'll make it disabled by default
but I can take a few seconds when I need it
Elia Schito
@elia
Jul 03 2015 22:48
I think disabling them for all the black box code is enough
most app code is not that big
things changed for me when I collapsed opal.js in one file without sourcemaps
Ryan Stout
@ryanstout
Jul 03 2015 22:49
interesting. Yea, I might try that.
if I collapse volt and opal, that would lower the number of files quite a bit
Elia Schito
@elia
Jul 03 2015 22:50
indeed
Elia Schito
@elia
Jul 03 2015 23:05
@ryanstout can you do a quick search in rubyspec for that particular bug? that would help clarifying the behavior a lot and if it's missing it'd be an improvement for rubyspec :)
Ryan Stout
@ryanstout
Jul 03 2015 23:05
@elia sure, does opal use a fork of rubyspec
Elia Schito
@elia
Jul 03 2015 23:06
no, it's on ruby/rubyspec thanks to… guess who?
@vais of course :tophat:
Ryan Stout
@ryanstout
Jul 03 2015 23:06
nice
checking for it
so I didn't search everything, but it looks like they are missing a spec for if there is no match and a limit
Ryan Stout
@ryanstout
Jul 03 2015 23:12
yea, I'm pretty sure its missing it. I went through all of the specs with .split in it
Elia Schito
@elia
Jul 03 2015 23:12
I'll add it
@ryanstout am I doing something wrong?
>> '/'.split('?', 2)
=> ["/"]
Ryan Stout
@ryanstout
Jul 03 2015 23:16
I get ["/", ""]
Elia Schito
@elia
Jul 03 2015 23:16
I see, I'll update the issue then
Ryan Stout
@ryanstout
Jul 03 2015 23:16
let me clear my changes real quick
sorry, one sec
sorry, yea, I put it wrong in the issue
let me update
ok, sorry, I updated it
was doing multiple assignment, so I got confused
still broke though
Elia Schito
@elia
Jul 03 2015 23:18
yep
Ryan Stout
@ryanstout
Jul 03 2015 23:18
running master now
can you duplicate?
Elia Schito
@elia
Jul 03 2015 23:19
yes, also this is curious (opal)
>> "xy".split("z", 3)
=> ["xy", ""]
Ryan Stout
@ryanstout
Jul 03 2015 23:20
yea, I ran into that one also
the problem is I don't really understand the code
something like this after match = pattern.exec(string); fixes my issue, but I have the feeling there's code below that was supposed to do that and just isn't working:
        if (match === null && limit > 0) {
          return result;
        }
like I have the feeling the if (limit < 0) { is broke (lt zero)
but maybe you can do a negative index
ok, yea you can
Ryan Stout
@ryanstout
Jul 03 2015 23:28
ok, I think I'll just wait until @vais comes back online :-)
@elia what's left before 0.8 gets released officially?
Elia Schito
@elia
Jul 03 2015 23:29
I guess the main problem was waiting for the dust to settle, then if I'm not mistaken it just depends on me
I think I'll push another rc with this fix and one for require_tree
master has already moved on with the new .JS syntax, it's really time to release
I hoped for a blog post explaining the changes but not really in the mood for it, let alone having the time
Elia Schito
@elia
Jul 03 2015 23:42