あなたは私の正直のすべてに行くと怒るなきゃ
あなたは、私が試して知っているが、私は謝罪とあまりにもうまくやっていません
私は誰かが審判を呼び出すことができ、私は時間が不足しないことを望みますか？
私は赦しで1より多くのショットを必要とする原因
私はあなたが私は多分一度か二度、それらのミスを犯したことを知っている知っています
一度か二度によって私は多分数百回を意味します
だから自分は今夜ああ、ああ償還、私は、ああ私は償還させてみましょう
私はちょうど第二のチャンスで1より多くのショットを必要とする原因

ええ、それは申し訳ありませんと言って、今は遅すぎるのですか？
私はちょうどあなたの体よりも欠けている原因
それは遅すぎるようになりまして申し訳ありませんと言うことですか？
ええ、私はあなたを失望させることを知っています
それは私が今、ごめんなさいと言うには遅すぎますか？

私はええ、ごめんなさい
ええ、申し訳ありません
ごめんなさい
ええ、私はあなたを失望させることを知っています
それは私が今、ごめんなさいと言うには遅すぎますか？

あなたは私がしたい場合、私は非難のひとつひとつのピースを取りますよ
しかし、あなたは2のために、このゲームには無実の1が存在しないことを知っています
私は行くよ、行くよ、その後、あなたが行く、あなたが外出して真実をこぼします
我々は両方の単語を言うと、これを忘れることはできますか？

それは遅すぎるようになりまして申し訳ありませんと言うことですか？
私はちょうどあなたの体よりも欠けている原因
それは遅すぎるようになりまして申し訳ありませんと言うことですか？
ええ、私はあなたを失望させることを知っています
それは私が今、ごめんなさいと言うには遅すぎますか？

私はちょうど私に戻ってあなたを取得しようとしていませんよ
私はちょうどあなたの体よりも欠けている原因
それは遅すぎるようになりまして申し訳ありませんと言うことですか？
ええ、私はあなたを失望させることを知っています
それは私が今、ごめんなさいと言うには遅すぎますか？


306c441

add


You don't need return here, just `Gender::Androgynous` without the semicolon.


develop (DavidGoldwasser) - i386-Windows-7-VisualStudio-12: Build Failed

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


Why does this field have the "default" access modifier? VisibleForTesting or some other reason?


Правильнее назвать "Update status.txt", ты ведь не добавляешь новый файл, а изменяешь существующий.


It looks ok


You took Yoinxs work and didnt give him credit. I see you can toggle this on and off which was something yoinx implemented. You just took it without giving proper credits. I can link you to the 2 separate commits if need be,


made changes to my repository

On Tue, Feb 16, 2016 at 12:53 PM, Emily Platzer <notifications@github.com>
wrote:

> Remove this cruft (1-8)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/courtm187/taxes/commit/65ff62cc63b79123bc75ab91208214578d4e68cf#commitcomment-16128527>
> .
>


Ha! I didn't even notice that I switched it the wrong way. Guess it is time for my next cup o' tea. I'll fix it. I'm monitoring Jenkins pretty closely now...


On the same machine. Yeah, I haven't checked (yet) whether it would track remote changes, that's not a use case I'm particularly interested in anyway.


Typo: `This should succeed.`


develop (DavidGoldwasser) - Win64-Windows-7-VisualStudio-12: Build Failed

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


Слишком длинное сообщение для коммита не удобно читать - постарайся в кратце описывать что именно изменилось. Но направление верное, молодец.


This too was taken from git without credits


@sameb , Can you clarify your concern?
Do you really expect this method to be used by third party code?

My initial implementation was following "keep backward compatibility, and make the code more obscure" (see https://github.com/cglib/cglib/pull/50#discussion-diff-44780894), however:
1) No one was defending "need backward compatibility for internal methods" position
2) I do not think it makes sense to add kludges to the code that no one uses except cglib itself


alright... I htought perhaps there could be a requirement to use google / gmail domains for copyright holder identification.


`REMOVE FROM picture WHERE position=LEFT OR position=RIGHT OR blurred=true`
:)


Btw, it's old commit. Refresh this page: https://github.com/python-rope/rope/pull/143/files
This module already has some comment at top of module.


@coreyfloyd I've changed it, but to be clear either method conforms to the best practice.  the difference is very minor. using `NSUInteger` as the **function parameter** means I need to cast it w/in the function. using a parameter type that's already printf-compliant means the function call does the casting for us. there are other places where i was doing the manual cast to `unsigned long` but was hoping to avoid it here by making the function parameter already suitable for printf, which is the sole purpose of this function.


What would happen if I call hasNext twice? Looks like you keep going forward as if next() have been called.


yep this is mine as well. you like stealing from people dont you? Your limiter.h file is exactly how I have mine set up.


mono_get_lmf_addr () is called from a _lot_ of places, some of it might be perf sensitive.


This is Yoinx's work you stole again. Nice try.


Oh I see, this was in a comment so I didn't change it.  I can put the erb in anyway, though


comment


@konklone makes sense - maintaining information at the speed of the technology changes is indeed difficult.

I think your language is excellent.


Now that I actually look at the test, this description isn't accurate. The account still has plenty for the reserve after the offer is *placed*. It would not have enough for the reserve after / if the offer is *taken*.


This also is not your work. You didnt just steal from us, you stole from other devs. Shame on you


GitHub pages change


Does this require that the user calls hasNext() before? next() should work even if you never called hasNext() even when in practice this would be a sign of a bug in the calling code.


This could be inline:

```elixir
def search(nil, _), do: nil
```
The same for the next two 'search'


@sameb, this is an internal interface. Why on Earth one would use this interface?

It is used in
```
private static final EnhancerKey KEY_FACTORY =
       (EnhancerKey)KeyFactory.create(EnhancerKey.class);
```

`private`.

Do you really want to keep those interfaces "just in case someone uses it"?


Seems, I added some lines


Octree (DeadParrot) - i386-Windows-7-VisualStudio-12: OK (2098 of 2098 tests passed, 9 test warnings)

 * 6 tests had: AUD diffs.
 * 2 tests had: EIO diffs.
 * 2 tests had: ESO small diffs.
 * 1 test had: SHD diffs.

![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)


This could be even more concise by making use of the `trailing closure` feature:
```swift
let filteredArray = moneyArray.filter { $0 > 30 }
```
Personally I think this make the code even more readable as it gets rid of unneeded brackets that only hinder reading the actual filter logic.


Not your work either


you can break these in two searches and use the when clause in the first one.


@americool Try `let(:factory_user) { create(:user) }` instead of `let(:factory_user) { build(:user) }`  NOTE: `create` not `build`


letter spacing is a bit far


cool, thanks

On Tue, Feb 16, 2016 at 1:56 PM, gcapalbo <notifications@github.com> wrote:

> yeah see the internal confluence page (overview of celery) - at the bottom
> in the convention section we have some general guidelines
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dimagi/commcare-hq/commit/1ad11d63d3427f296fb97353d0eeae5931c62dc9#commitcomment-16126885>
> .
>


Yes, that would probably be slightly better.


Хэндлинг исключения это то, что я передаю туда в качестве сообщения? Что не так с ними(кроме вероятно большого кол-ва ошибок в англ языке...)?

> файл log.txt данный примером в задании не парсится так как там есть нечитаемые символы.

У меня парсится все хорошо. Не знаю в чем проблема. Брал log.txt файл с сайта. Все работает. Можно подробнее о проблеме и желательно то, что пишет Loger?

> gson.fromJson(new FileReader(filePath), Messages[].class)

Оно просто считает одну строку или весь файл? Из исходников gson не понял как оно поведет себя.
А так у меня же несколько методов для чтения. Каждый метод в итоге обращается к методу readJsonStr для избежания повторов кода.


@faheemzunjani Wuthering Heights is already present in the list. Removing it,


It works here:

```
$ build/systemd-resolve -t TLSA --raw _443._tcp.fedoraproject.org |wc
      1       3      32
$ build/systemd-resolve -t TLSA --raw packet _443._tcp.fedoraproject.org |wc
packet: resolve call failed: All attempts to contact name servers or networks failed
      1       3      32
$ build/systemd-resolve -t TLSA --raw=packet _443._tcp.fedoraproject.org |wc
      1       4      82
$ build/systemd-resolve -t TLSA --raw=payload _443._tcp.fedoraproject.org |wc
      1       3      32
```
It even does what you wanted, i.e. treats `--raw packet` as two arguments.


This may be nit-picking, but Swift is generally written with an uppercase s. This is true for multiple occasions within the style guide itself and also Apple's own documentation.


There is nothing there, the remaining (desktop-only) resources are all in 'desktop/resources'


What is min_duration for? GetThroughput will return 0 because the route doesn't exist, right? Then there will be an overflow. Maybe just use 1Mbps to calculate the duration?


Nice. I had this in mind but I forgot ^^


not your code


Put sleep outside if and else.


Making use of `trailing closures` would also make this line even more readable.


What's the warning about? Which parameter? Without pulling this branch into my IDE, I don't see where the warning comes from.


s/him/it/, since you're referring to the user's _balance_ going below the reserve.


Waiting for a new build with the fix. Thanks for this!


when you make a commit do it in present tense... "Add advertisement for abby books"


I squashed 16e3c265ca61f308525e181cca59525e09673675 and 16bf0ea4deca7d0288946765eb97039698859833, and opened #257.


@dblotsky yes, of course, I am currently working to address them. Thanks for the review!


looks like this script has become stale or maybe didn't work with the latest changes in the first place.  I've opened https://github.com/openshift/origin/issues/7356 to address and reverted the script to get things copied for now after spending a bit trying to fix it.


I think before the cnt was working, it just didn't update the value in $display properly because it was async, it should be totally fine to keep this sync and just look at the output graph of Q in the simulator


el hack esta genial, por que no lo usas en el logger? es porque en este momento (setup) no está definido el stackId?


Ну в принципе не плохо всё конечно, но не до конца. Отчета (списка) нет. И все суммы, минимальное и максимальное надо было сделать через функции. А так всё норм. Надо доделать =)


:clap: Many thanks


Probably should suffix with `Exception`
And likely needs some more constructors with javadocs.


Yes


I notified upstream anyway: https://github.com/lwindolf/liferea/issues/317.

Note on github in general: generated tarballs contain `/archive/` and the real manually uploaded releases contain `/releases/`.


could probably add a metatable link for splashlib.new() → splashlib()


I'm wrong ? If we cancel kick, as there is a break statement, the login packets are not fully sent so the player doesn't connect. I didn't have issue with kick in the if statement, are you sure there is a problem with it ?


The structure below with three separate for loops is a bit less obvious than just checking over the outer and inner indices separately. Is there a performance gain from doing it like this?


Good catch. Yes, if dispose() is called during an async operation, but flagEndAsync executes right before the mDisposeAfterAsync flag is set (314), then we'll fail to call dispose() again to actually perform the cleanup. I observed this bug in the app after adding a few strategically-placed sleep() calls.

I believe the most straightforward solution is to introduce a lock object so mAsyncInProgress and mDisposeAfterAsync are accessed atomically. I've confirmed that this eliminates the problem. New patch forthcoming...


Tried appending `--`, still no dice (and this is on git 2.3.5)

