1. Yes, AudioTrack.isPassthroughSupported could be moved into MediaCodecAudioTrackRenderer. I think it's debatable which design is better because other audio track renderers might want to use this functionality, and I think it makes sense to put the utility method that maps MIME type to audio encoding in AudioTrack. I'll think about this more but please let me know if you can think of a good reason to move it. (An application can already avoid having to pass in AudioCapabilities by using the other constructor.)
2. This should be implemented already, except for the fallback and exception (work in progress).
3. We originally had an implementation like that, I think, but it doesn't work if you want to support PCM audio 'decoded' with the raw decoder, in which case we would need to look at the input MIME type to the decoder which is a bit messy.
4. This is related to the fallback/exception feature in (2). An application can use the AudioCapabilities and MediaCodecUtil to work out whether passthrough or normal decoding is possible, but that doesn't help if you want to start playback using a secondary audio track because the first one is unplayable. Fixing the MediaCodecAudioTrackRenderer TODO will avoid trying to play back a track for which playback is not possible, which would address this.


nlink should always be 1 for directory


**valid-jsdoc**: Missing JSDoc @returns for function.


@stephane : sorry, snicolet is also Stephane.
Thank you snicolet for good luck.


    for field_type in field_types:
        CONVERSIONS[field_type] = [(FLAG.BINARY, str)] + CONVERSIONS[field_type]

TypeError: can only concatenate list (not "function") to list

python version is  3.4.3
How should change，help me                       Thanks a lot  !!!!


Please note: There is still a bit of work to make this work. The OAuth tokens expire after 1 hour, so an assortment of keys (client_id, client_secret, access_refresh_token) need to be kept track of, so the access key is never expired.


It's documented in `ess-fill-calls` and `ess-fill-continuations`.
To use it, place the point on or in a call, or in continuation, and use M-q to reformat the code.

I'll try to find some time to work on the interface before the release but I'm quite busy atm. We need to find a way to setup the cycling at the different levels and integrate this in an alist of style as Vitalie suggested.

Probably something like:

```lisp
(ESS
  (ess-fill-calls-style1 . 'something)
  (ess-fill-calls-style2 . 'somethig-else)
  (ess-fill-calls-newline . nil))
(other-style
  (ess-fill-calls-style1 . 'hop)
  ...))
```

Then we cycle through the `ess-fill-calls-styleN` until no bound variable is found.

With `'original` as a possible style to go back to the original state (though probably after the release).


Ugh. You"re right.


@trondn typo :)


s/is it/it is/


удобнее использовать
for team1, team2 in match_results:
сразу распаковка по переменным


doesn't `bundle install` already takes care of setting up jekyll?


Could you elaborate? If we need a different keyboard layout for different type of robots (like drones vs ground robots), that is fine, but the keyboard actuator code need to take that into account.


improve the class Queue so that the calling speed is faster than ever


You're welcome


Fixed endif comments …


why is it defined like that @blaszczakphoto? :)


ок, только приведи все к PEP8 - в пайчарме есть меню Code/ReformatCode


The method signature has changed. You should throw a deprecation notice if the second parameter (formerly `$queryDB`) is boolean.


Could you explain a little why it's initialized like that?


ок + PEP8


i would rather clear tokens in one place (eg. save response from both branches and check the status code. thenthe method `clearTokens()` won't be needed at all)


I'll pass it so it won't break a build just for the time of explanation:)


The Seq extension is that something we want to use? or is this more something to avoid changing the tests?
As it only seems to wrap the functionnames of .NET to something different
(i also personally find the .NET naming a bit odd, but...)


drop the () after java


please also add a new line for pep8 compliance.


为什么nuget上还没有更新一键开通会员啊？


Shouldn't the function be named `is_older_than`?


That's a line comment on a commit! You'll lose it when you squash your commits!


测试


Sorry, the note about fixing #317 is wrong.


There's no need to create instace of Project to check its validations :)
```ruby
describe Project do
  describe "ActiveModel validations" do
  it { is_expected.to validate_presence_of(:email) }
end
```


Shouldn't be the comparison done with the `>` operator if we want to check that it's older than a given date?


```var root = null;```
```var newBody = null;```

or

```var root         = this.contentWindow.Hammerhead.ShadowUI.getRoot();```
```var newBody = this.contentDocument.createElement('body');```


про конверты и кирпичи - неверно. например отверстие 10х10 а конверт 9х100. это задача на вложенные if (или встроенные функции min/max)


There is a small typo in the example: it needs to be `os.system`, not `os.sytem`.


I understand. However, we see that attributes in alphabetical order is out of the scope of CTD2Galaxy (we want to reduce the use of third party libraries as much as possible).

Perhaps an extra script could be added. Once the conversion is done, said script would parse a ToolConfig and spit it out in alphabetical order for regression testing purposes?


``)``
``.appendTo('body');``


@mmaechler

You need to add `(looking-at ".*[ \t]*(")` to `ess-align-continuations-in-calls` if you want alignment there.

But then you'll have *all* continuations aligned when they're inside a call, which is maybe not what you want.

Not sure what to do here. Oh, maybe add the logical functions `all()` and `any()` to the aligned calls? cc @vspinu


You missed a c from currently.


doesn't that also need a require on dci?


warning, you have tab and spaces...

no tab on this project (except for TCPDF original class)


how do you install this?


It seems to me you changed the logic. See https://issues.jboss.org/browse/WINDUP-745 we would not like to have unordered list if the warnigs are just 1.


file should be moved outside kci dir now


ok


Sure, will do.


The thing is that we stopped treating logical operators and normal operators differently, so we need to find a rule suitable in all cases.


различные элементы - ок.
про сумму цифр - N надо задать константой и оптимизировать количество переборов (откуда взялось range(10,100500) ?)


I did the namespace decleration in the envAttributes:

```
envAttributes: {
    'xmlns:soap': "http://schemas.xmlsoap.org/soap/envelope/",
    'xmlns:w': 'http://www.win.eu/w'
},
````

Isn't it up to the user to make sure everything is declared?

Can this be implemented in an other way, for example, adding some extra config vars?


I did this exact thing in 1bd8d9dc2 and then reverted it in 96ad4ded after we confirmed the problem only exist in loop bodies that are not their own functions. Now you reverted the reversion - did you see actual problems?


This seems to break my +New button when trying to add an field from the Custom Fields list.


Why the global reference? :)


"There are a variety of ..." - methods is plural. Also, should "full life" really be "full health"?


Also, why is the new `$compositeFields` param passed in by reference, and how is it actually used? In this method particularly, the passed in value is discarded/overwritten. It seems to be a way to have methods like `database_fields()`, `custom_databse_fields()` and `composite_fields()` cause side effects on these values without changing their return signature. Given all those methods are public, it makes the API quite confusing. Shouldn't DataObject just maintain an internal cache of those fields and build it up on first access?


mi piace più questa versione


@wesleycho  @gorork I'm not sure why we've merged this one... Currently popup has the `typeaheadPopup` directive so people could create their own `typeaheadPopup` to add whatever attributes (including classes) to the pop window.

IMO this change only adds to the size of the library for a very specific case that could be solved in the user-land in a more flexible way. Could we re-consider / revert this one?


Why is there an ```\end{flushleft}``` while there is no ```\begin{flushleft}```? LaTeX complains about this and does not compile.


From the [docs](http://electron.atom.io/docs/v0.32.0/tutorial/quick-start/#write-your-first-electron-app):

> // Keep a global reference of the window object, if you don't, the window will
// be closed automatically when the JavaScript object is garbage collected.
var mainWindow = null;


Вынести определение из bfs, тк например граф может быть несвязным и нам нужно помнить visited.


关注这一规则，我想我找到了手机只能浏览play，而不能下载应用的原因


Great examples, thanks for sharing!

Quick question on the call to csu_kdp.calc_kdp_bringi() -- the documentation of that function says that it assumes PhiDP has already been unwrapped, but I didn't notice that step in your example.  Is PyART doing that automatically under the hood?

Thanks!


Not much here... just setting the compile targets.


Classes/Eat Now/Detail View Controller/EatNowDetailViewController.swift

Fixed 579d33a


The Java stub we were using doesn't work with Java 1.8. Had to switch to a newer plugin that does the same thing.


Fixed 579d33a


all these settings can be set in maven now. Don't need this file.


@nyetwurk
I wrote an integtest for CondVar. It seems to work alright. Could you test it as well?

#include "mt/CondVar.h"
#include "mt/Mutex.h"
#include "mt/Lock.h"
#include "base/Stopwatch.h"

#include "test/global/gtest.h"

TEST(CondVarTests, waitTimeout)
{
	Mutex* mutex = new Mutex;
	CondVar<bool>* condVar = new CondVar<bool>(mutex, false);
	float timeout = 0.3f;

	Lock lock(mutex);
	Stopwatch watch(false);

	while (condVar->wait(watch, timeout)) { }

	watch.setTrigger();

	EXPECT_GE(watch.getTime(), timeout);
}


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#aDhkkdf2SNSGhshxs9yrNQ


Fixed 090ef31


Whatever `array` is in this instance is obviously not implementing `==` properly then... Are you sure this isn't a delegator instance involved? They will often masquerade as the underlying instance (because `inspect` gets delegated, but implement `==` themselves).


Which file? In run.sh, there's no other way to run.

Somewhere different? I'd have to look.

On Wed, Sep 16, 2015 at 7:25 PM, Phil Rosegay <notifications@github.com>
wrote:

> peter, I don't understand, why would you change the default of useexport
> to true when we want to deprecate the use of the export tests?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/VoltDB/voltdb/commit/b560a97d05979a8783ac16908a486e868faecfd4#commitcomment-13275668>
> .
>

--
*Peter Shaw,* VoltDB Engineering
*VoltDB, Inc.*
*E:* pshaw@voltdb.com

www.voltdb.com <http://voltdb.com>
Follow VoltDB on Twitter <https://twitter.com/voltdb>, Facebook
<https://www.facebook.com/VoltDB?sk=wall>, and LinkedIn
<http://www.linkedin.com/company/voltdb>


Isn't this overriding the max age setting in Drupal?


strはpythonではstr型を返す関数です。
http://docs.python.jp/2.6/library/functions.html#str

関数の上書きとなりえるので、使用は避けてください。


this is sad but in order to build on windows we need to target x86 or x64. So...for now we are doing to force users to use the 32-bit java. We can revisit at a future date.


On Wed, Sep 16, 2015 at 04:57:13PM -0700, MercenaryJosh wrote:
> Then why not make session a $http.get rather than a full $resource?

It's consistent with AzureAPI.person.get().  I've tried to use
$resources whenever I can to be as consistent as possible.


必要のないコメント、つまり不要な情報は削除


Completely new thing here. This is very similiar to what the mac does. It's a stub exe that launches the jar.


It would work, I would probably consolidate the two into a single channel (via a struct{out []byte, err error}) and then check in the case.


I also update the test with a unit test that makes sure chart data is being assigned after its returned from the mock service.


fix indent


This function is used in both getDocumentState and when it returns None, again at call site (gotoDefinition).

I choose not to optimize this for those reasons:
* I assume it's not too expensive (one call to DTE and one call to projectFactory)
* at call site, the function has significant cyclomatic complexity, I didn't want to touch it now (but just to make it one step worse!) as I'm not familiar with all which is happening there

Let me know if you want me to give it another look.


It's probably worth bypassing the parser here and just doing `makeNode(DeleteStmt)`.


should be `Check if identity is allowed to add columns to the specified table.` (no extra space and pluralize column)


As a style thing, obviousness is preferable.

Eg.  if [ "$state" == "4" ]; then return 0; else return 1; fi

(Except you'd want line breaks :-)


Is it safe to assume `idx + 8` ? What about 10 char variants: `charset=ISO-8859-1` or `ISO-8859-8`

Would using [HttpHeader.getCharset()] (https://github.com/zaproxy/zaproxy/blob/develop/src/org/parosproxy/paros/network/HttpHeader.java#L556) be better?


Sorry, da hab ich feature und merging in einem branch durcheinander geschmissen... - Anfängerfehler...
Drum jetzt die wirre branch Geschichte...


You might as well combine into 'elsif' and remove a level of indentation


Te salvé de horas debuggeando al pedo, lo sé (?)


Does this really fix, hence close issue #1890?


By the way, since we are talking about simplification, I wonder if we should get rid of `ess-indent-from-lhs`, or leave it there but undocument it.


I'd agree that these differences couldn't possibly result in any performance difference that isn't just theoretical. The functional form seems fine to me, and it can be used more easily during iteration.

On the subject of that common practice, though: Angular dropped support for IE8 as of 1.3. IE8 was the last ES3 browser that permitted redefining `undefined`. Capturing a reference to `undefined` in the outermost closure was a bit odd even when IE8 was supported, though:

 1. It's silly; even in the ES3 era, who has ever seen code where undefined actually got redefined?
 2. It won't even necessarily work. That is, there's no guarantee that Angular gets loaded before the legendary redefining of undefined takes place. If this is truly something we want to be safe about, one would make the comparison against void {something} and never refer to `undefined` at all.


Super weird! Well, I'm glad we show it properly on our homegrown generator :)

I've tried to hunt for anything that might trip up the syntax, and saw nothing out of the ordinary. I suppose we could try a couple things to see if this would do the trick:

- Try indenting four spaces instead of two.
- Try a blank line before the opening tickmarks.

Part of me also wants to call this a *legitimate bug* in the GitHub Markdown parser and raise an issue with GitHub HQ.


Bio image is successfully replaced. However, after testing on both Chrome and Firefox, bizarre formatting issues arise:
![odd_spacing](https://cloud.githubusercontent.com/assets/10101246/9921762/f0951700-5ca6-11e5-8cec-538e76310036.PNG)
Also, font abruptly changes after end of the unordered list:
![odd_font](https://cloud.githubusercontent.com/assets/10101246/9921768/05690ce0-5ca7-11e5-8c29-826df9d84728.PNG)


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.593](http://teamcity.monogame.net/viewLog.html?buildId=10159&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 539, ignored: 6 Build time: 00:20:38


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


Also I'm still getting a compilation error, same as before. :V


This line of code got dropped.


Should've bumped revision as this is a functional change and not just enabling it on a specific arch.


fix for issue #458


Hi, will it be more efficiently to create objects like this dynamicly?


add a javadoc


@RandomAltThingy I have Java OCD of not having empty lines between methods unless there are many and very different.
@jython234 I just committed to let you see what I mean.


The sendfile declaration in socket.h is guarded by a check for
__BSD_VISIBLE, hence the define before socket.h. This is a hack and I'm not
at all happy about having to do this, but I don't know of any other way.

In sys/cdefs the following turns __BSD_VISIBLE on or off:

#if defined(_ANSI_SOURCE)   /* Hide almost everything. */
#define __POSIX_VISIBLE     0
#define __XSI_VISIBLE       0
#define __BSD_VISIBLE       0
#define __ISO_C_VISIBLE     1990
#elif defined(_C99_SOURCE)  /* Localism to specify strict C99 env. */
#define __POSIX_VISIBLE     0
#define __XSI_VISIBLE       0
#define __BSD_VISIBLE       0
#define __ISO_C_VISIBLE     1999
#elif defined(_C11_SOURCE)  /* Localism to specify strict C11 env. */
#define __POSIX_VISIBLE     0
#define __XSI_VISIBLE       0
#define __BSD_VISIBLE       0
#define __ISO_C_VISIBLE     2011
#else               /* Default environment: show everything. */
#define __POSIX_VISIBLE     200809
#define __XSI_VISIBLE       700
#define __BSD_VISIBLE       1
#define __ISO_C_VISIBLE     2011
#endif

and further up it depends upon the POSIX version if this code gets executed
at all. I haven't gone into any details in understanding the mechanics of
this but since Erlang is compiled using C99 mode I think it's clear that
this flag will never be set, even if the preprocessor reaches this block of
code.

Is there another cleaner way of enabling the __BSD_VISIBLE flag or should I
be doing something completely different?

Regards,
JP

On Wed, Sep 16, 2015 at 7:52 AM, Lukas Larsson <notifications@github.com>
wrote:

> Could you add a comment here on why we have to enable __BSD_VISIBLE?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/theom/otp/commit/043c49a6951add5bf7b05cda643c8b4874849e5a#commitcomment-13255348>
> .
>


I purchased the rails-stripe-membership-saas tutorial.  It was great!  Thank you.  I want to be able to create a trial subscription without having to put in your credit card number.   I can't figure out how to implement this latest commit.  I have modified the gem in my gemfile:

gem 'payola-payments', git: 'https://github.com/peterkeen/payola'

Do you have an example of how to implement this?  For example, what would the initial sign up form and controller action look like for to set up a devise user without a credit card.  Also, once the Stripe trial period has ended what would the form and controller actions look like.

I have been attempting to implement this myself without much luck.  I think that I having issues with the javascript validations.

Thanks!!!


Looks good. Question though, when I revisited nats-top it was updating the uptime but the server was gone, and it was showing old data for the server section. Also, terminal did not respond even to CTRL-Z, so something else going on too..


This leaves an empty `div` for site-title. That could look broken if there is padding on that .site-title or flexbox is used in title. Check the discussion and PR here https://github.com/Automattic/_s/pull/785#issuecomment-136523557
Should probably just mirror whats happening on the front-end, hide .site-title and color the link, right? https://github.com/Automattic/_s/blob/master/inc/custom-header.php#L61-L73


need this as a second line (instead of `-o -name '__pycache__'`) since otherwise the directory might not be empty and `find` won't delete it


Hi, @wasabeef.
Can you give me some information to show why can not get the mask from BitmapPool? I really not understand why about this. Thanks!


# include<stdio.h>


This was deleted by bb63bd5e3792455b77ce93e471158bed0cc884f0, which made a vendor-specific test fail.


Bacana, brother. Ficou show. Devíamos fazer do atirarFlechaRefactory?


Boa! Também testei atirarFlechasRefactory.


Semicolon missing :eyeglasses:


This doesn't change anything on Linux... It probably breaks something on Windows, though.


You can configure rubymine to add a new breakline at the end of the file. ( I think you have that setting for ruby files and html already)


@cpuguy83 is this a sane way to write go? Could I write the struct simpler somehow?


@dirk-thomas Where do these files live so that I may manually apply this change?


Nice data structure :+1:


FYI, here the SHA was changed to a newer one, however the URL was not changed to reference 3012 like the other files were.


yep, it's absolutely clearer


nono


mm this does not look good an id definition with `#`
@terryllowery can you double check this /?


nonono


Great work! You don't need to unwrap the context every time. When context became optional I wrote extensions that takes optionals and handles it there. Makes the code cleaner. See [`renderInOptionalContext`](https://github.com/danielgindi/ios-charts/blob/swift-2.0/Charts/Classes/Utils/CALayer%2BExtensions.swift) and [`UIGraphicsPushContext`](https://github.com/danielgindi/ios-charts/blob/swift-2.0/Charts/Classes/Utils/UIGraphics%2BExtensions.swift)

