That's almost where I was going.

Our new more complex spanners can incur a lot more score iteration than a once-per-leaf visit during a single depth-first pass. For example, I have a custom spanner class for my recent viola duet which, on every leaf formatted for the spanner, has to iterate forwards and backwards through some or all of the other leaves in the spanner, looking for combinations of attached indicators. It may even look to the leaves preceding and following the spanner in order to cover all of the formatting edge cases. That's O(n)^2 in the worst case. And more importantly it's not just slow but confusing to read.

This realization about spanner iteration now holds true to a lesser extent with our Glissando and TextSpanner classes: when formatting spanners, there's a good chance we're visiting each leaf covered by the spanner more than once in a single format-time session. And spanners may often have to do a lot of "pre-processing" logic to figure out exactly what contributions to add to each leaf (see Abjad's TextSpanner or ```_get_annotations()``` on Consort's StringContactSpanner: https://github.com/josiah-wolf-oberholtzer/consort/blob/master/consort/tools/StringContactSpanner.py).

If spanners could cache information they needed, a lot of this extra iteration and formatting complexity would go away. They should be able to cache their list of leaves (as they may span containers rather than leaves). They should be able to cache more complex data, like mappings of leaves to indicator inventories, or sequences of start/stop markup segments (for text-spanner-like formatting).

The right place to cache would be on a class I'll call LilyPondFormatSession - something that is created when you call ```format()``` and lives only for the duration of that operation. When you format a leaf with a spanner, the format session would be passed as an argument to that spanner's ```_get_lilypond_format_bundle()``` along with the leaf itself. The spanner can check if its leaves (and potentially other information) have been cached on the format session, and use that cached information rather than re-iterating over a chunk of the score. If the spanner doesn't have a cache, it will iterate its leaves and deposit the leaf sequence in the format session for when we format its next leaf.

This is not very exciting for working with Slurs, but for TextSpanner, Glissando and score-specific annotation-aware spanners like BowContactSpanner, StringContactSpanner and friends, it would clean things up a lot.


I wonder if this should also check the error message, just in case it is exiting for some other reason.


Wondering here the same thing as https://github.com/lonelyplanet/aws-mfa/commit/cf4bae7d32bd8a964d132ee5252752aaaea1faee#commitcomment-9156766


A misunderstand found in "Even the other denizens you encounter seem out of place here" when I try to find the missing lines in zh_CN.lang by comparing with yours.
Pls notice the “even” as conjunctive adverb in the words, and if you say sth is out of place, it means it's not in the right place but not another way of saying come from. I will check the other lines later.


Attempts to fix this error

warning: Error running '/lua/ui/game/gamemain.lua:OnSelectionChanged': ...d alliance\gamedata\mohodata.scd\lua\maui\bitmap.lua(47): Game object has been destroyed
stack traceback:
[C]: in function SetNewTexture' ...d alliance\gamedata\mohodata.scd\lua\maui\bitmap.lua(47): in functionOnDirty'
...orged alliance\gamedata\mohodata.scd\lua\lazyvar.lua(86): in function SetValue' ...orged alliance\gamedata\mohodata.scd\lua\lazyvar.lua(97): in functionSet'
...d alliance\gamedata\mohodata.scd\lua\maui\bitmap.lua(59): in function SetTexture' ...alliance\gamedata\mohodata.scd\lua\maui\checkbox.lua(43): in functionSetCheck'
...orever\gamedata\lua.nx2\lua\ui\game\construction.lua(439): in function CreateTabs' ...orever\gamedata\lua.nx2\lua\ui\game\construction.lua(316): in functionOnCheck'
...alliance\gamedata\mohodata.scd\lua\maui\checkbox.lua(45): in function SetCheck' ...orever\gamedata\lua.nx2\lua\ui\game\construction.lua(2280): in functionOnSelection'
...\faforever\gamedata\lua.nx2\lua\ui\game\gamemain.lua(419): in function <...\faforever\gamedata\lua.nx2\lua\ui\game\gamemain.lua:409>
[C]: in function SelectUnits' ...orever\gamedata\lua.nx2\lua\ui\game\construction.lua(1282): in functionOnClick'
...d alliance\gamedata\mohodata.scd\lua\maui\button.lua(133): in function <...d alliance\gamedata\mohodata.scd\lua\maui\button.lua:122>


```$basename```, ```$org``` and ```$db``` are text parameters, so they should be quoted.


Something in here should probably be async, shouldn't it? i.e. the completion might take some time to find? I'm tempted with the history api to make this method return an NSOperation so that we have some way to cancel in progress tasks. But I'd also like to just tie this to the list history api. I wonder if we can do something like:

locationTextField.textChangeDelegate = tabsViewController
tabsViewController {
    func textField(textField: UITextField, shouldChangeCharactersInRange range: NSRange, replacementString string: String) -> Bool {
       // When the text changes, fire off an async history request
       history.filter(searchString) { sites in
             viewController.data = sites
             locationTextField.autocompletion = findCompletion(sites, searchString)
       }
}


"Ruby" means the color not the material.


@9v-kz 出现什么错误？是否是最新版本？


I'll merge this change once I get a chance to test it with new reports. Thanks for finding the problem!


Why the second `skip_cache()` and the three checks?


Indiegogo campaign                                                                                Sent from my BlackBerry 10 smartphone on the Virgin Mobile network.                                                                                From: John F. LewisSent: Monday, January 5, 2015 12:29 PMTo: Orain/ansible-playbookReply To: Orain/ansible-playbookSubject: Re: [ansible-playbook] Add staff(at)orain(dot)org alias + one for me (5d553ae)Why? We have like 10 different emails for the same people now...

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


Thanks. `stapler-class` is a hidden form field in a large number of core UI components. The PR to resolve this, https://github.com/jenkinsci/jenkins/pull/1443, has been pending review by @kohsuke for two months now.


I think it's to test that it works even nested.


But why couldn't the others accomplish this goal?


This looks odd. If user supplies a path, it should be the full path - including the file name.

See other examples in Azure:
Import-AzurePublishSettingsFile –PublishSettingsFile "C:\Temp\MyAccountName-date-credentials.publishsettings"

Here you are taking folder name from user but generating your own file name - which seems odd.


Roger

Will send Some more patches tomorrow
On master right?

> Op 5 jan. 2015 om 18:37 heeft Scott Kingsley Clark <notifications@github.com> het volgende geschreven:
>
> Same reason :) We can add them now
>
> —
> Reply to this email directly or view it on GitHub.
>


Should we set developers to be `False` by default? Eventually we will, but we'll likely find out failures from developers that way. Then set to `True` for prod.


Please test this cmdlet to make sure RP errors and RDFE errors correctly get returned by cmdlet.


There is also no single other email address that goes to the people staff now does..


Thanks a lot.
I'll fix them soon.


Make sure you run FxCop / CodeAnalysis on this code - and see everything is green. We seem to be doing Crypto stuff - need to make sure it's all good.


Yeah, sounds like a good idea, I can do that.


Making the logging more explicit would probably be helpful. Something like:

    stbt.as_precondition caught a UITestFailure exception and will re-raise it as PreconditionError. The original exception was:


```$mc``` is a text parameter, so it should be quoted.


Fixed in a later unrelated commit, but I'll rebase anyway.


You've repeated this from `getParent()`, I'm assuming this entity is the child


Hi! Could you please create a pull-request at https://github.com/guard/guard-passenger ?


> Yeah for keybindings it'd work just fine but as @techee
> <https://github.com/techee> said for cases where we try and catch
> <Primary> with a mouse click we need something else. So we might as well
> go ahead and use the same <Primary> everywhere and not maintain the
> keybinding map in addition to it, especially as using <Primary> in
> keybindings seems really the easy part.
>
> BTW, this supposes that we should use <Primary> for button clicks and not
> <Control>, and while it seems to make sense I have no idea if it's the
> expected behavior on OSX :)
>
To confess, I have Ctrl and Command keys swapped by a software utility on
OS X, otherwise I'd get mad by hitting the wrong key all the time. So in
fact, what I'm trying to achieve is to get all Ctrls changed to Commands so
they can then be swapped once more and I get normal Geany keybindings.
Simple :-).

But yes, other editors behave the same way even for mouse clicks - a
feature similar to "go to tag definition" is invoked in XCode by
Command+click (i.e. Ctrl+click in my case).


Nope, there are always two entities involved in a breeding process, then the third entity is after a successful breeding.


it's sort of a hack to make the external-external-children sort after the external-children.


The example you quoted is for taking file as an input which is fine. In my case the output of the command is a file and hence we should not let user choose the file name.


I'm not sure there's any reason why a Play project needs a special syntax. When we build our Play projects with SBT we are able to treat Play projects just like any other project


...傳送門那部份我有注意到
但是我還是很難決定RUBY色要用什麼詞彙
畢竟作者在這邊用了一個不同的詞彙


So what entity is `getOtherParent()`?


Yes it runs


The other parent involved in the breeding process? (Again there are two entities to breed, a third is created after a successful breeding).


Well, it isn't time consuming the second time, true, but that first time is quite brutal.  If the second copy needs to be there on all machines there's probably nothing to be done for it.  I only bring it up because on my machine this step of setup took around 15 minutes (and actually failed on one machine the first time I tried it).


The parameters and name of the command are suggested by Rochak himself.


Stalls are unavoidable when a limited resource is over-subscribed. The best you can hope for in that situation is fairness.


This is an internal class, right. So could we keep access modifier as 'internal'. Can you please check the same for all other internal types.


