[00:01:42] *** jamezp_afk has quit IRC
[00:06:02] *** bleathem has joined #seam-dev
[00:06:02] *** ChanServ sets mode: +v bleathem
[00:06:10] *** kevinpollet has quit IRC
[00:07:35] <bleathem> lightguard_jp: ping
[00:10:11] *** jamezp has joined #seam-dev
[00:10:11] *** ChanServ sets mode: +v jamezp
[00:11:24] *** kevinpollet has joined #seam-dev
[00:11:25] *** kevinpollet has quit IRC
[00:11:34] <lightguard_jp> bleathem: pong
[00:11:41] <bleathem> Hi Jason
[00:11:50] <bleathem> looking at your object converter
[00:11:51] <lightguard_jp> bleathem: What's up Brian?
[00:11:53] <lightguard_jp> :)
[00:12:03] <bleathem> neat approach - I'm assuming it actually works?
[00:12:21] <bleathem> was this pulled forward from Seam 2, or is it a new idea?
[00:12:32] <lightguard_jp> It was Stuart's idea.
[00:12:36] <lightguard_jp> I think it works.
[00:12:46] <lightguard_jp> Haven't tested it, but the idea behind it is sound
[00:12:48] <stuartdouglas> it works
[00:12:55] <stuartdouglas> or at least the idea is fine
[00:12:56] <lightguard_jp> There you have it :)
[00:13:01] <stuartdouglas> I have used it in projects before
[00:13:15] <stuartdouglas> I have not actually looked at the impl for this one :-)
[00:13:19] <lightguard_jp> It's been tested in a round about way then :)
[00:15:55] <lightguard_jp> stuartdouglas: https://github.com/seam/faces/pull/60
[00:15:55] <lightguard_jp> Nothing special going on there.
[00:16:50] *** aslak has quit IRC
[00:16:50] *** sbryzak has quit IRC
[00:16:50] *** clerum has quit IRC
[00:18:28] *** aslak has joined #seam-dev
[00:18:28] *** sbryzak has joined #seam-dev
[00:18:28] *** clerum has joined #seam-dev
[00:18:28] *** wolfe.freenode.net sets mode: +ov sbryzak clerum
[00:19:03] *** jamezp is now known as jamezp_afk
[00:19:08] <bleathem> lightguard_jp: how about changing the int to a long?
[00:19:27] <bleathem> and starting it at long minimum?
[00:19:29] *** tsurdilo has quit IRC
[00:19:32] <bleathem> (rather than 0)
[00:19:40] <lightguard_jp> thought about it, but I really didn't think you'd get above Integer.MAX conversions in a conversation
[00:19:43] <stuartdouglas> bleathem: if you have more than int_max items in your conversation you are probably screwed anyway
[00:20:00] <bleathem> ah, didn't realize it resets in a conversation
[00:20:05] <bleathem> makes sense
[00:20:15] <lightguard_jp> It's conversation scoped
[00:20:17] <stuartdouglas> one thing that is wrong is that if getAsString is called twice it will break
[00:20:47] <stuartdouglas> actually it won't break, but you will get a different id each time
[00:20:57] <bleathem> I assume the conversation ahs to be long-running for this to work?
[00:21:05] *** aslak has quit IRC
[00:21:09] <stuartdouglas> which is probably not that much of an issue
[00:21:25] <lightguard_jp> stuartdouglas: You're right, but it probably should be fixed
[00:21:39] <lightguard_jp> bleathem: No, it'll run in a temp conversation if it isn't long running
[00:21:52] <stuartdouglas> actually I don't know, because to fix it you really need another map of a bi-map
[00:21:55] <bleathem> then on post-back, how does it "unconvert"?
[00:22:07] <stuartdouglas> lightguard_jp: if it is not long running it does not work
[00:22:31] <stuartdouglas> as the map does not live long enough to convert back
[00:22:33] <bleathem> should it log a warning or something the, if the conversation is not long-running?
[00:22:42] <bleathem> s/the/then/
[00:22:47] <lightguard_jp> stuartdouglas: You're right. Is that really much of an issue? A doc issue at least.
[00:22:55] <stuartdouglas> its a doc issue
[00:23:12] <lightguard_jp> bleathem: I'll add some docs to the pull request.
[00:23:24] <bleathem> I'd like to see a wrning in the log though
[00:23:39] <bleathem> if you attempt a conversion in a transient conversation
[00:23:41] <lightguard_jp> I could either do another reverse map, or search through things. proc vs memory on that one.
[00:24:38] <lightguard_jp> bleathem: Does guava have something in it's map impls that would help with this?
[00:24:49] <lightguard_jp> Or at least make it a little shorter on the code?
[00:24:52] <bleathem> guava has a bi-map
[00:25:00] <lightguard_jp> Do we have it in Solder?
[00:25:05] <bleathem> dunno
[00:25:13] <bleathem> I imagine so
[00:25:19] <bleathem> it's been in guava for a while
[00:25:33] <lightguard_jp> Okay, I'll update the docs, add a warning and take a look at the bi-map.
[00:25:41] <bleathem> great stuff - thanks!
[00:26:32] <bleathem> I suppose it's possible to make the conversation long-running after the converter is called
[00:27:01] <lightguard_jp> huh, no bi map in solder
[00:28:48] <lightguard_jp> Okay, no biggy, just two maps
[00:30:34] <bleathem> lightguard_jp: what's the status of JSFUnit testing in Faces?
[00:30:58] <lightguard_jp> bleathem: Haven't touched it since the latest testing structure change
[00:31:04] <bleathem> ok
[00:31:12] * bleathem goin' offline for a bit
[00:31:19] *** bleathem has quit IRC
[00:34:47] *** tsurdilo has joined #seam-dev
[00:43:57] *** jamezp_afk is now known as jamezp
[01:07:52] *** rruss has joined #seam-dev
[01:12:25] *** hannelita has quit IRC
[01:13:13] *** edburns is now known as edburns_away
[01:50:52] *** lightguard_jp has quit IRC
[01:53:20] <Diablo-D3> https://github.com/blog/967-github-secrets
[01:58:30] *** sannegrinovero has quit IRC
[02:14:26] *** rruss has quit IRC
[02:22:58] *** jamezp is now known as jamezp_afk
[02:38:12] *** bleathem has joined #seam-dev
[02:38:12] *** ChanServ sets mode: +v bleathem
[03:18:52] *** akazakov has quit IRC
[03:56:42] *** edburns_ has joined #seam-dev
[03:56:43] *** edburns_away has quit IRC
[03:58:59] *** sgilda has quit IRC
[04:09:34] *** edburns_ is now known as edburns
[04:09:56] <Diablo-D3> http://rss.slashdot.org/~r/Slashdot/slashdot/~3/oC4URIr2-eU/new-jboss-worm-infecting-unpatched-servers
[04:10:00] <Diablo-D3> well that isnt fun.
[05:50:04] *** edburns has quit IRC
[05:50:32] *** edburns has joined #seam-dev
[05:52:18] *** tsurdilo has quit IRC
[06:06:12] *** tsurdilo has joined #seam-dev
[06:23:23] *** tsurdilo has quit IRC
[07:02:40] *** edburns_ has joined #seam-dev
[07:02:40] *** edburns has quit IRC
[07:52:01] *** mehdi_ has quit IRC
[08:20:20] *** omid has joined #seam-dev
[09:06:06] *** clerum has quit IRC
[09:24:03] <lincolnthree1> Diablo-D3: only affects JBoss 5 and older.
[09:24:13] <lincolnthree1> Not 6 or 7
[09:24:15] *** lincolnthree1 has left #seam-dev
[09:59:10] *** omid has left #seam-dev
[12:02:53] *** alesj has quit IRC
[14:15:36] *** sgilda has joined #seam-dev
[14:34:07] *** ga_br has joined #seam-dev
[15:54:54] *** ga_br has quit IRC
[16:09:18] *** edburns__ has joined #seam-dev
[16:09:18] *** edburns_ has quit IRC
[17:41:53] *** aslak has joined #seam-dev
[17:54:51] *** alesj has joined #seam-dev
[18:51:59] *** tsurdilo has joined #seam-dev
[19:30:06] *** jamezp_afk is now known as jamezp
[19:32:01] *** jamezp is now known as jamezp_afk
[19:41:01] *** rruss has joined #seam-dev
[19:41:40] *** rruss has quit IRC
[19:52:35] *** tsurdilo has quit IRC
[20:43:56] *** aslak has quit IRC
[20:52:47] *** aslak has joined #seam-dev
[21:28:16] *** rruss has joined #seam-dev
[21:54:36] *** clerum has joined #seam-dev
[21:54:36] *** ChanServ sets mode: +v clerum
[22:04:10] *** bleathem has quit IRC
[22:32:00] *** rruss has quit IRC
[22:32:24] *** rruss has joined #seam-dev
[22:44:23] *** alesj has quit IRC
[22:45:53] *** rruss has quit IRC
[22:53:30] *** aslak has quit IRC
[22:55:38] *** alesj has joined #seam-dev
[23:22:15] *** alesj has quit IRC
[23:42:35] *** alesj has joined #seam-dev





top