@mib32 хорошая идея, + можно добавить возможность загрузки и редактирования кастомной иконки в категории.


LGTM


I've been using msysgit bash. The new files were undoubtedly shown;
it's a lesson for me about the pitfalls of selective reading... :crystal_ball: :eyes:


shizzle


@gonzalovilaseca So you don't have to override the `getTranslationMethod` in every extended entity.


the strncpy() problem. I wish I could use strlcpy() instead


tiens y'a des gens qui lisent ces changelogs :) héhé.

Je me suis rendu compte que je parlais jamais d'écologie sur ce blog, du coup je l'ai tej ;)

Biz !


I will add whitelisting for the template rendering and add a separate PR for it.


You can use `git reset -p` to only reset parts of a commit, it shouldn't be too hard since all of them are on separate lines. See http://nuclearsquid.com/writings/git-add/ for explaination, only it's the other way around, since its reset instead of add. If it doesn't work out, I can do it myself when merging as well.

As far as the pluralization goes, this is fine, the API is going to change quite a bit in the next release because of the refactoring anyways.


@cdaguerre Now you have to do it if for example you use a custom Product class, but want to still use Sylius default ProductTranslation.
With `get_class` you only had to override it if you wanted to use your custom Translation class.
Don't know which approach is better though.


I'm not sure why this is requird... Aren't the strncpy(ext_ip_addr, "0.0.0.0", INET_ADDRSTRLEN); enough ?
also INET_ADDRSTRLEN is 16, which is enough to contain the '\0' terminating char


It's crappy in some part, i advised you.
Thanks for caring about the draw lib.
Since rxTools uses different entrypoints, TOPSCREEN2 is the second framebuffer, in case we can't detetch what is active, like in mset.


sb


Hmm, this doesn't seem to work correctly. In a multi-module project, I get
```
Resolving with repositories:
 - maven: http://mavenrepository/artifactory/libs-release
 - maven2: http://mavenrepository/artifactory/libs-release
 - maven3: http://mavenrepository/artifactory/libs-release
 - maven4: http://mavenrepository/artifactory/libs-release
 - maven5: http://mavenrepository/artifactory/libs-release
 - maven6: http://mavenrepository/artifactory/libs-release
...
```
and that goes on for several pages (maven1-maven78 and multiple repetitions of each of those too). I guess, comparing using `equals` is not enough here. This isn't going to be fun I'm afraid.


Если главной целью задачи является добавления отсутствующих методов в логгере между enb и mock версией, тогда почему код разный? Сюдя по оригиналу, здесь не хватает проверки опции `hideWarnings` aли так задумывалось?) https://github.com/enb-make/enb/blob/master/lib/logger.js#L83


что это даст?


@gonzalovilaseca To my understanding `get_class()` returns the name of the class in which the relating call is defined, unless you call `get_class($this)`, so you will always get `Sylius\Component\Translation\Model\AbstractTranslatableTranslation`, unless you override it in every extended entity. Refer http://php.net/manual/en/function.get-class.php#example-5514.
Am I wrong?


Also, this optimized code for the splashscreen does not work, it messes up the image.
I'll push some commits today


The client retains ownership of variants that weren't successfully saved. That is why we wait until after the `emer_circular_file_save` call before ref sinking and unrefing these variants.


missing header.


Shouldn't stay UTF-8? Npp itself can't open this file correctly with this encoding.


dang it, that's what I get for being excited and releasing before you reviewed =)

Do you think we should still collect all repositories? #2 votes yes, and #30 votes no.


Unfortunately that's not sufficient as accessing this configuration value will now fail if it wasn't set: https://github.com/Graylog2/graylog2-server/blob/2adeded1de0baadfb4c6671ec3a401080755ad58/graylog2-inputs/src/main/java/org/graylog2/inputs/transports/AmqpTransport.java#L147


LGTM


`assert_circular_file_is_empty()` calls `emer_circular_file_read()` and then asserts that there was nothing to read.


this wasn't a good idea.. it is not portable when scram arch changes... I guess eval runtime should work too


Please double check that we can safely remove the two checks above.


@cdaguerre you're right. If we have static method we should use ``` get_called_class ``` in order to get name of the calling class. @gonzalovilaseca see example http://php.net/manual/en/function.get-called-class.php#example-5509


Nice work!


Thank you sir :)


I think the txtValidForHostnames should also be view.gone since you are not using it...


