```
size_t length() const   { return elements_.size(); }
```

already exists a couple lines up. Please remove this.


@bajtos - Depending on the environment and target server type (e.g. Portal vs API) and/or local preview, we mount SwaggerUI at different nesting levels, without these changes I could only get one scenario to work at a time.  For instance:
* https://developer.godaddy.com/static/developer_portal/swagger-ui/dist/images/throbber.gif
* https://api.dev-godaddy.com/doc/images/throbber.gif
* file:///Project/ApiSpec/swagger-ui/dist/images/throbber.gif


Thanks Darrell!
It seems likely and with the coming Minimed Connect I should be able to push my glucose information either through Nightscout or with Garmin ConnectIQ app. What do you think?!
BTW - Take a look at this watch face; I think it gives a good idea of how CGM data could be viewed on Garmin Fenix 3. Plus with the amount of caloric information available it could and has helped to to get me a 6.3 A1c.

Thank you,

Dominick D. Gallegos
603.785.2749 Cell
dominickgallegos@gmail.com

On Tue, Jul 7, 2015 at 2:48 PM -0700, "Darrell Wright" <notifications@github.com> wrote:

This was more of a proof of concept pulling the data from a nightscout server.  Someone else in the NightScout Facebook group used the alerts, like an SMS message to get it working during a workout.  The Garmin API doesn't let one communicate with the internet while an activity(e.g. Running/Cycling) is active.

Using the Nightscout notifications seems to be the best approach currently.

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


Added the check.


Por que você usou names como atributo do controller?


createSingleBuffered wasn't used in latest chromium. And according to this commit, it also not used in this version < KITKAT.  So what the purpose of this?


@bajtos - I disagree.  I made the changes in here as well in order to ensure `swagger-ui` remains reusable for our many teams that aren't using LoopBack, but still leverage `swagger*`


`sys/personality.h` seems to be missing from Android's libc implementation (according to [this](https://github.com/tias/android-busybox-ndk#config-options-that-do-not-build-missing-library)) and causes a [build failure on Travis-CI](https://travis-ci.org/reicast/reicast-emulator/builds/69901056):
```
jni/../jni/../../../core/linux/common.cpp:19:29: fatal error: sys/personality.h: No such file or directory
```


Por que a aplicação em dois controllers?


Oops....
Um....  Yeah, we're removing these


What is this line all about?


앗 감사합니다. 별 생각없이 and를 자주 쓰는 버릇이 있어서


added these three files which handle the table creation for each model type. as of now going to /content/all_people/ takes you to a page where all the people in the database and their attributes are displayed and is sortable!!


# git config --global user.name "kogei26220"
# git config --global user.email kogei_26220@yahoo.co.jp
# cd ~
# mkdir dotfiles
# mv .vimrc dotfiles
# mv .gvimrc dotfiles
# mv .xmodmap dotfiles
# mv .bash_profile dotfiles
# mv .vim dotfiles
# ln -s dotfiles/.vimrc .vimrc
# ln -s dotfiles/.gvimrc .gvimrc
# ln -s dotfiles/.xmodmap .xmodmap
# ln -s dotfiles/.bash_profile .bash_profile
# ln -s dotfiles/.vim .vim
# cd dotfiles
# git init
# git add .
# git commit -m "first commit"
# git remote add origin git@github.com:kogei26220/dotfiles.git
# git push -u origin master
Counting objects: 13, done.
Delta compression using up to 4 threads.
Compressing objects: 100% (11/11), done.
Writing objects: 100% (13/13), 9.96 KiB, done.
Total 13 (delta 0), reused 0 (delta 0)
To git@github.com:kogei26220/dotfiles.git
* [new branch] master -> master
Branch master set up to track remote branch master from origin.


LGTM. We should move away from the deprecated Get-WmiObject, which won't work on Server 2016, and use Get-CimInstance instead, but it doesn't need to be part of this change.


1. Yeah... we can add that check, but the only thing we can do is to assertion out, is my understanding correct?

2. Oh yeah, that is true, let me include a commit for that :) Thanks @dzafman .


is this in anyway related to the issue with that new crash with libhoudini?


This count down is new and i have not fully gotten it working yet. There is a bug where the countdown does not print there for does not happen.


Looks good, I see the hyphen needs to be excluded and I like your solution for that.


next time can we not wrap here? it's sort of strange to wrap


I don't think this fix is working. I still get nothing when I try to start a menu on a level higher than 1. I'm using the latest build and I've checked the file and the added lines are there.

Thank you.


Well, the crash of houdinin is due to no having x86, I'm working in trunk, still no x86 for stable. Feeling like going wild again?


We always wrap children when constructing trees.


sangre, sudor y lagrimas? :)


When is this method called ?


it's slower (extra multiplication by `3`) and adds a risk of overflow.
probably doesn't matter in practice, but for code correctness and elegance, I prefer my way.


@hvelarde please confirm if this code should be back, as you removed it in this commit https://github.com/collective/collective.nitf/commit/42202ebf38fdca750b6a2797f584a6a76ab6a244#diff-0343d67035040d10ffe1d495dc49a3aeL39


"Verify that"?


this was one of the problems with tests


Should use constants instead of hardcoded values.


"Verify that" is clearer, though it's less important than the previous instance :)


Can we add a `refreshServicePeer` method ?

```js
ServiceDispatchHandler.prototype.refreshServicePeer =
function refreshServicePeer(serviceName, hostPort) {
    var self = this;

    var peer = self.getServicePeer(serviceName, hostPort);
    peer.connect();

    self.exitServices[serviceName] = time;
}
```


런칭 전까지는 마이그레이션은 무시해도 되지 않을까 하는데... 아닐까요?


Replace `{}` with `Object.create(null)`


It's `destroy`; not `close`.

Close is only for something that closes a file descriptor like a TCP server.


in tests just use `setTimeout` directly instead of from `channel.timers`


Sweet!


this needs to be more then 5 because servicePurgeTime is 10


Looks good once those minor clarifications are made.


당장 모델 수정할 일도 많을거고 프로젝트 참여원들 대부분 깃 사용도 다들 능숙치 않을텐데 자기가 건드린 파일도 아닌 디렉토리까지 컨플릭
일으키는 것보다는 나을 것 같아서 일부러 빼두었습니다. 좀 더 적어둘걸 그랬네요.

문제가 된다면 되돌리도록 하겠습니다
2015. 7. 8. 오전 9:49에 "Seokju Hong" <notifications@github.com>님이 작성:

> 런칭 전까지는 마이그레이션은 무시해도 되지 않을까 하는데... 아닐까요?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sparcs-kaist/sparcssso/commit/bd4be9e7c445f50c7a757a8565d210d8f2411564#commitcomment-12051608>
> .
>


Ooops, accidentally commenting on the commit rather than the PR...


This change introduced a fundamentally different beahvior in the simulator.


This update make sense, but it broke my build when using gcc 4.4.7 and boost 1.49.

Building AbcOpenGL,
```
alembic/lib/Alembic/Util/Foundation.h:39:33: error: Alembic/Util/Config.h: No such file or directory
```
and when building PyAlembic,
```
error: no matching function for call to 'get_pointer(std::shared_ptr<Alembic::AbcCoreAbstract::v7::TimeSampling>&)'
```
The definition of get_pointer(std::shared_ptr) got added in boost 1.53 so right now the PyAlembic build is broken when using older versions of boost.

I was able to fix these issues pretty easily, however. Take a look, let me know what you think. If it looks good I'll submit a PR.

https://github.com/rsgalloway/alembic/commit/507b73a4895b0ead2f46432a478a6adbdeb4062f


it is called in advertise.

```js
        function advertise(serviceObj) {
            var peer = relayChannel.handler.getServicePeer(
                serviceObj.serviceName, serviceObj.hostPort
            );
            peer.connect();
            relayChannel.handler.updateServiceTimestamp(serviceObj.serviceName);
        };
```


Does this have anything to do with the SemanticChecker?  It'd be nice to have it in separate commits.


It would be nice to have test cases for the SemanticChecker under `test/semantic-error-test/` similar to `test/parse-error-test/`.


@netj Sure. Will do.


Some clean-up is necessary if we want to make progress.
One interesting side-effect is to give more value to our connected pawns on file A/B (and also, to a lesser extent B/C)
If this works, we will have removed all file dependency in pawns.cpp, and rely only on the PSQT for this.


Overall, the checker looks great!


This bug was found by semantic checker... yes it should be in a separate commit. I'll pay attention next time.


r+


Don't worry about changing anything now as I'm already integrating it myself, but for future reference, don't assign to parameters. Use a local variable instead.
Good explanation:
http://stackoverflow.com/a/2229437/4282499


would be nice to get this merged into the main version


Ah, then it makes perfect sense.  You could mention such things in the commit message to reduce confusion.


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


merging MonsieurNicolas/stellar-core/warnings = 1e690493 into auto


MonsieurNicolas/stellar-core/warnings = 1e690493 merged ok, testing candidate = 23621611


since the entire `docker-release.sh` (formerly `build-docker.sh`) file wasn't deleted (only the `for` loop over `GOOS`) it just showed it as if it was any other deletion, and since that code was copied over to `docker-build.sh` it showed that as a new file. either way makes sense I think


I don't know who you are
I don't know what you want
If you are looking for ransom,
I can tell you I don't have money.
but what I do have are a very particular set of skills. Skills I have acquired over a very long career. Skills that make me a nightmare for people like you. If you let my daughter go now that'll be the end of it. I will not look for you, I will not pursue you, but if you don't, I will look for you, I will find you and I will kill you.


@gorhill why remove an entire list over one entry? Isn't this a perfect candidate for the Unbreak filter list?


Oops, forgot the Message... I only added SearchTripsFragment.java and the fragment into the layouts


ROFL


1. Yes, for now we can only assert.  :-(

2. Notice that there are 2 ECUtil::decode() functions with the same issue.     :-)


Huh? where do I do that?

On Tue, Jul 7, 2015 at 5:03 PM, Alexander Strada <notifications@github.com>
wrote:

> Don't worry about changing anything now as I'm already integrating it
> myself, but for future reference, don't assign to parameters. Use a local
> variable instead.
> Good explanation:
> http://stackoverflow.com/a/2229437/4282499
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/MorbiusProductions/coinage/commit/bb4482ec42102878b501549184495519717a5c00#commitcomment-12051718>
> .
>


Interesting question. We should do an exercise and see if the security level increases if we do. The thing to keep in mind is that OTR (like other protocols) has been designed to have a consistent security level overall - that means the sizes of theses ranges are set to roughly lead to the same work load for trying to brute force attacks.

Another possible problem would be implementations that only allocate enough size for 320bits. You can interpret the spec as allowing that.


> Isn't this a perfect candidate for the Unbreak filter list?

No, it's the worst:

    @@||s3.amazonaws.com^

This would suddenly override all block filters based on `s3.amazonaws.com`.


This will remove the key from your inventory when you leave the room, cool.


ok~我建一个网盘哈，

在 2015-07-07 00:40:14，"ma haidong" <notifications@github.com> 写道：

很不错！ 视频的素材是不是可以放到一个大家可以下载的地方？

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


idiot


That will break paths like "/home/user/my folder with spaces/project/phantomjs"


I came more than a year late but is there a built version of the combined changes (func_detail_blocker + nodefaultcubemaps) still available?


Let's talk about this one tomorrow. I feel quite uncomfortable about it, which is why I didn't use ReadFull when doing this initially.


This is nice. Syntactically, it feels more familiar for anyone that's worked with https://ga-dev-tools.appspot.com/


