Found BROKEN by quick.
p:30:0,
                 from /home/casadibot/slaves/linuxbot/quick/source/casadi/core/mx/mx.hpp:28,
                 from /home/casadibot/slaves/linuxbot/quick/source/casadi/core/std_vector_tools.cpp:26:
/home/casadibot/slaves/linuxbot/quick/source/casadi/core/mx/../matrix/matrix_impl.hpp:3241:30: note:   ‘std::basic_ostream<char>’ is not derived from ‘std::basic_istream<_CharT, _Traits>’
                           >> n_col << "), but got " << Dwork.size() <<".");
                              ^
/home/casadibot/slaves/linuxbot/quick/source/casadi/core/mx/../casadi_exception.hpp:136:35: note: in definition of macro ‘casadi_assert_message’
       " failed. " << std::endl << msg  <<  std::endl;   \
                                   ^
In file included from /usr/include/c++/4.8/istream:879:0,
                 from /usr/include/c++/4.8/iostream:40,
                 from /home/casadibot/slaves/linuxbot/quick/source/casadi/core/mx/../printable_object.hpp:29,
                 from /home/casadibot/slaves/linuxbot/quick/source/casadi/core/mx/../shared_object.hpp:29,
                 from /home/casadibot/slaves/linuxbot/quick/source/casadi/core/mx/mx.hpp:28,
                 from /home/casadibot/slaves/linuxbot/quick/source/casadi/core/std_vector_tools.cpp:26:
/usr/include/c++/4.8/bits/istream.tcc:955:5: note: template<class _CharT2, class _Traits2> std::basic_istream<_CharT, _Traits>& std::operator>>(std::basic_istream<_CharT, _Traits>&, _CharT2*)
     operator>>(basic_istream<_CharT, _Traits>& __in, _CharT* __s)
     ^
/usr/include/c++/4.8/bits/istream.tcc:955:5: note:   template argument deduction/substitution failed:
In file included from /home/casadibot/slaves/linuxbot/quick/source/casadi/core/mx/../shared_object.hpp:30:0,
                 from /home/casadibot/slaves/linuxbot/quick/source/casadi/core/mx/mx.hpp:28,
                 from /home/casadibot/slaves/linuxbot/quick/source/casadi/core/std_vector_tools.cpp:26:
/home/casadibot/slaves/linuxbot/quick/source/casadi/core/mx/../matrix/matrix_impl.hpp:3241:30: note:   ‘std::basic_ostream<char>’ is not derived from ‘std::basic_istream<_CharT, _Traits>’
                           >> n_col << "), but got " << Dwork.size() <<".");
                              ^
/home/casadibot/slaves/linuxbot/quick/source/casadi/core/mx/../casadi_exception.hpp:136:35: note: in definition of macro ‘casadi_assert_message’
       " failed. " << std::endl << msg  <<  std::endl;   \
                                   ^
In file included from /usr/include/c++/4.8/string:53:0,
                 from /usr/include/c++/4.8/bits/locale_classes.h:40,
                 from /usr/include/c++/4.8/bits/ios_base.h:41,
                 from /usr/include/c++/4.8/ios:42,
                 from /usr/include/c++/4.8/ostream:38,
                 from /usr/include/c++/4.8/iostream:39,
                 from /home/casadibot/slaves/linuxbot/quick/source/casadi/core/mx/../printable_object.hpp:29,
                 from /home/casadibot/slaves/linuxbot/quick/source/casadi/core/mx/../shared_object.hpp:29,
                 from /home/casadibot/slaves/linuxbot/quick/source/casadi/core/mx/mx.hpp:28,
                 from /home/casadibot/slaves/linuxbot/quick/source/casadi/core/std_vector_tools.cpp:26:
/usr/include/c++/4.8/bits/basic_string.tcc:996:5: note: template<class _CharT, class _Traits, class _Alloc> std::basic_istream<_CharT, _Traits>& std::operator>>(std::basic_istream<_CharT, _Traits>&, std::basic_string<_CharT, _Traits, _Alloc>&)
     operator>>(basic_istream<_CharT, _Traits>& __in,
     ^
/usr/include/c++/4.8/bits/basic_string.tcc:996:5: note:   template argument deduction/substitution failed:
In file included from /home/casadibot/slaves/linuxbot/quick/source/casadi/core/mx/../shared_object.hpp:30:0,
                 from /home/casadibot/slaves/linuxbot/quick/source/casadi/core/mx/mx.hpp:28,
                 from /home/casadibot/slaves/linuxbot/quick/source/casadi/core/std_vector_tools.cpp:26:
/home/casadibot/slaves/linuxbot/quick/source/casadi/core/mx/../matrix/matrix_impl.hpp:3241:30: note:   ‘std::basic_ostream<char>’ is not derived from ‘std::basic_istream<_CharT, _Traits>’
                           >> n_col << "), but got " << Dwork.size() <<".");
                              ^
/home/casadibot/slaves/linuxbot/quick/source/casadi/core/mx/../casadi_exception.hpp:136:35: note: in definition of macro ‘casadi_assert_message’
       " failed. " << std::endl << msg  <<  std::endl;   \
                                   ^
make[2]: *** [casadi/core/CMakeFiles/casadi.dir/std_vector_tools.cpp.o] Error 1
make[2]: *** Waiting for unfinished jobs....
make[2]: *** [casadi/core/CMakeFiles/casadi.dir/generic_type.cpp.o] Error 1
make[2]: *** [casadi/core/CMakeFiles/casadi.dir/options_functionality.cpp.o] Error 1
make[2]: *** [casadi/core/CMakeFiles/casadi.dir/casadi_instantiations.cpp.o] Error 1
make[1]: *** [casadi/core/CMakeFiles/casadi.dir/all] Error 2
make: *** [all] Error 2

