Concerning the `Log.warnings()`
> Log.warning("subnet %s not found; skipping" % interface_settings.subnet_id)

Please use parameters (in mustache style), like so:
> Log.warning("subnet {{subnet_id}} not found; skipping", {"subnet_id": interface_settings.subnet_id})

`Log` is a structured logging system, which handles JSON documents, and can send them to document storage for later querying.  It is unfortunate that SpotManager only logs to text files, so you do not see the benefits.


Please add m before members names: mWidth;

This will permit to have parameters without the ending '_' as seen in  void Resize( size_t width_, size_t height_, bool is_rgb ); for instance.


Looks good. Of what I spotted, the only thing i don't like is the use of extract() that isn't allowed in the WP coding standards.
https://make.wordpress.org/core/handbook/coding-standards/php/#dont-extract


Thanks Madhura


Good!

To make this a proper unit test, you might want to use [`assertEquals(double, double, double)`](http://junit.sourceforge.net/javadoc/org/junit/Assert.html#assertEquals%28double,%20double,%20double%29), though.


@ldoktor I prefer to merge TreeNodeDebug stuff , back to the tree.py module. I see if I do it along the day.


Every todos here seem right to me.
Note that I'd rather add get* in getters name when possible.
i.e.: getWidth() const.

Simple functions like this one could also use inline.


That get messy.
- Could you extract handleChange()


Did you find a written solution for cpkc? I didn't find any vulnerabilities when I looked at it.

I was thinking that maybe in the challenge, using the LLL algorithm give the solution. But it may be difficult to find a vulnerability in the encryption without seeing the code they are using, because for me a non brute force attack would mean to find a way to break NTRU. They have to make a small mistake in the key generation or when they generate a random "r".


Need to test and some cleanups are still needed but it seems like a good step forward already. :)


кстати наверное нужно сделать +1 (для более привычного отсчёта), а то получается начинаем с нулевого дня


@phstc yes, added myself a task for this


The .jar file is built as needed, and isn't in the .git - regular build updates can be found at the [Magarena project page](https://buildhive.cloudbees.com/job/magarena/job/magarena/) on cloudbees


looks fine


statement is always true


@akva2 @stefansaraev we dont need libglfew.so (which is built) on our target? also we should set PKG_ARCH="i386 x86_64" if it dont works for arm, which seems to be the case, or we should do a "exit 1" if "! $OPENGL=mesa" ? (and remove "mesa" from PKG_DEPENDS_TARGET)


