My test using DFU mode on Taranis:

| fw on radio | flashed fw | result |
|------------|--------------|-------------|
| opentx 2.0.13 Taranis| 2.0.13 taranis | no problems |
| 2.0.13 taranis | 2.0.14 taranis plus | no problems (expected as old fw doesn't have flavour)|
|  2.0.14 taranis plus |  2.0.14 taranis | Alert box, can't flash|
|  2.0.14 taranis  |  2.0.14 taranis plus | Alert box, can't flash |
|  2.0.14 taranis  |  2.0.14 taranis | works |

So I say it is good. :+1:


>Паша, не совсем так: в начале уттилитой генерируются эталонные решения поиском в ширину (всего один раз) и дампаются в файлик.

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


There is no wrapping here. I suggest we give it a better name.


Looks good to me.


Memory leak? What if we replace this with "LastFrame := nil"?


Shouldn't this be

```yml
    - ruby_installed.rc != 0
```


umm. newRouter calls `router.Ourself.Start()` too, so this can't be right.


I would split this into 2 separate tests if we really want to assume that built-in `toString` functions behave differently from the hand-crafted ones:
* "should consider built-in `toString()` in non-strict comparison"
* "should consider custom `toString()` in non-strict comparison"

Personally I would find it easier to read that way


It does make things marginally better, but there are still some issues - for example, from the test case we were given for #141, "<svg version" is compressed to "<svgversion" with this patch in place.


"ssl" could be nil if raven-lua works in ngx_lua. It's better not to expose this globally because it's not used in all situations.


84694650_EnableCodeCoverage (lefticus) - Win64-Windows-7-VisualStudio-12: OK (1467 of 1582 tests passed)

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


yeeeah, two release in 9 minutes. I guess you beat the record of @dukeboard :-)


yes
and its not a float value :P :D


You're absolutely correct! The point is indeed not assigned at this point, so allocating memory for the dereferenced type won't work there! I've fixed it and committed the change!


Don't use JavaScript in your HTML. This is along the same lines as keeping the presentation logic out of the HTML as well. Here, we have tightly bound our semantic structure of our form with its behavior. We're also using a string of JavaScript (noted by `return` instead of just the function name). The JavaScript interpreter is going to `eval` this string (which is dangerous) and possibly take the return value and shoot it into the global namespace—although it more likely just be cast off into the ether.

A better path would be to have JavaScript/jQuery add an event listener to the submit button and then prevent the default action.


I don't see how that's possible. captcha.php just displays an image, it's not involved in the verification of whatever a spam bot may put into a registration form. At the same time the test string is a good way to debug broken captcha images, so even if it was exploitable somehow by spambots, it should probably be fixed elsewhere.


As you know noticed overrideScope works fairly well for applications, but less good for libraries.  I presume that deepOverride is subject to the same problems.

I have a no-so-nice idea, but I think it is feasible.  My suggested design would be for packages to publish (via an attribute) the "local" package extension that they desire.  A modified version of callPackage would gather the published "transitive" package extensions (defined below) of the dependencies, plus the "local" extension of the package and combine them to create a "transitive" package extension for the package under consideration.  This "transitive" package extension is published (via an attribute) and it is used as the default scope for building the package.  However care must be taken so that if some other package (higher in the dependency chain) wants to build in a different scope, then callPackage must defer to building in that scope.  This will involve a moderate change in the logic of how scopes are handled for our callPackage.

The processes of combining the "transitive" extensions from the dependencies with the "local" extension of the package can proceed with various level of sophistication.  At a high level, the list of "transitive" extensions from the dependencies should be combined "in parallel" meaning the combining operation should be commutative.  This will imply that when two dependencies try to override the same package (with different versions?) the result should be an error.  But the "local" extension should be combined "sequentially" so that it gets to override any desires of the dependencies.  In this way the "local" extension can even override the errors coming from conflicts of the dependencies.

A less sophisticated solution is to combine all the "transitive" extensions from the dependencies sequentially and end up with random resolutions of conflicts that the "local" extension will be responsible for fixing up.

As you can see I haven't really ironed out the details of this design yet.  I'm too busy for the next few weeks to flesh out an implementation, but eventually I should have time.  I probably have time to review implementations if you folks want to go ahead without me.


Good thing I didn't comment on that `data[message]` before—it's gone now. :smile:

I still don't love this line of markup.


Summary by treatment made for Split Plot, Strip Plot and Factorial Designs


@pepol I'd use "..." instead of ", "


Noticed this one a while ago too :-) Thanks for creating a pull request for it.


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


`Array.prototype.forEach` ([MDN documentation][fe]) is probably a better choice in this situation.

[fe]: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/forEach


@mperham  Do you have an idea why jruby started to fail ?


End of the week effect :-)


Looks pretty good. I would modify the first test a bit by adding a "test" instance to the child and asserting that that instance was *not* returned but rather the one from the parent.


@belph it no blow up :clap:.This will work wonderfully, the only reason this function should be used is if they want to send something to the server, it wouldn't be hard to add some convenience and have that function possibly take a message object, and call toJSON on it. Also since we have dict_to_cJSON we'll be able to directly convert to JSON and send the string to the server very easily if the plugins are running in their own threads, also since we have dds_sem we can just use that for thread safety :+1:


@elado Want to try and do that? It doesn't look that simple :)


Lines 4–8 have semicolons. Lines 9–12 do not. We can [argue](http://trevmex.com/post/21194966996/crockford-on-bootstraps-semicolon-omission) about [automatic semicolon insertion][asi], but consistency is more important.

[asi]: http://inimino.org/~inimino/blog/javascript_semicolons


Don't you want -O2 in both the PIC and non-PIC cases?


JavaScript is camelCase, not snake_case.


Hi! It was to make the height of the toolbar match the design spec (56 + 72 = 128). https://www.google.com/design/spec/layout/structure.html#structure-app-bar


I did exactly this in #303.  Notice that `LICENSE` was updated in #303, too (see [here](https://github.com/libqueso/queso/pull/303/files#diff-0) for the diff).  The only reason these files didn't get updated was because I forgot to `touch` them.


Please replace `var` with `let`. We use `let` whenever possible in our code for more predictable scoping. More details here: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/let


This looks good! I should have mentioned earlier, when the xhr timeout occurs, we need to resolve the favicon promise with an empty string (like we do in a 404 or 401 on load), otherwise we will continue hanging on an unresolved promise.

Bonus points if we can reuse this function for both `ontimeout` and `onerror` to ensure we always resolve the promise!


Never had problems with not mounted /sdcard or UMS too... but also there are reports that apps can't be moved to sdcard.


Whoops nevermind.


Two things:

* Avoid `==`.
* `null` is falsy and so is an empty string. You could just do `if (fields[i]) {…}`.


We were using this to log which version of the API was used. Is there a reason this was taken out?


@sirmordred Knowing my Galaxy Mini, if you remove the SD while the system is running, it instantly dies and softbricks.


`0` is also falsy, so you could just do `errorMessages.length`.


[CodeMagica/RailsBestPractices] remove unused methods (ApplicationSettingsHelper#extra_sign_in_text)
reference:


some tests failed:
failure: http://build.servo.org/builders/mac1/builds/716


Semantics
On Jan 9, 2015 12:47 PM, "anfrantic" <notifications@github.com> wrote:

> @Garbee <https://github.com/Garbee> I agree with you. But what i had in
> mind was more convinient naming for commands. Basicly a bus command is a
> task (or a job) that has to be performed, a collection of smaller
> operations grouped together. Console command - is a 'command' which user
> (or cron) gives to aplication to do something. Just like we use "Contracts"
> for interfaces for cleaner. But after all this is not a controversy, just
> my opinion =)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/laravel/laravel/commit/ce20ef22c1cc7a450c5499162ca97a3887963269#commitcomment-9222341>
> .
>


Looks like copy paste errors here.  ButtonName isn't relevant to AddCommand


But most important:
openTX 2.0.9 from FrSky (original firmware) to 2.0.14 should raise the alerts as well!


r+


I was very surprised to see that this was necessary. I was under the impression that marking the C function definitions as "nogil" automatically did this when possible.

Nope!

This isn't immediately cleanly merge-able with the pyio branch, otherwise I would immediately do so.


Commit https://bitbucket.org/gutworth/six/commits/1daeb523458d is after six 1.8.0. So this should be 'six >= 1.9.0'.


I don't love that we're effectively making a singleton by using class methods here.


r+ rollup

THanks!


Heh, second time I've done a rebasing pull only to see you've already fixed my problem


this is no submodule, this is a modified copy of uberserver.

for how to create a git submodule.... http://git-scm.com/docs/git-submodule


but your version based on mutiprocessing too block
I tested my version it's work
yes readline blocks until no data, but when data available it return value and unblock

you can test my code


This integration was updated to use the AIM API in this PR: https://github.com/Shopify/active_merchant/pull/1297

It doesn't look like that API requires a version parameter, which is probably why it got removed.


@pepol I don't think we need this line.


(1.8.0 is dated 2014-09-11, the view* fix is dated 2014-10-03, both on the main branch.)


Why did this have to be removed?

Also, it should be if (multibar.barColor()), not sure why you removed the () there, since barColor is a function so it would always evaluate to true then...


thank you too. :-)


Same as other comment.


@bors: retry
* #3228


Good. When do you plan to release .14?


We are used to advise http://btc.blockr.io to our italian users, although I don't know if it's the faster.


Why did you choose `\W`? `\W` will pickup any non-word character, which includes !, #, @, and any other symbol. Either this is not what you intended (`\s` might have been a better choice) or your variable needs another name.


@ziafazal -- can you bundle this work up into a PR so we can take a look?  Thanks!


Very cool update feature.


Comments.


saw approval from jdm
at https://github.com/ebalint/servo/commit/81f47344d6e121d86de368a25107415406de3c0f


merging ebalint/servo/canvas = 81f47344 into auto


ebalint/servo/canvas = 81f47344 merged ok, testing candidate = da400a7a


Vinzenz has noticed another issue in this function and is fixing it.


I wait you mark #1981 as fixed and I send an email to the beta testers distribution list. I plan to release on monday. Is this ok?


@lirainer This is the commit where I added the numeric range validation.


I would probably have just chained a few of these variables. You're declaring a lot of local variables that you're immediately disregarding. At the same time, you are actually chaining on line 7.

Maybe something along the lines of:

```rb
letters = string.downcase.gsub(/\s/, "").chars.uniq
```


Yeah, seems fine to me!


@victorluft - Will we actually be creating the index here in our use case?  Or will that occur somewhere else?


Yeap. Already fixed. Thanks!


where 20090102_20120629_nostockpnl.mat  file


@kirkshoop, @KozhinM, I'm happy to move this code back to Context if that is the decision.  I can leave the Capabilities.supportsEncryptedMedia() function as is, since it only checks to see if a mapping exists.  Just let me know what the decision is.


@ranjan Why did you remove this cert gen?


Ok, thanks.


