There was missing favicon for the admin panel.
Meta tag was added. Favicon's file was replaced by file with correct extension.


Do we need to spread this array, as opposed to simply calling `header.push("Description", "Install Metrics")`? It doesn't seem like we need the array for any other reason.


Hello guys, what do you think about my changes?
Thanks


OK~


Done.


Ack!  No, it should just be doing a jquery `.remove()` on the `li` element dynamically, no page changes were intended: https://github.com/GSBCfamily/bvcms/commit/ecd0ddf8d57b1a5c211ec5c1b9ed85e0e43d66a0#diff-f4b4d22fa7da9645b79381eba478f14fR253

I only tested under Chrome on Windows 10 and it was working fine.  I'll fire up the virtual machine tomorrow and test it under Edge and Firefox too.

Thanks for the bug report -- I'll get it fixed.


That works! Thanks so much. (Again.)


Not at all. I just decided to use the Pelican domain for SITEURL at the test instead my own, pxquim.com. Any other suggestion to use as "dummy" domain?


Good point! Changed the wording to what you suggested and wrapped the whole file (minus the command line below).


Please don't make unrelated changes in PRs like this. This is going to require all other Deployment-related PRs to need rebasing.


Почему реализация методов класса в заголовочном файле?
Для "MyUtils" нельзя придумать какое-нибудь другое название? Эту штуку вообще нужно выносить?


As discussed offline, 2 to 4 looks awesome and 3 is for the weakhearted.


Why do we need to support the old Java 6 version? I think we should focus on the new version instead.


What is this being used for? I'm not convinced it makes sense for an implicit expression like this to really have a range of the whole expression. You have the CallExpr containing the TypeExpr if you need the full range.


Sounds good to me, and agreed that it makes this commit nice and confusing! 🐙


Can't we just enforce that the new expression has a valid source range? I'm concerned about the memory usage here.


TeamCity Waterfall :: Build [Build 25](https://tc.demonwav.com/viewLog.html?buildId=214&buildTypeId=Waterfall_Build&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 43 Build time: 00:00:47


I would suggest calling this "getRawText" instead of "getFullText".


Not that this helps you but I ran into a similar issue and it turned out the scrambled text only seems to happen when the receiving terminal program was using editline instead of readline.


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.6-AddressSanitizer: OK (3041 of 3041 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)


Why?

Also, replication controller seems to have it in its own utils. Not sure why it shouldn't be with replica_set_utils.go


commited;


Hi @viswanav,

We recently moved this decorator to the topology core.

Check how this was refactored:

https://github.com/HPENetworking/topology_lib_iperf/blob/master/lib/topology_lib_iperf/library.py#L25

Could you please also do the same?

Thanks


This code reused the variable, so this is a no-op. I think you need to use "arg" instead of "argParen".


Why? If I write `nil`, the `()` being passed to the constructor isn't explicit.


You don't need quotations here.


My bad. Forgot to add `PR-URL`. Located at https://github.com/nodejs/node-eps/pull/2


The ``parse_pid`` function returns an integer, or an exception. It makes no sense to check this is different than the empty string:

https://github.com/HPENetworking/topology_lib_sflow/commit/fd12ac7acb12742969c20926d8ed47e86cabec81#diff-d985b9642ce9b0faacefb44cc53eaabfR47


dank commit fam


This got the test to pass. I think I got it right...was this what you were trying to explain: I needed to check that the item_from_other_department's department_id was not the same as item1's department_id?
@droberts84


No, it should only be a problem if multiple @ThriftField annotations for the same field (e.g. on setter and getter) specify different maps. If you use a map with multiple entries, but you only put it in one @ThriftField annotation or you put the same map in all @ThriftField annotations, it should work.

I will add a test to demonstrate multiple annotations.


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


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


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


@larsrh thanks for catching this :)


I don't think we can do this. Apart from having a *lot* of overhead, I don't trust that it will actually work the way you want. There's really no guarantee that a Token still exists once an AST is built, and there are a lot of places where we try to re-lex something without building a whole parser.

What is this for? Can you add tests? Unit tests, perhaps, rather than lit tests?


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


I don't actually see any code changes in this release. Am I missing something?


Yeah, I feel weird using cpp.ref. Partly because it starts with cpp, and partly because it actually has legitimate uses outside of recursion (e.g. when you just want to use a ref instead of a contained struct).


I'll have to have a look at that.  I'm unable to devote brain time to it right now, but I'll get back to this.


What's an example of a TypeRepr with a valid endLoc where the typeLoc is still implicit?


The tests use round-trip encoding, which does an encode, decode, compare. w/o equals implemented compare would fail because it would compare references (which won't match since we've rehydrated the object from serialized form).


Done.


This blew up in the test because the version of composer that Travis has by default is too old. Can you move it to after the self-update below or do we need to ensure that tls is disabled before then in the case of newer Composer builds?


This and line 154 are the actual bugfixes.


Oui mais je sais pas si je vais pouvoir faire mieux, j'arrive pas à débugguer. Les débuggueurs prennent plusieurs dizaines de minutes à fonctionner...


ah mince, tant que ça :/


