> The reallllly common problem with this install step is that people were installing into their toolchain, and then passing -L paths inside their toolchain, which confuses/breaks multilib toolchains

When you say toolchain do you mean 'gcc' or 'ld'? For 'ld' it is indeed required to use `-L` flag like this `-L /usr/arm-none-eabi/lib/`. The explanation is that default script includes a number paths where it searches for libopencm3 libraries:

```
$ arm-none-eabi-ld --verbose | grep SEARCH_DIR
SEARCH_DIR("=/usr/arm-none-eabi/lib"); SEARCH_DIR("=/usr/local/lib"); SEARCH_DIR("=/lib"); SEARCH_DIR("=/usr/lib");
```

I believe most users do not use default linker script and instead include stm32 one `INCLUDE libopencm3_stm32f4.ld`. That script does not specify any search paths:

```
$ arm-none-eabi-ld --verbose -T /usr/arm-none-eabi/lib/libopencm3_stm32f4.ld  | grep SEARCH_DIR
```

Thus path `/usr/arm-none-eabi/lib/` should be provided by the command line.

Other way to solve the problem is to add `SEARCH_DIR` directly to stm32 linker scripts. For example with following script instruction `SEARCH_DIR("=/lib");` in `/usr/arm-none-eabi/lib/libopencm3_stm32f4.ld` I do not need to use any -L flag.


Most examples are not using SOL_LOG, i just follow the pattern, as are just few prints here, i prefer not change if you don't mind.


@ericanaughton Hey! I'm sure it's wicked old. You can delete any of my branches. thxx


Обязательно исправлю


Good question, I'm not sure. I would guess no, but it might change as the set of remotely running jobs changes.


sorry about that, I had written the setup script "from head". in the future you can just let me know about such small issues without a meeting :)


OK, I will change the comment.

On Wed, Dec 9, 2015 at 10:02 AM, Aaron Lehmann <notifications@github.com>
wrote:

> This contradicts the docstring, which says "Payload returns the raw,
> signed content of the signed manifest. The contents can be used to
> calculate the content identifier.". If the function includes the signatures
> in its return value, it's not returning the signed content, and the return
> value is also not useful for computing the content address, because schema1
> content addresses are calculated on the portion without the signatures.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/RichardScothern/distribution/commit/2078e97678052e27e1481f32b38542bfca732e3b#commitcomment-14880625>
> .
>


**Code Quality - Critical**

You assigned  `Storage::disk('s3')` to `$s3` on line 182 and then on line 188 you still used `Storage::disk('s3')` . That's repetition, you should just use `$s3` on line 188.

Then on line 186, there's no need assigning `$fileName` to `$filePath`, just go ahead and use it on line 186.


Yeah I'd say don't attempt to catch exceptions this broadly; let them bubble back to the caller as is


Aren't those privileges required to read symlink information?


all tests pass:
success: http://buildbot.bos1/builders/test-riak_kv/builds/2908


salam khodet feker kon yad begir


@borshop merge


I don't know. How can we check it?


How will the content address be computed if `Payload` includes the signatures?


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


Remove the extra space on line 78


In this ticket, it seemed that we require those privileges to make our `stat()` work: https://github.com/git-for-windows/git/pull/220#issuecomment-114063135


**Code Quality - Nitpick**

Line 82 to 92 is not on the same grid line as the rest of the code in this function. Please fix!


This isn't your fault, but the singular "type" in the sentence above makes it sound like we're only talking about one type. Maybe "... specifies the relationship between an Iterator type and a Semiregular type whose values denote a range."


Feel free to delete this snippet. I don't think `BigQueryCall` has to be defensive against a client that initialized it with a dud `google_auth_config` param.


**Code Quality - Nitpick**

Line 95 shouldn't exist, just put the code on the same line as 94.


TeamCity pull requests :: netty [Build 813](https://garage.netty.io/teamcity/viewLog.html?buildId=3899&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 6239, ignored: 21 Build time: 00:42:33


**Code Quality - Nitpick**

Line 122 shouldn't exist, just put the code on the same line as 121.


Do you have an idea about when you plans to launch a release with these fixes?


s/Container/Pod


**Code Quality - Nitpick**

How does this even work?... Why is line 134 and 135 here? :flushed:


This does not work. I think the `--filter=r` should be `filter=r`.


I would remove `...in the stream processor space` at the end. It's repetitive.


Maybe say `further growing` to further stress the popularity point? ;)


developed *for* Storm?


`adaptations` => `changes`?


`minimizes` reads like it is possible in another way imo. Maybe `enables`?


I would remove this line, because it is distracting in the main text. Or maybe as a footnote?


On a first read this may sound contradictory, because you say that you have to change the submission code before (although I see that it is API compatible and hence it is minimal). Maybe we can rephrase this to be easier to understand? I would actually remove the second sentence (`Only the translation...`) as it became clear already in the sections before.


It's a little weird to have this module (telescope) raise an exception from a different module (external). Can we push this error wrapping logic down into a function in external.py?


I would remove the boilerplate code. We don't need to criticize ourselves for this awesome feature ;)


is it constant or continuous?


To times `even` shortly after the other sounds repetitive to me


beautiful!


This function was for translating atom_cmpxchg/atomic_cmpxchg to atomic_compare_exchange. After your change, will atom_cmpxchg/atomic_cmpxchg still be correctly translated to SPIR-V?


Great post! Very nice. I had some minor comments inline. Thanks for writing this up and the great addition to Flink.


@adamcooke  any specific reason for pulling this out...?!


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


The content address is calculated and returned on `manifestStore.Put` using canonical JSON.  If it is required elsewhere .canonical will need to be accessed.


@Argon- : well I use Linux/KDE  (chakraOS, I was one of their ex-maintainers also), but I do have high DPI screen on my ultrabook and I will buy a 4k monitor for my desktop when there is good /cheap'ish one with G-sync. (aka. 4k with good 10bit panel+ G-sync for a good price).

I wonder is it possible to get this working on Linux also (somehow) or do I just need to remove it


Ok so this how i add a line to this file?


`distribution.UnmarshalManifest` also returns the canonical digest when called on a schema1 manifest.


Nice, good clean test


sup


**Code Quality - Nitpick**

Don't you think it's better to use the `$request` instance instead of the `Input` Facade?.. For uniformity.

Example `$request->get('title')`


jg


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


Oops, looks like I did not test my change, sorry, This seems much better!


Эмм. Хитрость этого подхода (напечатать или свои внутренние фильмы, или переданные снаружи) от меня ускользает.


xDddddd


It might happen if kubelet has already deleted the pod.
 1. kubelet updates status
 2. kubelet deletes pod from the apiserver, and the corresponding entry in the status cache
 3. kubelet receives the status update (1) from watch
 4. kubelet sees the pod been removed from watch

Step (3) is the case where this happens.

I think this is tolerable, but to avoid that completely, we can chose not to delete the entry until we can't see the pod from watch.


Rather than passing an opacity into updateOpacity, it should just know based on `this.opacity` and `this.visible`. That cuts down on potential errors.


`ORDER`   shouldn't it be `ORDERS`


I can see what you mean. We're trying to find the rowIndex in the table. This method can return a -1 for a not-found. This is used for the `contains` method in the `RealmResults`. Christian and I discussed this and the first stab was `find`. Maybe it should be `findRowIndex` or something like that? Thoughts?


Correct! Fixed it.


ORDER shouldn't it be ORDERS


For instance here...you don't want to have to calculate the opacity here, but the way updateOpacity is currently written, it's an error to not pass anything in like this.

Please address in the next PR. :)