Please revert this one for now, until we can talk about it. The semantics are quite different.


Your #ifdef won't work, if the BOOST_VERSION, you need to make sure it isn't also using ALEMBIC_LIB_USES_BOOST

Something like: (probably isn't syntactically correct)
#if defined(ALEMBIC_LIB_USES_TR1) || (BOOST_VERSION < 105300 && !ALEMBIC_LIB_USES_BOOST)


Don't know how this happened, but it appeared I was missing one simple change of a variable -_-


Hi, would you consider making a pull request to get this into upstream (and thus NPM)?

Thanks!


Thanks for changing these as well; I hadn't spotted these in previous reviews.


PR's been open for a while. https://github.com/b3nj4m/hubot-react/pull/6


Oops, too late. We can talk tomorrow. It would be nice to have a test
case to expose the uncomfortable behavior. Given all the tests we
currently have, they just behave the same.

On 7/7/15 8:17 PM, Ola Bini wrote:
> Let's talk about this one tomorrow. I feel quite uncomfortable about it,
> which is why I didn't use ReadFull when doing this initially.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/twstrike/AwESome/commit/f0871005558c9003cc932bce3e98f5e724d3f921#commitcomment-12051855>.
>


Should this be in a finally, in case an assert in ModPow1Large2SmallInt fails and throws an exception as a result?


Yep, that's the idea!


Right, I thought of that, but this is in PyAlembic/Foundation.h, and you can't even build PyAlembic without boost, so it should be a given that ALEMBIC_LIB_USES_BOOST will be defined at this point, right?


@teoljungberg @gylaz @mcmire - Cool, totally reasonable. Thanks for the chat guys! Happy coding :palm_tree:


No they don't - you changed the tests from EOF to ErrUnexpectedEOF - for other things that use io.Reader, that change is the difference between expected behavior and unexpected behavior. Which is why the tests were looking for EOF.


Primeiro Commit.


Segundo commit, alterando o primeiro.


Rock-paper-scissors is a hand game that is played by two people. The players count to three in unison and simultaneously "throw” one of three hand signals that correspond to rock, paper or scissors. The winner is determined by the rules:

Rock smashes scissors
Scissors cuts paper
Paper covers rock
Rock-paper-scissors is a surprisingly popular game that many people play seriously (see the Wikipedia article for details). Due to the fact that a tie happens around 1/3 of the time, several variants of Rock-Paper-Scissors exist that include more choices to make ties less likely.

Rock-paper-scissors-lizard-Spock (RPSLS) is a variant of Rock-paper-scissors that allows five choices. Each choice wins against two other choices, loses against two other choices and ties against itself. Much of RPSLS's popularity is that it has been featured in 3 episodes of the TV series "The Big Bang Theory". The Wikipedia entry for RPSLS gives the complete description of the details of the game.

In our first mini-project, we will build a Python function rpsls(name) that takes as input the string name, which is one of "rock", "paper", "scissors", "lizard", or "Spock". The function then simulates playing a round of Rock-paper-scissors-lizard-Spock by generating its own random choice from these alternatives and then determining the winner using a simple rule that we will next describe.

While Rock-paper-scissor-lizard-Spock has a set of ten rules that logically determine who wins a round of RPSLS, coding up these rules would require a large number (5x5=25) of if/elif/else clauses in your mini-project code. A simpler method for determining the winner is to assign each of the five choices a number:

0 — rock
1 — Spock
2 — paper
3 — lizard
4 — scissors
In this expanded list, each choice wins against the preceding two choices and loses against the following two choices (if rock and scissors are thought of as being adjacent using modular arithmetic).


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


Oh, I leave it intentionally to illustrate that customers can call the Fluent API to set the capabilities vocabulary.


Incorrect, the ALEMBIC_LIB_USES_BOOST is for the Alembic lib only,
PyAlembic could be linking against a version of Alembic that uses tr1,
C++11 OR even boost.

Lucas

On Tue, Jul 7, 2015 at 6:22 PM, Ryan Galloway <notifications@github.com>
wrote:

> Right, I thought of that, but this is in PyAlembic/Foundation.h, and you
> can't even build PyAlembic without boost, so it should be a given that
> ALEMBIC_LIB_USES_BOOST will be defined at this point, right?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/alembic/alembic/commit/2f260c47a4dbf3419b759cfa3fbf5cfb1ad3a87f#commitcomment-12051903>
> .
>


You might consider modifying this to take an action that gets executed with the threshold, e.g.
```C#
public static void RunWithFakeThreshold(string name, int value, Action action)
{
    FieldInfo field = s_internalCalculator.GetDeclaredField(name);
    int lastValue = (int)field.GetValue(null);
    field.SetValue(null, value);
    try
    {
        action();
    }
    finally
    {
        field.SetValue(lastValue);
    }
}
```
Then the usage sites would look like:
```C#
[Fact]
public static void ModPow1Large2SmallInt_Threshold()
{
    BigIntTools.Utils.RunWithFakeThreshold("ReducerThreshold", 8, ModPow1Large2SmallInt);
}
```


Ummmmm


This is my very first program - a very inefficient way to code this project. I realize that I could have condensed a lot of the code that is redundant!


Just dump the old file


looks like guru is getting jelly...


@psytronx DWMR (done with my review). Press on!


fast-forwarding master to auto = 23621611


no, that code is no longer needed


97643186-ABUPS-total-row-not-match-water (mjwitte) - x86_64-MacOS-10.9-clang: OK (1673 of 1673 tests passed)

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


MORPH_CROSS 마스크 만들어서
이리저리 깍고 erode시키고
ㄱ런데 minMaxLoc은 mat의 최대최소원소위치구하는건데
최대값 max가 0일때까지라니 무슨말인지 모르겠음


Can you add more information


Official solution, very pretty !
five line into one line, so beautiful
```javascript
function countWords(arr) {
  return arr.reduce(function(countMap, word) {
    countMap[word] = ++countMap[word] || 1; // increment or initialize to 1
    return countMap;
  }, {}); // second argument to reduce initialises countMap to {}
}

module.exports = countWords;
```


あ


maybe not. osxfuse package contains both library and header.


no worries, fixed with 007d44b2d6f71dfe9a9999e3d0a0a5257020291e


Also add a `-y` to `apt-get update`


Fixed my importer. Thanks!


Por que eu achei que ficaria mais fácil
Eu tentei colocar numa controles só, ai parou de funcionar.
Ai resolvi fazer duas controlar

Laís Vidal de Oliveira

> Em 07/07/2015, às 21:41, Edy Segura <notifications@github.com> escreveu:
>
> Por que a aplicação em dois controllers?
>
> —
> Reply to this email directly or view it on GitHub.
>


ナイトメア特殊スキル：巻物使用不可 #422
ナイトメア特殊スキル：薬使用不可 #421


I'm not entirely happy with this submission. I would like to tweak parts of it for concision and rubocop warnings, but right now I'm just submitting it so it won't be late.


Why do you set MinMeleeDmg, MaxMeleeDmg to low values in your commits? According
to https://github.com/cmangos/issues/wiki/Creature_template#MinMeleeDmg

"NOTE: This Field Is No Longer Used By The Core - Maintained For DB Developers
(Nice Field To Have Still)

Minimum Melee Damage Is Now Calculated Using DamageMultiplier"

MinMeleeDmg, MaxMeleeDmg fields should be pre-calculated damage for all creatures,
but you change them by dividing by DamageMultiplier.

Although, looking to current repository code... there is one place,
in Creature.cpp, Creature::SelectLevel(), where cinfo->MinMeleeDmg is
actively referenced. May be it is incorrect current code in repository, which
should be fixed (I do not use this code).


Just found this for Adibags. Is there a way to change the looks of the free space slot? Currently it looks like this for me (Using Flatsquares) http://i.gyazo.com/12581a2ce20f8692e4cfa60eb7339857.png


Any chance this could make it into Sage? I'd love to use the sage `ruleset.xml` for my projects.


I don't believe this change should have been made.


@unclecheese should this be set to the latest tagged stable version of toolbar?


Why do this?


No worries.  Just curious :-)  If there's value in keeping it, by all means, keep it.  If not, I vote for aggressive deletion.


Fixed! https://github.com/rsgalloway/alembic/commit/d1892f0a4e416a4ed4b2bce2f3f05ba1c5ebeb0f


Decrypting fails for key 3 on some of the tests. Not all. Will need more research.


What's the story with this file? It has the header from the symfony project but it doesn't look exactly the same as that version from the same time this was imported (just doing a diff).

Should this be something we extend from sf?


```return``` is unnecessary as the last statement in ```void``` method.  Please remove.


tag description is missing


Mali


having these fixtures is a good thing? why delete them


OK, I just reverted.

I can't understand why we want to avoid this different error in the
padding. I also can't understand why it is OK to use io.ReadFull when
reading from the pseudo-random generator (in otr3) but it's not
desirable to use it here.

For me it still seems that, in the end, we only have it because we want
to handle ErrUnexpectedEOF errors as regular EOF errors?

To make my point, I've made #31. The test changed has no particular
reason to worry about io.ErrUnexpectedEOF
(917432bddbf4367484394bbf5556593245b7faa8) and io.Reader does not
mention io.ErrUnexpectedEOF.

Does it make sense?

On 7/7/15 8:23 PM, Ola Bini wrote:
> No they don't - you changed the tests from EOF to ErrUnexpectedEOF - for
> other things that use io.Reader, that change is the difference between
> expected behavior and unexpected behavior. Which is why the tests were
> looking for EOF.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/twstrike/AwESome/commit/f0871005558c9003cc932bce3e98f5e724d3f921#commitcomment-12051906>.
>


https://github.com/reicast/reicast-emulator/commit/45eba7843f2cb0ee615b56fee3b61fab5e14106c ?


Could break this into a non-anonymous function (meiEditor.something) so it can be reused, as you had it before. Not necessary, especially if you don't plan to reuse the code.


18行目のログイン判定、レコード数が1件かで判定しているけど、
テーブルの構造上、 同じメールアドレス・パスワードのレコードが登録される場合がある。
DBの定義を修正して、PKをメアドにするか、0でないなら成功と判定するかしないとバグる。


Could also make this generic and add to utils


97643186-ABUPS-total-row-not-match-water (mjwitte) - i386-Windows-7-VisualStudio-12: OK (1679 of 1679 tests passed)

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


merging now.  see https://github.com/aurelia/logging/blob/master/dist/amd/aurelia-logging.d.ts for auto generated .d.ts files (which are a work in progress - at the moment)


__NIT__: Forgotten commented out code


What about the different skin tones? Line eight will work fine with different skin tones, but line 18 won't


Can you explain what this is supposed to achieve?
```
		// announce our max payload size as the max payload our client supports
		maxLen := getMaxReplyLen(r, proto)
		rcopy := r
		rcopy.SetEdns0(uint16(maxLen), false)
```


...except for the tcpPort/udpPort distinction. Argh. They are the same though, aren't they? Alternatively the helper could select the port based on `client.Net`.


Fixed an compatible issue with mozilla firefox, where some buttons and icons was not working/showing properly.


__NIT__: typo aftger -> after


Looking good \o/


@JedWatson You forgot to add `href: this._path.append('.href'),` to the `paths` in LocalFilesType.js around line 88


The previous commit failed to push, so had to make some small changes for it to be pushed. You silly git.


Thanks @MihaelaStoica. From the context, I think the type should be DataType.Disk for all VBD data sets, and DataType.Storage for all SR data sets. I will revise the PR.