This project is just a build harness and set of patches for building the upstream plugin for libpurple-2. The code change resides [upstream](https://bitbucket.org/pidgin/main/commits/92885e0456edad1a04d6dedb418e5d86bd7a3be6).


Why does the workaround you posted work? It seems like the hash gets reinitialized, but isn't this an issue with MIME types getting cached after boot in response to malicious requests? It seems like the default value proc the hash receives is the same too. Just curious!


Thanks bmrussell, these changes seem useful, I just have a few comments before merging.

1. I get a build error saying "Properties" doesn't exist in the current context on line 26 and 52.
2. Please make sure all code follows the One True Brace style to be consistent with the rest of the project.
3. The config file isn't currently being used to it's full potential, perhaps instead of setting the picture position manually on line 16, we can fetch the config file settings (also please change the default to "Fill" instead of "Fit").
4. Move the "GetPicturePosition" method above the "SetPicturePosition" method just to have relevant methods together.
5. Move the "GetProxy" method above the "DownloadBackground" method for the same reason as above.

Thank you!


maybe use the feature name so `std_has_cstr_borrow`:

https://github.com/rust-lang/rust/blob/master/src/libstd/ffi/c_str.rs#L317-L320


See https://github.com/milkypostman/melpa/pull/663


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.6-ThreadSanitizer: OK (383 of 383 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)


LGTM


Checking 6x-1,6x+1 is faster, but the current version is already 0s on all the HR test cases.

    isPrime n = all ((/=0) . mod n) . takeWhile ((<=n) . (^2)) $
                2:3:concatMap ((\v->[v-1,v+1]) . (*6)) [1..]


the naming convention is quite common for private vars IME - I think made more popular by tools like resharper.

It helps distinguish between class and local vars but without resorting to hungarian notation `m_`.

I don't know if sonar has such a rule but it probably should - or we can make one :)


TeamCity Waterfall :: Build [Build 26](https://tc.demonwav.com/viewLog.html?buildId=215&buildTypeId=Waterfall_Build&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 43 Build time: 00:01:23


@KiminRyu Thank you!
Quite helpful!


Can you type your parameters even if they accept multiple values? Such as `\Symfony\Component\Console\Helper\TableSeparator|array`.

Also, please prefer the FQN in the phpdoc.

Thanks!


Also see [Issue 122](http://github.com/weidai11/cryptopp/issues/122).


4-varandstring incomplete


Good catch, @esYon!  I'm planning on resolving the issue of the multiple index.html files now-- since the frontend folks were working on one separate from the one we were working on previously (which resulted in this issue upon merge).  I'd recommend also to get an entirely new API key if you haven't already done so for the GoogleMaps API since this old key will still exist in the commit history.


this is compiled for any Linux board and board not based on BBB shouldn't know about BBB's gpios. So here you have 2 options: a) add bbbmini to the ifdef or b) allow the scaling to be configured in the constructor so whoever is constructing it pass the scaling factor. (b) is preferred since it prepares the ground for other boards as well.


Thanks fam


TeamCity Waterfall :: Build [Build 27](https://tc.demonwav.com/viewLog.html?buildId=216&buildTypeId=Waterfall_Build&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 43 Build time: 00:00:49


@Atahan-Turkoglu Looking at it like that is misleading. You have to look at the numbers mathematically:
Suppose you decide to change the bounds from (-2.94,2.94) to (2.30, 2.30). That increases the probability of Type I and Type II errors from 5% to 9%.

What this means is that tests that reach LLR +2.30 can fail quite often when (-2.94, 2.94) bounds are used. And most importantly, that's a good thing, because the +2.30 test that failed is more likely bad and "deserved" to fail.

Intuitively (not in a statistical way, because it introduces conditional probabilities), if a test went from +2.30 to -2.94, then if you considered the test to have started at +2.30, it would have failed (-5.24, 0.64) bounds. So when you say "I was sure it will pass," but then the test fails, that can be a good thing because the +2.30 was likely misleading in the first place, and we want patches that are not improvements to fail.


What's the use of this?


So, by the end of this week, I'm really hoping to get a lot of these loose branches cleaned up.  We really should only have one branch per feature (outside of develop/master), rather than one or multiple branch(es) per person since it seems some of the old branches are really not up to date with develop.


По идее, чтобы понять авторизован или нет пользователь нужно смотреть localStorage['token'], это плохой вариант?


erleboard is using this ifdef so this cannot be changed. However as I said above, we should use the minimum amount of ifdef possible.


넵 res.send()로  http response 받아야 다음으로 넘어갑니다.


Can you update the comment to be a bit more specific with what's happening? Ie the formatting difference.


"Все игры"?)


TeamCity Waterfall :: Build [Build 28](https://tc.demonwav.com/viewLog.html?buildId=217&buildTypeId=Waterfall_Build&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 43 Build time: 00:00:48


Done.


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (389 of 389 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)


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.6: Build Failed

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


Could line 290 onwards simply be written as:
```
if let capability = capability {
  tokenParams = ARTAuthTokenParams()
  tokenParams!.capability = capability
}
if let ttl = ttl {
  if tokenParams != nil { tokenParams = ARTAuthTokenParams() }
  tokenParams!.ttl = ttl
}
```


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


This doesn't compile.


придумывание имен классам и картинкам с непривычки отнимает уйму времени :)


Изменение max-width у .container делает окно pages уже, чем главная страница. Это можно выпилить?


> asserts or llvm_unreachable instead of silent acceptance for unexpected conditions

Please do not do that for this code - or at least make it so that clients of this code can choose their preferred behavior

A node visitor that asserts and dies is going to be practically unusable for LLDB; and I really really want to avoid having to maintain our own copy now that this code lives in the compiler


I didn't think creating those extra variables would have a noticeable impact on performance.
Is it worth it to go through and delete those local variables?


TeamCity Waterfall :: Build [Build 29](https://tc.demonwav.com/viewLog.html?buildId=218&buildTypeId=Waterfall_Build&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 43 Build time: 00:00:52


This project hosts a map-based interface that simplifies the process of identifying the Address API endpoint given a specific neighborhood name. After a neighborhood is chosen, a endpoint is provided that supplies all addresses within that neighborhood and associated variables.

![image](https://cloud.githubusercontent.com/assets/3280502/12569321/5fab6862-c393-11e5-90b5-6ff3208fd03b.png)


I think the first value here should be `APS2_COMMS_ERROR`


aha, I see. That makes sense.


thanks!


If you are writing django code yourself, and are seeing this same error, wondering why it shows up if it's been fixed:

I got here today.    Are you using an interactive python prompt, and doing a 'reload' on a module where you defined models? That's what got me here.  The warnning you see in the prompt ("don't reload the models.py file, it can break things") - this is one of the things that breaks.

 You can fix the problem just by exiting the REPL and starting a new one.


uhm... i don't remember my original logic for this. It might have had something to do with gx though (although gx only uses patch for publish, which also requires add so i don't think thats it.)


I'm probably missing something obvious here, but if the token has expired and there is seemingly no way for the client library to renew the token as it only has a `tokenDetails` option, how is the test passing?


Again, probably missing something as 2am!  But why does this test fail and the other pass?


Leave this `const`.


@sgkang can you branch off dev in the future and merge in using a pull request?


The API changed, so I think that it should now be corrected to something like `Container.BindIFactory<FooMonoBehaviour>().ToPrefab(prefab);`.


Don't use implSymbol. use implForWrapper.


develop (lefticus) - 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-warnings-yellow.svg)


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

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


Yes. will do.

> On Jan 25, 2016, at 4:54 PM, Rowan Cockett <notifications@github.com> wrote:
>
> @sgkang <https://github.com/sgkang> can you branch off dev in the future and merge in using a pull request?
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/simpeg/simpeg/commit/32f3ef301aaa54e98589e07507c87fb665b73cda#commitcomment-15665845>.
>


Right on, thanks for the heads up


The same here, we need to remove the `<TMonoBehaviour>` part.


do I need to call jv_free(input) before these errors?


I'm new to `const` / `let`, thought the change was required because i'm adding a new property to the object further down. But I guess it's only required if I replace the full object?


this is uglify code, not ours, please add an `// XXX EMSCRIPTEN` comment


style


Exactly.


It was not pushed to gcr because of the fact that Kube repo has not been
updated yet. But there is no harm in pushing all the images to gcr in
addition to dockerhub. So I went ahead and did that.
Can you try again?

On Mon, Jan 25, 2016 at 3:10 PM, Elias Levy <notifications@github.com>
wrote:

> Shouldn't GCR have been updated with a new image v0.6 tag?
>
> $ curl  https://gcr.io/v2/google_containers/heapster_influxdb/tags/list
> {"name":"google_containers/heapster_influxdb","tags":["v0.3","v0.4","v0.5"]}
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kubernetes/heapster/commit/669c5c388304d475be841a8edb108223c420504d#commitcomment-15664086>
> .
>


I've already changed that in a subsequent commit. Do you prefer a new PR with all changes in one commit in the end? There is more coming I think. There are still issues with "data:" URLs.


reported by: Hunger
idea by: Hunger

hardened/current/unstable branch first


@paddybyers?


np fam


Reverted.


at the next time please use git pull --rebase when pull in the new hardened/{10-stable,current}/master


@gakimball I was getting a branch setup to work on the ES6 migration and noticed there's a merge conflict here.


It is wxi, typing mistake. Will be corrected


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 12703](https://teamcity.mesosphere.io/viewLog.html?buildId=112500&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:18


We can just rebase after you're done.


I need to debug the problem, whatever it is.


Is TTL in seconds or milliseconds elsewhere?  In Ruby durations are in seconds, but then transformed to milliseconds when communicating with the API


I'm pretty sure I'm using tabs here. Is that wrong?


It was a pull but it didn't pull everything for whatever reason. Does 05f4b55 lg2u?


Also see [Issue 122](http://github.com/weidai11/cryptopp/issues/122).


I messed up the rebase, the commit title should be `httpd24: 2.4.18` :disappointed:


One more for luck


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


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


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


Though Hunger brought it up, this is something that I've had in my head and wanted to do for a while. Next time I'll commit to unstable first.


OMG what is this.


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


holy SHIT


どの修正です? 一括置換しているので、そうそう抜けはないと思うんですが、cherry-pickしそこねたので否定はできません。


TeamCity Waterfall :: Build [Build 30](https://tc.demonwav.com/viewLog.html?buildId=219&buildTypeId=Waterfall_Build&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 43 Build time: 00:00:52


ROGUE AS FUCK


### ROGUE
## AS
# FUCK


Why are you commenting out glasses?


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


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


Should $runner default to 'auto' or FALSE? Param needs doxygen please.


I'll update the help text. Thanks J.


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


確かに `サーバー` に関して、大量に抜けてますね。
指摘、ありがとうございます。

いま作業手順を再確認したのですが、なぜ抜けたのかは不明。
修正します。


But I worked so hard to keep everything out of the function...


ага, перемудрил. не подумал, что js'у не трудно будет выдавать сразу "найдено: 3" или "найдено: 10". думал оставить "найдено:", а от сервера получать только "3"


I don't mean asserting on an invalid mangled name. I mean things like a null ASTContext. Swift code should not be wasting a branch on that.


LGTM


Same for other things that are AST invariants, like "all unbound generic types have nominal type decls". It's fine that LLDB doesn't want to know all of our AST invariants, but we should not hide invalid ASTs behind silent failure.


I *think* doing doing that will reproduce the original issue if the field is required. So it will be invalid by default, you enter the month - then it will still be invalid and we won't pickup the fact that it is also `badInput` now...


fixed in https://github.com/vim-jp/lang-ja/pull/23


@zacscodingclub This looks great, well done!


Weird, there seems like as an white-space issue. Btw, it would be nice to bump hardenedbsd related defines to higher value, to avoid future merge conflicts. If it possible, for example start from 200 or so..


А зачем целый core? Для Spring Data нам hibernate-entitymanager вдоволь хватит


@aleneum and @wtgee, I pushed directly to master since this is just a minor README change, but let me know if you want me to open PRs for everything, including minor tweaks like this.


What is the class used for anyway? As far as I know, it hasn't changed anything.


Наследуем от BaseEntity


Sure. Would you like me to change that with this feature?


We already pass a bunch of different events in so I'm not sure if this is worth it. IE passes `keydown`, IE might also pass `cut` and `paste`. I feel like normalizing all those is a bigger task... (but probably a good idea)


TeamCity Architecture Team :: Java Platform :: otj-components :: otj-data-structures :: build [Build 75](http://teamcity.otenv.com/viewLog.html?buildId=4039570&buildTypeId=ArchitectureTeam_JavaPlatform_OtjComponents_OtjDataStructures_Build) is now running


TeamCity Architecture Team :: Java Platform :: otj-components :: otj-data-structures :: build [Build 76](http://teamcity.otenv.com/viewLog.html?buildId=4039571&buildTypeId=ArchitectureTeam_JavaPlatform_OtjComponents_OtjDataStructures_Build) is now running


TeamCity Waterfall :: Build [Build 31](https://tc.demonwav.com/viewLog.html?buildId=220&buildTypeId=Waterfall_Build&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 43 Build time: 00:00:48


Ehh, nothing. It's good as is.


ciestupinan: cristina estupiñán


TeamCity Architecture Team :: Java Platform :: otj-components :: otj-data-structures :: build [Build 75](http://teamcity.otenv.com/viewLog.html?buildId=4039570&buildTypeId=ArchitectureTeam_JavaPlatform_OtjComponents_OtjDataStructures_Build) outcome was **FAILURE**
Summary: Tests passed: 37; exit code 1 Build time: 0:0:0


TeamCity Architecture Team :: Java Platform :: otj-components :: otj-data-structures :: build [Build 76](http://teamcity.otenv.com/viewLog.html?buildId=4039571&buildTypeId=ArchitectureTeam_JavaPlatform_OtjComponents_OtjDataStructures_Build) outcome was **FAILURE**
Summary: Tests passed: 37; exit code 1 Build time: 0:0:0


TeamCity Architecture Team :: Java Platform :: otj-logging :: build-config [Build 81](http://teamcity.otenv.com/viewLog.html?buildId=4039583&buildTypeId=ArchitectureTeam_JavaPlatform_OtjLogging_BuildConfig) is now running


TeamCity Architecture Team :: Java Platform :: otj-logging :: build-config [Build 82](http://teamcity.otenv.com/viewLog.html?buildId=4039584&buildTypeId=ArchitectureTeam_JavaPlatform_OtjLogging_BuildConfig) is now running


TeamCity Architecture Team :: Java Platform :: otj-logging :: build-config [Build 81](http://teamcity.otenv.com/viewLog.html?buildId=4039583&buildTypeId=ArchitectureTeam_JavaPlatform_OtjLogging_BuildConfig) outcome was **FAILURE**
Summary: Exit code 1 Build time: 0:0:0


TeamCity Architecture Team :: Java Platform :: otj-logging :: build-config [Build 82](http://teamcity.otenv.com/viewLog.html?buildId=4039584&buildTypeId=ArchitectureTeam_JavaPlatform_OtjLogging_BuildConfig) outcome was **FAILURE**
Summary: Exit code 1 Build time: 0:0:0


Looks like this got added back again in your last commit.


Have already fixed this (didn't save file in editor!) in next commit.


コメントありがとうございます。
たしかに、else: ルートに入ると、エラーしますね。。修正が必要です。

ただ、もともとやりたかったのは

else: ルートに入る
　→type_に不明なプロパティ種別が入っている
　　→処理が不可なので次のプロパティ種別に遷移する

だったので、
pass ではなく continueを入れるのが良さそうですが、いかがでしょうか。

処理のイメージとしては、
propertiesリストの中身が
[ OFPPDPT_ETHERNETのパラメタ、謎のタイプのパラメタ、OFPPDPT_EXPERIMENTERのパラメタ]
となっていた場合、
　1.OFPPDPT_ETHERNETのパラメタ→propにappend
　2.謎のタイプのパラメタ→continue
　3.OFPPDPT_EXPERIMENTERのパラメタ→propにappend
となるイメージです。


good job!


対応が年を越してしまいました。新規対応なので２０１６年に致します。


Traduction complète terminée BO et FO
Enjoy ;)


I stand by my suggestion of

> make it so that clients of this code can choose their preferred behavior

It's fine that the compiler wants to not waste a branch to protect itself from crashing, but for the debugger, a crash is far worse than emitting an error message and continuing. Yes, I know, this is a long-standing culture clash, but at least let's not make the situation worse by starting to deliberately crash in places where we used not to.


はい。ofp_util.py作る前です。
本モジュールの目的はOF14対応＆OFCTL改革だと思っているので、
ofp_utilを考慮した実装にしたいと思います。


good question. This update was to fix a specific issue with the rhel image that didn't affect the centos image.

I'll post another pull to update centos to keep the images in sync.


in


こちらは 01-26 朝ミでお話したとおり「対応ネタ」として起票します。


AUTO-MESSAGE - JSHint detected erros in the following files: lala.js


You're already using the Swift AST all over the place. Not Crashing here isn't going to get you anything better anywhere else, and may be the cause of existing bugs.

We're definitely *not* going to add a flag deciding whether to crash or not.


Yeah, that looks pretty good.  I see there is an essential problem in FlowTypeChecker, which is essentially the existing problem as for break duplicated.  That seems reasonable all things considered, as it's actually quite a job to fix that.


Just use `add()`?


Reference: [issue#2](https://github.com/seb86/WooCommerce-Skip-One/issues/2)


use six.iteritems


Why not leave this with a URL constructor? Makes its purpose much clearer imo.


:white_check_mark: :thumbsup:


I did this because I fund case where
```
modulePath == C:\WINDOWS\assembly\GAC_32\mscorlib\2.0.0.0__b77a5c561934e089\sorttbls.nlp,
assemblyName == mscorlib
```


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


Thank you for fixing the problem. Maybe you should correct your German Lang
file. Its not very good German.
Am 20.01.2016 22:02 schrieb "Marcin Wieczorek" <notifications@github.com>:

> Fix /ng error found by @IceteeMen from spigotmc.org
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/MarcinWieczorek/NovaGuilds/commit/67affb869ada87cd9e7256393e1bf913b44821ee
>


[Lab 1 PostgreSQL.pdf](https://github.com/ulicnir/database-meng/files/104290/Lab.1.PostgreSQL.pdf)


Not sure about this yet. These comments were useful for us to fully understand the logic here.


很好，你把这个提交给我吧


canols should be canola.


cristinaestupinan:ciestupinan


if you rebase this, it would be appreciated if you could drop the `@` from the commit message, so github doesn't decide to email me this commit again


Why did you update autoprice and put moneymillion outside of the loop???


Indent for function


Ah I didn't know as there was no test for it. Do you propose an alternative fix that meets both needs?


> You're already using the Swift AST all over the place

We do (try to) check that we are in a safe state before calling out to Swift, exactly to prevent crashes

If we are using the AST without properly guarding against malformed/invalid states, that is a bug that should be fixed. But, as the debugger, not crashing is a good thing in and of itself.


はい。対応を試みてみます。
できるだけ共通部はUTILに追い出したいですね。

こちら、以前対応ネタとして頂いており、
当時はリリース優先のために「ofctl_v14リリース後の対応」としていましたが、
同様に「ofctl_v14リリース後の対応」としていた to_action->to_instruction も対応実施中なので、
対応進めても良さそうですね。

ただ、以前対応した時に、一部のメッセージで上手く動作させられなかった経緯があるので、
その際は処理方法等相談させて頂くかもしれません。


"Not crashing" is good for release builds but not for debug builds. This is in fact precisely what assertions are for.


как увидеть, что название отеля - это ссылка? в задании это тоже не прописано


Rachel Ulicni
1/25/2016
Lab 1: PostgreSQL

[Lab 1 PostgreSQL.pdf](https://github.com/ulicnir/database-meng/files/104294/Lab.1.PostgreSQL.pdf)


Bananas are good.


LGTM


Thanks for organizing the code in the Controllers directory. Can you make it consistent with the naming? Some are plural, some singular, some with '_controller' on the end, others not.


Sorry, I have realised this myself. Commit has been reverted https://github.com/openenergymonitor/emonpi/commit/988ffa27871f29e9fa4d4b0a7fb13a7338eb7918

Will need a more elegant solution to solve the read-only random-seed errors in the logs. Do you think have a working random-seed is important? It's never worked on the previous emonPi / emonBase images due to read-only FS. As far as I can tell it's main use is for generating strong encryption keys.


Expanded markbook_view.php to view up to 12 columns at once. Uses CSS to fix the student name column to the left side. Uses JS to add a double-scroll to the top and bottom for easier viewing.


Consider using != instead so that you avoid the IF statement entirely.


Let's keep semi-colons consistent. Either use them or do not. I prefer no semi-colons, but I think we have them throughout the code.


hoge


Looks good.


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


double the `fileContents` double the fun?


danke m8


lol wtf


Can you explain what a factory or the factory folder is for?


lollll. i tried to do this for literally 30 minutes to an hour. and then gave up. then reached out to you lollllllll.


Not quite.

The goal of this test is to check that the response from your controller does not include data about `item_from_other_department` (which is from a different department)

The code you have right now is only testing the the two items do not belong to the same department.  Since you created them both in the spec to belong to different departments.. we would hope they dont.

You need a test which would fail if `result` includes data about `item_from_other_department` and fail if it doesnt


In the future, let's link these file deletions with the file creations for better understanding.


so the fix was to not use `static` i take it? good old static variables...


What is life?


I'm taking a peek. Won't be able to finish tonight, but I can bring to class.


I see... this is the case for passing `innerSubscriber` to `notifyNext`. Makes sense.


cc/ @trxcllnt


In the text file, specifying grid size


Ya by adding the static stuff it meant the `fileName` parameter had no effect after the first time the method was called. iirc this was my bad.


@nbradbury Great comments! For the SqlUtils call I usually prefer separating out the args but I've changed it for readability. Other comments addressed in 083b8b5


はい。同意見です。

ただ、

>この dp.id のキーってそもそも要らないのでは？と思いました。
>リクエストのパラメータで、 dpid は指定するので、

については、dpidを指定せずにofctlを使う場合もあるかな？
と思いましたが↓どうでしょう。

例えば、

    @set_ev_cls(ofp_event.EventOFPPacketIn, MAIN_DISPATCHER)
    def _packet_in_handler(self, ev):
        msg = ev.msg
        datapath = msg.datapath
        desc = ofctl_v13.get_desc_stats(dp, self.waiters)
        ...

なので、ユーザはdpidを初めとした
datapathの中身を知らなくてもofctlの関数を使える
というメリットはあるのかなと思います。


Think https://github.com/ably/ably-ruby/blob/support-more-presence-members-for-client-lib-tests/spec/acceptance/realtime/connection_spec.rb#L594-L633 should provide the clarification yo uneed.  Checking that the serial is zero before any message is published is wrong, it should be -1


同じく、こちらは 01-26 朝ミでお話したとおり「対応ネタ」として起票します。


Sorry, I'm not working on this stuff anymore. The commit probably still works, so you could try applying it to a fork, but I'm not sure what's needed to make the tests pass.


Thanks for the commit, it's not easy make up a story without my native language, and I didn't know till now the past tense of slain.


What are the most useful fields from [x509.Certificate](https://golang.org/pkg/crypto/x509/#Certificate)?


#4 First step: in order to produce different fog on different player, each player must have his/her own data. @bakaxyf.


たしかに！
ありがとうございます。参考にさせてもらいます。


Sorry to seem dense, but isn't this the official repo?   https://github.com/rossp/django-helpdesk
If not, where is it?


yup. Nice to move the secrets out so these can be tested locally and excluded!


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


Yep, the site has changed location, it is not on SF.NET anymore.


Do we need this constructor at all?


@refractionpcsx2
This print is only used for EE and IOP message. It won't impact perf if you don't enable them. And I guess games don't print a full book ;)

@ramapcsx2
Great. There isn't an option anymore as I do the diff manually. Could you use the `--show` option and copy past the GOT/EXPECTED line.


It's just something like
```
class list
{
  Item start;
  Item end;
  void init()
  {
    start.next = end;
    end.prev = start;
    end.elem = nullptr;
  }
}
```


这两个spider应该merge到一起;


@svetlyak40wt, yeah, I'm using an small script that collects and groups git changesets by tag, which I know doesn't work well on some places, it's in my TODO list to improve the changelog generation (even if that means manually doing).


Are we talking about the same REPL?  This command is about the SBT REPL where I can enter and execute SBT tasks like `compile`.


However, this will break source maps. @gagern Do you have any idea how to best deal with this?


Either that, or also use ```$request``` directly in ```if (WMCommonRegistry::get("wmrequest")->hasVariable("sort") ) {```


Ah i didn't see it said 16 bytes not 16 bits, thought that was a bit wrong lol


Ok, but that means you merge the configurations of different mime types, no? I don't think that is ideal. Sounds like surprising behavior to me. For instance, what do we do with conflicts, i.e., key `foo` being set for two different mime types?

So, but to clarify for me, it is intentional that a language can have multiple mime types, right? (looking at https://github.com/graalvm/truffle/blob/master/truffle/com.oracle.truffle.api.vm/src/com/oracle/truffle/api/vm/LanguageCache.java#L72 and https://github.com/graalvm/truffle/blob/master/truffle/com.oracle.truffle.api.vm/src/com/oracle/truffle/api/vm/LanguageCache.java#L138)


This looks like a a bug fix that hasn't been applied consistently. I'd vote for logging a separate issue and reverting this change. You can also see it's referenced in `ComponentRepositoryTest` line 149 and 156.


Hi there, may I ask why type of `array` and `object` didn't apply this method like others? Is there any concerns? Thanks :)


Ditto here - half a bug fix which we should revert from this branch and handle in a separate ticket.


这里叫" class FeatureItem(BaseIte): "会好一点，因为包括了平台标签和用户印象信息;


ah, no. Not the same, I thought you meant the `console` in sbt, which is what most people people mean by "the REPL".


This is a change in functionality that should be reverted as it is not a simple code clean-up.


It's a simple algorithm that addresses the classic "glass of water" problem, in which you have two volumes: the glass itself, and the water. One way to model this is by introducing a small "air gap" between the glass and the water, but this technique produces artifacts (because of total internal reflections in that gap) and is relatively inefficient.

With the "nested dielectrics" approach, you make the water volume intersect the glass volume, and you assign priorities to each: say priority 1 for the water, and priority 2 for the glass. The renderer then takes care of considering the water volume only once the ray has left the higher priority glass volume.

The paper that explains things very well: http://graphics.idav.ucdavis.edu/~bcbudge/deep/research/nested_dielectrics.pdf


Hello,

You may want to create a new key.snk file.

I created this one specifically for this projet, but, if you want to create your own (And I strongly recommend to never trust a stranger, even, if it's me :-)), you can do it using the .csproj property view, under Signing tab.

Thank you.


Bug fix. Please revert


Bug fix, please revert.


@pestOO I don't understand you. We pass utf8 string to `QFile` directly and that's all. What do you propose?


that is a define at the start of the file remapping that name from syslog.h


@pestOO, what kind of conventions you're talking about?


cleanupVenezuelaFiles for "Angola"?


These tests will need to be ignored again when you revert the fixes.


The generated code lines
```systemverilog
function example_struct_type write_example(bit [31:0] data, int address,
                                        example_struct_type registers);
   example_struct_type r = registers;
```
gives this error
** Error (suppressible): output/example_sv_pkg.sv(135): (vlog-2244) Variable 'r' is implicitly static. You must either explicitly declare it as static or automatic or remove the initialization in the declaration of variable.

Replaced with (generated) code
```systemverilog
function example_struct_type write_example(bit [31:0] data, int address,
                                        example_struct_type registers);
   example_struct_type r;
   r = registers;
```


Good point, I guess I was having too much fun with the new Java 8 streak API ;).


Well I was trying to match the new change in code from what was changed before.

We used to match against `assemblyName` but you made a change against the path which led to this defect.

Perhaps we should revert the original change and look into your scenario/issue that caused the change?


这里 'start_url_prefix' 是一个我没来得及去改的问题，用 string 的format 方法会好一点，否则在代码中修改url会confused。把'start_url_prefix' 替换成 ‘start_formated_url’;


Thanks @timler - I'd prefer to resolve this in a new issue, if you're okay with that.
My concern is that looking at #179 where this code was initially added - it seems like the IdNumber class and commented out code was added in with the intention of providing future capability to assign multiple identifiers to a Donor. I'd prefer to discuss this requirement with the team before making a decision to remove the code. Sound okay to you?


我用CC官方源码搭配你的package和luci源 这样改反而过不了编译 改回原来的就能过 然后用你的源码来编译就提示qosmon.c里面出现eerror: expected ')' before 'finish'
         signal( SIGFPE, (__sighandler_t) finish );这样的错误 然后qosmon.c删除了  (__sighandler_t)之后就能通过编译 但是不知道会出什么问题


test comment.


Hi! Have created a patch to fix this regression and to fresh the layout of TOC in affix mode (301b6ae).


Press, you already started working on it in 2013?


这里‘commentInfo’ 习惯用 ‘comment_info’，驼峰体我习惯用于类名; 这是个人习惯，问题不大;


This commit is causing a constant stream of 404 errors for me in the app dev console looking for built/info.js


yes


Now the status indicator in the topology summary are gone.

Shouldn't the if statement be inverted ? Now you show the indicator, when self is None or deleted


To be checked: what is the exact fee for exactly 10 coin, because the documentation was blurry before editing


MessageBroker is still accessed through `getInstance`. Use cached pointer.


This was not included, so must be checked


这里需要加一条log 信息;


Feature request: have the "return change" option selected by defaut


Use

```html
<code>block</code>
```
For block names. :)


It was been fixed on 7e06fcf5ee2b90fbc7895f1625a4bb15511291c8


ок


Нет. Это исправление не пойдет. Надо разобраться почему при первом открытии одни настройки грида устанавливаются, а при Ф5 -- другие. И сделать так, чтобы нужные настройки сразу грузились.


This were attempt at styling code with markdown, but I may have failed. Again, new to github. I should have use a backtick (`) instead of a space.


合并为 "class PlatformFeature(Lolly)“ 吧，参看 p2peye;


Ditto


do we need this file??


@montaga I cannot reproduce your issue. When I open `/examples/` locally in Firefox and click on `cindy(gl|3d)`, the browser follows the symlink and redirects to `/plugins/cindy(gl|3d)/examples`.

This might be different when using a server, depending on the server configuration.

For example, if you use Apache to serve the repository, then
```
<Directory /examples>
Options FollowSymLinks Indexes
</Directory>
```
should do.

Can you please provide more detail on what you are doing?


@knalli Does this mean I am forced to upgrade my build to Node 4.2? How risky is it to stay at 0.12.x?


这两个spider 需要合并，因为是一个request，否则会浪费资源，如果可能的话，合并到档案页也不是不可以。


Здесь неправильно закрывается тег, на следующем коммите поправлю и добавлю заглушку.


In https://github.com/LuxoftSDL/sdl_core/commit/b02aa765607c410b1696aa81ce982fad60bf4868 this method was removed. Also it used similar to AddConnections method logic.


a taky vraci "ok" ... musi se kontrolovat http kod, pokud to neni 200, tak doslo k chybe ne?


Is pastebin only used when errors occur?


In https://github.com/LuxoftSDL/sdl_core/commit/b02aa765607c410b1696aa81ce982fad60bf4868 `RemoveConnections` method was removed.


we do need a document detailing the model of the return; this had me for 1min or two until I reviewed the source code ;)


Bad copy/paste here.


Koukni na soubor Overeno.php na řádek 133, tam se právě kontroluje ten http kod


ок


Warto sobie zapisać `win[i]` w jakiejś zmiennej, aby nie powtarzać tego kodu non stop ;)


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


I'm talking about the [Meta-Data-File](https://f-droid.org/manual/fdroid.html#Metadata). Isn't the App-Name fetched from the .txt-File instead of the strings.xml?
![screenshot_2016-01-26-11-16-10](https://cloud.githubusercontent.com/assets/4741199/12578061/d41efbe2-c41e-11e5-829d-2435a6f32fda.png)


Ber tento soubor jen jako wrapper nad cURLem, jen aby se to dalo jednodusse testovat.. Tahle trida by nikdy neměla být použivaná samostatně.


I have no problem w/ this patch and hope it passes but the two tests below show that 4MB at STC is NOT high pressure but is actually just the smallest size BEFORE high pressure.  2MB or 1MB would be high pressure at STC.
http://tests.stockfishchess.org/tests/view/569adc340ebc594a0871d910
http://tests.stockfishchess.org/tests/view/569b14d50ebc594a0871d925


#approved


Fajnie jak by użyć `forEach` do tego!
Ciekawym zabiegiem jest użycie `call`-a do zmieny kontekstu:

```javascript
[].forEach.call(win, function (item) {
    console.log(item);
});
```


neprijde ti divne, ze do errorMessage se muze ulozit cokoliv co neni chyba?


Good point. I have no idea how f-droid is retrieving the name. Someone else submitted it to f-droid.


This change would force a type annotation in the MonadTransformer sample code.


:ok:


@return int HTTP code


test comment


Super. I didn't notice. @romansklenar Thank you for correcting me.


Zamiast operować na stylacha (CSS) lepiej (praktyczniej) jest nadawać odpowiednie klasy w CSS.
W DOM Level 4 jest properties `classList`. Więcej tutaj: https://www.w3.org/TR/dom/
To jest oficjalna dokumentacja DOMa w JavaScript.


why does this line have a comma, and the other ones have dots? ^^


ок


Fixed :blush:


When do you find the new packet?


ES UI install 用 sudo 會錯


No comment :+1:


Great! Can you send a CLA following our [contribution policy](http://essentia.upf.edu/contribute)?


This was exactly what I needed to put my pinned cert verification code :+1:


I am sorry for bothering, Dreamweaver told me that there is a syntax erron
on that line, it told me it on multiple files, but everything works as it
should..

2016-01-26 11:21 GMT+01:00 Tobias Reich <notifications@github.com>:

> Here
> <https://github.com/electerious/Lychee/blob/master/src/scripts/lychee.js#L35>
> ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/electerious/Lychee/commit/9b702cfcc7aa006412542325b913975edf403a02#commitcomment-15672956>
> .
>


@malirod, @anosach-luxoft I mean that the correct way to use `QString::fromUTF8`.
But I get that we will *not* re-write all `QString` construction usage, so it's ok.


TU PUTA MADRE.


Dobrą praktyką jest podział na pliki.
Nie trzeba wszystkiego trzymać w 1 długim pliku - ciężko później się odnaleźć.
Elementy związane z RWD fajnie jest mieć w innym pliku (albo zaraz pod tym samym selektorem który modyfikujemy w zależności od wielkości okna.


Это как вообще? Как мы будем переходить на главную страницу? Если тут нет активного элемента?


SALUDOS


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


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


А вот тут ты наверно хотел обернуть футер в…ммм…footer, ты ещё не знаешь, но ты уже хочешь


I'm sorry, I think I don't get exactly which mappings you refer to. What do you mean?


See https://github.com/LuxoftSDL/sdl_core/commit/f9ec9a547eff806ea01d619b32385d8efe6611a4


+1 al push con un par


Proponuję wczytać ten plik przed Twoim plikiem `style.css`.


Grrr i forgot to ammend commit before push


    Why should we allow this when its not being assigned to a variable?
Because both assignment and action are the 2 cases where it happens. Assignment are the obvious and action also happen here and there, [see this](https://tracker.moodle.org/browse/MDL-52332?focusedCommentId=384268&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-384268), for example. So I think both have the same point, in practice.
    We do not seem to be asserting these lines in moodle/tests/moodlestandard_test.php
Correct, those 2 lines (82 and 86) won't be resulting in an error (Inline doc block comments are not allowed; use...). They are valid inline doc comments now. Note that the testing system does look for the exact errors and warnings defined and, once all them are checked to match results, any other remaining error/warning not defined in expectations is reported as an "excess". Not sure if that's what you're asking. In other words, it's verified that those lines do not lead to any error/warning.


Est-ce que tu fais l'usage de *StateMixin* ?


я не стал переименовывать, т.к. не знал какие критерии к именам, поэтому оставил index


может быть в начале проверять, есть ли такая кнопка вообще, а потом проверять уже ее видимость, чтото одно оптимизируем. в другом месте опять лишнии телодвижения https://github.com/yiisoft/yii2/issues/9796


Maybe something was otherwise dividing by zero?


**The code does not comply to PEP8.**
*PEP8Bear, severity 1, section `autopep8`.* *Related to file `coalib/tests/results/TextRangeTest.py`, line 134.*

The issue can be fixed by applying the following patch:

```diff
--- a/coalib/tests/results/TextRangeTest.py
+++ b/coalib/tests/results/TextRangeTest.py
@@ -132,6 +132,7 @@

 class TextRangeExpandTest(unittest.TestCase):
+
     def test_expand(self):
         empty_position = TextPosition()
         file = ["abc\n", "def\n", "ghi\n"]
```


**The code does not comply to PEP8.**
*PEP8Bear, severity 1, section `autopep8`.* *Related to file `coalib/tests/results/TextRangeTest.py`, line 142.*

The issue can be fixed by applying the following patch:

```diff
--- a/coalib/tests/results/TextRangeTest.py
+++ b/coalib/tests/results/TextRangeTest.py
@@ -139,6 +139,6 @@
         full_range = TextRange.from_values(1, 1, 3, 4)
         self.assertEqual(empty_range.expand(file), full_range)

-
+
 if __name__ == '__main__':
     unittest.main(verbosity=2)
```


**Line contains following spacing inconsistencies:
- Trailing whitespaces.**
*SpaceConsistencyBear, severity 1, section `python`.* *Related to file `coalib/tests/results/TextRangeTest.py`, line 142.*

The issue can be fixed by applying the following patch:

```diff
--- a/coalib/tests/results/TextRangeTest.py
+++ b/coalib/tests/results/TextRangeTest.py
@@ -139,6 +139,6 @@
         full_range = TextRange.from_values(1, 1, 3, 4)
         self.assertEqual(empty_range.expand(file), full_range)

-
+
 if __name__ == '__main__':
     unittest.main(verbosity=2)
```


``spaces=3`` -> ``spaces: int=3``.


**The code does not comply to PEP8.**
*PEP8Bear, severity 1, section `autopep8`.* *Related to file `coalib/tests/results/SourceRangeTest.py`, line 71.*

The issue can be fixed by applying the following patch:

```diff
--- a/coalib/tests/results/SourceRangeTest.py
+++ b/coalib/tests/results/SourceRangeTest.py
@@ -69,6 +69,7 @@

 class SourceRangeExpandTest(unittest.TestCase):
+
     def test_expand(self):
         empty_position = SourcePosition("filename")
         file = ["abc\n", "def\n", "ghi\n"]
```


Pour le formatage ReST, il faut une ligne vide entre la description et les paramètres.


LGTM


``spaces`` est disponible dans le scope de ``to_tree``, tu n'as donc pas besoin de le redéfinir comme paramètre. Idem pour ``statechart`` en fait.


@desci esses espaços servem para quebrar linha no markdown. hehehehe espaço, espaço, tab. =)


A vérifier, mais je pense que tu peux retirer le ``list()`` pour ``trees`` et ``flat_tree``.
Si tu retires ``statechart`` et ``spaces`` de ``to_tree``, tu peux directement faire ``map(to_tree, children)``.


adapter


Zamiast pisać tak długie linijki można przenieść zachowanie (czyli logikę do pliku JavaScript).
Złapać sobie selektorem odpowiednie elementy i wg tego co niżej:

```
var $spans = doc.querySelector('.teams li.stats .window .tournaments span');
[].forEach.call($spans, function ($span) {
    $span.addEventListener('mouseevnter', function () {
        enter('goldIoPol')
    });
});
```

Oczywiście musisz do bardziej rozbudować, ale wtedy ten kod będzie o wiele czytelniejszy od tego co jest obecnie w pliku HTML. Jedna z ważnych reguł brzmi, aby rozdzielić zachowanie od wyglądu.


`majora_generator` should be installed as devDependencies only, then you should put this in `config_dev.yml` instead


What is this check for? I see similar check in CandidateResolver...


Typo in version number? and for 5.5.1 too ;)


A vérifier, mais je pense que tu ne dois pas distinguer le cas ``if len(children) == 0`` car si ``children = []``, le reste du code fonctionne tout de même.


Si le puedes forwardear el purto es mejor ademas si tienes un dyndns podemo adjuntar al direccion a la llave tambien para que los demas tambien puedan conectarse a ti mas facilmente


../src/mmcheckingpanel.cpp: In member function ‘void TransactionListCtrl::OnCopy(wxCommandEvent&)’:
../src/mmcheckingpanel.cpp:1352:52: error: ‘GetColumnsOrder’ was not declared in this scope
         const wxArrayInt columns = GetColumnsOrder();


This variable seems to stay at 0 indefinitely. Is this intended?


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


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


I've reverted the change, now we need to decide what is the issue with as it is normal and expected.

> I did this because I fund case where
>
>    modulePath == C:\WINDOWS\assembly\GAC_32\mscorlib\2.0.0.0__b77a5c561934e089\sorttbls.nlp,
>    assemblyName == mscorlib


Should the default value be true, given we fetch templates remotely and don't do any checks?


LGTM!
[![LGTM](http://lgtm.in/p/6LQIHpSOp)](http://lgtm.in/i/6LQIHpSOp)


@nateabele could you please update npm registry with this version? I can see 0.2.16 being the latest there.


I'd prefer
public static String cp(String logicalFileName, String localFileName, long fileSize,	 SE closeSE)

be careful with variable names (camelCase is applied)


Why T1 was unresolved and why it is resolved now?


インデントを入れてください。
forやwhile,ifなどの中身は可読性向上の為、インデントが必要です。


TeamCity Civic Assembly :: Build backend + frontend [Build 26](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=449&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) outcome was **FAILURE**
Summary: Tests failed: 3, passed: 1734, ignored: 7 Build time: 00:23:18
### Failed tests
```
Document tests: Should take a alert when try to upload new version for locked document: <no details avaliable>

User Management: Should submit a form for user creation: <no details avaliable>

Venue tests: Venue type: Should Not Edit a Venue: <no details avaliable>

```


I try to keep indentation (cut at 80 characters) and white lines neat. It's just details, but I prefer ;)


don't you need to like take `len(file_contents[end_line-1])` here or so?


改成和上面一样 'Permission already exists.'


Because `convertWithImpliedThisAndNoReceiver` can filter out all candidates, so no check will be performed via CandidateResolver


"Este é um manifesto que visa convocar Piratas do Brasil e \#hackers, peguem seus passaportes, precisamos nos unir porque temos muitas coisas para mudar."


We might need to truncate these to avoid spurious regression failures. Let's see...


plesae add a test with an already fully defined source range and one with a range that covers e.g. line 2 and 3 from a 4 line file.


`elif` would be better style


Il y a essentiellement trois choses à tester (qu'on peut d'ailleurs tester séparément) :

1. Que chaque état du statechart est repris dans l'output et que l'output contient uniquement ces états. Je propose de parcourir chaque ligne de l'output, de faire un ``.strip()`` dessus pour retirer les espaces, et de mettre l'ensemble des noms que tu obtiens dans un ``set()``. Ensuite, tu compares cet ensemble à ``set(statechart.states)``.
2. Que le nombre d'espaces est correct. Je propose, pour chaque ligne, de compter le nombre d'espaces ``(len(line) - len(line.lstrip())`` vaut bien ``spaces * statechart.depth_for(state)`` (la racine étant en depth 1, il faudra sans doute soustraire ``spaces`` une fois).
3. Que les états sont imbriqués correctement. C'est sans doute plus délicat à faire sans ré-écrire une partie de l'algorithme.


unack, same as on the other commit


Oh, I understand you


Tu peux sans doute utiliser ``with self.subTest()`` ici (le mieux étant d'avoir le nom du fichier en paramètre de subtest).


I am trying to setup new site and after creating database I see this error

syntax error, unexpected 'new' (T_NEW) in /var/www/html/ppemr/gacl/adodb/adodb.inc.php

So we have to remove '&' sign from gacl


smoothed


:ok:


To be totally clear, ask the original author. But, I think the Idea was that SEQAN_FORWARD_ARG and SEQAN_FORWARD_CARG has the same text.

If that is the case, I can copy the text. Otherwise, provide me a text for this deprecated feature.


Ok, vu :)


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


`T1` was not unresolved, but `Inner` was. The main reason is that no candidates left after `convertWithImpliedThisAndNoReceiver` call. https://github.com/JetBrains/kotlin/commit/da927ec917434c788796be7076c10e67508ad5d4#diff-b0f51b10ece5eb636d51823649313ef4R386


If there is a conflict, then the key entered as last wins. At least in my implementation.

Yes, there can be multiple MIME types associated with a single language. I see them as synonyms. Merging options for the same/synonymous object makes sense to me.


@piotrd Well, this is the spec for our build process (ensure that the dev tooling/dependencies work as expected).

If you use npm/webpack/ands o on for loading only, but have a regular browser environment, you can stick with a node whatever you want.

I don't know how to desperate this two..


good catch - thanks Kyle


Are you sure?
1.3.6.1.4.1.232.1.1.1.1 = Hex-String: 00 0a 0f ff ...
The key of $response is the oid
$response->{'1.3.6.1.4.1.232.1.1.1.1'} = "0x00 0a..."   (old Version)

You set $k to this oid, then you don't use $k any more. So your forth line should be
$response->{$k} = pack('H*', $h);
right?


Color code for "ostrygowy" is wrong (7 digits instead of 6).


これなくても動作しませんか？


Opps.. seems that I updated it right before your pull request. @RePoChO


yes I'm literally stupid


可読性向上の為、開業を入れた方がよいです。


А почему хосты в разных местах разные?


Du coup corrigé 49a7d29


Wrong color code (7 digits instead of 6).


Idem : 49a7d29


Опечатка. В словах типа "сет" лучше давать ссылку на глоссарий, где будет объяснено, что это категория билетов.


Rajouté dans les cas nécessaire : 49a7d29


The PHP programmers'


OK


For a moment there are no tests, so, yes, it makes sense to delete these roots.


Changé pour IStackableItem, donc plus de problème à ce niveau normalement.
49a7d29


tensorflow.org webpage is not in sync, it still refers to Bazel 0.11.


Проверяются те кнопки, которые указаны в шаблоне. Довольно странно указывать кнопку шаблоне, но не указывать ее в buttons.


@JustAkan 음... EAS 는 보류 해야 할 듯 싶습니다.
conflict 가 수정해서 될만한 양이 아닐 뿐더러 DQ 님이 구레포에서 댓글 업급 하신 걸 보니 64비트 에서는 아직 정상 작동 하지 않는다고 하시네요.


zou je hier niet input type "number" van maken?


Да, верстал вчера, когда еще не знал о необходимости перехода. Поменяю.


I like "kotlin-build-common" better, put sort of emphasis on kotlin build helpers. Just build is too general.


Ok, except of the test source roots.


Is this will working? moduleHandlerFactory.getTypes() will return the old types not the new one.
The modifiedService is called when you update properties of ServiceRegistration object
The new types have to be get from referece object as new registeation values.


@leifurhauks thanks for spotting - i've added back the deprecation notice in 6aa1a4930ed412b8bdab6731d169fafbd7e59fd2


"the the"


These changes are because the way that `toEqual` matcher works has changed in jasmine-node (probably only to match own properties)


Fixed. Thanks! :)


Is there a bit missing here?


これで全データ含まれていますでしょうか？
fgetsは１行だけ取得する関数です。
while文で読み込むのが一般的なので、while、feofを使用したバーニョンもやってみてもらえますでしょうか？


А тут забыл спросить в скайпе. Футер - обязательный элемент? И как понять, где его логичнее начинать? В нашем примере лучше с карты или сразу после карты?


Hi Thomas, I updated my Fiji, but the new plugin does not appear...Does that take some time till it makes it on the update site?


make levels of abstractions consistant


+ typo in view-source:http://easyappointments.org/ line 219:

```<li>Set zour server information to the "config.php" file.</li>```

should of course read:

```<li>Set your server information to the "config.php" file.</li>```


я не против, просто опусти вэту логику скрипт будет отрабатывать чуть быстрее


Семантически - да, обязателен


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


`if ((e.admin && isAdmin)||e.admin==false)` -> `if (!e.admin || isAdmin)`


Goed idee.
On Jan 26, 2016 11:55, "Marcel Goethals" <notifications@github.com> wrote:

> zou je hier niet input type "number" van maken?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/CodeForEindhoven/buurtpreventie/commit/6a8b622eed2c861bb0db305e02d684cc775e7c0e#commitcomment-15673698>
> .
>


Hi. You may want to take a look at this one again. It is deleting the name of the last two recipes instead of just one. I keep ending up with a blank name instead of "Yarrow - Wormwood - Tulips Extract" if I delete a duplicate "Ring of Narlik".
Also maybe automatically decrement the slot count by one. Otherwise, it's not different than a manual clear + sort.


Sorry, I edited the string quickly after copying and pasting it and I forgot to update it.
I'll fix it.


Опечатка. Плюс глаз коробит все эти "Запрос на создание, запрос на указание", грамотнее и доступнее для читателей писать: "Создание заказа", "Указание платежной системы". Не надо держать читателя за идиота и при этом предлагать ему HTTP листинг далее.


There's no need for this complicated `declval` code. Simply use `make_response_promise_impl`.


remove duplication btwn read_player_option


inline to start then refactor


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1240](http://teamcity.monogame.net/viewLog.html?buildId=14015&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 584, ignored: 6 Build time: 00:30:26


But isn't it up to the languages to define the semantics of the mime types?

I don't know, I don't see a reason why merging would be desirable. On the other hand I see a lot of confusion from a unclear definition of last-one wins.


кусок с футером я просто скопировал  из стартовой страницы


Sounds good to me. If we need to support multiple Id Numbers in the future, this will be handy.


Добавил интеграцию со Slack


begin around the integer block


Is there really a jasmine-node 2.x ? I can't find it :confused:


inline


look as the efficiency average, it's was 0 all the time = driver never did anything.
now it's state 25% to 70% when RAM is low and i bash it hard with more and more things to load.


I just discussed this with @cmaglie a bit more, perhaps we should have a closer look at the pin manipulation API, I actually think that a lot of improvement can be made without having to change the sketch-facing API at all (and the variant-facing API can probably also keep backward-compatibility). #1285 would probably be a good basis for this. I'm planning to have a closer look at this in a while, after I've finished some of the things I'm working on now.

As for the address spaces, that would only solve part of the problem, which is probably also easily done using `__builtin_constant_p()` to bypass the `pgm_read_byte()` calls, without having to wait for a new compiler version.

On the short term, we should probably revert this PR and re-apply it when/if we improve the pin API later.


begin block around integer


Review of basics in HTML and CSS


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


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


why not simply put the `self.start.line or 1` and so on in the invocation directly? Shorter, and certainly possible since it's so simple now.


以前の訳ですが、「与えませんでした」と過去形にする必要はないと思います。


Are these two lines necessary (since we have the next expectation) ?


Что тут происходит?


IMO a regression test has to pass the worst case scenario and that's in this case low-pressure not high-pressure. With high-pressure you simply kill the effect of the patch.
But this are just my 2 cents.


Fixed, thanks


and bloat the diff? no


実際に使われているところを見たわけではないので何とも言えませんが、
英語の方ではあるので残したほうがいいのでは？


ok.. forcnul jsem to


IMHO, we should not be adding version attributes, meta or otherwise, until Nix actually supports these and the semantics are clear. We really don't want to have everybody doing this in a different way.


critical!
size() returns size_t - architecture dependent


Remove


dont ref simple types, ignore him


Yep.

> BETA 2.0.0 Support is available in the Jasmine2.0 branch.


:sheep: :it:


dont flush in reading


@casey-collab


dont flush in reading, also there are no bits here


Thanx for the enlightenment. It was a typo mistake.
In general 0.2.01 is converted to 0.2.1. But I saw that you had problem installing 0.2.1 anyway.

If you'll run the below command you'll see that 0.2.1 is already taken, thus we need to change the version to 0.2.11

Modify your pull request and I'll commit and publish

-Arik

---------------------------------
npm view  js.utils versions
[ '0.0.1',
  '0.0.2',
  '0.0.3',
  '0.0.4',
  '0.0.5',
  '0.0.6',
  '0.0.7',
  '0.0.8',
  '0.0.9',
  '0.1.0',
  '0.1.1',
  '0.1.2',
  '0.1.3',
  '0.1.4',
  '0.1.5',
  '0.1.6',
  '0.1.7',
  '0.1.8',
  '0.1.9',
  '0.2.0',
  '0.2.1' ]


Probably breaks with the recent release of `4.2.5.1`. Would you mind making this comparison a bit more relaxed?


Not really, I added them when I was debugging the jasmine-node upgrade.
But they don't do any harm.


sorry about autocorrection in what I wrote: `I` should be `i`

I'll be interested to see how the timing results compare.


same here


aren't primitive types passed by value ? (in that case a plain array of objects) I think immutability apply to objects only (at least in java) but I might be wrong, let's ask the others


verify sniffs for CONNECTION_TYPE (wpp outputs it), probably should be INSTANCE (but client will accept this)


Hmm, for some reason this does not work. Any ideas?


いいえ。残すべきではありません。

これはセンタリング制御のための空白なので、訳文が元の幅を超えた時点で残すべきではありませんでした。
このあたり一帯の空白の維持 or 削除はubuntu側では機械的に元の空白数を維持しようとしていますが、
本当は見た目を考慮したものなので訳文と原文との比較をする必要があります。

ja.po では後者の方式を採用しています。


Thanks to this on latest Kali Rolling, it changes back interface to Managed mode right before Channel Hopping and sudden script exit. Changed it back to "arg.skip" to make it workable together with --world. Please revise.


přejmenoval jsem to na $body ;-) dává to takto víc smysl, ne? :)


Why do ```addCategoriesCriteria``` and ```addBlocksCriteria``` return something, but you never use that?


Yes but this is for the criteria in a moduleinmode. For example, the contentblock in the mode special only adds activities that have special: true.


wre


Add new line


Yes, but that was wrong because then you would should more activities in those blocks on the homepage than you would in the activities overview.


Pronto. A imagem é domínio público se entendi direito (as artes originais dos livros da Alice).


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


Note that `os.name` on cygwin is `'posix'`, but we don't care, since *pycryptodome* fails to install on cygwin.


Hi, after update elasticquent in my project I have an error about these changes:

```
FatalThrowableError in ElasticquentTrait.php line 270:
Fatal error: Class 'ElasticquentResultCollection' not found
```
If I add the "\Elasticquent" namespace another time, then all works correctly.
It's possible solve this problem in your code? Maybe you have some different configuration in your project when you test the changes.

Thanks for share your work ;)


Just saw this:

https://twitter.com/tjernlund/status/691738876868714497

> On Jan 25, 2016, at 2:58 PM, Koichi Shiraishi <notifications@github.com> wrote:
>
> @timsutton Thanks, I understand.
>
> radar is I knew, but I did not know the meaning of the sentence. Sorry.
> This problem maybe has been posted radar. This issue mist64/xhyve#83
> I tried to leave a comment on his said number's thread, but could not be found :(
>
> —
> Reply to this email directly or view it on GitHub.
>


dhddddddddd


用 resolve 方法沿着当前的样式路径去找就行了。


As per https://github.com/symfony/symfony/pull/17540#issuecomment-174940207, this should be:

```
if (request.statusCode >= 400) {
    statusCode.setAttribute('class', 'sf-toolbar-status sf-toolbar-status-red');
} else {
    statusCode.setAttribute('class', 'sf-toolbar-status sf-toolbar-status-green');
}
```


We can also add a leading `/` for stuff like `mx.mx`, `mxbuild` or `workingset.xml`


testing comment


Проставить labels для чекбоксов


For what it's worth, I also feel like these changes happen prematurely. We have no consensus how package name and version number ought to be specified in the future, and this patch turns a lot of straightforward build expressions into recursive ones for no obvious reason: fn terms of Nix, these changes have no observable effect (except complicating the evaluation somewhat), and whether that $name / $version format used in the patch is actually more in line with the future format is something we just don't know.

The right place to further this cause, IMHO, is the *Nix* repository. First of all, we need an implementation that takes advantage of these version attributes, then we can worry about refactoring Nixpkgs.


Why `@withStyles(s)` changed to `withStyles(ContactPage, s);` ?


***Code Quality - Nitpick***
In Javascript, you can use one `var` to declare / define many variables like so
```var firstNameState, lastNameState, emailState, passwordState = false; ```

In cases were the line is too long, you can break it like so
```
var firstNameState = false,
    lastNameState = false,
    emailState = false;
```


Storing in a different table would be cool but likely add significant complexity. Should be fine to just add another state property and update the indexes.


This is a bug. The key is actually `\spec`, not `\specs`.

PR coming...


@vishalvijay it seems like class-level decorators don't work in Babel 6, if so, this solution might be better than workarounds.


このメッセージは `gvim -y` で起動すると表示されるようですね。
表示位置の調整用かなぁ？


Yes. It's just saying that there shouldn't be infill overlap in the raft..
Should I make it const? or constrexpr?


LGTM! :+1:


:sparkles:


I have no idea, but I will come across it when finally fixing the prime tower.


Согласен.
Мне явно тоже надо приянять это к сведению.


I think this convenience feature is very much unexpected; it tricked me to believe that httr uses some kind of magic to cache the credentials. Why not export `protect_cache()`?


***Code Quality - Nitpick***
There should a single space after `}`, before and after `else`


Это факт. Но статья про сет в глоссарии пока не написана, к сожалению.


Suggestion to add:

* Arguing for or against ideas, not people


@newoga Look at this PR :laughing:


Потому что @deterok недостаточно внимателен, а я ещё просмотреть не успел.
_Думаю_ что он поправит.   :-)


Would be useful to offer some suggested 'boilerplate' clarifications, whether on branches or in another file.


Second mistake :laughing:.


This commit undo a lot of changes, including recent changes for benchmarking (normalized PR, detail timing, pred verification, etc.) please verify


@newoga Looks like it's was added back ^^'


Please update the mli as well


Again, might be useful to offer some suggest 'boilerplate' clarifications of this, and dropping the 'Examples' wording for something more firm: 'Representing the project or community is defined as...'


>em produção não tem problemas com isso; a alternativa seria rescrever os testes para tomar em conta a proteção >CSRF

Sou a favor de corrigir os testes.


I also prefer `kotlin-build-common`


Aside note: maybe we can come up with a tool to help people to generate a Covenant suitable for their project, with a list of boilerplate clarifications for what constitutes 'unprofessional' behaviour and what constitutes 'representing' the project.

Simple web page to take some options for those, and an e-mail address, spit out markdown to drop in their git repo (or use GitHub API to do it for them if they want).


I think haskell currying of ```fmap :: (a -> b) -> f a -> f b``` make more sense. I know scalaz also use that signature but I wonder why?


@avluis no search issues found. Tested on 4 devices.


I guess master branch is wrongly merged into dev branch, resulting in lost of edits


OK, I will fix this. I was intending to keep `make_response_promise_helper` private to `make_response_promise_impl()`.


Hmm, sorry about that. I can't even pretend it compiled on my own computer.


这是干啥的～～？


test comment.


otherwise it will match these names at any depth in the repo right?


Если вызывать `$bh->apply([...])` внутри шаблонов — всё сломается.


cool! :)
wouldn't we want to clean up the temp file?


Pull request please


Wouldn't it be more appropiate to use a map instead of an array to store the styles?
`clearStyles(styles)` could get rid of the `includes` method call, if a map was used instead. That would probably perform better?

I don't agree with the naming `clearStyles(styles)`. The name is misleading, as the method isn't clearing the styles provided as argument.


check if this is really equivalent...


Was it intentional to comment this out? (the same thing has happened couple of times further down)


Put the description for this algorithm.
```
This algorithm implements Constant Q Transform employing FFT for fast calculation [1].
```


FYI https://github.com/moneymanagerex/moneymanagerex/issues/667
~~~~~
/home/dawid/Development/moneymanagerex/compile/bk-deps g++ -c -o mmex_mmcheckingpanel.o -I../lib/wxsqlite3/sqlite3/secure/src -I../lib/wxsqlite3/include -I../lib/lua/src -I../lib -I../3rd -I../3rd/LuaGlue/include -I../3rd/cgitemplate -I../3rd/csv-parser/include -I../src -DNOPCH -I/usr/lib/x86_64-linux-gnu/wx/include/gtk2-unicode-3.0 -I/usr/include/wx-3.0 -D_FILE_OFFSET_BITS=64 -DWXUSINGDLL -D__WXGTK__ -pthread -fPIC -DPIC  -g -O2 -std=c++11 -g0 -O2 ../src/mmcheckingpanel.cpp
../src/mmcheckingpanel.cpp: In member function ‘void TransactionListCtrl::OnCopy(wxCommandEvent&)’:
../src/mmcheckingpanel.cpp:1352:52: error: ‘GetColumnsOrder’ was not declared in this scope
         const wxArrayInt columns = GetColumnsOrder();
                                                    ^
make: *** [mmex_mmcheckingpanel.o] Error 1
~~~~~


läuft nun durch beim start. RouterInfos werden geladen.

Scheinbar gibt es noch ein Problem mit dem VLAN Modul.

test_Server_VLAN.py:
*Test raised an Exception: 'eth0'*

Vllt erstmal ohne Bridge ausprobieren? @Symen
Bridge in eine separate User Story auslagern.


Awesome!


@springmeyer Can you publish a new version that includes this fix?


if -> switch


type inference

```scala
Welcome to Scala version 2.11.7 (Java HotSpot(TM) 64-Bit Server VM, Java 1.8.0_72).
Type in expressions to have them evaluated.
Type :help for more information.

scala> trait Functor[F[_]] { def map[A, B](fa: F[A])(f: A => B): F[B] }
defined trait Functor

scala> val list = new Functor[List]{ def map[A, B](fa: List[A])(f: A => B) = fa map f }
list: Functor[List] = $anon$1@7188af83

scala> list.map(List(10))(a => a + 1)
res0: List[Int] = List(11)

scala> trait Functor[F[_]] { def map[A, B](f: A => B, fa: F[A]): F[B] }
defined trait Functor

scala> val list = new Functor[List]{ def map[A, B](f: A => B, fa: List[A]) = fa map f }
list: Functor[List] = $anon$1@30331109

scala> list.map(a => a + 1, List(10))
<console>:13: error: missing parameter type
       list.map(a => a + 1, List(10))
                ^

scala> list.map((a: Int) => a + 1, List(10))
res2: List[Int] = List(11)
```


Thanks. I've effected your suggestions.

On Tue, Jan 26, 2016 at 12:25 PM, Makinwa Olubukola <
notifications@github.com> wrote:

> *Code Quality - Nitpick*
> In Javascript, you can use one var to declare / define many variables
> like so
> var firstNameState, lastNameState, emailState, passwordState = false;
>
> In cases were the line is too long, you can break it like so
>
> var firstNameState = false,
>     lastNameState = false,
>     emailState = false;
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/andela/getmyshop/commit/5637f7860484f579a7755be1cc054dd64f235019#commitcomment-15674262>
> .
>

--
*Tobi Oduah /** Andela.com
<http://andela.com/?utm_source=andela&utm_medium=email&utm_campaign=signature> *
*Software Developer / +2348037946755 <+2348037946755> / @Andela
<http://bit.ly/andelatwitter>*

*"The startup that's harder to get into than Harvard" - *CNN
<http://snip.ly/HfId>
See what The Wall Street Journal <http://snip.ly/u9AI> has to say.


In this version, two main changes are proposed:
-- better managing the case where a sample of the regional pool is directly generated by coalesc (when m=1 and filt=NULL)
-- including in output both the composition of the community and of the pool (the help file has been updated).


Thanks. I've effected your suggestions.


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


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


Почему не `if (isset($declByBlock[$thisBlock])) ...` ?


@yger might have forgotten to mention this during the intro. A useful site for backing up documents, webpages which change frequently etc. Also good for tracking issues associated with the website: https://github.com/OCNS/Website/issues


Nice one, I was just about to open an issue suggesting this :)


Done for #3


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


bitte hier die ganzen IF-Klauseln durch <input ... require> ersetzen


И этот кусок тоже стоит поправить


@holden Y U do dis? ;_;


:octopus: :sparkle:


@malirod see a11dc714a5d76f00d4a1bb03586c73b26957417e


。。。求拆个create出来


求立个flag


Partially fixes #39, fixes #50


Closes #1.


Changed it.


よりプリミティブにいうと

* `:set insertmode` していて
* `gui_running` (`gui.in_use`) の時に
* 8秒中の2秒くらいの確率で

表示されるメッセージでした。これ、普通に使ってたら、まず見ることはありませんねw


I wasn't far off doing this myself! Looks like you submitted a PR just in time for me to not to learn PHP :)

What I did as a stopgap measure before was offload the API to `chronometer.sh -j` on the main pi-hole files (which just outputs the same info as as JSON string)

Is there any performance advantage of having these calculations done in PHP over bash?


> can we just write things so they don't depend on unsupported features?

Sure, but I guess I don't understand the issue..? Why is it painful to include for all browsers?

The polyfills all start with `if (!Object.keys) {...}`, `if (!Array.prototype.forEach) {...}` etc. so if the browser already supports the feature,  they're a noop.

Is the issue that we don't want to modify global objects? If so I can abstract them in `Utils` functions (though since they're the official MDN guaranteed-to-match-the-official-spec polyfills, modifying global objects doesn't seem that bad).

If is the issue lib size? They're mostly comments so don't add that much once minified. If we're worried about the size of unminified ably.js I can just remove the comments (they're just copied from MDN). (And I'm not sure rewriting things to not use the feature would save much space over the polyfill -- we have to specify the work either way, whether in utils.js or polyfills.js).

(In at least one case the polyfill's only used by test code, so I can move that to a test-only-polyfills file at least)


Nice!


No entiendo que pasó, abrí ambos arachivos con el editor de Eclipse, eliminé en ambos el caracter fantasma y luego hice "add, commit y push"; desde casa lo arreglo.


返回值好任性。。


> The polyfills all start with if (!Object.keys) {...}, if (!Array.prototype.forEach) {...} etc. so if the browser already supports the feature, they're a noop.

Simply because we're sending bytes to data that is then discarded.

> Is the issue that we don't want to modify global objects? If so I can abstract them in Utils functions (though since they're the official MDN guaranteed-to-match-the-official-spec polyfills, modifying global objects doesn't seem that bad).

Nope, that doesn't bother me that much.

> If is the issue lib size? They're mostly comments so don't add that much once minified. If we're worried about the size of unminified ably.js I can just remove the comments (they're just copied from MDN). (And I'm not sure rewriting things to not use the feature would save much space over the polyfill -- we have to specify the work either way, whether in utils.js or polyfills.js).

Yes, agreed, I just meant it's not ideal, but then working with old browsers is not ideal!


@nicklockwood Just tested on Android in 0.18 and it works fine. There's a slight flicker though. I now feel stupid coz I was rendering a absolutely positioned phantom view to measure height, and then update the height of the TextInput!


spacing


Thanks and very sorry I did not see that. Sent.


This is grey area to me, I know basically nothing about this `Java compiler API backed implementation`. Is this something we could use directly here, instead of creating debt?


This should be $secret = bin2hex(random_bytes(20))

This generate a 40 character length.


Same.


Ensure spacing after colon are consistent.


Shouldn't this be _after_ we write the file?
Here we don't preallocate


Why are you assuming there will always be a match between every player and the specified journal pointer? If the player hasn't done the related quest yet, would they still have the pointer? Because you're not checking for ```p``` to possibly be null here.


Enderman?


for the record, I did initially wonder the same @jbgi, and I did wrote them in haskell style, but after a little why I faced the exact issues @xuwei-k described here and I went back to use the following pattern: take the last parameters and put it at the begin.

In way it make it more consistent with how we call method on object in scala (the first parameters being the "this" sort to say, like in python).


Not needed.


Тут дело в том, что настройки грида изначально на диалоговой форме устанавливаются правильно, но в момент открытия формы остатков они слетают


Shouldn't this be in PosixHelper.AllocateFileSpace ?


@Jean85 I understood it that codes 4xx and 5xx are treated as errors and showed in red. Codes 3xx in XHR are not errors, but shouldn't be used. So code is marked as warning (yellow)


yes it would be better to handle all these parameters within a container. I think I will open another issue for that problem because it impacts a lot of code on this repo but also on the scheduling project


add space after:. Should be:

public function hasGrant($grantType): bool

We'll do another PR that adds typehinting everywhere (including on parameter), but for now just fix the CS on what you've done :).


it would be nice if deliver and supply were a bit more similar in terms of options (naming, etc)

in supply `:skip_upload_apk, :skip_upload_metadata, :skip_upload_images, :skip_upload_screenshots`
in deliver `:skip_screenshots, :skip_metadata, :skip_upload`(with this commit)

Maybe rename this new option to something like `skip_package` or `skip_upload_package`? (I would prefer all to be skip_upload_... but that is another story)


Nit: space between 'this' and the period.


Ups xD


I see!
Haskell type inference is really great!


I cant join isnt fixed


Yes, you're right, forgot that. Thanks ^^


Interesting. When I added multiple extra path components, the first of those extra path components was swallowed up by the filter spec (the last ``(.*)``) which caused the signature to be invalid.

eg ``/images/sHBpEKTDHyF2s2IFGvSFZaf9/1/fill-800x600/test/test.jpg`` was parsed into:

```python
signature = b'sHBpEKTDHyF2s2IFGvSFZaf9-Ss='
image_id = 1
filter_spec = 'fill-800x600/test'
```

So I'll also change those ``(.*)`` to ``([^/]*)`` to prevent this from happening.


Good question!


i have to think about this commit a bit


Does it works?


Keep consistency in mind when choosing either single or double quotations.


Not sure what we can do about the flicker. Perhaps @andreicoman11 has some ideas?


It was something in the micron N25Q032A series


Fixed in: https://github.com/kaedroho/wagtail/commit/9d7dc78a676160cb6dfd8f15ded867ae94e90fbc


use 0xF for the collision flags as 15 implies that it is a number.


or rather, using implicits to pass around typeclass instances is not powerful enough. they should really be part of the language to show their full power!


D'uh. Should just be "Code of Conduct" again. Will fix this, thanks!


@carlaschroder can we rephrase this a bit? I have a hard time parsing the grammer above.


Thank you very much!


cheers.


It takes care of `ByteBuffer`'s position and limit, yes, but this class does its own separate read and write positioning in order to not need flip(). These positions do not necessarily correspond to the `ByteBuffer`'s internal ones, and are reset for each read and write.

For fun, I tried removing these two lines (both and individually), and `DataBufferTests.asByteBufferDatatests()` started failing. So take a look there to see why it's necessary.


The signaling layer is built on top of Apple's Push Notification network. So short answer is no.


@kucharovic 3xx codes in XHR are not only never used, but never shown. The W3C specs about it talks of "transparent"; so, your browser will treat them transparently, as in you will never see the 3xx request, not even in the browser's developer tools.
So, if you will check for 3xx codes, you will have unused code, hence my comment.


음... 그렇군요. 근데 kexec은진도가안나가는건가요? 지금까지아무언급이없더군요


¿es para las búsquedas esto?


good :)


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


Probably best to mention it can only delete items from online players, judging from the code? I'm not sure what you intention was/is with those items for players who aren't currently online, as they're not included in the iterators.


I do not think so.

Simon

On 26-01-16 11:14, ipelupessy wrote:
>
> do we need this file??
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/amusecode/amuse/commit/3b72f2da0ab0d8501ea49dc0a2ad66c03ccc3765#commitcomment-15672775>.
>


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


I see.

We would embed a  in the string I think, and update it with $(‘#seconds’).text(count);  I can do that shortly.

> On Jan 26, 2016, at 12:13 AM, Phil Davis <notifications@github.com> wrote:
>
> Doing this is easy - and it works for now in most European languages. But, for example, in Nepali (or Hindi or...) the verb comes at the end of the sentence. So in Nepali we would use a word order like:
> "Page" "60 seconds in" "automatically" "will reload".
> Obviously with each of those parts of speech translated to the Nepali word and script!
> So for the general case there needs to be a way to have the whole sentence as a single gettext() with a replaceable parameter:
> gettext('Page will automatically reload in %s seconds')
> is fixed and can be translated at the server using the PHP. Then in the JavaScript the%s has to be replaced with the current value of the countdown timer each second. I will have a play with how to do that.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/pfsense/pfsense/commit/9a4dad76fcdd5522db4ca35100de90237f329938#commitcomment-15668487>.
>


Isn't that implied by what it says right now?


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


I would prefer to have it different per it device. But as long as I typically don't connect to several devices i recovery mode simultaneously, LGTM.


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


For me, this change fixed Python3 compatibility.


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 228](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=716&buildTypeId=XbimEssentials_XbimEssentials) outcome was **SUCCESS**
Summary: Running Build time: 00:01:34


Use instead ```nextPowerTwo``` method which is defined in essentiamath.h ;)


Nothing to do with the PR, but I changed it as it is not the correct way of documenting the return value


New clean-up function added to the interface


tu as testé in browser?

Quand on autocomplete, il ajoute un trailing space ou pas?


We want to run the cleanup always, no matter whether we succeed or not


no-op cleanup (nothing to cleanup in the read scenario)


Test to ensure we do cleanup in the write scenario


Delete the dataset created by the setup


fix error.userInfo.


Yes I think so, however it won't work anymore on x64


Oui, testé, et pas de trailing space.


@melix I think what Adam is referring to is the difference between forking the `javac` executable and using the Java compiler API. Take a look at `org.gradle.api.internal.tasks.compile.DefaultJavaCompilerFactory`: the `CommandLineJavaCompiler` is the implementation that forks `javac`, and the `JdkJavaCompiler` is the implementation that uses the compiler API.

So ideally we want to prefer using a daemon compiler process using the `JdkJavaCompiler` (configured with the right Java home), instead of simply forking javac, as the spec states.


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


You meant to write ```everything_renamed```, the entry is now duplicating ```everything_removed```.


This won't boost your precision, but will lead to loosing time for creating this vector. Convert to double when necessary at the place where computation happens.


Rename to `limit` please


@bweston92 Dockerfile-alpine file especially for docker-compose usage


Does not make ESP8266 any happier. Now we get a
Now with neomatrix we get a compile error because swap is not defined.
Adafruit_NeoMatrix.cpp:178:24: error: 'swap' was not declared in this scope
To correct, I changed swap to adagfxswap in adafruit_NeoMatrix.cpp


Also run beautify on the code which added a lot of changed rows.


Remove these comments as they are obsolete?


I wasn't sure if the AES key was encrypted with ECDH or RSA, so i preffered to let it out.
@Googulator How is it encrypted?


Will this be patched in the 1.9 branch?


Any chance to release a new version with that fix to rubygems?


毎回読み込み直す必要はないかと思いますので、
require_onceがよいと思います。


Страница товара должна называться точно не index - копия.html Лучше назвать её по аналогии с .psd макетом, например item.html


Замечание по вёрстке: внутри ссылки должна быть картинка


Remove?


I thought @edolstra had recently made `__DATE__` more useful. Putting ad-hoc changes into individual packages seems like it'll turn into a mess quickly, especially if people start doing slightly different things every time.


图表的清单该叫什么中文名字啊？
图清单？图目录？图索引？


Читабельность: отступ дочерних блоков внутри .container отсутствует. Ты в Атоме пишешь код?


Not longer support ruby 1.9.3


I copied that from `place_maze_design`...


http://project.e-catchup.jp/issues/9929
の修正です。


Put this into protected methods, no need to have it public


Зачем тут эта замена https на http?


"Number"


Thanks a lot!


спасибо, все исправила и закоммитила снова с изменениями, очень помогла подсказка по созданию рэндомного числа, спасибо))


"constantq.h"


Looks good, @nicobn, thx for the contribution. Will merge.


I prefer to handle this in NSString category


默认样式里面有的，自己再写一遍其实也是挺好的，更安全一些


It was only recently that I worked out what the flags were for. I guess at some point will have to go through and make sure its used consistently.


@edolstra @peti even with no consensus or native Nix support, do either of you think it's likely that we won't end up with a distinct `version` field in the derivation attrset? I can't really think of a design that wouldn't give us that, but am curious if you have one.

Either way, it doesn't seem premature for us to move to one standard across the board before migrating to something else. Thousands of packages already have the

```nix
rec {
  name = "foo-${version}";
  version = "bar";
}
```

pattern today, so adding it to even more packages doesn't seem like it would hurt anything.


Suggestion by @marcelhollerbach.


Thx


@Jean85 never shown?
![out](https://cloud.githubusercontent.com/assets/1418898/12580579/fa98782c-c430-11e5-8af1-a7b1b29aa054.gif)


It seems like a simpler fix would be to not make `GetForLosSize` return a table at all, and just change `for (const CLosTables::LosLine& line: table) {` to `for (const CLosTables::LosLine& line: instance.lostables[li->radius]) {`?


GPL or LGPL ?


`exec()` returns either null or an array (where the first element will be the extracted sub-match).
The error probably comes from here.


I would say let the Grid class handle manipulation of its' members. That is - create four new functions in Grid for the four operations you specify here: moving forward, backward, first and last. Also, try to generalize common functionality: I am seeing unnecessary duplication in the above four operations.

On second thought, you could create only one function in the grid for handling the movement operations. The function would be called something like `updateGrid` taking an [enum](https://github.com/IMQS/www/blob/master/js/js-map/map-defs.ts#L671) specifying the particular movement.

The window.onresize function above can also make a call to Grid telling it that the window has been resized: `myGrid.onResize()`


Please do not name this class `SpecialCasesExport`, 'Export` is already part of the name of this class, due the fact that it is inside the `CppExport` namespace. `SpecialCases` is enough.


Will check on this tomorrow. :)


I know it's not directly related to your changes but I noticed that in `SchedulerClient#waitForAllJobs` the ArrayList instance that is created could be initialized with its final size.


Convert to double here


Let the calling function handle the error; pass in a `failureCallBack` parameter.


I guess you intended on calling this `Import`, not `Export`. Either way, remove that suffix entirely.


lgtm


Looks good to me


Remove?


tosi kiva juttu hei


Number


@EranOfer I'm reviewing the units and noticed this line looks wrong, it is a duplicate of the `IntegrationCompressionTopic` line above - was this intended?

The original jroland code has different topic names here.


Not seriously


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


You can let the ajax function be responsible for instantiating the `XMLHttpRequest`.


causes a deadlock with cli due to remote class loading during logging.


Doesn't work - so what's the point...


后面进行删除，不过这样效率貌似很低的。看看jdk中remove(object)是如何实现的？效率如何？


Why?


it looks nicer


Just pushed 821814c50b4 dealing with Unicode issues under py2, and fixing problematic `os.path.join()` with lone drive-letters under py3.


outstanding


Posted here to track:
https://github.com/brave/browser-laptop/issues/329


rien ne s'obtient sans souffrance


Coudl we use [`fromNativeSeparators`](http://doc.qt.io/qt-5/qdir.html#fromNativeSeparators)?


TeamCity Civic Assembly :: Build backend + frontend [Build 27](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=453&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) outcome was **FAILURE**
Summary: Tests failed: 3, passed: 1736, ignored: 7 Build time: 00:22:55
### Failed tests
```
Document tests: Should take a alert when try to upload new version for locked document: <no details avaliable>

User Management: Should submit a form for user creation: <no details avaliable>

Venue tests: Venue type: Should Not Edit a Venue: <no details avaliable>

```


Awesome :+1:


A typescript programmer might be confused upon encountering this variable (`firstRecordId` and `lastRecordId`) - he might mistake it for referring to a DOM element ID, which is not the case. I would name it something like `firstRecordNumber`


Added for 0.14.....


Fixed in 821814c50b48e.


The pull request is towards v1.10 but the patch will be back-ported to the upcoming maintenance release (v1.9.13) in 1.9 series.