Not a lawyer either, but I believe (perhaps falsely) that:

1. The language here doesn't "hand out copyrights to *everybody*".  In fact, because there was no copyright assignment as part of the contribution process, all of the contributors do in fact hold some copyright already, no matter what language you decide to use here.

2. Because copyright is held by all contributors, you need consent from all contributors to change the license.  For whatever it's worth, I give my consent to change the license to the MPLv2 (and I think it's a great license for a project like this), but traditionally I've seen this handled by making an issue ticket that pings all of the contributors to get their consent.  This can take some time, but it's worth covering all the bases here, and I think it would be appropriate.


Seems odd to specify one obj file in inputs and another by piping it, why the difference?


#2
MVC패턴 적용
- 사용자의 요청과 응답을 처리하는 Servlet (C), 데이터(게시물의 값)를 담고 있는 Model(M), 결과를 노출시킬 JSP(V)로 구분
- 테스트 편의를 위해 로컬 DB사용


@flarum/core This change was demanded by a WordPress integration I'm working on, where logging into WP will request a token via the Flarum API and then set it to the `flarum_remember` cookie. Do you think it is a security risk though, to allow tokens to be created with arbitrary lifetimes?


## ここに画像を置く

ウサギとカエルの画像


To me level makes sense here. When you log something you specify the level (think of it like a severity level: notice, warning, error) and when you run a server, you specify the verbosity (what level of verbose-ness you want to see in the logs). Anyway, I'm fine with either term so it won't block me from accepting this PR.


I would just put the var here and save a line:

    srv = new Server(customOptions.host, customOptions);


.. ditto ..