@danmcd Yes, we should be using [[ everywhere.

@lotheac sorry for the latency on your followup.  AFAIK [[ is compatible with all uses of [, and extends it in some useful ways.  For one, it handles whitespace better, so you don't have to double-quote your variables, etc.  As I said, this is just for consistency.  You can change to [[ without any other changes.  Re: pkg list, sounds good, thanks for the clarification.  Re: pkg exit code, also, sounds good.  I'm satisfied.

Thanks for doing this!


This causes a deprecation warning for `@which`. It does not seem trivial to fix, since the `@which` macro is defined by metaprogramming that assumes the existence of a similarly named function, in `gen_call_with_extracted_types`


access modifier is 'internal'...


in the ios version previously there are tabs before the device names, it would be great to have the same look & feel.


Have already added a todo for the same as get vault is not yet implemented in PowerShell.


should be `newCandidateObj.election` not `newCandidateObj.incumbent_challenge`


> On Jan 5, 2015, at 9:06 AM, Brad Davis <notifications@github.com> wrote:
>
> Does the audio device get created if these are removed? It is worthwhile to remove the extra devices to trim down overhead.
>
>

No, and even if so, I would argue for minimal Vagrantfiles.  And to that end, removed a couple of lines from each Vagrantfile.


Why moving to another repository instead of just instantiating a SikuliX2 branch? This is indeed what git is made for, and it helps keeping a clean history.


Right, so `getParent()` == `getEntity()` then? There's no need for `getParent()` in this case.

I thought the 3 methods returned 3 different entities you see.


FTR: here's the proper fix 060cda77729cb1c4a827560e09e89a7b22078ba9


I guess this commit should have a message like `style(ngRoute): move comment inside relevant function`, right?


  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;
      }
    }

  Find your perfect job in 2015

  There are more than 240,000 jobs for you to find the perfect role in 2015. Don't 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


Yeah. Plain text, so no HR. Didn't know if jason wanted this or the other way where there isn't anything at all.


Oh right, it was there already. Looks dodgy to me, but whatever.


I was considering creating some exception classes; then I could check the name of the exception raised instead of checking what is written to stderr. Does that sound good?


Tested?


No need for the explicit type here. I don't love this inline JS though. Can we add this as a resource file? I realize it will be tiny (it will probably grow).

Eventually we may want to show actions on non link things (like videos or images?) I wonder if it would be good to expand this to return the entire chain of DOM elements under your finger


just trying to keep up with the times...


rax retry tests


This space make source listing files from " components" instead of "components". That the first cause of the issue.


Thanks, fixed.


The restyling makes sense, and I now know how to update block schemas. Wasn't aware of the requirement for calling a refresh. Thanks for the assist!


yes, @lgalfaso can you please fix this quickly


This change is odd...your iteration counts change, but mine do not.  Can you reproduce this on master?  Seems like a machine-specific thing, as they basically find the same values.


Yes; I guess that's simpler, but this code is not "strange" :smile:

Will fix and merge.


Its  a typo:
All Other lines are:
A:B:C:D
Line 4 is:
A:B:C:D:E


Adding this made the operator E work with variables. Because otherwise, variables and operator names would clash... Variable names still cannot contain operator characters however...


Had to run `composer update` to have this version used by Piwik: https://github.com/piwik/piwik/commit/0dc425d01d15025bf719a02d74bfac315c29bb61


I messed up here, this can be closed. You can import relative to the project's root (`@import client/stylsheets/...`) without a problem without adding `process.cwd()` to the paths array.

I also actually tested what would happen when passing in an empty array and everything still works the same... You can import from relative to the file's directory and the project root without an issue. So I'm confused what this code is doing at all.

Anyway, this can be closed. Sorry for the premature pull request.


See https://docs.google.com/document/d/12k2eFGWP-jEJ_0aHk8sdbh-A6je1-Qm0EIiWT2cta18/edit# for task details.


Sure thing!

best,
Andrew

On Mon, Jan 5, 2015 at 10:02 AM, Joe Meyer <notifications@github.com> wrote:

> The restyling makes sense, and I now know how to update block schemas.
> Wasn't aware of the requirement for calling a refresh. Thanks for the
> assist!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/concrete5/concrete5-5.7.0/commit/0e83dc5bbb34c1b21e7c4eb0304aad1fec89b9a8#commitcomment-9157130>
> .
>


3.0.2 版本，官网上发布的最新版本，3.1.0 没有升级。。。。就因为3.1.0的编辑器还不完善。脸书，推特直接返回404页面。3.0.2 Apache环境上设置了伪静态，官网上下载的3.0.2应该不缺文件的。为什么出现页面找不到的提示我也搞不清楚了。。。毕竟技术方面也很薄弱 。。。。咳。。。


The third param is still unaligned, you should revert this or align all.


@wmvanvliet thanks a lot -- I will once I'm at the lab tomorrow.


Way to kang commits and claim them as your own. Bravo!


cc @ericavonb @jefff :-/


True this is inconsistent, I think I will remove ubam from every wrapper. Do you have a usecase for ubam in Galaxy?


`reversed` - right!  That's what Python does, same argument.  I'll make that change.


Somehow it keeps sneaking back in...


The VM is 32 bit :-)


FYI.  This update has lost the ability to connect to a single 32 bit 8 channel board.
Gary


It does not anymore, using vhost, but before I needed to be sure that the docroot directory is created.


What do you mean?


True. Please run "Get-Command -ParameterName Path" to see all Azure cmdlets that use the parameter Path. Most of their intereptation is that Path is the full path. Pls confirm with Rochak aslo.


Awesome! Thanks Phil


Committed at https://github.com/openmrs/openmrs-module-allergyapi/commit/cc7fdd4de37c9745ab4eeeb3cd9e3c16021332cb


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


Excellent, tests are working for me now!


Didn't notice that before. Fixed in #75
Thanks


расширение не обязательно указывать, можно просто @import "specific";


Because I will start completely new with SikuliX2 while staying API compatible as far as possible.
A history in the sense of Git is not necessary or might even make things worse.
Since I want to use e.g Git-subrepos and switch to gradle only branching is not enough for me here:
I want to have a clear cut.

This 1.1 repo will live in parallel, to allow minor improvements and bug fixing with versioning 1.1.x in parallel with the development  of  version 2.


@tchx84 Its need to be aligned with the tuple no with the content of the tuple.
BTW, if I move it I get pep8 error.
*server.py:133:37: E127 continuation line over-indented for visual indent*


Looks good.


typo in comments


Whether you can do (non-rolling) upgrade without #2 depends on what's in your cluster state (e.g. if you have snapshot repositories configured) - since there are only version checks reading the state in a couple places if you don't hit one you're fine.  As for the unassigned shards on bouncing master, still hard to reproduce but I've hit it twice now, though don't know that it came in with 1.4.1 vs earlier (we've been running 1.1.0 & haven't seen it, but also don't restart nodes frequently.) On plus side recovery is kicked off if you subsequently stop another node, but...


https://gist.github.com/DmitrySharikov/e8e6bae3d7f5cc62cbf4
смотри как разметку нужно оформлять. И не забывай про отступы


I think this call should be after the error check


Sure. I changed a little bit more than I think most maintainers would like, but it's worth a shot.


Break constantEquals out into a separate module with its own test suite. Don't expose it from credential. Every export from credential is potential an attack vector.


r+ p=22222


hey man can u tell logano i want to join or what his response was (i got banned from mmocentralforums for suggesting something from infinite :( do u have skype i can add or something


Change lgtm. Test...I'll take your word for ut


strand2 is already included.


meters work like before now. TODO: Why did one testcase break as result?


* ссылки не нужно оборачивать в тег <a>, тк link_to и так компилируется в тег <a>
* , role: "button" удали


rollup


output:
Enter a float value :5.7
Given float value :y = 5.700000
After round off:y = 6
By adding 0.5 to a float value, the float value gets next higher integer part. Assigning the float to an int resutls in truncation of the fractional part and the result is as required.
To print exactly one decimal place after the decimal point and round off to one tenth position, add .05 and use %.lf .Some compilers automatically round off the value for %.lf.
Rounding off an integer to its tenth position can be done by adding 5 to the interer.Then the resulting number is divided by 10. The quotient is then multiplied by 10 resulting in rounding off the  the given integer to its tenth position. The same concept can be applied for rounding off an interger to its hundredth position by adding 50 and then dividing the resulting number by 100 and then multiplying the quotient by 100.


looks good.


The indentation should be like this, to make markdown work.


In C a static function is not visible outside of the file it is written in (or translational unit with some fiddling).
This will not allow calls that are using this function.
Was this the intended behavior?


Three alchemy fixes. Forgot that I moved the scale grist and add grist conversion out methods of the ingredient loop.


I have sent you the checksums of the demo files for the registered version(s) in one of my mails today. I guess they are the same for all commercial versions.


Please, tell me how to name the new branch? I don't want to confuse other Contributors.


Wonderful, thanks!
On Jan 5, 2015 4:53 AM, "Markus Wick" <notifications@github.com> wrote:

> Thx for this backtrace. So I was able to fix this bug in 4.0-4924.
>
> --
> Reply to this email directly or view it on GitHub
> <https://github.com/degasus/dolphin/commit/6e3b2712d2fafcd8fae7ba3653207980685708e6#commitcomment-9151622>
> .
>


The "Downloads" folder is localized. for non-english systems, it is not called "Downloads". This might be problematic for customers running in non-english OS.


@Listbd the whitespace destroyer...


Wrong command?


Having a hard time maintaining authorship? Original author here: https://github.com/Cl3Kener/UBER-L/commit/0438f74510ebf696b351a87a715cea6c32393e04


82420068_GridViewMultiEdit (evanweaver) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2068 of 2188 tests passed)

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


We're supposed to handle the layout ourselves


* во-первых, J**a**nuary 2015
* во-вторых, `<%= Time.now.strftime(%B %Y) %>`


don't use `console.error`. Code should throw or nothing.


some tests failed:
exception: http://buildbot.rust-lang.org/builders/auto-mac-32-opt/builds/3021
exception: http://buildbot.rust-lang.org/builders/auto-mac-64-opt/builds/3015
exception: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-t/builds/3016
exception: http://buildbot.rust-lang.org/builders/auto-linux-32-opt/builds/3007
exception: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-t/builds/3007
exception: http://buildbot.rust-lang.org/builders/auto-linux-64-opt/builds/3010
exception: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-t/builds/3003
exception: http://buildbot.rust-lang.org/builders/auto-linux-64-x-android-t/builds/2998
exception: http://buildbot.rust-lang.org/builders/auto-win-32-opt/builds/2674
exception: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-t/builds/2672
exception: http://buildbot.rust-lang.org/builders/auto-win-64-opt/builds/1511
exception: http://buildbot.rust-lang.org/builders/auto-win-64-nopt-t/builds/1502


I broke it on purpose, then fixed it later.  I was trying to accommodate a multi-line prompt that would support the 4 line input of the dialog (a script).


TeamCity MonoGame :: Develop (Win) [Build 3.3.0.1782](http://teamcity.monogame.net/viewLog.html?buildId=4181&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 293, ignored: 6 Build time: 00:16:37


Unnecessary blank line. Not critical, but code quality is nice.


Weird.  Are we using the same tab settings?  I believe I am using "Keep spaces" with an indent of 4.


r+


Please add a comment above the function name, referencing the issue you're fixing. This is nice, so we can see why this test exists.


Again with the tabs...


Cool.


what no this isnt the default tabbing that robotc does

i like having the two spaces or whatever robotc does


looks good.


This all block can be re-written:
```js
return request_with_auth(endpoint, 'POST', data).then(function(response) {
  return JSON.parse(response).id;
}).then(function(question_id) {
  // for loop
  return Promise.all(metadata_updates).then(function() {
    return question_id;
  });
}).then(function(question_id) {
  return SumoDB.get_question(question.id);
});


what the hell is this


I understand that @davkean and team have been coding from since way before a great IDE even existed to edit C# code, but every version of Visual Studio (back to VS2010 as far as I can remember) has provided refactoring, "generate from usage" code expansion and code snippets that do not use underscores. That's as much of a public *official* guideline as it can get.

Having different teams within Microsoft not follow their own IDE guidelines is not only weird, it forces anyone contributing to more than one Microsoft OSS project (i.e. Rosyln AND CoreFX) to constantly remember to change what the IDE already generates by default so that it matches alternating coding styles.

I can only imagine how annoying that would get really soon....


We need to be able to set this in one go to. Please ask @mythmon to help us with this.


I did not see this was available for MacOS X, too. Thanks for the link! This makes this repo somewhat obsolete :)


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


@bors: retry


Not sure if this was intended, but with this change AdHocCommand is unable to utilize Bytestream session negotiation (or anything else relying on Discovery services for that matter), as the discovery reply never gets processed while command is in the execution stage.


Since it became apparent that the regular expressions had to be slightly modified to work correctly with Boost.Regex it is best to remove the script. Users should not directly import the filters anymore!


would change --output <input file
-> --output < file name


@peti This Cabal library is failing to build for me (this happens at the end of compiling all the modules) with:

```
Linking Setup ...
configure flags: --disable-split-objs --disable-library-profiling --disable-executable-profiling --enable-shared --enable-library-vanilla --enable-executable-dynamic --disable-tests  --with-gcc=/nix/store/jk6130sfnnpnpd22bai77acbk93lmlm3-clang-wrapper-3.5.0/bin/clang --ghc-option=-optl=-Wl,-headerpad_max_install_names --with-gcc=clang
unrecognized 'configure' option `--disable-executable-profiling'
```

I've searched for this option, but it seems that we are not setting it explicitly anywhere.


r+


arcs are implemented slightly different in browsers. Even though the math the implementations are based on is the same the different implementations can even cause different numbers of cubic curves to emulate an arc. Rounding during the computation accumulates pretty quickly. (Behind the scene all browsers convert arcs to cubics again btw.)

I reimplemented the arc logic in WebKit and I know that there are visual differences for certain arcs. They became visible if you have big arcs (or scaled-up paths). Using cubic curves almost certainly let paths draw the same in all browsers. So I am not sure if we even should proceed or if it should rather be exported by the application directly. Furthermore, the path data is longer in most of the examples in your patch. Which surprises me a bit.

However, the math looks very interesting. It seems that you split the path into line segments and compute the angles. However, I am not entirely sure what you do exactly. Does the logic detect more than partly circles? (Sorry, was too lazy to check the logic on first review :(.) Could you provide some comments please?


r+ rollup


The trampolines are generated in a different context (i.e. by the fault handler, not the JIT), so I'm slightly worried about incrementing the far code emitter pointer outside of the actual JIT. What's a clean/safe way to do this?


r+ rollup

Thanks!


ouch...


It compiles and runs for me on the PPPL cluster


[CodeMagica/RailsBestPractices] remove unused methods (Refinery::SiteBarHelper#site_bar_edit_link)
reference:


Can you show an example of tuple indexing here?


I might be confused, but you seem to be passing `jvOffers` in to `getBookPage()` both times - when you're computing the bids and the asks.  Won't the second one overwrite the first one - or won't they get merged together?


Thanks ;)


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


Hi @slava77 thanks for spotting this.
We are going to commit a fix in our ongoing PR.


There is one flat global namespace for filenames. If the file has the same name but different contents, you can't use the same name for it: you have to do something like DEMO3_3.DMO_huntbgin, DEMO3_3.DMO_darkwar, and mark them both as "install_as: DEMO3_3.DMO" and "look_for: [DEMO3_3.DMO]" (I should streamline it at some point so that the default for look_for includes what you install_as). See the handling of quake2 that I've just pushed to wip/doom-quake2 in git://git.debian.org/git/users/smcv/game-data-packager.git.

This should make gdp crash with an assertion failure because of the size mismatch, in fact...


I see :-) OK.


Looks good, but what compiler option did you use to get this?

I tried with `CC=clang make clean all` and with scan-build from `make check`, and get no warnings ...


82420068_GridViewMultiEdit (evanweaver) - 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)


My stats is poor (to say the least :)), so I posted a question to stats.SO: http://stats.stackexchange.com/questions/131304


try declaring
> $scope.tree_data = [];
before retrieving data. After this you can write a function to fetch data using http.get().


should be singleton


Yes - I broke constantEquals out, too. :)


This should probable use checkedCast


its really strange in here oooo


I don't know which syntax to use a.t.m. ; diggin it


Thanks.


@narfbg It's step 11 actually. But as this would be bothersome to maintain, I'd suggest « (see *Usage of XSS filtering* step above) ».


List them separately (as sodemo.zip_e32d61f and sodemo.zip_d581a3e or something, if you don't have any more descriptive name), configure them both to "look_for: sodemo.zip", and the Python code should do the right thing: unpack whichever one you happen to have. (And if it doesn't, we should fix it so it does.)


Er, are these precisely the same files as for -wl6?


Are you sure that it's this commit which is causing this? Would you mind posting this a bit more elaborated on the Smack forum: https://community.igniterealtime.org/community/support/smack_users

I think I know which situation you describe. But it would be helpful to know which APIs are exactly involved.


Found because of https://github.com/psypanda/hashID/issues/20


And, if the general accepted idea is that style is not that important, why the trouble of coming up with yet another one that deviates from that?


Don't use assert, since it only turns on if the VM runs with `-ea`. Do the check as before, but throw an AssertionError, instead.


r+ rollup


fixed sorting algorithm, replaced with std::sort


Why pin it to a version number at all?
How about just "latest"?
And if we bundle and get warnings or breaks, we deal with them.


Tanks for your pull request @romulostorel in the old code return false or true, and is more explicit, but i think the your way is more ruby like.


Otherwise, looks great! 👍


typo: "ahdnler"


I'd be tempted to split this into "a separate game" (a separate YAML file) if that's how users are going to think about it, the same way Doom 1 and 2 are separate games.

I'd also be tempted to call it spear-of-destiny, because <https://en.wikipedia.org/wiki/Sod_%28word%29> :-)


r+


he he he


This looks like a copy/paste error. This file is identical to `newrelic_sysmon/init.sls` but should match something more like https://github.com/theirc/Service-Mapper/blob/develop/conf/salt/project/newrelic_webmon/init.sls


@Konfekt
What's your use case?
I have resorted back to not re-mapping `.` myself.


Did you not get a `config.serve_static_assets` => `config.serve_static_files` DEPRECATION WARNING?


Thanks. Fixing it now.


They have different name, I will add this
```
http://www.classicdosgames.com/game/Spear_of_Destiny.html -> sodemo.zip
http://www.dosgamesarchive.com/download/spear-of-destiny/ -> destiny.zip
```


I don't see how these files can have a concrete size, but have alternative versions. Does this mean both the alternatives are the same size but with differing content?

I should add cksums or sizes as a top-level thing like md5sums, to make it easier to assign concrete sizes to the wl6_steam and wl6_alt variants of these files.


oops, thanks


Well I remapped Ctrl-R to :redraw! as ctrl-l/h are just too convenient for buffer switching.


I don't know, I don't have those; I just put those there to make `make check` happy. We need help here to locate the files.


>> I'm not sure we're allowed to do that though, the current description in formats.h

> This was the exact reason why prepare was made, to alter the format string, using other fields that are NOT available either before or after prepare.

As currently defined in formats.h we *return* a new string. Altering any of the input fields is not mentioned.


This looks great! Will cherry-pick.


@bradenmacdonald So does this commit prevent a Studio author form initially referencing a library for which they do not have a permission? I'm a little confused because the code below is specifically testing _refresh_children, but perhaps it is the same code path.


Looks good, will cherry-pick it.


Ah yes, this is leftover from when I was going to package a misc fork of RTCW-SP because I hadn't found iortcw yet.


I think it should be
```
* Safe serialize() replacement
```


Others are seeing this outside of Nix, for example https://github.com/rrnewton/haskell-lockfree/issues/40.


Okay, I'll change it back...
Still I think it's a bad idea to blindly trust something you don't understand.


@kaspth noted! Thanks by the way.


@dmitchell Did you see this commit?

Having something in library_content_module.py "xblock.want" something in cms makes me quesy. @bradenmacdonald How would you feel about putting StudioPermissionsService in student.auth?


Ok I like it, this way I can get `spear-of-destiny` soon complete without risking regressions in wolf3d


Yes it is in the vendor blobs from Google. The only blobs I pulled from factory were extra audio effects stuff. The blobs in factory image are slightly different as they don't have debug symbols.


oops.


I overrode the `getEntity()` so that casting would not be necessary, though, getting rid of `getParent()` would make sense to avoid confusion.


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


Also noted!


Oh I see Thanks. But actually I have a lot of problem. First libbacktrace doesn't work(can't exec) So I am trying to change with Android's one becuz it is working fine on MacOSX. Thanks! @pbeeler


