These are chat archives for opal/opal

3rd
Jul 2015
Jikku Jose
@JikkuJose
Jul 03 2015 02:45 UTC
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 UTC
@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 UTC
@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 UTC
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 UTC
\o/
Ryan Stout
@ryanstout
Jul 03 2015 22:27 UTC
@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 UTC
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 UTC
yea
I'm not sure it will ever happen :-)
Elia Schito
@elia
Jul 03 2015 22:34 UTC
eheheh
Ryan Stout
@ryanstout
Jul 03 2015 22:34 UTC
if I had more time I would build a replacement :-)
@vais you around by chance?
Elia Schito
@elia
Jul 03 2015 22:35 UTC
I think he's asleep
Ryan Stout
@ryanstout
Jul 03 2015 22:36 UTC
ok, no rush
trying to understand the split method.
on string
Ryan Stout
@ryanstout
Jul 03 2015 22:46 UTC
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 UTC
yeah, it's a bliss
Ryan Stout
@ryanstout
Jul 03 2015 22:47 UTC
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 UTC
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 UTC
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 UTC
indeed
Elia Schito
@elia
Jul 03 2015 23:05 UTC
@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 UTC
@elia sure, does opal use a fork of rubyspec
Elia Schito
@elia
Jul 03 2015 23:06 UTC
no, it's on ruby/rubyspec thanks to… guess who?
@vais of course :tophat:
Ryan Stout
@ryanstout
Jul 03 2015 23:06 UTC
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 UTC
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 UTC
I'll add it
@ryanstout am I doing something wrong?
>> '/'.split('?', 2)
=> ["/"]
Ryan Stout
@ryanstout
Jul 03 2015 23:16 UTC
I get ["/", ""]
Elia Schito
@elia
Jul 03 2015 23:16 UTC
I see, I'll update the issue then
Ryan Stout
@ryanstout
Jul 03 2015 23:16 UTC
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 UTC
yep
Ryan Stout
@ryanstout
Jul 03 2015 23:18 UTC
running master now
can you duplicate?
Elia Schito
@elia
Jul 03 2015 23:19 UTC
yes, also this is curious (opal)
>> "xy".split("z", 3)
=> ["xy", ""]
Ryan Stout
@ryanstout
Jul 03 2015 23:20 UTC
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 UTC
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 UTC
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 UTC