@littleguy77 this is breaking compilation on MSVC, here's the log:
```
1>  main.c
1>d:\notbackedup\gsi\projects\perso\github\mupen64plus-ui-console\src\main.c(629): error C2375: 'SDL_main' : redefinition; different linkage
1>          d:\notbackedup\gsi\projects\perso\github\mupen64plus-win32-deps\sdl-1.2.14\include\sdl_main.h(57) : see declaration of 'SDL_main'
1>
1>Build FAILED.
```


Hi, it seems the code now uses generics and depends on at least Java 5. Would it be possible to reimplement the enum support please?


how about `andThen`, like in futures?


Done


Probable something like this is needed (taken from https://sourceforge.net/p/gnutelephony/bayonne/ci/master/tree/configure.ac):
```
AC_CACHE_CHECK(whether sigwait has 2 arguments,
    ac_cv_libc_sigwait2,
    AC_TRY_COMPILE([
        #define _GNU_SOURCE
        #define _POSIX_PTHREAD_SEMANTICS
        #include <stdio.h>
        #include <signal.h>],
        [sigset_t sigs; int signo; sigwait(&sigs, &signo);],
        ac_cv_libc_sigwait2=yes,
        ac_cv_libc_sigwait2=no
    )
)
if test "$ac_cv_libc_sigwait2" = "yes" ; then
    AC_DEFINE(HAVE_SIGWAIT2, [1], [2 argument form])
    AC_DEFINE(HAVE_SIGWAIT, [1], [have sigwait])
else
    AC_CACHE_CHECK(whether has any sigwait support,
        ac_cv_libc_sigwait,
        AC_TRY_COMPILE([
            #define _POSIX_PTHREAD_SEMANTICS
            #define _GNU_SOURCE
            #include <stdio.h>
            #include <signal.h>],
            [sigset_t sigs; int signo = sigwait(&sigs);],
            ac_cv_libc_sigwait2=yes,
            ac_cv_libc_sigwait2=no
        )
    )

    if test "$ac_cv_libc_sigwait" = "yes" ; then
        AC_DEFINE(HAVE_SIGWAIT, [1], [have sigwait])
    else
        AC_CHECK_FUNCS(sigwait)
    fi
fi
```


css » CSS
webkit » WebKit
Scss » SCSS

If this goes in the readme, then I think it should be explained _why_ someone should add this. Personally, I’m not convinced it needs to be noted at all.


Можно комментарий к переменным? Не понятно, что за  `key` и где используется.


No I would just add a reference to the plugin inside plugin_thread. That way we could just call its add message. I think that would be easy to implement and if we aren't planning on changing the api we are going to have to call the addMessage eventually


`s"$key $process".trim` ?


84694650_EnableCodeCoverage (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (596 of 596 tests passed)

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


Паттерн-матчинг как в основном коде?


@barbalberto Commits of the type ``Merge branch 'master' into <xxx>`` are usually useless, it is way better to **rebase** your branch, especially when the branch is one single commit.
The only reason to have this kind of commits in a feature branch is when you have a shared branch that is developed for a long period, and a rebase would be more complicated than a merge, otherwise, please **REBASE**!!!


Extra line


if host start with "http://" i think we need to replace http with https


I'm actually wondering what the reason behind this change is as the basicOperation handler in the view should already normalize the path.

Do you remember the reason?


bien vu


Oui !


oops :)


`fs.access()` provides the same functionality as `fs.exists()` and more:

- `exists()` does not follow the error first callback style, which confuses people and causes issues with `promisify` and similar tools.
- `exists()` can only check for file existence, while `access()` can also check for specific permissions (read, write, execute).
- `exists()` performs a `stat()` inside of a `try...catch`. This causes any potentially useful error information to be lost. The full `stat()` is also slower.


That is my take as well. Although the commit into the compat branch doesn't close anything, it seems good to keep the same comments as the cherry-picked commit for consistency.


I rely on @akashivskyy and @paweldudek :neckbeard:


@hashem78 just an heads up for being a jerk. Let Us Finish! Before Pointing At Us! "Don't Be Jealous Of Us, Coz You Won't Be Better Than Us"
Meanwhile Credits have been given to the twrp team!
https://github.com/Epic-OS/bootable_recovery/commit/a6068a6338a88b96637924d85a83300461825428


Indeed, types are :
```
t = np.array([int])
```
or
```
t = np.array([int]).T
```
For the first case, the f_contiguous flag is true too as it contains only one elements. So we will always include this extra header even if it is not required.


This removal caused Columns Titles to disappear.


Какова цель?
```
foreach(x => inputManager(x.key) = tmpInputManager(x.process))
```
`inputManager` вернет ссылку на поток, и потом ей зачем-то присваивается другая ссылка? Не понимаю.


hi @cdujeu
and I have to admit that I over reacted: This change did not solve the problem. So sorry for being so harsh!


2: M102 unrecognized bullet/signature
3: M102 unrecognized bullet/signature
4: M102 unrecognized bullet/signature


First impression: I would say maintaining a larger plant is slightly cheaper than maintaining a small one, so fixed O&M is lower per kW for buildings than for residences. Will check research on etdataset.


WIP xD


Much better :+1:


Prefer hex, `#000`


:abc:


> The original refs are fine. Especially since we're using a single issue to track multiple versions.

We can track issues in multiple version, you have the "Ref" which references to the commit in master branch, which have all the needed links, it a bit bugs me that we would say we close something or fix something without github actually doing any actions, but we're mention this "Closes" and "Fixes" in this format specially for that reason, so i would strongly suggest do this how github documented it.

> Definitely do not change cherry pick comments to Refs, it would make it impossible to track what's been cherry-picked.

Not sure the reason why you would need to track "what's been cherry-picked", since cherry-pick command is definitely not the only way you could port changes from one branch to another, whereas "Ref" is universal way to reference relevant commits.


It's called submodule. I would prefer not, because the maven Pom in
BlockServer automatically downloads IO lib and level db.

On Tuesday, January 13, 2015, PEMapModder <notifications@github.com> wrote:

> Suppose we could submodule it/mirror it/whatever?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/BlockServerProject/MavenRepository/commit/2484b5a45dce7957a58f204b6b0a9c9ec3276dfa#commitcomment-9254593>
> .
>

--


Extra line


I think this could be optimized a bit.

we can replace the "select" by a "INSERT IGNORE " which would not create new values if already existing for same storeid/optionid tuple.

i agree with the if $avalues[i]!=$svalues[i] test.

what do you think about this ?


@derekeder
My guess is that you commented out this filter in order to display the point data.

However, you'll notice that on my version, the point data does NOT display by default. We require users to turn on each point layer. Why? Because the organization's primary goal is to emphasize the opportunity levels of the census tract polygons, and as you can see, the 6,000+ points clutter up the map.

Question for you: Does my solution to turning off the point checkboxes make sense?
I could not find a way to do this on the index.html page (because I don't really know what I'm doing)
Instead, I commented out the penultimate line in the reset filter portion of maps_lib.js:
```
//reset filters
$("#search_address").val(MapsLib.convertToPlainString($.address.parameter('address')));
var loadRadius = MapsLib.convertToPlainString($.address.parameter('radius'));
if (loadRadius != "") $("#search_radius").val(loadRadius);
else $("#search_radius").val(MapsLib.searchRadius);
// $(":checkbox").prop("checked", "checked"); //if active, all checkboxes on by default
$("#result_box").hide();
```
Does that make sense, or do you recommend a different approach to turning off point checkboxes by default?


Muy bien!


the owndata flags should not be check. You don't care if you own the data or not. You just want to know if it is transposed or not.
I think you did it because t contains only one elements so it is f_contiguous AND c_contiguous. But hopefully, you use ```type.T``` for the transposed type in the spec expander. Thanks to this, you create a view on a numpy array and because it is a view, the owndata flags is false. But it is only a side effect :-(


I see. So it is no problem to use the editor build right into Github for small fixes like this, right?


Also I know that you don't know how to use maven in eclipse. Or are you
using Intelij IDEA now? I'll look up maven support for that.

On Tuesday, January 13, 2015, Alejandro Zeise <atzeise@gmail.com> wrote:

> It's called submodule. I would prefer not, because the maven Pom in
> BlockServer automatically downloads IO lib and level db.
>
> On Tuesday, January 13, 2015, PEMapModder <notifications@github.com
> <javascript:_e(%7B%7D,'cvml','notifications@github.com');>> wrote:
>
>> Suppose we could submodule it/mirror it/whatever?
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/BlockServerProject/MavenRepository/commit/2484b5a45dce7957a58f204b6b0a9c9ec3276dfa#commitcomment-9254593>
>> .
>>
>
>
> --
>
>

--


not enough methods for cache


Sick!


Oh great, looks like this somehow broke the Travis build...


Per the style guide, this should be written as `padding: 0 ($breadcrumb-height / 4) 0 ($breadcrumb-height / 2);`


> Not sure the reason why you would need to track "what's been cherry-picked", since cherry-pick command is definitely not the only way you could port changes from one branch to another, whereas "Ref" is universal way to reference relevant commits.

Because cherry-pick is a specific action and a reference is generic. If there's a cherry-pick comment, you know for a fact that the commit was applied, if there's a ref, you just know that it's in someway related. I can assure you that there are cases where this info is very useful, but I don't want to get into a huge debate about this here.


Default gear weight rather be 250.


Per the style guide, this should be written as `padding: 0 ($breadcrumb-height / 2) 0 ($breadcrumb-height / 1.5);`


r+ rollup


could you please add the `@return` annotation to ensure correct autocomplete


Perché inizialmente non mi era del tutto chiaro dove mettere i file Markdown per creare l'ebook, le pagine HTML (che sono la vecchia versione dell'ebook, non sapevo se le avremmo modificate a mano o no) e le immagini.

La risposta finale è che il problema (quasi) non esiste. Ci sono solo i file Mardown, i file HTML sono generati dal plugin [`jekyll-ebook`](https://github.com/lmullen/jekyll-ebook) e quelli attualmente nel repo si potrebbero buttare.
Rimane comunque il problema delle immagini che devono stare dentro `gh-pages` ma devono essere disponibili quando si genera l'ebook.

Detto in altre parole, non si possono fare due branch separati, così:
```
[branch: master]
├── e-book
│   ├── e-book-stylesheet.css
│   ├── manifest.yml
│   ├── metadata.xml
└── README.md

[branch: gh-pages]
├── gh-pages
│   ├── _config.yml
│   ├── Images
│   ├── index.html
[... tutto quello che serve per le gh-pages ...]
└── README.md
```

A meno di non avere una copia della cartella `Images` nei due branch (che, converrai, non è una soluzione ottimale).

Rimango convinto che sia meglio, _rebus sic stantibus_ tenere le gh-pages in una sottocartella e fare `git subtree push --prefix gh-pages origin gh-pages`.

Fai conto poi che queste cose le abbiamo capite facendole, quindi il fatto che il repo sia un po' incasinato è il minimo.

Inoltre, volendo mettere i puntini sulle "i" è più facile dire a un neofita. "vai su questo repo i file sono dentro la cartella gh-pages" invece che digli che deve selezionare un altro branch.


Double quotes


`black` should be `#000`


`white` should be `#fff`


Suggested alternative here: https://github.com/square/okhttp/pull/1320


85606174-ImproveIGValidation (kbenne) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2167 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-98.99497487437185%25-yellow.svg)


Yes I guess, but I'm not really sure how that works. Doesn't that only work for UNIQUE fields?


This should be written as `padding: ($base-spacing / 4) ($gutter / 2);`


Lowercase hex colors


Make zip optional please, with a `pack` argument or something, because it is already tedious building it and restarting Firefox after most of the changes, but waiting for zipping too for no good reason...


add methods to pour/out water


I like the simplification of the sort and map.  Reads better.


hey you're right, i was convinced there was a unique index for option_id,store_id for eav_attribute_option_value , but it seems there is not ! so we cannot avoid to compare with existing store values :(


Nice!


sounds good


class TCon


The comment here affected http://bugs.jquery.com/ticket/1987 because the commit hook for Trac is still active. (I deleted the comments there.) It's possible that we may still want to reference Trac tickets for new commits but more likely that it will just annoy us. How about if we use URLs to reference the old trac tickets in commit messages and I remove the Trac commit hook? Or if URLs are too ugly we could use trac-1987, it's pretty easy to manually construct the url from that.


Interactive shells have job control.  Here's a recipe that spawns an interactive shell to create a new process group.

https://gist.github.com/groner/51acca4fb08d45582035


hello
yay! hahaha


All by myself. I've just finished Tunisia, Morocco, Egypt, Algeria and South Korea. Only 2500 teams in a week... somebody said a small number.


whops :)


