Where the hell are you guys?

Sent from my iPhone

> On Apr 1, 2015, at 9:33 PM, MaxSpiegs <notifications@github.com> wrote:
>
> Sending out this email cause I know you guys have great photos. Anyone have
> a pic of fleck making a rather interesting face? Preferably one with rather
> squinted eyes? Thanks!
>
> On Tue, Mar 31, 2015 at 7:02 PM, johndunaske <notifications@github.com>
> wrote:
>
> > I bought it
> >
> > Sent from my iPhone
> >
> >
> > > On Mar 31, 2015, at 6:42 PM, MergeyMergeMerge <notifications@github.com>
> > wrote:
> > >
> > > Also, spread the link to anyone who wants to play gmod with us
> > >
> > >
> > >
> > >
> > >
> > >
> > > Sent from Windows Mail
> > >
> > >
> > >
> > >
> > >
> > > From: Ayyyyyyyy lmao
> > > Sent: Tuesday, March 31, 2015 6:36 PM
> > > To: Team1559/RecycleRush2015
> > >
> > >
> > >
> > >
> > > Sure I guess idk
> > > On Mar 31, 2015 6:34 PM, "johndunaske" <notifications@github.com>
> > wrote:
> > >
> > > > And about Gary's mod?
> > > >
> > > > Sent from my iPhone
> > > >
> > > > > On Mar 31, 2015, at 6:31 PM, MaxSpiegs <notifications@github.com>
> > wrote:
> > > > >
> > > > > Better yet, get "8 Crazy Nights". Adam Sandler, Deers eating
> > "droppings"
> > > > > and with the MOST ANNOYING VOICE a character can ever have
> > > > > On Mar 30, 2015 8:55 PM, "Cody Burrows" <notifications@github.com>
> > > > wrote:
> > > > >
> > > > > > I can't edit the code right now which makes me sad.
> > > > > > Pete:
> > > > > > Sudo check your email so you can edit it for me!!!
> > > > > >
> > > > > > —
> > > > > > Reply to this email directly or view it on GitHub
> > > > > > <
> > > >
> > https://github.com/Team1559/RecycleRush2015/commit/e988916aa0486744c6b367f2f78142639b2b0c67#commitcomment-10479823
> > > > >
> > > > > > .
> > > > > >
> > > > > —
> > > > > Reply to this email directly or view it on GitHub.
> > > > >
> > > >
> > > > —
> > > > Reply to this email directly or view it on GitHub
> > > > <
> > https://github.com/Team1559/RecycleRush2015/commit/e988916aa0486744c6b367f2f78142639b2b0c67#commitcomment-10507032>
> >
> > > > .
> > > >
> > >
> > > —
> > > Reply to this email directly or view it on GitHub.
> > > —
> > > Reply to this email directly or view it on GitHub.
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <https://github.com/Team1559/RecycleRush2015/commit/e988916aa0486744c6b367f2f78142639b2b0c67#commitcomment-10507367>
> > .
> >
> —
> Reply to this email directly or view it on GitHub.
>


We shouldn't render a page that isn't SSL if force_ssl is enabled.


Você deveria trocar o nome do pagamento de "Gratuito" para "Mensal", já que você está selecionando a opção de "Assinatura mensal".


This may be a little more concise as:

```rust
match match_digits(s, 2, 2, false) {
    Some(h) => hours = h,
    None => failed = true,
}
```


Also, could the `failed` local variable be omitted for just `return Err(ParseError::InvalidZoneOffset)`?


This could be just `let _ = parse_char(s, ':');`


Just a note, if you truly want for this to be an s2-common repository, then this overlay should go in the i9100/n7000 device repository. The SGH-I777 & SPH-D710 & SGH-S959G, which are also members of the smdk4210 family have a different key layout (+ assistant/search button, - physical home button) so these values differ per device.  Admittedly, there are FAR fewer people building for these other devices.
```xml
<!-- For SGH-I777 & SPH-D710 & SGH-S959G... -->
<integer name="config_deviceHardwareKeys">79</integer>
<integer name="config_deviceHardwareWakeKeys">64</integer>
```


**[unitTests_js_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_js_branches/10735/testReport/) for this commit**


Hmm, I think part of what's going on here is that we're not using our state monad in a very state-monad-like way.  Instead we're using it like a Reader monad together with `local` function to modify the environment within a local scope.  That is, changes to the environment never persist after a recursive call ends.  I wonder if we could actually change to using `ReaderT` + `local` (and whether it would help).


We're at bettas Italian food.

On Fri, Apr 3, 2015, 11:41 AM johndunaske <notifications@github.com> wrote:

> Where the hell are you guys?
>
> Sent from my iPhone
>
> > On Apr 1, 2015, at 9:33 PM, MaxSpiegs <notifications@github.com> wrote:
> >
> > Sending out this email cause I know you guys have great photos. Anyone
> have
> > a pic of fleck making a rather interesting face? Preferably one with
> rather
> > squinted eyes? Thanks!
> >
> > On Tue, Mar 31, 2015 at 7:02 PM, johndunaske <notifications@github.com>
> > wrote:
> >
> > > I bought it
> > >
> > > Sent from my iPhone
> > >
> > >
> > > > On Mar 31, 2015, at 6:42 PM, MergeyMergeMerge <
> notifications@github.com>
> > > wrote:
> > > >
> > > > Also, spread the link to anyone who wants to play gmod with us
> > > >
> > > >
> > > >
> > > >
> > > >
> > > >
> > > > Sent from Windows Mail
> > > >
> > > >
> > > >
> > > >
> > > >
> > > > From: Ayyyyyyyy lmao
> > > > Sent: Tuesday, March 31, 2015 6:36 PM
> > > > To: Team1559/RecycleRush2015
> > > >
> > > >
> > > >
> > > >
> > > > Sure I guess idk
> > > > On Mar 31, 2015 6:34 PM, "johndunaske" <notifications@github.com>
> > > wrote:
> > > >
> > > > > And about Gary's mod?
> > > > >
> > > > > Sent from my iPhone
> > > > >
> > > > > > On Mar 31, 2015, at 6:31 PM, MaxSpiegs <notifications@github.com
> >
> > > wrote:
> > > > > >
> > > > > > Better yet, get "8 Crazy Nights". Adam Sandler, Deers eating
> > > "droppings"
> > > > > > and with the MOST ANNOYING VOICE a character can ever have
> > > > > > On Mar 30, 2015 8:55 PM, "Cody Burrows" <
> notifications@github.com>
> > > > > wrote:
> > > > > >
> > > > > > > I can't edit the code right now which makes me sad.
> > > > > > > Pete:
> > > > > > > Sudo check your email so you can edit it for me!!!
> > > > > > >
> > > > > > > —
> > > > > > > Reply to this email directly or view it on GitHub
> > > > > > > <
> > > > >
> > >
> https://github.com/Team1559/RecycleRush2015/commit/e988916aa0486744c6b367f2f78142639b2b0c67#commitcomment-10479823
> > > > > >
> > > > > > > .
> > > > > > >
> > > > > > —
> > > > > > Reply to this email directly or view it on GitHub.
> > > > > >
> > > > >
> > > > > —
> > > > > Reply to this email directly or view it on GitHub
> > > > > <
> > >
> https://github.com/Team1559/RecycleRush2015/commit/e988916aa0486744c6b367f2f78142639b2b0c67#commitcomment-10507032
> >
> > >
> > > > > .
> > > > >
> > > >
> > > > —
> > > > Reply to this email directly or view it on GitHub.
> > > > —
> > > > Reply to this email directly or view it on GitHub.
> > > >
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> https://github.com/Team1559/RecycleRush2015/commit/e988916aa0486744c6b367f2f78142639b2b0c67#commitcomment-10507367
> >
> > > .
> > >
> > —
> > Reply to this email directly or view it on GitHub.
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Team1559/RecycleRush2015/commit/e988916aa0486744c6b367f2f78142639b2b0c67#commitcomment-10564752>
> .
>


This one


TeamCity xBIM :: XbimExchange :: Build [Build 188](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1285&buildTypeId=XBIM_XbimExchange_Build) is now running


So I saw this guy, but can't figure out how to fix him without messing up people with suffixes. Better to special-case for exactly that name, or just deal with the fact that the source sometimes screws up?


I agree... I'll change it back.


I think the wording here is not correct. So with my comment I tried to clarify how our APIs work


This is amazing, thanks!


Ah ok


Судя по тестам isset быстрее в несколько раз
Вот http://juliusbeckmann.de/blog/php-benchmark-isset-or-array_key_exists.html


In this case discussion is a dictionary, not the object.


This line is redundant, already have it at L279 above. Or if it has changed, it needs a name change, I get a warning on it


TeamCity xBIM :: XbimExchange :: Build [Build 188](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1285&buildTypeId=XBIM_XbimExchange_Build) outcome was **SUCCESS**
Summary: Running Build time: 00:02:31


@JeffreyWay regarding standard behavior, why? How is that consistent with coffeescript folder structure processing and how can that possibly qualify as behavior to be expected? If I've got a folder structure for my sass files, why wouldn't I want to keep that structure once they're processed into css files?

This is your thing, and there's loads involved, I get that. It just feels like sometimes things are hacked on willy nilly, considering you're making tools used by thousands of developers.


I loop replacement first because it seems natural.

What do you think?


there weren't, but it doesn't mean there couldn't be. fixed.


I guess so. You'll need to change https://github.com/dualspiral/Lottery/commit/5b7565916d4409a26cba0c84399c8013edba5eef#diff-8c93ed918e18bbc57b0feb7f651d7cb7R384 to be this limit, and change the logic in

https://github.com/dualspiral/Lottery/commit/5b7565916d4409a26cba0c84399c8013edba5eef#diff-8c93ed918e18bbc57b0feb7f651d7cb7R393

and

https://github.com/dualspiral/Lottery/commit/5b7565916d4409a26cba0c84399c8013edba5eef#diff-8c93ed918e18bbc57b0feb7f651d7cb7R404

need to be changed to be the `>=` operator. I think.


also fixed another error. should have been checking e, not e-mail here.


Did you leave?

Sent from my iPhone

> On Apr 1, 2015, at 9:33 PM, MaxSpiegs <notifications@github.com> wrote:
>
> Sending out this email cause I know you guys have great photos. Anyone have
> a pic of fleck making a rather interesting face? Preferably one with rather
> squinted eyes? Thanks!
>
> On Tue, Mar 31, 2015 at 7:02 PM, johndunaske <notifications@github.com>
> wrote:
>
> > I bought it
> >
> > Sent from my iPhone
> >
> >
> > > On Mar 31, 2015, at 6:42 PM, MergeyMergeMerge <notifications@github.com>
> > wrote:
> > >
> > > Also, spread the link to anyone who wants to play gmod with us
> > >
> > >
> > >
> > >
> > >
> > >
> > > Sent from Windows Mail
> > >
> > >
> > >
> > >
> > >
> > > From: Ayyyyyyyy lmao
> > > Sent: Tuesday, March 31, 2015 6:36 PM
> > > To: Team1559/RecycleRush2015
> > >
> > >
> > >
> > >
> > > Sure I guess idk
> > > On Mar 31, 2015 6:34 PM, "johndunaske" <notifications@github.com>
> > wrote:
> > >
> > > > And about Gary's mod?
> > > >
> > > > Sent from my iPhone
> > > >
> > > > > On Mar 31, 2015, at 6:31 PM, MaxSpiegs <notifications@github.com>
> > wrote:
> > > > >
> > > > > Better yet, get "8 Crazy Nights". Adam Sandler, Deers eating
> > "droppings"
> > > > > and with the MOST ANNOYING VOICE a character can ever have
> > > > > On Mar 30, 2015 8:55 PM, "Cody Burrows" <notifications@github.com>
> > > > wrote:
> > > > >
> > > > > > I can't edit the code right now which makes me sad.
> > > > > > Pete:
> > > > > > Sudo check your email so you can edit it for me!!!
> > > > > >
> > > > > > —
> > > > > > Reply to this email directly or view it on GitHub
> > > > > > <
> > > >
> > https://github.com/Team1559/RecycleRush2015/commit/e988916aa0486744c6b367f2f78142639b2b0c67#commitcomment-10479823
> > > > >
> > > > > > .
> > > > > >
> > > > > —
> > > > > Reply to this email directly or view it on GitHub.
> > > > >
> > > >
> > > > —
> > > > Reply to this email directly or view it on GitHub
> > > > <
> > https://github.com/Team1559/RecycleRush2015/commit/e988916aa0486744c6b367f2f78142639b2b0c67#commitcomment-10507032>
> >
> > > > .
> > > >
> > >
> > > —
> > > Reply to this email directly or view it on GitHub.
> > > —
> > > Reply to this email directly or view it on GitHub.
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <https://github.com/Team1559/RecycleRush2015/commit/e988916aa0486744c6b367f2f78142639b2b0c67#commitcomment-10507367>
> > .
> >
> —
> Reply to this email directly or view it on GitHub.
>