```


Okay.  But it's fine to leave the 3 clauses like this?  Or should I have an empty, not empty case and then a 1 and >1 case?


> If the identity of an app is defined by a URL instead of an origin, doesn't that mean that the Web's permission/security model would break for installed Web apps? If you give one app permission to do something, and that is tied to the origin, wouldn't that mean that another app, on the same origin but with a different start URL, would be able to use that permission, too?

The Manifest for Web Applications specification currently doesn't have anything to say on the topic of permissions, and therefore this doesn't change anything.

However, if permissions are ever defined in the manifest in future it would be interesting to discuss whether those permissions only apply to an application context and are therefore limited to the scope of the app, or whether they apply to the whole origin. In Firefox OS for example privileged permissions are keyed by origin + manifest URL and are only granted to same-origin resources loaded inside the application context with a manifest applied.

I think permissions is probably a separate topic to identity though.


Cool... will be aware of this, thanks for the heads up.


Why return $content here?


You could also use tempfile.mkdtemp() to do the same thing. Start out by creating a temporary directory, like so:

import tempfile
import shutil

tmp_dir=tempfile.mkdtemp()

Now, open the file:

sys.stderr=open(os.path.join(tmp_dir, "stdout.log"), "w")
...

Once the temporary log files are no longer needed and have been closed, you could delete them in one line:

shutil.rmtree(tmp_dir)

This results in more elegant/readable code, in my opinion. Why do things in more lines than absolutely necessary? Also, this avoids the possibility, however remote, that someone has os.environ["TEMP"] set to the empty string.


@pecko - we will put the "this" object into the map each time we add a GeoToolsLayer - which will be multiple times.

I suggest you just do it once, in the GtProjection constructor.


We have a lot of problems with Lucene, and we really want to switch from it.
The problem is that it is very much a non trivial task to do.
I spiked it using Voron, but we are currently stuck with how to handle
updates efficently.

*Hibernating Rhinos Ltd  *

Oren Eini* l CEO l *Mobile: + 972-52-548-6969

Office: +972-4-622-7811 *l *Fax: +972-153-4-622-7811

On Mon, Jan 12, 2015 at 6:13 PM, Federico Andres Lois <
notifications@github.com> wrote:

> Dont get me started on Lucene :P
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Corvalius/ravendb/commit/19a94a4713a21198d410f8cb9af3cf2949334088#commitcomment-9243883>
> .
>


> I believe the places where specific quote handling was already added, we should leave it at that to not possibly break things that were there intentionally...

At least for the 2 trackback occurences it make sense to switch to null instead of ENT_COMPAT, since that was coded long before knowing the new Bits with 5.4+ versions, I believe.
```
serendipity_entity_decode(strip_tags($matches[1]), null, LANG_CHARSET);
serendipity_entity_decode(strip_tags($body), null, LANG_CHARSET);
```


This is WIP on #16 not #17 as the commit msg suggests


Yes, I see that now.


Thanks, it’s a good suggestion. I will take it into account.
From: Tyler Kavanaugh
Sent: Monday, January 12, 2015 5:17 PM
To: manuelcortez/TWBlue
Cc: José Manuel
Subject: Re: [TWBlue] Installer: updated website. Application.py: changed version to 0.51. Main.py: redirect stdout and stderr to log files located in temp folder. (05d6245)

You could also use tempfile.mkdtemp() to do the same thing. Start out by creating a temporary directory, like so:

import tempfile
import shutil

tmp_dir=tempfile.mkdtemp()

Now, open the file:

sys.stderr=open(os.path.join(tmp_dir, "stdout.log"), "w")
...

Once the temporary log files are no longer needed and have been closed, you could delete them in one line:

shutil.rmtree(tmp_dir)

This results in more elegant/readable code, in my opinion. Why do things in more lines than absolutely necessary? Also, this avoids the possibility, however remote, that someone has os.environ["TEMP"] set to the empty string.

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


It may be better to contain the neighbor nodes in the network.


I am a bit confused on this?

so any keypress will bring up the prompt? what if it connects somehow?


No its not allowed. If it is it should be removed
On 12 Jan 2015 05:41, "tamland" <notifications@github.com> wrote:

> @MartijnKaijser <https://github.com/MartijnKaijser> Is this the sort of
> thing ok for addons in the official repo? It writes to advancedsettings.xml
> and contains hacks to write itself into the confluence skin.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/xbmc/repo-scripts/commit/6edaac249d5540893b784b48bc7acdafed9a41e4#commitcomment-9241247>
> .
>


Hey,

Looks like russian locale is broken now.

```javascript
// main file index.js
var _definitions = {
  "name": ["first_name", "last_name", "prefix", "suffix"],
...
_definitions[d].forEach(function(p){
    Object.defineProperty(exports.definitions[d], p, {
      get: function () {
        if (typeof locales[exports.locale][d] === "undefined" || typeof locales[exports.locale][d][p] === "undefined") {
          // certain localization sets contain less data then others.
          // in the case of a missing defintion, use the default localeFallback to substitute the missing set data
          return locales[exports.localeFallback][d][p];
        } else {
          // return localized data
          return locales[exports.locale][d][p];
        }
      }
    });
  });
```
definitions.name.male_first_name this won't be defined.


84632652-PIUFans (kbenne) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2164 of 2190 tests passed)

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


Я бы задекларировал эту функцию перед https://github.com/B-Vladi/Response/blob/master/Response.js#L50
Всякие фичи javascript, которые позволяют этому работать это круто конечно, но код читать не удобно.


We used the Z version in our first batch of Mulles, and I have about 10 of
those units left. I would prefer if they would work in riot as well, which
is why I made these modifications in the first place. I am not sure if
@Farthen 's device has the latching LPTMR0 implementation or the early
implementation either.
On Jan 12, 2015 4:39 PM, "Johann Fischer" <notifications@github.com> wrote:

> Do you or your customers use a Z-Device?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/gebart/RIOT/commit/52a14de12dd12071f81400942c7e7cc3cef938fd#commitcomment-9243265>
> .
>


MAMMAAAAAAAAAAAAAAAAAAAAAAA xDD


We have the guard for Rational, but it's more complex than an instanceof check because we only have a handle to a RubyClass for any of the Rubinius classes.


pfiuuuu… tired.


to be compatible with the binary distribution, the bios should be copied over to the install folder, and then copies to RetroPie/BIOS during the configure stage (This may change in the future, but right now that's how it has to be done to work).


Read the `giveItemRandomly` code. If item limit reached, it plays QUEST_MIDDLE.


This is consistent with the definition. When the status is given, it is set and the method returns $this.


OK, but it just put one object to map.


also if you open and then close and open and close... you will now have two events bound right?


Sorry, but what does "natively" means? How does it enable anything on windows? Do we really want to tackle with python and all on windows, or I missed the point? Ansible can not run on windows, meaning the control machine can not be windows, but if you run it from posix machine, it can configure windows machine, too.

Just to be clean, I pointed out how AWS works just to give an example. Inventory of machines can be simple text file with every line corresponding to one machine.

It's possible that your trying to ask something totally different and I don't see it. If that's the case, please point me in the right direction.


I'm assuming it wouldn't reconnect if beaker has crashed, is that right @bgrinning


you can do this!! quale è il problema?


rps_retronet_prepareConfig no longer exists - was all changed last night so sorry about that - you can just remove it and the netplay parameters.


On 12/01/15 16:07, David Disseldorp wrote:
>
> This assumes that "net ads join" supports your newly added
> osServicePack argument. I don't think that's a safe assumption to make
> without a corresponding samba-client version dependency.
>
I don't think there needs to be a *hard* dependency, the information I
got earlier is that it is possible to "include multiple packages in one
single maintenance incident" by informing via "Bugzilla (NEEDINFO
maint-coord@suse.de)"
>
> "net ads join" appears to handle unsupported parameters as if they are
> the "domain name" argument.
>
not sure what you mean, if you mean you think net ads might silently
ignore the new parameter then I seem to recall that it didn't an
triggered an error (but I could be wrong)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/noelpower/yast-samba-client/commit/7b5069707c6a146544c347a402efada99d0a148d#commitcomment-9243754>.
>


word, sorry about the premature merge :no_mouth:


I think the UI stuff fits a lot better in the data repository as it was so far. Also `<!DOCTYPE html>` is missing.


(the netplay parameters is added automatically by runcommand if enabled in the runcommand.sh menu). this means you dont need to configure everything when changing netplay config, nor have to go to retropie-setup to enable/disable it. you can just launch a game with it enabled when required


I know.  But, there isn't a logical connection between "add a new layer" and "embed a this pointer in the map".

We both know you do it because you rely on it to support GeoTools objects, but it doesn't seem to logically fit in this method.  The constructor is better.


I was thinking the tests would be a new file, but maybe it's fine if we repurpose this one to be "all unit tests" and make separate files when we need to.  In that case, can you just change the test prefix to "versioning/auto" or something?


I believe we don't want this block, since it's redundant.


Please check the encoding format of README file. Everything in Chinese cannot show properly in Markdown but can be seen in pure text.

I am using the GitHub official Android app, and my phone#s default language is Chinese.

There must be some mistakes, whether it is in this file or in the app.


Do you mean make it nested for empty-not-empty and then 1-greater-than-1? Yeah that's a good idea.


We don't really need that, we can use git blame to know the author ;)


  Jobandtalent mail

    /* Client-specific Styles & Reset */

    #outlook a {
      padding:0;
    }

    body.outlook p {
      display: inline !important;
    }

    body{
      width:100% !important;
      min-width: 100%;
      -webkit-text-size-adjust:100%;
      -ms-text-size-adjust:100%;
      margin:0;
      padding:0;
    }

    .ExternalClass {
      width:100%;
    }

    .ExternalClass,
    .ExternalClass p,
    .ExternalClass span,
    .ExternalClass font,
    .ExternalClass td,
    .ExternalClass div {
      line-height: 100%;
    }

    #backgroundTable {
      margin:0;
      padding:0;
      width:100% !important;
      line-height: 100% !important;
    }

    img {
      outline:none;
      text-decoration:none;
      -ms-interpolation-mode: bicubic;
      width: auto;
      max-width: 100%;
      float: left;
      clear: both;
      display: block;
    }

    center {
      width: 100%;
      min-width: 580px;
    }

    a img {
      border: none;
    }

    p {
      margin: 0 0 0 10px;
    }

    table {
      border-spacing: 0;
      border-collapse: collapse;
    }

    td {
      word-break: break-word;
      -webkit-hyphens: auto;
      -moz-hyphens: auto;
      hyphens: auto;
      border-collapse: collapse !important;
    }

    table, tr, td {
      padding: 0;
      vertical-align: top;
      text-align: left;
    }

    /* Media Queries */

    @media only screen and (max-width: 600px) {

      table[class="body"] center {
        min-width: 0 !important;
      }

      table[class="body"] .container {
        width: 95% !important;
      }

      table[class="body"] .row {
        width: 100% !important;
        display: block !important;
      }

      table[class="body"] .wrapper {
        display: block !important;
        padding-right: 0 !important;
      }

      table[class="body"] .columns,
      table[class="body"] .column {
        table-layout: fixed !important;
        float: none !important;
        width: 100% !important;
        padding-right: 0px !important;
        padding-left: 0px !important;
        display: block !important;
      }

      table[class="body"] .wrapper.first .columns,
      table[class="body"] .wrapper.first .column {
        display: table !important;
      }

      table[class="body"] table.columns td,
      table[class="body"] table.column td {
        width: 100% !important;
      }

      table[class="body"] .columns td.one,
      table[class="body"] .column td.one { width: 8.333333% !important; }
      table[class="body"] .columns td.two,
      table[class="body"] .column td.two { width: 16.666666% !important; }
      table[class="body"] .columns td.three,
      table[class="body"] .column td.three { width: 25% !important; }
      table[class="body"] .columns td.four,
      table[class="body"] .column td.four { width: 33.333333% !important; }
      table[class="body"] .columns td.five,
      table[class="body"] .column td.five { width: 41.666666% !important; }
      table[class="body"] .columns td.six,
      table[class="body"] .column td.six { width: 50% !important; }
      table[class="body"] .columns td.seven,
      table[class="body"] .column td.seven { width: 58.333333% !important; }
      table[class="body"] .columns td.eight,
      table[class="body"] .column td.eight { width: 66.666666% !important; }
      table[class="body"] .columns td.nine,
      table[class="body"] .column td.nine { width: 75% !important; }
      table[class="body"] .columns td.ten,
      table[class="body"] .column td.ten { width: 83.333333% !important; }
      table[class="body"] .columns td.eleven,
      table[class="body"] .column td.eleven { width: 91.666666% !important; }
      table[class="body"] .columns td.twelve,
      table[class="body"] .column td.twelve { width: 100% !important; }

      table[class="body"] td.offset-by-one,
      table[class="body"] td.offset-by-two,
      table[class="body"] td.offset-by-three,
      table[class="body"] td.offset-by-four,
      table[class="body"] td.offset-by-five,
      table[class="body"] td.offset-by-six,
      table[class="body"] td.offset-by-seven,
      table[class="body"] td.offset-by-eight,
      table[class="body"] td.offset-by-nine,
      table[class="body"] td.offset-by-ten,
      table[class="body"] td.offset-by-eleven {
        padding-left: 0 !important;
      }

      table[class="body"] table.columns td.expander {
        width: 1px !important;
      }

      table[class="body"] .right-text-pad,
      table[class="body"] .text-pad-right {
        padding-left: 10px !important;
      }

      table[class="body"] .left-text-pad,
      table[class="body"] .text-pad-left {
        padding-right: 10px !important;
      }

      table[class="body"] .hide-for-small,
      table[class="body"] .show-for-desktop {
        display: none !important;
      }

      table[class="body"] .show-for-small,
      table[class="body"] .hide-for-desktop {
        display: inherit !important;
      }

      table[class="body"] .button,
      table[class="body"] .tiny-button,
      table[class="body"] .small-button,
      table[class="body"] .medium-button,
      table[class="body"] .large-button, {
        display: block!important;
        width: 100%!important;
      }

      /* Custom media-queries */

      a[class="call_to_action"], a.call_to_action { display:block!important; }
      div[class="column"], div.column { width:100%!important; float:none!important; display:block!important;}
      div[id="pad_on_mobile"] { padding:0 10px!important; }
      div[class="column"] td { text-align:left!important; }

      img[class="companies_image"] { width:100%!important; }
      div[class="alternative_suggestions"] a[class="company"] { margin-left:auto!important; margin-right:auto!important; }
      div[class="job_opening_invitation"] div[class="column"] a { width:auto!important; display:block!important; margin-bottom:20px!important;}
      div[class="job_opening_invitation"] div[class="column-wrap bottom_bordered"] { padding-bottom:0px!important; }
      div[class="opportunities_newsletter"] .image_cell { width:65px!important; }
      div[class="opportunities_newsletter"] .image_cell img { width:65px!important; min-width:0!important; }
      div[class="opportunities_newsletter"] .bordered .variable_pad { padding:10px!important; }

      div[class="notification_review"] .image_cell { width:65px!important; }
      div[class="notification_review"] .image_cell img { width:65px!important; min-width:0!important; }

      div[class="welcome"] div[class="wrapper"] table { margin-bottom:0px!important;}
    }

    /* Targeting clients that accept media quieries */
    @media all {
      div[class="opportunities_newsletter"] .image_cell img,
      div[class="notification_review"] .image_cell img {
        min-width:150px;
      }
    }

  New jobs released

  To date, there are more than 2,500 new job opportunities posted on jobandtalent. Do not miss these opportunities.

                View jobs

    -- The jobandtalent Team

                        You are receiving this email because reply@reply.github.com is registered at Jobandtalent.com (3rd floor, 36-37 Featherstone Street London EC1Y 8QZ) and agreed to receive email from us.

	If you would like to unsubscribe and stop receiving these emails click here


I mean use logger to log it.


Why is it necessary to convert the ID to hex?


I just used fsmx-libretro as a template. https://github.com/petrockblog/RetroPie-Setup/blob/master/scriptmodules/libretrocores/fmsx-libretro.sh

Or is mupen64plus-libretro the right way? https://github.com/petrockblog/RetroPie-Setup/blob/9d4a2e05f1de73cfe5731396f439f24e75eadc92/scriptmodules/libretrocores/mupen64libretro.sh#L38


Node? Too generic a name


It seems that "https://www.meethue.com//en-US/api/getaccesstokengivepermission" will hit 404 error.


@Ocramius Why do you prefer sending more than 1 then?


Pull request with failing tests?


Do we need to GetDocuments? There's usually more than one.


this comment should read: `like Fn<(A, B,), Output=C> to `Fn(A, B) -> C`


I think t5/core/dom and t5/core/events are now useless.
And you can maybe replace "tjq/vendor/ui/jquery.ui.datepicker", "tjq/vendor/ui/jquery-ui.custom","datepickerLocal" by only "datepickerLocal"


@smarquespt The translation source is: "hh:mm:ss". The translation is "HH:mm:ss" AFAIK.


Hi
  The french tokenizer is not exported to be public at the API Level. Is that normal ?


Y'avait déjà `var model = this;` plus bas.


Okay.  I'll do that then.


This could do with a better name. I was looking at it thinking it was the normal `[n]` case when in fact it's the `[n, m]` case isn't it?


Thanks it is working!


84632652-PIUFans (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)


@dkayiwa Yes, you are right .form.formId in the line number 22  isn't subjected to XSS attacks since the user's inputs are mostly subjected to XSS attacks. Thank you and  I'll make the required changes in the pull request.


Appreciate this.  I might later give a bit more thought to the service selections (eg WeldSEBatchArtifactFactoryImpl and thread pool) used in the default.   But will still merge in now.


use single quotes


dito


Thinking out loud here. Why not fix Lucene a bit instead in the meanwhile?
Meanwhile, as in, have the proper time to invest in owning that part of the
stack too. Lucene in itself is a good solution, a bit underdeveloped lately
(last big update is 3.0.3 and almost 1 year old already), but serves the
purpose quite well. Armed with our knowledge on how it is used on RavenDB
there are many optimizations that would certainly impact us dearly.

While rewriting and stabilizing the whole Lucene functionality may take 1
year or so. In 1 or 2 months of work we could be able to at least optimize
the common paths.

From: Ayende Rahien [mailto:notifications@github.com]
Sent: lunes, 12 de enero de 2015 01:20 p.m.
To: Corvalius/ravendb
Cc: Federico Andres Lois
Subject: Re: [ravendb] Aggresively inlining methods and properties with
millions of executions when bulk uploading 25000 freedb records. (19a94a4)

We have a lot of problems with Lucene, and we really want to switch from it.

The problem is that it is very much a non trivial task to do.
I spiked it using Voron, but we are currently stuck with how to handle
updates efficently.

*Hibernating Rhinos Ltd *

Oren Eini* l CEO l *Mobile: + 972-52-548-6969

Office: +972-4-622-7811 *l *Fax: +972-153-4-622-7811

On Mon, Jan 12, 2015 at 6:13 PM, Federico Andres Lois <
 <mailto:notifications@github.com> notifications@github.com> wrote:

> Dont get me started on Lucene :P
>
> -
> Reply to this email directly or view it on GitHub
> <
<https://github.com/Corvalius/ravendb/commit/19a94a4713a21198d410f8cb9af3cf2
949334088#commitcomment-9243883>
https://github.com/Corvalius/ravendb/commit/19a94a4713a21198d410f8cb9af3cf29
49334088#commitcomment-9243883>
> .
>

-
Reply to this email directly or
<https://github.com/Corvalius/ravendb/commit/19a94a4713a21198d410f8cb9af3cf2
949334088#commitcomment-9243995> view it on GitHub.
<https://github.com/notifications/beacon/AA_zowHd8_Rqu-jSjS01Th8beWnpvAGwks5
ng-uSgaJpZM4DRNQw.gif>


When aria-hiding icon fonts, I like to ask myself if the icon is conveying meaningful information that is not being conveyed in the adjacent text.  If so, I like to add SR text.  In this situation, I think screen reader text "Error" would be appropriate.  I think it should be added to the ```<h3>```.


Maybe better to make directive instead - to be discussed.


`out` is a `QString` here and is not safe to pass to vcs. IMHO this is the cause of UV-CDAT/uvcdat#957.


all tests pass:
success: http://build.servo.org/builders/linux1/builds/517
success: http://build.servo.org/builders/linux2/builds/515
success: http://build.servo.org/builders/android/builds/553
success: http://build.servo.org/builders/mac1/builds/734
success: http://build.servo.org/builders/mac2/builds/734


yeh more like mupen64plus-libretro currently. fmsx-libretro needs fixing up also.


84632652-PIUFans (kbenne) - i386-Windows-7-VisualStudio-12: OK (2179 of 2190 tests passed)

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


Yes, `engineName` & `customPath` are passed from the calling function to `requireEngine`. Then `requireEngine` returns `engine`, and the calling function assigns that returned value to `Adapter.engine`. I changed it so that the variables are accessed directly by `requireEngine`, and the result is assigned directly to `Adapter.engine` by `requireEngine`.

I'm not sure how that's a controversial change... Unless you want to discuss the merits of object oriented vs functional programming, then I could totally see your point. But since we're dealing with classes, and it's a function that's specific to this class, I don't think that's what you're getting at.

And the error handling that it previously had took any error that was thrown by L159-160 and rethrew it as a generic "engine not found" error. We could probably change it to parse the error code and make sure it's really "MODULE_NOT_FOUND" like we're expecting to be thrown there. But the error thrown by `Module._resolveFilename` is probably nicer anyway because it makes it clear that the search is being done by require, rather than some other module-finding logic we have.


My pleasure ;)