А вот как бы здесь без if обойтись, м? Предоставив «блок по умолчанию», если пользователь не передал блока.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 8814](https://teamcity.mesosphere.io/viewLog.html?buildId=89306&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 0, ignored: 205 Build time: 00:09:07
### Failed tests
```
null: Kubernetes e2e suite.Kubectl client Guestbook application should create and stop a working application [Conformance]: <no details avaliable>

```


**Code Quality - Nitpick**

You can just use `$this->uploadFileToS3($request)` and `$this->getImageFileUrl($request->cover)` straight on line 144 and 145 because assigning it to a variable is needless since you'll use the variable just once.


shouldn't be ETCH_R_ONE        %r28?


Does this commit belong on the Implement Tables branch?


needsUpdate is *NOT* thread-safe.

You would need to either add a lock around m.apiStatusVersions, or add a new channel to tell the sync thread to rset the apiStatusVersion


I think this diff would make more sense if it was the superTask version that had the "+" to show how the new changes the old. It was confusing to be reversed. Also, trim all the path info from the top line and start at "python/lsst/..."


не совсем разобралась с применением `<label>` и вот.


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


`data` isn't being used in this test and can be removed


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


@larsgrefer, thanks again for the clarification via email. Please update this PR to keep that line for now, then I'll merge.


seems you only used 3 spaces for indentation here.


Блок можно передавать дальше (в следующий метод ожидающий блок) с помощью оператора `&`. Use it!
http://www.skorks.com/2013/04/ruby-ampersand-parameter-demystified/


Looks good. Thanks for fixing it.


All the api.Pod were changed to pointers in a bulk update PR (#7004). You are right about the potential data race.


There you go @CairoLee c28c9f6


I see, my bad. I'll revert this

On Wed, Dec 9, 2015, 19:36 srob650 <notifications@github.com> wrote:

> It does exist, but only when doing "people_search()" returning a list of
> people and their "relevancy-score", like with show_search().
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/liiight/pytvmaze/commit/4aaa6d07f6189a570f97a3437811becb38beb2e9#commitcomment-14879943>
> .
>


Hey Guysoft, thank you for the pointer. I'm probably going to reference the link it as I get more complicated.


So for this, I was sort of thinking something more along the lines of:

```python
string_classifier_ctor = classifier_services.StringClassifier.__init__
using_string_classifier = false
def string_classifier_tracker(cls):
    using_string_classifier = true
    return string_classifier_ctor(cls)

with self.swap(classifier_services.StringClassifier, '__init__', string_classifier_tracker):
    response = reader.classify(self.EXP_ID, self.EXP_STATE, answer, {'answer': answer})

answer_group_index = response['answer_group_index']
rule_spec_index = response['rule_spec_index']
answer_groups = self.EXP_STATE.interaction.answer_groups
if answer_group_index == len(answer_groups):
    return 'default'

answer_group = answer_groups[answer_group_index]
if answer_group.get_fuzzy_rule_index() == rule_spec_index:
    return 'classifier' if using_string_classifier else 'soft'
return 'hard'
```

Sorry that wasn't clear. I'm also not 100% sure the code above will work for swapping the StringClassifier constructor, but that's the basic idea.


Эээ, товарищ! Это ж ты навсегда меняешь instance variable класса, после одного такого вызова в @movies так и останется отфильтрованный список.

И, к слову сказать, это одна из (немногих будем честны) реальных ситуаций, где будет полезен метод `reduce`.


Let's see what I can do here as an alternative.


This is ugly, and I worry about the compile times caused by this disjunction. I suppose it's needed here so we can drop `Iterator` in favor of `WeakIterator`, but I found `WeakIterator`/`Iterator` to be a cleaner solution. Let me think about this some.


Maybe we can skip this step if `Complperc <= 40` ?


Not sure if this is an old comment, but this is a really good test. This was really complicated to mock and you got it.

The only thing I'd add is something like this to the end:

```
mock_job_collection_insert.assert_called_once_with(
    project_id='measurement-lab',
    body={'configuration': {'query': {'query': 'dummy_query_string'}}})
```

This lets us know that it's passing the right query to BigQuery.


mysterious?


The redirect URI is where you receive tokens ya? Not the sign on URL.


"The domain of `==`"


The note & footnote are intended to point out that this requirement is effectively purely semantic and need not be validated - apparently they need clarification.


No worries!


Xin, please include a description of the change that you are making in the body of the commit message.  Something like "rename variable X"


109806134-AbsChlrBlankWaterFlow (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (909 of 909 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-25.74%25-green.svg)


this function is too long, this is one possibility of shortening it. I personally wouldn't expect the function to handle splitting for me, a function signature is a contract, just require it splitted.


this will break on django 1.8


Review that attributes must be 'private'.


I want to note this approach also guards against code skew, since ``_get_classifying_rule_type`` is currently duplicating functionality from reader.classify. The suggested approach above does have one drawback in that it slightly exposes functionality for the classifier and how it's used (ie, it makes the assumption reader.classify will call the constructor to StringClassifier when using a classifier for evaluation).

The above sample is also not 100% complete, since there's no guarantee in this case whether a rule is actually being classified to the string classifier just because its ctor is called. You actually may instead want to swap out StringClassifier.predict_label_for_doc with a proxy, call the real one, check the return value for whether the prediction is non-default (you shouldn't need the actual document index, since reader.classify will return a soft rule index anyway), and then return the result in your proxy. That should certainly work (versus the strange constructor swapping above).

We can discuss this later, as well!


sorry, I don't get this...


@stanhu sorry, I am confused. If you generate a new password, we should clear up the token, no? Why wouldn't we?


Is this info not available from dev.office.com/app-registration?


When you check in these examples, you should make these files executable (`chmod a+x examples/drawing-ml`)


**Code Quality - Critical**

Why are you assigning 0 to `view_count`? What do you think about just making `view_count` have a default value of 0 in the database? ..So, you'll have to change that by creating a new migration file to effect that change.

Why are you also assigning 1 to `channel_id`? Does it mean that every episode uploaded will belong to just 1 channel?


I assume it's more like this now:

    data = CppcheckData.parsedump(...)
    for cfg in data.configurations:
        print('cfg:' + cfg.name)
        for scope in data.scopes:
            print('scope type:' + scope.type + ' name:' + scope.className)


First of all, do use collection initializers (``var a = new List<string>(); a.Add("abc"); …`` → ``var a = new List<string> { "abc", … };``)

You are providing static list. To do so, lazy concept (``Lazy<…>``) is ideal.

Consider creation of a new entity and storing this value in DB. It could be easily administered by an admin – this would make it very easy for non-programmer to completely amend glyphicons used in the app.


you could put this into a well named function


This breaks the rest controller, mime types can have a forward slash in them, which is now no longer escaped. :-(


**Code Quality - Critical**

Please remove all unused functions in the file


is `redirect_uri` required in this request? I forget.


self.name.. does not seem to have the same indentation as the next line


Когда я тестировал разные методы, я использовал метод print_short, который постоянно переписывал, в зависимости от того, что я хочу увидеть в первую очередь. А когда я написал этот метод, я решил для удобства использовать в своих целях. Убрать?


Thanks, I'll do a copy here. :)


Had some bugs with getting the package to compile on a Mac. Not sure if I had just not tested that it built from the last time or if this is one of those times differences expose themselves between OS. I exposed the current and proposed CSP parameters in the RFP model since they could be reconstructed and graphed from "getParmeterForCategory." I'm not sure that this is necessary, but I went ahead and exposed the vectors themselves.

I now believe that all the RFP parameter/trace variables that are exposed to R have been written correctly. Now I am working on reading in and initializing the Parameter and Trace object from these files. I will need to create a new parameter object in R to do so (thinking just call parameter's blank C++ constructor in this case - which I have exposed now). I am starting by testing the current alpha and lambda prime variables. It seems like Rcpp has a List type that will allow me to circumvent using SEXP objects between C++ and R. I have verified that I can at least access the first layer of the list by checking it's size. More testing will be needed, but production should speed up if this approach works.


please look at the indentation here.


The administrator might want to set an initial, random password to avoid keeping the password as a blank value and then send the reset the token so the user can set a proper password.


move everything below out:

```python
if begin_match:
    end_match = find_end_of_comment(...)
    if end_match:
        pass  # save the comment
```


Please delete waste and be careful.


Я думал, что это и есть цель задачи )


**Code Quality - Critical**

The indentation here is flawed. Please fix!

Just use `$3`  on line 190 and also use `$fileName` also on line 188, the variable assigned on line 185 is needless.


I did, but only because we were testing the `--pull` flag to build, which expected the pull to be cached.

I think in most cases this is safe.


It doesn't need to be checked because there is no subsumption relationship between `WeakEqualityComparable` and `EqualityComparable` as concepts exist today, is that right? Then it may not matter in practical terms, but it's still a bit of a wart.


It's ok.


Nitpick: Please don't cram enum declarations into a single line like this.


Why not push the `EqualityComparable` requirement into `ForwardIterator`?


@osantana have you tested these changes in your environment ?

I have been integrating these on my projects for the last days to make sure they don't break anything.

If you can confirm it works for you on Django <1.9 and 1.9 that will help a lot :)

If everything goes well I plan to create a new release tomorrow or the day after.


fhffh


Array


lgtm


Obs: Esqueci isso dentro da Main, muito pouca coisa para mandar outro commit....
                v = new View();
		vc = new ViewController(v);
		vc.control();/*  START INTERFACE */

só pra chamar a janela....


**Code Quality - Critical**

You have just killed us. Exposing our access key and secret to the whole world!  :open_mouth: . These values should come from environment variables.


辛苦了，能把调用方法记录在 README.md 里么


Why is ENABLE_LOGCAT better than __ANDROID__? It seems like this disables logging on non-ARM versions of android. (But maybe that didn't work anyway)


Did you try doing this as a non-admin in the tenant?  Make sure that works.  The app needs to be actually assigned the permissions to do Calendar.Read, Mail.Read, etc.  And only admins should be able to do this, I think.  Unless the officedev portal has some sort of powerful permissions and is doing so itself.  Just try to do this flow as a non-admin and make sure that works.


Looks great Joe.  If/when you want to publish instructions for doing this in a multi-tenant fashion (i.e. a service/daemon that works across AAD tenants), I'd like to review as well.  That's where things get a bit tricky.


For sure.
I also want to test the output produced some more.
In any case, work in progress :smile:


hmm, so I have to change it here too,


Code Quality - Critical

You have just killed us. Exposing our access key, secret, bucket and endpoint to the whole world!  :open_mouth: . These values should come from environment variables.


you'll usually want to remove the common indentation, i.e. the whitespace before the `@` right?


Right. It's ugly, but not syntactically significant. The compiler doesn't have quite enough knowledge to see the subsumption, I think it could if `Same` was a builtin.

The cure for "ugly" may be to simply remove the syntax requirement and add a prose requirement to the effect that `==` must mean "equals" when `S` and `I` are the same type.


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


This call is not very generic, and not much simpler than just invoking dns_packet_set_flags() directly. Maybe it would be nicer to just call dns_packet_set_flags() directly from the mDNS code instead?


you know that python can do that? `inspect.cleandoc(docstring)` I already pointed you to that one IIRC


lgtm otherwise


@bpostlethwaite how would write it?


**Code Quality - Nitpick**

The url here can be shortened a smaller syntax by just using the `route()` function and passing in the route name for this url.


Yeah...You are right! That's really a fancy design, I just noticed it. Thank you very much!
@yujuhong Why not just increase the version in the internal status cache? :)


HT @idgserpro


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


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


Many typo, should be 'abstract' not 'abstruct'


if you really want this, put it in a decorator like `arguments_to_lists`


The function is just there to decide whether the currently used screen resolution is too high to perform certain heavy actions on it, therefore just shell out to whatever returns you your correct screen resolution and perform some comparison to decide the outcome.


Перенесла `<label>` для name, email, country и `<textarea>` в первую колонку (где писала их изначально). Теперь при клике на Name фокус переходит на соответствующее поле. Оно?

a83148d15bbb441937882316b6118996ada565ba


Just wanted to add, I re-read our original discussion about this fix and I said there that we *should* catch unexpected exceptions. Seeing it in code, I'm realizing that was wrong and we should just let them go.


This should be `5.0.0-alpha.14`.


dddd


you can keep the password as nil. devise is not going to allow users to log in if the password is nil.


Ok, good to know. Thanks.


Нееее. Цель сказать «мой любимый список всех фильмов! дай мне список комедий 2000-го года!» а не «мой любимый список всех фильмов! превратись навсегда только в список комедий 2000-го года!» :)


@mdtusz https://jsperf.com/number-vs-plus-vs-toint-vs-tofloat/14


This will explode, as you free the items while iterating, hence when determining the next entry you already destroyed the current one which you need to figure out the next one...


辛苦了，不过现在的弹出方式好像不是滑动，而是 width:0 -> width:200px
可能有几个方面需要改动：

*  移动方式改为左右滑动，后面会有在左侧的侧栏

* 稍微离开一点侧栏，现在是完全贴在旁边的

* 圆角稍微大点，箭头的角度也稍微加大点，让圆角和箭头能稍微感觉连在一起


Whoops.  :)  I'll fix later today.
On Dec 9, 2015 1:12 AM, "Lars Brinkhoff" <notifications@github.com> wrote:

> Did you mean #define OBJDIR struct ObjDir?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sam-falvo/kestrel/commit/ae5f286a125f0f07fe01502c66a932a3c768edc8#commitcomment-14868033>
> .
>


Noted. Thanks.


Para #3


This is largely the same thing. The address is only 4 byte long (size of a register), and the function is inline, so no copying should happen. We also generally want 8 byte or less objects to be copied by value and not by reference where possible.

Unless you can prove otherwise? Did you look at the assembly?


I'd say a monster should be to drawn to the light before you can see it. Or does this make it too hard? I haven't played in a while.


FYI, this is now possible in [atom-terminus](https://github.com/drelyn86/atom-terminus), using wild cards in the commandPrefix setting.


Ah, I'm using LLVM locally.  That likely explains the… substantially greater warning verbosity.  ^_^;


So the ArrayList hazyConstManager should probably be the constants declared in code, not whats in the file. We can populate it by doing hazyConstManager.add(this) at the end of the constructor here. Then we can look through the file and update the values (if they're different) with a setVal(double newVal) method here. The way its set up now we have to search through an Arraylist every time we want to use the value of a constant. That'll get kinda bad when there are 150+ things in that list. Instead we want to be able to do SomeFileWeDeclareConstantsIn.exampleConstant.getVal().


Hmm, I wonder whether we really want this to be a double linked list. I think it would be simpler to just make this a singly-linked "pseudo-list". Specifically, replace this with a single "DnsPacket *more" field, and then make sure that this field takes a ref on the more field. Hence you chain them up, properly reference counted, but there's no way from a message to the previous one, just one to the next one.

I think this would be much simpler, and all we need no? This also makes the freeing logic much simpler, as you don't need to iterate anything, but can just call dns_packet_unref(p->more), and this does the right thing automatically.


For me the "~/Desktop" is not working. So not file / output is being written. Is this related to php setting? Did anyone test the above?


lgtm


yeah I did guess that from the beginning that it was something like that. I just been wondering how I will get it working here on linux also, I'm no MPV expert only reason why I started to configure mpv was because the [bomi](http://bomi-player.github.io/) dev told that he is taking a sabbatical from that project


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


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


Ups. Uses seeds 0... instead of the specified ones!


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


rearrange


I suggest:

    data = CppcheckData.parsedump(...)
    for cfg in data.configurations:
        print('cfg: ' + cfg.name)

Those 3 lines of code will more or less fix http://trac.cppcheck.net/ticket/3555


You use 3 different types of string formatting in this file:

```python
        print("Add Temperature to database: %.2fC" % tempC)
        print("Querying data: " + query)
        print("Result: {0}".format(result))
```

I would recommend using one consistently.  I'm also going to recommend using ```.format``` as it is the powerful and current.

For the three statements above:

```python
        print("Add Temperature to database: {:.2f}C".format(tempC))
        print("Querying data: {}".format(query))
        print("Result: {0}".format(result))
```

The "Querying data" may not make sense immediately but format is MUCH faster than string concatenation and we type protection from it as well, so less crashing.  :)

Also check out this site:
https://pyformat.info/
it is neat


Maybe that can be an iteration on this once it's published. I tried replacing the tenant ID with *common*, but that doesn't seem to work for app-only apps.


I didn't include in the little sample I built and it works properly, so I think the client ID and secret are enough.


@tarakreft good documentation!


No, nothing has changed for what macros are imported. This just makes lookup more efficient.


Actually, I'm still confused how #3846 fixes the issue that #3774 was trying to address. For example, if I do something like this:

```ruby
user = User.new
user.email = 'test@test.com'
token = user.generate_reset_token
user.save

# Send reset token instructions to user
send_reset_email(token)
```

Since the `save` clears the reset token, the user will receive a stale token that no longer works.


Ah, it probably is. I typically go through the portal still so that's the instructions I went with. Not sure if the token issuing endpoint is available or not, but I will double check.


It creates a big swarming mess of monsters. It amps up the difficulty immensely.

I agree though.
Eventually monsters will have their own perception variables as well as speed. That way you can try to out run monsters that you see. Or run from monsters that are chasing.

It would take some balancing too.

> On Dec 9, 2015, at 11:50 AM, Byrån Zaugg <notifications@github.com> wrote:
>
> I'd say a monster should be to drawn to the light before you can see it. Or does this make it too hard? I haven't played in a while.
>
> —
> Reply to this email directly or view it on GitHub.
>


You are right. I see what you mean. Wouldn't the fix then just be a matter of changing before_save to before_update?


done


Yes... but for whatever reason the portal (and the registration tool) expose this mysterious sign on URL value that serves as the first redirect URL. Either will work (because they're the same AFAIK).


I'm not Ben, but I might do it like this.
``` javascript
function createComputeError(symmetric, arrayminus){
    var val2 = parseFloat(array[index]),
        val1 = (symmetric || !arrayminus) ? val2 : parseFloat(arrayminus[index]);

    return function computeError(dataPt, index){
        return [val1, val2]
    }
};
```

You'd probably want to change the naming (I don't know the full context here...) but that would work, and it's easily composable in other places if needed.


VALUE_TYPES could/should be derived from the VALUE_TYPE_LENGTH map


I have not tried that. Can non-admins sign in and change application permissions? I'll check that out.


@josevalim Yes, that would work. :)


are there actually files with fat/vfat/ntfs extensions?


This looks inconsistent with your other files. Consider making this a one liner

```
import { shouldShowProgress } from 'lib/site-settings/exporter/selectors';
```


(FYI, I scratched something wrong I said in my previous comment)

Regarding the subnets, maybe some confusion (actually let's call them address pools): When user creates a network, he can specify multiple pools for it.
When we'll enable user to select a preferred IP when running the container, we will check if the user specified IP belongs to one of the pools configured for the network.

At network construction time, we expect user to let libnetwork know all the address pools from which IPs will be picked for its containers. This because we want to reserve that IP, because we are assuming that an IP (alias or not) given to this container should not be given to any other container on this network. Uniquely assigned.

But the virtual IP is a special alias IP and breaks that assumption, it will for sure be configured on multiple containers on the same network.

To solve this I was proposing that either we relax the `ip already allocated` check on alias IP (not on the primary) or we have a new `endpoint.iface.virtualAddress` field on which only we skip the above check.

Need your help on this question: Would introducing a virtual IP field cover all cases, in other words, besides the VIP use case, is there another case where ip are not uniquely assigned to containers in the same network ?

Thanks


Derp. This is why I don't like writing things in English! Will fix in a sec.


![image](https://cloud.githubusercontent.com/assets/6675409/11695444/32f6ee9e-9e7d-11e5-8d73-1c91302dc232.png)

no `parseFloat` please.


... clauses regardless of the table being read-only ...


--configurations.end() looks strange


This class name should be `export__spinner-button` to match the file name and the react class display name


Прошу прощения,не могу добавить issue, поэтому пишу здесь

№1
Шаги по воспроизведению проблемы:
1. Запустить программу
2. Ввести ник, подтвердить
3. Подключиться к боту на files.litvinov.in.ua
4. Отправить сообщение

Ожидаемый результат:
Обратное сообщение в формате "Back: < сообщение>"

Фактический результат:
Сообщение не получено

№2
 В списке контактов напротив показано, что юзер онлайн,что не соответствует действительности
( Проверялось на юзере asd, который пользовался чатом более недели назад)

№3
Кнопка Clear, которая, как я понимаю, должна удалять строку веденную в поле для отправки ( прошу прощения, если это не так), выводит в поле IP, к которому подключена программа


Sorry for asking but could you please send a new PR with your previous test? Thank you for keeping me in check.


If we keep our own buffer, I'd be inclined to raise this, perhaps to 4k, 8k, or larger, which might substantially decrease the per-byte overhead.


Fair point. I suspect that whoever wrote that code originally (could have been me) wasn't aware of the existence of the `std::vector` assignment or equality operators.


I'd prefer

    if (_settings.dump_all && fdump.is_open())


I'd make this one line so that that trailing `>` doesn't get lost while someones reading it.

`<Button disabled={ loading } { ...this.props } >`


I don't think so, but they were to complement filemnt, I'll remove them

I also added other enhancements to filemnt

  case ${Ext} in           #exceptions
    2fs) Type='ext2'     ;;
    3fs) Type='ext3'     ;;
    4fs) Type='ext4'     ;;
    sfs) Type='squashfs' ;;
    iso) Type='iso'      ;;
    *)
		info=$(file "$imgFile")
		echo -en "\n${info}\n"
		case $info in
			*DOS*)  Type='msdos' ;;
			*FAT*)  Type='vfat'  ;;
			*NTFS*) Type='ntfs'  ;;
			*9660*) Type='iso'   ;;
			*UDF*)  Type='iso'   ;;
			*ext2*) Type='ext2'  ;;
			*ext3*) Type='ext3'  ;;
			*ext4*) Type='ext4'  ;;
			*Squashfs*) Type='squashfs' ;;
			*) echo "Unrecognized format" ; exit 1;;
		esac
		;;
  esac


I think you forgot to add these libraries to `make.inc`.

`LIBRARIES += AP_NavEKF`
`LIBRARIES += AP_NavEKF2`
`LIBRARIES += AP_RangeFinder`
`LIBRARIES += AP_SerialManager`


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


this dumpfile variable can be moved into the if


You might want to consider handling the case where someone provides the `disabled` prop as that might make this component not work as intended.


ошибки лучше выводить в stderr
Например: System.err.println("Несовместимые режимы Stream и Limited.");


When would the task instance id ever be missing?


this is starting to smell like we need a class hierarchy/cluster and/or some protocols: a bunch of properties which are only valid based on the factory method you used -> base class + subclasses with their own properties.


Huh! I really messed up on this one. Why does Travis did not punched my face?

Anyway, thanks for fixing it!.


Maintenant je sais qu'il faut éviter les \n dans les massages de commit x)


I'm not sure what was the reason for using `quote()`.
I replaced all SET, EMITS, RETURNS, SCALAR symbols with strings and now I have much cleaner code.


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


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


looks good to me!


yes, will use pgdb.escape_string() to escape all the command shell and sql query string.


lol, yeah I was pulling parts from influxdb example scripts. I had been compiling for a few hours and was excited to get it working, so I didn't bother to review the code for consistency. It's fixed now.

Does this help you at all or did you already have an idea of the code?