I will look into this. I changed the labels since the type checker complains about a double declaration of field x as str.


I used `Mouse`, because that's what was already in place for the `NessyBackend` and I wanted to conform to the `Backend` role that lives next door.


Thanks, it's fixed.


ya erase him and his friend ....they both r burden for community ....


TeamCity CI Experimen :: PHP Symfony2 project :: Symfony Project Test [Build 18](http://192.168.44.2:8111/viewLog.html?buildId=64&buildTypeId=CiExperimen_PhpSymfony2project_SymfonyProjectBuild) is now running


Homework completed


login through social sites has been removed.


This sounds great, but shouldn't it be the following instead?

```bash
export NODE_PATH=$PWD/lib:$NODE_PATH
```


s/ceph_objectstore_tool/ceph-objectstore-tool/


@scytacki Thanks,
Please refer this commit https://github.com/concord-consortium/lara/commit/bb7e3501ff4203e478c390ab0d0599e37bd3eb61 for the fix.


TeamCity CI Experimen :: PHP Symfony2 project :: Symfony Project Test [Build 18](http://192.168.44.2:8111/viewLog.html?buildId=64&buildTypeId=CiExperimen_PhpSymfony2project_SymfonyProjectBuild) outcome was **SUCCESS**
Summary: Tests passed: 3 Build time: 00:00:50


Why are you removing this?


Done


stoi return int not int64


I ran it as a local script. I should probably convert it into a function runnable on the browser. What is the proper way of adding the 'penn-sdk' library to this app? I am new to require.js and don't quite understand how it's working.


85606174-ImproveIGValidation (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-warnings-yellow.svg)


As I don't have a mac to test this on, I will trust you that you have tested this and it works. It looks right.


1: M190 line is too long (51 > 50)


does it actually need payment? or is this the same direct connect needs `nil` thing


This introduces some compiler noise:

```
[warn] /home/roberto/guardian/membership-frontend/frontend/app/views/support/Social.scala:9: method encode in object URLEncoder is deprecated: see corresponding Javadoc for more information.
[warn]   def encode(str: String) = URLEncoder.encode(str)
[warn]                                        ^
```

You just need to use the non-deprecated version of this method, passing in the encoding as "UTF-8":

http://docs.oracle.com/javase/7/docs/api/java/net/URLEncoder.html#encode(java.lang.String,%20java.lang.String)


IMO powinna byc tylko jedna tablica LUT. Obaj gracze korzystali by z tej samej, ale inaczej ja interpretowali. Np. gracz X traktowalby -1 jako X, a gracz O jako O. Co o tym sadzisz?


hm, sorry, I've grepped for this ID, but with no success.


This is Alex


Who cares, you shouldn't be throwing exceptions from foreign namespaces anyway.


1: M100 needs more reviewers
1: M101 signature is missing


Se l'idea è avere un unico posto con dei file .md che vanno modificati e quindi un modo semplice semplice per generare un sito statico e l'ebook, suggerisco di usare https://github.com/GitbookIO/gitbook che mi pare più aggiornato del plugin che state provando a usare.

> meno di non avere una copia della cartella Images nei due branch (che, converrai, non è una soluzione ottimale).

A dire il vero che due branch condividano cartelle e file è abbastanza normale. Nell'istante in cui esistono 2 branch distinti per gestire "sviluppo" e "pubblicazione", credo sia abbastanza corretto pensare che @Aubreymcfato periodicamente va a fare un merge del branch di "sviluppo" in quello di "pubblicazione", aggiornando così la cartella immagini.

Ad ogni modo pensavo sul serio aveste sbagliato il merge, capita. Ora che ho capito meglio come pensate la cosa forko e provo a usare gitbook :shipit:


all tests pass:
success: http://buildbot.rust-lang.org/builders/auto-mac-32-opt/builds/3128
success: http://buildbot.rust-lang.org/builders/auto-mac-64-opt/builds/3121
success: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-t/builds/3122
success: http://buildbot.rust-lang.org/builders/auto-linux-32-opt/builds/3113
success: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-t/builds/3113
success: http://buildbot.rust-lang.org/builders/auto-linux-64-opt/builds/3116
success: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-t/builds/3109
success: http://buildbot.rust-lang.org/builders/auto-linux-64-x-android-t/builds/3104
success: http://buildbot.rust-lang.org/builders/auto-win-32-opt/builds/2781
success: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-t/builds/2779
success: http://buildbot.rust-lang.org/builders/auto-win-64-opt/builds/1617
success: http://buildbot.rust-lang.org/builders/auto-win-64-nopt-t/builds/1610


fast-forwarding master to auto = 6ba9acd8


1: M111 unrecognized component name: collections


This value should be written as `($base-spacing / 2) $gutter`


Normally you clone it and send it as one merge request ...


:fearful:


`white` » `#fff`


`white` » `#fff`
`black` » `#000`


`black` » `#000`


The corect spelling is weight.


This value should be written as `($base-spacing / 3) ($base-spacing / 2)`


We have a helper at OSD::init_op_flags() that iterates over ops and sets flags on the OpRequest.  Let's add a flag to that structure (must_promote or similar) so that we avoid an additional pass over the ops vector?


This should be written as `padding: $base-spacing / 2;`


Prefix with `0`


There was previously no spacing in between post images.  This should only affect post images (uploaded from a post).  Also, if someone someone sets image to be say 400x400, where the images display on top of each other, it also adds that spacing as well.


That's fine to use. No worries :)
On Jan 13, 2015 8:51 AM, "alawatthe" <notifications@github.com> wrote:

> I see. So it is no problem to use the editor build right into Github for
> small fixes like this, right?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jscs-dev/node-jscs/commit/8bd2ffd7bd1fc53a49d017f06322b178124534fa#commitcomment-9260032>
> .
>


Should be double quotes


this looks good!


This is me - I didn't config Git.


@sferik you are perfectly right, my mistake here. As you saw, there is another PR


Ich glaube es ist besser wenn das in bt_get_or_create passiert, d.h. als Default für alle neuen Typen gesetzt wird. Null ist kein guter Default.


It is actually most probably caused by RAML parser change


This should be written as `padding: ($base-spacing * 2) $gutter;`


saw approval from nick29581
at https://github.com/nikomatsakis/rust/commit/487a4a174afb21de5ce5573e16dfe876f6face91


merging nikomatsakis/rust/assoc-types-enum-field-access = 487a4a17 into auto


status: {"merge_sha": "e94a9f033eb79b3b4fb037722366644650e8e16c"}


nikomatsakis/rust/assoc-types-enum-field-access = 487a4a17 merged ok, testing candidate = e94a9f03


Does this need to be more general? If it's going to be used for Protein facilities and so on..


Could also just be:

```diff
diff --git a/src/cluster.c b/src/cluster.c
index 79359ee..f7b280b 100644
--- a/src/cluster.c
+++ b/src/cluster.c
@@ -1093,7 +1093,7 @@ int clusterHandshakeInProgress(char *ip, int port) {
  * EINVAL - IP or port are not valid. */
 int clusterStartHandshake(char *ip, int port) {
     clusterNode *n;
-    char norm_ip[REDIS_IP_STR_LEN];
+    char norm_ip[REDIS_IP_STR_LEN] = {0};
     struct sockaddr_storage sa;

     /* IP sanity check */
```

looks cleaner to use compile-time array/struct initializers :)


These two lines above should have parenthesis around `($grid-item-border-size / 2)`


Ah I've left that for the next upgrade because of lack of time at the moment


This should be written as `padding: ($base-spacing / 2) ($gutter / 2);`


Ok, I just read further and it makes sense as it is, apologies..


Done


There isn't any config file in powershell


@luan This doesn't the database time from moving forward and this test is now flakey. I've hit the following failure multiple times this morning:

```
Failures:

  1) VCAP::CloudController::Dea::Respondent#process_droplet_exited_message when the app crashed the app described in the event exists adds a record in the Events table
     Failure/Error: expect(app_event.timestamp.to_i).to eq(time.to_i)

       expected: 1421158322
            got: 1421158323

       (compared using ==)
     # /Users/sykesm/.rbenv/versions/2.1.4/lib/ruby/gems/2.1.0/gems/rspec-expectations-3.1.2/lib/rspec/expectations/fail_with.rb:30:in `fail_with'
     # /Users/sykesm/.rbenv/versions/2.1.4/lib/ruby/gems/2.1.0/gems/rspec-expectations-3.1.2/lib/rspec/expectations/handler.rb:35:in `handle_failure'
     # /Users/sykesm/.rbenv/versions/2.1.4/lib/ruby/gems/2.1.0/gems/rspec-expectations-3.1.2/lib/rspec/expectations/handler.rb:48:in `handle_matcher'
     # /Users/sykesm/.rbenv/versions/2.1.4/lib/ruby/gems/2.1.0/gems/rspec-expectations-3.1.2/lib/rspec/expectations/expectation_target.rb:54:in `to'
     # ./spec/unit/lib/cloud_controller/dea/respondent_spec.rb:63:in `block (6 levels) in <module:CloudController>'
     # /Users/sykesm/.rbenv/versions/2.1.4/lib/ruby/gems/2.1.0/gems/timecop-0.7.1/lib/timecop/timecop.rb:141:in `travel'
     # /Users/sykesm/.rbenv/versions/2.1.4/lib/ruby/gems/2.1.0/gems/timecop-0.7.1/lib/timecop/timecop.rb:115:in `send_travel'
     # /Users/sykesm/.rbenv/versions/2.1.4/lib/ruby/gems/2.1.0/gems/timecop-0.7.1/lib/timecop/timecop.rb:50:in `freeze'
     # ./spec/unit/lib/cloud_controller/dea/respondent_spec.rb:54:in `block (5 levels) in <module:CloudController>'
```


Hi there,
I did that because I didn't want any gateway to manipulate the submit order button text (to meet german law, especially texts like "proceed to PayPal"). I've just committed a small fix which manipulates the order button text for gateways by a hook. You may now force your payment gateways' order button simply by adding a variable to your gateway class e.g.:

class Gateway extends WC_Gateway_BACS {
   public $force_order_button_text = true;
}

hope this will help you out!


Dont know what is happening!!!


saw approval from patrickt
at https://github.com/helium/rbhelium/commit/11fb4805f854caa22de565015ecb761d69306c09


merging helium/rbhelium/adt-bignum-macs = 11fb4805 into he1000-integration-18-adt-bignum-macs


helium/rbhelium/adt-bignum-macs = 11fb4805 merged ok, testing candidate = 77f4c3da


Ok. Keep trying to understand


Ok, changed


IMO nic nie stoi na przeszkodzie i mogą korzystać z tej samej tabeli. Nie muszą jej interpretować inaczej, gdyż gracz pyta się tabeli o konkretny stan planszy. Co jest jak interpretowane to kwestia konwencji, my przyjęliśmy, że krzyżyk zawsze zaczyna, i reprezentujemy go 1. Tylko technicznie trudniej nauczyć sieć 5400 stanów niż 2700 chyba że jednak będziemy mapować stany ( już sie przymierzałem do tego są funkcje dokonujące przekształceń i mapująca). Możemy równie dobrze wygenerować dane na dwa sposoby i oba je zaprezentować i dokonać analizy porównawczej.


all tests pass:
success: http://buildbot.helium.io:8010/builders/test-rbhelium-build-debian-7-64/builds/12
success: http://buildbot.helium.io:8010/builders/test-rbhelium-build-centos-7-64/builds/12
success: http://buildbot.helium.io:8010/builders/test-rbhelium-build-debian-7-32/builds/12


I can not confirm this statement


My Ruby’s pretty rough, but I think this has broken ENV & command line passwords. I get:

    error: private method `password' called for "<password>":String

It's not great to have your password echoed in plaintext in the command output either.


I thought so, too, but:
* it is code (at least the JS) and the javascript code brings more than just a better looking UI
* it is required (inexor wouldn't be usable without it)

It would be possible to move this to data though.


fast-forwarding master to he1000-integration-18-adt-bignum-macs = 77f4c3da


This (and the previous commit) look good, thanks. Might be worth adding a little comment in the "down" migration saying that the migration is not reversible, but it's not critical to do so.

Could you commit this to master?


If we do not wait power shell is exiting, which means it start showing the prompt again rather than waiting.

Moreover I am not firing the task in parallel as of now because of below reason already mentioned in code:
"Making sure we can generate the file, once the SDK and portal are inter-operable upload certificate and fetch of ACIK can be made parallel to improvve the performace."


Fixed


see https://github.com/guardian/membership-frontend/pull/63


Done


Will find out and bring it in next iteration if it exists.


@bors: retry


@kbenne I think this is a bug, the set should be outside the if block, it looks correct on line 687


No good reasons. I will change that.
(I initially move the code to a separate package but then realized there was no good reasons for it)


@Hackwar Oki, fair comments. Additional feedback :

- I still don't feel category handling should be in the this router. It would make more sense to me that this router is part of the categories component. It could be an abstract router one can extend from. By the chosen implementation you are tightly coupling categorie handling with routing.

- I would consider renaming `JComponentRouterAdvanced` to `JComponentRouterView`. Advanced isn't a very concrete name. This router deals specifically with component routing based on registering views. Would make sense the name reflect this, this also matches with the `JComponentRouterViewconfiguration`Object.


weterterreteerty


Actualy, just run into this. Why it is required at all?
In my case:
```php
dd( $results );
```
```
array(1) {
  ["ctr"]=>
  string(4) "0.10"
}
```
... which gets lost after this conversion, and result is:
```
float "0.10"
```


This was actually a fix for  issue #43
The error looked something like
 in_mem_shas['files'].... KeyError: 'files'
It was because the 'files' key was missing from in_mem_shas in the cases where in_mem_shas was emptied (like if there is no .shastore–like when it is first cloned). You couldn't replicate this?


I used to use `>:` which was a nice prompt indicator. Now it is simply a blank line. Was it your intention to remove the possibility to define a custom prompt?


Looks good. Maybe make "shut down" _italic?_


Perhaps we should say "use access instead"?


I guess this is a race condition.  We need to insert and catch the exception rather than check, then set the outputs.  That might be a significant change that doesn't need to happen in this PR, but we should at least make a `bug` issue for it.


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


Bummer. I've been using the "assignments" endpoint to grant roles to users on definition points as part of search testing: https://github.com/IQSS/dataverse/blob/master/scripts/search/tests/grant-spruce-admin-on-birds

@michbarsinai is there any plan to re-introduce this at some point? Maybe under the "/s" secure area?


@martynjames I changed settings like this, since I couldn't find envs/*.py that contains settings for common. I did, however, find settings for lms/cms and copy/pasted content from there.

Can you please review this and see if it's acceptable?


I think we had ARM64TODO elsewhere.


is this still being used at all?


> I've moved the config headers
Ahh, OK. I see now.


will this affect any other module (assuming it was here for a reason)


Awesome snippet.


Ah ok. I integrated it this way: https://github.com/humhub/humhub/commit/0042c779643addaff69b519c3b3d24a37f7735ba
So please check it out, if it works for you.


Из "/var/folders/.../tmp42..876" в tmpInputManager получаю "-d" в inputManager
apply(path: String, key: String): InputStream можно реализовать


here you could throw an IllegalStateException


jep, recently seen in many cases. Moving it into the else block as a second chance before throwing an exception might be acceptable compromise, performancewise?


destroy this semicolon and add a logger warning if possible


I might have pushed this, but it was originally written by @mastertheknife .  Seems to me that Monitor needs to be defined.  Perhaps the line that defines it has been lost.


Great addition, thank you! There's one thing I noticed which was a bit confusing to me:
Use of event entity to pass start and end date in an object to the microflow for the render view.

IMO it would be better to define extra poperties in the widget configuration(calendar.xml) to avoid confusion:
- Render view Entity
- Start attribute (of Render view Entity)
- End attribute (of Render view Entity)

Creating a separate category in calendar.xml for these properties would also make sense. We would really appreciate if you are able to implement it, described as above!


@Hackwar I understand, it would still be good to have an interface for this so we don't need to work against an implementation. Can we move the API as defined in the advanced router into an interface ?


should we add more of the SparkConf properties here? (we could still override them if provided explicitly in a module implementation via getSparkConfProperties())


Then I guess the `moduleDefinition` is not needed either. Can remove on merge


Hi Stefan,
Can you clarify is IE8 is officially supported by ember-cli-6to5?
Thanks


Can't we keep using the same variables:
```
box-shadow: 0/*{offsetLeftShadow}*/ 0/*{offsetTopShadow}*/ 5px/*{thicknessShadow}*/ #ddd/*{bgColorShadow}*/;
```
Maybe rename thicknessShadow to blurShadow to make clear that it sets the blur radius and not the spread radius.

Or does this give problems with ThemeRoller support for previous versions of UI?


FWIW GCC 5 uses 67 as FIRST_PSEUDO_REGISTER


Why do you need to edit the component? If editing is necessary to reproduce the bug, please add a comment explaining why, as it appears on the surface to be unnecessary (and it is not mentioned in the BDD description).


can you add the `Contingent` to check for correct input parameters and correct dimensionality


LGTM


The onEntityExplode method needs some love, thats a insane amount of if statements.


TESTS FAILED
 Mr.roboto url : http://jenkins.plone.org/roboto/get_info?push=8c3e7955057745fda4793149038cb25c
 plone-5.0-python-2.7 [FAILURE]


So what happens when ~/.tue.old already exists?


Thanks for your PR.

object.pp is named wrong and should be objects.pp


That's fair.


Status was removed from the report schema.


We need to use param callback result, if we have one. Currently empty string is displayed.


This is not at all what I had in mind, will need to think about how it behaves.

What do you think about having a variant of `ModuleUtils.createModuleClassLoader` that does not take `nested` jars into account:

```java
public static ClassLoader createModuleClassLoader(Resource moduleLocation, ClassLoader parent) {
		try {
			File moduleFile = moduleLocation.getFile();
			Archive moduleArchive = moduleFile.isDirectory() ? new ExplodedArchive(moduleFile) : new JarFileArchive
					(moduleFile);
			List<Archive> nestedArchives = moduleArchive.getNestedArchives(new Archive.EntryFilter() {
				@Override
				public boolean matches(Archive.Entry entry) {
					return !entry.isDirectory() && entry.getName().startsWith(LIB);
				}
			});
			URL[] urls = new URL[nestedArchives.size() + 1];
			int i = 0;
			for (Archive nested : nestedArchives) {
				urls[i++] = nested.getUrl();
			}
			urls[i] = moduleArchive.getUrl();
			return new ParentLastURLClassLoader(urls, parent);
		}
		catch (IOException e) {
			throw new RuntimeException(e);
		}
	}
```

This would avoid a lot of useless scanning IMO


I don't think it's a good idea that we're editing directly in these files. Usually if we need to extend features or functionality or fix bugs, we polyfill the methods, which also gives us a better chance to submit a pull-request in case of a bug. Is this something you guys have been discussing? If we continue down this road, we will end up with the same problems Podio had just a while ago with versioning of dependencies. This is also specific to what we're building, so not a good idea to have this here.


Needs to return undefined; used to return list of operations to execute.


> transpose is ridiculously fast in Vectorz BTW because it just swaps a SparseRowMatrix into a SparseColumnMatrix
That's true, but there are cases where one needs to operate on columns (e.g., normalizing as mentioned earlier) and then go back to row accessing. At some point I have to build column vectors from row vectors or vice-versa.

I wouldn't be surprised if you're correct about the performance of LinkedSparseMatrix. Another option is a SparseDualRCMatrix where both row and column vectors are maintained to represent a matrix. This would use more space and have additional implementation complexity but would have trivially efficient row and column access.


omg the API throws a 404 when a list doesn’t exist. And the API interprets it as "Well, something is wrong with YOUR url buddy" instead of handeling it properly and returning false or NULL or and empty object or something.


why not? this was actually convenient, when i add new js files, it automatically embeds them when i run w.sh


I vote to undo this change. This is the name displayed in maven (command line and more) and not just some "pretty id" as I thought.


Don't need to monitor the animator; that's what the supervisor is for.
Since the animator will notify the websocket handler what it's PID is, we don't
need to get it from the supervisor when we start the child as well.


Also, this was already posted on their git as a bug and it was fixed. back in 2013. so I went all like "Well, it's fixed then, it won't be that."
#UGH


Removed monitoring


Bummer.  I wonder why it wasn't an issue for me or travis ci.  Maybe an SDL 1.2 vs 2.0 difference?  I'll try to take a look.


I agree with Søren. If we use external dependencies we need to keep them pristine unless we actually intend our changes to be accepted upstream. Otherwise we've just closed off our future upgrade path and wasted a lot of future time


Animator notifies websocket handler of its PID when it starts


Switched to using subject 3 rather than 1 in 479beb96cd31da4b6b562d128a4c0b7e5ea562c2


fixed in c74b41aad7f28d33c61ba60d4e8d1f5c9a4f43a4


Ok, thanks for the info. I am going to try that right now!


Incorrect title.
Correct:
24885 - Roundy's - New Mariano's Opening - Add Amenities Icons to Evergreen Park


85606174-ImproveIGValidation (kbenne) - i386-Windows-7-VisualStudio-12: OK (2171 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.17770671539515%25-yellow.svg)


websocket_terminate seems to be required or else something in cowboy crashes when the the handler stops


This LICENSE file is only for dependencies that we ship with the source (like css and js and fonts and copy/pasted third party source code under for example BSD and MIT license)

Remove the `scopt` line for that reason


Improvements worked out with @dmandrioli:
* New way to style the arrow. Benefits:
  * The width of the popup is no longer a bit larger than the root node (on desktop) (the visual width of the root node was smaller than the real one because of the way the arrow pseudo-element).
  * Vertical centering no longer a bit wrong on IE11/Win7.
  * Better behavior with custom vertical-align specified on the widget, in particular on IE.
* Not using anymore type=search on input, due to inconsistent behavior across browsers (pathological case: Safari/MacOSX), apparently without impact on accessibility.


TeamCity Bij12 Voorgangsrapportage Natuur - LCV :: CDS BIJ12 :: Build [Build 2.3-SNAPSHOT-72-1503a194b1863978e5ec1e4cefac04c534b4ee27](http://buildserver.geodan.nl/teamcity/viewLog.html?buildId=73881&buildTypeId=vrn_CdsBij12_Build) outcome was **SUCCESS**
Summary: Tests passed: 61 Build time: 00:03:31


Thanks! Fixed in https://github.com/aosabook/500lines/pull/97


Discussed in chat, but for canonical reference, it's not necessary because 'before_field' is called before text_money in the _render method. This is simply a fallback saying.. if nothing else was declared, let's throw in a $ sign.


Add here `  - scopt (http://github.com/scopt/scopt) - Copyright (c) scopt contributors


This is really wrong.

Lots of build environments do the build + install in a sandbox, and then merge to the real install dir.

If you do a sandboxed install in, say, /sandbox, you will end up with a symlink like /sandbox/usr/bin/phar pointing to /sandbox/usr/bin/phar.phar, and when merged to /, you will have /usr/bin/phar still pointing to /sandbox/usr/bin/phar.phar.

An easy solution, since the symlink is in the same directory as the target, is to make a relative one, like

$(LN_S) -f phar.phar $(INSTALL_ROOT)$(bindir)/phar


This can be rewritten to use `typ.walk()` and a for-loop now, if you like. Tends to make things nicer.


There are some double `;;` here


(See def'n of `maybe_walk_ty` for an example)


could simplify this name probably (and make it more general): ContainerFilter


some tests failed:
exception: http://buildbot.rust-lang.org/builders/auto-mac-32-opt/builds/3124
exception: http://buildbot.rust-lang.org/builders/auto-linux-32-opt/builds/3109
exception: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-t/builds/3109
exception: http://buildbot.rust-lang.org/builders/auto-linux-64-opt/builds/3112
exception: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-t/builds/3105
exception: http://buildbot.rust-lang.org/builders/auto-win-32-opt/builds/2777
exception: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-t/builds/2775
exception: http://buildbot.rust-lang.org/builders/auto-win-64-opt/builds/1613
exception: http://buildbot.rust-lang.org/builders/auto-win-64-nopt-t/builds/1606


I think to avoid confusion, this flash should only be displayed if the tickets weren't free, and a separate flash used otherwise with no reference to payment


method name should be plural: applyFilters


Will that work on Windows?


This commit was reverted via 9548054e21a3c611dde40b2035fb0e0e4ba405e6 and didn't make it to 1.7.1


You're right; I'm kind of disturbed that this actually works.


Ah it was fixed in the next commit :)


> Switched to using subject 3 rather than 1 in 479beb9

Of course I meant it the other way around, i.e. switched to using subject 1 rather than 3!


Alright then!


On Windows users should better type the path to their Python 3 executable, e.g. `C:\python34\python.exe`


:sob: fixed in 0a3e35a0fce8f8380d4a990592efa965dd30e8d0


> I remove the Trac commit hook?

I think it would be best

> Or if URLs are too ugly we could use trac-1987, it's pretty easy to manually construct the url from that.

Either way is sound good to me, although "use trac-1987" sounds a little bit better


@gogregor can you please review?


what happens when you call  `fs.accessSync()` on something that doesn't exist?


I'm not 100% sure but I think that this looks_like_crypt function: http://search.cpan.org/~sharyanto/Crypt-Password-Util-0.09/lib/Crypt/Password/Util.pm#looks_like_crypt($str)_=>_BOOL  might be of use to you here.

I'm swamped at the moment--I know an implementation, test, and pull request are the right thing to do here, but I can't swing it right now.  Just wanted to let you know in case you end up getting back to this.


Maybe it is a good idea to add the values of other invariants (chi, pg, q, maybe more) to the class as well. We have to do this computation anyway, when a surface is selected.
The current situation is a bit more memory-efficient, but I don't think we have to worry about that yet.
It would also make it a lot easier to put other coordinate systems in the table.


Awesome. Seems to work. Thanks. Can you put it in a separate PR?


85606174-ImproveIGValidation (kbenne) - Win64-Windows-7-VisualStudio-12: OK (2174 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.31475559616263%25-yellow.svg)


I actually think this isn't needed anymore, since `smallest_prime_of_bitlength` is gone.


Whoops! I'm sorry! You're right, this change did break reading the password from the ENV. I'll prepare another pull request to fix it.


Ah, you mean there would be a set of possible inputs ({c_1^2, c_2, h11}, or {h11, h20, h10}, or {h20, h10, chi}) and all others are computed from that. Makes sense, and isn't too hard. I'll try to do it this weekend.


2: M102 unrecognized bullet/signature
3: M102 unrecognized bullet/signature
4: M102 unrecognized bullet/signature


In fact, I believe this is the ONLY check that is necessary for trait objects. You can remove the iteration over the substitutions above. To see why, see [this comment](https://github.com/rust-lang/rust/blob/master/src/librustc_typeck/check/regionmanip.rs#L336).


@vk26
Здесь необхоимы пробелы, дабы соблюдался стиль кода.


If I understand this method We do an update if the model doesn't exists and an insert if it exists.
So a code like that will update all the table :
```php
$model = SomeModel::firstOrCreate([
  'column' => 'unmatching value'
]);
$model->update($attributes);
```
Is that an expected behaviour ?


The first error you reported per the type in kinetic_pdu.c:50 is due to the lack of a cast. This was fixed shortly after the release was tagged. I pushed a fix for it, and then relocated the 0.10.0 tag to the fixed version. If you pull again, and update to the 0.10.0 tag again, you should get the change. If not, you should be able to update to 'master' branch. In either case, the git hash should end up being: 5d7dee6edee054c03d35acc7b5c8d7209644d57e

The second issue is likely due to you not updating the repo's submodules. You can do one of the following:
1) git submodule update --init
2) make clean # performs the above command to ensure all is in sync.


@jdevalk You can remove this foreach and rtrim loop and simply do:
```php
$tags_out = implode( ',', $tags );
```


I've just done one - let me know if that code is what you were intended.


In general, rather than continuing to recurse here, I think it'd be better to use the `type_outlives` method that `regionck` provides. (This also handles the trait object and region pointer cases below.) Unfortunately, I think that we still need this kind of recursive unfolding walk to uncover the things that have destructors -- I was originally envisioning we would use type contents or something, but (a) that does an unfolding walk too and (b) that will only create additional, spurious errors.

Note that using `type_outlives` is *slightly* more restrictive than the code that exists here now, but I think that's by and large a good thing, as it will handle (e.g.) phantom lifetimes correctly.


Походу да.


shouldn't this be `Please provide your API key using the API_KEY env variable.`?


EnumMap maybe? Since key is enum.


Hello, when I find the commit for black lollipop? notification, setting, menu, ecc.. thank you!


You guys should use a CDN for this; it'll be faster, more secure, more up to date, and it'll lower bandwidth for our server.


TeamCity Bij12 Voorgangsrapportage Natuur - LCV :: CDS BIJ12 :: Build [Build 2.3-SNAPSHOT-73-c880bf1c30eb35ee43a845dd73620e969db91c9a](http://buildserver.geodan.nl/teamcity/viewLog.html?buildId=73886&buildTypeId=vrn_CdsBij12_Build) outcome was **SUCCESS**
Summary: Tests passed: 61 Build time: 00:02:17


Reviewed.Ready to be merged.


AllegroGraph
http://franz.com/
Weka3
http://www.cs.waikato.ac.nz/ml/weka/downloading.html


Thanks!


It didn't work at my end. FrontEnd (webgui) was ordering as a string (1,2,22,3,3333,4, etc..). If Invoice numbering is numeric then the following would be a better approach:
$this->db->order_by(CAST(ip_invoices.invoice_number AS UNSIGNED) DESC');


:+1:

Looks good to me! Thanks!

@mattt what do you think?


Well, maybe even that. My initial thought wasn't even too change the input, but to calculate redundant information. I.e., just moving computations from updateInvariants (https://github.com/jcommelin/superficie-algebriche/blob/master/js/form.js#L22-L62) to the class. That is in some sense even more conceptual. And if at some point we want to display the surfaces via other coordinates then (c_1^2, c_2) it will be easy to have the computed values of the other invariants ready at hand.


Here is a solution:
in libhwcomposer/Android.mk
Use Os to compile this so, it will be no errors found.

@arter97


Try building ui-console with C linkage rather the C++.


Ticker is an abbreviation of company name used ONLY in stock exchange. Краткое название might not be a correct field name? I might be wrong as studied Russian long time ago :-) Just alarming to make sure you guys will have 100% perfect translation. Na zdarowje!

'Ticker Symbol'                => 'Краткое название',


I wonder if we could do something like take a hash for all of the boolean switches rather than continue adding parameters to this list...


@dtsn I had come up with one much the same, but both the one I came up with and yours here break the test-case `dataurl-base64-doublequotes.css`, but thanks for continuing to help me dig into this one.


Yes, this migrate call doesn't check anything - you was right


I would remove this comment, it relates to the code before this commit.


@javisantana  We said to not touch old models, but it is extremely necessary to have those data.


@sagarrakshe Is this correct change ? Assume its breaks planned thing.


```
`validator_class` instances from
```


Thank you so much @glouppe !


Yeah, these "toExist" checks bite us all the time.


@xavijam don't worry as far they are compatible changes


<center> tag is deprecated; use a CSS class instead


yeah, true


> I can assure you that there are cases where this info is very useful, but I don't want to get into a huge debate about this here.

Don't get me wrong, if this is useful, then i'm all for it. Just wanted for us to decide with all the possible information.

So @dmethvin, @scottgonzalez just let me know how you want to go about this, since i did this differently, obviously, my commits should be coherent with the others.


不错不错，只要是基本功能会用了就 OK 了，其他变量等一些技巧暂时可以先不用。


@dan-f this shouldn't say "content group", should it? What about when it is the experiment group configurations?


Logging from a filter looks rather impossible.


With Ymodem being an optional library, we need to make a note that users must compile this into their firmware by adding `#include "Ymodem/Ymodem.h"`.  On the photon, we should be able to include this in the system image by default (storage space permitting.)


we should wrap this in a try/catch and if getBeansOfType throws an Exception proceed with a null filters list (or perhaps better, always ensure an empty list so we do not need to do a null check above)


@JustGregory: This has nothing to do with Java version. Generics are supported since Java 5. But due to the nature of generics they are not present in the decompiled code we get from the Mojang code.


classname needs to be generic


Merci pour ces traduction (et aussi pour la correction). J'ai fais pas mal d'erreurs et surtout trouver comment tourner la phrase pour que la traduction soit correct en français tout en étant suffisamment petit pour ne pas être "hors écran" n'est pas toujours facile, donc félicitations à toi :p


"initial commit 2" -> waiting for the 3rd one :watch:


Бекграунд у нас індекс пейджа! Це лишнє і не красиво! Обговорювали...


Shouldn't it be `.label`?


maybe this method should be named "listKeys" (or seomthing similar), since A) the class itself is called ConnectionProperties and B) what's actually returned here is a comma-delimited list of the property keys


On my laptop Thinkpad x200(with SSD), running allBalance will take over 4 hours.(338K blocks)


Not a lot of data here… I've got an issue with this line! Plz help!!!1!!1 :sos: :construction_worker: :helicopter:


FloatCol servait à prendre en compte les demi parts !


Shouldn't it be `label`?
And what about the language? Is it working even though the language is not specified?


Done!


Yeah me too, man that's weird right ?


Left during merge.


The test depends on some ed services which do not get generates otherwise. Its interesting though that the python tests actually work while the catkin_make fails. Maybe we can ignore that for now.


Oh bien vu !


Typo in function name: should be Disabled not Diabled.


You're awesome man, i was wondering. This damned bug is annoying as hell :) Well done


OK


This variable is never set false, so once it becomes true, it stays true for ever.


The flow is competition->rounds-> section, right?
What is meant by category then?
On 13-Jan-2015 8:40 PM, "Sanket Sudake (संकेत )" <notifications@github.com>
wrote:

> @sagarrakshe <https://github.com/sagarrakshe> Is this correct change ?
> Assume its breaks planned thing.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/QuodeIT/Quodeitv1.1/commit/4b9e30fdd0aa7129a5f23711ea21f6b722b56bd0#commitcomment-9261320>
> .
>


> How about if we use URLs to reference the old trac tickets in commit messages and I remove the Trac commit hook?

+1 but we should never use `#` in commit messages (just `gh-`), leaving it for past commits or mistakes/lack of knowledge about our process of contributors.


This won't work MSVC2010 has no support for C99 :-(


For cherry-picks, I'd say always use `-x` and leave the reference.


The easy fix would be to include browser/noflo.js in the repo.


Should the changes in DialogPVRGuideOSD.xml be back-ported to official?


"allows robots and drones"?


r+


Should these be backported to official?


En fait ici j'avais un problème.

Concrètement, avec l'ancienne implémentation, pas de problème pour interpréter un programme, seulement, dans le cas d'un programme interprété, pas possible de changer la destination des outputs et inputs du programme, ils étaient settés en dur ici.

Du coup j'ai fait un workaround (vu que les std streams sont pas des FStreams) pour qu'ils aient le bon comportement quand même s'ils sont utilisés dans un cas normal, et un mode fail-safe quand ils sont wrappés par un socket ou autre chose.


2: M102 unrecognized bullet/signature
3: M102 unrecognized bullet/signature
4: M102 unrecognized bullet/signature


@zyllorion some features are not polyfillable to es3, others are.


needs javadoc to explain why this level-of-indirection is necessary


Co nas to obchodzi w tym momencie? Jeżeli się jakimś cudem udało zalogować graczowi "     ", to dajmy mu szansę się też wylogować. Najwyżej okażę się, że nie ma takiego loginu, co jest innym przypadkiem, ale można traktować go tak samo jak próba wylogowania niepoprawnego loginu.


Is there any howto to build gulp? I managed to install nodejs and nmp but I get a gulp error (is there a specific file for rainloop) ?

382/31738# /usr/bin/gulp
[17:18:45] No gulpfile found


Hi MZ,
The SDK zip contains some 3rd party libraries not appropriate for github. Try this link:
https://www.dropbox.com/s/bx7oy9afla1ji0h/MojoSdk.zip?dl=0
Cmor.


not sure if this could be package-protected rather than public (given the serialization), but we should check


```bash
npm install gulp -g
```


> We pull down pull requests, rebase them off of master – editing the commit message if it doesn't conform to the guidelines, and then locally (fast forward) merge, then push upstream

Wow, sounds like a ton of work to do :-).