Good job :+1:


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.594](http://teamcity.monogame.net/viewLog.html?buildId=10163&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 539, ignored: 6 Build time: 00:20:56


@lukeolson first of all, thanks for replying to Ryan just now! Based on your response, I played around with the example you pointed to. And I noticed that "submit for grading" was actually not spectacularly great wording for the button on the top right, especially given that this particular flow is actually not graded. So I've played around with his phrasing some more, and now it only says "submit for grading" if a permanent grade is actually being recorded. Otherwise, it just says "finish".


is depth ever 0 without the table being empty? I wonder if this condition will ever be true basically. even if does mean, what I believe is the intention, that there's only a single bucket, I think it should probably be a bit more aggressive. If you only have two buckets, you're still not connected correctly. I could imagine setting the limit to m_table.depth() < 4 and be happy with keeping the longer timeout (because it may take more than a minute to fully bootstrap)


SOS狀態下需要重試，普通狀態下不需要重試，否則在極端網絡條件下，數據更新快過上傳速度，導致堆積


@keawade So I went looking as a result of Joel's comment on his library commit. I get why you rolled that back, but why the email function? That didn't use mongo at all.


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


Killed player items are not "pick-up-able". I was farming the T1 village and I was not able to pick-up the dead player's gear. I think the gear was his armor and 1st slot weapon. If so, they should not be seen.

![2015-09-15_17-39-53](https://cloud.githubusercontent.com/assets/14245232/9922111/75c8ecbe-5caa-11e5-8987-1fb7828be11b.png)


Mine is:
```
0x456E676C69736820D6D0
0x456E676C69736820D6D0CEC4
0x456E676C69736820D6D0000000000000000000000000000000000000000000000000000000000000
```


r+


Thanks.


saw approval from MonsieurNicolas
at https://github.com/graydon/stellar-core/commit/4e1b76c14b7b4ebd1ca5551e3ad1990605190901


merging graydon/stellar-core/bug-762-merge-peer-and-validator-seeds = 4e1b76c1 into auto


graydon/stellar-core/bug-762-merge-peer-and-validator-seeds = 4e1b76c1 merged ok, testing candidate = 0b845128


Wow, the tab situation needs to be fixed.

I will do this soon.


Renamed this, right here.


Sorry for that typo, please merge this: https://github.com/TrinityCore/TrinityCore/pull/15527


Why'd you revert my logger changes in this?  They're completely unrelated...


windows phone, not windows!


This project needs better review practice before hitting the green button.


Very well explained of the results! I think one of the most important thing in multithread programming is how to synchronize between threads.


log_cache() isn't needed since Log->update_cache() is being called.


Ah, yes.  This the problem of naming things for a particular rule set.  I think "finish" makes sense.


102497980-Improve-waste-heat-function-handling-for-CoilCoolingDXMultiSpeed-and-CoilHeatingDXMultiSpeed-#5162 (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1336 of 1336 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-70.22%25-green.svg)


actually, I looked at this commit just now.  I don't think it makes things easier to understand!  I may have had a drink when I made the pull request lol


The language tags are case-insensitive, you can refer with the w3c documentation http://www.w3.org/Protocols/rfc2616/rfc2616-sec3.html#sec3.10
```
White space is not allowed within the tag and all tags are case- insensitive. The name space of language tags is administered by the IANA. Example tags include:

       en, en-US, en-cockney, i-cherokee, x-pig-latin
```


Thanks for the catch @purplecabbage


I've noticed it's safer (no research) to proceed the string with /: "/admin/delpost.php".

By default, CORE_DIR is `_core/` but it doesn't seem to break when the string results in `_core//admin/delpost.php`.


Should use CORE_DIR.


Whoops, I didn't mean to pull that out. Got overzealous I suppose.


Muito bom! hu3


`locate find_in_workspaces.py` will tell you where the file is in your system.


Bacana! :3


O melhor queijo - NOGUEIRA, Flávio.


LGTM


- For keyPoint 2 (topic 0) you should use builder pattern
- You shouldn't commit empty classes (like Buildings.java, KeyPoint03.java and Patients.java in keyPoint03 package)
- For keyPoint 3 (topic 0) you should use patterns, builder, factory, etc
- You shouldn't duplicate classes in different places (for example Patients.java in src folder and then also in keyPoint03 package)
- You should make a project (Java project) for each Topic
- I will close the PR but you can correct these items in the next commit


Realmente muito bom, equipe!


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 641](https://teamcity.labkey.org/viewLog.html?buildId=272614&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 33, passed: 9 Build time: 00:27:20
### Failed tests
```
org.labkey.test.tests.CDSMeasureStoreTest.testDemoMeasureStore: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyAntigenScatterPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyColorStudyAndTreatmentVars: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyStudyAndTreatmentVars: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyBoxPlots: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifySubjectCountsWithFilters: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyTimeAxisWithMultipleSchedules: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifySubjectCountsWithFiltersAdvancedOptions: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyScatterPlotColorAxis: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyAntigenBoxPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyBinnedPlot: <no details avaliable>

##### there are 23 more failed tests, see build details
```


Astonishing work Heinzz ! What design pattern did you use ?


Será que este construtor ainda é necessário?


Please move `lsm_plug_system_fw_version sys_fw_version;` to new
`struct lsm_ops_v1_3` as  1.2 already released.
As also changed version number in `configure.ac` `AC_INIT`.
Thank you.


I did in fact try to loop over these, but it's not trivial in bash. I decided it was too academic of a concern and left as-is.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.595](http://teamcity.monogame.net/viewLog.html?buildId=10177&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 539, ignored: 6 Build time: 00:20:05


Uma dica: o nome do teste tem que ser mais expressivo.
Use por exemplo: criacaoDeElfoComNomeCerto.


101280704_lowTempRad (nealkruis) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (0 of 0 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)


102497980-Improve-waste-heat-function-handling-for-CoilCoolingDXMultiSpeed-and-CoilHeatingDXMultiSpeed-#5162 (lgu1234) - Win64-Windows-7-VisualStudio-12: OK (1844 of 1870 tests passed)

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


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

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


101280704_lowTempRad (nealkruis) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: Tests Failed

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


Why is this test being disabled?


Remember to remove unused package imports (you can do this with Ctrl + Shift + o in eclipse)


@dominictarr wrote this already, it's in a gist somewhere, although would be much better as a repo.


all tests pass:
success: https://travis-ci.org/stellar/stellar-core/builds/80749878


removed in next commit...


question about what this code meanby


The style of annotation as following, It will be document in https://crosswalk-project.org/apis/embeddingapidocs_v5/index.html
```
/**
     * About the API
     * @param
     * @since 6.0
     */
```


@droric given how close you are, seems worth waiting to release RP-0 until you're done. Do you have any guesses as to when that might be? (Just released RO v10.4, so RP-0 should go out 'soon' :) )

Thanks again!


Yes, I do.


Other LGTM.


not related to this line, Commit message should be split in multiple lines (each less than 80 characters)


Because madness. :black_joker:


fast-forwarding master to auto = 0b845128


#5167-ExampleFile-VariableRefrigerantFlow_FluidTCtrl_5Zone.idf-Warnings (rongpengzhang) - x86_64-MacOS-10.9-clang: OK (1873 of 1874 tests passed)

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


- Wrong package name, should be topic1


Are the paths correct for MacOS?


here it is: https://github.com/dominictarr/pull-stream-examples


How about move this  up
```
mDIPScale = DeviceDisplayInfo.create(mViewContext).getDIPScale();
mContentsClientBridge.setDIPScale(mDIPScale);
mSettings.setDIPScale(mDIPScale);
```


Que tal pesquisar por algo chamado StringBuilder?


這邊一樣需要再 save 前把資料都清除喔


We should also tell the user what vmm types are currently supported. This is useful in case of typos or wrong capitalization (If the user puts 'openstack' instead of 'OpenStack' this will fail, and it will be quite hard and frustrating to understand why :) )


Could you [check out](https://github.com/nfarina/homebridge/blob/master/platforms/HomeAssistant.js#L410-L433) the more modern API for returning services and implement this way? It'll be a lot cleaner and shorter too!


I will move ```mContentsClientBridge.setDIPScale(mDIPScale);``` down since XWalkSettings instance was created here.


@benhurott :
Conheço esse cara, mas quis manter mais simples.
Tem alguma diferença de performance entre concatenação e usar o StringBuilder?


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#EP_5QY5nT_Of91U4yGuwsw


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#bz7NHhmoRtm-kSKbmzOScA


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#GrYgC6i4TN636C-zpS8MJw


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#AgdeypRlQLu6vW_tUEs00Q


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#IDY-SX_5ReuWIOX6WAtBug


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#SpQNN-nxQIuhVy_yNydyFg


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#U6BtXWkXTL-11l8RThKw3A


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#OE2R5kSoQxiNnHB2beekKQ


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#fIqwRqOvQDa1Qx0xCCTb6Q


Can you explain this revert? Unless it was an unintended side-effect of conflicting.

For all intents and purposes, `[]` is the same as `array()`. Also 'dummy' throws an error as it's not defined, so it definitely does not need to be there.


i'd probably use `get_class($lastInstance)`.


Why is this name == `Debug` check hardwired?   Is there a way this could be specified in the Podfile?

For example:  If a user has a project that uses 2 separate build configurations that we want to be treated as a :debug configuration it would be nice if that could be set.


Just a reminder you said you'd revert this.

`ereg*` is deprecated in favor of `preg*`.


On the same category as the comment below, I think you should check this instead:

APIC_VMM_TYPE_OPENSTACK.lower() == vmm_type.lower()

And then use the constant on the below calls. This will avoid failures caused by wrong capitalization. Another possibility would be to normalize this values in the init and assign it to some instance attribute.


Can we refactor more of this code into the ORM? There's a lot of ORM internal plumbing being dragged into the GridField here.


#4702-AirloopHVACUnitarySystem-autosizing-incorrect-for-heating-only-systems-when-using-FlowPerHeatingCapacity (rraustad) - i386-Windows-7-VisualStudio-12: OK (1851 of 1873 tests passed)

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


This feels like it crosses over with the SearchFilter stuff, for example. Can we somehow make use of that?


yes, that's why in the ApicVmmTypeNotSupported exception class, The only 2 options are spelled out exactly for this purpose.


Could you please explain about setting the argument to -1?


Should be CORE_DIR.


2 whitespace are needed here. In general, there are a bunch of PEP8 issues here. Please run [0] to catch and fix them.

[0] tox -e pep8


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


Ok, i see your change


Yes, sounds really bad. Let me look into it.  Sorry!


You can make this contructor private so that this class use Singleton Pattern


Most uses of Log are bad an pointlessly add overhead.

It'd probably be best to `require_once` as early as possible (probably before execution).


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


change "branch": "master" to "tags": true


yes, I've modified the msg directly in the pull request:
https://github.com/noironetworks/apicapi/pull/3

Let me know if you didn't see it then. Thx.


That we will have no more diffs like this:
```diff
-BUNDLED WITH
-   1.10.6
```


BTW: are you running `make test` as root?  I can't see how /dev/null can be deleted otherwise.
```
$ ls -ld /dev
drwxr-xr-x 17 root root 4280 Aug 18 09:58 /dev/
```


Did I make a spelling mistake?


@bradtheappguy it's already possible to do that in the Podfile, but please don't comment on old commits


@iagosilvany na master né?

Pra prevenir erros assim a minha equipe sempre usa isso no sublime, em **preferências > Settings user**:
```
"ensure_newline_at_eof_on_save": true,
"font_size": 13,
"ignored_packages":
[
    "Vintage"
],
"tab_size": 2,
"translate_tabs_to_spaces": true,
"trim_trailing_white_space_on_save": true
```


MiscReleaseFixes (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)


This is deleted in future commit 93e0f0f5f96fb0ab88c280fd2fe34bb8812c520d


Other LGTM.


Should this have cloudflare in the class name? Following the pattern from `CloudFlareCacheTagHeaderGenerator `


TaskCluster: [object Object]


Classes/Eat Now/Detail View Controller/EatNowDetailHeaderView.swift

Deleted later 93e0f0f5f96fb0ab88c280fd2fe34bb8812c520d


Determine the number of characters in a string.


Label deleted in next commit 93e0f0f5f96fb0ab88c280fd2fe34bb8812c520d


I'd rather put a parameter here instead of hardcoding it... This way we don't have to remember to change this message every time we add a new VMM domain :)


How do I use quicklisp with multiple lisps?  I already installed it for use
with ACL.   Now, I'm trying to use it with SBCL and I get:

* (load "quicklisp.lisp")

  ==== quicklisp quickstart 2015-01-28 loaded ====

    To continue with installation, evaluate: (quicklisp-quickstart:install)

    For installation options, evaluate: (quicklisp-quickstart:help)

T
* (quicklisp-quickstart:install)

debugger invoked on a SIMPLE-ERROR in thread
#<THREAD "main thread" RUNNING {1002C7E633}>:
  Quicklisp has already been installed. Load
#P"/home/layer/quicklisp/setup.lisp" instead.

Type HELP for debugger help, or (SB-EXT:EXIT) to exit from SBCL.

restarts (invokable by number or by possibly-abbreviated name):
  0: [LOAD-SETUP] Load #P"/home/layer/quicklisp/setup.lisp"
  1: [ABORT     ] Exit debugger, returning to top level.

(QUICKLISP-QUICKSTART:INSTALL :PATH NIL :PROXY NIL :CLIENT-URL NIL
:CLIENT-VERSION NIL :DIST-URL NIL :DIST-VERSION NIL)
0]

On Wed, Sep 16, 2015 at 7:56 AM, Zach Beane <notifications@github.com>
wrote:

> New problem..."Package UTIL.DATE-TIME does not exist" on SBCL.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/e40/cl-quakeinfo/commit/819c02af4f1c3c4693e14b8a28347a29aff8f116#commitcomment-13264548>
> .
>


Number of characters in a string.


This is wrong --- I would have sworn we had a fix merged, but I guess not!

```php
elseif ($C = $this->collaborators->filter(array(
             'user__emails__address' => $mailinfo['email'])
           )->first()) {
.....
}
```


Yes. I am an administrator and was building an AMI (ec2 image).


Forgot to open a pull request for this, but for open source posterity, the author signed off on it.


TaskCluster: @sw337y-314159 does not have permission to trigger tasks.


this will makes me crazy


If it's been installed once, by any Lisp, you should be able to load it
into your session with (load "~/quicklisp/setup.lisp") in any other Lisp.

On Wed, Sep 16, 2015 at 9:24 PM, e40 <notifications@github.com> wrote:

> How do I use quicklisp with multiple lisps? I already installed it for use
> with ACL. Now, I'm trying to use it with SBCL and I get:
>
> * (load "quicklisp.lisp")
>
> ==== quicklisp quickstart 2015-01-28 loaded ====
>
> To continue with installation, evaluate: (quicklisp-quickstart:install)
>
> For installation options, evaluate: (quicklisp-quickstart:help)
>
> T
> * (quicklisp-quickstart:install)
>
> debugger invoked on a SIMPLE-ERROR in thread
> #<THREAD "main thread" RUNNING {1002C7E633}>:
> Quicklisp has already been installed. Load
> #P"/home/layer/quicklisp/setup.lisp" instead.
>
> Type HELP for debugger help, or (SB-EXT:EXIT) to exit from SBCL.
>
> restarts (invokable by number or by possibly-abbreviated name):
> 0: [LOAD-SETUP] Load #P"/home/layer/quicklisp/setup.lisp"
> 1: [ABORT ] Exit debugger, returning to top level.
>
> (QUICKLISP-QUICKSTART:INSTALL :PATH NIL :PROXY NIL :CLIENT-URL NIL
> :CLIENT-VERSION NIL :DIST-URL NIL :DIST-VERSION NIL)
> 0]
>
> On Wed, Sep 16, 2015 at 7:56 AM, Zach Beane <notifications@github.com>
> wrote:
>
> > New problem..."Package UTIL.DATE-TIME does not exist" on SBCL.
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/e40/cl-quakeinfo/commit/819c02af4f1c3c4693e14b8a28347a29aff8f116#commitcomment-13264548
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/e40/cl-quakeinfo/commit/819c02af4f1c3c4693e14b8a28347a29aff8f116#commitcomment-13276981>
> .
>


Tem sim, não se preocupa com isso agora. Só te comentei pq sei que vc saberia usar hehe.


? we read it in the `if head_shape_fname` above, but then set it to identity here?


I will investigate... Thanks for the feedback, I will look into fixing these couple of issues.


Si bien el tag "li" permite omitir el tag de cierre, es recomendable cerrarlo.
También es recomendable usar lowercase en los tags.


Ah, OK.  Thanks.  It's been a long time since I've used ql.

On Wed, Sep 16, 2015 at 6:26 PM, Zach Beane <notifications@github.com>
wrote:

> If it's been installed once, by any Lisp, you should be able to load it
> into your session with (load "~/quicklisp/setup.lisp") in any other Lisp.
>
> On Wed, Sep 16, 2015 at 9:24 PM, e40 <notifications@github.com> wrote:
>
> > How do I use quicklisp with multiple lisps? I already installed it for
> use
> > with ACL. Now, I'm trying to use it with SBCL and I get:
> >
> > * (load "quicklisp.lisp")
> >
> > ==== quicklisp quickstart 2015-01-28 loaded ====
> >
> > To continue with installation, evaluate: (quicklisp-quickstart:install)
> >
> > For installation options, evaluate: (quicklisp-quickstart:help)
> >
> > T
> > * (quicklisp-quickstart:install)
> >
> > debugger invoked on a SIMPLE-ERROR in thread
> > #<THREAD "main thread" RUNNING {1002C7E633}>:
> > Quicklisp has already been installed. Load
> > #P"/home/layer/quicklisp/setup.lisp" instead.
> >
> > Type HELP for debugger help, or (SB-EXT:EXIT) to exit from SBCL.
> >
> > restarts (invokable by number or by possibly-abbreviated name):
> > 0: [LOAD-SETUP] Load #P"/home/layer/quicklisp/setup.lisp"
> > 1: [ABORT ] Exit debugger, returning to top level.
> >
> > (QUICKLISP-QUICKSTART:INSTALL :PATH NIL :PROXY NIL :CLIENT-URL NIL
> > :CLIENT-VERSION NIL :DIST-URL NIL :DIST-VERSION NIL)
> > 0]
> >
> > On Wed, Sep 16, 2015 at 7:56 AM, Zach Beane <notifications@github.com>
> > wrote:
> >
> > > New problem..."Package UTIL.DATE-TIME does not exist" on SBCL.
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/e40/cl-quakeinfo/commit/819c02af4f1c3c4693e14b8a28347a29aff8f116#commitcomment-13264548
> > >
> > > .
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/e40/cl-quakeinfo/commit/819c02af4f1c3c4693e14b8a28347a29aff8f116#commitcomment-13276981
> >
>
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/e40/cl-quakeinfo/commit/819c02af4f1c3c4693e14b8a28347a29aff8f116#commitcomment-13277010>
> .
>


Why is it needed though? Using ssr doesn't mean you are using pushstate.  Anyways, I removed it because it was causing a problem somewhere... bad commit message and I can't remember where.


You can always use https://github.com/be5invis/Iosevka/releases/latest actually.


This is NOT a real exposed API for developer, we just use the annotation for reflection generation and the function is used for delegate event handling to XWalkView only, that's why no formal API comment appended.


Tranquilo Fer jaja. Lo que queria decir es que hay que hacer source del .rc para que nvm y rvm (creo que para los dos es) queden accesibles en el PATH


Oh, it was for: https://github.com/canjs/can-ssr/issues/16


- The link should have the name of the war project, in your case instead of href="/Exercise4_5/HelloWorld" should be href="/Exercise4/HelloWorld"


develop (mjwitte) - Win64-Windows-7-VisualStudio-12: OK (1342 of 1342 tests passed)

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


I think this will work a little better here, keeps original use cases and only act upon the alarm after initial run


I think we discussed changing this name to include Enyo if you want to continue to use the dispatcher or convert it to a generic method that just uses the DOM parent.  Also, don't forget to add 'Id' onto the end.


Muito bom Biel ja tenho aqui seu commit. Sent by Windows Phone


Ótimo Lucas. Ja tenho seu arquivo aqui. Sent by Windows Phone


Fixed in:
https://github.com/JohnLangford/vowpal_wabbit/pull/785


102497980-Improve-waste-heat-function-handling-for-CoilCoolingDXMultiSpeed-and-CoilHeatingDXMultiSpeed-#5162 (lgu1234) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1844 of 1870 tests passed)

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


`Cannot add a column`


`columns`


At the moment, I'm still working on v0.96 to make sure it's stable to release, but I'm close to being able to release it. I can commit all of the changes I've made in the next few days and see if I can make it release-able.


Hello Swmitra,

you did a great job, I'm impressed. I have a problem with the template: when I right click on the toolbar like you did in the video, I can't get the "Add Template" menu item. And by the way, clicking on the yellow "Settings" icon does nothing.

Thank you!

Serge B.


Pas si difficile que ça


102497980-Improve-waste-heat-function-handling-for-CoilCoolingDXMultiSpeed-and-CoilHeatingDXMultiSpeed-#5162 (lgu1234) - 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)


Should this be in the leanback folder?


unless wiki.private


<img width="252" alt="graylinetable" src="https://cloud.githubusercontent.com/assets/386877/9922892/ffc2247e-5cbb-11e5-84f0-4948c7921fd6.png">


4967-SZRH-spm (EnergyArchmage) - i386-Windows-7-VisualStudio-12: OK (1846 of 1879 tests passed)

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


device/motorola/victara/sepolicy/system_app.te:2:ERROR 'unknown type display_sysfs' at token ';' on line 18094:
allow system_app display_sysfs:file { { getattr open read ioctl lock } { open append write } };
#line 1 "device/motorola/victara/sepolicy/system_app.te"


Given that there's also some new contracts coming in, probably best to do a release now, and then another release in a few days/week with this and them.


Also note the inconsistency between the (much more attractive) LaTeX table style and the Word table style in the sample PDFs.


Vẫn lỗi anh ạ


Atualização do Master!


Apparently the `+` is supported, I take that back - http://docs.julialang.org/en/latest/manual/strings/#man-version-number-literals

Still not sure I get exactly what it means from reading that a few times though.


Acho que dá pra criar o Elfo Légolas aqui em cima pra não precisar repetir


nice placeholder


remove the two spaces above this.


Yeah, I do like this better. There is some advantage to only sending a `Core` rather than a whole `GlobalAddress`. For global completion event, since we already assume it has to be statically allocated, I think we can just send the `Core`.

There's some abstraction benefit to hiding this detail from users by making some more opaque `CompletionTarget` type. Or at least making a typedef. But it could just confuse people...

~~~cpp
class GlobalCompletionEvent {
  using CompletionTarget = Core;

  ...

  CompletionTarget enroll() { ... }
  void complete(CompletionTarget t) { ... }
}
~~~


Down the up two codes around the  `mSettings.setDIPScale(mDIPScale);`?


I'm merging but wanted to point out there are a number of places where " Red" was replaced with "$Projectname" and the space between that and the previous word are lost.


Hi.
BitmapPool can only specify the width and height.

In the case of the following source code:
What we can get cash?
```
Bitmap mask = mBitmapPool.get(width, height, Bitmap.Config.ARGB_8888);
```

```
Bitmap bitmap = mBitmapPool.get(source.getWidth(), source.getHeight(), Bitmap.Config.ARGB_8888);
```


Update


Duh, I definitely did that wrong!


2016? :+1:


@sanbornm Looks like the email address used for these commits is not associated with your GitHub account. If you add that address to your account, you'll be properly credited as a contributor. :+1:


I know this works but for consistency's sake, can you call with `()`? :)