Also, This seems like it could be a useful component elsewhere in calypso. Consider moving it to the [components](https://github.com/Automattic/wp-calypso/tree/master/client/components) directory and giving it a README file.


By using `where = "true"` I covered the branch which includes the where clause -> better test coverage


This is awesome, thank you.


Shouldn't these be `<ion-slide-page>`? `<ion-slide` has just been deprecated.


Pushed a fix in develop

Best,
Anand Doshi

ERPNext

> On 09-Dec-2015, at 11:25 PM, Maxwell <notifications@github.com> wrote:
>
> Hi, this change give me a error
>
> Traceback (innermost last):
>   File "/vagrant/frappe-bench/apps/frappe/frappe/app.py", line 67, in application
>     response = frappe.handler.handle()
>   File "/vagrant/frappe-bench/apps/frappe/frappe/handler.py", line 77, in handle
>     execute_cmd(cmd)
>   File "/vagrant/frappe-bench/apps/frappe/frappe/handler.py", line 94, in execute_cmd
>     ret = frappe.call(method, **frappe.form_dict)
>   File "/vagrant/frappe-bench/apps/frappe/frappe/__init__.py", line 788, in call
>     return fn(*args, **newargs)
>   File "/vagrant/frappe-bench/apps/frappe/frappe/handler.py", line 29, in runserverobj
>     frappe.desk.form.run_method.runserverobj(method, docs=docs, dt=dt, dn=dn, arg=arg, args=args)
>   File "/vagrant/frappe-bench/apps/frappe/frappe/desk/form/run_method.py", line 41, in runserverobj
>     r = doc.run_method(method, **args)
>   File "/vagrant/frappe-bench/apps/frappe/frappe/model/document.py", line 551, in run_method
>     return Document.hook(fn)(self, *args, **kwargs)
>   File "/vagrant/frappe-bench/apps/frappe/frappe/model/document.py", line 706, in composer
>     return composed(self, method, *args, **kwargs)
>   File "/vagrant/frappe-bench/apps/frappe/frappe/model/document.py", line 689, in runner
>     add_to_return_value(self, fn(self, *args, **kwargs))
>   File "/vagrant/frappe-bench/apps/frappe/frappe/model/document.py", line 545, in
>     fn = lambda self, *args, **kwargs: getattr(self, method)(*args, **kwargs)
>   File "/vagrant/frappe-bench/apps/bios_mx/bios_mx/bios_mx/doctype/sat_settings/sat_settings.py", line 31, in test
>     self.sign(rfc, xslt, cfdi, **options)
>   File "/vagrant/frappe-bench/apps/bios_mx/bios_mx/bios_mx/doctype/sat_settings/sat_settings.py", line 34, in sign
>     cfdi = self.__xml_signature(cfdi, xslt)
>   File "/vagrant/frappe-bench/apps/bios_mx/bios_mx/bios_mx/doctype/sat_settings/sat_settings.py", line 78, in __xml_signature
>     self.certificate_key).startswith('/') else self.certificate_key)[1]
>   File "/vagrant/frappe-bench/apps/frappe/frappe/utils/file_manager.py", line 294, in get_file
>     file_path = get_file_path(fname)
>   File "/vagrant/frappe-bench/apps/frappe/frappe/utils/file_manager.py", line 309, in get_file_path
>     file_path = file_url
>  UnboundLocalError: local variable 'file_url' referenced before assignmen```
> —
> Reply to this email directly or view it on GitHub <https://github.com/frappe/frappe/commit/8a5addaae759837126530a34e9e45074b7396543#commitcomment-14880423>.
>


Consider adding a line break above


I'm assuming that this would link back into the reviewing part of the tool, possibly at the federal loans section. Any reason not to add an ID to the `offer-part` and set that up now?


Такие большие и значимые функции, как сам стрим(или просто запрос), наверное, лучше выносить в отдельные классы.


This version is quite old and older than the version i have introduced before. Yes you cant enable disable it. You have to manually add that function to do that if you have the skill for that.


This actually be handled in an global ExceptionFilter – it is nothing hard a makes the code a bit shorter and allows you to handle all exceptions at once. Following this tutorial: http://www.codeproject.com/Articles/731913/Exception-Handling-in-MVC


Yeah, that's what I thought might happen.

I like the idea of perception. Maybe a like a low random chance of noticing the light from a distance.


Do you need a build of this just yet?? Or do you want to wait for a release?


s/which meet/that meets/


@electrum any thoughts on the error code and message?


Мне кажется, что лучше не выводить e.printStackTrace();, а пробросить исключение, поймать его в main и вывести e.getMessage();


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


The only method of resolving conflicts that's currently used is `-resolveConflictsFavoringSourceIdentifier`, here's the implementation:
https://github.com/etoile/CoreObject/blob/7d3cacf1141ae1f37e4675e4e5d6cd55d05427e0/Diff/COItemGraphDiff.m#L1083

The COItemGraphDiff code is kind of convoluted because an edit can be in multiple conflicts. I think the idea behind that was that an edit can violate multiple constraints, e.g. it's inserting an object in a range the other user deleted, but it's also an "embedded item insertion conflict" (both users inserted the same object at two different places, for a collection that doesn't allow duplicates).

So it depends what you want to do - are you thinking of a conflict resolution UI? I think the first step is write tests for the behaviour you need, the current code is not well tested (iirc, it's only tested as a black box when resolveConflictsFavoringSourceIdentifier is used. the implementation should be tested in detail). The COItemGraphDiff implementation may not make sense, I just randomly wrote some stuff that seemed to make sense at the time ;)


good catch, fixing


Translate


I can't remember now. I'm sure there was a reason :smile:


I have no idea why is this the right thing to do. A comment with an explanation might help.


check `isNullOrEmpty`


The dream never dies... Not even after the due date


If the user will be seeing these states then lets translate them


It helps immensely.  I have to get back to work but I am very excited to get this all tied up soon!


По-моему, если найдено недостаточно твитов, то можно еще раз запустить twitter.search(query); и он в tweets найдет другие твиты. И так делать до тех пор, пока требуемое кол-во твитов не будет набрано.


The argument name is `rowIndex`, but it is not a row index of the receiver object, a row index of the underlying `Table`.
This naming is easily misunderstood when reading the code.

I think we should use clear argument name (`rowIndexOfUnderlyingTable` or something...).


Why these nodes won't be added by simply visiting children?


I agree that exposing the progress through the task object is useful. We are considering wrapping the `NSURLSessionTask`'s properties in an `NSProgress` object. (We also need to update the progress feedback blocks.) Do you have any preference for one approach over the other? Thanks.


This looks strange.  Is there no `BYE`?


Splitting the function out as @mdtusz suggested makes the statement less daunting. My instinct is to use vanilla if and else conditionals and ternarys sparingly. Readability is vastly more important IMHO than keystrokes and there is no perf cost to if and else vs `?`


I would move this message to the previous commit.


formatting


This message is different than the others


Oh, for Recurrent Complex structures...not sure how to do that. It's one of the problems with spawn structures.


closes #4


Yes. I've added #3847.


this change should be in the previous commit


I figure we should drop the check for the old magic entirely.


If (something)
     return function () {}
else return function () {}


fixes #7


fue CRUD no cruz


Copy the message from the next commit to here


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1112.9456f0f1960f471b1c94f385334f262c6e14abd4](http://build.palaso.org/viewLog.html?buildId=67103&buildTypeId=bt431) outcome was **UNKNOWN**
Summary: Canceled (Tests passed: 503, ignored: 4; exit code 1 (new)) Build time: 00:15:38


Why is the cast to BoundObjectInitializerExpression safe?


I don't see `menus` being modified below, any reason why this changed from `const` to `let`?


- Use `$(SKETCH)-v2.px4` instead of `ArduCopter-v2.px4`.
- Does solo create it's own network? If so, is `10.1.1.10` guaranteed to always be its IP address?
 - If that's the case, it would be nice having that explained in the commit message :-)
 - If that's not the case, we can't use the hard coded IP address like that and we should think of another approach.


Just to be safe, check `source` for null


These 3 methods/properties assignments repeat multiple times. Rather than copy-paste them around, create a private method for them in the controller (or consider introducing a new public method in the service to populate these properties of the transaction, …)


We only log error codes that we receive, not those we generate. Hence, please do not pass EACCESS to log_error_errno() like this. Instead write this as:

```C
{
        log_error("Own control group unreach...");
        return -EINVAL;
}
```
or so.


lgtm.


Fixed manually in 2.8.3 and it works ;) Thank you!


What does CastArray do?


Hmm, has this stuff been merged upstream? I wasn't aware...

Is presume this is specific to the unified hierarchy? Or is it not?

Why make this an option? Sounds OK to just do this always in the unified hierarchy.


Consider moving this to line [27](https://github.com/Automattic/wp-calypso/commit/b73c88d04bafd7a701e0e47a416d9b69980ff0be#diff-302329fdf6f5ca6bb595e990472e6765R27) and using the "Module Variables" comments like so:

```js
/**
 * Module Variables
 */