add_warnings (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-AddressSanitizer: OK (348 of 348 tests passed)

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


I agree, and it complicates things, but I don't see a way around it since `key` is now data.


develop (lefticus) - i386-Windows-8.1-VisualStudio-12: OK (347 of 348 tests passed)

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


Yeah, I saw that... I didn't realize that the dict argument was for structured logging and not just as a syntax for parameter replacement.  I wonder, perhaps you could additionally support fields in **kwargs, so that I could do this?

  Log.warning("subnet {{subnet_id}} not found; skipping", subnet_id=interface_settings.subnet_id)

Just a tad easier to type.


lesson/lesson 4 - advanced oop/www/inner/js/Helpers.js

ugye az volt a lényeg, h vagy string vagy egy tömbbel, amibe ojjektumok vannak, azzal térjen vissza. Amúgy a
```javascript obj.constructor.name``` itt mé nem müxik?


we certainly don't need the .so file. it should use the static lib..


Whoops. `getLastCorrection()` returns a `Vector3D`, of course, so you want three `assertEquals()` calls...

```java
final double DELTA = 1e-5;
final Vector3D correction = proj.getLastCorrection();
assertEquals(12, correction.getX(), DELTA);
assertEquals(12, correction.getY(), DELTA);
assertEquals(12, correction.getZ(), DELTA);
```

BTW I would probably use different offsets in x, y and z, just in case that the algorithm mixes up axes at some stage.


Please rewrite the commit, skipping this dead code.


Our test suite seems resistent to bravery. :wink:


Static properties of React component classes are mostly meant for exposing functionality to consumers of the component.


Its better to define things in header files instead of passing a bunch of defines on the command line.


Alright, a fix has been implemented, give that a shot!

There's no in-SDK way to determine portrait mode, but as it turns out it's
fairly simple just to check the bounds of the current GLX context.

On Wed, May 13, 2015 at 5:37 AM, Feilen Haksan <feilen1000@gmail.com> wrote:

> Hmm, I'll have to see if we can detect if the Rift has been rotated in the
> SDK. Thanks for the heads up.
> On May 13, 2015 2:05 AM, "Zachary Cook" <notifications@github.com> wrote:
>
>> Unfortunately the fix for portrait mode messes up landscape mode, maybe
>> only swap if portrait mode is detected?
>>
>> --
>> Reply to this email directly or view it on GitHub
>> <https://github.com/feilen/dolphin/commit/1a0c832dc639a640c19aacf59fb19174d3229177#commitcomment-11169923>
>> .
>>
>


I tried to make it a habit of mine to use the `final` attribute whenever it is possible. This helped me prevent many a programming mistake plenty of times. Maybe you want to do the same?


不错呀


롤백하고 다시 migrate해야 `schema.rb`에도 반영됨


You can't modify the schema (which represent the desired format of current documents) to support migration functions. I have implemented a more rigorous solution in a70ce65c52c3533a7fc828fb55a72e5c82b70949.


here too: we should do a exit 1 if $OPENGL is not mesa


Whoops! Sorry. I was clearly too tired to perform a merge last night! Thanks again for the patch.


add_warnings (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-ThreadSanitizer: OK (348 of 348 tests passed)

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


lesson/lesson 4 - advanced oop/www/inner/js/Models.js

Rusnya de müxik, alapvetőleg a felülírásra törekszem. Tudom, h nem ezt kellett volna, de valahogy a másik ```map```-os oldal (tudod mert az élet mapos oldala) most nem akart az agyamba tuszmákolódni.


test comment


;-) I should have seen it coming.
https://travis-ci.org/matlab2tikz/matlab2tikz/builds/62387636 though shows that by including also the mocking PR, the test passes as reliable (I did not revert the commit there).


1line comment


error: AC_CONFIG_MACRO_DIRS([m4]) conflicts with ACLOCAL_AMFLAGS=-I build/autotools.


Sets shift modifier


add_warnings (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (348 of 348 tests passed)

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


@mgruner but wouldn't it be enough to place a warning in the docs? Every admin/developer is responsible for his/her system. Or just prohibit pre-filters for the heavy templates... pre filters are very handy for restructuring templates without overriding framework templates. Maybe  a caching for filtered templates is also possible.


+1 for 0.1.0


Oh, thanks for the tip!  I'll add that in.

Actually, you're seeing all of the code.  `valid_until` is a param to boto's `request_spot_instances`, so if I understand this correctly, setting that attribute causes that argument to get passed.  The argument is required to be in ISO8601 format (YYYY-MM-DDTHH:MM:SSZ), which corresponds to `%FT%TZ`.

I would have liked to use times.ISO8601, but that gives me YYYY-MM-DD HH:MM:SS, which is lacking the "T" separator (and the timezone specifier).  According to the ISO8601 spec, the "T" can only be omitted by "mutual agreement" of the parties in the communication, and the EC2 API specifies that T is required for DateTime arguments.


:beers:


Just found this:
```
// @formatter:off
...
// @formatter:on
```
Seems to be supported by both [Eclipse](http://stackoverflow.com/questions/1820908/how-to-turn-off-the-eclipse-code-formatter-for-certain-sections-of-java-code) and [IntelliJ IDEA](http://stackoverflow.com/questions/3375307/how-to-disable-code-formatting-for-some-part-of-the-code-using-comments).

Maybe it's not ideal, but looks pretty close to me, since regardless of rule, there will always be exceptions.


You get something like **kwargs:  You can pass whole structures; the message template can be used to reach deep into the structure (using dots as a path separator) to print specific values:
> Log.warning("subnet {{subnet_id}} not found; skipping", interface_settings)

The structured log will have the whole `interface_settings` structure, but the printed message will just have the `subnet_id`


add_warnings (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.6: OK (348 of 348 tests passed)

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


I see your point. I'll remove it.


I have just checked and `System::getReferer()` is the only method which is used in both the back end the front end, so we might also leave the scope check there. Anyway, I'll be creating a PR for the changes.


this is a ui lib


Pooja I think this is not correct...... it sou..


test


add_warnings (lefticus) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.67: Build Failed

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


It's up to you whether you think it's worthwhile. Thanks either way.


kraj


The reason that they didn't add "virtual" there is because they added the function body directly inside the class declaration. That is the equivalent to adding the function body inside a .cpp file like normal (remember that in source files you don't add things like virtual to functions.


+1 as well, but have in mind that this looks like it was pushed by mistake. Check [this](https://github.com/inuitcss/trumps.spacing-responsive/issues/12#issuecomment-99523714) and [this](
https://github.com/inuitcss/trumps.spacing-responsive/issues/11) issue.


@ebrehault please add a changelog entry for this.


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


develop (lefticus) - Win64-Windows-8.1-VisualStudio-12: OK (347 of 348 tests passed)

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


This fix is wrong. `read()` can return any type of value when it's in objectMode.

https://nodejs.org/api/stream.html#stream_new_stream_readable_options


That doesn't necessarily mean it won't be GC'd. Python garbage collects all classes and modules during the interpreter shutdown sequence in order to ensure that finalizers are invoked properly for anything referenced from those objects. If Ruby does something similar this could cause problems on shutdown depending on the order in which things happen.


It's like this:

struct A {
    virtual int getArea() = 0;
};

struct B : public A {
    int getArea() {
        return 4;
    }
};

That's the same as:

// B.h
struct B : public A {
    virtual int getArea();
}

// B.cpp

int B::getArea() {
    return 4;
}


Holnap fogadóóra?

2015-05-11 11:11 GMT+03:00 Márk Csősz <mark.csosz@gmail.com>:

> var Models= {
>     Vehicle:function(numberOfWheels, fuelType) {
>         Vehicle.count = ++Vehicle.count || 1;
>         this.numberOfWheels = numberOfWheels;
>         this.fuelType = fuelType;
>     },
>     //cars
>
>     Car:function(brand, model, performance) {
>         this.brand = brand;
>         this.model = model;
>         this.engine = new Models.Engine(this.brand, performance);
>         this.toString = function() {
>             return this.brand + " " + this.model; //part 5, nyincs beégetve
>         };
>         if (arguments[3] != undefined) {
>             this.production = arguments[3];
>         }
>         if (arguments[4] != undefined) {
>             this.isNew = arguments[4];
>         }
>         if (arguments[5] != undefined) {
>             this.price = arguments[5];
>         }
>         if (arguments[6] != undefined) {
>             this.numberOfWheels = arguments[6];
>         }
>         if (arguments[7] != undefined) {
>             this.engine.fuelType = arguments[7];
>         }
>         if (arguments[8] != undefined) {
>             this.engine.displacement = arguments[8];
>         }
>         if (arguments[9] != undefined) {
>             this.engine.strokes = arguments[9];
>         }
>
>     },
>     //Car.prototype = new Vehicle;
>
>     //beállítjuk a car prototípusát egy új vehicle-re
>
>     //és itt meg beállítjuk egy új carra a konstruktort
>
>     //engines for cars
>
>      Engine: function(brand, performance) {
>         this.brand = brand;
>         this.performance = performance;
>         this.horsePower = function() {
>             return performance / 1000 * 0.75;
>         }
>         if (isFinite(performance) == false || performance < 1) {
>             throw new performanceException(performance);
>
>         }
>
>     },
>
>      Motorcycle: function(numberOfWheels, brand, fuelType, cm3,
> terminalVelocity) {
>         this.numberOfWheels = numberOfWheels;
>         this.brand = brand;
>         this.fuelType = fuelType;
>         this.cm3 = cm3;
>         this.terminalVelocity = terminalVelocity;
>     },
>
>     //other vehicle classes
>    // Motorcycle.prototype = new Vehicle;
>
>
>      Bike: function(numberOfWheels, brand, bSize, speedCount) {
>         this.numberOfWheels = numberOfWheels;
>         this.brand = brand;
>         this.fuelType = "nincs";
>         this.bSize = bSize;
>         this.speedCount = speedCount;
>     },
>    // Bike.prototype = new Vehicle;
>
>
>     //specific cars
>     //mitsubishi
>
>      MitsubishiLancer:function() {
>         this.engine = new Engine(this.brand, this.performance);
>         this.toString = function() {
>             return this.brand + " " + this.model; //part 5, nyincs beégetve
>         };
>
>     },
>     //MitsubishiLancer.prototype = new Car("", "", 100);
>
>     //Renault
>
>
>      RenaultFluence:function() {
>
>         this.engine = new Engine(this.brand, this.performance);
>         this.toString = function() {
>             return this.brand + " " + this.model; //part 5, nyincs beégetve
>         };
>     },
>    //RenaultFluence.prototype = new Car("", "", 100);
>
>
>     //wartburg
>
>      Wartburg353:function() {
>
>         this.engine = new Engine(this.brand, this.performance);
>         this.toString = function() {
>             return this.brand + " " + this.model; //part 5, nyincs beégetve
>         };
>     },
>     //Wartburg353.prototype = new Car("", "", 100);
>
>     //other objects
>     //part 5
>     //part 6, eccerűen megoldom, nem baszom bele a tömbbe, ignorálom. Ja
> meg kiírom, h aki bevitte az fasz
>     //part 7 na ezt bogozd ki
>     //part 8
> /*
>     function VehicleContainer() {
>
>
>     }
>     VehicleContainer.prototype = new Array;
>     VehicleContainer.prototype.push = function() {
>         for (var i = 0; i < arguments.length; i++) {
>             if (arguments[i] instanceof Vehicle) {
>                 this[this.length] = arguments[i];
>                 this.length++;
>             }
>             if (!(arguments[i] instanceof Vehicle)) {
>                 throw new notCarException(arguments[i]);
>             }
>
>         }
>
>     }
>
>
>     VehicleContainer.prototype.filterByType = function(getType) {
>         var match = new VehicleContainer();
>         for (var i = 0; i < this.length; i++) {
>
>             (getType == this[i].brand && match.push(this[i]));
>         }
>         console.log(match);
>     }
> //exceptions
>     function performanceException(perf) {
>         this.perf = perf;
>         this.message = " :ez nem felel meg a nem szám és nem negatív
> szabálynak";
>         this.toString = function() {
>             return this.perf + this.message;
>         };
>     }
>
>     function notCarException(obj) {
>         this.obj = obj;
>         this.message = " :ez nem Vehicle típusú ojjektum";
>         this.toString = function() {
>             return this.obj.constructor.name + this.message;
>         };
>     }
>
>
>     //prototypes
>     //car
>     Car.prototype.production = 1980;
>     Car.prototype.isNew = false;
>     Car.prototype.price = 200;
>     Car.prototype.numberOfWheels = 4;
>
>     //engine
>     Engine.prototype.fuelType = "petrol";
>     Engine.prototype.strokes = 4;
>     Engine.prototype.displacement = 900;
>
>
>     MitsubishiLancer.prototype.brand = "Mitsubishi Lancer";
>     MitsubishiLancer.prototype.model = "A70";
>     MitsubishiLancer.prototype.production = 1973;
>     MitsubishiLancer.prototype.isNew = false;
>     MitsubishiLancer.prototype.price = 200;
>     MitsubishiLancer.prototype.numberOfWheels = 4;
>     MitsubishiLancer.prototype.fuelType = "petrol";
>     MitsubishiLancer.prototype.displacement = 1.378;
>     MitsubishiLancer.prototype.strokes = 5;
>     MitsubishiLancer.prototype.performance = 64000;
>
>     RenaultFluence.prototype.brand = "Renault";
>     RenaultFluence.prototype.model = "Fluence";
>     RenaultFluence.prototype.production = 2009;
>     RenaultFluence.prototype.isNew = true;
>     RenaultFluence.prototype.price = 17500;
>     RenaultFluence.prototype.numberOfWheels = 4;
>     RenaultFluence.prototype.fuelType = "diesel";
>     RenaultFluence.prototype.displacement = 1.461;
>     RenaultFluence.prototype.strokes = 6;
>     RenaultFluence.prototype.performance = 110000;
>
>     Wartburg353.prototype.brand = "Wartburg";
>     Wartburg353.prototype.model = "353 ";
>     Wartburg353.prototype.production = 1966;
>     Wartburg353.prototype.isNew = false;
>     Wartburg353.prototype.price = 299;
>     Wartburg353.prototype.numberOfWheels = 4;
>     Wartburg353.prototype.fuelType = "petrol";
>     Wartburg353.prototype.displacement = 0.993;
>     Wartburg353.prototype.strokes = 4;
>     Wartburg353.prototype.performance = 60000;*/
> } //Model end
>
> 2015. május 8. 18:26 Márk Csősz írta, <mark.csosz@gmail.com>:
>
>> miko lesz kövi fogadóóra? Megint szarúl lettél?
>>
>>
>> 2015-05-07 14:30 GMT+03:00 fogasl <notifications@github.com>:
>>
>>> Ezt a mezőt érdemes lenne átnevezni, mert a size védett szó
>>> JavaScript-ben! (kerékátmérő, ha jól gondolom...)
>>>
>>> —
>>> Reply to this email directly or view it on GitHub
>>> <https://github.com/MarkCsosz/webapp-lessons/commit/82df0051d384fd07486dd98fbf142382b00ed899#commitcomment-11084743>
>>> .
>>>
>>
>>
>


I think this was mistakenly removed.


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

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


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-LibC++: OK (348 of 348 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)


Looks good, just one comment about style. Most of the code base just checks for truthiness like "if (component.propTypes && ..."

See the line with nativePropsToIgnore just a few lines below for example.


this is fine as-is, I may tweak PKG_ARCH and so later if it would make you happy @sraue ;)


Oh, well that's neat.


Sorry, I missed you usage.   I use the format defined in boto.utils.ISO8601
> from boto.utils import ISO8601


Try to do without ViewBag.  Is it necessary?  Can you just put the text in the page directly?


qualified flag doesn't seem to be needed here anymore, is it?


It's a little weird that "postId" comes from the "PostTitle"


Perfect, I'll use that, thanks!


Legbonyolultabb osztály


This fix is wrong. `read()` can return any type of value in `objectMode`.

https://nodejs.org/api/stream.html#stream_new_stream_readable_options


Weird regression: https://envisage.ifi.uio.no:8080/jenkins/job/ABS-Bucky-Tests/469/testReport/junit/abs.frontend.delta/DeltaAttributesMixedTest/deltaParserNPE1/


In my tests this morning, this breaks non-Windows provisioning for more than one site.  In order to get my sites working, I had to rollback the changes to roles/wordpress-install/tasks/main.yml


ezer hála és kartácstűz, erre sose jövök rá magamtól


Noted. I will correct it.


I don't understand why the port numbers are hard-coded and not used later.


테스트의 세계에 온 것을 환영한다

그렇지만 이런 애들은 테스트 하지 않아도 되는데, swindler의 비지니스 로직을 테스트한다기 보다는 레일즈 자체를 테스트하는 것에 가깝기 때문임


I agree with including electronic instead of manual pp


As above: I don't understand why the port numbers are hard-coded and not used later.


LOL


Bravo! Çok iyi düşünmüşsünüz!


And therefore it does nothing. This if statement will never execute. This was in anticipation for when there is, if, or are those columns because they are date types.


了解


@evanfuture thanks for catching this and saying something! can you please open a new issue?


done (and rebased)


Thanks! Look forward to the release!


I've set the default branch to be master for the repo, so an explicit checkout shouldn't be required moving forward.


:cake:


그렇네.

처음에 association하면서 헷갈려서 seed db만들고 이거저거 바꾸면서 해서 테스트 연습도 할겸 만들어봄~ 다음에  커밋할때 삭제하겠삼.


@bmneely DWMR. Press on.


부르는 상황에 따라서 `me`는 ambiguous할 수 있으므로 `stories_of` 따위를 추천


I am going to re install from scratch and update the readme accordingly. It should be pretty straight forward if you have any experience building apk's


Thanks a lot dude, for this fix! Its working now ! (y)


> Disclaimer: Please don't see any of my comments (which are sometimes wrong, btw) as offences.

Of course not. Discussing is always a good thing imho.

> As for guidelines, please, make sure to:

Err... I thought I had followed those, but I use completely different ones, so I'm fighting my habits. I'll fix that.


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


이걸로 충분할까? 예를 들면 글자 수 제한 따위가 있을 수도 있음


@nenadjelovac Uh. I was just adapting my code to the new style but with the current inconsistencies (`.u-mb0-lap` vs `.u-palm-pt0`) it makes little sense. :-/


On 13. mai 2015, at 15:06, Volker Stolz <notifications@github.com> wrote:

> Weird regression: https://envisage.ifi.uio.no:8080/jenkins/job/ABS-Bucky-Tests/469/testReport/junit/abs.frontend.delta/DeltaAttributesMixedTest/deltaParserNPE1/
>
I changed something recently in that area; investigating now.

Rudi


type-o: in **a** Julia data set


Taxel positions extracted from the CAD with respect to the right/left foot FT sensors


@bmneely `9223372036854775807` - is that random or a special value? my hunch is that you're having fun or proving some point to yourself.


The `tls`. Sounds good.

Can't wait to see all the confusion "the tls authors" are going to cause now, though. :smile:


Huh... my apologies, don't remember why I did that.


@bmneely DWMR. Press on!


자기 스스로한테 쓸 수도 없어야겠지? `Followship#must_not_follow_oneself` 참고


...and I fixed times.ISO8601.  Thank you for noticing that.


That would be good, yes.  When I wrote this, I was thinking either a specific AZ or all AZs, but now I know that I have to allow a list of AZs anyway because of [not all accounts can access all AZs](http://serverfault.com/questions/573862/amazon-vpc-availability).  I'll work on that this morning.


Замени на метод прогонки.


No problem :) It _is_ legit ISO8601 -- just not the common format.


ok pushed and published - this really needs (some/any) cross-platform tests thanks guys.
 npm v0.1.12


This is interesting! I thought you had to whitelist apt sources -- does any PPA work?


Gut geworden!


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


>  Hasn't std::search a undefined behaviour when not finding the desired element? (in c++98)

No, it returns the end iterator which was given, when not found.

> As we got: linear * linear + constant here vs linear * linear before.

According to cplusplus.com (sometimes I use this one, sometimes I use cppreference, both are quite good) std::search have a linear * linear complexity.


Unfortunately, this code doesn't unify.  I think you are missing a comma before the 'P' in Across1.  However, even with that change, it still doesn't work.  In particular, you need to be sure that you use a given variable only for a particular square in the puzzle.


add_warnings (lefticus) - i386-Windows-8.1-VisualStudio-14: OK (348 of 348 tests passed)

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


this is not needed, was from an earlier attempt


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


typo: 对于 应为 多余


No, but they've whitelisted `avsm`: https://github.com/travis-ci/travis-ci/issues/3705


This should say Melissa Dominguez


hehe, love it.  I'm going to follow your lead - @Dan, I'm going to charge for good intentions now :P


I'll stick with the current implementation for now.


@reneeb: I am not sure if any of your proposals is practical. How would you selectively disable? Caching based on content is also not possible because you never know how many permutations there will be, for example if someone adds the system time to the HTML output.


앗 논의가 필요하겠지만 전통과 신뢰의 140자가 어떨까?


after the colon:

including a 30 person software startup to IBM.


а зачем это расширение убирать? Как же Gemfile.lock?


Out of curiosity, have you considered having `pyLibrary/` in this repo be a git submodule?


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


remove `.only`


extra newline?


Достаточно одной пустой строки между методами


Er... this looks like just a whitespace change, unless I'm mistaken?


Copy and paste the abstract here?


So `React.Children.only` doesn't like string children? That seems wrong :/


This will cause `set -x` and `set -o pipefail` to be set at the root level of Travis' script which will mess with their error reporting system. Please revert this.


Not sure we need to say this. If they have cloned the repo they have it. :)
If not having it could stop the processing that should be a message to the console and/or in the log.


Нужна пустая строка в конце файла


do you know how long will we have to wait until support for mapleader is added in new version ?


@yjun2 DWMR (done with my review). I enjoyed our call last night - thank you. Press on!


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.67: OK (0 of 0 tests passed)

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


You mean like this?
http://mslc.ctf.su/wp/volgactf-quals-2015-cpkc-crypto-400-writeup/

Ah, yes you are right. It will always be brute force. I think they
didn't make a mistake in generating the keys. Just used too small values
to be secure.


I think we're ok without this.


@HmHmmHm (정색) 전혀 흥미롭지 않은데요


My bad ;)


Hi Gokul,
Thank you very much for the prompt reply and I apologize for disturbing you since 4 days.
I have downloaded the pykafka and started working on the same then , I got the below
Error  File "log2kafka.py", line 46, in <module>
    main()
  File "log2kafka.py", line 40, in main
    producer = kafka.producer.Producer(options.topic,options.host,int(options.port))
  File "/home/veeresh/tests/kafka-storm-cassandra-master/kafka/producer.py", line 29, in __init__
    self.connect()
  File "/home/veeresh/tests/kafka-storm-cassandra-master/kafka/io.py", line 21, in connect
    self.socket.connect((self.host, self.port))
  File "/usr/lib/python2.7/socket.py", line 224, in meth
    return getattr(self._sock,name)(*args)
TypeError: coercing to Unicode: need string or buffer, int found

can you Please let me know the the reason and I have doubt that error is related to socket  ,please clarify this doubt as well.

Thanks in advance.


1836664-ReturnWaterTempReset (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1655 of 1655 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)


For why this is problematic: https://travis-ci.org/mapbox/mapbox-gl-native/jobs/62394555#L1882 ends abruptly and never executes the `after_failure` script.


I get the following output for a partitioned device:
```
$ sudo lsblk -nb -o SIZE /dev/xvda
171798691840
171792149504
```

So you should probably only parse the first line.


Doh!


I've attempted to address this in 9c710bebc2aa5dd7cc620d93a9001d3914876865. I don't feel as if it's worth adding a new test, because all that test is checking is that the option is accepted, not that anything is done with it. However, I have renamed the existing test to make it more generic.


@cedricmessiant this robot test line is fragile and will most likely fail randomly. You are loading the page and clicking on an element without waiting for the element to be present in the browser. You have to add a "wait until page contains [element]" before clicking on the overlay button to make sure the element you are clicking on is really present. cc @hvelarde


Maybe add a switch to control if it should be packed?


"change the copy bath at the "

Should be "path", not "bath". :)


In bash, this should be simply `=`, not `==`.


Hey @duff,

The way you described it is exactly how I first implemented this change, but I think it will be used by enough gateways in the future to justify the top level property. I will be implementing this for Authorize.net today.


Do you have any Infos of the value? Maybe we could create an enum :)