```diff
diff --git a/spec/integration/protected_branches_spec.rb b/spec/integration/protected_branches_spec.rb
index 402be38..ad1759c 100644
--- a/spec/integration/protected_branches_spec.rb
+++ b/spec/integration/protected_branches_spec.rb
@@ -53,7 +53,7 @@ describe Overcommit::Hook::PrePush::ProtectedBranches,
   end

   shared_context 'local branch up-to-date' do
-    before { `git rebase --keep-empty origin/#{remote_ref} > #{File::NULL} 2>&1` }
+    before { `git rebase --keep-empty origin/#{remote_ref} -- #{remote_ref}` }
   end

   shared_context 'ProtectedBranches enabled' do
```


Considering `isDone` could be more complicated, we should consider adding a `setDone`.


Nice job on the testing specifics :metal:

Parameter name `n` isn't very descriptive, and although the code looks "tidy" it's not super obvious what the intent of the code may be. Refactoring at this point will be fairly easy (Green to green) since you have good test coverage.


this function is a stub atm


consider `splash:setDone(true)`


Update with repo info


yup that worked...strange nobody else has had this problem...

On Tue, Feb 16, 2016 at 3:09 PM, dothework <notifications@github.com> wrote:

> @americool <https://github.com/americool> Try let(:factory_user) {
> create(:user) } instead of let(:factory_user) { build(:user) } NOTE:
> create not build
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/americool/bloccit/commit/585b631884e34638e25e1d8ce8225a02ead0ad68#commitcomment-16129003>
> .
>


This can be pushed to a file in `splash/` now if you like.


Sorry, perhaps I wasn't clear: the `--` should go *after* `#{remote_ref}`, i.e. at the end of the line.


I agree with you, let's use `fetchFromGitHub` instead.


Also, make GetThroughput to return bool indicating whether the throughput can be found, just like FindRoute. Otherwise, you will initialize the table entry to be 0 when the throughput doesn't exist. This is unexpected.


I think this would be better placed in `splashlib:draw()` in case someone resizes the window mid animation.


Sorry about the disruption caused by the replaced tarball!!! Overhead for updating a release notifications everywhere (or issuing another release) was so great I would have not been able to fix it before 1-2 weeks later so I choose to dirty-replace the tarball :-(


Some example psuedo-code:

```java
import static org.spongepowered.api.text.template.TextTemplate.*;

TextTemplate template = of(
    TextStyle.ITALIC, "Welcome to ", arg("serverName"), " ", arg("playerName")
);

void onPlayerJoin() {
    Map<String, Object> params = new HashMap<String, Object>();
    params.put("serverName", Text.of(serverName));
    params.put("playerName", Text.of(player.getName()));
    template.send(player, params);
}
```

Alternatively:

```java
import static org.spongepowered.api.text.template.TextTemplate.*;

TextTemplate template = of(
    TextStyle.ITALIC, "Welcome to ", arg("serverName", String.class, s -> Text.of(s)), " ", arg("player", Player.class, p -> Text.of(player.getName()))
);

void onPlayerJoin() {
    Map<String, Object> params = new HashMap<String, Object>();
    params.put("serverName", Text.of(serverName));
    params.put("player", Text.of(player.getName()));
    template.send(player, params);
}
```


stub


Should we specify some kind of formatting here? Personally, I think that
```swift
if let
    widget = widget,
    url = options.url,
    host = options.host
{

}
```
makes statements like this more readable. This is true especially when unwrapping a lot of optionals in the same `if let`. Note that this would also make debugging easier, as a developer would be able to step through each assignment and see exactly which variable is `nil` - if any.

Are there any downsides to this approach that I forgot to mention?


Leap year?


I've pointed this out to the curriculum team.
Glad we got it.

On Tue, Feb 16, 2016 at 3:16 PM americool <notifications@github.com> wrote:
yup that worked...strange nobody else has had this problem...

On Tue, Feb 16, 2016 at 3:09 PM, dothework < notifications@github.com > wrote:

> @americool < https://github.com/americool > Try let(:factory_user) {
> create(:user) } instead of let(:factory_user) { build(:user) } NOTE:
> create not build
>
> —
> Reply to this email directly or view it on GitHub
> < https://github.com/americool/ bloccit/commit/ 585b631884e34638e25e1d8ce8225a
02ead0ad68#commitcomment- 16129003 >
> .
>
—
Reply to this email directly or view it on GitHub .

Not using Mixmax yet?


Пользователь может создать несколько сообщений с одним и тем же id. такого быть не должно, поэтому не доверяй пользователю создавать id :)


Не ясно в каком виде вводить дату когда пользуешься программой


This commit is to address the following exceptions thrown by react v0.14:

![screen shot 2016-02-16 at 12 18 59 pm](https://cloud.githubusercontent.com/assets/6956310/13089849/81364b4e-d4a7-11e5-9a2c-7f23d5633a48.png)


De acuerdo , añado modificaciómn


OK.
I thing i made a mistake anyway ...

import os
import subprocess
try:
  name = os.environ["PLUMA_CURRENT_DOCUMENT_NAME"]
except KeyError:
  exit(1)

f, ext = os.path.splitext(name)
nxt=''
if ext == '.c':
   nxt = '.h'
if ext == '.h':
   nxt = '.c'
if nxt:
   subprocess.call(['/usr/bin/pluma', f+nxt])


Okay, I can help. Where is your branch?


It looks like you wrote a lot of code before your tests were passing (calcTax versus taxCalculator). Be sure to use the red-green-refactor cycle to write test driven code.


LGTM!


А зачем второй раз устанавливать зарплаты?


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


thanks for this comment
www.e-mobl.com


Pushing to master


если ввести не число (например, лишний раз нажать Enter) то программа падает с NumberFormatException


release-5.x (lefticus) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


not your code


А в этот файл хорошо бы примеры всей функциональности, м?


I think that there is a lack of records in any reader it would result in a NullPointerException in the if above (```if(!record.getTarget()...)```) and not in a IOException. An IOException here means that sothing really bad has happened).


release-5.x (lefticus) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

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


Don't think this would be  a fix.

Must check it. Tagging it as bug


Whoops yeah didn't mean to do that. I'll fix it / document / polish soon, just wanted to get the concept out there to see what people thought.


This is usually onelined.


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


Right right right okay cool with it :+1:


mejor `frameworklogger.setEndpointId(stackKey)`
en el logger del framework va: `logger.WithField("StackId", helper.stackId)`
ojo: `frameworklogger.Setup` debe copiar el logger, si no se sobrescribe el stackId para todas las instancias de la fabirca


Is this used? Can we remove?


В принципе всё ок. Только я имел в виду в задании, что функции - это должны быть именно _функции_, а не просто посчитать.
Надо доделать функции )


The postprocess.vertex is only used after rendering a 3D view so that wouldn't help. These 2D post process effects cause additional render views but they are only 2D without any scene entities and the used shader is texture_color.vertex or texture.vertex. It should be enough the flip the UVs there. Maybe id Software did another .vertex -> hlsl generator similar to the glsl generator and approached this by replacing the tex2D command with a function like float4 tex2D( sampler2D sampler, vec2 texcoord ) { return sampler.Load( float2( texcoord.x, 1.0 - texcoord.y ); }


You can't (portably) use bitwise operations on `bool`.  You need to say
```C++
equalOuter = equalOuter && /* other condition */;
// ...
```


I tried both actually (though I didn't mention it); same result.


No need to allocate an array. Just use 1.0 / stats_[client_ids_[i]].throughput in the second loop. Also, even if you need an array, vector is a better choice.


Interesting choice. I always wanted to do this too (name them .twig), but we were catering to people whose text editors wouldn't open automatically open the files. I think .twig is better.


This doesn't generate CSV files. That's other code elsewhere not committed. OP's wrong. :disappointed:


nice!


why?


Welp, I'm out of ideas. No idea what's going wrong here :frowning:


ditto!


зачем setZp возвращает что-то?


"Specifying"


твой геттеры-сеттеры - не работают!


I prefer "written to" instead of "dumped to"


Ok but the issue comes from the Vhost used for disabled sites.

Closing it as we will fix it where it should be


For casting, double(round_interval_)/client_ids_.size(). The associativity can sometimes mess up in your approach.


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


Again, double(round_interaval_)/...


`hack/copy-kube-artifacts.sh` was that script copying files needed for testing.


Fix for
<img width="698" alt="screen shot 2016-02-16 at 12 32 15 pm" src="https://cloud.githubusercontent.com/assets/5640772/13090074/bc5336be-d4c1-11e5-9d40-d034c79d9431.png">


I see now. I used did a "push -f" to overwrite this branch with the best method I could come up with:

 https://github.com/doanac/libsoc/commit/5aa918c6c422254a4bcaed1f71bb287df4834905


This should live within an `it` block


Move free above return statement


using abs would cause undesirable results on samples which are of type acc (double). The bug was detected with negative sample values.


getting *rid of verify_globals!


and now pushed to the repo..

nodejs-v0.12.x-legacy takes over /usr/bin/node & /usr/bin/nodejs


whats so important about this?


if (settings.autorun !== false)


Look good Jed, keep up the good work!


i dont know you tell me


is it  `> >`?


Instead of sorting in the calling code and throwing an error here if the sorting was wrong, why not sort the reads here, thereby obviating the error-check?


Yep, that's exactly what happened. This works now.

On Tue, Feb 16, 2016 at 2:19 PM, Nick Skelsey <notifications@github.com>
wrote:

> Hey it looks like you did not include a file when you made this set of
> commits. When I load prototypes/bubbleChart/index.html this error is thrown:
>
> http://localhost:8000/data/data_master.csv Failed to load resource: the server responded with a status of 404 (File not found)
> d3.min.js:3 Uncaught TypeError: Cannot read property 'length' of undefined
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/codefordc/school-modernization/commit/ba82c23cdb1f449e66529a2c7ac3cc4f9a23e8da#commitcomment-16127526>
> .
>


Lets have `calcTax()` return cents... in this case 1.70


Do you mean that I do not have to do this?

```
class MyRule
  include Rulezilla::DSL
  include MyRuleSupport
...
end
```
if there is a module:

```
module MyRuleSupport
end
```
already defined?


This whole return statement needs to be indented two spaces to the right. You know you can do `npm test` locally (which runs tests + nsp (node security project) + standard), or simply `npm lint` if you only want to check coding style.


А всё остальное даже вроде ничего так ))


auto it_o = spawned.begin();


@floam, I’m unable to reproduce the issue you're describing. If I type `brew d` into fish and then hit the tab, I’m offered the following suggestions:

```
deps                       (Show a formula's dependencies)
desc            (Summarize specified formulae in one line)
diy  (Determine installation prefix for non-brew software)
doctor                    (Check your system for problems)
```

This is true even when I’m in my own home directory, with `Desktop` (and `Documents`, and `Dropbox`) as valid completions.

I'm running version 2.2.0 of fish, under Mac OS 10.11.3.


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


TestCustomCommand (jasondegraw) - x86_64-MacOS-10.9-clang: OK (2072 of 2073 tests passed, 546 test warnings)

 * 547 tests had: AUD diffs.
 * 1 test had: EIO diffs.
 * 1 test had: ESO big diffs.
 * 1 test had: MTR big diffs.
 * 1 test had: ZSZ big diffs.
 * 1 test had: Table big diffs.
 * 1 test had: ERR diffs.

![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)


Все баги и недочеты устранены.
Остался вопрос с возможной проблемой чтения стандартного input файла с сайта. У меня такой проблемы нету.


What's the reasoning for `max` versus `min`?


No that was just my first hit


the old logic for this is super complex and seemingly does a simple task... makes me nervous


Done. You need to remove the assumption that  hasNext() and next() are going to be called in the usual manner. You cannot assume that this is the case.


