If that's a reference to an explanation of the reason for the change, it is somewhat vague, apart from a statement about patents and a statement about "who owns what".  The "and so forth" means nothing to me.

Rather than clutter up the contributor survey there, I'll share my thoughts here:

1. There are more widely compatible licenses than the MPL that deal more explicitly with patents than the Revised BSD License.

2. MPL itself does not provide any guidance on "who owns what" beyond the absolute necessities of the license's restrictions, so far as I recall (or notice on skimming the license text once more).  What it does do is take a little time to explain how the MPL interacts with a few other licenses, to carve out some level of compatibility with a few, very specific licenses that are, themselves, broadly incompatible with other licenses.  This is not surprising, but offers no benefit in and of itself over the Revised BSD License, which does not have the same compatibility problems, and thus does not need that explicit treatment.

Apart from that, I am not sure why you recommend reading #3550.  As you can see, I am not picked out as part of the list of GitHub contributors.  I think I did contribute to some documentation at one point, but I'm about as close to certain as I can be (without some serious research) that it has been obsolesced in the years since, so you should not need my permission for any relicensing as a contributor.  A recent renewed interest in Rubinius might have prompted some contributions, but I have not made any yet, and am now unlikely to do so given the constraints of the new project license, so no worries about me, as a contributor, having any legal standing on the matter of relicensing.


Thanks.  Unfortunately I ran into more trouble with x64.  Should I post the log (200k) or is there a better place for me to post?


ah bai, Danobat-ekoak, blogean idatzi zenuten bai :)

Eskerrik asko

2016-01-22 16:24 GMT+01:00 ierrasti <notifications@github.com>:

> DANOBATGROUPen. Zuk idatzitako blog sarrera bat irakurri ostean hasi ginen
> Cabot erabiltzen.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ierrasti/cabot_alert_telegram/commit/53dcc93ba0095b505929b439952a2c584a53e84b#commitcomment-15615438>
> .
>

--
Mikel Larreategi
mlarreategi@codesyntax.com

CodeSyntax
Azitaingo Industrialdea 3 K
E-20600 Eibar
Tel: (+34) 943 82 17 80


Hi Steffen, thank you for contributing/sharing this.  I have been following your progress with interest and borrowing liberally for a project of my own.  I wanted to ask you, is your intent to add unix file descriptor support to dbus-sharp?


@stevewest @frankdejonge Any idea what the rationale behind this change was?

The original removed double slashes and `../` parent directory references from the URI, the new code simply strips all dots when followed by a slash, which I think is not what should happen here. Am I correct?


Image checked :)


This code block still contains indents.


Thanks for fixing this error!

Have a nice day!


Check the already opened issues, might be already covered with a fix somewhere but if not post a new issue.


It is too early for our scenario since we initialize the plugin very early on in our app, but also have checks to see if a user's location has been turned on (using the new IsLocationEnabled method) and provide some prompts before then calling OnAppStarted.

If you think this is going to be an issue for other users we can take this interface method out and figure out a different solution.


Wow, this is old code. And I am not sure I really see the advantage here, has an issue come up regarding this?


its ok .


What is wrong with "suffixes"?


:+1:  LGTM


I prefer to insert one empty line between text and code block.


// dsds


Could we drop the extra `<p>` tag here and point the styling at `.find-nearest { label { ... } }` ?


T won't be in scope here?
Can we really get mixed types in the same set of values?


테스트


Will test as soon as I get near the Mac again. Maybe @jcome or @genete can also help? ^__^


Yeah, there's an issue posted on the forum, where someone needs to use a URI like `/controller/method/param1./param2./param3`, and the dots are stripped of the URI segments...


Python 还需要学习，用requests是因为不会处理 Unicode

发送自 Outlook Mobile

On Fri, Jan 22, 2016 at 7:27 AM -0800, "Liu Yuyang" <notifications@github.com> wrote:

不妨用python3 的future （Promise?）来实现下，我来学习一下Orz

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


Merged here.


some lines down:
totalFrames = samples->pkt->pause_burst_ms * m_sinkFormat.m_sampleRate / 1000;


Do we really need this fanciness in such a commonly-used function?
Surely better to keep light (we can always create another function if we get a need to encode into alternate encodings)


In that case unless @frankdejonge can shed any light on this I'd say we go back to the old behaviour. Either that or the person having an issue has to re-think their design a little.


意思是sync失败后，再次sync就行？

发送自 Outlook Mobile

On Fri, Jan 22, 2016 at 7:26 AM -0800, "Liu Yuyang" <notifications@github.com> wrote:

！应该叫长连接更合适些，

我这里看到的都是只要synccheck断开就连接，但服务器会保持连接，只在有推送的时候返回断开。客户立即重连

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


Not really sure why we need to filter this anyway. Fuel URI's don't point to a position on disk, they are controller method mappings, so having a URI like `/controller/../../../../method/../../passwd` would simply not do anything...


Of course.
Podemos falar em pt-BR também.
Acabo colocando o título dos commits em inglês por costume, Hehe
Abraços.


Checked


hm but sourcelink.fsx should handle this?


test


can you explain us this one, please?


Not sure how I am the system manager. Whoops :/

Roisin


I can live with an import step, but we must ask for more opinions on how to handle this; then we would need to document the canonical approach.


啥域名

发送自 Outlook Mobile

On Fri, Jan 22, 2016 at 7:21 AM -0800, "Liu Yuyang" <notifications@github.com> wrote:

域名为什么会这么多呢？我一直很奇怪，请指教

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


I think QDir::toNativeSeparators  beter to use


Why didn't use `skipCheck` here?


the payload of a pause burst is 32 bit


if you test this, please let me know either way


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 221](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=707&buildTypeId=XbimEssentials_XbimEssentials) is now running


Если в этой строке произойдет ошибка, мы получим сообщение о неправилином вводе параметра, хотя на самом деле не открылось даже окно перехода.


Any updates on this? I'm looking to see if this will have long term support or not, I think it's a fantastic feature.


Yeah. And it did.

But something changed
On Jan 22, 2016 16:35, "Matthias Dittrich" <notifications@github.com> wrote:

> hm but sourcelink.fsx should handle this?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/fsharp/FAKE/commit/0d830d65c7d5ab999811e3b78af0242757325f58#commitcomment-15615748>
> .
>


谢谢关注！


@apollo-h Changed the DB path


Eskerrak zuri.


 Ahahahaah. Ainda mais que português é o meu fraco.
 Beleza, valeu mesmo; seu nome está no README.md pela contribuição.
 Alias anunciei hoje essa nova versão. Confere o vídeo no canal :-)

   - https://goo.gl/nfF8rl

----

   - https://www.youtube.com/user/tocadotux
   - http://tocadotux.blogspot.com.br/
   - http://www.tsu.co/tocadotux
   - *Caixa de Ferramentas do Unix*
   <https://github.com/tocadotux/caixaDeFerramentasDoUnix>

Em 22 de janeiro de 2016 13:35, João Batista <notifications@github.com>
escreveu:

> Of course.
> Podemos falar em pt-BR também.
> Acabo colocando o título dos commits em inglês por costume, Hehe
> Abraços.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/gabrielscosta/caixaDeFerramentasDoUnix/commit/57ec8ef51eb586624fbaa3deadd59139dc0789a7#commitcomment-15615738>
> .
>


This is a nice way to tidy this stuff up. :+1:


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 221](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=707&buildTypeId=XbimEssentials_XbimEssentials) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:01:45


\o/ Feels like visiting Graceland \o/


Having `this.subject` feels weird. The dependency chain should be:
```
AppClass --> View --> Accessors --> Marionette
```
A view depending on the AppClass creates a circular dependency which doesn't look graceful.

I'd remove that line and implement the wait in the constructor of `ContactsDetailsView`. When initialized, that view should make sure it's ready to be used. So the transition should be over.


Cela me semble indépendant du Fillon :
https://www.urssaf.fr/portail/home/employeur/calculer-les-cotisations/les-taux-de-cotisations/la-cotisation-dallocations-famil/la-reduction-du-taux-de-la-cotis/comment-declarer-la-cotisation-d.html

Mais cette page n'est pas totalement claire.


I was lazy and just used a generator.


Oh, yeah, you are correct. Sorry about that. There's no issue here.


I mean there is *not* GEODEL. So has no*t* maybe.


Nit: `view.js`


Good catch 👍


@puzrin my bad, thanks!


On garde en double et on gère plus tard sauf si t'arrive à te retrouver dans ce maquis (ou que tu trouves un bon contact à l'URSAFF)


Indeed, I misread my own line of code and thought that that was what I was doing ;)


Consider using `ng-submit` for `saveUser()` -- it will interact better with your validations/other form features you may implement later.


Misschien moeten we hier de array filteren en enkel de checkRedirectUrl en sanitizeDestination subscriptions er uit halen? Kwestie dat als we zelf later subscribers willen toevoegen, we niet weten waarom die niet werken. (Of als Drupal of een andere module andere handige subscribers zou toevoegen in de toekomst we die niet per ongeluk unsubscriben)


I suppose it could be a possible issue if the uri parameter in the route has a greedy regex and the parameter is passed through to something that loads a file. But arguably that's also a bad developer issue.


После логина должно пройти некоторое время, чтобы кнопки появились на экране. Необходимо добавить ожидание.


Is it possible to refactor the usage of the `Open` to remove the `new` and pointers? E.g. pass stream as out param or rely on NRVO and return stream by value (but I am not sure about how this technique works with streams).


Consider making this an abstract state: https://github.com/angular-ui/ui-router/wiki/Nested-States-&-Nested-Views


TeamCity Civic Assembly :: Build all [Build 26](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=424&buildTypeId=CivicAssembly_BuildAll&guest=1) outcome was **FAILURE**
Summary: Tests failed: 144 (85 new), passed: 28, ignored: 4 Build time: 00:17:07
### Failed tests
```
Blog categories tests: Should not delete blog category if cancel pressed: <no details avaliable>

Blogs tests: Should delete blog: <no details avaliable>

Document tests: Should add new version of Document: <no details avaliable>

Document tests: Should be able to revert from new version of Document: <no details avaliable>

Basic Events CRUD Actions: Should navigate to Page page: <no details avaliable>

Basic Events CRUD Actions: Should Not Add a Page if Cancel button pressed: <no details avaliable>

Basic Events CRUD Actions: Should Add a Page: <no details avaliable>

Basic Events CRUD Actions: Should Not Edit a Page: <no details avaliable>

Basic Events CRUD Actions: Should Edit a Page: <no details avaliable>

Basic Events CRUD Actions: Should Not Delete a Page: <no details avaliable>

Basic Events CRUD Actions: Should Delete an Page: <no details avaliable>

##### there are 134 more failed tests, see build details
```