Yeah probs not. we should probably just prepare headers nice and early so neither methods need to deal with parsing them


At least, \ of PATH is converted to / at startup. See  commit 4076b74ae81c6e6c70203335dac95873b9305bf3.
For mapping, kLIBC has those features, path rewriter. But it requires an additional dll installation and settings. And I know, path rewriter is prior to a real entry. That is, it applys its rewriting rule without checking the existence of a real entry. In case of // notation, it is reserved for UNC.

Nevertheless, this is worth considering.


I suggest posting this around midday US east-coast time on, for instance, coming Tuesday (Eric should be back by then, I think).

I always set the time-zone as well. e.g. `date:   2015-06-16 17:00:00 +1`


* why using "this" e.g. here?
     this->setFixedSize(300, 200);

* set fixed size & title from the designer
* check methods names (on_cancelButton_clicked()) according coding standard


We announce our max payload size as the max payload our client supports. So if the client support 65535 bytes, we query to the fallback server announcing that we support that, and we receive (up to) 65535 bytes that are forwarded to the client... However, if the protocol is TCP, we always use the maximum payload size.


Does board.at check bounds? If not, L309 is scary. @Lieuwex


Haha, yeah. What the hell happened here?


Mach mal ein Re-ordering. Erst primitive (absteigend von 64 Bit) Und dann Strukturen. Das verbessert das Alignment und vermeidet damit Fragmentation.


+GoCoEditServer-Connector


Are the blocks remapped from chisel -> chisel compat on world load? I can do that if it isn't already done.

Also, is stuff like oredicted glass/stone chiselling going to be moved over to this mod? I feel like it could be argued either way.


Where did this namespace come from?


@lieuwex lolwat


Done!


Die können auch const


嗯，这个索引很有必要，我才跑了一晚上，数据已经1万条了。


我就试试


Done, although its not that much of a simplification (see https://github.com/weaveworks/weave/commit/4c94abdbee77aa4e8eb664f8c05774ffb3083478)


:shipit:


@tomsmeding dunno, but nice timing, I already had f1df73718c135d028b123c959c4793c5f2f9cf62 locally. Which doesn't use `Board::at`


Also how do I compile this? Trying to test it out, but it has lot of missing files and errors?


ruthlucas


@oliward Could you be more specific, what needs to go after what ? Use line numbers if that helps.


@saurabhsjoshi: First off, because of @karthikb351 being lazy yet again, this isn't compilable at the moment. It is also very messy, so we are trying to fix it.


cheers. It doesn't save any lines but makes it easier to see what the difference is between the three tests.


Ok. I just assumed there are no supported systems out there using docker-io already, but updating the commit to keep both just in case.


Well done


hmmm


This is not proper change,please use name properly


@NobodysNightmare without this `references`, deprecation warnings will be emitted. I believe though that the deprecation warning is incorrect in this case - the `changesets` association defined on `Repository` uses a SQL string for `order`, not `where`.

I'm not really sure what to do here: I can't set `config.active_record.disable_implicit_join_references = true` until plugins are dealt with. On the other hand, _extra noise_ from core classes will make upgrading plugins tougher. I think the best solution is to wait until Rails 4.1 and then do another pass, removing unnecessary references.

Even better, once we're on Rails 4.1 - thanks to a fix for https://github.com/rails/rails/issues/15405 -, we'll be able to rewrite scopes/associations `order` statements

```ruby
order("#{Changeset.table_name}.committed_on ASC, #{Changeset.table_name}.id ASC")
```

as:

```ruby
order(committed_on: :asc, id: :asc)
```


No need to test against dots because they all belong to your adress anyway.


asaaaadxasc


parsingConditionsChildrens parsing should be made using for loop for reducing code duplicating


Given that rcopy isn't a copy (r is a pointer), rcopy already contains the EDNS size fields that r contains.  So why copy it over?


@NobodysNightmare

> Probably we should agree on a best practice to always put the references immediately after the corresponding where clause.

I've tended to put it at the end of the chain, but you're suggestion seems better.


dsdasdas


__NIT__: basically all files are missing newline at end.


Ok I can help out! First off what's up with the Network class, most of the imported classes are deleted like FriendEvent and SuccessEvent and then the Network class is used in the login activity so it does not compile without that.


Line 1-4 of this file:

https://github.com/kevin-chilcott-sift/vagrant-project/blob/126500cb00037baab3721785ebbe9754359dc400/homework-form-processor.php#L1

Looks like this:

<html>
<body>

<?php

So HTML tags are passed to the browser before you do anything at all in PHP, which means you can't then do the redirect in your PHP code.

See here for info: http://www.w3schools.com/php/func_http_header.asp

So what you'd need to do is start your file like this:
<?php
// if something redirect, else continue
?>
<html>
<body>

BUT, as I say, you don't need any HTML output at all from this script, as it is just doing processing for you and either forwarding/redirecting to /homework-form-index.php or /homework-form-success.php.

So I'd remove <html> and <body> tags entirely, and open your script straight into <?php


Working with 2 nodes! Thanks


Sorry, forgot to change it.


Draw, por qué un return, y después + código?


@saurabhsjoshi: @karthikb351's one job was to clean up the Network class and implement the dispatch() method. He said he'll finish by Saturday or Sunday, but still hasn't started, I'm sure.


OS/2 support LX and NE. In fact, MZ is dos stub. LX is 32 bits maigc. NE is 16 bits magic.

execve() in os2.c already checks those magics. By the way, exec() of other OSes does not check the executable magics ?

BTW, the reason why this commit is not needed, is that this fork is specific to OS/2 ? I think, other OSes as well should not execute OS/2 executables as a script.


@oliward  Thanks, that makes perfect sense.


* remove the code that should not  be in the destructor
* why using this-> while you can call methods directly?
* use const QString &titleText because of optimisation where needed
      void ErrorDialog::setTitleOfWindow(QString titleText)


Most of them were just wrong, standard damage output for elites, or worse. I am not afraid to abuse the field to set proper damage multipliers.

e.g. Rend was hitting harder than Ragnaros, with an attackspeed of 11xx ms, the Chromatic Elites next to Drakkisath like sissies.

There is no need to keep this, but if you really, really want to, you can look at older releases.


Last one: if you document the type of the field here, your IDE won't show 'method not found' and also support code completion for it


__NIT__: case would be nicer.


regionfree.bin is missing!


debugging


Err, last last one: one whitespace too many before closing brackets


done


okkkbut


done und gut zu wissen


Is this the case in Celery 3.1? It seems that queues is missing there


I'm fine with using the wrap class. I like it and I want to implement namespaces as well.
It's unnecessary here though: you already have  $wrapperelement, you don't need to make jQuery search the DOM again.


The problem is that we need tab separated values, while I think that printf aligns values with spaces.


Yes, I did it using my username and password right before pushing the
changes, so I did it using HTTPS protocol instead of SSH.

I could be wrong because I don't know anything about the network
configuration here at ESAC, but it seems that the firewall doesn't allow
any incoming connection using port 22 (SSH). That could be the reason why
we cannot use any public/private key authentication but we can push changes
using HTTPS protocol (which works on port 443)...

2015-07-08 11:47 GMT+02:00 J. McAuliffe <notifications@github.com>:

> So did you do this commit with a password?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/johnnycakes79/epys/commit/286f445b56060fc86d87b186dfc748daf768b27d#commitcomment-12057524>
> .
>


can you explain the logic, that executes in this if-else block


should be moved to constants


You are right: that is a bug.


Ok let me look into it. So what do I do about the events in Network? And what does dispatch() do?


Not particularly keen on decrementing i so it can be incremented again.  I'd prefer:
```
for i := 1; i < len(ids); { // shuffle down any duplicates
	if ids[i-1] == ids[i] {
		ids = append(ids[:i-1], ids[i:]...)
		continue
	}
	i++
}
```
If it compiles...


jo hatte ich an ein paar stellen vergessen.


@oliward I mean the validation of the submitted form data is not working, as even if the fields filled in are incorrect or missing, the user automatically gets transferred to the success.php file. This comment is relating to the first line of the one above.


Oh, I thought that the difference between breaking and non-breaking is that breaking uses `Meteor.defer` and non-breaking throws an error. ;-)


In the unlikely event that it passes LTC, I'll rewrite this w/o `reinterpret_cast`. In the best solution is to carve out CMH into a separate class. Will add a few lines of code, but will make the code easier to understand and to modify, which is more important.


using contains method of map is more suitable solution


done


totally cool, no worries


Why would a defer help?


oops, yes it should be