(at least for future commits; not sure about the ones you've already pushed)


Cambio de Blog por Tips Issue


good job today :+1:


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


It is not documented in out api documentation?


In ListView, the ImageView's size may equals. It think it will work this time.


Sim, dá sim, já corrijo e dou push


MiscReleaseFixes (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (802 of 802 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-20.78%25-green.svg)


is the spelling `cal-cultures` intentional here?  Currently lives at http://calcultures.cdlib.org/ aka http://www.calisphere.universityofcalifornia.edu/calcultures/


Na master é pra dar emoção rsrs

tab_size: 2? Eu uso 4...
Essa é a config que eu uso, JP que me passou:
{
	"color_scheme": "Packages/Color Scheme - Default/Monokai.tmTheme",
	"ensure_newline_at_eof_on_save": true,
	"font_size": 10,
	"ignored_packages":
	[
		"Vintage"
	],
	"tab_size": 4,
	"translate_tabs_to_spaces": true,
	"trim_trailing_white_space_on_save": true
}

Mas por algum motivo, nos arquivos JS não pegou de primeira...
Enfim, obrigado pela dica!


TaskCluster: @sw337y-314159 does not have permission to trigger tasks.


it is safe to delete nullptr


use implicit condition, should be
```
if not out.writable:
```


@ZdrowyGosciu Add me on Hangouts or get one if you don't have one already. My gmail is ireallylikeshrek@gmail.com

Still getting the panel issue, last_kmsg here: http://pastebin.com/S0L4qGmy


No. it's not a XWalkView specific API.


I write a issue to `Glide`: https://github.com/bumptech/glide/issues/633

I am waiting for answers.


Bold this: **This is not software for everyone, this is software for our intended use.**
(I'm already getting tweets about why I'm not doing this for everyone)


Это значение по умолчанию. Можно не указывать.


I hope no one minds if I comment but as an avid user and supporter of Flyspray I feel slightly downhearted that after the great efforts made by the various contributors that the project now finds itself at a cross roads. I have watched over the past few months how the project has become more fragmented as people have struggled to meet the requirements set by the volume of work. This has had the knock on effect that the deadlines contributors have tried hard to meet have come and gone increasing the stress levels of all involved. The argument over fundraising, in this instance, using adverts has clearly hit a raw nerve and given rise to a whole heap of pent up emotions clearly showing just how much stress and frustration there is within the group. So this argument in one way or another was always going to happen. My personal feelings  about this whole thing is:-

the inclusion or validation of 'ads' in the software should be consigned to a separate fork for testing out these and other ideas. I'd probably prefer that you spent the time on closer integration with Git and Subversion and doing a plugin for Visual Studio so that I don't have to keep telling people that Flyspray is better that TFS and them saying but it's not integrated with the rest of the development environment, etc,etc.

the work that people are already committed to doing needs to finished off or at tied off so that it is at least usable if not completed to the full specification.

I do also feel that some of leading contributors do actually need a 'holiday' away from the project for a week or so. Tempers are frayed currently so now would be good time to have a break and then to come back and to have the discussion about moving the whole thing forward.

In short - don't let your hard work and good intentions go to waste having invested so much time and energy and have come so far.


Nit: odd indentation


Hi @spcui

1. According to the request, need test both vnc and spice scenarios.
2. How about the status checking after migration?


This is the single canonical place where the local config schema is specified.


> On 17 Sep 2015, at 10:50, Michio Honda <notifications@github.com> wrote:
>
> Hi,
>
> Sorry for delay because of vacation, I got reproduced your situation and understood why it happens.
> netmap recently changed such that VALE virtual ports (where app attaches to with MultiStack) do not have non-NULL ifp.
> Therefore, your patch on ms_host_na() makes sense.
>
> I pushed equivalent patches to the master branch.
> Let me know if you have any other problem.
Hi Michio,

thank you very much!
@Felix: Can you double check that your applications work?

Best regards
Michael
>
> Cheers,
> - Michio
>
> From: Michael Tüxen
> Reply-To: nplab/multistack
> Date: Monday, September 14, 2015 at 17:01
> To: nplab/multistack
> Cc: Michio Honda
> Subject: Re: [multistack] Checking for null pointer (8a2503b)
>
> > On 14 Sep 2015, at 16:48, Felix Weinrank <notifications@github.com> wrote:
> >
> > System info
> >
> > FreeBSD bsd1.fh-muenster.de 11.0-CURRENT FreeBSD 11.0-CURRENT #0 r287778: Mon Sep 14 13:07:33 CEST 2015 root@bsd1.fh-muenster.de:/usr/obj/usr/src/sys/FELIX amd64
> >
> > ix0: flags=8843<UP,BROADCAST,RUNNING,SIMPLEX,MULTICAST> metric 0 mtu 1500
> > options=8407bb<RXCSUM,TXCSUM,VLAN_MTU,VLAN_HWTAGGING,JUMBO_MTU,VLAN_HWCSUM,TSO4,TSO6,LRO,VLAN_HWTSO>
> > ether 00:1b:21:5e:bc:1c
> > inet 10.9.8.1 netmask 0xffffff00 broadcast 10.9.8.255
> > nd6 options=29<PERFORMNUD,IFDISABLED,AUTO_LINKLOCAL>
> > media: Ethernet autoselect (10Gbase-Twinax <full-duplex,rxpause,txpause>)
> > status: active
> > ix1: flags=8843<UP,BROADCAST,RUNNING,SIMPLEX,MULTICAST> metric 0 mtu 1500
> > options=8407bb<RXCSUM,TXCSUM,VLAN_MTU,VLAN_HWTAGGING,JUMBO_MTU,VLAN_HWCSUM,TSO4,TSO6,LRO,VLAN_HWTSO>
> > ether 00:1b:21:5e:bc:1d
> > inet 10.9.8.3 netmask 0xffffff00 broadcast 10.9.8.255
> > nd6 options=29<PERFORMNUD,IFDISABLED,AUTO_LINKLOCAL>
> > media: Ethernet autoselect (10Gbase-Twinax <full-duplex,rxpause,txpause>)
> > status: active
> > em0: flags=8c02<BROADCAST,OACTIVE,SIMPLEX,MULTICAST> metric 0 mtu 1500
> > options=4219b<RXCSUM,TXCSUM,VLAN_MTU,VLAN_HWTAGGING,VLAN_HWCSUM,TSO4,WOL_MAGIC,VLAN_HWTSO>
> > ether 00:19:99:63:af:af
> > nd6 options=29<PERFORMNUD,IFDISABLED,AUTO_LINKLOCAL>
> > media: Ethernet autoselect
> > status: no carrier
> > igb0: flags=8843<UP,BROADCAST,RUNNING,SIMPLEX,MULTICAST> metric 0 mtu 1500
> > options=403bb<RXCSUM,TXCSUM,VLAN_MTU,VLAN_HWTAGGING,JUMBO_MTU,VLAN_HWCSUM,TSO4,TSO6,VLAN_HWTSO>
> > ether 00:1b:21:73:a2:e8
> > inet 212.201.121.81 netmask 0xffffffe0 broadcast 212.201.121.95
> > nd6 options=29<PERFORMNUD,IFDISABLED,AUTO_LINKLOCAL>
> > media: Ethernet autoselect (1000baseT <full-duplex>)
> > status: active
> > igb1: flags=8843<UP,BROADCAST,RUNNING,SIMPLEX,MULTICAST> metric 0 mtu 1500
> > options=403bb<RXCSUM,TXCSUM,VLAN_MTU,VLAN_HWTAGGING,JUMBO_MTU,VLAN_HWCSUM,TSO4,TSO6,VLAN_HWTSO>
> > ether 00:1b:21:73:a2:e9
> > inet 10.0.1.201 netmask 0xffffff00 broadcast 10.0.1.255
> > nd6 options=29<PERFORMNUD,IFDISABLED,AUTO_LINKLOCAL>
> > media: Ethernet autoselect (1000baseT <full-duplex>)
> > status: active
> > igb2: flags=8c02<BROADCAST,OACTIVE,SIMPLEX,MULTICAST> metric 0 mtu 1500
> > options=403bb<RXCSUM,TXCSUM,VLAN_MTU,VLAN_HWTAGGING,JUMBO_MTU,VLAN_HWCSUM,TSO4,TSO6,VLAN_HWTSO>
> > ether 00:1b:21:73:a2:ea
> > nd6 options=29<PERFORMNUD,IFDISABLED,AUTO_LINKLOCAL>
> > media: Ethernet autoselect (1000baseT <full-duplex>)
> > status: active
> > igb3: flags=8843<UP,BROADCAST,RUNNING,SIMPLEX,MULTICAST> metric 0 mtu 1500
> > options=403bb<RXCSUM,TXCSUM,VLAN_MTU,VLAN_HWTAGGING,JUMBO_MTU,VLAN_HWCSUM,TSO4,TSO6,VLAN_HWTSO>
> > ether 00:1b:21:73:a2:eb
> > inet 10.102.0.1 netmask 0xffffff00 broadcast 10.102.0.255
> > nd6 options=29<PERFORMNUD,IFDISABLED,AUTO_LINKLOCAL>
> > media: Ethernet autoselect (1000baseT <full-duplex>)
> > status: active
> > em1: flags=8802<BROADCAST,SIMPLEX,MULTICAST> metric 0 mtu 1500
> > options=9b<RXCSUM,TXCSUM,VLAN_MTU,VLAN_HWTAGGING,VLAN_HWCSUM>
> > ether 00:04:23:c7:c3:0c
> > nd6 options=29<PERFORMNUD,IFDISABLED,AUTO_LINKLOCAL>
> > media: Ethernet autoselect
> > status: no carrier
> > em2: flags=8802<BROADCAST,SIMPLEX,MULTICAST> metric 0 mtu 1500
> > options=9b<RXCSUM,TXCSUM,VLAN_MTU,VLAN_HWTAGGING,VLAN_HWCSUM>
> > ether 00:04:23:c7:c6:3e
> > nd6 options=29<PERFORMNUD,IFDISABLED,AUTO_LINKLOCAL>
> > media: Ethernet autoselect
> > status: no carrier
> > em3: flags=8802<BROADCAST,SIMPLEX,MULTICAST> metric 0 mtu 1500
> > options=9b<RXCSUM,TXCSUM,VLAN_MTU,VLAN_HWTAGGING,VLAN_HWCSUM>
> > ether 00:04:23:c7:c2:d3
> > nd6 options=29<PERFORMNUD,IFDISABLED,AUTO_LINKLOCAL>
> > media: Ethernet autoselect
> > status: no carrier
> > em4: flags=8802<BROADCAST,SIMPLEX,MULTICAST> metric 0 mtu 1500
> > options=9b<RXCSUM,TXCSUM,VLAN_MTU,VLAN_HWTAGGING,VLAN_HWCSUM>
> > ether 00:04:23:c7:c4:f1
> > nd6 options=29<PERFORMNUD,IFDISABLED,AUTO_LINKLOCAL>
> > media: Ethernet autoselect
> > status: no carrier
> > lo0: flags=8049<UP,LOOPBACK,RUNNING,MULTICAST> metric 0 mtu 16384
> > options=600003<RXCSUM,TXCSUM,RXCSUM_IPV6,TXCSUM_IPV6>
> > inet6 ::1 prefixlen 128
> > inet6 fe80::1%lo0 prefixlen 64 scopeid 0xc
> > inet 127.0.0.1 netmask 0xff000000
> > nd6 options=21<PERFORMNUD,AUTO_LINKLOCAL>
> > groups: lo
> >
> > Prepare netmap and multistack
> >
> > Create a vale switch with igb1 attached to it
> >
> > /usr/src/tools/tools/netmap/vale-ctl -h valem:igb1
> >
> > double check
> >
> > weinrank@bsd1:~ % sudo /usr/src/tools/tools/netmap/vale-ctl
> > bdg_ctl [149] bridge:0 port:0 valem:igb1
> > bdg_ctl [149] bridge:0 port:1 valem:igb1^
> >
> > Load multistack module
> >
> > kldload ./multistack.ko
> >
> > double check dmesg
> >
> > 602.709218 [1673] netmap_interp_ringid deprecated API, old ringid 0x0 -> ringid 0 reg 1
> > igb1: link state changed to DOWN
> > igb1: link state changed to UP
> > MultiStack: loaded module
> >
> > Run pkg-gen
> >
> > ./pkt-gen -i valem:gen -s 10.0.1.201:9899 -f tx -d 10.0.1.202:9899
> >
> > System crashed
> @felix: Thanks for providing the information.
> @michio: Can you reproduce the problem? It would be great if we could get the stuff stable...
>
> Best regards
> Michael
> >
> > —
> > Reply to this email directly or view it on GitHub.
> >
>
>
> —
> Reply to this email directly or view it on GitHub.
>
>
>
> —
> Reply to this email directly or view it on GitHub.
>


Этот бин необходим для настройки Spring Security, поэтому его лучше перенести в Security Config или еще куда, но точно не смешивать с общими настройками. Необходимо стараться делать компоненты независимыми и отчуждаемыми друг от друга.


This is the single canonical place where the global config schema is specified.


does this actually change the startdate of the event? cause I can see it changes the endTime, not the startTime though


Woops, this was meant for the Facebook branch, not a commit on the master. We can leave it in the master though if @tomkr agrees.


Опять же это настройки Spring Security. И в данном случае, я даже думаю, что этот бин не нужен совсем, т.к. он настраивается по умолчанию.


you are doing awesom. if you can tell me what those instructions do, i can fill the esil-expressions for you


we are stuck on iojs@2.x until we can update `sodium` to use the new `nan`, which changes quite a bit.


Бины с дао лучше вынести в отдельный конфиг и @Import-ить в главный.


... аналогично и с бинами для DAO.


sir inheritance aur upcasting downcasting kaha hai


huba ^_^


Basically you pass in `issue_tracking` and it gives you what is stored in `modules.issue_tracking`. Bit of a magic in Go when the config is a struct. Could have used a map, but whatever...


The vehicle is getting a short impulse and based on its physical characteristics (gravity...) it stays on ground or not.


star.


Probably better to check `load()` down there to understand what these functions do and what the args are about :-)


Why we need new source for each client? Single source works perfectly fine, see:
- my commit: [make messageEventsSource application-scoped](https://github.com/pawelkaczor/reactive-flows/commit/6ba6859fd3f8e1087ff619d1ab9e9a0a75305478)
- my test tool: https://github.com/pawelkaczor/sse-test


goo shit code


shouldn't this resultset be `TrackingVariant`?


gogog


@majianpeng this change fails http://pulpito.ceph.com/kchai-2015-09-16_07:54:44-rados-wip-kefu-testing---basic-multi, to be specific,

`test_xattr()` in `./qa/workunits/rados/test_rados_tool.sh`. you can rerun it in your local env to reproduce it.

we need either update the tests or remove this change? what do you think?


Glad this idea worked)


Actually implemented by `config.LocalConfig` and `config.GlobalConfig`...


@vilchik-elena Could we assert something here?


would `Sub::Name` be useful here?


Тут надо определиться -- вы используете чистый Hibernate или все-таки Hibernate, как провайдер JPA?

Если первое, то можно убирать некоторые зависимости, например, `hibernate-entitymanager`, если второе, то нужно изменить настройки и настраивать как JPA и не использовать hibernate-specific настроек.


What's the reason for splitting over two lines? I find the original line more readable.


!mustfix Put some comments in the code to document the interfaces.


thanks for the update, this caused trouble cross-compiling in cray environments and this fixes it.


**Code Quality - Critical**

 - `Books` is not defined, and will be treated as a global variable.
 - `Books` isn't defined as a class, and should be in lowercase.


Так делать в любом случае никогда не нужно. Если вы не знаете, что сделать с исключением, то просто пробрасывайте его выше, а не "глушите" его вот так.


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


Этот бин точно нужен? Для чего?


I do not think we need different layouts, this was just a comment on your comment of natural or not. If your personal perspective is a differential robot it could be unnatural, if your perspective is anything else, like a UAV or a omnidirectional wheeled platform it is more natural. I think the new configuration is more close to games, where you usually separate between movement and rotation. But anyway it is not a problem to control any robot with this configuration, everyone should be able to figure this out in seconds.


`users_count.times do ...` ;-)


