If we add several variables we can do this other approach:

https://github.com/keymon/cf_manifest_template_spike/commit/d9c0e86a15435b76720799abe233046e08248647

That will save us the overhead of explicitly having to specify each variable as an output, extract one by one in the make file, and pass it as an argument.

Also, depending on how we manage the manifest, having all the variables in one file would ease everything (e.g. having a .yml with spiff or a .rb file with .erb).


This seems to have broken the "trash-patch", I get a patch failure while running `brew install mutt --with-trash-patch`:

```
patching file globals.h
Hunk #1 FAILED at 139.
1 out of 1 hunk FAILED -- saving rejects to file globals.h.rej
```


A customer has a resource configured as multiple-active="block" in a resource group. They don't mind if it's found being active on multiple nodes,  and want the resource group to keep running there.


1: M111 unrecognized component name: kwalitee


The abbreviation should be ZC, right?


That is really interesting. One question: Does the rendering happen in the server or in the client (`bosh_cli` side)?


Не сможет. Потому что целочисленный параметр ресурса может изменяться произвольно. Чтобы перевести арифметические операции `+`, `-`, `*`, `/`, `=fun()` в enum, надо отследить все изменения, расчитать значения дульт к enum. Но это можно сделать только в рантайме, прогнав модель со всеми базами. В общем не надо даже начинать на существующем железе писать такой конвертер.


I really like this.


you should remove `var u *User` line 86


Could someone please merge this?


1: M110 missing component name


const


This commit is causing a pane spec to fail (https://github.com/atom/atom/blob/45694dcda0b6dec132979960d47fa874fa591aa6/spec/pane-spec.coffee#L347-L352), so I'm going to try partially reverting it until I can figure out how to fix the broken spec.


You are right, lets leave them until we remove them by acting on the corresponding modifications.


Add a product


Looks good, personally I've added alias with absolute path to avoid reinstalling when I make changes in the tool.


I probably did not express myself well. My remark was to only change the example. The response section should tell the user that `bugzilla_component` is a list of strings, but the example should show specific values. There is no point in showing the same JSON twice.

Also, since the example should show actual data, the values should be wrapped in double quotes so that it is valid JSON.


The change looks good but why is pkcs15_bind called twice in the first palce? I only see this for different apps, but a card with multiple slots (like eID-s) that should not happen ?

I find the p11card abstraction quite confusing to be honest...


Checkmarx scan results 07 Sep 2015 12:15 UTC +03:00

High
Medium
Low
Info

New
1
0
0
0

Recurrent
0
0
0
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

d3d1c0b068aa21a2aa829d5411ca63fda694392a	@dortal


Done. Thx


@uhafner I'm getting a different exception: `java.lang.ClassNotFoundException: org.jenkinsci.main.modules.sshd.SshCommandFactory`


you could remove the notify callback


Sure, but fetchResults returns a Future, and I want to return a Buffer (could be Buffer.empty if no results, but should be the same buffer for every update.


LGTM


pls cache result


Hi Ian, thanks for fixing that. What exactly caused the timeout?


эту строку можно пернести в условие


Dir.chdir {block}


git nazi


be sure you can run each test on it's own by running it with this pattern

`goapp test -v -run=^TestUserById$`


Yep, we could. I like it this way to make it very explicit which calls need to be atomic.


Please take in consideration the default naming convention on methods https://en.wikipedia.org/wiki/Naming_convention_(programming) -> in this case the method should be called: openViewMenu()
"<i>Methods should be verbs in lowerCamelCase or a multi-word name that begins with a verb in lowercase; that is, with the first letter lowercase and the first letters of subsequent words in uppercase.</i>"


Pourquoi en private ? Autant ne rien en faire si on peut pas l'utiliser, non ? (Pareil pour la lib iOS)


单元测试


Solved adding:
```
 <dependency>
    <groupId>org.jenkins-ci.modules</groupId>
    <artifactId>sshd</artifactId>
    <version>1.6</version>
    <scope>test</scope>
</dependency>
```

Now I'm getting the expected `java.lang.NoSuchMethodError: org.dom4j.io.HTMLWriter.setEnabled(Z)V`


Согласен, арифметические операции упустил.

Фактически, есть модели, которые творят такое? Если нет, можно сделать в конвертере обработку наиболее распространенных частных случаев использования `ranged` интов.


les hairfies coifféS


Do not remove initial author!
Just add your name as a contributor.


just wondering: do you use python2.x to build the docs?

I considered adding this change upstream, but then remembered that u"..." is not valid in python 3.2 (which is currently our minimum requirement for running borg).


Hello Dad :+1:
first : I love Linux then
I have a little dream ...just a little can you follow me ?It so funny to see you have 30K follower but you follow  nobody so please ,for fun follow me 1 day ?


yea, without explicit cast it didnt work so ... ;-)


@PeterHsieh For the moment, just do a database query to get the current propertyId for the tenant. I have done something similar in https://github.com/nirrek/wallflyServer/blob/master/handlers/userMessages.js

In the longer term though, we will want to have some sort of local model on the client that grabs all these pieces of data on load to simplify this stuff.


un ternaire en 3 lignes ? pas très joli je trouve


One best approach is to separate your tests (class methods)  into suites (classes)

If this methods are testing substractions then move them into a new class SubstractionTest.java


I could have sworn there was a repo here a minute ago!


The same approach for divide, multiply, etc.


@r-tock unfortunately, I have no way right now to add support for persistent storage (GCE PD, EBS volumes, etc.). Please, open an issue and let's see if anyone's willing to contribute. Perhaps yourself?


HTTP GET method는 body를 쓸 경우 문제가 생길 여지가 많아서 쓰지 않는 것으로 알고 있는데 특별한 이유가 있어서 사용한 건가요?


:+1:, merging


Also, you should make an alias for the payload `var payload = request.payload;` at the top of the postHandler function, just to make the values array more readable.


Please wait with changes like this next time, if you know there are unmerged modifications. Merging will be complicated.


Seems that `terminal status-line (TS) support` has broken the patch, changing it to include the new:
```
WHERE char *TSStatusFormat;
WHERE char *TSIconFormat;
WHERE short TSSupported;
```
lines works locally for me.


I know I can. I chose not to. :D


OK:D


 A solution here could be:

results: ReadBuffer[Something] = inputChannel.flatMapBuf {
   Buffer( fetchResults(_).recover(..)   )
}.orElse( Buffer() )

Just need a Future to Buffer convertsion: https://github.com/MetaStack-pl/MetaRx/issues/1


Works fine - fixes the bollinger bands on changing data.


Well... not all of them, unfortunately. I can't find anything from Synology about the Video Station.


产品名称：下载器
版本：2.0
实现：Java
功能摘要：
    使用Java开发的一个文件下载器。
    1）使用单线程机制实现，以后抽时间改成多线程的
    2）支持断点续传
    3）支持本地文件自动重命名
    4）下载文件创建使用先占式模式
    5）支持异步调用
    6）支持任务中断续传


Any reason why the Eclipse files have been deleted?


Главное, чтобы первая лаба такие параметры только через равенство выставляла. Другие модели сейчас не так важны.


Thanks @RuthC-C :-)


Yeah git is being a pain for me atm, so I'll put it up later. :)


Don't want to change the name so it's default for all users?


LGTM


Husk # headlines :)


It is a bug.
if fk_unit is null, then test was false, but it should be true to set "null" into fk_product.


Reviewed #soumyajit 3.32 pm 7.9.2015


Headlines?


could you please?


기존의 api 구현을 최대한 덜 건드리기 위해서입니다. 원래 GET 인 걸 method 를 변경하면 라우팅 테이블을 바꿔야하는 관계로 수정 범위가 커집니다.  query syring param을 덧붙이는 것도, documetation 수정해야 하니 귀찮은 일이 됩니다.


ack


Is this function supposed to be a module function or an api type utility function to be used anywhere else. If this is the case you must be aware that the module file will not be available during the install of other modules.


Oh, true. Didn't notice that. Sorry


Jep, and is even unsafe to do as an enums size is undefined. I used it correctly on reading, but missed the writing part. Also constants are dangerous without explicit cast.


Ja # Integrate with Zapier i toppen


review done


Can this not go in `local-carton` or would that cause problems?


Checkmarx scan results 07 Sep 2015 12:36 UTC +03:00

High
Medium
Low
Info

New
1
0
0
0

Recurrent
0
0
0
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

d3d1c0b068aa21a2aa829d5411ca63fda694392a	@dortal


Only current Alpha users will need to change it.


Тут есть две сущности:
1) название файла
2) basename

Сначала ты назвал и то и другое `TypeMismatchMessages` -- это не совсем корректно, т.к. название файла завязано на единственное сообщение внутри него и как только добавится другое сообщение файл уже не будет отражать содержимое. Поэтому я предложил переименовать в более общее `просто Messages или SpringMessages`.

Потом ты переименовал basename в `Messages_ru` и тем самым захардкодил русский язык.

>добавил ru_RU чтобы файл находился быстрее.

Здесь идея была не в том, чтобы быстрее работало, а в том, чтобы добавить русский язык и впоследствии также лего и просто добавлять локализацию для других языков.

Итого, как я это вижу:
- basename `Messages` -- не содержит указания на язык
- название файла `Messages_ru.properties` указывает, что в этом файле содержится локализация для русского языка.

Почему ru, а не ru_RU, потому что ru подойдет и для ru_RU и для ru_ЧТО_УГОДНО_ЕЩЕ, но не наоборот.


- 初回ローカルリポジトリをプッシュがうまくいかない。
- 下記で解決
    - git push -u origin master
    - GitHub アカウント名を入力してエンター
    - パスワードを入力
        - （そのときには何も表示されませんが、とにかく打ち込みます）して、エンター


`createAngel`?


No icon shows for the "Send correction" link in the list of tasks (next to the eye). This breaks the design.
![work-correction](https://cloud.githubusercontent.com/assets/609839/9713918/22556f2c-551a-11e5-8427-602db2dac2c6.png)


My bad :-)  Thanks


@JeromSar why is this command blocked from CF? I use it to unban myself from false bans.


커밋


Ahh. Ja det kan jeg godt se at du har sat på alle de andre. Troede det
skete automatisk :)

On Mon, Sep 7, 2015 at 11:35 AM Jesper Strange Klitgaard Christiansen <
notifications@github.com> wrote:

> Ja # Integrate with Zapier i toppen
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/elmahio/documentation/commit/e8d3d9be6900758b8ab3cb1f3bdd39eb9b908798#commitcomment-13093168>
> .
>


@shivapoudel working fine here. Any more info for me?


No


Version 1.1r11


LGTM


it's not the solution, because a new command could be sent between updateChannel and ReconnectListener.invocation


**no-unused-vars**: test is defined but never used


**quotes**: Strings must use singlequote.


**no-var**: Unexpected var, use let or const instead.


**semi**: Missing semicolon.


En mi plataforma funciona bien (server_type = test) pues se usa los SVG.
Cual es el estado de los SVG se van ha usar o no?


java沒有像C#那種property直接在constructor指定名稱就設定的方式，所以改成把builder寫的更descriptive一點


Update


So close. Yes, you have to check if it's a number but the default value should be `NULL` (represented by `DEFAULT_VALUE` constant) not `0`. Don't worry about types, in ruby you can assign both number and string to the same variable. The same goes for the tests, default value is `NULL` not `0`.


I guess we are just including all the mappings that were there. The ones "non schema" are provided by Drupal's default installation. Probably it is good to keep them here, but knowing why are they here and where they are coming from. I guess we will remove them in future versions.


Where are these permissions defined? Why not use the hook_permissions() to retrieve them and reduce the chances of mistakes?


Hi Simon,