@victorluft Is this correct? For a full copy, we're want the settings and mappings for the source index; for a non-full copy (a reindex), we won't want either? Or will we still want the settings?


Not in our use case, no. In ours we want to load mappings from model file for new index. and create appropriate aliases before we start migrating.


where 20090102_20120629_nostockpnl.mat


http://paste2.org/aHp1dOnX
This is the full check the client does (from build 4.3.4.15595)


Thanks, not sure why this was in there :-)


For the record, I'm also experiencing the same behavior as @matwright


I'm assuming for cases where the gem is gonna be handling new index creation, we want to copy all settings and mapping, and otherwise let user do what they want with that prior to copy.


Thats better :)


Quotes are needed wrapping $mysqlCmd if we have fancy characters in our MySQL password.


No Oxford comma!? ;)


Yes i partially agree i can deffinatly fit a lot more without getting those errors. However currently i have ~40 apps installed and only have 400mb left. And while this sounds like alot i did have more apps installed but removed some because the insufficient space error happen at the high amount of 200mb free. Which i also think it kinda of silly but idk if we can change that at all or not. And i also agree that it will probably be slower but considering the biggest app is the same size as the speed of my sd i think we are more limited by processing power than storage speed as though the one app is the same speed id say half if not more of the remaining apps are less than 1/2 that size. And i also agree to an extent that there is risk in what if the user remove the SD however...the last time i removed my sd was...when i got the phone so to me this isnt really an issue ive considered much.


Why are we coercing this into a string?


still coruption and frezze


Thx ! :)


@tarelli I've decided not to use the (automatically generated) DepthFirstTraverser (because of #13 ). Do you think that doing things this way instead is too ad-hoc?


last without frezze


Hah! And I thought I'm the only one who saw this :heart:


@doadin Space detection limit used to be changeable with Mounts2SD, although that app's ancient by now. It allows changing the 10% limit to 1%.


Done


develop (lefticus) - i386-Windows-8.1-VisualStudio-14: OK (331 of 331 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)


I'm not sure you need to be this specific in your test. If this test fails, it's hard to tell if it's because your implementation is wrong or if you just counted the letters wrong.


cbecc-integration (kbenne) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2068 of 2188 tests passed)

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


Done.


@borismarin why not to simply overriding the method to add the missing stuff? Also keep in mind that with a visitor you can start visiting at any point of the tree, not necessarily from the root!


I think we need to more explain why you use a builtin server of php and why you can't launch the install without this.


cbecc-integration (kbenne) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


PTAL @juberti


Good plan.


I fear this line breaks windows and osx builds, since it's not implemented in the `glut.h` we provide with oF.
It seems that glut on linux has this method, but in OSX and Windows don't have it in their glut headers.

I'm on Windows now where it breaks, but i assume


It does, we (Samsung and Sony msm7x30) use this setting on CM12.


develop (macumber) - i386-Windows-7-VisualStudio-12: OK (2172 of 2188 tests passed)

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


Would you update the release notes? I also need to send an email for the translators


file_name = '20090102_20120629_nostockpnl.mat';


Travis reported JRuby problems recently: https://twitter.com/travisci/status/552465304442634240


[CodeMagica/RailsBestPractices] remove unused methods (Spree::Stock::ContentItem#eql?)
reference:


where  file_name = '20090102_20120629_nostockpnl.mat';


Too much space in the picture


So we need to add a method that let's us `#load_mappings` from a passed-in block...

In that case we could still do the
```ruby
dclient.indices.create index: didx, body: {settings: settings, mappings: mappings}
```
the same way for a straight copy and for a reindex; the only difference is that the `@mappings` variable would be block-loaded instead of read from the sclient?

For the callback we'd need to create the aliases, where would that go in the flow here? Just before or just after the new `#load_mappings` populates `@mappings` ?


Refs #628


Works for me, thanks.


It seemed to work OK but I was under the impression that I should not be using it for PHPSpec


That would be after, assuming we want mappings in place before we update alias so writes happen to new index, which I would assume would virtually always be the case.


Something weird happened on this commit.  The message doesn't match the contents, and I'm not entirely sure where the contents came from - looks like some work @jonathanolson did on assertions.


Ok


@updateing This seems strange but now is working here XD


ok, well thats interesting 10% considering todays storage capacities seems like a lot of waste oh well gg google i guess lol. So sorry bout the long story but also i was asking if you had considered it and how to do it. Can anyone tell me how one would do it if they didnt care about he risk or slowness? Was i correct in how to make the change?


I also agree that the expectation would be that the flags are parsed automatically (especially since there is a `SkipFlagParsing` attribute on `Command`).


great that you used validators here !


Thanks a lot, Corvis! It worked!


Have you analysed that removal of safety check "(cap_cnt < repl_comp->max_pmatch)" doesn't result in accessing some invalid index later?


r+ rollup

Thanks!


I was wondering about this.  Good info, well documented!


Also a check that my stops are in flash (not in RAM) is needed


getActions() is enough IMO. It's hard to imagine a cause which comes from a transient action.
I'll modify the logic to make it compatible with 1.509.3

If you have any use-cases for transient actions, I can add their support as well


I basically copied and pasted it form the same Readme section I wrote for bubblewrap, lol.


I'd keep the delimiter as a constant and call it FIELD_SEPARATOR instead (all constant are spelled in capitals in java)


Bad practice, which strongly decreases the performance.
instanceof should be used


good!


I'd extract a variable for getAttributes call and give it a good name


cherry picked into develop


good idea to extract this part into a function in order to save us a useless comment :)


Typo, you forgot the ; on the query


This has to be
```
const Vector<Integer> configs(*rit);
```
ie, with `()` instead of `[]`, because you are calling this constructor of the class `Vector<Integer>`, not declaring an array of `Vector<Integer>`


Yuck. This is a pretty terrible thing to ask you to do.


lots of string concatenations are really slow in java because strings are immutable. I'd again point to an open source CSV framework: it should make work easier.


Nit: Should there be a space after '}'? Also, should else be  on a separate line or the same line after '}' . Looks like you need some autoformatting in this file overall.


@xylv this looks very fragile, and it won't work in the case of multipart uploads. I pushed a different fix to wip-10311


r+


I personally don't have a clue, but colored sand allows for more rainbow colors, which... was requested... I think removing torches and warning signs may be a bad call, but Cricket might have a justification.  I'll ask later this evening.


84694650_EnableCodeCoverage (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-Debug: OK (1090 of 1090 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)


... and this should be just `(*rit)[0]`, because `*rit` is already the row itself


Class names should start capitalized ;)


I have no idea what is going on here. The only reason I could figure out what you were trying to accomplish was because I scrolled down and looked at the test.

Ultimately, this method is just a variation of what you did for Scrabble in Module 1, right? This entire method could have been written in a few lines.

```rb
letters = "Foo! Foo!".downcase.tr('^a-z', '').chars
letters.each_with_object(Hash.new(0)) do |letter, letter_count|
  letter_count[letter.to_sym] += 1
end
```


r=Aatch


@maspwr :tongue:uage :)
I think, that I'll let the choice to the Ansible maintainers :books:


It is already fixed in their source, so now we're just waiting for robovm to update dependencies.
https://github.com/shrinkwrap/resolver/commit/f28418141df3ecc22dcd1051ed8fdc58b73d50a4


use 1L instead of (long)1


This should probably be `continue;` to execute another iteration


Reconsider to use SOCDEV_PA/VA instead


This could actually be `getHasEnabled2FAAttribute()` then where you currently call `Auth::user()->hasEnabled2FA()` you can now do `Auth::user()->hasEnabled2FA` - saving two whole characters :+1:


Yeah, that looks shiny. But still, _cleanup()_ works in main thread. Don't you want to put **&** after parentheses?


I used the code from the newest abtsys branch but without userland specific parts.


Where are the tests for the unhappy path? We strip out whitespace, but we don't test that functionality. What about numbers? What about symbols and punctuation? Emoji?


Coloured Sand really doesn't fit with the theme of the mod now that I think about it, and others have agreed. Awesome idea, but doesn't quite fit in with Chisel. I'd like the justification too for torches, but especially signs, people have already protested their removal. Btw, do you have Cricket's email, he wanted me to contact him that way?


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


@hadley I should've listened to you. `writeChar` behaves differently on Linux, even with `type="bytes"`.


zxombie:
```
Looks good
```


Why not just do

```ruby
origin = Location.new(params["origin"])
```


my concern here is that this is seemingly blocking the main queue that could hamper the ux. How does this fix the crash?


Why did you make this change here?
This means that the installation step of kazoo fails at
`/opt/kazoo/utils/sup/sup crossbar_maintenance create_account gridmobile sip.kazoo.lan admin kazoodev`!
At this point the AccountDb for the user does not exist because you validate both account and user before creating account (and account database) in couch!
ie, this line of code is hit before https://github.com/2600hz/kazoo/blob/master/applications/crossbar/src/modules/cb_accounts.erl#L272 !


Used PCB_LR value instead of hard-coded "30"


Removed wrong kdb_trap() from EL0


zxombie:
```
Could we also use PCB_LR here? It should be defined if pcb_x is.
```


rax retry tests


So, we've done some refactoring, but `letter_count_array` is still gnarly. Also, it's still a singleton class. It's just acting like hanger for random functions—and not an object-oriented design.


Now the pause is at the last switch of a given type, IMHO it should be at the first different switch type so see my comment in the code above.


emaste:
```
Yes ARM64TODO please
```


For this, I wrote the following code:
```
bool cocircuits_or_fail(const Matrix<Rational>& G,
                        std::vector<Set<int> >& positive_parts,
                        std::vector<Set<int> >& negative_parts,
                        bool allow_fail=true)
{
   const int n = G.rows(), e = G.cols();
   for (Entire<Subsets_of_k<const sequence&> >::const_iterator sit = entire(all_subsets_of_k(sequence(0, n), e-1)); !sit.at_end(); ++sit) {
      const Matrix<Rational> ker = null_space(G.minor(*sit, All));
      if (ker.rows() >= 2) continue;

      Set<int> plus, minus;
      for (int i=0; i<n; ++i) {
         const Rational val(ker[0] * G[i]);
         if (val > 0) plus += i;
         else if (val < 0) minus += i;
      }
      if (allow_fail &&
          (plus.size() == 1 ||
           minus.size() == 1 ||
           plus.size() == 0 && minus.size() == 0)) // not full-dimensional
         return false;
      positive_parts.push_back(plus);
      negative_parts.push_back(minus);
   }
   return
      positive_parts.size() > 0 ||
      negative_parts.size() > 0;
}
```
It is called with
```
int non_convex_ct(0);
   std::vector<Set<int> > positive_parts, negative_parts;
   if (!cocircuits_or_fail(G, positive_parts, negative_parts)) {
      ++non_convex_ct;
   }
```