Got it. So one question is left - what to do with snprintf. Part of logic is in the `pcap-int.h`, another part is in the `pcap-stdinc.h`


It's actually almost funny that only now that I'm knee deep in Paper Walls do I realize how much of a good ACT 1 That-Day actually is. It's self-contained enough to stand on it's own but it can stand as the first third of a larger story.


Как смысл в этой константе? Название сервлета почти никогда не меняется.


Not sure if this method should be made `protected`

I don't really see what would be the use case where we would have the need to access this method specifically.

The thing we may provide as protected method would be an extracted method of :
```
            resp.setLogLevel(RestxLogLevel.QUIET);
            resp.setStatus(HttpStatus.OK);
            String contentType = HTTP.getContentTypeFromExtension(relativePath).or("application/octet-stream");
            Optional<CachedResourcePolicy> cachedResourcePolicy = cachePolicyMatching(contentType, relativePath);
            if(cachedResourcePolicy.isPresent()) {
                resp.setHeader("Cache-Control", cachedResourcePolicy.get().getCacheValue());
            }
            resp.setContentType(contentType);
```
in `handle()`

WDYT ?


Might make sense to check one of the actual implementations of these interfaces in the modules directory to see how it all works. Just pick one of the commits later in the list.


As soon as this pulrequest https://github.com/E2OpenPlugins/e2openplugin-Foreca/pull/23 has been honoured, and the OE-A version has been updated accordingly, Foreca should be working again for everybody.


Please remove console.log.


**Code Quality - Critical**

- Missing semicolon, detrimental in production environment when **strict** Javascript is required.


!mustfix Use `prompt.Dialog` for this. There is the `secret` parameter now that can be used for this.


:ok:


Excellent catch, didn't occur on my machine because I've got the hook installed. You actually uncovered a different issue: travis doesn't fail when tests fail, I reckon this has to do with  run_tests.sh return status, I'll look into this.


is this necessary?


Ive picked just the important ones :D


testing


Возможно, что имеет смысл отнаследоваться от `AbstractAnnotationConfigDispatcherServletInitializer`. Либо, как минимум, отрефакторить `onStartup()` и извлечь 3-4 метода.


I remove them altogether for now


why simple when there is complicated ;-)
but do won't do (also a good one)
 we should use map because you probably want to investigate what you have for more elaborate tests


Tämä ei varmaan tuota jUnit muotoista xml-raporttia? Jos tuottaa, varmaan Jenkinsillä ei ole otettu tuloksia mukaan JUnit report post-steppinä, joka analysoisi tuloksen. Aiheuttaako faili Jenkins buildin failuren?


Thanks! It is simple and useful!


Код выше -- кандидат на вынос в отдельный метод.


removed as you suggested.


ㅁㄴㅇ


**Code Quality - Nitpick**

 - `key` is already defined in as an argument, you shouldn't define it again.


1) Все методы в интерфейсах, всегда имеют неявный модификатор `public`, поэтому писать его не нужно.
2) Использовать непараметризованный `List` (и любые другие коллекции) плохо. Так делали до Java 5, пока не появились generic-и, сейчас везде нужно указывать явно тип содержимого.


Hyvä juttu, lisään tämän! :)


yeah, right. return value is different.


Thanks Bill,
your argument is very valid, I will add exception handling around socket.gethostbyname() to use domain name as is if it fails to resolve.


Этот комментарий должен быть в формате javadoc (т.к. начинается с `**`), но он не в формате javadoc и вдобаво бесполезен. Авторство и дата мало нужны, а когда нужны всегда есть в git-е. Если хотите указывать авторство и дату, то тогда нужно использовать теги из javadoc -- @author и @since. Настройте IDEA, чтобы она либо не вставляла вот такие бесполезные комментарии, либо вставляля сразу нормальные.


@vilchik-elena What's the reason for giving a higher priority to LABEL?


Не указан модификатор. Добавьте `private`.


@ccx-testerbot runtests master


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


@adegroote @pierriko any thoughts?


Thank you for the request. It has been added to the queue (id: 91). To see the progress you can go to the [status](http://ccextractor.canihavesome.coffee/test/91) page. Please note that depending on the current queue, it could take a while before any results will be visible. In each case I will report back here once the tests are done.


Я вижу, что здесь вы используете чистый Hibernate... Это не JPA.


the concern inclusion!!!


@vilchik-elena Should we match against STATEMENT()? The existing PHPGrammar uses INNER_STATEMENT_LIST which can match a FUNCTION_DECLARATION.


Is `returnsEnumeratorIfNoBlock` not working here?


only try to require module to include if explicitly told so!


Как правило, использовать импорты со звездочкой не рекомендуется.


@MEZk
I see, but we need to solve that problem. We cannot allow new option that produces false positives. Can you propose some solutions? We'll decide together what to do.


scope name is the name of the current context is `ns: true`


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


or namespace can be defined explicitly by string value from one of these options


Название сущности мало связано с названием таблицы. Советую переименовать класс. Правда, в конкретном случае, класс придется переименовывать в `Person` :)


@vilchik-elena why an empty method?


star.


@pynicolas ah, indeed, forgot to fill it. Thanks!


hi rahul


Если у вас всего один конструктор и он без аргументов, то указывать его явно не имеет смысла (т.к. JVM создаст его для вас автоматически).


hfd


An error occurred while running the tests. Please check the [status](http://ccextractor.canihavesome.coffee/test/91) page, and correct the error. If you have a question, please contact @canihavesomecoffee.


@philss hey, thanks for the mention :D


Эта стратегия используется по-умолчанию, нет смысла указывать ее явно.


looks good to me


A GtTagValueMap always ends on `\0\0` so it should be fine.


@ccx-testerbot runtests master


@brendanashworth I removed them because, later on if we decide to remove the `events.EventEmitter`, then we anyway have to do this change. Apart from that, we practice what we preach and stuff ;-)


failureUrlにはdeleteしかないので、このcrudModeは不要です。
ただ、動作確認はまだなので、残しておきます。
これを削除する際には、confirm.jsp内の参照されている箇所も修正する必要があります。


Если название поля сущности соответствует названию поля в таблице, то `name` можно не указывать явно.


I'd vote for keeping current behaviour for arrow keys.
There is no problem adding new keys for different control though.


GroundDomainUpdate (mmatts) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1864 of 1880 tests passed)

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


Acho que são temporários, são os aces? Podíamos por está aba para o pessoal meter links para test. Pk qada sistema é proprio tipo o mesmo app ssiptv na lg ou samsung não responde da mesma maneira aos links.

--- Mensagem Original ---

De: "LuisAraujo1" <notifications@github.com>
Enviado: 17 de Setembro de 2015 00:24
Para: "adrianoafonso/myIPTVchannels" <myIPTVchannels@noreply.github.com>
Cc: "reivan85" <reivan_mgl@hotmail.com>
Assunto: Re: [myIPTVchannels] Create Beta links ON\OFF (67cbfcd)

Esta on com uma qualidade excelente.

---
Reply to this email directly or view it on GitHub:
https://github.com/adrianoafonso/myIPTVchannels/commit/67cbfcd6659560143af49ce80d7b31e9b6caa7c1#commitcomment-13275650


eUrlにはdeleteしかないので、このcrudModeは不要です。
ただ、動作確認はまだなので、残しておきます。
これを削除する際には、confirm.jsp内の参照されている箇所も修正する必要があります。


Скорее всего, имеет смысл пометить поля surname/name/lastName/age как `nullable = false` и аналогично обновить поля в БД. Может быть некоторые и могут быть NULL, но явно не все сразу, ведь в этом мало смысла.


#5167-ExampleFile-VariableRefrigerantFlow_FluidTCtrl_5Zone.idf-Warnings (rongpengzhang) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1344 of 1344 tests passed)

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


@gabest11 could you please review this patch? There could be nuances which I've missed, specifically, is there a reason for PCSX2 in NTSC Progressive mode to use 30/60 fps (before this patch) instead of 29.97/59.94 which the patch does?