add_warnings (lefticus) - Win64-Windows-8.1-VisualStudio-14: OK (348 of 348 tests passed)

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


No answer to these questions.


Correct, a nice idea.


@dennisfrank as I said, my hunch is that this is a mistake.


@cedricmessiant "Log in as site owner" should be explicitly mentioned in the "Given" clause, e.g. "Given a logged in site administrator". See https://github.com/plone/plone.app.robotframework/blob/master/src/plone/app/robotframework/user.robot cc @hvelarde


The controller gets the current coordinates from the server every second. Does the connection fail it will generate example coordinates. Every previous coordinate is visibile on a red path drawn into the map...


@Kibrael - might want to walk through how to set up the database. Almost a step-by-step. Does the order matter when loading data? Can be simple.


@alexblair1 DWMR. Press on!


yep, thanks, fixed.


Need to add to ceph-test package too


@Kibrael - we don't need the log file here. Should add it to the .gitignore and remove it.


Should be define ULLONG_MAX    (LLONG_MAX * 2ULL + 1)


![Uploading 屏幕快照 2015-05-13 下午9.28.36.png…]()


we currently don't need this and it would make the impleementation a bit more complex. It is easy to add in case its needed later but I think its better to not have it right now.


`s/remote/remove` :confounded:


I should remove this comment, as I tried to remove the operator, and it have generated compilation errors. But, seriously, I would be very tempted to use wstring.


ダウンロード


//getConfigEduroam("senha", "usuario@ufg.br");


this lie doesn't belong here


thats irrelevant, we have single quotes here in other tests too, maybe in all of them


add_warnings (lefticus) - i386-Windows-8.1-VisualStudio-12: OK (347 of 348 tests passed)

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