Nit [style]: I prefer either keep it on the same line (I don't really mind if this exceeds 80 char) or if you want to brake it then put the closing paren on it's own line:

    server.suppressExpecteErrMsgs.push(
      'getRoot signaled error [Error: intentional failure]'
    );

Again, not a blocking issue, but I thought I'd mention it.


임시 암호화 & 토큰 생성


Also, not a lawyer, but my understanding is 1. contributors hold their copyright to their specific work, 2. notwithstanding any individual's copyright, the combined work is also copyrightable, and if anything, that's what the notice refers to.

Any particular contributor who disagrees with MPL-2.0 can request their specific, verifiable contributions be removed and we'll rewrite those parts.


We can remove any jslint directives.


Independent versions is really needed. FWIW lodash is a monorepo and I individually version all [lodash packages](https://www.npmjs.com/browse/keyword/lodash-modularized).


Does it really perform defaulting? I haven't traced the code closely, but I thought defaulting is only performed in conversion.


Error, meant to fix #2


root はpopularじゃない？


hi,

that issue is due to cinnamon early devellopment versions.

in boxpointer.js (usr share cinnamon js ui there is twice x + 10).
why isn't that value defined in themes or in dconf.settings.?

well perhaps a cinnamon dev will fix this issue soon.

 x + 0  there and -arrow-border-radius greater than 16 in theme file works.

you are right it could also be done in cinnamon configurable applet.
However just changing it for one isn't a solution either.

regards bluedxca93


In Sadie I need to use a different build directory for each platform I support so working in that directory seeems reasonable. Otherwise i think I'd have to specify the directory each time.


Thank you, @kgoedecke, for accepting the patch and for the quick action!


感觉良好


Still not a lawyer, but okay by me. :+1:


Sorry, I didn't realized that this is just testing code. I guess that in this case you can do whatever you want.


I suggest switching this to an options hash, as several positional values with defaults is tricky to use correctly. Also not sold on the name, `keepUnprefixed` would be preferable IMO.


Typo, should be "does not", UniversalDecoder provides defaulting.


Looks like there is a missspelled "canola"


So we'd have to build the project three times. @Cheesebaron, @martijn00, do you know more on how to do this?


@ljharb true. My mistake, sorry.


Since we only support 2.0+ I think this would be better as a lambda fat-arrow: `-> { "yml" }`.  Does you disagree? What about @jekyll/core?


Any particular reason for this?  I generally prefer to operate on existing values before calling methods, that way I let short-circuit evaluation save me some work on the easy cases.  I realize this isn't really a performance critical method so I'm fine either way, though.


Error in line no. 748.


Could you add a short godoc?


what the fuck?


See https://github.com/spalger/rcloader/pull/11.


tukarlah nama $carilah kepada lain


I thought I disabled that rule.


I removed the back button on both the playlist and webview so the user will have to swipe left to right or press the playlist icon twice


This breaks the script on linux machines. As the test also succeeds if the file does not exist. Try:
> if [ -x `which DOESNOTEXIST` ]; then echo I have DOESNOTEXIST; fi

if [ -x "$(which name)" ]; then should work more reliably.


Canols? Did you mean Canola?


Wow, thanks for this analysis and fixes! I'll get these tweaks pushed to my repo today or tomorrow if I can find the time.

Interesting about clockpro. My code is a translation of a python implementation, rather than from the paper directly.  Is either one consistently better or are they just subtly different?


> I have no intention using my Puppytime arguing. - I find codelines to be the fun part.

I'm sorry if you see the "optional vs mandatory" change in a block of code as arguing. Particularly in a collective project that is trying to minimise "hard"-required apps (rootfs-skeleton), as this code does with hard dependencies.
I believe we had a similar argument back when rootfs-packages was first generated but so far works just fine. Why not keep with rootfs-packages for new code?


**Style/BracesAroundHashParameters**

I think it is kind of a stupid rule as well. I only changed this when the _only_ parameter being passed was a hash; what Rubocop really wants is for the braces to be left off anytime the _last_ parameter is a hash, but this looks confusing to my eye.

I would rather hash parameters _always_ have braces, which can be set with `EnforcedStyle: braces`


Do you prefer `->` over `lambda`?

```ruby
default = %w(yml yaml).find(lambda { 'yml' }) do |ext|
```


Yup, if it's a single line, go ahead remove the braces IMO, but if it's going across multiple lines add the braces!


mmm so in that case, we should leave the % as you did, and if the len is undefined, use value like
```
value = current + ' bytes';
```
or something like this, so it's always a string, whether we know it's length or not


As an exercise to use lambda, it looked as if isFactor could be replaced with

if lambda elem, j: elem%j==0
...but somehow that is resulting in the wrong solutions.

(Of course, the whole section also could've been a simple if (elem%j==0)...to determine if J is a factor of elem, even without lambda.  Though arguably that makes code less readable.)


现在已经有很多bug是靠+0.x的方式解决的了我觉得这不是个例，应该是PM的entity中的某个包含offset的环节出了问题


one semicolon too many


What do you think of this implementation?
```javascript
len = parseInt(res.headers['content-length'], 10) || 0;
```
So now if the header is undefined, len will turn 0. When you later do ```if (len){ ``` you are also checking if len is not 0, NaN, or undefined, so you could reduce this line a bit.


Thanks for the fix, I can confirm this works on my end. It would be good if I could set the token in the .env file, or another method which avoids hard-coding the token in. Once I know how this can happen, I can accept this request and make necessary changes.


as far as I can see, this isn't used anywhere.


I also based [my version](https://github.com/ben-manes/caffeine/blob/master/simulator/src/main/java/com/github/benmanes/caffeine/cache/simulator/policy/irr/ClockProPolicy.java) on the Python implementation. I made some fixes, but I don't recall what they were other than using linked lists instead of position indexes. I did notice that ClockPro can be very slow for large caches on large traces, so either mine has an issue or it has a poor worst case (as expected in a CLOCK algorithm). I spent a lot more time on LIRS since that was what I had expected to use before realizing the windowing trick fixed TinyLfu. LIRS required debugging with the author's C version to determine which steps differed, find if the paper discussed that (sometimes it didn't), and correct mine.

| Trace | go-clockpro | clockpro |
|---|---|---|
| multi1 | 49% | 52.69% |
| gli | 11% | 16.61% |
| ps | 52% | 53.99% |
| cs | 1% | 15.69% |
| loop | 38% | 10.66% |

Interestingly how our `cs` and `loop` runs behave so differently. We'd probably need to ask for the author's original version to debug properly.


On my linux box and my mac notebook the time+year is in column 6 and 7 and this change breaks it (I do use homebrews corutils package though, not sure if the macports one is different here).


是的 :D 以前修传送掉出世界也是把y加个0.1  刚刚好就是会往下掉 边界问题挺烦躁的 XD


Ah, it still needs to be in the form:

            isfactor = lambda elem, j : elem%j==0
            #if isFactor(elem, j):
            if isfactor(elem, j):


Prevents the 'loaded' class from being appended multiple times to lazy-loaded images.

Also added a space, in order to append 'loaded' as an additional class instead of appending it to an already existing class name. This is optional and based on the fact that the class is being appended via '+=' in the original code instead of setting it via '='. It will work without the space as long as there won't be any other classes added to a lazy-loaded image, but adding the space will make it work with any additional classes being added to the image in a potential future update.


Please take a look


The intent of this change is to fix issue #2357.
The problem in that issue is the interaction between the way bindings are evaluated and how functions get their arguments.

Take the example binding cited in that issue:

    bind t forward-jump and backward-char

Let's say that we're in normal-mode and I type the characters: `tx`.

The `t` is read from the input stream, it's mapped to the binding above and the binding is evaluated by `input_mapping_execute()`.  A function is detected, `has_functions` is set and `m.commands` is iterated through in reverse (I've added some comments to the code from `input.cpp`):

```cpp
// for each function in the binding, in reverse
for (wcstring_list_t::const_reverse_iterator it = m.commands.rbegin(), end = m.commands.rend(); it != end; ++it)
{
    // lookup the function's code (aka it's R_* constant from input.h between R_MIN and R_MAX)
    wchar_t code = input_function_get_code(*it);

    // if it has any args, read them from the input stream and set them aside in a separate stack
    input_function_push_args(code);

    // push the R_* code onto the head of the input stream
    input_common_next_ch(code);
}
```

At the start of this loop, processing the `t` binding, our state is:
m.commands = { "forward-jump", "and", "backward-char" }
Input stream = { 'x', ... }

The first iteration through the loop maps "backward-char" to R_BACKWARD_CHAR.  It is a 0-arity function so no arguments are processed and R_BACKWARD_CHAR is pushed onto the head of the input stream leaving us with:
Input stream = { R_BACKWARD_CHAR, 'x', ... }

Similar story with the second iteration, it ends with:
Input stream = { R_AND, R_BACKWARD_CHAR, 'x', ... }

In the final iteration, "forward-jump" is mapped to R_FORWARD_JUMP which has an arity of 1 so `input_function_push_args()` attempts to read the argument from the input stream.

**Without this patch:** the next character from the input stream is read as its argument.  Erroneously, this is R_AND instead of the intended 'x'.

**With this patch:** the input stream is iterated through until a real, non-function code, input character is found.  Along the way, any function codes are pushed onto the front of the `skipped` deque:
Iterating through the "skip loop" (starting with the precondition) the state is:

| skipped | input stream | arg | result |
| ---------- | ----------------- | ---- | ---------------------- |
| {} | { R_AND, R_BACKWARD_CHAR, 'x', ... } | undef | loop starts |
| { R_AND } | { R_BACKWARD_CHAR, 'x', ... } | R_AND | `arg` is between R_MIN and R_MAX, loop continues |
| { R_BACKWARD_CHAR, R_AND } | { 'x', ... } | R_BACKWARD_CHAR | loop continues |
| { R_BACKWARD_CHAR, R_AND } |  { ... } |  'x'  | `arg` is less than R_MIN, loop breaks |

'x' is saved as the argument then then `std::for_each` pushes first R_BACKWARD_CHAR then R_AND onto the front of the input stream leaving it looking like this:
Input stream = { R_AND, R_BACKWARD_CHAR, ... }

This leaves the functions in the correct order, ready to be read off the input stream and processed and has stored each of their arguments correctly in the argument stack off to the side.


这种实现方式参考 MUI 的官方文档网站的源码，不过其最新的代码已经改版


Please use clear commit messages, "Update penbox.py" is not a clear one.. You should use a message that summarize the modifs made in that commit.


Please update the comment as well `// 3 seconds`


We can remove the whole `.config` block., if the patch resolves the msie caching issue.


In case `ThrowOnOversizedBrokeredMessages` is used, then yes. When a custom strategy is registered and it won't throw, then it will be hard to know what happened.


Related to a question in another commit (https://github.com/smarterclayton/kubernetes/commit/e66fa27ca492173fb08bf13fb1645334aa5f3c7d#commitcomment-15214735), why unlike the test in the other commit, the TypeMeta of `into` in this test is not set?


hope its missing equals...


Add "Expect error: " at the start of the error message?


What problems exactly?
What debug form? There is no debug form in Delphi by default on exception, you can either create your own one http://stackoverflow.com/questions/15890029/delphi-obtain-stack-trace-after-exception or use 3rd party libs like https://www.eurekalog.com/ or http://madshi.net/madExceptDescription.htm


Why NotFound? If web service is able to send a response, I'm expecting it to be `{success: true}`


Latest version is fine. Sorry I merged and then reverted a previous one.
Anyway glad to hear that you are working on something big.

If you have something big, then please at least open issue and put
reference to your git repo so we can review the work. Also if you
do not feel confident in some changes we can help finish that task. Do not
hesitate to contact me directly at mmicko@gmail.com

On Sun, Jan 3, 2016 at 4:22 AM, ajrhacker <notifications@github.com> wrote:

> There are no comments on GitHub for #512
> <https://github.com/mamedev/mame/pull/512> or #529
> <https://github.com/mamedev/mame/pull/529>; if there were any, they must
> also have been made somewhere else. Anyway, I've submitted a new version
> which eliminates not only the hack but also the global variable it relied
> on.
>
> By the way, it's only barely evident, but I added verbose validation to
> help test a major revision of devfind. I'm certainly not going to be
> submitting that the way I originally did it, which broke hundreds of source
> files at once; though I eventually got that version to compile and I still
> consider it a righteous featurectomy, I don't feel confident that every one
> of my hand fixes was appropriate.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mamedev/mame/commit/f8eb29b9bd92a585995258504c8f56b6679c9f87#commitcomment-15232068>
> .
>


I am not sure the last three sections should be in the README. PyPI is just a well-known python package repository (and is not even our primary source of distribution since debian package is there); and the binding specification and the ev3dev-lang project should not concern the *users* of the project.


use my ms013g boot.img .. i have updated that too for the f2fs :+1:


When there are no arguments on proc, and lambda I do, because `->` is the quick way to do `lambda { "yml" }` (`->` doesn't take arguments) but if I'm honest that code is drastically confusing anyways, I would have opted for `Dir.glob(Pathname.new(".").expand_path.join("_config.{yaml,yml}")).first` and I would explicitly choose that because Find will require me to join again and ensure it's depth is one which is annoying.


duplication of line 339?


Code monkeys gonna generate getters and setters


1. The original design intent was to have the `window` parameter be fundamental to code verification.
2. I prefer consistency for the `window` parameter, but the RFCs recommend different behaviors for HOTP and TOTP. At least within the two sets of verification functions, the default parameter values should be the same.
3. Correctness in `passcode` was hardcoded to `-window <= verify.delta || verify.delta <= window`. This is the recommended behavior described in [RFC 6238][].

I looked over what [RFC 4226][] (HOTP) and [RFC 6238][] (TOTP) recommend and this brings up something I put off addressing in writing `passcode`:

- For HOTP, RFC 4226 recommends using a look-ahead window during verification. `0 <= verify.delta <= window`
- For TOTP, RFC 6238 recommends a time-step of 30 seconds with both look-ahead and look-behind windows. For a 30 second time-step, a window of ±2 will accept codes generated within a 2-minute time frame.  `-window <= verify.delta <= window`
- Security implications are described in more detail in the RFCs.

1. Should this be something that should be addressed now?
2. What would be sensible defaults for `window`?

[RFC 4226]: https://tools.ietf.org/html/rfc4226
[RFC 6238]: http://tools.ietf.org/html/rfc6238


We can do that. I was in fact contemplating using `--force` but then had second thoughts about the use case.  Solver uses `--modify-stack-yaml` for force overwriting so we may want to normalize that.

I missed the resolver help, will correct that.


Thanks for the update!


Yes. No problem and already merged.


```sh
wall_state() {
+ if [ `iptables -L -n |wc -l` -gt 10 ];then
+ return 1
+ else
+ return 0
+ fi
+}
```
First look, should do the same :
```sh wall_state() {  [ `iptables -L -n |wc -l` -le 9 ]; } ```


Ah cool, danke!


next update this can go back. canJoin only returns a straight true or false


Spacing


If you're really gonna change it, change it in the configuration as well


it's a revbump so pls mask ~amd64 and ~ppc


I hate to say it, but this isnt the way other dialogs do this...


hmm j'avais oublié cela. Je l'utilise ici
https://github.com/karlcow/ymir/blob/899f7edfcae590d576eaac6ccd5cc573417b3aef/ymir/ymir.py#L400


Don't you think this will break peoples browsers who visit http://www.aliexpress.com/ directly?


Took a bit from each pull request. Made me laugh, not sure why...


outside src_prepare I'd gather all patches together in a PATCHES array and then call the epatch function whilst expending the array. Example here: https://gitweb.gentoo.org/repo/gentoo.git/tree/dev-java/colt/colt-1.2.0-r3.ebuild#n30


This isn't always going to work because language word order. You need to do something like `"Download_File": "Download {{fileName}}"` with dynamic localizations.


Bah, well, that will take some work.


line:11-14
이미지 파일들에서 보행자 인식 작업을 할 때 이미지파일들을 제공하는 부분일 뿐,
실시간 보행자 인식을 하려고하는 나에게는 필요 없는 코드


这行真是 excited，`weibo/weibo/weibo`，隐藏得很深嘛


Thank you for this Mike!


Added Autostart


Why is hash set to non-default value of 16?


nothing


aaaa


test


Yeah. :P


Relevant issue: https://github.com/commercialhaskell/stack/issues/1597

I don't really like using `--force` to mean this, and `--modify-stack-yaml` seems long.  How about deprecating both of them and just having a flag like `--modify` or `--modify-files`?  Should probably just create a new issue for this.


Great! thanks.


이거 결국에는 부질 없었음


The window manager generally deals with it. It varies a bit between window managers, but they generally try to do something sensible.


Our convention is to avoid wild card imports. :) https://wiki.openmrs.org/display/docs/Coding+Conventions


I assume you meant to ping #834 :). Why are you multiplying both values with 31? There are a lot less possible hashes now


I changed it in config during the meeting today.


This seems to be mixed up with lots of formatting changes. Can you separate the two? :)


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 170](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=602&buildTypeId=XbimEssentials_XbimEssentials) is now running


masked ~amd64, ~ppc, and ~x86.


this is exactly as it is for now
'''javascript if(len){
        value = (100.0 * (current / len)).toFixed(2).toString() + "%";
      }
      else{
        value = current;
      }
'''
i'll add bytes to the end


done.


Is there something wrong with the content in <property name="label">?


我在看您的那篇文章的参考文献时，发现这样一个项目LTP：https://github.com/linux-test-project/ltp
之前我查资料的时候看到过，这里面有的一些脚本（c形式）与lkp包含的测试程序是共同的，不知您是否了解？是不是lkp是某种意义上LTP的一个壳？


`addToNode` must be a member of `TreeLeafNode` I think. Why move it to `BTree`?


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 170](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=602&buildTypeId=XbimEssentials_XbimEssentials) outcome was **SUCCESS**
Summary: Running Build time: 00:02:43


I tried and it did not run.
But since version 3.7, it does not support VN.
I can not login because the invalid account token


Brilliant, thanks for the report!


#79 hoàn tất


why cherrypicked?
spell 70725 doesnt exist in 6.x and 70726 fully changed


In china, this server can't use
change a server?


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


Now everyone sees `Visible as "Bastien's computer"` in their settings panel..


Suggestion: Separate the adding key to `TreeLeafNode` and adding data at offset  `addValue`


> FWIW lodash is a monorepo and I individually version all lodash packages.

This is not nearly as simple of a problem for Babel. Lodash works with different major versions of different lodash packages, not to mention that we aren't always in control over our own changes since we have a committee to listen to.


not exactly, pythonocc-core 0.16.x requires any oce-0.16 version.

For instance, pythonocc-core 0.16.2 works either with oce-0.16 or oce-0.16.1.


Thank you :+1:


If it no longer works, we need to update the [conditions section for the mission-making guide] (https://github.com/endless-sky/endless-sky/wiki/CreatingMissions#conditions).


Yep! See #1959


I agree, both the existing options are a bit ugly. `--modify` sounds very general and 'files' in `--modify-files` is also pretty general, how about `--update-config`?

Also, we may want to take a step back and rethink the command organization. For example `stack config` becoming the master command for all config related operations? init and solver could be subcommands of that?


error CS0136


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 171](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=603&buildTypeId=XbimEssentials_XbimEssentials) is now running


Hey,

I've actually fixed some of these already. I'll see what I can change for my next commit and I'll give you credit on README or something.

Thanks.


Neat, I'll add this in with the protocol update and put you in the README. Can't really merge the PR since I've already changed most of this code a lot. :P


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 171](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=603&buildTypeId=XbimEssentials_XbimEssentials) outcome was **SUCCESS**
Summary: Running Build time: 00:03:12


We have to use the `buffered_file_data` and `buffered_file_instance` to deal with the blocks right ?
That is why I thought that moving it to the `BTree` class would make sense.
Anyways, it is a private method.


I was trying to have the videos in the web view play inline or at least not automatically play, but the above code didn't work


dup dup.

And the path is fixed location, not as the README said:

> Save the API key to a file named `OWM_API_KEY` in the rainman git directory and everything should work.


BTW this is not the right commit, I have rebased some more over this.
So see https://github.com/ExternalMemoryDS/external-mem-ds/commit/a01493dfc91eb8be33ca72f3bdda04699660b1d9 instead.