Thank you for the request. It has been added to the queue (id: 92). To see the progress you can go to the [status](http://ccextractor.canihavesome.coffee/test/92) page. Please note that depending on the current queue, it could take a while before any results will be visible. In each case I will report back here once the tests are done.


Зачем использовать локализованный вывод в `toString()`?


Fixed #3


This is one refactoring too far. This should actually be named just `@user`. The same is true of this whole controller and its views.


Этому методу здесь не место. Как минимум, нужно вынести его в какой-нибудь класс Utils.


Indentation is wrong here, using tabs and mixing them with spaces.


@fbq thx for reminding~ :D


Didn't need this


можно просто this, думаю будет работать


I am confused whether we're using one common PropertiesPanel or two separate entities. They use different attribute name, so need different selectors. I didn't want to introduce conditions into common.PropertiesPanel, so I just select a panel here and verify if it exists.


Thanks for this initial contribution!
A couple of notes:
- check for double spaces after almost every full stop or semicolon (should be single space)
- I would prefer if we use a more impersonal approach when writing docs (for instance switching the "I" with "we")
- make sure you take advantage of the rst markup features for bold and italic
- you can preview your pages here http://rst.ninjs.org/

Cheers,
F


It's an encoding issue, not related to code.
try changing the encoding setting in your TV.

please note that some languages are not supported, for example Hebrew is shown as dots .


В чем причина использования собственных валидаторов? Почему не использовались стандартные `@NotEmpty/@NotBlank` и `@Min` из hibernate-validator?


Лишняя строка.


Looks like a great start Thomas! I'd suggest also mentioning that encryption can be enable on a per-file basis using chattr +c, which is not dependent on the -o compress mount option.


I left some old code here, just commented it. Might be useful for further deployments testing, but it's outside of the scope of current task.


Could you try `system("echo $PATH") ` in R and make sure perf is in one of folders returned?
That should fix the problem but I don't like the solution. Environment manipulations from R seems to be platform dependant. Also perf may conflict with perf monitor tool from `linux-tools-common`.
Would be better to refuse perf usage and use one of AUC functions provided by R's packages. I think ROCR package should suit or some other available from public CRAN. It's much better to depend on another R's package than on external tool.
@martinbel Martin, could you possibly make such improvement?


We need casting since the module objects as passed around are of `loader.Module` type, but we need to return the interfaces as specified in the `modules/common` package.


> This is wasteful. You'd be much better off creating a composite operation

I lowered the priority of doing it (by deferring it indefinitely  :smile:) since I had a feeling it'd not make a visible difference, since the admin client is reused across operations and network is local anyway. I will send a PR to fix that.

> This would have better discussed in a pull request,

Sorry for that, I thought this change would fall in the "trivial" category


Yeah, all `GetXXX` methods basically look the same. Didn't find any way how to get rid of the duplication more.


Ни здесь, ни выше, я не вижу смысла в использовании групп. `@NotEmpty`/`@Positive` должны воспринимать `null` как валидное значение и не выдавать ошибок. Если вы хотели не вызывать их лишний раз, то это бесполезная оптимизация (вы сделали код менее очевидным, добавили 2 интерфейса) и сэкономили пару тактов, которые пользователь никогда не заметит.


On the other hand, it's way better than the duplication before :-)


Снова пустой и бесполезный no-arg constuctor.


Бесполезный комментарий/ненужные явные `public` модификаторы. Убрать :)


!mustfix Use `prompt.Dialog`


Возможно, что имеет смысл добавить также атрибут `readOnly = true`.


Corrected.


!mustfix This is not too pretty, forgot to refactor. Also, make sure there are no duplicate labels.


I don't like this line. But I'd like a failsafe to avoid unnecessary if-else after the pointer *data has been exhausted.


Если вы создаете этот бин "вручную" через джава конфигурацию, то почему бы не создавать его сразу с зависимостями? В чем преимущество сделать `new` и потом Spring будет инжектить зависимости за вас? Тогда можно отказаться от создания бинов в джава конфигурации и использовать сканирование.


You're right. It is docker_port in line 57. Corrected.


In the current implementation, when nat_aware is TRUE, LISPmob uses the RTR for all the interfaces even if they are not behind NAT. Therefore, the process to check for repeated RTRs should be done for all interfaces.


Following the draft, interfaces using private addresses should be added with R = 0. In this piece of code, the first locator using the RTR uses as the address of the locator the address of the RTR, but then is not added again using its own private address. The other locators using the RTR use as a locator address their own private address.


It may also be worth linking to this new content from the Samba + Btrfs compression section at documents/SLES12-SP1/SLES-admin/single-html/#samba.advanced.compress .


@mkordas
As I sad, only add new option to exclude some bundles.

Just imagine the situation:
we have the following property file:
````xml
checkstyle=com.puppycrawl.tools.checkstyle.ant.CheckstyleAntTask
````
Does it require translation?

And what about this example:

````xml
checkstyle=com.puppycrawl.tools.checkstyle.ant.CheckstyleAntTask
checkstyle.errorMessage="Checkstyle error message text"
````
Does it require translation? How can we distinguish these files?


This module is probably going to be abandoned in the future, so no need to be too strict. But it's not like I didn't try to do a proper job ;-)


BTW `<pcap-stdinc.h>` is included twice in `scanner.l`.


i don't know


No need for this any more, the user is prompted.


"You can't step into the same river twice". That means, once an event has been consumed by sending it via HTTP to one client/browser, it's gone, and no other client/browser will receive it.

Your test only opens a single SSE connection (`client.js`). Try to open more ...


Commit message is wrong. Should be columnchart.


