@sbrunner Can you please elaborate why you choose to parse the less file directly in the browser when in dev mode?
The performance are really bad on my side. It takes more than 5 sec. for the browser to load and parse the less to css. which is not comfortable. I'd prefer using something like `watchmedo` to detect less files change and compile the css on the server and use the build css file in the html.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 10901](https://teamcity.mesosphere.io/viewLog.html?buildId=101435&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 2 (new) Build time: 00:00:40


ach ja :confused: natürlich, WIP...
klaro, muss ich mich dran gewöhnen. ich finde es halt unklar, was beim rebasen und pull-request etc. passiert. Ich wollte gerade die SQL-Einrückung korrigieren, da waren plötzlich die Dateien wieder auf dem alten Stand. Für sowas habe ich halt zu wenig Gedult...


so so unrelated change also :cop: ... und mein rechner ist schuld.... hehe mag daran liegen, dass ich 2 postgresql versionen drauf habe... aber das ist eine andere geschichte... können wir ja dann wieder auf 5432 standardisieren. :-)


You probably also want to handle `OOModel::ReferenceType` in the exact same way you're handling pointer type.


Ah, I didn't the that you now sourcing file. So, then it's not a problem.


Thank god.


Haha +1


For CPP, you cannot have ' (prime) and __IMPOSSIBLE__ on one line, so just introduce a line break after the =.
__IMPOSSIBLE__ is better than matching non-refutably against Just.


Yes, you're right, now I see.


This project is to test uploading eclipse web application to GITHUB


that explains things :sweat_smile: thanks Brad!


Why `||=` for a local variable which is certainly not initialized?


Как-то слишком сложно. Проще же ((CheckBox)v).toggle();


Thanks!


"cardinality violation"を「カーディナリティ違反」と訳されてますが、濃度違反」と訳したほうが良いかどうか、迷うところですね。
# Oracleのマニュアルでは「カーディナリティ」のようです。


@niklasravnsborg Stimmt, aber dafür ist längst nen Fix raus. Denn im Moment wird beim Speichern ein falscher Wert für requireImage übergeben. Das wurde gefixt, und dann ist validateImage auch immer definiert.


It was mobile that made me switch it back... It just looked completely horrible.

I've never thought about eye tracking though in regards to content - that's really interesting.


А это же не checkbox. Тогда в каждой группе чекбоксам надо дать одинаковые ID. И здесь просто будет:
```
((CheckBox)v.dinfViewById(R.id.checkbox)).toggle();
```


I think another approach would have been to just provide csv field instances for `SignupAttempt`, and inline the parser and renderer functions for the `Bool` there.  Then we wouldn't need a new type, and we wouldn't need any instances for `Bool` either, whether orphan or not.


After adding this, I can not use any path modeling; latent variable modeling is still lovely.


Is the problem that a second run puts the `const { PropTypes } = React;` below the import block? Would it help to just do
```js
const React = require('react');
const { PropTypes } = require('react');
```
instead?

I'm just now starting to look through your changes, so apologies if that assumption ^ is incorrect.


@niklasravnsborg: Ah ok, seh den Fehler. Ja stimmt, das ist beim Testen wohl untergegangen, sry.


handle-arena.h depends on MonoHandle, and handle.h depends on MonoHandleArena. We can either inline handle-arena.h in this file (handle-arena.h declarations have to be as public as handle.h declarations, so that's not an issue), or we have to declare MonoHandle before including handle-arena.h.


So, as I was ranting yesterday, my current implementation of the types is really bad. Whenever you call the `type()` method of an expression, you receive a pointer to a brand new object, which you now own and must `SAFE_DELETE` yourself. above you first get a `type` and then based on it you perform some casts which potentially overwrite the `type` variable. Thus the original object that you were returned is now a memory leak, since you don't delete it.

To make things even more convoluted, you should be aware that once you receive a type pointer, that type might contain other types which it owns. So for example a pointer type will contain and own a filed that represents the base type. So if you get the base type of a pointer, you must make sure to delete the pointer only after you are finished with the base type. Since the pointer own the base type, you should also be careful not to delete the base type.

Here, make sure not to overwrite the type variable once you set it, and then just `SAFE_DELETE` it at the end, once you done exploring it or its children.


@t-ionut can we keep the code name var convention like in the rest of the code?
https://github.com/PressLabs/gitium/blob/master/gitium/inc/class-gitium-submenu-settings.php#L126
https://github.com/PressLabs/gitium/blob/master/gitium/inc/class-gitium-submenu-configure.php#L54


don't use cm trees =(


:+1: I remember having this name in the early stages of a commit, but I then changed it in the end. I think this might be better after all.


В любом случае ещё надо добавить Locale.US, иначе на арабских языках будет хрень


Is the comment above still relevant?


Можно


Hello world I'm starting to learn r and sharing my repo in future


:+1: potayto potahto :)


Why dsn-core-init if u already have dsn-run later?