Comment


Sorry I did not realize middle alignment with the computed height of the two elements results in a break of the baseline alignment. This reinforces my opinion because I think baseline alignment should be the default.


I see where you're coming from. But given that it's an upgrade routine, so it's only going to run once, and that it's actually a relatively quick query, since we're not actually selecting any data, I think it would be appropriate to make it an upgrade routine. FWIW, running the query on a posts table with > 1.2 million rows took < 1s.


@plumpudding  What do you think of this compromise? It's short while also seeming like a real licence written by someone who knows law, in case there are juristical problems.


4 files are the same, 4 files differ: https://lists.alioth.debian.org/pipermail/pkg-games-devel/2015-January/033271.html


Good note, thanks.

2b5682d4f39f7d16ce47e2f45ea19ffab615cc60


While touching this code, can you rename newTask to something sensible? buildTaskWithPortsForOffer or something similar. Took some time to grasp what the Seq[Long] is.


Indeed...


Hmm #1001 might be caused by that other function you mention.

Feel free to re-work the allocs. I doubt much will happen upstream, almost nothing were changed since my last look years ago.


So, what do you think?


Content for non-telepathic humans.


Then maybe ```getOtherParent``` ought to be ```getOtherEntity``` just for symmetry


We didn't come up with another one - we already had one internally, we've just never had a chance to go back and apply it holistically across the code base.