Not sure how much work to change this (if a lot don't worry about it), but for consistency in naming the return values ... You have:
`aggregate_report_31`
`aggregate_report4x`
`aggregate_report12_1`

Maybe make them all follow the same pattern:
`type_report_number_sub`


Probably: Normal = 0, IgnoreLightsAndLanes = 1, IgnoreStreets = 2, IgnoreLights = 3


reack, unchanged after rebase


Cyanide lunch Combo


Fixed ages ago.


done


:metal:


remove the second comma here, the English reads more smoothly.


Maybe you remove this comment here ...


1836664-ReturnWaterTempReset (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (629 of 629 tests passed)

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


Should we note autoconfig here? or is there no way out but to go completely manual if we have such packages?


This should probably be at 1.2 now.  It *may* work with 1.1.1, but we haven't been testing with that version.


@alexblair1 `return 3` this may make the tests pass but it's not a good solution. consider this: `return dividend % divisor`


1836664-ReturnWaterTempReset (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1661 of 1661 tests passed)

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


I think it's better to translate "paint" into 「绘制」than 「描绘」, as far as I know, Google Chrome doesn't have a Chinese interface, so we might refer to the Firefox timeline:

![screen shot 2015-05-13 at 21 32 37](https://cloud.githubusercontent.com/assets/1091472/7611612/1c3430aa-f9b8-11e4-86ca-adc69d389df9.png)


@alexblair1 DWMR. Press on.


Hi Fabio,

I am currently using nfs on the warpboard for my rootfs. To be able to use usb port for nfs, I have to put this modification in dts file:

--- linux-patches-4.0.old/arch/arm/boot/dts/imx6sl-warp.dts	2015-04-07 18:00:16.656819047 +0200
+++ linux-patches-4.0/arch/arm/boot/dts/imx6sl-warp.dts	2015-04-07 18:00:36.828818182 +0200
@@ -122,7 +122,7 @@

 &usbotg1 {
 	vbus-supply = <&reg_usb_otg1_vbus>;
-	dr_mode = "host";
+	dr_mode = "otg";
 	disable-over-current;
 	status = "okay";
 };

Can I use nfs with peripheral mode ? I am not very familiar with dr_mode stuff.

Thanks,
Arthur.


Ok. Thanks for the info :)


Well, implementing all operators twice (one for ::operatorFoo, another one for operatorFoo=) is quite tedious. So, just use one to implement the other. That's just easier to do and maintain.

> Needs to test for copy uponself before, right?

I do not see why. This code does not change *this at all. Did I missed something?


Oh, didn't notice the date. Sorry! I guess the [PR](https://github.com/illuminate/support/pull/28) can be closed then :)


Please don't send PRs to this repo. They will NEVER be merged. This repo is read only.


@datakurre I think you have to start using [EditorConfig](http://docs.plone.org/external/plone.api/docs/contribute/conventions.html#editorconfig) ;-)


FWIW, I recently tried with Spark 1.1 but didn't have any success.

On 2015-05-13 9:38 AM, Chris Bethune wrote:
>
> This should probably be at 1.2 now. It /may/ work with 1.1.1, but we
> haven't been testing with that version.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/unchartedsoftware/aperture-tiles/commit/52ff084f6d00b46383a18f585282dd36dd2c8493#commitcomment-11175854>.
>


add_warnings (lefticus) - Win64-Windows-8.1-VisualStudio-12: OK (347 of 348 tests passed)

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


ack


should be covered by the ```%{_bindir}/ceph_test_*``` and ```usr/bin/ceph_test_*``` lines in the RPM and DEB package lists.


Workaround from https://bugzilla.mozilla.org/show_bug.cgi?id=1164447


Yes, I had tried git submodules, and I have found them very difficult to use:  The merge conflicts were endless.  I never did figure out how to just synch sub directories of a repo either.  Instead, I use Subversion to synch the sub directories of various repos, including the main pyLibrary, where its tests are.

I am open to removing the pyLibrary code and including it in requirements.txt
> pyLibrary==0.7.15132


Please add detailed comparison report as in unit tests reporter (`mocha` etc...)


type-o: comma**-**


yep, wrong commit message =/


nvm, I see it's wildcarded


This is slightly unrelated but we could define app name in one place https://github.com/kasper93/mpc-hc/commit/7a6015f3806791dd465ce3e13c79fc81e1180b6a and use it.


removed


Should it be 1.2.X, or are we pegged to a specific point release? It looks like there's 1.2.0, 1.2.1 and 1.2.2.


No that is for checkboxes in the body, the checbox (this one) in the header is generated in a different way, and that the reason for this hack


Please try. You could even find some enhancements since I bet that assign is smart enough to *not* do all those push_backs without reserving the correct amount of memory before.
If really performance in this class is a problem, and if the compiler can not optimize this by itself, it could be fastened by precalculating "_str.begin()+pos", too. But I doubt that modern compilers are stupid enough to not do this. And in debugging it's fast enough. I have spent lot of time in cgdb those days.


sorry, I see a project with meteor-i18n-admin, I think I could test that by your solution. I try to back this commit about i18n.


ummm don't remember


I believe this is the homepage for a GUI application called ImageOptim. The homepage for the `image_optim` gem appears to be here: https://github.com/toy/image_optim


The PR is here: https://github.com/contao/core-bundle/pull/254


please adding test `expected exception` to check changed Exception throwing, just like such below.
https://github.com/zendframework/zf2/blob/master/tests/ZendTest/Http/Client/StaticTest.php#L249


Hi If you have some time can you update spring security plugin?


Sounds good!

Leave the start and end columns.. we need time bounds for annotations.


ok, cool.


Woohoo! I'm legit!


ruby.sh not exist !!


I haven't, but I've noticed there are [open pull requests](https://github.com/illuminate/support/pulls) on this repo.

If they are never to be merged, then it'd make sense to close them all down to reinforce the point.

I'm new to the Laravel eco-system, so I'm still trying to find my feet, sorry to be annoying.


I thought you'd say that ...

We can ditch it if you're deeply opposed. But here's the argument for using it:
- As you'll see in some of my responses, there aren't always simple native JS alternatives, which means you'd have to write your own functions (to do things like `_.mapValues()`) -- so why not use a battle-tested library for those, instead?
- Including lodash as a dep would mean that contributors would be encouraged to use *its* util functions to perform tricky little tasks instead of writing their own. Might make contribution easier, less buggy.
- I like the functional clarity of, for example, `_.includes(string, char)` over `string.indexOf(char) > -1`.
- Even if we opt for using native alternatives where the native alternative is simple (e.g. the `indexOf` example), what do we lose by using lodash functions when the native alternatives are *not* simple (e.g. `_.mapValues()`)?


![2015-05-13 9 28 36](https://cloud.githubusercontent.com/assets/7954078/7611553/b5a7c2a2-f9b7-11e4-8a1d-307365fdf740.png)


Que trabalho lindo.


Sure, sounds good.


I realise this isn't gardening now, Would rebase but 500000 commits came from master


ack


...but then bug fixes are much harder


Using a for loop is only my habit.

But, in your version, there one bad practice: using uint32 to describe a size, because this will make program buggy on 64 bit machines.
Plus, using one instruction loops, without {} is something I hate to do and see. Why? Remember the goto fail, last year? This is the reason I never use while/for/if without {}. I had my own time lost for nothing before that apple fail appeared, too.


I meant if a dev (like me) is stupid enough to do `s += s;` (empty or not).

By looking at the code, it seems it will actually work anyway. Nevermind. :)


Strings do not have the `forEach()` method. But it turns out that the ES6 `for (x of y)` *does* iterate over strings, so that could be an alternative.


You can check with @dgiesbrecht, but I think this (and the other similar instances) should be:

```bash
java GraphClusterApp
```


I think we could simplify all this by using momentjs time formatting for strings. Reducing props just to one string and a switch to set it.


Objects do not have a `.map()` function.


Skeleton file. Leave it alone. Use if for reference.


Thanks! Glad you think it's a good approach.


Pradek IT karjera kursu kodai.


А почему удален вызов sendConfirmationMessage ?
В дев версии не приходят подтверждения.


removed


The change was made to improve readability. Previously the visible part ($dashboardPart) was slightly messy, now it's just the key. Note that the actual link didn't change, just the text.


This is cool!

My worry is that this will cause the Python version to diverge (even more) from the Java tokenizer. The original motivation for this port was to allow Python scripts to interoperate with ark-tweet-nlp (Java), without worrying about differences in tokenization. That said, I'm not sure how other people are using this tokenizer, and maybe it's worth including enhancements beyond the Java version. What do you think?

Another option: would you be interested in making a similar pull request to the Java tokenizer?


Yes, I find that when text is justified to the left it's more readable.


ok イメージどおり！


Para adicionar o jar do postgres:
1- Botão direito sobre o projeto
2-Propriedades
3-Java Build Path
4-Aba Libraries
5- Add JARS


As discussed previously, I think we can just include the instructions to run this from Gradle.


ここは、self.user_idとcurrent_user.idだけで比べたほうが省エネだね。tipとして。


1836664-ReturnWaterTempReset (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1661 of 1661 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)


The only reason was src/rules/declaration-bang-space/index.js line #31: I return `false` in order to break out of the iteration; but use a simple `return` (which is returning `undefined`) for the early return above. The early return could just as well `return true` if you want to keep this rule.


Ah ... good idea.


Slight formatting issue, extra space here. Otherwise, ack


Thanks Madhura.
I looked at some of your other related commits too.
I realized many things that were missing in my patch. Will take proper care the next time. :)


I don't get how this change improve things over what @Slava originally did. One of the point is that if a third-party package want to test if a string of text is a potential Blaze identifier (let's imagine that meteor-jade wants to implement JavaScript-like expressions) it will have to test:

```js
BlazeTools.parseIdentifierName(scanner) || run(/^(?:@index)/);
```

And if Meteor adds more @-identifiers (as in #4266) the third-party package will have to update its identification code to support them.

I think that `BlazeTools.parseIdentifierName` (or it may be another public method) should match supported @-identifiers (even if they are not valid JavaScript identifier).


Just wanted to point out that the 'container-entrypoint' (instead of bash)
here was initially _expected_, it was not a typo.  In the patch itself it is not
documented.  That was a way how to enhance environment (and run some scripts
possibly) the same way as it is done during 'docker run' --

-- without additional hacks like touching the BASH_ENV variable or ~/.bashrc.
Those hacks in turn does not help e.g. with 'docker exec -ti /bin/sh'.  Or with
'docker exec arbitrary_binary'.

You would be able to do 'docker exec cont-entry arbitrary command line'.

But yes, this is just a documentation.  I'm not blocking this patch - just
wanted to document the original purpose somewhere :).


ack about the iterator, implemented in https://github.com/fallen/artiq/commit/e8ddfab0368e67eecb40abbaa5b65924bfefd7db
It's much cleaner this way, thanks!


Double check the line numbers after we merge.


ack, as it's autogenerated


thats meant that way because one thing is piped into the other and that whole is piped into the third


Okay, maybe not BILLIONS of embeds, since it's only for websites that do some tinkering with their API before reposting links, but you know what I mean. They could just as easily have redirected the old v2 URLs to the v3 equivalents, it's so sloppy on their part...


ah, alright


That would screw with the before/after order of the characters (and no forEach on strings). Here's another place where lodash provides functionality that doesn't have a super clear native alternative (though I think the alternative would be to write a old `for` loop backwards that proceeds backwards ...)


ack


@hvelarde Agreed...


Would you mind if I considered merging this into core next time I make some updates?


Espresso10: Set SELinux to permissive


ack, largely generated.


That would be a perfectly valid usage. It would work, because:

First, we create a copy of s, s2.
Then s2's size is increased to reach s.size()+s.size().
Finally we add s at the end of the s2, and return s2. No conflit here. Remember: we are working on a copy. This is probably why this operator is usually defined outside the class :)


No it doesn't support polkit ATM, it does need to handle permission errors seperately


That's weird; I don't get how using a submodule would cause a merge conflict.  The only thing in this repo would be a reference to a specific commit hash in the pyLibrary repo.

Honestly, it's not a big deal to me either way, since I'm not going to package this up as a deb for puppetized deployment in my infrastructure.  If I did, though, I'd probably prefer the requirements.txt route.  That way, I could just use fpm to build packages for everything I needed.


The cart-summary and typewatch is not added in step 1 so you can't update units by typing in the input. Is this the correct behaviour?


Great work man ,
Im working in MIUI 6 for samsung device ....
My device use CM 11 same build number as your base rom .
i modifited your patchrom so i add my stock rom and other important files
The framework folder of your base rom same my framework folder expect the framework-res.apk different , so i decompiled my file and put it in patchrom ...

When the rom boot it stuck in MIUI logo but adb work , so i need a help to fix the problem ?

I use my own kernel and the kernel boot MIUI 5 , do i need to modify it to boot the MIUI 6 ?

And because your rom already work so i think my problem in kernel ?
What do you think ?

Thank you my friend :)


降低样式计算的范围与复杂度


We also need to make this window on top our crashed window. Probably only way is to make it topmost in case our window already is topmost.


# A confession about the fixed issue

I was aware of the connections should be expected to fail and I didn't made it right from beginning. Why?

Decades ago internet was a slow, quirky place. All of a sudden it improved a lot. It might have taken a year really but before the connections were really unreliable with packet loss being easily double digits. Then it improved a lot, to the point you can assume the network will just work.

I decided to use this as a testbed to measure this improvement. As expected people from advanced countries would not see disconnections.... Russia, China, Spain, Brazil would report this.
And me... since I've switched to a supposedly affordable ISP.

# Known issues

1. Right now the web monitor is in a transitioning phase as it cannot detect if something is off because connection failed or because the pool is not-activated... well, it can but I must think at it again.

2. If a connection cannot be had, the icon still goes white as the mining process starts instead of staying red to signal no remote servers. Uhm...


No problem. Following the coding style of a project is important, otherwise the code quickly become unreadable. I had enough proofs of that at my last job.