Why not skeleton?


So, what about keeping just line 32 of https://github.com/mlocati/concrete5-5.7.0/blob/bfad81e4a78af689744ff26872b93622bdb493f9/web/concrete/src/Url/Resolver/CanonicalUrlResolver.php#L29-L33 ?


<img src="http://media.giphy.com/media/ZxrpYTevDLmmY/giphy.gif">


Cool Thanks... I'll adjust the wording.


This changes the styling for these circles, adding a class of `col-sm-3` that they previously did not have.

Do they still look the same at all screen resolutions?


Yes

On Fri, Apr 3, 2015, 11:47 AM johndunaske <notifications@github.com> wrote:

> Did you leave?
>
> Sent from my iPhone
>
> > On Apr 1, 2015, at 9:33 PM, MaxSpiegs <notifications@github.com> wrote:
> >
> > Sending out this email cause I know you guys have great photos. Anyone
> have
> > a pic of fleck making a rather interesting face? Preferably one with
> rather
> > squinted eyes? Thanks!
> >
> > On Tue, Mar 31, 2015 at 7:02 PM, johndunaske <notifications@github.com>
> > wrote:
> >
> > > I bought it
> > >
> > > Sent from my iPhone
> > >
> > >
> > > > On Mar 31, 2015, at 6:42 PM, MergeyMergeMerge <
> notifications@github.com>
> > > wrote:
> > > >
> > > > Also, spread the link to anyone who wants to play gmod with us
> > > >
> > > >
> > > >
> > > >
> > > >
> > > >
> > > > Sent from Windows Mail
> > > >
> > > >
> > > >
> > > >
> > > >
> > > > From: Ayyyyyyyy lmao
> > > > Sent: Tuesday, March 31, 2015 6:36 PM
> > > > To: Team1559/RecycleRush2015
> > > >
> > > >
> > > >
> > > >
> > > > Sure I guess idk
> > > > On Mar 31, 2015 6:34 PM, "johndunaske" <notifications@github.com>
> > > wrote:
> > > >
> > > > > And about Gary's mod?
> > > > >
> > > > > Sent from my iPhone
> > > > >
> > > > > > On Mar 31, 2015, at 6:31 PM, MaxSpiegs <notifications@github.com
> >
> > > wrote:
> > > > > >
> > > > > > Better yet, get "8 Crazy Nights". Adam Sandler, Deers eating
> > > "droppings"
> > > > > > and with the MOST ANNOYING VOICE a character can ever have
> > > > > > On Mar 30, 2015 8:55 PM, "Cody Burrows" <
> notifications@github.com>
> > > > > wrote:
> > > > > >
> > > > > > > I can't edit the code right now which makes me sad.
> > > > > > > Pete:
> > > > > > > Sudo check your email so you can edit it for me!!!
> > > > > > >
> > > > > > > —
> > > > > > > Reply to this email directly or view it on GitHub
> > > > > > > <
> > > > >
> > >
> https://github.com/Team1559/RecycleRush2015/commit/e988916aa0486744c6b367f2f78142639b2b0c67#commitcomment-10479823
> > > > > >
> > > > > > > .
> > > > > > >
> > > > > > —
> > > > > > Reply to this email directly or view it on GitHub.
> > > > > >
> > > > >
> > > > > —
> > > > > Reply to this email directly or view it on GitHub
> > > > > <
> > >
> https://github.com/Team1559/RecycleRush2015/commit/e988916aa0486744c6b367f2f78142639b2b0c67#commitcomment-10507032
> >
> > >
> > > > > .
> > > > >
> > > >
> > > > —
> > > > Reply to this email directly or view it on GitHub.
> > > > —
> > > > Reply to this email directly or view it on GitHub.
> > > >
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> https://github.com/Team1559/RecycleRush2015/commit/e988916aa0486744c6b367f2f78142639b2b0c67#commitcomment-10507367
> >
> > > .
> > >
> > —
> > Reply to this email directly or view it on GitHub.
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Team1559/RecycleRush2015/commit/e988916aa0486744c6b367f2f78142639b2b0c67#commitcomment-10564847>
> .
>


saw approval from jdm
at https://github.com/Ms2ger/servo/commit/eefb0773a053422ecd5c7a5b0deaf1558f53ef22


merging Ms2ger/servo/impl-send = eefb0773 into auto


Ms2ger/servo/impl-send = eefb0773 merged ok, testing candidate = 74f8c0ee


Missing param tag


lodash supports this too. Woop woop. It's handy when dealing with arrays of properties to not have to use `.apply(_, blah)` and it aligns with other methods, like `_.omit` and `_.pick`, which allow individual args or arrays.


I think this line can go away.


failed


You can probably delete this line.


Yea I don't know where that is lol

Sent from my iPhone

> On Apr 3, 2015, at 11:43 AM, JeremyWeed <notifications@github.com> wrote:
>
> We're at bettas Italian food.
>
> On Fri, Apr 3, 2015, 11:41 AM johndunaske <notifications@github.com> wrote:
>
> > Where the hell are you guys?
> >
> > Sent from my iPhone
> >
> > > On Apr 1, 2015, at 9:33 PM, MaxSpiegs <notifications@github.com> wrote:
> > >
> > > Sending out this email cause I know you guys have great photos. Anyone
> > have
> > > a pic of fleck making a rather interesting face? Preferably one with
> > rather
> > > squinted eyes? Thanks!
> > >
> > > On Tue, Mar 31, 2015 at 7:02 PM, johndunaske <notifications@github.com>
> > > wrote:
> > >
> > > > I bought it
> > > >
> > > > Sent from my iPhone
> > > >
> > > >
> > > > > On Mar 31, 2015, at 6:42 PM, MergeyMergeMerge <
> > notifications@github.com>
> > > > wrote:
> > > > >
> > > > > Also, spread the link to anyone who wants to play gmod with us
> > > > >
> > > > >
> > > > >
> > > > >
> > > > >
> > > > >
> > > > > Sent from Windows Mail
> > > > >
> > > > >
> > > > >
> > > > >
> > > > >
> > > > > From: Ayyyyyyyy lmao
> > > > > Sent: Tuesday, March 31, 2015 6:36 PM
> > > > > To: Team1559/RecycleRush2015
> > > > >
> > > > >
> > > > >
> > > > >
> > > > > Sure I guess idk
> > > > > On Mar 31, 2015 6:34 PM, "johndunaske" <notifications@github.com>
> > > > wrote:
> > > > >
> > > > > > And about Gary's mod?
> > > > > >
> > > > > > Sent from my iPhone
> > > > > >
> > > > > > > On Mar 31, 2015, at 6:31 PM, MaxSpiegs <notifications@github.com
> > >
> > > > wrote:
> > > > > > >
> > > > > > > Better yet, get "8 Crazy Nights". Adam Sandler, Deers eating
> > > > "droppings"
> > > > > > > and with the MOST ANNOYING VOICE a character can ever have
> > > > > > > On Mar 30, 2015 8:55 PM, "Cody Burrows" <
> > notifications@github.com>
> > > > > > wrote:
> > > > > > >
> > > > > > > > I can't edit the code right now which makes me sad.
> > > > > > > > Pete:
> > > > > > > > Sudo check your email so you can edit it for me!!!
> > > > > > > >
> > > > > > > > —
> > > > > > > > Reply to this email directly or view it on GitHub
> > > > > > > > <
> > > > > >
> > > >
> > https://github.com/Team1559/RecycleRush2015/commit/e988916aa0486744c6b367f2f78142639b2b0c67#commitcomment-10479823
> > > > > > >
> > > > > > > > .
> > > > > > > >
> > > > > > > —
> > > > > > > Reply to this email directly or view it on GitHub.
> > > > > > >
> > > > > >
> > > > > > —
> > > > > > Reply to this email directly or view it on GitHub
> > > > > > <
> > > >
> > https://github.com/Team1559/RecycleRush2015/commit/e988916aa0486744c6b367f2f78142639b2b0c67#commitcomment-10507032
> > >
> > > >
> > > > > > .
> > > > > >
> > > > >
> > > > > —
> > > > > Reply to this email directly or view it on GitHub.
> > > > > —
> > > > > Reply to this email directly or view it on GitHub.
> > > > >
> > > >
> > > > —
> > > > Reply to this email directly or view it on GitHub
> > > > <
> > https://github.com/Team1559/RecycleRush2015/commit/e988916aa0486744c6b367f2f78142639b2b0c67#commitcomment-10507367
> > >
> > > > .
> > > >
> > > —
> > > Reply to this email directly or view it on GitHub.
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <https://github.com/Team1559/RecycleRush2015/commit/e988916aa0486744c6b367f2f78142639b2b0c67#commitcomment-10564752>
> > .
> >
> —
> Reply to this email directly or view it on GitHub.
>


Que opinas sobre la decadencia del pais en lo que al barra bin barra bash se refiere?


If you grab role first, and then remove `things_to_replace`, and then resolve name, and then `strip(" ,"`, then the trailing comma on Schwartzkopf will be removed, I think.


I got my pre ordered lunch, so I'm not coming

Sent from my iPhone