Guys, I've think we've ridden this horse as long as we can, thanks for the feedback on our coding styles. The feedback has been noted, but we're going to stick with what we have.


84202670-DuplicateNodeInZoneConnCausesCrash (JasonGlazer) - x86_64-MacOS-10.9-clang: OK (1576 of 1576 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)


Nice!


Iyiymis direk site uzerinden de yaptigin degisiklikler gorunuyor. Profil resmin guzelmis bu arada :smile:


Agreed.


Why was this removed from the scope? I had wrapped the {{ windowClass }} expression in another element, for a custom template, but now this option is too rigid to be of much value. Why force the class to only be added to the root element, instead of just making it available on the scope?


I typically see this written as `#ifdef _WIN32`

Perhaps the underscore is some kind of old school relic and you no longer need it, but I still do it to be on the safe side.


This explicit order deserves a comment.


is this safe? is it sure that ZIN0(3) != out[0]?


No reason. Do you think it is better practice not to create superfluous variables?


Changes made on desktop. Will commit at COB>


Agenda must be updated too. See https://github.com/jquery/meetings.jquery.org/pull/18.


OK, got it now - I missed the `else`.

Now there's even more code duplication.  You can remove that duplication with a tiny lambda - something like this:

https://github.com/rec/rippled/commit/695e932f9bb9ec59151d4ca224d271b5edc874a7

(note: wrote it quickly, I know it compiles and tests but it needs a skeptical eye.)


This changes the behaviour. Is this intended? Before the tasks in the queue with the given app id were counted? Now the version is filtered as well.


Entirely a matter of style, but just to be clear that I'm calling something from the global namespace I tend to preface functions with `..`, for example `::PQencryptPassword`


This went due to "Organize imports"


I fiddled with step_x and step_y... new commit coming up shortly.


I use a trailing underscore to denote member variables (e.g. `conn_`) as it's very helpful when writing code to know when you are mutating shared state. I also preface pointers with p, e.g. `pConn_`


As @stts mentions, the bare tuple return value is pretty ugly here.  However, this change is already quite large so let's do it as part of another change set.  Filed #937 to track it.


r+


az önce ekledim onu senin icin :D

2015-01-05 21:07 GMT+02:00 0014 <notifications@github.com>:

> Iyiymis direk site uzerinden de yaptigin degisiklikler gorunuyor. Profil
> resmin guzelmis bu arada [image: :smile:]
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mrgenco/strategy-game/commit/a29c790e6212961f51b824b11c64c0e5dcbffe0a#commitcomment-9157956>
> .
>

--
Computer Engineer
Mehmet Rasid Gencosmanoglu


See #937, let's keep this PR smallish if possible.


    git checkout experimental
    git checkout -b <new branch name>

This -b creates the new branch. To push this upstream after committing changes:

    git push origin <new branch name>


@peti: Thanks! (I cherry-picked to release-14.12.)


Sure, can do.


Thanks


Was that right?


Here is a commit that fixes the problem: https://github.com/aburgm/openclonk/commit/2f69aa4850ab7a4b621e5f30cbc537d40f32c0df

It might make sense to keep it as a separate commit, or you can also squash it into yours if you like.

I have still seen some of these `WARNING: Animation 'ShieldStandDown.L' from Clonk is not included. A newer version of the animation exists in the destination file.` warnings when reloading ClonkIncluder. But the crashes are fixed, and reloading now seems to be valgrind-clean.


Great. This is definitely the problem to solve. The potentially quadratic complexity of the call to TextSpanner._get_leaves() hasn't bit me in either the recent cello piece or the viola duo (which were the pieces during which I added the method call). But the potential is certainly there. And so I'd been waiting for an opportunity to investigate exactly these points.

The idea of a format session will work. But would it make sense to think through other options first? In general it probably makes good long-term sense to avoid implementing a session model of anything until we're certain we can't come up with a nonstatal implementation that would work (and read) just as well. It's not that there's anything inherently wrong with sessions, of course. But the ability to articulate the steps in the formatting process (or any other process) potentially decrease when the potentially variable states of objects-to-be-formatted (in this case spanners and their leaves) will have to be taken into account in the verbal description of the process. I don't have an alternative suggestion yet but thought now would be a good time to mention.


Highland has `.fork()` and `.observe()` for multicast. Some discussion of the drawbacks here would be good. I have found forking streams with highland to be confusing, for instance. It would be good to call this out more specifically.


The `apps` should never be really big, so the sort will not have a big impact.


Looks good @joelotz


Good catch, will change.


The `readlines` call is unnecessary.


TeamCity MonoGame :: Develop (Win) [Build 3.3.0.1783](http://teamcity.monogame.net/viewLog.html?buildId=4184&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 295, ignored: 6 Build time: 00:16:18


Should we add all hosts here, instead of just the first?


This is great. =)


The result of `hashlib.sha256()` isn't a string. Are you sure this is right? When I submit the string `"password"` it seems to work but if I submit the sha256 hash of password it doesn't authenticate.


oops it was in next; merged into master now.


This doesn't handle commented lines.


So this is preflighting the save with individual requests to the OSM API for each modified entity. This seems potentially expensive / slow. Is it what JOSM does?


82420068_GridViewMultiEdit (evanweaver) - i386-Windows-7-VisualStudio-12: OK (2170 of 2188 tests passed)

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


... or res.isDefined is false, i.e. the offer doesn't match. Are you sure about the removal from the queue below?


Waarom een abstract class ipv interface? (program to an interface)?


Aha. I did not realise that dart pub provides a bootstrap script to automatically look for the javascript files as long as they are following the *.dart.js filename convention. So, yes, you are right, I can remove index.js if I adapt my devmode script.


This needs to be fixed


There is an issue for the org-mode version: https://github.com/dengste/org-caldav/issues/56

@stvharman: Your issue sounds different, you should probably raise a new issue (with a backtrace `M-x toggle-debug-on-error`)


What about using integers for absolute number of lines and float for fractions of the available height?


r+ rollup


Um... Is it really necessary? For example when OF 0.9.0 will be released, a lot of people won't compile openFrameworks by themselves, they will be using precompiled one.

If you guys compile release build on machine with GLEW_ARB_direct_state_access extension, these macros will be irrelevant...

But the other way around, if one will compile it without GLEW_ARB_direct_state_access extension, but at some point will install driver that supports 4.5, he won't get speed up by using this extension since macro removed the code at compile time.


Where did you find that the width and depth should be >= 3 ? I don't see this test in wine sources.


When you remove @(at) this is a case of invalid params, you can put this expect inside on spec of line 19.


The man page says:

```
       The   wolf4sdl-wl6,   wolf4sdl-wl6a,   wolf4sdl-wl1,  wolf4sdl-sod  and
       wolf4sdl-sdm binaries start  the  Wolf3d  Full  v1.4  GT/ID/Activision,
       Wolf3d  Full  v1.4 Apogee (with ReadThis), Wolf3d Shareware v1.4, Spear
       of Destiny Full and Mission Disks and  Spear  of  Destiny  Demo  games,
       respectively.
```

So what we need is the version that was sold by Apogee, and it seems nobody involved in the bug has it.

I'd just comment out the support for wolf3d-data-wl6a until someone who has that version talks to us, tbh.


`git revert 91f7cd1`?


Refac this please :)

These lines need indentation


With that change:
```php
v::arr()->check(''); // true
```


if you reinstall the driver (although in this case it depends on the version of GLEW) the headers will change and the OF parts that use them will get recompiled too as soon as they do.


I think you are right that while assert is cleaner syntax it can be turned off. I just switched to using an if/throw block rather than introducing a dependency.


r+ p=22222


My solution for this has been "LD_LIBRARY_PATH=$(pwd)/.libs python -mpdb ceph", but this seems slightly easier.  I'd probably do "if it exists in the env", though; requiring 1 just seems needlessly strict, when someone forgets whether it's 1, 'yes', "Y", 'present in env', or whatever.  I'd also not call it 'execv_bin', since it's more than that when it gets a flag added.


also we don't ship the binary library with OF anymore it gets compiled the first time you compile an example so this would fail in several platforms


This is already accomplished with options.modelName.


the sort is fine complexity wise of course. I meant the reason for the sort. It's not obvious from the code why we sort by constraint size.


Perfect! :stars:


Ok, but I'm talking about end users of applications: they won't get feature if I compile app on machine without GLEW_ARB_direct_state_access...