I finally have a module with problems, and I see

     [10:26:49.251] DEBUGAPI: <datapipe.cc/evalReadback:L565> Channel 3 ROC 2: first readback marker after 15 readouts. Ignoring error condition.
     [10:26:49.251]  WARNING: Channel 3 ROC 3: Readback start marker after 15 readouts!
     [10:26:49.251]    ERROR: <datapipe.cc/Read:L164> Dumping the flawed event +- 3 events:
     [10:26:49.251]    ERROR: <datapipe.cc/Read:L166> ====== 0 ====== a242 8000 4388 322 2a86 4388 322 2a49 4389 322 2a6d 4389 322 2a85 e022 c000
     [10:26:49.251]    ERROR: <datapipe.cc/Read:L166> ====== 0 ====== a23c 80b1 4389 322 2a84 4389 322 2a49 4389 322 2a6d 4389 322 2a86 e022 c000
     [10:26:49.251]    ERROR: <datapipe.cc/Read:L166> ====== 0 ====== a23d 80c0 4388 322 2a84 4388 322 2a4b 4388 322 2a6d 4388 322 2a85 e022 c000
     [10:26:49.251]    ERROR: <datapipe.cc/Read:L166> ====== 0 ====== a23e 8000 4388 322 2a85 4388 322 2a49 4388 322 2a6d 4388 322 2a85 e022 c000
     [10:26:49.251]    ERROR: <datapipe.cc/Read:L166> ====== 0 ====== a23f 8040 438a 322 2a85 438a 322 2a4a 438a 322 2a6d 438a 322 2a85 e022 c000
     [10:26:49.251]    ERROR: <datapipe.cc/Read:L166> ====== 0 ====== a240 80b1 4389 322 2a82 4389 322 2a4a 4389 322 2a6f 4389 322 2a86 e022 c000
     [10:26:49.251]    ERROR: <datapipe.cc/Read:L166> ====== 0 ====== a241 80c0 4388 322 2a84 4388 322 2a4b 4388 322 2a6d 4388 322 2a87 e022 c000
     [10:26:49.251]    ERROR: <datapipe.cc/Read:L170> Reached 100 dumped events, stopping now...
     [10:26:49.251]    ERROR: <datapipe.cc/Read:L164> Dumping the flawed event +- 3 events:
     [10:26:49.251]    ERROR: <datapipe.cc/Read:L166> ====== 0 ====== a342 8000 4388 322 2a4e 4388 322 2a63 4389 322 2a86 4389 322 2a65 e022 c000
     [10:26:49.251]    ERROR: <datapipe.cc/Read:L166> ====== 0 ====== a33c 80b1 4389 322 2a62 4389 322 2a63 4389 322 2a85 4389 322 2a65 e022 c000
     [10:26:49.251]    ERROR: <datapipe.cc/Read:L166> ====== 0 ====== a33d 80c0 4388 322 2a60 4388 322 2a64 4388 322 2a85 4388 322 2a65 e022 c000
     [10:26:49.251]    ERROR: <datapipe.cc/Read:L166> ====== 0 ====== a33e 8000 4388 322 2a61 4388 322 2a64 4388 322 2a85 4388 322 2a65 e022 c000
     [10:26:49.251]    ERROR: <datapipe.cc/Read:L166> ====== 0 ====== a33f 8040 438a 322 2a61 438a 322 2a63 438a 322 2a85 438a 322 2a65 e022 c000
     [10:26:49.251]    ERROR: <datapipe.cc/Read:L166> ====== 0 ====== a340 80b1 4389 322 2a60 4389 322 2a64 4389 322 2a85 4389 322 2a65 e022 c000
     [10:26:49.251]    ERROR: <datapipe.cc/Read:L166> ====== 0 ====== a341 80c0 4389 322 2a60 4389 322 2a65 4389 322 2a85 4389 322 2a65 e022 c000
     [10:26:52.401] DEBUGAPI: <datatypes.cc/decodeRaw:L14> invalid pulse-height fill bit from raw value of c026b8: ROC 7 [22,14,88]
     [10:26:52.401] DEBUGAPI: <datatypes.cc/decodeRaw:L14> invalid pulse-height fill bit from raw value of 6b06b8: ROC 7 [61,14,88]
     [10:26:52.401]    ERROR: <datapipe.cc/CheckEventValidity:L484> Channel 2 Number of ROCs (2) != Token Chain Length (4)
     [10:26:52.401] DEBUGAPI: <datatypes.cc/decodeRaw:L14> invalid pulse-height fill bit from raw value of 60135c: ROC 11 [22,14,172]
     [10:26:52.401]    ERROR: <datapipe.cc/CheckEventValidity:L484> Channel 3 Number of ROCs (2) != Token Chain Length (4)
     [10:26:52.401]    ERROR: <datapipe.cc/Read:L164> Dumping the flawed event +- 3 events:
     [10:26:52.401]    ERROR: <datapipe.cc/Read:L166> ====== 0 ====== a32e 8000 4188 35c 2060 4188 35c 204f 4188 35c 2089 4188 35c 2065 e022 c000
     [10:26:52.401]    ERROR: <datapipe.cc/Read:L166> ====== 0 ====== a328 80b1 4389 35c 2060 4389 35c 2060 4389 35c 2089 4389 35c 2065 e022 c000
     [10:26:52.401]    ERROR: <datapipe.cc/Read:L166> ====== 0 ====== a329 80c0 4388 35c 2060 4388 35c 204f 4388 35c 208a 4388 35c 2065 e022 c000
     [10:26:52.401]    ERROR: <datapipe.cc/Read:L166> ====== 0 ====== a32a 8000 4388 35c 2060 4388 35c 2060 4388 35c 2089 4388 35c 2065 e022 c000
     [10:26:52.401]    ERROR: <datapipe.cc/Read:L166> ====== 0 ====== a32b 8040 601 235c 60 4189 35c 2060 4189 35c 2089 4189 35c 2065 e022 c000
     [10:26:52.401]    ERROR: <datapipe.cc/Read:L166> ====== 0 ====== a32c 80b1 4188 35c 2060 4188 35c 204f 4188 35c 2089 4188 35c 2065 e022 c000
     [10:26:52.401]    ERROR: <datapipe.cc/Read:L166> ====== 0 ====== a32d 80c0 4189 35c 2060 4189 35c 204d 4189 35c 2089 4189 35c 2065 e022 c000
     [10:26:52.402] DEBUGAPI: <datapipe.cc/evalReadback:L565> Channel 2 ROC 2: first readback marker after 15 readouts. Ignoring error condition.
     [10:26:52.402]  WARNING: Channel 2 ROC 3: Readback start marker after 15 readouts!
     [10:26:52.402] DEBUGAPI: <datapipe.cc/evalReadback:L565> Channel 3 ROC 2: first readback marker after 15 readouts. Ignoring error condition.
     [10:26:52.402]  WARNING: Channel 3 ROC 3: Readback start marker after 15 readouts!
     [10:26:52.402]    ERROR: <datapipe.cc/Read:L164> Dumping the flawed event +- 3 events:
     [10:26:52.402]    ERROR: <datapipe.cc/Read:L166> ====== 0 ====== a332 8000 4188 35c 2060 4188 35c 2060 4189 35c 2089 4189 35c 2065 e022 c000
     [10:26:52.402]    ERROR: <datapipe.cc/Read:L166> ====== 0 ====== a32c 80b1 4188 35c 2060 4188 35c 204f 4188 35c 2089 4188 35c 2065 e022 c000
     [10:26:52.402]    ERROR: <datapipe.cc/Read:L166> ====== 0 ====== a32d 80c0 4189 35c 2060 4189 35c 204d 4189 35c 2089 4189 35c 2065 e022 c000
     [10:26:52.402]    ERROR: <datapipe.cc/Read:L166> ====== 0 ====== a32e 8000 4188 35c 2060 4188 35c 204f 4188 35c 2089 4188 35c 2065 e022 c000
     [10:26:52.402]    ERROR: <datapipe.cc/Read:L166> ====== 0 ====== a32f 8040 418a 35c 2060 418a 35c 204f 418a 35c 2089 418a 35c 2065 e022 c000
     [10:26:52.402]    ERROR: <datapipe.cc/Read:L166> ====== 0 ====== a330 80b1 4189 35c 2060 4189 35c 204d 4189 35c 2089 4189 35c 2065 e022 c000
     [10:26:52.402]    ERROR: <datapipe.cc/Read:L166> ====== 0 ====== a331 80c0 4189 35c 2060 4189 35c 204d 4189 35c 2089 4189 35c 2065 e022 c000
     [10:26:52.935] DEBUGAPI: <datatypes.cc/decodeRaw:L31> Invalid pixel from raw value of c3c9aa: ROC 10 [72,196,218]
     [10:26:52.935]    ERROR: <datapipe.cc/CheckEventValidity:L484> Channel 2 Number of ROCs (3) != Token Chain Length (4)
     [10:26:52.935] DEBUGAPI: <datatypes.cc/decodeRaw:L14> invalid pulse-height fill bit from raw value of 79354: ROC 14 [22,30,164]
     [10:26:52.935]    ERROR: <datapipe.cc/CheckEventValidity:L484> Channel 3 Number of ROCs (3) != Token Chain Length (4)
     [10:26:52.935] DEBUGAPI: <datapipe.cc/evalReadback:L565> Channel 2 ROC 3: first readback marker after 15 readouts. Ignoring error condition.
     [10:26:52.935] DEBUGAPI: <datapipe.cc/evalReadback:L565> Channel 3 ROC 3: first readback marker after 15 readouts. Ignoring error condition.
     [10:26:52.935]    ERROR: <datapipe.cc/Read:L164> Dumping the flawed event +- 3 events:
     [10:26:52.935]    ERROR: <datapipe.cc/Read:L166> ====== 0 ====== a370 80b1 4389 354 2848 4389 354 2849 4389 354 2882 4389 354 284c e022 c000
     [10:26:52.935]    ERROR: <datapipe.cc/Read:L166> ====== 0 ====== a36a 8000 4188 354 2847 4188 354 2849 4188 354 2881 4188 354 284c e022 c000
     [10:26:52.935]    ERROR: <datapipe.cc/Read:L166> ====== 0 ====== a36b 8040 4189 354 2846 4189 354 2849 4189 354 2881 4189 354 284c e022 c000
     [10:26:52.935]    ERROR: <datapipe.cc/Read:L166> ====== 0 ====== a36c 80b1 4189 354 2847 4189 354 2849 4189 354 2881 4189 354 284c e022 c000
     [10:26:52.935]    ERROR: <datapipe.cc/Read:L166> ====== 0 ====== a36d 80c0 4189 354 2846 4189 354 3849 4389 354 2887 79 2354 84c e022 c000
     [10:26:52.935]    ERROR: <datapipe.cc/Read:L166> ====== 0 ====== a36e 8000 4388 354 2846 4388 354 2849 4388 354 2881 4388 354 284c e022 c000
     [10:26:52.935]    ERROR: <datapipe.cc/Read:L166> ====== 0 ====== a36f 8040 438b 354 2846 438b 354 2849 438b 354 2882 438b 354 284c e022 c000
     [10:26:52.935]    ERROR: <datapipe.cc/Read:L170> Reached 100 dumped events, stopping now...
     [10:26:59.169]     INFO: 2973590 events read in total (110819ms).
     [10:27:13.082] DEBUGAPI: <datatypes.cc/decodeRaw:L31> Invalid pixel from raw value of cfc22e: ROC 7 [79,198,30]

It is not clear to me that the observed printout pattern is a feature:
  * the 100 events printout appears multiple times (also afterwards, the above is just an excerpt)
  * there seem to be less than 100 events between the "stopping now" printouts.

Neither is a real problem, of course, and the log file size is also not a problem (even when running with DEBUGAPI in a fulltest). Just to mention it in case it was not the intended behavior.

Cheers,
--U.

PS: The above is part of the production branch, which I intend to tag soon (I have tested a few modules with the HEAD of the production branch, still 'hoping' maybe for a problem with the readback test).


Yes, this all is a mess...

- https://github.com/OpenSC/OpenSC/blob/master/src/pkcs11/slot.c#L272-L297 calls `pkcs15_bind` for each PKCS\#15 application on the card
- then there is this thing where virtual slots are created if a card has multiple PINs

In theory it would be correct to register the mechanisms by slot (because different applications on the card may have different capabilities). But in practice `pkcs15_bind` throws away the information regarding the application and registers all mechanisms globally in the `p11card`.

The root of this problem can be found in #538, I think. Because most of the code base assumes that the hardware (i.e. the physical card with a static layout) defines what you can do with the card instead of applications that may or may not be found on the card.


遇到过的最诡异的问题！！！为毛updateProject（使用HQL）就不行！！最后无奈自己单独写了个sql来更新mock_num就没问题了。

查了project.hbm也没发现mock_num列有什么奇怪的地方啊，为毛updateProject就不行！！！就不行！！！就不行！！！