let lastSiteId = null;
```


Wrong link text here (".NET Core Framework").


I think the original wording here is correct, no?


D'oh thanks


<3 this solution!


:+1: Thanks James!


Awesome, thanks!


@FernetMenta sorry for accusing you - this one broke testsuite building


I found it. I could never reproduce the issue with just my own markup+css. I had to dig through the computed styles for all of the parent elements to track it down. One of my 3rd party dependencies was setting position:relative on the body and clearing that did it.


Nice


109806134-AbsChlrBlankWaterFlow (JasonGlazer) - Win64-Windows-7-VisualStudio-12: OK (1983 of 1983 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)


@jpfr: No, the compilers failed, that's how I found it. I always compile examples :)

I think we should review travis's instructions...


Consider renaming `option.key` to `option.value` since the primary thing its used for is a value.
Just a suggestion


I was about to submit the same PR. I'm glad I checked the un-merged PRs.


What's so unreadable about a ternary?


I'd have preferred to pass around an interface that defines the print method rather than passing the channel and redefining the method everywhere. The interface itself would need to be in a separate package to prevent circular references.


test


TeamCity LabKey :: Sprint :: Premium :: Git postgres 9.4 [Build 9](https://teamcity.labkey.org/viewLog.html?buildId=290564&buildTypeId=LabKey_Sprint_2_LabkeyPremiumTrunk_GitPostgres94) outcome was **FAILURE**
Summary: Tests failed: 18 (6 new), passed: 221 Build time: 00:33:02
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSVisualizationTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.ScrumtimeTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSHelpCenterTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.util.JUnitHeader: <no details avaliable>

org.labkey.test.util.JUnitFooter.afterJUnit: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testDatasetExport: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testSelection: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testSelectingEmptyMeasure: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testStickyFinderFilterOnDataset: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testStickyFinderFilterOnStudyNavigator: <no details avaliable>

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


When i release because there will be a couple more changes this week.


Remove markup on IteratorCategory here.


to answer your question more clearly, what the implementation of -resolveConflictsFavoringSourceIdentifier does is
 - call `-removeConflict`
 - then, re-add the edits that it wants to keep from that conflict with `-addEdit`.

I think your idea to change _updateConflictsForRemovingEdit was good but your addition should be changed to:
```
if ([conflict allEdits].count == 1)
{
    id edit = [conflict allEdits].anyObject;
    [self removeConflict: conflict];
    [self addEdit: edit];
}
```
However, you have to be careful because this would modify the conflicts while iterating through them. I would let the original loop in `-_updateConflictsForRemovingEdit` finish, then collect a list of all conflicts with zero or one edits, then iterate over that list using the above code fragment to delete them.


АрхитеРКтурой


We've lost the original "For input iterators" text. We need to show where we're removing parts of the existing standard.


FYI I'm not sure I'm going to stick with this. It has side effects and security implications.


Can do, but it sort of requires the call site to keep 'track' on the other flags. I was thinking of hiding that by a specific call. But yeah, I had it the other way around, and can just go back to that version.


ПеРСмотр


Why we are using ExpressionSymbol instead of MemberSymbol?


Воу. Гит - для коллаборации и версионирования исходников. Терминал то это так важно. Тогда даже gnome-terminal.


Ditto here. "Value type T", "CopyAssignable", etc. are all from the original standard and need to be shown as removed.


Awesome! However, you might also want to check that it's not nil, before calling `.blank?`, since it may not be in the form POST'ed to the server


The path finding right now is very in intelligent.
If distance to player is < value move towards player.

I need to use an A* pathfinding algorithm. To make it more reasonable.

The thing is the player won't notice the wandering to the light. To the player it just looks like more monsters are coming into view.

> On Dec 9, 2015, at 12:10 PM, Byrån Zaugg <notifications@github.com> wrote:
>
> Yeah, that's what I thought might happen.
>
> I like the idea of perception. Maybe a like a low random chance of noticing the light from a distance.
>
> —
> Reply to this email directly or view it on GitHub.
>


The `if` is much more common and parses easily to natural grammar. It is more expressive. Many devs will go many months without writing a ternary. If you have a choice I feel its good to err on conforming to easy to read code as a chunk of code is written once and read many times.


not a constant


Resolves #288


@gmacdougall: @philbirt and I worked on a commit here to pull a bit more of the logic back into order cancellations. the reason being that a reimbursement has never before had return items on it that were not already accepted, and we wanted to maintain that existing behavior. once that was done, there wasn't really any logic left in the previous method we were adding to reimbursement.


Shouldn't the Kafka properties here be for the `consumer` rather than producer ?


Erm, yes. Staged the wrong hunk in fact. This needs a _SAFE version of course.


логирование


Hmm, the TC flag always has the same meaning for all three protos, right? Maybe leave the call as is, but make it generic and replace it by:

```C
h->flags = htobe16(be16toh(h->flags) & ~TC_BIT | (truncated ? TC_BIT : 0));
```

if you follow what I mean. Then it would work for all protocols, and be generic, and substantially better than dns_packet_set_flags() as it only alters one specific bit...


yes! you are right.
Fix on e2ce404fbc5260f21ebe388bcb2e07c45d042b4f


Alright, well thanks for considering it.


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


could you possibly specify the upper bound here for correctness? I think it's 2.4.9? I only tried with 2.10.4, and that failed, because of a missing `.h` file, which was also not found in any of the versions between 2.4.9 and 2.10.4.

also, would it make sense to open an issue for adding support for ==2.10.4 or even anything >=2.5.*?


Should I raise here?


Since those `IArgument` nodes are not associate with any bound node therefore the tree walker has no idea they exist.


соответсТвующим


соотетсТвии


Also I may have fixed the issue with jenkins. You username is all lowercase and in the permission manager it had an upper-case 'S'. So hopefully it should allow you to build things.


см выше


In Kona I was asked not to use concept names as the names of unconstrained template parameters like this.


проБлем
отсутСтвия


запоЛнение


In error scenarios the setter can be null, it looks like the IPropertyInitializer won't have any useful information about the property.


Ternarys are great for simple
a ? b : c
but you asked if I thought function declarations nested in a ternary was gnarly and that is my reasoning ;)


the problematic line was this one from `cbits/event.h` IIRC:

```c
#include <webkit/webkitversion.h>
```


The zero start position is fine for 16bit and 32bit values but not for 8bit (int8 and uint8) nor "bit" valuetype. For those the old behaviour was to interpret "readRegister" as index of the byte, or index of the bit.

For example, with valuetype=int16 index=0 corresponds to first byte in first register, index=1 corresponds to second byte in first register, index=3 corresponds to first byte in second register etc.

The item syntax in this branch is not verbose enough for this currently. One option would allow defining the register as "REGISTER.POSITION" where POSITION is byte index (for int8 and uint8 valuetypes) or bit index (for bit valuetype). Similar addressing is [used by Siemens](http://www.plcdev.com/siemens_s7_indirect_addressing) in their PLCs.


Good morning sweet prince.


Well, yes. In fact, I never used the `LIST_*` macros to access the fields, as they don't make sense in this case. I still left this one in because the fact that we have a _prev pointer actually allows us to assert whether an operation is happening on anything else than the top-most packet.

Regarding the ref counting, I'm not sure whether we should allow handing out pointers to a child in any case. Hence, just treating the children as reference-less is actually simpler. But I can redo this your way, let's see :)


I changed the ednote to indicate removing paras 1-3 and table 106, which now that I think about it is the entire section. I'll change the ednote to "Replace the entire content of the section with:"


Точка стоит, с большой?


чёт тоже с точкой там


Oh, I wasn't aware that plantUML can also handle graphviz natively.


ЛОЛ


#1 Говорят, бот не работает. Раньше нормально подключалось + мы так и  не выложили адекватную версию.#2 Возможно, проблема в том, что при закрытии программы юзер не уходит в оффлайн. #3 Наш баг, исправим, спасибо.


не знаю, щюпщюп


This was to fix OPSK-602, not 601!


ДАТАН. Ну чёт, это стандарт такой?


> @osantana have you tested these changes in your environment ?

Not yet. We see only one exception on graph model command and see this PR that fixes the issue.

We'll run our tests against master branch of django-extensions and put the results here ASAP.


We should override the `List#contains(Object)` instead of overloading `contains(RealmObject)`.


```java
private static final MethodHandle killQueryMethodHandle = methodHandle(getClass(), "killQuery", Slice.class);
private final MethodHandle boundKillQueryMethodHandle = methodHandle.bindTo(this);
```


Holy cow. I prefer fewer overloads at the expense of more involved descriptions. Fewer overloads is a win for users of the library, even though it makes the specification a bit uglier.


You need to have subdirectories, otherwise you'll get clashes.


No need. Pushed to vendor_aicp already.


Can you put the `POST /images....` in back-tics?


I'll change them to `T`s.


Update Phpcs to PHP Code Sniffer, not sure why but that is how it's listed under packages in the preferences menu.


Well, but if you don't refcount, then what is "owning" a ref to the packet? It appears as if the ref counting stuff is without effect if a DnsPacket is part of such a linked chain? That sounds weird, as there would be two parallel ownership schemes, and if somebody takes a ref on one of the linked packets (which might be an OK thing to do, maybe just for debugging purposes, where you want to keep a ref to all or specific msgs you send), this would result in memory corruption sooner or later... I think it's a good idea to stick to as few ways of managing object ownership as we possible can for the same object, hence doing ref counting, and only ref counting for DnsPackets sounds like a good idea to me...


This is wrong!! :D
```
def create
     @board = Board.new(board_params)
     if @board.save < save checks validations too and return true or false
         redirect_to boards_path
     else
         render :new
     end
end
```
Please read this:
http://guides.rubyonrails.org/v3.2.13/getting_started.html#getting-up-and-running-quickly-with-scaffolding


How does `indent` interact with `builder.append` above


