TeamCity Architecture Team :: Java Platform :: otj-components :: otj-data-structures :: build [Build 8](http://TeamCity/viewLog.html?buildId=1177507&buildTypeId=ArchitectureTeam_JavaPlatform_OtjComponents_OtjDataStructures_Build) outcome was **SUCCESS**
Summary: Tests passed: 33 Build time: 0:0:0


See inline comments on the changes.


You could also take this approach:

````php
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
<title>Control Statements</title>
</head>

<body>
<?php
    // Initalize two variables: $a and $b
    $a = 15;
    $b = 10;

    // Test whether $a is bigger than $b
    if ($a > $b) {
        echo "A is Bigger than B";
    } else {
        echo "A is not Bigger than B";
    }

    echo "<br/><br/>";

    // Initalize some variables
    $x = 20;
    $y = 25;
    $z = 15;

    // Check whether $x is less than $y or equivalent, or neither.
    if ($x < $y){
        echo "Y is Bigger than X";
    } elseif ($y == $z) {
        echo "Y is equal to Z";
    } else {
        echo "Y is Bigger than Z";
    }

    echo "<br/><br/>";

    // While Loop
    $i = 1;

    while ($i <= 10) {
        echo $i++;
    }
?>
</body>
</html>
````


Will this affect things like parsing modifiers for other constructs (beyond classes/functions)?  If so, do we need to report errors for doing things  like "export default enum { }" yadda yadda yadda?


Yeah I think so also


Yeah, I believe 3 is the ultimate solution, otherwise ideally we should check that every XXX Producer has a XXX Consumer and there're a lot. I'll add the code back for now and do 3 later.


@dzt help me make it


TeamCity Architecture Team :: Java Platform :: otj-components :: otj-data-structures :: build [Build 10](http://TeamCity/viewLog.html?buildId=1177509&buildTypeId=ArchitectureTeam_JavaPlatform_OtjComponents_OtjDataStructures_Build) outcome was **FAILURE**
Summary: Tests passed: 33; exit code 1 (new) Build time: 0:0:0


TeamCity Architecture Team :: Java Platform :: otj-components :: otj-data-structures :: build [Build 9](http://TeamCity/viewLog.html?buildId=1177508&buildTypeId=ArchitectureTeam_JavaPlatform_OtjComponents_OtjDataStructures_Build) outcome was **SUCCESS**
Summary: Tests passed: 33 Build time: 0:0:0


Great work so far @tanyalang - impressed to see that you're learning a lot :)

I think you'll be an amazing PHP developer before too long!


Pourquoi tu as supprimé KillCash de PocketMine ?


We should call these something better.  I would hav eno idea from looking at the code that calling emit would be wrong.  Maybe something like emit_doNotCallDirectly.  With a comment on it explaining the only callers that have permission to call it.


Where are you using this variable?


@istx25 Sweet, that makes thing easier for me because I'm more familiar with Swift than Objective-C so I'd be able to contribute easier.


TeamCity Akka.NET :: Akka.NET PR Build [Build 102](http://petabridge-ci.cloudapp.net/viewLog.html?buildId=156&buildTypeId=AkkaNet_AkkaNetPrBuild&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 822, ignored: 47 Build time: 00:09:36


Can we avoid slicing multiple times?


This function is completely pointless *if* `C` and `T` are the same type, or `T` has a completely larger range than `C`. Should we check for that, or is that the intent?


An ExportAssignment creates a default export, here we are looking for non-default exports (as part of enforcing mutual exclusiveness of regular and default exports for pre-ES6 targets).


We should actually fix this up so we show this in the navbars.  I'll let @DanielRosenwasser know about this so we can update things properly.  Also, shouldn't we also have code to handle the function case?


I don't think so... the optionsRect starts at triggerRect.left, triggerRect.top. At least, this is how it has previously worked.


i like the constrain helper, but i don't know if it's necessary here -- react gives us some built-in stuff to achieve the same thing:

```javascript
propTypes: {
  keepOpenOnSelect: React.PropTypes.bool,
  preferredHorizontal: React.PropTypes.oneOf(['left', 'right']),
  preferredVertical: React.PropTypes.oneOf(['top', 'bottom'])
},

getDefaultProps: function() {
  return {
    preferredHorizontal: 'right',
    preferredVertical: 'bottom'
  };
},

getInitialState: function() {
  return {
    id: uuid(),
    active: false,
    selectedIndex: 0,
    horizontalPlacement: this.props.preferredHorizontal,
    verticalPlacement: this.props.preferredVertical
  };
},
```


Why not just roll this code into getDeclarationName?


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1140 of 1140 tests passed)

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


An ExportDeclaration with no exportClause is an `export *` declaration.


Great comments.  THanks.  It's super helpful.


Look that : ( my warp is "ressources" ) ![image](https://cloud.githubusercontent.com/assets/10122049/6495506/2bc88b16-c2cb-11e4-925d-c2635b178e0d.jpg) :/


Module names needs to be corrected. It should be "app.services.healthKit" to follow the same convention.
Check others as well.
:bug:


much clearer.  thanks!


It's not better. In fact, we did some benchmarks and it's a bit slower (it turns out openssl/md5 is implemented in assembler). But it's like 2 times slower, and openssl/md5 is very fast, so for now it's acceptable.

The problem was that the compiler uses MD5 after [this commit](https://github.com/manastech/crystal/commit/e7a54eefd56f1bea0227809c0f789838bba89b13) and so it started depending on libcrypto. That meant changing our [omnibus recipe](https://github.com/manastech/omnibus-crystal) to take that into account. That, or implementing MD5 in Crystal. So we did this last thing, which is easier for us for now, and also makes libraries using only MD5 free from the libcrypto dependency.


I tried "essentials.sign.use.warp.ressources" but doesn't work xD


still not getting why this only applies to ES5 and below.  could you claify in the comment why we do things differently for ES5 vs ES6.  Thanks!


Once we add inline assembler support we can improve this ;-)


No, `default` is considered a modifier only when it immediately precedes `function` or `class`.


Sorry about that. I tried with `Pkg.publish()` twice, but it rebooted my machine both times, so I just gave up and pushed directly. Should be fixed now.


Need to fix this.


develop (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1140 of 1140 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)


Does this include type aliases?  If not, can we clarify that in the comment?


We no longer have the annotation on the last expression of `body` here. Shouldn't we put this both places?


I have : "essentials.sign.use.warp" and "essentials.warps.ressources". He Always Tell me : [Error]... :(


If you're using EssentialsPE 1.0.2 or lower, then wait for the new version to come out, it should include that error


Awesome


@samth Isn't that redundant?


Hi. The script isn't working. When I installing script I see next info -  Warning Violent Monkey - script sketchfab-scene-exporter has no @garant lines


Because we'd have to pass an extra (parent) argument just for this one case.


jordansaints, thanks for the downgrade instructions. I am back to the last working commit and all is well.

I am never updating sickbeard again, lol


Currently, I think we'll propagate that information via some special cases in `tc-app`, but I'd rather not rely on that.


I'm pretty sure the `length` in this signature is just a type variable, which may cause issues with name-resolution. I.e. if we import this signature in a different module, `length` will not be fully qualified.


:-) I am not debating the race, i am saying the race exists either way:

What I want to avoid:

Thread 1: tenantDao.addTenantKeyValue(key, value, isSingleValueKey(key), internalContext);
Thread 2: retrieve the key, which DOES not trigger triggers a cache miss but returns the old value
Thread 1: tenantKVCache.remove(tenantKey);


If you have an association on `Topic` like `belongs_to :user` you could use it here like:

```ruby
@topic = @user.topics.find_by(title: params[:subject])
```


actually the events are called
draw:drawstop

so kind of double prefix as well ...


See comments in final code.


75127368_DCFSEC (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1644 of 1651 tests passed)

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


Sooo annoying when thumbs are included


Definitely comment this :)


@jmilum Thank you for sharing and adapting the code for CALCview. I have tried to compile theCALCviewwin on my windows PC but had no luck (unfortunately I am more a process engineer than a software engineer). Do you know or there would be by any change a .exe file for the CALCview?
I read the description you gave on the previous page and this program would be very helpful for me.
If you would have anything to help it would be great!
Best regards,

Maarten
maartenworld@hotmail.com


She will come soon or not soon ? xD


In JS, the final item in list should not have a comma. It causes some browsers to crash.


![violent monkey](https://cloud.githubusercontent.com/assets/11303756/6495656/c0af6fc0-c2d4-11e4-8843-5522e1b50062.jpg)


JavaScript


Maybe I'm missing something but in your scenario, there is a race condition indeed but the end result is correct (the cache is populated eventually with the new value). However, in mine, we end up with the wrong value in the cache?


You could also look into the `.first_or_create` method to do this without the `nil` check.


:+1: I endorse this sidequest


Not that soon... I'm very busy to work on the missing things and to fix the actual bugs


@samth It can't hurt to keep it in, so I've reinstated the annotation.


75127368_DCFSEC (Myoldmopar) - 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)


develop (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1140 of 1140 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)


Being able to change resolutions would be a good addition. Instead of using winforms, may I suggest the following usage pattern:

* The game defaults to a resolution of 800x600.
* There is a "Settings" button control at the bottom-left / bottom-right of LoginGump.
* Clicking on the settings button loads SettingsGump.
* SettingsGump has a drop-down box. The drop-down box is populated with common resolutions. Possible settings might be 800x600, 1024x768, and 1280x800. The drop-down box is automatically set to the current active resolution when the Gump is Initialized.
* Upon selecting a choice in the drop-down box, the resolution changes, and a MsgBox gump opens allowing the user to confirm or cancel the resolution change. If the user does not select either confirm or cancel, the program returns to the previous resolution.
* I would recommend not creating a new State for displaying SettingsGump, but rather, an entirely new gump that opens on top of the LoginGump (in effect, LoginGump is still running in the background). This way, we could potentially use the same SettingsGump throughout the program, no matter the current active running state or model.
* To make certain that no other Gump can be clicked while SettingsGump is active, set SettingsGump.IsModel = true.


Bad copy/paste, the method is `getLastName`


develop (jasondegraw) - i386-Windows-7-VisualStudio-12: OK (1140 of 1140 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)


Makes sense.  Thanks for clarifying.


This has a bug where updating an existing mailing will inadvertently change the mailing_type.

It's better to set defaults using _civicrm_api3_mailing_create_spec() -- the default will only apply to new records, and the default will be inspectable (e.g. for tools like API explorer).


maybe this shouldn't be here? Is this for later?


The beta versions can we crash the server?


.... which is really unfortunate. In Drupal/PHP conventions, the trailing comma is required. The trailing comma is actually a nice practice (because it makes for smaller diffs); but it's incompatible with (IIRC) IE.


That worked, thanks. But check out this build failure: https://travis-ci.org/company-mode/company-mode/jobs/53113914

Looks like there's a stray paren somewhere. Does `make test` run fine for you locally?


Hey @tute , this line %i is ruby 2 code. Is doorkeeper still intended to run against 1.9.3?


It seems the right way for adding binary data to the project like webfont files must be done by releases on github: https://github.com/blog/1547-release-your-software


Quite a lot of stuff. The GUI, where it downloads mods from, some more cosmetic improvements, and a small API that uses FMLInterModComms messages that mod authors can use to add/edit their mod in Beacon. :)


This beta version "3" Fix this bug ?


They're expected to be stable enough to run on a server, but we provide no warranties that it will not crash, bugs may be there, that's why they're "Betas"


We do need to clear the tag where the // should we clear this tag? comments are, right, per the discussion last Fri?


88734010-AFN-horizontal-openings (lgu1234) - x86_64-MacOS-10.9-clang: Tests Failed

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


The*


good call with the err.


this form looks good to me, random question is there a validator attribute we're supposed to add to forms? like ng-required or something?


`gender`?


there is no return in that function, void? is it possible?


actually prints:

  var obj = {
    a(/*before*/ param) /*after*/ {
    },
  };

should print:

  var obj = {
    a(
      /*before*/ param
    ) /*after*/ {
    },
  };


pekny ;]
nejakej tip, proc to dava jiny vysledky?
Je dobre to prelozit s DEBUG2 a u Dijkstry se objevi podrobny vypis behem vypoctu.

U F-W se muze pridat vypis matice behem k-nasobeni, ale podle me, je to tak jednoduche, ze tam snad chyba byt nemuze...


weather app gh pages


Redundant import in https://travis-ci.org/BuildCraft/BuildCraft/builds/53100098


a thousand pardons, good sir, but i beg to differ.  using `.top`, and opening a menu with the trigger positioned near the bottom of my viewport, i sometimes get a render underneath which stretches beyond the bottom of the viewport and results in a "scroll-jumping" behavior, but using `.bottom` that issue goes away.  i can illustrate it with a screen capture if you'd like.


This specialization is created in order to declare T buffer. Without it we attempt to declare a void buffer...


why are we injecting $state? sorry if it's here I don't see when we use it


Semicolon


cool sammiches. :shipit: :8ball:


I think this setting is currently buggy on mainstream kernels, I'm not sure we should enable it.


what's a side sandwich?


This looks pretty good. Unfortunately, due to my delay in reviewing, we need to move over to 4.6.beta3.


Very good plugin


Update from original


If you want to PR this, I'll merge/investigate/fix it.


Wow this is da best y delete :P - Gegy1000troll


@semarie: This patch is needed for DragonFly. Please apply! Thanks!


This change eliminates requests' batching for all applications not limited by available network bandwidth, because clientWriter goroutine won't block neither on Flush() nor on Write() calls - it will only block on the "<- c.requestsChan" when it is drained (and it will be drained all the time). The OS usually has internal per-connection write buffers, so it doesn't block on send() syscall if these buffers have enough free space. And these buffers almost _always_ have enough free space if the network has free bandwidth to use. If the network has no free bandwidth for RPC, then the c.requestsChan will be always full and the majority of Client.Call() requests will eventually end up in timeout error.

While requests' batching has negative sides such as increased latency and 'thundered herd' problem, it has the following benefits, which have higher priority in throughput-oriented applications:
1) better compression ratio;
2) lower number of send() syscalls;
3) lower number of network packets and network packets' overhead.