Marathon prioritizes scheduling tasks with constraints under the assumption that they are harder to place as utilization increases.


@Respect/troubleshooters, anyone can explain the reason what this change was made?


Not sure the previous behaviour was correct. Compare the second patch in #932. If expect your new semantics is correct. But I would move it to another separate patch, mixing behaviour changes and refactoring is not good....


But so far that's been implicit behavior, eventually it might be nice to have a separate priority calculation.


84202670-DuplicateNodeInZoneConnCausesCrash (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Tests Failed

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


seems like this would be a valuable PR to contribute back


That's a good point...


why did you remove the role here?


there's no way to do this without forking the repo?


I think if you change this to `hashlib.sha256('password').hexdigest()` it should store the actual password.


There won't actually ever be another host at the moment.  We would want to if we were dealing with a ClusterComputeResource, then we'd want to iterate over all of the hosts in the array.  On that note, there's a separate bug that doesn't allow us to find a HostSystem in a cluster by name.  Will file that one, but I have no need to fix it right away.  I will need it shortly though.


Great! Thanks. =)


Yes it should be length not -1.  Silly mistake thanks for catching that and the dublicate number.  Since start was listed twice I think I missed this.   Also added the efficient on the loop.   Normally I do this, not sure why i left it off this time. Maybe because the list was small but nonetheless..


Perfect fix,
I think you forgot one (line 728) :

var fileObject = event.srcElement.files[0]
->
var fileObject = target.files[0];


Please make sure these debug "console.log" are removed when requesting review.


Not sure this test is still testing what it claims to test, since `log` has only ordinary macros and no procedural syntax extensions.


try to have more meaningful commit messages. Next time you have a merge conflict, come find Omid or I and we can walk you through that process.


If this is really two-space or tab indenting, please fix that to 4-space indenting


This is done to throw the proper exception: `checkedCast()` would throw IAE, but `saturatedCast()` will result in an invalid Unicode code point.


'explicitly'


oh i see, but i guess that's less prioritary than making OF compile in as much platforms as possible. if we remove the ifdefs that potential OF user won't even be able to compile it anyway. also if the application is tested using the >4.5 features it's ok if it remains like that. the new syntax only makes it easier or more natural to use GL buffers but i don't think it's faster or anything like that so there won't be any benefit for an application use in having the app internally use the new calls


Could you please clarify how xplat compatibility will be achieved?
I thought we will have runtime compatibility, but now I see in code compile-time compatibility with partial classes (*.Windows.cs & *.Unix.cs) and conditional compilation. How will it work in case of Nuget & package restore?


@alvinlai: might be worth looking at the past couple commits to see what I changed up!


do we actually have a get method? I thought that was just on the input facade that called the input method?


I was thinking about a percentage progress, 0% to 100%, and sometimes it could be 120%, because the user pulled too much.

It's a great feature, I was waiting for it.


Esqueci de comentar também que eu transformei o antes dicionário Exercise num objeto de fato.
Isso quer dizer que type(IfBaseUnit.createExercise) agora é "Exercise", não "dict".


after this pull i see a lot of 'hostname: Unknown host' in my build output.
```
hostname: Unknown host
hostname: Unknown host
hostname: Unknown host
hostname: Unknown host
  CXX    webserver.o
hostname: Unknown host
hostname: Unknown host
  CXX    ws-api.o
hostname: Unknown host
hostname: Unknown host
  CXX    ws-auth.o
hostname: Unknown host
hostname: Unknown host
  CXXLD  pdns_control
hostname: Unknown host
hostname: Unknown host
  CXXLD  pdnssec
hostname: Unknown host
hostname: Unknown host
  CXXLD  zone2sql
hostname: Unknown host
hostname: Unknown host
  CXXLD  zone2ldap
hostname: Unknown host
hostname: Unknown host
  CXXLD  zone2json
hostname: Unknown host
hostname: Unknown host
  CXXLD  dnsbulktest
hostname: Unknown host
hostname: Unknown host
  CXXLD  dnsgram
```
It looks like we generate version_generated.h over and over again.


I have no idea where this is coming from. Is there maybe a setting like that active in your `~/.cabal/config`?


Sorry, leaving this here was a mistake - I've removed this node now.


Talvez eu vá começar a fazer isso retroativamente com métodos do tipo, porque eu acho a notação com ponto mais elegante do que a notação de acesso por colchetes.


Yup, that was my mistake.  Going too quickly.  Should be changed now.

On Mon, Jan 5, 2015 at 2:30 PM, Mike Nolan <notifications@github.com> wrote:

> I think if you change this to hashlib.sha256('password').hexdigest() it
> should store the actual password.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/thequbit/yellr-server/commit/2092bb755b5389fb9bf28c242fe41d3d45c46ee2#commitcomment-9158314>
> .
>

--
*Tim Duffy*
http://timduffy.me/
(585)-210-8353
@arbiterofbits


Rather than add this additional state to the context, I'd prefer one of the following approaches:

* For conflict resolution, create and load data via a second `iD.Connection`. Nobody will listen to events from this connection so the data won't flow into the default history.
* Remove eventing from `iD.Connection` and pass result data exclusively via callbacks. The context has wrapper methods for `loadEntity` and `loadTiles` that push the data into the default history. Everything uses these wrappers except conflict resultion, which calls the connection methods directly. (This is the approach I favor.)


I guess he aimed for `empty()` compliance, so `v::notEmpty()` would be required if you'd like strict checking. I'm not sure what went wrong though, 'cause I seem to have debugged this code before.


You say they should be handled by the caller, but, this is the caller of CephName.valid(); do you mean "the caller of find_cmd_target"?  if so, I'd make that clearer ("our caller" would do, or "find_cmd_target's caller")


84202670-DuplicateNodeInZoneConnCausesCrash (JasonGlazer) - 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)


I took this method from kernel/common/string.rb in Rubinius 2.1.4 and added the copyright message. I also noted that it's not a full copy of the file (so merging is easier if we want to do that).


This is from kernel/bootstrap/string.rb, imported in a similar way.


r+, thanks again for doing this!


@aleks-sidorenko, in cases like this, the current plan is for multiple DLLs to be built, one per platform, all of which will be in the NuGet package.  In some other cases, where it makes sense, one DLL may be able to support multiple platforms, if the differences are minimal and selecting an implementation at runtime makes sense, e.g. differences between flavors of a particular OS.  I'm not sure what you mean by "runtime compatibility" vs "code compile-time compatibility"... can you clarify?


This is the implementation of the new primitive.


We then have to add the new new files to the core.rb file, which is the only one that the VM loads.


You can't offer a GPL script that has to be merged into non-GPL one.


Yeah, sounds like a nice approach.


And with that we pass all of the `String#include?` specs.


Are you sure it won't compile? I checked the GLEW docs for that method - and it should only return true if the extension is present.