I just hope you are not introducing any dangling Fiber references. ;-) Sometimes it is useful to set references explicitly to `null` to be sure, see [here](https://github.com/laverdet/node-fibers#garbage-collection).


Yes, I'm using gmail. Does this matter ?

I know gmane and newgroup of mksh. However it is full of SPAM. So I think that it is not maintained. Anyway, if I send something to gmane and newgroups, it is delivered to mailing list as well ?

What I want to say at that time is that you can accept OS/2 codes again. Because I could not contact you, I forked this repo.


default initialization of an object of const type ... without a user-provided default constructor


@saurabhsjoshi: Can remove all traces of events in Network, we are using Event Bus only for Activity signaling the Fragments. dispatch() calls the HTTP requests in the order it is meant to and based on the NetworkConfig object that it gets. Also, it takes care of Request Throttling as outlined [here](https://github.com/CollegeCODE/VITacademics-Glass-iOS/issues/1) and the errors. Talk to @karthikb351 about this, it is complex but not difficult.


i cant make it work, please could u tell me how i can use this module just to extract 1 gmd, i am trying to use it for phoenix wright 5, 3ds version, the gmds probably are the same but i cant run the module.

So please, what i need to do to extract the gmd what changes need the module?


Or maybe `Meteor._sleepForMs` and then retry. Because in meantime guarded sections might exit and free dependencies. No?


Hello Christian,

Man we can't thank you enough. You have been a great help, probably just saved.


No. The deadlock exists because all the fibers are blocked on each other. So none of them will run unless the deadlock is broken somewhere.


I've removed the truncation of recursive responses (in https://github.com/weaveworks/weave/commit/71594ddc83c4e105b68544cede6e27d5e6977449)


TeamCity Jarvis :: Jarvis.DocumentStore :: Debug / Test / Release [Build 573](http://build.prxm.it:8811/viewLog.html?buildId=3425&buildTypeId=Jarvis_DocumentStore_CI) is now running


Yes, I did set all the fiber containers and variables to `null` after use. Ok, I see that I didn't do that in tests, but that shouldn't matter.


Dann lass eine DEBUG Konstante einführen. ;)


Sorry, pushed it too early. I'll get back with a decluttered version.


Shall I open another issue for it? Why exactly `AM_PROG_AR` is failing?

I've seen this mentioned in https://github.com/sass/libsass/pull/973#issuecomment-86930415 only...


broke the windows build

```codegen.cpp: In function 'void jl_dump_bitcode(char*)':
codegen.cpp:422:35: error: 'globalvars' was not declared in this scope
     jl_gen_llvm_gv_array(bitcode, globalvars);
                                   ^
make[1]: *** [codegen.o] Error 1```


in大阪の告知を更新


Nice.


@kevin-chilcott-sift I think your issue is that you are unlikely to get to this point, as it will have redirected them above. E.g. line 31-36. If you have an empty $_POST['firstname'] it sends them back, else it sends them forward.

So, it looks like you don't need line 31-57 if you're instead checking for empty in your foreach loop here.

You also have an issue in your foreach loop that if there isn't an issue in the first field it checks it will immediately redirect them, without checking the later fields.

I'd suggest changing it so if there is no issue it does nothing (but goes to next item in loop), so it only redirects them back if there is an issue.

THEN, once your foreach loop is complete, it can redirect them to success, as, at this point, you will have done all your checks, and must have found no issues.

And, FYI, you want an exit; on the line after the redirect, as at that point you don't want it to continue to process the script. You are done, and just want to send them back, or forwards.


This is just a poor workaround. Any symbol defined by the compiler may break our code this way. Another example is `<windows.h>` breaking `std::min()` and `std::max` (https://github.com/saper/libsass/commit/fa45c3cc19466e0ff22c492a7095fad1af0ceecc)


there is no need in empty line before if
if-else statements marked out only when they are placed on one level with other instractions


Why rename the class there?
I can't see the reason :confused:


Broke the OSX build as well: https://travis-ci.org/JuliaLang/julia/builds/70015237


Ah, so tabs are necessary. I thought from the commit message it was simply changed to remove fixed length. Thanks for the explanation :thumbsup:


should be an empty line before and after is statement


:+1: :shipit:


`if (!match)`?


Wasn't this commit, was 050eedb5ff0a072cb05e91ac6e827e8322ba2c2b, but this was the only push.


wat


ok


Maybe not return only unique values @sindresorhus @arthurvr?


you realise bom isn't going to work anyway right


browsersim yes, but should I chane this line for VPE features?


My first appearance in other's tests! :laughing: :rocket:


I think awk would also work with non-tab spaces. Because I believe awk processes tabs and whitespace the same as 'empty' in between the 'words' it counts.


Thanks for making this useful update. Much appreciated!


I'll do that
This actually looks buggy to me too, since having a double db resource set...
```php
mysqli_query($serendipity['dbConn'], "SET NAMES " . SQL_CHARSET, $serendipity['dbConn']);
```


Just read a bit about what inline functions are, and I agree with the change now ;-)


if @current_user_roles is seted up - you can avoid executing query using structure `||=`
Final code will be `@current_user_roles ||= user.team_roles if user`


This try/except will not have much to do. Only if creating the thread fails it will do something. Any exception thrown in the library scan will be caught by [this](https://github.com/rembo10/headphones/blob/master/headphones/logger.py#L201). I suggest to remove the try/except or to catch a more-general exception (which is always good instead of catching Exception).

I also see you create the target incorrectly. You now first run the scan, and use the result as a target. Guess you mean this:

```
threading.Thread(target=librarysync.libraryScan, kwargs={"dir": artistfolder})).start()
```


TeamCity Jarvis :: Jarvis.DocumentStore :: Debug / Test / Release [Build 573](http://build.prxm.it:8811/viewLog.html?buildId=3425&buildTypeId=Jarvis_DocumentStore_CI) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 289 Build time: 00:13:40
### Failed tests
```
Jarvis.DocumentStore.Tests.dll: Jarvis.DocumentStore.Tests.SelfHostIntegratonTests.DocumentControllerIntegrationTests("v1").verify_de_duplication__not_delete_original_blob_before_15_days: <no details avaliable>

```


These changes have nothing to do with the task you're solving. Get rid of these spurious changes


What's the use-case for this?


Well, create it then I guess?


why was this? I think you need the boundary treatment before doing streaming or anything if you want the correct behaviour of it actually being there


@Kubik-Rubik see: https://github.com/joomla/joomla-cms/pull/7377


To make this nicer https://github.com/kevva/viewport-list-cli/pull/2. Basically I want to convert an array of objects to a table.


You should remove `?>` from end of the file.


can we keep method objects? they are more debugging-friendly :L)


ok


test


please, answer can you use methods from standard string class like replace() instead of for loop


yes, thank you!

On Wed, Jul 8, 2015 at 3:59 AM, Adrian Wielgosik <notifications@github.com>
wrote:

> couldn't they be just this?
>
> eqTests["add"] = ()=>pair(parse("1 + 2").ToString(), "3");
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/joebo/microj/commit/7d0501ff01c30ef1b854724164947ddd685f7cb0#commitcomment-12055548>
> .
>


think you've left this `.only` in the tests by mistake?


^^wenas:
¿¿montamos la SDK de kinect aqui??


Thanks ;)


Ik heb de hook naar register geactiveerd. Als je nu een release maakt zou de boel automagisch overgeheveld moeten worden naar /staging.


I've decided it's best to follow Paulo's suggestion (via email) to have two separate rules for `makeglossaries` and `makeglossaries-lite.lua`. I think that's the most sensible route. That way users know what they're asking for.


a have already mention about considerations about for loop instead of code duplicating for creating parsingConditionsChildren objects


Why inventing your own warning text? The compiler message is "Illegal pattern" - what's wrong with that?


1)no need in empty line after result.repeatElement.addAll(wrapper.repeatElement);
2) need empty line after for loop


I thought of it, but it adds slightly more complexity. Well, I'll think a bit more how to do it


The duplicate has been removed in another pull request.


Never execute queries in a `for` loop. Use `prefetch_related` to fetch attribute values together with attributes.


What section is this under?


918ee650539fbe3b50ea5e1efb915ebe7cbdfe31 fixes the build failure, though there are still warnings `not a recognized processor for this target (ignoring processor)`


And done !


Can it both be valid AND return `None`?


Can we get some more info on BoZoN, before I merge this pull request?


例えば、
```python
class filterOperation:
    xmlCodes ="""
XMLのコード
"""
    def __init__(self, im_input):
        self.width, self.height, self.dim = im_input.shape
        self.im_mask_k4qq3nj4 = np.zeros((self.width, self.height), dtype=np.uint8)
        cv2.rectangle(self.im_mask_k4qq3nj4,(140,10),(650,470) , color = 255,thickness = -1)
        self.im_mask_k4qq3nj4 = ~self.im_mask_k4qq3nj4
        return
    def filterFunc(self, im_input):
        im_output = cv2.threshold(cv2.bitwise_or(cv2.cvtColor(im_input,cv2.COLOR_BGR2GRAY),self.im_mask_k4qq3nj4),110,255,cv2.THRESH_BINARY)[1]
        return im_output
```
というファイルが保存されるようにしてみれば？


this lambda is never executed, resulting in the coverage going down. One solution would be moving out both lambdas into one function because later in execute_section that function gets actually called. So a `def do_nothing(): True` would do the job


pull 추가한거야


Line 22:
```        $this->assertTrue($integers[2] >= -1000 && $integers[2] <= 1000);```
should be
```        $this->assertTrue($integers[3] >= -1000 && $integers[3] <= 1000);```


why not just `expect(error).to be_kind_of Transproc::Error`??


you have to remove the get_metadata method also right?


IE 9 is not catched but it seems to work fine. However IE 8 is not catched too although the interface is broken.
So I will remove the commented lines in a new commit but you need to fix the unsupported browsers detection :/


Because it needs to be available in the execute phase and `c_string` will not be valid at that point and we also tear down the `import` object.


test


Please, do not interpose functions and fields, it makes it harder to read your code. Also, name your entities properly - this 'visitor' is not just a visitor - it does something.


Seems legit, thanks!


We can remove this if it is not being maintained.


**NEG** *rd*,*rs*/**NEGU** *rd*,*rs* are not instructions per se, there are special aliases to **SUB** *rd*,*$0*,*rs*/**SUBU** *rd*,*$0*,*rs*. There is no reason to handle them as they is an option to enable/disable them. And honestly, you should disable them to make your life easier (no need to handle NEG[U], NOT[U], LI, LA, etc.).


no need in empty line


Can we please keep the demo in there?


So good job you proof read that then, what with my multiple routers and copy paste errors; plus I was really careful to get the Rock-on naming standard and still missed it.  I worked perhaps a little faster than I was able but thought it important and timely to get it done. Thanks for sorting.


HTTP Server OS X/Server/Controllers/UserController.swift

Hi @paulofaria. You should read https://github.com/groue/GRMustache.swift/tree/Swift2#note-to-library-developers.


D'oh! This is because I used to declare subjects and (the shame, I know) implicit validation ;)


no need in empty lines before and after instructions inside the if block


Can we keep the source code in there as well please?


This whole spec could be written as a single test checking that `Transproc.[]` raises proper error and it could regex-match message rather than being so strict. Otherwise it's gonna be quite brittle and too close to implementation details.


Isn't this visitor exactly the same as the one assigned to 'visitor' field below?


Once again, use the compiler's messages when possible.


Does this not help show the parameter is optional?


whoops, if I had any coffee yet I would've rearranged the ifdef to avoid copying the two lines that are the same now - there's still a test failure in `linalg/pinv` though https://travis-ci.org/JuliaLang/julia/jobs/70041689

```
exception on 3: ERROR: LoadError: mismatch of non-finite elements:
  a = Inf
  0.0 = 0.0
 in test_approx_eq at test.jl:132
 in test_approx_eq at test.jl:150
 in anonymous at no file:298
 in include at ./boot.jl:254
 in runtests at /tmp/julia/share/julia/test/testdefs.jl:197
 in anonymous at multi.jl:837
 in run_work_thunk at multi.jl:590
 in anonymous at task.jl:837
while loading /tmp/julia/share/julia/test/linalg/pinv.jl, in expression starting on line 294
	From worker 2:	     * linalg1              in 104.76 seconds
	From worker 5:	     * linalg/cholesky      in  35.35 seconds
	From worker 4:	     * linalg/triangular    in 110.26 seconds
ERROR: LoadError: LoadError: mismatch of non-finite elements:
  a = Inf
  0.0 = 0.0
 in anonymous at task.jl:1403
while loading /tmp/julia/share/julia/test/linalg/pinv.jl, in expression starting on line 294
while loading /tmp/julia/share/julia/test/runtests.jl, in expression starting on line 5
	From worker 3:	     * linalg/pinv
```


Hi - what is the namespace about? Can you let us know who you are?


I prefer the old one. This commit breaks the notion of responsibility.


zzzz


isn‘t the contract more like
```js
assert.calledWith(shallowRenderer.render, new ComponentDouble());
```
?


Can we change name/group/etc to something like `my-cool-microservice` so that it's obvious that it's for the use project?

And add spaces before and after `=`.


Also, this is a constant - use a static final field for it.


Done in b93806fed189e289e1d52fa88fee9570909fa5e3


We talked about this a while back but I kinda lost track of it. Anyways, it is probably save to close this issue now. There is a small possible performance gain with a variadic method I suggested if you are still interested.

See http://dev.clojure.org/jira/browse/CLJS-890


+ some other stuff.


this is right version


if you cover some statements in try-case block, it's allowed to add logging instructions in catch block. so, please, add it.


 Посмотрел я qwflash. Сама утилита работает правильно, просто я не исправил описание ключа -w в хелпе. На самом деле так надо указывать только имя файла и имя раздела. Хелп я исправил.
Выглядит процесс так:

$ ./qwflash -s ptable/pt-mf825-w.bin -w 00-0-MIBIB.bin:SBL1 -w 01-0-SBL2.bin:SBL2 -w 02-0-RPM.bin:RPM

 #  --Раздел--  ------- Файл -----
00  0:SBL1          00-0-MIBIB.bin
01  0:SBL2          01-0-SBL2.bin
02  0:RPM           02-0-RPM.bin

 secure mode...ok
 Отсылаем таблицу разделов...ok
 Запись раздела 0 (0:SBL1): байт 268288 из 268304 (99%)  ... запись завершена
 Запись раздела 1 (0:SBL2): байт 263168 из 263276 (99%)  ... запись завершена
 Запись раздела 2 (0:RPM): байт 105472 из 106316 (99%)  ... запись завершена