Hint: the error's here: `$(CC) $(FLAGS) -c $(SRC)/%.c -o $(BIN)/%.o`
Hint hint: [use `$@` and `$^`](https://www.gnu.org/savannah-checkouts/gnu/make/manual/html_node/Automatic-Variables.html)


Should wrap this in a try/catch, otherwise on a failure the test will timeout rather than reporting the actual failure.

```
setImmediate(function() {
    try {
        expect(...);
        done();
    catch(e) {
        done(e);
    }
```


No. LWG is adamantly opposed to the concise template introducer syntax.


Same as above comment


Why you added `required` ?


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


@plribeiro3000 Can you check this commit? I pushed to my repo (which I thought was a fork), but it redirected here automatically (since I moved my repo to the fog org). Testing locally this fixed the issue.


Probably worth adding codemirror modes here as well?


Sorry for being annoying, but it's probably a good idea to rename the module folder, too
(from msdk-features/msdk-row-filtering/ to msdk-features/msdk-features-row-filtering/)


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


You can omit "Note that". Perhaps;

"The pull is cancelled if the HTTP connection is closed before the pull completes."

(or even "The pull is cancelled if the HTTP connection is closed.")


Looks good! Please correct a few namings.


or to turn this around: if two parallel ownership schemes shall coexist (i.e. ref count + part of a packet chain), then this should be clarified with asserts, for example _ref() and _unref() should hit an assert if the caller tries to ref/unref a packet that is part of such a packet chain, if you follow what i mean...

(but anyway, still think going ref counting all the way is the nicer approach)


I'm going to attempt a background sub. Looking at this snippet, I can't tell if it's doing what I expect. Is it comparing the current depth info to a depth image of the "empty" room? (in order to subtract furniture, floor, wall, etc?)


Same here


перенес цену и кнопки перед спойлерами


Typo.


Can't this be done with something like:
```java
node.getArguments().stream()
        .map(argument -> process(arguments, indent))
        .collect(Collectors.joining(", "));
```


When you finish this part ( create board ) please merge all changes to master.
We should using our flow with Pull request. It would be better.


изменения выше не нужны


This was done by Zachary Yee


Спасибо  за ответ. По поводу бота, тестирую другой чат,  отвечает нормально


Wouldn't `git clean -fd` work? It removes all files that are untracked and ignored.

(Don't run this command if you have work that hasn't been committed)


Well done, thanks @liila  !


Exit Select is not required at the end of a Case block.


109806134-AbsChlrBlankWaterFlow (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1447 of 1447 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-70.52%25-green.svg)


:fire: long live satoshi nakamoto :fire:


Or maybe it will add unneeded extra step when adding new category, dunno.


Use DirectCast instead of CType.


This json schema contradicts with what I described in the document 😑


109806134-AbsChlrBlankWaterFlow (JasonGlazer) - i386-Windows-7-VisualStudio-12: OK (1983 of 1983 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)


As far as I know, no.. after `continue` the `while` clause is checked. Also, it's covered by unittests so it should be good.


looks good


Thanks for reading the commit log, fellow bloodsucker.


Is baseline just a line on screen so we know that the speed is higher or lower than `stats_base`?


Should these fields be readonly?


Should these fields be ReadOnly?


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


Looks good, thanks.


Would be nice if C had namespaces.


Yes, you can try it with the latest build:
https://github.com/M66B/NetGuard/releases/tag/0.55


Old habits die hard! thank you!


Merged


@jnv Where is this options object used? It looks like dead code.


The alternatives here are:
```c++
template <class I1, class I2>
  EqualityComparable<I1, I2>()
```
and
```c++
template <class I1, EqualityComparable<I1> I2>
```
I think there's a very good case to be made that template introduction syntax is clearer in the particular case where one is choosing between `template <class, class, class> Concept<...>` and `Concept<...>`. "class class class" doesn't provide any additional information. It's *not* the same case as `Mergeable{I1, I2, O, C, P1, P2}` which I agree is unreadably terse.

I've been considering submitting a design guidance document for how to use TI and abbreviated function template syntax properly in the library specification. I've tried almost every possible permutation of syntax at one time or another in cmcstl2, it's given me some useful experience.


Had 0.55-201512090**4** but I did not see anything, will retry with the latest.


    $(function() {
      ...
    });


I think you can do
    .keydown(function(e) {


Ah, sorry, yes, I even commented the missing `}` and got confused nonetheless. >_<


I'm specifically annotating here only content that has been struck from N4560; the ednote replaces this section entire from C++14.


I think it's still broken on my case. Results:

```
$ pip freeze | grep -i django-extensions
-e git+https://github.com/django-extensions/django-extensions.git@8ef6f5e659b36a92b8dbb6ab3a1f3b735f84272a#egg=django_extensions-master
(vas-partners)$ ./manage.py graph_models license partner
/home/osantana/.virtualenvs/vas-partners/lib/python3.4/site-packages/django/core/management/base.py:265: RemovedInDjango110Warning: OptionParser usage for Django management commands is deprecated, use ArgumentParser instead
  RemovedInDjango110Warning)
/home/osantana/.virtualenvs/vas-partners/lib/python3.4/site-packages/grappelli/urls.py:20: RemovedInDjango110Warning: django.conf.urls.patterns() is deprecated and will be removed in Django 1.10. Update your urlpatterns to be a list of django.conf.urls.url() instances instead.
  url(r'^switch/user/(?P<object_id>\d+)/$', switch_user, name="grp_switch_user"),
Raven is not configured (logging is disabled). Please see the documentation for more information.

Traceback (most recent call last):
  File "/home/osantana/.virtualenvs/vas-partners/lib/python3.4/site-packages/django/apps/registry.py", line 148, in get_app_config
    return self.app_configs[app_label]
KeyError: <module 'apps.license.models' from '/home/osantana/Work/vas-partners/vas-partners/vas-partners/apps/license/models.py'>

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "./manage.py", line 10, in <module>
    execute_from_command_line(sys.argv)
  File "/home/osantana/.virtualenvs/vas-partners/lib/python3.4/site-packages/django/core/management/__init__.py", line 350, in execute_from_command_line
    utility.execute()
  File "/home/osantana/.virtualenvs/vas-partners/lib/python3.4/site-packages/django/core/management/__init__.py", line 342, in execute
    self.fetch_command(subcommand).run_from_argv(self.argv)
  File "/home/osantana/.virtualenvs/vas-partners/lib/python3.4/site-packages/django/core/management/base.py", line 348, in run_from_argv
    self.execute(*args, **cmd_options)
  File "/home/osantana/.virtualenvs/vas-partners/lib/python3.4/site-packages/raven/contrib/django/management/__init__.py", line 41, in new_execute
    return original_func(self, *args, **kwargs)
  File "/home/osantana/.virtualenvs/vas-partners/lib/python3.4/site-packages/django/core/management/base.py", line 399, in execute
    output = self.handle(*args, **options)
  File "/home/osantana/.virtualenvs/vas-partners/src/django-extensions/django_extensions/management/utils.py", line 57, in inner
    ret = func(self, *args, **kwargs)
  File "/home/osantana/.virtualenvs/vas-partners/src/django-extensions/django_extensions/management/commands/graph_models.py", line 77, in handle
    dotdata = generate_dot(args, cli_options=cli_options, **options)
  File "/home/osantana/.virtualenvs/vas-partners/src/django-extensions/django_extensions/management/modelviz.py", line 105, in generate_dot
    appmodels = get_models_compat(app)
  File "/home/osantana/.virtualenvs/vas-partners/src/django-extensions/django_extensions/compat.py", line 140, in get_models_compat
    return apps.get_app_config(app_label).get_models()
  File "/home/osantana/.virtualenvs/vas-partners/lib/python3.4/site-packages/django/apps/registry.py", line 155, in get_app_config
    raise LookupError(message)
LookupError: No installed app with label '<module 'apps.license.models' from '/home/osantana/Work/vas-partners/vas-partners/vas-partners/apps/license/models.py'>'.
```

My `INSTALLED_APPS` at `settings.py`:

```python
INSTALLED_APPS = (
    'grappelli',
    'django.contrib.admin',
    'django.contrib.admindocs',
    'django.contrib.auth',
    'django.contrib.contenttypes',
    'django.contrib.sessions',
    'django.contrib.messages',
    'django.contrib.staticfiles',

    'django_filters',
    'raven.contrib.django.raven_compat',
    'rest_framework',
    'rest_framework.authtoken',
    'kombu.transport.django',
    'django_extensions',

    'commons',
    'apps.partner',  # <-- maybe the cause of the traceback is the fact that we
    'apps.license',  #     use our django apps inside of an apps package.
)
```


I'm not quite sure I understand what this plugin does. Just curious.


The first one is because I'm using an IEEP which doesn't get synced to the client. That shouldn't be too hard to fix. The second is probably from that, too. For the third, I'll just be changing how backpacks are found. The last just requires a few changed lines.


There are however nodes representing argument expressions. Are they going to be added during the normal walk? Does it mean we will have some kind of duplication in the result?


Теперь можно делать SVG. Альзо, нынешнюю error иконку нужно перерисовать, она абсолютно не смотрится наложенной на оригинальные фавиконы. Это я пока промежуточно запилил.
Можно нарисовать, например, в виде красного крестика х по тому же типу, что и новые плюсы, внизу справа.


I think so.


man, I just made on the basis of what Behnaz did


`src` нужно ставить после `onload`, иначе последний может не сработать.


528e81f3941fb40c75921537c0444f56a87c1314 merges your patch. I think I've got an idea for why `event.cpp` has problems--it passes callbacks to the CL API, and I imagine those need to be `stdcall` as well. (There's a `clSetEventCallback` or some such.) I'll look into that in a bit.


I think importFolder is not needed for this picking...we need this "trick" in Xmipp auto-picking because it is base on a previous step.


To simplify the implementation and make it easier on the users, i would propose to have only have IPAddress and IPAliases. To enable the virtual ip use case, I would prefer to have a relaxed check for allocation on the IPAliases group.

I don't think that libnetwork itself should care about allocation or uniqueness. This is very much driver(network and IPAM)  specific. This is in many respects similar to the gateway/default route issue I raised earlier.
This would allow the implementation to be more future proof, as I would not claim to know all the use cases, specifically when looking at IPv6 where making assumptions will bite us.


ok, i'll fix it


Can you post a screenshot? Also, can you send me the following?
1. Version of KeepassX you are using?
2. Result of
`find /usr/lib -iname "*libexpat*" -type -f`
3. Architecture of your system
`uname -a`


But then just the getter is needed I think. The setter will enforce all implementations of the interface to be mutable. And to expose mutating behaviour. We prefer to encapsulate as much as possible.


It generates this file: https://github.com/fkling/astexplorer/blob/gh-pages/manifest.json . Otherwise the `moduleid => filename` mapping is stored in one of the entry files (`vendor` in this case), which means that file would change every time any other file/chunk changes.

Somewhere it says this plugin should be used ;)


+treinamento


Yeah, probably not worth it...


Is this supposed to be used by Snippet? Currently duplicated in both places.


@chrisndodge The closing strong tag should come after "exam" but before the period.


just a moment, i will push the updated version, this one has some JSON issues. Then you can fix it...


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


This change broke the portability of this library which was not dependent on Liferay during runtime!


Hmm, I'd say add a rule into the css for this


what is p?


All the above newly added code go inside if block of file is not empty.


Если это переместить после img.onload, то не будет работать 0_о. Поэтому перед. Справедливо для всех последующих итераций.


Closes #1 and #2


Nice - I complained loudly about the lack of this for ages and then forgot to build it...

Couple of nits inline.

Also can you try hooking the submit action on the form instead? If that doesn't work then we'll keep this as-is but using form gives us more flexibility if the UI changes in future.

    $('#layoutJobSearchForm').submit(function(e) {
      ...
      e.preventDefault();
      e.stopImmediatePropagation();
    }

You'll want to add `stopImmediatePropagation()` as well so that any other handlers installed by Beacon are bypassed. Make sure those calls are inside your `if` so that it doesn't prevent the default/beacon stuff if the format doesn't match.


is Ka actually 0.8? or 0.5/hr ?


LOL you have to look at it on the console... NOT HERE


@zenithtekla GitHub isn't quite the right place to solve this kind of issues. If you found a bug, please submit it to the [bug tracker](http://dev.ckeditor.com). You may also try to post your problem on [StackOverflow](http://stackoverflow.com/questions/tagged/ckeditor) and ask the community to help you.


я уже написал причину.


This one is wrong.


Ah interesting, never heard about it. Thanks :)


Well that's good to hear it's not an angular-material issue. I ran into a similar issue with d3 breaking svg icons a while back.


Fixed. f3113f2


I lied when I said these commits are mutually exclusive, this is actually fixed in next commit. Sorry :)