all tests pass:
success: http://build.servo.org/builders/linux1/builds/500
success: http://build.servo.org/builders/linux2/builds/498
success: http://build.servo.org/builders/android/builds/536
success: http://build.servo.org/builders/mac1/builds/717
success: http://build.servo.org/builders/mac2/builds/717


After looking into this further, the error was being caused by having `vendor/` in `.gitignore`. I published a new release that fixes this issue. This import should function correctly now.


No handling of possible exceptions


zxombie:
```
Could you add a comment here so we can find this later when we have cache support.
```
emaste:
```
We used ARM64TODO in the past
```


all tests pass:
success: http://buildbot.rust-lang.org/builders/auto-mac-32-opt/builds/3085
success: http://buildbot.rust-lang.org/builders/auto-mac-64-opt/builds/3078
success: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-t/builds/3079
success: http://buildbot.rust-lang.org/builders/auto-linux-32-opt/builds/3070
success: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-t/builds/3070
success: http://buildbot.rust-lang.org/builders/auto-linux-64-opt/builds/3073
success: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-t/builds/3066
success: http://buildbot.rust-lang.org/builders/auto-linux-64-x-android-t/builds/3061
success: http://buildbot.rust-lang.org/builders/auto-win-32-opt/builds/2738
success: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-t/builds/2736
success: http://buildbot.rust-lang.org/builders/auto-win-64-opt/builds/1574
success: http://buildbot.rust-lang.org/builders/auto-win-64-nopt-t/builds/1566


fast-forwarding master to auto = b5571ed7


zxombie:
```

You should also have:

  .if !empty(DDB_ENABLED)
  CFLAGS+= -fno-omit-frame-pointer -mno-omit-leaf-frame-pointer
  .endif

in sys/conf/Makefile.arm64 to ensure future compilers include the frame pointer.
It may be the default, but we shouldn't rely on it.
```


Fixes #1


The two variables were used as class variables instead of local variables as they are being used in both the functions at line numbers
85,86,87, and 89 in onSubmit()  function
222 in getRandomFakeSecretQuestion() function


Removed fdt_addrsize_cells() in favor to OF_getncprop


OK thank you for clarifying.


fast-forwarding master to auto = da400a7a


done


Inverting the two expressions of  the conditional operator makes the stop working as it should:
`bool pause = (newval > val ? !stops.contains(newval+1) : !stops.contains(newval-1));`


Removed excluding devices not compatible with "gic-v3-its".


Used central function for that


Hey @micolous - Please don't commit to this repo directory. Send a pull request and we'll merge it in.


good call!


There's enough logic in this controller to justify a class to manage the responsibilities of retrieving and comparing prices (OO design!). Short of that, using private methods could help make this easier to understand. For example:

```ruby
response_hash[:ups_rates] = sort_response_rates(response.rates)
# down below
def sort_response_rates(rates)
  rates
    .sort_by(&:price)
    .collect {|rate| [rate.service_name, rate.price] }
end
```


This was added in response to:
```
zxombie:
   This code looks familiar, it would be nice if there was a central function to merge this,
   ofwbus_setup_dinfo and simplebus_setup_dinfo.
```


I have a gui program that does all the pyav stuff in a bg thread and the jank is really noticeable without. I might have done it a bit excessively...


I'd use the `rc` package instead. Executing `npm` is slow.


Based on the previous pull request, I had made changes  to the file. In the process , I forgot to remove line number 212 as it does not affect the code and have created a new pull request with the new change :)


Twig bridge is working well with the latests L5? Wondering because I need it for a new project.


zxombie:
```
Looks good
```


придумал способ обойтись без внешнего модуля:
```
if (require.resolve(process.cwd() + '/node_modules/shark-core'))
///
```
таким образом ищем установленный пакет именно в текущей директории из которой запущен скрипт


Looks good


We're working on it now - looks like all is well but for some Twig functions (html_ I think)


saw approval from jdm
at https://github.com/saneyuki/servo/commit/d8fed6dbd463fa432d2dde1e884dba55609e85bc


merging saneyuki/servo/event = d8fed6db into auto


saneyuki/servo/event = d8fed6db merged ok, testing candidate = 0f12704b


zxombie:
```
How similar is this file to sys/arm/arm/busdma_machdep.c? Could we just use the same file on both?
```


saw approval from Aatch
at https://github.com/dotdash/rust/commit/6fba2a962417991a2c9236148f537f1443ae7e83


merging dotdash/rust/fca = 6fba2a96 into auto


status: {"merge_sha": "6b179741db7717acfc8b3051d413869a3bf0aa2e"}


dotdash/rust/fca = 6fba2a96 merged ok, testing candidate = 6b179741