Yeah.  These are tests for the the functionality that you are adding.


True. But utils_pch.h is including it AFAIK.
I know those types are somehow a bit redundantly defined. But as soon as we'll move to c++11, we'll drop all that. :)


Maybe because you closed it and none of us reviewed it?


This broke unit tests. I think we should start getting better about that. We don't have a lot of tests, but we should really try to keep Travis green as we're going to be adding more soon.


saved me now...


May want to rebase this with 7f0acab2465e61b0759885cfb8f7d65843c707a3 as that's a minor modification on this


You are welcome!
Following links should be useful for all the GSoC students if you haven't already seen them
https://ci.phpmyadmin.net/job/phpMyAdmin-continuous/
https://wiki.phpmyadmin.net/pma/Developer_guidelines
https://wiki.phpmyadmin.net/pma/CodeSniffer


Done: https://github.com/roots/bedrock-ansible/issues/217


Peg it to 1.2.  I don't think there's an issue using 1.2.1 or 1.2.2 as those didn't introduce any API changes.


I always try to avoid deps if I can

for mapValues, I will do something like

```js
const obj = { 'a': 1, 'b': 2 };
const map = Object.keys(obj).reduce((map, k) => {
  map = {...map, [k]: obj[k] * 3}
}, {});
console.log(map) // same result as the lines below

// from lodash doc
_.mapValues({ 'a': 1, 'b': 2 }, function(n) {
  return n * 3;
});
// → { 'a': 3, 'b': 6 }
```

I personnaly do that (vanilla when possible even if more verbose), but I am not against using this one since it have nice perfs and is correctly maintained :)

Note: we can rely on this `Array.prototype.includes()` since we are using babel https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/includes


最重要的，获得绝对高度之后，如何实现z轴自稳。也就是z轴平衡系统是如何实现的。


ack


This explains my pain better:

https://codingkilledthecat.wordpress.com/2012/04/28/why-your-company-shouldnt-use-git-submodules/


> No, it returns the end iterator which was given, when not found.

In c++11, i agree.In c++98, not according to the doc. But yeah, I know, I'm nit-picking. ;)


@alecive don't forget to update the [documentation](http://wiki.icub.org/yarpdoc/group__yarpmotorgui.html) as well :grin:


yeah sorry. Object.keys(obj).map(k => obj[key] ...  )


No problem for inlining things. I think it's useless (the compiler can optimize thing by itself anyway) but no problem.
Anyway, I try to avoid putting code in header files because it makes it mandatory to recompile every file which includes that header when you make a change for a reason or another. VT's compilation time is long enough.
Plus, it avoid me to need to open 2 editors to see all class's methods implementations :) (I could use the tab feature of vim, but I don't like this feature. Text editors should only edit text, and let window managers work with tabs.)


There is no `$sInstanceQuery` variable in this method.


원래 있던 pending 복구해주세요


Nice use of title attribute!


no, we want changes be small and atomic. If we later don't want the enum for some reason we can just revert the other commit.


We already have a fix for that in my repo. Let me know if it's not good enough.


Для тех кто использует simple-core путь должен быть таким:
```
include /var/www/project/config/dynamic/nginx.config;
```


changed it to tracker_id. it is used to pass the tracker id to the stats popup


Thanks for this fix! Would you open a PR so that I can merge this in?


This didn't apply, but that series of wrappers could be useful for debugging.


의도한대로 동작하는지 로컬 콘솔에서 실제로 해보지 않았군?


I just created a jsPerf and it looks like the code would be much faster if instead of using this regex test we maintained an array of whitespace characters and used a isWhitespace function that checked characters against that array by index: http://jsperf.com/check-whitespace-regex-or-index


To be entirely fair, it's now obvious to me that I broke tests without realizing it too. So, I clearly need to get better as well.


using a bound constraint might be safer though


Hi,

On addreses step ?

https://github.com/PrestaShop/PrestaShop/blob/1.6/themes/default-bootstrap/order-steps.tpl#L58

Regards


I won't merge that, binary-only.


I did not changed the properties' names. Never. I have noticed that, sometimes they have a _ at start, sometimes nothing. So, I did not changed anything. I'm fine to put m befores. That's what I usually do, too (except that I do not use CamelCase, but underscores, as C++ standard functions does. Just my tastes, and I'm fine with Camels.)


1: M111 unrecognized component name: base


Is there any reason you stored `t` as a variable, rather than just making this `this.clock.tick(100)`? :open_mouth:


Did you submit a PR to pinax/django-user-accounts for this?


Does this need to live in the same place as the wifi driver? Might be better in another temporary repository, until we merge it.


I get the hesitancy to add deps, totally ... Well, I can eliminate it for now and we'll see if the need comes back.


I guess I'll release the next version in a few weeks. You can download CI builds of IdeaVim with mapleader implemented from [here](https://teamcity.jetbrains.com/viewType.html?buildTypeId=bt299&guest=1).


I couldn't merge those patches, loads of conflicts with cleanups I did as well. Would be interested in updated patches.


ack


コメントテスト


This looks like spaces instead of tab? Does it come like that from upstream or maybe it got modified by accident?


This is strange, git's diff failed for some reason to recognize the actual diff. The actual diff is

```
diff vlsv_reader_parallel.h ../vlsv/vlsv_reader_parallel.h
123c123
<       success = (masterSuccess > 0);
---
>       if (masterSuccess > 0) success = true;
```


@adaptivity new line for the mob bottle localization


Again, would be better in a separate repo.


=========

##Conveners

* Christine Harlow (University of Tennessee, Knoxville)
* Jennifer Eustis (University of Connecticut Libraries, Storrs)

##Terms of reference

* The name of the interest group is the Islandora Metadata Interest Group (IMIG)

* The purpose of the IMIG is to investigate and provide metadata solutions that help improve metadata creation, maintenance and enhancement in Islandora.

* Specific goals, activities, and outcomes include:

  * Metadata best practices for Islandora

    * Metadata profiles for multiple metadata standards
	* Documentation

  *Investigate metadata tools

    * Creating and editing metadata in Islandora
	* Pre and post processing of metadata including harvests, transformations, and migrations

  *Investigate features that enhance metadata creation and maintenance

    * XML form enhancements
	* Use and application of taxonomies
	* Alternatives to XML forms

  *Investigate linked open data

 * The interest group will meet once a month virtually (eg. via Google Hangouts). A call for agenda items will be posted to the Islandora Google Group at least 1 week prior to the meeting. The convener will appoint a note taker for the meeting and meeting notes will be made available at [IMIG](https://github.com/Islandora/Islandora-Metadata-Interest-Group).

 * The convener will report to the Islandora Roadmap Committee following the IMIG's meeting.


@jonasschnelli feel free to submit a change to the release notes


Ah, I see.  Yeah, definitely some rough edges there.  I largely work around those issues by wrapping `git` in a bash function that automatically does `git submodule update` after various operatiosn like checkout, pull, rebase, etc.


`storify` 이상함 (있는 동사도 아니고 swindler에서 유의미한 고유 용어도 아님) 굳이 한 단어로 줄여서 써야 할 필요없어 이거는


We could do, would it need editing again?


If $use_taxes at line 131 and if $use_taxes at line 132 are double if conditions, because $use_taxes is a boolean value.  If $use_taxes at line 131 is false the condition is false and the second if condition at line 132 is not necessary.


haha :8ball:


:trollface: @ myself :wink:


Isn't it dangerous?
While you are waiting the old master could became available...
I think the best what we could do in this case - write recovery.conf without primary_conninfo and restart it in readonly mode...

P.S. I was working a lot on governer during past two weeks and did some cleanups and refactoring of your code.
The basic ideas were:
- do as small requests as possible to etcd (do one recursive query to fetch all information about cluster)
- start up postgres with pg_ctl
- create and delete replication slots only when list of members has changed
- do not write primary_conninfo into recovery.conf when master is not defined

If you are interested, please have a look at: https://github.com/CyberDem0n/governor


On my build system (Cubieboard2) there does not exist the directory `armv7l-linux-gnueabihf-native`, it's name is just `armv7l-linux-native`
Maybe it is better to link `bin` to `tools/depends/armv7l-linux-gnueabihf-native/bin`


`minimum: 3`은 좀 랜덤한데... `ㅋㅋ`는 안 되고 `ㅋㅋㅋ`는 될까? 이건 굳이 필요하지 않은듯


Yes, it's still a work in progress, as I said. Still needs cleanup: TODOs to do, comments to remove, messages to update, and, the most tedious task, that I consider useless but I'll follow the guidelines, writting that the parameters named hello means hello, and those named world means world :)

I personally stopped documenting every single parameters a long time ago, now I simply choose their name, and describe the pre-/pos- conditions of my functions. Not invariants, because in my usual code things are small enough to make it clear when a function changes or not stuff in the class, and what it changes, with the const keyword.
Honestly, documenting every single parameter is such a pain. So often useless, finally, because names can often be self-describing.


why?


1836664-ReturnWaterTempReset (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1661 of 1661 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)


The hyphens we've been using throw an annoying wrench in this. Any chance we're considering a switch to camelCase for rule names?


BTW Are you going to add `vessel:cores` and `list cores`?


I agree for the `strings.HasPrefix(identifier, needle)`, but I think the name should be more permissive

 on the javascript version I did this: https://github.com/scaleway/scaleway-cli/blob/master/lib/utils.js#L109-L118

I replace the -_ chars in the input with ".*" so, `ubuntu-trusty-dev` will become `ubuntu.*trusty.*dev`, then I try to match the regex against the `realname.replace(/[^a-zA-Z0-9]/, '')`

Also, you need to raise an exception if your lookup resolves more than 1 item, so the user need to be more explicit to only match one


Look at line 56. It's now directly binding the size and position of the window view to the window properties. Now need for extra Connections.


Wouldn't `camelize` be the right choice here?


@chelmuth: please add to staging


To demo in the sample compositor how workspaces work. It's an easy way to test that windows open on the right workspace, switching to a window on another workspace opens the right workspace, etc.


I guess this is made static to avoid allocation? But I think CString will internally still alloc space for string.


Does this template already exist?


This was changed back to your wallpaper in a later commit. You'll see that if you look at the PR diff.


How far away are we from Jekyll being usable?


``` ruby
[:foo, :bar_beta].join("/").camelize # => "Foo::BarBeta"
:bar_beta.to_s.camelize # => "BarBeta"
```


mispaced )


leftover


Hi. I was using this django-nailbiter. It is working fine if it is in a local repository. But when it comes to scenario where static images are served from remote place such as AWS s3 bucket, This Api fails in generating an unique file name as it searches in the local repository(upload to remote place such as AWS s3 bucket)


As discussed offline, we reference gradlew (and ensure that relative path is correct)