hey @NoahCarnahan i think these functions will be for you to implement unless i'm mistaken


Хороший и лаконичный код, зря не взялась за дополнительные задание. Мне кажется 4 не твой потолок, но в целом решай как знаешь.

Итого: все сделано на отлично - ну, поправить id чтобы не задавал пользователь и ввод даты не нужен (можно брать ведь текущее время) или тоже понятнее сделать. Доделаешь сможем поднять оценку. Коммиты красиво названы, первое задание тоже хорошо.

Принято задание 1.1 и 1.2.


Of course this isn't documented.  I've opened [an issue](https://github.com/cloudfoundry/cloud_controller_ng/issues/537)


AFAICT, EmitUtils is public, and hash_code is a public static method.  So this is a public API, not an internal method.  Changing the method signature of a public API is an issue.


overhang with a 'g'?


resulting


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


"if the blocks"


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


for(var i=0;i=10;i++){


I guess that's fine for now, thanks. lgtm


Cambierei `reiterano` con `permettono di iterare`


overhang etc


No reasoning, just need to give a number and I opted for the max; happy to change to the min or something else that you feel is appropriate.


-what


So what is your take on that?

Note: `public` != `published`. See http://martinfowler.com/bliki/PublishedInterface.html


(8) définir la vue html d'un premier jeu de données


`sugli` invece di `gli`


Ahh sorry about this.  Get so used to 64 bit everywhere I should have realised that the tests would need to be adjusted.


`itera`


Ah, I see the comment above this (collapsed by the diff view, so I missed it) says this is internal and public only for classloader issues.  So I agree it's OK to change it.


Thanks, fixed. (Does seem to work however)


I think we need to push a 0.4.1 which reverts the osx sed thing.


@Guillaume351 Can you reproduce this error?


Are there any comments on EmitUtils describing that users shouldn't use it?  If not, how is a user supposed to know not to use it?


> Really I think the fact you are specifying the class you will accept precludes the event that you would be returned an NSNull, correct?

It's not `NSNull` I'm worrying about, but `nil`. `NSNull` is actually already handled by `!isKindOfClass` so i can remove the special error handling.

> Also its not really a type… its a class (you can't pass a protocol or a primitive)

Fail enough, but this was already enforced by the method signature. I'll change the name to match.


@jasonsurratt This PDF is 297.7KB on my machine, causing the test to fail.  Is that a bad thing, or can the range be adjusted a little?


Repos forked from hof-example-form are polluting the BRP GA tag analytics - see UKHomeOffice/hof-example-form#18

This patch removes the hard-coded tag script and includes only if the tag id is exposed as a local.


> Объект Date удалится сразу после создания

Не удалится.

В классе Loger есть поле
`private Date date;`
В конструкторе при создании логера создается объект
`date = new Date(System.currentTimeMillis());`
При вызове
```
    public String getTime() {
        date.setTime(System.currentTimeMillis());
        return date.toString();
    }
```
Согласно спецификации
```
    public void setTime(long time) {
        fastTime = time;
        cdate = null;
    }
```
Изменится содержимое объекта Date. После чего вызовится toString() и вернется строка для вывода.
Объект Loger за все время ни разу не пересоздается(собственно внутри него соответственно и Date тоже)
Преимущество есть.
Или я что-то не понимаю?


great thanks


too small


Also, I "forot" how to spell "forgot"


The problem is solved by explicitly require the library 'subr-x'.


Qui cambierei il discorso perchè secondo me in Italiano si capisce poco:

```
// `range` sulle stringhe itera sui singoli codici Unicode
// della stringa. Il primo valore rappresenta l'indice posizionale,
// il secondo valore rappresenta la `rune` Unicode in sè per sè
```


How about `final AlignmentBlock rightOverhangingBlock = left.referenceEnd < right.referenceEnd ? right : left;` followed by `System.arraycopy(rightOverhangingBlock.bases . . .` thus eliminating the `if . . . else` and a few lines of code?


left block's call


I wonder why there is a difference. I think I was seeing 200KB when I first made the test and gave it a 50KB buffer on either side. I'm re-running it locally now. Can you post your PDF?


As it turns out I actually did have this check in PyEMD a few versions ago, but I took it out for speed. Never really tested if it had a real impact though, so I'm open to putting it back in.


unintuitive


The errors were from coefficients / bugs. They have since been minimized (after the report) to quantization issues of the reference bit depth.

The link was provided as a means to test transfer curve.


hai dimenticato una `<`


It could be fine if you could add some test result samples. You can find some examples from other xunit tools in src/test/java and src/test/resources.


value=None

No spaces between = sign


Glitrende. Takk for innspill! Er 4 -> 2 mellomroms innrykk noe nytt etter overgangen fra pandoc? Mener å huske man trengte 4 der? Jeg kan oppdatere tilsvarende i nye og gamle oppgaver etterhvert :)


I don't have any error, only if I want to cancel a player kick (if, for example, he has VIP and can join even if server is full), I can't do it anymore now.
Before, the kick return true if it was successful and false if it was canceled for some reason. In a plugin, you could cancel the kick if reason = "disconnectionScreen.serverFull" , but now even if we cancel the kick, because of the "break;" the login process stops so the player is kicked


It's missing the var.


Nice job :metal:

Now that you have good test coverage, you can refactor that function!


Location.php file


@coreyfloyd actually to address both of these, about about `instanceOfClass:forKey:error:`? pretty unambiguous there, with the only caveat that it's covariant (i.e. returning a subclass is also valid, which in cocoa terms is a "kind of" class)


either both continuous or both contiguous.  I prefer both contiguous.


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


That would certainly be possible; it might be nice for a bunch of similar tests to share more specific utility code.


@normanmaurer That should also work. :+1:


Is this right?  You want the id where it's already `true`?


I think a mere sanity should be enough.
Google produces no results for "import net.sf.cglib.core.Customizer".

Well, I'm fine if we name it as "backward-incompatible change" and release as 3.3.0, however it looks like a hair splitting.

I think the main aim of keeping `EnumUtils` `public` was to allow usage of the class across different cglib packages.


In any case my intention was in no way to halt progress here, it was just a
good context to do the proposal in.
15. feb. 2016 20:03 skrev "Razzeee" <notifications@github.com>:

> I like that idea. @martijnkaijser <https://github.com/martijnkaijser>
> will also.
> @fetzerch <https://github.com/fetzerch>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/tamland/xbmc/commit/78263fe05bb8b2769c7cf98876cfe5d6e42d2601#commitcomment-16102616>
> .
>


porrra john


Ah yes, it's a little quirk of snescom... as you see the offsets are normally within range of a short branch (<=127) but snescom subtracts 20 bytes of "security margin" (that code part is labeled "// ugly hack" btw :D ) in case it needs to substitute some code within the range.
That said, as long as a warning is issued instead of an error, snescom automatically substitutes the (conditional) short branch with a "skip if false; branch long" so it works anyway.
Personally on one of my machines I got annoyed and disabled the 20 byte margin completely :D


ricorsione


Do you have a github version of hxcpp?


Fixed.


Maybe. How did you feel about the text for the other platforms?


I think that's better left to someone who has interest in investing time to keep such old versions working and is able and willing to test with them. I don't, and FWIW note that even the extended support for 1.6 ended almost a year ago and 1.7 is already EOL too for 3+ months (meaning not even security or data loss bugs will be fixed). https://www.djangoproject.com/download/#supported-versions


OK, вопросов у меня тут нет


Word. I can't imagine it would make a huge difference, but since EMD is the main bottleneck...who knows. I'm definitely going to leave it in for the rest of the macro refactor so I don't get burned by mis-sized repertoires. I'll benchmark it after and see.


`_valori di ritorno multipli_`


Remove above line


че за херь?


I will fix these shortly, working on stream-stats stuff right now.


This is a hack that works around a CSI bug that breaks global-variable setting.

When a script gets run from csi.exe, there's a global variable called "Args" that contains the script arguments.

But when you build a script interpreter, you don't get Args by default.  Instead you need to have a global variable for the script.

The correct way to set global variables is broken, so this is a hack that defines a global and then sets it by inspecting the state.


How is this used with Swift?


Inizierei con
```
L'indicazione `(int, int)`
```


Before committing, I checked the English language. For this language no action is needed.

Today, I have checked some French and Italian examples. In these languages the suggested fix would apply. However, I can hardly think of a scenario in which sentences can start with a newly added word... By the way, right now, these languages do not have any additional words in spelling.txt.

Right now, I do not think it is necessary to include the rule in the superclass' function.

But it would be pretty easy and probably would hardly cause additional calculations:

  protected boolean ignoreWord(List<String> words, int idx) throws IOException {
    String word = words.get(0);
    boolean ignoreUncapitalizedWord = idx == 0 && ignoreWord(WordUtils.uncapitalize(word));
    return ignoreUncapitalizedWord || ignoreWord(word);
  }


Done.


Why did you do this, by curiosity? When it comes to tracking down griefers, extra info like this could have helped alot.


Riga 21 e 22 da cancellare


Missing a test to ensure avoidance where SSH is already  enabled.


depends


Jankily added the chart images


highlightMarkersByEditor is not used, remove


> Er 4 -> 2 mellomroms innrykk noe nytt etter overgangen fra pandoc?

Ja, det stemmer.

> Mener å huske man trengte 4 der?

Det stemmer også.

> Jeg kan oppdatere tilsvarende i nye og gamle oppgaver etterhvert :)

Det er ikke kritisk, men synes 2 mellomroms ser bedre ut. Utfallet blir det samme, bortsett fra ett tilfelle (som jeg mener å huske jeg søkte etter):

```
+ listepunkt avsluttes med to nye linjer

    ```js
    function fireMellomromBlir (tolket) { /* som kodeblokk */ }
    var ergoFårMan = 'med ```js i kodeblokken';
    ```
```


`OverlappingSection` would be better


This is an internal method. `ARTDataEncoder` is not used by end users.


Rename to searchHighlightPattern for explicitness.


Now that the method is public.
Do we want to add a null check?


Ah, so you're doing that server-side. Fair enough. For some reason I had assumed you'd parse and render Markdown in JavaScript, the way reveal.js does. But this is perfectly fine too.


Nice job :metal:

There are some whitespace issues in main.js, but great job getting this functional.


IMO these comments make the code less readable because the method names are already explicit.


I don't understand how this commit accomplishes what it actually says it does... my original comment was that for a set of `n` promises we'll recurse `n` times (once per promise) when we really only need to recurse once at the end...


Fixed at c3ca53cbfb8508e1551e5a312a72152cf8bca82c.


Something like [that](b3fa24c9c883645f898b9d120f83474741a8c66e)?


Unfortunately iam using mingw that came with Qt package, and for while it doesn't supports C++ 11


Related to the previous story, you only want to operate against applications where SSH is already disabled?


The help string probably follow the standard Markdown filename convention, using the `.md` extension.


I think npm failed to update eslint correctly (it probably forgot to remove an old version somewhere) and that caused eslint2 to fail with warnings coming from an outdated eslint-config-xo. When I changed it, due to the failure and the eslint 2.0 release notes, I assumed the change in the way eslint load dependency had changed and was breaking this.


Missing a test to ensure avoidance where SSH is already disabled


not `store__pootle_path`?


Takk for info! Enig i at 2 mellomrom ser bedre ut i lister. Da får man en mer naturlig flyt :)


cruft


ой, а зачем тебе countEmployy параметр нужен?


Would have taken ~2 hours in Ruby, now takes ~25 seconds.