Вместо таких конструкций лучше использовать [`np.logaddexp`](http://docs.scipy.org/doc/numpy/reference/generated/numpy.logaddexp.html):

```python
log_a = np.logaddexp(log_a, log_ksi_t)
```


## This is a markdown file


@edubxb would love your feedback here if you have any other ideas on how to do a multi-line prompt like this.


Are `map` and `compact` the right enumerable tools here? So, we're mapping into an array that's go a bunch of `nil`s in it and then compacting them out. `select` is what you want here.


`response` is a pretty generic name for this -- what is a this a response from? Is the response the application's user that Rails is about to send back? It's hard to tell without a more specific name. `ups_prices` may be a better name for this, though it's hard for me to know without more familiarity iwth it. Naming your variables carefully and accurately is tough but totally worth it: http://martinfowler.com/bliki/TwoHardThings.html


This was added in response to:
```
zxombie:
   There is no need to prefix everything with arm_. We know it's arm as it's in a GICv3 driver.
```
There was no sense to merge this change with the ITS driver since there were arm_ prefixes in GICv3 driver too. Therefore there is this commit.


Again—why are we coercing this into a string? We want to test our implementation in its purest form.


something's wrong here, I can't tell what you mean. I think you want to access the `e+i`-th element of that row; however, you can't do it this way, because the entries are not of the same size as a pointer, so you can't just add an `int` to the beginning as with an array of `int`.
I think maybe you want `(*rit)[e+i]` here.


Added without arm_ prefixes


Add spaces between anchors do they don't end up looking like a single word


w00t `master` build is passing again!


@csms
In any case, I want to switch the implementation to User::getAuthorities().
Seems it will have a better performance due to the internal caching.

Do you agree?


There should be a blank line between imports that come from this project and imports from 3rd party libraries.


His [git profile](https://github.com/TheCricket) lists his mail as bcord@me.com right under his username and icon.

If that's not it, then check git logs, as his email tied to the repo will be in there.


```py
Figure.print_grid()
```

outputs the grid generated by `make_subplots()`


these three lines do exactly the same thing as lines 36-38. You should remove the duplicated logic either with a class (preferable) or at least with one or a few private methods.


With new imports, you no longer need this.


I would also leave the ```figure``` margins under the Elements:

```
figure {
	margin: 0;
}
```


It survived somehow the trip in the constricted car and now got a cold shower. (Who actually now is cleaning up my bath room, hmm @skalk ?) Later it gets a weekend vacation on my balcony.


You are using Wordpress right?

I just commited a change to the ```(CRM.$)``` to ```(cj, CRM)``` to follow other core JS files.
(This was breaking JS in my case - in Drupal).


This constant should be moved above, next to `CONFPATH` and other constants.


I'll make a PR to try to fix.


Are you using tabs or spaces?


@Nyami looks like this caused the problem.  Although running ```rake generate``` locally on my own machine results in a success (I am running Jekyll 2.4.0), the version of Jekyll on the build server 2.5.3, seeing this as an error.  You can see this in the build log here:

https://ci.appveyor.com/project/GaryEwanPark/adnuguk-github-io/build/1.0.28#L92

I also pushed another small change (which I thought was the problem when I saw the failed build) and is documented here:

http://help.appveyor.com/discussions/problems/1137-problem-with-installing-gem

I found this problem on my own blog, but didn't move it over to this one.


You are only using `in_pkg` once for `STATICDIR`. Why not use it for other constants (e.g., L27-28). Personally, I think it's overkill.


Nice!


Aha, sent off the email I had drafted to that adress, ty. I didn't think to check github *facepalm*


Count was wrong cause this was above and thus this hook was unusable


May I ask why you introduced a new table for delivery_time? I did a similar thing, but I just added a simple string field to Order table to omit an additional query. I admit that your solution looks more elegant.


Well, that is more or less what that method is used for, right? See e.g.

    visitList(lems.getComponentType(), new BuildNameComponentTypeMapVisitor(lems));

It seems to me that having to override the automatically generated traversing visitor for Lems (because it is wrong, see #13 ) defeats the purpose of automatic generation.


Just confirmed - she works fine. Add these if you need em though: https://github.com/anomalylabs/streams-platform/commit/c86e03a9b90d4522ec2e011f103fa306d458a939


ok. If it didn't helped, feel free to drop it :)


This is probably an error in the crossbar_maintenance routine that hadn't been encountered yet. `create_account/4` should probably validate the account, create the account, and then validate/create user. Crossbar can't consider a user valid if the username matches an existing one for a different doc id.

Bug in `crossbar_maintenance:create_account/4` afaict. Thoughts?


This can be simplified a bit by returning inline (we often add a separate method for switch statements for just this reason):

```javascript
function shortErrorType(errorType)
{
    switch (errorType) {
        case "USER_ERROR":
            return "USER";
        case "INTERNAL_ERROR":
            return "INTERNAL";
        case "INSUFFICIENT_RESOURCES":
            return "RESOURCES";
    }
    return errorType;
}


Won't this always set `devnull = open(...)`?


Looks good:
```
 $ grep stops100 opentx.map
 .rodata.stops1000
 .rodata.stops100
 .rodata._ZL9_stops100
 .rodata._ZL10_stops1000
stops100                                          allsrc.o
stops1000                                         allsrc.o
```


You can launch the installation on apache/nginx too ;)

We use PHP builtin server here only to avoid an apache/nginx configuration as the BackBee Installer give you the full configuration on last step.


@theengineear @chriddyp

```py
tls.make_subplots(rows=2, cols=2)
fig.append_trace(Scatter(), 1, 1)
```

Exceptions when:

- grid_ref is not defined (e.g. when `make_subplots` hasn't been called yet)
- row or col is out of range
- axis objects corresponding to 'x{}' and 'y{}' ref are not found in fig['layout']

Adding an exception when `xaxis{}.domain` is modified **might** be too much in my mind. What if a user want to fine tune an axis domain after calling `make_subplots`?


Retry what 10 times? Please be more specific in the commit message.


Awesome, thanks!

On Fri, Jan 9, 2015 at 1:48 PM, Ryan Thompson <notifications@github.com>
wrote:

> Just confirmed - she works fine. Add these if you need em though: c86e03a
> <https://github.com/anomalylabs/streams-platform/commit/c86e03a9b90d4522ec2e011f103fa306d458a939>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/anomalylabs/streams-platform/commit/858dfae55d0b86618308a43f6b5c186e0dc71e0d#commitcomment-9224239>
> .
>

--
Osvaldo Brignoni


spaces


inside the picture, or in the markdown file ?


Of course!

Ryan Thompson
President / CEO
AnomalyLabs, Inc.
www.anomaly.is

t (877) 705-0806
m (309) 752-8581

> On Jan 9, 2015, at 1:51 PM, Osvaldo Brignoni <notifications@github.com> wrote:
>
> Awesome, thanks!
>
> On Fri, Jan 9, 2015 at 1:48 PM, Ryan Thompson <notifications@github.com>
> wrote:
>
> > Just confirmed - she works fine. Add these if you need em though: c86e03a
> > <https://github.com/anomalylabs/streams-platform/commit/c86e03a9b90d4522ec2e011f103fa306d458a939>
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <https://github.com/anomalylabs/streams-platform/commit/858dfae55d0b86618308a43f6b5c186e0dc71e0d#commitcomment-9224239>
> > .
> >
>
>
>
> --
> Osvaldo Brignoni
> —
> Reply to this email directly or view it on GitHub <https://github.com/anomalylabs/streams-platform/commit/858dfae55d0b86618308a43f6b5c186e0dc71e0d#commitcomment-9224267>.
>


Closes #76


s/make/Make/ in the commit message. Can we have the same easily for wget, too?


Thanks for that thought - I'll test out that change and make a PR if it works out.


develop (macumber) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2161 of 2188 tests passed)

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


is there any reason we can't just add _grid_ref as an attribute instead of a key?


thank you @crouillon to have back ported my PR, but can you make PR instead of directly push on ``0.11`` branch ?

A review is always positive :+1:


Ah? I would have said (and therefore I implemented) the opposite! What other say about this? @kilrah @projectkk2glider ?


The commit message title is a bit long, it wraps. Please explain the details in the commit message body, including why we now use an absolute path to `strip`.


I know `efficiencies.py` is a work in progress, just a thought that having one file per efficiency might work nicely, e.g.

```
├── efficiencies
│   ├── generator.py
│   ├── reconstruction.py
│   ├── stripping.py
│   ├── ...
│   └── utils.py
└── fitting.py
```

Each efficiency file can define the same method, e.g. `efficiency`, for a script in `analysis/` to loop over.


:clap:


merge


sdfsad as sdd


These ones are missing the `/`. Also I think we don't need to write the name if it's the same value as the route.


This is added to have a way to show information about order delivery time in e-mails sent to clients (and in other places where it might be required). Such information in e-mail is required by a new consumer law that was recently introduced in Poland. This new act is based on EU regulations so it is likely that other countries might require such information too.


re: domain modification. yeah, i think that's overreaching. good call.


I'm glad it didn't explode.
>the only reason this function should be used is if they want to send something to the server

Well, maybe not. We should allow inter-plugin communication (maybe have a trusted senders list?). I was thinking that the syntax would ultimately be something like this:

```python
    dds.send("Grandma",<Message>) #Send to Server
    dds.send("MyCoolOtherPlugin",<Message>) #Send to MyCoolOtherPlugin
    # ===== OR, Alternatively: ======
    dds.send-server(<Message>) #Send to server
    dds.send-plugin("MyCoolOtherPlugin", <Message>) #Send to MyCoolOtherPlugin
```

Also, I should note that the plugins are only expected to send the serialized `content` portion of the message; we'll add the needed info to make it a fully-fledged socket-message.


Yeh, the "GrahamCampbell" for actually redirects to this one. Try it out yourself: https://github.com/GrahamCampbell/sentry.


where did you find robot builder, I can't find it anywhere :l


This is for the node.js REPL env? How is it not working given 0.1.5-SNAPSHOT? (I added a REPL interaction log using the node.js REPL env to the README this morning.)

I don't think the compiler environment is being rebuilt; it's just being dereferenced from whatever REPL environment has been "set" on the current nREPL session (or defaulting to the…default).


Line 77, update the doc to reflect new status column.


See Trello for a discussion


@lvangool I think this might be erroring out on some stacks:

```
E: Sub-process /usr/bin/dpkg returned an error code (2)
```


add pages


@alviteri I disabled SELinux on my phone so I'm not sure if this needs some extra policy...


Why this was included?


Nice. Good catch.


FWIW... https://github.com/eric239/planout/commit/4bf9f40f7091a64e70beb8a2b4181699dc2ebb36


Hi guys... I understand #1 (PRs without tests are not merged) but how does #2 impact contributors (not maintainers like yourself).  Should people outside your team stop submitting PRs?


ack, it was a mistake. I'll remove it.


type assertion should be better. it will be faster.


awesome !


I follow the naming rule of First10of5K.txt.
/n/scratch2/pfister_lab/thouis_jones/big_dataset/Connectome/Pipeline/0Control/Python/../../../../../../../amelio/data/conn/main_dataset/5K_cube/input_images/z=000001.tiff
Do your team have open  dataset include these image files? can you provide the website?
Or the rhoana workflow samples?


I'm not sure why git thinks I deleted every line of ```json_module.f90``` and then added it back...


1277852-CommandLineInterface (nealkruis) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1587 of 1589 tests passed)

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


@Hypercubed , I think you're misunderstanding. PRs *can be merged*, no matter who they're from. It's just that they're merged manually using command line.


https://github.com/linuxmint/muffin/commit/8039f9ec6eb4286228e4317e4e3a32f22cdd0beb

Desklet appears to work now - please take a look when you can.


@Christopher83 @arco Hello, with this configuration there seems to be no move app to sd option is this correct? Or did i miss something? Also idk if you guys got my messages but have you considered a configuration using external sd card for /data and sd as this give potential to load alot more apps, as a option?


Thanks @devnatash !!


4660-GLHE_Vertical_Input_Fixes (mmatts) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1590 of 1590 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)


Removing this while fine and making perfect sense also creates inconsistencies with other areas of the framework, for example Controllers and Seeders.


My apologies, I thought I had uploaded this previously.  I will go back tonight and make sure it is up.


Nu trebuia sa elimini ebuild-urile vechi neaparat, da' meh. Daca asa trebuie sa fie :-j


1277852-CommandLineInterface (nealkruis) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


How did this event happen...


No, i meant this... how did i do that...


why positive listener finished the activity.


I got it… we run Mac’s at work and I’ve been tasked with figuring out how to do some JMeter automation against SURO and a few other specialized tools. It was nice finding your SuroMeter sample as a JMeter plugin.

Maybe the shell runner scripts aren’t all that necessary – I think a better solution would be to give proper instructions on where to find your JMeter installation and where to copy the JAR into – or even link back to Apache’s JMeter tutorial on plugins – that tells where to put the JAR…

Regards,

Greg Patnude

Cell:                        <tel:%28253%29%20320-9751> (253) 320-9751
E-Mail:  gpatnude@hotmail.com
Web Site:            http://greg.patnude.com <http://greg.patnude.com/>
ICQ:                       232407433

“…I never let being smart stop me from doing stupid things…”

cid:image002.jpg@01CDB669.F49FB120

From: starzia [mailto:notifications@github.com]
Sent: Thursday, January 08, 2015 3:26 PM
To: gpatnude/surometer
Cc: gpatnude
Subject: Re: [surometer] Update compile_and_run.sh (9b1bc17)

@gpatnude <https://github.com/gpatnude> , good catch. Unfortunately my script is kind of a hack to begin with, since it only works with a specific version of JMeter installed on a Mac.

—
Reply to this email directly or view it on GitHub <https://github.com/gpatnude/surometer/commit/9b1bc17981d1414988fc5b920df6d7594c653fa9#commitcomment-9211245> .  <https://github.com/notifications/beacon/AJ0IrWdHKRgX0kdZ6ElMVlLreVe72ixsks5nfwmagaJpZM4DQIhk.gif>


`null|string` or strict constructor


Same as above


develop (macumber) - Win64-Windows-7-VisualStudio-12: OK (2174 of 2188 tests passed)

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


Looking at other lines - first letter probably should be in uppercase


The test need to guarantee that the db index will not change if any error
occurs when trying to select a new index, no matter what error.
What about change the method name to be something like
shouldNotUpdateDbIndexIfGotSomeError.

I'm sorry I'm answering from iphone.
On qui, 8 de jan de 2015 at 22:27 Jungtaek Lim <notifications@github.com>
wrote:

> And setting negative value and exceed value has same error message, so we
> could be fine with negative value, explaining developers that we're trying
> to select "invalid" db.
>
> 127.0.0.1:6379> select -1
> (error) ERR invalid DB index127.0.0.1:6379> select 100000
> (error) ERR invalid DB index127.0.0.1:6379> select 100
> (error) ERR invalid DB index127.0.0.1:6379> select 10
> OK
> 127.0.0.1:6379[10]> select 120
> (error) ERR invalid DB index
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nykolaslima/jedis/commit/285841ce7b6eb9ee536ed5babb9901e2f3dcaef1#commitcomment-9211850>
> .
>


❤ Looks so nice


Local phpunit is screwed so cannot verify if the test passed or not. I suspect `unlink()` will fail...


I'd suggest `RepositoryDTO` to not be confused with the entity.


okay.


la barra ploma en la portada del disco quiero poner botones de favoritos (corazon) y el icono de compartir


Assuming you've ran tests, I'd say it's fine.

As a somewhat unrelated note, I've read in more than one place that it's
considered "cringe-worthy" to do something like 'if (cur != NULL)' or 'if
(cur == NULL)', rather than 'if (cur)' or 'if (!cur)'. If you agree with
this, I'd still say that we shouldn't bother actively seeking to eliminate
these supposed bad-practices. But rather, just modify them as we encounter
them.

On Thu, Jan 8, 2015 at 3:05 PM, Robert Cochran <notifications@github.com>
wrote:

> @JoshuaBrockschmidt <https://github.com/JoshuaBrockschmidt> - after this
> quick bandaid patch, everything looking good to merge into master?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/TheUnderscores/protolang/commit/99a02c178df396143aaf8c76d3f6d353fab3905d#commitcomment-9210918>
> .
>


관리자 페이지로 가는 링크도 ssl 적용이 되게 알려주신 부분 반영했습니다.


@collinss I was looking at the calculator app - it looks like the buffer handleKeyPress method is translating numeric keys incorrectly (or maybe I need a keypad?) - the 'key' value seems to equal the actual numbers I type (rather than 49, 50, 51, etc...).  I checked with and without this patch set, with the same behavior - was just checking to see if there were new issues caused, but it looks like it was pre-existing.


I think we should create a tag "VERSION", just for fun of it :smiling_imp:


sorry for this breaking change.


r+


Needed scene changes.
1) Attach lootTable.cs to the prefab: Beehive, PicnicBasket, BerryBush, Boulder.
2) Attach LootItem.cs to LootWindowControl empty game object in scene.
3) In the hierarchy place the player game object on layer 9.
     The layer can have any name such as  "Player". It's a similar process to adding a tag to an object.


If we call f.find with TraverseLeafs=true, we can remove both the {Cluster}ComputeResource cases and remove setResourcePoolInventoryPath.  I think its cleaner, but still end up with a extra trip to traverse leafs if given a path to host or cluster.  What do you think?


PS: Talked to @tkrugg on IM and he agreed the fix isn't right (but there's no automated test to check for it).   He had a suggested fix at https://paste.kde.org/p6zc21o8j.


oops. InfiniteScrollItem ;)


Also, hopefully I can actually get some work done on protolang this
weekend. I've been trying to mix A-level homework, a social life , 8 hours
of sleep and some other various commitments, so I'd suppose you could guess
why I'm struggling to find free time. I know you'll probably tell me it's
excusable, but I want to let you know that /I/ really do want to work on
this project. I'll probably start cutting sleep from the equation. Maybe a
"No-Sleep Thursday" or "Cat-Nap Wednesday" so I'm not /constantly/
sleep-deprived.

On Thu, Jan 8, 2015 at 6:26 PM, Joshua Brockschmidt <
joshuabrockschmidt@gmail.com> wrote:

> Assuming you've ran tests, I'd say it's fine.
>
> As a somewhat unrelated note, I've read in more than one place that it's
> considered "cringe-worthy" to do something like 'if (cur != NULL)' or 'if
> (cur == NULL)', rather than 'if (cur)' or 'if (!cur)'. If you agree with
> this, I'd still say that we shouldn't bother actively seeking to eliminate
> these supposed bad-practices. But rather, just modify them as we encounter
> them.
>
> On Thu, Jan 8, 2015 at 3:05 PM, Robert Cochran <notifications@github.com>
> wrote:
>
>> @JoshuaBrockschmidt <https://github.com/JoshuaBrockschmidt> - after this
>> quick bandaid patch, everything looking good to merge into master?
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/TheUnderscores/protolang/commit/99a02c178df396143aaf8c76d3f6d353fab3905d#commitcomment-9210918>
>> .
>>
>
>


@ldacosta you expect the same constants to be used everywhere in the code base? i think putting them in one place is a good idea, but the way it is currently is not extensible. the way to make this extensible would be to encapsulate these in a trait which was either mixed in, or passed on in the form of a typeclass implicit (the latter being similar in terms of flexibility to passing the defaults explicitly). there's a tradeoff between the two option, and neither is needed for this pull request (or it'll never get done lol), but something for you to keep in mind


Can this be a map of an empty struct instead of uint8 since it's used just to test for the presence of a path and the value has no significance?


Wrong comment name (leftover from previous commit and push)


I don't see a definition or explanation of needles.


the returned error strings should consistently begin with lower case


case


init the project


Why do paragraphs get it's own spot in the cascade hierarchy? We should be able to put this into /components


Adding two more test cases to check correct usage of EPSILON


Why not just sum the values for the queen?


@ldacosta i see now what you were doing with the protected method. i should avoid reviewing code on the phone


Protect against the case where `left_edge` and `right_edge` are equal, and hence `width` will be infinitesimally small.


Ok, tomorrow i will report, thanks @mtwebster and don't take to much care of what i say... I'm not an enemy... And finally agree or not i will try to help, how you see that i do for example on the Super_R behavior. I'm not agree with this behavior, but i try to help, to not be removed by clem, reporting thats is working good for me...


@ldacosta any reason why these aren't ``implicit``?


What is the reasoning behind using a mixin instead of an @extend? Also, "paragraph" is the name of the module. "paragraph-tile" would be a new module altogether. "paragraph__tile" would be more suited to naming conventions


See commit 2aa02c3 - if you're good with it, I'll squash that commit.


When points are infinitesimally close together.


just add a space before "{" (on Image and Timer, also on ShaderEffectSource.qml line 33 too ;))


Change to test for being within `EPSILON` of the edge.


Actually, I think the most recent version of Git (currently 2.2.0, although this change may have been made in 2.0) defaults to `git push --simple`, which only pushes the current branch. I'm not sure if this point is relevant anymore. See the following from the Examples section on [Git's `git push` page](http://git-scm.com/docs/git-push/).

>GIT PUSH
>Works like git push \<remote>, where \<remote> is the current branch’s remote (or origin, if no remote is configured for the current branch).

>GIT PUSH ORIGIN
>Without additional configuration, pushes the current branch to the configured upstream (remote.origin.merge configuration variable) if it has the same name as the current branch, and errors out without pushing otherwise.

Specifically, "pushes the current branch," meaning that only one branch is pushed.

Now that this is `git push`'s function, what would you say are the advantages over an alias for `git pull` to `git pull --rebase`?


Define `EPSILON` to a difference we consider points to be 'arbitrarily close'.


Import the floating point absolute value function for use.


i'm guessing you meant 85c8830 instead


is there any way that we can run this manually, for testing purposes?


If you mean for speed reasons this all gets evaluated at compile time.  If it passes I'll leave it to the maintainers to decide what form they like best.  I only wanted it to be clear where the queen values came from.


Can you remove the "WIP:" from the commit message?


not always an error, so maybe handle_ceod_response is a better name


anulado


It would be ideal if the styling that goes along with these paragraphs were extracted into separate display related classes. The word "paragraph" may mean something in the Drupal sitebuilding world but it tells us nothing about what display patterns are being extended. Attaching a mixture of layout, colouring, and paddings without a method to the madness to sitebuilding classes supplied by Drupal makes it very hard to manage the cascade.


I am fine with leaving this publicly exposed if there's no way to specify message or recipient from the URL


Don't we need to close the reader?


Please PR changes in the future so that we can review (and are notified of changes).


`if err = i.Unmarshal(buf); err != nil {`


I'vr always done NULL checks explicitly. Thus... Uh, well, if you don't like it, it gets fixed. No questions.


Add col-highlight


I think he was getting an error. In that case, we need to use branches, guys


The thing is, that will slow down slide logic. What we have now is that plugins get a dict of other plugins receiver functions via the setup method. That allows plugins to directly communicate with each other, no middle man. i think this lib should only be for sending things to the server


some tests failed:
failure: http://build.servo.org/builders/mac1/builds/718


Is it necessary to manually invoke the functions? If I'm not mistaken, handlebars does that for you, so maybe you can replace with this: ```_.extend(opts.collection._transform(doc), doc)```


IDFXT (Unknown) - x86_64-MacOS-10.9-clang: Tests Failed

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


@doutriaux1 do I get the car keys back now?


@jcheng5 crazy! But doesn't `writeBin()` write 0-terminated strings?


Would it be an option to allow for logging ($error['type'] == E_ERROR) for 2.2 , that should cut down on the amount of logging generated.. and still log why a crashing package installation (for example 'tinc') happens?


Totally missed the first error and version, just saw what I guess was the effect of the problem with the missing folder...

I don't fully get octopress etc just yet, I've set it up for a blog of my own, making a little more sense as I encounter the problems :-)

Sent from my Windows Phone
________________________________
From: Gary Ewan Park<mailto:notifications@github.com>
Sent: 09/01/2015 19:46
To: adnuguk/adnuguk.github.io<mailto:adnuguk.github.io@noreply.github.com>
Cc: Nyami<mailto:nyami@hotmail.com>
Subject: Re: [adnuguk.github.io] Corrected sponsors link (e636a84)

@Nyami looks like this caused the problem.  Although running ```rake generate``` locally on my own machine results in a success (I am running Jekyll 2.4.0), the version of Jekyll on the build server 2.5.3, seeing this as an error.  You can see this in the build log here:

https://ci.appveyor.com/project/GaryEwanPark/adnuguk-github-io/build/1.0.28#L92

I also pushed another small change (which I thought was the problem when I saw the failed build) and is documented here:

http://help.appveyor.com/discussions/problems/1137-problem-with-installing-gem

I found this problem on my own blog, but didn't move it over to this one.

---
Reply to this email directly or view it on GitHub:
https://github.com/adnuguk/adnuguk.github.io/commit/e636a8431636ee54c183b66d7fcb1e94a57aff24#commitcomment-9224206


## This is a markdown file


I see your interaction log.
(evaluated (.-version js/process)) ... that will work of course.
Try it on 0.1.5-snapshot and reference some cljs.core ... ah ... 'vars'
cljs.user> (map inc (range 10))


@doutriaux1 this is now going to break some other things as your obs are no longer copied.. I think we need to work on this one together..


Phobos has a decent FFT implementation using the standard Cooley-Turkey algorithm.
https://github.com/D-Programming-Language/phobos/blob/bef24a3bee33fce6f5b418fed052d565a626e556/std/numeric.d#L2492
The linked-to library doesn't precompute sin/cos lookup tables and will already have performance problems because of this.


I agree, however i based this on the fact that there are year table headers when there is no picks (see: http://jupiterbroadcasting.com/laspicks/android.php)


Is this also supposed to be videouploadconfig?


Yeah, but this commit is just a cleanup since in my version the nav_* variables do not exist anymore.


trackstar


some tests failed:
failure: http://buildbot.rust-lang.org/builders/auto-win-64-nopt-t/builds/1567
exception: http://buildbot.rust-lang.org/builders/auto-mac-32-opt/builds/3086
exception: http://buildbot.rust-lang.org/builders/auto-mac-64-opt/builds/3079
exception: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-t/builds/3080
exception: http://buildbot.rust-lang.org/builders/auto-linux-32-opt/builds/3071
exception: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-t/builds/3071
exception: http://buildbot.rust-lang.org/builders/auto-linux-64-opt/builds/3074
exception: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-t/builds/3067
exception: http://buildbot.rust-lang.org/builders/auto-linux-64-x-android-t/builds/3062
exception: http://buildbot.rust-lang.org/builders/auto-win-32-opt/builds/2739
exception: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-t/builds/2737
exception: http://buildbot.rust-lang.org/builders/auto-win-64-opt/builds/1575


I just checked, and it still exists as specified. https://github.com/autopkg/hjuutilainen-recipes/blob/master/GitHub/Atom.pkg.recipe#L8 Plus, the download recipe won't make a package. Do you have hjuutilainen's recipes added to your list of repos?


hi


The year thing was not clear to me at the time. (See my comment in the other commit about the android app picks).


Yip, been there.  Was a bit of a learning curve, but once you are up it, I really like it :smiley_cat:


I was just moving code around for inet_ptop. No plans to fix it for now.


I've tried both and slightly prefer @davxlw way.


the issue is env/*compiler* is getting bound to a new compiler env on each pass through cljs-eval ... https://github.com/pmbauer/piggieback/blob/49ed03808328a4e94edca8308b4588baf36af0a2/src/cemerick/piggieback.clj#L88
tools reader needs that compiler env to fully resolve names in the context of the current namespace:
https://github.com/pmbauer/piggieback/blob/49ed03808328a4e94edca8308b4588baf36af0a2/src/cemerick/piggieback.clj#L222

... but that compiler env is thrown away on each call to cljs-eval unless it is attached to the ::env/compiler key

Compare to how env/*compiler* is preserved on each pass through the repl loop in cljs/repl.clj
https://github.com/clojure/clojurescript/blob/master/src/clj/cljs/repl.clj#L342
https://github.com/clojure/clojurescript/blob/master/src/clj/cljs/repl.clj#L369


saw approval from alexcrichton
at https://github.com/brson/rust/commit/df059020c4c1ff61a70288cf32c2259dabb4fecc


merging brson/rust/installer = df059020 into auto


status: {"merge_sha": "73a25f55ad748b4d3516417c711b99ce446591af"}


brson/rust/installer = df059020 merged ok, testing candidate = 73a25f55


Oh, that's for inet_ntop. Ugg, god damn windows.


Any reason you're adding this blank line? :)


... now I might be totally solving this the wrong way.  But the above patch is working for us anywho (clojure 'vars' resolve, (require ... :reload) works, etc)


Let me explain how I would use the pauses:
- if I want to use SC, I short press [+] and I see SA... so I'm in the right kind of switch, don't need to long press.
- if I want to use tEd, I long press [+] and it will pause at tRl, I can now choose my trim switch with short press.
- if I want to use L5, I long press [+] and it will pause at tRl, then at L1, I let go the button and continue with short press.

If it pauses before the change of switch type, I don't know what switch type will follow, so should I maintain the button pressed or not ?


develop (macumber) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Yes that is his email

Sent from my iPhone

> On Jan 9, 2015, at 11:43 AM, CptSpaceToaster <notifications@github.com> wrote:
>
> His git profile lists his mail as bcord@me.com right under his username and icon.
>
> If that's not it, then check git logs, as his email tied to the repo will be in there.
>
> —
> Reply to this email directly or view it on GitHub.
>


We usually put a prefix into the subject of the commit message to identify the area that the change affects.  This is the package name, "daemon", "bridge", "ws", or "test".  The package is "server-systemd" here.  I'll add it when merging.


Is there good reason to do more than just seed the zoneinfo file in the appropriate place as the original code did, and let nspawn continue creating the symlink?  I'd prefer we limit the change to fixing the relative symlink case.


drops ALT text for pages like /training/


develop (macumber) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2068 of 2189 tests passed)

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


Hehe, I'm seeing it in many projects since "flat" style is the new default. For almost all badges...


works for me too, thank you


Is this necessary?  Can't we just readlink /etc/localtime like the original code and use the results when successful?

Keep in mind this is just to reproduce the /etc/localtime _target_ and let nspawn do its setup_timezone() thing, not reimplement it.


With this change, the "NPE" error still occur if ngModel did happen to be undefined. It seems safer to check for both defined and non-null. So either:
```
if (angular.isDefined(ngModel) && (ngModel !== null)) {
```
or the shorthand syntax:
```
if (ngModel != null)
```
(note the use of !=, instead of !==)


P.S.
```
$ git log | head -n 10
commit b619424f9bfd4c5abe090ff236a1a486ff97f425
Author: Chas Emerick <chas@cemerick.com>
Date:   Fri Jan 9 09:27:39 2015 -0500

    use scratch analysis environment for special functions, not compiler environment, gh-38

commit 208f2765225a6532d6b0578e965e54f929fbe952
Author: Chas Emerick <chas@cemerick.com>
Date:   Fri Jan 9 09:19:33 2015 -0500

$ lein repl
nREPL server started on port 48423 on host 127.0.0.1 - nrepl://127.0.0.1:48423
REPL-y 0.3.5, nREPL 0.2.6
Clojure 1.6.0
Java HotSpot(TM) 64-Bit Server VM 1.7.0_72-b14
    Docs: (doc function-name-here)
          (find-doc "part-of-name-here")
  Source: (source function-name-here)
 Javadoc: (javadoc java-object-or-class-here)
    Exit: Control+D or (exit) or (quit)
 Results: Stored in vars *1, *2, *3, an exception in *e

user=> (require '[cljs.repl :as repl])
nil
user=> (require '[cljs.repl.node :as node])
nil
user=> (cemerick.piggieback/cljs-repl
  #_=>          :repl-env (node/repl-env)
  #_=>          :output-dir ".cljs_node_repl"
  #_=>          :cache-analysis true
  #_=>          :source-map true)
ClojureScript Node.js REPL server listening on 58124
Type `:cljs/quit` to stop the ClojureScript REPL
nil
cljs.user=> (map inc (range 10))
WARNING: Use of undeclared Var /map at line 1 <cljs repl>
WARNING: Use of undeclared Var /inc at line 1 <cljs repl>
WARNING: Use of undeclared Var /range at line 1 <cljs repl>
"Error evaluating:" (map inc (range 10)) :as ".map.call(null,.inc,.range.call(null,(10)));\n"
SyntaxError: Unexpected token .
    at Socket.<anonymous> ([stdin]:30:80)
    at Socket.EventEmitter.emit (events.js:95:17)
    at Socket.<anonymous> (_stream_readable.js:746:14)
    at Socket.EventEmitter.emit (events.js:92:17)
    at emitReadable_ (_stream_readable.js:408:10)
    at emitReadable (_stream_readable.js:404:5)
    at readableAddChunk (_stream_readable.js:165:9)
    at Socket.Readable.push (_stream_readable.js:127:10)
    at TCP.onread (net.js:526:21)
nil
cljs.user=>

```


I don't think this really makes much difference. People can just rename/delete classes to match their own preference. :)


develop (macumber) - i386-Windows-7-VisualStudio-12: OK (2169 of 2189 tests passed)

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


Have we decided to just do suffixes in lowercase now?  I realize it doesn't matter too much since the case matching is insensitive, but I just always liked the aesthetics.


Creo que este metodo sobra si se hizo lo del registrations controller, que dice? @orendon


First runs:
Israfel vs Bodmall was pretty evenly matched - Israfel died 47 times, Bodmall died 53 times.
Bodmall vs Miranda: 0 to 100.


@joepestro - thanks for the heads up. The parts I added shouldn't cause any issues - they just enforce rules for non-interactive install defaults (I actually changed it because another user had issues with it hanging on an upgrade config-file update query.

Is this an existing issue on your server? Do you have any other error info? Would be useful to know if apt-get update or apt-get install -y newrelic-sysmond work alone? Also, can you confirm your OS/versions?


nice find!


rax retry tests


Ehh quiay, cambiame el codiguito @MIsabelZM


develop (macumber) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


`Network` refers to the whole networking of the container. `Net` refers to a single network that the user defines. I'm calling it `Net` because I want the user to specify it on cmdline via `--net=abc`. `--net` would be consistent with Docker flag and qemu-kvm. But I'll change Network to Networking.


People might want to type hint the params in their implementations, but php won't allow this if we have an untypehinted method defined in an interface.


Typo here. The != NULL part was incorrect and break the builds:
~~~
--- all_subdir_arcnet ---
/jenkins/workspace/HardenedBSD_Master/sys/modules/arcnet/../../net/if_arcsubr.c:170:30: error: comparison between pointer and integer ('int' and 'void *') [-Werror]
                if ((m->m_flags & M_MCAST) != NULL)
                    ~~~~~~~~~~~~~~~~~~~~~~ ^  ~~~~
~~~
more info here:  http://nyi-01.build.hardenedbsd.org:8180/jenkins/job/HardenedBSD_Master/128/consoleText


This will add a README


Dynamic programming != Dynamically-typed programming


Creo que podriamos haber hecho el user:references para que tambien nos cree el index :)


I had to read that sentence twice. Just an FYI.


is it able to report scan progress every seconds ? like here:
(pydev)[dev@bouteille python-nmap-lib]$ python examples/proc_async.py
Nmap Scan running: ETC: 0 DONE: 0%
Nmap Scan running: ETC: 1369433951 DONE: 2.45%
Nmap Scan running: ETC: 1369433932 DONE: 13.55%
Nmap Scan running: ETC: 1369433930 DONE: 25.35%
Nmap Scan running: ETC: 1369433931 DONE: 33.40%
Nmap Scan running: ETC: 1369433932 DONE: 41.50%
Nmap Scan running: ETC: 1369433931 DONE: 52.90%
Nmap Scan running: ETC: 1369433931 DONE: 62.55%
Nmap Scan running: ETC: 1369433930 DONE: 75.55%
Nmap Scan running: ETC: 1369433931 DONE: 81.35%
Nmap Scan running: ETC: 1369433931 DONE: 99.99%
rc: 0 output: Nmap done at Sat May 25 00:18:51 2013; 1 IP address (1 host up) scanned in 22.02 seconds
(pydev)[dev@bouteille python-nmap-lib]$


What confused me is there were already creates and removes just a bit further down in the code. Why did they work? Perhaps they did not? I guess this dyna_salt_init() was needed all the time so should be moved outside the `#if FMT_MAIN_VERSION` block (although that one is bloody unlikely to ever be false so we could just as well drop that #if/#endif).


Strangely, I'm getting a checksum mismatch on this file because it appears to only be a prefix. The checksum I'm seeing is `c9577bba0321eeb5358fdea29634cbf124ae3742e80d729f3bd98e0e23726dbf`, which is just a longer version of the checksum here.

I fixed it by manually overriding this attribute, but I'm curious why your checksums appear to be prefixes of full sha256 ones?


This will take a sentence in a string with a single word as an argument and return the instances of the argument


I want LoadNet to work with arbitrary structs because JSON config can have plugin-specific keys. Therefore LoadNet's second arg is of type interface{}. This is the same as json.Unmarshal.


r+


Ah I see now, it's called in fmt_self_test() which is called inbetween.


This was purely an optimization to avoid splitting something that's known at compile time. It's a private function so I thought I'd sacrifice awkwardness.


yes, but this is only happening on the server side. So while it's sending over the data, it ignores functions, since those cannot be transfered over JSON as they contain logic.

On the client side we lost the connection to our original collection, since we use the 'esSearchResults' collection now.


I think we might have a function for this already, just not in packet.h? I'm not sure, could be mistaken.


You're right, it is just the initial part of the full 256 checksum. As far as I know Chef only needs a portion of the checksum in order to match (not to say you can't use the whole string.) I used the same length that was there for the checksum of the older jamm version - maybe I should have just put the full length in there. What version of chef are you running?


"running" here should be made conditional to the visibility of the splash image; otherwise we could fall again in the high cpu usage scenario.


Where the active come from?
I don't expect the `getattr(self, "type_")` work too.
And there is no variable `object_` in `object_.name`


Hai, you forgot to sign this, kthnxbye :)


the extra brackets around this whole statement can be removed too


Much easier then switching all the logic around 😀


thumbnail need alt text


Removing thanks!


We really need something better than "2FA". This method is hard to type, lol.


Ouch. I never saw this because both the stock browser REPL and Austin capture a compiler env automatically, e.g. https://github.com/clojure/clojurescript/blob/master/src/clj/cljs/repl/browser.clj#L284

Please send a PR! :-)


img tag needs ALT text - along with others on this page


`getHasEnabledMFAAttribute`?


Fixed in r 276896, thank you!


<img> needs alt text


Yeah, he's on IRC now, but thanks anyway!


It does not work with slash? You really need to use backslash here?


Using a Loader would also be an alternative to the "hidden"/"visible" states of the splash object. Worth a try, as it may consume a little bit less memory.


On 01/09/2015 09:25 PM, magnum wrote:
> What confused me is there were already creates and removes just a bit
> further down in the code. Why did they work? Perhaps they did not? I
> guess this dyna_salt_init() was needed all the time so should be moved
> outside the |#if FMT_MAIN_VERSION| block (although that one is bloody
> unlikely to ever be false so we could just as well drop that #if/#endif).

If you drop it, you should drop all other #if FMT_MAIN_VERSION > 11 that
are outside of a format implementation. Otherwise things will break.


@brixen Downgrade? Was this intended?


IDFXT (Unknown) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


The continuous integration system doesn't like this commit: https://travis-ci.org/viennacl/viennacl-dev
Looks like the matrix-vector product has some issues.


I am sorry, I had to add some `xmlbase` parameters to `text_construct`, you may want to push again the documentation.


CentOS 6 installs MySQL 5.1 so I've added instructions to get the version 5.5 installed.


True!


Erm... @edwinb was the `Effect.Protocols.idr` filesupposed to be part of this commit?


All the "with" values with the exception of "args" is coming from rocket code and is not supplied by end user. I'll do {args} replacement last to make sure it doesn't interfere. It also will allow the user to actually use curly braces if needed.


but it could be maybe possible to call the transform on the client.


Should this be `<= 1` to handle empty pages?


I tried to make submodule but when i tried commit it wanted to commit to uberserver. I need something that can be kept "different"


What is exactly your idea? do you have any example of something similar? I checked it and looks weird in my terminal...

Oh-my-zsh have some themes that can be what you are looking for:

Agnoster → https://gist.github.com/agnoster/3712874#file-screenshot-png
This Agnoster "mod" → https://gist.github.com/smileart/3750104


now that I look at it, this commit adds some commented out error statements where I use them frequently
also you will notice that sanitizing now trims so people like branden can't just throw a space after vmutti and call it a username


It worked and I submitted a PR with the change: https://github.com/2600hz/kazoo/pull/699


@rjcruz esto hace que el menu se devuelva bajo un nodo llamado "arrays", podemos nenombrarlo a "menuItems" asi el array tiene el mismo nombre que cuando se usa con expandedMenu=false.
Gracias


Thanks for the quick reply @dcparker88.

We're using Chef 12.0.0 (installed fresh by the omnibus installer). It does seem to expect the full checksum for us. Once we overrode the attribute, everything went smoothly.


nice.


Yep it's the new style


I am beginning to like the fact that each module has its own `print`s.


@lvangool This was only happening on initial provisioning, and I initially thought it was an issue with my `deploy_hooks.yml`, since it was failing silently every time. It had been working for me prior to this commit, though.

Yes, apt seems to be configured fine, e.g. `apt-get install -y newrelic-sysmond` works without issue.

Tested on Ubuntu 14.04.1 LTS.


Seems like the dislike was only a glitch, works ok now on my machine.


The fragment:  "and it'd be your code will communicate its purpose better"   needs reworked.


Yeah, I was swapping between true and false while debugging.


rax retry tests


I like being able to reuse our current functions. Thanks.


develop (lefticus) - Win64-Windows-8.1-VisualStudio-14: OK (331 of 331 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)


Use version in Airlift `Assertions`


I have a form component that renders a `mui.Input` with no `defaultValue` prop.  I want to provide a `reset` method on the form component and was previously using `input.clearValue()`.  When the form is submitted, I was using `input.getValue()`.

What is the suggested alternative to the `clearValue` and `getValue` methods?


`assertGreaterThan`


`assertLessThanOrEqual`


:smile_cat: cat smiley


One useful bit of command-line code I use when I goof this up is to call `git commit --amend`.


IDFXT (Unknown) - i386-Windows-7-VisualStudio-12: Build Failed

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


ignore the "hello" comments in the program, used them for debugging


See discussion here:  https://github.com/calabash/calabash-android/pull/517

I am not in favor of keeping 2.3.3 dependency.


This creates an error when trying to build Osirix


Ugh, yes. Thanks for catching that!


Great! glad this fixed a real problem.
I sent a pull request with a commit message that more accurately reflects the reasoning behind the change.


IDFXT (Unknown) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Tests Failed

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


LGTM


This seems to have broken the daily builds on Ubuntu.

CMake Error at CMakeLists.txt:3 (cmake_policy):
  Policy "CMP0043" is not known to this version of CMake.

https://launchpad.net/~otter-browser/+archive/ubuntu/daily/+build/6702199


@bors: retry
* #3228


Is the next version going to be 0.5? I was thinking of skipping to 1.0.


I thought we would need to wait longer ? Well, if 0 is sufficient, that's all good :)


r+, looks good!


Ok, interesting. I can make a PR with the full checksum, it doesn't necessarily affect lower versions of chef.


It seems to have slipped from the API docs, but the upper limit on `per_page` is 200.


Right, I forgot I was looking at server code. Client-side transform might be a little more involved, since we're going to have to deal with docs of different collections mixed together. If you have something in mind we can try it. This should also work for now.

Btw, is it necessary for `opts.collection._hasCollectionHelpers` to be true? What if there are custom transforms set up (independently of collection helpers)?


@gzm0 wanna review this commit before I push it to scalajs/master?


I'm testing the following code in Windows:

```r
  if(.Platform$OS.type == "windows"){
    dir.create(file.path(.libPaths()[1], "compiler"))
    file.copy(to = .libPaths()[1], from = compiler.path, recursive = TRUE)
  } else {
    install.packages(repos = NULL, pkgs = compiler.path, type = "source")
  }
```

I'll send an update after the weekend.


saw approval from jdm
at https://github.com/saneyuki/servo/commit/d8fed6dbd463fa432d2dde1e884dba55609e85bc


merging saneyuki/servo/event = d8fed6db into auto


saneyuki/servo/event = d8fed6db merged ok, testing candidate = 21fd1120


Thanks!


Makes sense.


This can now just be `liftM (mappend m)`


ah, of course


@2langnic here we go! now it results more clear?


Hmm, with this change now, spec that checks for deep contract violation passes due to different reason. Now basically there is no such a thing:

|argument|result|
|:-:|:-:|
|Success(string_with_hello)|first pattern successful|
|Success(string_without_hello)|deep contract violation - pattern matching fails immediately|
|Failure|second pattern successful|
|bad_input|pattern matching failure|

If with this spec, you will go and break deep contract violation detection logic, it becomes:

|argument|result|
|:-:|:-:|
|Success(string_with_hello)|first pattern successful|
|Success(string_without_hello)|deep contract violation - first pattern fails and jumps to next one - and second pattern fails, which results in pattern matching failure|
|Failure|second pattern successful|
|bad_input|pattern matching failure|

In this case 2nd line of the table says that it will still fail, whereas in the real world example with broken deep contract violation detection logic, it should have been succeeded.

Now, we need either transform last pattern match to something that will match `Success(string_without_hello)` and `Failure`, or use different fixture for failed pattern matching (with cause of failure = no patterns matched).


:+1: Have you thought about making a pull request to mitchellh/packer?


Cool :) :) :)


This reads much better now :+1:


And you didn't write Cohen's' haha ;)


- [ ] Check the things
- [ ] Check more of the things
- [x] Done


I liked this snippet, I wrote a little variation ready fine tuned to meet my js structuring:

```javascript
$.fn.selectedText = function(){

            this.on('mouseup', function(){
                var t = '';
                if( window.getSelection ){
                    t = window.getSelection();
                }else if( document.getSelection ){
                    t = document.getSelection();
                }else if( document.selection ){
                    t = document.selection.createRange().text;
                }

                if ( t != '')
                    alert (t);
            });

        };

        $( function() {
           $( '.contentt' ).selectedText();
        });
```


This commit actually breaks detection of zlib for me, in a weird way (it says missing and found at the same time ?!?):
```
-- Found OpenSSL: /usr/local/Cellar/openssl/1.0.1i/lib/libssl.a;/usr/local/Cellar/openssl/1.0.1i/lib/libcrypto.a (found version "1.0.1i")
CMake Error at /usr/local/Cellar/cmake/3.1.0/share/cmake/Modules/FindPackageHandleStandardArgs.cmake:138 (message):
  Could NOT find ZLIB (missing: ZLIB_LIBRARY) (found version "1.2.5")
Call Stack (most recent call first):
  /usr/local/Cellar/cmake/3.1.0/share/cmake/Modules/FindPackageHandleStandardArgs.cmake:374 (_FPHSA_FAILURE_MESSAGE)
  /usr/local/Cellar/cmake/3.1.0/share/cmake/Modules/FindZLIB.cmake:110 (FIND_PACKAGE_HANDLE_STANDARD_ARGS)
  libraries/fc/CMakeLists.txt:94 (find_package)
```

Moving it back where it was fixes it, and the ``if ( ZLIB_FOUND )`` has no influence on compilation, though.

Is there a way to revert this commit?


there is an ``s`` missing here, right? Also, The example seems pretty contrived ^^


Should I go back and amend that now or follow the golden rule of not messing with public commits?


IDFXT (Unknown) - x86_64-MacOS-10.9-clang-Debug: Tests Failed

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


Should this be `__PIC__`?


This should be fixed now.


Yeah, already fixed in 195b24aec30ab2be792e857e4e71c5428eff168e.


develop (macumber) - Win64-Windows-7-VisualStudio-12: OK (2176 of 2188 tests passed)

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


<3 library mini-posters.
Also really appreciate the serie-home docs :+1:


Yeh. I actually copied your name and email from the info included by git in your commit. :)


I don't understand why are needed these timeout_add lines. Could you explain?


Cool


And there we go ...


Case-sensitivity is handled by the parser (and is configurable, see SQLToIdentifierCase in the grammar).


The arrow looks way better, but it has alpha which isn't really a thing anymore with Material.

I found a back arrow in the material icon pack and rotated it, how do these look? https://cloudup.com/c1YJDZpK8bu


sef.sounds_enabled should be private


I would appreciate it if you didn’t use amend, since it was a few commits ago the problem was introduced.

I usually don’t use amend if the problem was created a few commits ago, as amend will only amend to the past commit.
> On Jan 9, 2015, at 2:03 PM, Drew Mills <notifications@github.com> wrote:
>
> Should I go back and amend that now or follow the golden rule of not messing with public commits?
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/FranticRain/360Controller/commit/28c74283448e8a86ca1b93a8ca0efe0cdbcb6f63#commitcomment-9225254>.
>


rax retry tests


Nightly: https://scala-webapps.epfl.ch/jenkins/view/Scala.js/job/scalajs-nightly/273/
Weekly: https://scala-webapps.epfl.ch/jenkins/view/Scala.js/job/scalajs-weekly/16/


Is looking very good.
Testing this, I found a few issues:
* At activity startup in the log I see:
```
Traceback (most recent call last):
  File "/home/gonzalo/sugar-build/activities/Paint.activity/Area.py", line 492, in __event_cb
    self.tool_start(x, y, button1_pressed)
  File "/home/gonzalo/sugar-build/activities/Paint.activity/Area.py", line 559, in tool_start
    if self.sounds_enabled:
AttributeError: 'Area' object has no attribute 'sounds_enabled'
Traceback (most recent call last):
  File "/home/gonzalo/sugar-build/activities/Paint.activity/Area.py", line 830, in tool_end
    if self.sounds_enabled:
AttributeError: 'Area' object has no attribute 'sounds_enabled'
```
* The button used to enable/disable the sound does not display any change to show if is enabled or not,
we can change the icon or toggle the the button and change the palette text. (Could be I don't see the change because I am on F21?)
* The brush sound have a end, if i draw a very long line without stop, the sound start and stop many times. Is possible cut it to make it more constant?


Don't use `note--warning` when it's not a warning. Use `note--info`. Or `note--success`.


It seems that @Tobschinski is developing Divination powers.


Don't leave dead code.


No `s` to `information`. It's invariable.


It's well-known that [npm comes with node.js](http://stackoverflow.com/questions/20128584/npm-comes-with-node-now-what-does-this-mean). I was testing on Ubuntu 14.04 LTS; maybe 12 picks up an older node package? Anyway, please accept my apologies.


@emfrias


d'no, just as bad tbh


You are right, thank you, some SQL client might not like it. Since this update file is now archived in the latest release, there's no longer a need to fix it.


Why did you remove the date patch? (A comment in the commit message would have been nice.)


@shadowzcw Can you please clarify what you mean?


Ammended to use `note--info`.


Not needed anymore.


darwin toolchain already seems to set -O3


@zeckalpha Fixed! Thanks for the catch :)


Np, merged.


Lol do you have any suggestion?


I'd normally just type out the full words.


In the case of booleans, I normally prefer starting methods with "is" or "has". So, "isTwoFactor" might do?


@mtwebster  I update cinnamon now, and i not note much change... I can not write text on the example. The example is a pretty standard sample, so need to work. Maybe i need to wait more time?


Made the corrections but I left `getattr(self, "type_")`.  It does work as intended.


84694650_EnableCodeCoverage (lefticus) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Testing the HTML runners...


Another option to fix it, is to add this to deep contract violation spec:

```ruby
  expect(PatternMatchingExample::Failure).not_to receive(:is_a?)
```

/cc @egonSchiele


Did you run the tests?


Oops, I changed this while I was debugging and forgot to change it back.


That's responsible for the added space after the header links, because D code in ddox is highlighted using  [prettyprint](https://code.google.com/p/google-code-prettify/).


I ended up fiddling with this test helper a little to ensure that for some of the tests (where the vendor pattern starts with `^`) we make sure we're passing in a path relative to the `samples_path`.


all tests pass:
success: http://build.servo.org/builders/linux1/builds/502
success: http://build.servo.org/builders/linux2/builds/500
success: http://build.servo.org/builders/android/builds/538
success: http://build.servo.org/builders/mac1/builds/719
success: http://build.servo.org/builders/mac2/builds/719


@dcparker88 please do, it would be more obvious to some.


I didn't try with that, I tried with your desklet, and it's working.  I'll try your example when I get a chance.


Yeah me too, it's just that here we are using the Eloquent mutators. I think `hasTwoFactor` it's better?


Initialize it with -1.


Yeh. Hmm.


fast-forwarding master to auto = 21fd1120


@myleshorton This brings in a flashlight release that includes the fix for #2121.


The tests fail as the ant-task doesn't support XInclude. I was thinking of adding an XInclude task to fetch the data but that your fetch the data once and not on every request (or once a day).

The idea was to enable the usergroup to host their data on their site and to hook up their own tools to create the data on-the-fly. At phpugffm we have the event-plugin for wordpress that we actualise. So why not use that data to dynamicaly feed the XML-file?

The main question is how the page is rendered in the background


Awesome @oxtoacart thanks!


@godiard I'm not sure why.. But if I dont disconnect/reconnect the event it will not work.
and the stop sound.. With the stop_sound, sometimes its still playing sound..


saw approval from jdm
at https://github.com/jimrhoskins/servo/commit/e8036accd6b4c2d5c3e94ca7e304492cc2ec23af


merging jimrhoskins/servo/sandbox-tokenlist = e8036acc into auto


jimrhoskins/servo/sandbox-tokenlist = e8036acc merged ok, testing candidate = ebb702ab


Wouldn't a `Let` here be easier to read and also slightly more efficient? Better yet, merge the two checks into one expression as in `(x != null && x.foo != null) ? x.foo.bar : undefined`.

I think it's worth some effort to keep this stuff readable.


I have just checked out current master and reverted only my changes to `lib/contracts.rb` and `lib/decorators.rb` and there is no failure in rspec run, which means it is not covered now, as I expected. This is a commit https://github.com/waterlink/contracts.ruby/commit/e4a9ad795dc9842443f7147969d5f7c9138040bf that shows it


The button toggle.. Its because we are at F21..

Can you explain me what is a long line without stop?


Forgot to mention that this fixes mlaa/cbox-mla#140.


Same nitpick, the example needs to have status.


Can l use this to spawn ores in a lavacooled area?


I think we can use this link for it https://docs.acquia.com/network/install


ı didnt test it with instagram but ı tested with stock cm12 camera and camera360 ultimate and removing these hacks are solving the issue which ı described in pull request
here you can also try it: http://www.mediafire.com/download/msfhz3hxy2lhltt/camtest.zip


Thanks so much, guys :)