fixed #121


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


I had to stop using it; I got warnings for simultaneous chdirs...


@jameskoster don't know why spinner is still loading, but seems working if I remove the url arg and left return as it is :)


This test checks if old cursor stays opened, while the new one is remaining opened. (it is implementation dependent)

The tests should check wether one of the cursors yielding same results remains opened.


ja das stimmt muesi nu luege wiemers anderst chönnt mache


Wouldn't it make sense to put those paths in their own variables?


Nope. The MatrixCursor implementation in Robolectric has changed and `moveToFirst()` calls `moveToPosition(0)`, so we have to change this verify call.


cool mate


I have found that we may need to loosen the precision of the geolocation, in case the coordinates returned by the geo services vary. I added the following to the tests for OE in other pull request (https://github.com/Southbank-Centre/OpenEvent/commit/1eb4aaac6e36daf36b8bd3a89922e55af306e172):
```
expect(json.address.streetAddress).toBe("Southbank Centre, Belvedere Road");
// geolocation & map link as expected for the address input
// added approximation calculation in case the
expect(json.geo.latitude).toBeCloseTo("51.5066566", 1);
expect(json.geo.longitude).toBeCloseTo("-0.11511270000005425", 1);
expect(json.hasMap).toBe("http://www.openstreetmap.org/?mlat=" + json.geo.latitude + "&mlon=" + json.geo.longitude + "#map=15/" + json.geo.latitude + "/" + json.geo.longitude);
```

We must use the way we can guarantee that the data provided will fit within the tests.


Checkmarx scan results 07 Sep 2015 12:54 UTC +03:00

High
Medium
Low
Info

New
0
0
0
0

Recurrent
1
0
0
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

d3d1c0b068aa21a2aa829d5411ca63fda694392a	@dortal


Indeed, there is a problem. I read you changes and already reimplemented it: https://github.com/TelepathyQt/telepathy-qt/commit/8001a42e55b2106eaaa2db6fa3236eb7a436f30d but found out, that it isn't correct to have client socket in setState(Open) condition:

1) http://telepathy.freedesktop.org/spec/Channel_Type_File_Transfer.html#Method:ProvideFile says "The channel ... will change state to Open when this method is called if its state was Accepted." No any word about client socket.
2) I checked TpQt client code: it waits for (State() == Open) before client socket connection.
https://github.com/TelepathyQt/telepathy-qt/blob/master/TelepathyQt/outgoing-file-transfer-channel.cpp#L222
https://github.com/TelepathyQt/telepathy-qt/blob/master/TelepathyQt/incoming-file-transfer-channel.cpp#L272


# masfmkd'k!?


Oeh, I'll steal this for Gandalf.


@monsta @sunweaver I've push a change to pkg-mate that adds --with-cpufreq-lib to `debian/rules`.


Smarty requested a closing tag before `<?xml-stylesheet` (AFAIR) and then could not compile the template. I think it may have been a matter of some changed Smarty internal regex between 3.1.21 and 3.1.27....(?)
```xml
<?xml version="1.0" encoding="utf-8" ?>
<?xml-stylesheet href="{serendipity_getFile file='atom.css'}" type="text/css" ?>
```

Can you confirm it is working now? [Ctrl]-[r]-[F5]


Review done!


Checkmarx scan results 07 Sep 2015 12:56 UTC +03:00

High
Medium
Low
Info

New
0
0
0
0

Recurrent
1
0
0
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

d3d1c0b068aa21a2aa829d5411ca63fda694392a	@dortal


wa na naabot ug DAO oy, grabeha nsad hahaha! service layer xa taman. akong controllers wai logic, ig receive nia respond ra xa. haha! unmaintainable na ang kadaghan sa akong scenarios, mas sayun i-centralize ang validation sa service layer. haha @reylibutan


New module, new API => new major version.


Review Done.


Test


Install hooks belong to install file.
keep in mind that not all modules are available during the installation process. That includes module hooks and other module API functions.


You may want to reconsider how you start the process, as starting your service via rc.local is not a best practice and provides no process management for system administrators. rc.local is also not enabled by default in many distributions (eg. Red Hat, Cent OS).


Ah! Good catch!


Second argument should be any() intead of eq() as this is not a subject of this test


@SimonCropp The name of that class is actually wrong. The name should be `XmlMessageSerializerMessageTypesInitializer`. Then it would clearly state the fact that this only works for XmlSerializer. To answer your question for the Json one. You have to write a similar task currently or we make this again a top level concept of the `IMessageSerializer` interface.


We do not need strongarm if we get rid of features.


Wonderful job!


shouldn't this be `rug_text_all = [rug_text_1, rug_text_2]`?


This source is cm based... So what since is what you are saying? lol


On Windows, fd_t is a HANDLE, so casting to an int is not appropriate. Should the correct fix be to change the prototypes of event_{accepted, closed, listening, connected, disconnected} to take a fd_t instead?


*sense


Thanks for catching these things.


ahh that's awesome. Cheers mate.


Photovoltaics (macumber) - i386-Windows-7-VisualStudio-12: Build Failed

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


102867486-FixRectangleShapeTest-5173 (mjwitte) - x86_64-MacOS-10.9-clang: OK (1823 of 1832 tests passed)

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


`Once you're` => `Once you're ready`


102867486-FixRectangleShapeTest-5173 (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (774 of 774 tests passed)

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


Alright, next meeting I'll give a demo and some people how to connect to the robot and control it.

PennApps was INSANE!!!!


But the Data Vis zip file that contained the SEUS data wasn't in the repo. So you guys had downloaded a new one and were using that, right? I'm guessing yes.


Oh does it? If that is so, it looks like a bug.


What am I missing?


Indeed. That is why I delayed working on it. Hmmm... you are right. I wonder what the impact of this though.


Waiting for PHP 7 and `$options["hosts"] ?? true`...


Many thanks good sir.


@aharley40 why did you add all these too? They're already set in other places, did you have a problem with them not being here or just copy them for fun?


@agoose77

1) Yeah, String.endswith might be a better alternative.
2) I'm not sure what you mean by "a real-time input logger" and a "replay (file) based logger". A logger logs inputs and is basically a replaying system. There shouldn't be a "realtime input logger" as the logger just replicates gameplay. It's not an input system in itself.

A problem with the current approach is that there's no way to "turn off" non-replay-based controls so that you can't play during a replay, so that's something to consider.


OK. Working on this.

Once I create the data set with the 0's, where does it go? Do I add the 0's to `dat`, or do we create a new object with the 0's?  If I remember correctly, I just want to create a new one that Lucas will use, right?


102867486-FixRectangleShapeTest-5173 (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1829 of 1838 tests passed)

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


LGTM


This is the prefix that is supplied for complete. It's not an identifier so I think we should treat it as just Text.


While we're at it... Load also takes a ModuleIdent


Typo


Make this a Type


If you could get in these few corrections, that would be great!


@chriddyp both result in `[[list1], [list2]]`


Does this make the plugin  grab a new xml file every 10 minutes..  most xml EPG providers, even stalker provided source  only update once or twice a day, having every download the same file every 10 minutes puts a lot of unnecessary strain on the servers, those xml files are not small, can this update be reverted/modified..


그냥 개인적인 의견이지만,
+HEAP USAGE RATE
는 큰 의미가 없어 보입니다. ^^;


hi


@chriddyp should I make this: `hist_data = [x1, x2, x3, x4]` ?


any 1 there


Can this also be ignore $resource requests @abombss


You call this "fix a bug"...


No! Did you even test this? `self.root, "bin", "castle"` will not work.


if tanan function nakos controller butngan nako validation dmd oy, hahaha! apil gani ang construction sa response message like "you have successfully..." kay toa sa service layer. ang service layer may kebaw jud if na-fail ba or ni-success ang request. nia if ni-fail, what type of failure sad, daghan sad kaau ko scenarios on different types of failure. hahaha! @reylibutan ag ferry booking lagi daw ingn roy, pwede to nato sideline puhon after ato mga project. haha nindot unta if lahi nsad nga language, haha suggest ko python framework. hahaha!


Ah yes.


you can always overload any method to do custom stuff

```
var emitter = energy()
emitter.emit = function() {
  var result = energy.prototype.emit.apply(this, arguments)
  // do extra stuff
  return result
}


I forgot write `fi` and...I hate the bash syntax...

On 9/8/15, FiveYellowMice <notifications@github.com> wrote:
> You call this "fix a bug"...
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/K-Guan/Arch-quick-install-and-config/commit/256a79c6ede3e8887f1fd9b3f489cb09d017a09c#commitcomment-13102714


Very interesting! Let's walk through it tonight, @snugthejoiner.


Покрутил - просто отлично стало это всё!

Алсо, похоже теперь для `#de-video-list` можно наконец задать `overflow-y: auto;` вместо костыльного `scroll`, вроде всё норм.


-f <filename> -b <block size>. buggy. block_size of 4096 will crash prog.


I mean, if you call this commit "fix a bug", nobody would know what you did
without looking at the diff.
On Tue, 8 Sep 2015 at 10:33, Kevin Guan <notifications@github.com> wrote:

> I forgot write `fi` and...I hate the bash syntax...
>
> On 9/8/15, FiveYellowMice <notifications@github.com> wrote:
> > You call this "fix a bug"...
> >
> > ---
> > Reply to this email directly or view it on GitHub:
> >
> https://github.com/K-Guan/Arch-quick-install-and-config/commit/256a79c6ede3e8887f1fd9b3f489cb09d017a09c#commitcomment-13102714
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/K-Guan/Arch-quick-install-and-config/commit/256a79c6ede3e8887f1fd9b3f489cb09d017a09c#commitcomment-13102723>
> .
>


Yeah, but I don't know what should I call this commit...Any ideas?

On 9/8/15, FiveYellowMice <notifications@github.com> wrote:
> You call this "fix a bug"...
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/K-Guan/Arch-quick-install-and-config/commit/256a79c6ede3e8887f1fd9b3f489cb09d017a09c#commitcomment-13102714


Hi @aslakknutsen how are you?

When this code are merged with master branch? I need configuration suite for my aplication and this code doesn't exists in my current version of arquillian-core

```
        <dependency>
			<groupId>org.jboss.arquillian.core</groupId>
			<artifactId>arquillian-core-api</artifactId>
			<version>1.1.9.Final</version>
		</dependency>
```


Ok. I did consider that btw but, on the other hand, the prefix must obey the same rules as a `DeclIdent`. You might also say that `DeclIdent` really ought to be called `PartialDeclIdent`, where a full identifier is a `PartialDeclIdent` that happens to be 'total'.


:heart:


Like "Fix if syntax".
On Tue, 8 Sep 2015 at 10:38, Kevin Guan <notifications@github.com> wrote:

> Yeah, but I don't know what should I call this commit...Any ideas?
>
> On 9/8/15, FiveYellowMice <notifications@github.com> wrote:
> > You call this "fix a bug"...
> >
> > ---
> > Reply to this email directly or view it on GitHub:
> >
> https://github.com/K-Guan/Arch-quick-install-and-config/commit/256a79c6ede3e8887f1fd9b3f489cb09d017a09c#commitcomment-13102714
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/K-Guan/Arch-quick-install-and-config/commit/256a79c6ede3e8887f1fd9b3f489cb09d017a09c#commitcomment-13102735>
> .
>


Yes. I guess it was a refactoring mistake, this should be UserManagementController


Prueba de TestNG