I take the speedup argument (as in favour for the #ifdefs) though, since it would add a couple of jumps to each draw call.

I wonder if the GLEW method as it is resolves at preprocessor stage, though, can look into it tomorrow when I'm back at work...


Unfortunately, I haven't been able to reproduce the crash myself as it is triggered by this weird inability to initialize the audio device, which I don't have an easily reproducible case for.

Your fix might help though, also you may want to look at the `setInterrupted:` method in the same file which seems to have very similar code, though it hasn't been triggering crashes for me. But better safe than sorry.


@mgracer48 No to all.


@kevinykchan Two notes on that:

1. aren't you doing toLowerCase() *twice* on acmp and bcmp now? E.g. at line 181 and line 183.
2. The `var acmp = a[f].toLowerCase()`  could throw if `a[f]` is null or not a string, right? E.g. if we accidentally sort on a field that is null or an integer.


This code is unreachable since ex.getCause will throw null exception.


Again, fixing path to ping on FreeBSD 9.3, 10.x, 11.x


fix related to code review


boop


saw approval from nikomatsakis
at https://github.com/nikomatsakis/rust/commit/842fb1cf8dfea210e9b09e8f5986c7d8c3f42fb3


merging nikomatsakis/rust/nick29581-arrays-3 = 842fb1cf into auto


status: {"merge_sha": "7c748664ed203cdcbe660f5cfe67cc45e884aa54"}


nikomatsakis/rust/nick29581-arrays-3 = 842fb1cf merged ok, testing candidate = 7c748664


Also, thoughts on `if (a[f] === undefined)`  vs `if (typeof (a[f]) === 'undefined')` ?


r=aturon


@eregon @nirvdrum this is an example of how to implement a new method using code from Rubinius. I took a method that @nirvdrum implemented recently, `String#include?` and re-implemented it using Rubinius.

This should be the first approach to implementing any new method you need to get something working. We can have exceptions for code that is critical for performance, but unless you're running a benchmark you can't make that decision, and even then we should first look to improve how the VM uses the primitives rather than remove them. I think we can also make exceptions if your are basically just implementing a shim to get stuff working and the Rubinius code would actually be far more complicated, and if the Rubinius model is fundamentally incompatible with the JVM (things like FFI calls).


"Framework for true mock in c#". Que bosta, hein?


should this be ``protected``?


There are a lot of tests including `''` as valid value in many data providers and such.

I'm not sure about the intention behind this but with this change our rules are not reliable any more because every time you have an `''` it is a valid value unless you use `notEmpty()` rule.


I thought of that too, so I moved `.cabal` out of the way entirely, but this had no effect on the behavior.  Since this doesn't seem like the sort of thing that would be a Darwin specific issue, I'm wondering why you don't see it on your side.  Can you show me the build output from rebuilding the Cabal library?  I'd like to see which flags get passed your configure command.


This fixes the linking problem: https://github.com/aburgm/openclonk/commit/b03a332b73463b378c94e4e92b66d45b923b3b9c


PHP 5.3 support has been ended. Please upgrade to PHP 5.4+ http://php.net/eol.php

@ilendemli reverting this would be great and also makes people upgrade their PHP to the maintained versions.


[SideCI/RailsBestPractices] law of demeter
reference: http://rails-bestpractices.com/posts/15-the-law-of-demeter


[SideCI/RailsBestPractices] remove unused methods (Static#irt_incomp)
reference:


No :)


I glanced at how Chrome does this type of stuff actually. It looks like they have an injection manager for each tab, and would inject (pieces of) a larger script. Then they just call something like (https://code.google.com/p/chromium/codesearch#chromium/src/components/translate/ios/browser/js_translate_manager.mm&q=__gCrWeb.translate&sq=package:chromium&type=cs&l=49):

self.webView!.eval("__moz__.utils.findElementsAt(\(x),\(y))") { result in
}

i.e. just a tiny bit of javascript mixed in the ObjC/Swift code. I think I'd like something like that (with a smart manager to limit how much JS we load into pages...)


Also wouldn't it help to change the declaration of that property so that it's `weak` instead of `assign`? I know that this project is trying to be backwards compatible to the dark ages, but hardly anybody is targeting anything less than iOS 5 these days (if that).


can u tell me his full response like why he said no


correct, thanks


Ja, ich wollte ein wenig aufräumen.
E2 nutzt die Funktion taxes einfach nicht, und stört sich daher auch nicht daran wenn die noch dran hängt.


removing from line 50-109 soon


Are you actively working on these ?   I do not want to have both of us walking all over each other.


@mvanotti This cast should be `(process_handle_t)`. That way this will automatically keep in synchrony with the definition of that type.


I've added some tests, source surfaces with 2x2 or SourceRect with 2x2 doesn't work on wine and Windows 7, except for same src and dst surface dimensions. I don't know where to find it in wine.


in ubuntu GLEW is still an older version and those defines don't exist yet so i had to add those ifdefs to be able to compile


But what I want is to set my defaults globally for my entire app through the provider - I can't do that anymore if this has been removed.


I just wrote this!


Ah, ok.  Just making sure it wasn't a problem in the official release.


r+ p=1


Frag bloß nicht, unterm strich hab ich nur das (void) bei der Deklaration vergessen, GCC, clang und VS haben da jeder eine eigene Meinung zu, das hat mich echt fertig gemacht. Jetzt ist der code für alle 3 OK, aber der Test ob eine zu große Einheit noch Burgenschutz bekommt oder nicht schlägt immer noch fehl, obwohl ich das schon angepasst habe. Muss ich morgen  noch mal schauen warum.


Right, sorry.


some tests failed:
failure: http://buildbot.rust-lang.org/builders/auto-win-64-opt/builds/1512
exception: http://buildbot.rust-lang.org/builders/auto-mac-32-opt/builds/3022
exception: http://buildbot.rust-lang.org/builders/auto-mac-64-opt/builds/3016
exception: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-t/builds/3017
exception: http://buildbot.rust-lang.org/builders/auto-linux-32-opt/builds/3008
exception: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-t/builds/3008
exception: http://buildbot.rust-lang.org/builders/auto-linux-64-opt/builds/3011
exception: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-t/builds/3004
exception: http://buildbot.rust-lang.org/builders/auto-linux-64-x-android-t/builds/2999
exception: http://buildbot.rust-lang.org/builders/auto-win-32-opt/builds/2675
exception: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-t/builds/2673
exception: http://buildbot.rust-lang.org/builders/auto-win-64-nopt-t/builds/1503


This changelog entry was not supposed to be in this commit.


the official limitations are listed here: http://msdn.microsoft.com/en-us/library/windows/desktop/bb205857%28v=vs.85%29.aspx

are you sure in your test app that you didn't fall into one of these ?


removed!


How do you know that a factorial of a nonpositive noninteger is not composite?


[MH]
My understanding:

a "remotable" encapsulates any body (function or method) to make it serilaizable.
Therefore, no need for spores any more.

 Question:
What is about references outside such a remoteable?
i.e. BigFunction.CallsOutsideFunction() /// How is it handled?


Ahh, I see!

That makes sense.

 I thought we're using the oF supplied glew (the one in libs) for all platforms.


Align the arrays.


Looks like you were trying to round up to a multiple of 4. This will add 4 if `strlen(label) % 4 == 0`. One common idiom that works is `(strlen(label)+3) & ~3`


Why remove the interface?


Is it worth moving this into its own file, just for separation?


No, we should NEVER alter teh input fields at all. They are (and should be) considered constant data, that you can not legally alter.  NOTE, in the test cases, they 'were' static.  I do think we have altered that lately, BUT the concept should be maintained.  Alter a string you are going to return all you want. BUT the field data should be golden upon return.


FYI putting M200 in the gcode file pretty much makes using volumetric useless, as the whole point is that you can define the filament diameter in the firmware not in the gcode file, so one gcode file can be used with different filaments. I would recommend making  the prepending of M200 optional.


Thanks for the fix, Aliaksei.

In the future, could you create an issue and tie the commit to the issue? Thanks!


I believe it was this change, and specifically removal of processing of packet listeners via executorService. New flow is like this :

1) client receives an AdHocCommand, which is processed via a PacketListener
2) AdHocCommand sends a request to establish a Bytestream session, which procudes a Disco request and a Collector is created to handle the response
3) AdHocCommand is still blocking the loop inside processPacket() method until Bytestream request times out
4) the Collector is now processed since the blocking PacketListener (AdhocCommand) has completed

In terms of APIs, my code has a LocalCommand which invokes Socks5BytestreamManager.establishSession() inside its next() method, which is where the problem takes place.

I will follow up through the community a bit later, like you suggested.


I'm done with #1001 so I'm not touching it anymore for now.


It's not serving any real purpose at the moment.

On Mon, Jan 5, 2015 at 1:54 PM, Romain Lanz <notifications@github.com>
wrote:

> Why remove the interface?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/laravel/laravel/commit/caa166f5d6ded18f2d3f11612433e90f5a1150e9#commitcomment-9158781>
> .
>


With the `dashboard.php` routes, should we wrap each "section" within a group?


As @noordawod suggested, you should be able to do something like:
```
var defaults = Provider.getDefaults();
defaults.property = 'something';
```


I can't? why? it has to be merged with what? is a base script, you can merge your script into mine if you want.


Für die Spieler ändert sich nichts. Da die .xml auch angepasst ist, gibt es nur noch 6 Stufen und nicht mehr 7.
Ob gehandelt werden kann oder nicht wird direkt über die Burgengröße abgefragt, so das die einzige Änderung für den Handel die ist, das man erst ab Größe 10 als Burgenbesitzer was von fremden Handel ab bekommt. Dann aber auch wieder die bekannten Werte. Das dürfte in E2 keinen Stören, den entweder sind die Burgen groß, oder man Handelt selbst in der neu erschlossenen Region. Sonst muss ich wegen der unterschiedlichen Zählweise (E2 ist 3 der Turm, bei E3 die 2) wieder zwischen den Spielen unterscheiden, und das tut wegen dem Handelsposten nicht not.


misspelled canola!


> if we remove the ifdefs that potential OF user won't even be able to compile it anyway

Then `if (GLEW_ARB_direct_state_access) {}` will return false in this case and it will be skipped. I'm on Mesa driver right now with, my top OpenGL version is 3.3, I compiled OF without ifdef's an hour ago.


@mgracer48 "Because all the "work" I have seen from you is eiter directly ripped from Toontown Infinite, or just simple tweaks to small files that are mostly irrelevant. I have also seen some of the questions you have asked on the P3D forums, and the answers to them should be common knowlege to someone that is a developer on this team. Thank you for your interest."


This should give people the freedom to typehint the methods how they want too.


feels wierd doesn't it?


Nice! :D


Now this working without problem ( i register more 200 number )
https://github.com/Jorit/New-Whatsapp-Wart-2015
Thinks @Jorit , @shirioko


@dgoodlad it is better to release this as 9.0.0?


last is recommend to revised to last_off


@SharpNoiZy that doesn't really matter as you can always change that to whatever term you prefer. Just update the config view path.


edit comment


should add the codes below to avoid to waste time on processing the remaining of diff 1
if (s_end)
  break;


this is my githup!


This fixes the 500 error, on updating.


FRC begins!


I'm not super sure, but I'm afraid this might cause the actual form to not get submitted.


Entry sysf for level vibrations isn't merged I will try add it soon


You can do something like
```
find . -name '*.cpp' ! -type d -exec bash -c 'expand -t 4 "$0" > /tmp/e && mv /tmp/e "$0"' {} \;
```
in an Unix system to only expands tabs that at the start of the line (not in the middle since it could kill some code)
or
make it git do it for you automatically via a .gitattributes file as mentioned in

https://gist.github.com/eevee/6721177

or forget about it and leave the tabs.


You should use constant when assigning state it would make the code more readable.


all tests pass:
success: http://build.servo.org/builders/linux1/builds/459
success: http://build.servo.org/builders/linux2/builds/458
success: http://build.servo.org/builders/android/builds/494
success: http://build.servo.org/builders/mac1/builds/675
success: http://build.servo.org/builders/mac2/builds/676


@peti @copumpkin As an interim solution for me personally, I can install a local `git-commit` hook that prevents me from committing changes to any file which says "do NOT edit".


Can we add:
```
if (error) {
  done(error);
}
```
So this will properly error


Node.nextSiblingが、要素ノード以外(空白文字を含んだテキストノードとか)も対象としてしまうので、次の要素ノードのみを取得するメソッドをprototypeに追加しました。後でここは別ファイルに分割します。


Is this the same with above webview_shell?


NodeListのprototypeに、Arrayの各種メソッドを追加してしまってもいいかもしれません。(これも切り出します)


I don't think the return false is needed here since Throw will end the execution. Also you could potentially clean up by using:

if (this->state != STATE_CONNECTED) NanThrowError(...);
return true;


fast-forwarding master to auto = 499d17f5


Can we add:
```if (error) {
  done(error);
}
```
So this will properly error


Ohh, I see. How about parseInt() instead? I know it's more verbose, but it's also more clear to me that it converts a string to an integer. At least, that's what I always do.


Note that I opted to default to a set default seed of 31337. The rationale is that when TS fails you really really want know what seed it was using. But it might be better to really seed with some variant of wall clock, and output whatever seed that was in the TS header when applicable.