There won't always be a `local_authority_name` whereas there would've always been a translated message.  Is this ok?  Should we provide a default placeholder value for the case where there isn't one?  I don't know enough about GA to know if an empty label will be ignored or not.


我自己抓了协议研究，就复用wdf的代码了。

发送自 Outlook Mobile

On Fri, Jan 22, 2016 at 7:21 AM -0800, "Liu Yuyang" <notifications@github.com> wrote:

域名为什么会这么多呢？我一直很奇怪，请指教

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


Через псевдо-селекторы(nth-child, first-child, last-child)


Здесь написано "Примеры значений". Видимо, некорректно перевели с английского.


Адрес не меняется во время выполнения теста, его можно сделать статической строкой и переместить в объявление класс..


TeamCity pull requests :: netty [Build 1092](https://garage.netty.io/teamcity/viewLog.html?buildId=4989&buildTypeId=pulls_netty&guest=1) is now running


mozesz przekazac od razu `TalkItemHolder`


Oui. Idéalement on annoterait les Variables avec les codes ou intitulés URSSAF :sunglasses:


@joecaintic DWMR. Ship code. Ship it!


Please leave this in


no test related to this ?


"Olha eu aqui traveis" :P hehe
O Rumbler Soppa me falou do projeto.
Enrolei muito para ajudar em algo.


Декоративное) исправил


tu nie wiem skad sie to -1 wzielo, daj jakis koment


@apotheon You are not on the list as the GitHub API did not return your username in the list of contributors.

The MPL 2.0 is a perfectly valid/fine license, probably better than most out there. If you have any specific concerns please address those either in Gitter or via an issue, it's hard to keep track of them via commit comments. So far I'm not seeing any solid arguments, other than vague statements about compatibility (which isn't even relevant since we're fully swapping over the license with permission of past contributors).


Thanks, I used the github web editor to switch this from 'bdemers' to 'openstack-packages'. (sorry about that)


Yeah :shipit:


Successfully created the [rsync-optim](http://rsync-optim-static-docs.pantheon.io/docs) environment. New commits to this branch will automatically be deployed.


I did, but it only ever installed with on 4.


I will check to see if jQuery has a function for that. For now, this will do.


Yeah :trollface:


Is it functionally mandatory to call filed with a type?
It seems like `observers` will be enough, not?


@davidworn DWMR. Shipping code. Good. Hope you are safe this weekend. :snowflake: :8ball:


Eventueel kan je hier gewoon zowel de ServiceProviderInterface als de ServiceModifierInterface implementeren. De ServiceProviderBase doet dat nu ook en verder niets dus het effect zou hetzelfde zijn op dit moment, maar het kan potentiële neveneffecten vermijden in het geval ze later code toevoegen aan ServiceProviderBase die de werking van onze CulturefeedUdb3ServiceProvider zou beïnvloeden. Maar die kans lijkt me vrij klein omdat dat een backwards-compatibility breaking change zou zijn dus het is eerder iets om te overwegen / in het achterhoofd te houden.


Lazy prick :P


I think it might be worth putting an href in here as a single word like "back" doesn't really seem enough to disambiguate this link from any others that might appear.


зачем пробел?


зачем в отдельной переменной?
Или он где-то используется?


昨天开一晚上都不会sync失败，但我发消息量增加就会。

发送自 Outlook Mobile

    _____________________________
From: Liu Yuyang <notifications@github.com>
Sent: 星期五, 一月 22, 2016 23:26
Subject: Re: [wechat_robot] 可以勉强工作的版本。回复太快会链接错误。。。 (976d78e)
To: lyyyuna/wechat_robot <wechat_robot@noreply.github.com>
Cc: Li Yiyang <lyyyuna@gmail.com>

！应该叫长连接更合适些，
 我这里看到的都是只要synccheck断开就连接，但服务器会保持连接，只在有推送的时候返回断开。客户立即重连

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


May be `AutoLock`?


faild


The `date` filter can take a formatting, just in case you didn't know.


I would rather see :
```
MailboxQuery.builder(session).privateUserMailboxes().build();
```


Oh, I see, this gives us a default value, but I guess my point still stands, does this label make sense for the situation where we don't have the name of the la?


Deze use statement (ServiceProviderInterface) lijkt me overbodig als je ServiceProviderBase extend? (Maar zie ook mijn comment hieronder)


all refers to what ?


Those three lines is a part of build_dist.sh what is used to create configure file.


nope, existing tests are already enforcing it


Perhaps we could have a GET/POST listener in index.php that when set shows a modal with "settings saved" and then fades out... don't worry 'bout that right now though, I can do it if you just wanna get this out of the door :-)


could be a builder constant


fyi, I found that if/else was often less efficient than pre-evaluating the else{} clause.

eg. check if this is smaller:
sp_temp = profile_step_sp;
if (t >= profile_step_dur) {
 t-= profile_step_dur;
 sp_temp = profile_next_step_sp;
}


You could squash this commit down into the original.


const operator
const parameter


Written this way, the assertion is done on the list. We want to make sure this particular contact appears in the list.


saw approval from javajolt
at https://github.com/basho/riak_pb/commit/ee73a3d9de496b2aaf6cd77484cb5cb1164c7c7e


merging basho/riak_pb/features/lrb/e2e-ts-remove-python = ee73a3d9 into borshop-integration-177-features/lrb/e2e-ts-remove-python


basho/riak_pb/features/lrb/e2e-ts-remove-python = ee73a3d9 merged ok, testing candidate = 142b9e9e


ok


Is there a another way to detect nested accordions than just assuming the first match is the one we want? I'd rather rely on something more explicit than that.


Incorrecte "contains" class? Is deze @file annotation eigenlijk vereist in Drupal? Zo nee dan zou ik die weg laten want is vrij obvious welke class er in de file zit, aangezien we maar 1 class in een file definiëren normaal gezien (en die moet dezelfde naam hebben als de filename)


true, removing using your previous comment


Indeed Jenkins notifications are shomehow lost, I will look at it. But I'm getting errors from Travis CI as well...


More effort that on the following commit as these changes come from two commits not one, but you could also squash these changes down into the original commits.


[smart drive.zip](https://github.com/himayatchirag/SmartDrive/files/101044/smart.drive.zip)


so the change is useless ? I don't understand why it's needed then


@shucknjiveturkey DWMR. Noce work here. Clear and concise. Ship code mate!


Looks grrrreat!


Btw. you can use `.substring(0, 24)` as the substring 2nd param is exclusive :)


still not indented correctly.


I prefer to call it like `IDCheckFunctor` or something like that.
And it seems to be a non-google-style naming (low case and `_`)


Also could be const


Could this be pulled out into a function? The code looks duplicated from above.


because the next commit break it else


I prefer to split to getting an contaner (`it->sec`) and updating this container at the next line


@ericmj we need to a new release. Sorry about that.

Meanwhile, it would be nice to write a test for this.


I think there is a stray single-quote after minimum.


на линуксе тоже в 1251?


Remove `which`


There is an extra space on the last edit.


@pekohl DWMR. Getting the `{ } ` aligned is worth the effort. There is a brackets add-in/extension that will help with that. It makes debugging so much more simple. Ship code.


I still don't think you want OutputVersion here. From env, we compute this twice, once for the patch:
```
objects, err := resource.AsVersionedObjects(infos, clientConfig.GroupVersion.String())
```
and once for the output:
```	if len(outputFormat) != 0 {
		outputVersion, err := cmdutil.OutputVersion(cmd, clientConfig.GroupVersion)
		if err != nil {
			return err
		}
		objects, err := resource.AsVersionedObjects(infos, outputVersion.String())
...
```