Учти, что для записи используются не просто дампы разделов, а дампы с отрезанными хвостовыми ff (с помощью ключа -t в qrflash), или mbn-файлы из фирменных прошивок. А разделы efs и файловые разделы линукса вообще должны иметь специальный формат.

Я как-то давно уже не пользовался qwflash, проще записать сырой дамп через qwdirect.

--
Alexander Stepanov


Looks like the Trello plugin version changed.


why aren’t you returning the result?


dist is not updated


You have a lot of visitors using one another. It's somewhat hard to read. Try naming them, it will make your code clearer.


Also this line should come after
```
this._rendered = true;


The inspection doesn't work in some cases. For example:
```erlang
foo(foo() + 1) -> ok.
```


Make sure all tests are passing before you submit a pull request.


Thx... So all pseudo instructions can be disabled? But how then?


Aici „fă cunoștință” mi se pare o variantă mult mai bună. „Cunoaște” nu e la fel de sugestiv și pe aceeași linie cu „meet”. Dac-ar fi să reformulez, aș face de genul „Fă cunoștință cu cel mai personalizabil Firefox pentru Android de până acum.”


You are also overriding `appScheme` here. I that intentional?


The recursive resolver is tested in the integration tests.


我设计了设备ID，还是直接用设备ID吧


Rather interesting because build 12880 does not seem to exist, whatsoever.


Hi Antony,
This commit prevents the translations.


能直接用Json


maybe ````onComplete```` or ````completionFunc```` instead of ````done```` (which is nice and short, but reads a bit like a boolean)? Unfortunately, we don't have a standard name across all of sclang for such callbacks.


It'd be good to remove the import of `fake_geocoder` from this file, since it'll now be shadowed by this definition


does this fix moving apps to sdcard? are there other commits needed?


This visitor will visit the whole tree. Is this what you want it to do?


I merged upstream in, it apparently wants to get merged in again.


ok


I cannot see what is freeing `abs_path` ... that's why I'm asking...


It's a set.


Gesinnung – siehe [CONTRIBUTING](https://github.com/runiq/dungeon-world-deutsch/blob/master/CONTRIBUTING.asciidoc#Übersetzungen-für-häufige-begriffe).


`>>> tuple([1,2,3,4])`
`(1,2,3,4)`
`>>> (1,2,3,4)`
`(1,2,3,4)`

`tuple()` constructor is only required when you want to create tuples out of a different datastructure like list, same as you have here. you can create tuples directly if you have elements.

No need to create first a list and then convert them to tuples


`test/290_*` only checks that we can send a DNS query to Google's servers. That cannot be considered a valid test for the recursive resolver. Those tests [did make sense some time ago](https://github.com/weaveworks/weave/pull/392#discussion_r24504744), so I think the same requirement still applies...


ahbkjk


What if we don't free user-supplied strings and the path pointer stay as provided by the user?


@bfogg1987 did you write this file? Looks great! If You didn't and its automatic, make sure to look over it how looks. This is how Controller tests should look/be laid out :)


we should now store this value in a class variable and `__init__` it accordingly.
so that we can change the path our file just by changing the parameter while creating class


Same here, this is really cool if it generates these. That must be new in Rails 4!


Something about Windows. close #1


How so?


`/usr/local/bin` - `/usr/bin` - `/bin` missing ?


chybí středník?


seš si jistej že tohle tam pošle ten element? nemělo by to být spíš `settings.nette ? settings.nette.el : null`


Oh right, I got the order wrong. Will fix!


As a tiny bit of DRY refactoring, you could have:

```python
post_1_url = reverse('info_blog', kwargs={'slug': 'post'})
post_2_url = reverse('info_blog', kwargs={'slug': 'post2'})
```

... at the top of this section


> So does "Atom's packages management" not mean going to Settings->Packages?

It does. Go **File** / **Settings**, click **Packages** tab, then under **Community Packages** section you find *keyboard-localization*. It has 4 buttons, click on the **settings** one. A new page will open, showing the plugins details. You find the **Settings** section, there you go


Nop, those are under $PATH


jsem, JS z `&&` vrati posledni truthy hodnotu


Argh !!!! you cannot :( because it is done inside captstone : https://github.com/aquynh/capstone/blob/master/arch/Mips/MipsGenAsmWriter.inc#L5325

geez, they dare to mix pseudos with real instructions :(


The `PATH=$PATH` should be adding those


:+1: for the use of `F` :)

I'd split this as:

```python
            ViewCount.objects.filter(page=self.object, date=date.today()). \
                update(count=F('count')+1)
```

... to keep under 80 columns


`s = '#u\tUsername\t'+str(x)+'\t'+str(y)+'\t\n'`
or
`s="\t".join(['#u',"Username",str(x),str(y),"\n"])`
I find these two more readable.


true. done is a node convention

this is a complicated app to build in supercollider. it needs async io and promises.


I'll make a readme.


got it, ty


This approach is a failure yet to explode.
The problem is not within the module, but within the Core API Media::addJsDef that SHOULD ALWAYS use json_encode to store the pass argument.

Because Media::addJsDef() need an urgent patch (affects various places) it will be receive an emergency patch soon and this PR will cause a double escaping


An alternate fix for 'Route Id not defined' issue (https://github.com/libis/CAG/issues/210).


In angular it's common to work with modules API using method chaining.


You're right sir! Ja myslel ze to funguje jen na OR, ale to by asi bylo hloupe :)

![snimek obrazovky porizeny 2015-07-08 13 24 00](https://cloud.githubusercontent.com/assets/158625/8569149/9c5a9098-2574-11e5-9341-f32d7f58684e.png)


Thanks! Do I have to add that command -nodefaultcubemap to the vbsp commandline now? And How i should create my func_detail_blocker entitites in hammer? Trigger texture only?


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


If you make this `.order_by('-viewcount__count__sum', '?')` then if there are posts with the same number of views they'll be in random order, which seems quite nice to me.


This doesn't work, it's not a template string.


Hello There,
I had begun and already implement the same ideas. How and where would you define for this assignment?


Can you please change it to https://github.com/black-perl/ptop#fork-destination-box , then i'll merge it. Thanks for this, I set the page up in hurry :)


Other than adding more tests for the DNS code, do you have any more feedback?


Done


(vím že tady není potřeba, ale všude je píšeme)


Please revert this change.
On Android this change means that the plugin is used even though the implementation of console on Android is good and does no enhancement by this plugin.

The consequence of this change is that now all console output looks like this:
I/chromium(28763): [INFO:CONSOLE(173)] "..I.. (myApp:MyApp) closeApp()", source: file:///android_asset/www/plugins/cordova-plugin-console/www/console-via-logger.js (173)
That is the file and linenumber information are worse than without the plugin.
Now the developer does not see where the console.log statement is in their code but the always see console-via-logger.js (173).


Typo in commit message: Wrong issue ID.


- test 1


Instead of re-writing this disposal logic, shouldn't we just inherit from DisposableObject and override DisposeResources ?


These seem like good contributions to add, though I'm not going to ask we do them now. @antoviaque thoughts on this?

I guess I'm ok with leaving out bok_choy for now, since you're at least using the same code structure, since that means if the things you're looking to do are eventually implemented, it shouldn't be /too/ hard to switch over. Even so, it's a bit bothersome we're reinventing here.


shouldn't this be setIdentity rather than setDefaultIdentity ? Is there a reason that you keep the old commented code ?


It looks like that compilation fail because this file is missing:
[ERROR] Line 29: Failed to resolve 'org.traccar.web.client.i18n.Messages' via deferred binding


Ah, Sorry. I ll fix it now.


you don't have to describe the day in a chronoligical manner, you can just have items with the various topics you worked on with possibly some explanations, tables, figures, results associated
this way you have also things to put later in your final report


I only have an incomplete list, which I just updated: <https://github.com/vadmium/python-altium/blob/master/format.md#record31-sheet>. I have only seen A4, A3 and A so far myself, and I don’t have Altium or Windows installed at the moment to check for any other formats.


Doing seek and overwriting the file contents is not a good idea.
What if the file already has contents that are longer than what you are writing then. Then you might overwrite partially. Its better to close the file. And reopen it in [`w mode`](https://docs.python.org/2/tutorial/inputoutput.html#reading-and-writing-files).


doesn't "exist" - fix grammar :)


`digit` is a single numeric character. In this case more appropriate word is Amount


Thanks for the feedback. It was really helpful. I will update my changes and open a PR.


why `index()`?


OK, makes sense.


this file handling class should return an error. (it shouldnot be aware of to show error). Instead I would suggest why not put this check even before calling this function.


yep that would be better, unfortunately then you cant use JSX, because that results in `render(React.createElement(ComponentDouble, {}))` and not `render(new ComponentDouble)`


@piquadrat IMO `ClientState` should not be logging. It is there only to keep track of success/failure rate and to stop us from sending when failure is high. Why not log from `client`? it already has the logger etc.


good point ... trying to use it in a couple of tests so the API can take shape


You should decrease visibility scope as match as possible. This kind of constant should be declared inside `directive function`.


It would be interesting to know what is the source of stat values in latest commits?
Is it printed manual from TBC time (with data before 2.3.0 patch), which Patman64
mentioned once? Or some other source?

And just for information, comparision of damage values:

Chromatic Elite Guard:
from this commit: (137+101)/2*9.1 / 2.0 = 541.45 dps
from Classic DB 1.1.1: (528+681)/2*1 / 1.175 =  514.46 dps

- not very much difference

more from Classic DB 1.1.1:
Warchief Rend Blackhand (2064+2309)/2*1 / 1.0= 2186 dps
Ragnaros (181+253)/2*35 / 2.8 = 2712.5 dps

Interestingly, YTDB uses different values for Ragnaros, but they lead to almost the same DPS:
Ragnaros (164+246)/2*26.4 / 2.0 = 2706


Very well


Este cambio hace que los bower-assets no se actualicen correctamente.


I see, so it is not only about the content of the block, but also about the usage of it :)

````ruby
values.reject { |k, v| k.to_i < 1 || !v.is_a?(String) }
values.reject! do |k, v| k.to_i < 1 || !v.is_a?(String) end
````

Thanks for clarifying :)


I guess we could have `handle_success` and `handle_fail` on `Client`, which logs and then sets the correct state?


Corregido con https://github.com/chamilo/chamilo-lms/commit/900d04ebb130c179b444d8b775b53ed06055475e


Not sure what print statements you mean, but there already is `screen:redraw_debug` that prints both the redraw events as well as all resulting intermediate states.


@flavour Please review


Oh, you can ;-) https://www.mirbsd.org/jupp.htm is a good editor for this, for example (would be interested if you were to OS/2-ify it too).

Otherwise, well, that’s just going to be a point of divergence between mksh and mksh-os2, nothing wrong with that either.


Hi @doekia,

Thanks for your comment. But I believe you are wrong since htmlentities does not escape:
http://php.net/manual/en/function.htmlentities.php

e.g:
"'" (single quote) becomes ''' (or ')  - No escape will be applied on this
http://sandbox.onlinephpfunctions.com/code/127cf9728298e5bf1290723c5bd1f2d56ded1df6

Best regards,


Thank you :)


That is cool with me. They didn't have one in atlas though and I just
wanted to try something out. :) Please feel free to totally rewrite.
On Wed, Jul 8, 2015 at 3:28 AM John Ericson <notifications@github.com>
wrote:

> IMO if would be easiest/best to do have nixos VM. We can also easily rig
> up node as daemon/service. What do you think of that?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/unisonweb/platform/commit/35e947dc4cebbae750a9bc67ed62ba7c0c2db24e#commitcomment-12055046>
> .
>


Thanks.


OK. I just saw your repo and thought to comment on a few things “offhand”, i.e. without thinking much beforehand, but to start a dialogue of sorts. Please just consider this as food-for-thought. It’s just a “couldn’t we do this” question, with an “if not, why not?” so I learn more about the kLIBC setting.


working


> The tradeoff however is that if we take too many entries per-thread, there is a risk of killing the level of parallelism because now all these entries are processed sequentially by that thread

Yup yup, however, I think this only applies if there are not too many entries in the queue. In that case, assuming a 200ms processing time per event, dequeuing it all would take ~1.5 minutes (upper bound). I'm not sure this scenario is likely to happen though, unless there is a sudden spike (otherwise, another thread would have picked some up, we wouldn't have waited until the queue has 500 entries to start processing them).


This file is used in mobile version of web UI, which is written in pure javascript + html. This warning is about GWT compilation, thus is not related with this deleted file anyhow.


I think that we can safely remove this since now the deleteDirectory doesn't throw any exception.


We should still keep this exception becuase it can be thrown for some stores (like CMISStore)


I think that this can be removed.


Don't indent stuff based on name... Put a newline before "map fst".

Use $ $ $ at end of lines to have uniform style with the rest of the code.

Use M.filter to make your filter simpler. Or better yet: use a list comprehension like:
[ k | (k, Db.FileDescNonExisting {}) <- M.toList elInputDescs ]

This may leak a lot of memory if compat makefiles aren't needed?  Perhaps this data should only be collected conditionally?


Yeah, i'd go for that :+1:


Warning: Not working currently


Rename to paths?


Right now, in mksh itself, this code prevents the shell from executing MZ as scripts. I will add LX and NE (I knew about them, and LE of Win32) to mksh, so it prevents trying to run OS/2 executables as scripts everywhere. The patch will be like this then:

--- exec.c~     2015-07-08 13:45:04.089402037 +0200
+++ exec.c      2015-07-08 13:45:58.693130592 +0200
@@ -970,3 +970,4 @@ scriptexec(struct op *tp, const char **a
                    (m == /* ECOFF_SH */   0x0500 || m == 0x0005) ||
-                   (m == /* "MZ" */ 0x4D5A) ||
+                   (m == /* "MZ" */ 0x4D5A) ||
+                   (m == /* "NE" */ 0x4E45) || (m == /* "LX" */ 0x4C58) ||
                    (m == /* gzip */ 0x1F8B) || (m == /* .Z */ 0x1F9D))


I thought `order` statements would not require `references`, according to [What's new in Active Record [Rails 4 Countdown to 2013]](http://blog.remarkablelabs.com/2012/12/what-s-new-in-active-record-rails-4-countdown-to-2013):

> Also, ordering string SQL snippets on included associations will still work the same way without the need of references:
> ````ruby
> Team.includes(:members).order('members.name')
> ````

But since you receive the deprecation warning I take your point. I am just careful when it comes to new stuff I didn't know until now ^^

Lets wait for Rails 4.1 and see if things improve :)


camelcase?


Je pars pour Liverpool aujourd'hui, je risque de ne pas toucher au repo pendant… longtemps :)


Space missing inside }.


Prefer single-quoted strings when you don't need string interpolation or special symbols.


Redundant curly braces around a hash parameter.


Separate every 3 digits in the integer portion of a number with underscores(_).


Space missing to the left of {.


Avoid apostraphe for non-colliding names. Extract lambda to its own "where" function with pattern-matching True/False instead or name it "tIsDir" or such?


Yes, Google is incapable of doing eMail right, so it matters.

GMane Loom (webinterface), GMane NNTP and the mailing list share the same messages. It is not required to be subscribed to post, hence the spam (trying to cut it down, got over 90% less than before the anti-spam measures, but some will still go through). Anything you send to either the ML or GMane (via web or NNTP) will be posted to the mailing list, and the reverse.


`Attachment.find(..., _join=AttachmentFolder)`


Perhaps replace this with partitionM into two lists?


I forgive you ;-)

Overall you made the styling there much more consistent than before.


Sure but that rule will look for dynamic_big_crypt.c which doesn't exist. You need a rule for dynamic_big_crypt.c too and I think it can be identical to the one in Makefile.in


Could you add what you do in the commit message, update just makes me look at what happened here? just "add kltedv" will do


Why do we need this?


That's not a good idea. It works for `FileSystemStorage` but any other storage that's not local will delete its temporary file when the context manager exits (so `fs_path` should also be a contextmanager or simply `return self.storage.get_local_path(self.storage_file_id)` directly)


remove several conditions for displaying editors.
    <!--<? $editors = $this->driver->getEditors(); ?>
    <? if (!empty($editors) && empty($authors['main']) && empty($authors['additional'])): ?>-->


Manda bala


`await*` !! ?


Why did you change the list comprehension instead of adding a clause to it?


wouldn't `posixpath` instead of `os.path` be better or does zipfile handle both `/` and `\` properly (not that any sane person would run indico on windows...)?


Use `sortNub $` or such instead of `Set.toList $ Set.fromList $ ...` ?


damn


no need to include the folder id - using the attachment id would be enough


The parenthesis are redundant (use hlint?)

The line is too long.


```
with attachment.file.fs_path as tmp_path:
    self._addFileFromSrc(dst_path, tmp_path)