> On Apr 3, 2015, at 11:43 AM, JeremyWeed <notifications@github.com> wrote:
>
> We're at bettas Italian food.
>
> On Fri, Apr 3, 2015, 11:41 AM johndunaske <notifications@github.com> wrote:
>
> > Where the hell are you guys?
> >
> > Sent from my iPhone
> >
> > > On Apr 1, 2015, at 9:33 PM, MaxSpiegs <notifications@github.com> wrote:
> > >
> > > Sending out this email cause I know you guys have great photos. Anyone
> > have
> > > a pic of fleck making a rather interesting face? Preferably one with
> > rather
> > > squinted eyes? Thanks!
> > >
> > > On Tue, Mar 31, 2015 at 7:02 PM, johndunaske <notifications@github.com>
> > > wrote:
> > >
> > > > I bought it
> > > >
> > > > Sent from my iPhone
> > > >
> > > >
> > > > > On Mar 31, 2015, at 6:42 PM, MergeyMergeMerge <
> > notifications@github.com>
> > > > wrote:
> > > > >
> > > > > Also, spread the link to anyone who wants to play gmod with us
> > > > >
> > > > >
> > > > >
> > > > >
> > > > >
> > > > >
> > > > > Sent from Windows Mail
> > > > >
> > > > >
> > > > >
> > > > >
> > > > >
> > > > > From: Ayyyyyyyy lmao
> > > > > Sent: Tuesday, March 31, 2015 6:36 PM
> > > > > To: Team1559/RecycleRush2015
> > > > >
> > > > >
> > > > >
> > > > >
> > > > > Sure I guess idk
> > > > > On Mar 31, 2015 6:34 PM, "johndunaske" <notifications@github.com>
> > > > wrote:
> > > > >
> > > > > > And about Gary's mod?
> > > > > >
> > > > > > Sent from my iPhone
> > > > > >
> > > > > > > On Mar 31, 2015, at 6:31 PM, MaxSpiegs <notifications@github.com
> > >
> > > > wrote:
> > > > > > >
> > > > > > > Better yet, get "8 Crazy Nights". Adam Sandler, Deers eating
> > > > "droppings"
> > > > > > > and with the MOST ANNOYING VOICE a character can ever have
> > > > > > > On Mar 30, 2015 8:55 PM, "Cody Burrows" <
> > notifications@github.com>
> > > > > > wrote:
> > > > > > >
> > > > > > > > I can't edit the code right now which makes me sad.
> > > > > > > > Pete:
> > > > > > > > Sudo check your email so you can edit it for me!!!
> > > > > > > >
> > > > > > > > —
> > > > > > > > Reply to this email directly or view it on GitHub
> > > > > > > > <
> > > > > >
> > > >
> > https://github.com/Team1559/RecycleRush2015/commit/e988916aa0486744c6b367f2f78142639b2b0c67#commitcomment-10479823
> > > > > > >
> > > > > > > > .
> > > > > > > >
> > > > > > > —
> > > > > > > Reply to this email directly or view it on GitHub.
> > > > > > >
> > > > > >
> > > > > > —
> > > > > > Reply to this email directly or view it on GitHub
> > > > > > <
> > > >
> > https://github.com/Team1559/RecycleRush2015/commit/e988916aa0486744c6b367f2f78142639b2b0c67#commitcomment-10507032
> > >
> > > >
> > > > > > .
> > > > > >
> > > > >
> > > > > —
> > > > > Reply to this email directly or view it on GitHub.
> > > > > —
> > > > > Reply to this email directly or view it on GitHub.
> > > > >
> > > >
> > > > —
> > > > Reply to this email directly or view it on GitHub
> > > > <
> > https://github.com/Team1559/RecycleRush2015/commit/e988916aa0486744c6b367f2f78142639b2b0c67#commitcomment-10507367
> > >
> > > > .
> > > >
> > > —
> > > Reply to this email directly or view it on GitHub.
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <https://github.com/Team1559/RecycleRush2015/commit/e988916aa0486744c6b367f2f78142639b2b0c67#commitcomment-10564752>
> > .
> >
> —
> Reply to this email directly or view it on GitHub.
>


@deepthawtz good catch. Russ fixed it.


Need to add a new line here


It actually has a different value.


I don't think there's anything we can do about the comma in the middle of his name; that was a red herring.


It happens if `timer._repeat` invoked `timeout.close()`. This was the reason for segfaults.


I removed calls to *destroyed.push_back()* because I saw no calls to *pop_back()*, *erase()* or *clear()* on *destroyed*. I suppose that *destroyed* grows with each call to *Core::updateProperty()*.
Is the situation when a plugin deletes a property before it has been processed a relatively rare occurrence?


New line here, too


I'll change replace example.com with localhost:9000....


Probably because assigning structs isn't ANSI C. Should be done through memcpy() instead (ideally through some kind of SET_CHEAT() macro to hide the unpleasantness).


Transform this after -> Complete transform now that

(Imagining JS if 'this' had been called 'self').


That's a pretty long commit message ;)


New line here as well


This doesn't handle the need to also alter the port and would end up with a url like: https://example.com:80/, I'll say again if force_ssl is enabled, we should never render a page in non-ssl.


I'll add this to the doc


Jesus - swagger_core depends on Scala that adds 25MB worth of deps. We need to get rid of that somehow.


Consider adding `padding: 0 2px` or similar here. Will change this
![selection_084](https://cloud.githubusercontent.com/assets/1042475/6984546/f91bf1a0-d9f7-11e4-9d06-e1a1949a13d8.png)

to this

![selection_086](https://cloud.githubusercontent.com/assets/1042475/6984548/fda29fbc-d9f7-11e4-8da7-91353bebbf22.png)


I'll change the wording, but I do think this is important for the guide.


I thought that might be the case, so I propose it gets a new name the
second time, to make it clearer what is going on

On Fri, Apr 3, 2015 at 5:51 PM, Matthew Pickering <notifications@github.com>
wrote:

> It actually has a different value.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mpickering/ghc-exactprint/commit/7157a9cc1ba11cd1b818ba7fb6b95655e9de0063#commitcomment-10564922>
> .
>


rip


Cool, I like it.


Attractive buttons, will do. Thx


Could we include the fact that this fix is specific to `es_repo_mgr`? Perhaps, "not being passed to es_repo_mgr sub-commands".


full config (latest version) - https://github.com/Intelworks/OpenTAXII/blob/master/opentaxii/defaults.yml


So this should read : .... the system libraries for `libtaxii`` will be installed....


Nice!


Interesting it might be as simple as changing `<<<>=`  to `<<<>~` ;-)


Given that listener registration is low traffic and usually done only during setup, whereas event dispatch is frequent, wouldn't it be much better to not introduce the possibly performance-critical `.clone()` but to use an immutable collection for the listeners? There are immutable collections which have constant time append, and we wouldn't need a clone at all during event dispatch.


TeamCity MonoGame :: Develop (Win) [Build 3.4.0.130](http://teamcity.monogame.net/viewLog.html?buildId=5985&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


Нет проверки на введенное имя (есть оно вообще или нет)


aha, ok. just checking )


@chrisseaton Both of these modifications were made in 2014, so I left the copyright header intact.


wait, no. system libraries will not be installed by default. and that is the main point. system libraries need to be installed manually!


Yep that's correct, well noticed. The "base build" changed though, so s2protocol.py needs this new protocol file to exist so it can __import__ it.


because I'm stupid idiot


If I understand correctly this is not connected to the destructuring and is covered by previous tests. The refactored form does not declare `${varName}` it seems. That has no consequences and affects no test?


@maikany


no!


Ah, shoot, my bad, it looks like the build on Travis is still broken. This fixed it on my machine, and I should be on yesterday's nightly. I'll check it out.


:) no!


Infinite thanks for the change... exactly for the network applet.


Hi Ryan,

Thanks for taking the time to look at this. The problem is with any green on an active high-contrast modeline.


+1, I think there's actually a comment in one of the files where popup.css is explicitly not included


I don't like "here" links.

Maybe something like: "7. You can [download the logo from Google Drive](https://drive.google.com/open?id=0B7xWXqwxUXt4LXJtcXZFbmtUVDQ) in various formats" ?


Пожалуйста, называй коммиты осмысленно


I just sent everything in.


Pushed a new version which should clarify this section.


yep it's better :-)


mine, we are using on your shard slightly modified version of hues
![hues](https://cloud.githubusercontent.com/assets/2104849/6984667/76d50b80-da2b-11e4-9d6f-a91e4075eb60.png)


Такие проверки лучше делать через regex


@alandonovan How can we use go-oracle without minor mode ?


@siusin 这里是不是删错了？应该删的是 `]` ？


@siusin 同上


Vundle is easy, clone Vundle, add the setup line to vimrc then add a line for each GitHub repository (e.g. `Plugin 'raehik/my-awesome-repo-on-github'`), no need to commit submodules etc.


Closes #47


Done


не очень понятно, произошло ли что-нибудьпосле нажатия кнопки "Войти" - имя пользователя просто сбрасывается.


Closes #102


Looks perfect :)


Or just "Transform tree ..."


The OpenStack Object Storage API v1 mentions only about _X-Account-Meta-Temp-Url-Key_ and _X-Account-Meta-Temp-Url-Key-2_ headers. I doubt that the number would be extended in the future -- AFAIK the goal for having a second one is to orchestrate key change.

Two _ifs_ are easier to read during analysis than loop, so I propose to leave it at is.


For now, yes. But I'm planning to drop them from the arch.h files later and clean this up.


How about making the last sentence more precise: "The file must contain the full certificate chain and the private key". I don't think you need to mention that the format is PEM -- it's pretty obvious from the extension.


Oh xD why avoid though?


My bad. Didn't see it in your defconfig and assumed it wrong, forget about it and sorry!


I didn't mean to add this file in this commit. It was supposed to end up in the next one.


I cleaned this up to make the code as well as the generated code more consistent. The comment on line 521 explains the new form.


Thank you for being on top of adding them after the patch !


its probably the same.. it a bitset operation
https://www.elastic.co/blog/all-about-elasticsearch-filter-bitsets


`@LTLIBINTL@` is required for Linux build.
Probably, ifdef or use --disable-nls configure option will help.


@rec Gonna be honest when I saw "clean up Sustain" I was imagining something much bigger in scope, along the lines of using Boost.Process to make the code cross-platform (http://www.highscore.de/boost/process0.5/).


Please avoid changing this library, to ease future upgrades.


Changing the version number here to be hard-coded is just wrong.


TeamCity MonoGame :: Develop (Win) [Build 3.4.0.130](http://teamcity.monogame.net/viewLog.html?buildId=5985&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 469, ignored: 6 Build time: 00:14:46


TeamCity MonoGame :: Develop (Win) [Build 3.4.0.131](http://teamcity.monogame.net/viewLog.html?buildId=5987&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


Good catch!


you have no idea the rabbit hole that colons sent me down. but fixed.


Changed the way AGREE cancels to avoid needing a change in JKind


Fork me?


C++ code of my data structure projects.


Не пишет текст сообщения, надо проверить иерархию


That might be a possibility :-) but this is just small stuff extracted from my attempt to find the shutdown deadlock.


fixed - code stolen from elsewhere, no idea why they didn't do this in the first place since it works.


Thanks for your comment. `zone.js` cannot be `async` because it must be executed before angular. app.js might be a candidate for `async` but it would be nice only if the page has certain amount of HTML outside of the root component.


> if force_ssl is enabled, we should never render a page in non-ssl.

if `force_ssl` is enable and also `redirect_to_canonical_host` is enabled?


Uh no, this should be removed


And all of this removed


lalalala


:shipit:


Does this work, because this is a category on `NSURLSessionDataTask` which doesn't exist on iOS 6 / OS X 10.8?


Done, see 075fac365c5ee7e4c78c0266c3cac1aae12e7616.


the stripping I am doing. fixed.


![qq20150403-1 2x](https://cloud.githubusercontent.com/assets/501861/6984664/ba6ff378-da5d-11e4-9609-05d060046031.png)
Because manager.config.CookieLifeTime is 0 here. If you wants to use it as a session cookie, why adding an expire to it? It's life time = explorer's life time.


This was the piece of code that was causing a problem in the jumbotron.


Should be `!Strings.isNullOrEmpty(key)`.


I wonder why this issue didn't come up in the original PR though.


This was part of the solution. I've added the row class to provide a little more structure.


This would mean we can't install 2.7 python


I think that simple to see, but if not here an example: two versions of interface implementation that difference by exception in method signature. Core code that use that interface. 2 versions  of extensions that implement that interface. Second version of extension become not portable to version 1 and it hard to trace/understand.


If you're going to change this one…


… why not change this one too?


rax retry tests


ignored per Miles verbal confirmation


Awesome! Thanks!

On Fri, Apr 3, 2015 at 10:57 AM Artur Malabarba <notifications@github.com>
wrote:

> Yes , it's a mistake. Sorry about that. will fix it again. :-)
>
> --
> Reply to this email directly or view it on GitHub
> <https://github.com/Malabarba/smart-mode-line/commit/30d561aa9782399fc50b0a6abb5728233e805654#commitcomment-10564142>
> .
>


The `WorkUnit` returned from `TaskState` is an `ImmutableWorkUnit`, so `setProps` will thrown an exception here.


Yes. Thank you rurban.


No worries man.


Just in case you wanted your plugin to react in a different way to a specter than to a player. Like you could teleport a specter to a certain coordinate.


(We should be checking the result of `waitpid()` instead of calling `kill()`, but that's out of scope for this branch.)


I want the class to have a name when debugging


This breaks the setOptions and setCallback functions resulting in
````Method Illuminate\View\View::__toString() must not throw an exception````

Reverting back to the original ````javascript.blade.php```` file fixed the problem


Does adding a preceding underscore not indicate the field is "protected"? If so, then this is not correct


This is a big win for readability.  I'm sure the code is smaller and faster, too.


You didn't commit anything kiddo


Oh, boost.process "isn't ready for primetime" yet.  Sigh. ..


J'aimerais accepter ce PR uniquement (et pas les autres), mais Github ne me propose pas cette fonction :/


nice one :D


TeamCity MonoGame :: Develop (Win) [Build 3.4.0.132](http://teamcity.monogame.net/viewLog.html?buildId=5989&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


TeamCity MonoGame :: Develop (Win) [Build 3.4.0.131](http://teamcity.monogame.net/viewLog.html?buildId=5987&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 470, ignored: 6 Build time: 00:14:33


We shouldn't mix short and long options. In line 17 we are already using `--system` so we should use `--quiet` instead of `-q` or isn't that portable?


Done, see 075fac365c5ee7e4c78c0266c3cac1aae12e7616.


I would like to keep citeimg, as we discussed, temporarily for backward compatibility, and deprecate it, to be dropped in subsequent release. I see no reason to put in an unused geojson class that should never be implemented except as a CITE Extension.


shouldn't 204 and 209 be ensure => positioned

?


Test case?


You can also mentioned [`hardcore-mode`](https://github.com/magnars/hardcore-mode.el). Using that made me stop using arrow keys :)


why not, it can be useful (:


I added this to my project (https://github.com/symfony-cmf/symfony-cmf-website/pull/70) and got:
```
Github integration pushed to platformsh 3 minutes ago > Failure
LOG:
Found 1 new commit.

E: Error parsing configuration files:
    - http://{default}/: Upstream 'php:php' of route 'http://{default}/' doesn't exist.

E: Error building the project: Invalid configuration files, aborting build.

1a28257  moving to platform.sh
```


trying this in 836426fc0e0ee523c350e160a0ea85a813daf18d


These constants belong in the .cpp file


@basmaelgeziry the comment should be above the method and right above it without leaving a line , start commenting by ## then #Destroys ...


this is a great diff


:D an extra line :dash:


Yeah, generalising `withGroupOpacity` could get messy. I can get the types to match up but I'm not confident it's doing the right thing.

Moving to `ReaderT` makes sense here I think. I'll code it up.


test


Yes, lol, I could have replaced the entire thing with `$valgrind pornography$`!


I removed this code in a future commit, so I'll just drop this part of the diff.


I see that variable chunking is currently not supported (above). Please take out these block-size printf statements or comments them out for now.


If the test does not run and fail, what might happen is the following:
- commit test, but test doesn't even compile.
- commit implementation, test runs and passes.

This provides no validation that the implementation is the reason the test passes, but only that it is the reason that the test compiles. For example, it might hide a situation where the test isn't truly testing the implementation. A future, refactoring of the implementation could then go untested, and that might never be discovered.


Would it be more appropriate for this to be an exception if the preference is not of the requested type?


Done.


r+


It should be rare.  It was common at one point.  So I think in the meantime we should produce a DebugOut::Error so that plugin writers will see that deleting properties is bad.  It'd be a bit more graceful than a seg fault.


Busybox sysctl only supports -q, not --quiet, that is why I used -q.
Also, busybox sysctl does not support --system, but I opened a bug for that, because it is a useful option and should be added.


Poltergeist looked to be the better option - it can be set to fail integration tests on javascript errors, which is nice.

I've added it - see here:

https://github.com/europeana/europeana-channels-blacklight/commit/bd4d2f8605d665a3eb15f9a91383103fa1fdfa9e

(and see the updated README for how to install the PhantomJS dependency)

Unfortunately to get the tests to run I've had to start putting in 'impure' code like the following:

```ruby
      # without the following 'expect' assertion the subsequent 'find' will fail
      expect(page).to have_css('.searchbar button.search-submit')

      find('.searchbar button.search-submit').trigger('click')

      # without the following timeout the subsequent 'expect' will fail
      sleep 2
```

That aside - it works.


You are welcome! I'm glad to have given you a starting point.


Actually. I redid the commit. didnt know if falcon had it or not so ddid it n=my self


there seems to be a syntax error here as the rendered link points to ``http://symfony.com/doc/current/cookbook/deployment/configure-services``


So are we going to have a massive refactoring to switch to annotations? The only downside to using annotations is that it heavily lets DAO(hibernate) implementation creep into the service layer which defeats why we use spring's dependency injection to loosely tie components in the DAO and service layer. But again one might argue that to some minimal extent we are already doing it.


typo?


@eraviart : let me know if we can/should mutualize this with openfisca-france counterpart


This line can just be `;`.


Same here.


I'm still using the same script.  This script actually dates from sometime around 2000 or 2001the same time I switched from freebsd to OS X. I don't remember exactly which year.  I realized recently that I need to update the mailcap a bit. I'm not handling some attachments properly with modern software.


By modifying the generated code string, you are invalidating all the mappings in the old source map. Line numbers and columns are changing, so the lines and columns from the mappings in the old source map no longer point to the same things that they used to.

Drawing on an analogy, it is like expecting this loop

    var i = 0, length = array.length;
    for ( ; i < length; i++) {
      var x = array[i];
      ...
    }

to iterate over each element `x` within `array` exactly once, while simultaneously it is removing elements from the front of the array within `...`. Without correcting `i` for each element removed, the loop will skip over some elements in the array, `length` no longer refers to the correct length, and `i` can be incremented past the array's new length.

In the same way `i` becomes an invalid index when the array was mutated, the mappings become invalid mappings when the generated code they point into is mutated.

--------------------------------------------------

What you/uglify-js should do if you want to transform the generated code and maintain the mappings in the process is either

* Transform the code into a `SourceNode` tree with `SourceNode.fromStringWithSourceMap` and ***then*** run your transformation on the `SourceNode` tree with `SourceNode#walk`, rather than the generated code string.

* Keep running the transformation on the string directly, but as you go keep track of line and column offset adjustments made by your transformation, and apply the offset adjustments to the mappings as you go. This approach requires a lot of careful bookkeeping and seems pretty easy to mess up.


You can uncomment ``debug_toolbar.panels.profiling.ProfilingPanel``. The output is similar to ``debug_toolbar_line_profiler.panel.ProfilingPanel``, just not quite as easy to understand.


there's also `beast::Process`


It would be great to fill up a buffer instead. That should be done eventually, but this is a nice stop-gap. Provided the emulated GBA supplies enough samples, no sound distortion should happen. This gets rid of all the nasty pops and audio crackling that were present in the DMA channels.

All in all, a very good Friday :)