wwwwwwwwwwwww (///w///)


Most of the interesting stuff happens in the loader package anyway...


Boa, parabéns!!! Funciona muito bem, pena não dar para smartTV :(


Checkmarx scan results 17 Sep 2015 13:43 UTC +03:00

High
Medium
Low
Info

New
0
5
11
0

Recurrent
0
0
0
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

90003e56d4511b9e1f1b097c10578cb41f77ca30	@CxTest


this is


@xhero and @cgueggi : translations


@xhero : translations


@cgueggi : translations


ks=cdrom:/dev/cdrom:/  - так задумано?


I see at least couple of solutions:
1. Do not require translations, if no translations are provided (require all only if at least one is present)
2. Whitelist of bundle names to be checked
3. Blacklist of bundle names
4. Whitelist/blacklist of directories/paths to be checked

In our case 2. probably would be the best. We could set it to 'messages'. But what about other projects?


The test suite finished running the test files. Below is a summary of the test results:

| Report name | Tests passed |
| ----------- | ------------ |
| _Broken | 15/15 |
| _DVB | 5/5 |
| _DVD | 3/3 |
| _DVR-MS | 2/2 |
| _General | 24/24 |
| _Hauppage | 3/3 |
| _MP4 | 2/2 |
| _NoCC | 10/10 |
| _Teletext | **20/21** |
| _WTV | 13/13 |
| _XDS | 34/34 |
| _Options | **83/87** |
| _DTVCC | **0/14** |
It seems that not all tests were passed completely. This is an indication that the output of some files is not as expected (but might be according to you). Please check the [results](http://ccextractor.canihavesome.coffee/reports/92) page, and verify those files. If you have a question about this report, please contact @canihavesomecoffee.


Gitgud, the M.radiation = max(0,M.radiation -= 3) part might not work as intended.
That line is trying to assign M.radiation twice for fuck sake. I'm not even sure if that's legal.


this is me not making any comments about javascript
@samphippen does not work


Это для нахождения ksboot.iso


@andreiterbea I am adding something to the scrum exercise project


CoCSharp is the core library. It shouldn't have the plugin API in it.


I need your opinion on this change. Here's mode details: https://github.com/j3k0/logbookr/commit/a9fcfdae3cac8626d28ecf9f5a382a8859c55757


@FroMage this isn't good style. We don't directly address the reader as "you".


@ddiss Thanks! This is very helpful and I'll integrate it into the file. :+1: Many thanks for all the tips. :smile:


LGTM
Thank you for removing checker from state creation helper \o/


Sure, but that's incompatible with the interface currently mandated by `Match`, which supposes that we can discover
existing resources exclusively from tags, by supplying parameters to `Rq infra`. AFAICT there's no parameter to [`DescribeLaunchConfigurations`](http://docs.aws.amazon.com/AutoScaling/latest/APIReference/API_DescribeLaunchConfigurations.html) that will say "only match launch configurations who's name has this prefix", and the obvious thing to do seems to be extending the interface to admit the application of optional post-request filters in `discover`..

I like the `Hashable` idea.


eigenartig bei meinem Revertversuch hier, erkennt er meine gemachten tatsächlchlichen Veränderungen


GCC 4.8.5 installed on my Snow Leopard Mac, thanks @evil-at-wow
Now, I hope I will still be able to build the core once you pushed those commits.


在TransferingMilitary裏已經自行由建築取得列表
`faction.TransferingMilitary.LoadFromString(this.Militaries, reader["TransferingMilitary"].ToString())` 這句是沒作用的


@mkordas
>In our case 2. probably would be the best. We could set it to 'messages'. But what about other projects?

Yes, and what if they have xxx-messages.properties and xxx.properties and they both require translations.

3 will be the best I guess. Users can define which bundles will not be checked. But what about the name of the option? We should point that the check will not check only existence of translations in the bundle, but not absence of keys


Can you explain me this stuff "[tests,docs,github,invoke]"?


OK, I changed it, is that better?


Hello, the "problem" of this fix is that's depends always how you individually named the gateways ...
I had used "flex-o" before e.g. and also mk.de uses this kind of DID (when I tested it 4 years ago).
The "new trunk" (from Flex-O / now PBX-Network) is also over 3 years old and based on Teles NGN hardware.

Perhaps it's better for common usage to define the preg_match as variable which could be setup in global config file with a nice default matching rule ?


Much better. Thanks.


GroundDomainUpdate (mmatts) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


This change caught my attention because it goes in another direction than all other changes related to changing packages names: instead of adding `workflows` part, you removed it. However, I checked that this is a leftover from one of the two initial commits made on GitHub: apparently this `workflows` part was introduced by mistake in 63feac07627e0b6edb4ab8d0fde8005f6d63a052. So it's good that you fixed it up. The same situation is in the `iis-workflows-metadataextraction` project.


Is there a reason not to trust Hydra.nixos.org? (both `trustedBinaryCaches` and `binaryCachePublicKeys`) I don't mean it should be *used* by default, but IMO it makes sense to *trust* it.


感谢你


- パーミッションキャッシュは ユーザーパーミッションと、 ユーザーの所属するグループのパーミッションをマージ

~~py
>>> from django.contrib.auth.models import *
>>> User.objects.first()
<User: admin>
>>> u = _
>>> u.user_permissions.all()
[]
>>> Permission.objects.filter(group__user=u).count()
562
~~~

- パーミッションキャッシュは set

~~~py
>>> from django.contrib.auth import get_backends
>>> from django.contrib.auth import models
>>> get_backends()[0].get_all_permissions(models.User.objects.all()[1])
set([])
~~~

- has_module_perms では、 app_label.codenameの一覧をキャッシュしておいてそれで判定
~~~py
>>> models.Permission.objects.values_list('content_type__app_label', 'codename')
[(u'admin', u'add_logentry'), (u'admin', u'change_logentry'), .....
~~~


Naah, Made it all backwards compatible ;)


Ok, I use tests for command injection to test taint analysis, so just do not remove priorities from there. Ideally, taint analysis should have its own tests not testing sinks and priorities, but the Taint instances at various locations; however, this kind of testing wouldn't be so easy to implement. In every case, I have already verified several times, that passing the tests is not enough to find bugs after refactoring, so I always run the analysis with the real code to derive many summaries and check the diff with previous versions.


Добавь к классам nav nav-pills nav-stacked еще один, affix-top


GroundDomainUpdate (mmatts) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (806 of 806 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-20.86%25-green.svg)


Exactly, there were two places where `workflows` part was mistakenly added to `iis-core` class package name. So this change is kind of a `revert`.


Já tem na lista smart_tv

--- Mensagem Original ---

De: "aminhaarroba" <notifications@github.com>
Enviado: 17 de Setembro de 2015 11:42
Para: "adrianoafonso/myIPTVchannels" <myIPTVchannels@noreply.github.com>
Cc: "reivan85" <reivan_mgl@hotmail.com>
Assunto: Re: [myIPTVchannels] Create Beta links ON\OFF (67cbfcd)

Boa, parabéns!!! Funciona muito bem, pena não dar para smartTV :(

---
Reply to this email directly or view it on GitHub:
https://github.com/adrianoafonso/myIPTVchannels/commit/67cbfcd6659560143af49ce80d7b31e9b6caa7c1#commitcomment-13283656


this adds double slash


gg


ეს ფაილი არ უნდა იყოს ატვირთული


fix for dropbox/hackpad#25


А для Spring Security, где?


@mkobos can you check whether previous version of this test passes on your computer? I had to change it to make it passing in my environment although Krzysiek claimed Today all unit tests passed on his computer which means it worked without the change introduced in this branch.


@rcrichton Wouldn't `every` be better than `reduce` in this case? https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/every


ეს არასწორია მერე `npmjs.com` იდან რო მოგინდება რეპოზე გადასვლა მანდ მოხვდები


Нужно стремиться к тому, чтобы в коммите небыло смеси из различных изменений. Одно изменение -- один коммит.


.idea დირექტორია არ უნდა ატვირთო, ეცადე დააიგნორო და ამოშალო რეპოდან ყველა ფაილი და ფოლდერი რომელიც ამ დირექტორიაშია.


I think this could be simplified.
Instead of using ObjectList where it was. We can just declare most of the variables the same once. Then call objectlist for each list of files. That way the common variables are kept together. Does that make sense?

Also, this only supports one level of directories... I guess that is enough to begin with.. It's not a simple problem.


Argh. So what's the right thing to write to say 3.x or 4.x where x is
anything?
On Sep 16, 2015 7:24 PM, "Tony Kelman" <notifications@github.com> wrote:

> #11 <https://github.com/madeleineudell/ParallelSparseMatMul.jl/pull/11>
> still applies
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/madeleineudell/ParallelSparseMatMul.jl/commit/15b53b300e946cd1786cb23e96f25c665382c687#commitcomment-13275655>
> .
>


მგონი სწორადაა, რეპოს ლინკია.

<img width="372" alt="screen shot 2015-09-17 at 15 06 49" src="https://cloud.githubusercontent.com/assets/1139050/9931572/d8e92d56-5d4d-11e5-92ee-fe58a1b4f90f.png">


არც .iml ფაილების ატვირთვა გჭირდება


ესეც დააიგნორე


It's more conventional to throw exceptions but its not necessarily good. I agree that optional is no replacement for error reporting since it has no information about what went wrong but i think its ok as long as there is only one failure condition that is denoted by an empty optional so its unambiguous.


develop (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1344 of 1344 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)


დააიგნორე


Should probably use this instead http://php.net/manual/en/function.http-build-query.php

Also, I'd name the 'args' argument either 'parameters' or 'params', seems more common.


Thanks for catching that seems like i missed that while refactoring :smile:


Please write python3 compatible code. print without braces is not python 3 compatible.


Did you commit the class AutomationIntegrationTest2 on purpose? Looks like a copy of AutomationIntegrationTest.


this seems to render the first frame twice?


Good idea to reuse that `-Dakka....`!


I mean, it hurts discoverability, but the relations should be enough to go on.


41 39c4 763e (QTnEdj4=) 改成 41 39c4 773e (QTnEdz4=) 版本 10.11


Sorry, removed my comment as I just saw https://github.com/marinmitev/smarthome/issues/85#issuecomment-141026116.


41 39c4 773e (QTnEdz4=)


develop (mjwitte) - 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)


Would you mind at least adding your comment inside empty catch block. Empty catch block just seems suspicious to me.


Correct! Need to make it smarter


Cool, thanks! Will try it out and merge later today.


For reference the `master` branch already has support for `VS2015` including a `VS2015` version of `libcef_dll_wrapper.lib`.


@beledouxdenis
I still don't get it why almost all devs at Odoo SA (at least the code gives me most of the times the impression) do use increments of one for order sequences whereas it is smarter to use for example increments of 10? Especially this can be used to avoid collisions and massive recomputations by logic (=more freedom to put something in between two sequences). Any idea or is it just random and nobody cares about this basic principle?


Increment version for modification.xml?


I moved this line from line 1468(two lines above). Had to read a bit about the function.
When the flag HOEDOWN_EXT_SPACE_HEADERS is enabled the hash prefixed headers require a space after the hashes, probably to differentiate the hash in the header content from the formatting hashes. If the flag is not enabled too, the function will count the levels, which is an additional overhead for is_atxheader .

The other option would be to restore the is_atxheader and make it not call is_atxheader_level everytime.


update on top of 4.1.3


maybe we should show this after decryption actually failed?


in learning process.


Do you need the retry loop also for ```add_slave```, ```remove_slave``` and ```destroy``` ?


Maybe the name should be in plural because returns a list of measures


Send a pr as this code is already merged


no need to call the default constructor of i_parser, since that is done automatically upon the owning class constructor (game_engine())


i done it


(͡ ° ͜ʖ ͡ °)


This patch did the trick :+1: (Tried on Archlinux) Thanks @rholy


`julia 0.3 0.5-`

Lower limit of 0.3.0 release, upper limit of 0.5 prerelease. The upper limit is exclusive if i remember the docs correctly. You could leave off the upper limit for now and go with just `julia 0.3` and not worry too much about 0.5 yet.


<!DOCTYPE html>

    <meta charset="utf-8" />

    <title>Q&Atopic</title>

    <link rel="stylesheet" href="http://static.zhihu.com/static/ver/c63e03aef14231ac7a2e29e60204be55.z.css">

                Topics

Commerce

                               »

Financial

                                Pay attention

                                Cancel attention

                              »

Travel

                                Pay attention

                                Cancel attention

Design

                                Pay attention

                                Cancel attention

Home furnishing

                                Pay attention

                                Cancel attention

Construct

                                Pay attention

                                Cancel attention

Chemistry

                                Pay attention

                                Cancel attention

Health

                                Pay attention

                                Cencel attention

Moives

                                Pay attention

                                Cancel attention

Fashion

                                Pay attention

                                Cancel attention

Natural science

                                Pay attention

                                Cancel attention

Vehcile

                                Pay attention

                                Cancel attention

Economy

                                Pay attention

                                Cancel attention

Food

                                Pay attention

                                Cancel attention

Environment

                                Pay attention

                                Cancel attention

            More

Contact us

Links

        © 2015 Q&A


An update for iOS app on App Store is required for supporting this?


typo: 'leaste' should be least


The same for the other properties of all classes: meter, concentrator, parameter... all of them are lists.
Isn't more readable `report.concentrator[1]` than `report.concentrators[1]` ?


Fair enough, i think I've seen those warnings


Se devo SOLO fornire un servizio tanto vale che iscrivo il provider direttamente a Services

Se invece il mio provider ha bisogno di prendere i riferimenti di altri servizi (Initialize), di calcolare logica (Update) o di disegnare (Draw) deve estendere la classe GameComponent (oppure DrawableGameComponent che è un GameComponent col metodo Draw) e lo iscrivo ai componenti


When are there exceptions from the "default"? Or do we mean simply "All parameters can be overridden"?


I am actually seeing a `stack level too deep (SystemStackError)` when trying to fetch from a site which has 301-redirected its `robots.txt`.

I think what is happening is
* Try to download `/`
  * Download `robots.txt` to check if we can download `/`
  * 301 redirect, download `robots_foo.txt` instead
    * Download `robots.txt` to check if we can download `/robots_foo.txt`
    * 301 redirect, download `robots_foo.txt` instead
      * here we go again...

Perhaps https://github.com/sparklemotion/mechanize/blob/v2.7.3/lib/mechanize/http/agent.rb#L998-L1002 (or the `fetch` in `get_robots`?) needs some kind of short circuit to allow `robots.txt` to be somewhere other than `robots.txt`?


yo


As a matter of style, one usually avoids writing "using" statements in header files, since this will propagate the "using" statement to any file who includes that header, thus *silently* poluting the scope with possibly unwanted definitions from that namespace.

For this particular case this means you'll have to write std:: on every std member used in this header file.
You can still write "using namespace std" on the implementation (cpp) file, since no one usually includes implementation files in other files.

Some people will argue that having full qualified (std::) items in the header file and not in the implementation file (by writing the using statement there) is inconsistent. I don't find that a big problem, and am ready to live with that lesser evil. Nevertheless, avoiding "using namespace" statements altogether is also an option.


Is this removed intentionally with the idea of perhaps deprecating config_path?


i don't think this check makes sense now, since love.physics is either nil or has getMeter anyway


Just saw this. This is NOT ok for me and has to be immediately reverted


Petite erreur d'inattention de ma part @yaf . Un commit et un push qui est parti trop vite ...
Ce commit et celui d'après corrigent les specs des controllers qui etaient cassés depuis les updates des factories users et accounts.

Le travail avait déjà initié avec le welcome_controller_spec qui avait été validé dans la dernière PR.
Donc petite erreur d'inattention pas bien grave.


**Comments/README - Nitpick**
Update the `readme` to reflect the changed directory structure, i.e. `one` to `1A`


yes I know, you're right, but my idea is to move the button in all those forms. I find confusing to have more submit buttons, people may wonder if it is necessary to submit changes before changing section.
That is my idea, at least.


Saba Baig Authored this commit.


Probably, this should be return value.lower()?!


Qui c'è un pasticcio! forse è roba scritta di fretta l'altra notte a malgioglio?
cmq gui o lo inizializziamo, oppure lo peschiamo dai servizi ( se l'abbiamo già creato prima)


Could even omit the entire intro about the old, "conventional", BAD way. The aim is for the new way to be conventional after all, good not only because it has a new feature (overriding by command line) but also because removing internal clutter by object orientation. Hopefully.


lelelel


yo bro


erledigt.


Seems like idea messed up my indentation. There are 4 cased even in your code the last line encodes 2 cases.
The same cases are encoded in my code (see comments below). I admit its not what you usually see in java code and it might be a bit harder to understand if you are not familiar with functional programming but its definitely worth learning and utilizing those concepts. What it does it naturally transforming the optional into the result we want instead of poking a the optional with a combination of `ifPresent` `get` and early returns.
```java
 return this.getReleaseCandidate().map(lhsRc ->
                        other.getReleaseCandidate()
                                .map(lhsRc::compareTo) // left hand side has rc and right hand side has rc so we compare
                                .orElse(-1) // only left hand side has rc so we (left hand side) are smaller
).orElseGet(() ->
                        other.getReleaseCandidate()
                                .map(rhsRc -> 1)  // only right hand side has rc so we (left hand side) are bigger
                                .orElse(0) // neither left nor right hand side have rc so we are equal
);
```