@SharonVarghese by the way, it is not the only one. You have a couple of other places where you need to also remove this. :)


My initial agreement was that the author would not do that. So if he has
now done so he breaches that agreement and our Repo rules.
On 12 Jan 2015 08:21, "Martijn Kaijser" <machine.sanctum@gmail.com> wrote:

> No its not allowed. If it is it should be removed
> On 12 Jan 2015 05:41, "tamland" <notifications@github.com> wrote:
>
>> @MartijnKaijser <https://github.com/MartijnKaijser> Is this the sort of
>> thing ok for addons in the official repo? It writes to advancedsettings.xml
>> and contains hacks to write itself into the confluence skin.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/xbmc/repo-scripts/commit/6edaac249d5540893b784b48bc7acdafed9a41e4#commitcomment-9241247>
>> .
>>
>


at some point I may rethink / rework this to use the separate install_binary stage to install the requisites like bios etc, but for now it's best done in configure.


@TomasVotruba I didn't optimize that, but feel free to PR


Any concrete reason why this is not active?


fast-forwarding master to auto = e6fe9f14


My device uses the latching implementation. It does actually read as reproducibly 0 if you don't write any value to it.


It may be better to add another get method taking multiple spatial as an argument.


You might want to change the lines on `defaultsettings.cpp` and `minetest.conf.example` too.


Modify `Eh::Settings#releases_dir` to take the additional path as an argument and let it construct the path, also use File.join instead of String interpolation


+++ Vitaly Puzrin [Jan 12 15 07:51 ]:
>No need to add `use strict` to each function, if you write in node style. Place it on the top. Each file will be scoped anyway on `require` (and after browserify too).

Good to know!


an early commit from me, well done


No, this is the current version number from esprima. I would rather keep the version numbers in check with esprima and just use the text portion for our own needs.


Is that OK? Otherwise, we risk losing where we really are on esprima?


Sounds good, I think we’ll stick with one file then. And fewer property files means fewer files to copy to the robot too.

On Sun, Jan 11, 2015 at 5:54 PM, jeffjensen <notifications@github.com>
wrote:

> Multiple property files can help organization but also can make it more tedious.  If we simply name properties with prefixes per area and keep them together in the file, then one file can work well and keep it simpler.  Even with multiple files, we would name properties with good, clear prefixes.
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/frc-2175/2015RobotCode/commit/507b58dcaad10c5eba29a3d629984badb73dac09#commitcomment-9235293


lyiliiy


lyilyi


Yes, agreed, let's name it `sliceIntegerFixnum`.
Or `sliceInt` or `sliceIntArray`?
Also, for the node name, maybe GetNode or SliceNode since it does more than just indexing?


Ok, AFAIK, it is also check when gold changes, like when there is no more gold on a tile, most notably.
The first thing would be to check whether this kind of comparison is no more than every turns. If so, I think we can on with removing it as I've seen nothing more using it so far.


@cptvitamin so I would add aria-label="Error" to the h3 element?


Ah, that makes a lot of sense. Thanks for clarifying that.
Other than http://www.cse.chalmers.se/~rjmh/QuickCheck/manual.html, is there a more detailed guide somewhere? I searched and couldn't find.
I've been using QC a lot over the years, but I didn't know those (very useful) details you just mentioned.


No.


r+


Slice-whatever is fine. I'm not so sure about the second idea. I'd like to be able to tell at a glance what method this is in a stack trace, and for operators I think uniformity is good.


```role="link"``` is unnecessary here since we're already using an ```<a>```

> "In the majority of cases setting an ARIA role and/or aria-* attribute that matches the default implicit ARIA semantics is unnecessary and not recommended as these properties are already set by the browser."

Cite non-normative note: <http://www.w3.org/html/wg/drafts/html/CR/dom.html#sec-strong-native-semantics>


I´ve checked worldclock plugin explanation and this is the way to go.


> That's what's left - more less...

> When these are sorted out, we can try to boot/test etc.

did you mean this ? https://github.com/ChronoMonochrome/kernel_common/commits/3_0_72
Finding changes from 3_0_72_2 in ste kernel isn't so complicated, but is it ok that it doesn't applies cleanly?


We would lose the ability to update to the 4.x version, which is what the
lucene.net community is working hard on.

*Hibernating Rhinos Ltd  *

Oren Eini* l CEO l *Mobile: + 972-52-548-6969

Office: +972-4-622-7811 *l *Fax: +972-153-4-622-7811

On Mon, Jan 12, 2015 at 6:31 PM, Federico Andres Lois <
notifications@github.com> wrote:

> Thinking out loud here. Why not fix Lucene a bit instead in the meanwhile?
> Meanwhile, as in, have the proper time to invest in owning that part of the
> stack too. Lucene in itself is a good solution, a bit underdeveloped lately
> (last big update is 3.0.3 and almost 1 year old already), but serves the
> purpose quite well. Armed with our knowledge on how it is used on RavenDB
> there are many optimizations that would certainly impact us dearly.
>
>
>
> While rewriting and stabilizing the whole Lucene functionality may take 1
> year or so. In 1 or 2 months of work we could be able to at least optimize
> the common paths.
>
>
>
>
>
> From: Ayende Rahien [mailto:notifications@github.com]
> Sent: lunes, 12 de enero de 2015 01:20 p.m.
> To: Corvalius/ravendb
> Cc: Federico Andres Lois
> Subject: Re: [ravendb] Aggresively inlining methods and properties with
> millions of executions when bulk uploading 25000 freedb records. (19a94a4)
>
>
>
> We have a lot of problems with Lucene, and we really want to switch from
> it.
>
> The problem is that it is very much a non trivial task to do.
> I spiked it using Voron, but we are currently stuck with how to handle
> updates efficently.
>
>
> *Hibernating Rhinos Ltd *
>
> Oren Eini* l CEO l *Mobile: + 972-52-548-6969
>
> Office: +972-4-622-7811 *l *Fax: +972-153-4-622-7811
>
>
>
> On Mon, Jan 12, 2015 at 6:13 PM, Federico Andres Lois <
> <mailto:notifications@github.com> notifications@github.com> wrote:
>
> > Dont get me started on Lucene :P
> >
> > -
> > Reply to this email directly or view it on GitHub
> > <
> <
> https://github.com/Corvalius/ravendb/commit/19a94a4713a21198d410f8cb9af3cf2
> 949334088#commitcomment-9243883>
>
> https://github.com/Corvalius/ravendb/commit/19a94a4713a21198d410f8cb9af3cf29
> 49334088#commitcomment-9243883>
> > .
> >
>
> -
> Reply to this email directly or
> <
> https://github.com/Corvalius/ravendb/commit/19a94a4713a21198d410f8cb9af3cf2
> 949334088#commitcomment-9243995> view it on GitHub.
> <
> https://github.com/notifications/beacon/AA_zowHd8_Rqu-jSjS01Th8beWnpvAGwks5
> ng-uSgaJpZM4DRNQw.gif>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Corvalius/ravendb/commit/19a94a4713a21198d410f8cb9af3cf2949334088#commitcomment-9244261>
> .
>


saw approval from jdm
at https://github.com/evilpie/servo/commit/d54a45a2b1e4fe5109d8292483e538252da240f1


merging evilpie/servo/window-proxy = d54a45a2 into auto


evilpie/servo/window-proxy = d54a45a2 merged ok, testing candidate = a227faa4


lgtm


Hi there.  This uppercase Q in the require is breaking builds on TravisCI.

Can you make it var Q = require('q'); instead? Thanks


So "Index" is `[]`?
RubySpec calls it `element_reference`, MRI calls it `aref`, HashNodes uses `GetIndexNode`, etc.
Behavior of Array#[] is like index + slice so I don't know what would be a good name based on semantics anyway.