I think we prefer to put the 'then' on the same line (if [[ ]]; then), but just something for next time.


Might be slightly neater to redirect to the /(page_id)/ view if there's only one page which should remove that bit of duplicated code.


Done in 9f7446d (and I added also other options that were not there).


Our EventRouter is to be used on single thread by desing.


Do you plan to do something with string.other.INC.xys?
i.e: contextual_help to open the included file?


You're absolutely correct. I'll submit a fix internally and have that rolled out next time we sync the repo. Thanks!


This was me, not sure why it says unknown :/


This was me too, not sure why it says unknown :/


1836664-ReturnWaterTempReset (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1655 of 1655 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)


Yes, it might be able to be handled by C++ (like I said, this was just a first pass - no point in working on a lot of new code if you end up rejecting it).

I don't think it should be unique to all screens - some desktop shells, such as OS X, use a per-screen list of workspaces. It should be up to the desktop shell to use workspaces shared across screens or per-screen workspaces.


argh, no, forgot it. It's pushed now.


When will you build a cm12.1 rom?I can test it for you


I was having issues with it picking up ClientWindow due to installation bugs, so I was using this temporarily. If you look in the overall PR diff, you'll see I switched back to ClientWindow.


Will do.


Yeah we can roll this back since it will be handled by Nginx before it gets to WordPress.


See lines 58 & 59.


Oh, exact. My mistake, thanks. Good to know. Then, should I implement an alternative version for C++98 users? Or should we (you know that I would prefer that) just not mind about 17 years old compilers? I wonder who can not afford to use GCC or clang.
And, I am using the current Debian old-stable: clang provides a repo with the clang v3.5 for me, so I can have a nice compiler even on my outdated OS :)
For normal Debian users, the g++ version is 4.9, which supports C++11 correctly enough. And as usual, Debian stable is outdated.

> But yeah, I know, I'm nit-picking. ;)

I do that soooo often. It's quite fun :)


No worries, there's no rush. When you next get round to working on the code, let me know if you want to chat about the use case for this (and if there's a better way of doing it).


This is a work in progress and does not reflect the final state.


I think we want to merge $tags and $tagsEvent here; rather than logging $tags then actually setting $eventTags


Can we convert this into a scope so it can be re-used by both the `published` and `unpublished` methods.


cool, thanks! :)


这个版本的翻译有问题，放到游戏中整个翻译失效了，估计翻译了不该翻译的内容或者xml文档格式存在问题。


/// <summary>


@mpalmi looks good. I didn't see any denials in permissive for udev_t either


Too late. I'd expect it in sanity_checks function of mkcloud, and just comparing with the value of nodenumber (no need for search)


in particular, this line should be logging the tags that get set on the image -- with the patch as-is, it will log the tags it generated based on the filename, and then throw those away and assign tags based on the upload form


format:
SHA1, pehash, file name


I see that you preallocate space for strings in `CCrashReporterDialog`. So CString probably will do shallow copy, but you could just get LPCWSTR from WaitForUserInput.


The following text in en.json:
"One there from the polling excluded device cannot create a history"
would read better as:
"A device excluded from polling there, can not create a history"


That's right but maybe the problem is on another side,

when you are in the summary (step 0) and click "**proced to checkout**" you are linked to */order?step=1* but if you have a order value less than the minimum order value, you still in the summary with a huge warning, not in the addresses, here it is when you miss the cart-summary.js that handles what i ask before


Should this be a div?


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


Any chance you could send that patch (and the following) as a script?


should this var_dump really be part of the codebase ?


Missing <thead> and <tbody>. I'm not sure if this matters though


I don't think so. This structure is the same in the edit forms, and I simply took it from "the designer"s templates. =)