That's what I've been doing, leaving the whole commit message unchanged from the original except for the "cherry-picked from" and "Conflicts" where they occur. I think that's all good info to have there.


@kostja, ping


Why isn't paramtype set to "light" automatically if the drawtype is e.g. plantlike or nodebox and paramtype is nil and paramtype2 is not "light"?


P.S. You should submit this to AOSP :)


@bausk Thanks alex.  Should the '5' in the equations above be '7'?


Ok let me check it out.  I will checkout the head revisions of audio-sdl, input-sdl, rsp-hle, video-glide64mk2, video-rice, and win64-deps.  Then I will checkout the head revision of Gillou68310/mupen64plus-core.  Then use the solution file in ui-console/projects/msvc2010.  Is that the correct procedure?


Zapewne to "something" to będzie akcja wyjścia z pokoju, czy nie tak? Poza tym, można to zrobić inaczej - players.remove(login) powinno zdarzyć się zawsze, więc informowanie kontrolera bym wrzucił do ifa z warunkiem (players.get(login) != null), a #remove wyjął poza if.


84694650_EnableCodeCoverage (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1094 of 1094 tests passed)

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


Done https://jira.spring.io/browse/SPR-12622.


To gracze są zalogowani, a nie loginy :) Taki minor, jakiś inny komunikat bym tu zastosował.