but i detect bug if use setFlippedX, some sprites not flipped :(


ok


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 12410](https://teamcity.mesosphere.io/viewLog.html?buildId=110590&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 215 Build time: 00:04:21


Do you know which commit changed it? (I'd like to look into it)


Sounds good to me :D :+1:


There is an extra space.


Could you also add a `{pull}803[803]` here.


`was created for`


Oi, Juscelino. Blz?
Sou Felipe, trabalho na SEA e eu estou avaliando teu projeto.
Vc usou ASP.NET né? Vc consegue subiir essa aplicação em algum servidor pra gente ver ela rodando?


Could we use from this method the previous one?
IT will reduce code duplication


- Wendy Cicotte <br/>


test update


@pekohl DWMR. Refactor to simplify your code - that's what you've done here. Get it working - then improve it. Ship code.


This looks wrong. There's no guarantee that `this.password


Just to get travis passing for now


Will it realy lead to issue without `empty` verification?


@pekohl Hope you are safe this weekend. :snowflake: :8ball:


This needs to go below the `// Module-level variables` header.


in fact, renaming to matchesAll, as this is an interesting shortcut to build an expression that matches all responses.


I would avoid `require_relative` as long as your code has a unique entry point as we do have in a gem.


Use of "easy", "easily" considered harmful.


This looks really scary :(


If the object that defines `applies_from` needs to come before the default config object, then that would definitely be a problem. However, that ordering seems pretty counter-intuitive. I would normally expect the last thing in the list to override the first thing. Is that something we could change?


Can you please explain why you removed the extra check?


We should make "Starting Chef Client (FIPS mode), version 12.4.3" a little more explicit (e.g. on, enabled, activated, engaged, made so, etc.)

How about making this a second line (which would read easier) like:
```
puts_line "OpenSSL FIPS 140 mode enabled" if Chef::Config[:openssl_fips]
```
or whatever?


Hallo Reining,

beim Nachvollziehen des Codes sind uns ein paar Fragen gekommen:
1. Warum wird der Code in Factory.java ausgeführt, wenn Message2 injected wird? (Woher weiß Zeile 40 in CDIServlet.java, dass sie dort nachsehen muss?)
2. Wann ist es Pflicht, mit @Produces eine Instanz zu initialisieren, wenn man @Injected nutzt? Ist das immer Pflicht?

Viele Grüße,
Worm und Burkhardt


mhm seems #I doesn't solve it completely. I'm getting it again on master


Could be replaced by a link to the build status at the top of the document (as is custom for GitHub projects)

```
[![Build Status](https://travis-ci.org/ypg-data/eigenflow.svg?branch=master)](https://travis-ci.org/ypg-data/eigenflow)
```


I think it might be caching related


@tomsmeding why do you explicitly cast to bool?


On s'y croit un peu plus ainsi.


When you are fixing in this part anyway - can you please use plural variable name `generated_headers`?


Yep looks like it's working if I remove the .fake folder
On Jan 22, 2016 16:57, "Matthias Dittrich" <notifications@github.com> wrote:

> This looks really scary :(
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/fsharp/FAKE/commit/0d830d65c7d5ab999811e3b78af0242757325f58#commitcomment-15616306>
> .
>


Because at this point the container isn't started, so we need to apply the change to the LXC container config so that those keys get set by LXC on container startup.

The later code is executed on update of the config for a live container, at which point changing the LXC config won't do anything and we instead must directly modify the cgroup config through the set_cgroup_item LXC API.


Does the openstackclient really ever look at the maas openrc?


Then it should be reproducable calling FAKE the second time


please follow me on github! thank you!


TeamCity Civic Assembly :: Build all [Build 27](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=425&buildTypeId=CivicAssembly_BuildAll&guest=1) is now running


ah, good to know. I think I'm ok with dropping 3.2, I've seen other projects do the same recently, and that seems easier than downgrading coverage. Thanks!


oh


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 12411](https://teamcity.mesosphere.io/viewLog.html?buildId=110591&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 215 Build time: 00:04:24


ah that's frustrating. also have you seen this: https://github.com/spulec/moto? i played around with it briefly when thinking how to test S3


I think when running the script regulary and then running the SourceLink task with the cached version makes it fail


But if this is true I think the bug should be there ever since we introduced caching


Will test this


[Now](https://github.com/neowutran/ShinraMeter/tree/7658bb963658fa8c601b7a729395e34a6fc3f726) you have only heal/hot dealt statistic (how much who is healing), but not received one (how heal/hot is distributed between targets). May be you forgot to publish it?=)
If you don't know, who applied hot to user (haven't seen priest, that buffed your party member with balder since meter start), this hot should be forged as a skill with unknown source and therefor not counted as hot to anyone (not tested on real data, just reading the current code).


Thanks. I just installed a bunch of extensions for brackets. let's see what
happens.

I'll be heading up to the country this weekend, supposed to get less snow
than the city.

I'm sure we'll be fine.

-Peter

On Fri, Jan 22, 2016 at 10:55 AM, dothework <notifications@github.com>
wrote:

> @pekohl <https://github.com/pekohl> Hope you are safe this weekend. [image:
> :snowflake:] [image: :8ball:]
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pekohl/bloc-jams-angular/commit/fb911cd17d2b080c6e38a351529751ee59fb70bf#commitcomment-15616279>
> .
>


Runs stable?


The last argument to `soundio_instream_get_latency` should be called `in_latency`, or am I wrong?


Seems like the assembly cache is empty what is going on
```
<?xml version="1.0" encoding="utf-8"?>
<FAKECache>
  <Assemblies />
</FAKECache>
```


Are you suggesting that I should document that it can take a ModuleSymbol as well?


Ideas?
On Jan 22, 2016 17:06, "Matthias Dittrich" <notifications@github.com> wrote:

> I think when running the script regulary and then running the SourceLink
> task with the cached version makes it fail
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/fsharp/FAKE/commit/0d830d65c7d5ab999811e3b78af0242757325f58#commitcomment-15616483>
> .
>


Is it a good idea to have inline=True in the base class and inline=False in bokeh's version?  Seems confusing.


This is the most basic sanity check on the minimum Python version. Not yet obvious that setuptools provides anything to place a requirement on minimum python version.


Maybe just merge this into the first leaving out "continue a process from a stage". We could argue that stages allow for quicker recovery, better code reuse.

```
`Eigenflow` was created for managing periodic long-running ETL processes with automatic recovery of failures.
```


pour naruto, tu pourrait voir  sur dx12 ou avec le llvm pour accélérer la compilation ? j'ai eu du 30 fps sauf que ca ma pris quand mème 5 min a compilé  ^^ domage que le sdtorm 2 le fasse pas, l'instruction qui manque pause vraiment probleme :/ utilise la demo pour le debuger, c'est NPUB90746 sera plus simple que le storm 2


:+1: Awesome


Thanks for catching that, that wasn't intentional.


I just tried out reversing the configuration order and it does seem to work correctly.


Give me a moment


@zinigor these 2 strings should be available for translation.


There seems to be a extra node <MessageText> in the response message from my Exchange2013_SP1 environment, compared to the msdn EWS reference samples. This addition seems to help, but I don't fully understand how the reader actually works its magic


Fixed in f18295c2aa61b70f299634c37fff026be244e9dc.


Please integrate the existing set up and alter them all together when we make a new file. It gets confusing as we go along since the ul & ol set ups had to be made page specific for rich text editors. The current layout exists in the _tds-type.scss.


está ok. Considerá usar `_.map()` para estas cosas:
```
response.labels = _.map(response.values, function(column) { return '';});
```


I was having issues with getting netwmpager and obsetlayout to cooperate with openbox and have everything functional.  Typically GoToDesktop up/down didn't function, or openbox needed to be restarted or reconfigured and that would in turn break netwmpager.

Anyhow, this provides openbox with the facility to configure a row X column virtual desktop setup and supercedes <number> in a sane manner.


propose to add typedef for this type. E.g. FileSizeType.


TeamCity Civic Assembly :: Build backend + frontend [Build 22](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=428&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) is now running


The [npm documentation](https://docs.npmjs.com/files/package.json#version) says:

> Version must be parseable by node-semver, which is bundled with npm as a dependency. (npm install semver to use it yourself.)

So I think it's safer to keep using 3 numbers in `package.json`, even if we only use 2 of them publicly?


Imagen Laboratoria


Well, I suppose it could be forwarded to gperf maintainers as it does extend gperf to allow supporting arrays of structures with tagged elements as needed for infinality's built-in settings (examples in the ftinf_[rs]h.gperf).

The modified gperf is only needed to regenerate the corresponding .c files and this is truly required only if changing the "name" fields or adding/removing array elements as changes in other structure fields can still be done on the resulting c files (although would be nicer to keep these fully is sync with the .gperf sources).


Can we cite the new A&C ORAC-DR paper (either in addition or instead of the AN one)?


Reversing the configs seems to work.

```diff
diff --git a/lib/import_js/configuration.rb b/lib/import_js/configuration.rb
index 4038b24..ab082c5 100644
--- a/lib/import_js/configuration.rb
+++ b/lib/import_js/configuration.rb
@@ -24,6 +24,7 @@ module ImportJS
       @configs = []
       user_config = load_config(CONFIG_FILE)
       @configs.concat([user_config].flatten) if user_config
+      @configs.reverse!
       @configs << DEFAULT_CONFIG
     end
```

Probably worth adding a comment that explains why we are reversing.


@aquamatthias It's part of the changelog but there's obviously no documentation yet.


Gosh, this looks pretty goofy—a Dr. Frankenstein's class name, combining a Bootstrap class with one of our own.


Should these be prefixed with MailTip or something?
I don't like adding quite generic words like _Recipients_


<!DOCTYPE html>

			<title>{{ page.title }}</title>

			<link rel="stylesheet" type="text/css" href="/css/main.css">

Home

About

CV

Blog


It might also make sense to reverse while concatenating:

```diff
diff --git a/lib/import_js/configuration.rb b/lib/import_js/configuration.rb
index 4038b24..28f4f6f 100644
--- a/lib/import_js/configuration.rb
+++ b/lib/import_js/configuration.rb
@@ -23,7 +23,7 @@ module ImportJS
       @path_to_current_file = normalize_path(path_to_current_file)
       @configs = []
       user_config = load_config(CONFIG_FILE)
-      @configs.concat([user_config].flatten) if user_config
+      @configs.concat([user_config].flatten.reverse) if user_config
       @configs << DEFAULT_CONFIG
     end
```


waa iga tijaabo, halkani waa halakan


You are using the `code-generator` interface and not the `node-factory` interface. `code-generator` is the correct choice, but you need to update your function signature to be the following:

```javascript
module.exports = function codeGenerator(elNode, codegen) {
```

For the most part, your code below does not change other than renaming `context` to `codegen`. However, I've added a few more comments below...


Ok sending PR in a minute, was a stupid mistake :(


return `0u`. Since type has changed from signed to unsigned, then use everywhere unsigned literals. Check all places. at least in these files.


I've just scanned, but it looks good to me.  I'm personally keen to move away from utime_t, so I'm more than fine with the work to clear it away systematically.


For long messages, could you use this style as done elsewhere:
```
msg = (
    "...."
    "...."
)
with self.assertRaisesMessage(ImproperlyConfigured, msg):
```
That avoids funky indentation.


 ahahahha

 Eu me lembro que ele tinha dito de você. Mas não tem problema, o bom é que
já está ajudando :-)

----

   - https://www.youtube.com/user/tocadotux
   - http://tocadotux.blogspot.com.br/
   - http://www.tsu.co/tocadotux
   - *Caixa de Ferramentas do Unix*
   <https://github.com/tocadotux/caixaDeFerramentasDoUnix>

Em 22 de janeiro de 2016 13:45, João Batista <notifications@github.com>
escreveu:

> "Olha eu aqui traveis" :P hehe
> O Rumbler Soppa me falou do projeto.
> Enrolei muito para ajudar em algo.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/gabrielscosta/caixaDeFerramentasDoUnix/commit/57ec8ef51eb586624fbaa3deadd59139dc0789a7#commitcomment-15616009>
> .
>


The `col-md-4` is a [Bootstrap class](http://getbootstrap.com/css/) that presumably is there for a reason. By kludging it into a different classname here, we break whatever Bootstrap was doing here.


Do we need to add `sbt` to the list of requirements?


Yes `.ini` need to die.


Change these lines to the following:

```javascript
codegen.addError('Invalid <include> tag. Template path argument is missing. Example: <include(templatePath[, templateData]) foo="bar">');
return;
```


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 12415](https://teamcity.mesosphere.io/viewLog.html?buildId=110602&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 215 Build time: 00:04:49


cf #17


Hooking okhttp2/3 should consider these situations: multi-dex, dynamic loading dex & packers. In these situations, OkHttpClient class can't be found by lpparam.classLoader sometimes.
A temporary solution is hooking android.app.Application.attach, then use the context.getClassLoader() to load some class. This solution works on multi-dex. But I've just tested on one single app. And that's why I did not create pull request.
I'm a little busy these days. I'll work on this issue later.


Exactly, then you're talking about either segments as input, or get variables as input. Both should be validated separate from the URI.


Can't we just attach the style to `remove-page-buttons`? Afaict that class is otherwise unused from CSS or JavaScript, and only shows up in the various KO branches of the `modal-footer` in this template.


Can you please add description, why this function is redundant.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 12414](https://teamcity.mesosphere.io/viewLog.html?buildId=110601&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 216 Build time: 00:06:28


can be a single line: `getattr(Article(), "objects")`


:tada:


> why not round (or ceil or floor) the value unconditionally

@jepler , The idea is to not change existing unit test that works for v8 but to fill in holes to make it work with chakra engine.


не уверен насчет ```static ```
привык, что у нас в ```AssemblyFactory return ModuleAssembly()```
плюс можно сабклассить для проверки, чтобы через мок проверять, что роутер обратился в ассамблею после вызова презентером метода вида ```showNextModule()```


Agreed, in that case I don't think I have any issue with changing this behaviour.


return 0u


That Rx-Main should go into a group with no targetFramework to be consistent


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 12413](https://teamcity.mesosphere.io/viewLog.html?buildId=110593&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 215 Build time: 00:08:44


I think `r` message prefix isn't needed with `assertRaisesMessage`


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 12412](https://teamcity.mesosphere.io/viewLog.html?buildId=110592&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 213 Build time: 00:08:54


don't put it into the uap10.0 one, it's not needed there


Testing basic github setup


Díky, já se snažil :)


It may be cleaner to use [pytest.mark.parametrize](http://pytest.org/latest/parametrize.html) or just write a doctest.


there should be a

`
<group>
<dependency id="Rx-Main" version="2.2.5" />
</group>
`


testing basic github setup


Consider taking out "But first"? Does it sound weird starting a new section that way? Maybe it's part of the way you want the blog tone to come across.


It looks like you can just delete these lines rather than comment them out!


Hallo Herr Reining,

da im Skript keine Folien zu Websockets enthalten sind, möchte ich gern fragen, inwieweit dieser Stoff relevant für die Klausur ist.

Viele Grüße,
Ronny Worm


[iBuffalo Classic USB Gamepad.txt](https://github.com/Charco79/retroarch-joypad-autoconfig/files/101098/iBuffalo.Classic.USB.Gamepad.txt)


with this line:
https://github.com/neowutran/ShinraMeter/blob/7658bb963658fa8c601b7a729395e34a6fc3f726/DamageMeter.Core/DamageTracker.cs#L342

I will add some code commenting soon x)


@openhardwarecoza so it's actually quite easy, if you have a loop that looks like this:

```javascript
for (var i=0; i<1000000; i++) {
 doCalculationIntensiveThings(i);
}
afterLoopStuff();
```

convert it to something that looks like this:

```javascript
function GreatCalculator() {
  this.currentIndex = 0;
  this.timer = null;
  this.count = 1000000;
}

GreatCalculator.prototype.run = function() {
  this.timer = window.setInterval(this.loop.bind(this), 10);
};

GreatCalculator.prototype.stop = function() {
  window.clearInterval(this.rasterIntervalTimer);
};

GreatCalculator.prototype.loop = function() {
  if (this.currentIndex < this.count) {
    doCalculationIntensiveThings(this.currentIndex);
    this.index++;
  } else {
    this.stop();
    afterLoopStuff();
  }
};
```

what this does is use the window timer functionality to asynchronously callback in fixed intervals, e.g. 10 milliseconds. Which gives the browser the time it needs to handle it's UI Events. 1 milliseconds would probably suffice too, depends a bit on how long your loop needs to calculate. An calculation dependent solution (probably better solution) could be:
```javascript
function doLoop(index, count, callback) {
  if (index < count) {
    doCalculationIntensiveThings(index);
    window.setTimeout(function() { doLoop(index + 1, count, callback); }, 1);
  } else {
    callback();
  }
}

window.setTimeout(function() { doLoop(0, 1000000, afterLoopStuff); }, 1);
```


The spec is a bit confusing, but shouldn't this be still -1 until the ACK arrives? "Is always 0 when the first message has been sent **and** an ACK has been received confirming receipt of the message." Also, that makes the first message not an special case but just the same as any other message, if I'm reading this correctly.


:+1:  I shall try to put the ```switch``` into a function


Thanks for working on this @laggingreflex.  Please take a look at the expected syntax for the `<include>` tag in Marko v3: https://github.com/marko-js/marko/issues/178

In Marko v3, we are using tag arguments to pass the template path and, optionally, a template data object.

Also, an `<include>` tag supports nested body content. The code to handle a custom tag will actually have some similarity to the code to handle the include tag. Please take a look at the following: https://github.com/marko-js/marko/blob/htmljs-parser/compiler/ast/CustomTag.js

NOTE: The CustomTag AST node is going to be more complex because it supports a "tag definition" that can be used to control how attributes are handled.

Couple things to note:

__Here is the code used to import and render a template:__

```javascript
let templateRequirePath = context.getRequirePath(tagDef.template);
let templateVar = context.importTemplate(templateRequirePath);
let renderMethod = builder.memberExpression(templateVar, builder.identifier('render'));
let renderArgs = [ inputProps, 'out' ];
let renderFunctionCall = builder.functionCall(renderMethod, renderArgs);
return renderFunctionCall;
```

In the case of an `<include>` tag, the template path can either be a literal (static) string, or it can be a JavaScript expression that resolves to a loaded template. Your code would need to handle both.

__Here is the code used to build the `renderBody(out)` function__

```javascript
renderBodyFunction = context.builder.renderBodyFunction(this.body);
```


TeamCity pull requests :: netty [Build 1092](https://garage.netty.io/teamcity/viewLog.html?buildId=4989&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 6387, ignored: 23 Build time: 00:42:09


I included a [test](https://github.com/openmrs/openmrs-core/blob/95b3ab3de2d8ec488d0ca0fcbcb1e8969d491313/api/src/test/java/org/openmrs/api/DatatypeServiceTest.java#L41), if i have a handler that extends a generic abstract class it would fail, in my test i have an abstract class [BaseDatatypeHandler](https://github.com/openmrs/openmrs-core/blob/95b3ab3de2d8ec488d0ca0fcbcb1e8969d491313/api/src/test/java/org/openmrs/attribute/handler/BaseDatatypeHandler.java#L15) that implements CustomDatatypeHandler, then i have [LocationDatatypeHandler](https://github.com/openmrs/openmrs-core/blob/95b3ab3de2d8ec488d0ca0fcbcb1e8969d491313/api/src/test/java/org/openmrs/attribute/handler/LocationDatatypeHandler.java#L17) that extends BaseDatatypeHandler, before my change the location datatype handler can't be found because the original code only inspected the generic interfaces for type parameter values and but LocationDatatypeHandler extends an abstract class so the type parameter values can't be read, so we need to inspect the type parameter values on the generic super class too.


This was already fixed.


@apotheon you may find [this post](http://www.gatsbymedia.net/development-articles/why-we-use-the-mpl) interesting. There are also other resources available. From my understanding, MPL-2.0 provides better protection against patent trolls and is a reasonable license. Again, IANAL and we're not having a legal debate here. More importantly, we're not debating the license. If you are a contributor, we've asked that you consent to the new license. Anyone is free to contribute or not based on their understanding of the license.

Thanks for taking the time to express your concern.


Ahh so the function stays the same, but you do the UI updates OUTSIDE the original function, and thus the timer fires the event if its busy?

Going to do this one myself (learning exercise)


Should be
Address change = changeAddresses.get(key);


Just curious, are they really hit points? I always thought they are health points.


Please use @After public void tearDown()


См. коммент выше


Should this be `recoveredClient.channel`? Or is `recoveredClient`'s serial supposed to increment even when the it's another client instance who publishes the message?


Yeah currently it just fades in a "settings saved" text at the bottom of
the modal.  Think it would be a bit weird to have another modal come up
over that one.  As some people may want to save and continue editing.
That's why I left the "close" button on the modal.  Figured I'd have close
save and reload page.  Gonna do that here ina bit and then make sure the
sorttable is working fine and then do a couple more try catches for
possible errors and then I'm ready.  Should be good tonight.

On Fri, Jan 22, 2016, 9:49 AM mescon <notifications@github.com> wrote:

> Perhaps we could have a GET/POST listener in index.php that when set shows
> a modal with "settings saved" and then fades out... don't worry 'bout that
> right now though, I can do it if you just wanna get this out of the door :-)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/SyNiK4L/Muximux/commit/ef66bdf6de20bcede2cd445ed9356f38189ff5f1#commitcomment-15616106>
> .
>


What was the status of these tests?


It's not a big deal.  I'm just one disappointed guy.  While many consider it a "perfectly valid/fine license", it does have some license incompatibility issues, and that's a big chunk of the reason I'm disappointed -- but that's surely of no concern to the larger project or community.

> I'm not seeing any solid arguments

I'm not making arguments.  I'm explaining why I'm disappointed by the news, because someone responded to my comment and asked.

> vague statements about compatibility (which isn't even relevant since we're fully swapping over the license with permission of past contributors)

They are not vague; I gave some rather specific explanation.  They're not a problem for the project relicensing per se: they're a problem for what would have been my potential use of the software, and may affect other users, in possible future edge cases (such as, perhaps, certain techniques in language design that might aim to take advantage of the unique design of Rubinius itself).

Please don't take this as some kind of campaign to change your mind or make you look bad.  I made a somewhat spontaneous noise of dismay, and replied as fully and politely as I could to requests for clarification.  That is all.


By "this", I wasn't clear. I only meant the code in the `default` block. But if the whole switch can be done that's cool too.


(∩´﹏`∩); you changed source encoding


How about replacing `Getters/Methods` here with:

> * Methods and/or logical groups of methods.


No, it should be `recoveredClient.channel`. Thanks.


TeamCity Civic Assembly :: Build all [Build 27](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=425&buildTypeId=CivicAssembly_BuildAll&guest=1) outcome was **FAILURE**
Summary: Tests failed: 154 (11 new), passed: 1567, ignored: 12 Build time: 00:26:28
### Failed tests
```
Document tests: Should be able to propose Document to review: <no details avaliable>

Document tests: Should be able to publish proposed Document: <no details avaliable>

Document tests: Should take a alert when try to upload new version for locked document: <no details avaliable>

Document tests: Should be able to unlock and revert to draft locked proposed Document: <no details avaliable>

Document tests: Should be able to propose reverted Document to review: <no details avaliable>

Document tests: Should be able to decline proposed Document: <no details avaliable>

Document tests: Should be able to propose, publish and expire Document: <no details avaliable>

Media type tests: Should navigate to Media Types page: <no details avaliable>

Media type tests: Should not create new media type if cancelled: <no details avaliable>

User profile tests: Should navigate to user profile page: <no details avaliable>

User profile tests: Should not change user profile info: <no details avaliable>

##### there are 144 more failed tests, see build details
```


Is that related to sprite sheet packer? That might also be a cocos bug.


Yes the openstack client does. These variables are more a hint later for the keystone client for example to use other domains than "default"


`channel` should be checked against `PDU_ADC_CHANNEL_COUNT` or maybe a new macro, `PDU_CHANNEL_COUNT`.


I'll be honest this is all french to me. Maybe because I have not looked any any of the math behind the breed timer. Perhaps this is something for @zininzinin . But I will say this, currently about half the time we are one stack short of the target anticipation stacks (assuming target stacks == #seconds in geneticist timer).


Same here.


Maybe this should just be `PDU_CHANNEL_COUNT` in `pdu.h`?


This should probably only sample channels that are turned on. Not much point in sampling a 0.


+ add bootstrap


Perhaps you just mean "If you set CONSOLE, you...".


www


Weirdest C++ code I have written.


It looks like this broke the stdlib build:

swift/stdlib/public/core/Arrays.swift.gyb:787:12: error: cannot invoke initializer for type 'Mirror' with an argument list of type '(ContiguousArray<Element>, children: _ArrayTypeMirrorCollection<_>, displayStyle: _)'


Oh, ok. I might also try to put the ```switch``` if it can keep the code clean.


Remove the version tag, and indent properly please. If you are using Dropwizard it also provides Logback and slf4j so maybe you could remove this dependency.


Perhaps make it clear here that PAM can use a plugin to read /etc/shadow, but overall PAM is just a system by which logins are centrally processed. I would mention using the LDAP plugin here because that was configured at both the CCDC and PCDC if I recall correctly.


sorry...didn't realize it was in there.


well, you wrote them for something. I wanted to leave deleting to you, just in case they were not doing anything by accident.


TeamCity pull requests :: netty [Build 1091](https://garage.netty.io/teamcity/viewLog.html?buildId=4988&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 6387, ignored: 23 Build time: 01:21:54


Great. I will give that a try and let you know if that fixes it.


With an appropriate what?


will change that.


@chrisseto Think I got it. How's this look?

```python
    @asyncio.coroutine
    def read_as_gen(self):
        self.file_pointer.seek(0)
        while True:
            chunk = self.file_pointer.read(self.read_size)
            if not chunk:
                self.feed_eof()
                chunk = b''
            self.chunk = chunk
            yield from asyncio.sleep(0)

    async def _read(self, size):
        self.file_gen = self.file_gen or self.read_as_gen()
        self.read_size = size
        self.chunk = None
        next(self.file_gen)
        return self.chunk

```


I think this is not used anywhere


Maybe you're right.
@mattheworiordan Can you clarify this?


Watch out with the transitive dependencies here, I'd recommend you to analyze collisions with ```mvn dependency:tree```  because adding this dependencies probably require exclusions.


Remove the version tag  on non root projects.


@ebrehault by doing this you're also forcing people to run manually this step on real installations


Well yes, the browser handles UI Event's itself. You can imagine it like a big loop in the innards of the browser:

(disclaimer: blatant simplification)
* route click event
* draw mouse move
* animate css effect
* execute javascript
* draw css mouse hover effect

if your execute javascript takes a long time, the following events will be affected. Also you only get one javascript thread, so other js won't be called while one executing is running.
So what you want to do is give the control back to this main-loop. The way to do this is just to let your code run out of the current scope (e.g. last method on stack ends).
But since we want to continue working, without the user having to rhythmically click the UI, we can use the timer function to let the browser queue the next javascript execution in after the current tasks. Even if you enter an interval of 0 milliseconds you will just be the next in the queue.


Fudidão


Why you are not doing this in the constructor of the resource?


Hello, consider your _edit/commit_ useless, simply because I used "type hinting" so if you pass anything else than array PHP is going to yell at you :).

to test it try this PHPSandbox I wrote test code for you just pick PHP version and hit run.
http://sandbox.onlinephpfunctions.com/code/51d0db7aff03765cd557b44647dec97383ff2ee1

documentation
http://php.net/manual/en/functions.arguments.php


@ebrehault this is not needed neither


@gajus While I tried the change locally, I came across more cases. It looks a bit complex. Let's think of following cases and let me know what is expected and accepted here.
```js
// 1.
_.chain({}); // IMO, we shouldn't force user's to enter new line here

// 2.
if (obj.method()) { // Same here. we shouldn't force new line here
//...
}

// 3.
ob1.obj2.obj3.method2(); // Should there be newline after obj3?

// 4.
ob1.method1().obj3.method2(); // How about this?
```
And may be there are more cases to think of.


Why are you using a private class instead package one? I'd use a package or even a public class instead an inner one.


Hey Pavel, this issue is reported in #6, and we do not have a solution for that so far. Hence, bob.learn.em requires numpy version 1.9.2, the tests won't pass with 1.9.1.

I will adapt the bob.wheels script to compile and upload wheels for numpy 1.9.2 for all supported platforms. AFAIR, I had done that before, but someone seems to have deleted the files, they are no longer listed at:
http://www.idiap.ch/software/bob/wheels/travis/


This commit is useless after the commit https://github.com/janetournois/cgal/commit/f89a8c87fcd880439834d7fb44cae7eb089e8963: the dialog is no longer destroyed. No need to restore its geometry.


me also @Dreddogpe


Does the weave proxy listen on `/var/run/docker.sock` now?


@TomBaxter I'm not a huge fan of the passing around result by assigning them to `self.chunk`

An async generator work pretty well here.

And shameless self pitch, [Agent](https://github.com/chrisseto/Agent) (should) do exactly what we want.
Or just a manually created async generator.


TeamCity Civic Assembly :: Build backend + frontend [Build 22](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=428&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) outcome was **FAILURE**
Summary: Tests failed: 12, passed: 1711, ignored: 12 Build time: 00:28:51
### Failed tests
```
Document type tests: Should not create new document type if cancelled: <no details avaliable>

Document type tests: Should add new document type: <no details avaliable>

Document type tests: Should not change document type: <no details avaliable>

Document type tests: Should change document type: <no details avaliable>

Document type tests: Should not delete document type if cancel pressed: <no details avaliable>

Document type tests: Should delete document type: <no details avaliable>

Media type tests: Should add new media type: <no details avaliable>

Media type tests: Should not change media type: <no details avaliable>

Media type tests: Should change media type: <no details avaliable>

Media type tests: Should not delete media type if cancel pressed: <no details avaliable>

Media type tests: Should delete media type: <no details avaliable>

##### there are 2 more failed tests, see build details
```


Looks good to me......¯\_(ツ)_/¯


Great. Could you use empty instead of isset because is more general.  If jsFormat is empty it should use Format because the validation should be the same by default.


You can remove apache http client if you are using Dropwizad. Check http://www.dropwizard.io/0.9.2/docs/manual/client.html to further information.


I considered adding a check here that client_t (which is a mongoc_client_t*) is not NULL.  If it is NULL, then we will crash the C driver on the call to `libmongoc::client_set_read_concern`.  This shouldn't ever happen, which is why I left out the check, but if we had an invariant-like thing I might add it here.


`&` does have higher precedence, but I added the parens for clarity.


I appreciate the input, thanks.


because bool is, in a sense, not a boolean type. It's an int. I have no idea what the c++ semantics for this are, but I think at least in C, when you say bool a = 7, a doesn't equal 1 -- it equals 7. Here I want to explicitly indicate that I want a 1 or 0 value, and the bool cast should do that IIRC. I'll look up what the actual semantics are


I think there's a kwsys function which does this. @bradking?


@chrisseto Will do
On Jan 22, 2016 11:42 AM, "Chris Seto" <notifications@github.com> wrote:

> @TomBaxter <https://github.com/TomBaxter> I'm not a huge fan of the
> passing around result by assigning them to self.chunk
>
> An async generator work pretty well here.
>
> And shameless self pitch, Agent <https://github.com/chrisseto/Agent>
> (should) do exactly what we want.
> Or just a manually created async generator.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/TomBaxter/waterbutler/commit/30f8317838a2e984e54fe22042fd3ffff10f82e6#commitcomment-15617297>
> .
>


It is removed in later commit. I will squash it in superseding PR.


Hahaha sind die zahle eifach usem duume zoge oder was? xD :D @zperee @Phong6698


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 12416](https://teamcity.mesosphere.io/viewLog.html?buildId=110603&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 215 Build time: 00:03:53


The problem is "I'm not going to add or remove stored properties" doesn't buy you anything, while "I'm not going to add or remove cases" does.

(By the way, in this document "fixed-layout" means something specific that's different from "I have full knowledge of this struct", because of its recursive nature. I don't consider a fragile framework's structs to be "fixed-layout"; they're just "not versioned".)


@mar4uk Sorry for the long delay. I've just published v0.0.4.


Ah yes, true - needs to be brought into scope. Will fix.

Yes, we can get mixed types indeed ;)


There's a `cmAlgorithms.h` this would probably be better in.


I use it line 62 (we need to commit the transaction so the BrowserTest can actually see the changes when it navigates the site)


true, i'll remove it


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 12417](https://teamcity.mesosphere.io/viewLog.html?buildId=110604&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 174 Build time: 00:05:24


doesn't the "" indicate that there are no input parms?  I'm surprised this works, but even so does it work via the schema Brad introduced?


Hmm, good point.


No, although they'd benefit if it is. I think an inlineable property is just shorthand for inlineable getters and setters, which is why I didn't pick another name.


Неизвестно. Надо эксперимент проводить.


can't we register a post-uninstall step ?


OK so this explicit cast is as useless as a comment I suppose. But then again, a comment wouldn't be that bad, so I'll keep this


Hi, Blair. The HTML and JS quizzes need to be submitted separately from their own repos created by following these links Steve put in the Slack channel:
https://classroom.github.com/assignment-invitations/c3db57cfb77e7fc10d8bdd4389fe6de5
https://classroom.github.com/assignment-invitations/36c0d875dd22a2720ed0decfb4f40988

Also, this is not the proper JS quiz. The one you did is a leftover from earlier classes. We'll be removing that today to avoid further confusion. Make sure you follow the directions from the front-end-milestones repo:
https://github.com/nashville-software-school/front-end-milestones/blob/master/the-static-web/quiz/JAVASCRIPT.md


Ahm, no fanciness here, but consistency with the s3_unicode pendant. In fact, the utf-8 string case is lighter that way than the decode-encode combo for the default case, so actually the simplification would be heavier ;)


I'd just let this return an error and terminate the receive loop at the call site.


How can it be heavier than:
if type(s) is str:
    return s


Pooof Mind blown!

Now I get it

Instead of the for i++, i=1,  i< max which kicks off and doesnt let you do anything until it is done, this replaces that by running however many it can between a stop and a start timed event, then comes back and continues where it left off the next start event. Thats what 'releases' the browser a little bit to allow you to do the update!


That's roughly what's going on, but it seems weird to make private and internal things inlineable, and yet you definitely want to be able to reference them from public inlineable functions. The important thing is that you can't leave behind any references to these functions, because the private members can't be versioned (in this proposal). So they'd become "always-inline into inlineable code".

We could do it; it just seems like it's easier to start with the simple thing that covers the 95% case.


Looks good.


Interestingly, it passed the tests - how could that happen? (something else must have brought T into scope there)


This was where the bug was


exactly.


Bug here too


@JDutil Hey, I know this is an old PR but I was wondering if you could elaborate why the StripeJS component was removed? Looks like it's related to #169.


How to start? where are the steps writeen to practice Swirl exercises?


Hm. We don't actually get any of the fixed-layout benefits from this, though, other than non-dependent-typed fields having fixed offsets. On the other hand, it's a generic struct, so it's not going to be *confused* for being efficient.

(And clearly we need Optional<some concrete type> to be fixed-layout-compatible.)


By the way, thanks a lot @slavapestov for reading these updates carefully!


`config/test.js` doesn't exist. Only `config/test.json`


Wait, disregard, I somehow missed lib/views/frontend/spree/checkout/payment/_stripe.html.erb


maybe we should change the condition into
```
if (Yii::$app instanceof yii\web\Application)
```


Buhu. You broke it. It was working before. Maybe there was a temporary problem due to some syntax change of the json file changes (bad pdbe). I will try to fix it again....
B


I think the idea of custom is because we are engineers at heart. We want to provide all the bells and wistles that the hardware can do.  Don't forget we are trying to make this a standard.  Maybe there will be a chip that doesn't support a custom.  What benefit does this drive to the client and to what expense?
I would suggest you remove the BlinkCustom and modify the GetCurrentBrigthness method to be that of GetLEDState (on, off, blink_fast, blink_slow)


Thank you for this. Is this because the 3 has been removed (i'm not sure why by the way) or is it because the label "van collects broken" isn't a suitable for label for a commit?


Thanks!


Shouldn't this be `new byte[0]`?


Awesome, cheers!
I'm free tonight (its 6 at night here now) and will be free to fiddle tonight and tomorrow all day/night.


I'd prefer `Collections.emptyList()` to prevent those ugly "unchecked compiler warnings".


Nitpick - Move this after GPSHeader.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 12418](https://teamcity.mesosphere.io/viewLog.html?buildId=110605&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 215 Build time: 00:04:12


There is no need to memoize ``operation``. If rails decides it's ok to cache controller instances, the first ``operation`` will be cached, and therefore the first ``article_model`` will also be cached.
Also it's 3 lines vs 1 and it doesn't help the comprehension.


uwsgi-plugin-python3 ;)


Hey @GammaC0de, you should really consider using the recommended `.format()` function. Plus, building a single element tuple like `"%s" % (result,)` for formatting looks pretty ugly, as stated in many sources ;) Notice my pull request on that.


I think it would better to rename this to `generateHeader`


This change needs to be made for the activity summaries in the emails as well:

```
You have been invited by <%= inviter.canvas_full_name %> to collaborate in the whiteboard <a href="<%= whiteboardLink %>"><%= whiteboard.title %></a>
```


I think this is fine, but it's hard to be sure.


Couldn't we use `TEE_OperationMode mode` or similar instead of `bool crypt` to make it more clear when reading the code?


But still #17


Yes, KnownPrivate, sorry. This maps to ASTNameLookupFlags::NL_KnownNonCascadingDependency.

I think I've convinced myself that this flag should always be present in error-handling code rather than making it contingent on the DeclContext.


we don't use `;` in lua for ending a line.


mogę to oprogramować ?
dźwięki pojedynczych akcji też będziesz robił ?


extra `;`


Looks good.


It certainly does, otherwise you'd get `$CFLAGS` set as `-defaultopt` rather than `-O2` ;)


The name of accessor and mutator should be set_client_id and client_id. Check other pkt header and Google style guide for example. You do need an accessor too. Otherwise, the id can't be extracted by a client to figure out whether the pkt belongs to it.


should be `options.includeEmail`


revert


In that case we need these unclean additional parameters in `PUT`.


The `com_maxfps` lagscript is unfortunately still enough to move through laser mines without dying :disappointed:


Could you move this to the same line with `local switch =` to keep the style consistent?


You need an accessor for type too.


should be `options.enrollmentStates`


It's cool. Just trying to make sure all the stuff stays together or it becomes a house of cards and then we will all perish! :skull:


Yep


Hello,

I'll submit the quizzes again then. Sorry about the mix up!
I'll get the proper javascript quiz in sometime before the end of the day.

On Fri, Jan 22, 2016 at 10:49 AM, Joe Shepherd <notifications@github.com>
wrote:

> Hi, Blair. The HTML and JS quizzes need to be submitted separately from
> their own repos created by following these links Steve put in the Slack
> channel:
>
> https://classroom.github.com/assignment-invitations/c3db57cfb77e7fc10d8bdd4389fe6de5
>
> https://classroom.github.com/assignment-invitations/36c0d875dd22a2720ed0decfb4f40988
>
> Also, this is not the proper JS quiz. The one you did is a leftover from
> earlier classes. We'll be removing that today to avoid further confusion.
> Make sure you follow the directions from the front-end-milestones repo:
>
> https://github.com/nashville-software-school/front-end-milestones/blob/master/the-static-web/quiz/JAVASCRIPT.md
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nss-day-cohort-12/blair-mcmillan-quizzes/commit/aeb5e334d292eb6473243cb0c69d2ffadbe8acc7#commitcomment-15617471>
> .
>


Can we add a test that ensures that email addresses are not included in at least the engagement index, the asset list and an asset comment?


? why not simply ignore it. We have the workerID, thats all needed.


I agree with you in general, but not for the specific case of missing a raw type, because you can't add that in an extension without manually writing the switch yourself. :-/ But it's true that it doesn't matter right now.


Variable names do not seem to reflect their purpose


k


LGTM


`GossipBroadcast` never errors in the current implementation. And I am not sure what the absence/presence of an error would indicate.


I get a segfault. Perhaps you meant buffer[0]+12?


I do plan to add a section on this, but I have never used either PAM or Kerberos before, and I wanted to test what I think should work before I do it.


getTries


Just wondering, would this work or not?

```php
    public function getLoader() : \ClassLoader{
        return $this->autoloader;
    }
```


configuration.  I'll fix this tonight.


no need to set stars_container as a self attribute here since it's not used outside of this function. you can just declare it as local inside the function and return it.


Shall I change the method signature to remove the error?


yes, I will fix that.


NB: I am not happy with this commit in general --- it changes too many things at the same time. It is good practice to keep commits concise, without mixing several topics in one and to avoid having feature development history in pull requests.

If it is not too much work, I would appreciate if you re-commit your changes. If not, no problem, I can do this myself when I will have a bit of time.

(I will now get back to reviewing the code .)


instead of (ok == false) use (!ok)


Jak zmieniłam na issue.title to kogoś testy się nie wykonywały. Moje testy działają


Yup, they are called that in Wyrmsun/Stratagus (the engine calls them hit points internally).


Aha, so is this a bug in the test code? The print above says "profiling addr queries", but name query is actually profiled.


`in`, `fek` and `nonce` could be `const uint8_t *` instead.


No problem, thanks!


sure, updated


You only changed `self.starts` here, which is not used/watched by the UI framework. What you need to modify is `stars_container`.


Why 200 ?


I tried to come up with an example but it ended up being much worse than I expected (and mostly unrelated). See [SR-599](https://bugs.swift.org/browse/SR-599).


Whats wrong with your identation?


@tp-clickapps yes I'm using windows for development.


Reword: Eg. "Custom monitoring and notification systems can be easily developed.  Messages are pushed to a message queue (Kafka is supported out of the box) and can be consumed by a message queue consumer."


No improvement. Now after resuming from pause it seems to produce stuttery audio for several seconds...


Change the function to bool FindMaxThroughputBS(int client_id, int radio_id, int *bs_id). So we don't rely on an inherent assumption bs > 0 to check whether it exists later. Also, no need to put const before int as it is not a reference. It's ok to use int/double etc for simple type.


Well, that was easy (just a tad annyoing that functions like this are nowhere to be found in the docs -- or I'm just too bad at searching...)! :)

Should we make the text in results selectable and introduce this kind of copying there as well?


`_defaultProfile.ClearPrefixes()` should work as well.
It gets rid of some of the code duplication since the implementation is the same as what's here.
Also gets rid of 3 lines of code


Simply ignore what?


This is working :|


The icalls in jit-icalls.c use mono_set_pending_exception () instead of mono_raise_exception ().


We need rewrite this. for `set` and `get` properties.


This will cache the layer even if the registry isn't configured as pullthrough, right?


Use range based for loop here, e.g., for (auto id : client_ids)


@tp-clickapps you're right. I use the ternary operator I think I miss it here


Added Tinker Rearm into key combo (disabled by default)
Added key combo switchable modes: Orbwalk, Move to mouse, Attack target, Do nothing
Added configurable key combo no-move range, where if your mouse is near your hero in the configured range it will not move
Added canceling for nukes single killsteal usage in case target is already dead or is not killable anymore while the ability is being casted
Added canceling for nuke skillshots in case enemy changed their direction while the ability is being casted
Fixed sometimes block checking returning wrong values.
Fixed sometimes skillshots not being used when enemy is out of castrange but the ability would still hit them thanks to its radius.
Fixed OnAttack AutoUsage sometimes not working
Fixed Tusk Snowball beign auto activated after manual usage (now it will be activated only if it was used by the assembly)
Fixed Omniknight Purification cast range check


You don't want to include this function.


Oddly enough, that compiles to one instruction bigger. But, thanks, I'll (try to) keep that in mind :)


Please use `framecontainer` for border styling. `centercontainer` should only be used for positioning. You should be able to achieve the same effect by using a `centercontainer` with `framecontainer` inside.


Use the constant defined above.


This is a bit strange -- there'll be a mismatch between the copied result and the displayed result.
Try e.g. the output of evaling `sin`, which will be much more verbose than the copyable view.


Use constant define above.


"https://dukesrg.github.io/?mset/rxinstaller.bin"

It only works when rxinstaller.bin is placed on SD://mset/

If the rxinstaller.bin is placed on SD root, this URL should be changed to "https://dukesrg.github.io/?rxinstaller.bin".


What's the correct audio format for a caf file, I'm not sure this is the right one.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 12419](https://teamcity.mesosphere.io/viewLog.html?buildId=110609&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 216 Build time: 00:04:08


What if I enter not an integer, but something else instead? Perhaps it might be useful to consider the use of exceptions.


Yeah, this is a defect in country DB. City DB code does it right.


You mean not empty? I'll make that change in a bit, sure.


@tp-clickapps Actually I'm using MySql with rails .. here I just want to try sqlite. What DBMS you use frequently?


"If you have used a previous version, please delete SD:/rxTools, SD:/3ds/rxTools, and rxTools.dat, if present on your (m)SD card."

Not only these files, but also SD://firmware.bin.


Again, variable name "x" does not reflect its contents. e.g. ( Hangman instance = .... ) looks better.


Created an issue re. this: https://github.com/agladysh/lua-geoip/issues/14


Actually can someone opt-out from a pullthrough regsitry?


I think you accidentally added this commit to BigstickCarpet/swagger-express-middleware#28.
Maybe you want to use a different branch?


Yes right. thanks!!!


Hum…


Yes


it works!
thank You and best regards!


-            debugCommand('ASK command, %s', id, '\n', command)


TestCustomCommand (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


-        if (connection.isLocalHost || replicationIds[0].indexOf(LOCAL_HOST) === -1) {


-  debugSocket('socket write, slot %s, node %s, length %d',


First check ids is empty or not and let the main logic fall straight through. E.g,,
if (ids.empty()) {
  Perror("WspaceController::ParseIP: ids are empty!");
}


Not a good practice to import the whole library if u need only a particular function


Caching doesn't matter, the defaults never change (that's what makes them defaults :) ).


here is not q&a page 〣( ºΔº )〣


Mark!


could do. It's mainly there for a) symmetry with `GossipUnicast`, and b) because all go functions must return error ;)


Hala


I'm just concerned about getting issues reported because users calling it incorrectly and plugin not working correctly because of it. What are your thoughts on this?


If an exception is thrown, it would be logical to catch it somewhere else. beta version?:D


I'm deducing from the build failure that starting up an alternate version of PostgreSQL uses `sudo`'d commands under the hood making it incompatible with Travis' container-based builds. Try removing this line to turn off containers.


對外連線網址
http://twentywork.southeastasia.cloudapp.azure.com:8080/TwentyWork/


Specify the function name in Perror to help debugging later, e.g., Perror("WspaceController::RecvFromBS: Receive invalid BSSStatsPkt");


ah, ok forgot who that this is a put. then alter identify to put and this method a patch.


@josejulio that is correct. Using min will result in 1 second being used unless 1/60th of timeout is less then a second... which is not what we want. For example, if timeout was 30 seconds, we would end up with a waitTime of 500L (1/2 second).

This needs to be changed to Math.max.


fdsf


Might be useful to implement GUI in order to easily abstract from the implementation details. Clearly, variable names could be more descriptive. And there is something wrong with indentation.


Move the header out of while loop. No need to allocate a copy each time


These are Italian. I have no translations for these. Should I delete them?


This plugin looks very cool and appreciated
good addition
https://fr.wordpress.org/plugins/ultimate-responsive-image-slider/


Instead of making it three times as long, could one not instead find other letters for the remaining two? Say for example "e" for editor and "c" for commentator/reviewer?


You're welcome, keep up the great work!


Good change to REQUIRED but pretty sure if REQUIRED is set this is not required so the if check is superfluous since a fatal cmake error will be generated.


Closes #5


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 12420](https://teamcity.mesosphere.io/viewLog.html?buildId=110610&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 215 Build time: 00:04:18


Should this really only happen when deactivating? Seems a bit brittle to me.

Also, this is a bit different than what normal Julia writes into the `.julia-history`:
```
# mode: julia
	include_string("sin()")
# time: 2016-01-21 22:48:06 Mitteleuropäische Zeit
# mode: julia
	@which include_string("asd")
# time: 2016-01-22 18:11:03 Mitteleuropäische Zeit
# mode: julia
	sin
# mode: julia
	1+1
```
First two items are from plain Julia, the other two from Atom.


This seems like excess overhead. This combined with getLog().debug results in 5 method calls. Although this overhead may be better then constructing a log message, in this case, I can't imagine the log message construction would be worse.

Am I basing my concern on outdated logging implementations and a non-optimized JVM?


This is the third time this pattern is repeated, please use a helper function instead.


Oh yeah this is my mistake, didn't fully clean out the "opt out" strategy for the opt-in strategy


@Charlotteis thanks


Don't assign type this way - appointing on raw pointers is generally a bad idea. That's the whole point of creating a header type:-)
Initialize the type in the constructor as it has a single type. See GPSHeader for example.


when will you port this to sponge?


because sponge is now available in beta form


IMHO `insert` should be a tactic as well, not a tactical.  It's easy to write a combinator that applies a tactic(al) to all subgoals separately (let's call it `onAll(insert(proof))` or even `insert(proof).everywhere`--then you can also do `decompose.everywhere`, etc.).  Applying `insert` to only the current goal is both what you usually want and makes the implementation much simpler.


Isn't it somewhat confusing to use both ```JOY_MAX_BUTTONS``` and ```JOY_BUTTON_MAX``` as enums? There's no way to tell from the names which one is meant to be used where.


When invoking os.listdir() on OS X the hidden file ".DS_Store" will be included which breaks the check if there are as many folders as classes.


i think the behavior as described in these tests is a little confusing as compared to its assertions, which causes this to seem like a magic number assertion.

we might want to consider changing the `It` description to read
`then it should configure backups for console, cc & uaa as postgres persistence`
and
`then it should configure backups for mysql, director and nfs`

even with that behavior change, these magic numbers still confuse me :)
not sure what behavior these assertions prove or disprove. perhaps it needs its own `It` block or `Context`


Nitpick - Put the implementation in .cc. and Move it before ComputeRoutes just like RoutingTable. In general, only accessor/mutator or locking with single line can be defined inline.


Ok, this part I think should be in the server.js.
``` javascript
function handleRequest (request, response) {
   try {
     //Disptach
     dispatcher.dispatch(request, response);
   } catch(err) {
     console.log(err);
 }
}
```
And even the definitions of the routes should be in server.js.
Maybe the name routes.js was poorly chosen by me :( shall we call it logic.js?

```javascript
dispatcher.onGet("/callback", function(req, res) {
  logic.getToken(function(err, token){
    if(err){
       res.end(err);
       return;
    }

    welcome(token, function(err, user){
       if(err){
          return res.end(err);
       }

        checkMembership(user, function(err, isMember){

             // And so on till we call AddStudent. Lots of callback hell! but since our callback functions are just mostly on line we won't lose our sanity. :)
         })
    })
}


Mi erano sfuggite un paio di inesattezze


Could you replace this with `_defaultProfile.CreateMap<TSource, TDestination>();` and get the same effect?
I can't tell, by just changes, if it helps eliminate, or add to redundancy in the code.


`hexCharToByte('A')` will fail, despite being valid. Perhaps you should use `aHexChar = tolower(aHexChar)`?


also added descriptions


```js
// 1.
_.chain({}); // IMO, we shouldn't force user's to enter new line here

// 2.
if (obj.method()) { // Same here. we shouldn't force new line here
//...
}
```

Looks fine.

Regarding:

```js
ob1.obj2.obj3.method2(); // Should there be newline after obj3?
```

I would say that nesting should happen after every `.`, i.e.,

```js
ob1
    .obj2
    .obj3
    .method2();
```

The same applies to the last example too:

```js
ob1
    .method1()
    .obj3
    .method2();
```


And every function in the logic module is tested.


char d = input.nextLine().charAt(0); What happens if I just enter a new line without entering any characters? I suppose the program crashes, since no exceptions and try/catch blocks


@flippmoke : qu'Est-ce que c'est?  :) Should `ogr.cpp` be testing ogr plug-in?


**TODO:**
日マタギをどうするか
一応、日を跨いだレコードは別セッション扱いできる

```ruby
row[label['session']] ||= "#{row[label['date']]}_#{row[label['client']]}_#{row[label['user']]}"
```


te muzyczki dodałem, ale daj znać co z pojedynczymi :)


same here. exception handling?


It's actually completely user controlled. Nothing in the plugin calls this method, it has to be called from the client app, and the implementation of the method is also handled by the user in the GeofenceListener. You could probably call the function "OnCustomEvent" and it would have a similar effect.

I think the only confusion that might come up is if a user expects this to be automatically called by the plugin for some reason, which is why I made the comment in the Readme that you have to manually call this method from your app.

If you think that might be too confusing, though, I could take it out and we can figure out a different solution for our scenario. Or, if you think renaming to something like "OnCustomEvent" works better, that is another option.


strzał i skok chcę jeszcze dodać. Coś jeszcze by się przydało?


TODO is what you are going to do:-) E.g., TODO: Enable dynamically assignment of a radio number.


Ups, my error, missed last update, but was sure that already downloaded it.


готово


Red highlighting on Github. Something wrong?


we're cooler :P


I don’t see any changed .ceylon in [this diff](https://github.com/ceylon/ceylon/compare/1142ef5c9425...28a0b31cbeef), was that done earlier?


Can we put this in the same config that we have the scavenge polling?


This shouldn't have been removed... See: 9f2f1ddaaac1b479049af943a268e88af8d1acbe

Bad merge?


Nice use of design pattern


I still do not understand the reason for :(


I'll fix this manually. Thanks


Please, write dscripting commit messages... "Let there be light" tells me nothing! About this commit. Can you describe here to me, what is thi socmmit all about? Thank you!


I would use `self.metric_type` just to avoid mistakes.


Sorry, going back to numpy 1.9.2 doesn't work any more as the version installed at Travis is now 1.10.1 (which seems to be binary incompatible with 1.9.2).

I have disabled the test, so that the build should pass on Travis now, but the bug is still open.


semicolon


ah ok.. worked for me in some spots.  maybe bank switching interleave ate up any savings.


Great!!


Can you update the list of table alters that are required for this PR ?


Because the 3 has been removed :)

> On 22 Jan 2016, at 16:56, Russell Vaughan <notifications@github.com> wrote:
>
> Thank you for this. Is this because the 3 has been removed (i'm not sure why by the way) or is it because the label "van collects broken" isn't a suitable for label for a commit?
>
> —
> Reply to this email directly or view it on GitHub.
>


Eine Schleife oder ein Split, die nach \n die reinkommenden Nachrichten zerteilt, macht es nicht einfacher, da mehrere Varianten beachtet werden müssen.


ok got it.
yes not a big deal. Always a matter of compromises !


How to remove in stars_container (CenterContainer) their children? The WidgetContainer has not such methods.


Can this constructor call `this(clock,recovery,Collections.emptyMap())`?


__API Description file syntax error on line undefined__: ETIMEDOUT (Autogenerated message from apiary.io)


Операторы ветвления


I'm working on "fuzzy" matching instead of exact string matching because there are some answers that have frustratingly close guesses that don't match i.e. "snake" and "a snake".


That's what I wanted to do but I couldn't figure out how to get past the issue of importing the Interop.Sys.GetEUid file. I suppose I could just declare the extern in the test file manually and not use the Common/Interop files.


Using exception as flow control mechanism rather than error handling is a bad practise. (As Henry says, losing a game is not an exception even if it only happens to you exceptionally)


same as `centercontainer`, no need to add bordersize.


I don't mention firmware.bin because rxTools is not the only firmware to ever have it in the root, and it won't affect anything one way or the other.

Link for the rxinstalled.bin... Never used it, didn't write it, not relevant to my pull request. Which was merged already, so if it bothers you, make another pull request, or start another issue, and I'll fix it.


Maybe add a little comment about what the class does?


I also get it from Travis now..


This is not javascript style avoid underscore and hyphens - change this to 'creationDate'


Good practise to comment your code (y)


What buffer overflow? `i` can have only values from `0` to `_rankColor.size()-1` in that line. With of this values cause read outside of buffer? Cast in that place will not change any thing because value is already cast to `size_t` by vector subscript operator.


fantastic :rocket:


Usually in Java, a good practice is to restrict your variables as much as you can so these should be private


I don't see why it's needed either, by avoiding it the code could be simplified.


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#Te2OxXknTTivhroq4iiqDQ


Done.


Overall, I very like your code. Nice abstraction from implementation details. Although you could create an interface and then implement TextUI which implements that interface, so that if u decide to develop graphical UI the migration to graphical from text UI could be performed very very easily.


how to click on fancy button android


There seems to be 'gofmt' missing on these lines.  You have to run 'gofmt' to fix these..

```
// ObjectInfo container for object metadata.
type ObjectInfo struct {
        // An ETag is optionally set to md5sum of an object.  In case of multipart objects,
        // ETag is of the form MD5SUM-N where MD5SUM is md5sum of all individual md5sums of
        // each parts concatenated into one string.
        ETag string `json:"etag"`

        Key          string    `json:"name"`         // Name of the object
        LastModified time.Time `json:"lastModified"` // Date and time the object was last modified.
        Size         int64     `json:"size"`         // Size in bytes of the object.
        ContentType  string    `json:"contentType"`  // A standard MIME type describing the format of the object data.

        // Owner name.
        Owner struct {
                DisplayName string `json:"name"`
                ID          string `json:"id"`
        } `json:"owner"`

```


Mhh.. it's private, so this can only be used inside InputDefault and describes how many buttons the remapping system can keep track of.
`JOY_BUTTON_MAX` from InputEvent can be accessed from everywhere and is the number of buttons our "standard gamepad" has.

I agree it's somewhat confusing, what should this be named instead?


Do you need a getter if it's anyways protected?


для ul можно класс вообще не писать, так как особых стилей к этому тегу применяться не будет, будут только обнуляться стили, которые по умолчанию идут.


Did you not like my suggestion of creating a `contains` function for a string so we can do:

```
assert.ok(contains(simonsEmail.email.html, reply1.body));
```

:cry:


ok I got the point. Good catch.
However I'm not 100% sure of the count check yet. Have to give it an additional thought. maybe useless but when it comes to options, I'm always scared... :)
I'll merge this one later.


There's a ton of places where we could then make the tests a bit cleaner


I dont think what Henry suggests is necessary as the comment above explains quite clearly what difficulty means


Ah, I see why, nvm


Actually, I think the previous version with bs_id is more clear.


I might be wrong, but isn't it supposed to be:
`click-to-open-select-control select`

This way I'm getting the error I was getting in the browser without the if condition. Otherwise the tests pass either way.

I see the same thing for the "delimiters" test also.


As Henry says, this is not the way exceptions should be used and also in this particular case, it is totally unnecessary!


new version


My personal opinion is - RN devs actually want to add system support of many things that are missing in Android, eventually. But up to that time, they encourage other to create modules like this that create a replacement (maybe they than actually integrate that code themselves, after some changes). I remember there were a project for WebView on Android, and from 0.16 or 0.17, it was added to RN core.

As for using both this module and internal thing - I would not care about it until it happens, and then you can make changes that remove that clashes, should they happen (and increase the version so that it is breaking according to semver).

Pokai Chang wrote:
> (I'm new on both the npm ecosystem and maintaining open source
> projects, so any suggestions on the next move will be very welcome!
> And also sorry about my bad English.)
>
> We're planning on merging these two modules, but before that we
> thought this feature might best belongs to React Native Core, I think
> it
may be good to ask React Native maintainers for suggestion, if they
> have any plans on |PushNotificationsAndroid|, or the API design, etc.,
> before to proceed on development.
>
> But another concern is that people might want to use both GCM and push
> notifications on Android at the same time, if GCM is wrapped into the
> notifications module, I'm not sure if |gcm-android| and
> |system-notifications| will work together without conflict. Or it may
> probably leads to duplicated services running on the system?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Neson/react-native-system-notification/commit/57d23a7120d57f83f1d7be6755c1334431fcd4cf#commitcomment-15608720>.
>


That is incorrect. That patch would make only comments made by this author come up. Not what is wanted here.


@gkalpak here you go:
feat($resource): add support for timeout in cancellable actions #13824


Good practice catching potential errors


There are still a few places where this is a problem:

```
node_modules/col-activities/lib/api.js: * @param {User}            user                The user for whom to update the timestamp at which the last activity took place
node_modules/col-activities/lib/api.js: * @param {User}            [actor]             The user who triggered the activity
node_modules/col-activities/lib/api.js: * @param {Function}        callback            Standard callback function
node_modules/col-activities/lib/api.js: * @param {Object}          callback.err        An error that occurred, if any
node_modules/col-canvas/tests/model.js: * @param {CanvasUser}    user          The user who created the discussion entry
node_modules/col-canvas/tests/model.js: * @param {Number}        [parentId]    The id of the parent entry in case this is a reply
node_modules/col-tests/lib/util.js: * @param {Number}    userId      The id of the user to expect a total number of emails for
node_modules/col-tests/lib/util.js: * @param {Number}    total       The number of emails to expect
public/app/shared/utilService.js:     * @param {Object}  data    The data for the parent's hash container. Each key will be prefixed with `col_`. For example, `{'user': 1, 'category': 42}` would be serialized to `col_user=1&col_category=42`
```


:bug: Yeah I'm not sure that sorts anything at all...


Thank you, Manuel, for the looking into this. I'll try to address issue #6 when I have time.


Its okay, I am here to confuse everyone.


yeah. Rock n roll. Like this and your explanations on #477


TestCustomCommand (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (2050 of 2050 tests passed, 0 test warnings)

 * 3 tests had: AUD diffs.

![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)


Change to view.js


This might fail if the email sending ever takes longer than an hour. It might be best to use the configured email sending time explicitly.


delete init file


TestCustomCommand (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (956 of 956 tests passed, 0 test warnings)

![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-26.85%25-green.svg)


$webMemoryLimit in CLI message..


Nice use of the anonymous inner class


Consider making this zoom (tile coord) 12 or 13 instead. Otherwise the tile rank test is almost always going to be 1 because there are too few other stations to compare with.


This is a prerequisite, put into set-up


For these 3, same comment:

Consider making this zoom (tile coord) 12 or 13 instead. Otherwise the tile rank test is almost always going to be 1 because there are too few other stations to compare with.


You should also change the license badge to match the new license.


Written this way, the assertion is done on the list. We want to make sure this particular contact appears in the list.


I pushed another fix: https://github.com/FernetMenta/xbmc/commit/c38469ae59a24910f8f2575066f3d0b5225e9201

this packet is responsible when player is paused


OK. And 3 and 4 should be forced all the time even when used in `if/while/for` etc.?

Should this
```js
a.b.c.d
```
be forced to
```js
a
    .b
    .c
    .d
```
?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 12421](https://teamcity.mesosphere.io/viewLog.html?buildId=110613&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 215 Build time: 00:04:22


The indentation is a bit inconsistent across the PR. For a matter of consistency, I'd recommend to use an editor that supports the .editorconfig file defined for the project.

After that, you won't have to mind about 1/2 or more spaces.


You're using exceptions for flow control and integers for error-handling, that's not exactly the best practice!


Fix #166.


Nit: Unused.


See, this is a nice use of exceptions! :D


Why the extra factor of 4 here, `(2ULL << SPA_MAXBLOCKSHIFT) * 4)` instead of `2ULL << SPA_MAXBLOCKSHIFT)`?


JavaScript won’t be bundled in a giant file, but delivered as single files in assets. This avoids having to parse the whole code on app start, if you have a large app and use techniques like lazy requiring parts of the code or an inline-require transform.

It’s not quite finished yet, but close, and the name will change to something sensible. So please don’t tweet etc. about it just yet :-)


Nit: In Firefox OS, we use tap() instead of click().


it’s opt-in, of course :)


Here, you return the function itself, instead of its result. This makes assertion done there => always pass. That is to say: the test is prone to be a false-positive, which can be harmful.

IMO, we don't need to return anything. this.waitForFavorited() will wait until we make sure the icon is there. If the icon never comes, it'll time out with a nice and meaningful stacktrace.

As a consequence, the assertion assert.notEqual(favorited, false); is redundant with the wait, so we can get rid of it :)


Катя, никаких инлайновых стилей. В файле с  html только html. Надо разделить на два файла - style.css и index.html и подключить в теге head через соответсвующую конструкцию. (<link rel="stylesheet" type="text/css" href="css/style.css">)


What is the advantage of declaring a temporary variable, here?


:warning: Confusing indentation


говорят что еще и грузинский надо ;-)


Yeah I see your point.
Anyway I'm pretty sure it's fine :)
But consider that is just a check to not try to diff something which doesn't need to be diffed :) - meaning it checks just that it's already fine - it can be removed and diff it anyways.
Also the diff acts on the pair "key" => "value", if the value differs between the two arrays, the pair of the first array will be kept -> http://sandbox.onlinephpfunctions.com/code/38d1a6a8ea7613f15ae85eab30d140878e870c34


@davidaurelio Ah, really awesome. Looking forward to it. BTW it was already in the RN newsletter :)


Hehe :trollface:


I agree, the selector is kinda screwed :smile: . It doesn't really return the list as 1 single element. Instead it returns many elements.

Then, if you wait for a single element, it'll return the first one that matches the selector. In other words, we should either rename the function to firstContact or use this.client.findElements()


Added.


$webMemoryLimit in CLI message..


Having this.subject feels weird. The dependency chain should be:

AppClass --> View --> Accessors --> Marionette

A view depending on the AppClass creates a circular dependency which doesn't look graceful.

I'd remove that line and implement the wait in the constructor of ContactsDetailsView. When initialized, that view should make sure it's ready to be used. So the transition should be over.

Then, we can remove this.subject in views.


Test Messages.


ok got it for the delay.
Could be a localized parameter in the future. We'll see.


No, it is used in other tests.


Отступы надо выравнивать. Напиши мне, какой у тебя сейчас редактор, найдем тебе плагин, который будет ровнять за тебя. Сейчас h1 и div под ним выровнены не корректно


@afelisatti applied changes based on comments 7b00940e5c20f72033eecd14ba360b1fb3f96189


I have the same issue as kwirk, only contacts with an email address are returned after doigna reload or using the search.

I first noticed when the contacts synced on my phone via DAVDroid are much less than they should be. So I reloaded the contacts in webmail and bam, most were gone. After a sudden panic I realized I good restore them from backup. After doing so I still seemed to lose them, but only after a reload.

After changing the @"." to @"%" as kwirk suggests, it makes no difference for me, still broken. I also use postgresql. But in a way I'm glad it's not DAVDroid at fault, and it should be fixable fairly easily. I've been concerned that I wouldn't have reliable open source CalDAV/CardDAV sync tool for android, after working well for so long. It disturbs me that as soon as it got mature, they swapped the engine out and it breaks sync with SOGo.

I'll try to patch it myself and report back.