haystack is never incremented. I suggest using haystack[ret] instead of *haystack.


OK for me.


From MS.Net Calling Dispose will result in a call to Dispose(bool) that is virtual and will allow other classes to dispose.

From Ms.Net calling close should be the smae a calling dispose.


Use size_t instead of int.


you mean because its active for PEPMASS?

Technically, \t is not allowed, but I guess we have seen it in the wild?

I don't see a reason not to enable this relaxation (except for the performance penalty).


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)


This is true


LGTM


@bors: retry


I think you could cache the actor system name in a private field


This is my first commit

![ic_launcher-web](https://cloud.githubusercontent.com/assets/8609449/5706752/1cfef762-9aa5-11e4-80e0-89b716578caa.png)


lgtm, feel free to land separately

nit: in the commit message, you say tuple, but I think you mean "struct with two fields"


Adding a comment here to say that this is just to fix a compiler warning, and maybe referencing the conditional check above would be helpful for future readers.


It would have been nice to leave the simulator mode methods in, but return false on get, and raise an exception on put. However, the tests inherited from the core test suite (`GatewayTest::testDefaultParametersHaveMatchingMethods`) do not allow this. Whether the tests are wrong and need overriding, or this is just the way the drivers are expected to behave, is up for debate. I suspect the tests are wrong and I may need to put these methods back in with appropriate exceptions if anyone attempts to use them, but we'll see.


84632652-PIUFans (kbenne) - Win64-Windows-7-VisualStudio-12: OK (2176 of 2190 tests passed)

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


Dead code.


This is wrong. strspn("b", "ab") gives the wrong result, 0 instead of 1 because it stops too early. It needs to check the entire accept string before deciding this character in haystack isn't in accept.


Dispose should be safe to call more than once and never throw in finalization. Right now Ms.Net will only throw on Dipose in 2 very specific cases. An one of those will be in the GC Thread... We should not try to copy this behavior.

Close and Dispose should never throw....


Cool thing about looking through the commit log for this repo is how the colors of yours and cala's avatar pics are complementary to each other.


This is actually a good reason for hewing to the legal line as opposed to implementing a supra-legal moral line. We can't help but discuss some matters privately, because not all information is public and we have to respect that. The discussions in the violations repo are private, yes. They are also kept to a minimum because they only bear on questions of legality.


I had started messing around with it myself back in the day. The idea I had in mind at the time was to check the adjacent subsectors' floor height difference  as *n* -- if all differences were less than a predetermined value  (I think I chose anything < 8) the water effect would activate at the height of the shortest difference - 1. if the difference was 0, the water effect shared the same height of the water texture. I don't remember if the code at the time did any clipping, but it worked in MAP02 pretty well.


y Jhoni?


Agreed!  https://github.com/wordpress-mobile/WordPress-iOS/commit/0a089f652302d17d7877956379ae365d38773129


Hey @jamesaimonetti can you please confirm this is in the *kazoo-R15B-3.18-50.el6.x86_64.rpm* build?


lol wut


Built the 3 icon middle section and the footer on my own before the walkthrough


* No está la licencia todavía
* Los programas en shell complejos te acaban volviendo por detrás y mordiendo en el c*lo.
* Los tests están mal organizados, hay un .tar.gz, un directorio dentro de un directorio... Si se trata de descargar cosas, un cliente lo puede hacer directamente desde git


Hi,

Had to withdraw index.php when url is not rewritten.

Regards


I think ideally these strings would be some sort of constants/enum, but that can cleaned up later


@paulmillr care to chime in?


yes, good catch


sg_persist_resource is effectively unused (used only when logging). Since the cluster resource name is anyway logged with every message, this parameter can be safely dropped.


I was thinking more on the line of pushing the changes to 3.0.3 to release an optimized 3.0.4 (obviously the Lucene team must agree to it) and as I bet many of the things we are going to do are going to remain in one way or another in 4.0.x, they can even port the changes into the 4.0.x branch without too much trouble…

We can even focus on things that are to be present in 4.0.x branch. Do you think that would be doable?

From: Ayende Rahien [mailto:notifications@github.com]
Sent: lunes, 12 de enero de 2015 01:45 p.m.
To: Corvalius/ravendb
Cc: Federico Andres Lois
Subject: Re: [ravendb] Aggresively inlining methods and properties with millions of executions when bulk uploading 25000 freedb records. (19a94a4)

We would lose the ability to update to the 4.x version, which is what the
lucene.net community is working hard on.

*Hibernating Rhinos Ltd *

Oren Eini* l CEO l *Mobile: + 972-52-548-6969

Office: +972-4-622-7811 *l *Fax: +972-153-4-622-7811

On Mon, Jan 12, 2015 at 6:31 PM, Federico Andres Lois <
 <mailto:notifications@github.com> notifications@github.com> wrote:

> Thinking out loud here. Why not fix Lucene a bit instead in the meanwhile?
> Meanwhile, as in, have the proper time to invest in owning that part of the
> stack too. Lucene in itself is a good solution, a bit underdeveloped lately
> (last big update is 3.0.3 and almost 1 year old already), but serves the
> purpose quite well. Armed with our knowledge on how it is used on RavenDB
> there are many optimizations that would certainly impact us dearly.
>
>
>
> While rewriting and stabilizing the whole Lucene functionality may take 1
> year or so. In 1 or 2 months of work we could be able to at least optimize
> the common paths.
>
>
>
>
>
> From: Ayende Rahien [mailto:notifications@github.com]
> Sent: lunes, 12 de enero de 2015 01:20 p.m.
> To: Corvalius/ravendb
> Cc: Federico Andres Lois
> Subject: Re: [ravendb] Aggresively inlining methods and properties with
> millions of executions when bulk uploading 25000 freedb records. (19a94a4)
>
>
>
> We have a lot of problems with Lucene, and we really want to switch from
> it.
>
> The problem is that it is very much a non trivial task to do.
> I spiked it using Voron, but we are currently stuck with how to handle
> updates efficently.
>
>
> *Hibernating Rhinos Ltd *
>
> Oren Eini* l CEO l *Mobile: + 972-52-548-6969
>
> Office: +972-4-622-7811 *l *Fax: +972-153-4-622-7811
>
>
>
> On Mon, Jan 12, 2015 at 6:13 PM, Federico Andres Lois <
> < <mailto:notifications@github.com> mailto:notifications@github.com>  <mailto:notifications@github.com> notifications@github.com> wrote:
>
> > Dont get me started on Lucene :P
> >
> > -
> > Reply to this email directly or view it on GitHub
> > <
> <
>  <https://github.com/Corvalius/ravendb/commit/19a94a4713a21198d410f8cb9af3cf2> https://github.com/Corvalius/ravendb/commit/19a94a4713a21198d410f8cb9af3cf2
> 949334088#commitcomment-9243883>
>
>  <https://github.com/Corvalius/ravendb/commit/19a94a4713a21198d410f8cb9af3cf29> https://github.com/Corvalius/ravendb/commit/19a94a4713a21198d410f8cb9af3cf29
> 49334088#commitcomment-9243883>
> > .
> >
>
> -
> Reply to this email directly or
> <
>  <https://github.com/Corvalius/ravendb/commit/19a94a4713a21198d410f8cb9af3cf2> https://github.com/Corvalius/ravendb/commit/19a94a4713a21198d410f8cb9af3cf2
> 949334088#commitcomment-9243995> view it on GitHub.
> <
>  <https://github.com/notifications/beacon/AA_zowHd8_Rqu-jSjS01Th8beWnpvAGwks5> https://github.com/notifications/beacon/AA_zowHd8_Rqu-jSjS01Th8beWnpvAGwks5
> ng-uSgaJpZM4DRNQw.gif>
>
> —
> Reply to this email directly or view it on GitHub
> < <https://github.com/Corvalius/ravendb/commit/19a94a4713a21198d410f8cb9af3cf2949334088#commitcomment-9244261> https://github.com/Corvalius/ravendb/commit/19a94a4713a21198d410f8cb9af3cf2949334088#commitcomment-9244261>
> .
>

—
Reply to this email directly or view it on GitHub <https://github.com/Corvalius/ravendb/commit/19a94a4713a21198d410f8cb9af3cf2949334088#commitcomment-9244485> .  <https://github.com/notifications/beacon/AA_zo4RQKUhveVJ7opKjUkhBIMaPGf_Lks5ng_GUgaJpZM4DRNQw.gif>


By the way:  eslint gives me

```
js/lib/node.js
   65:4   error  'Node' is read only
   no-undef
```

Node is not a global variable.  I think it's treating this as if it were `node` (uncapitalized), because when I change it to `Noode`, I don't get the warning.

Is that a bug in eslint, or is it problematic for me to use the name `Node` here?


The "returning object" behavior is new behavior, so that can't be backported.

The documentation (sans the "returning object" part) could be backported however.


sizeof won't work for functions.


I created an "issue" for your crafting, go see it.


Use size_t instead of int. This can be implemented in terms of strcspn, btw.


can't


I know that there is no problem resolving that value at runtime. But it could be, that this value has a completely different meaning on a lower level API, e.g. returning a different sensor here. I'm not aware of such cases in real life, but that is what this Lint warning is about.


I think quotes should do it.


Hey @jleandroperez - I tried something like that (I think you meant width, not height above), and I still needed the conditional because the table row widths on iPads appear to be constant driven, but on iPhones are superview pinned.


I don't understand why we need to change the code :(


So it is a different module :-) - https://github.com/jquery/jquery/pull/1912

>  is "wrap" a full fledged module or it's still a "manipulation"?


So,  explain me the problem :/


cur_size = 0?


r+ rollup


@samccone this is based on prompt logic I reverted in fc9ac9d and now partially brought back, so it's how the prompt logic works in  beaker-solo. It can be improved but I just took it as it is and changed only the message.

@buob it reconnects after server crashed and restarted, but it reconnects to a new server instance which doesn't know anything about the previous ipython kernel session.


It should not. Privately extending an Android API with values not reserved for private uses would be a serious mistake of the vendor: it would mean that previously compiled applications would refer to the wrong sensor if the OS is upgraded to a version which now supports the new sensor whose identifier was misused. I can't believe this would happen, this is why I would prefer not to add overly defensive code to it.


the intent here is to remove the mapping that corresponds to the component's react-id from the :state-map map, correct?  if so, i'm about 42% sure that we still have a typo here, and what we really want is:
```clojure
          (swap! (get-gstate this) update-in [:state-map] dissoc (react-id this))
```


@aberke The [build job](http://build001-fmt.2600hz.com:8080/job/Kazoo/445/) has the commit and [the RPM build artifact](http://build001-fmt.2600hz.com:8080/job/Kazoo/label=CentOS-6-x86_64-R15B/) is 3.18.50 (which is in the [staging repo](http://repo.2600hz.com/R15B-staging/6/x86_64/Kazoo/origin/3.18/kazoo-R15B-3.18-50.el6.x86_64.rpm)). So you should be good to go with a 'yum update' if you have the staging repos configured.


no work :(


Na verdade não.

O seu teste estava testando se o `ActiveRecord` está funcionando.

O meu, se o **comportamento** esperado está ocorrendo.

Percebe a diferença?

Precisa prestar atenção se o que você está testando é pertinente, nesse ponto, temos de confiar que o pessoal que desenvolve o `ActiveRecord` testou que passar todos os parâmetros salva o registro... :grimacing:


Actually, I thought about that one too, but then it hurts your ability to use more sucint function names, etc., whereas you can type anything you want in the description. Flexibility. The code that parses the function description involves several pipes and writing this inside `log` is not very intuitive.

Anyway, it's OK. I will add the function to this file and remove the other.


IMHO it's overly splitted. For example, ego data could be combined with zones/cities because they are about lat/lon coords. Alien races, missions and trajectories are also quite tied together.  Scripts and terrains are parts of mapgen, they should be together.


This had been pulled as 42bdc31921e1c758a4062a2e477214f2697d5aec


Might have to be a bit of refactoring to make it happen, but I don't have a problem with the code changing.


A friend once suggested that a regex should always have a name -- IE be pulled out to a method. So here you could do something like ...

```
if chunk =~ starts_with_number_sign then Heading.new.render(chunk)
```

and...

```
def starts_with_number_sign
  /#+\s.*/
end
```


:+1: I was just going to submit a PR for this.


It depends on your conventions...

1) The DNF's conventions say that you shouldn't do "from foo import bar" unless "bar" is used more than five times. We also sort the imports alphabeticaly.
2) And Pylint complains about:
```
C:  1, 0: Missing module docstring (missing-docstring)
C: 26, 0: Missing class docstring (missing-docstring)
W: 29, 4: __init__ method from base class 'Plugin' is not called (super-init-not-called)
```
3) Also the naming of variables is little bit confusing. E.g. "for pkg in package"...
4) And of course, the tests are missing :-P