Searching through all google's code, I do see some usage of EmitUtils.  Usage is limited mostly to begin_method, null_constructor & constructor_switch... but I view that as an indication that users will do what we let them do, and absent any instructions that they shouldn't do it... there's no reason they shouldn't have been doing it.

Since EmitUtils was public and *not* in any "internal" package or documented as being internal-only, then it's part of the public API and we shouldn't change it.

Is it really necessary to make this backwards incompatible change?


Nice job :metal:

Be sure and remove cruft before committing.


Sorry, found one more.
"Make the printer"


I see your point so in the save dialog box test for ``if (pRTS->m_provider != "Local")`` ?
Would this suffice ?


класс - в порядке ))


Rename event name according to globalState name change


Opened https://github.com/flynn/flynn/issues/2463


Hows does this show the news on the display? can the link be changed to any RSS feed? i can't seem to get it working.


Probably.


Thank you for your contribution. However, the download URL for the latest version (`http://downloads.findingsapp.com/findings`) redirects exactly to the same versioned URL (`http://downloads.findingsapp.com/Findings_3077_1.3.2.zip`).

The versioned casks are currently more preferred, and `:latest` should be used only if there is no other official versioned URLs available. So there is no point in these changes at the moment.


В принципе нормально, но я бы разделил нахождение каких-то параметров и их печать.


What about making this into a constant? Something like `Quarter.MAX`?


I think the menu should be available to at least the host/admins of the server. Like say if someone requests a ride type, they could be able to use the object selection to get it. Also with the console being available, it would increase potential for people to start working on scripting mods etc. Personally once I get home from work tomorrow I'll be installing 8.1 on a virtual machine and then downloading vs2015 hopefully and start working on more console commands along with maybe an option to execute scripts kinda like batches etc. I'm rambling on from my original question, but yeah was there a reason this was removed? Does the debug commands cause mp glitches?


Because its far too complicated to manage and also unnecessary. For example what is the point in allowing raise water but not lower?


Remove any debugging logs before comitting.


This shouldn't have ng-controller


>Is it really necessary to make this backwards incompatible change?

Can you suggest which change would make you happy?

I can easily copy&paste `EnumUtils` to `EnumUtils2`, however that would make me a sad panda.

I can easily make new customizer `extends Customizer` and add `instanceof Customizer2` here and there (as I did initially). However it looks awful for no benefit.


Reuse input var once you get.


Should getters replace using computed properties to store references to store items in components?


Nice job :metal:

Whitespace could be condensed in main.js


This was just a debug message. This is for test purposes only.


ok i'll make hte changes now


5420-OSCM-Docs (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1536 of 1536 tests passed, 0 test warnings)

![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.71%25-green.svg)


Yep. I agree


Rename to highlightRanges since it support no timed out highlight(not necessarily frash-ed).


You're moving the function argument position of the line above, this line was aligned, and now isn't any more.


Ah, neat. I didn't know, Node.js is all new to me. I'll send one more
commit to fix the lint errors.

On Tue, Feb 16, 2016 at 9:30 PM, Lars-Magnus Skog <notifications@github.com>
wrote:

> This whole return statement needs to be indented two spaces to the right.
> You know you can do npm test locally (which runs tests + nsp (node
> security project) + standard), or simply npm lint if you only want to
> check coding style.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/hintjens/prebuild/commit/31b3687530bb9aff62d751c3e7536d20288c43a9#commitcomment-16129476>
> .
>


@wolfreak99 they are disabled because they break the game, like many other things such as placing track designs. Until they are made to work with multiplayer correctly, they have to remain disabled.


LGTM


fixed in 1dcb34cd3677772f8b84b812baa7bdf2c2d07cd4
thanks


It do, Qt5 uses c++11 already. You're probably missing this in your .pro: `QMAKE_CXXFLAGS += -std=c++0x`
And your mingw having gcc > 4.6 has full c++11 support.


Use $USER environment variable.


TeamCity Open-source Libraries :: ConfigInjector :: CI [Build 2.0.516.0](http://build01:8888/viewLog.html?buildId=14278&buildTypeId=ConfigInjector_CI) is now running


I didn't understand everythind yet, but in my opinion logically `$htmlResult` cannot be `self::button` _and_ `self::text` at the same time(!?)


This transformation should be implemented in similar way like `sumo_store_riak` (https://github.com/inaka/sumo_db/blob/harenson.201.add-checks-for-all-sumo-data-types-in-tests/src/sumo_store_riak.erl#L398). Because conditions comes in different forms, and in this case this only evaluates one of them (`{field_name(), operator(), field_value()}`) and in the first level of the list, remember, conditions might be nested. So the recommendation is to follow the same approach as riak store pointed before.


TeamCity Open-source Libraries :: ConfigInjector :: CI [Build 2.0.516.0](http://build01:8888/viewLog.html?buildId=14278&buildTypeId=ConfigInjector_CI) outcome was **SUCCESS**
Summary: Tests passed: 38 Build time: 00:00:36


The change that would make me happy is preserving the existing method signature.  You can add new methods, and you can change the implementation of existing methods -- but changing method signatures is a backwards incompatible change and makes life harder for users.

EmitUtils2 is not necessary and a fairly absurd idea.

Is it not possible to leave the existing signature and make its implementation delegate to your new method?


when editorElement is not attached endRow become undefined.
Better to accept callback to be called when editorElement is attached?


Octree (DeadParrot) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (0 of 0 tests passed, 0 test warnings)

![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)


perhaps call $predicate only once to speed up and avoid side effects.

sometimes they are not expected the predicate is called twice for same item.


Octree (DeadParrot) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


See comment below about where the responsibility for sorting reads should be.


Let me review this today...


Octree (DeadParrot) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: Tests Failed (0 of 0 tests passed, 0 test warnings)

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


Octree (DeadParrot) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: Tests Failed (0 of 0 tests passed, 0 test warnings)

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


this is pretty neat, but I would really prefer using pure functions as before. If that's too hard to put in, then this is a fine base abstraction


@srhise it depends on whether the computed property is specific to that component. Getters are intended for reusing the same state retrieving logic across multiple components.


dfghj


It seems this this line is the reason for switching from `ReadsContext` to `Stream<GATKRead>` elsewhere.  Do you do that because after `filter(readFilter)` you have a stream and you don't want to convert back?  I see that the old code avoided this by making filtering the responsibility of `counter`.  IMO working with `ReadsContext` is more natural even though it requires putting the filtering in the counter, which I agree is less natural.


pnhhn


Nice job :metal:

Now that you have good test coverage, you can try your hand at refactoring. That nested if statement madness might be a good thing to detangle...


This change requires you to add a pageTitle to each ui router route when porting existing modules.


I can't recall now, but I we can change it back to see if it still works. I think I did that as part of a debugging process and I'm not sure if it can be changed back.


You can replace this with `1.3.2,3077` to use string interpolation in `url`:
```ruby
version '1.3.2,3077'
```


No worries, I just released https://github.com/davidchin/react-input-range/releases/tag/v0.6.2


Thanks, @emacsway , I suspected something like this and that's a good explanation. Glad you have work all this work.


It's not that it would be too difficult to do it's that I thought this was something worth adding in general, not just just for templates, but as a convenience when the type of a given TextElement is unknown and there for you would have to do a series of `instanceof` checks to find the correct method to add it to the builder.


Using the `version '1.3.2,3077'`, the correct download URL should be:
```ruby
  url "http://downloads.findingsapp.com/Findings_#{version.after_comma}_#{version.before_comma}.zip"
```


1) Customizer is used in lots of places: hash_code, hash_array, hash_object, not_equals, append_string, append_string_helper,...

cloning all those function is not fun. It is essentially equal to `EmitUtils2`.

2) New method is required since "existing Customizer" interface had no way to convey `public Type getOutType(int index, Type type) {` information.

I think we are wasting time here.


The current sha256 is `e76a1c67b0dfaf5807e45c2d0276385afb875612f2693b945cf1e97939b2256f`.


załadowanie pliku z klasą nie powinno mieć efektów ubocznych


Done!

![image](https://cloud.githubusercontent.com/assets/138494/13091019/0ddb93d2-d4ae-11e5-9cda-5a26a673de2c.png)


ok, based on explanation in email, this looks correct


I considered Javascript, but this was simpler to implement. :)


ack


Of course, good catch.  Fixing it.


Simple == good.


Hey @phippuuuu love the concept!

Just a heads up, we’re definitely going to want you to eventually switch to using `addEventListener` rather than setting `window.onblur` and `window.onfocus` here. Let us know if you need any help.


maybe something a bit simpler will do. Totally untested:

const char *hostname = pn_connection_get_hostname(connection);
if (!hostname) return 0;

const char *end = strrchr(hostname, ':');
int len = end ? end - hostname : strlen(hostname);
char *retval = malloc((len + 1) * sizeof(char *));
if (!retval) return 0;

strncpy(hostname, retval, len);
retval[len] = '\0';
return retval;


interesting, i didn't know that. I think that there are a couple of locations where I use this, especially in the new code in the view tool


@aoifec: I've updated the doc now. Thoughts?


This leaks implementation details. I suggest making `index` a parameter of `SGEN_ARRAY_LIST_FOREACH_ENTRY_RANGE`, or keeping an index in this function.


obsługa błędów – co jeśli aktualnie nie gra żadna piosenka?


you already included it in commit 5fd6ccb97086d62e350eab63892240e006675035 .


See if the alternative above works for you.


nom, imho nie ma się co silić na skróty, normalnie `position`


This is set up correctly for testing :metal:, but the business requirements are not accurate. $15 should result in 1.35 in taxes.


przenieś to np do konstruktora


@vlsi, I'm not sure why you think we're wasting time.  This is a public API and we change the method signature -- that isn't something that should be done lightly.

Re (1) -- who is asking to clone the functions?  You don't implement backwards compatibility by copy/pasting, you implement it by delegating the functionality.  I see a fairly simple solution here: re-introduce the old methods and have them wrap the Customizer in a custom registry whose .get(Customizer.class) returns null if the Customizer was null or a list of size 1 if not.  AFAICT, that would be 100% compatible and not require any code duplication.

Re (2) --  No one is saying that the new method isn't required.  Yes, it's required.  That doesn't mean the old method needs to disappear.


Oh, forgot about that.


Nice job :metal:


bang


That's some Blue Steel right there


This should be moved into `sgen-array-list`.


generalnie nie wiem jak tam ten cały pygame mixer działa, więc nie wiem na ile musisz to zrobić:

obsługa błędów: co jeśli nie ma załadowanej żadnej piosenki? mixer walnie jakimś wyjątkiem? oleje? co Ty chciałbyś, jak wrapper, żeby się działo? może sam chciałbyś rzucić wyjątkiem? albo po cichu olać?


Hmm.. I'll fix it


Looks straighforward.


Now the focus is on ulx4?
Go go go compatible with ulx3 :smile:


so nice to get rid of it.


ddgfdgdfg


BTW the proper thing to do is to modify bitcoin-tx to allow it to take the tx version as a parameter. Maybe it already can do this, but if it has that ability it wasn't apparent from the help text. That way we wouldn't have to do this every time the tx.nVersion changes.


12 emit


neeeeeeee


co jeśli nie ustalono katalogu z muzyką?


Reported in issue #18, just about to commit fix.


What does this comment mean?


There are #guaranteednationthemes and #freenationthemes commands for tinkering too. By default 1 is guaranteed and a couple more may spawn.


lgtm


Perfect. Of course, it is perfectly OK to leave TODO notes in the code (especially, if you add that code later ;)).