Need ```local $@;``` before the eval, or use Try::Tiny


Cool, will do that from now on.


쩝 이런거는 SQLite3같은거 쓰시지 그랬어요


the StorageLevel should be configurable, perhaps via "spark.storageLevel" as a recognized key in the moduleConsumerProperties?


hi also habs probiert und tauchen bei mir keine fallschirmspringer auf. auf was für eine höhe hast du die gesetzt? ab welcher höhe spawnen die? danke


Honestly, since there isn't anything else done to the action, you could just inline the whole thing.
```
changeBeamAction->addSubAction(AMActionSupport::buildControlMoveAction(beamSelectionMotor_, (beamPositions_.value(beam))));
```


This is my first time


I agree with @dmethvin & @scottgonzalez. Also, [most](https://github.com/jquery/jquery/commit/4cbf02df84dbcaa44b75a64ed832f7dbff2231dd) [of the team](https://github.com/jquery/jquery/commit/e07c00be724855a0af41e9cd083e7122485e4f2f) [have been](https://github.com/jquery/jquery/commit/abfb10c82ef91fef99ba92c287a6c5a9e8045e80) [using](https://github.com/jquery/jquery/commit/d588c852f06d3a2731aebd038f16e9e3206b399d) this format.

BTW, [you, too](https://github.com/jquery/jquery/commit/f3b89ba0820882bd4ce4404b7e7c819e7b506de5). :)


Just to be sure we're on the same page here's both definitions:
Definition in main.c
```__declspec(dllexport) int main(int argc, char *argv[]);```
Definition in sdl_main.h
```extern "C" int SDL_main(int argc, char *argv[]);```


Why was unistd required in the first place?


I think this is worthy of a rubygems release since the latest version is a blocker for a ruby2.2 upgrade for any project using it.


!

Nice catch ;)


I just did that...


391/31747# npm install gulp -g
npm WARN cannot run in wd gulp@3.8.10 marked-man --name gulp docs/CLI.md > gulp.1 (wd=gulp)

> v8flags@1.0.8 install /usr/lib/node_modules/gulp/node_modules/v8flags
> node fetch.js

/usr/bin/gulp -> /usr/lib/node_modules/gulp/bin/gulp.js
gulp@3.8.10 /usr/lib/node_modules/gulp
├── pretty-hrtime@0.2.2
├── interpret@0.3.10
├── deprecated@0.0.1
├── archy@1.0.0
├── minimist@1.1.0
├── tildify@1.0.0 (user-home@1.1.0)
├── semver@4.2.0
├── chalk@0.5.1 (escape-string-regexp@1.0.2, ansi-styles@1.1.0, supports-color@0.2.0, strip-ansi@0.3.0, has-ansi@0.1.0)
├── v8flags@1.0.8
├── orchestrator@0.3.7 (stream-consume@0.1.0, sequencify@0.0.7, end-of-stream@0.1.5)
├── gulp-util@3.0.2 (array-differ@1.0.0, object-assign@2.0.0, beeper@1.0.0, array-uniq@1.0.2, lodash._reinterpolate@2.4.1, replace-ext@0.0.1, vinyl@0.4.6, lodash.template@2.4.1, through2@0.6.3, multipipe@0.1.2, dateformat@1.0.11)
├── vinyl-fs@0.3.13 (graceful-fs@3.0.5, strip-bom@1.0.0, defaults@1.0.0, vinyl@0.4.6, mkdirp@0.5.0, through2@0.6.3, glob-stream@3.1.18, glob-watcher@0.0.6)
└── liftoff@0.13.6 (extend@1.3.0, flagged-respawn@0.3.1, resolve@1.0.0, findup-sync@0.1.3)


Я ок


When you merge this into `master` it should auto-close your [Pull Request](https://github.com/thoughtbot/refills/pull/196)…


Good point. We have more image related tweaks coming up, so maybe we should have our own custom block type.


We use more gently approach, but fact is, behaviour with broken urls is not specified in rfc. So, each developper can do anything he wish. href is html-escaped in renderer and should not break output.

The only thing, that make sense - don't add this patterns to commonmark spec.


Provide both Gill Sans and Gill Sans MT, they seem to be called differently depending on Windows version...


> Then I will checkout the head revision of Gillou68310/mupen64plus-core

You can use the head revision of mupen64plus-core, the MSVC2010 project file has already been merged


we should avoid the toString here (was fine for PoC only), and probably just make this take the payload as is (especially if the inputType is applied as usual)... otherwise, we could accept a MessageConverter somehow (via an optional getMessageConverter on the module?)


IMHO changing space --> tab should be very fast. So just enable the substitution by default.
Better than calling split() twice if \t is truly present (should be much more expensive).
If you have the time, measure runtime with vs without the substitution but  I think it might be better to enable it -- also avoids code bloat.


@jeremy Two questions
(1) I am very interested in what tools or techniques you used to identify and optimize the string allocation issue in per_thread_registry.rb.  Was it just ObjectSpace, or more?
(2) Can you point to any work done on "per thread registry" designs might be replacing this one?

I often find myself optimizing our code base, but have a harder time documenting the gains in speed or memory--at least in the time I can squeeze in for optimization tasks. so best practices would be a welcome bit of information :)
I am looking for thread-friendly ways to cache data even closer to the app than memcache, and am wondering if I should be looking at this built-in functionality, or are there other techniques to look at (including the one presented at Railsconf 2014 in Portland)

Thanks.


 $(**"#consumable_item"**).parent().attr("href","/items/"+data.consumable.id)
вот здесь?


Ok.
now go to rainloop root folder and:
```
npm install
gulp
```


yes


Just curious, what happens if the normal class methods are not ignored by JSON? Are they used in some way?


The worst part?  We're probably not done here...


The ```die``` calls in here would be better as ```Carp::croak()``` so the source of the error can be found easier


LGTM


even though this is a private class, I think we should make the name more explicit... BlockManagerChannel or maybe something more vague like MessageStoringChannel (technically it's going to buffer until it writes a block, either the name should capture that somehow and/or we should mention it here in JavaDoc)


There is already a method for this


I've disabled the Trac commit hook.


Returning the excerpt or the summary at this point makes it impossible for WP themes to customize thge excerpt through custom trimming or adding a custom "Read more" link after the text.

I propose the changes in the commit which limit the handling of the the excerpt from the used hook to posts of the type "podcast" and apply all the filters registered in Wordpress under the hook "wp_trim_excerpt" so themes can use this hook to further customize the text returned by Podlove Publisher.


It would be nice to be able to move to `#` references at some point, it gets a bit confusing when committing to other repos.


r+


One solution is to remove ```#include <SDL_main.h>``` and remove SDLmain.lib from the linker


I'm not sure where this line has come from?


1: M111 unrecognized component name: collections


try...cat...에서 로거 동작 테스트 완료했으므로 불필요한 로거 삭제


OIC why it's been moved here, but this is deliberate - we don't want to use source_url when looking up existing things (as it might well have changed), but do want to use it when creating it.


@bausk, With...
*Allowable Moment:  5.885 kip-ft
*Allowable Shear:   3.160 kip
...is it safe to say, any member to the right of the red line below, is sufficient?  (just a little confused by "Minimum design section is 3 x Base Section" reference above)
![screenhunter_202 jan 13 09 30](https://cloud.githubusercontent.com/assets/507113/5723101/ff3c2404-9b06-11e4-985d-4102af98419e.jpg)

Thanks.


Not a bummer at all - just consolidation. This functionality is available under the dataverses endpoint.

Sent from my iPhone

> On 13 בינו׳ 2015, at 16:35, Philip Durbin <notifications@github.com> wrote:
>
> Bummer. I've been using the "assignments" endpoint to grant roles to users on definition points as part of search testing: https://github.com/IQSS/dataverse/blob/master/scripts/search/tests/grant-spruce-admin-on-birds
>
> @michbarsinai is there any plan to re-introduce this at some point? Maybe under the "/s" secure area?
>
> —
> Reply to this email directly or view it on GitHub.
>


yes , "%Y-%m-%d [%H:%M:%S] is good idea


Improvements worked out with @tkrugg:

* New way to style the arrow. Benefits:
  * The width of the popup is no longer a bit larger than the root node (on desktop) (the visual width of the root node was smaller than the real one because of the way the arrow pseudo-element).
  * Vertical centering no longer a bit wrong on IE11/Win7, and also in other browsers in case the user sets a different vertical-align on the widget than the default (middle)..
  * Better behavior with custom vertical-align specified on the widget, in particular on IE.
* Not using anymore type=search on input, due to inconsistent behavior across browsers (pathological case: Safari/MacOSX), apparently without impact on accessibility.


Thanks. I'll change this once we get @frrrances' new classes.


> You can use the head revision of mupen64plus-core, the MSVC2010 project file has already been merged

Actually, no it hasn't.  That's ok.  I pulled your fork with the MSVC2010 project files.


Eventually we will probably move to the pom files in version 3.4 or later.


> You can use the head revision of mupen64plus-core, the MSVC2010 project file has already been merged

Oups sorry actually it's not merged you have to use Gillou68310/mupen64plus-core like you said ;-)


hmmm on solaris, we should actually get sigwaitinfo ... I am linking with librt for this purpose ... is this not working ?


Shouldn't we check for existence before?


Job pr-scala failed for 54c8d720 Took 251 min. (ping @som-snytt) [(results)](https://scala-webapps.epfl.ch/jenkins/job/pr-scala/9467/):

To retry exactly this commit, comment `PLS REBUILD/pr-scala@54c8d720baa5e6890ad38f4b3ae2b9576e07645f` on PR 4246.
NOTE: New commits are rebuilt automatically as they appear. A forced rebuild is only necessary for transient failures.


I think it's probably fine. People are using source control.

On Tue, Jan 13, 2015 at 9:37 AM, Ibrahim AshShohail <
notifications@github.com> wrote:

> Shouldn't we check for existence before?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/laravel/framework/commit/164206bf2daced533d011e321b36d193f91c625d#commitcomment-9261849>
> .
>


could we get rid of NoOpMessageBusBinderModule and instead simply check if the moduleExecutionFramework value is not null here? (if it's not null, it's a custom execution framework type and the binding is the responsibility of that framework's corresponding plugin implementation rather than the core functionality)


Is it possible that ```_parse_pipestatus()``` can detect something bad happened but ```$exception``` is false?

It might make more sense to move the call to ```_parse_pipestatus``` inside the eval (or try block) and have it throw exceptions instead of returning error strings.


This is only a commit. Which is OK. A pull request happens when you push a branch other than master to github and then create a pull request through the GitHub user interface.

This code looks good.


Wouldn't it be possible to write a similar test using the public API instead (that is the TSNE class)?


@stefanpenner Doesn't 6to5 transpile to es5 ? "6to5 maps directly to the equivalent ES5." [source](https://6to5.org/docs/compare/). So to even get started on IE8 you'd need at least an es5-shim, correct?


This classname should be generic.


noté je corrige


Um guys, that's in error.  The default port for collectd _is_ 25826:

https://collectd.org/wiki/index.php/Networking_introduction
> By default UDP port 25826 is used.

Can we roll this back?


@fredboudon, @pradal  : This line doesn't work if eigen is not installed with port (for example manually or from homebrew).
I don't know why, but if we set inc_path to "/usr/include", scons uses path defined in options.py else, it searches only in '/opt/local/include/eigen3' and fails:
Path for option eigen_includes does not exist: /opt/local/include/eigen3


@stefanpenner Thanks.  My concern is that with the 0.1.6 ember-cli and latest ember beta / canary I am getting issues specifically with IE8 that I've not resolved (and need to before release) - most seem to be outside of core ember.js.  Trying the above change gave a completely different, new, error which will also need to be tracked down.  Other than ember-cli's blueprints, modules and npm dependencies I'm not using anything other than ES3 js in the code I write.  I think it's really important to see some stabilisation in the entire dependency suite to prevent regressions.  A big selling point of ember-cli was the es3 compiler, in theory making everything IE8 compatible and removing development effort.  I'll have to hold off of esnext>6to5 until the ie8 issues I have open are closed.

I would rather see a commit to fix the version of the broken regenerator version until 6to5 is stabilised on all supported platforms with release versions of major modules.  Maybe a bit over cautious but I think that all secondary/tertiary dependencies should not be able to break ember-cli.


oops


for me it's 2 differents profiles.
x264hq (longname: x264 High) -> libx264
h264hq (long name: H-264 High) -> h264 encoder provider by ffmpeg/libav


But it's removed now, right?


[VH 單位](http://www.w3.org/TR/css3-values/#vh-unit)


We've started using 1 var declaration per variable. See:
https://github.com/airbnb/javascript#variables


If these values are always numbers, ```<=>``` is better than ```cmp```


TeamCity MonoGame :: Develop (Win) [Build 3.3.0.1849](http://teamcity.monogame.net/viewLog.html?buildId=4333&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


@jdesboeufs  c'est mieux ?


Yep, I screwed by ``` git commit -am '_'  && git push ``` without thinking I had sensitive data -_-


r+


r=alexcrichton rollup


Ok... Sorry to be such a pain in the ass. This is what I did, step by step:

git clone https://github.com/RainLoop/rainloop-webmail
cd rainloop-webmail/
npm install gulp -g
npm install
gulp

Finished building, bla bla, no errors. now what?


@robinedman is the indentation ok?


кооммент


This is definitely a bug, good catch. There are a couple ways to sort this out. The way you have here is one of them, but I think there are two options that are slightly better.

The first is to use use [done](https://github.com/cujojs/when/blob/master/docs/api.md#promisedone) instead of then in the last statement and keep it with 2 arguments. The second is to put the `catch` statement after the `then`. That way, if the promise rejects, it will skip the `then` and go straight to the `catch`. Of these two, I think using `done` is slightly better.


@yevgmotov This seems a little heavy handed. You should be able to just wrap the `-[AVCaptureDevice setFocusPointOfInterest:]` call with the `-[AVCaptureDevice isFocusPointOfInterestSupported]` check, instead of skipping over the entire block.


normally the return can be simply done with:
return jsonify(plugin=bundle.__dict__)
Ask to @HugoGarrido, he done it on the catalog part to extract datas to a json.


Sigh. Fixing.


If ledger is of type number and legal why is Number called?


That's all.


solo semicolon?


Ouch non, is_period_size_independent = True marche pas sur SimpleFormulaColumn...


Hi David,
I'm confused about the previous statement:
can I get rid of the "info-not-available" in general or do I need to replace them by a structure cariing the valid flag?
A second question is related to the traction field:
  M-traction-signal-status ::=
        ENUMERATED { info-not-available, traction-on-traction-off }

What is meant with this value?


I'd be interested to know why that _3754 is not affected. That seems really strange.


Puhycz,

You can access a sample code at the following link:

http://codepad.org/u8li6VsX

It uses the GLCD library to display data from DS18x20 temperature sensor.  To compile the code you have to create a project in MPLABX and add the source files from 1wire library, GLCD library and the one I'm providing.  To run the program it's better to try it in the simulator fist.
You can change the code if you don't have a GLCD (KS0108) display.

Regards,
Iván


Actually, we do use it - this makes deploys fail, since we do health checks against the supervisors with timeouts.  Good catch; you probably saved me about twenty minutes of trying to track this down :)


Looks like we don't need set the uri here (will be set when the user select a tool or when the sound is enabled)


this test passes?


I didn't find any regression with this but I'm not sure this is a correct fix because it's working fine on ARM without it. I'm suspecting the x86 backend to run out of register when trying to recompile the block. Maybe you should ask @Ari64 what he think about it.


Ok it's 2 differents codec, so it corresponds to 2 profiles.


That makes sense, I'll work with my pair first thing today to fix this up, thanks @sykesm.


Haha im just gonna nix it because I stopped using webstorm. Thanks man :p


Thanks a lot man, really love your work too :)

I was wondering about submitting it to AOSP too, would I submit it to their master or lollipop-release branch?


Aieieie, "cross platform" my butt lol. Let me know when you guys are agreed on the correct way to procede (looks like you're nearly there?) and then I'll merge.


r+


For statistics cursors, clearly compare() to know relative positioning does not make sense.  But it seems like we could provide equals() to know if two statistics cursors point to the same stat field.  I'm not sure it is terribly useful but equals() does make sense where compare() does not.


Tu regarderas ce commit. J'ai trouvé ce qui nous cassait la tête. On avait besoin de la syntaxe ```apply ... with (n := ...) (m := ...) ...```


It was moved to configuration.php, since front-end does not need it, but I mistakenly left it commented in configuration. I will put it back on next update.


Restored two spots to original versions, or at least really close.


Why checking `strlen()`? Is it intentionally not `if (empty(...))`?
It gave me a warning when I made custom composite field with waContactBirthdayField subfield, cause in case of waContactBirthdayField `$params_subfield['value']` contains an array, not string.


> BTW, you, too, @markelog. :)

Doesn't count! I was very young back then :-)


Yes but you can just redirect output to a file like:
linux:
./lyrister.py -s 'one love' > one_love

windows:
python lyrister.py -s "one love" > one_love.txt

As I understand it this is the preferred way of writing apps so that you can pipe the output into other apps.


Do you still support this project? I think it is very useful for Front-End developers


Oh I see that firmware was released after the initial disclosure. Gonna dig in later =)


`done` won't help here. May be because `return cb(...)` evaluates to true, I'm not sure. I tried it and it worked not so good.
One more way of course is to change condition to `(res && res.result !== undefined)`, but Promises has good error handling, so why would you do that?


This fix needs to be applied to the v0.3 branch also


Weird alignment here


TeamCity CI Experimen :: PHP Symfony2 project :: Symfony Project Test [Build 19](http://192.168.44.2:8111/viewLog.html?buildId=65&buildTypeId=CiExperimen_PhpSymfony2project_SymfonyProjectBuild) is now running


I normally just return on object without the hashes
```coffee
getOriginalImageDimensions: ->
  content = @component.content[@name]

  width: content.width
  height: content.height
```

or the short version:
```coffee
getOriginalImageDimensions: ->
  content = @component.content[@name]
  { width, height } = content
```


Te prends pas la tête, revert le tout.
@eraviart bosse déjà sur le sujet avec sum_calculate !


TeamCity CI Experimen :: PHP Symfony2 project :: Symfony Project Test [Build 19](http://192.168.44.2:8111/viewLog.html?buildId=65&buildTypeId=CiExperimen_PhpSymfony2project_SymfonyProjectBuild) outcome was **FAILURE**
Summary: Tests failed: 3 (3 new), passed: 11 Build time: 00:00:54
### Failed tests
```
Project Test Suite: Acme\DemoBundle\Tests\Controller\DemoControllerTest: testIndex: <no details avaliable>

Project Test Suite: Acme\DemoBundle\Tests\Controller\DemoControllerTest: testSecureSection: <no details avaliable>

Project Test Suite: Api\TestBundle\Tests\Controller\DefaultControllerTest: testIndex: <no details avaliable>

```


okay


@yevgmotov This needs a check on the return value. If the lock cannot be obtained, we shouldn't be attempting to change the configuration.


new markdown file


@eaquigley why did you take this stuff out? md5? size_in_bytes?


ok


The Number() cast is no longer necessary, true


> it gets a bit confusing when committing to other repos.

Yeah, especially when those conventions similar but has little differences and you have to commit to those repos practically every day/week, bound to have something weird on your hands.


Could I get a test case that fails with the current implementation but succeeds with the new?


Nevermind, @davidChevrier told me about it.


TeamCity MonoGame :: Develop (Win) [Build 3.3.0.1849](http://teamcity.monogame.net/viewLog.html?buildId=4333&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 359, ignored: 6 Build time: 00:17:21


Hi Pedro, thanks a lot for the note. My intention was to move it to
configuration.php,
since front-end does not need it, but I mistakenly left it commented in
configuration. I will uncomment it back on next update.
Thanks so much.
-John

On Tue, Jan 13, 2015 at 5:48 AM, Pedro Mendonça <notifications@github.com>
wrote:

> This removal caused Columns Titles to disappear.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/qTranslate-Team/qtranslate-x/commit/eb4abf40bcd330fe877c72ef6bd36deacc52f4f6#commitcomment-9259986>
> .
>


For the time being replace `EXPORT` with `extern C_LINKAGE`.  But this will only work properly on windows.  I'll continue to look for a real solution.


Since you're querying the DOM twice to see if it has the class, this should be saved to a var. It's costly to keep checking the DOM for the same thing in multiple locations :)

Something like:

`````
var isMosaicGallery = $gallery.hasClass('-mosaic');
var initialItems = isMosaicGallery ? 5 : 6;
var stepItems = isMosaicGallery ? 8 : 6;
`````


Master for sure, but i would submit to lollipop-release as well so they "maybe" include in 5.1 lol


your code is very tight and clean.

I am trying to find the line between committing fast and often, and refactoring to make it clean.


This can be declared after the following if statement


Is this running the ASUS Merlin firmware?


Won't this require all the files in pipeline/tutorial to also be changed?


1: M100 needs more reviewers
1: M101 signature is missing


What's the extra `a` at the end? typo?


Dear patrickmeiring,
what happen after all 50 Epoch was completed ?
the training is complete ?

and now how I input new Number's image to Network and then say to me the name of the number ?


You may also want to remove this line  :)


1: M111 unrecognized component name: collections


Is it safe to ignore this exception?


I think I see what buildscript this was based off of :P


Further, ASUS doesn't recognize this as a product they support - http://support.asus.com/Search.aspx?SLanguage=en&keyword=rt-16n


винести в js файл


Nevermind. I see it should be RT-N16.


Ok... I can't figure it out so I give up. I am using the version I just build but I see no green check mark for DKIM


We can't hard code https into the protocol like this. Some portals will continue to be http for a while. So the `app_config[:host] = app_config[:site_url].gsub("http://","")` code should also set a app_config[:protocol] based on the app_config[:site_url]


@bskinner There's no point to make any of this configurations if the focusPointOfInterest is not supported. That's why I wrapped the entire block. In fact, the focus manipulation here has very little effect on the scanning ability. It was an attempt to make scanning faster by manipulating with the focus. I didn't find any proof that it actually helped with scanning. I left it here as it brings no harm either.


TESTS PASSED
 Mr.Roboto url : http://jenkins.plone.org/roboto/get_info?push=2b2939e7a30049ca8bb719601daab0ce
 plone-5.0-python-2.7 [SUCCESS]


This change seems to cause problems in the Travis CI environment (at least). Was it intentional, or an accidentally committed dev environment setting?


info: https://github.com/mathiasbynens/dotfiles/issues/149


не залишайте оцих порожніх лінійок


Исходник старой утилитки


I think the braces aren't needed here?


@jduck yes, it's merlin build and I know, there is no official support anymore :(


Десь такий метод має бути вже імплементований... Якщо немає, то, думаю, було б добре мати його десь в userServrice, наприклад, щоб можна було використовувати по всьому проекту


Didn't know that. Quite quirky, -1 points for user interface :)


@bskinner I agree that it's probably a good idea to check for the return value of `lockForConfiguration` call. However, it won't really hurt if the lock is not obtained. I believe it just won't persist any configuration updates.

I'll make the change, but I'd suggest to mark it as a low priority and not hold anything because of that.


For the time being just remove `EXPORT`.  I'll continue to look for a real solution.


Accidental


Не забувайте про equals і hashcode


Looks good.


I'd split this up into two regexps, and then add a second clause to the list of tokenizers below, `{name: "COMMENT", val: blockcomment}`.  Names of tokens don't have to be unique in the list of token-recognizers.


r+ rollup


Should this be documented under the events section?


Very nice adding the length check to the regex!


Sure, will do...


Looks good to me.


I get an error when running the modified python script:

```
Traceback (most recent call last):
  File "/home/cs/suse/trollolo/scripts/create_burndown.py", line 81, in <module>
    plt.figure(1, figsize=(11, 6))
  File "/usr/lib64/python2.7/site-packages/matplotlib/pyplot.py", line 423, in figure
    **kwargs)
  File "/usr/lib64/python2.7/site-packages/matplotlib/backends/backend_tkagg.py", line 79, in new_figure_manager
    return new_figure_manager_given_figure(num, figure)
  File "/usr/lib64/python2.7/site-packages/matplotlib/backends/backend_tkagg.py", line 87, in new_figure_manager_given_figure
    window = Tk.Tk()
  File "/usr/lib64/python2.7/lib-tk/Tkinter.py", line 1740, in __init__
    baseName = os.path.basename(sys.argv[0])
IndexError: list index out of range
```

Do you know where this could come from? I assume you don't see it on your system.


Везде где ты забыл поставить пробелы)


Clearly it's a bug. Thanks for pointing it out.


вирівняти


These things happen - no need to ruin existing clones by rebasing the master.


> One solution is to remove #include <SDL_main.h> and remove SDLmain.lib from the linker

What do you think about this? I don't understand why we are using SDL_main anyway?


Lgtm. +1 for merge

On Jan 13, 2015, 16:41, at 16:41, Gael Varoquaux <gael.varoquaux@normalesup.org> wrote:
>0 is a bit crap. Let's have 1 or 2, so as to have a beautiful image on
>the front page.
>
>Sent from my phone. Please forgive brevity and mis spelling
>
>
>
>On Jan 13, 2015, 13:32, at 13:32, bthirion <notifications@github.com>
>wrote:
>>OK, but subject  1, or even subject 0, is good enough.
>>
>>---
>>Reply to this email directly or view it on GitHub:
>>https://github.com/nilearn/nilearn/commit/49c41be564f9a5b366fa064adcfe6fe3ae265832#commitcomment-9258935


не дуже гарно мати такі "магічні" числа, як оця сімка, скажімо)


TeamCity Ybackend :: AnnaPonomareva/ybackend [Build 110](http://192.168.30.225:8111/viewLog.html?buildId=516&buildTypeId=Ybackend_AnnaPonomarevaYbackend&guest=1) is now running


TeamCity Ybackend :: AnnaPonomareva/ybackend [Build 110](http://192.168.30.225:8111/viewLog.html?buildId=516&buildTypeId=Ybackend_AnnaPonomarevaYbackend&guest=1) outcome was **FAILURE**
Summary: Exit code 127 Build time: 00:00:01


Hi there, is this Dockerfile something you'd be happy to merge into our `master` branch?


Inicio de switch para generar ceros: /*Por el momento dfinimos haciendo uso de "switch" el largo del Byte, es decir que si decimal 2 a binario es igual a 10
+ entonces "completará" el Byte con 6 ceros faltantes por delante del resultado osea = 00000010 */


Saw the revert, sorry about that, I missed the `->name` completely... being a dyslectic is not always helpful ;-)


TeamCity Ylab :: zmactep/ylab [Build 149](http://192.168.30.225:8111/viewLog.html?buildId=517&buildTypeId=Ylab_ZmactepYlab&guest=1) is now running


A release will follow shortly (this week/next week) once I have confirmed the latest changes work in Shopify.


Make yourself a variable for `allKeys` rather than calling it all the time.


¡Post leído!
Me encanta la idea de Moloch, aunque fallo al ver como la ciencia y el arte son frágiles. La ciencia y el arte son armas tremendamente poderosas, y por eso persisten.
Ese final, aludiendo a la muerte de Dios... me puede.


всі лейби виносьте в ресурси (spring messages)


Ok thanks will do that tonight :) and thanks for your comments, really appreciated.


Thanks!


### [Codacy](https://www.codacy.com/p/4380/b/2037431/commit/6037059) detected an issue:
#### Message: `Avoid unused local variables such as '$phpbb_root_path'.`

#### Occurred on:
+ **Commit**:  4710fb25c0608cd4e5265115bb849793271fbab8
+ **File**:    [acp/main_module.php](https://github.com/Un1matr1x/phpBB-EXT-oGame/blob/4710fb25c0608cd4e5265115bb849793271fbab8/acp/main_module.php)
+ **LineNum**: [19](https://github.com/Un1matr1x/phpBB-EXT-oGame/blob/4710fb25c0608cd4e5265115bb849793271fbab8/acp/main_module.php#L19)
+ **Code**:    `		global $config, $phpbb_root_path, $phpbb_admin_path, $phpEx;`

#### Currently on:
+ **Commit**:  7a3a38289a41bc3701f7ac372e704bb5134a97f1
+ **File**:    [acp/main_module.php](https://github.com/Un1matr1x/phpBB-EXT-oGame/blob/7a3a38289a41bc3701f7ac372e704bb5134a97f1/acp/main_module.php)
+ **LineNum**: [19](https://github.com/Un1matr1x/phpBB-EXT-oGame/blob/7a3a38289a41bc3701f7ac372e704bb5134a97f1/acp/main_module.php#L19)


This deals with entity escaping in attribute values and node data. I know virtually nothing about neko and my C is a bit rusty, so I'll need @ncannasse to review this (in particular my buffer usage which looks a bit too Haxeish).

This passes my new Haxe unit tests, so we can re-introduce the native parser if we make a neko release.


Was this an oversight? Can we remove the "print" here?


Seems like this message should go somewhere that you can print the URL that the bad response came from. Otherwise this message isn't very actionable by the developer.


And it's an error?


Would fail on first install `composer create-project laravel/laravel` since `.env` is not available yet.


I was thinking the same thing.  ui-console is not built against SDL2 in mupen64plus-ae.  I'll take a look.

Otherwise, the "best" cross-platform solution I have is to replace `EXPORT` with
```
#ifndef WIN32
__attribute__((visibility("default")))
#endif
```
Which is the only reason I needed EXPORT in the first place -- to make it externally visible to other .so libraries.


@zyllorion although the name is 6to5 many aspects work fine in 3, just use the repl and see if the output of the feature you want is something you feel comfortable with.  If you don't want to take any chances, ember-cli-6to5 is an optional dependency, feel free to remove it from package.json and you wont be tempted.

Please note, our module transpiler will remain es3 safe for the foreseeable future, its merely the extra fancy features that may require es5.

Additionally, we are all busy, and could use some more IE8 testers and fixers. This issue appears near and dear to your, I would welcome the extra man-power on this topic.

> @stefanpenner Doesn't 6to5 transpile to es5 ? "6to5 maps directly to the equivalent ES5." source. So to even get started on IE8 you'd need at least an es5-shim, correct?

large chunks of 6to5 work in es3, as es5 is a superset of es3.


Related to #70


Yes. The pubkey_node is for extra logging detail. In recent versions of rippled this is provided in the initial server subscribe response, so this extra call to server_info is only necessary for old versions of rippled. This function will probably be deprecated entirely soon.


Those 5 commits fixing notification lockscreen error or quick dialer?


@samuelbeek Thanks for adding these! Will you create a PR for these next time please?


aufpassen, das hast du jetzt doppelt, Zeile 1, 6, 7 löschen und einrücken


Ah Ill check if file exists.

On Tue, Jan 13, 2015 at 10:17 AM, Mior Muhammad Zaki <
notifications@github.com> wrote:

> Would fail on first install composer create-project laravel/laravel since
> .env is not available yet.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/laravel/framework/commit/164206bf2daced533d011e321b36d193f91c625d#commitcomment-9262532>
> .
>


We were when using esprima-fb. Espree uses JSX instead of XJS, so had to do this.


Nevermind, realized I was looking at your fork not my repo :p


This is a generated file, and shouldn't be uploaded to the repository (Don't worry about it for now, I'll fix this)


@pdurbin did I? If so, it was not intentional at all and me having a slip of the hand when inserting the linking.


There is no possibility to check, if the deskewAngle = 0 is due to correct analyzeLayout function call or failled call.
The same for all other 3 properties.
I mean, I run tesseract, set image, call tesseract.deskewAngle, get 0. Why is this 0 here? Did analyseLayout run till the end or finished due to iterator == NULL condition?
There should be something to distiguish those two test cases.


getRow function u can look about it


Class name should be TitleAndAuthor (this wont' compile)


Looking at wpilib, it seems that the gyro's angle should be multiplied by a proportional scaling constant (Kp) in order to have better correction speeds.


This is totally wrong


Could you rename the file to just `COPYING` please?


It should be blue fire stones


good job


For what it's worth, initiative and hitPoints were designed to be integers.


This only works on OS X, see my Main.java for a cross platform solution


Using
```
let g:latex_vim_executable = v:progpath
```
seems the cleaner solution.


refactored tests, u can look about it


[Test] But a real comment :)
Please order the using statement alphabetically


Look's good to me ;-)


що це?


Please include some additional spaces before the $ sign, like this:

    $ bash run-report.sh

It makes it look a little nicer in the generated [README.md](https://github.com/LalatenduMohanty/gluster-bugs-webui/blob/678efb46669c0f0ad8b33dd9c00590f7f7572468/README.md).


TeamCity Ylab :: zmactep/ylab [Build 149](http://192.168.30.225:8111/viewLog.html?buildId=517&buildTypeId=Ylab_ZmactepYlab&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 121 Build time: 00:10:29


Should be unnecessary


This bug still here because, in some circonstances (for e.g. when rotate the device), the ```getMeasuredWidth()```method returns 0; and then, the result of the subtraction is negative leading to the ```java.lang.IllegalArgumentException: Layout: -36 < 0 ```

If you enclose the ```if (tempErrorText != null) ... else if (helperText != null)``` with a ```if (getMeasuredWidth() > 0) ...``` it will work.


lgmt


Should return `this`


Where's the method that reads the file line-by-line? (is the other guy doing this?)

The interface should be like:
```
public static List<String> readFile(File file) {
   ....
}
```


я думаю, що тут можна вказати папку, яку слід сканувати на наявність sql скриптів.


With all the comparisons that go on between cards, it would probably be better to cache the rank index and suit index on the card instead of computing it each time a card is compared.


all tests pass:
success: http://buildbot.rust-lang.org/builders/auto-mac-32-opt/builds/3129
success: http://buildbot.rust-lang.org/builders/auto-mac-64-opt/builds/3122
success: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-t/builds/3123
success: http://buildbot.rust-lang.org/builders/auto-linux-32-opt/builds/3114
success: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-t/builds/3114
success: http://buildbot.rust-lang.org/builders/auto-linux-64-opt/builds/3117
success: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-t/builds/3110
success: http://buildbot.rust-lang.org/builders/auto-linux-64-x-android-t/builds/3105
success: http://buildbot.rust-lang.org/builders/auto-win-32-opt/builds/2782
success: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-t/builds/2780
success: http://buildbot.rust-lang.org/builders/auto-win-64-opt/builds/1618
success: http://buildbot.rust-lang.org/builders/auto-win-64-nopt-t/builds/1611


fast-forwarding master to auto = e94a9f03


could simplify to: (SparkModule.MODULE_EXECUTION_FRAMEWORK.equals(moduleExecutionFramework))


Java variables names are camelCase: http://docs.oracle.com/javase/tutorial/java/nutsandbolts/variables.html


Not guaranteed, but it is only instantiated one time as a static variable in this process in the getGraph method.  Also, CUniquePath definition is private to CRRDGraphWrapper, so it is not intended for use outside the CRRDGraphWrapper class.


I don't like using "_" in a namespace (although our generated namespaces might include "_" as well, because it is being generated automatically from the REST-JSON files, I can elaborate more if you are interested...)

Anyway,
why not using something like:
Daimto.Google.Calendar

* you don't need to include dotnet in the namespace it looks weird.
* I don't like avoiding 2 letters to create Clndr, why not just Calendar?


Please see my last comment: https://github.com/drolbr/Overpass-API/issues/175

I opted for a different option there (csv option list).


This method doesn't actually throw FileNotFoundException, or IOException (it catches them)


perhaps we should refactor so that we can avoid all the super.supports() calls, but also allow the execution framework to be passed along... maybe with a ExecutionFrameworkPlugin interface that provides a method to be called from the superclass (AbstractStreamPlugin or StreamPlugin, which by the way it seems we could flatten into a single class)


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)


Before

![](http://i.imgur.com/rpvLoEb.png)

After

![](http://www.quickmeme.com/img/db/db62f31d5cbfebc64edf2bd0e627ff28a5741ed5147282d6472c1e9d69fd7c44.jpg)


The change in FormValueWidget that this commit copes with: 00f0b74736e4b1a9473ef0c1a1d8ed5971f15bcd.


actually, checking the framework here seems unnecessary given the supports method as it is now


Perhaps, we should run this test in the cycle to check all the possible page segmentation modes? Also there is no checking for the values, which are different from G8PageSegmentationMode. Should we add those tests and the proper code to the setter? Should we create a separate ticket for that?

@kevincon, what do you think?


So I
 - Kept `EXPORT` as it is
 - Removed `#include <SDL_main.h>` from this file
 - Removed the SDL*.lib linkage from the MSVC2010 project file.

This builds fine on both Android and MSVC2010.  Have checked the Unix makefile but the SDL association already looks weak there.  Looks like a dependency that no longer exists.  I think this is the right solution, as EXPORT can provide the same benefit to all platforms (namely, allowing ui-console to be built as a .DLL/.SO and called from another binary).

I can put in a pull request to remove the SDL dependency.  All you have to do is fix the MSVC2010 linkage to SDL.

 does the trick.  We can keep EXPORT in there (which provides the same benefit to Win32 as it does to Unix/Android).  Builds and compiles in MSVC2010 and Android.  Haven't tried the Unix makefile, but its association with SDL looks weak already.  So

Looks like a dependency that no longer exists.


1. Always write cycle with brackets
ex:
for (int k = 2; k < 2_000_000; k++) {
/**code*/
}
2. Declare and use variable "k" in cycle
3. Use an underscore as a divider for readability


Même remarque sur le msg de commit "add" au lieu de "Add"


Leonardo tuvo una emergencia familiar y tuvo que viajar a casa de su mama. En cuanto regrese se comunica de nuevo.


Idiomatic ruby drops the `is_` and `has_` prefixes, so this method should just be `old_maid?`


whats the problem?


This all looks fine!


This should be split into two methods, each taking in a `List<String>`, and returning the important bits:

```
public String getTitle(List<String> lines)

and

public String getAuthor(List<String> lines)
```

where `lines` is the output of `readFile()` (see above, `readFile()` should return a `List<String>`)

**protip**: If your method has an *and* in the name, it should usually be two methods


Like I said in the email, it looks like in your tests the Old Maid isn't always assigned, so this should take that into account. Something like `old_maid && card == old_maid`, since that would always return false if the Old Maid is not assigned.


Yeah that assert seems superfluous.


I don't think this is the right place to do this; it makes more sense to do it as part of rendering.


it was nice :)


Nice!

Just a question. Why you are doing in https://github.com/k-nut/leaflet-layer-overpass/commit/30687c44be4cc9f3499f3c625559c77ba005d302?w=1#diff-257679dfcf4fc435798b7afe904d3d5aR237
```
var self = this;
```

and then you use `this` again here https://github.com/k-nut/leaflet-layer-overpass/commit/30687c44be4cc9f3499f3c625559c77ba005d302?w=1#diff-257679dfcf4fc435798b7afe904d3d5aR242
```
if (this.status >= 200 && this.status < 400) {
```


Looks like leftover debug code too, doesn#t it?


We could use IsFinite instead of IsNan here.


This recurses infinitely, the parameter should actually be maxlinelength, and the parameter is not used in box2text-string.


By the way. Why checking for that? To guard against humongous functions?


What happens to your code if null is returned?

Hint: it crashes at line 58: `TitleAndAuthor.split("\n");`

If `TitleAndAuthor` is  null here, we get an NPE by calling `split()` on a null variable.

My suggestion is to remove the try/catch from your method (after you split them into two methods), and let the program crash if the file is not found or the user's disk does something stupid (fail fast: http://en.wikipedia.org/wiki/Fail-fast)

![Fail Fast and Carry On](http://social.eyeforpharma.com/sites/default/files/fail-fast-and-carry-on.png)


Given the code change, this no longer needs to have multiple cases


this should be optional with AC_ARG_ENABLE(), you can default to "yes".


Is this zero supposed to be there? Seems like a lot of unnecessary work for a vector of ones...


Yeah, hoping to introduce the v0.4 branch today, and leave v0.3 with a very short life.


Crap, I might have spoke too soon.  I get a segfault in Android.  This might just be a problem in the Android edition's java-c bridging library.  I'll check that out now.


since the fields are static, this won't make a difference, but the arg should be truly NULL rather than the string "null"


trop cool! oui j'ai vu ça c'st une bonne syntaxe. Je regarde là pour la
suite d ela preuve

On Tue, Jan 13, 2015 at 4:52 PM, Théo Zimmermann <notifications@github.com>
wrote:

> Tu regarderas ce commit. J'ai trouvé ce qui nous cassait la tête. On avait
> besoin de la syntaxe apply ... with (n := ...) (m := ...) ...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/stratSystemF/lambda/commit/60fdf71c40ae425c27463ef527cca50933f7d714#commitcomment-9262109>
> .
>


![screen shot 2015-01-13 at 10 29 09 am](https://cloud.githubusercontent.com/assets/705860/5724244/fce2de42-9b0f-11e4-86ad-39169a52af3c.png)


I like these changes :+1:
However, I think that we should update the Wiki accordingly: https://github.com/adsbypasser/adsbypasser/wiki/DOM-module-reference


In this case, I'd start by initializing `out` to `np.zeros(x.shape)`, then just do line 215.


actually we could probably ignore this (should never happen)


Yeah, that's better.