Nitpick: I like prefixing all jQuery objects variable names with $, could you rename mapDiv -> $mapDiv and everything else as well?


@peti I think the overrideCabal function eliminates this problem in your haskell-ng branch. I hope your branch comes to master soon as it'll make this not a problem.


What if I demote to high instead of highest?


which


yes, i am not sure to keep it or delete it.


Maybe Frodo should only be able to do this once, probably by checking his item. For bonus points, use the item "Ring Target" instead of Eviolite and up the boosts that Frodo gets in defenses to compensate, and also only give boosts if he holds the Ring Target.


I don't know what the right answer is. We could go either way here and either require that an authority be tolerated, or we could say that SRI URIs need to start with the case-insensitive string `ni:///` and therefore are required to not have an authority.

If we think we might want to use the authority field in the future (e.g. abusing it for versioning or some other reason) then we should require that it be ignored by user agents. Otherwise we won't ever be able to use it.


Like I said, we're not going to auto/manual post-process every file we're committing. It will break too many existing files, or at the very least highly likely to introduce changes which author did not intend. Leave it be.


Thank you, much appreciated.

If you're still making more improvements, one request from me: if possible please eliminate all "  (you already found and fixed half of them...).  Another translator added those in recently but they have already caused some game crashes because the game code is not expecting "


I've just noticed that !MIMETYPE is defined in [template.erb](https://github.com/w3c/webappsec/blob/fa576baca28f134c70dcca9a953b1e0d1e68fd0f/specs/subresourceintegrity/template.erb#L61) so I've taken this commit out of the pull request.


網域名稱透過bi-grams的方式分類


If others want it, this file in `.git/hooks/pre-commit` (marked executable) should do the trick:

``` bash
#!/bin/bash

for file in $(git status --porcelain | awk '{print $2}'); do
    if grep -q "do NOT edit" "$file"; then
        echo "Attempting to change an auto-generated file"
        exit 1
    fi
done
```


Need parameter passing to add additional .js files.


ちょっとわかって居ないので、
どういうケースで困るかの具体例をもらえると嬉しいです。


> should add the codes below to avoid to waste time on processing the remaining of diff 1
> if (s_end)
> break;

No, I need to read the last part of diff 1, and to check it.


Not including general sign up functions because my sister is the only user.


A producer can send message to every topics, so I think we can share it across workers. The producer is ensured ready by the time a new instance of `kafkaWorker` of created


Fixed.
https://github.com/agencyrevolution/ar-workflow-helpers/commit/daac5664bf4d7fb00d719436aa792ffa036f709f


Seems like a good direction, even if it ends up just for some significant code simplification.

Two things to check:

1) The color format of the numpy images returned by opencv is suppose to be BGR, but the glTexImage2D is saying it is RGB. So this may need to be updated.
2) the reason for the numpy.copy that is now removed was because on Linux a tester of the code reported that without this line the image was not being drawn. On Windows it was OK without it though. Need to retest to see if this /oddity/ still exits, or if the new code fixes it somehow.

Also, another speed up maybe achieved by using glTexImage2D for the first frame draw, but then use the glTexSubImage2D call in all other frame draws. At least on Windows I think this is faster because the video card reuses the video memory allocated by the glTexImage2D call, instead of freeing and recreating it every time a frame is transferred.

Thanks for helping on this Jon.

Sol


rpmbuildでこんな警告が出たので

    警告: %changelog に偽の日付: Tue Mar 7 2011 Robert Duncan <robert@robduncan.co.uk> - 1.9.2-p180-1


CentOS7.0ではruby-debuginfo-2.2.0-1.el7.centos.x86_64.rpmが作られた。
CentOS6.6では作られなかった。


```html
<tr>
  <th>段位認定</th>
  <td>SP</td>
  <td>十段</td>
</tr>
```
の例で説明します。(一部簡略化してます)
今、`elem`が`<td>SP</td>`の要素を持った`Node`オブジェクトであるとします。
ここで、`elem.nextSibling`すると、`<td>SP</td>`と`<td>十段</td>`の間にある'改行+スペース'を取得します。
これは不自然なので、`elem.nextElementNode()`で`<td>十段</td>`のNodeが帰ってくるようにしています。


(+ @disnet)


you can revert my commits


If you want to include this into regular tests, put this into chromium-perf.py. Don't change this config file, as it's just an example.


Why does it need to read the last part of diff 1, it seems to just moving the file pointer in this case


merge upstream


Why do we need to do this ?


Please help!Thank you!

I want to use   "get_certificate"  to print  the information of certificate，but I have a problem:

When I edit line 60 in androguard/androapkinfo.py :+1:
                if a.is_valid_APK():
                    a.show()
+                  print a.get_certificate(a)
                    display_dvm_info( a )
And then I run androapkinfo.py  ,the result is :+1:

（False,<chilkat.CkCert; proxy of <Swig Object of type 'CkCert *' at 0x24114e0>>）

How can I fix it? THX very much!


okay, i'll check chromium-perf.py.


Did you created the DT3Linux/HAL.hpp? @procedural


@adderly no, but feel free to PR if you have one


Get -> Gets


Raw text of my resume, not formatted


This is for the add-on I created.  I contemplated making the class inheritable, but it has all of these helper methods that are public that would have to be overridden, making an inheritable class very prone to errors.  I chose to make a global variable that stores instances of the classes that a third-party plugin can access some class variables.


Just got into this, ill have it soon then.
The lack on documentation will kill me. :skull:


all tests pass:
success: http://buildbot.rust-lang.org/builders/auto-mac-32-opt/builds/3011
success: http://buildbot.rust-lang.org/builders/auto-mac-64-opt/builds/3005
success: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-t/builds/3006
success: http://buildbot.rust-lang.org/builders/auto-linux-32-opt/builds/2997
success: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-t/builds/2997
success: http://buildbot.rust-lang.org/builders/auto-linux-64-opt/builds/3000
success: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-t/builds/2993
success: http://buildbot.rust-lang.org/builders/auto-linux-64-x-android-t/builds/2988
success: http://buildbot.rust-lang.org/builders/auto-win-32-opt/builds/2664
success: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-t/builds/2662
success: http://buildbot.rust-lang.org/builders/auto-win-64-opt/builds/1501
success: http://buildbot.rust-lang.org/builders/auto-win-64-nopt-t/builds/1492


fast-forwarding master to auto = ad9e7593


@rafaelfranca I think it only warrants a minor version bump, since it should be backwards compatible


The class isn't ideal to be inheritable as it includes several public helper methods.  If this class were to be inherited, it would have to override these methods, which is not ideal and is very error prone.

I created a global variable in index.php that stores instances of the class, and we can access class variables from within that.


setting a baby to breeding false will make it grown.


Add-ons my friend :D


Yeah... The code is structured well, though...


saw approval from Gankro
at https://github.com/pczarn/rust/commit/841b458667eed5ccdc0f47fd54398acfe2cd676c


merging pczarn/rust/ring_buf-collections-reform = 841b4586 into auto


status: {"merge_sha": "875bcbdec697eba49b20fa68568c7e7417bb6d2a"}


pczarn/rust/ring_buf-collections-reform = 841b4586 merged ok, testing candidate = 875bcbde


Here just only to decrease f_len.
But when f_len equal zero, the checking will happened in the read process.

BTW, this commit have already removed.
You need to refresh the web to comment.

2015-01-05 10:40 GMT+08:00 Li Wang <notifications@github.com>:

> Why does it need to read the last part of diff 1, it seems to just moving
> the file pointer in this case
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kylinstorage/ceph/commit/e75510287ce6668153aa7228e496e283f2aa399a#commitcomment-9148307>
> .
>


Probably need to handle the carry bit better as well in these cases. At any rate, this fixes the wonky text in Megaman Battle Network 4 and (apparently) prevents the intro video in Kingdom Hearts: CoM from freezing.


Bootstrap 3 nya theme ra dayon vic haha kay para lahos na


Oops, misread this line; it should be fine (though the outer parentheses are unnecessary).


updated code based on your review/comment. we have a slightly messed up setup in our config dir where we dont import secor.common.config so any new config breaks the setup. which was the motivation behind (trying to) introducing defaults. will fix our configs separately.


82420068_GridViewMultiEdit (evanweaver) - Win64-Windows-7-VisualStudio-12: OK (2172 of 2188 tests passed)

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


Thanks. Changed here: https://github.com/angular/protractor/commit/a8d07a67acbbfcff00d0b8f509f2607162aa4a9b


@leejones this look better for throwing and testing errors?

I'm wondering if I should move the error classes into an AwsMfa::Errors namespace and/or put them in a separate file.


Ah, makes sense; reverted.


OK, thanks. I should ask "what name I have to give to new branch" :). I don't known if i.e. "Auto-G1 bugfix" will be ok?


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


what do these packages do to interfere with retropie?


That is what is happening.  Look at unrar around line 725. That is where the over read is starting at.

Now, as for making it 'not' happen. That I am not sure.  We need to have a large enough buffer, BUT was also need to have the filled with some data that will in some how cause the unrar process to short circuit quickly enough. So pre-pad with NULL, or 0xFF, etc (not sure what is best). Then when decompression walks off end of buffer, the algorithm would quickly bail out.  BUT this may require a larger buffer (64k??)

I really think this one should simply be ignored.  It is going to happen, the times when we do have to rar-decompress garbage data.


I would check to make sure that @slideshows isn't being used in the view.


Consider wrapping the next five lines into a utility function.


variable name


It was transient and is gone now.


Fwiw, addresses in Brooklyn end in `Brooklyn, NY`, those in Queens end in the name of the town, for example `Maspeth, NY`. Not sure about Bronx/Staten Island. My experience with most user-facing geocoders is that they are smart enough to fix this stuff up.


Did you mean to commit this print statement?


r+


Do you have a test for something like
```ts
`${foo}${bar}`
```
where there is an empty head but the first span literal is empty too?


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