I mentioned in the original pull that I had added a *ton* of `advance` overloads to capture every distinct combination of syntax and preconditions. I agree that twelve-ish overloads is ridiculous; I'd planned on refining them but was distracted by the explosion over the sentinel semantics. I'll do so now.


Add new Movement of dragon once it detects crack


How do you disable mpdecision as default and let user to choose to turn it on?


Scrollbar won't work on Chrome. It says on console perfect-scrollbar is not initialized.

![Console log](https://i.gyazo.com/c7c59c08f2f7555a34d0265c0baae0a7.png)

Haven't tested on other browsers


OK I will split it into two commits .


@bosborne thank you for getting this file...I forgot to ask you for it.


*shakes head*

`SELECT a.id, a.project_id, a.name, a.is_admin, a.is_view, b.name FROM groups a LEFT JOIN projects b ON a.project_id = b.id`

If you really need the project name... do it in SQL if you can.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1110.d803e3325b8b20c04fcad9c56fb5dff6f6ca5ddd](http://build.palaso.org/viewLog.html?buildId=67043&buildTypeId=bt431) outcome was **UNKNOWN**
Summary: Canceled (Tests passed: 501, ignored: 4; exit code 1) Build time: 00:09:32


Looks good to me!


How are you trying to load DubX? Chrome Extension, user.js, bookmark? It seems like `perfectScrollbar` plugin didn't load in time from dubtrack


Link to http://guide.meteor.com/collections.html#local-collections (I'll update the id later today)


Yeah it is :) It's Github aware so I'm really liking it

I can just do:

git merge --ff-only <url_to_pr>

James Mills / prologic

E: prologic@shortcircuit.net.au
W: prologic.shortcircuit.net.au

On Tue, Dec 8, 2015 at 2:52 PM,  Florian Best <notifications@github.com>
wrote:

> https://github.com/spaceone/dotfiles/blob/master/gitconfig
> oh, the github CLI thing seems nice.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/circuits/circuits/commit/ab51c96d601898bb912e53930ae286935818bd29#commitcomment-14860959>
> .
>


There is a small difference between these two: PTC in signature doesn't include Pointer and ByReference. These are constructed types in signatures. However, IL instructions like ldelem.i, stelem.i, ldind.i, etc. treat the pointer as a primitive value.

At some point we might want to provide helpers like this: http://source.roslyn.io/#Microsoft.CodeAnalysis/CodeGen/ILBuilderConversions.cs in the IL writer.

So do we want 2 different PTC definitions?

The only usage in signature decoder is

```C#
return _provider.GetPrimitiveType((PrimitiveTypeCode)typeCode);
```

What about eliminating the enum and instead adding ```GetXxx()``` method on the interface for each primitive type?


I think it's a good example.

Re the `onResultReceived` I would at least call out that it exists in the "method call lifecycle" step 4


I don't see any conditional trace statements in this code. It is essential to have proper error/information tracing in order to properly debug any component. Also make sure there is some small test harness for each of these components.


Looks like it should work


Taking it one step further, do we need a separate IPrimitiveTypeProvider (rather than merging it into ITypeProvider)?


Canolie should be Canola


Appreciate your work! ^__^


The diff doesn't show it super clearly - but all the red comes from me abstracting the checking of the state to its own function `compareMT(...)`.

Also added is speedTest, which shows a significant boost when compared to TreeMap:
```
java.util Put: 2,696.66 K/s RemYes: 2,871.00 K/s Yes: 3,317.50 K/s No: 2,938.45K/s AddTo: 3,999.55 K/s IterFor: 39,411.76 K/s

fastutil Put: 2,687.11 K/s RemYes: 3,198.49 K/s Yes: 4,584.76 K/s No: 3,821.69K/s AddTo: 6,285.56 K/s IterFor: 41,176.47 K/s
```


Spanish to English

Spanish to English is a number vocabulary exercise. It was written in Verilog using Quartus and implemented on a DE2-115 Altera FPGA.
How to Play:

The numbers 0-10 are represented by switches 0-10, 11-15 are represented by 10 and 1 for 11, 10 and 2 for 12, etc. Numbers 20-90 are represented by the number and 0 switch, switch 2 and 0 for 20, 30 and 0 for 30 etc.

To begin flip switch 17 on and off to reset the device, then flip switch 16 to start the excercise. When each Spanish number is displayed on the LCD screen flip the 0-10 switches accordingly and flip on the 16 switch to submit your answer to be checked. If your answer is corect, correct will be displayed on the LCD screen and a green LED light will turn on, if your answer is wrong, wrong will be displayed and a red LED light will turn on. Flip switch 16 off to go to the next question if your got the answer correct or back to the same question if you got it wrong. You can't procede to the next question until you get the current question correct. Once you go through all of the questions you are returned to the home screen and can start  agian. You can also restart by toggling the reset switch, switch 17.


That is horrible :) Why not use an MSBuild conditional construct?


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1111.d803e3325b8b20c04fcad9c56fb5dff6f6ca5ddd](http://build.palaso.org/viewLog.html?buildId=67045&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 743, ignored: 6 Build time: 00:05:42


@knocte It's only temporary. The fsharp plugin won't be using these .orig files for much longer :)


thanks

gclub   sbobet


Chrome Extension.


The last_insert_id is associated with the connection so

```php
$i1 = $db1->insert_id();
$i2 = $db2->insert_id();
```


you added a line here


@sellout: The best thing I can say for this is it seems to work, at least with the `Id` monad. :stuck_out_tongue_closed_eyes:

It could probably be more compact, and I'm sure there's a cleverer way to require `G: Traverse`. Actually, it would seem to make sense to require that only when using this fold, but I couldn't get that to work. Maybe with a partially-applied type?

Actually, it's not clear to me why we don't assume the bindings are collected in a Map and just make the key type the parameter.

Anyway, I'm going to run with this but just felt like sharing.


aa


You are right, Julian. RexLiteral itself does not support ARRAY, instead it will be an array constructor. So no point to make this change. I'll go ahead and commit.


Hey man, looking good - let me know if I can help get things rolling here!


Can you go to this link: https://travis-ci.org/javelinjs/mxnet-scala to activate Travis for this repo?


saw approval from kuenishi
at https://github.com/basho/riak_cs/commit/76f53c6a4f58c2a6d8f1a69c0cbc216f067c1e85


merging basho/riak_cs/ku/merge-2.0 = 76f53c6a into borshop-integration-1275-ku/merge-2.0


basho/riak_cs/ku/merge-2.0 = 76f53c6a merged ok, testing candidate = 1a1a56f5


I don't know about this, Steve...


closes #2


pull: DENIED!


w3c/spec-generator#21 is about that (at the leve of Spec Generator)


:smoking: :beer:


fixes #: 1


Hey, thanks for checking out the Mozilla Study Group project! How are things going? Let me know if you have any questions or if I can help get you set up!


你后来又试过了嘛？真机上效果还理想吗？我这边试了还行啊


aadfdfddg


So this dis-allows #4 serverside, but now theres an issue client side