Boehm should never call this code, as it would assert at `mono_gc_make_root_descr_user` (https://github.com/mono/mono/blob/master/mono/metadata/boehm-gc.c#L1424)


変更部分を一通りチェックしました。


@DamienCassou could you please revoke my permissions for this repository, so that it no longer shows up in the list of my repositories. Thanks.


Changes to LD_LIBRARY_PATH after the process has already started are ignored. From dlopen(3) (emphasis added by me):
> If, **at the time that the program was started**, the environment variable LD_LIBRARY_PATH was defined to contain a colon-separated list of directories, then these are searched. (As a security measure this variable is ignored for set-user-ID and set-group-ID programs.)

Now, it's true that I can distinguish between the instance where the build process was started from outside the bindings via make, because the Makefile target runs node-gyp with the environment variable ```SOLETTA_FROM_MAKE=true```, whereas, if someone starts with ```npm install```, this variable will be absent from the environment.

So, I can distinguish two cases:
  0. If the environment variable is present we can avoid using ```dlopen()``` only if we assume that eventually, when the bindings will be needed by an application, the soletta library *will* be available on the library path, meaning it's sufficient to link with ```-L$(pwd)/build/soletta_sysroot/usr/lib -lsoletta``` and, most importantly *without* ```-Wl,-rpath```.

  0. If the environment variable is absent and the build is started from npm install, we can avoid using ```dlopen()``` only if we assume that, when the bindings will eventually be used, the soletta library is guaranteed *not* to be in the library path, because it will be in ```./build/soletta_sysroot/usr/lib/libsoletta.so```, meaning that we *have to* link with ```-L$(pwd)/build/soletta_sysroot/usr/lib -lsoletta -Wl,-rpath $(pwd)/build/soletta_sysroot/usr/lib```.

So, if these assumptions are reasonable for all install/distribution scenarios, I'm happy to remove the dlfcn stuff and replace it with linker flags.

@nagineni, what say you?

OTOH, if it turns out that we have to keep the custom library search code, I'm prepared to change it so that it first tries to load soletta from the library path, i.e., with ```dlfcn.dlopen( "libsoletta", dlfcn.RTLD_NOW | dlfcn.RTLD_GLOBAL )```, before trying the specific path ```path.join( __dirname, "build/soletta_sysroot/usr/lib/libsoletta.so" )```.


i just checking how its work


@OttoKurschner Our “new” filter has now regained the ability to tune it. (Specifically, these two filter functions are now virtually identical, differing only by a constant factor on the parameter)

See the notes in 82e8142 for more information including how to port your old gaussian settings to the “new” function.


こちらは使われていますでしょうか


second commit for check


@notspiff: Not sure if I understand what's going on here. Could you help explaining?
In line 11-27 you set SHAIRPLAY_FOUND to 1 if we have the include dirs and the libraries and the code compiles.
But then in line 30 FPHSA sets SHAIRPLAY_FOUND to 1 without the compile check.


To hide some of the complexity here, I think this could be an `empty?` method in the ImportStatement class instead. Alternative names: `consumed?`, `exhausted?`, `invalid?`, `has_run_out_of_variables?`.


@peaseloxes @BaasT Voeg dit toe boven je MainClass om die "Make this a private constructor" in je sonar te surpressen


Думаю, проверка не нужна, так как createAdapter() вызывается из onViewCreated(), а setRetainInstanceState() нету.


The MONTHS variable can be made as a constant.


Yeah I totally forgot.


Typo Decmeber -> December


改過來了，謝謝


:+1: makes sense even without the destructured-default merge commit that's coming up.


Yup, much nicer.


There is a TODO comment two lines above. Is that still relevant?


oops


Das Problem war das es eigentlich 2 Fehler waren, die die gleiche Fehlermeldung lieferten, jetzt hat jeder von uns einen davon gefixt^^


Note that this doesn't handle cases where the client needs the traversal for children _before_ it can finish processing the parent.


saw approval from hmmr
at https://github.com/basho/riak_kv/commit/8832bb5ed0e56d984ff03479130fb8f9726bf13e


merging basho/riak_kv/at/do_not_squash_null_cells = 8832bb5e into borshop-integration-1321-at/do_not_squash_null_cells


basho/riak_kv/at/do_not_squash_null_cells = 8832bb5e merged ok, testing candidate = 93e18641


fast-forwarding end-to-end/timeseries to borshop-integration-103-feature/jrd/ts-spark-coverage = 6cb12b8f


taskRouterWrapper


@phpmaps - The items added I think would be better if you set the default variable to None, and the value is only added to the parameter dictionary if the value is update?

Example:
     if resultOffset is not None:
          params['resultOffset'] = resultOffset

Thoughts?


@23
Delete.


ja ich war am Anfang auch oft nahe an der Frustgrenze...


The lambda you define here is almost identical to the one you defined above. Please unify those, e.g. by using `bind` http://en.cppreference.com/w/cpp/utility/functional/bind or something similar.


:+1:, though see related comment in https://github.com/lencioni/import-js/commit/4798b0172e28b3be5a9ecfb7340e60f3ef723577#commitcomment-15313481

I worry a little bit about the `destructured?` method changing slightly (someone might re-introduce the boolean). In such case, the list of variables could be empty, but it's still considered `destructured?`. I guess that's covered by tests though, so no worries.


jaja, kommt mit meinem PR


thanks to@Fred Oh


recent GHC don't pick old `bytestring` :)


asdf


Hmm, ideally we shouldn't talk about plug-ins at all, but just "top-level modules" whatever that is.

Also this hardcoding should ideally not appear in C++ but in the configuration Json.


修正不要ですが補足までに
bladeのコメントは {{-- コメント --}} で書くとhtmlのソースとしてレンダリングされなかったりします。
http://readouble.com/laravel/5/1/ja/blade.html


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


I'm not seeing the problem. The call to sender is established in line 32.


Experimental note: Create the blips on serverside and use the arena as argument for visibleTo, because the players are children of the arena, it should work (see description of Blip.createAttachedTo)


$scope.data.videoPath is earlier in the script defined as a string. `var files` was also defined as string but used as array on line 126. Here I have redefined it to be an array, so `files[0]` on line 126 will work. You should consider to define `$scope.data.videoPath` as array to start with.


On this page,,,,
$ cd mvn install
     should be
$ mvn install
or
$ mvn clean install


`$scope.data.videoPath` was `videoPath` which is not defined anywhere.


Oh, just forgot to add `, buffer = nil` to the argument list.


Because there is no / in fron of this link


This is a module dependency to load ixgbe on most systems (depending on kernel configuration)


After looking through your changes, I think my suggestion above might work and be good enough. I don't have time to look at this myself during the day, so I'll leave it for you to amend.


It generates some warning too.


Why would you want to set buffer to anything else than `range.source_buffer`?


I only changed those with a / in front because they didn't work correctly. Chrome identified them as absolute paths, and couldn't follow. Without / the links are recognized as relative paths.


これはコンパイルエラーを起こしていたので、直後のコミットで修正済みです。


But you connect the `heatCapacitor` to port_b, i.e. the inner surface, so the (not steady state) temperature of the capacitor will be the inner surface temperature.


これは何のソースでしょう? 見た感じだとnetbeansを使ったことで発生したソースのようですね。
今は問題ありませんが、業務の開発では不要なソースが上がったら大変なことになりますので、不要なソースが上がらないようにローカルでのgitの設定ができたら尚良しでしたね。


#LuiDerp


Noone saw this before?!
And actually, doesn't every block drop itself if its broken and getDrops is not defined in the block?


I took the original repository version and did not change the quotes.


"api_version" should be string


#SriDerp :v


А зачем тогда вообще имплементировать этот интерфейс? Или ещё будет какой-то функционал?


So far I have not encountered such a case. I'm not sure if a variable declaration can ever be in it's own unit.


「カーディナリティ」などという日本語になっていないカタカナ語を出した時点で翻訳としては敗北なのですが、「濃度」は私のボキャブラリーでは「溶液の濃度」しか想像しないので、RDBの用語としては意味がわからず、使いたくないと思っています。


I'm not sure, I just followed the existing fellow methods like `range_with_surrounding_space`.


Есть ещё одно решение, чтобы не производить замену на &amp;
Добавить функцию javascript
```javascript
function startup(ctrl) {
	if (ctrl) {
        f = ctrl.form;
    } else {
        f= document.forms['mutate'];
        if(!f) return;
    }
	f.properties.value = "<?=$content['properties'];?>";
	showParameters();
}
```
И в самом конце файла
Запуск рендера параметров заменить на
```html
<script type="text/javascript">setTimeout('startup();',10);</script>

Работает на ура.


.setPositiveButton(R.string,ok, null)


I like you can never be jealous


They are not the same. The source file dependency collection written here only considers names and treats soft and hard dependencies the same. The member "dependencies()" on the other hand contains a set of header code unit parts containing the declarations used (hard dependencies only) not just by name but via member access and ignoring the source part. Basically everything but what we want.


@adamdougal @dominicmullings for the record this commit looks good (apart from the fact it bypassed the PR process :smile:)


Зачем запрещать закрывать диалог тапом мимо него?


Yes, thanks.


```
if (tts != null) // ёптыть, опять tts)
  return null;
```


What’s this? Don’t seem to recall ever seeing a `sauce.yml` file.


added


change title


don't send stats*


merged conflicts


Wie eben angesprochen, wir sollten noch einmal mit Martin darüber sprechen, wie wir für ADMINs und Project-Owner den Status "Deferred" und "Deferred Published" auf der Übersichtsseite darstellen. Nach den Änderungen kann man m.E. die 2 Zustände nicht ausreichend voneinander unterscheiden.


Ansonsten habe ich aber keine Anmerkungen! :+1:


Greets, it seems that the code originated in mpv in the first place so this abstraction is kind of pointless (apologize for using the edit function instead of making a new post to point this out)

The problem I encountered with the Gaussian filter is the appearance of fine stair-stepping or blurring if you modify the blur by any factor from the default, I could never produce any good results with this filter by modifying the sharpness so I only use or even recommend it strictly for chroma scaling and as a window function for Jinc (large radius, it works extremely well)

If you will make this change permanent you will probably have to hard code the 1/0.94~deblur into the windowing function as it is (currently) not user tunable, but I am no expert on those things


Лучше оперировать одним объектом, реализующим общий интерфейс (или выделить общего предка).


some tests failed:
failure: http://buildbot.bos1/builders/test-riak_kv/builds/3070


I chose not to write existent files to disk to try and experiment with "Raise-Rescue" and at_exit.


Octree (DeadParrot) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


If the `.so` loading fails here, we will never get to the ReLinker part, right?


This code will crash nunit runner with undefined exception, that were very hard to track down.
In my case I have TestCase with some arguments and Explicit=true, this cause NunitFramework to pass null.

I've try to just add a check

``` csharp
    if(str == null) return null;
```
and everything worked without an issue


LMAO :D


fix for bbc/microbit-extras#2501


1.delegateKMeans:简洁一下
2..idea目录是每个人特制的配置目录，不用上传上去，用.gitignore忽略
3.spark1.5版本在未编译情况下不兼容scala 2.11.x 系列，改成2.10.x
4.删除没必要的注释
5.肖师兄求带


What's the reason for adding query string parameters to the payload at all, rather than always keeping them only as part of the query string parameters?


I added comment in github, it does not have a send button, can you give a
suggestion ?

On 6 January 2016 at 21:48, hamnan1945 <notifications@github.com> wrote:

> we also can notify each other by @
>
> @NocusNibbes <https://github.com/NocusNibbes>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/hamnan1945/test/commit/87f1f733397d672309e2a1ee84da6817d0b09bdc#commitcomment-15301765>
> .
>


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1133](http://teamcity.monogame.net/viewLog.html?buildId=13407&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 560, ignored: 6 Build time: 00:24:10


@oliverlockwood thanks!


I updated the script, it should be fine now :P

I left the `.format` since citing [the docs](https://docs.python.org/release/3.1.5/library/stdtypes.html#old-string-formatting-operations):
> **Note**: The formatting operations described here are obsolete and may go away in future versions of Python. Use the new String Formatting in new code.


Testare


I'd still suggest:
```
OMEROVER=${OMEROVER:-omero}
WEBAPPS=${WEBAPPS:-false}
```
to allow overriding the properties without changing the files:

```
WEBAPPS=true bash linux/install_ubuntu1404_apache24.sh
```


The list of test arguments is a subset of the all application supported arguments. The test has only the arguments previously described in the autotest test. All supported arguments in test are in the yaml file, but yes, I agree we should describe them better.


Ack.


Ma perchè?


Don't work ..


no. merge?


测试中


Exactly, I don't know, I'm guessing it's irrelevant now and is a hangover from when the wrapper only dealt with `application/x-www-form-urlencoded` payloads. But the worry I have is it's something to do with the signing.

I know for example, the qs is taken into account when generating the signature hash, but the postbody isn't so maybe it has something to do with that. That's the bit that needs a little investigation.... :-)


@cableman: Do you have any idea why this is necessary?

[We have a request to update this](http://platform.dandigbib.org/issues/1346).


As discussed, because we have repository method *countBy...*, which returns the count directly, so we can keep the implementation


@thecapdan, although it seems strange that we've removed a `catch` block that we previously relied on, I've taken a look at Java's `Map.get()` method, and it only throws `ClassCastException` if the key is of the wrong type (which is impossible in this case), whereas it's much more likely to throw `NullPointerException` since the `getOperatingSystem()` could return `null`. I think it's therefore to accept this change as it currently stands.


Triggering local scan was meant for fixing watchdog issues in windows platform. But in Mac this not required. The triggering of an additional local scan could result in duplicate entries in local sql-lite db and as a result we these files in server with version number > 1.


Npm added that automatically when I did `npm install --save-dev walmartreact/assert-transform`


Most of the comments have been addressed:
- Moved workflow_list from hook_workflow to hook_manager
- Moved runpath_list from hook_workflow to hook_manager
- Moved path from hook_workflow to hook_manager (deprecated)

I would prefer to postpone the following to the next merge:
- Generalize hook_workflow in hook_manager to be a list.
- Possibly eliminate path.

Please let me know if the internal tests pass.


This is used in the BABEL_OPTIONS config


The rest of these unused ones are apparently used in the expect-transform but weren't installed with it. Any ideas?


This should be in 'other-modules', because it is reexported from `Web.Users.Persistent`?


I suggest to reduce Figure 1 to 3 inches width, it still is readable and increases space.
I'm now reading the whole new draft in order to review it, and before proposing a title.


When drive uploads a file, the server will soon send a documentCreated, documentModified event back to drive during the next NuxeoDrive.GetChangeSummary API. For this scenario we do not need to mark the document as remotely_modified.


Nah I wanted the right capitalization


靴靴大神，现在还能用吗？


in addition, I think it would be simpler all round to treat the qs and postbody as completely separate concerns. So if we can remove this bit of code, then it's wins all round


Started adding flow defs to this but didn't have the plugin installed yet. This is the type def of the options this plugin will take


custom


Yeah don't worry.


Shrug. Makes no practical difference. Would that be preferred?


"directly get started" -> "just get started"


If the `.so` doesn't load it will throw an exception, which would crash the app. Also this part doesn't really use ReLinker as it doesn't have a Context.

So in case of errors here the app will crash. The Android ClassLoader only loads classes on demand, so there is only a chance of crashing if you call `Util` before creating a `RealmConfiguration`. That will only ever happen if debugging or running test code. All normal usage should work fine.


Check out the new title and suggest changes...


"editors manual" -> "Editor's manual"


Just a balance suggestion, when the dynamic phase cost is enabled, if the value is below the normal Phase cost, it should use the normal phase cost. Like, it uses 5% of the dimension tick cost with a minimum value of 100.

Without that, any dimension that uses less then 2k RF/t its cheaper to use the Phase Field then having the dimension powered....


"familiar with wagtail" -> "familiar with Wagtail"
"disable this section" -> "hide this help section"


Here, "inner" means "non-top-level". `inner == null` is for filtering out anonymous classes.


This is cool, we keep it in this way then.
So my vote for less surprises when 1.10 is released.


Right, sorry! I did not mean "proposing a new one" but "suggesting changes".


Thanks bro👍


I guess this renders this commit useless? 5bf25e4
If so, feel free to ```git revert``` it.


Based on the `BuildOAuthHeader` method, it looks like the body is only signed when the body is Form Url Encoded, there is a body and there are 0 parameters.

Do you know of a reason why we can't sign the body when there are one or more parameters?

I wonder if Anthony Steele knows? This was explicitly added in commit #d5a48e1a.


I think the original reason this was done is to allow the choice of storage but it was unnecessarily complicated and needed removal XD


You can comment on commits (i.e, any changed or updated file) in two way:
First: inline commenting: if you hover your mouse over "+" at the first of each line you can add comment an push comment button, then i'll be notified about your comment.

Second: if you have a more general comment on the commit, you can put your comment at the end of the page, just like what you did in your last comment: "I added comment in github, it does not have a send button, can you give a
suggestion ?"

Sorry if you received many emails and notification alerts in your email. But we are just playing and warming up for a private account. You can change the notification setting of your Github account, to prevent sending the alarm emails.


ab


Не получится ли адская мешанина в США и Западной Европе? Может, оставить только `city`?


regex win :+1:


Why `positions`? Don't you want to show the selected `internship_applications`?


Hey, super danke hast du eine bestimmte code cleanup einstellung, vielleicht kannst du mir die irgendwie zukommen lassen damit wir vom selben Format sprechen :D
Ich komme ja blöderweise aus C# und da sieht der Laden einfach ganz anders aus und ich kämpfe noch recht mit der formatierung ;)


I guess you meant to use `internship_applications` as `positions` don't have any student related information?


You don't need to do this BTW, it's valid from the year the project started


PS also tested for stats::heatmap (labCol, labRow) i.e. not an issue specific to S3 etc.


Tu as bien la dernier version de la branche ? J'ai dû oublié de la mettre à jour


Nice!


I see. What bothers me is the mapping is adding a level of indirection (another virtual call in the chain), and I think we don't need that. Two different thoughts:

1. Let the implementations override `Device::set_speed()` and they can call any function/map helper themselves (that given a generic speed gives back the specific impl speed).
2. Is the generic interface actually used? Or when setting speed I already know whether is I2C or SPI? If the generic is not used, we could drop it.


Because this has diferent meta values, it is more confident, if added


There is no need to describe all params, only the ones that are supported by the test should do.


Hi,

could you be more specific about integrating problem?

What does not work, error received, etc..

Thanx


oui oui j'ai bien tout a jour


If you use struct, the members are public by default.

> I'd like to use struct only when it's a real POD.

Fair enough.


Octree (DeadParrot) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1917 of 2041 tests passed)

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


Octree (DeadParrot) - x86_64-MacOS-10.9-clang: OK (1896 of 2003 tests passed)

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


It used to work with the "view" action too. That's the URL tested by the webstandards validation (see https://github.com/xwiki/xwiki-enterprise/blob/xwiki-enterprise-7.4/xwiki-enterprise-test/pom.xml#L300-300). Can be seen as a regression.


or include License
person right, that priority is me.


Met toi a jour avec la master aussi ^^


Can you make the fact that the warning is only shown _if_ an extension was appended more explicit, i.e. by separating the extension appending and the existence check concern?


Licence Piriority.
First Piriority . writer: Choi Hyeonil ;ee32320@hotmail.com


ack :)


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


@borshop: retry


Twice.


Hi @CFXd
I am very interested in this change. I would like to move my jquery library to the footer of my web
My jetpack version (downloaded from wp repository) doesn't have it yet.
Any idea of when is it getting through?
Thanks!


Just have this error when i break a Flower:

Fatal error: Undefined class constant 'FLOWER' in src\pocketmine\block\Flower.php on line 98

Regards

Kiyo


Why is this change? On 1.2.x there used to be a way to modify the logic behind freezing routes extending the `ControllerCollection` (just overriding `flush` would do), but now seems impossible to do, as `doFlush` is `private` and won't call the parent's `doFlush`. Am I missing something?


Эти две строчки можно схлопнуть в одну. Или даже так:
```bash
if [ -n "$OPT_COAST" ]; then
  cp "$INTCOASTSDIR"/WorldCoasts.*geom "$INTDIR"
  cp "$INTCOASTSDIR"/*.csv "$INTDIR"
fi
```


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


ich hab derzeit nur eine ganz kleine Änderung an den Default-Einstellungen gemacht :)
und zwar hab ich die Formatierung von 'Chained method calls' auf 'Wrap if long' gestellt.
die Einstellungen dazu findest du in File->Settings->Editor->Code Style->Java im Tab 'Wrapping and Braces'

der Shortcut zum automatischen Formatieren ist Ctrl-Alt-L, und soweit ich weiß, kann man in Intellij
auch einstellen, dass er dass vor jedem Commit automatisch macht ;)


The window function is tunable (via `scale-wparam`).


I've created the corresponding issue: http://jira.xwiki.org/browse/XWIKI-12972


`$ROADS_SCRIPT` в кавычки, как и `$OSRM_URL`.


leave just url="/api/tasks/list"


:heart:


Проник лишний пробел.


и на мен не ми звучеше добре, сменям го


missing OOM.


But they (manager and the clients) shouldn't be called in the wrong order, if the manager goes out first, calling unregister will lead to incorrect results (potentially a crash).

We can ensure ordering of shutdown if we move all the static managers and such to be part of the AP_HAL main object instead (for practical effects, they already are part anyway).


Octree (DeadParrot) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (935 of 935 tests passed)

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


Well, this was not meant to be a supported extension point. If you need an extension point, please open an issue explaining your need


This is meaningless
https://github.com/temasek/android_art/commit/72cdd24dcc106f52b21372a972cdb72042f4632a


reack


Sub-directories like this were generally working directories of the artists. They're not actually included in the build directories. (Build doesn't run recursive.)

Same with "vv2" directories which have "higher resolution" originals, generally double resolution of the proper textures but not very sharp anyway since they're just the artist painting resolution. :)


I see how that would help developers transition, in particular when they copy paste code that expects `args`. But for new developers and when creating new controllers isn't kind of weird to create this unneeded reference? We don't alias `$.addClass` etc as well ;)


First commit added by Ivelin Yovchev


Не вижу, где инициализируется `qtasks`.


Nope, this is not OK. The `is_system` bool should only be set if we are connected to a real system bus, and we base authentication decisions on that. This should never be set on direct connections, which the ones to PID 1 sometimes are, if dbus-daemon is not up yet.


This doesn't match what you have above.


Please, update the this title:
`Share Milligram on Facebook` -> `Share Milligram on G+` or `Share Milligram on Google Plus`


Hmm, no, I don't know the reason, only thing I would do now is test against the api behaviour. If the api allows it, then we can do it. I seem to remember that the body is included in the sig hash when form url encoded, but would need to double check against the api behaviour.

There are integration tests that test a POST with params against an api endpoint (oauth/accesstoken I think), which will let you know if the sig is wrong, as the api will 401 if so. They are the ones I needed to change `foo` to `userid` for.

Otherwise, you'd need to look at the internal test project which (I think) tests against the dummyendpoint which reflects back the params it received and that would tell you if all params (qs or postbody formurlencoded) were received.

He may know, but it was back in March so may well have forgotten...! We could ask him?


jQuery provides wrapper functions for `on` that I would use instead.  In fact, you're using one above.

```javascript
$('.rating-circle').mouseenter(function () {
```
does the same thing and looks much cleaner.


Why? Which functions are still defined as inline when they aren't?
Commit https://github.com/raspberrypi/userland/commit/e0ecdb98a59a70048b64f8a5196d423680fbdc7e went in last week that shut up most of the ones being complained about under #220.

vcos_types.h is used as an OS abstraction and has about 5 different implementations. If the pthreads one is wrong, it is the implementation that needs fixing, not the definition.


I have no idea.


reference:

http://stackoverflow.com/questions/28371814/using-getelementbyid-inside-a-function-javascript


@aleffert now I'm able to refresh comments by initializing Paginator in  viewDidApear but now the issue is if I scrolled to bottom of table and came back by opening and closing NewCommentVC, it generates two requests for first page and one for second page, I'm getting duplicate results for first page(got combined results for both first page requests).


Он много где встречается, ага) Называл по аналогии с ядром.

> On Jan 7, 2016, at 2:46 PM, Alexander Marchuk <notifications@github.com> wrote:
>
> if (tts != null) // ёптыть, опять tts)
>   return null;
> —
> Reply to this email directly or view it on GitHub <https://github.com/yunikkk/omim/commit/eb03ac0449fde7f421cea575b74eb79217f7c672#commitcomment-15313943>.
>


FYI, my app started firing an error (`jsPlumb function failed : TypeError: Cannot read property 'offsetLeft' of null`) since this method changed.
The issue is the fact that my root element is `position: fixed`. It seems `offsetParent()` returns `null` for fixed position elements.. A possible solution is change fixed to absolute, but I still haven't tested for collateral issues..


Кроссплатформенное ядро уже переделали, в jni руки просто не доходили абсолютные сделать.
> On Jan 7, 2016, at 9:57 AM, Alexander Marchuk <notifications@github.com> wrote:
>
> Вроде везде относительные пути
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/yunikkk/omim/commit/db542acfac14d9ffca6ce69f4855afa24f098488#commitcomment-15308975>.
>


Yes, thanks for notifying me. I also jousting noticed there is no Flower.php in item


@kammaron 可以的，有问题可以发邮件


HAL_BARO_MS5611_I2C_BUS used to point to the bus we use in the instantiated hal. I.e.: HAL_BARO_MS5611_I2C_BUS==0 -> hal.i2c, HAL_BARO_MS5611_I2C_BUS==1 -> hal.i2c1 and so on.

With this commit HAL_BARO_MS5611_I2C_BUS now means the bus number itself. So I had to take a look on each board to know the mapping and put it here.


Thanks for the clarification!


Ah ok, that makes sense :) Thanks !


Typo on "should should".


// FIXME: Catch an error here and rollback model if it fails


True


:heart: [2]


Планировался, угу.


:clap: :clap:


Удалю


Commit this


Это временное решение, угу.


Was removed in a later commit.


See bugzilla bug http://bugzilla.qooxdoo.org/show_bug.cgi?id=9243


У tts по умолчанию есть уже элемент, его сначала заменить нужно.


Should we include single quote too?


No it is not and on top of that I think we should not create module units at all. We can have each individual macro a code unit and merge them using the configuration file thereby completely removing the module cases.


Why is the if-block in `constructUrl` necessary, when it already sets the parameters in `match` method and apparently correctly? This seems to break the link generating #87


Хз, я был отчасти согласен=) Добьём фичи - попробуй ещё разок реквест запилить:)


@astachurski If `ResultFeatures` and `SaslsMechanisms` are lists, then:
```
case SaslsMechanisms of
    [_H|_T] -> lists:append(ResultFeatures, SaslsMechanisms);
    _ -> ResultFeatures
end.
```
is equivalent to
```
ResultFeatures ++ SaslsMechanisms
```

PS: it's SASL not SASLS / SASLs


`OAUTH`?


Looking a bit about the original point, it seems is not an issue (it is "fine" as long as we don't use the original object anymore). However the function assumes the caller doesn't own the object, just a reference to it. I still feel a bit awkward with the arrangement, but this is all internal details of Linux I2C implementation, we can tweak it later if needed.


@tarsius done


Не думаю, мб он как-то меняться сможет в будущем при каких-то условиях (время суток, например, или изменения локали какой).


esi is a map_element of type path. Offset 6 will be edges (that are connected to other paths).


Why PropertyTaxUtil is not injected ?


I think it would make sense to clean this up properly by using the same names for namespaces and plugins. This will elevate the problem of mapping, avoid configuration file stuff and simplify the _api import things.


On my first review, if I remember correctly, you told me that we don't need `else` block after `Errorf()` because the letter will fail the test. If it's true than we could remove it :)


This breaks compatibility.
`blank?` considers whitespacing and empty string.


Why not #include "config.h", it is confusing. People may think that the "<config.h>" is system header file.


Couldn't we directly use `layer.maximumValue(shpKeyIdx)`?


Не относится к этому PR, но кажется, тут лучше не создавать явно список, а передавать генератор.
Странно, если quantifiedcode не нашел это.


indentation


It could look like this

```
ldscript = path.join(project_name, 'link.ld')

def render_ldscript():
  with open(path.local('Craftr.link-template.ld', 'r')) as src:
    content = src.read()
  content = content.replace('${STARTUP_OBJ}$', ldscript)
  with open(ldscript, 'w') as dst:
    dst.write(content)

# Target definitions

if session.export:
  # Only render the new linker script if also a new build.ninja manifest is exported.
  render_ldscript()
```


1) The implementation either support setting the speed or they don't. The speed itself doesn't change for them. That's why it's implemented in the generic interface, otherwise each implementation would need to provide it. So implementations either implement the `set_speed(I2CDevice::Speed)` or they turn them into empty methods (that's what we do in Linux).
2) Not sure about what was the question


This can also be made much cleaner.  Can you use `index` without any arguments?
http://api.jquery.com/index/


Thanks :-)


> I really want to prefix (and would have a liked a more clear startswith() function).

Having such a function would make my confusion disappear by making the intention of the code clearer. I think is a good idea to have it (even if just local to the file and promote it up later if need).


@astachurski `binary_to_atom` on a system boundary risks crashing the system. See _Run out of atoms_ at: http://prog21.dadgum.com/43.html


We should set `format$inherits = "pdf_document"` here


Кажется, что стоит добавить комментарий к этому методу, т.к. он называется `sort_candidates`, что подразумевает только сортировку, но при этом он может изменить `self.candidates`, выкинув из него элементы (`cg.group_id not in gu for cg in self.candidates`).

Можешь объяснить логику здесь, почему разрешается добавлять подобные элементы, которые потом будут выкинуты?


Actually, the last line wasn't understood by the piqi compiler and discarded with a working. The second to last line, is redundant, as the name of the imported module will be `stmt`, so there is no name to state it twice


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


I think we should derive from html_document using the `html_document_base` function. This will among other things ensure that pre and post processors are chained rather than overwritten.


Do you plan to update this patch to the latest version available from https://github.com/jkakavas/creepy and push it upstream?  Seem like a good change to get included in the official version.


ACK.


Grrrr forgotten about that.


Если вдруг будет меняться, можно будет вернуть.


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


Done


Is this change reasonable?


提交


Will the quickstart work with JDK 1.6 ? We still support it.


I would go for

```js
return (function loop(page) {
     ...
     if(res.body.hasMore) {
          return loop(page + 1);
     }
     ...
})(0);
```

Looks cleaner to me but it's just a suggestion ;).


Should this be `parts << views_path if views_path`?


Maybe a warning then?


Should have checked my new commit first :/


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


Didn't you read the code in server.py?

It's calling a method `response()` that will return a dict with the relevant data (chat_id and text). Why would you change this?!

Change it back.


```
- '5'
- '4'
- '0.12'
- '0.10'
```


if `map_element + 1` is type banner. Note that this is being checked incorrectly it should really have done `&0x3C` so possibly source of original bug but then banners don't have rotations so in this case it is fine.


This commit is public :\


Возможна ли ситуация падения приложения в случае вызова метода остановки потока (и, как следствие, обнуления задачи) и продлолжения исполнения этой ветки?


As above but map_element + 2. I made a function that gets all banners on a path tile and that should really be used instead but will require understanding the rest of the code first.


PR welcome :). Same on the following packages:

https://github.com/kevva/github-gists
https://github.com/kevva/github-repositories


The new one wouldn't drop the correct flower


It should, but did not try it. Let me check.


#include "std/string.hpp"
#include "std/vector.hpp"


Раз уж вы всё равно используете 11 стандарт, то похорошему здесь лучше атомарный флаг использовать


Maybe rename `reg` to `first_reg` or similar?


Только с categories.txt, или ещё и с vehicle_model.hpp?


Hello!
I already have a Keithley 2002 driver at https://github.com/herm/python-ivi/tree/k2002 which is quite similar to the Keithley 2000. Perhaps we can join our development work to make later merging of these drivers easier.


Me being curious:
Why and how doesn't this commit show up on https://github.com/opbeat/opbeat-js/commits/master?

PS: Just noticed: Other tags use a "vX.Y.Z" instead of "X.Y.Z".


if the direction from the start is not a valid available edge...


And btw, speed means it'll optimize for dex2oat's speed instead of runtime
speed.

On Thu, Jan 7, 2016 at 9:47 PM, Janson Kang <notifications@github.com>
wrote:

> Grrrr forgotten about that.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/temasek/android_device_samsung_hlte-common/commit/c1364c82b98aba313fbc2f3f0968d88957228d47#commitcomment-15314910>
> .
>


Why does this even have to be called if it does not want to use a modified message? (which is happening in `create_dict()`).

Why not simply use `self.message_params`? That `messageDict` variable is unnecessary.


I think we should use `before_create` here instead of `before_save` as the later is fired both in new record create and record update

http://stackoverflow.com/questions/6249475/ruby-on-rails-callback-what-is-difference-between-before-save-and-before-crea


Thanks @stof, I will do it asap.


Yes, it can. For example:
https://github.com/dimitar-asenov/Envision/blob/development/FilePersistence/src/simple/GenericNode.cpp
`MAX_DOUBLE_PRECISION`


The Chinese translation I got didn't say this. Let's wait for KONAMI's ruling.


Why not put `dispatch_to_core()` logic in here?


GetToken?


Hm... Do they drop the correct one now (without any commit)?


а, там с синонимами


I'd prefer CAT_OTHERS


done


TeamCity Xbim :: Xbim Exchange :: XbimExchange [Build 123](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=634&buildTypeId=Xbim_XbimExchange_XbimExchange) is now running


delete 20131105832黄倩倩.doc,i do


ack


I hope the right thing is

public function getDrops(Item $item){
		return [
 			[Item::RED_FLOWER, $this->meta, 1],
 		];
 	}

But for me it drops the right flower after that


Noted, I'm correcting that right away. Thanks!


TeamCity Xbim :: Xbim Exchange :: XbimExchange [Build 123](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=634&buildTypeId=Xbim_XbimExchange_XbimExchange) outcome was **SUCCESS**
Summary: Running Build time: 00:00:54


const?


I thought this was the lesser of two evils, getting a warning instead of requiring webpack users to have `json-loader` configured, but it seems this breaks for browserify users, too. Will revert later today and release a new version.


What about &&?


Dandelion? Tulpic? How about all of these?


Yes, it should be, did @thebigsmileXD check that commit?


Dandelion is not considered a flower


is this project work on .ccc extensions?


comments like these are supposed to go in docstrings ABOVE the method.


maybe we need a warning in "debug mode". We don't even add warnings in some cases (like in the transfer method)  that can be called thousands of times per second since that would flood the uart. Notice this is a "non-logging" function. We return false and let the caller decide if it can log or not.


thats why it has its own file (because of having a different ID)


```ftypes::IsStreetChecker::kPaths``` -- это константа.
Может быть, сделать по ней этот вектор индексов и не ходить каждый раз в классификатор?


variables should be snakes


not true... Look again :-)


> implement function to transfer data. (typo on commit title)

done


cool!


Why isn't the ordering of write operations consistent? It seems odd to me that changes to files would be first to base, then overlay... but open handles uses the opposite order.

I've been thinking about this a lot. I think it would make a lot of sense to have either order as an option (but consistent) since there are valid cases for both... and it's really just a question of performance vs consistency.


This looks more like you are removing some sections and adding new ones.


Greedily как вариант


1) Think this way: how much you would lose by dropping the I2C and SPI versions of set_speed()? IOW, Linux would implement `set_speed(Device::Speed)` itself.

2) The question was whether client code is calling the Device::set_speed (with device enums) or the SPI/I2C variants.


Несколько нестандартно конечно использовать ожидание на условной переменной с одним примитивом, а уведомление во время захвата другого. Технически в вашем случае это работать будет, однако не пишите так в боевом коде - это ни разу не ожидаемое поведение. Корректно было бы инкапсулировать логику очереди с блокирующим poll()/getTask() в отдельном объекте или на худой конец оставить в пуле потоков, чтобы условная переменная за его пределы не вылезала


ppc


I'm wondering if it'd be better to have a "media" style sheet instead of including the youtube one here?


Good catch @piotrowski
That's what happens when I try to sneak a new release out while there are no adults around ;)


derpedyderp


Showing the amount due in the top of the statement


If you think this macro should be in q_platform, I can change that. But will it be ok for you if I'll just keep pushing new commits, and not doing nasty things with git to make it all in first one?


Maybe?


I don't think "withMatch" is useful here -- it can even be counter-productive because it wouldn't check anything actually here, because content is an empty object. `calledWith` is better in this case because it will check if the same content object was used.

but maybe a better way is to actually add stuff in `content` :) (and keep `calledWithMatch` so that we don't check if it's the same object, because we don't care)


[IMP] Typo


OCD is kicking in, can we fix the indentation


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


Thanks. We'll fix that.


@rmackay9 does this work? Was it on purpose?


After installing using npm (v0.20) I have this **dist** folder:
```
.
├── client
│   └── src
├── preboot
│   └── src
│       ├── interfaces
│       └── server
├── server
│   ├── src
│   │   ├── directives
│   │   ├── express
│   │   ├── http
│   │   ├── platform
│   │   ├── render
│   │   └── router
│   └── test
└── universal
    ├── client
    │   └── src
    └── server
        ├── src
        │   ├── directives
        │   ├── express
        │   ├── http
        │   ├── platform
        │   ├── render
        │   └── router
        └── test

```
 then, I removed the dist folder and ran:
```
tsc
```

And this is the **dist** folder after **rm -rf dist** and **tsc**:
```
.
├── client
│   └── src
└── server
    ├── src
    │   ├── directives
    │   ├── express
    │   ├── http
    │   ├── platform
    │   ├── render
    │   └── router
    └── test
```


Oops,  forgot to edit after copy paste.


This is an API change that may impact projects that depended on this method, please keep the original and mark it deprecated


[FIX] Typo


We are going to need some mechanism for making this route dependent. Judging by the one of the open issues we haven't settled on a routing approach. Maybe this is something we should focus on next?


Yes


I see the constraints now, and given the nature of the issue, I'm convinced that `return false` is enough.

> We don't even add warnings in some cases (like in the transfer method) that can be called thousands of times per second since that would flood the uart.

For the future: one alternative for those cases is to keep track of how many of those you get, and dump a warning at specific points or let this information available somehow.


Works with these config.py snippets:
```
CHATROOM_PRESENCE = (('huone@chat.example.invalid','salasanakki'),)

CHATROOM_PRESENCE = list()
CHATROOM_PRESENCE.append(('huone@chat.example.invalid','salasanakki'))
```


It returns '9', ie the maximum string value
Στις 7 Ιαν 2016 1:43 ΜΜ, ο χρήστης "Matthias Kuhn" <notifications@github.com>
έγραψε:
>
> Couldn't we directly use layer.maximumValue(shpKeyIdx)?
>
> —
> Reply to this email directly or view it on GitHub.


I would just remove the `for PWM` and `-pwm` from the files and just keep `I2C Callbacks`. Since we are using a built-in component.


used it to move the text down you can remove it if you want


Its still a bit complicated.. because I'm depended on the NPM package


TeamCity Xbim :: Xbim Exchange :: XbimExchange [Build 124](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=636&buildTypeId=Xbim_XbimExchange_XbimExchange) is now running


The field is of type string
I have just created a new shapefile with integer field, will push soon.


Может ли всё упасть из-за || удаления потока из _queue_workers_, так как здесь и в методе удаления потока захватываются разные примитивы?


minor: put the `<includes.h>` before the `"includes.h"`


Вопросы:
* Поясните, обоснованно ли ругается _valgrind_ и в случае обоснованности, надо и можно ли этого избежать?
```
==29617== Thread #1: pthread_cond_destroy: destruction of condition variable being waited upon
==29617==    at 0x4A09098: ??? (in /usr/lib64/valgrind/vgpreload_helgrind-amd64-linux.so)
==29617==    by 0x4A0B9E8: pthread_cond_destroy@* (in /usr/lib64/valgrind/vgpreload_helgrind-amd64-linux.so)
==29617==    by 0x40ECB3: boost::condition_variable::~condition_variable() (condition_variable_fwd.hpp:84)
==29617==    by 0x40E084: ThreadPool::~ThreadPool() (ThreadPool.cpp:233)
==29617==    by 0x40E0D3: ThreadPool::~ThreadPool() (ThreadPool.cpp:234)
==29617==    by 0x41631A: main (main.cpp:82)
```


line 170 gives the opportunity to have something in this so it is not blank


TeamCity Xbim :: Xbim Exchange :: XbimExchange [Build 124](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=636&buildTypeId=Xbim_XbimExchange_XbimExchange) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:00:55


TeamCity Xbim :: Xbim Exchange :: XbimExchange [Build 125](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=637&buildTypeId=Xbim_XbimExchange_XbimExchange) is now running


How are you compiling this?


Well, but wouldn't it still make more sense to have another member method, like `sourceDependencies` and still use it here, and still unify the two sorting methods, since the rest of the bodies are the same?


Should be >= here and also in cases 3-4-5 and in computeCDF.


done (much better !)


you can remove this. It will be delt with in a later statement revision


nit: maybe name the test "initiate a new message with predefined content"


Thanks for merging!
When do you plan the next release to the pip?


TeamCity Xbim :: Xbim Exchange :: XbimExchange [Build 125](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=637&buildTypeId=Xbim_XbimExchange_XbimExchange) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:01:01


Alright.


I forget the hyperlinks into the commit comment!

Sorry, by I am not yet able to do the @zbeekman magical commits modifications :-) The links are the following:

https://groups.google.com/forum/#!topic/comp.lang.fortran/ykAodjcSPqc

https://gist.github.com/szaghi/97245928de2082d0c289


perhaps use an underscore instead?
like `/:dbname/_all/entries`


yeah, but this is different :-) i continue to another item when the error occured as the error is expected there


done


Sounds good, I will look into the name change a bit later.


> Can you please review this commit message and see if it correctly summarizes the cause of the problems I observed and described in that message?

LGTM


it is using the description of the codes instead of "Office Visit"


Definitely. My thinking was to do the simplest for now until routing is decided upon and implemented. We can get away with this for now until we add other pages.


TeamCity Xbim :: Xbim Exchange :: XbimExchange [Build 126](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=638&buildTypeId=Xbim_XbimExchange_XbimExchange) is now running


Values are already printed by computePDF, so this one is redundant.  And debug line just above is useless too, it does not clearly show which branch is being used.


Done. Changed for " The verifies that each of them returns on success (i.e. exit code zero)."


Octree (DeadParrot) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: Coverage Too Low

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


Hello Guys,

Just a sidenote:

There is a typo, hasPermisssion method has 3 "s" character ;)


Red alert... Red alert... Red alert...


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


TeamCity Xbim :: Xbim Exchange :: XbimExchange [Build 126](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=638&buildTypeId=Xbim_XbimExchange_XbimExchange) outcome was **SUCCESS**
Summary: Running Build time: 00:01:36


@damiarita We've had to revert this change in d396079f89e402de4610dccd8ac7e488288b6a8f, as it caused the sharing library not to be enqueued at all in some cases. You can follow the discussion here to find out more: #1223


Making it vendor agnostic would be good.


The rom is now pre-rooted? ;) 👍


ddoonnee ;-)


done


I guess the same could be said for the naming of the component. Maybe `video` instead of `vimeo` would be better?


og-Header raus, da die von Plugins gemacht werden sollen


Great input, thanks!

I've experimented with a similar syntax in https://github.com/stasm/l20n-syntax-experiments/blob/da539f16a0a76374d30e21782e8203b124ef6ba3/pattern-matching.example#L63-L94  Any thoughts on this and its readability?

Sidenote:  Using `0`, `1` and `_` as patterns to match in `betterNotify` works well for English, but other languages might have more complex plural rules which are harder to express concisely.  Hence the `plural` abstraction which returns a finite and well-defined list of possible variants.


Yeah but this one is specifically for vimeo - a the other one is specifically for youtube.


This project is about Monitoring distant locations over Internet. Monitoring pramateters ara:
-Temperature
-Air Pressure
-Moisture
-Sound
-Light

Project has two sides:
1-Ethernet Arduino with sensors and set to web client mode.
2-Server Side (Microsoft windows server 2012) with Classic ASP set on and any kind of Database.
 On this prototype I used Access as a database.

On the DashBoard side of project I used HTML5 for dynamic monitoring. Best Browser for this is Google Chrome.

Arduino client sends the sensor datas to server over a asp page. Server writes the data to Database with uniqe IoT ID and also writes the last data to Station's text file. Chrome DashBoard Client Read teh data, using ServerEvents (HTML5) and displays the data on Browser using Gauges and Graphics.

You can also set upper and lower thresholds for data. And get alarm if data over or lower that thresholds.


Sorry, I'm confusing. Did you mean we cannot use `is_system` to decide systemctl was attached either system or user bus? Or only `is_system` have to be not set here? (i.e. `is_user` is OK)

When calling `log_job_error_with_service_result()`, I could not find hint about currunt `systemctl` command is running with "--user" option or not. If `is_system` or `is_user` could not be used, is there any proper way to get hint?


Они позже отфильтруются по критерю близости. Если убрать town, то мы в Сибири городов недобираем.


Its interesting that you have chosen to write RCT2_GLOBAL for every single register address offset rather than using the correct structures and members as you go along. Often doing it as you go along makes it much easier to understand what the code is doing and allow you to predict logic or check if there is already a function that does the code you are just about to write.

Both me and Duncan have made timelapses that you could have a look at to see how we do this.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1134](http://teamcity.monogame.net/viewLog.html?buildId=13421&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 560, ignored: 6 Build time: 00:34:31


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1135](http://teamcity.monogame.net/viewLog.html?buildId=13425&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **FAILURE**
Summary: Snapshot dependency failed: MonoGame :: Develop (Mac) Build time: 00:00:08


Ok. I still have a few things to fix, I will look at it tonight.
Στις 7 Ιαν 2016 2:08 ΜΜ, ο χρήστης "Matthias Kuhn" <notifications@github.com>
έγραψε:

> The field is of type string
> I have just created a new shapefile with integer field, will push soon.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/opengisch/shpsync/commit/dfe2965e9cc44f80c4283af0cf143fdbb772350b#commitcomment-15315233>
> .
>


True!


GHT_Bugs (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1995 of 1995 tests passed)

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


I have that listed under https://trello.com/c/TnPxaDfl/6-install-scripts-parameters
I was first going to make it work (since I had to clean some repo i.e. gallery webtest)
thanks again for the suggestion


My bad
On Jan 7, 2016 8:22 AM, "Alx31" <notifications@github.com> wrote:
>
> 🙌😅
>
> —
> Reply to this email directly or view it on GitHub.


First work with plots in R. Nice job!


I used homebrew to build ffmpeg. It didn't install libffaac for patent reasons I believe. So my version defaulted to libmp3lame, which is free. I'm not sure is this was the right thing to do or not.


@f41c0r @JavaJens This is one of the diffs I found, while comparing upstream and this, I'd love to understand what it does and why :). Perhaps you should also encourage people to write bette commit messages?


https://github.com/sch3m4/pyadb
Nothing,I use this package.Maybe could help you.


brauchst du wegen mir jetzt hier nicht zu ändern, aber als info:
besser wäre `rex_url::backendPage('mediapool/upload', $arg_url);`


Yes


Show de bola! :clap:


I figured I would help the developer out. What do you think of this?


Poke @Laizerox and @X-Savior for the eventAI core patch.
I think it is better if we add the core patch first, before the script changes, isn't it? :wink:


@ricardok só lembra depois de converter os ícones pra `.svg` :guitar: :


Seems indeed like a typo.


Can you help me understand why the original said that a timeout should NOT raise an error? I *was* getting an error...thus this change.


@JavaJens Seems here are open comments that haven't been fixed considered. Does this need fixing?


Do you think it's worth to have this and L77 as reusable piece of logic?


It does work  somewhat better for me


org.drftpd.commandmanager.StandardCommandManager - Command slaves failed: ''
java.lang.NullPointerException
at org.drftpd.commands.slavemanagement.SlaveManagement.doSITE_SLAVES(SlaveManagement.java:167)


Number of lines contributed doesn't count. And, I did ask stack exchange if I should do it before this commit. http://programmers.stackexchange.com/q/306360/209547. Probably its wrong but I was just following the answer. I don't really know enough about this to defend myself here.


Sorry these already are fixed ;).


I change the cache to the vendor directory of this library in a previous commit, i think it's the best option for fastest build, WDYT ?


proc


Could do, but I don't know how to avoid double callbacks. This route would also be matched by the attachment route, wouldn't it?


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

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


Lol. That would explain a lot of things ! :-)
Thanks @xfurry for clarifying.


@JavaJens Is that App version? Why do these 2 differ between this branch and upstream?
2.8.2 (websocket) vs. 1.8.3 (upstream)


Yeah, thanks.


no change!


一些模块的control脚本写的比较挫，restart总是各种失败，这里在中间添加sleep和wait for尽可能避免因此而出错


等待端口启动


it's great to find the "std::cout<<"初始化车辆位置："<<siu_i<<"   "<<std::endl;"!!!!!!


Ok... let's do (1)


ahahahah


@ramakrishnaippili  all these Service are working without @Transactional annotation ?


I would probably also call the file abstractscript but adjust the import above to name it BaseAbstractScript


Rust supports conditions on patterns, like this:

```rust
let val = 15;
match val {
  3 => println!("Three"),
  n if n % 5 == 0 => println!("{} is divisible by 5", n),
  _ => println!("Other"),
}
```

Which could perhaps be stolen as:

```rust
betterNotify => match val {
  3 => "Three"
  n if n % 5 == 0 => "{{ n }} is divisible by 5"
  _ => "Other"
}
```

Which seems like it should be able to adapt almost any possible case. Supporting pattern guards would perhaps be overly complex, though, and since variable capturing would also be required, plain string patterns would need to be different.


由于open-falcon.git在link gitmodules的时候是指定了特定版本号，导致使用递归模式clone的时候获取不到最新代码，因此这里干脆改成人工枚举各个git库来下载


saw approval from lehoff
at https://github.com/basho/riak-erlang-client/commit/5aa86acf74a606dccd0a2abac5d4b110525857d5


merging basho/riak-erlang-client/feature/jrd/ts-spark-coverage = 5aa86acf into borshop-integration-256-feature/jrd/ts-spark-coverage


basho/riak-erlang-client/feature/jrd/ts-spark-coverage = 5aa86acf merged ok, testing candidate = 5d733c95


Ignore that, I was blind.


엌ㅋㅋ


Well the organisation is cyanogenmod in this case, so you should go with the first answer (aka no, don't update the headers).


Wow. How could I miss it? It should be somewhere else (maybe in a catch block?). I'll fix it tonight.


I'll try to get some docs!


I have the case of a class named `List$impl`, which contains an anonymous class `List$impl$1`, and when the visitor visits `$1`, `inner` is `null` so the algorithm detects `List$impl` as an inner class, which is wrong.

What I meant in my previous comment is that given the precedence of Java operators, the current code is in fact the same as

if (inner == null || **(** everything else **)**)

Is it really what you meant?


Hello Mr Beans,

First thanks for this great app, ive totally overhauled it but i run into an issue.
I would like to ask if you can help me out here.
I am stuck at implementing dslv from slim, as ActionListViewSettings(alvs) must be loaded as a subfragment i have trouble loading it. i used the displaySubFrag method but the preference in the xml contains extra values for alvs. as we remove the fragment from the preference that now is assigned in arrays we need to remove the extra too. So now i get NPE as these values are not found by alvs ofcourse. So how to solve this issue

This is how the original looks like:

        <PreferenceScreen
            android:key="navigation_bar_button_settings"
            android:title="@string/customize_title"
            android:summary="@string/navigation_bar_button_settings_summary"
            android:fragment="com.android.settings.vrtoxin.dslv.ActionListViewSettings">
                <extra android:name="actionMode" android:value="0" />
                <extra android:name="maxAllowedActions" android:value="5" />
                <extra android:name="defaultNumberOfActions" android:value="3" />
                <extra android:name="disableDeleteLastEntry" android:value="true" />
        </PreferenceScreen>

Current looks like:

        <Preference
            android:key="navigation_bar_button_settings"
            android:title="@string/customize_title"
            android:summary="@string/navigation_bar_button_settings_summary">
        </Preference>

Btw sorry for bothering you here, couldnt find your hangouts or a other way to contact you.
You can find me on hangout, Stephan Schols or NL-BlackDragon.

Best regards,
Stephan


Because the libraries differ ;)
The official textsecure lib doesn't allow to set the 'fetches_messages' flag


Wouldn't it be better to just make `len` into an `unsigned int` to begin with?


nOOOOOOOOOOO :trollface:


:tada:


:fireworks: :fireworks: :fireworks:

Thank you!!!


C'est dommage de faire disparaître le ```role```.


Wer mergt am Ende in Master rein?


@yale2a1
```bf
++++++++[>+++++++++<-]>++++.+++.---.
```


Currently the RedPhone server offers no way to enable signaling of incoming calls without using GCM.
Therefore, the call registration is disabled in this fork. You can enable this and will see that outgoing calls work, but incoming calls will fail (at the caller side).


OK.


I am not entirely sure we actually **have** to set a specific Ruby version. IIRC it was mostly needed to get one specific gem to work which I have thrown out since.

And because it's the version I am running on my machine but that should not actually be a valid reason.


I think this is what you want:

https://ci.appveyor.com/api/github/webhook?id=9afuk16ggu7168cc

Thanks!

On Wed, Jan 6, 2016 at 11:33 PM, Tony Kelman <notifications@github.com>
wrote:

> Do you want to host it on your AppVeyor account? If you've signed up for
> one and try to enable this repo, go to the settings and copy the "webhook
> url" here and I can do the rest.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/JuliaLang/Roots.jl/commit/da017b2cc7b11c493f902d6245de8985b40ca19a#commitcomment-15307817>
> .
>

--
John Verzani
Chair, Department of Mathematics
College of Staten Island, CUNY
verzani@math.csi.cuny.edu


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


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1176.9ad04c381b1f032a557f889d60e7f6ed5d91c7cf](http://build.palaso.org/viewLog.html?buildId=67596&buildTypeId=bt431) outcome was **UNKNOWN**
Summary: Canceled (Tests failed: 2 (2 new), passed: 809, ignored: 7) Build time: 00:27:21
### Failed tests
```
GlyssenTests.dll: GlyssenTests.ProjectTests.UpdateProjectFromBundleData: <no details avaliable>

GlyssenTests.dll: GlyssenTests.ProjectTests.UpdateProjectFromBundleData_BundleDoesNotContainLdmlFile_MaintainsOriginalQuoteSystem: <no details avaliable>

```


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


@Alfredo-Delgado this is published in steal-bundler 0.2.4.


Do we really need it here?


It makes me a little nervous when we explicitly invoke System.exit without qualification like this.

Historically, there have been MANY cases where we SHOULD NOT invoke System.exit because we cannot assume the invocation to main is actually the initial and only Class main that is going to be run.

For example, it is our test subclasses that are invoking this SeleniumPlus.main method.  It is entirely possible there will be subclasses that will want to invoke this main AND THEN continue with other things--like post-test processing.

We should consider adding support like exitAllowed (set/get) and ONLY perform System.exit *IF* the setting is true.


`changed_when: False` would be better, because the `locale -a` command never actually changes anything on the system.


It doesn't look like you need to pass the start date as an argument since the subscription object has it already?


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1177.3d2ec64abbd578ec29ade7de81168f499fdee32d](http://build.palaso.org/viewLog.html?buildId=67599&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 812, ignored: 7 Build time: 00:18:32


awesome


I would use `command: locale -a`.  You're not using any shell features (pipes/redirection/etc), so why spawn an extra shell?


IMHO this fact should be mentioned somewhere, as I saw tickets about that. A first-start message, in the readme or something?


Martin meinte mal wir sollen alles in WorkInProgress pushen und er mergt danach.


@NocusNibbes  : adding the boolean $Blocksave is ok, however, I was handling that in another way
but :+1:


Looks good!


I suggested `... not in locales.stdout.lower().splitlines()` on IRC because if `postgresql_locale` is something like `en_GB` (i.e. Latin-1), but `locales -a` returns just `en_GB.UTF-8`, you would want to run `locale-gen`, even though `en_GB` is a substring that appears in the output.


Looks like job will trigger 5pm every Sunday, right ?


@ddur I noticed that the `this.Equals(that)` was recursing - I believe this is what you were intending - thoughts?


I missed myself few fields ;) But the roundtrip test uncovered everything.


Let me guess, you just copied that ;)
I think at least the maven artifact ids are wrong.


Looks good.


Done


Not threadsafe


Maybe add a `assertListenerStatus()` after creation?


OK, Done.


So this version reflects the version of your modded libtextsecure-java? Thanks!


Oh my... Is it means that MC still alive? Is release have planned? Thanks.


I'll change the bug ID. And the original title of bug 39384 is "QQ 7.6 can't be started after the installer first start it." QQ 7.6 still can't be start because of ntoskrnl.exe under wine 1.9.0.


typo


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


主题


all tests pass:
success: http://buildbot.bos1/builders/erlang-client-test/builds/256


TeamCity pull requests :: netty [Build 981](https://garage.netty.io/teamcity/viewLog.html?buildId=4571&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:06:14


why is this field required ? booking.ticket.count can get you the same result.


@Arcanemagus
I just copy it from https://github.com/atom/ci/blob/master/appveyor.yml#L8-L9


okay


yes..:)


saw approval from gordonguthrie
at https://github.com/basho/riak_kv/commit/db0adfe762c9efafdc571f0f321544c4fa246be7


merging basho/riak_kv/at/do_not_squash_null_cells = db0adfe7 into borshop-integration-1321-at/do_not_squash_null_cells


basho/riak_kv/at/do_not_squash_null_cells = db0adfe7 merged ok, testing candidate = d641ff67


you can move this to a different model altogether call payment, in addition you can keep card details and stuff as a different model to save cards (for future scope)


Solved !! About to push...


For date-based cancellations, I suppose the code already does something similar (i.e. if requested date is before the future start date, the effective cancellation date is the start date)?


Translation .mo file for Slovak language (SK) has been lost with this commit. Please add it again.


A common denominator here is a set of type definitions in the `Stmt_piqi` module, that is generated by a piqic compiler. So the OCaml compiler see them as equivalent. You can see that I reuse `Bil_piqi.exp_of_piqi` in the IR serialization module.


the idea for keeping payment model separate is that you can then maintain transactions , so if a payment fails you wont need to modify the user document.


@jmcphers It looks like they changed the semantics of `path` at some point to just mean the directory. `full name` seems to work fine though.


You may want to use `Array#some` here: https://developer.mozilla.org/fr/docs/Web/JavaScript/Reference/Objets_globaux/Array/some

If you're using short function syntax, that gives:

```js
if (users.some(user => user.user_id === user_id)) {...
```

>  As @leplatrem suggested, about using the http api instead of Kinto.js (as used above), maybe it would be easier that way, your thoughts?

As we're an offline first client, we should try to avoid querying the server whenever it's possible :)


done


@musamusa  though not part of your changes but i don't think there is `isnt` keyword in Python, i think they meant `is not`


how will you handle a condition where capacity < screen.seats validation ?


This basically drop the config needed for https://github.com/geonetwork/core-geonetwork/pull/426


GHT_Bugs (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2033 of 2033 tests passed)

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


"Improvement" ? This crashed l4-l5 session communication :p


It appears that this change sometimes causes failures.


do you need an association here ? you can just keep a reference of seat on ticket, coz seat having a ticket is a little illogical.


@gorhill
Just a quick observation (on v1.5.3 release version), the settings icon is hardly visible when your monitor is slightly titled above the eye level. It is visible properly when the cursor is hovered over the black bar.

For exp. users, this isn't a problem. However, i am not sure for the beginners/1st time users, if the icon opacity is visible enough when the pop-up is opened (and the cursor is not on the black bar)..


There should be no debug statements


Done


@musamusa please can we replace these magic numbers with a constant such as COUNTRY_LEVEL, STATE_LEVEL etc and also update other places they appeared in


Macarena más guapa :-*


Hah! The `'A'` case is missing. Good catch, have to add this.


Just making it good for reading....


Why increment by 2 ?


How about ```ASSERT(m == NULL || m == mp)``` and remove the ```if (m)```.


It could be a lifetime issue with an array object that is being used by the distributed evaluator.


The problem with this (I think) is that the HTML and epub versions would break.   This is a super-good idea.  If we could make the epub and HTML versions work.


Octree (DeadParrot) - Win64-Windows-7-VisualStudio-12: OK (1901 of 2009 tests passed)

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


I think i have problem running qunit versions newer than 0.7.2 for this change. on travis i get this error:

Testing  /home/travis/build/kangax/fabric.js/dist/fabric.js ...
[Error: Process blocked for too long]

Could it be related to this new function? Can i change "maxBlockDuration: 1000" for my test?


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


yangzaiqing.cn


I know that the ocaml compiler will see it within the scope of the BAP project. The Bap.Std interface doesn't expose those types, so if you ardently want users to strictly consume from only that, that would be a complication.


@steelbrain
I just saw it [somewhere](https://github.com/search?q=%2F%5Ewin%2F.test%28process.platform%29&type=Code&utf8=%E2%9C%93)
I'm not very sure in the future it will be win64.


The project URL should be `https://github.com/jonschlinkert/micromatch`, I think.


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


TeamCity pull requests :: netty [Build 982](https://garage.netty.io/teamcity/viewLog.html?buildId=4572&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:01:55


Definitions should be https://github.com/DefinitelyTyped/DefinitelyTyped (the repo was moved a while back)


root->value must be not collectable?


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


some tests failed:
failure: http://buildbot.bos1/builders/test-riak_kv/builds/3072


Enabled. Did you trigger the first few builds manually? For what I did to the webhook, see https://github.com/JuliaOpt/JuMP.jl/pull/362#issuecomment-70610293


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


Yes, see the README for how to install.


Yeah the piqi compiler barks a meaningless (and possibly misleading) a warning, but the module name that is produced within the output _piqi.ml does in fact change if you use .ocaml-module. I see what you mean that the final result made it in this case redundant, just making sure to clarify in the advent of future mishap with piqi.


**Code Quality - Nitpick**

You should use status codes that reflects the error. In this case, the error was generated by the user and not the server, hence the error should fall in the 400 range.

Most similar to this error is 409 CONFLICT, you should do a little research to verify the status codes akin to the errors you send to the user.


in my VM running FC22 x86_64, ping often failed (but sometime okay) without the following patch.

``` c
diff --git a/tools/lkl/lib/posix-host.c b/tools/lkl/lib/posix-host.c
index caac1df561c4..391219fb524e 100644
--- a/tools/lkl/lib/posix-host.c
+++ b/tools/lkl/lib/posix-host.c
@@ -242,7 +242,12 @@ static int net_poll(union lkl_netdev nd, int events)
 	if (events & LKL_DEV_NET_POLL_TX)
 		pfd.events |= POLLOUT;

-	poll(&pfd, 1, -1);
+	while (1) {
+		int err = poll(&pfd, 1, -1);
+		if (err < 0 && errno == EINTR)
+			continue;
+		break;
+	}

 	if (pfd.revents & (POLLHUP | POLLNVAL))
 		return -1;
```

https://gist.github.com/thehajime/cc5e8b712066c32b11e7

we may adjust ret


@parvatihiremath thats a bad news, thats not how it should behave... Let me check


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


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

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


done


Thanks
what the<ui-clockpicker> tag is actually representing?


TeamCity pull requests :: netty [Build 983](https://garage.netty.io/teamcity/viewLog.html?buildId=4573&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:01:51


test is failing.
Probably adding
`contents = contents.replace("<script src=\"resources/libraries/flot/jquery.flot.resize.js\"></script>", "");`
helps.


'csv only allowed' => 'Charger uniquement un fichier CSV, s\'il vous plait.',
should be ‘Uniquement des fichiers CSV sont permis.'


Why are you changing these to underscores?


Can you elaborate here?


It's a good suggestion indeed. I'll postpone this change as an improvement to be done later.


nice


I think you forget to replace '/dev/ttyUSB0' with device here


why does Constraint has its own type?


@Nagarei
mergeしない方向でcommitしていたからびっくりしました。
``SFINAE_test_add_insert_binary_operator``ブランチどうします？消します？


Whitespace? There originally was no whitespace. Should I add whitespace?


@smcv

> Explaining the rationale for some of my recent commits:
> ideally I'd like to make everything in g_d_p/__init__.py able to round-trip losslessly to/from the YAML

So that means adding these calls to substitute() all over the place to decode install_to just-in-time everywhere ?
Well I can do it, if that's needed.


I don't know what to do here... I'll let it ther for now.


Fixed in e5b1d68b7baa4eef011b5138f628e14cb5c7b782


But this is my project containing some CyanogenMod bits which have been given proper copyrights to which is what I asked about in the stack exchange question. I did mention using android git source code and modifying that there. Apache license section 4 b also States "You must cause any modified files to carry prominent notices stating that You changed the files" so does most licenses used for open source projects.


do you have a usecase for welch on short segments like epochs?

for short signals you should use tapers.

we should not advertise the use of welch on epochs. Especially in examples.


I think this can be made prettier like this:

```sh
  cat >~/.bash_profile <<-\EOF
	# generated by Git for Windows
	test -f ~/.profile && . ~/.profile
	test -f ~/.bashrc && . ~/.bashrc
	EOF
```

(Note the `-` after the `<<`...)


This is a bad idea:

- it invalidates the Travis cache on each build (as composer will still regenerate some files, even though the content is the same, and the autoloader won't even have the same content)
- if a build corrupts the vendor folder, all subsequent builds will be broken (until you delete the Travis cache entirely)


I have no part in this but I also love both of you


**Code Quality - Nitpick**

In your routes, you want to have only resources, all modifiers of the route should be added as query parameters.
Here a suggested route pattern
.
```
/api/documents?limit=2
```


@musamusa part of the feedback from UNICEF of rapidpro phase 2 part 1 i.e geojson import will remove `in_country`  property so i suggest something else maybe using Django Mptt get_root() which returns the country of any boundary location or extending the previous solution to cover this new level, whichever is easier and more flexible.


humn... considering that unregister would only happen during initialization (if the device wasn't really found) or on exit... I'll add an assert here.


couldn't we do this?
```python
for _, m in loaded_modules:
    m.init()
```

Looks simpler, and avoids a lookup.


Sigh, I confused zval_dtor() and zval_ptr_dtor(). Thanks for spotting, will fix.


:( premature suggestion - still failing


这里直接设置jiuhaoused flag啊。。。用不着给card加flag了


The same property is declared in the header too.


nice squash! too long subject though :)


Oh yeah. Shameless typo fixes.


> "tohle všechno je skvělé. Pro webové aplikace.". Tato formulace na mě také nepůsobí dobře. Asi bych to spojil do jedné věty.

Souhlas.

> a teď jsem si všiml, že to znamená, že pak s vyprávěním přecházíš do bodu 2)

Dal jsem radši pryč. Tyhle přechodníky nejsou moc obvyklé, nechci lidem zbytečně motat hlavu.


@fsaintjacques

> Do we want to maintain a non-avx2 implementation?

I think it is helpful to know how well we do compare to whatever the compiler does with generic code. Don't you?


just wondering: comment is about mysql, but change is only for mssql ...


@musamusa how about a more descriptive name e.g get_location_field(type)


You can add validations to multiple fields life the code below.
```
status = validator.checkForMandatoryFields(userRequestDto, UsersRequestDto.UsersRequestFields.password, UsersRequestDto.UsersRequestFields.email);
```
This can reduce the number of lines of code.


You need to at least change this


Ah, this is a `readwrite` redeclaration internally. But I guess you're right that we just touch the ivar internally and don't actually need the property to be writeable.


UUID


This part of logic (compound condition) can also go into a reusable part of code, i.e isTreatedAsEmpty($node)


rotted. just remove this timeout and let the test runner do its thing


Is that the IP of the instance ?


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


You should fiddle with the signs in https://github.com/core-dd/libpredict/blob/master/tests/orbit-t.cpp#L148, not the signs in the raw testcase data.


/*
Author: Mette Espedal
Student ID: no idea
E-mail: mette.espedal@gmail.com*/

#include <stdio.h>
#include <stdlib.h>
#include <math.h>  // så langt virker det likt enten denne er med eller ikke.

int main(void)

	{
		int integer1; // first number to e entered by the user.  kunne også vært kalt var1 ettersom det er første variabel
		int integer2; // second number to be entered by the user. evt. var2 istedenfor integer2
		int sum; // variable in which the sum will be stored

				 //adding
		printf("\n Enter a number\n "); // prompt, displayed on screen. Fant at ved å bruke spacebar etter \n på slutten ungikk jeg at tallet kom helt til venstre.
		scanf("%d", &integer1); // read an integer

		printf("\n Enter a second number\n "); // prompt, displayed on screen
		scanf("%d", &integer2); //read an integer

		sum = integer1 + integer2; // assign  total to sum
		printf("\n when added together the sum is %d\n", sum); // print sum

									//subtracting
		printf("\n Enter a number\n "); // prompt, displayed on screen
		scanf("%d", &integer1); // read an integer

		printf("\n Enter a second number\n "); // prompt, displayed on screen
		scanf("%d", &integer2); //read an integer

		sum = integer1 - integer2; // assign  total to sum
		printf("\n when subtracting the sum is %d\n", sum); // print sum

									//multiplying
		printf("\n Enter a number\n "); // prompt, displayed on screen
		scanf("%d", &integer1); // read an integer

		printf("\n Enter a second number\n "); // prompt, displayed on screen
		scanf("%d", &integer2); //read an integer

		sum = integer1 * integer2; // assign  total to sum
		printf("\n when multiplied the sum is %d\n", sum); // print sum

									//dividing
		printf("\n Enter a number\n "); // prompt, displayed on screen
		scanf("%d", &integer1); // read an integer

		printf("\n Enter a second number\n "); // prompt, displayed on screen
		scanf("%d", &integer2); //read an integer

		sum = integer1 / integer2; // assign  total to sum
		printf("\n when divided the sum is %d\n", sum); // print sum

		printf("This is how far I've come so far.\n All works exept dividing, so I know I have more work to do \n to figure out how double works. \n Scrapped the first attempts as they made sure nothing worked. \n To be continued...");
		scanf("%d");  // når jeg skrev denne inn fikk jeg teksten på slutten til å bli stående. Hvorfor virker ikke neste kommandoen, den skulle jo ha tatt seg av det.

		getchar();  // trodde denne skulle gjøre at teksten ikke forsvant så fort, men den så ikke ut til å hjelpe. Da må man bli oppfinnsom...

		return 0;

	}


regarding repro: have you tried `make stressrace STRESSFLAGS='-p 16'`?


Just out of curiosity, why did you rollback the jquery ui version? Thanks for your work


I agree with @sebbacon
Given this is just for the fetchers, we should implement this within the #fetch_registry_page method, or create a private method called _http_get_with_retry which handles the retrying.

I prefer the school of thought that monkey patching should be only ever used as a last resort if there's no other way of doing it.


Should this line be `mm.isMatch('*.md', {dot: true})('.verb.md');`?


You can just remove this method altogether


done.


GHT_Bugs (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (2001 of 2001 tests passed)

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


static


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


Should this: ```system.version.ProductVersion >= '10.10'``` be ```system.version.ProductVersion >= '10.10'```


It was broken anyway because of the addition of `$this->name`.


Ahah, nice one, sorry...


TeamCity Xbim :: Xbim Geometry :: XbimGeometry [Build 115](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=640&buildTypeId=Xbim_XbimGeometry_XbimGeometry) is now running


Nonsense is going on. It looks like a leftover from before i realized that fpha is case sensitive in first argument. These lines should go.


Right Sir I was not aware of this. I will make the proposed changes.


These lines as in the manual set to 1.


Nope, wrong fix.

If you set the "usync/" folder as source, the importer will use the dotted separated file names as path in the graph, so actually variable.yml / site_name: will become variable.site_name in the graph.


TeamCity testProject :: build [Build 29](http://192.168.0.5:8111/viewLog.html?buildId=55&buildTypeId=TestProject_Build&guest=1) is now running


Shouldn't this be just record.name, as the type is stored in the document type?


Something that would be good to add to the tests is a way to verify the correct return type.  Although there are a number of ways to do this, one would be to declare a `boolean` type variable and assign the result of `isMatch`, `contains`, etc. to it.  If the type definition changed in the future and started returning something else, it would catch the error.

Example:
```js
let boolResult: boolean;

boolResult = mm.isMatch('verb.md', '*.md');
boolResult = mm.isMatch('verb.md', '*.md', {dot: true});
...
```


So were there bugs? :8ball:


A more RESTful approach could be:

* like: `POST` - /projects/:id/likes
* unlike: `DELETE` (or `PATCH`) - /projects/:id/likes


@Ken-Lo fyi!


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


TeamCity testProject :: build [Build 29](http://192.168.0.5:8111/viewLog.html?buildId=55&buildTypeId=TestProject_Build&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:02:30


why disable tests?


Hm, did I capitalize Copyright wrong? Oh well...


оставьте такой нейминг для комментов и 2 строчек коммитов


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


Are there known problems with your version in terms of WhisperSystem rejecting it from their server or something like that?


Review done


to keep the old behaviour it should set to `LANGUAGE_CODE` - i assume from settings
```
Before Django 1.8, get_language() always returned LANGUAGE_CODE when translations were deactivated.
```


I consider `== true` a bad practice (Java'ism). LHS can be an int (a valid representation of bool), thus in comparison `true` gets promoted to int yielding `1`, which may not necessary be equal to a truthy `!0` on left side.


this -fPIC is duplicated


That's indeed a much better to get the arguments, but unfortunately this change would break any existing scripts. I wonder if there's a way to support both the old way of passing argument and this new one?


No, i gave up. Did you get any further with this?


@dillaman So they are displayed in a more uniform style and looks better.


We now include Python code as well.


I would have thought toBool() would only return true or false, but the whole comparison is unnecessary anyway so I'll get rid of it.


TeamCity testProject :: build [Build 30](http://192.168.0.5:8111/viewLog.html?buildId=56&buildTypeId=TestProject_Build&guest=1) is now running


It would be good to declare a `type` or `interface` for `((filePath: string) => boolean)` so that it can be reused, and type annotations can be more easily added where it is used.  Perhaps something like `MatchFunction`?


lol... happy new year!


I still find it questionable behavior.


Ah, I see now, thanks. Somehow failed to consider that anonymous classes may belong to other parents, too.


fixes #238


Where is the data of the raw testcase coming from?
IMHO the raw data is wrong in terms of sign:
E.g. looking into 'tests/data/sat_VELA-1_201509261800.test' the range rate is 132158.75. Therefore the distance between object and observer is growing. Hence, the Doppler shift must be negative as the wavelength relatively seen stretches. Any comment on that?


@dillaman As you can see, most of the options use underscores, but a few of them use dashes. I just want to make them all the same and not to confuse caller.


Didn't saw that it's `readonly` in `.h` and `readwrite` in `.m` when I wrote the first comment though. I didn't know that you can redeclare a property as `readwrite` in the `.m` so you can use it, but in this case, the code should either use `self.cachedRequestBody`, either remove the declaration from the `.m` file, right?


Hey! Did you supply your recaptcha login to the joomla recaptcha plugin?
The modifications I did are still working for me. You have to understand however, that this not a fully tested bulletproof solution. So you may have to fiddle around a bit. I wrote the jcomments guys a message and asked them to include my modifications in the next official jcomments version. Sadly they didn't release one yet.

If you get recaptcha 2.0 working for you. Please tell me what was missing so I can update the code here! Thanks!
Thomas


In this case it is bool and it will not create an error.
But still a bad practice.


TeamCity testProject :: build [Build 30](http://192.168.0.5:8111/viewLog.html?buildId=56&buildTypeId=TestProject_Build&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:02:45


This commit seems to have broken many tests, they fail with `java.lang.RuntimeException: DownloadPath(interface de.flapdoodle.embed.process.config.store.IDownloadPath) not set`.


TeamCity MahApps :: MahApps.Metro PullRequests [Build 1.2.1.22](http://builds.brendanforster.com/viewLog.html?buildId=396&buildTypeId=MahApps_MahAppsMetroPullRequests) is now running


It would be good to add a type annotation onto this line to verify the return type, and/or to use that variable below in a similar way that it shows in the example code in the docs.
Also, it says that the pattern for `matcher` can be `glob/string`, `regex`, or `function`, so it would be good to add a test for each of those:

```js
let isMatch: MatchFunction = mm.matcher('*.md');
isMatch = mm.matcher(/md/);
isMatch = mm.matcher((filePath: string) => true);
```


nope, probably needs Raft elections slower than anything imaginable.


TeamCity MahApps :: MahApps.Metro PullRequests [Build 1.2.1.22](http://builds.brendanforster.com/viewLog.html?buildId=396&buildTypeId=MahApps_MahAppsMetroPullRequests) outcome was **SUCCESS**
Summary: Running Build time: 00:00:34


Yeah it seems weird, but the wiki suggests that it can be used for more than one space.
http://wiki.openstreetmap.org/wiki/Tag:amenity%3Dparking_space


I've seen a very elusive and very hard to track bug, where LHS in such comparison was an uninitialised `bool` class field. It could yield both `== true` and `== false` comparisons false. :D


This is dangerous, IMNSHO. You create the class in `else` clause, but register it anyway out if `if-else`? Is it ok to do it this way?


TeamCity MahApps :: MahApps.Metro PullRequests [Build 1.2.1.23](http://builds.brendanforster.com/viewLog.html?buildId=397&buildTypeId=MahApps_MahAppsMetroPullRequests) is now running


将字符内容移至strings.xml文件中，增加连接函数


No `import` for this?


Pattern guards would indeed make `match` much more versatile.  Here's another experiment with them: https://github.com/stasm/l20n-syntax-experiments/commit/bfd9e477bb461a3a810662ae700d3a7d12bb1ca4.  I don't think we need to bind the variable used in the pattern; the guard could only have access to variables already defined in the scope.

For plain string patterns I've been using the keyword notation with a leading colon (`:foo`).  Keywords would be interchangeable with quoted strings (`"foo"`) as long as no whitespace is present, which I think would work well for things like `:one`, `:other`, `:feminine` etc.


TeamCity MahApps :: MahApps.Metro PullRequests [Build 1.2.1.23](http://builds.brendanforster.com/viewLog.html?buildId=397&buildTypeId=MahApps_MahAppsMetroPullRequests) outcome was **SUCCESS**
Summary: Running Build time: 00:00:40


TeamCity MahApps :: MahApps.Metro PullRequests [Build 1.2.1.24](http://builds.brendanforster.com/viewLog.html?buildId=398&buildTypeId=MahApps_MahAppsMetroPullRequests) is now running


forbiden is for the U.S. Vice President ;-)


See #188


it's in next commit