If you like this name, take it. :-) Branch and name will go away as soon as the result is cherry-picked to experimental, anyways, he he :-)


Ok.


Oops... wrong branch...


Thanks, @pboling. :smile:


NO?... I mean no. :-) thanks for the catch


I am getting the following build error, while trying to build System.Console solution in Windows7 Operating system "Predefined type 'System.Object' is not defined or imported". is this a known issue if so is there any work around ?


Does that even compile?


use up diskspace. pulseaudio could potentially use some cpu cycles if running in the background unused.


Better throw an exception here


The issue with this versioning is that `[1FDB1D61]` in the URL is actually a CRC32 of the file that is being downloaded.


`hasJail` would've been a better named


This patch is very site specific, not portable, and may have other problems. I recommend against this patch being included in the main codebase.


That came over from the OTM implementation. Now that you call attention to it, it is probably just adding noise. I will remove it.


static import?


A curses client would be worth doing for sure, but I figured it would be easier to rewrite existing code rather start from new. Also I'm a noob and I suck at programming.

I wasn't aware that GetRegisteredContacts makes a call to the server, that's definetely unnacceptable for performance. I'll try to rewrite the pull request so that it caches the information in RAM like you said and does the lookups that way. I think you're probably right about the flag being the best way to do things - but I think the feature should be there because it's confusing, even if only for testing.


you can remove the space between </strong> and changing


This would be a nice util


here also you can remove the space between </strong> and changing


Wrong variable and simplifiable with `!jail`


84202670-DuplicateNodeInZoneConnCausesCrash (JasonGlazer) - x86_64-MacOS-10.9-clang-Debug: OK (1576 of 1576 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)


There seems to be something missing here, no? On the other hand, you're not using `indicators` anymore in the remainder of this function.


Can you provide an example? In the begin-rescue-else construct, the rescue clauses do not rescue exceptions raised in the else clause.


you misspelled canola


:+1: Thank you for this patch


r+ p=2015


A pawn is bound to its player, a `boolean` isnt enough.


"in der Nähe von dir"? Wohl eher "In deiner Nähe" - der Genitiv ist wohl diesmal des Dativs Tod...


One thing to also test is that binary plus and minus get parenthesized in template expressions. For example:
```ts
`s${1 + 2}`;
`s${1 - 2}`;
1 + `${1 + 2}s`;
1 - `${1 - 2}s`;
```
should emit:
```js
"s" + (1 + 2);
"s" + (1 - 2);
1 + ((1 + 2) + "s");
1 - ((1 - 2) + "s");
```


@hadley: not so soon, date/time formatting/parsing features are scheduled for v0.5-1
anyway, internally `width <= 0` is the same as `width = 1` in stri_wrap (the one in the repo).
So maybe `stri_wrap(..., ifelse(width <= 0, 1, width), ...)` as a workaround?


TESTS FAILED
 Mr.Roboto url : http://jenkins.plone.org/roboto/get_info?push=61212af4c5894066afbbe2e4d8760aea
 plone-4.3-python-2.6 [FAILURE]
plone-4.3-python-2.7 [SUCCESS]


[SideCI/RailsBestPractices] law of demeter
reference: http://rails-bestpractices.com/posts/15-the-law-of-demeter


r+ p=22


Updated PR.


nit: I would s/exponent/suffix.

<serialized>      ::= <sign><value> | <value>
<value>         ::= <number><exponent> | <number>
<number>    ::= <digits> | <digits>.<digits>


But then it would need to be paired with explicitly saying who you are writing your message to, in case there are messages from multiple sources. Hence this begs for a more intuitive UI.


All these values should be present in the config.json file.


I can not note a problem on expo, and i think that is not the same thing, while expo re-parent the screen and is in different mode than a desklet. Maybe what is needed is enter  on a modal mode temporary, while the desklet have the focus.

Is normal that you can not be able to lose the focus. Or this is what occurs on some oldest cinnamon implementation. What i do it's controlling the focus and release the text entry, when the desklet lost the focus. Otherwise what happens is that when you return to the normal state, the carried of text entry continues active on the text entry.


Why do we need to perform a save operation? The purpose of this module is to load the config values from config.json and make them available everywhere in your app.


Do you mind leaving `min_value` and the others below as `#[unstable]`? We have them for `Int` and they are somewhat more plausibly useful. (We'll need to make some final decisions during alpha.)


I don't understand "multiple of 3"


Our team is composed by 5 members. If we all push to master that would quickly become a mess. Next time, please create a new branch, then later merge it.


path encoding is automatically handled by template engines. It is the case
with groovy markup and thymeleaf. Have to look at handlebars. Thanks a lot.

On 5 January 2015 at 21:13, David Estes <notifications@github.com> wrote:

> Just double checking is the path url decoded first? if its not and you
> have asset names with spaces in them you might have a %20 situation. Just
> making sure.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/zedar/ratpack/commit/24f58d7714b7541e636ab875fb8eb0bc78fe1385#commitcomment-9159104>
> .
>


We need to export the nconf object from the module. All other modules will import config.js and will access the config values via nconf.get('key').


this has nothing to do with the PR, please undo


So 0 is not valid?  What does that mean for unspecified resources?


If HAVE_VTUNE is not defined, will this generate an empty shared library ?


Please put space between `[`s , i.e. `[ val ]`.


Are you sure the "is" is required?


Indeed


We should keep the middleware name ensureAuthentication for consistency.


spaces between `[]` here as well please


Thoughts?


ahh ok, would more modifications to the system setup be ok to reduce memory and CPU usage?
like i have a script that i run on my retropie setup after a fresh flashing of raspbian, would you like to see some of these tweaks implemented here? they do reduce ram consumption at least


There's no option to disable the automatic M200, but writing `; M200` in the custom start G-code achieves the same effect…
What's the status of adoption of M200 across firmware implementations and their LCD panels?


Fik du et performance gain ud af det? Da jeg kiggede på det sidst, så det ud til at det var deserialisering, der tog størstedelen af tiden. Jeg overvejede lidt om der var en måde at få rykket det ud til efter serialiseren. Måske sådan at serialiseren i virkeligheden er en adapter til en eventstore?

En anden lille detalje, som jeg funderede lidt over efter sidste gang, er at der jo i mange situationer kan være huller i starten af cachen. Det havde jeg forsøgt at løse i min halvgjorte udgave, men det ser ud til at være væk igen :)


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


`cbrt` is a bit... terse. Maybe unstable pending possible rename?


Copy paste inheritance? ;-)

IMO it should read
```
dica-developer-db
```
or something like this.


is this the recipe where you can just pump in lava and water to make molten obsidian?  Yeah that might be a bit much, although if we make it so that you'd have obsidian by the time you have the means to insert fluids into the smeltery, it shouldn't matter...  still, I don't much care for that alloy recipe, so we can probably remove.


@johanberonius inline styling with ttml and dfxp work. You can also style using configuration options http://support.jwplayer.com/customer/portal/articles/1482067-styling-captions-for-fcc-compliance.


:neckbeard:


`exp_m1` and `ln_1p` are pretty cryptic. Maybe unstable pending possible rename?


This isn’t correct. You are not verifying that the result of `getpwuid(getuid())` is non-NULL. You’ll need to have one more variable, and please add the code in line 689 :).


Example for clarity?  Why is this important?

It seems like this imposes complexity on all consumers of this data, rather than always emitting in normalized form.


don't use `angular.*` within angular. just use `extend`. this is so that we can have a theoretical chance of supporting multiple versions of angular side-by-side - which requires renaming window.angular  to something else.


I'm wondering whether the comments should be aligned... @scottgonzalez What do you think? (I was looking for something like this on the api docs site, but I haven't been able to find it...)


Also would simplify the rules here for quantization


Should be
```
git://github.com/Dica-Developer/db.git
```


Let's create a separate function that implements the work done on this branch. handleDashboardPostsRequest(req, res).


Proyecto nuevo


@yihui @gagolews fix pushed and I'm about to re-run revdep. Will submit after that, all going well.


Could/Should this commit be squashed onto the previous one?


I prefer to leave my history unaltered.


Let's create a function that implements the work done on this if branch too. handleProfilePostsRequest(req, res).


Fully supported by Smoothie, in addition to setting flow rate and filament diameter from the panel.
Currently I use it by setting filament diameter to 1.12838 in slic3r.
Then I set the real diameter in Smoothie with M200.

How about if the filament diameter in slic3r is set to 0 then you do not prepend a M200 ?


use `isString`


@RobertDaleSmith not sure you seen this, but check this out: https://github.com/keen/dashboards

Download the zip, and take a look at the customer-facing example. If you think it makes a good profile page, I could integrate it into the repo :)


In case anyone else is looking at Xbox audio - the sample rate can be calculated like so:
```c
( MagicValue >> 5 ) & 0xFFFF
```
The lower 5 bits relate to the channels (4 - 1 channel / 8 - 2 channel)


removing / disabling running services can reduce memory usage. samba for example. otherwise you are just freeing up some disk space. I made a beta retropie image which has a lot of stuff removed - you are welcome to test it if you like.


@gauntface Pull request?


I'm just wondering why you forked it just to do that. XD


I think rescuing the error will cause the command to return a `0` exit status:

```
> ruby -e "begin; raise \"An error message\"; rescue => e; puts e.message; end"
An error message
> echo $?
0
```
 How about replacing this line with `abort e.message` to get the error message output and a non-zero exit status?


@RobertDaleSmith Actually, here's a direct link to view it without downloading: http://keen.github.io/dashboards/examples/customer-facing/


Sorry if I misunderstood just re-read - you are offering some cleanup scripts ? I'm not sure we want to include much more here right now - at least I was just working on this so we dont remove stuff without users permission. I think a more detailed cleanup script could be useful - but less useful if we switch to using a slimmer retropie image in the first place.


is there a slimmer retropie image you have in mind?


no i think
the dummy entry is only saved when there is more updated snapshot. If we write dummy entry when SaveSnap in compaction, WAL will treat it as a revert-back entry and hope that the next entry follows the dummy entry.


hrm.. the first one is better than current one. will do it.