TeamCity MonoGame :: Develop (Win) [Build 3.4.0.132](http://teamcity.monogame.net/viewLog.html?buildId=5989&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 470, ignored: 6 Build time: 00:14:08


you meant #ifdef??

please use line break here.


pls make paths relative to the working directory


Better yet, write ```
if (ShouldLog(s, k) && (c)  \
    ::beast::Journal::Stream(…);  \
else
```
and similarly above.


relative paths


Just to go back on my words a little, if we are to use annotations, as @rkorytkowski has done, it would be best to use the standard JPA annotations as opposed to those in hibernate. But again this will mean more work since we have to rewrite most queries in the DAOs.


Maybe force Ssl is a misnomer because it's really only intended
To force ssl if you have redirect to canonical host checked

Think of it like the old base url - the force ssl setting is really just a way to set the scheme portion of your base url to https.

Not married to this approach - mostly just want settings that make sense, a canonical host name and some way to force that host name is the only one that renders.

Maybe we only show the ssl checkbox if the redirect to canonical host is checked ?

Sent from my iPhone

> On Apr 3, 2015, at 9:13 AM, Michele Locati <notifications@github.com> wrote:
>
> if force_ssl is enabled, we should never render a page in non-ssl.
>
> if force_ssl is enable and also redirect_to_canonical_host is enabled?
>
> —
> Reply to this email directly or view it on GitHub.
>


r+


It may just be me, but I feel the above logic is unnecessary convoluted.

I would prefer:

func isPacketForwardingEnabled(ipVer ipVersion, iface string) (bool, error) {
	switch ipVer {
		case ipv4, ipv6:
			         return getKernelBoolParam(getForwardingKernelParam(ipVer, iface)
			case ipvboth:
				enable, err := getKernelBoolParam(getForwardingKernelParam(ipv4, "")
				if err != nil | !enabled {
					return enabled, err
				}
			       return getKernelBoolParam(getForwardingKernelParam(ipv6, iface)
			)
		)
	}

Same for the function below.

If it is just me, feel free to discard this comment.


Not of priority but ideally all the comments describing what each of the functions do should be written in '.h' file which can be used as quick reference. Please include the same, preferably in the following format

/*
 * @description: ......
 *
 * @in: input variables, description of each
 * @out: output variables, description of each
 * @return: various return status and what they mean.
 */


Skomentujmy to sobie :dancers:


It's okay! it's easy to fix rustbook:

```
$ cp ~/src/rust/src/rustbook/*.rs src/
```

:wink: I'll let you submit a PR though if you want to, otherwise, I'll just do it!


This file is a part of screed


I'm not sure what you mean?  This code isn't to handle an empty pages index.  The code is here to handle end conditions.


By doing this, you have to re-do your config file and your Beer ID is reset. I didn't want that so I didn't delete the beer.


do you mind using constructor references from Java 8? In Java 8 you can pass around references to methods, and constructors too, and it will probably eliminate the need in reflection


jdcfg


Use capital letters while defining macros.


s/over/from/


pls make this final and uppercase its name: LOGGER


Please move it into `pandalone` package.
And check my inline-review comments


The method is final, so it is safe to call.  I'll make the class anyway.


as long as you have all the 'compress/uncompress' methods returning a `File` object (which it looks like you do), this should be all good.


Is this code really necessary here?
Mabe make  a TC out of it??


This is used by `WindowOperator` which is in a different package


``import networkx as nx``


Looks good, thanks for helping me with this. I don't know if it matters, but Human_face is capitalized in the code while the filename itself is lowercase. Just thought I'd point that out in case it causes an issue.


sweet  :cake:


#1  but it is not a good idea to use marked directly in filter, better to move markdown render a service, in case we want to change to another library


pls add this copyright notice at the top of all Java files:

```
/*
Copyright 2015 BarD Software s.r.o

Licensed under the Apache License, Version 2.0 (the "License");
you may not use this file except in compliance with the License.
You may obtain a copy of the License at
http://www.apache.org/licenses/LICENSE-2.0
Unless required by applicable law or agreed to in writing, software
distributed under the License is distributed on an "AS IS" BASIS,
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
See the License for the specific language governing permissions and
limitations under the License.
*/
```


Wait, Rustbook is included in the Rust repo? I had no idea. Disregard all my PRs on this repo, then, I guess, sorry.


Oversight, fixed.


use `trustAsHtml` is must for angular binding html, since `ng-bind-html-unsafe` has been removed.


@lsmith77 Thanks for noticing (see #5144 for the fix).


@KunalKene1797
https://github.com/BlackBox
Dosen't seem related to BlackBox-OS at all, and also dosen't have a repository called build
From where is the build repo coming?


it seems element.html wont have angular binding's security problem.


why do you use same fd_blk for reading too and change its current offset postition. you could as well re-open the file or use dup() call and close it once you read the contents.

Also as joseph has mentioned, make fd_blk global static variable.


e... phpstorm's default indent size for json is 4 space....


Something about this test is breaking tests down the line. It supposed to reset the URL (which would then make the tests resolve to http://www.dummyco.com/ – but it's sticking with derpco.com. Check the latest travis ci build.


Esto fue por linea de comando, si hay algo que tengo que implementar o que esta faltando por favor decirlo ¿que estoy haciendo mal?

![por linea de comando](https://cloud.githubusercontent.com/assets/11746092/6985339/a744a8fc-da08-11e4-967f-8ace0fe47da1.GIF)


Its Not Yet Initialized My Friend :smile:  Check again CM/AOSP Repos added Today(Check the latest commit). soon after my exam, i'll upload other customized repos on this git and get going with the first builds.


can't use trustAsHtml as default render, because it won't work it use `element.html`


why do you read the contents in this routine?


if you want receive emply String instead null. This string field must be qoublequoted. Else it will transform to null.


Please rename fields according to our naming convention

* member fields are prefixed with "my": `myConfiguration`
* static fields such as singletons are prefixed with "our": `ourSingleton`
* static fields which look like constants (e.g. immutable objects) are uppercased: `CONFIGURATION_PATH`


Okay


any sense to initialize it in the constructor and make final?


fixed this in 563f30b


Actually I dont. It is my commit. I couldnt remember if falcon had the patch so I redid it my self. @arnavgosain


咦？


It may have been introduced during the manual rebasing somehow (but not sure how that would happen).


you may find `Preconditions` class from Google's Guava library useful. They allow you to create, well, preconditions: `myField = Preconditions.checkNotNull(arg)`


So, if we don't consider `force_ssl` in this CanonicalUrlResolver, we shouldn't consider the `canonical_port` too (`canonical_port` may be valid only for ssl, whereas we may not be in ssl, or vice-versa)...

The best solution would be to have two different `canonical_port`s, one for http and one for https.


You can also try to rebuild it under the build menu, which fixed some errors for me.


This was very fast. I was just about to create a ticket reporting hyper being unable to build on Apr 2 Nightly. Great response time :)


The purpose of this commit was to change most of the source code from Java to Kotlin; this is a first pass attempt, and it is working (except JUnit tests), but the style might be improved.


Haha thanks


No active merge of candidate 9fef56ea found, likely manual push to master


merging yodalee/servo/issue5397-location-pathname = 9fef56ea into auto


yodalee/servo/issue5397-location-pathname = 9fef56ea merged ok, testing candidate = bc896be3


@hunterboerner hyper **shall** build on Beta.


I'm removing that class entirely since my overhaul makes it really unneeded.


@aembler oh, I was confused then. I think that we can get away with having a conditional for `if($force_ssl && !$redirect_to_canonical_host)` that will force redirect to https and clear the port.


:tophat:


I suggest to create interface `IndexerConfiguration` with read-methods and make this class implementing it (rename to `IndexerConfigurationXml` ). Another option is to create a simple immutable class `IndexerConfiguration` with simple `indexPath` and `directories` fields (no annotations) and create its instances from `IndexConfigReader` or whatever, which looks pretty much like this class now.

Long story short, we'd like to avoid being tied to some particular XML library :)


This reverts my fix from March 10. That fix would cause the terrain to rebuild when an extension calls one of the terrain engine's require* functions. (For example, when you call engine->requireNormalMaps() the terrain needs to rebuild existing tiles.) So we need to revisit this and find a common solution.


Looks good to me. However we should closely monitor the performance impact of this change in Azure Website.
Do we already have something in place for that?


Thanks, added.


Hey, you should check out
https://www.datacamp.com/

Pretty sure they're hiring and it seems kind of up your alley in data analysis. I'm going to a meetup that they'll be at: http://www.meetup.com/ny-tech/events/218809133/


Not keen on throwing from ctr(). will add to warnings, and immediately return.


Only setting to not closed on success path.


great


>If you'd like to fiddle with Wiremod, consider copying one of the existing entities like the Wire Igniter (wire/lua/entities/gmod_wire_igniter.lua and wire/lua/wire/stools/igniter.lua) and see if you can make them do something different, like toggle gravity.

If you do this, be aware that some entities in wiremod are reeeeeeeeally old and messy. I'd suggest taking a glance at the github history of the entity to see when it was last edited first.


are classes from this package copied from somewhere? If yes then you may want to add the source and license, if available


Whoops lolz Greg be one step ahead of me


Not sure, on windows it's no problem as I tested it. But linux is usually another thing. Though I suspect Byond doesn't really care anyway, since it's already being used like that in other places.


Sorry should have referenced https://ampersandjs.com/learn/data-hook-attribute, from where I mostly stole the whole data-hook idea.


This change is correct but can't actually go in until we fix signal handling. :-1:


ok.. I am removing this with my next commit. :)


I can think of *std::weak_ptr* for this use case.


most of that looks fine - but i'm a little confused by the changes to commands.config:
+UninstallerHelper.exe * uninstall-rpfcoordinator.bat ~RingtailProcessingFramework|UNINSTALL_EXCLUSIONS

it seems odd to pass in that bat file as a parameter to that exe every time... why not have the uninstallerHelper itself look for an exclusions file named by the name of the role that we pass in?  such as 'UNINSTALL_Exclusions-RPFCOORDINATOR.config'


there are some configs missing in new file


Hi @lsmith77

You named your app `website` instead of `php`. So for your upstream, you'd need `php:website`, not `php:php`.

NB: renaming an app should not be done lightly - the data and files will be deleted.


That's why it's a separate commit.  :-)

Want me to knock it off?


What if there is no build.sbt script? Can we fall back to 2.11.5 in this case?


I'd say something like "If you use the Safari browser, be aware that it..." - since Safari doesn't support search even if someone wants to use Safari. :-)


I was doing the same thing auto populate foreign keys and optimize the query from url(also I like your query helper)


I am only talking about line 39, IMO the indention is missing.


@FadyKesbra perfect


Don' you need to specify the deep path as well to test "before treating it as a path"? :stuck_out_tongue:


nice one!


This was a cool surprise, nice Job!


I think it turned out quite nice. I've tried it on the backend tests and it works fine. Could someone try to build the site with it please?


Updated as per Steve Gaul's updates on 4/3/2015 (from his v3 spreadsheet).


I figured that it would also be ok to update return type of this analogous function.


oops forgot m15 as well


Complete


When you add in the registration urls here, do you know what actual endpoints they provide?


@BenjaminSchubert  any chance of a README file somewhere describing the intended usage?


What about something like this?
![url-handling-proposal](https://cloud.githubusercontent.com/assets/928116/6985708/9dfaec60-da36-11e4-99e4-a7ff0c205301.png)


@khadijaibrahim


I don't have a Linux machine at hand to try this out. Would the patch be okay if like 718 was left without changes?


As noted in https://github.com/marshmallow-code/marshmallow/issues/185#issuecomment-89364771 , I'm not sure this is even necessary. Any type that implements `len` should also implement `iter`.


cant delete this tag


Are you providing UI anywhere that points to these views?  If not, why provide the URLs.  If there's no visible interface in the user's browser that leads them here, there's not a whole lot of point in providing the url at all.


Tests (on super):
* ./jtrts.pl raw-sha256 raw-sha256-ng raw-sha512 raw-sha512-ng xsha512 raw-sha256-opencl raw-sha512-opencl xsha512-opencl
* ./jtrts.pl raw-sha256 raw-sha256-ng raw-sha512 raw-sha512-ng xsha512 raw-sha256-opencl raw-sha512-opencl xsha512-opencl -passthru="-dev=cpu"
* ./jtrts.pl raw-sha256 raw-sha256-ng raw-sha512 raw-sha512-ng xsha512 raw-sha256-opencl raw-sha512-opencl xsha512-opencl -passthru="-dev=5"
* ../run/john -test -form:sha512crypt-opencl -dev=5
* ../run/john -test -form:sha256crypt-opencl -dev=5
* ../run/john -test -form:sha512crypt-opencl (on well)
* ../run/john -test -form:sha256crypt-opencl (on well)
* ../run/john -test -form:sha512crypt-opencl -dev=cpu (on well)
* ../run/john -test -form:sha256crypt-opencl -dev=cpu (on well)

****
* ../run/john ~/testhashes -form=raw-sha256-opencl -wo:/home/claudio/solucao.txt
* ../run/john ~/testhashes -form=raw-sha512-opencl -wo:/home/claudio/solucao.txt
* ../run/john ~/testhashes -form=xsha512-opencl -wo:/home/claudio/solucao.txt
* ../run/john ~/testhashes -form=raw-sha256-opencl -mask:mot?Arhead
* ../run/john ~/testhashes -form=raw-sha512-opencl -mask:mot?Arhead
* rm ../run/\*.pot   *(all cracked)*

****
../run/john ~/testhashes -form=raw-sha512-opencl --mask=[Pp][Aa@][Ss5][Ss5][Ww][Oo0][Rr][Dd] -dev=5
../run/john ~/testhashes -form=raw-sha512-opencl -mask:?l?l?l?l?l?l?l
../run/john ~/testhashes -form=raw-sha512-opencl -mask:?d2345?d?d?d
../run/john ~/testhashes -form=raw-sha512-opencl -mask:12345?d?d?d9?d
../run/john ~/testhashes -form=raw-sha512-opencl -mask:1?d3?d5?d7?d90123?d5?d7?d90?d2?d4567890
../run/john ~/testhashes -form=raw-sha512-opencl --mask=?u?u?uCAPS
../run/john ~/testhashes -form=raw-sha512-opencl -mask:mot?Arhead
../run/john ~/testhashes -form=raw-sha512-opencl -mask:123456789012345678901234567890123456?d -min=40 -max-l=40
../run/john ~/testhashes -form=raw-sha512-opencl -mask:john?a?l?l?lr

../run/john ~/testhashes -form=xsha512-opencl -mask:?l?l?l?l?l
../run/john ~/testhashes -form=xsha512-opencl --mask=[Pp][Aa@][Ss5][Ss5][Ww][Oo0][Rr][Dd]
../run/john ~/testhashes -form=xsha512-opencl --mask=boob?l?l?l


@cjolif do you want me to update it?


@norine-badr


Fixes #6


According to some developers this still makes sense ;)


Why wasn't this necessary before?


Discovered this bug while testing my own code using the httpAuth middleware. This can be reproduced by using multiple credentials, and attempting to log in with a set of credentials that is not the first item in the array. The line:

    allowed |= options.credentials[i] === request.headers.authorization

is using a bitwise OR operator (not logical) and so allowed is becoming a 0 or 1 (not true or false). Using the strict equality operator (===) in the for loop is then resulting in a comparison of 0/1 === true/false, which will always be false, resulting in a premature exit from the loop after the first iteration.

If you attempt to log in with the first set of credentials in the array, the loop is still exiting prematurely due to the allowed === false being evaluated as (1 == false), but since allowed is 1, the code after the loop works correctly.


rebase 1-url-error onto develop1fb9fd41feb500d93c38fbb21ac78ddfb94efc4b


The default doesn't work. According to RawGit, "Files are cached permanently after the first request" for `cdn.rawgit.com`.


yes, looking at some ruby best practices this is what most people advise.


better! thanks


in this case if it was a path it would return `undefined`.


I can remove the second and third test, but they are parsing different directories, and checking to see if they were parsed correctly. cmdExec.SetOutput() is inserting the different directory strings.


Not 100% convinced someone would actually want that, but sure :)


haven't used guard, but i've used listen and it seemed to be OK. sounds like a fun project!


Because something else that we depended on transitively depended on it.


I have a fix for it that I haven't been able to fully test yet, I added a
forgetInstance to the teardown
On Apr 3, 2015 9:53 AM, "Andrew Embler" <notifications@github.com> wrote:

> Something about this test is breaking tests down the line. It supposed to
> reset the URL (which would then make the tests resolve to
> http://www.dummyco.com/ – but it's sticking with derpco.com. Check the
> latest travis ci build.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/concrete5/concrete5-5.7.0/commit/796ed009208209123ff827ea9abe7fbb40bd2f36#commitcomment-10565711>
> .
>


+1 thanks for providing a fix for this


TeamCity Ybackend :: AnnaPonomareva/ybackend [Build 289](http://192.168.30.225:8111/viewLog.html?buildId=1071&buildTypeId=Ybackend_AnnaPonomarevaYbackend&guest=1) is now running


yeah yeah, I've switched over already. will update ASA-I can be bothered


TeamCity Ybackend :: AnnaPonomareva/ybackend [Build 290](http://192.168.30.225:8111/viewLog.html?buildId=1072&buildTypeId=Ybackend_AnnaPonomarevaYbackend&guest=1) is now running


Ah, good catch.


Yes, please.


@thorncp What I was referring to was a situation like: someone changes the code on the worker side to send more attributes, then they would need to know to update this line, otherwise this will raise.


crc32c.calculate(new Buffer(computed_ip_in_hex_format,"hex")) works too. It's not really more simple right now with the current code aligned on the c++ one, but I will modify it and revert to this.


Krag and NathanKell have a point.  With a reflection based approach, custom PQSMods could be implemented in secondary DLLs and it wouldn't require code changes from us for them to be supported.

> On Apr 3, 2015, at 7:05 AM, Thomas <notifications@github.com> wrote:
>
> Just got my solution for multiple mods working and tested it with your PFOblate Krag. ;)
> http://imgur.com/a/pLZvb
> Works fine, I think. ^^
>
> —
> Reply to this email directly or view it on GitHub.
>


Reference:
http://www.mcbbs.net/forum.php?mod=viewthread&tid=89089&highlight=highlands


My intention was to warn about "expected" failures that we don't really need to know about. The lines outside the try/except should not fail, and if they do I want a traceback. But the lines in the try/except are debatable. Detector probably should always have a TAN_PIXELS transform. As to the cast to TanWcs: a better solution is to construct a TanWcs from the wcs, but the present API doesn't support that well (though it can be done), and the WCS will likely always be a TanWcs, so I felt this solution was an acceptable compromise. I'll pull the TAN_PIXELS out of the try/except.


手残一个回车发出去了...


Woops! :+1:


The [Rule of Three](http://blog.codinghorror.com/rule-of-three/) probably applies here:

> To build something truly reusable, you must convince three different audiences to use it thoroughly first.

If you look at the examples in the DHH post they're things like `Trashable`, `Searchable` and `Movable` — mixins which apply to multiple models.  If you can see `BlocklyExportable` used as a mixin in multiple models then it makes sense as a concern.  Otherwise I think it makes more sense in the model.

DHH does mention they only mix the `Dropboxed` concern into a single `Person` model, but I don't think `BlocklyLevel` qualifies as a nonessential slice of the `Blockly` model.  `Dropboxed` adds an orthogonal behavior to *any model* whereas `BlocklyLevel` is a representation of core fields in *one specific model*.


I'm going to create a new issue: this discussion is starting being too much for a simple commit notes...


The PUP vulnerability only happens with /dev_bdvd/PS3_UPDATE/PS3UPDAT.PUP because all game discs include that folder. The remapping for /PS3/UPDATE/PS3UPDAT.PUP is not necessary and it breaks a feature in webMAN MOD that allows to install (CFW) PUP stored on the root of /dev_bdvd.

Please change line 187 to:
if (path && (strcmp(path, "/dev_bdvd/PS3_UPDATE/PS3UPDAT.PUP") == 0))


@mfischer can you try this with your ARM + pmt_smasher bug?


With my Parser-Patch, this is possible, we wouldn't have to change anything besides of that, but Krag and Nathan would have to write their own wrapper-classes for their mods.

But you're the boss. :D

----- Ursprüngliche Nachricht -----
Von: "Nathaniel R. Lewis" <notifications@github.com>
Gesendet: 03.04.2015 19:34
An: "BryceSchroeder/Kopernicus" <Kopernicus@noreply.github.com>
Cc: "Thomas" <dorianstoll@googlemail.com>
Betreff: Re: [Kopernicus] Mods, Mods, Mods... (7293861)

Krag and NathanKell have a point. With a reflection based approach, custom PQSMods could be implemented in secondary DLLs and it wouldn't require code changes from us for them to be supported.

> On Apr 3, 2015, at 7:05 AM, Thomas <notifications@github.com> wrote:
>
> Just got my solution for multiple mods working and tested it with your PFOblate Krag. ;)
> http://imgur.com/a/pLZvb
> Works fine, I think. ^^
>
> —
> Reply to this email directly or view it on GitHub.
>

—
Reply to this email directly or view it on GitHub.


TeamCity Ybackend :: AnnaPonomareva/ybackend [Build 289](http://192.168.30.225:8111/viewLog.html?buildId=1071&buildTypeId=Ybackend_AnnaPonomarevaYbackend&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:09:33


Thanks for the fix. :+1:


this syntax is verbose but not that bad


TeamCity Ybackend :: AnnaPonomareva/ybackend [Build 290](http://192.168.30.225:8111/viewLog.html?buildId=1072&buildTypeId=Ybackend_AnnaPonomarevaYbackend&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:09:30


That does look nice, I'll build the docs as soon as i get some free time


Have more coming, to print name entries as real Unicode text in the XML output.  Possibly, even load them as unicode strings in Python, but that might be disruptive to scripts.  Probably will add asunicode() or something...


this is really janky. probably will fix with some kind of viewport-dependent diagram abstraction.


It looks bad in the diff view, but the line looks good in the project main page and in the raw view of the README.md file.  A bug on github?


doh.. yea, habit ;d thanks!


ca prend un alt pour tes images `[Case crea....](imageurl)`
le nom du dossier en minuscule


Am I missing something? There's nothing in there stating using data-* as a css hook is a bad idea. It's just another HTML attribute. If anything it keeps the markup simpler, since a bunch of classes won't need to be made to make minor tweaks.

The only downside I see is if you plan on changing the data-hook values frequently, then you would also need to modify the css also.


@eteq Does this fix the issue you created ?


I tried that, I must have messed it up before. Thank you, it works.


This what we talked about today; just a reminder to remove the fixme.


Looks good, although, is this specific throwing setter only for throwing this custom exception object? Alternatively, you could just leave the setter as `nullptr`, and the NPH will throw automatically treating this property as a read-only.


Thank you for making a release 7.3!  Would you be able to help create a new release 7.3.1 including recent bug fixes?


Et ton url d'image est pas bon `/images/post/...` au lieu de mettre dans le dossier post tu pourrais créer un dossier pour ton post ie: `/images/2015-03-31-coveoSupportBehindTheScenes/caseCreationSearch.png`


Looks really nice to me too, ditto re: free time.  Almost done grading these midterms.


alt + url


MarkdownDocs (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1683 of 1683 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


I see it [without identation](https://cloud.githubusercontent.com/assets/365089/6986038/e45453ba-da52-11e4-90f8-4dd810fdb6f2.png) too.


You are correct, never throw from constructor


What does this change?


See #2166


ok, I'll revisit this if it is really needed


TeamCity MonoGame :: Develop (Win) [Build 3.4.0.133](http://teamcity.monogame.net/viewLog.html?buildId=5994&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


MarkdownDocs (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1677 of 1677 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


It is not important. But I would expect this:
```js
client.on('error', function (err) {
    var newErr = new NestedError("Failed to communicate with localhost:8080", err);
    console.log(newErr.stack);
});
```
instead of this:
```js
client.on('error', function (err) {
var newErr = new NestedError("Failed to communicate with localhost:8080", err);
    console.log(newErr.stack);
});
```


Seems like we should make an effort to respect the abstraction put in place by ActiveMQ (or JMS?), namely that connecting to a queue is an operation can fail. I can't tell you what we should *do* about that; the code below honestly seems fine to me, lacking information about the specific failure/repair cases for something like a remote queue.


Added some deep objects to make it interesting https://github.com/lodash/lodash/commit/32ef1bb7a5f6d395e56f48e46e0d1ddf5b443b2f.


the jmp_buf for windows needs to save a lot more registers, including MMX. in linux, it only needs long int alignment to save a few registers. on darwin, it only demands int alignment.

this doesn't seem like it should waste a full page, just `16 - (sizeof(jl_taggedvalue_t) % 16)` bytes at the beginning


Probably need a reworking for the intro.

ill just change some formatting : `I'm the second...` -> `I was the`
`objective of specializing in the support of our Coveo for Salesforce product.` -> `objective of supporting the Coveo for...`


@norine-badr


you forgot to move it to external writeup only fixed here de1afd62907c5a67842427a29ae3d98aeeebb331


okay :) unblock me on g+ maybe?


I'm not sure if we should flat-out remove the function; there are almost certainly scripts out there (for better or worse) depending on `source.with.encoding`. I think it makes sense to leave it there to ensure those scripts can still run.


If we're going to do this, we need to figure out a better way to match by name, since names can be slightly different across different sources. Two ways to do this: either store the possible alternative names (acronyms, slightly different titles), or do fuzzy matching. I recommend a combination of the two.


Good idea.


MarkdownDocs (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg)


Should change the staging specific rules to ensure that they are not present.


Gotta get 'em all!


This is going to fail on windows. Why not something like (pseudocode):

```
try {
    exec ("nodejs");
}
catch (e) {
  exec ("node");
}
```


Possible?  I squashed some commits, and maybe git ignored some whitespace changes (not what I would prefer it to do though...maybe a bug, or maybe a setting somewhere that I didn't know I had)


I'm not sure why this would need a fix, this was intentional to me..?


2 line breaks if you want a new paragraph


Thanks for all the updates!

One thing I forgot to ask. Did you create/update the unit tests for the updated() method?


The problem with this method (and the reason why I didn't do this in the first place) is now you can't aim diagonally unless you're moving. When you let go of the keys, you always let go of one of them first, so whenever you can only face 4 directions. Maybe we can make it so the direction responds directly to the controls, but it changes smoothly?


Fixes one bug and adds another..


Thanks. Fixed.


You are right, I was looking at the wrong line.  Thanks not giving up on convincing me =)


` Now, let’s start by the beginning shall we?` wasn't this the beginning ?


why is this commented out?


mention in commit is :imp: I will have my revenge


lgtm


If it makes review easier, it probably makes maintenance easier as well :)


You can remove these comments. `impl Copy` basically implies it's safe to copy the object this way. In fact it'd be weird to have a `Copy` type with a `Clone` impl differing from this one.


OK, indeed, new Buffer(ip) just works fine since ip is an array of bytes...


6 to 18 is outside business hours ?  isn't it the contrary 18h to 6 ?


+1

I thought I caught that..... Oh well.


this is a friend of a friend check and you are simply checking if author is friends with the list of friends the request provides.


"authors" thats not a key in the json


The `docker-linux-386` target creates a file named `lantern_linux_386`, I'm copying this file to `lantern_linux_i386` (note the `i` before `386`), I did this to be able to pass only one variable (architecture) to the `fpm-debian-build` function, Debian prefers to see `i386` with the `i` instead of just `386`.

So, for packing the amd64 Lantern I'd use:
`$(call fpm-debian-build,"amd64")`

While for 386:
`$(call fpm-debian-build,"i386")`


You can drop the braces when the list shrinks to one item.


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/13603/testReport/) for this commit**


what are you checking here?


TeamCity MonoGame :: Develop (Win) [Build 3.4.0.133](http://teamcity.monogame.net/viewLog.html?buildId=5994&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 469, ignored: 6 Build time: 00:14:12


That's not really too much of a problem, is it? Depending on the pace of the game, the player will probably always be moving, that or they won't be so on top of everything that they won't bother to try diagonal aiming anyway. The whole "move to aim" thing feels clunky anyway. I still think just making separate controls for movement and aiming/shooting would be a fine option.


This feels like a "with-mq-transaction" or something.


Any idea on when you'll bump the version and republish to npm?


putting `Right off the bat` at the beginning of the sentence might not be the best choice
`here is some information that any product specialist likes to see,Right off the bat,  when he is assigned a new case.`


This bit of indirection (`connection-schema` -> `java.jms.Connection`) feels unnecessary; I had to go look it up.


Just a heads up, you'll want to use `l.custom?` here when resolving conflicts.  (See: https://github.com/code-dot-org/code-dot-org/commit/13994b02a4a48c00de505a95728db694a3127bb6#diff-1f68e16ab551449bdbd0fb7f01004f68L264)


`given the brackets are adequately replaced.` don't think we need to say this this is a technical blog, people are used to this on technical blogs


No, but I don't think I changed that method, so the existing unit tests should still be valid.


GRR!  I just rebased away my shame on that one.  :-)


see *the*


Its a start @mattwynne, but unfortunately `@title` is used in the included module Names, so this change by itself breaks cucumber/cucumber, all the ast classes using the module Names need to be updated together (fixed in https://github.com/cucumber/cucumber-ruby-core/commit/ea29026eb7b9db1c428fffea3b489757845d654c)


One typo - otherwise :+1:


@radare so, what about this PR? is it useful or not?


Related issue: #7


The verbiage for Zabbix and New Relic look good, though I'd recommend being a little more specific for New Relic and calling it a "hosted service" instead of just a "platform".


Please make the GoTo capitalization consistent. I prefer Goto myself.

Also, I just pushed to BuildCraftCompat my idea on adding custom harvest handlers from BCC's side, starting with AppleMilkTea2 -> https://github.com/BuildCraft/BuildcraftCompat/tree/master/common/buildcraft/compat -> as you're the robot man, you should look through this... I was considering adding interfaces for custom harvesting and planting handling to BC itself myself, though, with the custom harvesting one returning an AIRobot to handle the harvesting type.


You could just do
`var client = new Riak.Client(['riak-test:10017']);`
here, unless you're trying to show what's behind the curtain.


Nvm. I lied about broken github username


91244204-HeatExchangerAirToAirSensibleAndLatent-Supply-Air-Outlet-Temperature-Control-problems-#4794 (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1167 of 1169 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.83%25-yellow.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-69.8%25-green.svg)


@basmaelgeziry approved


Related issue: #19


Related issue: #35


Should we add `Additionally, ` to the beginning of the sentence, to not make it sound like that's the only place we store the token?


Passive voice! Which is actually a little confusing... but I see by the code that this does indeed commit the message.

What happens if we `kill -9` the process after pulling messages off the queue but before they've been fully processed?


@cntrlwiz @ShaunOfTheLive @torf

You three have the most active forks that I can tell right now, so I would like your opinion on this implementation. In #19, @cntrlwiz and I discussed subcommand blocking and the best way to implement that. I came up with the idea of implementing it like this, where you can do "record clear" and it would block "record clear" but not "record" or "record date", etc. I also added in a feature where you could do "record *" and it would block every subcommand of, but not record itself.

I haven't done much testing on this yet, which is why it's on the nightly, but I wanted some outside input on this from the people it would be affecting. What are your thoughts on this?

Also, anyone else who uses this, feel free to comment your thoughts or potential changes to this method.


Liknk to (PostgreSQL) http://www.postgresql.org/?


This is a simple change :+1:


This change caused https://github.com/mapbox/node-mapbox-gl-native/issues/109

/cc @springmeyer


Oh I see.  The type parameter on `Optional.empty()`.  You're right; it isn't needed.


I think this doesn't work. If the protocol is valid, we start the connection loop here, which keeps running until the connection gets terminated.
Hence the SUCCESS will never be printed.


@billmeek I think these changes in this branch should fix this issue.  The Shows screen should now be pulling from the programs table instead of the title_infos table. Now, when you click on one of them, it passes both the title and the inetref (if it exists).  If only the name exists and the inetref is blank or null, it should be appending that criteria to the query so that only those programs matching that case get used to return just what is needed for this line.  You can play with these lines marked here if it is not working for you.  Let me know how this goes for you.


I found the [original patch](http://paste.isomorphis.me/YmD&ln) of this. The first problem is the exception `Ocsigen_Is_a_directory` is used in `Ocsigen_extensions` (l. 190 of original patch) and in `Ocsigen_server` (l. 290 of original patch) - and in `Ocsigen_local_files`.

The second problem is the type of exception. The old type of `Ocsigen_Is_a_directory` is `Ocsigen_extensions.request`. So, `Ocsigen_server` is dependent with `Ocsigen_extensions` by the exception and by the type of the exception. In this case, it's no possible to extract `Ocsigen_extensions.request` (the type of the exception) from `Ocsigen_extensions`, it's too complicate and there are just a little part of this record whom is used by `Ocsigen_server`. This part is `Ocsigen_extensions.request_info`. So I decided to get off this type in the new module `Ocsigen_request_info` (best module ever) and change the type of `Ocsigen_Is_a_directory` to get off the dependence of `Ocsigen_extensions.request` in `Ocsigen_server`.

Now, `Ocsigen_server` has just need to apply the function of this exception without know `Ocsigen_extensions.request`.


Initializing this as `source(v8::Persistent<v8::Object>::New(source_))` fixes #109, does this look alright @springmeyer?


Looks good.


In some cases, I find it cleans up code to import qualified names of particular types / functions. If you're only using a couple things from a module, often they don't clash with the other "couple things" you're using from other modules. And the code reads a bit better when not every single value or type is qualified (IMO).


Potentially a problem becaues it would change with the name of the folder the package is in?


zut : (GPL ? CC-0 ?)


Wouldn't this also waste a full page because of data at the end ? For e.g. a lot of 8 x 2k objects (gc) pages you'd either have to restrict every one of those to 7 objects or waste (most of) the last (vm) page right ?
Could we just treat the jmp_buf as a special case ?


eh?


@basmaelgeziry will you change the <<welcome >> to be <<view>> and add "welcome/" to the homepage.html.erb


@sradu
I would suggest changing this to:
npm install to install dependencies

because npm install ejs express does not install the full set of dependencies.


need to close this factory somewhere, maybe in the finally block.


I followed the pattern of the existing getIntegerPreference method. But may be you're right, yes. What kind of exception would be appropriate? InvalidArgument?


Посмотрите Font Awesome :-) http://fortawesome.github.io/Font-Awesome/


MarkdownDocs (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1683 of 1683 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


Hopefully fixed now.


Isn't there supposed to be a pushback on the attacked as well?

 "[The Dragon Spear's Special]...Shove, which pushes an opponent back and stuns them for three seconds."


Prefer 'auto' here ;-)


LGTM


MarkdownDocs (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1683 of 1683 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


nit: indentation


r=jdm


No, it's good: this is the version I publish to crates.io. They may diverge over time, we'll see.


I like longer names.


To be clearer: I understand q and sess, since those are common, but 'pro' and 'con' are surprising.


@armonr @jsutterfield


Git blames @chris00 :-) 8c7824caba8bfd331621bc5e3ee7c38f821dac2e


Alternatively, `NanAssignPersistent(source, source_);` here would work too.


@GavinNishizawa what was your thought process re: including the entire rit_object.py file?  So people who took the old CS1 know what it is?


`shutdown` is unused.


> Turns out, most of the challenge is that there's no obvious way to know whether a class, id or certain HTML tag types are being used by the JavaScript or not. So someone working on design and markup can inadvertently break the JavaScript app and vice versa.

The point of the data-hook is to separate JS and CSS references in the HTML.


How to identify the specific tunnel encapsulation type using this? More specific tunnel types like 6_in_4, 6_in_6, 4_in_4, 4_in_6 should be defined. Alternatively, we can have the new attributes "SAI_TUNNEL_ATTR_PAYLOAD_TYPE" and "SAI_TUNNEL_ATTR_PROTOCOL_ID" to set the ip payload type (v4/v6) and the protocol number in the outer header (4, 41 etc) respectively so that SAI implementation can find out the specific tunnel types.


There's a small glitch with a shadow element.


This is because the formatter on Rails.logger was already extended with the module. Setting a new formatter directly, no luck.

The typical way is to use the `config.logger = ActiveSupport::TaggedLogging.new(SyslogLogger.new)` route. That wraps the logger and extends the formatter.


I think SAI implementation needs to use the SAI_TUNNEL_ATTR_SIP and SAI_TUNNEL_ATTR_DIP attributes to setup both encapsulation and decapsulation on a tunnel node (assuming bi-directional) always. Will it be useful to have separate attributes SAI_TUNNEL_ATTR_ENCAP_SIP, SAI_TUNNEL_ATTR_ENCAP_DIP, SAI_TUNNEL_ATTR_DECAP_SIP, and SAI_TUNNEL_ATTR_DECAP_DIP? User can be able to create special cases like receive-only tunnels which can be set with just the DECAP SIP and DIP.


Nice work.

A nice to have would be if you could stick the whole m4_esyscmd_s in an external script, like http://git.savannah.gnu.org/cgit/autoconf.git/tree/configure.ac#n22


because the i is pulled from ->resume in the FT struct and if resume is == 0, we don't resume


@houkanshan

Sorry >_<

Fixed!


Shouldn't it be
`nodes = numpy.array([(cent - gap/2) - width/2, cent - gap/2, cent + gap/2, (cent + gap/2) + width/2])`


MarkdownDocs (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (661 of 661 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-5.02%25-green.svg)


@pgkirsch, your editor is inserting tabs -- we want these expanded into 4 spaces. Check your editor settings -- in vim this is controlled by turning on the `expand tab` setting.


Yeah that *might* be a valid point... ***if*** I had changed the interface. I haven't the signature of `createAccountInDatabase` has not been changed. Also, this is in the OAuthModule not some sort of API!


Almost certainly not rethrow; that'll bubble up and kill the thread, right? I think catching it here and logging is _probably_ the right thing to do, though I would want to test it with a synthetic exception to make sure that's really the behavior we want.


@beeselmane He just, without any investigation, moved two files...
Made the links and the ENTIRE SYSTEM not work :D


If i add provided scope back, I get this warning in WF https://gist.github.com/Jiri-Kremser/4d358142dad00912fb7c

the server seems to work well, but it looks ugly.


Ok, I will move this to the new_extension PR.


Thank you for the clarification. From the Instructions section of the [upstream repository](https://github.com/mikekelly/hal-browser).

> All you should need to do is copy the files into your webroot. It is OK to put it in a subdirectory; it does not need to be in the root.
>
> All the JS and CSS dependencies come included in the vendor directory.

The HAL Browser is a SPA and consists of HTML, CSS and a JS library with some dependencies to other ones. The current behaviour of copy and paste the files should not be broken, i think.

One possibility might be to patch the main `browser.html` file while generate the Maven artifact. The patch should be included in the webjar repository and processed by [Maven Patch Plugin](https://maven.apache.org/plugins/maven-patch-plugin/). All dependencies to 3rd party libraries are replaced by load require.js and added as Maven dependencies to webjar `pom.xml`. As consequence each update to a new upstream release must include a proper patch file and up to date dependencies via webjar `pom.xml`.

Is this possible? Thanks in advance.


10e8 * 2 * 10e-10 = 2 * 10e-2 = 2/100s = 20ms, no? I'm a little confused with all the units, sorry.


I think this wants to be

    importData.plates[plate].rows[plateRow].columns[plateColumn] = { labels: {} };


it's actually not just a display value, but i agree this was not the correct fix.  the account hash needs to change for demo mode because some data is keyed from that.  but this demo account ('0xDEM0') is actually in the demo.js, where it should be.  we just need to override the real address in the case where a ethereum client is running but the contract fails to load and we still want to demo.


And this wants to be

    importData.plates[plate].rows[plateRow].columns[plateColumn].labels[category] = value;


retested with Windows 8.1 pro (x64) and Firefox 36.0.4 (and Waterfox 36.0.4) and the problem I had with086ceeeb7ee33e51d3bc69be078db9cda82bd3fd still occurs.
very strange. I'll tests more further (rebuild gateone, restart the service) and keep you informed of the results.


LGTM


hopefully this demo nonsense will become less important soon.


We could, but why would there be no build.sbt? It's part of the repository: https://github.com/lampepfl/dotty/blob/master/build.sbt


Please read the document I linked in the first reply. I doubt no would run
into this problem if they had read the node.js document throughly.

By the way, your code is still looks odd in my point of view. Considering
IPv4 is uint32, and crc32 map this space into another uint32 space, which
means no data is digested. Or to be more precise, there is a huge
possibility that crc32 maps different IPv4 address into the same one.

And masking IPv4 is also looks strange, why not [convert](
https://github.com/ashi009/bestroutetb/blob/npm/lib/prefix.js#L22) address
into its uint32 format? Then you can do it in one line of code?

ip &= mask;

Afterwards you can use buffer.writeUInt32LE(ip) (please check the method
signature) to get the buffer you need.

On Sat, 4 Apr 2015 at 1:57 am Aymeric Vitte <notifications@github.com>
wrote:

> OK, indeed, new Buffer(ip) just works fine since ip is an array of bytes...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Ayms/bittorrent-nodeid/commit/0644f68d2b9e99569e842f864a083a6d39436a4c#commitcomment-10566598>
> .
>


I actually intended to open a pull request but accidentally pushed to master. Raise you voice here if you see any problems with these changes.


I believe this introduced a bug for calling anything from a derived class that uses the static keyword.  In my case, `forceCreate` tries to instantiate the abstract class instead of the derived model class. I think this is because it is executing within the closure and has lost the necessary context.


I do not understand: the `setup.ml` does not use dynamic mode, thus `oasis` is not required (but you may want to use `ocaml setup.ml ...` and not `make` to compile and install the package).


This means that port == stream. This isn't true. I think we would need to re-think the design of the serialization/deserialization of the address manager to take into account the existence of streams. A possibility is to have change the structure of serializedAddrManager so that it can handle multiple streams. Maybe add another struct called serializedStreamInfo that includes Addresses, NewBuckets and TriedBuckets as its fields. And then serializedAddrManager could contain a slice of type []*serializedStreamInfo which contains all this information for each stream. What do you think? @justusranvier Do you have any suggestions regarding how to handle this?


I never suggested that `BlocklyLevel` would be at all "reusable" in its current form, the point was merely to "slice up chubby models" by dividing the complexity into functionally-differentiated components. I want to avoid lumping the `ActiveRecord::Base`-oriented functionality together with hundreds of lines of code concerning a JavaScript transformation that is entirely separate from the database-modeling part.

My bar for when to introduce a Concern is not whether a component can be reused in more than one model, it's whether it's possible to carve out a logically self-contained "single slice of responsibility" from a complex object serving multiple purposes. So with this concern I'm just suggesting that we keep the "model the level database table as a Ruby object-graph" responsibility separate from the "export the level-object properties into the format that the Blockly runtime requires" responsibility.

If the use of `Concern`/`include` is your main issue with this approach, I can rework this in a less-Rails-y way using a stand-alone class. I don't want to just add this big chunk of code into an additional `level.rb` or `blockly.rb` method directly on the model class - in fact, I'd eventually like to see us separate out much of the XML import/export functionality from the core `Level` graph even further to make the whole hierarchy even more manageable.


Oh right, I'll change my settings. Thanks for fixing it!


No.

That's a perfectly valid definition, but not the one I find most natural.  The width is the total width from the first non-zero response to the last.


I feel framed.


I know it's a prexisting issue, but the fact that with-test-broker binds the `connection` var in its body was a little startling. Might be better if it looked like a binding form:
`(with-test-broker "test" [connection] ...)`

Or if it just took a function as the parameter.


Those macros are getting kind of unwieldy. Might want to format them.


I disagree. It's not a documented API and users can change their scripts to use the source function. I'm also not sure whether there are in-fact very many of these scripts as they'd be depending on a function that is *only* available within RStudio (not a great place to be, we might actually be helping them understand and remedy this dependency).

That said if you guys feel strongly I'm fine within leaving it in.


Up to you, but this could also stay

    this.experimentFeatures = { labels: {} };


This uses the WikiFont move icon instead of the Font Awesome paper airplane. We can change that back again, I just did it for consistency in the PHP.


Consider following [the spec] (https://github.com/bjorn/tiled/wiki/TMX-Map-Format), which states `id` is the local tile ID within its tileset, and `gid` is the global tile ID.


TeamCity Azmq :: Build (OSX) [Build 69](https://78.46.71.115/viewLog.html?buildId=451&buildTypeId=Azmq_BuildOsx&guest=1) is now running


TeamCity Azmq :: Build (Linux) [Build 85](https://78.46.71.115/viewLog.html?buildId=450&buildTypeId=Azmq_BuildLinux&guest=1) is now running


TeamCity Azmq :: Build (Windows) [Build 111](https://78.46.71.115/viewLog.html?buildId=452&buildTypeId=Azmq_Build&guest=1) is now running


TeamCity Azmq :: Build (Linux) [Build 85](https://78.46.71.115/viewLog.html?buildId=450&buildTypeId=Azmq_BuildLinux&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 19 Build time: 00:00:07


TeamCity Azmq :: Build (OSX) [Build 69](https://78.46.71.115/viewLog.html?buildId=451&buildTypeId=Azmq_BuildOsx&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 19 Build time: 00:00:11


TeamCity Azmq :: Build (Windows) [Build 111](https://78.46.71.115/viewLog.html?buildId=452&buildTypeId=Azmq_Build&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 19 Build time: 00:00:20


How much contention is there in practice when manipulating the block list?


Is this expected?  I thought we checked if they were a salesforce user before calling ```enrollment_submitted_crm_actions``` ?


Tip:
flush(); exit(1);

This will mimic the default behaviour of php in HHVM inside error and exception handlers.


@mikemorris yes, using a `v8::Persistent` is correct: otherwise the value needs a handlescope to get collected - so I'm not surprised it crashed without being a persistant.


We have done very minimal patching of WebJar contents.  It becomes a bit of a maintenance burden but might be the only option.  Ideally the upstream project would move to a more standard dependency loader (AMD, RequireJS, etc).

Here is how we did this for the jQuery WebJar: https://github.com/webjars/jquery/blob/master/pom.xml#L100-L121

I'm not totally opposed to just adding the `vendor` directory back.  It isn't how we usually do things but it might be the least worst option.


`timeout.close()` does not modify `_repeat`, only `clearTimeout` does.

On second look, I guess it could still be reached. Oops.

`if (!timer_repeat || !timer._onTimeout)` may be more comprehensive; in case someone actually tries to call `close()`.


@miffedmap I'm not sure I understand the reason behind this change - `dirname "$0"` works for me, and BASH_SOURCE won't work in non-bash shells.


ok


[1] Can we have additional CREATE_AND_SET attributes SAI_TUNNEL_ATTR_DSCP and SAI_TUNNEL_ATTR_IPV6_FLOW_LABEL for setting DSCP and IPv6 flow label in the outer header?

[2] Can we have an attribute to set the Next Hop list for reaching the tunnel destination address which could be multiple hops away? There can be single or multiple next hops (ECMP) in the transport network for forwarding packets to the tunnel destination address and the list can have single or multiple next hop object ids.

/* Next hop list for reaching the destination address at tunnel entry point [sai_object_id_list_t] */
SAI_TUNNEL_ATTR_NEXT_HOP_LIST,

Application can add or remove next hops to the next hop list when there are changes to the route resolution for reaching the tunnel destination address.


(I really hope someone sees this...)
I used to pull the whole numpy into the name space via `from numpy import *` instead of traditional `import numpy as np` and this resulted some strange errors in minimize function from SciPy. I tracked them down, but it seems like the whole class is not going to work, since once I fixed them, the new similar once poped up. I decided to switch to classical use of import without overflooding the namespace, which I suspect is responsible for the errors. The following describes the original problem.

The np.isfinite assert does not work with sympy floats, as the result you get an error even if xmin is actually finite.

Working example:
------------------
import sympy as spp
import numpy as np
x = spp.numbers.Float(1)
np.isfinite(x)

Fix:
------------------
import sympy as spp
import numpy as np
x = spp.numbers.Float(1)
xx = np.float(x)
np.isfinite(xx)


Is this just simplifying it so that you don't have to pass in a parameter every time you call, and it'll just use `self.logo_paths` in place of additional_permitted_paths?


Done :)


What does `GUARD` mean?


Add documentation for `IPRange#split`.


TODO: Assert that there are two new ranges after deleting the existing IP.


I think this should be a `!` function... core.async's `>!` is a pretty good indication that queue writes should use that convention.


it is not, because the current way to detect if an instruction is valid or not is by checking a string in the disasembled code. this requires a refactoring with lots of tests (will break pd -x) so i prefer to delay this feature for 1.0


Member está escrito errado.


you got it


Hello, i have a question...is it possible to sign the hash externally, thus without providing the private key to jXAdES. I have a situation were i need to sign the digest on a smart card but prepare the xml signature externally of the card.


E


I think it's so we can run our code to make sure it works. This commit doesn't seem to add it to the LaTeX file itself.


Hi,

The individual binary payloads are encoded the same on the unix socket as on the wire, yes. For bi-directional connections (i.e., unix sockets, and eventually TCP and TLS sockets), we didn't want the dnstap / Frame Streams sender to immediately start dumping the dnstap output into the socket unless it was sure it was connected to a dnstap receiver, so there is an extra handshake protocol between the sender and receiver at the beginning before the payloads begin to flow. This handshake was changed between `fstrm` 0.1.0 (and the current version of `golang-framestream`) and `fstrm` 0.2.0.

Basically I need to update `golang-framestream` to use the same protocol used in, e.g., the `fstrm_capture` utility (https://github.com/farsightsec/fstrm/blob/v0.2.0/src/fstrm_capture.c), or `libfstrm` proper (https://github.com/farsightsec/fstrm/blob/v0.2.0/fstrm/writer.c), and to document the actual handshake that occurs. Apologies for not getting to this before, I really should have gotten to it by now.

BTW, there is a public mailing list for dnstap as well: http://lists.redbarn.org/mailman/listinfo/dnstap.