I see referencies for `@Suite` [here](http://discuss.arquillian.org/t/arquillian-core-suite-subsuite-how-to-define-the-suite/163/3) but this implementation is open for all or not?


Looks good


Curious why you decided against the submodule route? [Heroku resolves submodules automatically](https://devcenter.heroku.com/articles/git-submodules#git-submodules).


Beautiful *.*


Yes, 1.2.4-2 installs cleanly on my Ubuntu box now. (The _VERSION string still says 1.2.0, though!)


Here is https://github.com/boostorg/fusion/blob/develop/include/boost/fusion/algorithm/iteration/detail/fold.hpp#L398.

OK, I'll revert this change and remove `add_const` from initial state.


Very clean well organized code. Also I agree with Paul good use of the commit message, think I may be borrowing from that in the future..


@lojack5: went ahead and reimplemented non shell ui path based on code from #96. Apart from general reviewing I would need feedback (docs) on two things:

- return values (especially mapping)
- possible exceptions raised

Ideally I would like the file operations to return to me meaningful values (for instance delete to return me which files were actually deleted etc) - and code to handle collisions (you started working on) - but apparently SHFileOperation does not return anything useful - see http://stackoverflow.com/questions/29053189/python-win32com-shell-shfileoperation-any-way-to-get-the-files-that-were-actua

If this seems ok though I would like to have it into 306 - as I do not want to lose WINE users (who would possibly be more programmer friendly than windows ones) - asked for feedback on #203 but I do not expect much


chop blank line


make it a docstring?


Isn't it? :)


It does mean that there's a pretty good chance it will be cached when a real user hits the page, though.

(Depending on how we deploy it, anyway. It might not depending on how many worker processes we have.)

The best solution is probably to use memcached/redis and have a cron job update this every minute. But I worry it's somewhat complicated and not sure I want to inflict that on whoever has to maintain this after me. Any thoughts?


In the future, you may want to shorten the length of the lines of code, just for readability sake.


Set it to 60 and reflect on your actions in future.


My suggestion is that you make it easier on the user and use simple numbers for choices.

On case 25 "birth year": The last statement will never be used because the first condition will execute because the variable will be greater than 1980 if its is greater than 2015. Also if the variable is from 1900 to 1980 there will be no statement or message to the user.


I don't think it would matter, assuming our own DNS is up? We shouldn't be hitting university DNS unless our server fails, and status.ocf.b.e is an A record in a zone our server thinks it's authoritative for.


r+


Ok, changed it :smile:


I am not sure what you mean here.

```
return Permutation(std::move(vec));
```
vs.
```
Permutation result;
// ...
return result;
```
?

Both should yield the same results (the first being ROV and the second being NROV). N/RVO constructs the object in the parent scope. The only difference between optimized and non-optimized code is the omission of the call to the copy constructor, so both should be safe.


Then it doesn't matter.


Is it possible and worthwhile to make `--parallel` and `parallel-num` mutually exclusive? It seems that `--parallel-num 99 --parallel` and `--parallel --parallel-num 99` will give the value stored by the last option specified.


Please add periods to help messages.


widih . . . rajin temen commit - e


The change to this download url should probably not be merged!


@filipedeschamps concordo =)


Cool. I made a fix that you'll see in the next version.


This isn't necessary anymore, as the migration uses the table name now (which is plural).


👍
--
Jeremy W. Sherman
http://jeremywsherman.com/

> El 07-09-2015, a las 16:15, archaeron <notifications@github.com> escribió:
>
> Hey
> Thanks for the catch :)
> It's already fixed here: e87a307
>
> —
> Reply to this email directly or view it on GitHub.
>


Clang warns that this is using ```abs(int)```, not ```abs(double)``` and suggest to use ```std::abs(position)``` instead.


Cute, Gary. I like 0 to exit. Might have to change mine.


:metal:


`luaL_checkudata` will raise an error when there is no `BIGINT_POINTER` userdata at position `index`, so `b` will never be `NULL` (`*b` still might). What you want is `luaL_testudata`, which returns `NULL` instead of raising an error. Sadly this function is available only in Lua 5.2+. For Lua 5.1 you could copy it from the Lua 5.2 sources, or from https://github.com/keplerproject/lua-compat-5.3.


Aqui o correto é *strings*, em itálico.

`string` é o tipo do dado, retornado usando `typeof`.

`String` é uma função construtora.


😦Can it be changed?
On Tue, 8 Sep 2015 at 10:56, Kevin Guan <notifications@github.com> wrote:

> Ok, changed it [image: :smile:]
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/K-Guan/Arch-quick-install-and-config/commit/256a79c6ede3e8887f1fd9b3f489cb09d017a09c#commitcomment-13102821>
> .
>


The code below is unreachable. Since the return value of `_checkBigInt` usually isn't checked for `NULL`, you probably want to raise an error instead of returning `NULL` when `*b == NULL` (the other cases are already handled by `luaL_checkudata`).


Sure, Why cannot? This is git, didn't you learn this?

On 9/8/15, FiveYellowMice <notifications@github.com> wrote:
> 😦Can it be changed?
> On Tue, 8 Sep 2015 at 10:56, Kevin Guan <notifications@github.com> wrote:
>
>> Ok, changed it [image: :smile:]
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/K-Guan/Arch-quick-install-and-config/commit/256a79c6ede3e8887f1fd9b3f489cb09d017a09c#commitcomment-13102821>
>> .
>>
>
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/K-Guan/Arch-quick-install-and-config/commit/256a79c6ede3e8887f1fd9b3f489cb09d017a09c#commitcomment-13102873


Yes, two new SEUS files were added, one small file is for STRATA area, and
the other has the catch info. These files are in 'data_updates' >
'data_updated'

> But the Data Vis zip file that contained the SEUS data wasn't in the repo.
> So you guys had downloaded a new one and were using that, right? I'm
> guessing yes.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/mpinsky/OceanAdapt/commit/67bcaf8c16a75ab4c7473c4168769f73ac7f0e28#commitcomment-13102617


Aqui no texto original o autor usa *array* em itálico, e creio que devemos manter essa formatação.


r+


OK, this is amazing.
On Tue, 8 Sep 2015 at 11:06, Kevin Guan <notifications@github.com> wrote:

> Sure, Why cannot? This is git, didn't you learn this?
>
> On 9/8/15, FiveYellowMice <notifications@github.com> wrote:
> > 😦Can it be changed?
> > On Tue, 8 Sep 2015 at 10:56, Kevin Guan <notifications@github.com>
> wrote:
> >
> >> Ok, changed it [image: :smile:]
> >>
> >> —
> >> Reply to this email directly or view it on GitHub
> >> <
> https://github.com/K-Guan/Arch-quick-install-and-config/commit/256a79c6ede3e8887f1fd9b3f489cb09d017a09c#commitcomment-13102821
> >
> >> .
> >>
> >
> >
> > ---
> > Reply to this email directly or view it on GitHub:
> >
> https://github.com/K-Guan/Arch-quick-install-and-config/commit/256a79c6ede3e8887f1fd9b3f489cb09d017a09c#commitcomment-13102873
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/K-Guan/Arch-quick-install-and-config/commit/256a79c6ede3e8887f1fd9b3f489cb09d017a09c#commitcomment-13102880>
> .
>


Nice! Have not felt the need to use `sbt console` yet but glad to know it will work when I do.

The commit name is a little misleading in regard to sbt though. It's just a minor version bump.


Podemos usar itálico aqui também na palavra array.


FYI this link is now a redirect.


I think you've learnt this, not?

On 9/8/15, FiveYellowMice <notifications@github.com> wrote:
> OK, this is amazing.
> On Tue, 8 Sep 2015 at 11:06, Kevin Guan <notifications@github.com> wrote:
>
>> Sure, Why cannot? This is git, didn't you learn this?
>>
>> On 9/8/15, FiveYellowMice <notifications@github.com> wrote:
>> > 😦Can it be changed?
>> > On Tue, 8 Sep 2015 at 10:56, Kevin Guan <notifications@github.com>
>> wrote:
>> >
>> >> Ok, changed it [image: :smile:]
>> >>
>> >> —
>> >> Reply to this email directly or view it on GitHub
>> >> <
>> https://github.com/K-Guan/Arch-quick-install-and-config/commit/256a79c6ede3e8887f1fd9b3f489cb09d017a09c#commitcomment-13102821
>> >
>> >> .
>> >>
>> >
>> >
>> > ---
>> > Reply to this email directly or view it on GitHub:
>> >
>> https://github.com/K-Guan/Arch-quick-install-and-config/commit/256a79c6ede3e8887f1fd9b3f489cb09d017a09c#commitcomment-13102873
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/K-Guan/Arch-quick-install-and-config/commit/256a79c6ede3e8887f1fd9b3f489cb09d017a09c#commitcomment-13102880>
>> .
>>
>
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/K-Guan/Arch-quick-install-and-config/commit/256a79c6ede3e8887f1fd9b3f489cb09d017a09c#commitcomment-13102888


saw approval from jedmccaleb
at https://github.com/MonsieurNicolas/stellar-core/commit/9e0923094b165b9331c443b80df9c5d1ea0e3a6b


merging MonsieurNicolas/stellar-core/windowsFixes = 9e092309 into auto


MonsieurNicolas/stellar-core/windowsFixes = 9e092309 merged ok, testing candidate = 4654e395


Nice work, when creating calculators do not forget to account for division by zero.


Wwww, I forgot.
On Tue, 8 Sep 2015 at 11:10, Kevin Guan <notifications@github.com> wrote:

> I think you've learnt this, not?
>
> On 9/8/15, FiveYellowMice <notifications@github.com> wrote:
> > OK, this is amazing.
> > On Tue, 8 Sep 2015 at 11:06, Kevin Guan <notifications@github.com>
> wrote:
> >
> >> Sure, Why cannot? This is git, didn't you learn this?
> >>
> >> On 9/8/15, FiveYellowMice <notifications@github.com> wrote:
> >> > 😦Can it be changed?
> >> > On Tue, 8 Sep 2015 at 10:56, Kevin Guan <notifications@github.com>
> >> wrote:
> >> >
> >> >> Ok, changed it [image: :smile:]
> >> >>
> >> >> —
> >> >> Reply to this email directly or view it on GitHub
> >> >> <
> >>
> https://github.com/K-Guan/Arch-quick-install-and-config/commit/256a79c6ede3e8887f1fd9b3f489cb09d017a09c#commitcomment-13102821
> >> >
> >> >> .
> >> >>
> >> >
> >> >
> >> > ---
> >> > Reply to this email directly or view it on GitHub:
> >> >
> >>
> https://github.com/K-Guan/Arch-quick-install-and-config/commit/256a79c6ede3e8887f1fd9b3f489cb09d017a09c#commitcomment-13102873
> >>
> >> —
> >> Reply to this email directly or view it on GitHub
> >> <
> https://github.com/K-Guan/Arch-quick-install-and-config/commit/256a79c6ede3e8887f1fd9b3f489cb09d017a09c#commitcomment-13102880
> >
> >> .
> >>
> >
> >
> > ---
> > Reply to this email directly or view it on GitHub:
> >
> https://github.com/K-Guan/Arch-quick-install-and-config/commit/256a79c6ede3e8887f1fd9b3f489cb09d017a09c#commitcomment-13102888
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/K-Guan/Arch-quick-install-and-config/commit/256a79c6ede3e8887f1fd9b3f489cb09d017a09c#commitcomment-13102894>
> .
>


Actually, `luaL_openlib` is a Lua 5.0 function. It is still available on Lua 5.1 if it was compiled with `LUA_COMPAT_OPENLIB`, but the native Lua 5.1 function would be `luaL_register( L, NULL, meta )`.


I'm laughing at your commit message. What a childish way to insult a ROM which your rom is 99% derivative.


All right. use git, we can change everythings.

On 9/8/15, FiveYellowMice <notifications@github.com> wrote:
> Wwww, I forgot.
> On Tue, 8 Sep 2015 at 11:10, Kevin Guan <notifications@github.com> wrote:
>
>> I think you've learnt this, not?
>>
>> On 9/8/15, FiveYellowMice <notifications@github.com> wrote:
>> > OK, this is amazing.
>> > On Tue, 8 Sep 2015 at 11:06, Kevin Guan <notifications@github.com>
>> wrote:
>> >
>> >> Sure, Why cannot? This is git, didn't you learn this?
>> >>
>> >> On 9/8/15, FiveYellowMice <notifications@github.com> wrote:
>> >> > 😦Can it be changed?
>> >> > On Tue, 8 Sep 2015 at 10:56, Kevin Guan <notifications@github.com>
>> >> wrote:
>> >> >
>> >> >> Ok, changed it [image: :smile:]
>> >> >>
>> >> >> —
>> >> >> Reply to this email directly or view it on GitHub
>> >> >> <
>> >>
>> https://github.com/K-Guan/Arch-quick-install-and-config/commit/256a79c6ede3e8887f1fd9b3f489cb09d017a09c#commitcomment-13102821
>> >> >
>> >> >> .
>> >> >>
>> >> >
>> >> >
>> >> > ---
>> >> > Reply to this email directly or view it on GitHub:
>> >> >
>> >>
>> https://github.com/K-Guan/Arch-quick-install-and-config/commit/256a79c6ede3e8887f1fd9b3f489cb09d017a09c#commitcomment-13102873
>> >>
>> >> —
>> >> Reply to this email directly or view it on GitHub
>> >> <
>> https://github.com/K-Guan/Arch-quick-install-and-config/commit/256a79c6ede3e8887f1fd9b3f489cb09d017a09c#commitcomment-13102880
>> >
>> >> .
>> >>
>> >
>> >
>> > ---
>> > Reply to this email directly or view it on GitHub:
>> >
>> https://github.com/K-Guan/Arch-quick-install-and-config/commit/256a79c6ede3e8887f1fd9b3f489cb09d017a09c#commitcomment-13102888
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/K-Guan/Arch-quick-install-and-config/commit/256a79c6ede3e8887f1fd9b3f489cb09d017a09c#commitcomment-13102894>
>> .
>>
>
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/K-Guan/Arch-quick-install-and-config/commit/256a79c6ede3e8887f1fd9b3f489cb09d017a09c#commitcomment-13102910