So this change won't go.


why make it client only?  Does this mean I cannot use this package on the server side?


See my comment for https://github.com/funny-falcon/gorpc/commit/d4f2053850f6402c2e0ffcdbb67f0e32ecc60cdf


@channel: Open to your thoughts/ideas but I was thinking we could

1. create github issues for these talk topics we want to see (group w/ a label)
1. send a mass email to the group and point them to the talks
1. research people in the area and reach out to them specifically


LGTM, (+1)


As example of the update bug -- when you work on an A/B Test, the A+B are initially created as "experiment", but they switch to "standalone" when you submit/send the mailing.


looks good my hope is that these merges don't affect ours but we'll just resolve them I suppose. Also should this novalidate, ng-required stuff be on your sandwich forms?


88734010-AFN-horizontal-openings (lgu1234) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


I think it's good to merge, I wrote some comments for things to think about in the future.


Basic soci question: How does soci match up the parameter names (eg `:bidu`) to the corresponding `soci::use(ubid)`?


@guzza MySql works fine with 7.2.2 and previous versions too... but you can't have case sensitivity on. IIRC the installer should give you a warning or put some error msgs in your log if you have that enabled. I just downloaded 7.2.2 and installed it on MySql without issue on windows without case sensitivity turned on.


@mattbookman

Per our discussion, please take a look at http://docs-test.readthedocs.org/en/task-oriented/index.html


Fixed all 3 copy-paste errors. I somehow in my mind interpreted the error message(s) as generic ("method is applied to a non-person") not realizing there is a method/action name there


here we go --
`triggerRect.top`: http://screencast.com/t/IOJOxxPm3Bz7
`triggerRect.bottom`: http://screencast.com/t/5aUNFNysP


This PR is of course to help fix that issue so it supports case sensitive queries... the trouble there is that many packages will probably not work for that scenario and the legacy APIs will probably not work for that scenario. It's just best to configure your db to be case insensitive.


yes we will do this later


Ah fair enough; I guess there could be platforms where it differs. In any case, no harm in turning the bit off anyway. Now done in latest source (-r356)


I guess that's fair, tho I'd still argue the common case would be section, and this pie in the sky implementation would then allow e.g.:

```
:rfc:`4462#p5`
```

(or `s/p/page/`) or whatever.


But it's so beautiful now!


s/vesions/versions/


QwtUpdate (macumber) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2086 of 2201 tests passed)

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


There's no i7 branch on Github btw.


Here we may add `if !User.current.didLoginWithFacebook` Depends on how we will solve multiple facebook login problem


Just fixed this :)


A sandwich that can only be eaten on the right side of the road.


wow u kewl


I think the answer here is yes, but I have one test program failure I can't currently explain and want to track it down first :) I'll uncomment it (and fix the identifier) soon

On Mar 4, 2015, at 6:24 PM, Isaac Evans <notifications@github.com<mailto:notifications@github.com>> wrote:

We do need to clear the tag where the // should we clear this tag? comments are, right, per the discussion last Fri?

—
Reply to this email directly or view it on GitHub<https://github.com/riscv-mit/riscv-isa-sim/commit/cb388c044095fda00a678598fb9610b07149f915#commitcomment-10041384>.


I can propose much better way - you just need to create one array of hashes with next keys:
```ruby
CONVERSIONS = [
  {from: :celsius, to: :kelvin, convert: ->(t) { t + 273.15 } },
  {from: :kelvin, to: :celsius, convert: ->(t) { t - 273.15 } }
]
```
and use
```ruby
CONVERSIONS.select { |x| (x[:from] == from_type) and (x[:to] == to_type) }
```


We will address this later (issue #32 )


@elizabethnguyen  ???


Thanks!  Could you add something that makes it clear that the new steps are only for NixOS users?  They don't apply to MacOS or non-NixOS linux, both of which are also supported by this script.


you can avoid this additional checking by using array of hashes with keys `from`, `to` and `conversion` (lambda) and `select` method of `Array`


thanks to Freek Wiedijk for pointing this out


@liz-cc  why?


IMHO, I think I like the individual assignments pattern better than the `soci::row` pattern anyway, because `soci::row` can separate the outputs from the query, which leads to errors over time.


@jobasiimwe , this is incorrect usage of scope.$watch. The first parameter in this case should be 'ward' (as a js literal string). Perhaps this is why things weren't updating as you expected...
cc @killera since I think he was going to be looking at the remainder of this story.


It would be useful if the error messages distinguished the post-create and pre-destroy cases, e.g. "Network plugin post-launch hook returned an error" or similar.


This is what happens when you use tabs and not spaces in Javascript </rant> :)


I was using Firefox latest version.


Nitpick: You're mixing initializations between here (703) and above (698). I think they're equivalent, but stylistically you should pick one or the other.


Leave some room for me, I plan to do some in the future :P


I think we should use rem() or pixels everywhere. Right now there is a mix of them. Let's chat tomorrow with Remi about what we want to do there.


@subroutines You can just change the name in the setting for this repo. No to create a new one (which already exists).


Removed for now


Oops, this should be --enable-fpm


Fixed. Thanks.


This is more of a style issue, but you want to have a space between key and value, like Video.create(title: 'Southpark', description: 'lol', category: comedy)}


what is generating all that stuff? I haven't seen it


Nitpick: Extra space between `*` and `db`.


Hi!

Can I ask why you need to build Java and Scala first? Do you need to have the business domain of your app in Clojure and call Scala from there?

In my  use case, I needed to call Clojure code from Scala, so I did it in that order.


just a little comment.


manifests is for data files, packages for python modules.


I noticed you keep hash without spaces before and after {should have_many(:videos)}, while its not incorrect i barely see it like that in ruby code. I would suggest to use this syntax { should have_many(:videos) }
And also should is older syntax - try using is_expected.to


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

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


:sob:


@nouc23 can I assume that PR?


will do.


Dependency injection is a design pattern in which a service (object) is injected in another. Dependency Injection decreases coupling between a class and its dependencies.
here, I really wanted to say that the jQuery object was injected into the $di object, not only used...
En français, " l'instance de JQuery est injectée dans l'objet $di en utilsant le mécanisme d'injection de dépendances"


I have more stuff that i need to merge in for this to actually be useful. I have it up and working on django 1.7, but none of that is visible yet.


develop (jasondegraw) - Win64-Windows-7-VisualStudio-12: OK (1140 of 1140 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)


When you describe a method in RSpec - there is a convention to use. For instance methods you prepend # and for class methods you prepend .
so this should be `describe ".search_by_title" do`
it helps massively to a person who is reading it after you and it will help yourself later on.


Here's the original: https://github.com/manastungare/google-calendar-crx/blob/master/src/_locales/en/messages.json#L63

    "message": "Google Calendar for Chrome lets you check your calendar while you visit other websites. For this, you need to grant it access to your calendar. This is a one-time setup step required before you can use this app.",


https://github.com/manastungare/google-calendar-crx/blob/master/src/_locales/en/messages.json#L93

"message": "Change options for this extension",


Causing error while statring app with config file `file.js` with content like
`module.export(/*...*/ static: ['*.js', '*.css'] /*...*/)`
Error like `Unexpected name 'static'`, throwed from [api.definition.validate](/tshemsedinov/impress/blob/master/lib/api.definition.js#L209).
Should be reviewev and tested.


Many thanks!


75127368_DCFSEC (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (634 of 635 tests passed)

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


Good job updating seeds ! :)


what's it?


Tommy, have you ever considered that perhaps TC2 should put the records in the table? We should probably go back to the drawing board with this. It looks like you've gone rouge here.


this could also be
` = image_tag "result.small_cover_url"`


Don't forget, to start the Putty session you'll need
![Multipass](https://31.media.tumblr.com/e34c51fc4a72ece1fa04e08dabb39108/tumblr_mzagk0EaKF1s4jjovo1_500.gif)


`length` is indeed a type variable. Once my modular verification work lands, when we turn `length`'s signature into a LiquidHaskell signature, we can resolve `length` in the context of the module the signature is defined in. Then when we refer to `bar` from other modules, we can refer to that LiquidHaskell signature instead of re-processing it in a different context. Maybe we could even use GHC Annotations for this.


you can also write order like this - `.order(created_at: :desc)`


This is great work Strott

![Dance](https://33.media.tumblr.com/46356cf6b6b85c2ae42a0d6f0d8abdc6/tumblr_mxmun96Dlj1slx273o1_400.gif)


:stuck_out_tongue: Sorry about the `\!\!\!`... Git Bash kept throwing errors because of the ending `!`, so I thought that the backslashes would break them out... :x:


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


r+


I dunno about this Tommy, I think we need to reconsider this.

![toliet](https://frenz1964.files.wordpress.com/2010/07/oie_borat.gif)


Sure, we can do that, but I thought the point of this idea was to make GHC do all of the work for us, transparently :)


The nice thing about using type variables here instead of type-level symbols (Niki's idea!) is that we can extract their sorts from the kinds GHC infers for the type variables.


You're welcome.  It's refreshing to see you finally using some DVCS.. :)


Dillon! Hey man you need to get off the boards! The admins are getting on my case!


Fix for interactive? http://pastebin.com/cysn0sgi Did you get this as well?


Shouldn't "px" units be included, "-9999" what?
This also looks poor form, probably better to apply a CSS class.


:heart:


T'es sûre que c'est la meilleure façon de compter? Tu utilises ton iterator et il n'est pas parallel. Il y a bien un meilleur meilleur moyen en utilisant l'API de Spark pour ça non? (Think WordCount, reduceByKey, count, ... ) :P


saw approval from Manishearth
at https://github.com/Adenilson/servo/commit/a31d3d7f701b0194d9d83d6b684ab7718b9a954a


merging Adenilson/servo/removeFlushLayout01 = a31d3d7f into auto


Adenilson/servo/removeFlushLayout01 = a31d3d7f merged ok, testing candidate = 34289943


Iterator? Why?


Yep, exactly. I use rule system written in closure [clara-rules](https://github.com/rbrush/clara-rules/):

 - facts are scala classes and need to be compiled before
 - they are used by rules written in closure.

Overall it looks like [this](https://github.com/tomaszym/izabela/blob/master/rules/src/main/clojure/cirkulerilo.clj)

I had no previous experience with closure though, so your plugin was really a huuge help, thank you! :-)


Yes, it was... I've fixed it, try it now.


We aren't using type-level symbols anymore, so I don't think we need `DataKinds` after all.


I don't think we need `KindSignatures` either.


...and I don't think we need this import.


The admin is like hey strott who's dillon? My friend. Why do you ask? He goes he's posting on database management systems github. Do you know what period he has database management systems? NOOOOO YOU GOT TO BOOT HIM OUT NOOOOOOOOOOOOOOWWWWWWWWWWWWWW!!!!!!!!!!!!!!!!


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


89438532_RadianceBugFix (rpg777) - i386-Windows-7-VisualStudio-12: OK (2193 of 2213 tests passed)

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


> All functions in this mode will be TESTNET-ONLY (eg. send_MP).

Running in testnet sort of implies it's testnet-only, but on the very first glimpse one could think Omni Core is inteded to be run on testnet only.


:clap:


Since you have the most mentioned users, and they are sorted/ordered ... here it's just a simple .first or take(1) ??


Sorry, didn't see your response at first. Type variables gives us more than type-level strings does (sort information), and with measures-as-functions, we get to use GHC's name resolution system to resolve them.


89224666-FixNANProblems-Issue4557 (mjwitte) - x86_64-MacOS-10.9-clang: OK (1639 of 1641 tests passed)

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


ERROR : Wrong commit status : Display red items and cancel send of the form when the mandatory inputs are empty


Wow, good catch. Can't believe I missed this.


Hey @mxw, I know you are probably really busy but I just wanted to check in on this in case it got lost in the shuffle. Is there anything I can do to help get this merged?


HVACSizingSim_CoincidentPlant (EnergyArchmage) - i386-Windows-7-VisualStudio-12: Tests Failed

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


```CombinedReportWriter``` looks like it could get a little messy. You're passing multiple kinds of things to the constructor now: instances of ```ReportBase``` and also straight up ```DataFrame``` objects.

I was hoping you would find a way to subclass ```ReportBase``` for all of your reports. But sometimes you just have to sit with a problem for a while, try anything that works, and then refactor later.

Let me know when you're "done done", and we can take the 10,000 foot view of the project and refactor everything.

Keep writing tests.


\v and \d need one more slash each here (currently this search fails), I'll fix it after merging.


IDFXT (refaqtor) - Win64-Windows-7-VisualStudio-12: Build Failed

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


Not necessary.


Why is this file and jankyDrivestick.h in the ROOT of our repo? And they look OLD as well. @kimboZ this was your commit...please resolve this.


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

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


Just getting started,ah ha~


code does not seem right


why would you do this in a loop? You want to poll several times and notify all tasks each time? What is the point? Is it because you are not sure how soon the metrics actually update? Then you better wait N * DEFAULT_FREQUENCY. Simply scheduling polling N times at the same time (FREQUENCY + 1) won't help much.


hry


.AppleDouble


rax retry tests


This will get called over and over and over again.... is that what you want?


where was this in the old code? I'm not seeing it so hard to say what it was meant for if anything


It doesn't work on WC 2.3.5 in a child theme. I tryed to put the same filter inside the parent theme but nothing happens.


@braver Hehe.. no no.. One is just trying to catch up to Isotope. It's great you explore all the risky stuff first. :stuck_out_tongue_winking_eye:

btw. `body` now inherits from `html`: https://github.com/atom/atom/commit/2dcdbb5a927e395653e00a7d5c2a104b77a0a6d9 So you don't have to style both anymore.

Also One uses only `em`s. That way you can resize individual areas and not just `html`. Not sure if you wanna try that for Isotope too. It could get tricky.. when things get nested they might blow up too quickly.. so sometimes I used `font-size: inherit;` just to keep it at `1em`.

And only the "main" `@font-size` is set to `1em`.. the rest just uses multipliers like `@font-size * 2`: https://github.com/atom/one-dark-ui/blob/master/styles/ui-variables.less#L149-L163 It's just so you could easily change to `rem` or back to `px` if it starts to get too problematic.

But yeah.. g


@judge2020 Yes, .ttf is the TrueType font extension.


Hey Nhu, can we revert this? I purposely didn't have the author html files in the same template folder based on Django 1.7 conventions: https://docs.djangoproject.com/en/1.7/intro/reusable-apps/

The lab tutorial is using an older convention I believe.


Oh interesting.  Wonder how many people this is actually affecting?


Is this a copy/pasta from line 42, or is this doing something I'm missing?


ohh.. wait. I think you use `rem`s to do the "compact mode". I guess that wouldn't work with `em`s everywhere since they are then tied to the element's `font-size`.


good work.


Crap... I just realized that this won't work since `/tmp/hsperfdata_root` won't be writable and the JVM doesn't support changing that directory...


Oh cool! No, I never really finished. I had a lot of trouble trying to set up the script. Mostly it kept firing and Google sent an email every time it fired and my inbox got flooded and even after I'd deleted the whole project it kept going. I just threw my hands up and gave up :flushed:


@vovkasm, i also find the new project created by templates can't find the resources file path correctly.


86605784-RefrigNoNodeCrash (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (623 of 623 tests passed)

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


On adding a new project, this order causes failure (Can't fetch before clone)

```
Beginning git repo setup
sh: line 0: cd: /data/samson/releases/20150302-230206-c383ad1/cached_repos/42fc6f414ad8786e5a57fddd0fe479ea: No such file or directory
Failed to execute cd /data/samson/releases/20150302-230206-c383ad1/cached_repos/42fc6f414ad8786e5a57fddd0fe479ea
```
from https://samson.zende.sk/projects/billing/deploys/65234


Note: I pasted the commit message wrong. The API to use is `server.address().port`, not `server.http.address().port`.


아래에 있는 내용은 내가 못봤네 ㅋㅋ 저대로 가자


*Just passing*


related/motivated by https://github.com/irods/irods/issues/2535


rax retry tests


It seems to me that `start_streaming` and `stream` could be collapsed into one call, rather than making two:
```
def stream(topic, partition, name \\__MODULE__, offset \\ 0, handler \\ KafkaHandler) do
  start_streaming(topic, partition, name, offset, handler) |>
  GenEvent.stream