```


Does it mean the payload is shortened by one byte, or is the first byte overwritten?


I think the default value returned by `get()` is already something falsy


`t` is a targetPath, why `t`?
Use newline before comma in list comp (if rename makes line long).


@mpele I have fixed compilation errors in https://github.com/vitalidze/traccar-web/commit/4204266ccf424036746035238af621075bdb7289 Please try latest `dev` branch.


Oh right, I was still looking at the legacy file but I thought it was Makefile.in.


isnt an `else` missing over here


What about moving the whole legacy crap into a `_offline_download_url` function in this file so `download_url` stays more readable?


This is a breaking change so should probably have been released in a major bump instead of a minor one right?


This should be extracted to some `Maybe CompatMakefile` structure, since the "Existing" thing here is a hack since `make` doesn't support non-existing inputs (and because we shouldn't collect this stuff if we're not making a compat makefile).


This should be http://deliver8r.github.io/deliver8r-parent/${site.path}


What is this supposed to do?  What prevents the device from updating while you're in the middle of the read cycle below?


Hi I would like to ask where can I get the value for StartSsoLink? Thanks. <newbie/confused with Portal2CRM and Integrating Parature to CRM>


Love the attention to detail :sparkling_heart:


This list is way too specific to be in an "expected" file?

Perhaps `make` should be run instead and seeing targets are generated correctly?


Why?


If the initclocks routine needs to run with interrupts enabled, why is it not called with interrupts enabled?


oops. Not sure what I posted here.

I agree with you that junk is hard maintain. Let’s make a list of scripts that you think do not work and/or should be removed. Than we will discuss on this list which ones to remove or fix. I believe web2py-scheduler.conf is useful and not less useful than some of of the other scripts so it can be part of a broader discussion.

This is a good time to clean up house.

On Jul 7, 2015, at 4:45 PM, Massimo DiPierro <massimo.dipierro@gmail.com> wrote:

> I agree that junk is hard to maintain. I
>
> On Tue, Jul 7, 2015 at 1:58 AM, niphlod <notifications@github.com> wrote:
> "the junk" is hard to maintain and frankly half of it can't be executed as it is.
> This too can't be copied and fired without prior modifications ...
>
> —
> Reply to this email directly or view it on GitHub.
>
>


Don't indent based on name length from previous line. Put newlines as needed to avoid it.

Consider use of new lines to split the list comprehension clauses.


Yeah, indeed. It's just that the path used to be the concatenation of the two.


Add a functionality test instead? That wouldn't work without the `rm` and should only work because `rm` is added correctly?


Newline before `f` so indent sizes are less arbitrary.


Should '\0' really become the empty string? You can't really shell-escape that, should be an error instead?


this syntax is deprecated.  prefer `current_user.update(role: 'standard')`


Why not use backslashes instead?


Great idea.


I know this is getting pedantic but see comment above - "mixed lookaround" :)
In short: move it to the appropriate section "set ops" or at the very least put it before any of lookahead tests.

These sections were made to simplify temporarilly enabling/disabling tests  of sub-features during re-design or big impact changes.


Missed that, thanks.


Maybe we should call it styles?


Этот метод практически полностью дублирует метод `DisplayWithEnter`. Чтобы понять, почему это плохо, представь себе следующую ситуацию: тебе приходят обновлённые требования к твоей игре, в которых говорится, что между ячеёками нужно "рисовать" решётку при помощи символов |, = etc. В этом случае тебе придётся менять код в 2-х местах, велика вероятность что-то забыть.


Can add a custom rule for copyright header:
```xml
    <Analyzer AnalyzerId="Microsoft.Web.StyleCop.Rules">
      <AnalyzerSettings>
        <StringProperty Name="FileHeaderText"> Copyright (c) Microsoft Corporation.  All rights reserved. Licensed under the MIT License.  See License.txt in the project root for license information.</StringProperty>
      </AnalyzerSettings>
    </Analyzer>
```


Maybe we should call it scripts?


That looks good.


@yjun2 DWMR. This is well done. Press on!


What is this about?


Yes, you can see that in the `gulp/config.js` – this is wrong... Let me fix that.


You open a block here, but the lines below it are at the same indentation level. Something is wrong here.


Yes, the Bestiary was used here.


Can add a line length rule - easier to review code in GitHub:
```xml
    <Analyzer AnalyzerId="StyleCop.KRules.LineLengthRule">
      <Rules />
      <AnalyzerSettings>
        <IntegerProperty Name="LineLength">120</IntegerProperty>
      </AnalyzerSettings>
    </Analyzer>