Ok then


Seems to work:
![screenshot from 2015-01-09 16 31 40](https://cloud.githubusercontent.com/assets/262776/5687668/3702605c-981d-11e4-9444-44ea1e06e364.png)


@edanaher this reads so much like a evaluator! Which it is. I guess I am saying it looks good.


Do we need to use `=` in an expression context? Because this seems pretty unnatural otherwise, although it does seem to work.

Also, arguably the else case should yield either the value of `baz`, or else `undefined`.


And I don't really understand what you said :-) Really: I don't know what multigraph is and I don't think this patch is *enhancing* the plugin? It's just a tiny bugfix?

Note that I'm just a stupid munin user, fixing a bug hit (but there may be other ways to fix it too, I guess). If you think this patch is weird, would you mind looking at, and comparing, the "autoconf" blocks in the multips and multips_memory plugins. Then I hope this patch makes more sense.


ı think yeah but not %100 sure ı will dig art sources to make sure and will report it later thanx


Also ist es nun 1.7 + 1.8 fähig durch die ReflectionUtils?


@ignaciouy, draw a line without stop, for a long time


I see why you did this, but I'm thinking a giant switch case is better than this regex.


because now FFMPEG and google's soft. decoders are handling mp3 playback without issue(ı tested it)
and also ı looked to logs and seems sometimes decoders are conflicting (Dozen of errors as far as ı can see from logs)


HTML runner tests pass.


Starting untagged source map tests.


It seems VS has added newlines in the constructor without me noticing.


If we did that, the only file in the tag should be a README that says "HAHA"


@dg Do you still think that this is a good idea? Because I have not changed my mind and still think that this should be reverted.


`hasTwoFactor`?


ggg


hmm yeah but ı tried it many times(locally-editing build.prop) and ı dont know but ı felt more responsiveness when ı removed them.It can be placebo but ı swear ı felt :)
your choice arco... :)