You're right that this is currently a no-op, so I don't object to removing it (though it might not be bad leaving it, in case it's needed later).

Also -- the convention is that pre-commit happens before the database transaction and post-commit happens after the database transaction. You could have both in a single API call (e.g. pre-commit, DB transaction, post-commit).


This one seems a bit forgotten, like you've moved on, but I'm suddenly interested in it. I've noticed it's a different version from the one Omni uses in the OPO, and also I can't find any enabled/disabled switch. Just asking in case you asked yourself the same question and already found a solution. Thanks!


您好，是不是现在下拉刷新，或者上拉加载更多的时候，都不能做其他操作了，要等刷新结束后？


ahh... yes thanks for reminding me  - totaly forgot about join - guess im not completely sql-fied


This needs to return `sm.Canonical` instead of `sm.all`.


Disregard this comment, it was meant for `Payload`.


This needs to return sm.Canonical instead of sm.all.


xD


iOS 真机测试上发现几个重要问题：

1. UI 不符合 iOS 规范，被强制改大了，不好看，这个我会慢慢修掉；
2. Navbar 返回的动画没了；
3. Statusbar 文字颜色需要处理，红色导航的时候需要白色，其他需要黑色，Sidebar 拉出来的时候需要盖住 statusbar，像 Inbox 一样，不然不同的 Navbar 风格导致 statusbar 的文字颜色不好处理；
4. 许多动画的地方不流畅；

—————

功能方面：

缺少很重要的：赞、编辑、删除、NoPoint、置顶、精华等功能，没这些，无法替代浏览器版本
Icon 还需新设计，@cassiuschen 可以尝试看看，我想了很久也没好的方案

在 2015年12月9日 星期三，10:17，Madz 写道：

> 你后来又试过了嘛？真机上效果还理想吗？我这


By far, the message "Empty work server assignment" from the Assignment Server is the most common question we get and it doesn't fit in any of the categories you've mentioned.  Moreover, it's a tough one to diagnose since most of the time, there isn't enough information to identify which work server or type of project is no longer available.  It's generally reported as a problem with the Assignment Server, but not a proplem _reaching_ that server.


@hiroki0325 スタイルシートの読み込み方の変更と修正点の変更しました。


1. 使用`boxmap[21][21]`存是否有箱子的记录；
1. 往左/右推的时候计算横方向最多可以推几格，往上/下推的时候计算竖方向可以推几格；
1. 注意墙每次推完格子之后都是回到原位的；
1. 四个方向函数大同小异，注意边界问题和连续有箱子的控制问题。


trailing whitespace


This change includes :
1. Importing DefinedSearch from compliance branch to current branch
2. Importing fixtures/initial_data.yaml from compliance branch to current branch
3. Adding predefined syslog search to yaml
4. Adding execute_query() to DefinedSearch
5. Adding search_by_syslog_error_type() to EventQueryDef
6. Removing uuid and other polyresource fields from EventQueryDef and AlertQueryDef


Do we need all these extensions ? I think the audio and video ones can go personally


On vibre maintenant? Booyah! Haha


all tests pass:
success: http://buildbot.bos1/builders/test-riak_cs/builds/1717


user:"xxxxx@gmail.com",


Okay I messed this up pretty badly.
>Oops


I've just taken what govcms out of the box allows content authors to upload.


Whoops, you're right.  I must've missed the first time around that it was post-indexed.  Please disregard.


ok


See earlier comment about purpose of post-/pre-commit operations.


more test cases === more good, right? :smile_cat:


Could do with a better commit message


Wow, that's super sneaky, how did you ever figure this out?


This is awful. Stop using `&&` like this.


Related to https://bitbucket.org/logilab/astroid/issues/276/testfailures-with-141 ?


Excellent


SHEBANG w/ SPACE **IS A LIE**. See AOSC-Dev/autobuild3#41 quotation.


I'm going to squash that commit later, just wanted to keep it separate
for now for review.

On 08/12/2015 6:29 PM, Daniel Cousens wrote:
>
> Could do with a better commit message
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ptschip/bitcoin/commit/24de0a2774de33dacfd683c32b4f3ba42b072bc5#commitcomment-14863895>.
>


@ligthyear is there maybe a way to prevent this plugin from running on this particular vagrant instance? it's funny, my work repo requires this plugin, so i have to uninstall it for beavy and then reinstall when i do work. it seems strange that vagrant runs all plugins on all instances, no?


!XXX/XXX/XXX/XXX/YYY/YYY/ZZZ/ZZZ/ZZZ = 36 characters
1+6*4+3*4=37


Hi Carolina !

Thanks for add yourself to the collaborators list, please put your name in alphanumerically sorted way and then make a pull request to verify that all is correct.

This is a very good exercise because we'll use this methodology for future commits. :smile:


Is there not an end delimeter: $ ?


You have many options to create Features. You can use the `harfbuzz.Feature` class as well. You also don’t need to append a ‘+’ sign.

Here is the full reference for specifying feature strings:
https://github.com/deepakjois/luaharfbuzz/wiki/Feature-Strings

Here is the documentation for API function `harfbuzz.shape` with some more details:
http://deepakjois.github.io/luaharfbuzz/api/#shape


Clening, dafuq LOL :laughing: :laughing:


And of course I meant #2 not 1.


does the split feature work with the experimental plugin? (I never tested it).


Oh hell go and use your `if`.
By the way your dear `|| true` will always make the `&&` at end-of-line useless.


Good catch!


Enough! Just use `*' 5.2'*)`.


`checkThreadSafety();` before calling content api


Oh, it seems that was solved by @aerendon ... XD


GO USE ARRAYS


looks good Tristan!!


I guessed it after observing that if I made a fake account with a large UID it didn't have a quota. It took a long time...

`quota` won't actually show you quotas for non-existent users, but `quotatool` will:
```
root@pandemic:~# quotatool -u :44444 -d /dev/mapper/vg-homes
44444 /dev/mapper/vg-homes 0 5242880 5767168 0 0 0 00
```

You need to specify a colon before the user ID to see the record (or it will complain the uid doesn't exist).


> I read some best practices guide about CMake

Quite often such articles actually contain "bad practices" :)
This SO answer is a good example:
* http://stackoverflow.com/a/1060061/2288008

Even if official documentation states that `file(GLOB` should not be used for collecting sources huge amount of peoples just "vote up" for doing it...

Anyway it's not an error, just a thought aloud.


You can. You have defined the format.
And NLs don't change the meaning of the things.
And it will be super interesting if a change in NL count gives a working collision.


Could you add a link to local::lib docs on metacpan here and line 98?


And line 52 I guess.


And you fscking removed the quotes.


@ChipLeo  Because it's for a non supported version.


what is this for?


Sure, see ca9fc0c348465bcaed20d078e010ecaf716f7645


The images still show the RadPushButton and have not been updated to RadImageButton. Please ping me when all of these figures are updated and I can copy edit.


Oh I also implemented the recipes.


@uberlaggydarwin umm, why exactly was this done ?


How about just `RUBY_VERSION.to_f >= 2.0`?


Done


Done.


`[ -d "$i" ] || continue`


`$group`. And can you be sure about newlines?


Missing sentinel value here.


Have I told you to use `grep -F` before?

This time you are using `'^'"$name"'$'` instead of `"^$name$"`. Looks worse.


```
return isOwnerActivityRunning() ? mFavicon : null
```


Once we already have the jquery included, we can use `angular.element` direct as `$` function


`bash`. You are a crazy baa-shit-ism writer.


The issue should be fixed now. Could you double check it?


`cd` can fail for a variety of reasons: misspelled paths, missing directories, missing permissions, broken symlinks and more.

If/when it does, the script will keep going and do all its operations in the wrong direction. This can be messy.


Why are you sorting `find` output?


I think you need to remove the file here.


This commit broke VerifyHeader's API in #5995. Now we will need a ```GetAncestor``` or ```GetSkip(CBlockIndex)``` additional function pointer.


这里存放平时前端学习时的各类笔记


work with 28f89fa1c287db14fdd5e92b5f8e8fc91edfaeb6


Il faudrait enlever les commentaires HTML histoire de voir les fichiers tels qu'ils seront présentés le 15.


tested by Lucas https://github.com/happybelly/zotero-factextractor/wiki/Test-Log-2


This doesn't make sense to me.. the whole point of calling encode/decode on ints (and not &foo, sizeof(foo)) is that it does the endian conversion.  If you switch encode to packed_encode then it doesn't do the endian conversion any more.

The case to change/optimize is not hte big endian path, but hte little endian one.  And IMO that should look like:

#ifdef LITTLE_ENDIAN
  p.copy((char*)this, sizeof(this));
#else
  ::decode(...);
  ...
#endif

Or,

struct inline_pg_stat_t {
   __le64 num_foo;
   ...
}

and point use an inline_pg_stat_t typed pointer directly into the buffer.  Then the endian conversion happens at runtime when you do x = p->num_foo or p->num_bar = y.


Is  the override onReceivedIcon function still leave?


this one makes sense, but add a comment in teh struct definition warning that the order/type can't get changed without breaking encoding


this one too


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


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


Oh :-/ that was accidentally committed. It's part of the parse in URL feature.


Oops, there is an extra ")" in this line.


#4853-Fully-autosized-DX-coil-should-not-fail-with-a-fatal-SHR-error (rraustad) - x86_64-MacOS-10.9-clang: OK (1966 of 1977 tests passed)

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


I think this should use `f.write` rather than `f.writelines`, no?


Instead of exporting the package versions here we should rely on Portal Java Contracts which would be `JavaJSONP` v1 and `JavaJAXRS` v2.

https://www.osgi.org/portable-java-contract-definitions/

Could you make that change instead and make sure the bundle exports all the packages of each contract?

Thx


Странно. На этом компе работает. А на работе нет.


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


fast-forwarding end-to-end/timeseries to borshop-integration-1302-at/lexer_error_messages = 9723e557


Not sure what feature you mean, but ok :)


Ctrl+Z


I'm not convinced this is the canonical way to get the geometry object but it is my best effort.


use git reset --hard or "discard local changes" in future


Keep up the work. Use the Clean Code text and the PEP-8 guidelines (linked on LearningSpace) to guide your code tidy.


Вот кусок работающего кода, в котором создается "внутренний" файл. Видимо, ошибка в работе с БД
 if (id == R.id.action_settings) {
            //Toast.makeText(MainActivity.this,getString(R.string.action_settings),Toast.LENGTH_LONG).show();
            File file = new File(this.getFilesDir(), "testFile");
            try {
                boolean b = file.createNewFile();
                Toast.makeText(this, "File Created = " + b, Toast.LENGTH_LONG).show();

                FileOutputStream fio = new FileOutputStream(file, true);
                fio.write("hello world".getBytes());
                fio.flush();
                fio.close();
            } catch (IOException e) {
                e.printStackTrace();
            }
        } else if (id == R.id.action_2) {
            try {
                BufferedWriter bw = new BufferedWriter(new OutputStreamWriter(
                        openFileOutput("testFile_2", MODE_PRIVATE)));
                JSONObject object = new JSONObject();
                object.put("textForTestFile_2", true);
//                if (c.moveToFirst()) {
//                    do {
//                        object.put("date", c.getColumnIndex("date"));
//                        object.put("summ", c.getColumnIndex("summ"));
//                        object.put("description", c.getColumnIndex("description"));
//                    }
//                    while (c.moveToNext());
//                }
                bw.write(object.toString());
                bw.close();
            } catch (FileNotFoundException e) {
                e.printStackTrace();
            } catch (IOException e) {
                e.printStackTrace();
            } catch (JSONException e) {
                e.printStackTrace();
            }
//                    File file = new File("file.json");
//                    file.createNewFile();
//                    FileOutputStream fOut = openFileOutput(file.getAbsolutePath(), MODE_WORLD_READABLE);
//

//                    OutputStreamWriter writer = new OutputStreamWriter(fOut);
//                    writer.write(object.toString());
//                    writer.flush();
//                    writer.close();

//                String FILENAME = "e:\\matfuck\\AndroidStudioProjects\\HomeBudget\\file.json";
//                FileWriter writer = new FileWriter(FILENAME);
//                writer.write(object.toString());
//                writer.flush();
//                writer.close();
            //break;
        }


No they won't. The default value of `ExpressionKind` property in `BoundExpression` is `OperationKind.None` (unless specific sub type override this) which would be ignored by walker (which is the case for argument expression).

On the other hand, I don't know if the argument expression you mentioned can be used as `IArgument`, maybe @johnhamby can provide more insight here.


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


I'm still using svn :dancers:


#55 `<-` this file is from working on this issue


Fixes #8


Formatting here is broken.


this is great!


eaere


Yes, that is it. It makes the BG subtraction, taking as background one of
the 3d images ( it is supposed to show a count down before taking saving
the BG, buy maybe the code I uploaded years ago just take the first frame)
El 9/12/2015 8:40 p. m., "Stephen Lindberg" <notifications@github.com>
escribió:

> I'm going to attempt a background sub. Looking at this snippet, I can't
> tell if it's doing what I expect. Is it comparing the current depth info to
> a depth image of the "empty" room? (in order to subtract furniture, floor,
> wall, etc?)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dasaki/ofxKinectBlobFinder/commit/8681525a0a453d904388e175a5d57722f906c084#commitcomment-14882982>
> .
>


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1113.9456f0f1960f471b1c94f385334f262c6e14abd4](http://build.palaso.org/viewLog.html?buildId=67112&buildTypeId=bt431) outcome was **FAILURE**
Summary: Execution timeout (new); tests passed: 504, ignored: 4 Build time: 00:37:21


Я точно помню, что у меня с этим пропадал ориинальный фавикон. Может, там была какая-то ошибка, которой больше нет.