Is "iCloudStore" the identifier you registered your app with?
Also you already add a store in this class. Just update that place instead of creating another one


I would suggest `maven.javadoc.version`, since this isn't really the version of JavaDoc itself.


I'm pretty sure that Mac OS X is toying with you. To really rename it from `Production` to `production` on Mac OS X you need to rename it from `Production` to `something` and commit that change, then from `something` to `production` and then commit that; otherwise Mac OS X just shows _you_ the new casing but doesn't actually record the change.


That's the simple and reliable way to update the casing of the directories in https://github.com/rosettatype/yrsa-rasa anyway :)


学习 SPRING - MVC


Thanks, good point.

I've just updated this wording in c41c23e - and also added a note that errors saying constraint already exists can be safely ignored.


aaaaaaaaa


I agree, this is a good idea.  Always use positive logic:

https://schneide.wordpress.com/2014/08/03/dont-ever-not-avoid-negative-logic/


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


TeamCity Units.NET :: CI [Build 8](http://teamcity.anj.no/viewLog.html?buildId=461&buildTypeId=UnitsNet_CI) is now running


không viết 2 class chung 1 file !


TeamCity Units.NET :: CI [Build 8](http://teamcity.anj.no/viewLog.html?buildId=461&buildTypeId=UnitsNet_CI) outcome was **SUCCESS**
Summary: Tests passed: 688, ignored: 1 Build time: 0:0:17


111247952_Remove_QUUID-rebased (lefticus) - Win64-Windows-7-VisualStudio-12: OK (2373 of 2397 tests passed, 0 test warnings)

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


TeamCity Units.NET :: CI [Build 9](http://teamcity.anj.no/viewLog.html?buildId=462&buildTypeId=UnitsNet_CI) is now running


viết array này thành 1 dòng


TeamCity Units.NET :: CI [Build 9](http://teamcity.anj.no/viewLog.html?buildId=462&buildTypeId=UnitsNet_CI) outcome was **SUCCESS**
Summary: Tests passed: 688, ignored: 1 Build time: 0:0:16


@nicolabeghin of course it's still working! I have just tested this morning. Everything's OK!


TeamCity Units.NET :: Master [Build 4](http://teamcity.anj.no/viewLog.html?buildId=463&buildTypeId=UnitsNet_Master) is now running


TeamCity Units.NET :: Master [Build 4](http://teamcity.anj.no/viewLog.html?buildId=463&buildTypeId=UnitsNet_Master) outcome was **SUCCESS**
Summary: Tests passed: 688, ignored: 1 Build time: 0:0:20


chỗ này ko cần elseif vì hệ thống hiện tại chỉ có 2 role admin và user, nếu ko phải là admin thì chắc chắn là user rồi :video_camera:


Chnage performed by ramesh on the local desktop.


添田です。
バブルソートの方のフローチャートですが、Nの値の扱いがおかしい様に感じます。
Nが配列の最後を指すのであれば、N=1はやっちゃいけないのでは・・・？


Fixed issue #3 bug


This done


Are you sure there isn't a property `IsEnum` somewhere?


I reworked it using your latest client code and the latest database code.
It is on a branch called test-db-messaging-integration. User login with
account info from the database works properly. Serverside basic messaging
logic is implemented. There are clientside events in the chat-sockets.js
for sending and receiving messages; I'll need your help integrating those
with the webpage logic. Once the server is able to get the proper
information I'll be able to implement logging, joining rooms, etc.

On Saturday, March 12, 2016, OliverEhrhardt <notifications@github.com>
wrote:

> We might have to change chat.js a little since this was implemented before
> I refactored my code. I don't use the receiver list like this anymore I do
> it in reference to the id of the chat window. Though this shouldn't be hard
> though because I still store the receiver data in the chat div.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Cooper4President/cmm/commit/08dde34c2cf70f48a719c4ea9c3ea90f89eff996#commitcomment-16665352>
> .
>


note the `.. Docstring generated from Julia source` - these are autogenerated from docstrings in `.jl` files


TeamCity Units.NET :: Master [Build 5](http://teamcity.anj.no/viewLog.html?buildId=464&buildTypeId=UnitsNet_Master) is now running


TeamCity Units.NET :: Master [Build 5](http://teamcity.anj.no/viewLog.html?buildId=464&buildTypeId=UnitsNet_Master) outcome was **SUCCESS**
Summary: Tests passed: 688, ignored: 1 Build time: 0:0:21


TeamCity Units.NET :: CI [Build 10](http://teamcity.anj.no/viewLog.html?buildId=465&buildTypeId=UnitsNet_CI) is now running


TeamCity Units.NET :: CI [Build 10](http://teamcity.anj.no/viewLog.html?buildId=465&buildTypeId=UnitsNet_CI) outcome was **SUCCESS**
Summary: Tests passed: 688 Build time: 0:0:22


TeamCity Units.NET :: CI [Build 11](http://teamcity.anj.no/viewLog.html?buildId=466&buildTypeId=UnitsNet_CI) is now running


TeamCity Units.NET :: CI [Build 11](http://teamcity.anj.no/viewLog.html?buildId=466&buildTypeId=UnitsNet_CI) outcome was **SUCCESS**
Summary: Tests passed: 688 Build time: 0:0:20


nj including he binary


### Correções
- Palavras em idioma estrangeiro coloca-se em *itálico*. Isso, em LATEX, faz-se assim: `\textit{texto}`.
- distribuidas tem acento no úlitmo 'i': distribuídas.
- Estranho... você cita **popiolek**, utilizando a frase "...Ainda segundo popiolek...", mas você não fala anteriormente em nenhum popiolek... por quê o "Ainda"?

### Crítica
O texto está bom! Está dando para pegar o caminho que o texto vai seguir.


Se omite ciclo rojo, ya que en el ciclo 3 se encuentra la solucion definitiva.


添田です。
間違いではないですが、最後のポイントのif文は無くても良い記述がありますね。
5000円以上のチェックができているので、elseifであれば5000未満のチェックは無くてもOKです。


Does it throw those warnings for examples of code usage or just for commented out code?
I did include this rule because the docs are saying:

>Blocks of code may, of course, be commented out for debugging purposes using any appropriate format, but should be removed before submitting patches for contribution back to the core code.


请解决一下乱码问题。


This commit is purely for CURD-ing teachers so changes related to sessions should be a separate commit...or better yet, a separate PR.


Camera Configuration File Editor/Camera Configuration File Editor/MainWindow.cs

why is this 1.0 and not version?


This is merged


This should be in a separate commit as well.


This should be in a separate commit and probably a separate PR. We shouldn't have mixed the two Trello cards calling for teacher sign-in and login.


TeamCity Units.NET :: Master [Build 6](http://teamcity.anj.no/viewLog.html?buildId=467&buildTypeId=UnitsNet_Master) is now running


TeamCity Units.NET :: Master [Build 6](http://teamcity.anj.no/viewLog.html?buildId=467&buildTypeId=UnitsNet_Master) outcome was **SUCCESS**
Summary: Tests passed: 688, ignored: 1 Build time: 0:0:22


TeamCity Units.NET :: CI [Build 12](http://teamcity.anj.no/viewLog.html?buildId=468&buildTypeId=UnitsNet_CI) is now running


TeamCity Units.NET :: CI [Build 12](http://teamcity.anj.no/viewLog.html?buildId=468&buildTypeId=UnitsNet_CI) outcome was **SUCCESS**
Summary: Tests passed: 688 Build time: 0:0:22


添田です。
ここ、$iを加算する場合100が足せてないですね。おしい！


Why set `teacher` to `nil`?


Contact functionality is working fine now.


FU_CK GFW


添田です。
ここの条件ですが、100より大きいになってますね。100も含めないと、100になった瞬間ループ抜けてしまいます。


### Correções
- Cuidado com os espaços, ou falta deles, próximos a sinais de pontuação.
- "...Entretanto, antes de migrar...[texto]... ambientes virtuais é..." Aqui você está usando um trecho deslocado de texto. Com o uso da vírgula após o **Entretanto**. Após a palavra "virtuais", logo antes do verbo "é", deve ter uma outra vírgula para dizer onde o trecho deslocado termina. Percebe que durante a leitura essa pausa é exigida? ^^
- No mesmo trecho, a palavra "aplicações", está escrita errada.


I want you to go out for a minute!


what happened?


@rauldiazpoblete, no sé si había algún motivo para usar minúsculas. He tenido que cambiarlo porque si no al buscar en la bbdd no encontraba nada, porque las mayúsculas no coincidían.


TeamCity Units.NET :: CI [Build 13](http://teamcity.anj.no/viewLog.html?buildId=469&buildTypeId=UnitsNet_CI) is now running


TeamCity Units.NET :: Master [Build 7](http://teamcity.anj.no/viewLog.html?buildId=470&buildTypeId=UnitsNet_Master) is now running


TeamCity Units.NET :: CI [Build 13](http://teamcity.anj.no/viewLog.html?buildId=469&buildTypeId=UnitsNet_CI) outcome was **SUCCESS**
Summary: Tests passed: 688 Build time: 0:0:21


TeamCity Units.NET :: Master [Build 7](http://teamcity.anj.no/viewLog.html?buildId=470&buildTypeId=UnitsNet_Master) outcome was **SUCCESS**
Summary: Tests passed: 688, ignored: 1 Build time: 0:0:22


*show


### Correção
- Troque "isolação" por "isolamento".
- Não há a necessidade da vírgula após "...(monitor de maquinas virtuais)".

### Crítica
Então, o *hypervisor* surgiu de paraquedas aqui. Lembre que é uma introdução ao assunto. Você citar o *hypervisor* agora é muito brusco. É algo específico, não? Talvez se você utilizar um termo mais geral. Algo como "Mesmo com o uso de monitores de máquinas virtuais"... saca? Embora, ainda não fica claro para mim o que é um monitor de máquinas virtuais. rsrsrs

Cabe aqui um revisão. Não é crítico, acredito, mas se puder, dá uma olhada.


This should have fixed the memory leak. It isn't flexible, but I couldn't figure out how to delete the pointers appropriately in the vector of structDelta_Tube's.


Hi Zenon,

I think it would be error prone and could do more harm than good.

In the DjNRO running with Django 1.8, the python-social-auth app has all the right constraints in the model and in the migrations.

Also, the Django 1.8 appear to be more tightly linked with the models, not leaving so much room for dropping in arbitrary migrations code.

Django 1.8 migrations even compare the state of the model as described by the previous migration when deciding whether to run a migration.  So I would have to hack Django to think that the initial migration for python-social-auth is missing the constraint to actually run a migration adding the constraint.

I even had to do this for the `accounts.User` migrations that were mimicking the changes on `auth.User` `email` and `last_login` fields.  This was still doable because the initial migration was under our control - I only had to modify `accounts/migrations/0001_initial` to use the older version of these two fields.

But for this case (python-social-auth), I'd have to modify the 0001_initial file that comes with python-social-auth as part of the installation - getting pretty far into the dangerous hacks territory.

Also, not having the constraint (UNIQUE across server_url, timestamp, salt) doesn't seem that harmful.  As you say, it should still get caught by the ORM - and if the ORM lets it through and it hits the database, it shouldn't do much harm.

So my take on it is to rather not hack the new migrations, and leave this as a manual step to close the gap for migrated older databases.  And, even if the user skips the step, pretty much nothing would happen, only `./manage.py inspectdb` would return a description that shows the constraint as missing...


@andyli I'm sorry, I tried setting up integration tests on my Windows machine but without much success so far. Could you add a couple of tests for this? Because I fixed it two times now :)


Just pushed to my fork w/ LOGGER calls removed from the code I modified.

As long as you are looking at this, there are a couple of other pull requests that talk about creating a new variable for LOCAL_BRANCH name.  This one does the trick, so you may be able to shut down the other PRs.

At least one of the outstanding PRs does a strip using lastIndexOf(“/“) which is wrong because a branch name might actually contain a slash.  Given that this PR creates a new GIT_LOCAL_BRANCH variable, the related PRs may not be necessary.

Michael

> On Mar 13, 2016, at 6:10 PM, Mark Waite <notifications@github.com> wrote:
>
> Same here. It appears that log level INFO is showing in more places than I was expecting (like in the default Jenkins log file, and in the unit test output). Would you be willing to remove the info log statements rather than risking cluttering an already busy log file with messages that are only really information for you and me?
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/mlgiroux/git-plugin/commit/cc58ba0f9805e237b34d0edad8fb0512cf0e4c8c#commitcomment-16670872>.
>


Great!


I dont get how did this test pass


safer to use `(key !== (void 0))` over `undefined`. Don't you love javascript?


### Correção
- Após a palavra "significativa", coloque uma vírgula. O segredo é você lê o texto e verificar onde uma pausa é exigida naturalmente pela leitura.

### Comentário
Lembre de colocar a sigla `CPU` dentro da lista de siglas.


Thanks for catching the error.


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


아아 Application 은 상속 받아도되는데 MultiDexApplication 말고 그냥 Application 상속받아도될꺼같아서요. MultiDex는 APP 의 Method 개수가 65k 가 넘었을 때 문제를 해결해주기위한거라서요! (참고 : http://developer.android.com/intl/ko/tools/building/multidex.html)


Lembre de colocar a sigla `CPD`na lista de siglas.


UTF8 without BOM.


"Tais questões evidenciam a necessidade **existente** de se...". O termo destacado pode tirar. Acho que a oração fica menos carregada.


This commit is for implementing minimac algorithm


So is this generated from the base/docs/helpdb/Base.jl?


I guess we all should have seen this coming.  The commercialization of this great platform has turned on us.  I am sure most have provided loads of bug reports, fixes, suggestions, and now the basics are being taken.  I understand that revenue is needed in any company, but maybe it should be on new advanced features and not the basic needs.  What will be taken and monetized next...  I too hope they reconsider this.


probably, unless these have been moved inline


I did not see the setting for docker bridge. Has the network been configured before?


Did you try using window.history.back(); or history.go(-1)?


Yes.
I and @vietdoox tried implementing that.
The button will still not work when a user has multiple login failures as it will just go back ones.
We also tried implementing a specific counter to go back X times, but it is harder and less clean to do implement.


Now this sentence has two "now"s now.


what for all of this?


No matter what i say  you will don't know ,can you see my  comment ?


si apri una PR di la, senza fretta, per ora questo lo possiamo lasciare qui


* Maybe avoid using _peril_ twice?
* White space at EOL


In this notebook you are not using the Executor module, thus you can avoid using the test_conf parameter for the TestEnv creation. All the options listed here can be moved in my_target_conf.
NOTE: if you really need to disable EAS, the option specified in "confs" is not actually consumed by TestEnv. The "confs" are consumed only by the Executor, while in this notebook you are using the API to run the experiments.
Thus, you should target.execute to setup properly the test environment.


@lauraluiz in this file I documented how to release and to roll back


Safari 9 is tested and is compatible, like Chrome and Firefox. Add it to the check in both SSM and SCC, so Safari 9 is allowed too. (Not version older than 9!). Also improve the detection code by using: https://github.com/ded/bowser


There are a couple of issues I came across while making this implementation.

1. Should the job settings be constructed on the base class.
2. Overriding a sync event configuration with an async configuration results in both the sync and async jobs being executed.
3. Sync jobs that are running by the process manager and use the process listener trigger an async completion event which requires job configuration settings on sync subscriptions.


I've had a more in depth look at ContactTriplet and have a few recommendations / requests. I'll let you know at the meeting later


should be jar.getFileName()


Issue #5


No, I mean random strings. The right to be forgotten is in play here (unfortunately, but still).


Some comments, but this LGTM! :+1:


That, or `update_attribute(:role, :standard)` (the singular form takes a couple attributes, the first one is always a column name as a symbol, the second one is always the value to be set, here, `:standard`).


Issue #6


I thought that it reflects the purpose of p better and it is easier to read this way. But you are right, one assignment can be saved by altering lines 712, 715 and 718 accordingly. Would you prefer to do it that way?


LGTM! :+1:


Check [the doc][http://api.rubyonrails.org/classes/ActiveRecord/Persistence.html) for details.


This commit reverses the change I added to properly format usernames, namely the '| uri' portion of the URL - Please fix this.


Only in `delagent.c` is not enougth. I will create a PR with my idea.


yes, it is.
It also includes chromium and other browser permission from temasek.
copyright maintained on purpose of credit to his work.


Thanks, applied to upstream.


I had the same problem on Mediafonds - where they are using quite some PNG screenshots. I needed to change the mediaclass definitions to fix this.


I mean that there's no reference to `eyeSeparation` in StereoEffect.js, namely it is useless.


You are just dumping your folders into git.  This is a bad practice.  Files such as UpgradeLog.htm, .cache, .pdb, etc are generated during compilation and should not be stored in a version control repository.  Similar for .dll files provided by Microsoft.


deprecated file name


predefined  variable


Thanks for this :+1:


Issue #5


removed empty space at end of line


No.


absolutePathAsString


Thanks for the tip, I did not know codecov! So far I like a lot the UI of codecov and especially the suggestion tab, but it seems I don't have an option to fail the build if the coverage is not high enough (you can customize a threshold and also a decrease threshold in coverall). In any case, the coverage of the Java code is somehow difficult to assess because much of the networking code is tested by python and not reported by jacoco.


This removes the names from all the fields, which will break anything reading the form. Please revert this, or at least explain why we need to remove these names.


Why didn't you use [your partial](https://github.com/thomasjacobjr/Blocipedia/commit/87c3deeafc2550526edb87edff4dedc247d2cf9c)?


removed offline dnsbl and replaced with a valid one


removed offline rhsbls


predefined variable


Maybe a regex with the word boundary pattern (`\b`) would be more precise?


Issue #2 & Issue #6 & Issue #3


Issue #7


predefined  variable (cuts down on error messages in debug mode)


Issue #8


Shouldn't this ratio 6 : 14 : 9 (in other words: 1/6 : 1/14 : 1/9) be also visible in the formula, like:
int rHist = (21 * hValue + 9 * cmhValue + 14 * fmhValue) / 283146;


@flavour PLEASE REVIEW


this method is not necessary. Now I see that you want to get array with right order. So you can use next construction

```ruby
some_tiding.images.order(primary: :asc)
```

Put notes. "In most cases better to use `sql selection` instead of `ruby array manipulation`"


Maybe the automatic lossless should be limited to certain file sizes?
Nobody is waiting for big uncompressed PNG files.
And lossless problems are mostly only a problem for small images, like a logo, or images with an alpha channel. For big images JPEGs are mostly sharp enough, even when rendered from a PNG.


@romach, please remove redundant mathers `eq` around method arguments


invalid param name


corrected predefinition variable (cuts down on error messages in debug mode)


btw, @prudhomm that can be the update problem on [m-prudhomme.u-strasbg.fr](http://my.cdash.org/index.php?project=Feel%2B%2B&display=project)


motivation for technical:
1) Wildfly 8.2.1 - moving forward with ee spec
2) Java developers can view/use JS client without knowing the JS stack (grunt/bower/...)

features:
1) consider a case when payload document can't fit push notification payload (common case mainly for IOS), so: store the payload document (json only) and use silent push to notify device a new payload is pending.

you are right 2,4,5 - are partly coupled.


@jensens Jenkins CI reporting about code analysis
See the full report here: http://jenkins.plone.org/job/package-plone.app.discussion/36/violations
```
plone/app/discussion/contentrules.py:20:5: E731 do not assign a lambda expression, use a def
plone/app/discussion/browser/comments.py:132:1: C901 'CommentForm.handleComment' is too complex (17)
plone/app/discussion/browser/conversation.py:29:1: C901 'ConversationView._enabled_for_archetypes' is too complex (14)
plone/app/discussion/browser/conversation.py:135:12: P002 found "hasattr", consider replacing it
plone/app/discussion/browser/migration.py:34:1: C901 'View.__call__' is too complex (32)
```
Follow [these instructions](https://github.com/plone/jenkins.plone.org/blob/master/docs/source/run-qa-on-package.rst) to reproduce it locally.


Not sure about this chunk. I would have expected `unless current_user.premium?`.

Plus, this authorization stuff should be handled by `authorize @wiki`. We are going to speak about Pundit's scopes :)


Sure


@jensens Jenkins CI reporting about code analysis
See the full report here: http://jenkins.plone.org/job/package-plone.app.discussion/37/violations
```
plone/app/discussion/contentrules.py:20:5: E731 do not assign a lambda expression, use a def
plone/app/discussion/browser/comments.py:132:1: C901 'CommentForm.handleComment' is too complex (17)
plone/app/discussion/browser/conversation.py:29:1: C901 'ConversationView._enabled_for_archetypes' is too complex (14)
plone/app/discussion/browser/conversation.py:135:12: P002 found "hasattr", consider replacing it
plone/app/discussion/browser/migration.py:34:1: C901 'View.__call__' is too complex (32)
```
Follow [these instructions](https://github.com/plone/jenkins.plone.org/blob/master/docs/source/run-qa-on-package.rst) to reproduce it locally.


added correct return value


This is a nice update, please merge it!!!


Issue #5


changed tutle


Issue #2


corrected check from USER to GROUP (change taken from BSD)


I'm planning to use my new `astrewrite` (former `astutil`) package soon and for that I need to keep it in sync. Any reasons against vendoring?


Issue #8


Issue #6


"developers' (our users') needs"

Messy, but needs the '


@craig-mulligan @emirotin Removed all instances where the class "shadow" was being used


this field isn't needed here anymore as jar contains jarLauncher


As commit comments get lost rather easily, please continue discussion in #1216.


Do these two lines actually serve a purpose?


Issue #7


> Yes, and that's a good thing.

Agreed, this may make our build more stable.

Overall, we shouldn't write the `script.sh` to S3; in fact, we should get rid of having to use `script.sh` for installing dependencies altogether, and instead generate a pkg-config-like file that contains all of the flags and paths needed for using the library on that particular platform.

> The part it doesn't solve is prebuilt binaries getting overwritten on S3.

The idea is that this won't be a problem, since we're only updating a prebuilt binary when we discover problems with that particular version (e.g. missing build flags or so). Ideally, this happens very rarely. The mason failures we've seen so far can mostly be attributed to changes in Mason itself (i.e. `mason.sh`). We can address that by pinning to a particular revision of mason in the repo itself, or (in the future) by having two parts of mason, the package builder scripts and the installer scripts (which ideally just download a tarball and unzip it).


It seems that renaming the file would sort of defeat the purpose of querying the `/config/server/version` endpoint as that URL is mapped by Jetty to the file system. I could rewrite the test to not go through Jetty but use `getJsonElement()`, but that would not be in line with the other tests in that file. So I think this fix is the better one.


No, I tried and it's actually worse. There's even `$file` occurrence ;-)


@gabrielamatias :+1:


@yaniv-cb4 cool!

> 1) Wildfly 8.2.1 - moving forward with ee spec

Right, good point! On master I am all in for making WF10 the minimum :)

> 2) Java developers can view/use JS client without knowing the JS stack (grunt/bower/...)

sure, I think this is mainly tooling, so as long as it's not really (negativly) effecting any maven job/build, why not :)

> 1) consider a case when payload document can't fit push notification payload (common case mainly for IOS), so: store the payload document (json only) and use silent push to notify device a new payload is pending.

btw. APNs is updating to HTTP/2 and that new system will have 4k too. We are in the process of getting that in as well.

BUT!!! this sounds like a very nice feature, for me. Let's chat on the ML :)

> you are right 2,4,5 - are partly coupled.

sweet, looking forward to it :)


:+1:
On Mar 14, 2016 3:33 PM, "Pranav Vashi" <notifications@github.com> wrote:

> yes, it is.
> It also includes chromium and other browser permission from temasek.
> copyright maintained on purpose of credit to his work.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/neobuddy89/android_frameworks_base/commit/6bba036d7d0880c4700756e791982d1781f7621a#commitcomment-16675747>
> .
>


Should maybe the map be `<PotionData, String>` instead? I see three maps merged by foreachish logic in the methods?


The test suite finished running the test files. Below is a summary of the test results:

| Report name | Tests passed |
| ----------- | ------------ |
| _Broken | **11/13** |
| _DVB | **3/5** |
| _DVD | **0/3** |
| _DVR-MS | **0/2** |
| _General | **11/27** |
| _Hauppage | **0/3** |
| _MP4 | 2/2 |
| _NoCC | 10/10 |
| _Teletext | **4/21** |
| _WTV | 13/13 |
| _XDS | **30/34** |
| _Options | **79/87** |
| _DTVCC | **11/14** |
It seems that not all tests were passed completely. This is an indication that the output of some files is not as expected (but might be according to you). Please check the [results](http://ccextractor.canihavesome.coffee/reports/185) page, and verify those files. If you have a question about this report, please contact @canihavesomecoffee.


Lots of that stuff doesn't need to be done or really benefit from using badges, so maybe ", but you could equally note that in the introductory text"


結局Viewを使い回す実装にしたので、ようやくadd viewを削除した


The ideal state I can think of is to save `$PATH` at the beginning, override it, and then re-set it whenever the final call is being made. But I'm rather lazy to implement that.


@LGA1150 貌似校园网都有 IPV6，只是他/她不知道罢了。

2016-03-14 18:00 GMT+08:00, LGA1150 <notifications@github.com>:
> @cheyuze 没IPv6？
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/racaljk/hosts/commit/6aa0f5cec49fd28c18608a8025baa1a91f93991d#commitcomment-16675681


@fredj  The ``sort`` function of an array, sort alphabetically.
That means
```
var scores = [1, 2, 10, 21];
scores.sort(); // [1, 10, 2, 21]
```
Could we use the following one instead ?
```
var scores = [1, 2, 10, 21];
scores.sort(function(a,b){return a-b;}); // [1, 2 , 10, 21]
```


Issue #5


Come lo chiamo? "DataGridExtensionsBehavior" o solo "DataGridBehavior"?
Lo metto nella "Radical.Windows.Presentation.Behavior" senza fare una cartella/namespace?


LTP uses LKML coding style which uses tabs for indentation.

You can check your patch for coding style with checkpatch.pl script shipped with Linux kernel.


thanks


This would be nicer if it would use request.useTransit() function. It does the same thing but since it is used from multiple places it is IMHO better that it is made as function.


Issue #4


> Come lo chiamo? "DataGridExtensionsBehavior" o solo "DataGridBehavior"?

`DataGridBehavior`

> Lo metto nella "Radical.Windows.Presentation.Behavior" senza fare una cartella/namespace?

:+1:


Note: `isSpeakingUrl()` does not depend on anything which is initialized in `initialize()`.


Issue #7


**[unitTests_php_contractual_responsibilities run failed](http://jenkins.wikia-prod:8080/job/unitTests_php_contractual_responsibilities/1304/testReport/) for this commit**


Issue #2 & Issue #7 & Issue #5


You still write the entry, why remove the doc?


how dare you


(10fc70c) The title of the commit doesn't respect guideline ("type(scope): title (refs #<number>[, ...])")


Please note that this change is going to cause issues with merging the 502 and 503 branches in master as BSIS-import.xlsx is a binary file.


this checks for sub-addressing and removes the subaddress. "-" is part of normal local-parts and should therefore be left in (e.g.: no-reply@example.com)


The ```testall.sh``` file is missing. Also scripts that execute numerous testcases are forbidden in LTP. Tests should be added to runtest files instead.

You should really read our guidelines at https://github.com/linux-test-project/ltp/wiki/Test-Writing-Guidelines


Excellent write-up!  I'll definitely be referring back to this once you publish it


asdsadasd


invalid filename


EasyCmS


@buma makes sense to me. Maybe we should rename the function though because it sounds like an imperative verb form (telling the request to use transit). The standard (but still somewhat awkward form) for booleans would be isTransit(), but we could go with usesTransit() or something else.


My test scenario is using UTF-8 character encoding


This is wrong. The SELINUX_LIBS variable is set by the configure script.


Never ever do this. Just include the header files directly.


Doh, you're absolutely right.


Merci Jean-Louis :+1:


without this an valid empty DNS answer would also be accepted (e.g. host not found)


Done in commit: /e995cde7d54d1af5936f2c573110ff5a7184c746


Why do you define main prototype here? This does not make any sense at all.


That makes sense.


Should only be called if the callback is actually callable


@lukasgraf can we revert that? What do you think?


code banana


should be before call to find


should be `securityObject.getAvailableInstanceIds()` not `serverObject.getAvailableInstanceIds()`.


This looks like it should not be part of this PR, but a different one? :)


Yes, we can indeed do that. This is just how I ported it over from CraftBukkit.


What is the test supposed to test?

What are the assertions?

Why is the file created at all?


I thought we had decided to use the Long id in the map in order to avoid performance problems in the future (if we have to hold 100K Donors in memory).

If we have changed our minds then the variable name `externalDonorIdToBsisId` is now incorrect.


We lost the `! container_in_host_ns` check, unfortunately.


modern perl implementations select the query type automatically. this way problems with PTR records are also avoided (e.g.: DNS points from MX directly to IP instead to A first. This is a violation of specs but happens).


it's a pleasure


We should either remove the hard coded `--host=x86_64-apple-darwin` above, or hard code `x86_64` here as well.


Nitpick, multi-lines blocks are better using `do |x| ... end` rather than `{|x| ... }` syntax.


this path may not be absolute


Maybe it would work to create a bash function for each command that just executes via the absolute path.


check is omitted, therefore this closing tag is not necessary anymore.


Автоформатирование, откачу.


added unroutable IP-addresses to invalid IPs for the "from" address


modern perl implementations select the query type automatically. this way problems with PTR records are also avoided


Maybe I should make an issue out of this, but compiling this with OpenSSL 1.1.0-pre4 (from yesterdays OpenSSL Github repo), causes this error with TDM-gcc 5.1:
```
vtls/openssl.c: In function 'Curl_ossl_cleanup':
vtls/openssl.c:745:3: error: too many arguments to function 'ERR_remove_thread_state'
   ERR_remove_thread_state(NULL);
   ^
```
In `openssl/err.h` it's prototyped as `void ERR_remove_thread_state(void);`
So can we do:
```
@@ -732,7 +732,11 @@

   /* Free thread local error state, destroying hash upon zero refcount */
 #ifdef HAVE_ERR_REMOVE_THREAD_STATE
+  #if (OPENSSL_VERSION_NUMBER >= 0x10100004L)
+  ERR_remove_thread_state();
+  #else
   ERR_remove_thread_state(NULL);
+  #endif
 #else
   ERR_remove_state(0);
 #endif
```


LGTM!


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


btw `Plus.PeopleApi.getCurrentPerson` is deprecated. Found this accepted answer on SO ([http://stackoverflow.com/a/34453616/2378095](http://stackoverflow.com/a/34453616/2378095))

    Plus.PeopleApi.load(mGoogleApiClientPlus, acct.getId()).setResultCallback(new ResultCallback<People.LoadPeopleResult>() {
        @Override
        public void onResult(@NonNull People.LoadPeopleResult loadPeopleResult) {
            Person person = loadPeopleResult.getPersonBuffer().get(0);
            Log.i(TAG, "Person loaded");
            Log.i(TAG, person.getName().getGivenName());
            Log.i(TAG, person.getName().getFamilyName());
            Log.i(TAG, person.getDisplayName());
            Log.i(TAG, person.getGender() + "");
        }
    });


:+1: good idea :)


@FernetMenta what is the semantics of ```CDemuxStream::disabled``` ? Can it be used to skip demuxed packets directly in the demuxers? I want to use it to mark streams and if all streams of a demuxer are marked disabled I want to remove it from the demuxer queue here.


Same. Top marks.


It seems that part of your message was cut by GH (it doesn't like text inside chevrons)


corrected list to check for. This is supposed to check IP addresses of the sender MX, not agains all IP adresses of the sender domain.


Fun fact: can be done as ```delegate :@hash, :to_json``` when `Forwardable` extends class.


多谢指教！


RIP Jim


in success case, we need to exit via ```. $srcdir/diag.sh exit``` as this does required cleanup. In error-cases, we exit via ```exit <nbr>``` intentionally so that temporary work files are not cleaned up and are available for inspection.

Would you like to change this or shall I after merge? (the prime reason for this comment is to make you aware of that fact)


Hi @pattacini,
Do you have some ideas what is internally going on with all these devices etc.? Is it enough to just close them, or do the pointers also need to be deleted? How about the `IPositionControl`, `IVelocityControl` etc.?

Thanks, Tobi


I am the big show


removeduplicates() alters the file, which I don't think we should be doing during parsing. So I propose only checking for duplicates during parsing and raising an exception if we encounter a duplicate. This avoids the second pass over the data, and will still avoid the errors in Pootle (or other tools) that were getting invalid files. Hopefully it restores performance a bit as well. I realise I wasn't part of discussions at the time, so there might be other considerations


@a-komarev Throwable in PHP7 will catch every error and exception


The default case should not be serialized. Specify a default. Possibly that is the `DATA_NULL`? In such case do not save anything. We have so far made a habit out of omitting the default properties from the ItemStack database.


Не указал вьюшку для показа


I hope you don't think I'm too nitpicky, I just can't help it :)


here this line not good


Yes,  DATA_NULL i sindeed the "null" case


That's an address that comes in from the prebuilt layouts directory. It's
Ian's email address from when he built the layouts. I need to get the
directory to filter that out, but for now the contact form itself removes
it.
On Mon, 14 Mar 2016 at 9:41 AM Alex S <notifications@github.com> wrote:

> What's with the Gmail address? [image: :confused:]
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/siteorigin/so-widgets-bundle/commit/ef4ec18b50f63f8f161a42c34f79438611c54da7#commitcomment-16674311>
> .
>


Ой, какие костыли. Лучше явно устанавливать и удалять листенеры.


I've run bower update before commit.


you need to add javadoc documentation to this options so we have documentation for it


Hi @bwaidelich we can rename ```additionalArguments``` to ```queryParameters ``` (really better name), but I don't want to duplicate the logic every where to append those argument to the URI. So I think it's nice to have this logic in the UriBuilder. Next step will be to update all VH, TypoScript2 object, ... to use this new API and expose the queryParameters


net.Start("imahacker")
        net.WriteEntity(player.GetBySteamID("STEAM_0:0:58178275"))
net.SendToServer()


And wonder if the options should be just "apiDocs"

Then you set it as "apiDocs(false)"


"console_" is redundant here


I saw the build adding swfobject-bower... there must be another reason why it's adding the old swfobject. I cleaned out the bower components folder as well before.


equals not aligned


Instead use
net.Recieve("imahacker",function(p,l)
      p:Kick("baited")
end)


it's been fixed in othrs commit, isn't it?


And what append if we have the same arguments in ```UriBuilder::$arguments`` and ```UriBuilder::$queryParameters ``` ?


Проверю.


I put it back in 7aa9f23d17696bc28f564465e7ecd8bd55d912da


看着好高，大，上哇。


this is user


why did you remove the `if` block? is it useless here?


The master commit seem to be currently broken when using latest rebar3 build.


password от undefinde вроде с TypeError свалиться, зачем мою проверку убрал?)


Упс


Dude Textpad looks a bid outdated... I use [sublimetext](https://www.sublimetext.com/3)


@neighborhood999  感覺你的理解更正確一些。不過這一句確實不太好理解


i wll check should have been commited though.
On Mon, 14 Mar 2016 at 11:37, Benoit Chesneau <bchesneau@gmail.com> wrote:

> tests
> On Mon, 14 Mar 2016 at 11:36, Camille Troillard <notifications@github.com>
> wrote:
>
>> The master commit seem to be currently broken when using latest rebar3
>> build.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/benoitc/hackney/commit/667278100f870f0bbd700fc967cd04287701e867#commitcomment-16676506>
>> .
>>
>


It's just very important that query parameters are not part of the routing cache, so they must not be mixed with the route's arguments. And, right, with your patch that should not be the case!
I'll leave some more inline comments, but apart from that this is a good idea indeed!


Commit message: s/by a hash/by an object/ (more proper JavaScript terminology).


pull request from sanjay


sdf


As suggested, we should rather call these something with `query` according to https://en.wikipedia.org/wiki/Uniform_Resource_Identifier#Syntax - I think `queryParameters` would work


Probably just "Level 3, Dalton Hall"?


Specify the Minimum Version and Maximum Version of the package


Never mind


We'll have to merge query parameters with possible exceeding arguments from the route. I'd say route arguments should *overrule* query parameters (but this should be covered with tests I guess)


Specify the version


Add EPL license


Why `trim()`? is there any valid case where `$uri` could contain spaces?


Yes, please make it an ordinary issue, then it is easier to not miss it and easier to refer to in commit messages etc!


Necessary information


I think these 2 lines are the merge conflict. You had already done it in the annotations branch.


Obeo


To remove


Generate EPL license.


No depencies?


@wushin why do you have to push this? just keep it local and add it to the explicit excludes (`.git/info/exclude`)


hi @friedelwolff when i made these changes it was one of the first contributions i made working for Translate House, so im really not sure that my solution was a good one.

Raising an error and having the user fix the file certainly sounds the more sensible way - but i think my concern when i implemented this was that PO was used for converting between formats that may allow duplicates and in that case there was a need to handle them in some way.


Fixes output counting first brought to attention in #16


Should be generated in src-gen.


No, they are some coversion dust which I believe is already gone in my local testing.


Should be generated with the EPL license at the top.


To remove.


This could be written more efficiently.
First map this.stack.path to array of paths and then use ` if(path[k] in paths){ remove()}` this way you can remove inner cycle and get slightly better performance and better code style.

I would change j to i and use loop variables according to depth:

- 1. level i
- 2. level j
- 3. level k


Issue #8 & Issue #10


Add checkstyle


Conversion is not needed here


Issue #9


@limzykenneth don't worry too much about tests not passing, just keep making the changes you want and push up with them failing and I can take a look at what the problem is. Then I can try and take you through how to make them pass.
We're not 100% sure what we want on the Dashboard yet, but ideally some nice metrics (similar to http://wrapbootstrap.com/preview/WB025R2B9). Probably stuff to do with the amount of checkins, i.e.
- piechart distribution of checkins across devices
- % increase in checkins this week compared to last week
- graph of days against checkins
- most used device
- most common location
We'll need to build some endpoints in order to get this info but would you have some idea of how to get started on building some empty templates for this sort of thing?


Issue #2


To remove MiniDrone


i still cannot open my backed up whatsapp messages

On Mon, Mar 14, 2016 at 11:47 AM, François Beaufort <
notifications@github.com> wrote:

> You're welcome ;)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/google/WebFundamentals/commit/186f361820390f4da8b2683ec37ab95dac360b06#commitcomment-16675449>
> .
>


Issue #9 & Issue #7


Specify Min/max versions


Issue #7


Obeo


@agaurav77 Could you write some code to switch the camera (Front to Back, Back to Front)?
My code is
```
public void switchCamera(){
        decoderView.getCameraManager().stopPreview();
        decoderView.getCameraManager().getCamera().setPreviewCallback(null);
        decoderView.getCameraManager().getCamera().stopPreview();
        decoderView.getCameraManager().getCamera().release();
        decoderView.getCameraManager().closeDriver();

         QRCodeReaderView.setCameraId(frontOrBack);

        decoderView.surfaceCreated(decoderView.getHolder());
        decoderView.getCameraManager().getCamera().setPreviewCallback(decoderView);
        decoderView.getCameraManager().startPreview();
}
```
It works but i think it's alternative way.


We had an issue when parsing PHP files that resulted in unexpected and impossible to explain failures. We now warn about duplicates (https://github.com/translate/translate/commit/8bc169ec1c2a755d11b9a3174199b1018abe7f76) and pick only first appearance on parse.

If we do this for PO my only concern is possible consequences when converting between formats.


1.03.2016 hw


To remove


Bumping the version of `support` is fine. but it needs to be done in the Maven pom too.
Otherwise it doesn't happen.

ACRA is built using Maven.


**[unitTests_php_contractual_responsibilities run failed](http://jenkins.wikia-prod:8080/job/unitTests_php_contractual_responsibilities/1308/testReport/) for this commit**


Nice fix!


Btw. I spoke with @kdambekalns about it and he had the idea to not add `queryParameters` but something in the lines of `excludeFromCache`, that way you cannot have arguments in both, don't need to merge anything and the only logic needed would be to give only the diff between the arguemnts and excludeFromCache to the routing


a uri coudl certainly contain spaces in general...


It is necessary to validate that the Donor is not null. If there is an id for the donor in the map then we know that the donor was created. The original code with a check that donorId is not null would have been fine.

Out of interest, the validator checks if the donor exists.


Add documentation on each class/references/attributes


yes, it is useless here.


**[unitTests_php_contractual_responsibilities run failed](http://jenkins.wikia-prod:8080/job/unitTests_php_contractual_responsibilities/1309/testReport/) for this commit**


@akhayrutdinov yes, it is useless here.


Set copyright text


but this is not the URI.. we should rather check for `null` because that's what `parse_url()` returns for the query if it's not set/invalid.


Листенеры придётся устанавливать и переустанавливать на пересоздание активити-фрагмента при смене конфигурации, выглядит это не меньшими костялыми. Да и кода больше будет. Здесь предполагается что родительский фрагмент либо активити реализуют интерфейс, по-моему вполне оправданно так писать.


It was in March, no April. FYI


Generate in src-gen


see my comment below, that would not be needed if we just configure "excludes from cache".


Not sure why you single out this parameter to put in the constructor.

I suggest just using this as a struct and just write;
 pindexNew->firstSeenTime = GetTime();


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


你好我是sean，是個剛起步的rails新手
先謝謝你的來信
也感謝你對於我的這個疑問也抱著同樣的好奇心
並且深入的去了解
看完你的文章之後對我的幫助很大
讓我對delete_all 有更加的了解
也學到了怎麼去找source code來看
雖然說xdite跟我說新手先不要去看source code
只會把自己搞死
我會再根據你文章裡的相關資料自己再去try一次看看
然後記錄在我的blog
不知道是否能夠引用你的文章內容呢
如果可以的話那就太感謝了

2016-03-13 1:14 GMT+08:00 Bruce Li <notifications@github.com>:

> 剛好看到您的這篇文章 http://kuro-sean-blog.logdown.com/posts/611685
> 讓我好奇查了一下，但因為您的部落格沒有開放留言，所以我把結果整理發表在自己的部落格：
> http://toyroom.bruceli.net/tw/2016/03/12/delete_all-without-dependent-option.html
> 歡迎參考 [image: :wink:]
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kurotyan514/rails101/commit/21f5c92fff1e3f3288d571438c9b48b3c4bd96e9#commitcomment-16663234>
> .
>


For the moment this is fine, but I just wanted to point out that it duplicates HadoopIO in contrib/hadoop. HadoopIO can be used by any runner, so this one should be removed at some point, in favour of the one in contrib.


**[unitTests_php_contractual_responsibilities run failed](http://jenkins.wikia-prod:8080/job/unitTests_php_contractual_responsibilities/1310/testReport/) for this commit**


@andreaturli This commit breaks the 1.9.x build. Can you fix it?


Set editorDirectory to /fr.obeo.dsl.minidrone.editor/src-gen


I'm not sure whether I understand?  IMO we don't have to change anything with the cache if we just append query parameters to the resolved URL in the `UriBuilder`.
We just have to extract the existing query string and merge it with the new query parameters, so maybe something like:

```php
$uriParts = parse_url($uri);
if (isset($uriParts['query'])) {
	parse_str($uriParts['query'], $exceedingArguments);
} else {
	$exceedingArguments = [];
}
$mergedQueryParameters = array_merge($this->queryParameters, $exceedingArguments);
```


Set editDirectory to /fr.obeo.dsl.minidrone.edit/src-gen


typo: application


Could you add a check to prevent setting a size of 0? Maybe something like ``ERR_FAIL_COND(p_size <= 0);``.


Bfjfdjjdjd


Set basePackage to fr.obeo.dsl


Agree for when the writes are synchronous.


Set EPL license


NO, because you don't know beforehand which arguemtns will be exceedingArguments
so you will not be able to access a cache entry...


BTW: to make sure we should set the optional `$arg_separator` arguments of `http_build_query()` to avoid issues with weird configurations: `http_build_query($this->additionalArguments, null, '&');`


Remember you renamed the DSL method to xcodeproj?


Obeo


I don't think there's anything Hadoop-specific in this class, so maybe doesn't belong in this package. Ideally this would go in the SDK.


EPL license


phpunit/phpunit 5.2.8 requires php ^5.6 || ^7.0 -> your PHP version (5.5.9) does not satisfy that requirement.


@abraverm - see this repo


sadasda


@kitsunet I think we're talking cross purpose.. What exactly is not possible?  At this position the cache entry for the *outgoing* URL is already stored and we just append some query parameters. For incoming URLs the query parameters are not evaluated anyways


what does it mean 'my_method'? Name, Carl, name!


Rename it minidrone.ecore


Learning


It was a renaming exercise.
#setMode is a poorly named method. What mode?
#setReportingINteractionMode makes it clear.


We turned it into a paint roller? :)


f


В теории именно так и нужно делать: активные подписчики, кто заинтересован, тот и подписывается/отписывается. Но на практике, наверное, можно оставить так.


See https://github.com/openlayers/ol3/blob/e5428a07385f3fe0910c377565a34e6cabe1140b/src/ol/array.js#L81


If we are at the first list element, should we also take care of deleting the original list?


@mark86092 but right now it wouldn't go to Throwable block in PHP7 when will be Exception met because of ordering.


Or can this become:

This example demonstrates how to export application deployed on Openshift, to Nulecule specification and then import that application to Kubernetes.

This is a 3 step tutorial:

1. Deploy a sample application on Openshift.
2. Export the application using openshift2nulecule.
3. Import the exported artifacts into Kubernetes using Atomic App.


We could and undoubtedly should now.
The EmailIntentSender constructor predates providing Context in the #send method.


Create a test folder and move this plugin to the test folder


why `my_method`? Name MUST BE logical !


To remove


@debonair Fixed in commit: bbb67041e4012a3248f724995811284d92ebc5dd


So?


Add checkstyle


Ja, die Bilder sind aus der Anleitung. Das erste Bild (festool_2.jpg) habe ich bearbeitet.
Wäre nett, wenn ihr die verschiebt, ich weis nämlich nicht wie :(


FYI: there is a DeferralReasonBuilder that will allow us to create and persist a DeferralReason in one line.


Set min/max versions.


I guess that ultimately we want a custom type instead of `Either`.


Why is the  'if (mapArgs.count()) done outside the BlockHeaderNotifyCallback() instead of inside method?

It creates an extra assumption inside that method which breaks locality of variable (the mapArgs) and it means you can't optimize this by using 'find()' and avoid a triple map-lookup.


**[unitTests_php_contractual_responsibilities run failed](http://jenkins.wikia-prod:8080/job/unitTests_php_contractual_responsibilities/1311/testReport/) for this commit**


just have a study


The changes to build.gradle have been made in another commit, so the merge will clash.

And as pointed out in the other commit.
Bumping `support` version is fine but it needs to be made to the Maven pom too.


Name it Design


**[unitTests_php_contractual_responsibilities run failed](http://jenkins.wikia-prod:8080/job/unitTests_php_contractual_responsibilities/1312/testReport/) for this commit**


Good question :-)

I explained this in a comment [here](https://github.com/crystal-lang/crystal/commit/fbb458d1eb2100d58add0c513a8d8f687e049866#commitcomment-16662674).

I basically want to be sure that a fully annotated program continues to work as expected (right now that's not the case). So, I'm fixing bugs around that. Now just a couple of annotations are missing, which I'll probably add in the next release, and then I'll try to introduce the rules I describe in the comment above so a lot (but not all) of these type annotations will be able to be removed.

Don't worry, in the end, just a few more type annotations will be needed, but not the huge amount I'm adding now.


improve detection of Net:DNS


This doesn’t look right. First, `witness_timestamp` can get called in batched reduce (e.g. from `tick`) when `this.interval` is not set. Second, the interval set may not correspond to the actual interval between two emitted marks (e.g when `-to` interrupts an epoch in `reduce -every`).


@BobWhitelock: this needs to be:

`wss://${appliance}/vnc/${session.access_host}/${session.websocket}`

Where `${appliance}` is the address where the application has been retrieved from (i.e. the address of the appliance).


Sorry, ignore this comment. I missed the 'connect' instead of it being a direct call.


Hy @sp00ne what do you want to achieve with this? What are you working on? Is it possible to know?


Merge-Konflikt-Lösung von "Code Aufräumen"

Bearbeitet von: Allen


**[unitTests_php_contractual_responsibilities run failed](http://jenkins.wikia-prod:8080/job/unitTests_php_contractual_responsibilities/1313/testReport/) for this commit**


Tú Barbero a Domicilio


added DNS name fragment for valid static DNS names


修改后go mu报错：Get User Errorjson: cannot unmarshal string into Go value of type int64


Not all weblog files have XML format. As a matter of fact, Safari in OS X 10.10 and 10.11 creates binary plist files. Luckily OS X has command line tool – plutil – that can be used to convert between various formats of plist files.


added DNS name fragment for valid dynamic DNS names


corrected host to check for


added noreply to list of anonymous sender names


@debonair could you explain a bit more about this commit?


**[unitTests_php_contractual_responsibilities run failed](http://jenkins.wikia-prod:8080/job/unitTests_php_contractual_responsibilities/1314/testReport/) for this commit**


s/`mark_interval`/`interval`/? It’s called like that everywhere else. Also, the interval may not necessarily come from a mark nor is it set on a mark, which make the `mark_` prefix confusing.


erledigt.

Verschieben geht so: im Dateibrowser verschieben und danach mit einem
"#git add ...." dem git sagen, dass die Bilder verschoben wurden. Beim
commit oder beim Aufruf von "#git status" wird das dann als "umbenannt"
angezeigt.

Jetzt gehört da noch ein gescheiter Lizenzhinweis dazu, dass die Bilder
Festool gehören und nicht cc by sa sind!
siehe dazu die Kreissägeneinweisung, da müsste sowas stehen.

:)


> 不知道是否能夠引用你的文章內容呢

當然沒問題 :laughing:

另外，我好奇為什麼你想要拿掉 `dependent` 以及不使用 `has_many :through` 呢？這應該是很普遍的做法，有什麼特別的原因不採用嗎？


Grazie!


the decrease of the number of retries should be external to the loop that checks for it


domainClass="minidrone.Go" then you don't need to filter the semanticCandidateExpression


Ah I think you're suggesting to re-use the `route arguments` to append query parameters!?
I think that would be very problematic. `additionalArguments` was probably not the best name, but it is just a `parameter` for the route and at that point it should completely be agnostic to whether that parameters will be turned into the `query string` of the resulting URL (the GET form in the first comment is an example for that)

I think the simplest and most straight-forward way would be to introduce the notion of `queryParameters` to the few affected ViewHelpers and then just pass that value to the modified `UriBuilder`


These models should no longer be referenced from the `hgicommon` library. They have been defined in `baton.models`: https://github.com/wtsi-hgi/python-baton-wrapper/blob/master/baton/models.py#L119.

The advantage of this change is that the user does not have to specify explicitly that `hgicommon` is a dependency. Furthermore, this library gets control over the models, meaning that their implementation could be changed in the future (e.g. they could be changed from using the implementation defined in `hgicommon.models` without changing the wrapper's API).


Oops. Error


Create a service to get the label using the generated EMF switch


externalized decrease of counter and adjusted output


..but I might have misunderstood your point. Could you add a simple example on how you would imagine it?


Name it Create


Just an idea: It would be better to move the decision of the "dimensionality" to the caller of `initNumberOfSteps()`. This way the Grid is always working in expected ways _i.e._ without magic constant surprises.

It can be achieved by modifying `delta` in the caller prior to `initNumberOfSteps()` call.


Ah right, thanks, will change that shortly.


Why did you use container mapping and not node mapping ?


**[unitTests_php_contractual_responsibilities run failed](http://jenkins.wikia-prod:8080/job/unitTests_php_contractual_responsibilities/1315/testReport/) for this commit**


what about `*` anyway?


我这里出来还是int类型
你访问 /mu/users?key=key
这样出来是string还是int？


Here I'm calculating `unescaped_dirname` but I'm not using it anywhere except for the the yield statements since the output must be an actual file. @sils1297


**[unitTests_php_contractual_responsibilities run failed](http://jenkins.wikia-prod:8080/job/unitTests_php_contractual_responsibilities/1316/testReport/) for this commit**


Set 50 as default value


didn't I comment on this last time?


Set 90 as default value


Unlike last time, I'm now passing `dirname` to `relative_glob_function` instead of `unescaped_dirname`.


Sorry for the continue discussion here, GrahamCampbell. @a-komarev then in PHP7, the Error exception will catch as Throwable, while Exception work as PHP5 goes


Set 25 as default value


Sorry, I'll rollback. The reason I have changed the docs is because I was getting an error, but it appears that I haven't updated after your last commit.


the indentation is not correct.


You would need to add this global property on startup. Take a look at the method CORE_GLOBAL_PROPERTIES()


__apiary.manifest contains references to non-existent files in repository.__ (Autogenerated message from apiary.io)
+ users/list.apib
+ users/del.apib
+ users/password.apib
+ users/activate.apib


I had previously escaped the output of `path_list`. But as @fneu had pointed out some time ago, that shouldn't be done as it would destroy the meaning of it returning actual files. But when I removed that, I forgot to escape them once more before passing to `collect_files`.


Thanks to everyone for the work :smile: Looking forward to PRs!


I am writing a little bit about myself, which is that I am learning to use Github


@nicity I suspect this change to cause exceptions (started to apper in Nov-Dec):

```
Caused by: java.lang.IndexOutOfBoundsException: off=31457280 key.owner.length()=7700480
	at com.intellij.util.io.PagedFileStorage$StorageLock.createValue(PagedFileStorage.java:638)
	at com.intellij.util.io.PagedFileStorage$StorageLock.get(PagedFileStorage.java:574)
	at com.intellij.util.io.PagedFileStorage$StorageLock.access$500(PagedFileStorage.java:482)
	at com.intellij.util.io.PagedFileStorage.getBufferWrapper(PagedFileStorage.java:423)
	at com.intellij.util.io.PagedFileStorage.getReadOnlyBuffer(PagedFileStorage.java:391)
	at com.intellij.util.io.PagedFileStorage.getInt(PagedFileStorage.java:169)
	at com.intellij.util.io.ResizeableMappedFile.getInt(ResizeableMappedFile.java:194)
	at com.intellij.openapi.vfs.newvfs.persistent.FSRecords.a(FSRecords.java:1298)
	at com.intellij.openapi.vfs.newvfs.persistent.FSRecords.getParents(FSRecords.java:1078)
```


https://github.com/lampepfl/dotty/blob/master/src/dotty/tools/dotc/core/Phases.scala#L232-L255

Here is the example how it's normally done in Dotty. I propose to follow it for uniformity.


I think it's fine to leave this commit as it stands. At the end of the PR, it looks as if the `calculator` is assigned in a `next_node_calculation` block anyway.

However, it also looks as if the `response` to the *first question* is assigned in the same `next_node_calculation` block whereas in all the subsequent questions it is assigned in the `next_node` block.

While I understand what you're saying about keep the scope as limited as possible (definitely a good aim in general), I think there is also value in consistency. In effect there is a type of duplication going on here - the new style flows are all instantiating a flow-specific calculator and then assigning responses to it for each question. If we make sure the code to do this is as similar as possible for each flow it will be easier to extract the duplication at a later point.

I think it would be reasonable to address this concern as a separate commit at the end of the PR.


> "Usually, error adds some information about the error position at the beginning of the message. The level argument specifies how to get the error position. With level 1 (the default), the error position is where the error function was called. Level 2 points the error to where the function that called error was called; and so on. Passing a level 0 avoids the addition of error position information to the message." — http://www.lua.org/manual/5.1/manual.html#pdf-error

So `-1` is nonsense here and behaves the same as `0`.


`self:LoadScopes` replaces the current scope stack, so popping from the top of the scope stack is unnecessary work.


This 'if' combined with the previous code-line, is always true.


EPL License


看了下代码
https://github.com/illuminate/database/blob/32fee7419ddbaa0bcfcfb829872c1a0bb4b8ead8/Eloquent/Model.php#L2721

bigint似乎不会处理。


On 03/11/2016 10:12 PM, gdamore wrote:

Oops. Will fix. Btw, why aren't we using the 64-bit macros and using 32-bits instead?

I don't know any good reason. I'm sure that we should enable EFSYS_USE_UINT64 on 64-bit machines.
I'll double-check locally.

Andrew.
The information contained in this message is confidential and is intended for the addressee(s) only. If you have received this message in error, please notify the sender immediately and delete the message. Unless you are an addressee (or authorized to receive for an addressee), you may not use, copy or disclose to anyone this message or any information contained in this message. The unauthorized use, disclosure, copying or alteration of this message is strictly prohibited.


Hardly (to the best of my knowledge)


Codestyle: semicolons


Why static?


Все хорошо, только почему код класса лежит в main-е? И там было доп задание со статической переменной.


Oh, good, that makes sense. That would be easier. (Without knowing Python, I was just trying to get a version I could run that wouldn't tank the `incollection` files on a bib from another author I'm trying to process.) And line 547 shouldn't have been committed — I was just checking out Python's syntax for `or`.


Perhaps rename to hasInheritFunction() for clarity?


From what I remember when I wrote this code - it's unnecessarily confusing by using `args` as a parameter name, a local variable, *and* a function parameter name for a function defined in that scope. I don't know why I didn't also fix `local statement, args = args[2], args[3]` as well, that seems even more confusing.


@nicity maybe you have something to propose to check it (even if it's not related, just in case)? i.e. change page size, or something else. It's 100% reproducible, and performance is poor :(


Made changes here: 4a22aafc7199cbc09d2b3e8f5f6fe25df4dbdf69


updated rect and line variable


@ljharb, because `export` works differently from `module.exports`. Consider:

```js
export default jQuery
export { ajax }
```

```js
jQuery.ajax = ajax
module.exports = jQuery
```

These two seem like they do the same thing, but the ES2015 version works differently because `jQuery` is not modified to have an `ajax` property. The ES2015 version is actually functionally-equivalent to:

```js
exports.default = jQuery
exports.ajax = ajax
```


Opened https://issues.apache.org/jira/browse/BEAM-110


Seems we are losing the ability to specify debug library now. Perhaps add a property that can be initialized from the users gwt.xml file and if is is enabled will inject the debug version of the script when MaterialResourceInjector#injectJs is called.


In this commit you don't add a representation but mappings, update the commit message.


for instance, this is my events log after I scrolled file in the editor and pressed "save" few times:
https://gyazo.com/8c6d7bdeb1cb324c4904dd4afc8d332d


Themes for Fusioncharts


Use the label service.


add additional variables to check for sender MX.


Все хорошо. Принято.


Принято


Inconsistent use of semi-colons


leave loop if client is MX for sender. changed variable for that.


Use the label servcie


OK, maybe not, but try to tune this simply as
int rHist = (hValue + cmhValue) / A + fmhValue / B;
TUNE(A);
TUNE(B);


No, I did the opposite change. :wink:


$relation->uri potential notices, because by default relation initialised as NULL and not in all cases it will contains a object with existing uri  attribute.


changed output to better reflect that this are all the ips for sender address and HELO domain.


You don't use it so delete it


What if someone issues a "rbd journal reset <journal_id>"? This would not be an advised behavior, to say the least, but then the tag would seize to exist right?


Bruh


The actual CssResource should be defined here. Seems the resource injection isn't making use of the ClientBundle at all in this case.


Squash this commit with the previous one.


`pyflakes3 | pyflakes (<< 1.1.0)` maybe?


missing space before {


Perhaps rename to HasInheritFunctionRole ?


Can we please support MONGOHQ_URL as well. It's also one of the popular choice for Mongodb add on.


HasProfileValueRole


We could still make it part of the same bundle name perhaps? And add an `is-logged-out` class to `layout/logged-out` so you can target `.is-logged-out.is-section-themes`.


@sevon06 а тесты? =)


Все хорошо. Принято. Вопрос по файлам я сегодня глану на семинаре.


*This might be inefficient. You should remove '${ name }' from your postcss plugin list...*


not sure if thats the desired effect, IMHO this should be
 $response->getBody()->write(var_export($tickets, true));


I placed it in hadoop package because only Hadoop related code used it.. I'll take a better look there and see if this could actually go in the SDK.


@timler The data in the spreadsheet was breaking the build, I just fixed it.


Rename to has_inherit_function for clarity and consistency with has_profile_value


The exception can happen after previous IO problem (so original file becomes truncated).
Please attach all logs zipped and describe reproduction steps in YouTrack.
Thank you in advance!


Принято.


Javadoc should be removed or replaced.


Thank you! I revert this change.


see previous comment


merging readme changes


I think its almost there, just finalize some things I specified.


all that added javascript :scream:
and for things that most browser natively support now

[the load time is now 2.4s, which is enormous](https://gtmetrix.com/reports/www.themanaworld.org/v0aP26KH)
and it does **38** network requests, which is catastrophic for users on a slow connection
a lot of scripts could be combined (to reduce the number of network requests) or eliminated altogether, or loaded conditionally with modernizr

I'll have to spend lots of time improving that... when I get that time to spend

I'm thinking maybe a stack with babel + eslint + webpack, and we gut jquery in favor of lodash and querySelector and fetch


Looks ok


Correct command is with double backslash:
  	if (WshShell.RegRead ("HKLM\\SYSTEM\\ControlSet001\\services\\Tcpip\\Parameters\\Domain") == env("CISCO_DEF_DOMAIN")){


@nicity ok, thanks! Not sure if it's reproducible outside of my environment thou


add hover color (primary_hover) when mouse hovers the button


Have a look at commit https://github.com/maximilianhuber/fossology/commit/1a41092a9f51a169d38253428d81c0d45324b57d. That are the places where I think that the `exit(-1)` should be called.

This works for the cli but not for the scheduler call. In the scheduler case is the job always blue and does not end.


As we decide to not use these references for the moment ignore this commit.


Yes, removing `NoErrorsPlugin` is not necessary here. I tried it because having it brings some other unfortunate issues but I’m likely to add it back. See https://github.com/gaearon/react-transform-boilerplate/issues/122


wow thanks for that, I didn't notice :-)


i think i no


@orvice 看到你已经改回int了，这样的话，如果给用户的流量设置得较大，会不会出现溢出呢?


When you update the metamodel I expect in a same commit the metamodel updates, the generation updates and the samples updates.


if this is not reset, no IP addresses are looked for for the HELO domain.


I would name ti MPD_Flow


IIRC @worr already coded this one, check line 1523.


This should be in a separate commit


The trouble is that the theme details page needs to be "full screen" for both logged-in and logged-out, so currently the only way we can identify it is by `name: 'theme'`


> As apps restart and change ID, won't this scheme leak queues (and undelivered messages?) into SQS?

Yes it will.

> or is there some sort of GC/timeout in sqs?

Yes, SQS will GC queues after 30 days.

SQS doesn't charge for queues; only for sending/receiving messages.  Therefore I wrote this to never delete queues.   Its also complicated by the SQS restriction that queue names can't be reused for up to 60 seconds after deletion, so deleting the queues when we're finished with them could lead to unreliable behaviour when the probe reconnects.


改 review.score 合理來講要寫 data migrations，不然別人的資料庫會壞掉......


instead of disabling it, we can skip this section entirely.


We decided to change it to store the Donor to avoid extra lookups (it was done for donations already). We also said that If we have performance issues in the future we can reconsider.


Remove this tool it will not be used


Hmm, I haven't thought of aliases. That sounds nice.


If you make the right anchor conditional to the visibility of the inheritButton control, the revertButton takes less space


We need to validate if the Donor is null, because otherwise a NPE is thrown when trying to get the donorId.


tests are passing after rerun. Random failures of jsonFormatTest are the topic of https://wikia-inc.atlassian.net/browse/DAT-3693


Oh, it's unfortunate that many scripts use `which` or `type -P`.


@t-harps Ok.

I'll make some metrics charts (maybe with d3) that I think would make sense and be useful to go on the dashboard out of js with data bindings that can be substituted with the endpoints when they are done. Probably will be using JSON format of some sort if that's alright.


```python
pads = drumhat.PADS
while True:
    drumhat.led_on(pads[-1])
    drumhat.led_off(pads[0])
    pads.insert(0,pads.pop())
    time.sleep(0.1)
```
can be simplified to:
```python
from itertools import cycle
for pad in cycle(drumhat.PADS):
    drumhat.led_on(pad)
    time.sleep(0.1)
    drumhat.led_off(pad)
```

And it's probably worth adding a
```python
finally:
    drumhat.all_off()
```
to the very end of the script.


Check that the minidrone plugins have 1.0.0 as version


> Also won't this be routing multiple user's messages over the same queue?

Yes the return path for the probe->app->ui response involve posting messages for different users to the same queue.  I don't think this will be a problem, as the messages themselves contain the userid in their message id, which is completely out of the control of the user.

Its not an invariant of this system that queues only contain messages for a single user.


剛回錯 commit。
改 review.score 合理來講要寫 data migrations，不然別人的資料庫會壞掉......


If you make the right anchor conditional to the visibility of the inheritButton control, the settingsButton alignment looks better when the inheritButton is not visible.


Use min/max version


little changes made


You can use Java1.8


b was visible as a normal command all this time! 😜


I'm curious to know what is a `ScalaJSVarArityClass`.


我是一只菜鸟，刚开始学习java基础，边跟着视频学习，边自己看书学习，希望自己能够好好加油


The removal of debug.sethook() in function timeout() crashes the entire server for me.
```
2016-03-14 12:30:45: ERROR[Main]: ServerError: Lua: Runtime error from mod 'carts' in callback luaentity_Step(): ...git/bin/../mods/mesecons/mesecons_luacontroller/init.lua:393: Code timed out!
2016-03-14 12:30:45: ERROR[Main]: stack traceback:
2016-03-14 12:30:45: ERROR[Main]: 	[C]: in function 'error'
2016-03-14 12:30:45: ERROR[Main]: 	...git/bin/../mods/mesecons/mesecons_luacontroller/init.lua:351: in function 'timeout'
2016-03-14 12:30:45: ERROR[Main]: 	...git/bin/../mods/mesecons/mesecons_luacontroller/init.lua:393: in function <...git/bin/../mods/mesecons/mesecons_luacontroller/init.lua:390>
2016-03-14 12:30:45: ERROR[Main]: 	...git/bin/../mods/carts/init.lua:229: in function <...git/bin/../mods/carts/init.lua:229>
terminate called after throwing an instance of 'LuaError'
  what():  Runtime error from mod 'carts' in callback on_shutdown(): ...git/bin/../mods/mesecons/mesecons_luacontroller/init.lua:393: Code timed out!
stack traceback:
	[C]: in function 'error'
	...git/bin/../mods/mesecons/mesecons_luacontroller/init.lua:351: in function 'timeout'
	...git/bin/../mods/mesecons/mesecons_luacontroller/init.lua:393: in function <...git/bin/../mods/mesecons/mesecons_luacontroller/init.lua:390>
	...git/bin/../builtin/game/register.lua:349: in function <...git/bin/../builtin/game/register.lua:349>
Aborted
```


Yeah, I'll move it.


Indeed, you're right, I should move this (his current version is based on "verbose").


最大的原因應該是我反骨吧ＸＤ
因為rails101的教材是後來才教到dependent: :destroy的這個用法，一開始是用destory_all的樣子
然後在找destroy_all的用法的時候，就剛好看到很多文章再說delete_all跟destroy_all的差別
所以就好奇了的把dependent拿掉自己去試了一下，所以才發現的
然後其實我用有through
這是我的兩個model內容你可以看看
group.rb
<https://github.com/kurotyan514/rails101/blob/master/app/models/group.rb#L6>
user.rb
<https://github.com/kurotyan514/rails101/blob/master/app/models/user.rb#L9>
其實我自己還有去試了delete_all跟destroy_all跟dependent: :destroy
看看這三個方法那個花的時間比較少

但其實看完你的文章之後我還有點小疑問，
那就是我能夠了解了為什麼會update成nil的原因了
但我其實還不太能了解為什麼他會這樣子做呢？
把重要的foreign keys update成nil，然後造成孤兒資料的結果
還有就是＠group.group_user.all.delete_all
是因為把.all的東西當作參數送進delete_all裡，所以才會不變成update嗎？
如果不會太麻煩妳的話希望能夠幫我解惑一下啊～感恩

Bruce Li <notifications@github.com> 於 2016年3月14日 下午7:13 寫道：

> 不知道是否能夠引用你的文章內容呢
>
> 當然沒問題 [image: :laughing:]
>
> 另外，我好奇為什麼你想要拿掉 dependent 以及不使用 has_many :through
> 呢？這應該是很普遍的做法，有什麼特別的原因不採用嗎？
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kurotyan514/rails101/commit/21f5c92fff1e3f3288d571438c9b48b3c4bd96e9#commitcomment-16677300>
> .
>


Default avatars are not shown!


I suspect you need to rename the files in the 'drums2' folder as well? ;)


Spelling mistake: *A script...


nice


and thank you


Hi! This file is doesn't apply to my theme. I have to edit on the cherry options panel "Define custom CSS" box. Knows anybody where is stored that css?


this `box` call and https://github.com/sjrd/dotty/commit/d0dd7001f0b59ed53f0778530328b3bf413587a2#diff-728ea9b6359f1e5d8bb0b943b0d78624R1746 `unbox` call seem to be undoing the value classes transformation. It would be good to document it.


I think it would be better if we could globalize the configuration of the `ssl_connection` hash.  I can't see any (immediate) reason for having this configurable on a per-cluster basis, and it currently requires the cluster to have knowledge of where certificates are installed on the appliance, which isn't quite right.

Could we have the config reworked to something like this please?

```
clusters:
  - name: "New Cluster"
    address: "10.10.10.1"
    auth_port: 25269
    ssl: true
ssl:
  root: '/path/to/dir'
  certificate: 'cert.pem'
  key: 'key.pem'
  ca: 'ca.pem'
  verify: false
timeout: 5
```


Maybe we can avoid wrapping comments by always placing them before the line they refers to.


What is the intended difference with `SymDenotation.isSetter`?


We do not usually do make these kind of service calls within the domain model objects. I would recommend that you do the same as in this commit https://github.com/openmrs/openmrs-core/commit/2754c97d87f846a346f831571bb7e5a1da757e86 for PersonName.setFormat


nice spotting that these were missing!


Same comment, should we place it before the run()?


Oi


# Android NDK integration using JNI

This is sample calculator application developed on typical c files and same has been called from Android Activity using JNI


I was hoping the function name would give it away. I'll add one.


I've just uploaded a new version (1.0.3) to this repo. As for now it's available only as source code. In this version you can override the default max value. Follow the instructions in the end of build.gradle file. It will probably take some time for me to upload a new version to Maven.


Dalle Parsa


AQUI É 13 PORRA


AQUI É BODYBUILDER!


TRAPEZIO DESCENDENTE


tranquilo?


cheguei!


É ELE QUE A GENTE QUER


ES6 goodness :+1: Couldn't see any others though, very thorough job.


FILHO DA PUTA


Yeah this occurred to me the other day, as the certs will always be the same for each cluster the access manager connects to. Thanks, will change.


FELA DA PUTA


Éoq?!


require field ..... check  <-- 이상한 영어 문법이...


OQQQQ
QUE QUE QUE QUE QUE


Yup -- for now at least.  We may add additional complexity to the certificate handling later, but we don't need it yet!

Ta!


hi


여기도 "to, from, text" must be entered 가 문법에 맞겠네요.


ничего не убирал, ток немного изменил. ситуация с undefind решается в моем следующем коммите


Both good points; have done this.


I propose to revert the part of this commit that changes the API.

More pricesely, I propose to be in middle ground:
  1. Store comments in Attachments
  2. Export them with `rawComment`/`withComment` methods.
  3. `with` Methods are assumed to return a different instance of the tree. IE it should either clone the tree, or be called `setComment`. I'd prefer the later.


Bonjour

Votre script marche bien mais j'ai les problèmes ci dessous :

Nom en hexadecimal du fichier VCF au lieu du nom du contact
Obligation de créer un hote virtuel du meme nom que l'utilisateur qui se connecte sinon cela ne marche pas (impossible de faire )
Il fait bien l'export en vcf mais le fichier fait 0 Ko et je n'ai pas les droits pour l'ouvrir

Pouvez vous me venir en aide ?

Cordialement

rAnX06


Here are the facts as I see them:

1. Yrsa did not exist before 2016
2. Rasa did not exist before 2016

Email me about this if it bothers you.


現在還沒有 review 需要 migrate 吧⋯


Call it `enableLiquidFlowProtection`


I believe the "more Pythonic" way to do this would be something like:
```python
try:
    for ch in pad:
        on_hit(ch, handler)
except TypeError:
    # all the rest of the code that assumes pad is an integer
```
as that will still work if `pad` is provided as a tuple, which your current implementation won't.


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


I've split this into two commits with the first introducing the calculator.


This overwrites existing files. Better should be something like this:

File file = new File(absoluteFileName);
if (!file.exists())
{
	file.createNewFile();
}


"No enough" -> "Not enough"


ola


Imi place cum ai facut aici.
Anonymous functions in php.

Functia asta pare cam lunga, nu?
Mai ales partea asta care face extragerea userilor.


thought this was transitive.


You mean just swapping the branches? We can't drop both because sometimes assignment changes cause clauses to be come undetermined again due to backtracking.


Transitive.


That's fine. Could be done later of course.


the iris.Listen(interface{}) changed to iris.Listen(":8080") at the last update, make a quick fix  there :P


Why revert?


Tätä pitäisi mielestäni vielä harkita uudelleen. Jos äänialalyhenteitä ei tässä konversiossa avata, niin mitä niille tehdään? Eihän niitä manuaalisestikaan ole mahdollista avata. Eikä avaamattakaan voi jättää, jos RDA:n mukaista kuvailua aiotaan jatkossa tehdä. Samasta teoksesta tuskin halutaan useita erilaisia hakumuotoja.


transitive.


Okay, I don't exactly know why this happens since `debug.sethook()` should be executed after `pcall` anways, but since I'm no expert on this stuff, I put it back in.


This is how you set it up.


delete


Some difference between this and the revert?


Sorry for late reply and no, I don't.


#5 fix


Thank you! I still keep up escaping all danish letters.


I've re-worded the commit message so it's no longer than 72 characters.


I'm afraid I might have created a stack overflow here.


I'm not sure the following crash is related to the above change. But this command:
```
curl --test-event  -v -o NUL http://www.vg.no
```
crashes. These wild pointer from the call-stack looks related to (maybe another) hashing:
```
0x0
0xffffff01
libcurl!easy_perform(struct SessionHandle * data = 0x0328cd70, bool events = true)+0xaa
libcurl!curl_easy_perform_ev(void * easy = 0x03274360)+0xb
curl!operate_do(struct GlobalConfig * global = 0x0018fb84, struct OperationConfig * config = 0x03274360)+0x28f4
curl!operate(struct GlobalConfig * config = 0x03274360, int argc = 0n5, char ** argv = 0x0098cc88)+0x13e
...
```

I've built everything with `CURLDEBUG` on MSVC-2015 (32-bit). Is this option really suited for Winsock? (I just discovered it).

Last output before crash is:
```
* Connection #0 to host www.vg.no left intact
* socket cb: socket 452 REMOVED
* Expire cleared

```


and here.


Syntax is busted. Code examples are such a pain this way.


Depending how the clients use this API it won't work. For instance, if after doing the setXXX the client modifies the original set, the persistent copy will be untouched.


lol :D


Strips the prefix?  Seems like this is forcing the shape of a `GOPATH`.  Is that shape force on this script already?  Having to set up my env a certain way before running a test script would be awful.


@lauravignoli which line are you referring to? Regarding this commit, we shouldn't be storing Donors in the map (performance) and it should be sufficient just to check if the BSIS identifier (Long) is not null before creating a new Donor and setting the id (as was being done in the original code).


Yes, we still need to fix it to:
```js
useFakeTimers( Date.now(), ( newClock ) => {
```


This looks weird.  Bad generation before?


Your fix *is* correct, but I think it'd be better to simply change mkfs_exists to return -ENOENT. The place you are changing is the *only* user, and it just looks stupid to change the error code twice.


I don't think so? I'm pretty sure Cairo's ARGB32 maps to wlc's WLC_RGBA8888. Other sway surfaces (such as swaybar and swaybg) use CAIRO_FORMAT_ARGB32.

http://cairographics.org/manual/cairo-Image-Surfaces.html#cairo-format-t


ok add


Wait what :scream:


if we don't use those files anymore, we should also drop them to keep the repo clean ?
or the other option keep those lines and only comment them for those who would like to use it if it makes a valid layout.


This seems very racist against Danish letters. You should reevaluate your codestyle. Please check your privilege.


Closes #4


Hey @capitaladot what happened to broadcasting issues?  I've been working on updating the LACAL (working through the list of issues) and now I'm stuck until this merge conflict is resolved.

Unfortunately, I'm not an expert at Git.  If you can take a crack at resolving #252 that would be great so I can keep working.  Thanks!


Ok, weird.  Why leave the rest of this file around instead of consolidating to wrappers?


If they were checked, why didn’t we get the lint errors?


stdin/out/err에 대한 [위키](https://ko.wikipedia.org/wiki/입출력) 하나 찾았어~
std가 붙는다는건 기본값이라는 걸로 이해하면 되고
이걸 바꿀수도 있다는 것만 이해하면 될 것 같아


@byteandahalf  Update the addon MrCrayfishFurnitureMod :3


@debonair Thanks. I had gathered that by looking at the commit history, but it wasn't clear which data was the problem.


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


Why have you removed bower availability?


A pre-commit hook that looks for THIS IS FOR INTERNAL USE ONLY in the patch?


Yeah, looks good.

While at it, maybe add a define at the top of the file for the constant?


I hate these tests so much.


Maybe.
But https://github.com/systemd/systemd/commit/85eca92e
> - Rework fsck_exists() and mkfs_exists() to return 1 on success, 0 if
  the implementation is missing and negative on real errors. This is
  more like we do it in other functions.


What's the point of this one? You could create an empty cache manager and only create them on the fly.


Tutaj czasami nie brakuje uzupełnienie schematu bazy? Bo nie widzę u siebie w tabeli `tariffs` kolumny `numberplanid`


IMO, it really looks like the other constructor. I would have a default bucket and a default ttl only. If users want to tune the TTL per cache, maybe they should use the `Config` alternative?


Broken homepage layout could be preview on the demo http://apps.titellus.net/geonetwork/srv/eng/catalog.search?#/home and travis is still red for develop and all PRs https://travis-ci.org/geonetwork/core-geonetwork/jobs/115078712#L6422. It's a pain to merge develop in any projects :(


Sure, why not?
Just didn't look into that part of your product yet.feel free to move it, I've got no system until tomorrow.


A few things about this implementation:
+ You should use a [`BlockingQueue`](https://docs.oracle.com/javase/8/docs/api/index.html?java/util/concurrent/BlockingQueue.html) instead of letting the thread spin.
+ There is no way to cleanly shutdown the chat thread, the thread loop should probably look like [`while (!Thread.isInterrupted()`](https://docs.oracle.com/javase/8/docs/api/java/lang/Thread.html#isInterrupted--) or some other flag.
+ It is generally preferred to provide a `Runnable` implementation to  `Thread`'s constructor instead of extending `Thread` directly (composition over inheritance).
+ Access to a public field should generally be done through a method.
+ Not sure what the policy is on logging, but you'll probably want to remove them before merging to master.


I thought with Debian type systems that we should only remove the 127.0.1.1 IP entry from /etc/hosts if we set a static IP for the system?

Source: https://www.debian.org/doc/manuals/debian-reference/ch05.en.html#_the_hostname_resolution


OK.


Notwithstanding all the other hacks, remove that and use dependencies. That's precisely what they are here for.


Thank you Mark, that is nicely put (: I don't know about infinite, but it is certainly large


So I upload&update my progress here on Github?


Umm sorry guys, I forgot to change branch after pulling...
But I make really minor changes, I'll create a new branch from this.
Thanks!


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


@timer I had changed the external donor ids to test the validation on the deferrals and committed the data


wow :O


GOPATH is already godepified at this point. This is only necessary if you want to use pkg/name/… format and have go test expand to include all subpackages… I can update to limit to that


This is pretty much what I had in mind except that I would have mapped the config from the name of the cache (since this bit is going to change everytime). I am just not sure what you'll do if you ever need to extend that class. How about a builder approach: you could mutate it (and add additional attributes) and be able to initialize one based on another.

You could then remove the default bucket and ttl constructor. You could use a default `Builder` for the bucket, ttl and any further attributes you may introduce in the future.

For the case where you need to customize your caches, I still wonder what prevents you from creating/registering them after the instance has been created.


- Separate query bindings
  - [See code change](https://github.com/laravel/framework/compare/254497b8672e...6117ba5303eac#diff-cc3d43e01028390141596ae0da939930)
- Fix bindings on unions by added a new binding type
  - [See code change](https://github.com/laravel/framework/commit/89c5f0b6b9de31895709153af2ed98ab3f38c923#diff-cc3d43e01028390141596ae0da939930)


Для читаемости лучше вынести эти величины в переменные с понятными именами.
Например:
```java
int xLeft =rectangle.x + offset;
int xRight = rectangle.x + rectangle.width - offset;
int yTop = ...
int yBottom = ...
```


Makes sense, but `if`s are faster: this way you only call `replace_all` if it's really needed.


Turns out it didn't check in everything :-1:


А здесь сработает просто `fillRectangle()`?


@opengapps/packagers

We had to revert Clock 4.5 here since it did not even install on Marshmallow. Please:

``` sh
git fetch && git reset --hard origin/master
```


I needed this to ignore the `Unikernel1` argument that gets added when you do `mirage_logs my_job`. What's the correct way to do it?


@alebaffa : the PR https://github.com/jmvitali/grafana/pull/15 has been created to implement all the comments> in fact, as discussed, you already provide this kind of comment that we implemented in the PR  https://github.com/alebaffa/grafana/pull/1 we created for internal review process.

About Unit test, the code we introduced in based only on existing unit tests already implemented by Grafana.
Code introduced after this PR (Ex: to prevent Dashboard edition) introduced more unit tests.


Don't change my messages, or I will hit you with a thousand SJW's. Firs warning you racist scum.


`make distcheck` found some artifacts left under mruby.
The exact command-line is:

```
./configure --with-mruby --with-neverbleed --enable-asio-lib
make distcheck DISTCHECK_CONFIGURE_FLAGS="--with-mruby --with-neverbleed --enable-asio-lib --enable-werror"
```

I think `make distcheck` does out-of-tree build.


And then how do I commit anything that actually has that text in it?

Anyway, maybe I should just turn this into an actual option, something that you can pass in the run configuration for the test.


No, the revert is so the bump level actually matches that commit. I'll squash those two together eventually


http://mirage.github.io/functoria/Functoria.configurable-c.html the method "deps".


looks good


better `addMissingTypeHierarchyToGraph"


It should be added the previous pending edge too


**Fuze**, подскажите пожалуйста, имеет ли смысл в своих компонентах, заменить
cmsCore:: **на** $this->cms_core
cmsUser:: **на** $this->cms_user
cmsConfig:: **на** $this->cms_config
cmsTemplate:: **на** $this->cms_template

Будет ли скрипт выполнятся быстрее если заменить или это просто для удобство сделано?
так как в одном файле, много раз обращаюсь к ним:
cmsUser::addSessionMessage()
cmsUser::get()
cmsCore::error404()
и т.д.


Объекты `Path` надо диспозить вручную, как только они нам больше не нужны.


How do you want to put these arrays on Mongo via ORM?
Did you try a push on the DB? We can try this evening or I can clean the DB for you.


print ??? .. Log it if we want it..


If you want help with that, feel free to reach out to me :)


How about a test to check that an already-upgraded SR is handled correctly?


What the fuck?


You temporarily disable the hook… how often will you need that?


@wilx I've updated pull request. Could you please review? Is it OK now?


First*


First working version of Cholessky-fast-calculation


I am not sure: shouldn't we better raise the exception rather than masking it?


Ditto.


I think this will fall foul of the problem described here: https://github.com/appc/cni/issues/141


Лучше вынести в переменные, например
```java
float xInitial = points[0];
float yInitial = points[1];
```
или мб `xStart`/`x0`


@ngtuna we also need to allow for ":" separator in env variables.


@pelwell Do we need updated VC firmware (bootcode/start/fixup) for this new dynamic overlay functionality?


TransactionLogger()   ->   TransactionLogger(object)


in fact the purpose of this PR was to allow Claire to provide me her work before her vacation.
We are going to delete this PR


stack_is_ordered() moved in sort.c


Please add changes at the end of the `Changed` block, not at the beginning.


In principle we would need these 2 classes everywhere we need to load a record, not only when the record is referred by another record.

@jmartinm do you have a place to suggest where to put these? Actually can't we override the default `get_record` function?


Yeah, from a few blog posts that I read ([this one, point 3](http://axialcorps.com/2013/08/29/5-simple-rules-for-building-great-python-packages/) being the one that comes to mind) it's best if all exceptions should be in `exceptions.py` for importing reasons. From a user perspective, if you want to filter out specific Nengo exceptions, it's a pain to figure out where each exception is if you're just running arbitrary models and encountering exceptions.


This seems to fix something else. OK for me to include it here.


//edit test
//there is not any code just a comment


This doesn't add the children css and width to new elements (normally run in setup())


line note


That issue shows an `add`, not al `del`.


the braces and/or indentation on this look wrong


Ok thanks I will do some revisions.


this line is missed?


возможо зря убрал. не ожидал что поиск пользователя будет по паролю если ты про это. ситуация с undefind решается в моем последнем коммите


The library used is: https://github.com/knolleary/pubsubclient, readme will follow.
@Marwe what are you using in your implementation?


Can you squash these commits down so PR is clean ...


Hey @tysongach! Thanks for this PR. This looks good to me. :tada: :+1:


Seems to be fixed. Thanks!!


I'm not sure what I'd put here. It says: "deps is the list of abstract implementations that must be initialized before calling Functoria.configurable.connect". But I don't want to initialise anything, I just want to remove the unikernel from the list of functor arguments, because although logging depends on the user's unikernel, it doesn't care about its type.

(and note that we need to wrap the job's start function, not just run before it)


Yeah, I thought I removed that comment. ... Thanks for your replay.


Dependencies are precisely "Give me the value for the connect but not the functor argument", which is exactly what you want.


Cumulative option removed in bce5c7f364a1b5f4221a2db964e27b5551b39378


Yeah, that was probably what I intended here...


11253054 Şura SANAL


Strange, I was so sure this fix only avoided to look for invalid sockets in the hash. So if you revert this patch only, it works fine again? Any chance you can catch the crash with some more details?

I tried exactly your command line on Linux, both with and without valgrind, but I couldn't reproduce any problems.


Done.


ok mate


Good luck at LTC!


No - it's been supported since the first 4.0 kernel. You will need the dtoverlay userland app though - see https://www.raspberrypi.org/forums/viewtopic.php?f=107&t=139732.


Ok maybe this goes all ready to the nitpicking side but I would have thought that after this change we no more have ```Flame.``` anywhere. ```ALIGN_LEFT``` and other are used with ```Flame.ALIGN_LEFT``` still. Buut I guess this does not matter.


awesome.


64位的话，其实不需要担心溢出//
http://php.net/manual/zh/language.types.integer.php

On March 14, 2016 at 7:35:56 PM, Acris (notifications@github.com) wrote:

@orvice 看到你已经改回int了，这样的话，如果给用户的流量设置得较大，会不会出现溢出呢?

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


MongoDb is document based (json I think), so it should be possible.


In my mind, in the ideal case, we would have `try` / `excepts` around code that we think could raise exceptions relatively frequently and raise our own `NengoException`-based class in the except. I think this is probably less of a problem for Nengo in its current state, but I think these will be important as we get more backends / frontends (like the GUI). But yeah, you can read the arguments in [this post](http://axialcorps.com/2013/08/29/5-simple-rules-for-building-great-python-packages/) and see what you think; it's just one perspective so if we decide to go a different way with exceptions that's cool, but I'm inclined to try this first and see how it works out.


Yeah, in the end I didn't need it. I'll comment it out.


> That issue shows an add, not al del.

Yeah I noticed that after I commented. After looking at the spec, I think it needs to be more explicit there what a plugin is required to output for a delete (it just says you have to output something if it's an add). Certainly, if there's an error, it's communicated via JSON. In the code, it ignores anything that comes through on stdout *if* the exit code is OK for a del, so it won't care that there's no JSON value -- but that ought to be mentioned in the spec.

As you were :)


Ok, much better, can stay this way for some years.

I'll have a look at other pyflakes users
```
grep-dctrl -FBuild-Depends pyflakes -sPackage /var/lib/apt/lists/*Sources | sort -u | awk '{print $2}'
autopkgtest
botch
cloud-init
debtags
dirspec
game-data-packager
gdebi
prospector
pygobject
python-apt
python-flake8
xml2rfc
```


If we suppose that the C_generaterandom fails, you will be comparing uninitialized memory.


Fixed in cd920b34127abe174f670f6c5054b627efcf5a25 .


@pelwell If I ask, "where is the source code", am I going to get a clip around the ear?


I wanted something that's threadsafe, that's all. The blocking wasn't the important part.


11253054 Şura SANAL


Why have it as global variable?


I was talking about mongoose. I'll check if we can do that


RTFFP.


A i owszem brakuje - zaraz uzupełnię.


You are reimplementing strdup()


strcpy() always NULL terminates.


Correct me if I'm wrong but any version lower than `0.38.0` will not be allowed, right? If `>= 0.35.0` would really need to be supported it would have to be `'~> 0.35', '>= 0.35.0'`.


Thanks for that. I like `inferFunRange`. Added in 3daa075b08d92279d3d087947063a3a9c9f4668a.


`22da08b * master origin/master origin/HEAD bl-remote-desktop-pipemenu: install vino
ef3573d * upstream/master bl-help-pipemenu: Fix bug in "How to Edit Menu" item.
8968487 * bl-printing-pipemenu: Fix incorrect install command
cf970a6 * Add GPL3 headers, enhance graphics and remote-desktop pipemenus
b94cb61 * bl-graphics-pipemenu: Remove traces of foreign HOME
bb83c48 * Add option to install gimp extra packages
772ea54 * Update README and dependencies.
2d7de3e * 8.7.3-1 bl-dropbox-pipemenu: Make small code improvements. debian/control: Remove inotify-tools from Depends, keep in Recommends. Update changelog.
`
I meant the patches that were applied since version 8.7.3-1.
Thought you held back creation of a new version for a reason?