thanks!


@Gillou68310 Is this worth a pull request?  What defect does it address?


Duplicate of line [343](https://github.com/NewsappAJC/gga/blob/develop/src/style/app.css#L343)


Thanks. No, it wasn't. Bundler gets picked up from the installed version. I'll add Bundler to Gemfile.installed, which should fix this.


Why? Shouldn't taxons always have a taxonomy.

If there is a reason to support taxonomy being `nil`, this should be `.try!` instead.


The problem with lets is that they are actually expensive (currently) because they expand into anonymous functions … and then also that anonymous functions don’t work in `$where` clauses. I had originally used a Let, but got talked out of it 😉


Qualé desse `thumb` aqui?


без toJSON все работает ....


Ah, I tracked it down.  Still, do you have any opinion on this? :)
https://github.com/ptitSeb/mupen64plus-pandora/commit/1ad0b75f4090a34fbc0ad7bfcbd04e5bff5f1e71


This was causing me a stack level too deep when I tried to use `rspec-mocks`. Turned out that it is not a good idea to define `call` globally, because it gets defined on singleton class of `Object` if my memory does not falters me. Which may actually lead to very weird stuff.


Do we need another case here for other RPM build targets that appends "rhel-"


doh! thanks ;-)


hi Hoffi,

just to note:

did copy the code for use in JEM3 but noticed an error.  Have to replicate to be sure but the error was poiting to line 172.  could not use in write context or something like that..
--> got the message upon installing a package (did create a new package locally so didn't test the 2.1.x)


And here is a next commit, that shows that adding this expectation reveals a failure: https://github.com/waterlink/contracts.ruby/commit/77077945c60e5338257e3870c44972f9eb9c2319


mustMakeDir


Oops, no, I thought I’d removed that to another branch but obviously wasn’t paying attention.

On 9 Jan 2015, at 20:37, Jan de Muijnck-Hughes <notifications@github.com> wrote:

> Erm... @edwinb was the Effect.Protocols.idr filesupposed to be part of this commit?
>
> —
> Reply to this email directly or view it on GitHub.
>


yeah this feature is allowing us to use ethernet connection through "USB to ethernet adapter"
ı think our kernel is supporting this feature but ı didnt test it yet(will do it)
and yep ı think we need some more changes to active it like this https://android.googlesource.com/device/lge/hammerhead/+/667f7d6%5E!/#F0
and activating eth0 service through init


@GrahamCampbell Of course they can, it's all down to preference really. I was only suggesting what I felt would round of some of the edges in consistency but of course not everyone feel the same which is totally fine.


develop (macumber) - Win64-Windows-7-VisualStudio-12: OK (2170 of 2189 tests passed)

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


This line doesn't make sense for an equality comparison


Again, the find_item was important here.


:-1:

Why was this merged? This never addressed our concerns.


@jbrooksuk This means we remove the mutator?


This should have OrderDto as the return type.  I believe you can get this by calling return this.Client.Post<OrderDto>(order);


not a problem @alviteri


For screenreaders


Untagged source map tests pass.


Yeh, so we should just leave it as it as then.


Pra essa tag `author` funcionar, tu precisa adicionar sua imagem a pasta `source/images/` e suas informações em `source/_layouts/_article.html.slim` (ex.: https://github.com/startae/startae-blog/blob/1245b994dd541b07dd022bbe0b673b8f60b243bd/source/_layouts/_article.html.slim#L66-L69).