Nie mogę dojść o co chodzi :-(


You realize that the current dandelion_white and viola in minetest_game exceeds the pots ? https://lut.im/oHxc26Q0/IuVmLqYX

I wanted our own textures for keeping the compatibility with the pots, without worrying if the patterns in MTG would change.


Agreed. Already updated for 74x in my PR. Also open for a more elegant solution.


The current implementation of graph assumes that samples are sorted by time (ascending).


Shouldn't the `#{host}` interpolation be `Regexp.escape`d?


83811490-Fault_DirtyAirFilter (rongpengzhang) - Win64-Windows-7-VisualStudio-12: OK (1681 of 1681 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)


Good point!


if you use breakpoint = image._width, why don't you use that here too?


Hi, this is not that easy in that case. The shasum verification is running on the workstation (local_action:) and afaik there is no sha256sum command on OS-X (which was the reason to go back to shasum command in the first place). I'd love to ignore that, but there are too many guys out there working on OS-X, at least two in my team, which means this PR would break the role for OS-X users. I need to check back with the other guys from the team how to implement a solution suitable to any workstation os, like lookup plugins to make decission for command to use more dynamic.


trailing whitespace


Great find @chrismccord, thanks! We can remove this comment also.


@marciomeschini
i think the #elseif is incorrect here, right? it should be #else
please fix


insert space after comma


This raises an error on my Redmine 2.3 install:

    NoMethodError: undefined method `matchers' for Method:Class


Never considered that (obviously), but I agree.


This class seems to be in the wrong module. It is used by the generated code, not by the compiler that generates the code, so it should live in the module that implements the SPI, not the module that defines the SPI.

The `CompilesConversionUtils` should also move to the module that implements the SPI.


Yes that's what I figured was causing the db lockup, however I thought it was because I was not closing before moving to cachetexture when I was doing a query for the fanart - I've been rerunning the scan on my test machine and the initial sync is not any slower. However, I don't see the db lock up on 14.2... it seems to be centered around Isengard. Marcelveldt, please correct it as you see fit or remove it. Thank you. :)


I'm not a fan of the name of this class.


The same comments that I had on "`CompiledNode`" applies here as well...


missing license header


I'm not sure what to do about this. It was done automatically by Qt Designer—I guess when I promoted teHistory.
Shall we leave it like this... or?


Yeah, I don't know how the whitespaces ended up there. I thought I used tabs :) . Anyhow, after double checking I see that those whitespaces (from my commit) doesn't work for me either.


@razor84 Just adjusting commits and matching up to their TeamBliss-LP counterpart. Thanks for confirming


fixed


@vhuber sorry, what do you want me to indicate?


The thing is, we reference "blueimp" in all sorts of places, not just here. When we clean things up we need to make sure it's cleaned properly. The following needs to be updated:

* https://github.com/umbraco/Umbraco-CMS/blob/dev-v7/src/Umbraco.Web/Editors/MediaController.cs#L432 ... is this still required?
* This still says it uses blueimp: https://github.com/umbraco/Umbraco-CMS/blob/dev-v7/src/Umbraco.Web.UI.Client/src/common/directives/imaging/umbimageupload.js ... still required/used?
* Still says it requires blueimp and options are still named blueimp: https://github.com/umbraco/Umbraco-CMS/blob/dev-v7/src/Umbraco.Web.UI.Client/src/common/directives/imaging/umbimagefolder.directive.js#L18, https://github.com/umbraco/Umbraco-CMS/blob/dev-v7/src/Umbraco.Web.UI.Client/src/common/directives/imaging/umbimagefolder.directive.js#L48


@michalmarcinkowski Are you saying that the fact that the method is `static` changes the behavior of `get_class()`?
I think `get_class()` returns what `__CLASS__` gives and it seems it can't be used with `get_class(self)`or `get_class(static)`.


Could you elaborate a bit more on that, the problem is `vector.hh` pulling `type_utils.hh` as well, I guess? We could for instance at least pull `vector.hh` in `type_utils.hh`. That would not give a nice separation but at least the user gets `is_vector` after including `type_utils.hh`, if that was desired.

Is this important enough to investigate how to break the circle?


@wkyoshe How can I set the option to true of false ? Do I have to open the CmakeLists.txt ? Can I do it another way ?


> 0 が良い


What is a "ternaryOr"?


What is a "ternaryNot"?


With the work on WASM (WebAssembly) we probably should ship NaCl :-) so no worries about that


En fait je voulais corriger des petits trucs dans ton dernier billet au départ :-)

des bizitou !


I've cleaned up quite a few of the event registrations to ensure they are cleared in the last couple of months, not sure if there are any more lying around but we need to fix them all up. Moving forward, we need to be real careful to ensure we unbind any event


Ah nice!


Hmm, I can't think of any _offline_ (when network is not available and content can't be fetched from remote server) capability that Service Worker can provide other than cache by any mean, while its window-less nature is really powerful :)

But yeah, you'll be there, so just bike-shedding :)


A web application runtime ?
A runtime for web applications
A runtime for packaged and hosted web apps ?

HTML5 is so two years ago


Dis line sucks.


OK, I will consider that


lgtm but im no bash expert


I fixed on my side with eval runtime, please removed it from your PR (otherwise I'll get conflicts)


yes! :)


Why ? (Now conflict with the throwing mod if we activate the obsidian arrow)


What was the actual conflict?


All is fine here, standard monochrome 8x8 font is extracted and drawed with foreground and background colors with transparancy support. The only thing can be changed here (along with the proper RGB order) is to use a u32 color high byte for transparency flag. Even software translucency can be add but this will be slow until we use a direct framebuffer access. Not of much use for service software though


you can add a line space her (not sure about the exact formatting conversion for this)


Have you fixed the type cast for TOP_SCREENx with (unsigned *) in the splashscreen?


The problem is that you're opening and closing a db connection for every image that gets added. That's not good and very slow. If you really want this in you should build a list of all textures that need to be refreshed and then refresh them all at once.

The other issue was that you're just removing the entry from the db but not cleaning up the old cached thumbnail so this will cause a mess and a very large thumbnail db.

For now I removed it from the code. Only if we think this is really needed we can add it in the proper way. What I did add is the cleanup in the full texture sync method, so if users have any complaints about seeing wrong images or something they can correct this theirselves by running the full texture sync. That one will now completely wipe the texture cache and refill it with the urls from the video and music db.


I checked in my changes just now btw


Ok, but the broomtick must be slow... (because it's a incredible easy way to climb and avoid monsters...)
and the recipe needs to be more difficult (it's a magic item after all)


Ohhh I see what you mean now! I thought where I made the addition was skipped during the initial sync and only applied for incremental sync. Well thanks for the info!


I'll have a look at the blueimp response on the server - as mentioned, it hasnt been touched.

None of the directives you refer too are referenced anywhere as far as I can see. To my knowledge, we only used file uploads in very few places, which is the folderbrowser editor, media dashboard and media picker dialog, the rest is handled as normal upload fields.

I'll get those directives removed as they are not used anymore.


weird, I ran `mvn license:format` and ran the build without failure?


I'm ok with this as well, for me ```Lifecycle-oriented methods``` is not really clear who has these ```methods``` (I turn off what I know about Messages when I read presentation to have side view :))


good!


suggestions?


great, but the changed lines are in the api.lua, you need to write a small comment because of futur merge


code layout, there is a space missing after the comma.


@flavour https://github.com/flavour/eden/blob/master/modules/templates/SAMBRO/gis_layer_feature.csv#L2 name shows attribute error...Why is that? I see no name field in alert too. took me a while to figure this out...

Filtering alert based on hierarchy is already there https://github.com/flavour/eden/blob/master/modules/s3db/cap.py#L387


:+1: :metal:


They're all bug fixes though, and I've been trying to test things moreso since I've had more time to work on this.


Three-valued logic version of 'not', returns `true`, `false`, or, `null`


Yes. Do you like it?


Очень полезный комит :)


Three-valued logic version of `or', returns `true`, `false`, or, `null`


May be worth including a link to http://eslint.org/ and possibly JSLint...


более красивый вызов - `this. logWarningAction('deprecated', target, msg);`


This looks good


Is it not better to provide the visitor directly in the constructor of AstScanner?


I'd suggest to revert this line as *MS Shell Dlg 2* is a Windows font and I'm not sure about the fall back capabilities of Qt on OSX and Linux where this font family doesn't exist.
I haven't had this kind of issue before, so I have no idea where it came from. I use Qt Creator most on Linux, extremely rare for editing on Windows.


If ```ref``` is not in ```solution```, this traverses the search tree twice: Once for not finding the key, then for finding where to insert (after which the tree has to be rearranged).
It would be cheaper to aggressively insert, and check the return value to find out whether an insert actually occurred or not.


关于 PDF 格式

[Sumatra PDF](http://www.sumatrapdfreader.org/free-pdf-reader.html) 这款阅读器也值得推荐，建议博主补上。


On ubuntu 14.04, with apache, the 2 default virtual hosts are : 000-default.conf and default-ssl.conf
So, this code did add the cookie to the SSL VHost only, leaving the HTTP defaut unmodified


That won't give real feeling of a Demo! :) But yeah, any way is good, deployed hosted version just for your convenience (don't forget to enable SW on device if you want to run it there).


I think here is buffer over flow, since it visits **format->params.tests[1]** while maybe the **size** of **format->params.test** is only **1**.


TeamCity Wordpress :: CorpSite :: CorpSite-Staging :: Launch Environment [Build 1](https://bsr.ict.je-labs.com/viewLog.html?buildId=29559&buildTypeId=Wordpress_CorpSite_CorpSiteStaging_LaunchEnvironment) is now running


I'm not sure why this seems to have changed, it may have something to do with my `init.gradle`, I'm still investigating.

I'm also not sure about the repository thing. I have to play those things through in my mind. I guess, if we're uncertain, we could always make it configurable.


This is really too verbose. It's ok if you leave it like this for this project, but for the next time you should implement ```VectorType::operator+``` and its relatives, so that you can write code like
```
return p - n * ( 2 * (p*n) / (n*n) )
```


There's a link in the references at the bottom pointing to the ESLint github, but yeah I will add links for JSLint and JSCS too.


This error message could be more specific since we know that it is related to ico format.


qwreqwr


@zyx0225 just two smaller comments


Good to know :) In my native languages it's ```ответственность``` or  ```адказнасць``` - so much easier to not mix :)


So this will do the trick:
/etc/apache2/sites-available/*default*.conf


added in a later commit


``get_class()`` returned the class where the method was declared, that's why the method was declared in ``Sylius\Component\Product\Model\Product`, I see it's been removed from there now.
That way, if you customised the Product class you didn't have to deal with translations, now you have to redefine ``getTranslationClass`` for it to work.
The old was less intrusive IMO.


Yes, I know, I did modify the file manually.
Just be prepared for bugreports from people installing it on Ubuntu with default configuration


If you use AppCode by Jetbrains. https://www.jetbrains.com/objc/ <https://www.jetbrains.com/objc/>

> On 24 Jun 2015, at 04:11, Wenchao Ding <notifications@github.com> wrote:
>
> Why ".idea" matters?
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/AntunS/FSCalendar/commit/bd94c88f0fbe9540d4b90a1ac7b658b4a1528afd#commitcomment-11829415>.
>


this was just a question, not a suggestion :-) i will change the path upstream.


<?
class sampleBitrixUtils
{

}


@ypadawann  gitを使っているので、コメントアウトは基本的に使わないようにしましょう。


nit: looks like it should be separate line, not as the part of the list "We want to reduce this latency through <b>two</b> ways" :)


OOps!  Well I can fix that easily enough; this was actually in response to a glitch in HDX caused by specifying flowers already in their pots.  Figured this was the best way, but I didn't think to look at those two flowers.


[test](http://73dcab3a.ngrok.io/w/3oTplHdpr3FHk9PAEsdVE0Kz) test tasks


@ypadawann bodyへの指定は`_all.scss`に書いてあるのに、htmlへのサイズ指定が`main.scss`内に意図はありますか？
後々の混乱をさけるためにも、クラスではなく、タグそのものへの包括的なスタイル指定はどこか一つのところにまとめておく方が良いでしょう。


>INET_ADDRSTRLEN is 16, which is enough to contain the '\0' terminating char

You are right, I will revert the `+1` part.

The problem lies in `strncpy(ext_ip_addr, use_ext_ip_addr, INET_ADDRSTRLEN);`, I am not sure whether `use_ext_ip_addr` is size limited or not, I was speculating it could have a length of 16 which will not terminate `ext_ip_addr` correctly.


@Tobion but browsers would treat ``///foo`` as a scheme-relative URL, not as an absolute path, which is an issue


Check out https://github.com/sindresorhus/electron-debug ;)


I just grayscaled the Cryotheum Dust texture from Thermal Foundation :p

I can't put this texture into a release


It think we could send just `options[:ico]` for validating ico (if ico present)


新增粽情端午节项目


hmm, link doesn't work :)


for #1, need to sync example file and the usable file


@ypadawann コミット内容とコミットメッセージが雑すぎます。
ざっと見た感じ、少なくとも

- テストの実装

- admin用テストファイルの中身を移動？

- alertをtoastで置き換え

- テスト用？ のコントローラーの動作を修正

の4つの作業を行っているように見えます。
それぞれコミットを分けて、ちゃんとしたコミットメッセージをつけるようにしてください。


On linux, nothing much can do, eh?


I think he should be indented 4 spaces.


@erik90mx you should make the spacebar move towards the ~~spider~~ cursor, and double-arrow-tap in the direction of that arrow (e.g. press W twice, and you will dodge up) (As discussed in the issue)


I'd maybe increase this introduction slightly.  Something like "We've been developing the [http://www.skystore.com] (Sky Store) site for the past 18 months..etc.  ...a repsonsive, large angular application with an embedded Silverlight player...etc.

I'm sure you can phrase it a bit better - but I quick introduction will help set the scene.


改变的地方


Thanks for submitting this PR.
Will not integrate it, because it is OS-specific (UNIX). SLRE is cross platform.


I have forgotten to change the #define guards in http.h. It was the same as in time.h. Therefore I got a conflict in parse.cc and I uncommented the time::duration unit test.

Now it is solved.

Von: Matthias Vallentin [mailto:notifications@github.com]
Gesendet: Mittwoch, 24. Juni 2015 10:05
An: mavam/vast
Cc: Robert Schmidt
Betreff: Re: [vast] fix conflict with time::duration unit test (d662950)

What was the actual conflict?

—
Reply to this email directly or view it on GitHub <https://github.com/mavam/vast/commit/d662950723988880d346651f5765608115970abb#commitcomment-11832795> .


A really like this article - relevant and interesting.  It may be worth dropping in a few more links, e.g. [http://guide.agilealliance.org/guide/pairing.html] (pair programming)


thanks, should be fixed with a6d2b2844c


*$isXtml* (lines: 605, 616) should be *$isXhtml*.


TeamCity Wordpress :: CorpSite :: CorpSite-Staging :: Launch Environment [Build 1](https://bsr.ict.je-labs.com/viewLog.html?buildId=29559&buildTypeId=Wordpress_CorpSite_CorpSiteStaging_LaunchEnvironment) outcome was **SUCCESS**
Summary: Running Build time: 00:14:16


this isn't going to work in general...


Makes sense; thanks for introducing me to `g_build_filename()`.


glibc is the problem :(


@ypadawann ミーティングの時にチラッと口頭で言いましたけども、keydownイベントでキーコードを判定するのではなくてformのsubmitイベントを使った方がシンプルです。


I think we should instead add a ext_ip_addr[INET_ADDRSTRLEN-1] = '\0'; just after the strncpy()
it is the usual pattern, so it is easier to understand and maintain


Sorry i didn't mean to push that!


how about specifying whole team's email ?
e.g. easyphi-int@andrew.cmu.edu for now


shipt it


OSM instead of OMS --> search and replace because there is at least 3 times this error


@yehudasa good progress here on my side. I am finishing/testing the whole feature. I hope to update the pull request this Friday 26th.


ok


Wow, that really must have been one of my brighter days^^ I think some local evluations in dune-gdt make use of those comparison operators...


@liancheng this sql will get the error
···
Caused by: java.lang.ArrayIndexOutOfBoundsException: -1
···


I think it's better to make it clear that it assumes there's a user called "daemon" in the rootfs (not in the host os).


@mosra: Could this work? It fixes the only warning output by i686-w64-mingw (gcc 4.9.2) when compiling magnum


It seems this code is broken... I had to rollback on my setup, because dns_get_record does not work correctly with DNS_A + DNS_AAAA. It can be reproduced with the following code sample :
```php
	print_r(dns_get_record("www.asahi-net.or.jp", DNS_A + DNS_AAAA)); # FAIL
	print_r(dns_get_record("www.asahi-net.or.jp", DNS_A)); # OK
	print_r(dns_get_record("www.google.com", DNS_A + DNS_AAAA)); # FAIL
	print_r(dns_get_record("www.google.com", DNS_A)); # OK
	print_r(dns_get_record("www.google.com", DNS_AAAA)); # OK
	print_r(dns_get_record("flets-west.jp", DNS_A + DNS_AAAA)); # FAIL
	print_r(dns_get_record("flets-west.jp", DNS_AAAA)); # OK
```

Only the middle host (which has both v4/v6) works. v4-only, v6-only is no good. Conversely, limiting the record queries to one type makes it work. It seems like this is due to the fact that dns_get_record fails the whole call if even one query fails :(

Seeing this behavior on pfSense 2.2.2 :
```
[2.2.2-RELEASE][root@pfsense.localdomain]/root: php -v
PHP 5.5.23 (cgi-fcgi) (built: Mar 27 2015 20:40:34)
Copyright (c) 1997-2015 The PHP Group
Zend Engine v2.5.0, Copyright (c) 1998-2015 Zend Technologies
   with Suhosin v0.9.37.1, Copyright (c) 2007-2014, by SektionEins GmbH
```

DNS_ALL also fails. DNS_ANY succeeds but is NOT a recommendable way to fix this issue.


@cdaguerre I'm saying that if you want to get calling class name in a static method you should use ``` get_called_class ```. ``` get_class ``` method accept object, so you can't pass ``` self ``` to it. Method call ``` get_class() ``` returns ``` __CLASS__ ```.


The function parseStringAsNum does not work properly in matlab 2014...

Why not use

function number = parseStringAsNum(stringIn)

number = str2num(stringIn);

if isempty(number)
    warndlg('could not parse all plan parameters');
end

Which works perfectly fine for me?! str2num will return an array if all characters in StringIn are numeric (that includes floatin point numbers). If there is a letter in stringIn it will be empty...


Hola Miguel Ángel, te he visto muy activo en el foro del curso y me ha parecido que podía ser interesante seguirte. He perdido un poco el ritmo con el curso, aunque espero seguir avanzando poco a poco.

En cualquier caso, y ya que estaba mirando el README, creo que estaría bien que apuntaras en él el enlace a la aplicación en Heroku para poder verlo en funcionamiento.

Un saludo.


Yes, I've updated the code to do so. Reload the PR if you didn't see it.


Also `android-7` seems to have disappeared from the Android SDK Manager. Earliest API version is now 8.


Show hangout request...I'm contacted u 2 days ago.


why was this change needed?


can we simplify these `match`es? I would suggest to use the following:
```
value match {
    case StringSeekRange(LowerBounded(lower)) => ....
    case _ => Iterator.empty
```


Yeah, I'll try to cook up some textures tomorrow.


I did not disappear, just marked as obsolete, so you have to enable it with *Packages -> Show obsolete packages*.  But i agree we should update to API 8.


Ouch yep, 4 spaces should it be. The menu item is moving to CommitInfo, but thanks anyway for spotting this on!


lgtm. thanks


Game Battle Royale


Better view to translation


je ne comprends pas pourquoi on rajouter ces fichiers.


The minimum frequency itself didn't cause any system issues that I encountered, but it was the cause of an error in Synapse with the minimum frequencies not displaying. Changing the min to 192000 restored the ability to tweak this in Synapse again.


s/indizies/indices/

seems it's the "indices" in german: indizes [0] =)

---
[0] http://www.dict.cc/englisch-deutsch/indices.html


@chelmuth: can you push this one? It fixes rasperry pi again


There are multiple ways to do client-auth: you can do it with a client certificate or with a JWT token for example. Checking that *the client has the right name* is an authorization concern and yes, this API is the base for implementing authorization logic.

As for the `final`, I call it base context in my comment only and its probably unfortunate... The design I have for extension (and that I will use for auth related metadata) is by composition/decoration as opposed to inheritance.
See the `chained` field in the C struct: https://github.com/grpc/grpc/blob/master/src/core/security/security_context.h#L45


you said that the other day lol


thanks :)


throw new GeppettoException(e1);


練習


@hygkui 因为想在借还书操作完毕的时候清空搜书页面的信息。


ah, the client ID ... that's something I have to (manually) add ?


Becuase, libvirt-sandbox-util.h already defined in sandbox.h


close [ddddd](http://73dcab3a.ngrok.io/w/2oTplHdpr3FHk9PAEsdVE0Kz) test tasks


OK, thanks, was fast


yep and I explained why in the PR description ;)


Hey @fblundun - this looks good, but I think we should fire an event for each jobflow failure, including bootstrap failures. It's useful data...


Can do!


An OPTION is define as follows in CMakeLists.txt:
option(<option_variable> "help string describing option" [initial value])
 - The initial value here could be ON or OFF.

At some point in our project we may need to change this initial value to ON or OFF and vice versa.
We won't need to open our CMakeLists.txt every time to reset this value.
 - For example for our USE_SHARED option defined as:
        option(USE_SHARED   "use a shared library" OFF)
- We might want to use a static library at some point.

Below is one of he ways this can be done in the command line.
 - In command line type(we have to be in the CMakeLists.txt directory)
- ```sh
        ccmake -D
        ```

Then a window like this where you can edit the initial value will appear(with all options and how to reset)
      ![Image](/Users/user/Desktop/terminal.png )


@orcc I forked this repo about a month ago because we started using the scan build plugin.  Somehow I got notified of this pull request, and started looking at it.

I must admit
1. I'm somewhat lost in understanding the workflow (didn't note that this is so old already)
2. My wordings may be a bit terse - I'm used to communicating the bare essence when reviewing - I'll try harder to write nicer proze

When reviewing code changes where persistent data is involved, I _am_ quite concerned with terminology and consistency, especially in the user visible interface.  This casing issue will for sure cause much cursing since it violates the principle of the least surprise.

That's why I thought it very important to not just sit back, and spit it out.

I understand now that it's too late after the facts...


it's convention that we use in the project. Look at other models.


@media가 가끔 나오는데 뭐죠? instance variable인가요?


what is it?


I don't get - what is it and what it means..
Why we need to commit temporary client settings and refrences..


git -b GED-5.0 --single-branch gives me - Unknown option: -b

So I do git clone https://github.com/MatthewBooth/JFLTE-GPE ROM and all works fine until I build... Then

thedude@thedude-VirtualBox:~/android/JFLTE-GPE/ROM$ ./build
Select to either BUILD the ROM, or just the modified apps/jars
1) Build ROM, Kernel and Files	4) Build Modded Files
2) Build ROM			5) Build Addons
3) Build Kernel			6) Exit
#? 1
arch/x86/Makefile:136: CONFIG_X86_X32 enabled but no binutils support
Makefile:652: Cannot use CONFIG_CC_STACKPROTECTOR_REGULAR: -fstack-protector not supported by compiler
scripts/kconfig/conf.c:718:1: fatal error: opening dependency file scripts/kconfig/.conf.o.d: Permission denied
 }
 ^
compilation terminated.
make[4]: *** [scripts/kconfig/conf.o] Error 1
make[3]: *** [silentoldconfig] Error 2
arch/x86/Makefile:136: CONFIG_X86_X32 enabled but no binutils support
Makefile:652: Cannot use CONFIG_CC_STACKPROTECTOR_REGULAR: -fstack-protector not supported by compiler
scripts/kconfig/conf.c:718:1: fatal error: opening dependency file scripts/kconfig/.conf.o.d: Permission denied
 }
 ^
compilation terminated.
make[4]: *** [scripts/kconfig/conf.o] Error 1
make[3]: *** [silentoldconfig] Error 2
Building JFLTE-GPE stock kernel

Creating config

Building kernel

./build_kernel.sh: line 71: schedtool: command not found

zImage not found, kernel build failed

Kernel failed to build


1: M111 unrecognized component name: elasticsearch


42: D103 Missing docstring in public function
53: D103 Missing docstring in public function
59: D103 Missing docstring in public function


Same goes if other subgames use large flowers textures, we can't be sure of that. But in hosting your own textures, there is no screw up.


From future? :smile:


好多乱码


I suggested to not touch the existing API and you agreed with that. It means that we need to wait for response from customer regarding the API.


Alright, the 1..n issue was caused by my {{init.gradle}}. The repetitions are still there though:
```
Resolving with repositories:
 - maven: http://mavenrepository/artifactory/libs-release
 - maven: http://mavenrepository/artifactory/libs-release


start


Je kan eventueel ook al je submodules op deze manier updaten: `git submodule foreach git pull origin master`


別の環境で試したら動きました！
勘違いだったようです。
ごめんなさい。


**로컬 커밋**에서 추가된 내용입니다.


To avoid the hash when it is not a search page (ask by client). I think its better not having it on every page. I am adding a comment.


The intrinsics also work on Win32 x86


The hash should only be here when we need it but we should not remove it on a page that does not belong to us and it should not appear if there's no search done etc..

Also location.replace do a reload in this case?


Hm... this are the necessary configs for our WCF but in any case I don't like that those configs a specific with predefined values which we want to change in run time.

:exclamation:  I am going to investigate other way of BaseClient generation.


Now changes are okay


Forgot in my commit comment ... but I changed alignment for some elements in the problem table ... left aligned the passive and active check information.


I prove to understand what this is


So I'm just testing this new file upload and it though it has less files, it also has less functionality and error checking.

* It no longer has progress bars, no indication of upload time - this used to work and was quite nice
* No longer has error checking for uploading flies that were too big


Closes #26


@yangshun Oops D: I think I thought it could work without the permissions because I was able to post and stuff after commenting it out. Sorry!


it should have loading bars, these have been in there since the beginning, so either a bug or a styling issue.

I'm looking over that old hacky way we detected large files and should be possible to port into this so we can show a proper error in those cases


will fix this css


So what we need to do is never overwrite any already present # in the page but rather reuse it or add it if not present. And we should not overwrite completely the hash but only add to it our prefixed properties like as_q= (algoliasearch query)


> if you disable assert() via NDEBUG you will run in strange effects and crashes.

Danny, i am not sure i follow you. by inspecting the code path, we are always safe here to dereference `s`. but it will take a human developer a while to figure out this. so IMO, an `assert(s)` would serve as a comment actually.


Yes indeed, I was porting the class directly from C++. I'll change them to Properties


yeah, it would. the format wasn't considered stable API, see the comment at the top of the file


Actually code seems to be much more readable after corrections


Then you'll need to do that change in another commit


I see the repetition too in both versions. I pushed some changes so it only gets logged once when the `Resolver` is created, instead of each time a configuration is resolved.


# awfw


good
useful


- acvasce
- bvhwv
- adjbvw;rv


Interesting 0.9 doesn't repeat, so it that should narrow it to a few commits.


This breaks compilation on ghc 7.8 (and probably below).
The Eq instance for Doc is not present in older versions of Text.PrettyPrint.


Got it, my eye didn't catch it in the diff. :smile:


the progress bars are actually fairly 'new', they were added in recent 7.2.x versions. They used the directive: umbImageUploadProgress which is no longer referenced.


Changed event priority to NORMAL as it shouldn't have been on MONITOR since it cancels the event.


The test did work on my machine before I checked it in but requires setting up NoSQL data store. Just disabling the project is enough for now.


Well, it's not code in the sense that it's just a metric on our code.
The setup is super recent 14 of may.

It's not causing any issue, the issue I was refering to here is that travis is highly unstable lately.
A lot of build time out or fail for no reason, so instead of just sending the info from one of the build(5.4).
I send the info from all the build hoping that at least one will not fail.
That way scrutinizer or any other third party service has more "chance" to get the info.

That said it would make more sense to to the same for coveralls. It would be simpler and more readable.


@Sjors @abhisharma2 We'll want this localised, what is the best way to utilise the strings file here? Good to know before I implement these help boxes in other places.


NameError: name 'run_for_collection' is not defined
Он не распознает функции из rnafold.py


いや、ひょっとしてZabbixのバージョンによって違う?

2.2.2では動いているが、2.0.3では動いていない。


xD thats why i did i commited such a useless commit! lol @hoholee12


This should be start with a lowercase letter, because obeoWrapper isn't instanciated with the ```new```-keyword:
```
oboeWrapper
```
There is also a eslint rule:
http://eslint.org/docs/rules/new-cap


These tweaks are supposed to be done in your relocation script, like...
```python
  relocator.set_forced_configuration_value("MINLNK", 4)
  relocator.set_forced_configuration_value("MINOBS", 1)
  relocator.set_forced_configuration_value("MAXNGH", 20)
  relocator.set_forced_configuration_value("MAXOBS", 100)
```


Lol no worries. I think we can still post because we gave the app the posting permissions before and it stayed even though the permission was removed.


@mehturt, ok then great!


I've implemented this and it works great, thank you for this!, but I noticed that when I disable a step that's already marked 'done' it does get skipped, but doesn't get the class / aria attribute 'disabled' anymore.
In my setup there's a checkbox that determines if a step gets disabled or not. I did it like this:

```
$("#checkbox1").change(function() {
        if($(this).is(":checked")) {
            $("#wizard").steps("disableStep", 2);
        } else {
            $("#wizard").steps("enableStep", 2);
        }
    });
```

If you first check the checkbox the step gets disabled. If you than go back and uncheck the checkbox, the step is enabled again. If you fill in the fields in step 2, it gets marked as 'done'. If you than go back again and check the checkbox again, you do skip step 2, but in the list of steps it is still marked as 'done' instead of 'disabled', and it's editable (so not disabled).

Do you have any advice for me how to keep the step disabled in this scenario? Thanks!


Looks good, will merge the request. Might refactor some to align with rest of the project.


you have to fill up now, good job continue


excellent it is what I want, you preserve encapsulation, the rule of C++ :+1:


The logical or causes a condition to happen where the develop-branch name is output twice as opposed to only once. This causes issue #72 .

```
Which branch should be used for bringing forth production releases?
   - develop
   - master
11:25:37.177: master
Branch name for production releases: [master]
Which branch should be used for integration of the "next release"?
11:25:37.440: develop
   - develop
11:25:37.584: develop
Branch name for "next release" development: [develop]
How to name your supporting branch prefixes?
11:25:38.148: feature/
11:25:38.225: release/
11:25:38.334: hotfix/
11:25:38.413: support/
11:25:38.482:
```

So I guess removing this logical operator, the above issue should be fixed. Is it possible for you to test this?


I do that in #2 by `current_event["event_id"] = str(event.resource_id)`


Should be straight forward to support both SEED and StationXML.. :)


How is this supposed to work? We want to call is_recursive on all nodes that are connected to the current node's sources. the way it is implemented now, it will check the own sinks, call is_recursive on itself and ultimately _always_ return true after the first recursion.

The purpose of this function is to determine if there are any cycles in the graph which we don 't want.


-wangjingjing.com
+Mylove xiaomo!


회사에서 less 쓰고 있는데 스타일러스 쪽이 기능은 같고 문법은 더 간결한 느낌?


update [test](http://73dcab3a.ngrok.io/w/SWcWS2gGmu9Kcp6Q59IZBSdt) and #2oTp test tasks


There is a plone5 branch already


Exactly :) Would you like to bite the bullet and implement that change? Also, keep in mind that the first release of Restcomm Android Client SDK is coming soon that will hide all the low level details and expose a higher level API that will allow you to utilize JAIN SIP much easier. The SDK sits on top of the low level API that you are now using.


I'm guressing the `sing System.Text.RegularExpressions;` is not really needed here.


update #SWcW and [test](http://73dcab3a.ngrok.io/w/3oTplHdpr3FHk9PAEsdVE0Kz): [Integration] GitHub Dialogue UI Design


update [test](http://73dcab3a.ngrok.io/w/SWcWS2gGmu9Kcp6Q59IZBSdt) and #3oTp: [Integration] GitHub Dialogue UI Design


What's the point of this unnessecary state variables?  Please fix/override MinUsableSize instead of adding clutter.


ok, thanks. Also i have to add autosleep... soy maybe tomorrow i will check all.

Have a good day !


Ah, yeah, it was in the initial commit, but I didn't take it out. Just leave it there for now. :D


1: M111 unrecognized component name: base


sorry, my mistake, had not read correctly


qqqq


close [test](http://73dcab3a.ngrok.io/w/SWcWS2gGmu9Kcp6Q59IZBSdt) #3oTp: [Integration] GitHub Dialogue UI Design2


close #SWcW [test](http://73dcab3a.ngrok.io/w/3oTplHdpr3FHk9PAEsdVE0Kz): [Integration] GitHub Dialogue UI Design2


TeamCity YaccConstructor :: YC.Core General Build [Build 459](http://teamcity.codebetter.com/viewLog.html?buildId=193913&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) is now running


Okay, I fixed it. This was introduced with the flatDir fix due to a surprisingly innocent change.

The RepositoryHandler performs the equivalence check to filter additions. This means that my original trick of using `project.repositories.add(it)` is required instead of using a Set, because a repository doesn't define an equals method.


Thanks - will update to use `jshttp/forwarded` when the other repo is deleted.


Og smid patchen på git, så den kan applies næste gang!


I just did some work to reduce the test running time from nearly 4 minutes down to under 40 seconds. I also added a `npm run test-quick` script if you just want a quick sanity check in 1 second. I think this is acceptable for now.


Causing an issue on my system
Removing  '''gridspec_kw = dict(height_ratios=[3,2,2,1])''' fixed it.
What is the purpose of  gridspec_kw??

'''
csv-out-file: V53M02_nonmoving_eval_usage.csv
Traceback (most recent call last):
  File "/home/scherers/Documents/Software/UKBB_EMG_Analysis/emg_visualizer.py", line 682, in <module>
    gui = VisualizerWindow(sys.argv[1])
  File "/home/scherers/Documents/Software/UKBB_EMG_Analysis/emg_visualizer.py", line 288, in __init__
    self.fig, (self.ax1, self.ax2, self.ax3, self.ax4) = plt.subplots(4,1, sharex=True, gridspec_kw = dict(height_ratios=[3,2,2,1]),figsize=(16,7.5), dpi=80, tight_layout=True)
  File "/usr/lib/python3/dist-packages/matplotlib/pyplot.py", line 1046, in subplots
    fig = figure(**fig_kw)
  File "/usr/lib/python3/dist-packages/matplotlib/pyplot.py", line 423, in figure
    **kwargs)
  File "/usr/lib/python3/dist-packages/matplotlib/backends/backend_tkagg.py", line 78, in new_figure_manager
    figure = FigureClass(*args, **kwargs)
TypeError: __init__() got an unexpected keyword argument 'gridspec_kw'
'''


I'm sorry, but I just refactored the graphics module as well, so it's probably hard to merge back into master.


どうもそういうことらしい。
ドキュメントには書いてないが、2.2.2ではhostidsが付いてくる。


fff


Works as expected (tested locally).


just ignore all exceptions is not a save course of action


@hoholee12 im with you as well, i wrote my first couple of scripts on my old xperia x10 mini pro, to be fair i did have a slide out keyboard built in the phone, but still coding on a mobile device is awful , no auto completion and tabbing is horrendous (mostly white spaces which caused not fondulated issues in bash scripts) and back then (2010-2011) very few apps had syntax highlighting...

@DeicPro im still a noob compared to @hoholee12 lol :) however i am learning, but ive mainly started to learn C and im very interested into the kernel ;)
Also yup i learnt from reading others code, i used to spend days on github scouring the bash scripts haha


@flavour please review this


Catch more specific exceptions, on reading this I have no idea what exception was thrown and why the exception occurred


あとこれとは関係無いけど、プラグインを動かしていて気がついた点として、プラグインを起動したとき、
queueに既に何らかのメッセージがあったとき、エラーを吐いて止まることがよくある。
exchangeProfileが終わるまで、関係ないメッセージは捨てるようにしないと駄目だと思う。


Thanx for pointing it out.

I m giving late response cause i did not open github.com last few days. Just saw your comment.
I reverted change in dscache.c file.
https://github.com/prasanna08/android_kernel_yu_msm8916/commit/3a2c7da7f461d404fe8e9c941f6214ffd5703b16


Doc update to come? ;)


neither fabspeedial nor fabtoolbar works. i don't know how they put perfectly on website but i tried everything it's not working properly. and also responsive is creating problem in mobile devices


Deployed. Thanks, Clay!


@prudhomm Indeed, I'll recheck the code


This function is no longer used, I think I write that for the very first version before writing specific constructors. I will remove it.


;-; im so outdated lel
@hoholee12 thanks for info m8 :3


The change from 5432 to 5433 should not have happened. We should rather have changed the exposed port.


TeamCity YaccConstructor :: YC.Core General Build [Build 459](http://teamcity.codebetter.com/viewLog.html?buildId=193913&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 116 Build time: 00:11:19


Aha. https://github.com/micropython/micropython/pull/1352


I have an issue related to this change due to nonexistent release 2.3 and higher in sonata-project/classification-bundle

Error log from composer:
requires sonata-project/classification-bundle ~2.3 -> no matching package found


Yes, I've improved a bit further to disallow strings to be used as keys as
well

On Wed, Jun 24, 2015 at 10:13 AM, Daniel Marbach <notifications@github.com>
wrote:

> Yes. Do you like it?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Particular/NServiceBus/commit/e8304ef8b70ee93dd0da7b9be7047295b1825ac4#commitcomment-11832902>
> .
>


It is just a set (name := "name") but I don't know exactly how to integrate that cleanly hook and source doc. If you think of something please help yourself :)


Removing the condition above is not correct - it causes that I can import the same DPU several times, having duplicities in the DPU tree


@mglukhikh would be nice to update the explanation here and in the next test, because the behavior has changed


eventのstatusもHAPI 2.0フォーマットになっていないようだ。


This is still required to run the tests, so this line should be left in the `README`.


OK. Thank you for the comment.


We would be unable to merge with file changes like this.


Add Quratulain to fam


For reference: https://github.com/jquery/testswarm/commit/57bc81ac5fe82438544dba99f807ee39ff64c6ec.


File a bug to clarify whether we want to disallow static fields and to do it if we decide so.


This exception would need to be better handled.


I actually prefer this:

```
while (list->entry)
        destroy(list->entry);
```

over the iterator. I mean, we literally fetch the first entry of the list and destroy it, until the list is empty. So lets write it as this. The safe-iterator variant kinda pretends we're leaving stuff behind, which we really don't.

But maybe just a personal opinion..


We need some general discussion area here. Using latest commits for random stuff I come up with feels kinda … dumb.

Anyway, I had a bigger face palm moment lately while examining TV/F3 stuff: I always thought the sun and chamber light direction vector in LVL files was, well, a direction in degrees * 182,044444444444 where the first value is the azimuth, the second one is the elevation, the 3rd one is unused because it is 0 in most cases and the light intensity is always 65536 for the sun… Wrong!

It's actually a position vector and the sun light intensity is the length of that vector. Don't tell me you already knew that :-/


Rename this field to `Name`


Media not checked in??


Also, update of .NET framework to version 4.6


This is too leaky. It should look something like this:

```python
def action(self):
	minions = self.game.minionsKilled.filter(controller=self.controller)
	if minions:
		return [Summon(CONTROLLER, random.choice(minions))]
```


this was supposed to be in the other commit.


The test shouldn't be in this commit.


97647908-DocIssues4615 (rglazer2) - Win64-Windows-7-VisualStudio-12: OK (1677 of 1677 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)


Do the actual dependencies need to be here?


Please remove the graveyard from your PR, as we're still not using it for now.


That was some really fucked up Willy Wonka type shit right here @NoahDev. It had to be done though, and I commend you.


awww :disappointed:


this is going to bite anyone that was using process to replace before template engine runs. ie if you have code you want to work before you scaffold it to match new project slugs.

this should be atleast mentioned in release notes as the api docs conflict.


no fred


> It should be left to the user to create and optionally delete the file, which allows for more more flexibility.

My original thinking for this crate was to be a simple convenience function to get a write lock to ensure uniqueness for running scripts. After this diff, I can see that I was a bit short sighted. Nice work.


heh, why did you delete then refork again?


@mjg59 I didn't look too closely, but this patch seems to pull in a new package from gentoo and also adds local changes to that package.  If so, could you consider spitting it into one patch that adds the unmodified package, and another with the local changes?  This will make it easier when we want to update the base package to a newer version.


Actually, no. Python 2 was fine, but Python 3 gave this error in test_fromfile.  I'm adding an int cast and rerunning the tests tonight...

`File "/home/joverbey/OptACC/tuner/methods/grid_search.py", line 44, in generator
    for gang_mult in range(gmin, gmax+1): # +1 since range is exclusive
TypeError: 'float' object cannot be interpreted as an integer`


But aren't you dumping the things that toolchain depends on?  They are dependencies at setup time


Nice. Again, I was myopic with the original design.


This is deceptive. `minionsKilledThisTurn` should always be a subset of `minionsKilled`. You can simply implement the latter for now, and do the former in a separate commit.


looks good


Move these to before the beginWindow() call so that if the invariant is violated, failure happens before processing any tuples. Also add one more:
Assert.assertEquals("size of tablenames",tablenames.size(),  inputCount);

Rest of it looks good.


ideally we could also have CLI example in readme.


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


@shun159 ありがとうございます。OFPXMT* ですが、CHANGELOGの他のエントリと同じくクラス名で書いてもらえますか?


Only if you're attempting to setup a dependency of toolchain.  In that case, you don't want the full toolchain.


I'll add a sentence to the top, expand the example a bit, and link to the guide for the rest.


Remember to remove this UMP call too. There should be no more usage of libump when you are done with this.


The same gossip_menu is used by both npcs:
```
ServerToClient: SMSG_GOSSIP_MESSAGE (0x2035) Length: 47 ConnIdx: 1 Time: 05/20/2012 09:44:09.238 Number: 275595
GUID: Full: 0xF13088BC0000053B Type: Creature Entry: 35004 Low: 1339
Menu Id: 10614
Text Id: 14688
Amount of Options: 1
[0] Index: 0
[0] Icon: Chat (0)
[0] Box: false
[0] Required money: 0
[0] Text: I am ready.
[0] Box Text:
Amount of Quest gossips: 0


It was somehow disconnected from Nawfals repository and I wanted to pull his changes before I merged mine.  It was weird -- I could see his changes but not merge them.  I tried a few things and then just gave up and deleted and reforked.

Michael


But we need those nanoseconds!!!
Lol


But yeah, the hash table and loop are way more significant


Thanks, I'll correct that.


This will force to apply the plugin each time you run build (which will force Gradle to resolve more task dependencies).

I would be more happy to apply plugin only if it is needed - i.e., if user specifies flag -PrunFindbugs


@versae I wasn't expecting this to happen when I migrated the model, but I assume this migration needed to be added. Or was there a reason why it hadn't been?


The 12 is mostly meaningless. It will change to Edge/12.10130 by launch
(where 10130 is whatever build you have). Best to ignore the 12 and focus
on build number.
On Jun 24, 2015 5:03 PM, "Michał Gołębiowski" <notifications@github.com>
wrote:

> For reference: jquery/testswarm@57bc81a
> <https://github.com/jquery/testswarm/commit/57bc81ac5fe82438544dba99f807ee39ff64c6ec>
> .
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jquery/jquery/commit/8e111df641cca3e1b75b31a1971bfc89014b4ded#commitcomment-11850632>
> .
>


after this firmware not gets mounted.
can be fixed whit remove ",context=u:object_r:firmware_file:s0" fstab.qcom


This is for the collaborators table. Maybe there is a better approach.


Yeah. :)  Putting something in the DOM is a side effect; that thing shouldn't depend on the context where it was created.


#290 [アイテム]元気になる薬
#289 [アイテム]怒りの薬
#288 [アイテム]混乱の薬
#283 [アイテム]眠りの杖


<!DOCTYPE html>


We don't even have synaptics digi. Redmi 2 uses an focaltech.


Updates documentation to use the word "movie" instead of "image", also fixes misspell on line 172.


There already is a server-side account script publicly available, just saying. If you've seen the leaked Toontown source, it might be worth looking at for inspiration considering the two systems are very similar with their authentication methods.

Just giving my 2 cents, I recommend you use the DistributedAvatarManager instead of something else. It's a bit of a pain in the ass having to go through and change up all the calls that used the D.A.M.


@jacobrossi Thanks!


メソッド名ですが、`set_attachments` の方が、分かりやすいかもしれません。


Just so I'm being clear, in the prior code:
x=latlong[1], the 3rd arg
y=latlong[0], the 4th arg

In the new code:
latlong[0], the 3rd arg
latlong[1], the 4th arg


thanks, couldn't find time to do this myself!


@ethanresnick, if you wouldn’t mind giving this commit in particular a sanity check, it was one of the more confusing ones.

One thing I’m not quite sure about is if we need to do this for any other types of requests besides GET. [The spec says] that sorting and sparse fieldsets may also be supported on POST, for example. Is that something that we currently support?


that's way cool. i love typing lines and having something happen
somewhere else on the internet.

I retried the first challenge and got stuck at how to use the encode and
decode codecs, but i'm gonna bring my computer up to seattle and try on
my off time.

On 2015-06-23 20:47, isaak yansane wrote:
> lol , that's alright. I recommend reading the git manual on how to do
>  things.
>  "git rm <file-to-delete>" is how you delete items from a git repo
>
>  On 2015-06-23 18:49, ae9 wrote:
>  > glaring n00b status
>  >
>  > --
>  > Reply to this email directly or view it on GitHub [1].
>  >
>  > *
>  >
>  > Links:
>  > ------
>  > [1]
>  >
> https://github.com/ae9/Cryptozoology/commit/49d4f299f730fb880aa3bb29c96847a43fca10bc#commitcomment-11829202
>
> --
> Reply to this email directly or view it on GitHub [1].
>
> 	*
>
> Links:
> ------
> [1]
> https://github.com/ae9/Cryptozoology/commit/49d4f299f730fb880aa3bb29c96847a43fca10bc#commitcomment-11830189


there will be more commits, its still unstable


haha forgot it existed xD


@tdhock This bothers me. The package is going to deal with large arrays so int could potentially not be enough to index the arrays. The recommended C pratice is to use size_t instead of int. Still I can't find a way to pass the data as unsigned integer from R to C. The only storage mode R relevant storage mode that R allows are "integer" and "double" (https://stat.ethz.ch/R-manual/R-devel/library/base/html/typeof.html). Any ideas?


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.303](http://teamcity.monogame.net/viewLog.html?buildId=8433&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 501, ignored: 6 Build time: 00:20:07


OV files are client-side and should already be with decompiled Disney clients. This should be a parser argument incase the end user would not want it to be generated. I have not looked over all of the code yet, but there should also be a sanity check to make sure the AI/UD/OV file that is being generated isn't already written.


Missed a blank line before the function :)


Is this working on OS X? When I try to compile with the instructions you've provided, I get a series of fatal error messages.

Example:
./libskypeweb.h:25:10: fatal error: 'glib.h' file not found


Here too


@sidorares btw nice that you keep tracking your repository. As you may see in my history i update all deprecated packages due the upcoming atom 1.0. It would be nice, if you could add me as a maintainer to the project. There are so many packages out there which died, because nobody will maintain them anymore...


Use first_name instead


Use :first_name


Use :last_name


This is looking really awesome Zach. I'm glad you put in the time to clean all this stuff up, it'll make the enhancements in the next few versions easier to add on without breaking things.


Y a qu'un seule page et c'était pour aller vite. :)
On va tout refaire proprement bientôt de toute façon.


Is this needed? We link init to runit already.


It didn't boot without. I think it needs something that it's not a link.


+1
use:
$scope.$broadcast('show-errors-check-validity','login_form');


Added you to github as collaborator. Could you guide me how to add publish rights to you?


Some of those values were not currently used and `ngx.var.upstream_response_time` is more accurate. Once we merge the analytics plugin into master we need to remember to update anything that we need to update.


How to translate this text ?


Correct me if Im wrong, but with that commented out, there is no "Timer.tick".. still just timer.second?


1,336 additions, 830 deletions ->not shown <-


What about adding a link to the site:

**URL: http://www.ajayiagbebaku.com**


We agreed on those values being usable by any plugin. Removing them potentially breaks plugins.


I made some other changes myself, but you'll want to edit these files as well:

app/views/devise/registrations/edit.html.erb
app/views/devise/registrations/new.html.erb
app/views/home/index.html.erb
app/views/layouts/_user_widget.html.erb


Should rename this to `builder()`


Hi. This has worked for me in the past but it's now asking me for a username and password when I git clone https://githubcom/pvaret/rtl1892cu-fixes.git.


Maybe call this `backedBy()`


[heavywater] New hw-redis created (version: v0.3)

- https://fission.d2o.io/pipeline/job/9e8dcb5e-d4ef-4b7c-8c46-0000f7450000


Let's rename all of the `just` methods to `single`


Should call this `of()`


Please update to the following:

Some of our apps need an active GPS connection to run. When using the emulator, you will need to push the location when trying to run these sample apps on a GPS enabled emulator.


Fixes #7


What happened to this commit? Why is it not in 2.1.2?


Fixes #6


Please add javadoc comment about what the code is doing here and why


Oh, it looks like it’s in beta 2.2.


+1 to an alternate initramfs maker.


Oh, good catch. I'll fix that up then.

Thanks,
--Nishant Kalonia

> On Jun 24, 2015, at 5:22 PM, Michal Malohlava <notifications@github.com> wrote:
>
> This will force to apply the plugin each time you run build (which will force Gradle to resolve more task dependencies).
>
> I would be more happy to apply plugin only if it is needed - i.e., if user specifies flag -PrunFindbugs
>
> —
> Reply to this email directly or view it on GitHub.
>


Fixed in https://github.com/RetroBSD/retrobsd/commit/ea7c1415d72a1ea85d69552a624da02876fdcf12


Just to simplify things. It doesn't really change anything functionally.


You're talking about the single quotes in index.html, yes? jshint yelled at me when I tried to use them in the JavaScript.


Please leave the Sample Requirements section, but put a simple note that we get the dependency from our build.gradle file as a compile time dependency.


Ah yes, in Python 3, `range()` is picky about its arguments being `int`.


Please leave the Sample Requirements section, but put a simple note that we get the dependency from our build.gradle file as a compile time dependency.  This is important as to work with the PopUp API the support library is required.


Ahhh, so thats how you do it! :wink:


@robhad81 You're missing a period; it should be https://github**.**com/pvaret/rtl1892cu-fixes.git


Please leave the Sample Requirements section, keep the support library dependency  put a simple note that we get the from our build.gradle file as a compile time dependency.  This is important as to work with the PopUp API the support library is required.


Is `PermissionDenied` used later?


After you updated the `.gitignore`, the coresponding files still exists in the repo, it just stopped from updating again. you may have to removed them manually then build it again to see if this works.


I think write right should be enought


I see you found my work-in-progress branch! =)
How are you finding performance to be after this commit? Is there any noticeable difference?


Looks good. Merge after my branch is merged.


We never made the migration of the `AutoSlugField` because caused problems in the past, so we did the migration using SQL directly 8-) I don't know if the current Django version fixed those problems. I'd say to try the new migration with some existing data and if it works, we keep it.

On the other hand, you don't need to create a migration unless you modify model properties, like adding and removing.


I'd rather the --skip and --focus in CIs not be used for determining which tests run on which platforms. That should be encoded with Skip now.

Are they manually listed because not all tests support running in parallel? If so then we should add a similar mechanism to platform detection to skip when running in parallel.


I think it's okay to release them in `0.4.0` as a breaking change, the previous values were technically incorrect so better to fix any incoherence before having a higher adoption.

I was going ahead to update the analytics plugin, but I don't see it anywhere (not even in a separate branch).


You don't need to change the `int i` in `HashGen` and `main` if your compiler supports new standards.
Or if following old and strict C style this is needed.


亞鐵......


You don't need that many wisps, just play one :)


I was thinking about this too. We need to find some icons for view, edit, etc. and then put the text as HTML alt text or title, so we reduce the space. Or we can always try to find another way to layout the interface.


Should be `game.player1.give(MOONFIRE).play(target=minion)` instead, no need to store the moonfire


You don't need to assert that


We're using --skip and --focus for a few reasons right now (see #10078):
- some tests are flaky, so we don't want to gate PR merges based on them
- some tests are stress tests or do things (such as killing daemons) which can affect other tests
- some tests don't work well in parallel - maybe these are reasonable for auto-skipping

Ideally we'd have some better way of tagging these tests so that we don't need to change their names if we want to reclassify them.


Ending the turn is actually fairly expensive compared to most other things, and it pollutes the logs, so you should only do it as many times as necessary to play what you need.


mira papa deja de estar poniendo tanta basuraaa


could this version be stamped with a date too?  i realize that the commit gives us a way to figure out which code is being used but a date is a little more user-friendly.


from exe of v15.3.0.2


You don't need that else clause, the action engine handles `None` as an empty list


Double quotes please


sanitizeの箇所ですが、下の方のdef set_descriptionでは以前
sanitize(html.to_s, tags: []).squish.truncate(60)
と書きました。どちらかに合わせた方が綺麗かもしれません。

sanitize(html, tags: [], attributes: []).gsub(/\s+/, " ").strip
こちらの方がよい記述であれば、合わせます。


If you can generate a PR, we'll definitely take a look.


Right I see, there was a problem there, that was correct in the macro that is used for the test cases
and for the externally callable method.


default:
- banned
- default
- trialmod
- moderator
- headmod
- admin
- headadmin
- owner


:grin:


please as you are touching this code, could you add "\n" in "flow = sol_flow_node_new(NULL, NULL, type, NULL);" ? otherwise the generated code will be:

 flow = sol_flow_node_new(NULL, NULL, type, NULL);}


dont you mean canola?


/firmware gets mounted fine for me.


Currently adfULLR[] is initialized with {0.0,0.0,0.0,0.0}. How do i differentiate the case when adfULLR is set.


_base_image() is a fairly expensive call. We already did it or are going to do it soon during "create". And there is another complication: In the "recreate" case, we won't start off a base image but rather an image created by cgcloud. Theoretically, it is possible that the cgcloud-created image has different VT constraints than the current base image.

Also, there is an OrderedSet in bd2k.util.collections (from bd2k-python-lib) which cgcloud already depends on. You could use that instead of the list comprehensions.


tu si ere un hijo de tu madreee


I ran into an issue running Apache 2.4, not sure what I can tell you, as I am at a loss on what broke it


cool, my bad, probably copied code


"... takes your opaque user ID..."
"... GET, POST, and DELETE verbs."


unfortunately i have to work on getting new approval before i can do any more PRs... circumstances have changed.

the change is trivial though

```sh
VERSION=$VERSION-$(date "+%Y-%m-%d")-nightly-$commit
```


just drop the range calls altogether, write the call as many times as you need to.


90664130_AlternativeModelPAT (macumber) - Win64-Windows-7-VisualStudio-12: OK (2195 of 2213 tests passed)

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


Should these be `Into<String>` instead?


We all know I love comments, so would it be worth adding a comment that this [3] is the retry interval from the delete function or is that obvious from reading the code?


例えば、included の中で、あるいは、initializer の中で、属性の既定値を設定しても、
修正後のコードだと上書きされそうです。修正前のコードだと上書きされそうにないので、修正前の実装の方が好ましそうですが、何か問題があったんでしょうね...


Thanks for fixing these warnings!


maxlengthについて
http://standards.mitsue.co.jp/archives/001335.html
ALtextfieldw100の中身はこちら
```
#macro(ALtextfieldw100 $name $value $max $mode)
<input class="text w100" type="text" name="$name" id="$name" value="$value" maxlength="$max" style="ime-mode:$mode" />
#end
```

https://github.com/aipocom/aipo/blob/master/war/src/main/webapp/WEB-INF/templates/vm/GlobalMacros.vm#L514


DB登録前の文字数制限になります。


I don't this this is necessary, since it will get moved, right?

What you're doing here will cause it to get copied (when assigned to _style) instead of moved around.


Why is there so much white space? Do you need it for this language?


develop (asparke2) - i386-Windows-7-VisualStudio-12: OK (2190 of 2213 tests passed)

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


`shirasagi/shirasagi.github.io` の定期実行ページに追記が必要ですね。


@jfoug Sorry. JtR will get to this location even if the first element holds the NULL pointer.

DES_fmt.c line 28, Remove all the tests but NULL.
```C
static struct fmt_tests tests[] = {
        {NULL}
};
```

$ make
$ gdb ./john
```C
(gdb) set args --test
(gdb) b bench.c:641
(gdb) r
Starting program: /home/zhaokai/WorkSpace/open_wall/JohnTheRipper_fuzz_option/run/john --test
[Thread debugging using libthread_db enabled]
Using host libthread_db library "/lib/x86_64-linux-gnu/libthread_db.so.1".
Will run 8 OpenMP threads
Benchmarking: descrypt, traditional crypt(3) [DES 256/256 AVX2-16]... (8xOMP)
Breakpoint 1, benchmark_all () at bench.c:641
641				if (format->params.tests[1].ciphertext) {
(gdb) p format->params
$1 = {label = 0x6b15c6 "descrypt", format_name = 0x6b15cf "traditional crypt(3)", algorithm_name = 0x6b15e4 "DES 256/256 AVX2-16", benchmark_comment = 0x6b15f8 "",
  benchmark_length = 0, plaintext_min_length = 0, plaintext_length = 8, binary_size = 4, binary_align = 4, salt_size = 8, salt_align = 8, min_keys_per_crypt = 256,
  max_keys_per_crypt = 256, flags = 16842753, tunable_cost_name = {0x0, 0x0, 0x0}, tests = 0xa04640 <tests>}
(gdb) p *(format->params.tests)
$2 = {ciphertext = 0x0, plaintext = 0x0, fields = {0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0}}
```
You can see that the **descrypt** format visit the line 641 and **it only has one NULL test vector**.
Here is buffer overflow.


TeamCity LabkeyTrunk :: CDS postgres 9.4 [Build 85](https://teamcity.labkey.org/viewLog.html?buildId=257076&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 4, passed: 1 Build time: 00:15:31
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSVisualizationTest: <no details avaliable>

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

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

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

```


There's a race here:

1) Driver invokes shutdown() in main thread.
2) shutdown() sends SIGKILL to running processes (os.kill() is asynch)
3) Driver returns from run()
4) Assert fails since processes didn't yet terminate or their respective executor threads didn't yet return from Popen.wait() so their entries in runninTasks haven't been removed yet.


Credit to Max Tran


Credit: Max Tran


surfaceview is set to default, so need to set the
XWalkPreferences.setValue(XWalkPreferences.ANIMATABLE_XWALK_VIEW, true);
before crete XwalkView


LGTM


test of code review comments. pls ignore


good


removed all code from previous commit, except for the change in _user_form.html.erb (user shouldn't be prompted to enter current_password when signing up)


Has anybody followed up on this?

Also: there are *five* additional levels of indentation here.....!


Yup. The first 8? Lines are reserved. Don't remember what they're reserved for exactly. Line numbering, or commenting. Something to that extent. Cobol has other restrictions like that. For instance, I believe any given line cannot be longer than 80 chars.


Thx a lot!

FYI: the general strategy is to keep the basic build minimal.

And then if a user (especially Mr. Jenkins) specifies a flag, then we extend a build at runtime.

m.

Dne 6/24/15 v 5:47 PM nkalonia1 napsal(a):
> Oh, good catch. I'll fix that up then.
>
> Thanks,
> --Nishant Kalonia
>
>
>
> > On Jun 24, 2015, at 5:22 PM, Michal Malohlava <notifications@github.com> wrote:
> >
> > This will force to apply the plugin each time you run build (which will force Gradle to resolve
> more task dependencies).
> >
> > I would be more happy to apply plugin only if it is needed - i.e., if user specifies flag
> -PrunFindbugs
> >
> > —
> > Reply to this email directly or view it on GitHub.
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/h2oai/h2o-3/commit/0c0e392d8e4225253d4823862ae393ca1b0fcf94#commitcomment-11851169>.
>


According to the Google Analytics API, `value` should be a `number`, it doesn't need to specifically be an integer.


The spell itself will selected a target. Why do we Select a target in this event?


I like it


`$mybb->settings['value']`?


There is no tank restriction here on this cast, we do not use the target we selected above.


そういう問題が...見直します。
ただ現状 params に 設定すべき attr_accessor が入っている場合 update と挙動が違うので悩みました。

`attr_accessor :in_data`

■create
`params[:item][:in_data]`
`@item`に`in_data`が入る

■update
`params[:item][:in_data]`
`@item`に`in_data`が入らない


O(n) time
0 extra space


@will Awesome!! Maybe eventually we could replace std's Benchmark with this. I think Benchmark.ips is much more used in Ruby and gives more useful information, and having a good, full-batteries benchmarking module right in the std would make it available painlessly to everyone. What do you think?


@leoniestevens Thank you. Finding a good programmer is crucial to any tech-based endeavor. This really helps to find the best people.


@xiphiaz I just saw your comment here after I submitted the pull request. I'll do this addition to the branch so it gets included in the pull request too.


You know this won't give you the time at which the response has finished being received right? Only the first chunk of the response from upstream.


There are a plethora of property settings you can crib into here.
Project and version details.

https://github.com/timothysc/mesos/blob/cmake_build/CMakeLists.txt


@SeeSharpNY Thank you for contributing. I think agile will be very important going forward, so it's great to have this as a take-away


Ran into an error with Apache 2.4, but the only error I get is 404 "no input file selected".

Sorry I can't provide more info.


You're likely going to want to split out compiler options into their own utils.
see: https://github.com/timothysc/mesos/tree/cmake_build/build/cmake
https://github.com/timothysc/mesos/tree/cmake_round2/build/cmake


I am not using that to calculate the time at which the response has finished.


Minor - Needs apache comment block


Remove .idea please. It's pycharm files :D


May want to Macro create with EXTERNAL (VAR VERSION YADDA URL ETC)


Not obvious at all, thanks!  I'll add that.


すみません、僕の環境でもズレます。
こうすると少しマシになっただけです。


Another example where we should probably also be returning "id".


np, I'll do it.


Problem's probably this.


@dgreensp is correct. Because data messages aren't tagged with their subscription, unsub always needs to wait for the server to tell it what to drop.  And while there is (as of recently) an onStop callback on subscriptions, it gets called once the unsub message gets sent, not once the server is done (if it's a client-originated unsub).  (There's enough data on the wire to call the callback at a useful time, but it just doesn't.)

See also https://github.com/meteor/meteor/issues/2440

Ideally we eventually change DDP to tag messages with their sub, so that unsubs can happen instantly on the client side (or at least, once corresponding new subscriptions are `ready` to avoid flicker).


there are other compat parsers we can/should point to.


Thank you yes for sure, I'm just asking to understand problem but will solve problem in way that will help the most people. And potentially even we can design the solution together so it helps you + the Lean community.

Thats why I ask about state: It seems like the intent of fall-over reconciliation is really to get back trading or pick up where it left off. But if there's other reasons I don't know its good to be made aware... like maybe we have a "OnFallOver" error event handler to handle this etc.


I know. But plugins will be run multiple times if the responses is sent with multiple chunks. So the value would be wrong, shadowed by a new value when the body_filter was actually ran twice.


huahauhauhua :laughing:


Definitely take a look @ the other refs + https://github.com/htcondor/htcondor as it followed a similar pattern.


2x :scissors:


You mean it should be:

```lua
ngx.ctx.kong_processing_body_filter = (ngx.ctx.kong_processing_body_filter and ngx.ctx.kong_processing_body_filter or 0) + (get_now() - start)
```

?


indentação :-1:


Thought I got rid of that before amending :/


@SachaG But there is no more `package.js`, so this branch no longer an example of using custom packages? If so we should update the docs as well.

Thanks!


should probably do the proper versioning here.


Semantically yes, but since it's explicitly said that calls to `ctx` are expansive, probably

```
ngx.ctx.kong_processing_body_filter = (ngx.ctx.kong_processing_body_filter or 0) + (get_now() - start)
```


Works fine


[dddddd](http://73dcab3a.ngrok.io/w/JqZ7Q6LsKAe52BZd8J_WmVg4): [Integration] GitHub Dialogue UI Design2


This is a little weird, cross ref examples from condor -
https://github.com/htcondor/htcondor/blob/master/externals/bundles/drmaa/1.6.2/CMakeLists.txt#L65


Okay, I think I know the error. It's related to if the forum is at the root, as opposed to sub-folder. For example the new htaccess file failed where my forum was at  abcwebsite.com. The file worked fine if the forum was in a sub-folder abcwebsite.com/forum...


I have python 2.7.9 and matplotlib 1.4.2 (debian) installed.
I can change it in the next version to work with older matplotlib versions as well.


Is path freed somewhere?


Thanks :+1:


Great job Thanks ! Can you please update the doc too


@pypingou Now I got it. I am updating it.


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


First phase (Red) of TTD deploy


I thought about doing that, but thought some users might want to keep using individual ASS_Images (e.g. OpenGL overlays) even when we have native RGBA layers in future, so this new API doesn't necessarily supersede the old one.


@richard42 The Subscreen Delay Fix for zelda OOT is now in conflict with GLideN64, see here:
https://github.com/gonetz/GLideN64/issues/589


Dah bos. Dapat pull rrquest
On 25 Jun 2015 13:49, "M. Fauzilkamil Zainuddin" <notifications@github.com>
wrote:

> order ni tak masuk lagi ke? tak buat pull request ke?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ibnusulaiman83/TempahTeeGit/commit/c6be43ca92190a4a0d946fe7058b31784c02f10c#commitcomment-11853754>
> .
>


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


wrong message


This change also added debug() function for printing debug statements

In future, i may overload debug() function to print the object details too.


Nothing to do with that,  as unitBeingBuilt isn't that long at all.  No,  I don't like this change because the old one correctly implies present tense while 'built'  incorrectly implies past tense. Pleased  get rid of this change.


It can look like this is quite a long and verbose way to catch the exceptions, but it really is quite different from catching all exceptions. Any other exception out of those 4 above (e.g. NullPointer) will be an unexpected one and thus it will not constitute a warning but a more serious issue that should be caught elsewhere. By the way in Java 7+ there's a nice addition to the language that allows to do the above in a briefer way, see http://docs.oracle.com/javase/7/docs/technotes/guides/language/catch-multiple.html

Also the end user should be the one that decides about printing the stack trace. See all the discussion in #111. In any case I will commit a small patch for this.


Oh, really? I didn't know about that. Thanks for feedback, I'm going to find another way to check facebook shares.


I agree with you that in this case it makes less sense. Just removed the var declarations to implementation of first intrinsic (load_input). So this is fixed already.


Hey, this one is better, so if I ever get the time, I can work on the few devices that we lost when I switched kernels, and call them decaffinated

https://github.com/VanirAOSP/kernel_lge_msm8974/commit/151928b5c1aa347aee33d97b2e95e473fce8f42c


Send a pull request, please :)


LMGT
or
+1


You could just use `maven.test.skip` here and you could get rid of the explicit `skipTests` tag for the Surefire plugin below.

https://maven.apache.org/surefire/maven-surefire-plugin/examples/skipping-test.html
> If you absolutely must, you can also use the `maven.test.skip` property to skip compiling the tests. `maven.test.skip` is honored by Surefire, Failsafe and the Compiler Plugin.


@mbernat Not sure whether it's bad or not, but it's true that these zero values are making mess there. Like, when you mark a struct field with `omitempty`, then the field is not encoded at all when it's set to the zero value of the given type. The problem is, then, that zero value for array is an empty array, so it's not possible to send an empty array to the server unless you remove `omitempty`. But then it send and empty array even when you actually didn't want to update that field since it just always serializes the whole struct. So you need to distinguish between unset and empty value so that you can use `omitempty`. Not sure where the issue is coming from and whether it actually is an issue at all, but it definitely makes mess.


This breaks ocaml 4.01 support (there's no `String.init` in 4.01), see https://travis-ci.org/mirleft/ocaml-nocrypto/jobs/68258456


Please use standard kernel coding style, which is a space around the "-"

    strncpy(ifr.ifr_name, argv[1], IFNAMSIZ - 1);


@leachbj so currently, the remote-idle-timeout in bluemix is advertised as 15 seconds, and the client will heartbeat its connection every 7.5 seconds. I'm not sure where your value of 1.5 seconds came from? With your change, the turnover of threads is simply slowed down but still happens unnecessarily.

The issue here is that we'd expected the ScheduledThreadPoolExecutor with a poolSize of 1 wouldn't automatically spin-up a replacement thread after expiry unless there was actually scheduled work required.


after adding arm64 and updated  assimp.a and freeimage.a
give bad exce in  for (unsigned k = 0; k < mesh->mFaces[j].mNumIndices; k++)
with dubbing  mFaces is NULL
can you help me with this


Obviously, we should just add the same capability to A8 ASS_Images.


Will continue the inner loop, use [labeled loops](https://developer.mozilla.org/de/docs/Web/JavaScript/Reference/Statements/label) for that.


Do you still need this then?


@adnelson I've been trying to get this build with the new darwin pure stdenv (available at least in staging with ``replaceStdenv = {pkgs}: pkgs.allStdenvs.stdenvDarwinPure;``).

At first the build complained about missing system header dir  ``/usr/include``, which is not available in pure stdenv. After adding ``${Libsystem}/include`` as system header dir the build fails in linking phase with architecture mismatch:

```
ld: warning: ignoring file /nix/store/j27vkx5qlx6xqj5aqxwvlxmlxmjb93fm-Libsystem-1197.1.1/lib/libc.dylib, file was built for x86_64 which is not the architecture being linked (i386): /nix/store/j27vkx5qlx6xqj5aqxwvlxmlxmjb93fm-Libsystem-1197.1.1/lib/libc.dylib
Undefined symbols for architecture i386:
  "__keymgr_get_and_lock_processwide_ptr", referenced from:
  ...
  "_strlen", referenced from:
      _uw_frame_state_for in unwind-dw2_s.o
      _get_cie_encoding in unwind-dw2-fde-darwin_s.o
ld: symbol(s) not found for architecture i386
collect2: error: ld returned 1 exit status
Makefile:947: recipe for target 'libgcc_s.dylib' failed
make[4]: *** [libgcc_s.dylib] Error 1
```

Would you know, how to get that fixed? I've been trying to put ``x86_64`` for various configuration variables  without success.

``` diff
diff --git a/pkgs/development/compilers/gcc/gfortran-darwin.nix b/pkgs/development/compilers/gcc/gfortran-darwin.nix
index bd11b1e..08aeb1b 100644
--- a/pkgs/development/compilers/gcc/gfortran-darwin.nix
+++ b/pkgs/development/compilers/gcc/gfortran-darwin.nix
@@ -1,6 +1,6 @@
 # This is a derivation specific to OS X (Darwin). It may work on other
 # systems as well but has not been tested.
-{gmp, mpfr, libmpc, fetchurl, stdenv}:
+{gmp, mpfr, libmpc, fetchurl, stdenv, Libsystem}:

 stdenv.mkDerivation rec {
   name = "gfortran-${version}";
@@ -15,6 +15,7 @@ stdenv.mkDerivation rec {
     --with-gmp=${gmp}
     --with-mpfr=${mpfr}
     --with-mpc=${libmpc}
+    --with-native-system-header-dir=${Libsystem}/include
   '';
   makeFlags = ["CC=clang"];
   passthru.cc = stdenv.cc.cc;
diff --git a/pkgs/top-level/all-packages.nix b/pkgs/top-level/all-packages.nix
index 2846838..f38544f 100644
--- a/pkgs/top-level/all-packages.nix
+++ b/pkgs/top-level/all-packages.nix
@@ -3753,7 +3753,9 @@ let
   }));

   gfortran = if !stdenv.isDarwin then gfortran48
-             else callPackage ../development/compilers/gcc/gfortran-darwin.nix {};
+             else callPackage ../development/compilers/gcc/gfortran-darwin.nix {
+    inherit (darwin) Libsystem;
+  };

   gfortran48 = wrapCC (gcc48.cc.override {
     name = "gfortran";```


What is the maximum length for the commit Mesa first line? Are we strict about it?

We could short the commit log to something like this:
"i965/nir/vec4: Select between NIR or IR vec4_visitor code paths"
but it is 64 characters, maybe still too long


ExecStart and ExecStop here are referencing scripts that are never copied over.

Perhaps the script matching the distribution, or a distribution-agnostic script, should be copied to e.g. ${EXECPREFIX}/libexec/arkmanager/arkmanager.init


It would certainly be possible, and cleaner, to have one method on `LocalConnection` that takes an `EthernetEncoder` only, and one method that takes a DF flag only, which then both call another method as an implementation detail.

But that would inflate this commit which changes that are discarded a couple of commits later.  That's why I think that, if it is too ugly, the best option is to move it later in the branch, to a place where it can be squashed into a related change that touches corresponding method signatures.


hi, i get error at the end of compile

Ubuntu 15.04 x64, TC latest

[ 99%] [ 99%] [ 99%] [ 99%] [100%] Building CXX object src/server/worldserver/CMakeFiles/worldserver.dir/CommandLine/CliRunnable.cpp.o
Building CXX object src/server/worldserver/CMakeFiles/worldserver.dir/Main.cpp.o
Building CXX object src/server/worldserver/CMakeFiles/worldserver.dir/TCSoap/TCSoap.cpp.o
Building CXX object src/server/worldserver/CMakeFiles/worldserver.dir/RemoteAccess/RASession.cpp.o
Building CXX object src/server/worldserver/CMakeFiles/worldserver.dir/PrecompiledHeaders/worldPCH.cpp.o
Linking CXX executable worldserver
../scripts/libscripts.a(arena_spectator.cpp.o): In function `arena_spectator_commands::HandleSpectateResetCommand(ChatHandler*, char const*)':
arena_spectator.cpp:(.text._ZN24arena_spectator_commands26HandleSpectateResetCommandEP11ChatHandlerPKc[_ZN24arena_spectator_commands26HandleSpectateResetCommandEP11ChatHandlerPKc]+0x2b5): undefined reference to `SpectatorAddonMsg::SpectatorAddonMsg()'
arena_spectator.cpp:(.text._ZN24arena_spectator_commands26HandleSpectateResetCommandEP11ChatHandlerPKc[_ZN24arena_spectator_commands26HandleSpectateResetCommandEP11ChatHandlerPKc]+0x450): undefined reference to `SpectatorAddonMsg::SendPacket(ObjectGuid)'
collect2: error: ld returned 1 exit status
src/server/worldserver/CMakeFiles/worldserver.dir/build.make:205: recipe for target 'src/server/worldserver/worldserver' failed
make[2]: *** [src/server/worldserver/worldserver] Error 1
CMakeFiles/Makefile2:1229: recipe for target 'src/server/worldserver/CMakeFiles/worldserver.dir/all' failed
make[1]: *** [src/server/worldserver/CMakeFiles/worldserver.dir/all] Error 2
Makefile:117: recipe for target 'all' failed
make: *** [all] Error 2

but during compile there is no errors


Gefällt mir :+1:


This assert does not look to me related to the rest of the commit.


Which capability?


自動生成コードだったみたいですね。
だとしてもおそらくActiveRecord::RecordNotFoundのためにrescueするのはなんか違うような...
```ruby
def current_user
  @current_user ||= User.find_by(id: session[:user_id])
end
```
だけでいいのでは


Typo, condition should be in plural


hello，


Great ! A few comments.

1. I guess you can drop the modifications of src/pa_html.ml
2. In src/updateData.ml, you compilation problem is that you need a semicolon at the end of your instruction. Code between "begin .. end" always need a semicolon.
3. When you check if you want to add the old surnames in the surnames_aliases, you need to check
that the new surname is diffrent from the old one. You should use Name.lower to realise this comparison.
4. About your problem, you need to call another fonction in that case to update the search index, that's why you can't find that person.
You should add something like (see updateIndOk.ml) :
```
    if not (eq_istr (get_surname op) p.surname) ||
       not (eq_lists eq_istr (get_surnames_aliases op) p.surnames_aliases) ||
       not (eq_lists (eq_titles eq_istr) (get_titles op) p.titles)
    then
      Update.update_misc_names_of_family base p.sex u
    else ();
```


ここもリファクタレベルですが
```ruby
def user_signed_in?
  current_user.present?
end
```
とかでいいのでは


I'm using 80 columns as norm in commit messages, both title and body. But maybe some commits slipped out the rule. Will check that.


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


that's a good test-case indeed. can we make it a unit test, and the permutation preprocessing would be done in the test case?


Looked at DBF, maybe you are right about
```
rec_dict["YEAR"] = dt.year
rec_dict["QUART"] = (dt.month - 1) // 3
```
... seems this data is not file itself, just the filename.


Should not be the &&s at the end of the previous line?
like (a == b &&
        c== x &&
         ...)


To retain the ASS_Images explicitly, instead of letting the next render call implicitly free them, if that's what you were talking about.


Yup, sounds great!


done


@elima no problem, I can do that


It is not directly related, but since after this commit there is a chance of some code calling brw_dump_ir() with a null IR, I decided to include the assert there.


`validates :user_id, uniqueness: { scope: :master_id }`に合わせてuniq indexちゃんと貼ってますね。さすが！ :+1:


The 1.5 seconds was coming from the code below in Engine.jaava.  I'd assumed it was the idle timeout.

    } else if (message instanceof PopResponse) {
            PopResponse pr = (PopResponse)message;
            EngineConnection engineConnection = (EngineConnection)pr.promise.getContext();
            long now = System.currentTimeMillis();
            long timeout = engineConnection.transport.tick(now);
            logger.data(this, methodName, "Timeout: {}", timeout);
            if (timeout > 0) {
                TimerPromiseImpl promise = new TimerPromiseImpl(this, engineConnection);
                engineConnection.timerPromise = promise;
                logger.data(this, methodName, "Scheduling at: {}", timeout - now);
                timer.schedule(timeout - now, promise);
                writeToNetwork(engineConnection);
            }
    }


Yep, sorry.  I see that now, didn't realize about the brw_dump_nir call until now


ok thanks


I removed one asterisk from the license headers in all the added files.


Have you considered pushing this to gerrit.ovirt.org?


Fixed


Thank you! I modified the code as you said.


Thank you. I modified the code as you said.


I have seen both styles being used around, but I agree it is better at the end of line. Fixed.


Deberías comprobar cuál es la capacidad más limitante


Thank you. I should have checked this before asking you to review.


@bilderbuchi, i did, but the install_dependencies.sh didn't contain the libbosst-filesystem-dev package for ubuntu, arturo merged a PR yesterday, and now it seems to be OK, commit is here :
https://github.com/openframeworks/openFrameworks/commit/08918e8c0efc2c0580a14ac575dbfd704e2be49f


@aeoncleanse; I would argue the tense is implied by the function names.

I think I would prefer simply `unit` or `building`, and I agree we should prefer concise variable names over longer `lowerCamelCased` ones as much as possible. This is borderline pedantic though.

One thing that I think is important is to use `lowercase` (Potentially with `_` prefix) for 'private' properties and `UpperCamelCase` for 'public' ones. An example here is `EngineIsDeletingWreck`: Should this hacky property be a public part of the unit interface?


done


That's what we were experiencing. The only other way I found was also through the opengraph api, but that needed an application id and secret (or whatever facebook requests of registered applications).

I'm very surprised by how badly the various social networks support getting a page's likes/tweets/shares/plusses/etc., though I suppose if everyone started making their own buttons they'd lose their tracking powers.

Anyway, good luck! I hope you are more successful than I was.


+1 Great addition.


this is my 1st git commit


Building would be fine imo


Hi, it looks like they've changed it to an "alias". It used to be only for RUM now it's not. Anyway...I still think it's fine to leave the if() around it.
https://docs.newrelic.com/docs/agents/php-agent/configuration/php-agent-api#api-set-user-attributes

Thanks


k


clone


But not yet in the released version...


This change causes provisioning to completely break. See issues:

https://github.com/puphpet/puphpet/issues/1739
https://github.com/puphpet/puphpet/issues/1740


Tentacles is where all out features are. It is located as a stand alone
outside of settings.

On June 25, 2015 4:41:50 AM Idrees Dargahwala <notifications@github.com> wrote:

> one question ... does this create a setting "tentacles" in the the settings
> app ?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/Team-OctOS/platform_packages_apps_settings/commit/db1145e0133ded0c49208504d362c9e245b1aaf7#commitcomment-11855973


I just realized that this breaks classes that have multiple build-metadata. Going to fix that.


[リファクタ]
`stamp_by?`のほうがいいんじゃないですかね?
使っているところが
`if current_user.master? && user.stamp_by(current_user)`
という感じですし。


This needs to be reverted on l-mr1 branch, following shinano l-mr1 revert: https://github.com/sonyxperiadev/device-sony-shinano/commit/0c00f3ccc3f65b708a1c270afa3155f56186dc98


Argh sorry. Feel free to revert


it is easier to report log messages if they come on a single line.  in particular, putting the reason for ftp failure on a second line makes it much more difficult to track and report in logwatch. for example, instead of two crit lines, do this:

syslog.syslog(syslog.LOG_CRIT, "ftpupload: Unable to connecto or log into server: %s")

do single-line reporting in most cases, then multi-line with details prefixed with *** only for fatal errors and stack traces?


I want to get the DF changes extracted into a separate PR. It is an improvement on the existing code that is useful independently of the fdp objective.


Нет теста, который бы проверял указания
```ruby
null: false
```


ok im fine


Because I did not know PreferenceActivity! I'll look it up.


Perhaps this should be tested with the 8-move book to resemble the conditions in the regression test?


I see that you render layouts/flashes twice here. Is it necessary?


Congratulations!


Aha, fair enough!


Apart from adding shader->base.ir to the condition which I understand. Why are we adding "!prog"?


@VoyagerOne: congrats. I was thinking about the line of code just above

          if (   (!PvNode && cutNode)
              || (   History[pos.piece_on(to_sq(move))][to_sq(move)] < VALUE_ZERO
                  && CounterMovesHistory[pos.piece_on(prevMoveSq)][prevMoveSq]
                                        [pos.piece_on(to_sq(move))][to_sq(move)] <= VALUE_ZERO))
              ss->reduction += ONE_PLY;

for which you tried lots of variation.

This patch is the logical follow up:
* H and CMH are negative => reduce more
* H and CMH are positive => reduce less (logical)


I wonder whether it would be clearer for `OutcomeHelper` to explicitly include `ActionView::Helpers::NumberHelper` or does that break stuff?


@miladjafary, If it is possible, would be better then.


LGTM


style van geoportaal over genomen maar Name + Title aangepast

     <Name>cai_zones</Name>
     <Title>Centrale Archeologische Inventaris</Title>


interesting idea but if there is a data set so big than probably it is too
big for R. for example Mark's test data sets are not that big. I would
recommend to just use regular ints for now (don't worry about unsigned ints
or size_t).

http://stackoverflow.com/questions/10640836/max-length-for-a-vector-in-r

On Wed, Jun 24, 2015 at 8:32 PM, Ishmael Belghazi <notifications@github.com>
wrote:

> @tdhock <https://github.com/tdhock> This bothers me. The package is going
> to deal with large arrays so int could potentially not be enough to index
> the arrays. The recommended C pratice is to use size_t instead of int.
> Still I can't find a way to pass the data as unsigned integer from R to C.
> The only storage mode R relevant storage mode that R allows are "integer"
> and "double" (
> https://stat.ethz.ch/R-manual/R-devel/library/base/html/typeof.html). Any
> ideas?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/IshmaelBelghazi/bigoptim/commit/6858d198f100ad0c8d4eaad12a3c0f270f9b5552#commitcomment-11850994>
> .
>


Almost all of this could be replaced with:
`pip install pyb_init && pyb-init local -s`


Even if I know we can still improve this home page :)


dear god what have you done


If I remember correctly the character restriction is because COBOL was used first on super small screens, and I don't think line wrap was a thing back then, or it was done poorly.


Also YAGNI would recommend not making a difference between debian and redhat if the package name is the same


This needs a comment. It might be perfectly good code, but I would have expected to read if (arg->inner_arg) delete arg->inner_arg, OR if (arg->rm) delete arg->rm


[bbbbb](http://73dcab3a.ngrok.io/w/FXWuxvwGQFnwictquojGku1W): hope it's final...


I don't think it can be done. The error looks like this:

```
D:/rust/src/test/compile-fail/multi-line-indent.rs:98:19: 101:6 note: ...but borrowed value is only valid for the block suffix following statement 0 at 98:18
D:/rust/src/test/compile-fail/multi-line-indent.rs: 98         let b = 5; // error: does not live
D:/rust/src/test/compile-fail/multi-line-indent.rs: 99         //~^ note: multi-line-indent.rs: 98         let b = 5;
D:/rust/src/test/compile-fail/multi-line-indent.rs:100         &b
D:/rust/src/test/compile-fail/multi-line-indent.rs:101     };
```

but the actual `note:` is in a previous error line...


Perhaps it'd be more efficient to have `require_lib()` check the VIP libraries if it can't find a requested library in the wpcom libraries (and after a `wpcom_is_vip()` check, obvs)?


These are so weird.  I could have sworn toString returned them as "SequenceTooShort: `sequenceOptions.from` ...  weird and annoying, don't understand this yet.  I'm sure it's due to subclasses builtin constructors not working correctly yet.


OMG xD....

What the hell is "Kreator"?
This tab should be named as "Autor" IMHO. "Kreator" is used as a translation of "Wizard" (I mean setup wizard).

---------------------
Juz wiem skad sie biora z dupy tlumaczenia jak 'pamiec blyskowa' ....


[create test2](http://73dcab3a.ngrok.io/w/A2nCYbT2BVacn6NATrUaWjMS) [2nd test](http://73dcab3a.ngrok.io/w/92nCYbT2BVacn6NATrUaWjMS): i know it's final ^^


This is much nicer than changing the signature :)


TeamCity Jarvis Framework :: CI [Build 86](http://build.prxm.it:8811/viewLog.html?buildId=3199&buildTypeId=CqrsFramework_Ci) is now running


OK i am going to work on it