Other than that it seems ok.


+++ Vitaly Puzrin [Jan 12 15 00:59 ]:
>https://developer.mozilla.org/ru/docs/Web/JavaScript/Reference/Global_Objects/unescape
>
>Don't use `unescape` it will increase existing madness level

This doesn't change anything from before, since we were always using it.
I hadn't realized it was deprecated.  I'll replace it with decodeURI.

>Example: http://spec.commonmark.org/dingus.html?text=[foo]%28%25C3%29

What, exactly, is this example supposed to illustrate?  commonmark.js
currently turns it into

http://spec.commonmark.org/dingus.html?text=[foo]%28%25C3%29

both with unescape and with decodeURI.

>Also that's a reason why you don't have exception (and probably why you have strange results, as described in issue).

What I reported in the issue is that I got *correct* results using this.
I only got wrong results when I switched to `node-url`.


```elseif``` wasn't preferred?


> The "returning object" behavior is new behavior, so that can't be backported.

Well, I'd say it depends on the point of view. It fixes the bug that it's not possible to implement a custom `inspect` functions that honours the option `color: true/false`.

That's my two cents, I understand your POV too. If it did not take two years to release v0.12, I would absolutely agree with you.


That's 100% problem with your config. I guess, this line https://github.com/jgm/CommonMark/blob/master/eslint.json#L3

compare with my one https://github.com/markdown-it/markdown-it/blob/master/.eslintrc#L3

It's better to configure rules more fine for your coding style. With defaults you will not use many cool eslint features.


+1, worked for me as well!  Ubuntu 12.04 LTS, Ruby 1.9.3p550 (2014-10-27) [x86_64-linux] Brightbox, PG 9.1.


I'd rather this code be refactored so that identical code lives in a named function (the same code is repeated a couple of lines later).


xlizard: Ihr benutzt die gleiche ipv6, wie wir (westpfalz1).

Könnt ihr das ändern?
siehe auch: http://wiki.freifunk.net/Icvpn

Euer Teck-Kontakt müsste dazu eigentlich auch eine email bekommen haben.


Hi,
die einzelnen Fälle sollten hier mit einem Kommentar versehen werden. Ich verstehe beim Betrachten des Source-Code nicht, was die Anfragen genau bedeuten. Am besten du definierst was für groups es geben kann.


C'est aussi le cas des dossiers si je ne me trompe pas ?


This is a good first shot and covers a lot of cases. The challenge will be when there's nesting like `this is *empasized while **this is strong and em**.*`


you need to uninitialized this too, maybe you should put it in a RAII class so you can be sure it is always cleaned up properly


What about `spec.run.group` or `spec.run.set`? Simple and concise. You are right `spec.suite` is not good, I just wanted to use the word suite which is cool.


pourtant je l'ai mis juste en dessous o_O tes sur que tu l'as pas :p ?


TARGET is --device|--emulator|--target=foo

I'm assuming --list --target=foo doesn't work.

You'll need to change this, probably split TARGET into DEVICECLASS|--target and then reference DEVICECLASS...


hi guys :D


This sounds like some Gordon Ramsey situation ... CHOP! CHOP! CHOP! CHOP!

How about this...

```
chunk.reverse.chop.chop.chop.chop.chop.prepend(">5h/<").reverse.prepend("<h5>")
```

Becomes something like this...

```
"<h5>" + chunk[5..-1] + "</h5>
```


Die Frage ist, ob hier eine Exception nicht sinnvoller wäre. Hier solltest du einmal überprüfen was die Konverter von JavaFX machen. was passiert z.B. wenn man im CSS was falsches einträgt.


You could give `#wrap` a `min-height` of `90vh` or something, then just not position the footer (let it come after in the normal box model).


Cool. Maybe someday. :smile:


r=nikomatsakis


Agora entendi o motivo :+1:


no, because aria-label replaces the accessible label, you don't want to do that. I would add SR text:
```<h3 class="title"><span class="sr">Error: </span><%- errorTitle %></h3>```
Bonus if you translate it.


so it needs -DUSE_SDL2 flag?


Wenn du schon ein Int zurück bekommst dann musst du nicht auf Double überprüfen. hier kannst du auch result.intValue() nutzen


Correct. It will run twice because that's two different tests. If the latter fails, the whole test should fail.


Is there a reason this doesn't appear after ubuntu?

(we should probably consider being consistent and actually alphabetizing these things... -- looking at windows* ...)


we can remove all words. since i didnt get into utilizing nltk i removed it and all references


@mjhea0 ^


Insgesamt bin ich mir nicht sicher, ob allgemein bei nicht konvertierbaren Input nicht besser eine Exception geschmissen werden sollte. Hier einmal überprüfen wie der Standard das macht


Try a little string concatenation here rather than `.insert` twice. And if you know there's just a single `#` at the front you can use the bracket method which I think is a little easier to reason about than gsub:

```
"<h1>" + elem[1..-1] + "</h1>"
```


OK


Looks good


rax retry tests


I love the idea of this test, but I'd break it into 7 separate tests. Particularly the paragraph one is a different idea than the headers.


Umm... why?


Many thanks!


:) nice man


Let's undo this change because selUser.getOrganizations() is already only returning explicit associations to organizations


Why remove the comment (merge it with the above one)?


Add replace method for multiple nodes.


all tests pass:
success: http://build.servo.org/builders/linux1/builds/518
success: http://build.servo.org/builders/linux2/builds/516
success: http://build.servo.org/builders/android/builds/554
success: http://build.servo.org/builders/mac1/builds/735
success: http://build.servo.org/builders/mac2/builds/735


why not using the Process component for that ? proc_open has many quirks to implement a cross-platform code, and you don't cover them


Es gibt ziemlich viele Codezeilen in deinen Converter die sich wiederholen. Ich denke hier wär es Sinnvoll mal eine Klasse LessStyleConverter zu erstellen:
LessStyleConverter<T, U> extends StyleConverter<T, U>

Davon sollten dann alle Converter erben. Ich habe gesehen, dass du bereits einen MathematicsConverter gemacht hast. Das finde ich sehr gut! Der sollte dann auch vom LessStyleConverter erben.


* El gruntfile es incorrecto, y además tenéis que definir la tarea para test; grunt es un gestor de tareas
* No hay nada de código *todavía* y eso qeu el código está ya en el repo correspondiente. A estas alturas deberíais haber integrado ya ese código, hacer tests para ese código (el que está en el repo de la UGR)
* P3 no es P3, es un hito en el proyecto, como tal tiene que estar en el directorio principal, no copiar cosas de un directorio a otro. Además, no habéis conservado la historia del proyecto original: el código está ahora con vuestros nombres y es imposible sabe cuál es realmente vuestra aportación.
* Tenéis un dockerfile, pero no lo usáis para prueba. Estáis lanzando un shell con node. En general, no da la impresión de que sepáis lo que estáis haciendo.


Redundant `.ToList()`.


`prender` is kind of a funny method name. At first I thought "prerender" but I'm guessing you meant "paragraph render". We're already inside a paragraph object, so just "render" would mean "paragraph render".

You really have three ideas in this method: render strong tags, render emphasis tags, and render paragraph tags. Make that more clear by breaking each out into its own method.


That's ok, no worries :)

No problem - I'm currently using it in a proof-of-concept myself and don't have any stability expectations :) I'm not that familiar with how the CLJS compiler works under the hood so am not entirely sure whether the fixes I've made are the right thing to do (particularly 9db779d, regarding macro compilation -especially in light of your [1] reference).

Btw, really like the way that you've enabled programmatic access to the CLJS build - thanks! I'm currently using it in [Phoenix](https://github.com/james-henderson/phoenix) - a library I've been working on to help with configuration of systems based on Stuart Sierra's Component library. Shadow-build has allowed me to build a CLJS compiler 'component' (code [here](https://github.com/james-henderson/phoenix/blob/8e3494a68708d7ac9d7a71f32c3a8c440cea6bc0/modules/cljs/src/phoenix/modules/cljs.clj), if you're interested, or `lein new phoenix-webapp <app-name>` is a lightweight lein template)

Cheers,

James


This code is the only regression, it always was .NET 3.5 compatible before :) (Smuxi needs that)


Google image replacement


84694650_EnableCodeCoverage (lefticus) - x86_64-MacOS-10.9-clang: OK (1584 of 1584 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)


Conventions say `public` fields are grouped and separated from non-`public` fields, which are also grouped.


Fixes are here... https://github.com/mono/mono/pull/1247


> This doesn't change anything from before, since we were always using it.

No. `unescape` also consume latin encoding, that is invalid. Deprecation, is not a problem - you can use it to convers utf16 <-> utf8 for example. But it's written everywhere, that unescape should not be used with url.

> What, exactly, is this example supposed to illustrate?

Look at link `href`. `%C3` become `%C3%83`. I think, that adding one more byte is strage. Can't comment `node-url`, didn't checked.


val = Math.sin(new Double(matcher.group(1)) * (Math.PI / 200));

^^Hier finde ich die Schreibweise nicht so gut. Besser:
val = new Double(...)

Das führt auch intern dazu, dass ein Double Objekt weniger erzeugt wird und ist somit etwas Performanter


WorkingWaterfronts --> What's Fresh


fast-forwarding master to auto = a227faa4


It is, but I may use it soon again - pedigree initialization became noticeably slow and I sped it up in a few places, but was planning to do a few more things. Will remove all these timer lines together from all files at that point.


@ziogaschr are you using the beta that supports swift?


Is there a reason for the condition change?


Actually you can override resource definitions in your own project, as described in the issue that originally requested this change: [GPJQUERYUI-13](https://jira.grails.org/browse/GPJQUERYUI-13?focusedCommentId=68393&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-68393)

    modules = {
       overrides {
          'jquery-ui-dev' { dependsOn 'jquery-dev', 'jquery-ui-theme' }
       }
    }

However, I agree that using jquery-dev instead of jquery in this case is more convenient.


> the answer is "yes" in cmake modules

that is actually not 100% correct, usually the "_ROOT" var is used as a preferred hint similar to `CMAKE_PREFIX_PATH`.


In der neusten Maven Version sollte es reichen folgende Parameter (so wie den UTF-8 Parameter) zu setzten:

<maven.compiler.source>1.8</maven.compiler.source>
<maven.compiler.target>1.8</maven.compiler.target>

Dann muss man nicht extra das compiler plugin definieren.


all in small case for key: Clip => clip


suggest creating a `/public/admin` directory and moving this (and the `edit-cps-app`, etc.) there. then, change line 142 of `pegasus/router.rb` to include `'/admin', '/admin/*'` in the list of protected directories.

This way you won't need line 2 and 3 in every file - we should have everyone start grouping admin stuff into this common location.


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)