Same here. I'll revert the column numbers.


Changes seem to be merged now.


```text
Warning: unrecognized parameter Style/SpaceAroundOperators:AllowForAlignment found in jekyll/.rubocop.yml
```


Neat, thanks!


Yeah I noticed that too, I plan on filing a bug when I have time as that option was copied directly from their default configuration file, so it's gotta be a bug either in their documentation or in changes they made lately.


哈哈哈哈哈哈


帅气的侃侃


Just to note, that the return statement would immediately exit the current function. So there is no need for a if..elseif. Also don't forget braces, so the L+R combination wouldn't just be an R. BTW if you think the function is too long, better split it to more functions, to keep your brain clear.


+시간표 편집 다이얼로그 환경 개선


Hi @citibob , @boegel, my guess is that normally this goes under easybuild-framework repo, contrib area!


I tried to make the reference independent to PID.  With association to the internal collection id, the DSC can have different (or multiple) choices of PID's.


求豪哥心里阴影面积


yes


Just to be sure: you see all this on event mpm, I assume? Cause that is where the keepalive experimenting is currently happening... Do you have a KeepAliveTimeouf set and what value does it have?

> Am 03.01.2016 um 03:17 schrieb Leho Kraav <notifications@github.com>:
>
> 1.0.16 at least by default configuration still has the same effect. Firefox network panel shows a red square for these connections and Request tab has "Connection: Keep-Alive" field listed, so it's likely it's related.
>
> It messes up some AJAX controls, because the browser makes the app think it's an active but endless connection going on, and UI gets stuck until you reload the whole page.
>
> —
> Reply to this email directly or view it on GitHub.
>


Having to undo changes. Doesn't appear to accept messages anymore.


Weird, I don't have the GUI client but I tested with netcat and it accepts messages and shows them in the log. Does your client send newlines after the text?


Using mbedtls, you can just `sha_file` to get the correct hash, or still use `sha` to get the hash of a memory block.


This file mixes tab and space indentetion, please change to space indentation.


countNodesSharingContent


I would not decrement the size


Be aware of what you passed. buf is a pointer, and char is a primitive type. Are you trying to store the pointer (which is then freed)? also I suggest you to include `stdint.h` and use `uint8_t` or `u8` instead of `unsigned char`.


I would not remove the delegate because if you iterate over the collection it is better to have the node inside.


sharing their content


thanks!


Sorry for adding the note here. I don't know what if you just use `AM_DeleteTitle` instead of checking if that exists?


Copy the formatting for "select" here. This should be unindented, so it is not a code block. And highlight "mode":
```
where ``mode`` is one of


Make this a separate block (with a single indentation level)


I am sorry for nitpicking, but they currently don't (and if I recall correctly haven't yet, ever). And since those are subclasses, adding them later won't break any code... But it is your call, have a nice day :)


Fixed these comments in follow-up PR.


@mcostalba : any suggestions?


@bluedxca93
I understand the problem more clearly now; thanks.


TeamCity LabKey :: Trunk :: Community :: CDS postgres 9.4 [Build 55](https://teamcity.labkey.org/viewLog.html?buildId=294652&buildTypeId=LabkeyTrunk_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 14, passed: 34 Build time: 01:04:41
### Failed tests
```
org.labkey.test.tests.CDSHelpCenterTest.verifyHelpCenter: <no details avaliable>

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

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

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

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

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

org.labkey.test.tests.CDSSecurityTest.verifyUserPermissions: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyDensePlotBrushing: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyTimeAxisWithMultipleSchedules: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifySubjectCountsWithFiltersAdvancedOptions: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyGutterPlotBrushing: <no details avaliable>

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


if you press ctrl+i and delete what is added to the input line and press enter you still get the irc icon pushed out from nowhere...

I think support for text icons should be dropped, this clearly doesnt work right.


I don't get that much. Instead of printing the same text again and again, you may consider to print it just out of the while except the `Press (A) to continue`. And do your HID response in the same while.


To fully populate the cache this should be,
```go
if slru.Len() < slru.onecap + slru.twocap {
```

and similar in add
```go
if slru.one.Len() < slru.onecap || (slru.Len() < slru.onecap + slru.twocap) {
```

Then if there is free space in `slru.two` you can use it, which helps to increase the hit rate.


![2016-01-03 17_00_00-](https://cloud.githubusercontent.com/assets/3712729/12078033/fcd750f6-b23b-11e5-8f60-87a7c306d759.png)

search box appear on top of the one2many tree view. Is it what we want to have?

![2016-01-03 17_00_26-](https://cloud.githubusercontent.com/assets/3712729/12078039/1b038126-b23c-11e5-9f2c-0897ccb2b9cb.png)

Error happened when you want to filter to group...

After reverting this change, it seems to be fine...


Це все не треба почитай про Play Ebean integration, має виглядати так:

```
Client c = Ebean.find(Client.class)
                .where().eq("ip", ip)
                .findUnique();
  ```
 в сенсі - всі ждбс-кишки треба викинуть, це как би розрив абстракції


#include "stdio.h"
//二分查找
int binary_search(int* a, int len, int goal);

int main()
{
    const int LEN  = 100;
    int a[LEN];
    for(int i = 0; i < LEN; i++)
        a[i] = i - 50;
    int goal = 30;
    int index = binary_search(a, LEN, goal);

    if(index != -1)
        printf("%d \n",index);
    else
        printf("不存在");

    return 0;
}

int binary_search(int* a, int len, int goal)
{
    int low = 0;
    int high = len - 1;
    while(low <= high)
    {
        int middle = (low + high)/2;
        if(a[middle] == goal)
            return middle;
        //在左半边
        else if(a[middle] > goal)
            high = middle - 1;
        //在右半边
        else
            low = middle + 1;
    }
    //没找到
    return -1;
}


With your update the minibatch iterator generate the np arrays on-demand so there is no need to transform the a np.array here, remove this transformation step.


Fortunately the users don't have to wait actual 0xFF times (outer). canContinue?False. shouldNotChange?True. So it would enter after entering the first while. And this while can make it accidentially fall to 0. Also don't know if there is any stable way to get a timer value.


якщо зробити все через Ebean-и, цього теж не буде


це ваще жесть, якісь глобальні змінні, це треба мгновєнно прибрати


What if this doesn't exist? You don't take the return value currently. Hope you might use that.


дічь, ці штуки треба прибрати :)


typo in commit, should be "Found debug colors..."


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


Please remove banner comments (`#######`)


You really misspelled it, isn't it?


**Problems:**

Some kind of weird issue where the application would throw an error saying a package was missing when a non-developer tried to run it on their computer.  Probably means I just need to exclude it from some kind of packages list, I just wasn't able to find it in the lists made available to my by the IDE.

**Debug Form:**

I mean the custom form you guys have whenever there's an unhandled exception that has a stack trace and all that other pretty stuff.


Eh.. Did you try to mean `currentPack=(currentPack+1)%totalPack` or something alike? Similar for the `KEY_RIGHT`. Sorry if I've get it wrong.


* Move this to the same place as where positive samples are added.


This should be `.swapped` instead of `.list`


архітектура така:

```
get /client => getClient() {
  ip = request.getRemoteAddress();
  if (ip is of localhost) {
    //ask client for ip:
    // send json indicating that client should calculate ip itself and send us
  } else {
    ask ipinfo.io over http
    parse response
    save Client
    send Json of Client to client
  }
}
```

client:
if got ok response to GET /client, parse output and fill in the page
if got response indicating that client should calculate its ip itself, ask ipinfo.io and send GET /client/NEW_IP

server: GET /client/SOME_IP =>
the same as before, just get ip not from request.remoteAddress(), but from path

DONE


在GItHub上测试添加的一行~


Nope


Putting a var statement below an operation in the same scope obscures the fact that the variable's initialization happens at top of scope. If lines carry a cost to save, I'd rather move the `new` up to the `var srv`, have the options merge done by a helper function from lodash, and bring some very short "then" branches to the same line as their `if`s.


Wait for the next update in a bit, and see if you can implement it into that. :)


Eh.. I suggest you to test the known types instead of this. Dunno how would really do such a region name for the package (maybe? orz).


Might be a good idea to start a `contributing.md` abd there explain when to save lines.


The functions on the flex and fixed layout would need some documentation (or at least it need to be tested if they show up in the generated API docs). The documentation should also state that only the "first/oldest" instance of the widget is removed, in case it was added multiple times.
I'm not sure if having this function in `base` is really all that useful. Perhaps make a function which can only remove a single widget (that should be the most common case) and have it inline in the layouts directly?


Same API documentation thing as in the previous one. Besides that, fine by me since I can see where this is going to...


Solved #98, not #99.
Commit comment amended in ba16f2a257631acb31515f9bbad6cebbae0a1901.


checked against MQTT Version 3.1.1 OASIS Standard
patch is fine


WTF? Return 1? You'd better split some out to new functions. Or else, a return would corrupt all this function. If you need to quit the while, I suggest you to use `break`.


Forgot to actually fetch the data?


这不已经2016了么？


Yep Docker is only used to help building for different platform. And now, it works (at least it compiles) for each one.
And it's almost automatic. Did you succeed in building libtorrent-go?


Вот это коммит !!! :)


Seems a bit unnecessary(?)


Not a cricism, I just suspected that you might not know that the SDK uses it so there's no point in reimplementing it. It is just the old ASCII version I think [A-Za-z0-9], but worth checking.

My point about coding efficiency is that when we are coding for code that is going to be flashed into every firmware build, then it's worth taking a little time to code for space effiency.

I discussed the tasking issue in #871.


A few remarks:

- Please don't roll unrelated changes into pull requests that aren't cosmetic (you changed the inclusion behavior of GStreamer. Are you *sure* gstreamer support is available on *all* other platforms, by the way...? probably better to include only those where we know it's available)
- You're no longer carrying down MOZ_PULSEAUDIO_CFLAGS but you are configuring them
- You've removed the use and inclusion of MOZ_PULSEAUDIO_LIBS and its linking (LDOPTS) in makefiles. That seems to be wrong.


强行false？


Yeah! I got it working thanks to your work!
Here is my current [Makefile](https://gist.github.com/Quentin-M/7280f687855abad0dfab).
My small app is almost done I believe, will see tomorrow how it goes ~

Using libtorrent-go is kinda necessary as there is no great torrent client lib in go. I desperately needed webseed support (BEP19) and none does it. Encryption is a great-to-have too.


Looks like you introduced #25 again?


receiving the following error ```Server is connected
elementName=Boron&elementSymbol=B&elementAtomicNumber=5&elementDescription=Boron+is+a+chemical+element+with+symbol+B+and+atomic+number+5.+Because+boron+is+produced+entirely+by+cosmic+ray+spallation+and+not+by+stellar+nucleosynthesis+it+is+a+low-abundance+element+in+both+the+Solar+system+and+the+Earth's+crust.%5B12%5D+Boron+is+concentrated+on+Earth+by+the+water-solubility+of+its+more+common+naturally+occurring+compounds%2C+the+borate+minerals.+These+are+mined+industrially+as+evaporites%2C+such+as+borax+and+kernite.+The+largest+proven+boron+deposits+are+in+Turkey%2C+which+is+also+the+largest+producer+of+boron+minerals.
{ [Error: ENOENT: no such file or directory, scandir 'public/Boron.html']
  errno: -2,
  code: 'ENOENT',
  syscall: 'scandir',
  path: 'public/Boron.html' }```


давай, напиши и тут )


I'd suggest not to use `:draw` at all. Instead, you can use `for _, h in ipairs(h:get_children())` on a hierarchy to iterate over all children hierarchies. On a hierarchy `h:get_size()` gives you the width/height and `h:get_matrix_from_parent().x0` and `.y0` gives you the position (assuming that nothing tries to scale/rotate/transform-in-any-other-way-than-just-:translate). Oh and `h:get_widget()` gives you the widget that this part of the hierarchy manages. You'd have to somehow identify your wrapper widgets here.

I don't know if iterating through the hierarchy like this really is better, but it would definitely get rid of the useless surface and cairo context. Oh and it would completely avoid lgi and its overhead. :-)


TeamCity LabKey :: Trunk :: Premium :: Git postgres 9.4 [Build 907](https://teamcity.labkey.org/viewLog.html?buildId=294657&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 15 (1 new), passed: 248 Build time: 00:34:38
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.ScrumtimeTest: <no details avaliable>

org.labkey.test.tests.RedCapTest.testDataTypes: <no details avaliable>

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

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

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

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

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

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

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

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

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

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


Abit unnecessary checking realm for every player entry, just need to check it for 1 or the raid/raidgroup


t


// Neighbor


Oh sorry if my notes hurt your enthusiastic this time. Don't be afraid, everyone produces mistakes (grammar error for me now?).. I know that may not be a very good idea but I suggest you to read a breif document about C (just minutes) [Here](http://overapi.com/c/) (especially the FlowControl). Sorry if being rude.


Don't add the y here, this is done just before, this could eventually break the flow, although this is highly unlikely.


Minor improvement could be to use a dictionary of <realm, list<string>> since a player name is never relevant outside of the realm scope and thus can limit the iterations needed everytime its checked against.


Add footer, button's style, modal


Danke schön!


Apparently prime numbers are important, or so tells me @AtomicBlom ^^"


Curious if this doesnt create massive performance penalties since the list will be iterated a ton of time. Looks fancy though!


# Notes for @jmangel
* I recommend using a linter (I found a syntax error)
  - http://jshint.com/
  - http://eslint.org/
  - http://jslint.com/
* If you're not already using one, I also recommend a good text editor. I'm finding Atom to be awesome: https://atom.io/ (it also has lots of linting plugins)
* avoid abbreviations for top-level resources like `lexi` for `lexicon` - spell them out
* I removed all the function parameters for now - we don't know what they'll be yet (might have an options hash, or a callback function, etc.)
* I won't always see comments in your code - better to use GitHub's line-commenting feature (if you browse the code for a commit or pull request, you can hover over any line, and click the plus button next to it; you can also tag me that way so I get a notification about it)
* JSDoc syntax uses `/** comment */` rather than `/* comment */`
* I've removed `deleteLanguage` and `deleteLanguages` for now. We should talk about whether users should be allowed to delete languages from the database. I'm also changing `upsertLanguage(s)` to `addLanguage`, so users can't override existing language information either.
* I made a number of other high-level decisions about which operations should be allowed on which types of objects. They're all open to discussion, of course.
* I realized that instead of `getPhraseFromText` and other methods like that, we can just say `getPhrase`. Even the text isn't specified, I'll still be able to find the phrase by ID (it'll just be slower).
* It seems like a ton of methods to write code for, but keep in mind that about half of them are just syntactic sugar. And the other half likewise will probably call some sort of generic `makeHTTPRequest` function or something (we'll have to sort out the details), and so in essence will also be a kind of syntactic sugar.


打错了！而且 OS X 上也通过测试了！什么鬼


Still will have to go back to re-edit the personal.vb form. Test message box keeps popping up but then, a tad bit too tired, so will look at it again later on tonight after work. Don't mind it, unless you just comment or edit it out.


Pls restore all my videos personal videos


bah, txfmt not inputfmt


Will do some small changes to this, but thanks for the base implementation!


References https://github.com/kivy/kivy/issues/3863


should never happen due to prior check


I still don't understand why you are talking about `this` in the context of static variables. Could you show me some Haxe code that demonstrates this problem?

Note that TTypeExpr is not much more than a type identifier. You can't do anything with it unless you go through the reflection API.


why did you remove the todo marker?


LGTM, @gernotstarke what do you think


该 commit id 已经提交。
补充了 table 的 nil 空洞问题，以及 table 长度的不确定性。
并简单介绍了一下  `table.new` 和 `table.clear` 函数。


ja mässiii


you'll need to negate the essentials.nick.changecolors permission.


Sorry, I thought rebasing might fix this, but Git still makes it look like an add and a move.  I think it's functionally the same as what Joe says:  "I think the expected approach (correct me if wrong) is there would simply be new classes created in the utility module and then the existing classes in standard procs would be marked deprecated and the existing processor impl would use the utility module import instead (thus making the now deprecated classes just sit there unused until they are removed in 1.0.0)."

If we want this commit to be cleaner, I can restart it and explicitly do what's being described.


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


fyi,

```go
if ocount <= vcount {
```

```
multi1 tinylfu: 6.781081ms 15858 total 7265 misses (hit rate 54 %)
gli tinylfu: 3.708753ms 6016 total 4036 misses (hit rate 32 %)
ps tinylfu: 4.332445ms 10448 total 4480 misses (hit rate 57 %)
cs tinylfu: 4.019741ms 6783 total 4727 misses (hit rate 30 %)
loop tinylfu: 221.997709ms 505500 total 255804 misses (hit rate 49 %)
```

So about 200 more misses for `multi1`, 17k fewer for loop, and the same otherwise.

Furthermore, if I then increase the cm4 by 2x it evens out as

```
multi1 tinylfu: 6.529585ms 15858 total 7051 misses (hit rate 55 %)
gli tinylfu: 3.475724ms 6016 total 3981 misses (hit rate 33 %)
ps tinylfu: 4.260191ms 10448 total 4486 misses (hit rate 57 %)
cs tinylfu: 3.179314ms 6783 total 4726 misses (hit rate 30 %)
tinylfu: 206.794271ms 505500 total 253356 misses (hit rate 49 %)
```

This indicates that either your supplemental hash functions could be improved to increase the distribution or the sketch size could be increased slightly. I think your sketch is 8x more compact than mine (I add 8 bytes per entry [16 counters]; you add 1 byte [2 counters]) so your addition of the doorkeeper is doing its job nicely. The above hit rates are very close to the Java simulator's, which has only a very slight edge on gli.


it will, but only negligibly. if there is a difference, I doubt it will be more than a couple of milliseconds


more_clear_state (mbadams5) - i386-Windows-7-VisualStudio-12: OK (1985 of 1986 tests passed)

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


It is


Ooops, strangely enough I have no write permissions for the Number Field extension, so I can't pull your PR. I think that this is an oversight and @nitriques can fix this.


This and a couple other commits came from a fevered mind.  I recently rewrote the implementation of this.  Don't take this file too seriously.


16 is the ID of the size data flag


Oops. I'll work out what caused this to be added a second time.


Do we need VersionedObjects to contain an array of Objects? It seems we only need two slots.


substr **and** trim? Also, what about links like http://www.yiiframework.com/wiki/?sort=update.desc ? query part is stripped here.


So if the `data` doesn't contain a field, the original value in `into` (i.e., `Value: 2`) won't overwritten to the default value? Is this consistent with our current implementation?


I mostly just dislike maintaining the many workarounds we have in the code, since most don't have tests and may eventually break again.

But if we documented every time we "fixed" edit data in a central place, maybe that could still provide adequate debugging hints.

It's not something I'm working on or have thought about a lot, though. :)


Thanks!

`listClear` needs to also set `listSelectedL` to `Nothing` since clearing the list and leaving behind a `Just` selected index would be invalid.  `listReverse` needs to have a similar behavior, i.e., the selected element needs to remain selected even if it changes position.  So if index `Just 0` is selected before a reversal, index `Just (N-1)` needs to be selected afterward (for a list with `N` elements).

Also please mention these behaviors in the Haddock, too.


So, to be fair, I am in general fine with (itemid << 16 | metadata & 0xFFFF),

But Blu told me people didn't think that was a good idea, so I offered the solution that IntelliJ idea considers a safe hashcode, based on two properties.

There's quite a few resources about it
http://stackoverflow.com/questions/3613102/why-use-a-prime-number-in-hashcode

The general gist of it is that hashcodes aren't guaranteed to be unique - Even using memory locations an int isn't wide enough - but they should be generated in such a way that when being bucketed, should evenly spread. If HashMap is using a modulus function, then ensuring the least significiant byte is the itemid would do that. Using the metadata in the least significant byte will, as I understand it, mean that a modulus function will probably end up with an uneven distribution and impact the performance of the hashmap.


Correction, you're using 2 bytes per entry (4 counters) since you have 4 vectors and each at half the width (w32). So 4x more compact than mine, or 2x with the above adjustments. So still a nice gain due to the doorkeeper, et al.


Thanks but without your work and sharing, no translation to make ;-D
Happy to contribute.
Regards


Why the check excludes the `actual.Group`?


查看以下commits  我认为state notifier driver 比 POWERSUSPEND更简洁 好用
https://github.com/shminer/android_kernel_flounder/commit/bd8988f5b0c9ab1f4113563394495f64173d0757
https://github.com/shminer/android_kernel_flounder/commit/a50f5edbe28182ef8e446692dc622e52119a6eae
https://github.com/shminer/android_kernel_flounder/commit/86d298171f5e1d49b29028950b26eb4bba990d12
https://github.com/shminer/android_kernel_flounder/commit/59b69641e131dd0baa9fc46b272d54ab810d9f82

另外在丹佛k1上不建议使用任何热插拔   k1是同步构架   热插拔时会造成emc延迟增加


 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:679](../blob/e19a8e31ea330b2909cb234987339bacbea28c98/include/chaiscript/language/chaiscript_parser.hpp#L679) Expression is always false because 'else if' condition matches previous condition at line 677. (multiCondition)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:683](../blob/e19a8e31ea330b2909cb234987339bacbea28c98/include/chaiscript/language/chaiscript_parser.hpp#L683) Expression is always false because 'else if' condition matches previous condition at line 681. (multiCondition)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:685](../blob/e19a8e31ea330b2909cb234987339bacbea28c98/include/chaiscript/language/chaiscript_parser.hpp#L685) Expression is always false because 'else if' condition matches previous condition at line 681. (multiCondition)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:685](../blob/e19a8e31ea330b2909cb234987339bacbea28c98/include/chaiscript/language/chaiscript_parser.hpp#L685) Expression is always false because 'else if' condition matches previous condition at line 683. (multiCondition)
 - __information__: [src/chaiscript_stdlib.cpp:1](../blob/e19a8e31ea330b2909cb234987339bacbea28c98/src/chaiscript_stdlib.cpp#L1) Skipping configuration 'CHAISCRIPT_MSVC;max;min' since the value of 'max' is unknown. Use -D if you want to check it. You can use -U to skip it explicitly. (ConfigurationNotChecked)
 - __information__: [src/main.cpp:0](../blob/e19a8e31ea330b2909cb234987339bacbea28c98/src/main.cpp#L0) Too many #ifdef configurations - cppcheck only checks 12 of 13 configurations. Use --force to check all configurations. (toomanyconfigs)
 - __information__: [src/main.cpp:1](../blob/e19a8e31ea330b2909cb234987339bacbea28c98/src/main.cpp#L1) Skipping configuration 'CHAISCRIPT_MSVC;max;min' since the value of 'max' is unknown. Use -D if you want to check it. You can use -U to skip it explicitly. (ConfigurationNotChecked)
 - __style__: [src/chaiscript_stdlib.cpp:18](../blob/e19a8e31ea330b2909cb234987339bacbea28c98/src/chaiscript_stdlib.cpp#L18) The function 'create_chaiscript_module_chaiscript_stdlib' is never used. (unusedFunction)
 - __style__: [src/main.cpp:169](../blob/e19a8e31ea330b2909cb234987339bacbea28c98/src/main.cpp#L169) The function 'get_eval_error' is never used. (unusedFunction)
 - __style__: [src/main.cpp:140](../blob/e19a8e31ea330b2909cb234987339bacbea28c98/src/main.cpp#L140) The function 'help' is never used. (unusedFunction)
 - __style__: [src/main.cpp:216](../blob/e19a8e31ea330b2909cb234987339bacbea28c98/src/main.cpp#L216) The function 'myexit' is never used. (unusedFunction)
 - __style__: [src/main.cpp:158](../blob/e19a8e31ea330b2909cb234987339bacbea28c98/src/main.cpp#L158) The function 'throws_exception' is never used. (unusedFunction)


How do I specify the python packages?


Haha, np–glad it helped!

On Sat, Jan 2, 2016 at 3:06 PM, Christopher Pyper <notifications@github.com>
wrote:

> @calvinfo <https://github.com/calvinfo> If we ever meet in real life, I
> owe you a beer, and by a beer I mean a whole case! Thanks for this dude,
> your a life-saver.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/3rd-Eden/memcached/commit/4ee6da819a60267ffb71b40c6e10d50b8e7a6b4d#commitcomment-15231595>
> .
>


Ok, pls report back.

Also, would it be possible to get access to this errorneous container, so I can see what is going on?

b.


Could you please explain what the line `flag&os.O_RDONLY > 0` does?


do it like below:
```cpp
auto vector = CE3D2_CREATE_VECTOR(4.0f, 22.0f, 71.98f);
auto vector2 = CE3D2::Vector(vector);

uut.transform_vector(vector2);
CE3D2_CHECK_VECTORS_EQUAL(vector, vector2);
```


every plugin is c.generic by default i.e. nothing is strictly required. I'd like to look into having a single metadata file (plugins.js) at some point, we just haven't established all requirements just yet.


I have prepared the Hungarian locale hu_HU.UTF-8.


These are only meant to contain exceptions. I agree they shouldn't sit here, I'm still adding them ad-hoc the same way I discover the UI implementation requirements.


I have the similar confusion here. This seems to be different from what we currently have: https://github.com/kubernetes/kubernetes/blob/master/pkg/conversion/decode.go#L146-L170


I might actually shift these to their own helper space. It's a little weird since I think Three implements them too, only a little different.


Hm, that's quite a hard problem.

Some kind of tree would be nice, but that's probably not possible with MathJAX.

Maybe something like this:

```tex
\left\{\begin{matrix}
  \mathbf{key:} &  \mathbf{value}\\
  \mathbf{some\_object:} & \left\{\begin{matrix}
    \mathbf{key1:} & \mathbf{value1}\\
    \mathbf{key2:} & \mathbf{value2}\\
  \end{matrix}\right.\\
\end{matrix}\right.
```
![object-latex](https://cloud.githubusercontent.com/assets/4294942/12081699/187eff20-b282-11e5-815e-3f4fe80cfbf8.png)

Or maybe just print regular JSON?


I've reverted the changes


The current SUNPRO conditional compilations are all relates to version 5 and 6 from pre 2005.

I have attempted to build VXL master with Solaris Studio 12.4 compilers.  The reasons for compilation failures with 12.4 are not related to the removal of the current conditional compilation code.  The current conditional code based on SUNPRO are related to SUNPRO versions 5 and 6, and are not problematic for SUNPRO 12.4.

If support for current Soloaris Studio compilers is required, then the problems related to the current compiler need to be addressed independant of the old conditional code.

Do we have a current need Solaris Studio compilers?


Pinging @snoyberg .  Adding this because a package that started with `a` wasn't at the beginning of the packages list, and so I thought it wasn't in the snapshot.


Changelog entry LGTM. :stars:


@hashedhyphen
`self.status.started` は113行目で `.reel` がすべてスタート済みかを評価するのに使っていまして、
このキーは一度 `true` にしたら `false` には戻らない想定でした。

ちょっと設計があまいのでクリックカウントを組みかえてみます。
PRありがとうございます。


We should write a macro that does this.


Wow that was stupid of me, my bad


:beers: @fmeyer


@legacyhardware when I pushed the last commit, it had to mearch your changes to the readme before pushing


我是注释掉这段了，如果不注释掉，上传图片的时候会出现502错误，上传不了，不知道怎么回事


Thanks, looks good!


need a []


yeah, that's a good point.


agreed


TIL that this is a thing you can do with Go


Delete


:+1: The sorting in diffs is already case-insensitive, so it would be only consistent to sort everything case-insensitively.


Wow... It is probably considered fair use since we asked him, right?


more_clear_state (mbadams5) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1450 of 1450 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.54%25-green.svg)


tanuo


good


すみません、言葉が足りませんでした。

いずれかの `.reel` に `self.status.started`  `false` があると、
`judgement()` 内にある画像が一致してるかどうかの評価までたどりつかないので、
この方法ではもろもろ不足している、ということでした。


No, it is not. This is old, anyway.


Now fixed


Looks like a fluke - `jct.gov` enforces HTTPS for me now.


I did everything you said and the bitrate isn't changing. I have a z22 yi with 1.2.13 stock firmware.
Do you put the +'s at front of each line or not???
Any help???


Also forgot to mention that it now relies upon tilt to get built

and more importantly, _it chooses the last modified log file_ so that the most up-to-date log is being shown.


这个代码需要把鼠标放到bs 上面才能实现缩进。。。


You don't need to document the imports. They are just implementation details so putting them in documentation is not necessary


I am reviewing git and github.  I only have that one css grid file and will upload to github, or atleast attempt.
Thanks for recent commits.  Where does clearfix go?  Looking to talk about things to anybody in Tuesday AM see what I ca do.
Chris


Well, I didn't do anything XD  Those emojis came when I created the repository :)


This is redundant. What would be a better idea is to describe the result of the progressIcon, like you do below


Don't know if this is the place to address, but students might ask why we don't (or can we) use IIFE's in Backbone.


Thanks for the information. For me, the old way still works so I won't change until I have to. I also added my concern to the bug report...


Rather than have the example hardcoded as a docstring, make a literalinclude to a an example in the examples directory


Pull it.
Vytvoř pull request.


You know what? I changed my mind. "Salvataggio emulazione" *does* fix the ambiguity, I got used to it and now I like it :-)


haproxy-devel, change gui to bootstrap


the __init__ method is a generic initialization method for all of python. Also, I don't know if it makes a difference, but I usually define the constructor parameters outside of the init method, in the class description. It would be a good idea to put the datatype of the paramter as well. Have a look at http://wiki.sugarlabs.org/go/Features/Sugar3_Docs/How_To_Write for an example


Yes that was what I was thinking. maybe I should leave off and leave caching up to the developer.


Ahoj, priznam se, ze jsem se na to koukla az dneska pozde a i kdyz to neni,
ze bych nevedela vubec, co po me chces, e to pro me porad skoro u vseho
neco, co si musim najit, ja se dela a nevim, kde bude nejlepsi zacit.. jsem
trochu zmatena a ztracena... jeste nemam ani dojety tutorial a splnene
vsechny pozadavky... s tim tutorialem nevim, jestli to ma jeste cenu, kdyz
us ses mi pustil do komentu a zrejme podle tutorialu budou nejake veci
jinak/hur...

On Sat, Jan 2, 2016 at 11:28 AM, PetrSvirak <notifications@github.com>
wrote:

> This clearly shows you used default template. I strongly suggest either
> rename this controller and give it a purpose or remove the controller
> completely.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/NeprejetelnaSalinama/PersonalAccounting/commit/05f827c797bb92b384be812bd8a7b4b959d95cb5#commitcomment-15229553>
> .
>


Shouldn't we just remove "Site Handle" from the individual page and leave it to the Site Config?


@jnardone: no, I think the patch's quality is not enough to submit.


Version is currently the only field we have set that is required (in external values).  Group empty string is not distinguishable from unset.


I do not understand this because models [5]


What we have today would be incorrect - group may be empty both on the wire and via defaults.


I think the old version was correct, but please correct me if I am wrong:
In the case of compound nouns, only 1 noun takes the plural form. In this case,"parameter" acts as an adjective and does not take the plural form, while "names" is the base word and takes the plural.


In the case where we are using versioned objects (internally using v1,
externally v1), we would be checking for first and last of the array.
Doing it as first and last of the array is easier and clearer than having
two specific fields.

On Sun, Jan 3, 2016 at 7:12 PM, Chao Xu <notifications@github.com> wrote:

> Do we need VersionedObjects to contain an array of Objects? It seems we
> only need two slots.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/smarterclayton/kubernetes/commit/4769fbbed689ad4e90f45c8424526b6454799168#commitcomment-15235793>
> .
>


Remove this


@nrc This commit has numerous breaking changes to public APIs used by compiler plugins. Could we please be better about tagging such things with `[breaking-change]` **and** include a description of how to fix things?


It's possible today if the conversion allows the value to be copied over.  This test is explicitly calling out the behavior that the exact object for into is what is the result value, and that we place the values into the decodable object.


With the commenting of the `jQuery.when` promise above, this return block exits the route matching, affecting the handling of other routes...


I DONT KNOW D:


Oh, sorry, yes.  typeToGVK is the arbiter of the "preferred" type for a golang object.  The order which you register types establishes their priority (so you can register the same Go object for multiple names in multiple schemas).


@westonruter: removed in https://github.com/WP-API/client-js/commit/51e83eb604795e8b93b5f188b0f80099f620e87a


That has the disadvantage of polluting the global scope for development. Anyway this script could be improved to make it a recursive process (e.g. pug-lexer depends on pug-error etc.)


Confirmed this will not work. the 2nd parameter is a binding array.


That's what I started with, but I switched to putting the event logs in `addInternal`. The only error usually returned here is that the range needs no action (e.g. it isn't ready to split or doesn't need GC) which wasn't particularly interesting to see.


I kept the exceptions in case this is run in a release build


Yeah, I think so. And note that queues are only disabled in unit tests.


Done.


I have sort-of competed Menu Bar and Notification CSS components. I need to tidy them up. Menu Bar consists of rotating  menu items, with sub-menu items. There are 4 notifications with 4 different animations: move-out left or right, and fade-out up or down.
I am scratching my head about the other 2 components. Any suggestion? I am looking at the Image Mask right now.


Ok


this new cocumenr


now using the latest sphinx/sphinx gallery from github and I'm getting the
same error

On Sun, Jan 3, 2016 at 2:50 PM, Eric Larson <notifications@github.com>
wrote:

> > I think I'm just using the anaconda version of sphinx...
>
> Anaconda will be the latest release, which isn't the same necessarily as
> latest master.
>
> You might also need latest master sphinx-gallery.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mne-tools/mne-python/commit/4b866e009bbbaa3192906e4655280407f147deb9#commitcomment-15235169>
> .
>


Actually, can remove the $dbh, since the database has already been connected.:
````
if (sqlQueryNoLogIgnoreError("GRANT FILE ON *.* TO '$login'@'$loginhost'") == FALSE) {


This is tested by ATTs, but should it have a unit test of it's own?


“public” seems redundant here since we can’t have a collaborative private list, right?


:joy:


I followed these instructions using firefox after installing greasemonkey and got a message that my fitness pal had been changed to show net carbs.  But my fitness pal still doesnt show net carbs


+1 like, +1 prayer


Actually i think the problem is that in copying the originals back they
went into the wrong directory.  The used to be in a util directory but are
now with the standard procs themselves.  Don't want to waste your time but
it is probably best to just do a clean commit just doing just what we
mentioned.

Thanks
Joe

On Sun, Jan 3, 2016 at 7:02 PM, Joe Gresock <notifications@github.com>
wrote:

> Sorry, I thought rebasing might fix this, but Git still makes it look like
> an add and a move. I think it's functionally the same as what Joe says: "I
> think the expected approach (correct me if wrong) is there would simply be
> new classes created in the utility module and then the existing classes in
> standard procs would be marked deprecated and the existing processor impl
> would use the utility module import instead (thus making the now deprecated
> classes just sit there unused until they are removed in 1.0.0)."
>
> If we want this commit to be cleaner, I can restart it and explicitly do
> what's being described.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/apache/nifi/commit/57047ea46b12419d4e2da1e4297a9612d4e06291#commitcomment-15235769>
> .
>


@JustPingo This sounds good. Yes there is always guy that impatient and try a WIP.


I think your doorkeeper might be excessively large. Calculating the number of bytes per entry indicates that its pretty high (16).

Passing in `size` instead of `sample` reduces it down with much of an impact. You can probably keep refining that to find a good balance.


Since it covers both, would something like this be better?

`Microsoft X-Box One pad (Covert Forces / Standard pad with 2015 firmware)`


Thanks for the comments! I pushed a commit that fixes some of the issues you pointed out.


The reference to `this.dateFormatter.format(oBegda)` here must be replaced with a call to the new utility function `com.transfieldservices.utils.Conversions.makeSAPDateTime(oBegda, false)`.


@AnimbusDev Fair enough.


[Uploading Apis…]()
[Uploading database…]()
[Uploading Slim…]()
[Uploading tests…]()
[Uploading upload…]()
[Uploading .gitignore…]()
[Uploading .htaccess…]()
[Uploading .travis.yml…]()
[Uploading composer.json…]()
[Uploading CONTRIBUTING.md…]()
[Uploading index.php…]()
[Uploading LICENSE…]()
[Uploading phpunit.xml.dist…]()
[Uploading README.markdown…]()


**BUG HERE** With these lines only the NATIVE_FIRM for both old and new gets a tripe installation attempt. However what about the others?


Change to:
````
$rowsupdated = generic_sql_affected_rows();


 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:679](../blob/353a077c6b207938af6ca22aaff9d1d2a8f2f6f9/include/chaiscript/language/chaiscript_parser.hpp#L679) Expression is always false because 'else if' condition matches previous condition at line 677. (multiCondition)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:683](../blob/353a077c6b207938af6ca22aaff9d1d2a8f2f6f9/include/chaiscript/language/chaiscript_parser.hpp#L683) Expression is always false because 'else if' condition matches previous condition at line 681. (multiCondition)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:685](../blob/353a077c6b207938af6ca22aaff9d1d2a8f2f6f9/include/chaiscript/language/chaiscript_parser.hpp#L685) Expression is always false because 'else if' condition matches previous condition at line 681. (multiCondition)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:685](../blob/353a077c6b207938af6ca22aaff9d1d2a8f2f6f9/include/chaiscript/language/chaiscript_parser.hpp#L685) Expression is always false because 'else if' condition matches previous condition at line 683. (multiCondition)
 - __information__: [src/chaiscript_stdlib.cpp:1](../blob/353a077c6b207938af6ca22aaff9d1d2a8f2f6f9/src/chaiscript_stdlib.cpp#L1) Skipping configuration 'CHAISCRIPT_MSVC;max;min' since the value of 'max' is unknown. Use -D if you want to check it. You can use -U to skip it explicitly. (ConfigurationNotChecked)
 - __information__: [src/main.cpp:0](../blob/353a077c6b207938af6ca22aaff9d1d2a8f2f6f9/src/main.cpp#L0) Too many #ifdef configurations - cppcheck only checks 12 of 13 configurations. Use --force to check all configurations. (toomanyconfigs)
 - __information__: [src/main.cpp:1](../blob/353a077c6b207938af6ca22aaff9d1d2a8f2f6f9/src/main.cpp#L1) Skipping configuration 'CHAISCRIPT_MSVC;max;min' since the value of 'max' is unknown. Use -D if you want to check it. You can use -U to skip it explicitly. (ConfigurationNotChecked)
 - __style__: [src/chaiscript_stdlib.cpp:18](../blob/353a077c6b207938af6ca22aaff9d1d2a8f2f6f9/src/chaiscript_stdlib.cpp#L18) The function 'create_chaiscript_module_chaiscript_stdlib' is never used. (unusedFunction)
 - __style__: [src/main.cpp:169](../blob/353a077c6b207938af6ca22aaff9d1d2a8f2f6f9/src/main.cpp#L169) The function 'get_eval_error' is never used. (unusedFunction)
 - __style__: [src/main.cpp:140](../blob/353a077c6b207938af6ca22aaff9d1d2a8f2f6f9/src/main.cpp#L140) The function 'help' is never used. (unusedFunction)
 - __style__: [src/main.cpp:216](../blob/353a077c6b207938af6ca22aaff9d1d2a8f2f6f9/src/main.cpp#L216) The function 'myexit' is never used. (unusedFunction)
 - __style__: [src/main.cpp:158](../blob/353a077c6b207938af6ca22aaff9d1d2a8f2f6f9/src/main.cpp#L158) The function 'throws_exception' is never used. (unusedFunction)


I took another approach to achieve the same affect. PTAL at the latest commit.


Change to:
````
function generic_sql_affected_rows()
 {
   return mysql_affected_rows();
 }


Eh.. I suggest you malloc for the list of TitleIDs so you just scan the path with FS only once.


**BUG HERE** If this is freed already - when the first title isn't NATIVE_FIRM. deref a NULL?


Or maybe `procedure-rename` could be syntax that would do that if it can see what its second argument is?


Nice comment


I put instructions for updating our github repo on our github wiki.
There is a menu item 'wiki' at the top of our github project page.

@shengdarLee Look at the bootstrap web site under 'components' for ideas.  For the menu, we should supply a base plain vanilla menu with the opportunity for the user to add a class to enable the fancy styling.


just unexport these


Looks good.

On Sun, Jan 3, 2016 at 8:52 PM, Peter Mattis <notifications@github.com>
wrote:

> I took another approach to achieve the same affect. PTAL at the latest
> commit.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/cockroachdb/cockroach/commit/5980ed97bda9823ca59132eddd7ef3b7874caee0#commitcomment-15236059>
> .
>


Yup. PSS is the default signature method for RSA.php so to make it mirror that I did the same here. ie. this is essentially forcing you to manually set the mode.

Altho thinking about it...  maybe the notion that the "function signatures" need to match up 100% is antiquated. A hypothetical Crypt/DSA.php wouldn't have a PADDING_PKCS1 and I don't think it'd have a PADDING_PSS mode either. And since, in theory, SSH/Agent/Identity.php can have RSA *or* DSA keys expecting this or RSA.php to match is unrealistic.

But then again having it be `RSA::PADDING_PKCS1` by default is equally short sided. Maybe I'll just make that method take one parameter and that's it. It then uses whatever signature type is appropriate. If you pass two to it it's not like PHP is going to complain. as demonstrated by, among other things, the existence of `func_get_args()`


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


TODO
Change to:
function generic_sql_insert_id()
 {
   return mysql_insert_id($GLOBALS['dbh']);
 }


I think have a less pessimistic false positive rate (currently 0.01%) would be a good change. Switching to 0.1% seemed fine (1 byte per entry) as did 0.2% (0.5 bytes per entry). At 0.4% a slight impact to cs was observed (29%) which was merely 50 misses. So there is a lot of opportunity here.


more_clear_state (mbadams5) - Win64-Windows-7-VisualStudio-12: OK (1985 of 1986 tests passed)

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


TODO
change to:
   $body = add_escape_custom($body);
   $id = sqlInsert("INSERT INTO transactions ( " .
     "date, title, pid, user, groupname, authorized " .


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


@bluedxca93
So, in a nutshell, if one is on Cinnamon 2.8.6 and has starkmenu installed and has a gap between the menu and the left border of the screen, he or she has got to hack the .CSS file of the theme. That's all, right?


@jelledevleeschouwer :)


Instead of splitting this into two typedefs, did you consider `using pending_response = std::pair<const message_id, std::pair<behavior, error_handler>>` (the value type of `pending_async_responses_ `)? This might streamline the code a bit.


Title for the commit should actually have been "loops" or "iterators" or something


No Shift+Click because many players use tablets and smartphones, so dig and sneak only.


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


代码大全。。。。。

On Jan 4, 2016 1:54 AM, Harry Ji <notifications@github.com> wrote:

这是啥书？

—
Reply to this email directly or view it on GitHub<https://github.com/yuanhui-yang/Code-Complete/commit/e38cff17487be5b8e6d9ef09d57392a860504627#commitcomment-15238330>.


@jelledevleeschouwer : Test not working was my fault. I have to remember to add a multicast route to localhost like `route add -net 224.0.0.0/4 dev lo`


Please add ```LOG("ERROR - Out of range exception in Beam::setCustomLightVisible)")```


right, fixed.


Hmm ok, I was already wondering why this change was showing lol


This change break the text over the image. See issue #426


Fixed in 2147938d0fd41c86b70ba06cc66118b909dcedd6


@jelledevleeschouwer the multi-hop test is not crashing anymore, but now it seems frames are not making it through.
`[6LoWPAN]$ FWD!
[6LoWPAN]$ Suppressing duplicate frame!
[6LoWPAN]$ FWD!
[6LoWPAN]$ Suppressing duplicate frame!
`

...

`---- Ping6 timeout!!!
PING 1 to aaaa:6109:0000:0000:0200:00aa:ab00:0001: Error 1
`


Stephen Duke Solution: https://github.com/BMDuke/google_homepage/commit/8ba1f081c5cebcc52d263cea9af5757c382c5991


Also, we should assume that users use multiplexing more often than `await`:

```cpp
if (self->multiplexes(mid))
  return msg_type::async_response;
if (self->awaits(mid))
  return msg_type::sync_response;
return msg_type::expired_response;
```


Indeed, they are. I'll replace them with `run-command` params


Good catch. I'll add a check right away.


Please add ```
if (static_cast<unsigned int>(number) >= netCustomLightArray.size())
{
    LOG("WARNING - non-existent light number ("+ TOSTRING(number) + ", count is: " + TOSTRING(netCustomLightArray.size()));
return;
}


Right. Thanks


@EarthCitizen This is definitely not the good place to report issue :) I have created https://jira.sonarsource.com/browse/SONARGRADL-9 and I will see how we can do that during next sprint.


### Expected an assignment or function call and instead saw an expression.

```
asdfsdafasdsfafs123
^
```


I think I'd prefer to keep the divergence from Algebra to a minimum in this PR. Maybe create a separate issue for that once this is merged?


for now :+1:, but we should fix this latter on in all projects


Ничего страшного. Видел коммиты только с изменением одного символа)


This is done in the next commit (for consistency with the rest of this file).


@pazguille any idea when this will be merged?


Why you removed this?


sicne you already have imported a class from bouncycastle and added the license, you could get
https://github.com/bcgit/bc-java/blob/master/core/src/main/java/org/bouncycastle/crypto/digests/MD5Digest.java
https://github.com/bcgit/bc-java/blob/master/core/src/main/java/org/bouncycastle/crypto/digests/SHA1Digest.java

the look reasonable to modify so they work with netty. Let me know If I can help with that.


Мне кажется, такая одноразовая вещь там не нужна. Или я не понял твоей мысли?


This crashes when there are multiple teamd running. Also it fails when this line is executed before the `server_thread` spawns the `teamd` process. How about using `subprocess.Popen` to run the daemon. You get finer control over the process without the additional hacks.


An interesting additional test case is what happens when using `request().await()` while there are already multiplexed requests.


> I think we can avoid the duplication here by putting this at the top:
>
> `return unless CrowbarPacemakerHelper.is_cluster_founder?(node)`

True. I kept it that way for consistency with other files. But I don't mind either way. Want me to change that?

> Similarly we can get rid of the `if` block and one level of indentation with:
>
> `return unless node[:pacemaker][:notifications][:smtp][:enabled]`

Nope, there's a `else` :-)


It would be good to change messages here to reflect the unit dpinger expects for the parameters. For time period, example says default is 30s, user would naturally end up considering -t expects it in seconds and add something like '-t 50', when it's expecting it in msec


Yay! Another mysterious old var gone.


Maybe it would be better to use  strpos() instead of preg_match, if you're just looking for the : character, as it'll be always faster ;)


I suggest extracting some code from the `js.Return` because the expression is becoming a bit hard to read.


Needed to compile in Cadence Incisive


Are you sure when it fails it returns `1`? or does it returns a different code number?


It's cleaner to define boolean values as True/False


space nit here and in the line above


OK


it was an error in the release notes.

15.5.0 removes python 2.6 support

see http://labs.twistedmatrix.com/2015/12/twisted-155-released.html

thanks


no blank line needed here


Thanks. Fixed all notes except using 'pvscan' instead of 'lvm pvscan' because apparently it doesn't work in inaugurator's busybox for some reason.


neither here (blank lines)


in addition to adding the real svg logo in the images folder, I also deleted some unnecessary images in the same folder.


New function field not stored can be merged.


@philmmanjaro I'm in favor of supporting budgie but TBH this is the first time ever I'm hearing of it. What's the rationale behind the choice of having it supersede gnome?


@CodingFabian sounds good! Thanks!


> Nope, there's a else :-)

Ah, crap ;-)


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


May I also suggest optional chaining over forced unwrap ? Like `event.allTouches()?.first?.phase`. You don't want your app to crash for a print :)


this value is not used anywhere


I think that change shouldn't have been there, it reverts to an old version of fc that doesn't have the fix for segfault in the websocket code...


Great. Ship it.


zh_cn translations
effect= 效果
invite = 邀请
pvptoggle=开/关PVP
ally.cancel=取消同盟
ally.invite.send=发送同盟邀请
ally.invite.cancel=取消已发送的同盟邀请
buylife=购买生命
(btw what is the life here?)
buyslot=购买槽位
(i dont really get what a slot is)
sethome=设置基地
(assuming the home is more of a base)
teleport=传送至基地
openinvitation=查看邀请
war.invite.send=发送停战协议
war.invite.accept=接受对方的停战协议
war.start=向(guild name here)宣战
bank.pay=存款
bank.withdraw=取款


Maybe the submissionsurl could be `https://open.kattis.com/submissions/%s` instead, so that you would do `get_url(cfg, submissionsurl) % submission_id`, to avoid the assumption that the URL would always end in `/submissionid`


i wanna change some parts of it


Okay, I just tried the "dest" command with a city having umlauts, e.g. Nürnberg, and it did not work with the old function. However it worked when using Google.

Sent from my Windows Phone
________________________________
From: Asger Juul Brunshøj<mailto:notifications@github.com>
Sent: 04.01.2016 10:46
To: schmimae/Public-AutoHotKey-Scripts<mailto:Public-AutoHotKey-Scripts@noreply.github.com>
Cc: Schmidt-Korth, Michael  ST/HZA-ZAD<mailto:michael.schmidt-korth@schaeffler.com>
Subject: Re: [Public-AutoHotKey-Scripts] Changed uriEncode() function, as the old one did not always work (71451f1)

I just tried and the old one works for me with ü and ß. Maybe we are running different versions of AHK that do regex differently. I don't know? Weird.

—
Reply to this email directly or view it on GitHub<https://github.com/schmimae/Public-AutoHotKey-Scripts/commit/71451f1263c484dc4b20360d7130c4fec88013bc#commitcomment-15239556>.


Ok. Please open me a github bug report. I will look into it after the 1.4 release. Or if you manage to generate a short apitrace trace, you could open a bug report in mesa (put the one with the reverted screen). It would be easier to debug and put me in cc and Timothy.


After thinking a bit about this work flow, it seems quite inefficient:
- `filter_msg` scans both `sync_responses_` and `async_responses_`, which is a worst case of `O(N) + O(1)`.
- In `case msg_type::async_response` and `case msg_type::sync_response` we perform *the same* lookup again

Instead, `filter_msg` could simply return `response` by looking at the message ID only. Then our `case response` performs the `O(1)` lookup first, using the iterator immediately on success. Then it performs the `O(N)` lookup in the `forward_list` to see whether the ID belongs to a blocking request (using the iterator only if it's the most recent blocking request). If neither container contains the ID the code falls back to the current `case expired_response`.

Makes sense?


the same as with the pid_stream. I'd really prefer using `subprocess.Popen` for all `utils.system` calls. (note `subprocess.check_call` is new in python 2.5 and should not be used in autotest. Using `subprocess.call` should be safe.).


It would be nice to define these variables outside of the function(s) and then use them via ``global``.


How about simplifying this method to:
```php
if ($targetQuantity > $itemQuantity) {
    $this->increaseUnitsNumber
} else if ($targetQuantity < $itemQuantity) {
    $this->decreaseUnitsNumber
}
```


Actually, in [cvmfs](https://github.com/cvmfs/cvmfs/tree/devel/externals/pacparser/src) we do use a static parparser.  We currently create a static library ourselves.  I didn't check for a while for pacparser upstream changes, just seeing now that it would have been possible at some point to have a static library out of the box.


DCB report is working without

@ModelAttribute("dcbReport")
 -    public DCBReport dCBReportResultModel() {
 -        return dCBReport;
 -    }

?


done


Thank you


You should share the file inside /migrations
this files are auto generated using make migration


SC/13 should not be creating these files any more.  I removed them from the .gitignore file in the same commits where they were removed from the Makefiles:  VERSION was removed in ec3282b6ff3997e92fa5fa7ff3f8cce4afe59aaa and the XML files were removed in ea2b63af33e9cac40338bffe2d9dd16b9c5f96fb.


The cameranodepos[0] / cameranodedir[0] system is pretty arcane code. Coder needs to know RoR jargon to understand it. Truckfile "cameras" and rig_t::cameranode* have nothing to do with camera, really, they orient vehicle in space. I'd like the vars to be named differently at least. Further, do we even need this system? Couldn't the orientation be determined by node coordinates in truckfile?

Further, it looks like the code checks for cameranode* validity and only then uses them. Shouldn't trucks without "camera" not spawn at all?


Maybe more appropriate will be `$targetUnitsNumber` -> `$increaseBy`.


not properly tested new buy functions, I believe they work, I did identified the shredder issue #193


You should not put this here, should just give a link to that and that's enough. Taking advantage of something licensed in MIT without acknowledging its author first is violating that license.


@erikse perhaps this should be squashed with the commit that uses it?


Thanks for the feedback. I think we should be careful not to over-engineer this (after all, people can always override the default formatting if they want something special :) ). At least we should not rely on specific non-standard packages for indentation and stuff.

How about something that looks more traditional, with newlines and indentation blocks? I'm only not sure how to do that with LaTeX, at least the following doesn't work:

```tex
\{\newline
\indent key: value\newline
\}\newline
```


I think this PR is a good opportunity to get rid of this `sync/async` mess for good.

For example, `pending_sync_responses_` could be renamed to `awaited_responses_` and `pending_async_responses_` to `multiplexed_responses_`.

Also, I think we don't need to replace most uses of `.then` with `.await` in the unit tests (please correct me if I'm wrong).

There is probably a bug with cached multiplexed responses when a multiplexed response message is stored in the cache (for example because an actor used `.await` in the meantime). At least I couldn't see how this case is handled in the code.


It iterates the cached messages and calls function `invoke_message`, and multiplexed responses are handled in `invoke_message`.


so you've gone slim recents :p


What about this:
```scala
def isClassNameInAncestors = {
  val ancestors =
    obj DOT "$classData" DOT "arrayBase" DOT "ancestors"
  if (isStrongMode) {
    js.BinaryOp(JSBinaryOp.in,
        js.StringLiteral(className),
        ancestors)
   } else {
     ancestors DOT className
   }
}
js.Return(!(!(genIsScalaJSObject(obj) &&
    ((obj DOT "$classData" DOT "arrayDepth") === depth) &&
    isClassNameInAncestors)))
```


And it's overwritten by the C code... bugzilla.gnome.org is the correct location for bug reports anyway.


Conflicts !

Auto-merging modules/presence/notify.c
CONFLICT (add/add): Merge conflict in modules/presence/notify.c
Auto-merging modules/db_text/dbtext.h
CONFLICT (add/add): Merge conflict in modules/db_text/dbtext.h
Auto-merging modules/db_text/dbtext.c
CONFLICT (add/add): Merge conflict in modules/db_text/dbtext.c
Auto-merging modules/db_text/dbt_base.c
CONFLICT (add/add): Merge conflict in modules/db_text/dbt_base.c
Automatic merge failed; fix conflicts and then commit the result.


No. Comments on the mailing-list or file new bugs at bugzilla.gnome.org.


So, the behavior argument (`bhvr_stack().back()`) is ignored for multiplexed requests? Ok.


As long as OpenGLRaw keeps, well, raw, I’m okay with that :). Thanks for the PR!


Hvorfor kommentere i stedet for å ta bort?


wrong indentation in this block?


Agreed. See gitter comment.


there's a missing space between `+` and `k`.


Yes.


Just log out users at Keycloak after deleting their account.


Godt spørsmål! Skal bort.


Welph...f*cked up the history now...


You want
```
if [ $SAVEYN = "y" -o $SAVEYN = "yes" ]; then
```
You can't use | or || because those are very special chars to shell, and you have to use single(!) equals. See `man test` for the full docs on the `[` command.


I think you just expressed it there yourself. Changes pushed


Oh also I guess you're lowercasing too. Since you have to do two comparisons it's simplest to (destructively and non-functional-programmingy) edit:
```
SAVEYN=${SAVEYN,,}
```


We have no submenu more in the navbar


We have no submenu more


Would be defined in the ty.-load.php


Ah yes well spotted!


Why did you delete NONE.MP3? I actually needed this for various "no music" situations. Or is there a workaround?


`convertDate()` converts the datetime to the site Date format


This indention looks just ugly and not K&R... at least, the inner code should be at the correct indention level and the second line of the two-line-if should start with "range." and "range." vertically aligned.


Hi, a PR means that you have done some changes in your repository and you ask to me to include them in mine. So if I accept the PR (and I did it!), this means that your patch is now included also in my repository.
Git is different from SVN in the sense that there is no "central" repository.
In the future, just be sure that you sync your repository to mine, since I plan to commit some changes and if you do not do that we will end having some conflicts.

Thumbs up and thanks again!!!


Oops, reverted in master, thanks for spotting it.


Wanna take a look... :)


Yup, of course. I will do a orphan branch ;-).Thanks!


Are you sure about dropping the __bridged reference?


I think the `(B)` in `MAX BYTES(B)` is pointless. We are already stating this is `BYTES` after all, and the extra info may lead to confusion.


psst, these aren't actually passing variables in. Instead you're accessing a global which happens to be named the same as the local variable you're trying to use. What you *should* do is
```
customaddr() {
 ADDR="$1"
 recordmydesktop --overwrite --on-the-fly-encoding &
 castnow out.ogv --tomp4 --address "$ADDR" &
}
```

Every bash function is like a minature shell script. If you run
```
customaddr A B C
```
then inside it, $1="A", $2="B", $3="C" (and conversely if you just run `customaddr`, $1=$2=$3="").


Thanks! I see three issues I'd like you to address before I merge:

1. You're only installing the upstart file (you need the systemd file for Fedora and RHEL7).
2. The NetworkManager dispatcher script isn't included.
3. The manual page isn't included.


I'll work on this at a later point in time, for now I don't want peoples' schedules randomly disappearing. There seems to be an issue where you need to refresh the external site for this site to change, which defeats the purpose of an external site in the first place.


Right... fixing it.


adfadfa


missing space between `<<` and `si_t`


The width & height are tested with "==" in the first loop, so it should be safe to just use ctx->width & ctx->height here without another CM..GetDimensions() call.


Aliasing a bidder is useful but what's really needed is the ability to reference an adapter multiple times under different aliases, eg so that one could run AppNexus as appnexus and also AppNexus as bRealTime (in order to run both AppNexus and bRealTime or another AppNexus-based network simultaneously).


Also, [DRY](http://www.c2.com/cgi/wiki?DontRepeatYourself)!
```
maincast () {
  customcast 192.168.1.5
}

bedroomcast () {
  customcast 192.168.1.10
}

```


typo? => ANNOUNCE


No commented code should be in the review.


I'm not sure if `N/A` is the best to specify there's no quotas set, but I'm okay with it.

However, you need to drop the 'cherry-pick' line in the commit message, and change `Fix:14216` to `Fixes: #14216`.

Also, a test to make sure this is working as intended when quotas are set (or unset) would be much appreciated.


Yep, and if there's no regular behavior (bhvr_stack is empty), `bhvr_stack().back()` will lead undefined behavior.


:+1: For making ```Beam::interTruckCollisions()``` an actual Beam method rather than an obfuscated semi-global function.
:+1: For reducing complexity
:+1: For more straightforward code


margins, paddings and layout parameters should be set in the correspondenting fragment_notes_list_note_item.xml, shouldn't they?


This introduces inoconsistency - we should either stick to the existing formatDescription & dimensions assignments or consistently switch to the way you changed in the first loop (lines 256-257).


The extra quotes are a realllllly good idea. A+


YOU FORGOT SCROLLBAR.


hi brandon, is this the line for position?
measureControlOptions: {position: 'topleft' }
tks man, best


Should we mention the possible exception of Parent overriding with ParentCamera?


In the future, you should avoid such waits as much as possible. Since we are not going to waste time for google, we will leave it as it is.


PTY allocation request failed on channel 0


Just replace the 'old' `then` to the 'new' `await`.


Can you explain please, why you are returning the whole content instead of only one line? This method should only return an excerpt, not the whole content?


Please submit this as pull request. Thanks! :)


That should be possible with the current implementation of aliasing? I'm currently using two AppNexus aliasses right next to each other on the same ad slots. Works like a charm.
If you are seeing different behaviour you could create an issue on GitHub with a part of your setup/logs or, even better, an online page with the implementation.


Happy new year. I'm taking a look now…


Why did you switch to CardView? Can you please provide a screenshot? Can't imagine, what these changes will look like...


This should read "TextureOrigin", not "TextureCorner" (which still works for legacy reasons but shouldn't be referenced anywhere).


I'm pretty sure the correct approach here will be to just rebase the other commit on top of current master and repush the whole branch.


Missing period at the end. =)


Added .gitattributes & .gitignore files


p.s. Some other useful links re how the scoping works:

http://stackoverflow.com/questions/5464811/ruby-koans-explicit-scoping-on-a-class-definition-part-2

https://groups.google.com/forum/#!topic/comp.lang.ruby/t5rtDNol3P8


Since these strings have special meaning and are used multiple times throughout the codebase their names should be extracted to constants. Another option would be to save their vids as variables and then refer to these subsequently. That would avoid breaking the functionality if someone decides to change the name.


Should this say *transformation* instead of *translation*?


I can try, there are subsequent commits that modify this file - like snake casing the name, and one has a content change further down the history


no need for explicit `return;` here.


Same comment as above


Iam not sure what this is supposed to mean. Maybe u can use something like all_ingredients.repeated_combination(100).each ?


well, how about `os.ttyname`?


Of course it could. But I won't create a script services that does the exact same thing than `$xwiki.getSpaces()`. Otherwise, we should not have deprecated it!

Anyway, this solution don't fix the scaling issue. Maybe using the document tree macro would be better.


You can add calories counting here and return array [total_score, calories]


You could leave this test here but change the call to `$response->getErrorMessage()` to `$response->getFullError()`


Wouldn't it be slightly more readable to write:
* implies two things: that the object is both the owner (orxObject_SetOwner()) and the parent (orxObject_SetParent())...
Should we also mention the exception to the parenting rule, ie. ParentCamera?


missing space between `;` and `//`


> I agree that your arguments on API design are very appealing. Unfortunately, rust-utp follows TcpStream design closer than your suggestion. My little test shows that.

This is interesting, I was tracing where in the rust code this happens. On Linux, this is done implicitly here:

https://github.com/rust-lang/rust/blob/master/src/libstd/sys/unix/net.rs#L110

And it seems to be the [default posix behavior](http://man7.org/linux/man-pages/man7/pipe.7.html)

> If all file descriptors referring to the write end of a pipe have been closed, then an attempt to read(2) from the pipe will see end-of-file (read(2) will return 0).

The behavior I was describing seems to be default for windows as [this code suggests](https://github.com/rust-lang/rust/blob/master/src/libstd/sys/windows/net.rs#L131). I hate to say it, but I like the Windows one better :). But yeah, as you say, we should stick with what rust uses as standard.


Can we add some detail to the commit message explaining why the security groups have been removed here please.


This was a quick hack. I was hoping you'd come up with something more awesomer.


missing space between `,` and `1` ; and an extra space after `1`.


Why is the arbitrary lifetime important? I'm hesitant...


Wouldn't it be better to use the same coding style in the comment than in the code?
Ie. no spaces around parens, pstChild instead of child, pstObject instead of obj


This still looks odd.

We return `FALSE` if the functionality provided by this function completed and `2` if nothing happened.

How about:
- `TRUE` if the functionality provided by this function completed
- `FALSE` if it did not


The cast helper is unnecessary.


Yes, you should use the named query for now.

You mean using the document tree macro to list only spaces? Is that possible?


Missing period at the end. =)


I think we should say "without using the link graph" instead of "immediately", as graph transition can also happen immediately.


Missing newline here.


AFAIK we cannot use named queries in the script services.


Add caption to image asset


Maybe a line break would help (at least it would in the PR section of github ;) ).


According to our coding standards false should be in uppercase.


@lucasart : OK, I checked that nodestime=600 is not a big concern, using it or not.


Missing newline at the end of the file.


It's no good. It always returns `/dev/ptsx`:

```
The file /dev/ptmx is a character file with major number 5 and minor number 2, usually of mode 0666 and owner.group of root.root.  It is used to create a pseudoterminal master and slave pair.
```

See http://unix.stackexchange.com/questions/21280/difference-between-pts-and-tty.

BTW, I just noticed that the unittests live in the old location. Sending a fix...


Yeah, I see no problems. Have commented on a couple of style niggles but I'm not sure what your coding style guidelines are.


This should probably mention that orxObject_DeleteNeighborList() needs to be called in case of non-NULL result, and also add it to the example?
Same comment about the example regarding the coding standards (for example, spaces around the **<** operator).


I'd put ID in uppercase for both uses.


Yes these commits have been forward-ported in saas-6 but this had not to be done. Commit 64ad6d1674f8b84bb38cb6ca2dd193c3ae8bb540 fixes this.