(｡+･`ω･´) 다시보니 저도 그런듯


TeamCity xBIM :: XbimExchange :: Build [Build 283](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1624&buildTypeId=XBIM_XbimExchange_Build) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:02:11


Not exactly the same. The new `FilterStack` requires users (i.e. other functions) to run token processing filters, if there is any, before using it. It's better not to split the processing logic and also makes more sense to keep it in the `FilterStack`. Also it doesn't require additional modification. The `run` function takes in a stream of tokens. It only needs to apply the pre-processing filters on these tokens before they are grouped. The only difference now is this stream of tokens represents a sql statement instead of multiple sql statements based on the new change.


Create repository


I looked this up and yeah, band 4 is red.


this cause memory leak regression https://issues.dlang.org/show_bug.cgi?id=14573


Yrnir n pbzzrag


Why not use an RenderGameOverlayEvent.post here ?


> a bug report will be sent.

Is it mandatory to send a dump?


1836664-ReturnWaterTempReset (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1661 of 1661 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)


Pourquoi if messageListView?  S'il n'y a pas de messageList dans le groupe, on va rendre tout l'élément de navigation non fonctionnel, non?


noise-seed.wav seems to be missing.  (line 35 in studio.rb)
RP won't build


Sorry, I didn't see that and will correct the behaviour.

2015-05-13 0:54 GMT+02:00 killerstorm <notifications@github.com>:

> Why do you think it is unneeded? A transaction can become invalid (e.g. if
> it is double-spent), in that case it won't be present neither in block nor
> in mempool.
>
> Chromanode will give you {"status":"fail","data":{"type":"TxNotFound"}}
> and you should return None, False
> Which is interpreted as status 'invalid' by TxDb.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/chromaway/ngcccbase/commit/69ec92e88dcf73a16f023d312426e4a32af4a2f2#commitcomment-11165663>
> .
>


Well done :+1:


great!)


I have added the functionality to create toolbars on startup. The toolbars can use the configuration information from the yaml file with the anchor pointer syntax.


@klahnakoski, how's this look?  I was thinking of adding some errors, such as when the availability zone list contains members for which there's no interface, but things were starting to get a tad verbose.


c'est juste du code défensif...


Do we need this if sauce connect has already started in the Travis before script?


@CyberDem0n I think you should create some pull requests for the changes you have done so we can discuss them there.


1836664-ReturnWaterTempReset (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1661 of 1661 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)


Need to commit changes in present tense


@mpalmi I would drop unix from the interface names and use rw_socket_perms instead of { read write getattr }

I didn't get any denials in permissive for syslogd_t when testing


Metadata popup window in IE doesn't contain scrollbars by default, and the window cannot be resized so metadata can't be seen properly.


Good point.

I'd been meaning to remove the `primary_conninfo` from recovery.conf when a leader is unknown. This would be a good catalyst to implement that.

Nice code :)  I'll run through the differences today, but looks great.  Cheers!


The project URL is wrong.  The actual URL is https://github.com/marschall/jdeps-maven-plugin


Clicked the wrong button; will undo :-/


Why not just swap when BackBuffer.w < BackBuffer.h


:smile:


Is there a particular reason why you want to check the certificate? I put those options on both wget and curl
because I could get the image on some machines. Since I would like to preserve compatibility I am more inclined to use those options.


`chan interface{}` feels kind of awkward. Why not just `bool`? I mean, it's indifferent, you can throw whatever you want at it and it'll work (which I realize may be the point) but if it's going to be a "throw whatever at it" thing then I'd rather make it `bool` only — (or, ideally, `struct{}{}` 'cause it compiles to 0 bytes :neckbeard:)


第九行缺少一个“[”符号，已经修正


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

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


@josevalim thanks so much for catching this! I'll work on a new approach tonight.


I don't see this patch merged in upstream. Is there any reason?


Let me know if this happens regularly so I can reproduce and debug. I will add an assert here to catch it.


These date changes appear to be unrelated to this commit.


halt(404) if page >= pdf.count


Thanks for the comments @MoOx . I think all of them except the index import/export should be addressed in the latest commit. For import/export ... I wanted to see what @stylelint/owners thought about using camelCase instead of hyphens for the names first.


Neat.


原来github可以像博客一样进行评论的，very good！


Yes that's helpful to have the new docs up makes it easier to find something. Thanks


Je ne comprends pas avec ce code "défensif" comment cette ideaList peut réussir à changer l'étât d'un groupe où il n'y a pas de messageList


Works much better now :)

Still, I get errors:

```
Error GPU mining. GPU memory fragmentation?
...
ethminer  FAILURE: GPU gave incorrect result!
```

Though it does find solutions, geth says blocks are mined and the account balance does increase. Upon checking the main chain, it appears about 1 in 5 has indeed my account as miner (maybe the others are just uncles?)

Would you like an issue for the error messages above (I have the full log handy)?


When will you build a cm12.1 rom for aries?


I would be more inclined to output and error/warn instead of an info for this.


same


Upgraded YUI purecss to 0.6.0. Small format issue w/ TOC.


ok, on l'enlève


88737258-EPW-Ruby-Access (jasondegraw) - x86_64-MacOS-10.9-clang: Tests Failed

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


Upgraded font-awesome from 4.2.0 to 4.3.0. No apparent issues.


a bit useless assignation


never do that!


I think they're waiting for me to write a test case, which I've been postponing for months. Maybe we can hack something together at the next Perl Mongers? :-)


It kind of have to be refactored. I'd prefer to do it now. There is one possible implementation in the previous comment (https://github.com/changsi/sqlparse/commit/c89b819ded098a7c224866d0a2d628d51269ca10). If the implementation of `MySQLParser`is a little complicate than the simple one because of the design of `MysqlCreateStatementFilter`, at least encapsulate the related logic inside the class and refactor the class later. One  benefit is it keeps the implementation in this file clean, and secondly, when we refactor each individual dialect-specific parser or add one, we only need to work on these individual parsers without modifying the functions here.


It looks like this broke the tests.


А зачем удалили bg-color? А вдруг картинка не загрузиться? Надо прописать цвет, похожий на изображение.


True, we forgot to commit it separately until it was too late :|


Fixed here: 5ed820cf61cf14a38d94cc85a22fd1195631822b


cause i didn't knew it existed?


@HmHmmHm 사실 매우 흥미롭네요 다시 보니 흐헤헿


тоже самое: прописываем цвет, когда подкладываем изображение: `background: color url('../image/name.png') repeat 50%;`


The majority of the changes are codestyle related.
Not sure where they are coming from, like SQL re-formatted.


These three changes look good to me. Thanks for adding the description. It's very clear now how the function should be used. :)


Там прям такой диз? оО!


The docs came out rather nicely. Wasn't expecting it to work first time, good sign I hope.

So the idea of tagging a release now is so we can work on Lexicon without needing the dev version of Docile.


Depends on which style guide you consult... I think the suspended hyphen here makes the sentence a little easier to read, but I could be persuaded otherwise. I'll leave it be unless someone has a strong objection.


Packages actually.


Yeh, we just don't process stuff here.


LGTM, but I vote for wrapping this logic to try/catch as well so that it will work even if someone will make it async again. Plugin is responsible for catching exceptions occurred, we should not rely on some knowledge about how Exec logic works (so that it can catch unhanded exceptions in sync methods).


I've thought a bit more about this. I have 2 remarks/questions:

1) If this is really a clone then we should drop the Cloner interface, have EC and XWikiContext implement Clonable and move the clone code in their clone() methods.

2) If we consider it's different from a clone, then the interface should be named differently (Copier for example) with explanation as to why it's not a clone.

WDYT?


LibOVR has a function SDKWindow:getRotation in LibOVR/Src/Displays/OVR_Linux_SDKWindow, it should be a better way to ge the rotation info


It seems that copy is not needed here, is that right?


Ну, может я чего-то не знаю, но иначе ширина строки у меня получается не совпадает по буквам... Может я чего-то не знаю, но без letter-spacing я не смогла растянуть строку по буквенно как у них.


Поправлю


Here we should be throwing some sort of exception here. Exiting here is actually causing all the tests to pass when in reality its just stopping phpunit from completing.


I believe you can remove `_get_subnet_availability_zone()` and call `self.vpc_conn.get_all_subnets()` just once with all ids.  But that is an optimization that can be left to later.

At a high level this looks good.  I must run it through a debugger to have more comments about the details.


@tcz can you confirm that removing the additional reset is fine?


Parameters need `*`


А без картинки не обойтись?)


...I will get on that, frantically starts googling how to do that


> tomhurd commented on 9bc516b 15 minutes ago
>
>@mpalmi I would drop unix from the interface names and use rw_socket_perms instead of { read write getattr }

Good call on the name.

rw_socket_perms seems excessive (ioctl read getattr write setattr append bind connect getopt setopt shutdown), considering the fact that I was only seeing denials for the perms above.


Oh, good point.  And yeah, no testing at all yet -- for all I know this thing kills your kittens when you run it.  I'm hoping to do some testing today.


Yes, this was for the deprecated Ant build system. When I deploy the Maven package I plan to make a version which has Android support.


It might be fine to just recycle `desructiveTitle`.  IIRC only one copy of the string will actually make it into the string tables and our polyglots won't notice the different description.


Stop doing this


I work in a Windows Auth environment and really need this to be added, hopefully in the 1.4 release please!


Нужно убрать запятую перед "и"


two typos


You'll find it helpful to label these more descriptively.  Once you start getting into larger loops p isn't going to help you know what it is but original_path will. Or even just path.  Don't every worry about being too descriptive, the little bit of extra space you take up is well worth the added clarity.


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


exactly.


Baylon


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


Like here, I'm not entirely sure what i is just by reading it, if you told me with a more descriptive variable name it'd make it easier to read :)


Hmm weird - pytest is used as the runner, not nose. How are you running the tests?


hte => the


Good! Nice descriptive function name.


Can we make terms emphasized or bold?

Also I am not sure about the `Aggregation` - we talked about it here and we came to conclusion aggregation is bellow the Nulecule spec as we only compose apps. Aggregation could be used while speaking about containers in replica controllers or images that are actually run, but that's out of scope for Nulecule spec.


why ``return 0`` if key file can't be opened?


> I was a bit hesitating here. I think a static global variable might be enough for now since it's used only in the compilation unit.

It should be ok, but with singleton you at lest can control when and if it should be initialized. Though we call CrashReporter::Enable() unconditionally anyway.

> real full dumps are not an option because they can be huge but I have changed the settings we used to create somewhat more complete dumps now that we have automatic upload with compression.

Looks ok.

> I have kept WER enabled because it's what Microsoft suggests to do. However it will cause the Windows dialog to show in addition of the DrDump dialog which might be a tad unsettling for the user.

I think WER should be enabled. At least driver crashes might get reported. Any crash is unsettling for the user.


I will try using moment.js


Why is this be here instead of image?


@zetok it makes the function behave like the comment says it should.


I hadn't seen `self.fail()` before, that's cool.  You might also want the following pattern:
```python
self.assertRaises(
    MockFunctionException,
    xformplayer.load_form,
    self.xform,
    session_data=self.session_data,
)
# or
with self.assertRaises(MockFunctionException):
    xformplayer.load_form(
        self.xform,
        session_data=self.session_data,
    )
```


Whyyyyy @psychocoderHPC does this crash?
I just wanted to remove `void ionize()` from `Particles::` and add the functionality to the models like `BSI`


Did you pick this from nieltg repo? @CPedrini


FacilityTabGridView (evanweaver) - x86_64-MacOS-10.9-clang: Tests Failed

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


I don't mind the dev version: just download it and copy it into the folder .. no problem at all: But Docile came a long way so it deserves to have a Tag too ;)  Maybe someothers will use it and find out some issues or have other great ideas.

But the docs at  a first glance look good - also the index is helpful.


@victortxa the comments are much better now!


I think Proxy and InvocationHandler have been around since Java 1.3


TeamCity .NET :: NET - Redis4Net :: Continuous integration build [Build 0.5.0-DEV10](https://teamcity.coolblue.eu/teamcity/viewLog.html?buildId=236413&buildTypeId=Net_NetRedis4Net_BuildAndPublishPackage) is now running


fixed in 27e7ed9a8c9a4d47ced27c1a4b0036f9d2316dc0


TeamCity .NET :: NET - Redis4Net :: Continuous integration build [Build 0.5.0-DEV10](https://teamcity.coolblue.eu/teamcity/viewLog.html?buildId=236413&buildTypeId=Net_NetRedis4Net_BuildAndPublishPackage) outcome was **FAILURE**
Summary: Compilation error: src\redis4net\redis4net.csproj Build time: 0:0:58


Merge conflict good!


Looking good, keep on chugging :)


@mpalmi you're right, I would keep it as { read write getattr }

The java_role interface is similar where it only gets connectto, read, and write.

ssh_basic_client_template and init_rw_script_stream_sockets both use rw_socket_perms, so I don't think it is unprecedented (not sure if all of those permissions are actually required though).

Since we're only seeing read, write, and getattr, and the target is kernel_t, least privilege is better.


That's correct. I'm not sure this needs to be wrapped in a run loop.


eh, I see.. there's a problem though, when one wants to debug stuff - it's hard to tell, without looking at source code, why something doesn't want to work. I.e. it would be nice to have something more than just exit codes, e.g. when toxcore is compiled with logging, some debug info could be printed to stdout, or something


agreed. so i tried the first pattern and couldn't get it to work for some reason (might give it another go). the second one is the one i usually use but python 2.5 doesn't support `with` which is pretty annoying


this is just loading all the often used info in an object... in hind sight probably could just return the row object


I'm new here, so I'm not sure if the code is well readable and if something is missing. Please leave a comment, to help me to set it better.


Good call, it's because the initial function loops through the paths variable.  So when it was called again it started looping through the paths again and made this infinite loop ..... good call to split.


I wonder if GetComponent() could be changed to this.GetComponent()?


Thanks for removing the copyrighted information from the repository. Unfortunately, these files are still accessible through the history of your repository. You can remove them by following the instructions in [Remove sensitive data](https://help.github.com/articles/remove-sensitive-data/). In our original DMCA takedown request, we asked that the data be removed in this way. Please go through the steps in the GitHub article to get that data removed - we can't consider this issue resolved otherwise.


Looks like you misspelled 'canola'.


If you like you can also have a look at this commit @Flamefire :wink:
Thx in advance!


You need to reset gpu memory (reboot, or if you have a nvidia card: http://stackoverflow.com/questions/15197286/how-can-i-flush-gpu-memory-using-cuda-physical-reset-is-unavailable)


TeamCity .NET :: NET - Redis4Net :: Continuous integration build [Build 0.5.0-DEV11](https://teamcity.coolblue.eu/teamcity/viewLog.html?buildId=236441&buildTypeId=Net_NetRedis4Net_BuildAndPublishPackage) is now running


going to instead try something like https://github.com/groupon/DotCi/blob/master/src/main/java/com/groupon/jenkins/buildtype/dockercompose/DockerComposeBuild.java#L113

>  for dockerfile in $(find . -name '*Dockerfile*'); do
dockerfileName=$(basename $dockerfile)
dockerfilePath=$(dirname $dockerfile)
pushd $dockerfilePath >/dev/null
for filename in $(grep ADD $dockerfileName | sed -e 's/^\\s*ADD\\s*//g' ; grep COPY $dockerfileName | sed -e 's/^\\s*COPY\\s*//g' ); do
  test -d $filename && continue
  test -f $filename || continue
  sha=$(git rev-list -n 1 HEAD $filename)
  touch -d \"$(git show -s --format=%ai $sha)\" $filename ; popd >/dev/null
done
done


I also have some includes:
#include <iostream>
#include <fstream>
#include <sstream>
#include <string>
#include <iomanip>
#include <stdio.h>
#include <gmp.h>
#include <gmpxx.h>
#include <paillier.h>
#include<windows.h>
using namespace std;


Fix to https://bugs.internet2.edu/jira/browse/CO-1065


This line is dangerous:
> Log.error("No network interface specifications found for {{availability_zone_group}}!", settings)

you can also use `locals()`, which is just as dangerous:
> Log.error("No network interface specifications found for {{availability_zone_group}}!", locals())

These delightfully powerful forms should be avoided; they could hold passwords and other secret keys, which would be copied to the log.

I have long considered having the message template determine what properties will be allowed in the structured log, and that would mitigate some of the problems with logging sensitive information.  My hesitation is caused by my desire to sometimes include the whole structure in the log, even if it does not show in the text log.  I also wonder if it will not help with the security concern, and make it even harder to identify sensitive data leaks:

> Log.error("No network interface specifications found for {{settings}}!", locals())

In this case, I can not just look at the parameters, but also the template, to consider all the data that will be submitted to the log.

Suggestions?


TeamCity .NET :: NET - Redis4Net :: Continuous integration build [Build 0.5.0-DEV11](https://teamcity.coolblue.eu/teamcity/viewLog.html?buildId=236441&buildTypeId=Net_NetRedis4Net_BuildAndPublishPackage) outcome was **FAILURE**
Summary: Compilation error: src\redis4net\redis4net.csproj (new) Build time: 0:0:55


Thank you for noticing, here is the fix: https://github.com/cocos2d/cocos2d-html5/pull/2895


I also have some includes:
#include iostream>
#include fstream>
#include sstream>
#include string>
#include iomanip>
#include stdio.h>
#include gmp.h>
#include gmpxx.h>
#include paillier.h>
#include windows.h>
using namespace std;


Hi,
        I would like to include your PR in Rx swift and give credits for your valuable contribution and motivate you to continue to help me with this project. But to do so, we need to make this right :)

        Could you somehow set the xcode to use 4 spaces for tab. I think that you are using tabs. If I merge this as it is, I'll probably change your whitespaces when xcode indents my code, and you won't be marked as contributor for those lines.

        The reason why I use 4 space tabs is because those are the default xcode settings.

       API is a dependancy IMHO also. There is no difference between API and all of the other dependancies in sharedDependancies. However, I think I understand that compiler was probably giving you issues about using self in init, so I guess we could live with separate API singletons for now.

       I don't see any usage of $ in this class, so $ is probably unnecessary.


1836664-ReturnWaterTempReset (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1146 of 1146 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-69.81%25-green.svg)


The value of the `name` property is way too long. Would it make sense to keep the old value, but still add the new links?


88737258-EPW-Ruby-Access (jasondegraw) - i386-Windows-7-VisualStudio-12: Build Failed

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


cacti_value should be indexed by both num_of_bytes and wordsize since both matter here.


Don't think that dependencies is used anywhere else so you can probably just remove

let $: Dependencies = Dependencies.sharedDependencies


Whitespace issue


Whitespace issues.


no i didin't remove any thing
Date: Wed, 13 May 2015 02:00:08 -0700
From: notifications@github.com
To: cookie_stealer@noreply.github.com
CC: faizanharoon@hotmail.com
Subject: Re: [cookie_stealer] Cookie logger PHP script (b8dc28b)

Just remove that if statement with the brackets.. It will work even then. well, did you remove the '+' sign that is at the beginning, from every statement?

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


While I agree with WER, I don't find it user friendly to show the dialog 2 or even more times; it makes things too overwhelming.


Zde je jen změna v čárce - "nevyjímaje" místo "nevyjimaje"


Whitespace issues


#include "iostream"
#include "fstream"
#include "sstream"
#include "string"
#include "iomanip"
#include "stdio.h"
#include "gmp.h"
#include "gmpxx.h"
#include "paillier.h"
#include "windows.h"
using namespace std;


You have to change the superbuild to match, see: https://github.com/OpenChemistry/tomviz-superbuild/pull/15


no entiendo una mierda jajaja!


Those are the includes.


I believe this is the connection string that the previous website used for connecting and authenticating with banner


I don't think it belongs in version.h; it's not version related per se.


「新聞のやつ」ってだけでは、
いつどの講義に出したレポートかわからなくなりませんか？
レポートの文面にも講義名すら無いので気になりました。


I believe this was fixed a while back with this commit: https://github.com/fog/fog-core/commit/8cee51b886e5db5d007140a1bd26659596da6973


If you revert the rc2 and version.h changes, the idea of having this defined in one place makes sense.


helmano teni pasta?


rax retry tests


:tada: this is now in our history forever :+1:


How's this?


Care to PR this?


1836664-ReturnWaterTempReset (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (629 of 629 tests passed)

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


Fixed ... there was an override for the OrderController that replace the standard behaviour ...

Thanks


Wow scott, wow.


ファイル名から何をするプログラムなのかわからないので修正した方が良いと思います


lol nom publishing


プログラムにあるコメントがローマ字なのが気になります
せめて英単語に直しませんか？


Just trying to help out :smile:.


Just wondering - is there a reason to still call the c.scertWire.Verify() fn if ! acceptCert?


processingで日本語コメントが打てないのならば次回お教えしますので
言ってください。


Also made combat tkinter windows open on top.


Minor typo: Fleeting -> Flattening


Muito bom!! Correto!! :dango:


Done! https://github.com/MisinformedDNA/redis-rb-ssl


:+1: good spot, this commit is perfect except you didn't include the URL for the original ticket.


プログラム中にコメントをもっと書くクセをつけませんか？

setupにコメントが無いので付け足したらいかがでしょうか


Never mind - i will merge as is, but if you'd like to do a new merge request on top of this one to skip that call, it would safe some cpu time.  (I'd also prefer to see the block called if ! acceptCert be a separate nicely named function (queryUnknownCert()?)

Not critical, just nice to have.


> It should be ok, but with singleton you at lest can control when and if it should be initialized.

The constructor doesn't do much, the real initialization is controlled by `Enabled` function so it should not be a problem.

> While I agree with WER, I don't find it user friendly to show the dialog 2 or even more times; it makes things too overwhelming.

I don't think we can avoid the Windows dialog if we choose to use WER too. :/ I will ask the DrDump guy though. Maybe we can have a better control on when exactly it will show up though, like make sure we will never show DrDump dialog and the Windows one at the same time.


a point of real convenience to me here would be to change this to

```
res$children <- list(lapply(self$children, FUN = function(x) as.list(x, ...)))
```

since this would cause `toJSON` to treat children as an array of objects, which is just what `d3` expects.  Then we can take advantage of `networkD3` to view the tree structures interactively.


Not sure if it belongs to `VersionInfo`. I mean it already exists in fact. We could choose to use `AfxGetAppName()` everywhere if you think it's better.


I think the last changes broke this extension, I'm getting this:

    ext\mcrypt\mcrypt.c(350) : error C2365: 'RANDOM' : redefinition; previous definition was 'enumerator'
    ext\mcrypt\mcrypt.c(280) : see declaration of 'RANDOM'
    ext\mcrypt\mcrypt.c(350) : error C2086: 'RANDOM' : redefinition
    ext\mcrypt\mcrypt.c(350) : see declaration of 'RANDOM'
    ext\mcrypt\mcrypt.c(351) : error C2365: 'URANDOM' : redefinition; previous definition was 'enumerator'
    ext\mcrypt\mcrypt.c(281) : see declaration of 'URANDOM'
    ext\mcrypt\mcrypt.c(351) : error C2086: 'URANDOM' : redefinition
    ext\mcrypt\mcrypt.c(351) : see declaration of 'URANDOM'
    ext\mcrypt\mcrypt.c(353) : error C2365: 'RAND' : redefinition; previous definition was 'enumerator'
    ext\mcrypt\mcrypt.c(283) : see declaration of 'RAND'
    ext\mcrypt\mcrypt.c(353) : error C2086: 'RAND' : redefinition
    ext\mcrypt\mcrypt.c(353) : see declaration of 'RAND'
    NMAKE : fatal error U1077: '"C:\Program Files (x86)\Microsoft Visual Studio 11.0 \VC\BIN\amd64\cl.exe"' : return code '0x2'
    Stop.

Thanks


this is dead code, not a comment


Ooops! Thanks for pointing that out - added now :-)


Hi,

thank for the fix commit.
We will considering adding the fix on the next release.
Small hint: set the package name inside the if block to avoid the null reference.

thanks,
Mobiroo Team


Done

On Tue, May 12, 2015 at 10:33 PM, qiangdavidliu <notifications@github.com>
wrote:

> Want to bump this to 1.1.152 with the more efficient json?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/tbak/ribbon/commit/f1f67fe707b1f6004c2a7c0f2511dfdb1c48b30b#commitcomment-11168964>
> .
>


XYplorer.tmLanguage: At the end of that file, you missed "entity.name.function.CommandID.xys".
CommandID can appear such as get("#493")


Yes and no. Gimme a call when you have time and I'll explain whats going on.


@SimonWilks Looks good!


trailing spaces?


We should return the promise here, not catch local description errors and continue in spite of them, because subsequent steps will also fail, creating error noise.

Generally, we should return errors (promises) out until we hit code that can handle the error and move on (e.g. to the next test).


is this debugging output or should this be user-visible?


But C-u C-M-x was already bound to something very useful, how to invoke the original behavior?


I can do that if you prefer, I kinda liked the idea of holding a ref on the actual buffer.


この行は不要じゃないかな？


I will try this.


We're not waiting for setLocalDescription here which seems like a mistake. If setLocalDescription fails then the other side will receive a broken description, causing error noise.


Nicely done! That earned you a tweet and a mention in the upcoming Redis Watch edition :100:


maybe those headers should be moved to ./common?


To ensure everything is properly de-coupled, I would make sure all of the views are triggered by a message, not procedurally. So if, for example, you want to show a "loading" screen, you would show it in response to a "RoutesWillLoad" message, which would be triggered before the routes will start loading.


Not really no, I made sure that the caller is always the main thread. Messages are then passed to the crash reporter UI thread but it never uses `EventRouter` directly.


Indeed, because libcommon is currently not a library of its own but rather a set of shared sources, there may be dependencies that go both ways. Naturally these should be eventually cleaned out before libcommon can, at some point in the future, become an independent shared library.


If you use `BlazeTools.parseIdentifierName` in this way, that is good to know -- can you point to the code?  You are probably the number one external user of this API, so I don't want to argue theoretical points over practical ones.

That said, my thinking was that `@index` is a Handlebars/Spacebars concept, and would not necessarily be part of another template language.  JavaScript expressions are a whole 'nother thing, and we would have to find another way to access `@index` from a JS expression.  I wasn't thinking of there being a concept of a "Blaze identifier."  This helper function parses a JS identifier, and changing it would be a breaking change to the blaze-tools API.  At the very least, another template language that decides to expose `@index` as `@index` can opt into it, or just allow `@` anywhere in an identifier, or whatever it wants to do.

Spacebars happens to base its concept of a valid identifier on this helper that parses valid JS identifiers, and in retrospect it wasn't a very principled decision.  Handlebars lets you write `{{foo-bar}}`, for example, while in Spacebars you have to escape it as `{{[foo-bar]}}`.  Why this incompatibility?  The only reason, I think, was my hope that we could allow JS expressions there in the future.  However, it looks like that's not going to happen.  There are too many other syntax differences between Handlebars expressions and JS expressions.  I wanted to reserve parenthesized expressions in mustache tags to be JS expressions, but now we have implemented nested Handlebars expressions instead (I was sort of convinced/outvoted on that one).

Slava's original code also had a bug: instead of allowing `@index`, it allowed identifiers containing `@index` as a substring.


rax retry tests


How did you catch that? Code review, test case, or firing assertion?


One thing I found myself wondering here is: what is the pool argument used for, given we're comparing two strings?  The answer is in the source for `pool_evrcmp_str()` which has some conditionals around epoch handling.

If we assume that the pools have identical settings for that value, this should be safe.


@spb-yuriy

Upstream patches should be cherry-picked with git from upstream, not copied as text
```
git cherry-pick <git commit id>
```

And even when doing a cherry-pick for one or another reason, supply a link to email in upstream project mail list that explains patch in question.

Please resubmit with rewritten patch.


I hesitated to add it, but you are right!
On May 13, 2015 17:12, "Ricardo Gladwell" <notifications@github.com> wrote:

> [image: :+1:] good spot, this commit is perfect except you didn't include
> the URL for the original ticket.
>
> --
> Reply to this email directly or view it on GitHub
> <https://github.com/WonderCsabo/m2e-android/commit/5922d43c19ae3ad7b16188db64e338ea26d3920a#commitcomment-11178089>
> .
>