added comments not to ex20 but to ex21


Fordi det brukes lambda her, blir try/catch den "billigste" måten å gjøre dette her


can do!


`npms` or `npm`?


@rdavison I was just patching the same and independently verified the fix.


Your idea about storing the various tax rates is interesting, but I'm not sure how clear that function is. I do like the idea that these could easily be changed.

In your test, you should expect something to equal a value (not expression). In this case, we want the calculated tax to be to the penny.

I can see you understand testing, so nice job on that :metal:


Yep, leftover, will remove.


```
     public static void printMap(Map<Long, Message> messageMap) {
         if (messageMap.isEmpty() || messageMap == null) {
             Logger.addToLog("Warning: Something no messages \n\r");
             System.out.print("No messages \n\r");
             return;
         }
```
Разыменование объекта, а потом проверка на null. Нужно поменять местами.


tutaj pobierasz głośność od mixera i zapisujesz w polu

linijkę niżej ustalasz `0` dla zarówno mixera jak i pola => czyli bez sensu => mam wrażenie, że `unmute()` Ci nie działa…


no it doesn't use request


Nice job! :metal:


What would you define as compatible?
On Tue, Feb 16, 2016 at 16:13 Bruno Carvalho <notifications@github.com>
wrote:

> Now the focus is on ulx4?
> Go go go compatible with ulx3 [image: :smile:]
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/TeamUlysses/ulx/commit/9969c627218c8bdde5ac8f9fa18738183e9a91a9#commitcomment-16130473>
> .
>


creature is now casting the Spell not caster


oh I didn't mean to make it sounds like this was a mistake or anything, we don't have the style guide yet to explain these things :)


is this a _safe_ version of rails?


What exactly does the:
~~~~
+  volatile uint8_t head;
+  volatile uint8_t tail;
~~~~
tell the compiler?   Looking at: https://en.wikipedia.org/wiki/Volatile_(computer_programming) it says:

> In C and C++[edit]
> In C, and consequently C++, the volatile keyword was intended to[1]
>
> allow access to memory mapped devices
> allow uses of variables between setjmp and longjmp
> allow uses of sig_atomic_t variables in signal handlers.
>
> Operations on volatile variables are not atomic, nor do they establish a proper happens-before relationship for threading. This is according to the relevant standards (C, C++, POSIX, WIN32),[2] and this is the matter of fact for the vast majority of current implementations. Thus, the usage of volatile keyword as a portable synchronization mechanism is discouraged by many C/C++ groups.[3][4][5]

I'm not sure it guarantees the correct thing happens on
~~~~
FORCE_INLINE uint8_t available(void) {
     return (uint8_t)(RX_BUFFER_SIZE + rx_buffer.head - rx_buffer.tail) % RX_BUFFER_SIZE;
}
~~~~
~~~~
      uint8_t i = (uint8_t)(rx_buffer.head + 1) % RX_BUFFER_SIZE;
~~~~

I do think in general, making this index 8 bits is going to cause the right thing to happen.   But I don't think there is any guarantee about the code the compiler is going to generate and that it will be 'atomic'.   The last line is easiest to discuss.  Specifically, even with the volatile key word, I think the compiler would be allowed to increment the memory location where rx_buffer.head is   and then load this value into a register to perform the mod (% RX_BUFFER_SIZE) and then store it back where it belongs.    If an interrupt happens at the wrong time (after the increment, but before the mod operation is stored), the value is only partially computed and what ever grabs it will have a wrong value.

For the last line, I think this will guarantee that does not happen:
~~~~
    cli();
    uint8_t i = (uint8_t)(rx_buffer.head + 1) % RX_BUFFER_SIZE;
    sei();
~~~~

for the previous example, something like this might guarantee what we need:
~~~~
FORCE_INLINE uint8_t available(void) {
   uint8_t xyz;
   cli();
   xyz = (uint8_t) ((RX_BUFFER_SIZE + rx_buffer.head - rx_buffer.tail) % RX_BUFFER_SIZE);
   sei();
   return xyz;
}
~~~~


How about adding a LIMIT with EXECUTE (delete) that should fail too.


**Note** didn't get this one working yet (might need help)


This is not obviously right, but I guess it's a good starting point. Since these passes may be mutating the MIR, it's reasonable to make all the data they read be considered an incoming dep on the final MIR end-product. Might be better to change the pass structure so that they instead are just given a def-id and request the MIR for that def-id, which would cause us to track the edges automatically, and then those passes could push nodes for themselves as needed. But let's leave this for now.


The `status_code` is already available, but it can technically also change underneath us, because these objects don't sync automatically.


I guess I probably should have split the s/result/value/ change into a separate commit so it was more obvious


FixCrashWithZoneAndSystemERV (JasonGlazer) - x86_64-MacOS-10.9-clang: OK (2079 of 2079 tests passed, 7 test warnings)

 * 3 tests had: AUD diffs.
 * 2 tests had: EIO diffs.
 * 2 tests had: ESO small diffs.
 * 1 test had: SHD diffs.
 * 1 test had: ERR diffs.

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


sí, ese es el problema, no existe stackId al hacerle setup al logger


```
public boolean newMessage(String message, String author) {

         Logger.addToLog("Attempt to create new message\n");
         if (message.isEmpty() == true) {
```
Разыменование String. А если null?


Takie todo w językach „kaczkowych” są kontrowersyjne – jeśli kwacze jak kaczka to jest kaczką – jeśli zachowuje się jak napis, to spoko, nie musi to być faktycznie napis – `join` zadziała także dla liczby (nie–napisu), przeszkadza Ci to?

Ale piszę, że kontrowersyjne, bo faktycznie czasem chciałoby się mieć więcej kontroli nad typami. Tylko wtedy po co wybierać język, który w ogóle tego nie wspiera? :)


Ouch. That was unfortunate. Got a suggestion on how to change it back?


is `spin` supposed to be `span`?


do we need to nest flash inside a `flash` hash?


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


Thanks Clifton.  U rock.


node_modules so nice you gotta say it twice


Nice job :metal:


Although it is fairly self explanatory, maybe include a note to the user about how "Start English Game" and "Start Spanish Game" have different rules and implementations. What do you guys think? Any ideas on a clean way to display the different rules for the different versions, if even necessary?


Change name of `subject` to `merged` or something.


creature->CastSpell(GetHitUnit(), uint32(GetEffectValue()));


That's true the boolean result of $this->kick should be taken into account otherwise plugins that use PlayerKickEvent will not be able to cancel the event. I use a plugin on my server to allow regular players to connect even when slots are full, it wouldn't work with this.


We should just use Stream in error messaging. May be run all errors by Andrew.


Trams don't have a position then a npe will be thrown. Please fix this  :)
In the debugger I see position values only for DB trains.


how you would implement same functionality differently? What _include_ is doing exactly here?


It's incomplete (nothing has changed in index.html). The feeds are determined by a constant called RSS_FEEDS in config.js

All that is happening here is that it is fetching the RSS feed - I haven't had the time to complete it.


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1552 of 1552 tests passed, 0 test warnings)

![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)


This could have been removed ages ago.


@ljharb sorry but which C extensions are you talking about?


Thanks :)


This shouldn't get detected as a test.


Everything looks pretty solid to me, without actually viewing the html doc. I added a line note about how we might want to explain (have a subsection/header) to illustrate the different rules for each game. Any input on this idea, if even needed right now?


Can you see any possible issues here?


The refactor of everything to `ExecutionStrategy` is the bigger noise really (it is a good refactor though).


what does this line do?


Why do you think that?


hah will fix in a follow up. Works just as well apparently :)


When I right-click on a `tabPage` span, it appears that both `onTabsToolbarContextMenu` and `onTabPageContextMenu` are triggered. Only `onTabPageContextMenu` should be.


can we _optimize_ this line somehow? do you see any drawbacks here?


On 16-2-2016 17:58, Kefu Chai wrote:
> why would you want to test |atexit()|?

Because it looks like I'm not seeing the PID exit code being executed in
some of the tests... ceph-osd just exits without cleaning the PID
but it also does not core dump. So where is the atexit code??
And thus I'd like to be sure that the code is actually is executed.

Given the nature of the code, it is rather hard to test with gtest.
So for the moment I'm using visual inspection.

--WjW


Done


Yes, by definition any size can/must fit into a size_t


Just tested this. Is this the error message you're talking about?

`[IBM][CLI Driver] SQL30081N A communication error has been detected. Communication protocol being used: "TCP/IP". Communication API being used: "SOCKETS". Location where the error was detected: "9.30.115.13". Communication function detecting the error: "connect". Protocol specific error code(s): "10060", "*", "*". SQLSTATE=08001 QODBC3: Unable to connect`


@evancz do you mean `jump` instead of `continue` ?


shame on you!

