Skip to end of metadata
Go to start of metadata

You are viewing an old version of this page. View the current version.

Compare with Current View Page History

« Previous Version 16 Next »

[01:00:46 CDT(-0500)] * ajan (~ajan@89.205.17.150) has joined ##uportal
[08:09:10 CDT(-0500)] * jessm (~Jess@c-68-40-84-77.hsd1.mi.comcast.net) has joined ##uportal
[08:12:21 CDT(-0500)] * colinclark (~colin@74.198.28.32) has joined ##uportal
[08:37:47 CDT(-0500)] * colinclark (~colin@bas2-clarkson16-1176020144.dsl.bell.ca) has joined ##uportal
[09:17:08 CDT(-0500)] * lfuller (~sparhk@wsip-72-215-204-133.ph.ph.cox.net) has joined ##uportal
[09:22:39 CDT(-0500)] * EricDalquist (~dalquist@2607:f388:e:0:221:9bff:fe37:e768) has joined ##uportal
[09:24:21 CDT(-0500)] * JoeMoore (89d880aa@gateway/web/freenode/ip.137.216.128.170) has joined ##uportal
[09:55:54 CDT(-0500)] * holdorph (~holdorph@wsip-72-215-204-133.ph.ph.cox.net) has joined ##uportal
[09:56:12 CDT(-0500)] * bsparks (~bsparks@wsip-72-215-204-133.ph.ph.cox.net) has joined ##uportal
[10:33:27 CDT(-0500)] * athena (~athena@email.cupcakeroyale.com) has joined ##uportal
[10:44:33 CDT(-0500)] <athena> hey EricDalquist - how's crucible looking so far?
[10:44:40 CDT(-0500)] <EricDalquist> neat
[10:44:43 CDT(-0500)] <EricDalquist> did you get the review email?
[11:05:08 CDT(-0500)] <athena> yeah
[11:05:14 CDT(-0500)] <athena> i wasn't sure what one of them actually meant
[11:05:16 CDT(-0500)] <athena> but i got it (smile)
[11:05:23 CDT(-0500)] <EricDalquist> so I added a few comments to the commit
[11:05:26 CDT(-0500)] <athena> would be great to have that set up - i'd love the feedback
[11:05:34 CDT(-0500)] <EricDalquist> I think you can go and respond to them or something
[11:05:40 CDT(-0500)] <athena> oh neat
[11:05:55 CDT(-0500)] <athena> i saw your comments, but i was reading them from the bus stop outside the basketball arena (tongue)
[11:05:59 CDT(-0500)] <EricDalquist> lol
[11:06:31 CDT(-0500)] <athena> yeah
[11:06:34 CDT(-0500)] <athena> great game though
[11:07:00 CDT(-0500)] <athena> watching the WNBA team here - seattle won after coming from way-behind to maintain their perfect home record
[11:07:01 CDT(-0500)] <athena> great time
[11:07:09 CDT(-0500)] <EricDalquist> nice
[11:07:13 CDT(-0500)] <athena> ya
[11:07:19 CDT(-0500)] <athena> so this is the comment i had a question about: "You should be able to do m.group(1); and get just the contents of the matching group in the expression. Then there won't be a need for the additional substring"
[11:07:29 CDT(-0500)] <EricDalquist> ah right
[11:07:34 CDT(-0500)] <athena> does matcher.group(number) return something different than matcher.group()?
[11:07:37 CDT(-0500)] <EricDalquist> yes
[11:07:44 CDT(-0500)] <EricDalquist> group() returns the whole match
[11:07:53 CDT(-0500)] <EricDalquist> the regex has a group defined in it though
[11:07:57 CDT(-0500)] <EricDalquist> the part wrapped in ()
[11:08:09 CDT(-0500)] <EricDalquist> the defined groups are indexed starting at 1
[11:08:12 CDT(-0500)] <athena> oh!
[11:08:14 CDT(-0500)] <athena> i see
[11:08:18 CDT(-0500)] <EricDalquist> so if you want the contents of the first () you do group(1)
[11:08:31 CDT(-0500)] <athena> yep, makes sense
[11:08:32 CDT(-0500)] <athena> will fix

  • No labels