In module Events

def self.created_subscription_details
     Person::PERSON_CREATED_EVENT_NAME
   end

   def self.created_subscription_details                     [DUPLICATED CODE]
     Person::PERSON_UPDATED_EVENT_NAME
   end

line 31 references updated_subscription_details,  that  method appears to be undefined


Thx.

However, "Everything" is always singular, you need improvements on your
English.
On Tue, 8 Sep 2015 at 11:14, Kevin Guan <notifications@github.com> wrote:

> All right. use git, we can change everythings.
>
> On 9/8/15, FiveYellowMice <notifications@github.com> wrote:
> > Wwww, I forgot.
> > On Tue, 8 Sep 2015 at 11:10, Kevin Guan <notifications@github.com>
> wrote:
> >
> >> I think you've learnt this, not?
> >>
> >> On 9/8/15, FiveYellowMice <notifications@github.com> wrote:
> >> > OK, this is amazing.
> >> > On Tue, 8 Sep 2015 at 11:06, Kevin Guan <notifications@github.com>
> >> wrote:
> >> >
> >> >> Sure, Why cannot? This is git, didn't you learn this?
> >> >>
> >> >> On 9/8/15, FiveYellowMice <notifications@github.com> wrote:
> >> >> > 😦Can it be changed?
> >> >> > On Tue, 8 Sep 2015 at 10:56, Kevin Guan <notifications@github.com>
> >> >> wrote:
> >> >> >
> >> >> >> Ok, changed it [image: :smile:]
> >> >> >>
> >> >> >> —
> >> >> >> Reply to this email directly or view it on GitHub
> >> >> >> <
> >> >>
> >>
> https://github.com/K-Guan/Arch-quick-install-and-config/commit/256a79c6ede3e8887f1fd9b3f489cb09d017a09c#commitcomment-13102821
> >> >> >
> >> >> >> .
> >> >> >>
> >> >> >
> >> >> >
> >> >> > ---
> >> >> > Reply to this email directly or view it on GitHub:
> >> >> >
> >> >>
> >>
> https://github.com/K-Guan/Arch-quick-install-and-config/commit/256a79c6ede3e8887f1fd9b3f489cb09d017a09c#commitcomment-13102873
> >> >>
> >> >> —
> >> >> Reply to this email directly or view it on GitHub
> >> >> <
> >>
> https://github.com/K-Guan/Arch-quick-install-and-config/commit/256a79c6ede3e8887f1fd9b3f489cb09d017a09c#commitcomment-13102880
> >> >
> >> >> .
> >> >>
> >> >
> >> >
> >> > ---
> >> > Reply to this email directly or view it on GitHub:
> >> >
> >>
> https://github.com/K-Guan/Arch-quick-install-and-config/commit/256a79c6ede3e8887f1fd9b3f489cb09d017a09c#commitcomment-13102888
> >>
> >> —
> >> Reply to this email directly or view it on GitHub
> >> <
> https://github.com/K-Guan/Arch-quick-install-and-config/commit/256a79c6ede3e8887f1fd9b3f489cb09d017a09c#commitcomment-13102894
> >
> >> .
> >>
> >
> >
> > ---
> > Reply to this email directly or view it on GitHub:
> >
> https://github.com/K-Guan/Arch-quick-install-and-config/commit/256a79c6ede3e8887f1fd9b3f489cb09d017a09c#commitcomment-13102910
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/K-Guan/Arch-quick-install-and-config/commit/256a79c6ede3e8887f1fd9b3f489cb09d017a09c#commitcomment-13102925>
> .
>


Why the __ prefix?


why not unsigned char key_buff[8]; ?


Sei que não foi a intenção mas achei esse parágrafo bastante _passivo agressivo_. Na tradução pro inglês ( https://github.com/idmarjr/frontendpath/commit/ef68916c50c32e3c5ddc35ea443f998bb48f4895 ) alterei ele consideravelmente e acho que seria interessante ter a mesma abordagem aqui.

Acho que seria legal usar um gancho para a [ página sobre box-model do MDN](https://developer.mozilla.org/pt-BR/docs/Web/CSS/box_model), assim como foi feito na sessão de **Pseudo-class selector** desse mesmo arquivo. Assim quem quiser ir mais a fundo no assunto tem acesso fácil a uma boa fonte para isso.


How do I download it?


好是好。。。可是我下载下来都是空文件啊，怎么回事？？？


Just a type wrong...I've changed it.


Add a config option if on windows :)


Just seemed prudent to avoid any weird collisions... any alternative you recommend?


all tests pass:
success: https://travis-ci.org/stellar/stellar-core/builds/79207492


Typo :smile:


Anyway, you should start an HAML file with the following:

```haml
!!!
%html{ lang: 'en' }
  %head
    -# ...
```


Honestly you could probably do this with no ill effects:

```
r = is_premul ? 255 : set
// etc
```

It looks slow, but the compiler is usually smart enough to optimize it (loop inversion).


Fun fact:
En markdown los espacios esos sirven para dar formato (un salto de linea no se reconoce si no está precedido por un espacio en blanco)

Se me hace que tu editor, intentando ser buena onda, quitó los trailing spaces al guardar el archivo :B
Pero ni al caso haha no importa c:


I'm very sorry again.


I haven't fixed this but all else is fixed in 2.3.10-r1 when it is known what to do for the licence I'll change it (if needed).


I also saw that error, while running rake db:seed in development environment. somehow, I have no problem with rake db:seed in test environment. It is the reason. all test work well


fast-forwarding master to auto = 4654e395


102867486-FixRectangleShapeTest-5173 (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1307 of 1307 tests passed)

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


All roads lead to Rome :)


Photovoltaics (macumber) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


化けてるので、文字コードをUTF-8にしてください。


Thanks for the information.
I will list it also to my README. To guide people to right place. :)

I don't know where is the libsass implementation page and don't care about it.
Anyway, I think this project is a libsass "binding" not implementation.


102867486-FixRectangleShapeTest-5173 (mjwitte) - Win64-Windows-7-VisualStudio-12: OK (1829 of 1838 tests passed)

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


hey parker, I'm trying to test your code before reviewing but I am having some problems with doing test cases.
I started dropbox-server and try dropbox --d1 dnode://test-data/folder1 --d2 file://test-data/folder2
it says something about cwd(). Could you provide me with a test command I could try?
thanks.


add comment.......


@10se1ucgo Alright so I tested EVERYTHING on a testbench... The only things not working right now then:

[x] Windows Defender Registry keys: Access Denied
[x] OneDrive: List Pin entries no longer remove it from the explorer menu (thanks Micro$oft?)


同じく化けているので文字コードをUTF-8にしてください


I would make it start with a \000 character, as it's unlikely to be the first character in the serialization of a key.


Or make it 32 bytes and use a uint256 (this is a nit).


102817062_MultiSpeedFan (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1300 of 1300 tests passed)

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


There is same process in store method. As I told you below, I think you'd better to write the process as shared class.
https://github.com/Mulodo-PHP-Laravel-Training/tuan-nguyen-laravel/commit/4068793390d27bb534943246ed7780d695d4754f

e.g.
class MessageUtility
{
    public static function getErrorMessageForResponse (array $errorMessages)
    {
        $responseMessages = array();

        foreach ($errorMessages as $messages) {
            foreach ($messages as $message) {
                $responseMessages[] = array('message' => $message);
            }
        }
        return $responseMessages;
    }
}

$response['meta']['messages'] = MessageUtility::getErrorMessageForResponse($validator->errors()->getMessages());

How do you think?

P.S I have not do debug above code, so it doesnt work correctly.