![image](http://cdn.meme.am/instances/500x/61316231.jpg)


Good catch, will fix.


```
                 if (Unit* hitUnit = GetHitUnit())
                      if (Unit* caster = GetCaster())
                          if (Creature* creature = caster->ToCreature())
                          {
                              creature->CastSpell(hitUnit, uint32(GetEffectValue()));
                          }
```


`GetName().Version` returns stuff like `1.0.0.0`, I don't tag that way because I can't be assed.


Doesn't this break queries where the string end punctuation comes in the same line as the SQL comment?

Same issue as with HTML's `</script>` in JavaScript, which we talked about in another issue.


lgtm


What is happening exactly in database on this line - do you see any issues here?


Entonces se crearía una copia y tendríamos una instancia de logger diferente por cada framework? Lo había pensado pero lo evité por cuestiones me memoria y evitar posibles problemas de concurrecia con varios objetos escribiendo en el mismo log...


time?


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


Sorry, I don't think I understand your concern.  Can you please sketch out an example scenario?


release-5.x (lefticus) - x86_64-MacOS-10.9-clang: OK (389 of 389 tests passed, 0 test warnings)

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


Do we mean each ceph hosti n the cluster?


@evancz why do we need another trampoline when we have one from [core](https://github.com/elm-lang/core/blob/master/src/Trampoline.elm)?
http://package.elm-lang.org/packages/elm-lang/core/3.0.0/Trampoline


Oh, no I messed up! I guess ditching sed for awk wasn't as fool-proof as I thought it was. It appears I have gawk installed from Homebrew. I should have `brew unlink`ed before testing.


Perhaps it's time to merge these three unit tests now?


if target is not specified can this be null?


Rób mniejsze commity: zamiast sieczki „różnych zmian” powinieneś każdą „atomową” zmianę zrobić oddzielnym commitem. Łatwiej się potem analizuje zmiany, nazwy commitów mogą mieć większy sens, łatwiej jakiegoś commita wycofać (bo wycofujesz jedną małą poprawkę, a nie cały zestaw różnych zmian).

np mógłbyś mieć:

* „YT_API: get_search_results(): keywords bez wartości domyślnej (zawsze należy jakieś przekazać)”
* „YT_API: dorzucone escape'owanie wartości przy żądaniach do YouTube'a”
* „YT_API: upewniamy się, że każde słowo kluczowe jest napisem”
* „YT_API: wywalone zbędne zmienne”
* „YT_API: obsługa błędów przy żądaniach do YT”

itp – commit powinien być mały, atomowy, jasny, dobrze opisany – żeby było wiadomo co zrobiłeś, a i super jak jeszcze wyjaśnia _dlaczego_ to zrobiłeś


are all of those parameters _safe_?


why not use new syntax?


hass error =\
see #103


Chau, remember to add a extra "#" sign to any english that has been translated.


can it be done on _model level_ and how? should it be done inside a model?


Would it be possible to add an pointer in core_options::entry to the container?


@kuraga that's the message travis-ci displayed, and tests failed on the sudoless container that worked both locally and with the sudo container - I'm assuming one of [these](https://github.com/substack/tape/blob/master/package.json#L26-L30) dev deps is the cause.


What issues was this causing @benjamincaldwell?


Thanks; fixed!


Any reason you didn't pr the main repo?


It needs to be the date of today?


Done


What's in this new file? What changes between this file and the previous "_selected-samples_" version?

- here, all the samples are eligible for integration in Bgee (after quality check): aka, all the samples reported in this file are coming from "YES" subjects (default "YES" samples) or "PENDING" subjects (only "YES" samples). **BREAKING NEWS**: sample number in this file (8016) matches **exactly** the number of samples tagged as "YES" as reported in the (in)famous column "B" of _samples-track_  (aka, "YES" samples from both "YES" and "PENDING" subjects). In particular:
  - 89 samples (https://github.com/BgeeDB/expression-annotations/blob/master/annotations/gtex/review_of_GTEx_annotations_FILES/discard_from_selected-samples.tsv) previously annotated in _selected-samples_ were discarded (please refer to #110 for further explanations)
 - 17 samples (https://github.com/BgeeDB/expression-annotations/commit/2de3bc90965d0eba45c65961eee6805f95c64205) were present in _samples-track_ but not in _selected-samples_. After manual check, they were found to be eligible for integration in Bgee. The reason for their exclusion from _selected-samples_ might be that they were be some of the latest recovered, thus not yet reported in _selected-samples_. Now, they're reported and annotated here, ready to be integrated in RNAseqLibrary.
  - only one instance of duplicated samples (https://github.com/BgeeDB/expression-annotations/blob/master/annotations/gtex/review_of_GTEx_annotations_FILES/GTEXID_duplicate_selected-samples_annotsID.tsv) was kept here, following Anne's advice (source of duplication remains still unknown...?)

- two new columns were added, giving information about assay type (RNAseq, Affy....etc), pasted from _samples-track_:
  - $R "TYPE_OF_LIBRARY"
  - $S "Assay_Type_s_SraRunTable")

- three columns were updated, concerning the eventual correction made by Bgee on Uberon proposed by GTEx, and made more strict (a lot of "useless" free-text was previously included ):
  -  $L was updated following informations reported in column $J "Tissue quality according to SMPTHNTS (GOOD/BAD)" : if $J=="GOOD", then $L is filled; if  $J=="BAD", then $L, $M and $N are empty.
  -  $M "Explanation for precision estimation" was updated following informations reported in column $L "Precision estimation for Uberon mapping (GOOD/BAD)": if $L=="GOOD", then $M=="ok "; if $L=="BAD", then $M contains the new Uberon term.
  - $N "UBERON by ANE/AME" ": if $L=="GOOD", then $N is empty; if $L=="BAD", then $M contains the new Uberon ID

- samples are sorted by tissue type

----

- NOTE: some tissues need further verifications (to be commmitted soon)
  - thyroid gland
  - gastroesophageal sphincter
  - esophagus squamous epithelium
  - esophagus muscularis mucosa


nazwa metody nie mówi o jaki kontekst chodzi: dla HTML będą inne znaki specjalne niż dla URLa

np w HTMLu znakiem specjalnym jest `<` – gdy w URLu jest on bez problemu akceptowalny

np w URLu spacja nie może istnieć (powinna być zakodowana jako `%20`),  gdy w HTMLu możesz spokojnie wstawić ile ich tam chcesz

---

czyli jak masz taką metodę, warto, żeby wskazywała kontekst, czyli np `_html_escape()` czy też `_url_escape()`


Agreed.  Thanks.


No matter, I think I got it working now. `longopts` tests are passing so I just released 1.2.0 with this, and some other gems: https://github.com/kvz/bash3boilerplate#v120-2016-02-16


@mccalluc Not real obvious to me how to run this. Maybe add a hint comment? i.e...
```ruby
# Usage: cat path/to/csv_file.csv | ruby csv-to-redirect-map.rb
```
.. or, get rid of the `while gets` loop and...
```ruby
Usage: ruby -n csv-to-redirect-map.rb path/to/csv_file.csv
```


in directive page-title,
```js
if (currentState.data)
```
should be
```js
if (currentState.data.pageTitle)
```
?


what kind of callback would be more optimal here?


Well when saving subtitles, the default filename is the ``video finame.extension`` so what i am suggesting is
``video finame<.lang><.hi>.extension``, ``<>`` if available.

The original subtitle finame doesn't matter at this point, only the video filename...
So in the end i dont see why i should test for "Local"...

ie. you may load a subtitle from the download folder (downloaded separately) and save it to the video folder through mpc in order to have the same filename and load automatically...


Hmm, I suppose it does. It is almost like we need an alternative construct for handling embedding.


Thanks again :100:


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


can this be simplified somehow?


Clearer and less susceptible to Monitor.clean introducing a need for `this` in the future.


so actually it is not perfect lol


how exactly would it be 'perfect'?


na pewno uwzględniłeś wszystkie znaki specjalne? co np ze spacją?

nie ma jakiejś gotowej funkcji do tego w pythonie? może jakaś biblioteka? generalnie to są tematy, gdzie nie warto wynajdować koła na nowo (bo gazylion ludzi już je rozkmniniało, a i często laik nie uwzględni różnych aspektów bezpieczeństwa aplikacji)


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


Domyślnie jest `_music_directory = ""` czyli będzie szukać piosenki w folderze w którym uruchomiona jest aplikacja


develop (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1545 of 1545 tests passed, 0 test warnings)

![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)


> Хэндлинг исключения это то, что я передаю туда в качестве сообщения? Что не так с ними(кроме вероятно большого кол-ва ошибок в англ языке...)?
Это то, что ты делаешь когда возникает исключительная ситуация. другими словами все что происходит в блоке catch

> У меня парсится все хорошо. Не знаю в чем проблема. Брал log.txt файл с сайта. Все работает. Можно подробнее о проблеме и желательно то, что пишет Loger?
Попробуй в файле записать сообщение с переводом строки. для json такие переводы (если писать его рукани, например, абсолютно нормальны. Код который я прикрепила прочитает корректно весь файл, а не одну строку.

Методы для чтения файла и стрима можно завязать друг на друга.


This should probably be some var linked to the html, however, no idea how to balance this yet.


If it's how it currently works then ok. I thought it was keeping the local name when possible.


Something


s/abundant/redundant/


Plus


It might be better to title this commit with something more descriptive. Flag maintenance is very generic


Nigdy pygame nie używałem, ale to chyba ma być wywołane tylko raz - globalnie dla wszystkich obiektów, nie wiem czy ewentualne wielokrotne wywołanie przy tworzeniu wielu obiektów nie będzie powodowało problemów.


what this `include` is doing here exactly, could we achieve same result by doing things differently? Should we?


This is actually removing `window.initBookmarks`, not the function `initBookmarks`.


This import is masked by the argument in the test method.  Looks like a mistake to me.


Why is this getting removed?


Di prepotenza proprio

Cit. Sergio Mancini(h)
@SantX27


I would keep the current wording since this may apply to more than just non-deterministic ops.


zbędna zmienna


Good catch!  I forgot to advance consistency_check_client after line 84.


Same as above but for initSettings.


release-5.x (lefticus) - x86_64-MacOS-10.9-clang-Debug: OK (389 of 389 tests passed, 0 test warnings)

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


the time of the last verson, usually.


Long is not a portable type on 64bits systems. Never use it. Use __builtin_clz and __builtin_clzll instead.


Not clearer at all imo :|


кроме того, что риквест ни к месту, есть замечания?)


This is a pytest fixture. Without the import it's not available in this module.


Agreed.  Thanks.


Looks like you guys are making some great progress. Something I'm seeing here is that you may want to do a last minute check before joining a game/adding players that makes sure the game didn't fill up (or someone already chose that color) in the brief interim before you actually perform the server call. That can save you a lot of debugging trouble.


Թվեր դու մի գրի, գրի
#. HTML..
#. Java..
ինքն ավտոմատ կհաշվի


pb au lancement de `gulp`
```
[22:39:11] html en/index.html 11.94 kB
events.js:154
      throw er; // Unhandled 'error' event
      ^

Error: Error: File not found with singular glob: /Users/user/git/devFestSite-2016/scripts/main.js
    at DestroyableTransform.<anonymous> (/Users/user/git/devFestSite-2016/node_modules/gulp-useref/lib/streamManager.js:90:36)
```


I could rewrite the whole paragraph as:

```
   This enables ops that cause trouble with DebugMode with their
   normal behaviour to adopt a different one when run under that
   mode. If your op doesn't have any problems, don't implement this.
```


Yea, that will fail. I will add a test to show that.


մի շարք բաներ այստեղ քեզ պետք չեն


Why this membar is needed?

Could you explain what's needed to make the los list accessible without the GC lock and how this membar accomplishes it?


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


ok


OK, I will run it by Andrew.


This value isn't used.

Does this method have side effects?


:sparkles: :cake: :sparkles:


Utilise la fonction $this->assetUrl() pour les images etc qui prends automatiquement le repertoire "/public/assets"

C'est utile si le dossier principal change de nom par exemple


Wow, you've already got Map tests written? Great job, guys. Let us know if there's any help we can give you, but it seems as though you're really plowing ahead at a great pace.


ես ֆայլտ չկա ու  WARNING կունես


Yes, that is taken care of in the lines below.


That shouldn't be the case. From the report I suspected the `with_prototype` context to be errornous, which is why this "fix" was confusing for me, but I haven't verified that.


all ok


Yeah, partially my fault there. I should’ve documented why the previous code had `guint` and `__builtin_clz`.


Strange every distribution packaging system will force to use CMAKE_SKIP_RPATH
Is there any reason to not?


> nie wiem czy ewentualne wielokrotne wywołanie przy tworzeniu wielu obiektów nie będzie powodowało problemów

to sprawdź, dowiedz się

> to chyba ma być wywołane tylko raz

jeśli tak, to możesz ewentualnie mieć pole statyczne i w konstruktorze inicjalizować tylko raz

----

Nie wiem na ile w Pythonie tak jest, ale w PHP plik z klasą powinien definiować tylko tę klasę. Żeby autoloader spokojnie mógł załadować plik bez efektów ubocznych. Można załadować plik i nie chcieć odpalać mixera. Mixer jest Ci potrzebny tylko jak użyjesz klasy.


Yes, I can. This is testing deployment xml below, so I didn't pay attention to this.


@btovar unfortunately this is needed for the Confuga cluster. Can we a hot fix release for this?


New code makes it clearer that `Monitor.clean` is a function, while old code doesn't convey that (`setInterval` also accepts strings as a first argument). New code also makes it clear that `Monitor.clean()` takes no arguments, while old code doesn't convey that (unless you know both how the spec specifies `setInterval`'s API, as well as how closely our supported platforms follow the spec).


Looks like what you need here is mono_atomic_store_release


This does mean that the minimum android level is now Gingerbread rather than Eclair. The Google Play analytics said no-one was running the app on earlier than Gingerbread, so no-one's missing out.


This kind of verbose exception message is great. Keep it up.


Why do we need a write barrier here?


can you add an extra linespace here? our linter prefers two empty lines before the "main" line


So those are two things that the new code makes clearer. What does the old code make clearer?


Här kan du lika gärna skriva `self.avatar = character['thumbnail']`


prefiero tener una solucion mas limpia y centralizada que tambien funciona para swarm y otros sin apuntar a logrus en marathon.go. la idea tener un logger en mesos-framework era desacoplarlo de la implementacion final. La concurrencia es un buen punto... no sé que maravilloso esta logrus. Que opinas? es dificil probarlo?


This error is wrong.


Then it's a good thing I haven't wasted any time trying to add support for that, like I intended to.


I don't think this needs to be set back to 0. Same with loop_halve.


Do we know at this point whether the Pokémon is known not to have an item?


Nice

- myhomeworks.rst include in  index.rst
- write TODOs in empty pages and summaries them in todo page


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


Yes.


No, this looks like leftovers from some previous code. Landscape.io should catch it.


Multiple booleans makes this code really hard to read. Change it to be a single enum value with explicit
meaning for all the 3+ states you need.


Any idea when this changes will be in live. ?


Well, the thing is, FMOD doesn't handle it "properly" either, only its workaround is simply playing two independent mono sources instead of refusing to position (as openal does). So ZDoom can in turn just halve the volume in FMODSoundRenderer::StartSound3D.

Again, there's no "proper" way of dealing with multichannel sound, when you go into 3d, you can either pick your favorite guesstimate or refuse to handle such case.


doc/getting-started
doc/....


Necessary? No
Convenient? Yes
I shows the actual exit results, so you never are in doubt that the correct result is returned.
And thus exit 0, shows that in run-tox.sh the error has to be of a different nature.


Ahhh I see. TIL.


ну


`args[3]` would be `''` if the Pokémon has no item, and `undefined` if it's unknown whether or not the Pokémon has an item.


pay no attention to this commit; it is erroneous.


It looks like you've got the right idea going on...but are you sure you want to make your abstract state the same as a Map Controller? Or do you mean to swap in and out different instances of a map controller according to which state you're in? Bit of a red flag to see a state class that's the same super class as the object using it.


These two imports are masked by function arguments.

Do the imports have side effects?


I think the reason it has been shut off is mostly just deployment confusion. Is there a Debian/Redhat handbook for linux packagers using cmake that we can follow?


Did you consider if passing a dummy in the non pre-cleaning case?


amended: ddc83b47a60e46270d629f0fc85ba159d004c1e7


Well, ideally we shouldn't need to push patterns into every embedded syntax as an escape hatch back out to the master syntax.

In reality when embedding, as soon as the parent syntax end token is hit, parsing of the embedded language halts without exception. Currently the HTML -> JS embedding relies on finding `</script>`, which runs into issues when a JS comment is embedded inside of an HTML comment. Similarly here, we want to consume the rest of the line for the SQL comment, but to escape back to the embedding language, we can't without complex tricks.

Instead ideally we'd scan until we hit a valid Python close string token, then syntax highlight the embedded language using the SQL definition and prepend all of the SQL scopes with the scope of the Python string.


i fixed it


Yes, they are pytest fixtures.


Ugh. No, I really dislike this. Can you please follow a different approach here. Replace the IP addresses in `interfaces-openstack.cfg` with a placeholder, like `172.29.236.@IP_LAST_OCTET@`, `172.29.240.@IP_LAST_OCTET@`, etc. and then use the `str_replace` function in the corresponding section in `write_files`.

See http://docs.openstack.org/developer/heat/template_guide/hot_spec.html#str-replace for more details on that.


Say what!? Shouldn't this be something like:
```
+#if !defined(OPENSSL_NO_CRYPTO_MDEBUG_BACKTRACE) && defined(HAVE_EXECINFO_H)
```
`<execinfo.h>` is a POSIX abomination.


Much better. Thanks.


It's because we often used 0 for mu' in tests.


Nit (I know it wasn't introduced here): space around `=`.


Is this condition even possible? The nursery doesn't move the old gen and a serial major collection won't clash
with a concurrent one.


failed shutdown test !! https://travis-ci.org/brettwooldridge/HikariCP/builds/109710825#L494


Swap space in virtual machines is normally a terrible idea (double swapping, etc.). Did you run out of memory at any point? In that case, we'll just need a bigger flavor.


Suggestion, worth what you paid for it: it seems as though you've got a lot of duplicated checking going on between login and register. What if you made a common method for validation?


Investigate changing map to show locations shown as pins on map


This is wrong and should be undone.


Great :)

- fix warnings -> WARNING: Definition list ends without a blank line


Is this removed because it causes duplicated lang info?


Looks good as far as I can tell.


Hello, I've a trouble using 1wire in the domoticz :
Version: 2.4618
Build Hash: 8e3d9cd
Compile Date: 2016-02-16 18:10:54

The 1 wire Searching device function is bugged and it return multiple occurs of the same sensors causing a dramatic slow down of domoticz.

I report this bug on the forum bug it seems that the forum is crashed.

I've seen that this file has been modified few hours ago. Does it provide a correction for my bug ?

ex:
2016-02-16 22:50:19.361  1-Wire: Searching devices...
2016-02-16 22:50:19.364  (Sondes temperature) Temp (Chambre P.)
2016-02-16 22:50:19.434  (Sondes temperature) Temp (Chambre E.)
2016-02-16 22:50:19.476  (Sondes temperature) Temp (Chambre C.)
2016-02-16 22:50:19.518  (Sondes temperature) Temp (Piscine)
2016-02-16 22:50:19.548  (Sondes temperature) Temp (Chambre P.)
2016-02-16 22:50:19.589  (Sondes temperature) Temp (Chambre E.)
2016-02-16 22:50:19.606  (Sondes temperature) Temp (Chambre C.)
2016-02-16 22:50:19.624  (Sondes temperature) Temp (Piscine)
2016-02-16 22:50:19.642  (Sondes temperature) Temp (Chambre P.)
2016-02-16 22:50:19.660  (Sondes temperature) Temp (Chambre E.)
2016-02-16 22:50:19.686  (Sondes temperature) Temp (Chambre C.)
2016-02-16 22:50:19.708  (Sondes temperature) Temp (Piscine)
2016-02-16 22:50:19.743  (Sondes temperature) Temp (Chambre P.)
2016-02-16 22:50:19.780  (Sondes temperature) Temp (Chambre E.)
2016-02-16 22:50:19.809  (Sondes temperature) Temp (Chambre C.)
2016-02-16 22:50:19.849  (Sondes temperature) Temp (Piscine)
2016-02-16 22:50:19.878  (Sondes temperature) Temp (Chambre P.)
2016-02-16 22:50:19.902  (Sondes temperature) Temp (Chambre E.)
2016-02-16 22:50:19.925  (Sondes temperature) Temp (Chambre C.)
2016-02-16 22:50:19.943  (Sondes temperature) Temp (Piscine)
2016-02-16 22:50:19.961  (Sondes temperature) Temp (Chambre P.)
2016-02-16 22:50:19.979  (Sondes temperature) Temp (Chambre E.)
2016-02-16 22:50:19.996  (Sondes temperature) Temp (Chambre C.)
2016-02-16 22:50:20.013  (Sondes temperature) Temp (Piscine)
2016-02-16 22:50:20.029  (Sondes temperature) Temp (Chambre P.)
2016-02-16 22:50:20.049  (Sondes temperature) Temp (Chambre E.)
2016-02-16 22:50:20.066  (Sondes temperature) Temp (Chambre C.)
2016-02-16 22:50:20.099  Executing script: /home/pi/domoticz/scripts/domoticz_main
2016-02-16 22:50:20.089  (Sondes temperature) Temp (Piscine)
2016-02-16 22:50:20.149  (Sondes temperature) Temp (Chambre P.)
2016-02-16 22:50:20.179  (Sondes temperature) Temp (Chambre E.)
2016-02-16 22:50:20.209  (Sondes temperature) Temp (Chambre C.)
2016-02-16 22:50:20.227  (Sondes temperature) Temp (Piscine)
2016-02-16 22:50:20.245  (Sondes temperature) Temp (Chambre P.)
2016-02-16 22:50:20.264  (Sondes temperature) Temp (Chambre E.)
2016-02-16 22:50:20.282  (Sondes temperature) Temp (Chambre C.)
2016-02-16 22:50:20.301  (Sondes temperature) Temp (Piscine)
2016-02-16 22:50:20.319  (Sondes temperature) Temp (Chambre P.)
2016-02-16 22:50:20.337  (Sondes temperature) Temp (Chambre E.)
2016-02-16 22:50:20.355  (Sondes temperature) Temp (Chambre C.)
2016-02-16 22:50:20.372  (Sondes temperature) Temp (Piscine)
2016-02-16 22:50:20.389  (Sondes temperature) Temp (Chambre P.)
2016-02-16 22:50:20.405  (Sondes temperature) Temp (Chambre E.)
2016-02-16 22:50:20.421  (Sondes temperature) Temp (Chambre C.)
2016-02-16 22:50:20.438  (Sondes temperature) Temp (Piscine)
2016-02-16 22:50:20.455  (Sondes temperature) Temp (Chambre P.)
2016-02-16 22:50:20.472  (Sondes temperature) Temp (Chambre E.)
2016-02-16 22:50:20.490  (Sondes temperature) Temp (Chambre C.)
2016-02-16 22:50:20.507  (Sondes temperature) Temp (Piscine)
2016-02-16 22:50:20.525  (Sondes temperature) Temp (Chambre P.)
2016-02-16 22:50:20.542  (Sondes temperature) Temp (Chambre E.)
2016-02-16 22:50:20.562  (Sondes temperature) Temp (Chambre C.)
2016-02-16 22:50:20.580  (Sondes temperature) Temp (Piscine)
2016-02-16 22:50:20.598  (Sondes temperature) Temp (Chambre P.)
2016-02-16 22:50:20.615  (Sondes temperature) Temp (Chambre E.)
2016-02-16 22:50:20.634  (Sondes temperature) Temp (Chambre C.)
2016-02-16 22:50:20.652  (Sondes temperature) Temp (Piscine)
2016-02-16 22:50:20.671  (Sondes temperature) Temp (Chambre P.)
2016-02-16 22:50:20.689  (Sondes temperature) Temp (Chambre E.)
2016-02-16 22:50:20.709  (Sondes temperature) Temp (Chambre C.)
2016-02-16 22:50:20.728  (Sondes temperature) Temp (Piscine)
2016-02-16 22:50:20.745  (Sondes temperature) Temp (Chambre P.)
2016-02-16 22:50:20.763  (Sondes temperature) Temp (Chambre E.)
2016-02-16 22:50:20.781  (Sondes temperature) Temp (Chambre C.)
2016-02-16 22:50:20.798  (Sondes temperature) Temp (Piscine)
2016-02-16 22:50:20.814  (Sondes temperature) Temp (Chambre P.)
2016-02-16 22:50:20.832  (Sondes temperature) Temp (Chambre E.)
2016-02-16 22:50:20.851  (Sondes temperature) Temp (Chambre C.)
2016-02-16 22:50:20.870  (Sondes temperature) Temp (Piscine)
2016-02-16 22:50:20.888  (Sondes temperature) Temp (Chambre P.)
2016-02-16 22:50:20.906  (Sondes temperature) Temp (Chambre E.)
2016-02-16 22:50:20.924  (Sondes temperature) Temp (Chambre C.)
2016-02-16 22:50:20.942  (Sondes temperature) Temp (Piscine)
2016-02-16 22:50:20.960  (Sondes temperature) Temp (Chambre P.)
2016-02-16 22:50:20.978  (Sondes temperature) Temp (Chambre E.)
2016-02-16 22:50:20.996  (Sondes temperature) Temp (Chambre C.)
2016-02-16 22:50:21.014  (Sondes temperature) Temp (Piscine)
2016-02-16 22:50:21.033  (Sondes temperature) Temp (Chambre P.)
2016-02-16 22:50:21.054  (Sondes temperature) Temp (Chambre E.)
2016-02-16 22:50:21.073  (Sondes temperature) Temp (Chambre C.)
2016-02-16 22:50:21.091  (Sondes temperature) Temp (Piscine)
2016-02-16 22:50:21.110  (Sondes temperature) Temp (Chambre P.)
2016-02-16 22:50:21.128  (Sondes temperature) Temp (Chambre E.)
2016-02-16 22:50:21.146  (Sondes temperature) Temp (Chambre C.)
2016-02-16 22:50:21.164  (Sondes temperature) Temp (Piscine)
2016-02-16 22:50:21.181  (Sondes temperature) Temp (Chambre P.)
2016-02-16 22:50:21.197  (Sondes temperature) Temp (Chambre E.)
2016-02-16 22:50:21.213  (Sondes temperature) Temp (Chambre C.)
2016-02-16 22:50:21.230  (Sondes temperature) Temp (Piscine)
2016-02-16 22:50:21.248  (Sondes temperature) Temp (Chambre P.)
2016-02-16 22:50:21.265  (Sondes temperature) Temp (Chambre E.)
2016-02-16 22:50:21.283  (Sondes temperature) Temp (Chambre C.)
2016-02-16 22:50:21.302  (Sondes temperature) Temp (Piscine)
2016-02-16 22:50:21.319  (Sondes temperature) Temp (Chambre P.)
2016-02-16 22:50:21.337  (Sondes temperature) Temp (Chambre E.)
2016-02-16 22:50:21.354  (Sondes temperature) Temp (Chambre C.)
2016-02-16 22:50:21.372  (Sondes temperature) Temp (Piscine)
2016-02-16 22:50:21.389  (Sondes temperature) Temp (Chambre P.)
2016-02-16 22:50:21.406  (Sondes temperature) Temp (Chambre E.)
2016-02-16 22:50:21.423  (Sondes temperature) Temp (Chambre C.)
2016-02-16 22:50:21.441  (Sondes temperature) Temp (Piscine)
2016-02-16 22:50:21.459  (Sondes temperature) Temp (Chambre P.)
2016-02-16 22:50:21.475  (Sondes temperature) Temp (Chambre E.)
2016-02-16 22:50:21.492  (Sondes temperature) Temp (Chambre C.)
2016-02-16 22:50:21.510  (Sondes temperature) Temp (Piscine)
2016-02-16 22:50:21.528  (Sondes temperature) Temp (Chambre P.)
2016-02-16 22:50:21.545  (Sondes temperature) Temp (Chambre E.)
2016-02-16 22:50:21.562  (Sondes temperature) Temp (Chambre C.)
2016-02-16 22:50:21.579  (Sondes temperature) Temp (Piscine)
2016-02-16 22:50:21.596  (Sondes temperature) Temp (Chambre P.)
2016-02-16 22:50:21.614  (Sondes temperature) Temp (Chambre E.)
2016-02-16 22:50:21.631  (Sondes temperature) Temp (Chambre C.)
2016-02-16 22:50:21.648  (Sondes temperature) Temp (Piscine)
2016-02-16 22:50:21.665  (Sondes temperature) Temp (Chambre P.)
2016-02-16 22:50:21.682  (Sondes temperature) Temp (Chambre E.)
2016-02-16 22:50:21.699  (Sondes temperature) Temp (Chambre C.)
2016-02-16 22:50:21.716  (Sondes temperature) Temp (Piscine)
2016-02-16 22:50:21.739  (Sondes temperature) Temp (Chambre P.)
2016-02-16 22:50:21.762  (Sondes temperature) Temp (Chambre E.)
2016-02-16 22:50:21.779  (Sondes temperature) Temp (Chambre C.)
2016-02-16 22:50:21.797  (Sondes temperature) Temp (Piscine)
2016-02-16 22:50:21.813  (Sondes temperature) Temp (Chambre P.)
2016-02-16 22:50:21.830  (Sondes temperature) Temp (Chambre E.)
2016-02-16 22:50:21.847  (Sondes temperature) Temp (Chambre C.)
2016-02-16 22:50:21.866  (Sondes temperature) Temp (Piscine)
2016-02-16 22:50:21.884  (Sondes temperature) Temp (Chambre P.)
2016-02-16 22:50:21.902  (Sondes temperature) Temp (Chambre E.)
2016-02-16 22:50:21.921  (Sondes temperature) Temp (Chambre C.)
2016-02-16 22:50:21.939  (Sondes temperature) Temp (Piscine)
2016-02-16 22:50:21.955  (Sondes temperature) Temp (Chambre P.)
2016-02-16 22:50:21.973  (Sondes temperature) Temp (Chambre E.)
2016-02-16 22:50:21.989  (Sondes temperature) Temp (Chambre C.)
2016-02-16 22:50:22.007  (Sondes temperature) Temp (Piscine)
2016-02-16 22:50:22.025  (Sondes temperature) Temp (Chambre P.)
2016-02-16 22:50:22.043  (Sondes temperature) Temp (Chambre E.)
2016-02-16 22:50:22.063  (Sondes temperature) Temp (Chambre C.)
2016-02-16 22:50:22.080  (Sondes temperature) Temp (Piscine)
2016-02-16 22:50:22.098  (Sondes temperature) Temp (Chambre P.)
2016-02-16 22:50:22.116  (Sondes temperature) Temp (Chambre E.)
2016-02-16 22:50:22.133  (Sondes temperature) Temp (Chambre C.)
2016-02-16 22:50:22.149  (Sondes temperature) Temp (Piscine)
2016-02-16 22:50:22.166  (Sondes temperature) Temp (Chambre P.)
2016-02-16 22:50:22.183  (Sondes temperature) Temp (Chambre E.)
2016-02-16 22:50:22.201  (Sondes temperature) Temp (Chambre C.)
2016-02-16 22:50:22.219  (Sondes temperature) Temp (Piscine)
2016-02-16 22:50:22.237  (Sondes temperature) Temp (Chambre P.)
2016-02-16 22:50:22.254  (Sondes temperature) Temp (Chambre E.)
2016-02-16 22:50:22.273  (Sondes temperature) Temp (Chambre C.)
2016-02-16 22:50:22.291  (Sondes temperature) Temp (Piscine)
2016-02-16 22:50:22.310  (Sondes temperature) Temp (Chambre P.)
2016-02-16 22:50:22.328  (Sondes temperature) Temp (Chambre E.)
2016-02-16 22:50:22.346  (Sondes temperature) Temp (Chambre C.)
2016-02-16 22:50:22.363  (Sondes temperature) Temp (Piscine)
2016-02-16 22:50:22.380  (Sondes temperature) Temp (Chambre P.)
2016-02-16 22:50:22.397  (Sondes temperature) Temp (Chambre E.)
2016-02-16 22:50:22.415  (Sondes temperature) Temp (Chambre C.)
2016-02-16 22:50:22.433  (Sondes temperature) Temp (Piscine)
2016-02-16 22:50:22.450  (Sondes temperature) Temp (Chambre P.)
2016-02-16 22:50:22.467  (Sondes temperature) Temp (Chambre E.)
2016-02-16 22:50:22.485  (Sondes temperature) Temp (Chambre C.)
2016-02-16 22:50:22.503  (Sondes temperature) Temp (Piscine)


Set last_marked to -1 when pre_clean is set and we can simplify the logic here.


redundant to check course_Id


>  I'm not sure why you think we're wasting time. This is a public API a

My point is `EnumUtils` is not a published interface.
Please show where do you find `EmitUtils` usage in the wild.
I bet it is just a copy&pasted of cglib source code.

It would be much better to spend time on adding more reviewers/committers, designing features, etc.
The PR was submitted 3 months ago and everyone was silent except @elrodro83 who did some review.

Being able to release would be good as well.


It's left shift operation.
Please see [left shift](https://msdn.microsoft.com/en-us/library/336xbhcz.aspx) for more details.

This expression will result in a very large integer.


Nice:-)


SDL_GetPixelFormatName should give you what you want: https://github.com/flibitijibibo/SDL2-CS/blob/master/src/SDL2.cs#L2793


Work with ulx 3 custom addons... Or something like that kk


Who I will be in the future :) I am thinking ...


Great, thanks.


docs/getting-started.md


I am not unable to upgrade to your latest version because we aren't using typescript 1.8 yet (since it is still in beta). Can this be an opt in feature to use the new typescript 1.8 string literal types? Otherwise I imagine this will break most people using this library since 1.8 is still in beta.


there is no ikon.jpg

 WARNING: image file not readable: 'ikon.jpg'


It won't be compatible to that level, sorry. We may suck in ulx3
users/groups though.
On Tue, Feb 16, 2016 at 16:52 Bruno Carvalho <notifications@github.com>
wrote:

> Work with ulx 3 custom addons... Or something like that kk
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/TeamUlysses/ulx/commit/9969c627218c8bdde5ac8f9fa18738183e9a91a9#commitcomment-16131477>
> .
>


(Is this true once the game is ended? I don't want to accidentally upload a log showing only my exact HP!)


WARNING: toctree contains reference to nonexisting document u'Data Structure'


Yes if ``m_lcid`` is provided when constructing the menu will add ``tab`` and then language so it shows consistently in the "accelerator" area of the menu...
When ``m_eHearingImpaired == HI_YES`` it will enclode the language in brackets ``[ English ]`` to denote for hearing impaired...


MERGE THIS!


remove this overly verbose


I didn't think we were really losing anything by inlining this as the name of the `next_node_calculation` is repeated in the method name.

I hope that the call to the `data_query` will be made via the calculator relatively soon.


/home/nanenare/workspace/ITC-7/Smbat_Sargsyan/homework/Sphinx/New_Summary_Sphinx/Linux.rst:5: WARNING: Explicit markup ends without a blank line; unexpected unindent.


In a later commit, we should change these to a consistent order: `in_out`, then `in_prefix_len`, then `in_suffix_len`, the same order that `OpenOrSealFn` uses.


Great , thanks much !, very helpful. !
Best


WARNING: Explicit markup ends without a blank line; unexpected unindent.


I didn't think we were really losing anything by inlining this as the name of the `next_node_calculation` is repeated in the method name.


lol, you apparently completely missed my `commands.js` change. We don't use `getLogForUser` at all when uploading replays, we use `getLog(0)` directly.


Adding the closure solve the gulp stacktrace.

http://stackoverflow.com/questions/26614738/issue-running-karma-task-from-gulp


Did not use var in all places, only where appropriate.


watchify should be in a `start` script instead


Check out most recent commit - Replaced hard coded username with the %USERNAME% variable and also replaced the hard coded IP address with the one provided by the "docker-machine ip default" command.


Ok.


Wouldn't that be a bit of a catch 22?  Where would `HAVE_EXECINFO_H` come from?

Our thinking is that whoever configures with `enable-crypto-mdebug-backtrace` better know what they're doing.  And right now, it's limited to execinfo, but hey, if someone has an alternative implementation, it's always possible to propose additional code.


Although right now I can't quite see how to communicate that information to the battle tooltip.