learnLevel, spCost....


full name fo key: params => parameters


Yeah. Just run in source directory:

    cmake -DUSE_SDL2=1 .
    make


First, I really dig how there are 20 commits in this repo -- good job taking small steps.

I'm no RegEx wizard, so `/\A\W\s/` takes me a minute to think about. "It starts with a non-word character  then a space". I think that's a bit ambiguous. Don't we want to make sure it starts with an asterisk? How about this...

```
def asterisk_list_marker
  /\A\*\s/
end
```

Then in this spot...

```
list_items = lines.map do |line|
  line.gsub(asterisk_list_marker, '<li>') + '</li>'
end
```


Shame that this fails on the first time the hook runs, it seems less fiddly than shelling out


Btw, already found an issue with text input not working. Something wrong with iconv.


But `spec.ok` wil output something, so we are going to have two outputs for the same test.


Is this TODO resolved?


These are good, simple, straightforward tests.


r=michaelwoerister


 - __warning__: [include/chaiscript/dispatchkit/type_conversions.hpp:222](../blob/31ef683cedd35fb0ff10094f7c7649b97832b86d/include/chaiscript/dispatchkit/type_conversions.hpp#L222) Member variable 'Type_Conversions::m_mutex' is not initialized in the constructor. (uninitMemberVar)
 - __style__: [include/chaiscript/dispatchkit/boxed_number.hpp:587](../blob/31ef683cedd35fb0ff10094f7c7649b97832b86d/include/chaiscript/dispatchkit/boxed_number.hpp#L587) 'Boxed_Number::operator=' should return 'Boxed_Number &'. (operatorEq)
 - __warning__: [include/chaiscript/dispatchkit/proxy_functions.hpp:179](../blob/31ef683cedd35fb0ff10094f7c7649b97832b86d/include/chaiscript/dispatchkit/proxy_functions.hpp#L179) The class 'Dynamic_Proxy_Function' defines member variable with name 'm_arity' also defined in its parent class 'Proxy_Function_Base'. (duplInheritedMember)
 - __style__: [src/chaiscript_stdlib.cpp:18](../blob/31ef683cedd35fb0ff10094f7c7649b97832b86d/src/chaiscript_stdlib.cpp#L18) The function 'create_chaiscript_module_chaiscript_stdlib' is never used. (unusedFunction)
 - __style__: [src/main.cpp:170](../blob/31ef683cedd35fb0ff10094f7c7649b97832b86d/src/main.cpp#L170) The function 'get_eval_error' is never used. (unusedFunction)
 - __style__: [src/main.cpp:137](../blob/31ef683cedd35fb0ff10094f7c7649b97832b86d/src/main.cpp#L137) The function 'help' is never used. (unusedFunction)
 - __style__: [src/main.cpp:217](../blob/31ef683cedd35fb0ff10094f7c7649b97832b86d/src/main.cpp#L217) The function 'myexit' is never used. (unusedFunction)
 - __style__: [src/main.cpp:159](../blob/31ef683cedd35fb0ff10094f7c7649b97832b86d/src/main.cpp#L159) The function 'throws_exception' is never used. (unusedFunction)


the route will stay the same as the creator / getter:
POST /render
GET /render/<renderId>
DELETE /render/<renderId>


I see no reason to move this *or* change its formatting.


r+ rollup

Thanks!


Yep, I understand that. I just think this is one of the functions people don't need to see the source code when they are trying to understand the test, as the function name is very descriptive. So it's better to put in the end so we don't "force" them to read something they don't need to in advance.


return an status_code, not 400 ...
take a look on : http://fr.wikipedia.org/wiki/Liste_des_codes_HTTP


This would explain the problem I saw when I tried to upgrade to the latest Om. I didn't have time to dig in at the moment, so I aborted the upgrade temporarily.


This is a semantic argument, but in the [docs](http://docs.astropy.org/en/latest/modeling/compound-models.html) I consistently refer to the `'|'` operator as "composition", and the `'&'` is either "join" (as we've called it before) or "concatenate".  Most confusingly here is that you're calling `'|'` "concatenate".  I don't think there's a right answer here since I don't know if there is really a good name for what it doing (the closest I can think of is "cartesian product" which it is when you consider models as functions).  I started calling it "concatenate" just because of the use of the '&' operator for concatenating strings in some languages, and because if the inputs and outputs of each model are tuples, this requires concatenating the inputs, and returns a concatenation of the outputs.  In any case, whatever names we use we should try to settle on something consistent to avoid confusion.


Is there a specific reason why the filter functions only receive the edge value and vertex value and not the whole Vertex/Edge? For edges it may be useful to filter by source or destination...


Looks good, should go upstream.


return nothing. A DELETE has non content response, just a status_code.


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

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


very dangerous to give the possibility to delete alls ...
don't expose it.


Looks good


r+


I think I've mentioned this before, but one interface that is currently sorely missing from `_CompoundModelMeta` is the ability to create a new compound model directly from an already fully realized ExpressionTree (rather than having to build it up piecemeal one operator at a time).  In addition to being more efficient in terms of execution, it would probably help simplify a lot of this code.

It's been kind of low priority since things work without it, but it's something I'd still like to improve sooner rather than later, and I think would probably help here, right?


return a status_code != 400
look on html status code to get the message: not found


I'm outta of ideas here, let's use `set` and we can refactor it later.


This line of code is trying to do too much. Break out each component to its own method. Maybe like this:

```ruby
def render_strong(para)
  para.gsub(/\*\*(.+)\*\*/,'<strong>\1</strong>')
end

def render_emphasis(para)
  para.gsub(/\*(.+)\*/,'<em>\1</em>')
end

def render_ampersand(para)
  para.gsub(/\&/,'&')
end

def render_paragraph(para)
  para.gsub(/\n(.+)\n/,'<p>\1</p>')
end

def word_format
  stronged = render_strong(paragraph)
  emphasized = render_emphasis(stronged)
  ampersanded = render_ampersand(emphasized)
  @paragraph = render_paragraph(ampersanded)
end
```


Lovely. Is there so much errors?


same as DELETE, the update will be on the same route /render/<renderId>


http://shenkarlab.github.io/pensyaTeam/ <http://shenkarlab.github.io/pensyaTeam/> 		לינק

> On Jan 12, 2015, at 09:50, Mushon Zer-Aviv <notifications@github.com> wrote:
>
> @orpgol <https://github.com/orpgol> sure, put it up on Github pages and send me the link
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/shenkarlab/pensyaTeam/commit/b1e9c13e01b2b6d367fc15c3a2833cd7627e7dfa#commitcomment-9237155>.
>


I think that HEAD requests should be used to search for the live edge. In this case the head request is supposed validate the time sync, but no single request can do that. When this succeeds it could be a segment anywhere in the availability window.
I agree with Mikhail that this HEAD request should be removed.


@rmutter so, this isn't so great. We now resize the photos on every save (this started happening when we switched to DRF), and we also don't retry cloudfiles. I'm putting it in so we can move forward with something that works. Ideally, I would imagine photo resizing would happen client-side, and server wouldn't actually touch media files, which could be uploaded directly to cloud files.


now moved them to our stylesheet but seems to prioritize iconic's entries


Went with what I found in the old codebase... I will pick a proper name.


You might have to move them down or make them more specific.

On Mon, Jan 12, 2015 at 12:28 PM, Daniel <notifications@github.com> wrote:

> now moved them to our stylesheet but seems to prioritize iconic's entries
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/arenahq/ervell/commit/3132bfeb06efa2c90c73f7fc570537fc5b2ce927#commitcomment-9245324>
> .
>


you're a dick


Are you sure about the arm_ prefixes? I still see them.


Daaamn, I love the idea of using `count_hashtags` -- very crafty without losing clarity.

This line of code, though ... is doing too much. Looking at `text_splitter` I don't see what it's doing for you -- split cuts on space then you join with a space, right? I must be missing something.


Can you push to reviews.freebsd.org for wider review.


In SpeechRecognition.java missing ' nlpCommand.setReceiver("app.commands.interpreter.nlp") '


Faudra qu'on regarde les performances du chat un de ces jours, car sur xnova, c'était une vraie pompe à bande passante quand c'était implanté sur la vue générale. ;)


This is new file


this is another one


rax retry tests


loopd variable export was missing for early stages - now it should be ok.

btw: static binary (qemu) should run the same across all systems - so if we add one more system later, let's store one binary to our repo and use it for all systems.


r+ rollup


Use the OAuth token generated by the `hub` command to login with `curl` to obtain the list of repos of this user or org.


To be removed build.options and manifest.mf in speech-recognition plugin


Just read the latest updates on the xbian forum. We've got a (already working) debian vps.


thanks for explanation, but it gives errors:

>adem@adem:~/git/qfusion-sdl/source$ cmake -DUSE_SDL2=1
-- The C compiler identification is GNU 4.8.2
-- The CXX compiler identification is GNU 4.8.2
-- Check for working C compiler: /usr/bin/cc
-- Check for working C compiler: /usr/bin/cc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working CXX compiler: /usr/bin/c++
-- Check for working CXX compiler: /usr/bin/c++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Found OGG: /usr/include
-- Found VORBIS: /usr/include
-- Found THEORA: /usr/include
-- Found CURL: /usr/lib/x86_64-linux-gnu/libcurl.so (found version "7.35.0")
-- Found ZLIB: /usr/lib/x86_64-linux-gnu/libz.so (found version "1.2.8")
-- checking for module 'freetype2'
--   found freetype2, version 17.1.11
-- Found Freetype: /usr/lib/x86_64-linux-gnu/libfreetype.so (found version "17.1.11")
-- Found JPEG: /usr/lib/x86_64-linux-gnu/libjpeg.so
-- Found PNG: /usr/lib/x86_64-linux-gnu/libpng.so (found version "1.2.50")
CMake Error at /usr/share/cmake-2.8/Modules/FindPackageHandleStandardArgs.cmake:108 (message):
  Could NOT find SDL2 (missing: SDL2_INCLUDE_DIR SDL2_LIBRARY)
Call Stack (most recent call first):
  /usr/share/cmake-2.8/Modules/FindPackageHandleStandardArgs.cmake:315 (_FPHSA_FAILURE_MESSAGE)
  cmake/FindSDL2.cmake:87 (find_package_handle_standard_args)
  CMakeLists.txt:87 (find_package)
-- Configuring incomplete, errors occurred!
See also "/home/adem/git/qfusion-sdl/source/CMakeFiles/CMakeOutput.log".

CMakeOutput.log: http://pastebin.com/7TSgqTem


lgtm

tested ok in: Chrome 39.0.2171.95, Safari 8.0.2 (10600.2.5), Firefox 34.0.5


Only  that the TFTP related --include argument is omitted in this example


Unrelated to this line, I like the idea of namespacing files. But make it `lib/chisel/ordered_list`. If you really want to do it right you use a module to namespace the class like this:

```
module Chisel
  class OrderedList
    # .. code
  end
end
```

Which is instantiated like this: `Chisel::OrderedList.new`

Back on this line, the `line.length` is technically a bug but you get away with it. If I have the string `hello` then `line.length` is five. So slicing `(3..line.length)` would be equivalent to `(3..5)` but the index of the `o` is actually `4` -- you're asking for more characters than exist. Ruby let's you off the hook and just gives you what's there (you could ask for `(3..100)` and it won't complain).

To do it right you'd minus one from the length ... `(3..line.length -1)`. And that minus one is the inspiration for the "negative index" syntax: `(3..-1)`, which is what you should use here.


Np, let me clarify. I'm trying to do GGRC development work on a Windows machine. Before the switch to Ansible, I could do that with Chef (as you could see from the README instructions I've added). Given that Ansible cannot run on Windows, how can I still develop on Windows? Aside, of course, from running a VM just to run Ansible in it to create another VM for GGRC :)


Acho que nunca vi um teste para telas de listar...
Pensa que isso é uma funcionalidade muuuuito simples, exibir informações do banco. Melhor testar na mão mesmo, por isso removi o spec. :grimacing:

Em situações como esta de criar muitos usuários na mão, prefira sempre fazer algo com um loop, por exemplo, e não há necessidade de salvar o retorno do `create`... alias, `create` retorna só `true` e `false`... use `create!` para situações onde precise que retorne o objeto criado ou lance uma exceção se falhar.

Exemplo:
```ruby
5.times { |n|
  Usuario.create!(
    nm_usuario: "Banana #{n}",
    nm_endereco: "Pindamonhangaba",
    nr_cpf: "1234567891#{n}",
    nm_sexo: "M",
    nr_telefone: "5555555#{n}",
    dt_data: "2#{n}/08/1988"
  )
}
```

Exemplo (com nomes pré-definidos):
```ruby
["Banana", "Abacaxi", "Pipoca", "Pepino"].each_with_index { |u,n|
  Usuario.create!(
    nm_usuario: u,
    nm_endereco: "Pindamonhangaba",
    nr_cpf: "1234567891#{n}",
    nm_sexo: "M",
    nr_telefone: "5555555#{n}",
    dt_data: "2#{n}/08/1988"
  )
}
```


This buffer needs to be larger (`len*2+1`).
From `PQescapeStringConn` docs:
> `to` shall point to a buffer that is able to hold at least one more byte than twice the value of length, otherwise the behavior is undefined. Behavior is likewise undefined if the to and from strings overlap.


Cool!


They are called multiples times here to test that calling Close or Dispose more than once will not throw an exception....


well my personal idea would be that then you don't need to write separate
functions for migration and that you keep the logic for how to migrate
local to the class that cares about it. the interface would be
migrateFrom:(string) version toVersion:(string)version and anyone
migratable would handle that
 that said I will put it all in one place. look for a new pr shortly

On Mon, Jan 12, 2015 at 2:14 AM, Frederic Jacobs <notifications@github.com>
wrote:

> Why would it implement an interface? The goal is to have all the migration
> code in one place. Eventually, you could also use categories for
> migrations, if that helps. But I wouldn't put any additional code in the .m
> file itself.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/corbett/Signal-iOS/commit/9c81aab03d1e8bbb783bc55dcf6010a0f08dc675#commitcomment-9240070>
> .
>

--
Dr. Corbett Moran
Rockets @ http://www.spacex.com
Physics @ http://www.ICS.uzh.ch  <http://www.ICS.uzh.ch>
Dev @ http://circleof6app.com
Dev @ https://whispersystems.org
www.christinecorbettmoran.com


Change `is_identical` to identical`.


Because arm_ prefixes were also present in the GIC code, I removed all in 38894c0 (the subsequent commit)


In that case things should work then. We can test this in-depth later.


It looks like your editor is using a tab to indent rather than two spaces. Get that switched -- some people will freak out on you.

The `include?` here is a bit too soft of an expectation. Consider this example:

```
I was so over that joke. It's totally #playedout
```

This text does `include?` a number sign, but it's not a heading. Instead, try `start_with?`


Lol


This is not usefull in Jan 2015


setting to permissive to see if it'll even boot. Note: never booted on KitKat 4.4.4 with SElinux enforcing.


Does this modification change the behaviour of the function?

I'm asking because it looks like availableVms should not contain any terminated VMs [due to this code](https://github.com/malawski/cloudworkflowsimulator/blob/master/src/cws/core/WorkflowEngine.java#L116). Is there a case where that doesn't work as expected?


r+ nice!


all tests pass:
success: http://buildbot.rust-lang.org/builders/auto-mac-32-opt/builds/3117
success: http://buildbot.rust-lang.org/builders/auto-mac-64-opt/builds/3110
success: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-t/builds/3111
success: http://buildbot.rust-lang.org/builders/auto-linux-32-opt/builds/3102
success: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-t/builds/3102
success: http://buildbot.rust-lang.org/builders/auto-linux-64-opt/builds/3105
success: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-t/builds/3098
success: http://buildbot.rust-lang.org/builders/auto-linux-64-x-android-t/builds/3093
success: http://buildbot.rust-lang.org/builders/auto-win-32-opt/builds/2770
success: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-t/builds/2768
success: http://buildbot.rust-lang.org/builders/auto-win-64-opt/builds/1606
success: http://buildbot.rust-lang.org/builders/auto-win-64-nopt-t/builds/1599


fast-forwarding master to auto = 055cc2ee


Sure


To the end it goes.


r+


Why not `TestCase.assertFalse`?

https://docs.python.org/2/library/unittest.html#unittest.TestCase.assertFalse


Why not `testCase.assertTrue`?

https://docs.python.org/2/library/unittest.html#unittest.TestCase.assertTrue


Thx for the code!

I updated for my own use, took out a couple of errors in code, and added functionality to support PowerShell Credential objects.

You'll notice that I took out the scripts ability to connect to vCenter. That's just a personal preference, but it can be easily added back.

```powershell
function Set-GuestNetworkInterface {
<#
.SYNOPSIS
	Configures guest network interface on a virtual machine.

.DESCRIPTION
	This function provides the functionality to start an RDP session without having to type in the password

.PARAMETER vCenter
    Hostname or IP address of the vCenter server(s).

.PARAMETER VM
	Specifiies virtual machine name.

.PARAMETER Interface
	Specifies the network interface to modify.

.PARAMETER IPAddress
	Specifies the IP address for the network interface.

.PARAMETER SubnetMask
	Specifies the CIDR for the network interface.

.PARAMETER DefaultGateway
	Specifies the default gateway for the network interface.

.PARAMETER DNSServers
	Specifies the DNS servers for the network interface.

.PARAMETER GuestUser
	Specify the username to run the script inside the guest operating system.

.PARAMETER GuestPassword
	Specify the password to run the script inside the guest operating system.

.PARAMETER GuestCredential
	Specify the credentials to run the script inside the guest operating system.

.NOTES
    Name: Set-GuestNetworkInterface
    Author: Dean Grant
    DateUpdated: 1/12/2015
	LastUpdatedBy: Chris Arceneaux
    Version: 1.1

.LINK
https://deangrant.wordpress.com/tag/invoke-vmscript/

.EXAMPLE
	. .\Set-GuestNetworkInterface.ps1

Description
-----------
This command dot sources the script to ensure the Set-GuestNetworkInterface function is available in your current PowerShell session

.EXAMPLE
	Set-GuestNetworkInterface -VM server1 -Interface Ethernet0 -IPAddress 10.0.0.5 -SubnetMask 24 -DefaultGateway 10.0.0.254 -DNSServers 10.0.0.1,10.0.0.2 -GuestUser DOMAIN\admin -GuestPassword supersecretpw

Description
-----------
Configures the network interface Ethernet0 on virtual machine server 1 with the IP Address 10.0.0.5/24, Default Gateway 10.0.0.254 and DNS servers 10.0.0.1, 10.0.0.2

.EXAMPLE
	Set-GuestNetworkInterface -VM server1 -Interface Ethernet0 -IPAddress 10.0.0.5 -SubnetMask 24 -DefaultGateway 10.0.0.254 -GuestUser DOMAIN\admin -GuestPassword supersecretpw

Description
-----------
Configures the network interface Ethernet0 on virtual machine server 1 with the IP Address 10.0.0.5/24, Default Gateway 10.0.0.254

.EXAMPLE
	Set-GuestNetworkInterface -VM server1 -Interface Ethernet0 -DNSServers 10.0.0.1,10.0.0.2 -GuestUser DOMAIN\admin -GuestPassword supersecretpw

Description
-----------
Configures the network interface Ethernet0 on virtual machine server 1 with the DNS servers 10.0.0.1, 10.0.0.2

.EXAMPLE
	Set-GuestNetworkInterface -VM server1 -Interface Ethernet0 -DNSServers 10.0.0.1,10.0.0.2 -GuestCredential $Credentials

Description
-----------
A PowerShell Credential object may be used instead of typical Username/Password

.EXAMPLE
	Set-GuestNetworkInterface -VM server1 -INT Ethernet0 -IP 10.0.0.5 -SM 24 -DG 10.0.0.254 -DNS 10.0.0.1,10.0.0.2 -U DOMAIN\admin -P supersecretpw -CR $Credentials

Description
-----------
Aliases are also supported.
#>

# Specifies  parameters required for the powershell session.
	param (
        [Parameter(Mandatory=$true)]
			[String[]]$VM,
		[Parameter(Mandatory=$true)]
		[Alias("INT")]
			[String]$Interface,
        [Parameter(Mandatory=$false)]
        [Alias("IP")]
            [String]$IPAddress,
        [Parameter(Mandatory=$false)]
        [Alias("SM")]
            [String]$SubnetMask,
		[Parameter(Mandatory=$false)]
        [Alias("DG")]
            [String]$DefaultGateway,
		[Parameter(Mandatory=$false)]
        [Alias("DNS")]
            [String]$DNSservers,
		[Parameter(Mandatory=$false)]
        [Alias("U")]
            [String]$GuestUser,
		[Parameter(Mandatory=$false)]
        [Alias("P")]
            [String]$GuestPassword,
		[Parameter(Mandatory=$false)]
        [Alias("CR")]
            [System.Management.Automation.PSCredential]$GuestCredential
    )

# Conditional logic to determine if Username/Password or Credentials were used.
If ($GuestCredential -ne $null) {
	$GuestUser = $Credentials.GetNetworkCredential().username
	$GuestPassword = $Credentials.GetNetworkCredential().password
}

# Conditional logic to determine if IP addresses or DNS server addresses are being modified or BOTH.
If ($DNSServers -eq $null)
	{
	"" + (Get-Date).toString('dd/MM/yyyy HH:mm:ss') + ": INFORMATION: Modifying IPv4 address settings for " + $Interface + "."
	$ScriptText = "New-NetIPAddress –InterfaceAlias Ethernet0 –IPAddress `$IPAddress -AddressFamily IPv4 –PrefixLength `$SubnetMask -DefaultGateway `$DefaultGateway"
	}
ElseIf ($IPAddress -eq $null)
	{
	"" + (Get-Date).toString('dd/MM/yyyy HH:mm:ss') + ": INFORMATION: Modifying DNS server addresses for " + $Interface + "."
	$ScriptText = "Set-DnsClientServerAddress -InterfaceAlias $Interface -ServerAddresses $DNSServers"
	}
Else
	{
	"" + (Get-Date).toString('dd/MM/yyyy HH:mm:ss') + ": INFORMATION: Modifying IPv4 address settings and DNS server addresses for " + $Interface + "."
	$ScriptText = "New-NetIPAddress –InterfaceAlias Ethernet0 –IPAddress `$IPAddress -AddressFamily IPv4 –PrefixLength `$SubnetMask -DefaultGateway `$DefaultGateway;
	Set-DnsClientServerAddress -InterfaceAlias $Interface -ServerAddresses $DNSServers"
	}

# Modifies the IP address settings and DNS server addresses for the interface.
Try
	{
	Invoke-VMScript -VM $VM -ScriptText $ScriptText -GuestUser $GuestUser -GuestPassword $GuestPassword | Out-Null
	}
Catch
	{
	"" + (Get-Date).toString('dd/MM/yyyy HH:mm:ss') + ": ERROR: Failed to modify " +  $Interface +  " with the message: " + $_.Exception.Message + "."
	}
Finally
	{
	"" + (Get-Date).toString('dd/MM/yyyy HH:mm:ss') + ": INFORMATION: Succesfully modified " + $Interface + "."
	}
}
```


This is in master in Jan 2015


Lo habéis presentado casi un mes tarde con la correspondiente reducción en la nota.
* No uséis wiki. Es imposible de evaluar y no está integrado en el proyecto.
* La clave de esta práctica es el despliegue automático. No se trata de crear sólo una jaula y ejecutar cosas dentro, sino de automatizar el despliegue en la jaula y la prueba de la misma.
* La pruebas son pruebas que ejercen la funcionalidad de la aplicación. Que, por cierto, no encuentro por ningún lado. ¿Dónde están los fuentes de la aplicación que está funcionando?


Good job my last commit was a failure I guess...


@PrincessNyanara fyi, for some reason, somehow, the settings file was getting set with an empty value on some machines and causing a proper crash.


@Dominic28 Ah damn :) Very good eye for the detail! Thanks! I'll push a fix in the next minutes.


Right


This is also in master in Jan 2015


This actually introduced a code highlighting bug on the official doc site:

![2015-01-12 at 11 44](https://cloud.githubusercontent.com/assets/724/5707689/6097d8b8-9a50-11e4-80d1-583ba8544a43.png)


At first look this appears to change the ```seg.availabilityStartTime``` from a client wall clock to a server wall clock time, but does not change ```seg.availabilityEndTime``` to a server wall clock time. These two times must be consistent (both server or both client).
I would prefer that the times were client wall clock times so that they can be directly compared to ```new Date()``` but as long as they are consistent and the usage is clear I am happy.


In master in Jan 2015 all undefined values are ignored


Remove the single quotes around the names of directories (libs and app).


Remove the single quotes around the names of directories (jniLibs and app).


Yep, it was an older style we transitioned from. Not sure why I had not updated it yet.


sslCipher should now be a shared_ptr to a std::string.  The reason is that some handlers may try to use it even after the client-side SSL structures have been freed.


Gimme more small commits, not one mega commit.

I like the concept of this `classify` method. Maybe a little bit of grouping is in order...

```
case
  when separated.is_header? then @text.render_header(separated)
  when separated.start_with?("* ") then @list.unord_list(separated)
  when separated.start_with?("1.") then @list.ord_list(separated)
  else @text.p_tags(separated)
end
```

Then the `render_header` would have the same kind of structure you have here but just for the headings. `is_header?` would return true if the first character is a `#`.


Glad I helped here too :)


home should probably route to root_url?


```php

{
	/* vars */
	var $type;
	var $attributes;
	var $self_closers;

	/* constructor */
	function html_element($type,$self_closers = array('input','img','hr','br','meta','link'))
	{
		$this->type = strtolower($type);
		$this->self_closers = $self_closers;
	}

	/* get */
	function get($attribute)
	{
		return $this->attributes[$attribute];
	}

	/* set -- array or key,value */
	function set($attribute,$value = '')
	{
		if(!is_array($attribute))
		{
			$this->attributes[$attribute] = $value;
		}
		else
		{
			$this->attributes = array_merge($this->attributes,$attribute);
		}
	}

	/* remove an attribute */
	function remove($att)
	{
		if(isset($this->attributes[$att]))
		{
			unset($this->attributes[$att]);
		}
	}

	/* clear */
	function clear()
	{
		$this->attributes = array();
	}

	/* inject */
	function inject($object)
	{
		if(@get_class($object) == __class__)
		{
			$this->attributes['text'].= $object->build();
		}
	}

	/* build */
	function build()
	{
		//start
		$build = '<'.$this->type;

		//add attributes
		if(count($this->attributes))
		{
			foreach($this->attributes as $key=>$value)
			{
				if($key != 'text') { $build.= ' '.$key.'="'.$value.'"'; }
			}
		}

		//closing
		if(!in_array($this->type,$this->self_closers))
		{
			$build.= '>'.$this->attributes['text'].'</'.$this->type.'>';
		}
		else
		{
			$build.= ' />';
		}

		//return it
		return $build;
	}

	/* spit it out */
	function output()
	{
		echo $this->build();
	}
}

```


delete these if they're not real pages


Added contact info


Ok, well, let's cross that bridge when we come to it :)


Is there not a way you could fix your tools to avoid these repeating issues in the first place?


il est moitié cracra son code en fait


100 of them, cba to adjust all that JDoc right now :-P


Created an empty repository


> The Manifest for Web Applications specification currently doesn't have anything to say on the topic of permissions, and therefore this doesn't change anything.

I'm not talking about permissions in the manifest, but in the user agent. The way it works today is that they run a stripped-down instance without the Chrome (or with a minimal, changed Chrome), but keep things like localStorage, permissions, etc. (Web Storage is then actually another security-related issue with this).

If this changed nothing, then the manifest wouldn't make sense. This changes app installation in a way that you can install Web apps in a way that works similar to "native" apps and next to the "native" ones on the home screen. For these other ones the assumption is certainly that one app can't access things I gave another app permission to, same as it shouldn't access sandboxed data the other app is storing.


r+ rollup


saw approval from alexcrichton
at https://github.com/oli-obk/rust/commit/b262ab5c4510fe251c50aebb6035b0b2c894830d


merging oli-obk/rust/json_non_string_key_maps = b262ab5c into auto


status: {"merge_sha": "10f9ee78e1e8ab9467bcb39d91a9738d8922d148"}


Obviously you should install SDL2 dev package


oli-obk/rust/json_non_string_key_maps = b262ab5c merged ok, testing candidate = 10f9ee78


Gotcha. Thanks.


This method is difficult for me to understand. Some notes/questions/thoughts:

* `doc.split` -- it seems like it should be someone else's job to split the document into chunks
* `doc_arr` -- I think that means array of documents. Just call it `docs` or `chunks` or something. We don't like to put the type of the data in the name.
* What's a `key`?
* `/#+\s.*/` try breaking regexs like this out into a method so you can name them, as I'll show below. Then you get `if key =~ starts_with_number_sign`
* I like the approach of counting the `#`s

```
def starts_with_number_sign
  /#+\s.*/
end
```


These links should become user-accessible as soon as we're confident that doing so won't expose our infrastructure in a bad way. Any concerns with removing the admin requirement now?


https://github.com/opscode/chef-docs/commit/18c3007b8bf3c86ec5bbc79be844f7715c7403b7

^^ that fixes the code highlighting issue.


Just like you installed png, jpeg, curl, freetype etc. development packages


84694650_EnableCodeCoverage (lefticus) - i386-Windows-7-VisualStudio-12: OK (1590 of 1590 tests passed)

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


sad


Part 1 of API creation for Learning Journal application.


nakon importanja ovog u bazu, potrebno je u tablici tiket postaviti default vrijednost na NULL na stupce "ocekvrdolaska" i "vrijemeposluz"


Ouch, yes please do not add any CR :sunglasses:


Lots of merge conflicts throughout this commit! I'm assuming you fixed them up in later commits, but it'd be good not to introduce them in the first place.


thanks once more, installation of "libsdl2-dev" on my Kubuntu helped


/** or /// style comments here so it shows up in Xcode?


it gets worse with numpy 19.1 i think this whole bit needs to be rewritten correctly before we go to 1.9.1... lots of fun ahead


r+ rollup


Closes #1


On Mon, 12 Jan 2015 08:24:34 -0800, Noel Power wrote:

> > This assumes that "net ads join" supports your newly added
> > osServicePack argument. I don't think that's a safe assumption to make
> > without a corresponding samba-client version dependency.
> >
> I don't think there needs to be a *hard* dependency, the information I
> got earlier is that it is possible to "include multiple packages in one
> single maintenance incident" by informing via "Bugzilla (NEEDINFO
> maint-coord@suse.de)"

Without a proper dependency, users could update yast2-samba-client
without the corresponding samba-client update, even if they go out via
a single maintenance incident. In such a case domain joins would break.

Therefore, I don't think the argument should be provided until a
samba-client release is available and version checks are in place.

> > "net ads join" appears to handle unsupported parameters as if they are
> > the "domain name" argument.
> >
> not sure what you mean, if you mean you think net ads might silently
> ignore the new parameter then I seem to recall that it didn't an
> triggered an error (but I could be wrong)

The osServicePack argument would be interpreted as
a domain name parameter and the join would fail.


@borshop: retry


`IsInstanceOnlyOpcode()` means that specific opcode can ONLY go through instance connection. Others can go through both cons. However, sending opcode on wrong connection leads to many strange bugs (like the one with initial spells). All opcodes connections should be verified by sniffs.


Please elaborate on (or remove) this "crashes" comment.


Maybe it's bug of the app


There are many examples about this.


@4wheels Maybe it's a bug of the app.


Was it really "expand" that you meant? "export" maybe?


Closes #8


This seems to have caused [a regression](http://djangoci.com/job/master-trusty-reverse/lastCompletedBuild/testReport/) when the test suite is ran in reverse order.


lel


Looks like its coming out wonderfully!


There seems to be a lot more in this commit than just fixing the stat stub, but the stat part looks good.


r+, my  hero!


Practice pulling


I remember this was a different path altogether. LOL.


Yea, saw that. Will fix now.


@nurkiewicz Did you mean `(tweetId): Optional.empty()`?


Don't use `AU`, it means that the patient was diagnosed with it at an unknown age, and we should not state that. See #1256


We should figure out some kind of logging/error system for failures. We'll inevitably run into some problems with SQL, so getting those errors logged will be helpful.


Fail commits #[15]


/** or /// style comments (here and elsewhere)


You should use `AU` if the age is unknown instead of `1`.


@XhmikosR: Sorry, I completely forgot that this commit required updating Transifex manually. I will see if I can do update it now.


Hi,

Thanks for an awesome package.

Wanted to let you know how we've put it to work: In particular we used it to wrap the [Flot JS](http://www.flotcharts.org/) library and create a charting package we plan on using in many of our dashboards.

[Github Repo](https://github.com/dcenergy/rflot)

[Project Home](http://dcenergy.github.io/rflot/)

Thanks again!


@jdhebden heads up - this is the wrong repository for these changes. This should probably go on our Award Force repository once it's ready to go. A reminder that Shift is open-source and will involve/build a community at some stage, so anything specific or required by Tectonic, cannot go in the repository.


Cool Stuff!

Nice to see shadow-build used this way. Now I only need to convice lein-cljsbuild and boot-cljs to also do this and the world is mine. :imp:

Just pushed some changes which should address the issues you had, only caching is broken at the moment. Will address that later.


Ok, got it.


Actually I have been thinking on reworking this XML to follow some standard, but I'll wait until all Ertheia changes are done, those weird names are retail names, can be changed later.


No worries.


IMO the example isn't that useful with no image nor tftp, right?


Damn, you're right. Fixing now.


84694650_EnableCodeCoverage (lefticus) - x86_64-MacOS-10.9-clang-Debug: OK (1584 of 1584 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)


yep shader version is more updated and build date is newer.
you can also try it
everything is good but wallpaper on homepage is flickering (apps and other stuffs were normal)


seems fine, except that problem you have already mentioned and no raw mouse input. Also Alt-Enter doesn't seems to work, and there is no app icon in KDE:
![qf-sdl](https://cloud.githubusercontent.com/assets/4707112/5708030/baf32a1c-9a9e-11e4-802f-646287095b45.png)

P.S. it even alt-tabs like Dota2 with a bit a delay on redrawing of KDE panels :D so my guess Dota2 for Linux uses SDL2 too.


I've been searching for this! Why did you :]


LOis


I am using [grape-roar](https://github.com/dblock/grape-roar), and this change breaks my code:

    module ProductRepresenter
      include Roar::JSON
      include Roar::Hypermedia
      include Grape::Roar::Representer

      link :self do |opts|
        request = Grape::Request.new(opts[:env])
        "#{request.url}"
      end
    end

`opts` parameter, supposed to include the environment, is empty `{wrap: false}`. Any idea why?


The plugin gives script error messages