$(function() {
var number = parseInt(prompt("What number would you like me to fizzbuzz up to?")) ;

for (var index = 1; index <= number; index += 1 ) {
   if (index % 15 === 0 ) {
     document.write('
ping-pong
');
      document.write("
");
   } else if (index % 3 === 0) {
      document.write('
ping
');
      document.write("
");
   } else if (index % 5 === 0) {
      document.write('pong');
      document.write("
");
  }
     else {
     document.write(index);
     document.write("
");
  }
 }});


Last time I ran the script was before Lucas had updated the website, so I
had manually added the SEUS files to the appropriate folder. I will update
the data_download folder with a new Data_Vis file.

On Mon, Sep 7, 2015 at 8:10 PM, Ryan Batt <notifications@github.com> wrote:

> But the Data Vis zip file that contained the SEUS data wasn't in the repo.
> So you guys had downloaded a new one and were using that, right? I'm
> guessing yes.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mpinsky/OceanAdapt/commit/67bcaf8c16a75ab4c7473c4168769f73ac7f0e28#commitcomment-13102617>
> .
>


#5067-CoilCoolingWaterToAirHeatPumpVariableSpeedEquationFit-inside-AirLoopHVACUnitarySystem (rraustad) - x86_64-MacOS-10.9-clang: Build Failed

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


Yes: create a new object. You could even write it to a csv file, but that's
probably not critical.

On Mon, Sep 7, 2015 at 8:16 PM, Ryan Batt <notifications@github.com> wrote:

> OK. Working on this.
>
> Once I create the data set with the 0's, where does it go? Do I add the
> 0's to dat, or do we create a new object with the 0's? If I remember
> correctly, I just want to create a new one that Lucas will use, right?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mpinsky/OceanAdapt/commit/5488fba14d8aafd5ab9db1ecffc6e53ca751cfc0#commitcomment-13102643>
> .
>


xxxxxxxxxxxx


@mlubin Any qualms against me merging this?


Let me know if you run into trouble. I know you've put in a bunch of time
already, so I will take it on if this starts to eat up even more time.
Thank you for working on it.

On Mon, Sep 7, 2015 at 9:40 PM, Malin Pinsky <malin.pinsky@gmail.com> wrote:

> Yes: create a new object. You could even write it to a csv file, but
> that's probably not critical.
>
> On Mon, Sep 7, 2015 at 8:16 PM, Ryan Batt <notifications@github.com>
> wrote:
>
>> OK. Working on this.
>>
>> Once I create the data set with the 0's, where does it go? Do I add the
>> 0's to dat, or do we create a new object with the 0's? If I remember
>> correctly, I just want to create a new one that Lucas will use, right?
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/mpinsky/OceanAdapt/commit/5488fba14d8aafd5ab9db1ecffc6e53ca751cfc0#commitcomment-13102643>
>> .
>>
>
>


sss


$(function() {
var number = parseInt(prompt("What number would you like me to fizzbuzz up to?")) ;

for (var index = 1; index <= number; index += 1 ) {
   if (index % 15 === 0 ) {
     document.write('fizzbuzz');
      document.write("
");
   } else if (index % 3 === 0) {
      document.write('fizz');
      document.write("
");
   } else if (index % 5 === 0) {
      document.write('buzz');
      document.write("
");
  }
     else {
     document.write(index);
     document.write("
");
  }
 }});


will do!


oh, indeed: https://dev.gentoo.org/~ulm/pms/head/pms.html#x1-11800011.1


UTF-8への変更OKです。


Hi MarinS!

Does that little message saying "no newline at end of file" indicate anything. From what I've been studying it didn't seem like it'd make a difference if you added like \n or not...not sure though :)


En este punto ya he cambiado mucho código, hay cosas que se han implementado, que no estaban en planes, pero son funciones muy interesantes.


I doubt anyone would be confused, and what if they never notice it's not official?! DDDD:


Realized this wasn't needed because Ryan had already done this with 67bcaf8c16a75ab4c7473c4168769f73ac7f0e28


@rogersb11

Fyi. I think I know the issue here. Basically display font size and clock font size are interlinked. One is a string the other is an integer (I think so). So when one goes to display font size it triggers FC where integer cannot be cast on string. And when one goes to statusbar font size, it triggers FC where string cannot be cast on integer. Deleting /data/data/com.android.settings only helps temporary. Need to figure out which to follow. String or integer.

If you have any inputs......let me know. Thanks


No sé que le ha pasado al administrador del controlador de versiones de Android Studio, pero, esta maravilla me ha hecho hacer otra vez commit, thank's google.


Isso é contra as regras da licença do MIT. Por gentileza respeite todo o trabalho que tivemos e mantenha os nomes do criadores,


Sir, "wrong" is a adjective.
On Tue, 8 Sep 2015 at 11:22, Kevin Guan <notifications@github.com> wrote:

> Just a type wrong...I've changed it.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/K-Guan/Arch-quick-install-and-config/commit/256a79c6ede3e8887f1fd9b3f489cb09d017a09c#commitcomment-13102975>
> .
>


又不是看不懂

On 9/8/15, FiveYellowMice <notifications@github.com> wrote:
> Sir, "wrong" is a adjective.
> On Tue, 8 Sep 2015 at 11:22, Kevin Guan <notifications@github.com> wrote:
>
>> Just a type wrong...I've changed it.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/K-Guan/Arch-quick-install-and-config/commit/256a79c6ede3e8887f1fd9b3f489cb09d017a09c#commitcomment-13102975>
>> .
>>
>
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/K-Guan/Arch-quick-install-and-config/commit/256a79c6ede3e8887f1fd9b3f489cb09d017a09c#commitcomment-13103112


`fullscreen` is an unknown option in gvim on windows, :-1:


算了，不吐槽你的英语了……
On Tue, 8 Sep 2015 at 11:50, Kevin Guan <notifications@github.com> wrote:

> 又不是看不懂
>
> On 9/8/15, FiveYellowMice <notifications@github.com> wrote:
> > Sir, "wrong" is a adjective.
> > On Tue, 8 Sep 2015 at 11:22, Kevin Guan <notifications@github.com>
> wrote:
> >
> >> Just a type wrong...I've changed it.
> >>
> >> —
> >> Reply to this email directly or view it on GitHub
> >> <
> https://github.com/K-Guan/Arch-quick-install-and-config/commit/256a79c6ede3e8887f1fd9b3f489cb09d017a09c#commitcomment-13102975
> >
> >> .
> >>
> >
> >
> > ---
> > Reply to this email directly or view it on GitHub:
> >
> https://github.com/K-Guan/Arch-quick-install-and-config/commit/256a79c6ede3e8887f1fd9b3f489cb09d017a09c#commitcomment-13103112
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/K-Guan/Arch-quick-install-and-config/commit/256a79c6ede3e8887f1fd9b3f489cb09d017a09c#commitcomment-13103119>
> .
>


@xiphiaz Yes, though not immediately. Currently we sort of store the "raw" data for each change, and in the case of sync, we store the ids before and after. Could be slimmed down a bit by using this update, though not necessary for functionality. When it's time to define further exactly how and what to store for each change and for display purposes, this will most certainly come in handy and ease the loging of syncs. Though I'd say updating to use this would be appropriate to do when refining for what data exactly to return for display purposes.


Yes, the use of ellipsis instead of round() was intentional in model 1. For models 2 to 5, the results of the doctests were close to round numbers, like 1.0, 0.0, and 0.5, so I used round() rather than something like 0.99999... or 0.49999....

For model 1, the results were not "nice" to round to, like 0.32741... or 0.89123...,  so I left it as it is. Given more training iterations, the results would have converged to whole numbers like 1.0 and 0.0.


Olá amigo, desculpe mas o que eu fiz?
Nao me lembro de ter alterado nada! Caso fiz algo errado nao foi minha
intencao! Desculpe
Em 07/09/2015 22:48, "Paulo Miguel Almeida" <notifications@github.com>
escreveu:

> Isso é contra as regras da licença do MIT. Por gentileza respeite todo o
> trabalho que tivemos e mantenha os nomes do criadores,
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/danielperes/nfscan/commit/00a6de50c5cac2936494ed24b0107f0444b44ef9#commitcomment-13103110>
> .
>


I added it anyway, not like it hurts to have `//aasd`


Hmm, it still allocates memory in computing the hash so that may be why. Let me get back to you with either:

- an implementation of `hash_combine()`
- a (rough) implementation of this: http://www.open-std.org/jtc1/sc22/wg21/docs/papers/2014/n3980.html


@kennethlimcp would you please add a reference here to "We did some fixing where a Particle device is not detected with new OS (Win 10) etc." wherever it was fixed.  Even though it sounds like we don't base anything on the MFG strings, you seem to indicate the possibility that we did, or do.  Just want to make sure this doesn't slip through the cracks :)


Can you see the user input prompt? You should be able to type in help to
get all the possible commands to run

On Monday, September 7, 2015, Jacob Ho <notifications@github.com> wrote:

> hey parker, I'm trying to test your code before reviewing but I am having
> some problems with doing test cases.
> I started dropbox-server and try dropbox --d1 dnode://test-data/folder1
> --d2 file://test-data/folder2
> it says something about cwd(). Could you provide me with a test command I
> could try?
> thanks.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/cs4278-2015/assignment2-handin/commit/6b7851264160980d527183978d3b29ce8c875797#commitcomment-13103040>
> .
>


버튼 업데이트 하는 부분  btnStates 가 infos 로 데이타를 설정해주는데요.
플러그인이 다른 형태로 버튼을 만들었을 때는  상태체크를 이쪽에서 하지 않고 플러그인 내부에서 하면 되나요?

왠지 상태 체크가 분리되는 느낌이 들어서요.


According to [this comment](https://github.com/bruceravel/gnuplot-mode/issues/15#issuecomment-56550800), this can be fixed.  I'll still have a look at create an option so that on Windows, it has to be manually activated.


There is same code in "update" process. It's not good. If you need to write same code, you'd be better to consider to write the process as new class and method like below.

```
public function createValidator()
{
    return Validator::make($request->all(), [
                    'firstName' => 'required|max:255',
                    'lastName'  => 'required|max:255',
                    'email'      => 'required|email|max:255|unique:users',
                    'password'   => 'required|min:6',
             ]);
}

$validator = $this->createValidator();

```


This always does a full scan of the string. Will words ever begin with spaces? Can we just check the first character?


Now if only we could get it to actually be 2 taps instead of 5...


反正又不是看不懂，问题解决了。

On 9/8/15, FiveYellowMice <notifications@github.com> wrote:
> 算了，不吐槽你的英语了……
> On Tue, 8 Sep 2015 at 11:50, Kevin Guan <notifications@github.com> wrote:
>
>> 又不是看不懂
>>
>> On 9/8/15, FiveYellowMice <notifications@github.com> wrote:
>> > Sir, "wrong" is a adjective.
>> > On Tue, 8 Sep 2015 at 11:22, Kevin Guan <notifications@github.com>
>> wrote:
>> >
>> >> Just a type wrong...I've changed it.
>> >>
>> >> —
>> >> Reply to this email directly or view it on GitHub
>> >> <
>> https://github.com/K-Guan/Arch-quick-install-and-config/commit/256a79c6ede3e8887f1fd9b3f489cb09d017a09c#commitcomment-13102975
>> >
>> >> .
>> >>
>> >
>> >
>> > ---
>> > Reply to this email directly or view it on GitHub:
>> >
>> https://github.com/K-Guan/Arch-quick-install-and-config/commit/256a79c6ede3e8887f1fd9b3f489cb09d017a09c#commitcomment-13103112
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/K-Guan/Arch-quick-install-and-config/commit/256a79c6ede3e8887f1fd9b3f489cb09d017a09c#commitcomment-13103119>
>> .
>>
>
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/K-Guan/Arch-quick-install-and-config/commit/256a79c6ede3e8887f1fd9b3f489cb09d017a09c#commitcomment-13103126


#5067-CoilCoolingWaterToAirHeatPumpVariableSpeedEquationFit-inside-AirLoopHVACUnitarySystem (rraustad) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Yes. It was "byte" when I posted the comment.


I think that it will help my code better. That was my mistake because I thought that it was short and I didn't write it in shared class.


cygwin on windows falls here, can distinguish it via `has("win32unix")` :smiley:


From 1-2 FPS all the way up to 76 FPS. Woots.


We should keep a certain consistency here. if ``getfloat`` requires a float the default value MUST have the same type.


What? No license?


fixes #2


@piotrpalek I don't recall they claim it :-) I think Yehuda was commenting several times that it's a bit different to React (although some ideas were borrowed). I have created an example to compare approaches. So in case of selected attribute no rendering happens on selection at all.


why use double -_-


should call p*y*thon :)


I think it's fine xD


Yep right - added a squashme commit which should make the rename complete (latest commit...)


@piotrpalek https://github.com/emberjs/ember.js/pull/10501


Hmm I think I remember the opposite, afaik they said the implementation is different (glimmer takes a bit different approach to the virtual dom then react) but the main idea stays the same - re-rendering should be cheap and we should be able to throw away all of the old state, re-render the new state (in memory) and the rendering engine should figure out what to replace in the real DOM so it's still perfomant.


I've been thinking the resources that depend on it don't actually have to be in the same group. I'm asking the customer if an implementation of multiple-active="ignore" would meet their need.


This constant name is nonsense - 5729.577951308f = 36000/(2*pi) - conversion from radians to degrees*100


Building VirtualDOM is not cheap too. And Ember offers a way to optimize here too (that's different from React, although I have heard it's coming there too)


Indeed... it was 22h... my bad :)
thanks


/cc @panshul007


@sfarmar Can you provide a recommendation for batch size?


@willl https://github.com/TypeStrong/atom-typescript/pull/559


92行目でhtmlを修正できます


@expez I do agree that it would be nice to have linking in the documentation buffer.
But, just to be clear, this PR does not highlight the code in doc buffers. This PR only highlights symbols in the code buffer. So it doesn't create the expectation of links, I think.


hm I was thinking memes could be separated into a new language, that way there will be no limit as to which texts can be trolled, while keeping the `en` and other languages clean without duplicate entries, e.g. `EventHistoryRank1` vs `EventHistoryRank1X`


Wouldn't a single asterisk suffice?


Thanks for pointing this. That's legacy from previos code and will go eventually.


Hello, So if I make all the changes listed above will "curl loopback" problem be solved ??, please advice. Thank You for your time.


but isn't this info already in the audit log?


I don't know, I find the separation not so artificial. The current implementation has the advantage to clearly separate the behavior of a full handshake and an abbreviated one. I mean when you need to understand the code, you can focus on a single kind of handshake reading the corresponding class.

Personally I 'm more disturbed by the doProcessMessage which seems to be duplicated in all handshake class except the `switch (record.getType())` in all Handshakers.


Looks like the Hugo export is still labeled as Inform 7


@boothj5 -- I'm commenting on the 0.4.7.rc1 tagged commit because I'm an everyday user of profanity and big fan of it and all the hard work you've been putting in. I've seen a lot of commits going by the past two months on github and I appreciate the time you've dedicated to work on this. I'm looking forward to the 0.4.7 release. @changetip please pass a beer to @boothj5 !


Hi @boothj5, @vsergeev sent you a tip worth a beer (14,113 bits/$3.50), and I'm here to deliver it ➔ **[collect your tip](https://www.changetip.com/collect/892782)**.

**[Learn more about ChangeTip](https://www.changetip.com/tip-online/github)**


Does hugo have it's own extension we could save to?


This is to show that we can edit content in markdown and transform to html automatically. The benefit is content can be extracted from code, and anyone can use a markdown tool to edit content.


Useless: get() already calls wait()


You're absolutely right :-) Thanks for noticing, I'll give the testing branch a try.


Maybe extract this condition into a method?


What is the purpose of this line? Why hardcoded `640`?


I think the virtual dom is actually plenty cheap, since react is based on the idea and people are using it :P In general what I mean is no matter how they implement it, they want us to be able to do re-renders of the whole thing.
Anyway it's good this actually performs quiet well.
@OrKoN you think this is perfomant enough? I think we can accept a small hit in performance if this would mean the component is easier to maintain/extend, what do you think? Are there any other concerns you have with this?


I'm no security guru, but isn't this bad?


@snugthejoiner that makes a lot of sense with confirmed. I totally forgot that you didn't have confirmed in your factory.

I will definitely look at your helper but it will be much later tonight or tomorrow. Keep moving, don't wait on me.


I'm going to add this to `gds-api-adapters`, but that shouldn't hold this PR back.


Either this should be ```Ramsey``` instead of ```RamseyConstraints``` or the file should be placed in ```RamseyConstraints```


Oh, sorry, I did not see your messages. Yep, no problem, I'll add unit tests and create a pull request :)


I wanted to avoid a user being able to edit the original list of operations through this method (explicit interface implementation is currently only used by the JsonConverter)


Fixed in master


Just as a coding note, we could probably do something like (untested):

```c#
var regNameList = Settings.Regions.Select(p=>p.RegionName).ToList();
var regTextColorList= Settings.Regions.Select(p=>p.TextColor).ToList();
var regBkgdColorList= Settings.Regions.Select(p=>p.RColor).ToList();
```


Great :)


I can think of ways to improve this, but they mostly involve refactoring more than just this code and would be a much bigger architectural change.....they might be worthwhile doing at some point, but this should suffice for now.


这样做好吗？


Recuerdo algunos, pero no se dónde están...!


MFC ... :walking:


Alles wat we niet nodig hebben kan er uit. Als we het later toch nodig blijken te hebben kan je het altijd weer adden.

Vergeet ook niet alles te documenteren, dmv. Javadoc !


@richmidwinter We decided to keep the log message on debug level because malicious (or malfunctioning) clients might swamp the logs with warning messages.

Also see https://github.com/dropwizard/dropwizard/pull/1052#issuecomment-102771678


Attention, il me semblait qu'il ne fallait pas mettre d'accents ?


I'd say let's keep it in `constants.py`.


这个写错了吧? 应该是  `git branch -d`


`RcppGSL::Matrix<int>` also has a typedef as `RcppGSL::IntMatrix`, they are equivalent.


why???


why not??


This class is not necessary as it is a block..

Also please note that you should make one nice commit instead of 6 commits for each file, for each pull request.


@mosic what is the memory leak here and how does this fix it?


forget optional in here


Fix reference: https://github.com/libis/CAG/issues/210


Please don't turn on Travis on your fork without disabling IRC notifications. You're also welcome to open a WIP pull request to get a build.


Welcome to new group of Urdu Poets and Writers


It's great that now there is support SDL_Mixer. Thank you!


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


niubility!


forget $2 here.


@shankardevy Good question. When `Task.await` isn't used the results of `Task.async` are stored in the mailbox which expands to fill all available memory.


proposal:

```...will fail with ``cast`` if ``string`` is no valid integer but return ``null`` with ``try_cast``:```


Ah yes, I see, `.discard()` is invoking the local `.read()` method (https://github.com/eventlet/eventlet/blob/b443de322aed2b0f9f0d1aab3d24c1f7cd5bd2da/eventlet/wsgi.py#L191), not a lower-level one.  Thanks!


合並


你好，我自己做的Demo 跳转时为什么奇数次都卡在 正在加载 页面呀，求教


Avatar `https://avatars1.githubusercontent.com/u/14178114?v=3&s=200`
![14178114](https://cloud.githubusercontent.com/assets/4997583/9732821/0ef714d6-5627-11e5-8b71-e4a4d698be3d.png)


Ne pas tenir compte des modifications des pages blog, BlogController, showArticles, oneArticle, en cours de debbuggage grâce à un forum.


Looks like this commit broke build.
Please see http://ci.cgeo.org/job/c-geo/4021/console to remedy the problem.


is it needed?


why indent differs?


0.0.2 was submitted to the group for publishing. We are at 0.0.3 :)


invert if


Luckily these aren't our real credentials - as we have overridden them in the hosted app :lock:


Hello.
I have a board terasic SocKit cyclone 5.
I decided to do your job.
I repeated the first two parts.
In Part 3, the management of flashing LEDs delayed by the HPS,
In Putty I run command "./blinker 1", leds quickly blink only 1 second, and then again slowly, why is this happening? as though it frays the data that I recorded in the memory of lightweight bridge's region.
What could be wrong?
As I understand the LEDs should blink quickly until I started with the command "./blinker 15"...


DRY


test


Hey,

The issues you mentioned should be fixed in the latest release (0.5.3), To actually customize the backend you need to set the options as defined in the extensions readme.

Let me know if you run into more issues!


I would put that in your own, global, .gitignore


what is LsObject?


if we are saying that they can look pretty much like whatever as long as the icon is included, why not just add one line in the 'icon' section about the back icon?


good job! now for the next level...make a branch called new-content and add some text in body and merge it to master.


Whitespace?


I did not notice your comment, I did not get a mail for it. I noticed it now by accident.

For a) there is usually *no* space before the emdash in english. But in other languages there is. I'll test that case.

I'll test b).

The reason for adding this is a bug found for MDOC (man page macros). If an option is shown in a man page (like `-a`) and anything had been before the `-` (might it be quotes or even something non-printable) then `-` appeared on one line and `a` on the next. There had even been hyphens added which appeared as ` -` at the end of a line, simply because there had something non-printable before a word. This is an old roff bug--it is also in DWB2 and DWB3.3.


Whitespace again?


Does this actually test something?

All added code is dead code.


This works. I'll try `become_user` as well.


It looks indented when I open it in vim and not indented in eclipse.I will edit it anyways.


why not just lastModified?


And more whitespace, see also rest of diff


contentLen -> contentLength


Thanks a lot for making the PR. If you omit `@address` the OS should try to listen to all addresses. Can you try that?

I'll look at the rest of the PR later.


wrong indent


Dominio correcto sin www


thanks @SahAssar, will check it out :smile:


hello guys=)


Bloqueado por DNS


else if


Versión fucional... Retomar


of make this an option (through configuration?).  Will it break existing behavior?


tabs restored.


Isn't a `<MessageBox/>` going to be composed of several `<Message/>`? That is, the `<MessageHeader/>` etc. would be sub-components of a `<Message/>` one.


invert if


Pretty happy with this - I'm leaving the colours in for now as it helps me navigate through the code :)

haven't done the only min-width mod yet. would also like to do a bootstrap version. we'll see.

time for lunch.


no err handling


add class comment why this class exists, what is does?


no sense, no err handling


No, you cant see it because version 1.35 of new horizons is VERY old

It should be New Horizons v1.6.5.1
And KSP v 1.0.4

if you look again at the link its right here -> http://imgur.com/BKWOlsy

By the looks of it netkan-bot set it like that -> http://imgur.com/n2XPVFK


Really well spotted! :100:


done


:sweat: Just a reference. I guess I'd use something else at last.. @pezy


Nice, so the erb is still clean and correctly indented.


DRY


#!/bin/bash
# Check if pipeline is disabled

ON=`cat /etc/sysconfig/bbc-istats-controller | head -n9 | tail -n 1 | cut -f 2 -d '"' `

if [ $ON -gt 0 ]
then
echo "PIPELINE IS ON"
else
mail -s "PIPELINE IS TURNED OFF" osman.marks.ext@bbc.co.uk tom.collins@bbc.co.uk craig.phillips@bbc.co.uk david.hodgkinson.ext@bbc.co.uk iStatsAVSupport@bbc.co.uk < /tmp/problems
fi


Suele pasar... A veces creo, a veces no comento el código, a veces tampoco lo actualizo todo.


mail -s "PIPELINE IS TURNED OFF" osman.marks.ext@bbc.co.uk tom.collins@bbc.co.uk craig.phillips@bbc.co.uk david.hodgkinson.ext@bbc.co.uk iStatsAVSupport@bbc.co.uk < /tmp/problems


Ideal para los tiles en Windows8


what is 'WriteStreamTmp' for?


Ideal para Shortcuts en SO movil


what is fp?


Using `IN_SET()` would make that code more readable.


what's nn?


¿Y si devuelvo la amplia gama de caracteres acentuados?


what's nn again?


Shouldn't we deprecate the keys instead of changing their content?


isn't you who named it 'IsNotExist'?


Any reason why you use Channel instead of ReadChannel? As it is impossible to push any data back into the Future (the past).


This causes `test47` to fail.


I know you're just augmenting existing sources, but I wonder why this isn't a switch-case statement rather than repetitive else-if blocks. Anyway, that could be done in a separate patch.


### For your matrix pleasure

Now you can take the `repl` __AND__ the `bloopl`.

Our repl now has inline markdown readme capacity:

```
happner
>> node
...
__node> $local.README    <enter>
```


not related to this patch but could be fixed in the same PR: this isn't a thread anymore.


same second check?


fi -> fileInfo


spaces


:bow:


No. We introduce new keys (and deprecate the old ones) when there's a new UI somewhere or when the keys are wrongly located (ie not following http://dev.xwiki.org/xwiki/bin/view/Community/DevelopmentPractices#HTranslationPropertyNaming).

In this case the meaning of the key has not changed. It's just a reword.


path is not used here


\ref{secao}. O número pode mudar.


\ref{} aqui também


Outro \ref{}


Why would you want to introduce new keys?


Ficou sobrando as aspas


why it is named 'driver_test'?


testing what?


@mlschroe This broke our test suite. Can you please fix it?:)


they may not accept without tests covered


yea bump to 1.0.4 build and pusg the tar.gz to mirros, Ill update wiki asap after this is available.


Omar, You should put the documentation that describes the function and it's parameters in the headers files. This is easier for the user of code.
Also this code can't really compile. The struct rMatrices1D and rMatrices2D should be completely defined in the header file.
So that would be (replace the <SOME_MATRIX_TYPE> for the real type offcourse):
struct rMatrices2D
{
public:
<SOME_MATRIX_TYPE> pressure;
<SOME_MATRIX_TYPE> velocity;
};

End the line "rMatrices2D result = {};" can simple be changed to "rMatrices2D result;" without the "= {}"
This also holds for rMatrices1D. Btw, if the same types are used for rMatrices1D and rMatrices2D, you can make then use the same type offcourse.


you can directly construct `remoteCommandHandler` with `commandHandler` instead of calling `reset()`


La fireClickEvent(element) lo que permitía era cancelar el envió de los archivos o imágenes al compartirlos;  se detecto que esta función hacia que no se pudieran enviar varios archivos seguidos. Se decide quitar la función.


Maybe CFFI_MIN_VERSION instead of CFFI_VERSION?

Otherwise LGTM


Добавил модель файлы заявки


Добавил в модель TenderClaim файлы заявки


Fixed in e4f3a591279789ccabe029dd00140b3cb0e3a6ed. Adding more metadata back later on. US only right now, I'm not sure if this used to work outside of the US before.


Vale. Luego cuando revise el post lo recorto un poco.


Просто строка


Hi, I'm very interested in this fork.
Can you describe your changes then create a pull request?

Regards!


Исправил баг при загрузке позиций из excel


Does anybody know in which version it will be available? In 3.8.2 is not.


The extradata is 256M (yes a quarter or a gigabyte) for this file.
We don't have enough SDRAM to allocate a second 250M buffer. And copying 250M is not a cheap operation.

Ideally it would be discarded earlier and the first 250M buffer wouldn't be allocated and we wouldn't waste the time reading it (it takes 30 seconds on my linux x86 PC to open this file). But I suspect that is inside ffmpeg and it doesn't know whether anyone will want it.


live() is deprecated at this moment.


Done by ead534b


iframe работает независимо от типа протокола


done


What does the "FS" in "FunctionFS" stand for? "File system"? If so, that sounds like an implementation detail of the Linux USB stack, and should not be exposed in the settings, name. Maybe `ListenUSBFunction=` (or even `ListenUSB=`) would be a better name?

Also, I don't like the abbreviations in `ListenUSBFFSEndpoint=`. First of all "FFS" is again is suboptimal, since I think the "fs" part is a Linux-specific implementation detail. I also don't like that we abbreviate things as "FFS" where the earlier option used the non-abbreviated name "FunctionFS". We should stick to one spelling, not a mixture of sometimes abbreivated and sometimes not. Also, we should generally be careful with abbreviations unless they are very common knowledge. The abbreviation "USB" is pretty commonly understood. But "FFS" is one I have not heard before. In fact, I think apperviating "Endpoint" to "EP" would be more common than abbreviating "FunctionFS" to "FFS"...

Long story short: I think these should be called `ListenUSBFunction=` and `ListenUSBFuntionEndpoint=` (or even `ListenUSB=` and `ListenUSBEndpoint=`.


Warum :O (nicht dass es relevant wäre)


학교 홈페이지는 플래시 플레이어를 사용하지만 서브 홈페이지는 플래시를 사용하지 않습니다.


Certo, @idmarjr! Lendo de novo vejo que essa frase ficou meio *nariz empinado*. Anotei ela pra alterar pra tua versão também.


Добавил кнопку добавления файла заявки при созданной заявке.


commit


It breaks behaviour a bit, in the sense that the proxy does not internally follow the redirects. I think this is preferable for a proxy and leave the following of redirects to the caller.


These two b locks are completely identical, why not make them one, and do `if (IN_SET(p->type, SOCKET_USB_FFS, SOCKET_USB_EP)) ...`


2015년 기준 정솔 : 3 학년, 충남중  송강현 : 1학년 -느리울 중


Hi Varun,

On Tue, Sep 08, 2015 at 05:01:04AM +0100, varunw-nvidia wrote:
> @achingupta<https://github.com/achingupta>
>
> Achin,
>
> Do you foresee a scenario where end_pwrlvl and max_off_lvl would be different? If No, then we can use end_pwrlvl here itself. If yes, then do you think it makes sense to pass the incorrect level to the Trusted OS?

`end_pwrlvl` is the highest power level that was requested by the user to be
placed in a low power state. This is what the `power_state` parameter in the
CPU_SUSPEND API carries. However, this request is local to the calling
CPU. Other CPUs might not be able to tolerate power down of the highest power
level requested by a different CPU. The PSCI implementation resolves such
conflicting requests aka state coordination. It takes into consideration the
requests from all the CPUs under a power domain (e.g. cluster power domain) and
determines the deepest power state each level from the CPU level to the
`end_pwrlvl` can enter. It is possible that an intermediate level might not
enter a low power state at all since there is atleast one CPU which wants to
keep it powered on. `max_off_lvl` is the highest power level that will actually
be turned off and is an output of the state coordination. It is more accurate
than the `end_pwrlvl`.

I don't think we should use the `end_pwrlvl` itself. It will work but is going
to be inefficient and redundant. I would like to understand why you think that
we are passing the incorrect level to the Trusted OS.

Do let me know if I am missing anything.

Thanks,
Achin

>
> �
> Reply to this email directly or view it on GitHub<https://github.com/achingupta/arm-trusted-firmware/commit/3bf351c857df0c5a181cd21fdbfc8071569f1e47#commitcomment-13103962>.

-- IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium.  Thank you.

ARM Limited, Registered office 110 Fulbourn Road, Cambridge CB1 9NJ, Registered in England & Wales, Company No:  2557590
ARM Holdings plc, Registered office 110 Fulbourn Road, Cambridge CB1 9NJ, Registered in England & Wales, Company No:  2548782


Can you explain why FFS EPs are excluded here?


added


진심으로 감사합니다.


there is `LIST_FOREACH_SAFE` which allows removing items while traversing.


Und warum der Spieleabend nicht mehr hinterm FSR und stattdessen in ner eigenen Datei? Ich fand das an sich schon ganz sinnvoll, den als Nr.1-FSR-Veranstaltung (die ja auch im Text erwähnt wird) direkt dahinter zu haben - sollte doch auch nichts dagegen sprechen. Würde ich einfach wieder zurückrollen - die Späße mit forcierten Leerzeichen und leeren Seiten wieder entsprechend umzustellen würde uns das auch ersparen (auch wenn das vermutlich jetzt nicht soo das Problem ist).


Hmm, why the special casing here for FFS EPs?


@idmarjr vou aproveitar essa parte para dar um feedback sobre teus outros comentários.

Em resumo: concordo com todos eles. Anotei aqui as partes que tu observou e irei alterar na versão pt-br.

Muito obrigado!


Done this. The key (or password) of the root user is still required for the `root.yml` playbook, though.


done


See CODING_STYLE: please place opening bracket on same line as function name.


Since of the rewording, the message will not be consistent across languages.

Also according to http://dev.xwiki.org/xwiki/bin/view/Community/L10N+Conventions , these keys do not follow the agreed format (wrong module, submodule, part, etc.) so I guess it would be a good time to deprecate and introduce the correct format (maybe even put them in their dedicated page).

Either:
xe.common....title=
or
platform.administration..... description=

Anyway, since we have lots of places that contain the word 'space' I think we should agree on the deprecation/ rewording strategy.


same as above.


Should be hidden.


Spurious newline.


fixes #13


Add xmlns:jdbc="http://www.springframework.org/schema/jdbc"


Change to: xsi:schemaLocation="http://www.springframework.org/schema/beans
						http://www.springframework.org/schema/beans/spring-beans-4.1.xsd
						http://www.springframework.org/schema/context
						http://www.springframework.org/schema/context/spring-context-4.1.xsd
						http://www.springframework.org/schema/tx
						http://www.springframework.org/schema/tx/spring-tx-4.1.xsd
						http://www.springframework.org/schema/jdbc
						http://www.springframework.org/schema/jdbc/spring-jdbc-4.1.xsd"


This is Awesome!! you wrote everything that is required for the Web Archive to execute and render the page.


Update class and driverClassName


missing OOM check.


Add context:component-scan base-package="beaconManagement.tcc"


Leandro, make sure you're using lower camel case on your file. Also, you have a typo in the date.


I really don't like allocating new SocketPorts for this I must say, since they show up in the bus interface as if the user had specified them. I'd instead much rather see SocketPort extended to contain an additional, optional array of fds for these automatically opened fds. Thus, the SocketPort struct should have a primary "fd" field, as before, but also an array of additional ones that belong to the same Listen line, but are the ones that are automatically discovered and opened.


Caty, you're mixing too many things here...

1) You're suggesting to deprecate keys when not all 26 language translations are not in sync... good luck in deprecating all the keys then!
2) The existing keys may not follow the agreed format. Please be my guest and make the change but not now, this is way too dangerous (you should wait at least 7.3 and possible 8.0). However this point has nothing to do with my commit for RC1.
3) We do not need to agree on the deprecation/rewording strategy. It's already agreed and documented! If you want to suggest some new definition/new rules (or even clarifications) please send a VOTE email with your proposal. But what I do know is that there's no need to deprecate a key when you're just changing its wording.

BTW when the next person releases 6.4.6, he/she shouldn't merge this change since it's only valid for 7.2+ (and this issue exists because we don't have branches for translations on l10n).


Ixi, vou dar uma olhada.

[2015-09-08 01:04] Alexandre:

> Estou travado nessa parte. Por algum motivo, mesmo passando o viewstate, o
> request é inválido. Alguma ideia, @andresmrm ?
>
> —
> Reply to this email directly or view it on GitHub.*
>


I ran in the same error. Any hints?


git push wapit


May want to trow an Extbase Exeption to have a more specific type on it: http://typo3.org/api/typo3cms/namespace_t_y_p_o3_1_1_c_m_s_1_1_extbase_1_1_property_1_1_exception.html


Funtions should either log in no cases or in all cases, so that callers can either log instead or not, but under no circumstances an error condition is logged about twice or never. This function currently logs in some cases, but in others it does not. The caller also logs, a second time. Please decide whether to log or not to log in the function.


Nightly sounds better than master and most people understand that


*Wiki and FAQ probably looks better without the dots.


Hmm, why are these properties of services, not of sockets?


Don't like the naming of this. Should not include "FFS", but "USBFunction" or so, see discussion on other string.


Make sure your title is conforming to the guidelines in the wiki, and reformat your content using Markdown.


ベースURL関連


maybe somebody would feel confused :)
sorry for any discomfort and thanks for a great privacy fix app!


Make sure to update your reading log with last night and tonight's homework. Also, your title needs to conform to the guidelines in the wiki.


Could you extend on this, I have trouble understanding what precisely these files should contain. Also, isn't this stuff specific to USB, and not just FunctionFS? Should this even mention "FunctionFS" at all?


"Forums", with no '


Marketing fluff?


День добрый!
Переходить к пункту 6?

2015-09-08 12:43 GMT+03:00 Slava Semushin <notifications@github.com>:

> Удаляй смелее :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/AleksSPb/Spring-demo-App1/commit/91f8bd84000a3c2457bdf6316654d14d75f52c81#commitcomment-13107945>
> .
>

--
Всего наилучшего, Пархоменко Александр


Hmm, I have trouble grokking this. Can you explain this in more detail, please? What is "options of this type will change during runtime" supposed to mean?


OS X, not Mac


Ping. Please respond.


Map.toList is a much more widely-known name for Map.assocs


Spacing?


A space is missing between ) and {


Childish? Because I asked a question ?


What's the point of this module? It re-exports a small subset of Data.Map api with the exact same types and invariants.


@OrKoN @vanthome this works but I am not sure where to remove the observer, and now that I am thinking about it is it even worth supporting this use case? I mean we could simply state in the readme that you have to replace the whole object for it to work, would allow us to drop a whole bunch of observers :P


Тег, тег не забудьте запушить :)


saw approval from andytill
at https://github.com/basho/riak_ql/commit/3f48a5c1e2ab4a806da5c4111cf9c1e6d0a8da0f


merging basho/riak_ql/gg/timeseries = 3f48a5c1 into borshop-integration-19-gg/timeseries


basho/riak_ql/gg/timeseries = 3f48a5c1 merged ok, testing candidate = 6511cbd4


Put the close paren in its own line


renameFile does copyFile?!


Please include "Upstream-status: Submitted (https://review.tizen.org/gerrit/47754)"
here. Otherwise that information will not make it into our repo.


Maybe use `intercalate "\n\t"` instead of "\n\t " prefix (which has a weird space there).


Redundant parens


Avatar `https://avatars0.githubusercontent.com/u/7913438?v=3&s=200`
![7913438](https://cloud.githubusercontent.com/assets/4997583/9733859/03c1deae-562f-11e5-8eac-ec080b9f6966.png)


newline after "where" (line too long, and it's customary)


@mwfrojdman beware ! this code is a nice example of how to create a recursive lamdba. When the lambda will resolve the symbol ```protocol_factory``` it will call itself.
Fixed in https://github.com/Polyconseil/aioamqp/commit/645c6bb2f83d52c4c1af34f519752f2eeb916169


Сейчас сделать тег и запушить? До внесения изменений?
Или после изменений повесить тег и запушить?

2015-09-08 14:37 GMT+03:00 Slava Semushin <notifications@github.com>:

> Тег, тег не забудьте запушить :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/AleksSPb/Spring-demo-App1/commit/91f8bd84000a3c2457bdf6316654d14d75f52c81#commitcomment-13109922>
> .
>

--
Всего наилучшего, Пархоменко Александр


Review this


I think the method should be named 'referenceCount' without 's' to follow the Cocoa naming style. e.g. -channelCount in AVFoundation


Well I must say that using `debounce` is not what I expected. I have to agree with @vvo here, it is better to have a solution that is absolute and not relying on side effects or stateful mechanism. Here we have a behavior that is changed if a call was done before...

Still the hack is interesting :P


It seems unused anyway, remove it?


The spacing here is weird, does haskell-mode work with this format?


@jaundice:
please update your PowerCollections subrepository prior to creating a new pull request :)
see bb897cc5d6871b930e726b6a9f08fdee2f80ae0d


why?


Thank you! This commit fixed the build. It was broken since http://ci.cgeo.org/job/c-geo/4021/


This does not belong in this commit


We not selling and none is buying, this is free and its true.


singular


May be we should use xxx.count rather than [xxx count]…

I switched to the dot syntax in my recent changes for all collection accessors such as -count, -allObjects etc. since they are now declared as properties in Cocoa API since 10.9 or 10.8. I don't remember exactly.

What do you think?


The check if for `handleReadyNotification` above, so that it looks for other `IN_PROCESSING` notifications for that account and type (others but itself).


Suggestions for review?