end
```
So the example becomes:
```
Kafka.Server.start_link([{"localhost", 9092}], :stream)
Kafka.Server.produce("foo", 0, "hey", :stream)
Kafka.Server.produce("foo", 0, "hi", :stream)
Kafka.Server.stream("foo", 0) |>
Enum.take(2) #> [%{attributes: 0, crc: 4264455069, key: nil, offset: 0, value: "hey"},
%{attributes: 0, crc: 4251893211, key: nil, offset: 1, value: "hi"}]
```


`/public/system` 으로 하자 (다른 paperclip 애들도 저기에 들어가니까)


@TechplexEngineer
Thanks for the comment and interest in this project. It is not necessary to have the lib cache, the real problem was I accidentally removed the txs0108e lib. But for the easy of using, i've added both the cache lib and the txs0108e lib back, and pushed to github.


Steve,
  Bruce suggested:

    def three?(arg = nil)
      arg && self.respond_to?(arg) ? self.send(arg) && self.rank == "3" : self.rank == "3"
    end

just in case I accidentally defined a method called `no_color` in the future, what do you think?  Also, I am going to have to split my time between getting some semblance of the view in place and reviewing rails, so my commits to the model might slow down a bit.


88620944-FluidCoolerWithLowSpeedAuto (jasondegraw) - x86_64-MacOS-10.9-clang: Tests Failed

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


Good! I like the way that you are about to automate GDB.


ah i c. ok

On Tue, Mar 3, 2015 at 8:17 PM, Jessica Yuen <notifications@github.com>
wrote:

> Hey Nhu, can we revert this? I purposely didn't have the author html files
> in the same template folder based on Django 1.7 conventions:
> https://docs.djangoproject.com/en/1.7/intro/reusable-apps/
>
> The lab tutorial is using an older convention I believe.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nhubui/CMPUT404-project-socialdistribution/commit/800f9ad0f02160821759466d1b8ca32f584950a5#commitcomment-10021512>
> .
>


I'm confused. You're checking if the state is Idle ... and then changing states .... to Idle ... is that what's desired?


Not sure why this isn't a `call` instead of `send`. Is there a difference in semantics that we need here?


Hi, thanks for your interest in combine_pdf.

I think you might have found an important issue to resolve.

I wonder if the parser ever returned non hash objects? That might indicate an issue with the parser or a non-conforming PDF file (a corrupt file)... or both. So such a case might be great to help improve the parser.

Also, I wonder if the parser shouldn't fail (raise an exception) in case of non-Hash objects and weather this should be coded inside the parser logic rather then in this specific place in the code where the parse method is called?

Bo.


thanks!

On 3 March 2015 at 20:27, nb <notifications@github.com> wrote:

> ah i c. ok
>
> On Tue, Mar 3, 2015 at 8:17 PM, Jessica Yuen <notifications@github.com>
> wrote:
>
> > Hey Nhu, can we revert this? I purposely didn't have the author html
> files
> > in the same template folder based on Django 1.7 conventions:
> > https://docs.djangoproject.com/en/1.7/intro/reusable-apps/
> >
> > The lab tutorial is using an older convention I believe.
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/nhubui/CMPUT404-project-socialdistribution/commit/800f9ad0f02160821759466d1b8ca32f584950a5#commitcomment-10021512
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nhubui/CMPUT404-project-socialdistribution/commit/800f9ad0f02160821759466d1b8ca32f584950a5#commitcomment-10021578>
> .
>


I think we might need to send the topic and partition along too, in case a client uses the same handler for multiple topics/partitions


Please write down the meaningful GIT commit messages.


this commit seems unrelated to your changes. Can you pull the latest develop instead?


It's definitely non-conforming PDFs. Some of our customers are having trouble when we try to apply a watermark on top of the page - looks to be something generated by a scanner.

Right now we're trying to figure out the best way to handle PDFs like this. Running them through ghostscript sometimes works, and PDFtk sometimes works too. If you open up these files in adobe or the Preview app on Mac and then save them again, it seems to clean up those references as well.

I'll try digging deeper into the parser to see how this issue can be resolved closer to the source.


You should prepare a builtin test suite to validate the implementations.
Also, can you consider [Travis-CI](https://travis-ci.org/)?


89445170CoilVsSpacePeak_ExampleFiles (wfbuhl) - i386-Windows-7-VisualStudio-12: Tests Failed

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


Maybe add a comment, that this is injected via some linker trick. Someone reading this code might be confused otherwise.


why not directly fmt.Printf() ?


Compiling this with:
cargo build --release --no-default-features

and comparing with the original with the same flags I'm see more than 2x the performance on large directories!


Some smallish comments..


Sure.  It can be built however you think is best.  No matter which way it works, the documentation should reflect it.  May I suggest updating the documentation to match the build process for the moment?  Then once you rework the build process, adjust the documentation accordingly.  This way users are not led astray.

Speaking of which, the "make install" step is broken.  See https://bugs.launchpad.net/gearmand/+bug/1401857.


+1 to merge this.


develop (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1134 of 1134 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)


>Uncaught ReferenceError: f is not defined

Should be `iframe.contentDocument`, apparently.


this is not worked if it's used as a NOT nested addon.
"this.parent" would be undefined.


@alexborro I personally find `!(x & (1<<BIT)) != !(x & (1<<BIT))` simpler than `((x & (1<<BIT))>>BIT) != ((x & (1<<BIT))>>BIT)`.


`call` is synchronous `send` is `async` (non_blocking) the action would block the GenServer indefinitely, but we want to return with a pid to the user


("where id in @0",new string[]{"1","2","3"}) error
("where name in @0",new string[]{"a","b","c"}) error
("where id in @0",new int[]{1,2,3}) ok?


Don't add the generated files such as ` hello_c/hello.s` in GIT repository.


LOL, this is one of my favourite ways to fix a test too :wink:


I'd love to get an example file, but I know privacy sometimes won't allow for it.

Let me know how it goes and thank you for helping improve the gem.


:disappointed: would fix this


>Uncaught TypeError: styles.forEach is not a function

See the note above.


Add `styles = Array.prototype.slice.call(styles);` to enable forEach, see the note below.


:) BTW - if you do `git cl land` it will do it automatically for you (it collapses all the commits as a single commit with a description that includes the subject of your CL and the review URL)


Should this be `getAlignmentDuration` (no extra 'e')?


good point, though my thought was that you should create workers for streaming per partition/ per topic since it is relatively cheap to create workers


HVACSizingSim_CoincidentPlant (EnergyArchmage) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


As mentioned here #8, these specs will disappear once #9 is merged.

Now stop :trollface: me, **YOU ARE SO MEAN**.

:stuck_out_tongue_winking_eye:


Why the space between `"> Check`?


Why spaces around the text?


Also, why not add a space between `smeilen.com"target=`?


I'd rather have a self-standing commit message here that explains what is done and why. Just imagine you are looking at the logs without github


Would suggest no space between `color` and `:`.


Would suggest a blank line between sections.


You should encode spaces properly.


Может, сделать событие "процесс запущен", на которое подпишемся в провайдере, и там уже будем регистрировать новый процесс в мониторе? Зависимость-то осталась, на самом деле.


`@date.next_year` 로 충분하지 않을까? (위에 `monthly`, `daily` 에서도 `beginning_of_*` 없어도 되는 거 같음)


@ibara


develop (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1134 of 1134 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)


88620944-FluidCoolerWithLowSpeedAuto (jasondegraw) - 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-79.52%25-red.svg)


@keithbostic

I backed this change out, since it broke the build. I believe you needed to add a ```test/packing/Makefile.am```, when I created a naive version of that file with the three source files in it, I see:

```
/home/alexg/work/wiredtiger/build_posix/test/packing/../../../test/packing/intpack-test.c:35: multiple definition of `main'
intpack-test2.o:/home/alexg/work/wiredtiger/build_posix/test/packing/../../../test/packing/intpack-test2.c:35: first defined here
packing-test.o: In function `main':
/home/alexg/work/wiredtiger/build_posix/test/packing/../../../test/packing/packing-test.c:58: multiple definition of `main'
intpack-test2.o:/home/alexg/work/wiredtiger/build_posix/test/packing/../../../test/packing/intpack-test2.c:35: first defined here
```

The three tests are separate so it seems like a reasonable complaint. I also wonder why you chose to avoid using the public API for the primary ```packing-test.c``` file? I'd have thought all of these test cases would be better via the public API now that it exists?


Yeah! :+1:


@qbit


Best. Collaboration. Ever.
Go team!


德莫


88620944-FluidCoolerWithLowSpeedAuto (jasondegraw) - 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)


2/2
Good work!


I don't know why, no matter what number I add to Date.today, it's always saved to review in on 2015-03-10...


hmm, logically it makes sense to me to check for everything that is passed. the interpreter will automatically pass undefined for you, but i also think like this because i come from strongly-typed languages where we had to write our own proxies for variable argument lengths


I do like Bruce's approach. Whoever mentored him in the past did a fantastic job :-)

You can still make the method a little more readable by removing the `self.` calls. Also, I think since you will be the only one passing in either `:red` or `:black` to the method, you can probably ditch the `respond_to?` call as well. Generally you'll want to save something like that if you're allowing user input to determine which method to call.

One last thing I would change is `arg` to `color`, just so you know that you'll be calling a color-related method here.

    def three(color = nil)
      (color.nil? || send(color)) && rank == '3'
    end

I guess I would change a little more, but you get the idea. That should still respond true to any three, and only true to red or black threes given a certain color.


2/2
Good work! Ruby syntax is pretty confusing but there are a lot of built in functions that are pretty powerful if you spend the time to learn them.


:dancer:


:+1: thanks


Do packages conflict will break in future  building other python software that use Pygments. In this case I can't build both python2/3 software.


I'm still seeing a NPE on line 161 even after a `usState.getUs()` null check was added, presumably in version 1.5.4. Does 1.5.4 have the fix for this?


@nfultz ?
Nobody is expecting that. But I have noticed that sometimes a comment is attached, even if that comment could have been in a commit message.


2/2
Good job on question 2!


Need to add back in


yo, we gotta remove all thse legacy combos from here. no need


no need on this branch anyways.


2/2
Cool solution for question 2


@istx25 maybe something a bit more dynamic? as in your enter your data in the app, not the code? throw it your blocks and other bits of info (time frame, etc) and it'll bundle everything together for you, all pretty looking. or you could have it add each block to your Calendar on your phone (personally that'd be more helpful).


href=" 빠진 것 같습니다.


Well, it looks like "won't work" is relative. The JVM runs just fine, but since `/tmp/hsperfdata_root` is not writable, it skips writing the shared memory file for the pid. This means that tools like `jconsole` won't work in local mode.


Alright, I like it. We can go over why it did not work later and see if we can make it work.


86605784-RefrigNoNodeCrash (jasondegraw) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


Я бы оставил формирование текстового представления там, где оно было, а этот класс пусть только считает. Это позволит убрать немного странный метод `AiName`


2/2
Good work!


is that first line with just `DuckieTV` legit?


88621024-ElectricLoadCenterStorageBattery-Crash (jasondegraw) - x86_64-MacOS-10.9-clang: Tests Failed

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


This code was copied from PySequence.java but this line changed from pyget to __getitem__


 I had the same error - making this change fixed it for me as well.
:shipit:


2/2
Good work! Look into method chaining to get your solution to #2 to be really short.


Do not need to exclude homework.


Erratum:

- Fixes ugly usage of File.join in c07b5f7efce5a3689a08802ada59166fc6ac7534


Forgot comma; change to "ddd, MMM d, yyyy"


So I did some more code golf on the `is_natural?` method and here is what I came up with:

    def is_natural?(card)
      NATURALS.values.flatten.any? {|natural_card| card.same_rank? natural_card}
    end

What you're really trying to get at is whether or not the card is wild or not, so I just lumped all the natural cards together using `NATURALS.values.flatten`, and then tested to see if `card` in question matches the rank of any of them. A bonus of the `any?` method is that it will stop and return true when it finds a match.

So it is still basically like what you had, just a bit more readable.

One way to really improve this would be to just check the rank of the card instead of constantly spinning through the `NATURALS` collection and running the `same_rank?` method:

    def is_natural?(card)
      %w(4 5 6 7 8 9 10 Jack Queen King Ace).include? card.rank
    end

A really nice benefit of this approach is that the tests went from taking 0.11264 seconds to 0.0277 seconds :-D And you could always move that array `%w(4 5 6 7 8 9 10 Jack Queen King Ace)` up into the `CanastaConstants` module.


2/2
Good work! You don't actually have to return anything from your baz function (or set it equal to new_arr)


I tried to fix "q1" vs "ql" typo and accidentally commited my local copy of Gemfile that is not supposed to be in the repo.


2/2
Good work! Look into method chaining to get your solution to question 2 to be much shorter


Whenever creating HttpResponseMessage you must create it with Request.CreateResponse otherwise some of the model state gets lost


Hey, you know we tested that the correct transaction engine result etc was returned in the npm tests for tickets, but failed to test that the ledger entries were correct. The wrong account's owner count was decremented. There's a lesson there.


ALL HAIL LORD MORGAN


test comment


test hello


Может сделать `IEnumerable<Game>`? Тогда `for` в AiTester можно будет заменить на более читаемый `foreach`


It would be more pythonic to write:

```
features = [self._get_features(data) for data in sent]
```


RTFM


```
tagged_sent = [(sent[i],labels[i]) for i in range(len(labels))]
```


88620944-FluidCoolerWithLowSpeedAuto (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (623 of 623 tests passed)

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


@colinmollenhour This syntax isn't compatible with PHP 5.3. Has a decision been made about the minimum supported PHP version for this module?


The above can be made more pythonic


lol :)


Is it possible to use a type that is more restrictive than simply "Type*" e.g. AggregateType?
Possibly a shame that we don't have a ClassType.


86605784-RefrigNoNodeCrash (jasondegraw) - 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-79.52%25-red.svg)


Here's the log output from the test (A good narrative of the back-and-forth between the two threads).

![selection_001](https://cloud.githubusercontent.com/assets/311973/6478290/b3fdab34-c1fd-11e4-9ec9-e03f15f7cc88.png)


$!result.Url
はTestFormDataのgetUrl() を読んでいるのと同義になります。
$!result.getUrl()
と書けるけど省略して書けてしまうというような感じになってます。


 war/src/main/resources/com/aimluck/eip/modules/localization/AipoLocalization_ja.properties
の
TEST_URL
を呼び出している感じです。


TestResultDataのgetUrl()メソッドを呼び出す処理になります。


Сейчас мы безусловно запрашиваем новый `Ai` на каждой итерации. Раньше было не так


 public static void checkMarriageAfterDeathError()
	 {
		 System.out.println("\n\ncheck marriage after death\n\n");

			for (Map.Entry<String,Family> entry1 : getGedcomFamily().entrySet())
			{

				if(entry1.getValue().getMarriageDate()!=null)
					if(getGedcomIndiList().get(entry1.getValue().getHusband()).getDeath()!=null&&getGedcomIndiList().get(entry1.getValue().getWife()).getDeath()!=null)
				{
					if(!entry1.getValue().getMarriageDate().before(getGedcomIndiList().get(entry1.getValue().getHusband()).getBirthDate()))
						System.out.println("PassH");
					else
						System.out.println("Error:Marriage after death for"+getGedcomIndiList().get(entry1.getValue().getHusband()).gettID());

					if(!entry1.getValue().getMarriageDate().before(getGedcomIndiList().get(entry1.getValue().getWife()).getBirthDate()))
						System.out.println("PassW");
					else
						System.out.println("Error:Marriage after death for"+getGedcomIndiList().get(entry1.getValue().getWife()).gettID());

				}else
					System.out.println("Pass");
			}

	 }


@miked-mellanox Did you mean to push this branch to ompi?


Oops, disregard.

Can you sort the whole thing alphabetically?


Лучше вынести в отдельный файл


As far as I can tell, these are debug counters that purely local to this file.  If that's correct then suggest that they
be marked "static".


r+


add file


82420068_GridViewMultiEdit (evanweaver) - i386-Windows-7-VisualStudio-12: Tests Failed

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


EipTTestはデータベースと1対1になっているのですが、データベースから取り出したデータがEipTTestのrecordにはいっていて、それをTestResultDataにセットしなおしているような感じです。

なんでこんな手間をするかというと、例えばデータベースには入れておかなくてもいいようなデータ（例えば登録した人の氏名や今回の場合であればURLからドメイン名のみを切り出したもの）でも画面に表示するときには使いたいケースってあると思います。

そんなときにEipTTestはデータベースと完全に1対1対応にしておいて、TestResultDataの方にだけURLのドメインだけを取り出すgetUrlDomain,setUrlDomainなどを用意しておくとデータベースの方は特に変えなくて色々できるようになるよ、という感じになっています。


Interesting. Nice work! I'd love to see an OAuth2 version of this. I took a glance at the code and will look at it more over the next few days. The implementation isn't trivial (making it unlikely to be accepted as a "Pull Request" to the main jhipster--i.e. difficult to support going forward) but I'd love to see something like this in jhipster. Anyway, thanks for sharing this!


LGTM


這邊可以試著使用references，並設定index。


no se usar esto, jeje


saw approval from shino
at https://github.com/basho/riak_cs/commit/f755f7775daba4b1b4ee659f385e1caee396081c


merging basho/riak_cs/feature/cs-debug-cuttlefish = f755f777 into borshop-integration-1076-feature/cs-debug-cuttlefish


basho/riak_cs/feature/cs-debug-cuttlefish = f755f777 merged ok, testing candidate = 35bdf211


saw approval from glennw
at https://github.com/bjwbell/servo/commit/3b321789620aa7ae96ddfd8165a38a3d5db5ea0f


merging bjwbell/servo/inline_block_overflow_ref_test = 3b321789 into auto


bjwbell/servo/inline_block_overflow_ref_test = 3b321789 merged ok, testing candidate = 07a3ce2c


Can Type* be AggregateType*?


這部分看起來是前一個commit誤加上去的，請試著修正前一個commit來解決。


init


Did you mean to remove everything from here down? I don't see duplicate content for it anywhere, and there's not much left over when all this is removed.


這邊建議補上 only 參數，避免萬一之後有其他action誤觸。


Fine


86605784-RefrigNoNodeCrash (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1123 of 1123 tests passed)

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


K


That scala


That's beautiful


Just about all our sites use the kind of setup @adrianbj described above (one image field specialised for embedding images into an RTE), so definitely +1 to that :)


Here's the log output from the test.  This will help to understand the back-and-forth narrative between the threads:

![selection_001](https://cloud.githubusercontent.com/assets/311973/6478418/a5b51f50-c200-11e4-93df-e4b1332eb0aa.png)


This really bothered me for a long long time. I had to type `C-e` then `jj`/`Esc` almost all the time while coding.

So I decided to use `C-e` to do both things together and just use `C-y` for closing popup.

@ifyouseewendy


89445170CoilVsSpacePeak_ExampleFiles (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1636 of 1636 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)


這部分應該分開commit。


建議使用only，雖然會導致後面寫的參數變多，但比較不會有誤觸的疑慮。


Кажется, ты забыл положить какие-то файлы. Например, в папке `Factories`


@Rochet2
> "Change all sets and maps to unordered!"
NO!

here we have balance between fast for ordered set enumeration (std::set)
f.ex:
for (TYPE:itr::const_iterator itr = XXX.begin(); itr != XXX.end(); ++itr)

and fast for **unorder_set** search:
result = XXX.find(blablabla);

look at http://www.cplusplus.com/reference/set/set/
and
http://www.cplusplus.com/reference/unordered_set/unordered_set/

the best type we understand after profiling...

in R2, where i'm live =) already all variants maked very good (thx for @rsa)
f.ex: https://github.com/mangosR2/mangos/commit/d9d21a98d751e41d0b931d28badb87d7e00fa1dd
and so on...


IDFXT (refaqtor) - x86_64-MacOS-10.9-clang: Build Failed

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


88621024-ElectricLoadCenterStorageBattery-Crash (jasondegraw) - 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)


all tests pass:
success: http://buildbot.bos1/builders/test-riak_cs/builds/859


這邊看起來是之前打錯，請修正當初那個commit來解決。


請嘗試改用 reference 的方法。


89445170CoilVsSpacePeak_ExampleFiles (wfbuhl) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


I think that this is wrong. Well, it causes issues #142  because it turns CMP Rn, $0 into CMP Rn, ZR, which is wrong, because that becomes CMP Rn, R31.


gggg


前面修正成用 reference 之後這邊應該就不需要了。


這邊也可以使用 reference 並設定index。


:+1:

But not bothering me that much. I'm using `C-[` to escape, so all the time I press `Ctrl` then type `e` and `[`. This seems easy for me.


use 'data = open(sys.argv[1]).read()' is better and enough. Prompt is less preferable than command line arguments.
You don't need to handle non-exist file, python will do that.

For short identifiers imported from library, usually we do not use 'from sys import argv', but 'import sys', and use 'sys.argv'. Same for 'os.path.exists'.


r+


Why do we want to prevent duplicates in db?  Don't we want it to keep track of how many times a match was endorsed?


wander why you have difficulties with prebuild.bat? maybe you have no .git dir? I like it more than just copying over :(


I am probably missing something obvious but I don't understand the class CallGraph.  I have looked through
the file and it appears to me that there is a single call to new CallGraph() in narrowWideReferences().  This
then wraps the root Node.


test comment


If infinite is false, I would meet a problem when I apply autoplay. I want to set infinite as false is because it will stop when Carousel at the last page. But now, when the Carousel at the last page, it continue to display from end to front. I think it goes against to meaning of infinite. Thus, I delete the code that will change direction  by autoPlayIterator in my project. In the code of I commit to slick, to be safe, I add an infiniteAutoplay configuration that if the value is true, it will change direction, if the value is false, Carousel will stop.


Where are you testing this? Evidently my test page wasn't exercising this observer.


The question is not related to this commit. Please open a new thread by mailing aesop users group : http://groups.google.com/group/aesop-users. Closing this thread here.


See 'os.path.join'. since windows doesn't use '/' as file path separator.


Would it be hard to have a similar request logger like Adblock Plus? well in Firefox anyways


what about changing those to `collection_select` ?


this needs some validations (i.e. user and course are mandatory)


Same here. Catch an exception only when you know it is your fault. Otherwise, let python raise it.


course already haves a name, not sure why do you need a connector name, ??


This comment is not from me. It just moved.


:heart:


I think we should leave this check in since other callers (i.e. the file browser) may not necessarily check `isValidJSONData:` before calling this, and ideally they don't need to.


Nice! You need a space between `autorize` and `@post`, though (as it is now, it should fail with a syntax error I guess, did you check the logs?)


owner should be the user who created the course.

Changing this to a relationship will allow you to filter "courses created by others" on course enrolment list


all tests pass:
success: http://build.servo.org/builders/linux1/builds/924
success: http://build.servo.org/builders/linux2/builds/922
success: http://build.servo.org/builders/android/builds/960
success: http://build.servo.org/builders/mac1/builds/1139
success: http://build.servo.org/builders/mac2/builds/1139
success: http://build.servo.org/builders/gonk/builds/110


Notice that you will calculate 'max(wordcount.values())' a loooot of times! Calculate only once!


any good reason to comment this? what if save fails because of missing mandatory fields?


Avoid using non-ascii character in commit message~~


コミットコメントの訂正です
× editShapeオブジェクト
○ editShapeイベントオブジェクト


88621024-ElectricLoadCenterStorageBattery-Crash (jasondegraw) - i386-Windows-7-VisualStudio-12: Tests Failed

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


what if save fails due to missing required fields?


@clydei
Thanks for providing changes to Yun


* the `f` error: http://www.imdb.com/
* the `forEach` error: https://www.google.com/webhp?sourceid=chrome-instant&ion=1&espv=2&ie=UTF-8
  (actually I typed some words in the addressbar to google them but only a blank page was shown with this url)


fast-forwarding master to auto = 07a3ce2c


another comment


A silly style thing but I question use of  !strcmp to mean "match".  Purely personally but this is a case in which I think
strcmp() == 0 is the better choice.  But again, this is a personal opinion.


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

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


you can extract this into a partial, so you can reuse it on every form that needs model errors


At long last. I think what happened was I somehow missed implementing tableView: (UITableView *) tableView moveRowAtIndexPath: (NSIndexPath *) sourceIndexPath toIndexPath: (NSIndexPath *) destinationIndexPath within BNRItemsViewController.m.

Frustrating as hell, but I solved it.


Why a static function that takes a WideInfo* rather than a method on WideInfo?


Think it's ok


Hi Patrik Björklund,

We are in need to set default value to Taxonomy Field. I was thinking to assign value at field level. But seen that you have done with this on list level. Probably its lack of my awareness about how default value works for Taxonomy Field, so wants to know more about the need to implement this at list level an also your comments about if we can implement it at field level.

Thanks,
Shankar Gurav


saw approval from jdm
at https://github.com/bjwbell/servo/commit/a35ffe1c6eccb0822d83427bd52327dec02d0496


merging bjwbell/servo/inline-block_fix_min-width = a35ffe1c into auto


bjwbell/servo/inline-block_fix_min-width = a35ffe1c merged ok, testing candidate = 9d58c086


IDFXT (refaqtor) - Win64-Windows-7-VisualStudio-12: Build Failed

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


can you share the above code rather than duplicating it?


Here is another way to achieve the same thing:

```ruby
@topic = Topic.find(params[:topic_id])
@post = current_user.posts.build(post_params)
@post.topic = @topic
```

Does it make sense to you?


err is not guaranteed to have a `json()` method.


duplicated code, DRY it (i.e. extract errors into a partial and pass the given model as a local )


is err guaranteed to exist?


Can we get this applied to Spree 2.4 as well? Thanks @JDutil!


88621024-ElectricLoadCenterStorageBattery-Crash (jasondegraw) - 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-79.52%25-red.svg)


I moved it from assignWorkersToFragments to here because it has nothing to do with workers.


Merge it with AbstractProducerEncoding since all we need here is operator.opId.


Generate harder ones by first adding rotation!
PIL is cool, and opencv for python is also a good choice~  We usually use both together, cause their image format are compatible: numpy.array.


This is done in sanityCheckEdges().


I added "Local" since this method only verifies/propagates LocalMultiwayProducer/Consumer. Also updated javadoc.


EOSController is also a singleton.


Добрый день,
Игорь!

Вы советовали,
чтобы не заливать мусор в репозиторий, положить файлик .gitignore в корень проекта. Сразу так и сделала. Видимо не помогло,
т.к. вы опять пишете, что залит мусор в репозиторий. Подскажите, пожалуйста,
что сделать еще.

Анжелика
Артеменко, “Программирование на Java для тестировщиков”, группа 22.

Date: Thu, 26 Feb 2015 15:54:17 -0800
From: notifications@github.com
To: test-project@noreply.github.com
CC: angelica.nm@hotmail.com
Subject: Re: [test-project] ContactRemovalTests, ContactModificationTests 2.2 (6d71194)

Это "мусор" не стоит его заливать в репоиторий.

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


This worked pretty great for me. Sorry that other code snuck in there, that wasn't intended.


hmm, this endpoint seems correct but the payload still isn't working for me. only getting a blank response. any ideas?


iterativeInput can be an operator, e.g. TableScan.


develop (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1131 of 1131 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)


82420068_GridViewMultiEdit (evanweaver) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


Because the link from the iterativeInput consumer to its producer has been cut, we can't get the schema of its producer here.


The  current travis build is [here](https://travis-ci.org/avr-llvm/llvm/jobs/53002115).


88621024-ElectricLoadCenterStorageBattery-Crash (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (623 of 623 tests passed)

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


@walxfang if you want to do in you must use named parameters.

```csharp
db.Fetch<User>("where id in (@x)", new { x = new[] { 1,2,3 } });
```


My results so far:
https://travis-ci.org/RandomInsano/llvm


I added a few things/notes, please take a look :) @dhalperi  If you think they're good I can merge this PR then.


auto_increment doesn't work for some DBs. Please create separate scripts for other DBs


you can use __construct instead of add function.


you forgot to take the asyncronicity into account
you need to return the promise, to avoid throwing immediately


better safe than sorry


I don't have a good handle on this yet. What condition would lead to the "else return true" branch?


Hi, i am not able to resolve dependency and can i get it with grails latest version ?


Why a function that takes a WideInfo* rather than a method on WideInfo?


88620944-FluidCoolerWithLowSpeedAuto (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1123 of 1123 tests passed)

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


Во-первых этот коммит очень большой. Стоит разбивать свою работу на этапы, каждый из которых делать отдельным коммитом.


I know, no problem. I also tried escaping newline with backslash, doesn't
work either.

2015-03-04 0:08 GMT+01:00 Makman2 <notifications@github.com>:

> Note:
> I tried to directly wrap [documentation] into a link, but the link itself
> was longer than 80 chars and so I need to split it up like that, markdown
> doesn't support linebreaks inside a link^^
>
> --
> Reply to this email directly or view it on GitHub
> <https://github.com/coala-analyzer/coala/commit/7148d343c3c1bcf8b076642038218cdf29dd93fc#commitcomment-10018735>
> .
>


@suhasjog :+1:  test again


Этот файл нужно добавить в solution


@drodriguez It would be great to get this on master if you're interested in pull requesting it.


esta linea hay que eliminarla


В `.h` файлах желательно только объявлять переменные, но не инициализировать. Сейчас в каждом файле, который импортирует этот хедер будет создана своя копия строки. Более правильный подход:

```
(.h)
extern NSString *const errorDomain;

(.m)
NSString *const errorDomain = @"com.LabWork4.serializer"
```


@suhasjog : test


se deberia hacer algo similar como en lpc: $intList = array (...


two newlines needed at top level.


89445170CoilVsSpacePeak_ExampleFiles (Myoldmopar) - 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)


line below this one (can't comment there because its not changed in this patch):
how much -> how many


Coala -> coala. We actually deliberately chose the small c. (I like it better :))


Can you check to see if the update I just did fixes this? I was doing some tests as well. I'm running it through the function I had set up to check for dupes and clean out any empty strings.


sentence beginning with but indicates strong connection to the one before and is bad style, consider setting a comma instead of a dot?


По общепринятому стилю кодирования на Objective-C имя метода и его параметров (продолжение имени) должны начинаться с маленькой буквы. То есть ожидается видеть такое имя:
```
+ (NSString *)serialize:(id)dictionary withError:(NSError *__autoreleasing *)error;
```


eSpeak printer test actually... did you look into the file?


hi This is commit


Слово "параметр" по английски выглядит так: "parameter".


skip_test needs to be defined at module level, since it will skip execution of the whole file.


## this is a markdown file


for more information


@rogertoma very well done


that looks pretty useless, if you can't name me a usecase for this please remove the example.


so better remove the linebreak


Noted Isuruwan. I'll update when I'm including the scripts. Currently scripts are removed and in a stage of deciding whether to put the scripts in carbon-governance or carbon-registry.


Madan, is the Tencent's server die? Why I couldn't send message to you?


all tests pass:
success: http://build.servo.org/builders/linux1/builds/925
success: http://build.servo.org/builders/linux2/builds/923
success: http://build.servo.org/builders/android/builds/961
success: http://build.servo.org/builders/mac1/builds/1140
success: http://build.servo.org/builders/mac2/builds/1140
success: http://build.servo.org/builders/gonk/builds/111


@HamzaAltayri No, it's still WIP, I just added what should be.


submit upstream


S860w+ mt6582 with 4.4.3??? Or S860 with 4.2 or 4.3? I can contribute to w+ but not original.


The code is already committed to the master since last Thursday..
Thanks.


ALL HAIL SAVIOR EDMOND


hgjfhg


Использование типа `id` здесь неоправдано, т.к. позволяет вызвать любой метод без предупреждений от компилятора. Всегда следует стараться использовать конкретный протокол или класс чтобы избежать этого. Вот как это выглядит:
```
for (NSObject<Serializable> *obj in self) {
    ...
```


Mmm... yes, sorry. I'll correct templates in nearest day or two.


These unused variables (path1, path2, path3, path4) give me build warnings.


fast-forwarding master to auto = 9d58c086


По общепринятой конвенции если есть выходной параметр `error`, при возникновении возвращать твой метод должен `nil`.


Hi,
Thanks for your response. Can you just publish the current code for mosse and clean it up later on? or can you just guide me if I can generate the same data through some other library?

Thanks a lot


I'd expect a newline after short description. (Our documentation parser does expect that too, markdown like.)


Looks like a constant to me that IMO wouldn't need any documentation if it were `WORDS_PER_MINUTE`


Seems to be unused?


88620944-FluidCoolerWithLowSpeedAuto (jasondegraw) - i386-Windows-7-VisualStudio-12: Tests Failed

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


It's very obvious that it's optional, isn't it?


There's typo. You write 'canols'. Fix it.


нельзя
у нас не только мобильники!


Could this not just call `$this->calculateReadingSpeed($post)`? Or am I confused about PHP/WordPress callbacks?


`"sp_reading_speed"` should probably be a constant at the top


href-а нет
используйте emmet!


`rxc` is a bad name. In this line I know what it means but if I'm scanning the function fastly to grasp its functionality or so I would need to look up the declaration to see what it means. IMO you could be a bit more verbose. rxc is also not a common abbreviation.


планируете использовать SVG-картинку?


`-----------` 这个应该怎么用？我没太看明白


здесь и много где ещё — наспоблюдение кодстайла.


Could be returned right here.

Might need to be trimmed to one digit after the comma or so?


you don't specify return value by the way and thats most importtant here because you could do that in various ways.


нет текстов :(


модификатор без того, что он модифицирует


some nice refactoring done - more yet to come ;-)


по БЭМ это элемент, а не отдельный блок.


Since the read time only depends on the content as far as I can see, I reckon it would be better to have just `$postContent` as a param here.


это сильно похоже на табличные данные.
где табличные данные, там тег table.


в голом HTML я не увижу есть опция или нет.
плохо
надо чтоб видел.


тоже, что выше — класс-модификатор без того класса, который он должен модифицировать.


AFAIK `get_post_meta` returns an empty string, so should probably change this condition. I think `empty()` would do the trick, but unsure.


88620944-FluidCoolerWithLowSpeedAuto (jasondegraw) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


тут интерактивная карта\просто вставьте div#map


скорее всего, это и есть подвал, тег footer.


快点星！


Nice, but it's not a gradle plugin. It's two part library. The compiler is an annotation processor that generates the code at compile time, then the API is means to access this code at runtime. Probably belongs under "Code Generation" or "Injector".


@till  @cweiske  why is this commit lost  in the history of the main branch?
My composer.lock depends on d5a3e959 and composer install failed.


In #7565 I completely refactor these tests. On a first note they were used to assess performance, in that PR they are used to assess correctness.

Feel free to delete those variables if they disturb you, but they are harmless imho.


89445170CoilVsSpacePeak_ExampleFiles (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (632 of 632 tests passed)

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


@qjia7 can you updated README also as to how to set up libva now (Branch you tested etc)


clear


<html>
<head>
</head>
<body>
</body>
</html>


No biggie but I am surprised that this self-call needs to qualified by LoopStmt::


I don't think this is quite right. Previously, it was possible to add a dependency to an unresolved configuration during task execution. Now, that configuration might be in "TASK_DEPENDENCIES_RESOLVED" state, and we still need to allow mutation (forcing a re-resolve).

We should add an integration test that does this:
- TaskA has ConfigurationA as input
- TaskA depends on TaskB
- ConfigurationA has it's task dependencies resolved to build the task graph
- TaskB adds a dependency to ConfigurationA during execution
- TaskA gets ConfigurationA with the additional dependency at execution time

This test should pass with Gradle in it's current state. We'll need to keep it passing when we change the way we determine the Configuration task dependencies.


Glad to see you like my dotfiles :D


//test


This is a damn clever solution.


앗 감사합니다!!


WTF?


oops.  got one of my changes in there.  hope that doesn't cause you a problem, sorry.


develop (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1133 of 1133 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)


CNAME


done


@trommel Yup, I have hardcoded the entire application (for simplicity), but it would be super easy to point the content of the labels to `UITextField`'s. I personally do not have experience with the EventKit framework (http://ow.ly/JU9Rl), so I can't really evaluate how difficult it would be to make an application that automatically adds the block data to the stock Calendar app. I'd be interested in seeing how that would work. :smile:


Thanks!  I was coming up with very awkwardly-worded comments, because of my unfamiliarity with the usage of the code. Your comments are perfect.


@bors: retry


Assert the node value.


Good tests.


Interesting.


saw approval from jdm
at https://github.com/Adenilson/servo/commit/463f03826fb2e5459f8ce4e438f10b0b553fc62d


merging Adenilson/servo/cleanupWarning02 = 463f0382 into auto


Adenilson/servo/cleanupWarning02 = 463f0382 merged ok, testing candidate = 1da60235


Good catch! I think I copied that from the original version, but didn't notice it :)


You can try `git rebase` rather than plain `merge`.


[RFC4035] Arends, R., Austein, R., Larson, M., Massey, D., and S. Rose, "Protocol Modifications for the DNS Security Extensions", RFC 4035, March 2005.


Can you make the GIT commit messages more meaningful?


develop (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1136 of 1136 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)


Doesn't some of this checking on index.php allow us to chop out checks we are doing at `login.php`?


You can try `git rebase` rather than plain `git merge`.

Also, please delete the generated files.


<iframe src="https://www.google.com/calendar/embed?src=es-419.romanian%23holiday%40group.v.calendar.google.com&ctz=America/Los_Angeles" style="border: 0" width="800" height="600" frameborder="0" scrolling="no"></Iframe


I dont mind, but travis is upset.. Complains about undef references.

I am on the train heading for the NEAT workshop, so will tale me a few hours before i get my hands on a keyboard.

.-.
Pål-Erik

> Den 4. mar. 2015 kl. 07.23 skrev Joe Hildebrand <notifications@github.com>:
>
> oops. got one of my changes in there. hope that doesn't cause you a problem, sorry.
>
> —
> Reply to this email directly or view it on GitHub.
>


Why Hi not YO?


I think this click trap may have impacted on the ability to click on the calendar month label, which used to take you to the jump month/year table.


I woul lobve to see it im main hipster. otherwise i would have to merge it by myself if there are new features in jhipster i want to use..


Az if(number.isEmpty()}{...} ahogy nézem nem a metódus elején van, hanem a
List<Integer> numberList = getNumbers(number);
validateNumbers(numberList);
után


Igen, tartalmaz 3-ast, osztható 5-el és 7-el is, azaz a kimenet FizzBuzzWizz kéne legyen, ehelyett FizzBuzzFizz van.


How can i access this `error` object when rescue in Grape?

BTW, Doorkeeper's ErrorResponse already handles localization, `error.body` will have a localized error description.


зачтено


Test


develop (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1136 of 1136 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)


@s2gatev: Стани, бутни си публичният ключ тук за да може да деплойваш машинката и сайта.


Seems to work fine for me.


Good call.  I am still very new to bash scripting at this level.  Didn't know that was a thing!


Is there a test that can expose this buggy behavior? If so, can you add it to the test suite?


Çàáûëà íàïèñàòü,÷òî èñïîëüçîâàëà ýòîò ôàéë:

.settings/**
bin/**
test-output/**
.metadata/**

Àíæåëèêà Àðòåìåíêî,ãð.22 "Ïðîãðàììèðîâàíèå íà Java..."

Date: Thu, 26 Feb 2015 15:54:17 -0800
From: notifications@github.com
To: test-project@noreply.github.com
CC: angelica.nm@hotmail.com
Subject: Re: [test-project] ContactRemovalTests, ContactModificationTests 2.2 (6d71194)

Ýòî "ìóñîð" íå ñòîèò åãî çàëèâàòü â ðåïîèòîðèé.

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


FIx for https://wso2.org/jira/browse/APIMANAGER-3281


Изначально собирался держать объект-результат со всеми полями отдельно
Так, например, если пропустить шаг 3, будет проще передать объект-результат без лишних полей, даже если что-то в них написано


I'm not sure if `.txt` is supported on github as README. But I suggest you to use `.md` instead which is supported for Markdown.


I'd suggest `getUrlSuffix()` instead of `prepareUrlSuffix`


89438532_RadianceBugFix (rpg777) - i386-Windows-7-VisualStudio-12: OK (2193 of 2212 tests passed)

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


@Js41637 are you using the very latest github version?


Does this work?


I suggest you to include `.idea` in `.gitignore` as this is an IDE generated folder and best not to be committed to repository.


This change completely breaks `kpm bundle`.


Nice one. But I think the 'actor' is missing - a person who acted upon an event. E.g. what if the owner of the post is the one who left a comment?


poor me, I can figure out how to do it. sad...


init _self
my first time to join the github
hello everyone


В этом случае ты не должен явно конфигурировать аргумент settings, Ты же их забиндил уже kernel.Bind<Settings>().ToConstant(settings);


Can be `final`.


2x is about the best you will get for sha512 SIMD (well, for 32 bit systems, you might still get 4x).  This is due to only MMX_COEF being 2 for the 64 bit formats.  But this format should be about as close as we can optimally get, since the inner loop is simply crypts right into the input data to crypt again. There is NO wasted memory movement there.


原来的658 659行为什么要改？


https://github.com/mangosmop/core-test/commit/c35b1c785765eb871ddf8287a294d8fc3d59701f

DBC structure SpellMisc.dbc 26 fields add
AttributesEx13; // 16


It does. It has a problem however when loading the ActiveInspector as a Data Source in an app. This would (sometimes) make the debug window to crash (more specifically the ko.toJS-function). We removed it while we look into why this is the case.


could you please separate the tags with a comma?!


A bit weird that this is a generic method that can accept anything that implements the interface, but eventually it works only with topic-oriented stuff. But as for now I can't suggest anything better.


Présentation de l'association Brou Création


89445170CoilVsSpacePeak_ExampleFiles (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1642 of 1642 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)


Same here, please separate load and testing


Line, I will add your part of code in our system to join together in your name, rest of work are under progess....


We could provide both but mark them as conflicting with each other. Let the user decide..


Hey, I don't understand why you're bringing that new Thumbnailer back, isn't that inefficient, and why does the worker already create a new Thumbnailer then?


yes, i mean a comma :sweat_smile:


Thanks for the pull request @justafish! Just curious as to what this does that https://github.com/andreypopp/jstransformify doesn't already?

I really appreciate that you've solved this problem for browserify users, but if jstransformify does the same thing more generically (i.e. allowing other jstransform visitors as well) I'd prefer to take that approach.


Intended for Ajax requests to retrieve related artists


GUI design is under progress, lots of error to fix..... before build further...


![screen shot 2015-03-03 at 11 00 50 pm](https://cloud.githubusercontent.com/assets/6334979/6479395/8212bc30-c1f9-11e4-9f43-5797dcd9ee32.png)


it will break building python modules  that use Pygments. Because one run of makepkg build both python 2/3 modules.


Uh, why? If get_queryset is ever reimplemented in that class, we wouldnt want this behaviour. This isn't really a cleanup and it's more confusing than anything else.


@jiuzheng
Stuck on unity.


Ez itt micsoda?


Isnt there something like this in django itself?


This commit updated underscore to 1.8.2, but I didn't realize my text editor 'kindly' deleted any trailing spaces after I saved the files (my difftool is configured to ignore whitespace changes which is normally what I want).

So apologies this commit made those inadvertent changes. I've been running these changes for a while though and the trailing whitespace removal or the upgrade hasn't caused any issues for me.


The code added in 'if' and 'else' blocks looks the same to me. Why not just add it once after the 'if' block?


Yay!


Oh igen. Ez így nem jó! 35-nél csak annyi kell, hogy FizzBuzz.


Except that this unit test hits the disk. That's bad isn't it?


In this case, does it required a std::move for return?


Yeah right, this is an artifact of trying to hunt down the error behind the failing test. Feel free to revert.


Nope, closest is https://github.com/django/django/blob/stable/1.7.x/django/utils/timezone.py


Yep definitely a must have :+1:

Thanks


isn't this a constructor anyway ?


OoH Excuse my friend, I have to stop to translate the evening after work ....

Thanks


QwtUpdate (macumber) - x86_64-MacOS-10.9-clang: Tests Failed

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


88621024-ElectricLoadCenterStorageBattery-Crash (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1123 of 1123 tests passed)

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


It does,

https://www.youtube.com/watch?v=Al6bkiccFDQ


Actually any good compiler should because the function (`chdir` in our case) is declared with `attribute warn_unused_result`.
`cdefs.h` on my system says that GCC supports this starting from version 3.4.
Stackoverflow has discussion on this:
http://stackoverflow.com/questions/7271939/declared-with-attribute-warn-unused-result


aaaa


some tests failed:
failure: http://build.servo.org/builders/linux1/builds/926


Same thing as in Evernote changes, remove from parent before destroying


Why are you not using pythonic::types::str? It would avoid the std::move if I correctly understand previous discussion.


Ah. I see now.
Does not work when its the first thing you click on after starting up, or after a refresh.
But if you click any-place else first, then its working again.


1) yes, we put a check the if open fails it exit and skip the rest of the test.
2) harness_dev regression PolyDriverTest ... however other tests may have the same problem.


the copy is intended, as ``result`` is used as the internal buffer and should not be modified. So yes that's a constructor call, and no ``std::move``


@istx25 That'd be extremely helpful if we could get that working.
Also, forgive my lack of iOS knowledge, but is EventKit compatible with Objective-C, Swift, or both?


Move the lock file validation logic to some class on the lock file. Like LockFile.ValidateProject(project)
/cc @anurse


You can write your own equals function that looks at the X/Y values.


not sure how much these methods are called, but initializing the messagedigest is expensive, so good to cache if there are multiple uses of it.


@kalyankondapally Done


На самом деле здесь ты можешь избавиться от зависимости AiTester-а от ProcessMonitor, потому что это не прямая зависимость. Для этого тебе нужно сконфигурировать контейнер для класса Ai, а в конструктор AiTester-а передавать Func<Ai> createAi. Контейнер в этом случае поймет что делать


so probably don't want a return in a ctor :santa:


IntelliJ のやつとか追記是非……


TeamCity Toolbox :: Pull requests [Build 39](http://10.48.34.86:8111/viewLog.html?buildId=4949&buildTypeId=Toolbox_PullRequests) is now running


TeamCity Toolbox :: Pull requests [Build 39](http://10.48.34.86:8111/viewLog.html?buildId=4949&buildTypeId=Toolbox_PullRequests) outcome was **SUCCESS**
Summary: Tests passed: 161 Build time: 00:00:41


HVACSizingSim_CoincidentPlant (EnergyArchmage) - 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)


It is a commit


wow


Please never use printStackTrace - use LOGGER.log(Level.INFO, "Could not delete a resource", e);


For simple text entries, you can replace `<textarea`> with `<input>`. This will present a single-line entry box as opposed to a multi-line one.


89445170CoilVsSpacePeak_ExampleFiles (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1138 of 1138 tests passed)

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


@trommel I'll look into it and ask around. Frameworks included in the iOS SDK are always compatible with both languages because Objective-C frameworks are bridged to Swift.


学习


@trommel same with OS X frameworks (afaik)


isn't this only possible if the devices is an ephemeral. as far as I know.


The main reason behind the rems is to relate all paddings and line heights to the typography, but without getting into trouble with inheritance like you would with ems. I lie this idea of tweakable sections, but it's tricky. There is this ftp package that sits in between the tree view and the editor, and it would be weird if they don't match.
Anyway, keep up the good work, and I'll just go and reconsider the USPs for Isotope. Again. ;)


This doesn't seem right, same condition in if and elif clause.


Yep!


`bundle install --path vendor/bundle --without production`


Add me as a collaborator in project settings


When using gitignores use .name/ to block a directory. In your case with recommenders folder do ```.recommenders/``` To block the recommenders folder. Additionally .settings directory is already ignored (please see line 16)


Мы решили оставить `.idea` в репозитории, чтобы форсировать единый стиль код. Всё кроме игноров выше не специфично для пользователя.


Please refer PR #43. Changes are applied.


The underlying PHP API client is compatible with PHP >= 5.3 but we never really discussed about minimum support PHP version for the module. Not yet sure if it represents a lot of work, but are you able to submit a PR for 5.3 compliance?


@curran thanks for the submission.  We are actually about to release 0.5.0.
Instead of telling the user to check out a specific version, how about changing the instructions so that it is dependent on the current version of the project?   Something like

```bash
export JOBSERVER_VER=`sbt version | tail -1 | cut -f2`
curl ..... $JOBSERVER_VER .....
```
Many people indeed will rely on the newer one.


У меня есть подозрение, что без `gradle` не будет работать Gradle wrapper. Добавление файла в `.gitignore` не ведёт к его удалению из индекса Git, поэтому сейчас всё работает :)


Jenkins output:
```
DEBUG:   File "/usr/lib/python2.7/site-packages/webfaf2/reports.py", line 271
DEBUG:     unknown_packages = (
DEBUG:                    ^
DEBUG: SyntaxError: invalid syntax
```


Um. I would, but I can't, because now when I activate the plugin I get a fatal error:

> Fatal error: Call to undefined function syn_clean_urls_string() in /Users/maymay/src/syndication-links/syn-postmeta.php on line 80

Sooooooooo…? That's definitely new. :(


Why do you need stdin here?


The android SDK was working earlier today, but after this commit, I cloned it down and it starting to give me errors. It's not even specifying where the errors are in Eclipse IDE


@treyp there are many things that doesnt work yet, for now the working functions are: send, post story, get snaps

Regards


Hi!!
Trying to learn coding. :( Now I want to ask what is the difference between yours and this http://goo.gl/wTuw9p module?


Ok, will be integrated.


TeamCity Toolbox :: Pull requests [Build 40](http://10.48.34.86:8111/viewLog.html?buildId=4950&buildTypeId=Toolbox_PullRequests) is now running


TeamCity Toolbox :: Pull requests [Build 40](http://10.48.34.86:8111/viewLog.html?buildId=4950&buildTypeId=Toolbox_PullRequests) outcome was **SUCCESS**
Summary: Tests passed: 161 Build time: 00:00:16


這個版本有些沒有資料，晚點會補上 3/4 的資料


Please keep client specific changes outside of commits. Different people use different IDE's, respect that and keep the repo for source code. Also please be mindful of moving your workspace/working dir and your local git repo so as to avoid duplications as we see in this commit where the entire source of Delta Deathmatch is also in a folder called Delta Deathmatch. https://github.com/Enters-Domain/Delta-Deathmatch/tree/master/Delta%20Deathmatch is in the main game source. This is a duplication, is inefficient, confusing and dangerous for other commiters, viewers and users of the source. Please fix or I will take the liberty of rolling back.


@andreasohlund any reason why you are using `MessageMetadataRegistry.GetMessageMetadata(string)` instead of `MessageMetadataRegistry.GetMessageMetadata(Type)` ?
If there is no reason, I'll change it ?


I was kidding. It's mostly there to slow you down.