```


This is another indicator of indentation problem. `end` is never at the same level as the line before it, unless you have an empty block/method/class.


LGTM!


Этот метод много на себя берёт, он ответственен за:
* Получение ввода пользователя
* Определение координаты ячейки исходя из введённой строки
* Изменение состояния игрового поля

Почему это плохо: меняя один из аспектов поведения, ты рискуешь случайно внести ошибку в другой.


I there possibly a @ref to the declaration of this *reassembly buffer*?


You really should follow john-dev ML.

Core changed, so now we can't rely on db == NULL meaning "first call". We have to keep track of it ourselves.

For a test run we still have (for now)
```
reset(NULL)
self-tests
benchmarks
```

For a crack run we now have
```
reset(db)
self-tests
reset(db)   <-- this might still need to be handled differently in your formats
crack mode
```
This is still subject to change, see http://www.openwall.com/lists/john-dev/2015/07/08/3


`BS8.concatMap` sounds very expensive (building lots of 1-char bytestrings, each is expensive!). You can use `Data.ByteString.Builder` instead, see https://hackage.haskell.org/package/bytestring-0.10.6.0/docs/Data-ByteString-Builder-Prim.html and https://hackage.haskell.org/package/bytestring-0.10.6.0/docs/Data-ByteString-Builder.html for a discussion of `escape` functions and their performance.


It's good that you fixed up the indentation, but such cleanups should be done in separate commits. Right now it's hard to say which lines were changed because of indentation and which have updated logic.


Guess this was kind of silly.  I was attempting to show this as an optional parameter.


Is this the only place that requires escaping? Does the makefile target name not require escaping?


Smid også evt SERVER IP ud i en env variable


Closed Visual Studio Online Task [vso#4345](https://azurecom.visualstudio.com/DefaultCollection/ACOM/_workitems#_a=edit&id=4345): 'This is a test usersnap task submission'

VSO State: Done


On thinking about this, part of me is wondering if the whole program needs to be asking for input for jobs and dependencies ? Where as sometimes just a job or jobs will be inputted, one by one maybe and stored in memory, also each time a job is stored it will ask for a dependency. I'm not sure how this would work. But it is clear that certain jobs will depend on certain dependencies only for example  b => c   c => f d => a  .... ect...


Thanks.  I've been using this commit in production since it is needed and there is no workaround for the problem.


Can we add `StyleCop.MSBuild.Targets` to `RESTier.settings.targets`?


@levicole I've never had a student implement the `CIStripesGenerator` before. I love it when new things happen. Thank you for that!


Это очень неэффективный способ определения координаты ячейки. Правильный алгоритм выглядит примерно так:
* Получить строку - ввод пользователя
* Убедиться, что её длина равна 2-м сиволам
* Получить первый символ строки, и на основе него определить первый индекс ячейки (если a - то 0, если b - то 1 etc)
* Проделать такую же операцию для второго символа

Код будет понятнее, понадобится меньше памяти и процессорных операций.


@levicole DWMR. So close to done!


> But aren't most filters that affect S3 based on subdomains?

Yes, and these subdomains would match `@@||s3.amazonaws.com^`.

I see 17 block filters in EasyList and 15 block filters in EasyPrivacy which would no longer work. I can't just create such an exception that defeat blocking entries in EasyList, EasyPrivacy.


Try on SMTP Hostname add: tls://smtp.163.com <tls://smtp.163.com> see if is works

> On 7 Jul 2015, at 08:39, DaisyOne <notifications@github.com> wrote:
>
> Many thanks for your help!
>  <https://cloud.githubusercontent.com/assets/13131511/8541117/4608ccb6-24be-11e5-83e5-2d55ac32df3f.jpg>
> I use SMTP,and this is my settings.But I don't know where is the problem.Could you find it?
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/opencart/opencart/commit/b03149e4802cc4ac7e9f9ec59c7be4759840c98f#commitcomment-12033269>.
>


Если пользователь ввёл неправильную строку, ты не установишь ячейку и продолжишь выполнение, скорее всего это неправильное поведение.


@oliward ah I understand, nice and clear. Thanks


Maybe we should even use `$(shell ./dynamic_big_crypt_generator.sh)`


Using `expect(VALUE)` right next to `expect( VALUE )` is inconsistent. In general it's good style to not put spaces around the argument's parentheses.


Yes, but I don't care this early :).


Good use of warden helpers :+1:


hello world


No, this file must be added **after** `Microsoft.CSharp.targets` otherwise the StyleCop target will not get called forever.


Maybe we should ignore the assets folder all together?


BIANYIQI


DSFDS


Yeah, that green line was a bit unrelated and too attention grabbing. :grinning:


В целом в текщей версии кода использование двумерного строкового массива `blank` для хранения состояния игрового поля выглядит более-менее. Но попробуй написать логику определения выигрыша одного из пользователей, и всё станет не так радужно. Кроме того, давай попробуем представить следующую ситуацию: логика определения выигрыша завязана на стравнения элементов массива с символами x и o. А тебя просят изменить внешний вид игры, так чтобы отображать заглавные буквы X и O в ячейках игрового поля. Или скажем ноль и единицу. А то и вообще исходя из настроек пользователя :) То, что у ты можешь привнести ошибки в игровую логику, меняя UI - неправильно в корне.


Unconsciously following local conventions.


Yes, thats much better, this was before I had implemented the copy function. I will update accordingly.


可以


it's contagious!


For the copyright, the rule from Web API (`Microsoft.Web.StyleCop.Rules.dll`) does not work at all. I changed the header in the source of Web API and build but no warning showed. Also I tried to copy the rule to the StyleCop folder and wrote the custom rule you mentioned but it did not get executed. Will try StyleCop.KRules instead.


Not sure? They're showing up correctly for me.


@al-martinez DWMR. How can I help you before we speak on Thursday?


What about `~AttachmentFolder.is_deleted`, how would I specify that?


For StyleCop.KRules, I found its source code in ASP.NET's repository. But it is based on xproj which cannot be directly used in RESTier projects. I can port it to a .NET 4.5 csproj but is it OK to do that? @congysu


`find()` accepts both `filter`-style args and `filter_by`-style kwargs


As I wrote in the git commit log:
"Technically the first two are not required since interrupts should not
be running at this point, but play it safe in case someone reorders the
platform initialisation code later."


As per commit log.


Attachment.find(~AttachmentFolder.is_deleted, ~Attachment.is_deleted, AttachmentFolder.linked_object == obj, _join=AttachmentFolder).count()


Nice! :)


Welp, this was an accident. Thankfully didn't change anything?


Should we call SOL_FLOW_STATIC_TYPE_CHECK here the same way we did for the deleted sol_flow_static_del_type()?


Ah yeah, forgot I could use that.


Glad to see this commit! One suggestion may be to remove the "cloud" terminology (At least on the user-side. The code variables, etc can stay the same). That way the user doesn't feel they *must* use a cloud location.


Because we need tsc_base and time_base to be updated atomically, and (AFAIK) no one guarantees that bmk_cpu_clock_now() won't be called from an interrupt handler, even if indirectly.


LGTM


The reason I left bmk_isr_clock() in there was to make it obvious to someone reading clock.c in isolation that the interrupt does nothing.


Change to "never"


Thank You :+1:


ss


This test for the Windows version presupposes that major versions < 5 do not anymore occur. Is that safe?


ping @hadley @kevinushey can you have a look briefly and check I did not step on a butterfly.


This is partly due to my confusion about where bmk_x86_cpuid() belongs. e.g. it could also go in cpu_subr.c and be called bmk_cpu_cpuid()...


I believe quotes need to be escaped. I might need to revert this


Change to 2


On saying that as the exercise points out, there are different requirements per section. Only at certain times are dependencies even included. Hmmmmm deep thought ....


You reviewed an intermediate version of this file. Now lines contains a few other elements than the file content.


Dude, you dont need the s bro.


oops, it's "currentDraft.value" here


same


Not that i care, number was just between 12340 and first 4.0.1 live


sdaf


Based on this code: http://fxr.watson.org/fxr/source/x86/isa/atrtc.c#L336 and agrees with the datasheet that we have enough time after that point to read the device safely. And yeah, I should probably also implement the "is RTC present" check...


fixes https://github.com/fabric8io/fabric8/issues/4226


Anarchist!


There must be a symbolic constant for the 19 occurring here!


1. I have been working on  a "sockset" application that allows setting various socket options, e.g., even if scp/rsync/etc. does not support setting a congestion control, it allows you to do:
sockset -c cdg scp backup.tgz remote.host:

2. You might get away with adding code in tcp_input/tcp_output if RDB is not a module, but optionally compiled in. See http://patchwork.ozlabs.org/patch/491670/


Tenhle řádek je tu navíc - funkce se zavolá přímo při vykonávání podmínky o řádek níže.


@inikulin Should you update Babel version in the package.json file?


@pcon - very very very nice catch and fix ;)


#1 fix


Nope, we have a `"gulp-babel": "^5.1.0"` in `package.json` which means `>=5.1.0 <6.0.0` (see: https://docs.npmjs.com/misc/semver). So you need just run `npm update` locally.


Yea. I definitely  need some help. I finally realized that checkpoint 29 needed the files from the previous project. I tried to merge the two branches i had together and spent most of my time yesterday trying to figure it out why my project wouldn’t open again(finally fixed it). I won’t be merging again anytime soon.

An idea I just thought of is how about an outline of what I should be doing to get checkpoint 28 going?

For example you could say. You  need to add X, but to add X you need to do blank to Y and Z. Not sure if that made sense. If it’s easier to talk and you have time that would be great too. Obviously my first choice would be to talk but I know you’re busy. I’ll take any help that you can offer that is easiest for you.

I’ve been meaning to ask you. What is DWMR? lol I couldn’t find an acronym that fit for it.

> On Jul 8, 2015, at 8:15 AM, dothework <notifications@github.com> wrote:
>
> @al-martinez <https://github.com/al-martinez> DWMR. How can I help you before we speak on Thursday?
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/amar59/Blocstagram/commit/8d1d1044d55d947c4cec6e149d881809f10833cb#commitcomment-12060118>.
>


@bitemyapp Ok, I'll switch that up. I'm in the middle of working on the scan&scroll -> scroll workflow; once I commit that, I'll move forward on this issue.

What did you mean by "Also what is the HTTP method compatibility like?" ?


reviewed ymatsukawa
[優先度の件はレビュー通りです](http://qiita.com/sqrtxx/items/1eb5828a7acd821c3706)
が、本番機は不特定多数の Linux 環境になることを見込み
(つまり一々デプロイ・リリース側が共通化する暇はないと思っているので)
リポジトリの .gitignore に埋込みをすべきではないでしょうか


On 08/07/15 12:18, Martin Lucina wrote:
> Because we need tsc_base and time_base to be updated atomically, and (AFAIK) no one guarantees that bmk_cpu_clock_now() won't be called from an interrupt handler, even if indirectly.

It is currently guaranteed.  (did you look at the interrupt handler code?)


n


correct, that method was added later at my request.


Remove this will cause a dependency on specific version, such as 5.2.0, instead of 5.2.0+.


We need to path endpoint now, right?


@daniel-beck Put blank spaces between `for` / `if` and `(`


reviewed ymatsukawa
[共通化出来ました](http://stackoverflow.com/questions/1819614/how-do-i-globally-configure-rspec-to-keep-the-color-and-format-specdoc-o)
ローカル環境には共通化設定をしました
が、
個人的なシュミで、差分このままにしておきたいです


5. Я понимаю, что можно, но в задании сказано всё-таки, что хочется иметь возможность посылать сообщение по урлу. У тебя такого нет, вот и весь спрос :)
6. Как мне кажется, логгер должен иметь возможность установки, куда писать инфу. Именно не установку левела, а установку рендерера в твоём случае. Что-то типа метода setRendered на логгере.
7. А контракт можно какой-нибудь предоставить? Я же могу затереть какой-нибудь нужный объект из прототипа или уже существующий метод. И хотелось бы, чтобы все методы были собраны в прототипе в каком-нибудь объекте или массиве, а не просто разбросаны.
8, 9. Опять же, всё по заданию, и это как посмотреть. А может я не хочу вызывать в каждом обработчике события метод log, а хочу чтобы он автоматом вызывался для события, которые я укажу логгеру? Стоит подумать, как мне кажется :)
10. Это по большей части связано с 7. Ну и опять же, код типа new this.slog.messages.BrowserMessage('info message') меня, как конечно пользователя, напрягает. Чем меньше я знаю о внутренностях твоего логгера, тем лучше. Идеальный вариант - я знаю что есть метод log и я прокидываю туда сообщение, которое нужно залогировать. А ты уже сам делаешь всё, что нужно.
12. Должна быть возможность, чтобы твой логгер подрубился как надо если я использую AMD или global. В твоём случае если я использую AMD и подрублю скриптом твой логгер, он мне создаст глобальную переменную, что не есть хорошо. И кстати, насчёт глобальной переменной, что если она уже занята? Просто затрёшь её логгером.


On 08/07/15 12:25, Martin Lucina wrote:
> Based on this code: http://fxr.watson.org/fxr/source/x86/isa/atrtc.c#L336 and agrees with the datasheet that we have enough time after that point to read the device safely. And yeah, I should probably also implement the "is RTC present" check...

What guarantees that you don't hit the loop a nanosecond before the
signal goes up?


Let's discuss this during our next meeting. If I uncomment the call to ```should_receive_update_for?``` one of my tests fails and I can't figure out why.


ここのエラーログがあまり適切ではないと思います.
sheepdog: のprefixが確認できない、またはイメージ名が格納されてないまたは、フォーマットが不正な状態かと思うので、image_locationがSheepdogのlocationのフォーマットタイプに一致しないとかでしょうか。


This revision is clean.


Why not place ths in the default `ht.access` file located in the root directory? Not there are two `ht.access` files to rename and keep track of.


probably should be _TIRPC_ to match new define


ok


Yes you need others but note this is not used in cm12.1


I made this correction because it is the scaling recommended in Lasso litterature. Should I made it in a different PR ?


Realized I didn't need these modules now anyway, so yeah... lol.


We have verified if we omit these. NuGet will pick the version referenced in packages.config as the **minimum** version of dependency. So its OK to remove them. Also this is the way other RESTier projects handle NuGet dependencies.


I mean when Media::adJsDef will be corrected, the string 'PrestaShop's special' will became:
'PrestaShop\'s special' then 'Prestashop\' special'

But you are right the "border" effect does not appear here, however since Media::addJsDef need a patch and that this commit need its counter part commit to decode entities, better fix Media::addJsDef()

Implementing workaround rather than fixing the underlying core function is the path that lead to the infamous 500 while a shop owner change translation too quickly as an example


why do we need ```Array.from``` here?


Here's my proposal, how to do it (based off https://github.com/mgreter/libsass/commit/5776e9bde50082ac6d2c2773396c2725104c9016 the parent of this one) : https://github.com/mgreter/libsass/pull/5


Wäre es möglich, das Dokument noch mal als markdown oder pdf hochzuladen? docx ist immer so eine schwierige Sache unter linux.


Whoops, my bad - sorry for the noise. Looks like I misunderstood the double pipe operator for months.


为啥看你的笔记这么有喜感。


Local test after 1000 games is +5. Not much significance, but not bad too.

I've seen tests where isolated was simplified (one recent from snicolet)
but could not find any with double and isolated at the same time (now that backward is simplified)

Can you provide a link or two ?


This is an interesting and strong statement. Do you actually think that this would be more useful than the current MB algorithm?


"else if" should begin on the same line as closing curly brace (like else in line 122), that is Moodle convention.


M48T86 datasheet (not sure where I found it), page 15/29:

> UIP. Update in Progress. The Update in Progress (UIP) Bit is a status flag that can be mon-itored. When the UIP Bit is '1,' the update transfer will soon occur (see Figure 11). *When UIP is a '0,' the update transfer will not occur for at least 244µs*. The time, calendar, and alarm information in RAM is fully available for access when the UIP
Bit is '0.' The UIP Bit is “Read only” and is not af-fected by RST. Writing the SET Bit in Register B to a '1' inhibits any update transfer and clears the UIP Status Bit.

(emphasis mine)


Oops :P


The pluginName change was the most important, lets us track separately from mainline Ess


I think a function signature like
`safe_memcmp(x: &[u8], y: &[u8]) -> bool` would be better.


Release in 1.20.1


what happened?

make sure to sync I just updated yesnodialog....

Cheers man! nice job on the info stuff.


Commented code to be removed


<img width="1249" alt="tickets" src="https://cloud.githubusercontent.com/assets/186856/8570326/e61b7cbe-257f-11e5-851b-2683dcde39a2.png">

Includes a generic listing table


This could be:
```
if x.len() != y.len() { return false }
```
There is no side channel there (except for the length, and that is known)


<img width="354" alt="screen shot 2015-07-08 at 12 51 53" src="https://cloud.githubusercontent.com/assets/2766945/8570355/0328dc52-2580-11e5-83d4-2d4c9e1b9fa6.png">


Why are these added? Breaks the build.


No worries, :) 'make clean' does the job


this is what I got inspired by


Thanks!


That code looks scary! Isn't there a cleaner way to do that? like import all source modules and instanciate the proper class when needing it?


should probably use `&$newtype(ref other)` to make sure that there won't be a copy here (to avoid leaking secrets on the stack). A sufficiently smart compiler should optimize, but you never know.


I think I slightly forgot about raster maps, that's still doesn't contradict "OsmAnd" style which will be used by default. It is quite difficult to think in all dimensions, so I suggest to think "we always have vector map" just sometimes it is hidden by raster map


On 08/07/15 12:40, Martin Lucina wrote:
> M48T86 datasheet (not sure where I found it), page 15/29:
>
>> UIP. Update in Progress. The Update in Progress (UIP) Bit is a status flag that can be mon-itored. When the UIP Bit is '1,' the update transfer will soon occur (see Figure 11). *When UIP is a '0,' the update transfer will not occur for at least 244µs*. The time, calendar, and alarm information in RAM is fully available for access when the UIP

Thanks.  The comment talked about update-in-progress instead of "update
not happening for at least 244us", I adjusted it.


What is your opinion about 2 ways to do it? 1. way like i have it in https://github.com/project-ncl/dependency-analysis/commit/0459928e12ab7168a01e3995b633f40e8dbe022e i have interface with anotations and then implementation or 2. your way with just implementation


having two variables named `source` but containing differently typed values is rather confusing. Please change the name of one of them


I would prefer having a verb in the method name.


I do not understand the test case... Which space is missing in the corrupted slang file?


OK ;)

Bon voyage

 Joseph

2015-07-08 13:47 GMT+02:00 François <notifications@github.com>:

> Je pars pour Liverpool aujourd'hui, je risque de ne pas toucher au repo
> pendant… longtemps :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/larmarange/analyse-R/commit/c2b0066fffbaac6a1800babecdcb811e814f0835#commitcomment-12059508>
> .
>


I think the root cause of the problem is, that it is unclear whether `providedOptions.method` or `method` should take precedence in `mergeOptions`. I now agree that removing `RequestMethods.GET` here is not the smartest move but if we leave it in then we need to give precedence to `providedOptions.method` otherwise all requests will default to `GET`.


That means no Wayland already...


The error message is wrong for this use case:
*Are you sure you are not missing a space after colon(:) for one of the imports and that all imports have an alias?*
The problem in this case is not the imports as the error message states...


Flow name should be identical to file name. Same for the other flow below


@lucasart better this one, please

  void age(Value& v) { v -= v / 4; }
  void age(Stats<Value>& v) { v.age(); }
  void age() {

    for (auto& a : table)
        for (auto& v : a)
            age(v);
  }


It might be faster to replace the CDRs with CADR and use F-CMP instead,


Hi @mrjrrz, this value is provided for your specific environment, after requesting SSO setup on partners.support.parature.com


Fair enough, still a very impressive script. Good work


Txs Wim I removed the three of them.


I haven't bought it through MAS though.


How do you plan to use this property in regard to deleting a relationship?


メソッドの目的は何でしたっけ
こっから命名しましょう


@mpaladin good question. removed in b7e228099a0fd51f5b1be552871263c6f863bb98.


Yeah, I've must have missed. Note to self, don't merge after 20:00.


WAOUh !! Ton appli devient bluffante.

![image](https://cloud.githubusercontent.com/assets/8062286/8570583/aaf750ac-2581-11e5-83a9-90b80afca465.png)


And you probably need streamClient plugin here


Fixed 'gpu' argument action.  Previous setting would set gpu to always true.


ちなみにprefixを変えたいだけならこれでいけますよ（FAQに追加するべきか）。
```viml
nmap <Leader>c <Plug>(caw:prefix)
```


```yaml
barclamps:
  crowbar:
    - crowbar
    - cisco-ucs
    ...
```


@Sunilkumarr remove new line


This would be good, but some of this information (particularly supply and status) would need a much lower TTL than the characters end point currently has (5 minutes).

A commander using a wvw app needs to know how much supply he has now, not how much he had 5 minutes ago.

I think this information would be better in a separate endpoint with a lower TTL, at the most 1 minute, but preferably even lower than that.


thx


This constitutes a non-backwards compatible change. As I told earlier, I think we should err on the safe side and mark it as such but still allow it.


But a class cannot Start with a number and this is not checked.


Без `initNavFuncs` функция `readCfg` ниже не будет работать.


Hey thank you guy ! :) Si tu trouves de jolies images, ça m'intéresse de les collectionner pour en faire une page, un bestiaire ou que sais-je. Je suis intéressé par les liens plutôt que les images d'ailleurs. Je vais de ce pas créer un ticket.


Hey :) I'm wondering why we can't initialize a new cordova project in a folder where we already have a "www" directory ?
Say my folder only contains my app in a `www` subfolder, i guess we could create the cordova project without conflict ?


@drogus should we add a note that this feature may change in an upcoming update too?


There seem to be problems with this change.

The description of `resend_on_redirect` states that when `maintain_method_across_redirects` is true, 303 responses are still redirected with a GET. With the original code, this is not the case - there is no special handling for 303 responses.

Then on the new line 261, the boolean algebra doesn't make sense. The following two lines are functionally identical:

```ruby
unless options[:maintain_method_across_redirects] and options[:resend_on_redirect]
if options[:maintain_method_across_redirects] or options[:resend_on_redirect]
```

The description of `resend_on_redirect` states that when it is enabled, the HTTP method will be the original one - not GET. However the boolean logic dictates that it will function exactly opposite the way it is described.

From what I can tell, there is absolutely no point in having the `resend_on_redirect` option. It doesn't achieve anything more than `maintain_method_across_redirects` does.


This wouldn't have failed even before. I think, we should really count that its 30.

Well, it looks like its 31... Something seems to be fishy with the DOW30 tag?


I think it would be better to place this in parent class JavaClientCodegen ?


Ici, j'imagine que c'est type="javax.sql.DataSource" ?


Может лучше оставить. Чтоб было ясно видно, что функция возвращает булево значение. Да и `aib.init` испарвить, чтобы всегда возвращали `false`, а не неявный `undefined`.


Pareil, databaseHost:databasePort/databaseName ?


This may need to be revisited on mobile devices.
http://imgur.com/AJlMCL2


Eher Datenbanksysteme / Datenbankarten oder sowas?


Why does the ui not know the name. This is code smell. The code that starts the export should know the file name.


> I'm learning a lot by just reading up your usual ping-pong game :-)

That's how it works. ;)


CircleCI
```
Configure finished.  Now 'make clean && make -s' to compile.
/bin/sh: hell: command not found
Makefile:220: recipe for target 'dynamic_big_crypt.c' failed
make[1]: *** [dynamic_big_crypt.c] Error 127
make[1]: *** Waiting for unfinished jobs....
Makefile:188: recipe for target 'default' failed
make: *** [default] Error 2
```
Not sure what the heck happens.


Never mind... makes more sense if you read the PR in full #317.


Note that I also learn from you, too. :)
ack use is the most famous example in my mind.


:clap:


Надо взять в теги, например, в <span>


Key-Value-Stores fallen eher unter den Begriff "NoSQL"? Meinst du das als Bereich? Darunter sind aber nicht nur KV-Stores, sondern auch Dokumenten- oder Graph-Datenbanken angesiedelt.