>  That's how Linux versions of libpcap, for example, are built;

So, it is possible to have the same for Windows...

> because the code that supports them in libpcap calls APIs from vendor libraries that are, as far as I know, not open-source and not present on the machines on which the Linux distributions are built

That is why you talked about dynamic loaded plugins...  Makes sense.
Vendors can build plugins and install them in addition to standard WinPcap modules.

> pcap-snf.c might not work on Wndows;

It works on Windows. I tested recently.

> that might require the third-party software to be available on the machine on which the library is running even if you don't have the third-party hardware;

It is possible to control dynamic library loading and do not require Myricom Sniffer drivers/dlls/hardware to be installed.

> If all versions of MSVC provide some `snprintf()`-compatible function, whether it's called `snprintf` or `_snprintf`, so that we always use it, then, yes, we should define `HAVE_SNPRINTF` in the MSVC project.

This approach will over complicate the code. I believe logic must be following: if `HAVE_SNPRINTF` is defined, `snprintf` _must_ exist. Additionally, you can create `pcap_sprintf` which will be implemented
  * either via `snprintf` as macro (if `HAVE_SNPRINTF` is defiend)
  * or via `_snprintf` as macro (if `_MSC_VER` is defined)
  * or via vsnprintf (if `HAVE_VSNPRINTF` is defined)
  * or some internal implementaiton

And in the code you should use `pcap_snprintf` instead of `snprintf`.

> Are you saying that it doesn't need to be defined for those versions of VS?

Sorry, I do not have those VS version installed to check. But I know for sure, that `snprintf` does not exist in VS2010 if `__STDC__` is defined. To make code clear you can add some kind of table:
* If VS2008, define required macros for VS2008
* If VS2010, define required macros for VS2010
* if VS2012, define another macros for VS2012
* etc

MS change things all of the time and it is hard to predict how did it work before (without testing) and how will it work. I will come up with this table for VS2010 and VS2015 and then you will extend it if required (users will complain in case of issue).

> To quote the commit comment for scanner.c.

In the commit e9f1a5591c3d449ee1a1558fae9a1ada5bd52ba9 `%top` is used. But `cat scanner.c.top  > scanner.c` is different. It just adds `#include <config.h>` to the top of file. This way is similar to what did you have before commit e9f1a5591c3d449ee1a1558fae9a1ada5bd52ba9


Note that this also changes the ordering of the `substring` parameters.

Either order of works but the canonical order is:
`str.substring(indexStart[, indexEnd])`

https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/String/substring


Step 2 : dans le chemin vers le fichier config ./application/third_party/origami/config/bower.php tu as "origami" en trop ;)


Mostly fixed. I think there's an issue with static files config on Heroku. The [admin page](poopfactz.herokuapp.com/admin) isn't in the correct format.


What the purpose of the "!"?


Version: 0.1.0


TeamCity pull requests :: netty [Build 215](https://garage.netty.io/teamcity/viewLog.html?buildId=1430&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 5601, ignored: 21 Build time: 00:36:47


This modification in conjunction with the [latest](https://github.com/matecat/MateCat/commit/4a739a5f39d34256e6a63f26b8110ee60c09b7be#diff-d9e66f5a209f37f869b77d6faede18dfR144) will raise all curl errors ( of **ALL** engines ) up to the client.


@kennethlimcp yes, our customers didn't prefer the link above. the new link gives them the option of getting a version of node for the future. and we are building towards that from the CLI side


Disable and then enable!!!


Enable after disabling!!


This should be `_CONN = None`


@m-kuhn thanks!


Text looks good.  Can you wrap this to < 80 chars per line?


That's what 593 was created to resolve, so I'm very surprised if that is the case.


Sure should.


taskcluster-github: https://tools.taskcluster.net/task-graph-inspector/#undefined


Nit: this line can be moved out of the `switch` since it's used in both cases.


nit: ‘angular’ → ‘Angular’ to match upstream capitalization preferences.


Nice :)


Not to be an asshole or anything, but unless you actually start from scratch, so that your code doesn't have *anything* in common with ide-haskell, you can't just change this copyright notice willy-nilly, by the word of this exact license. You can certainly *add* your own copyright. Just not remove existing one, since then you lose all rights to ide-haskell code.


> In the commit e9f1a55 `%top` is used.

That works only with Flex, not Lex.  As long as we support using traditional Lex, we can't use that.  There are a variety of advantages to requiring Flex, although it does mean that people on at least some commercial UNIXes will have to download and install Flex if they want to build libpcap. Admittedly, they've already downloaded libpcap, so....

> But `cat scanner.c.top > scanner.c` is different.

Yes, unlike `%top`, it works with Lex.


This is weird name. Consider "GetExpressionFromExpressionStatement"


This change requires updating the call site at https://github.com/ansible/ansible/blob/devel/lib/ansible/inventory/dir.py#L112


yea i'm cool with that


When you will merge this?


Maybe replace this paragraph with “Invoke these functions and assign their result to the returned object.”?


What about erase?


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


:fire:


Actually the field we want to uniquely identify the user is `uid`. It will persist across all providers and sessions.


Maybe replace this paragraph with “Assign these directly to the returned object without invoking them.”?  That focuses on what we want people to do, and not what we don't want them to do ;).  I'm also not sure who this comment is for, since we've already done the work of using these functions in the return object.  Can you give more details about where you see things going wrong if we skip this comment-block entirely?


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 640](https://teamcity.labkey.org/viewLog.html?buildId=272580&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 5, passed: 0 Build time: 00:10:14
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSMeasureStoreTest: <no details avaliable>

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

org.labkey.test.Runner: org.labkey.test.tests.CDSTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSLoginTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSSecurityTest: <no details avaliable>

```


My [“who is this for?’][1] question applies to this block too, and the same answer will likely cover both blocks.

[1]: https://github.com/azurestandard/azure-angular-providers/commit/3177ef134588be50d9270760c85837c82d8ead67#commitcomment-13274398


Wait do emojis show up in gitbot comments? :new_moon_with_face:


initial gh pages branch


@salmazov What's the catch you were trying to point out?


```
@HD	VN:1.5	GO:none	SO:coordinate
@SQ	SN:chrM	LN:16571	AS:HG18	UR:/seq/references/Homo_sapiens_assembly18/v0/Homo_sapiens_assembly18.fasta	M5:d2ed829b8a1628d16cbeee88e88e39eb	SP:Homo sapiens
@SQ	SN:chr1	LN:247249719	AS:HG18	UR:/seq/references/Homo_sapiens_assembly18/v0/Homo_sapiens_assembly18.fasta	M5:9ebc6df9496613f373e73396d5b3b6b6	SP:Homo sapiens
@SQ	SN:chr2	LN:242951149	AS:HG18	UR:/seq/references/Homo_sapiens_assembly18/v0/Homo_sapiens_assembly18.fasta	M5:b12c7373e3882120332983be99aeb18d	SP:Homo sapiens
@SQ	SN:chr3	LN:199501827	AS:HG18	UR:/seq/references/Homo_sapiens_assembly18/v0/Homo_sapiens_assembly18.fasta	M5:0e48ed7f305877f66e6fd4addbae2b9a	SP:Homo sapiens
@SQ	SN:chr4	LN:191273063	AS:HG18	UR:/seq/references/Homo_sapiens_assembly18/v0/Homo_sapiens_assembly18.fasta	M5:cf37020337904229dca8401907b626c2	SP:Homo sapiens
@SQ	SN:chr5	LN:180857866	AS:HG18	UR:/seq/references/Homo_sapiens_assembly18/v0/Homo_sapiens_assembly18.fasta	M5:031c851664e31b2c17337fd6f9004858	SP:Homo sapiens
@SQ	SN:chr6	LN:170899992	AS:HG18	UR:/seq/references/Homo_sapiens_assembly18/v0/Homo_sapiens_assembly18.fasta	M5:bfe8005c536131276d448ead33f1b583	SP:Homo sapiens
@SQ	SN:chr7	LN:158821424	AS:HG18	UR:/seq/references/Homo_sapiens_assembly18/v0/Homo_sapiens_assembly18.fasta	M5:74239c5ceee3b28f0038123d958114cb	SP:Homo sapiens
@SQ	SN:chr8	LN:146274826	AS:HG18	UR:/seq/references/Homo_sapiens_assembly18/v0/Homo_sapiens_assembly18.fasta	M5:1eb00fe1ce26ce6701d2cd75c35b5ccb	SP:Homo sapiens
...
```


Remved basically duplicate definition.  The final patch actually remove v3_scts.c so I didn't spend time cleaning that up, just changed it to use the new structure.


Renamed.


Should be $1


It's just `AzureAPI.session.get()`, and it doesn't take an object as an argument.


I think you could just drop `indexForTab` and use `tabs.indexOf(tab)` here.


According to [this](http://developer.android.com/guide/practices/ui_guidelines/widget_design.html#anatomy_determining_size), four cells would be 250dp (but 300dp works fine on my 480x800 emulator).


And this is how we determine auth state as it happens so we can update the `users` list. I actually think users should never be removed just their state: offline/online etc.

https://www.firebase.com/docs/web/guide/user-auth.html#section-monitoring-authentication


This sort moves `chr1` to the first position.  So for reads mapped to `chr1`, the initial `referenceIndex=1`, while the reconstructed one has `referenceIndex=0`.


* Add something in the commit message about the purpose having to do with power.
* While you are at it -- nit: capitalize "d" in "disable" in the commit message

Other than that, LGTM.


commenting out all different tests to add them one by one using macro-trans


4967-SZRH-spm (EnergyArchmage) - x86_64-MacOS-10.9-clang: OK (1859 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-99.25%25-yellow.svg)


Stop


@chobberoni cool. I changed it as many were on 0.12 and 4.0 which doesn't work with CLI currently. This change reintroduces the issue (i know it's stated v0.10.x but people don't read) Thoughts?


You are testing against the same patch...


JOptionPane.showMessageDialog(null, "null, Oscar González (KokinG7)");


I tried a larger crawl and actually found a lot of cases without the `Date` header, so if none of this is found then we will need to skip the record, since no valid WARC-Date could be found. The `nutch.fetch.time` is initialized with the `CrawlDatum.fetchTime` property, on the `HttpResponse` class since now the CCDD only reads the content of the segment and doesn't read any other data. Any suggestion on this?


true. or we can change the symbol name from "vector" to "[]" :)


`AzureAPI.person.get()` doesn't take an object argument, so this should be the the more generic `AzureAPI.person.<action>(...);`.

And the `api/` prefix is just internal to Beehive.  As far as the API spec and this library are concerned, the URL is `<url>/person`, where `<url>` is [this][1].  This `api/` dropping applies to  a number of other comments added in this commit.

[1]: https://github.com/azurestandard/azure-angular-providers/blob/5cc1d1e9c8d5dd6f0bdf0c4fe2d7a9233d41ab1f/azure-providers.js#L49


What's the `max()` for? It's not possible to have 0 tabs is it?


Closes jamoma/JamomaMax#934


You might want to remove that one ;-)


Yep, Mea culpa :smile:  actually I need to remap my live template to use the logger instead.


like-stamp


and again. Apkcrawler is just too fast :-P
//It is not fair!!//


two things:

1. I'm not sure what `referenceIndex` you're referring to? Sorry if it's obvious but I don't see it in code anywhere where this would be an issue.
2. AFAIR from e.g. #794, we don't have good ways to preserve a specific non-lexicographical sort-order of the `@SQ` lines when we ADAM-ify a BAM, but ultimately didn't think that that was essential to have, so I'm curious to better understand why it's giving you trouble here.


Might be worth extracting these two lines into a helper method since it's duplicated several times.


Apparently you are correct.  Will look into and resolve this.  All my what.


Die Prüfung bringt uns eigentlich nichts, was soll die bezwecken?


Probably shift the bracket parenthetical to its own line, after which you can probably drop the parentheses:

    // AzureAPI['purchase-order'].<action>({...});
    // Must use bracket notation due to hyphen in key name.

The semicolon after the action call implies the action call is valid JavaScript (which it is modulo ellipses), and it's nice to keep non-JS off the line.  Using a separate line also keeps us under 80 chars ;).


pass request_method to get_request_kwargs make it possible to adapter instance to make some decision based on request_method, such as set aditional headers.


@shelar .. please note this change. I'm not sure why you were checking for `< 4`.. If my change is not sufficient, please let me know.


Np cheers.


Does this do anything? Looks like the transforms and opacity set by `animate()` will immediately be replaced by the following `completion(true)` call.


these files were created and updated automatically


MiscReleaseFixes (Myoldmopar) - Win64-Windows-7-VisualStudio-12: 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)


Why do you need this?  In this repository's example, I'm just using [remote depenencies][1].  The `bower.json` is just for downstream consumers.

[1]: https://github.com/azurestandard/azure-angular-providers/blob/v0.3.21/example.html#L228-L229


I can't build master after this commit:
```
make[1]: Entering directory '/home/truf/ml/vowpal_wabbit_current/cluster'
/usr/bin/g++ -std=c++0x   -D__extern_always_inline=inline -Wall -pedantic -O3 -fomit-frame-pointer -fno-strict-aliasing  -D_FILE_OFFSET_BITS=64 -DNDEBUG -I /usr/include  -fPIC  -I ../vowpalwabbit -std=c++0x  -c spanning_tree_main.cc -o spanning_tree_main.o
/usr/bin/g++ -std=c++0x   -D__extern_always_inline=inline -Wall -pedantic -O3 -fomit-frame-pointer -fno-strict-aliasing  -D_FILE_OFFSET_BITS=64 -DNDEBUG -I /usr/include  -fPIC  -I ../vowpalwabbit -std=c++0x  -c ../vowpalwabbit/spanning_tree.cc -o ../vowpalwabbit/spanning_tree.o
/usr/bin/g++ -std=c++0x   -D__extern_always_inline=inline -Wall -pedantic -O3 -fomit-frame-pointer -fno-strict-aliasing  -D_FILE_OFFSET_BITS=64 -DNDEBUG -I /usr/include  -fPIC  -I ../vowpalwabbit -std=c++0x  -o spanning_tree spanning_tree_main.o ../vowpalwabbit/spanning_tree.o ../vowpalwabbit/vw_exception.o
../vowpalwabbit/spanning_tree.o: In function `VW::SpanningTree::Start()':
spanning_tree.cc:(.text+0x1af1): undefined reference to `pthread_create'
collect2: error: ld returned 1 exit status
Makefile:26: recipe for target 'spanning_tree' failed
make[1]: *** [spanning_tree] Error 1
make[1]: Leaving directory '/home/truf/ml/vowpal_wabbit_current/cluster'
Makefile:90: recipe for target 'spanning_tree' failed
make: *** [spanning_tree] Error 2
```
Something got broken?


Taking Ian's comment [here](https://openproject.stackato.com/work_packages/301816#note-10) into account.
LGTM


It might be worth during an `rmi` before attempting the pull?


That is correct, see https://openproject.stackato.com/work_packages/302761


Si on execute bower install l'erreur suivante se produit :

bower                         EINVALID Failed to read /var/www/html/myproject/bower.json

Additional error details:
No name property set

Dans bower.json il faut rajouter la prpperty
"name":"myproject"


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)


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)


101280704_lowTempRad (nealkruis) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-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)


Trolol


If we're dropping tabs (which is in line with our [pro-space docs][1]), we should probably do that across this repository in a separate commit.

[1]: https://github.com/azurestandard/azure/blob/master/development-workflow.md#whitespace


Yes, I noticed after I already pushed the branch. Given that this is an abandonded repo for maintenance only (the new version is in https://github.com/hpcloud/logyard-apps-lgr), I don't think leaving them in hurts. I'll remove them if I have to do any other edits.


There are no issues with 0.12, it works perfectly fine.


Not sure. I have seen it fail for people and they had to rollback


I prefer sem-ver's `^3.7.0` to get everything up until the backwards-compatibility break in 4.0.  Unless we have reason to believe Algolia isn't following sem-ver?


We've been 0.12 compatible for several versions now. I even have Travis-CI testing on it.


It Appears you are storing the "Master Password" in plain text. A better implementation to consider would be something from System.Security.Cryptography have a look at this article from MSDN: https://msdn.microsoft.com/en-us/library/92f9ye3s%28v=vs.110%29.aspx storing passwords in plain text defeats the purpose of a password to begin with.

Kudos for adding unit tests!


drat..


Also I don't think such code need for PollStatus. It should end the block either they if it's "true", shouldn't it?


What about the header line:
`Measure:volume   M8710000`
?


And unless that patch release (3.7.7) impacts us directly, it seems better to use a patch-release of zero.  Folks consuming this library should be either auto-bumping and trusting sem-ver, or monitoring the underlying library for minor/patch releases and updating their installed dependency.  We don't need a patch-level opinion here.


You guys mind taking a look at this refactoring to see if you spot any potential problems?

Thanks

@jgautsch @GeekOnCoffee


NM: I remember that the header lines are removed on line 116 now


@louh - On line 275, do you know why this renders in GitHub on the same line as 274, with the js word displaying? It doesn't look any different than 236, which does break properly. Ugh.

![image](https://cloud.githubusercontent.com/assets/4380498/9920241/403a2e64-5c87-11e5-9192-9ad9b73ac698.png)

On the bright side, the line breaks OK on the dev doc site!

both @nvkelso and @kathleenld mentioned this in their reviews.


Huh, I thought we had that, but I guess not.


as per your suggestion,
I'm adding size check to `combineChunk` for server side validation
thanks


Maybe pick a different header here?


I've been trying to follow Python's [import organization][1] with an ordering of:

1. Angular stuff
2. Other external stuff
3. Internal stuff

so probably insert `algolia` before `AzureModelIdentifiers`.

[1]: https://www.python.org/dev/peps/pep-0008/#imports


Done.


Yes, it creates in almost every case a temporary file. See https://github.com/machinekit/machinekit/issues/754


OK cool. Thanks @sasrin


I still don't understand why you used `mine` instead of `index`. Is there any possibility, that non-admin user can see not his Addresses?


Yes, I noticed this, but found the value of 300dp in some examples and worked fine. I think 250dp is the minimum for a 4 cells. Will investigate.


Squash this into the commit that introduced it?  It's orthogonal to adding the Algolia dependency.


Airbnb says var declarations should be on separate lines. https://github.com/airbnb/javascript#13.2


Nope, this is caused by something else, will try to find the problem tomorrow morning. Perhaps I broke something on my pc


not any more, right? It's now space delimited?


why promisify and not use promises (i.e. you're using callbacks with request)


should the description be clear it's about matrix-vector multiplication


ah, I see


I don't think such prolifiration of new settings is a good idea. There are already loads of settings and if we can avoid new `align` settings (which are all exceptions from the rule) we should do that.

Where does this exactly ingnores offset? Only in continuations, everywhere or only in continuations after a call? There are too many overlapping parts and exceptions now.

What about?

```
ddd <-
    condition1 || condition2 ||
        conditon3 || conditon4

condition1 || condition2 ||
    conditon3 || conditon4
```


Could always just mock up a simple `User` class to demonstrate in the example, maybe?

```
<?hh

class User {
   public static function load(int $id): string {
      return str_shuffle('abcdefg') . strval($id);
   }
}
```

or something like that I guess.


Hi Nikolay,

My apologies -- this was a temporary repo I was just playing around with (and which I appear to have forgotten to delete!)

Sorry about that!

Ranjit.

> On Sep 16, 2015, at 3:02 PM, Nikolay Yakimov <notifications@github.com> wrote:
>
> Not to be an asshole or anything, but unless you actually start from scratch, so that your code doesn't have anything in common with ide-haskell, you can't just change this copyright notice willy-nilly, by the word of this exact license. You can certainly add your own copyright. Just not remove existing one, since then you lose all rights to ide-haskell code.
>
> —
> Reply to this email directly or view it on GitHub.
>


I'm going to remove this function from this commit.  Let's revisit later once we have the code that calls it ready for review.


102497980-Improve-waste-heat-function-handling-for-CoilCoolingDXMultiSpeed-and-CoilHeatingDXMultiSpeed-#5162 (mjwitte) - x86_64-MacOS-10.9-clang: OK (1865 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-99.63%25-yellow.svg)


:+1: to not putting `active` in `default_scope`


do you want these two function names to match? :-)


Probably use the unambiguous `algolia_application_id` for the variable name too (or `algoliaApplicationID`, depending on how our camelCase discussion works out).  That will let us add additional application keys for other services (if we end up needing them) without creating confusion about which app ID we're using.  Follow the same pattern for `api_key`.  And since we only use these variables for the `algolia.Client` instantiation, a longer variable name isn't going to result in much more typing.


It was "specced" in https://openproject.stackato.com/work_packages/300360 but never implemented. This task is still open for Metallica, but I don't think it will make it in. Definitely not for the September deadline...


Oh! Heh. Promisfiying it is indeed useless -- my first attempt went that way and I left it in place (since it leaves it working both ways)

Turns out it was hard to get right without more invasive change since there are two arguments to the callback.


Now I seeThis is supposed to contain an error. Maybe we should rename the fixture files according to their use, rather than just `1` `2` or `3`.


OMG! Thanks for catching that.


I assume this is not needed for Flink?


+1 looking forward to it!


If the intention is to save to disk, I'd suggest to use GUID as the name. Otherwise, on the same machine, the different UT runs may conflict with each other. (For example, the save failed due to new bug, but the load may still succeed because of the old data left from previous run, thus hiding the bug.).


I'd call this variable `resource`, since that's what it is.  `rtn` (return?) is what you're going to do to it later on ;).


According to Google the rule is `70 × n − 30`, where `n` is the number of cells.


Probably skip the two-step approach and just do:

    resource.index = algolia…;


Nice! Thanks for the contrib.

There are two comments to be addressed: single-line var declarations, and possibly renaming fixture files so that their use is clear.


Why not set `ess-align-continuations-in-calls` to align all multi-line epxressions in calls and be done with it? I think this would be a consistent and simple rule.


Ah, I see you drop it in ff44b026.  Probably just squash that drop in here.


Since we are using the random data, maybe we should generate the rows on node directly instead of distribute it.  We can control how the random data is generated, thus to make sure the local matrix have the same values as the distributed matrix.


what does spread do again?


OK, it's better to match the actual symbol in the language.


Acredito que a variável hora seja desnecessária. Logo mais, irei publicar uma modificação.


Hi, I see this is a html5 page style, I would recommend using Angular, and forget about explicit depandencies with jquery, Angular will manage that.


I'd leave that whitespace alone here.  Once we get a style-checker, we can just make whatever whitespace cleanups it suggests.


removed because this field is now in the base class


the 'error' field is deprecated.  Seems unsafe to rely on it alone.  Checking both seems safer.


reordered fields to match the Go code


I don't think we need a `TransactionScope`, there seems to only have one resource, is this a mistake ?


Since promises can only pass a single value, this is a way to pass two -- it's an array inside the promise, but it `.apply()`s it to your callback so you get them as separate parameters.


these fields are int64 in the Go code


This is an int in the Go code


i was thinking the same. i dont know why i just didnt do it. i knew it'd come up


[Spearman–Brown correction](https://en.wikipedia.org/wiki/Spearman%E2%80%93Brown_prediction_formula)


Sí, estoy al tanto de ese detalle.
Lo puse como una forma más evidente de decir CERRÁ TODO


hmm. i wonder why jscs didnt complain


You're right. It was a mistake to display this as ESLint errors. I replaced it by a simple `console.log`, so it should solve the problem while keeping the warning.

Related issue #220


I am thrilled to see this released, and can’t wait until there is a CIDER version that takes advantage of it! Many thanks. :grinning:


> if we can avoid new align settings (which are all exceptions from the rule) we should do that.

I felt the discussion with @mmaechler showed that we can't avoid it here.

> Where does this exactly ingnores offset?

Everywhere. Your example produces this:
```r
ddd <-
    condition1 || condition2 ||
    condition3 || conditon4
```
If I understood correctly, it's what Martin wants.

> Why not set ess-align-continuations-in-calls to align all multi-line epxressions in calls and be done with it?

personally I prefer to allow flexibility where this does not complicate the code too much. Emacs is about flexibility. And here it's fine regarding code complexity.


Methods should start with lowercase (`validatedSurveyTaskWithIdentifier`). ;-)


`ObjectPromiseCache` is completely ignorant of the sort of of model it's caching, and I'd like to avoid abstraction-leaking if possible.  Can we rename `index` to `algolia` and use JavaScript's version of `hasattr(AzureAPI[this.model], 'algolia')` for this if-switch?


TeamCity pull requests :: netty [Build 216](https://garage.netty.io/teamcity/viewLog.html?buildId=1431&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 5601, ignored: 21 Build time: 01:17:12


This is now automatically detected by https://github.com/mer-hybris/droid-hal-device/commit/0a09a40bd00e81b01b0c06ab1bccd8b6e2960efa


Done!


No longer needed due to https://github.com/mer-hybris/droid-hal-device/commit/5c8d19fcaf747d57b0be4efdec9955dd84e74947


@bgoodri We might want to consider adding an additional `prior_scale_for_dispersion` (called something else) because `prior_scale_for_dispersion` has to do double-duty here. There’s already a dispersion parameter `theta` in `count.stan` and we now also have `scaled_tau` for the group stuff. Right now both use `prior_scale_for_dispersion`, which is not ideal I think.


maybe `'("|" "&" "!=" "==")` is too much though. Do we only want `|` and `&` here?


Or `algoliaIndex` or whatever to make it both clear what that attribute holds (an `algolia_client.initIndex`) and unlikely that someone puts something else in the same spot (e.g. a non-Algolia index).


Yeah. It gets very confusing depending on what app you use. Sublime Text doesn't have RTL support. Try the form in Collect.


It's a pattern used in several other routes, but I'm not sure why. A non-admin user should always be able to see their AddressBook


What is this apt-cache thing?


Hay que reiniciar después de instalar un kernel nuevo, sé que hay formas de evitarlo incluso. Ahora con todo el lío de systemd parece que también hay que reiniciar para actualizar eso.

https://serverfault.com/questions/15782/how-often-should-i-reboot-linux-servers


This seems independent of the shift to Algolia, and needs to land with a matching Beehive PR that updates `apps.api.views.price` and callers to return this multi-price-level form from the non-Algolia endpoints.


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


> Why not set ess-align-continuations-in-calls to align all multi-line epxressions in calls and be done with it?

That would really bug someone using pipes. It's not uncommon to use pipes inside dplyr's `mutate()` for example. And maybe that person still wants the override for other things like logical operators or operations within parentheses.


Blimey should be banned from editing code at midnight after a long day at work! Corrected.


A variável "minutemhora" não foi declarada, mas acredito que seja desnecessário, assim como a variável "hora". Logo mais, estarei resolvendo isso.


Huh, yes, yes I do.


hybris-12.1 branch doesn't exist in that repo


Excelente, pronto podremos compartir experiencias entonces. Me avisas por favor, porque no hay notificaciones para cuando se pushea una rama :P


Just checking - this regex may be a little too restrictive for the naming of a Virtual Network?


This needs to happen in order to use the data from Algolia.


This commit looks good to me, but I'd leave out the `packaged_products` name until you have code that actually uses that index (possibly in a later commit in this PR).


So the helper in PSTAlertController+Selection.swift wasn't gonna work in this case?


This file contains the most interesting changes.


this one do not work.

```cpp
 0:46:43 "ExileClient - Incoming message: ["updateStatsRequest",["906749","840229",0,11]]"
 0:46:43 "Dispatching message 'updateStatsRequest'..."
 0:46:45 Error in expression <h{};
_pos = getPosATL _unit;
if(!isNull _nObject)then
{
_weaponsItemsCargo = wea>
 0:46:45   Error position: <_nObject)then
{
_weaponsItemsCargo = wea>
 0:46:45   Error Undefined variable in expression: _nobject
 0:46:45 File mpmissions\__CUR_MP.Altis\IAT\exile_overwrites\exileclient_object_player_event_onkilled.sqf, line 22

````


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


@dtolnay wrote:
> The new "may need parentheses" message shows up if anything between { and :<expression> fails to parse.

This would seem to confirm that the second error message is just an elaboration of the first one ("unexpected ':'").  That is, it seems that jq has really only found one error in this case.  (If there are other such cases, as you seem to believe, then that would in my view just confirm that it would be worthwhile addressing the problem head-on.)

Please note that I have no objection to one error generating more than one message.  I just don't think there's anything bizarre or even unusual about having an error counter that counts distinct errors.  But it's true there are more important issues to address (in jq and the world at large), so by all means, please ignore this posting.  Let me take this opportunity, though, to thank you for your contributions.


See [my][1] [previous][2] [comments][3] about the product resource version of this code.

[1]: https://github.com/azurestandard/azure-angular-providers/commit/adfec2c3460d300dda158ed7656feb8adaf5f048#commitcomment-13274984
[2]: https://github.com/azurestandard/azure-angular-providers/commit/adfec2c3460d300dda158ed7656feb8adaf5f048#commitcomment-13274993
[3]: https://github.com/azurestandard/azure-angular-providers/commit/ff44b026bc0393e6e3d8c0166fa3a501922ed74b#commitcomment-13275147


This is needed. The generators are not (and I don't think should) import pushstate at the moment.


@georgistanev:

Thank you!


Good point. Will fix and upload.


Thanks, I've cherry-picked the commit on my `decoration` branch, as 4a9b7f1816533db4df9c738e056fd9d214136571.


Agreed, but it also breaks the ability to use the existing API endpoint (which returns a different price structure).  I'm suggesting the following upgrade path:

1. Land an API (probably Beehive and api-spec) update to switch to this more detailed price structure.
2. Land this commit to adjust the library to that newer price structure.
3. Land this PR with the rest of the Algolia-specific stuff.


This is a unit test for just the controller. Still need to create one for the chart directive and new buildings_reports service.


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


sorry i dot use any of this three,


The models are ‘product’ and ‘packaged-product’.


Backticks around `.index` (or [whatever we end up calling it][1]).

[1]: https://github.com/azurestandard/azure-angular-providers/commit/ff44b026bc0393e6e3d8c0166fa3a501922ed74b#commitcomment-13275147


nope.


Seems orthogonal to this commit ;).


Hah yep, but you could of changed that if you wanted.


That English '-_-


@TallJimbo I think I'd prefer the `eupspkg.cfg.sh` patch to be done using the (more standard?) approach of a patch in the `patches` directory. Otherwise it's all too easy to not realise what is going on when the package is updated in the future.


Looks good to me (modulo some Algolia comments that are orthogonal to the refactoring here).  The bulk of this should be squashed into your original refactoring commit, since the removals will make that commit easier to read.


That would be nice, but I couldn't find a convenient way to do it. DSet.init and DSet.initS have complex signatures, including partitionSize, that I don't want to pollute the intent here.


Squash this whitespace fix into the commit that introduced it (c91af33).


Oops, sorry I missed that issue. Thanks.


> I felt the discussion with @mmaechler showed that we can't avoid it here.

We didn't even started a discussion actually.

> If I understood correctly, it's what Martin wants.

Martin wasn't talking about this case at all. His main concern was the indentation within calls. And I think he would agree that the following indentatioin is also ugly:

```
ddd <- some_call(var1 + var2 +
                     var3 + var4)
```

> maybe '("|" "&" "!=" "==") is too much though. Do we only want | and & here?

Either all or none. We don't want long expressions to be indented differently depending on which one breaks the line.

> It's not uncommon to use pipes inside dplyr's mutate() for example.

Well. that's very much debatable. I personally disagree.

```
ddd <- some_call(expr1 %>%
                 expr3 %>%
                 expr4)

## looks better than

ddd <- some_call(var1 %>%
                     var3 %>%
                     expr4)

```

In any case, having a special treatment for pipes would make more sense than having special treatment for logicals. It's just confusing to have different indentation for multiline expressions depending on the operator that ends the line. This shouldn't be the default.


oh! right!!

niiiiiice. :-D


just curious, shouldn't `fread` unlink file at the end?


How about use Source, or SourceI?


Sorry for the reformatting, was not really sure why it was formatted differently than before, so did a full reformat, but that actually changed all the lines.
Mostly the code in the gameObjects package has been changed. The most notable class changes are: GameLoop got split into 2 classes: GameObjects has been created, AbstractDynamicObjects class has been created, Projectile class got code from the Line class instead of extending it and all static variables which first belonged to GameLoop are now not static anymore and belong to GameObjects.


Oh oops I misinterpreted.

I have no idea who to allocate blocks of memory on C#. Shit.


These README changes should land in the commit that adds the Algolia-index attribute to the listed model (e.g. document `product` in adfec2c34, packaged-product in f01712fbc1, …).


@ykhodorkovskiy This gets us rebooting/shutting down in Enforcing.


Looks good to me


i think flax would be better


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


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


Just prints out the available versions. IMO, this is the most likely error if one is defining `$DOCKER_VERSION`


typo on canola


@ykhodorkovskiy Just need to get this into the refpolicy tree and get it reviewed and then we should be ready to go.


See #1


Let's try Measurement and MeasuredValue, perhaps.


I mean the `c[r] == c[g] && c[g] == c[b] && c[r] && c[r]` line, because an int* can't be treated as an array of any sort.

So I guess I'd either make a struct for just change the parameter to an int[].


Fixed.


Lionel, really. It just doesn't make sense to treat logicals differently than other operations. In R logical variables are automatically promoted to numericals whenever needed and the other way around.

As I said before, there are too  many rules and exceptions already and you are the only one who keeps track of it all. Let's make it consistent and simple for the user. No one will be spending hours understanding our indentation system. We came from a cluttered system and we are on the point of getting into clutter again.


It seems
```
.PHONY: help
```
is missing.


Yeah, that's a cut & paste error. I thought I had removed it  - Ill fix


Error: Validation failed against schema: http://schemas.taskcluster.net/github/v1/taskcluster-github-config.json#[object Object]


He has coding OCD.


Yeah, I'll fix the user API side of this and upload, but leave changing the test start triggers paradigm for another day.


> It's just confusing to have different indentation for multiline expressions depending on the operator that ends the line.

I think it's more confusing to have the indentation depend on the kind of calls, and the fact that it should be in a call to begin with. I think the new (though temporary until we decide on something) override is much more predictable. It seems that what is confusing or not depends on the person here.

> Martin wasn't talking about this case at all.

Yes I read between the lines. Maybe erroneously?

> having a special treatment for pipes would make more sense than having special treatment for logicals

But that special treatment would be an exception for an exception. That's not ideal. It's better to have two kinds of clearly defined overrides.

> I think he would agree that the following indentatioin is also ugly:

You can set the calls override to `.*(` to cancel indentation inside all calls.


a pyx file depends on all used pxds though, so ignoring them will lead to not-triggering a rebuild when the pxd was changed, or am i wrong?


This affects e.g. the list of reviewers on the Editor's workflow:
![style](https://cloud.githubusercontent.com/assets/200411/9921137/56239a28-5c8f-11e5-8b89-b87435f86c7b.png)


Yep, good point.


Esta on com uma qualidade excelente.


TaskCluster, Error: Validation failed against schema: http://schemas.taskcluster.net/github/v1/taskcluster-github-config.json# {"validation":{"version":{"enum":true}}}


#11 still applies


TaskCluster, Error: Validation failed against schema: http://schemas.taskcluster.net/github/v1/taskcluster-github-config.json# {"validation":{"metadata":{"schema":{"source":{"required":true}}},"version":{"enum":true}}}


Ok Boss)))


GHX_Vert_IDD (mmatts) - i386-Windows-7-VisualStudio-12: 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)


@carlbrugger No, I think master is fine. I was thinking we could put an `isAdd()` method in the pattern and just call it in all the subclasses, but this works too.


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?


Hi.

Ive managed to compile a 5.1.1. ROM with hush commits but i have a series of problems when it boots up (softreboots, no signal, batery gauge says 0%,etc)

I would like to see if we can help each other out and get this to work for lollipop


True, I'm actually thinking about moving the contents into test_record, since it's likely the only "custom" part remaining will be the test record oriented handler. I think pull request #23 by amyxchen is the right place to figure this out.


Yes, the parameters are different.
With 0 0 0 0, this is same as "master"


Will do.


@radar any chance we can get another version bump to fix this?


I don't believe this is the correct logic.

The first check `!$(a).hasClass($.DirtyForms.dirtyClass)` does not filter out non-dirty fields, it simply does a check to see if any of them are dirty. If none of them are dirty, it returns `false`, which is correct.

The second check `$(a).not(':dirtyignored').length > 0` simply returns `true` if any fields are not ignored. You are returning `true` if any of the fields matching the selector are not ignored without checking whether they are dirty. There still could be non-dirty fields in that batch that are ignored.


Thanks! It definitely took some doing.


GroundDomainUpdate (mmatts) - x86_64-MacOS-10.9-clang: OK (1865 of 1874 tests passed)

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


TaskCluster, Error: Validation failed against schema: http://schemas.taskcluster.net/github/v1/taskcluster-github-config.json# {
  "metadata": {
    "schema": {
      "source": {
        "required": true
      }
    }
  },
  "version": {
    "enum": true
  }
}


This will lead to having hundreds of cmake targets some day. Converting all pxds at once avoids that.
I'd of course like having idependent conversions, but i fear that will clutter the output of each single make as cmake says "yes i build target $bla".


It's not so simple. Things can run in parallel, so the values generated by the random would be different. I tried DSet.cacheToMemory, which would be ideal, but apparently has different semantics than I expected (i.e.: it's not an "eval-and-cache").
I am not happy with the saveToHDD thing either, but it's the only I could find that doesn't distract from the purpose of the sample.
The way to do this "parallel, random deterministic" thing is to generate as many seeds as there are partitions and start a new random for each partition, with the first fist random's values as a seeds. But that's a pain. A little less of a pain, but still a pain, would be to use something deterministic, like hash(node) ^^^ hash(partition), and BitConverter to generate values. Here, any of this distracts from the purpose of the sample.


https://github.com/USBhost/FIRE-ICE/commit/008e021ead9ee8a0d90f0b6d606c863722aa483a
this should help you out!
also take a look here https://git.kernel.org/cgit/linux/kernel/git/jaegeuk/f2fs.git/log/?h=linux-3.10


r+


Fixed.


saw approval from jedmccaleb
at https://github.com/MonsieurNicolas/stellar-core/commit/68a8f1ff1d0551d4a2fa90d4c199c907f3bf12c9


merging MonsieurNicolas/stellar-core/ledgerEntryFixes = 68a8f1ff into auto


MonsieurNicolas/stellar-core/ledgerEntryFixes = 68a8f1ff merged ok, testing candidate = 415b36ba


GHX_Vert_IDD (mmatts) - Win64-Windows-7-VisualStudio-12: 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)


¡Cuidado! Es "beginner", no "begginer"


TaskCluster, Error: Validation failed against schema: http://schemas.taskcluster.net/github/v1/taskcluster-github-config.json# ```{
    "validation": {
        "metadata": {
            "schema": {
                "source": {
                    "required": true
                }
            }
        },
        "version": {
            "enum": true
        }
    }
}```


I addressed the comments. Please look at the pull request now.


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


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


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


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


> In R logical variables are automatically promoted to numericals whenever needed and the other way around.

hmm I don't see the relevance. When you use a logical operator, that defines the semantic meaning of the variable no matter the underlying representation.

In the previous engine someone wrote specific behaviour for logical operators so it made sense to him as well.

> We came from a cluttered system and we are on the point of getting into clutter again.

That's not the same kind of clutter at all... If you read the code you'll see I'm trying my best to keep it organised and manageable. Though I agree certain part are much messier than I'd like :/ But here it's a simple branching, not a big deal.

> No one will be spending hours understanding our indentation system.

I don't know, when I use a mode I often look for obscure settings. When I can't find any for I need, I setup advices. But indeed the default styles will be good enough for most people.

Let's see what Martin thinks... Maybe he also prefers to have the override for logical operators working at top level and not only inside a call.


> You can set the calls override to .*( to cancel indentation inside all calls.

Yes. And this is what I want to be the default. No exceptions for if, (, [, pipes etc. Simple rule which everyone can understand - align inside calls, offset outside calls.


Hi m,
How about is it?


Hello! I am interested, do you remember me? I'm the one who added the ability to split extended classes through different javascript files. It has been proven to be really useful, I'd gladly maintain the repo for you if you wish. Let's chat!


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


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


I agree that this optimization is necessary. But please change the last line to be the same as line 151 was previously so it filters out the non-dirty fields.


See [here](https://github.com/aurelia/event-aggregator/pull/13) for a complete port.


TaskCluster, Error: Request payload must follow the schema: http://schemas.taskcluster.net/scheduler/v1/task-graph.json# ```undefined```


Should use CORE_DIR.


Please file an issue about it. Comments on commits are very hard to track.


`return $validate->verify($action);`


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


why hide :)


NinePatchMaskTransformation getId() is wrong, you can't remove the width and size, if not, when loading  many images with different size, their cache will confusing. Thanks!


since this is boolean we can just omit the attribute


dd


This doesn't seem right, word wrap has been set to 9999 instead of the default of 80?


@PEMapModder It's a good idea, but I would prefer it on a separate branch.


TaskCluster, Request payload must follow the schema: http://schemas.taskcluster.net/scheduler/v1/task-graph.json# ```undefined```


You are correct. I don't have anything against doing so, but I have no reason to do so, either.


See [here](https://github.com/aurelia/event-aggregator/pull/13) for a complete port of event-aggregator to TypeScript.


What?


See the related azurestandard/api-spec#29, which gives us enough information to pick the right price-level.  `retail` isn't always right, and we'll need a path for getting the person-specific information proposed by that PR down into this area so we can pick the right price.  The path we choose should be flexible enough to handle the future pricing model, which isn't very clear at the moment.  I'd probably just pass down a callback that takes the product and packaged-product as arguments.


Ta errado, faiz de novo....


ok I do agree that with the new operator override, it doesn't make sense to have a flexible call override. So I'll change it to a boolean. I didn't like the unpredictability of the former system so you're right, let's have it totally predictable.

As for the operator override, let's see tomorrow what Martin thinks about it, e.g., does he want the logical continuations aligned at top level.

Personally I don't use these overrides at all, I just tried to do the right thing.


ta ok agora...


This is the right version.


Huh, no idea when that happened.


So, @jenspinney and I explicitly did not use ReadFull so we could introduce failure cases and to avoid hanging if a custom prng was used that stalled. Further, rand.Reader already does it under the covers for the default path.


Forgot to add to the commit message, but this also fixes a (minor) bug where it used 60fps instead of 59.94 target frame rate for NTSC-progressive.

Now it uses 59.94 by default also for NTSC Progressive, and also respects the custom NTSC rate in NTSC Progressive mode.


This is a bit of an assumption.  I don't know for sure if this will work for other types of secure (passwordless) connections, but I don't we're losing any functionality.


TaskCluster, Request payload must follow the schema: http://schemas.taskcluster.net/scheduler/v1/task-graph.json# ```{
    "message": "Request payload must follow the schema: http://schemas.taskcluster.net/scheduler/v1/task-graph.json#",
    "error": {
        "validation": {
            "tasks": {
                "schema": {
                    "0": {
                        "schema": {
                            "task": {
                                "schema": {
                                    "metadata": {
                                        "schema": {
                                            "name": {
                                                "required": true
                                            }
                                        }
                                    }
                                }
                            }
                        }
                    }
                }
            }
        }
    }
}```


LGTM :+1:


> hmm I don't see the relevance

The relevance is that if you write an expression with `+` and `|` in it, you should not re-indent the following code depending on whether you ended with + or |.

> If you read the code you'll see I'm trying my best to keep it organised and manageable.

Sorry. I didn't mean your code at all. It's super clean. I meant the UI. You are very biased because you wrote it. I have to go to the docs each time I start a new issue. It's already too much configuration even for me, and I have been tracking it from the very start.

> I don't know, when I use a mode I often look for obscure settings.

Let's not waste users' time. Indentation should be consistent across programmers. We shouldn't enforce mambo-styles, emacs it or not. Especially in this case when it's not at all necessary.

As far as I am concerned `ess-align-continuations-in-calls` can even be boolean now for simplicity.


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


Looks like you don't need this class here?


nice change dude!


Je viens de voir tout le boulot, merci mais pense à nous en laisser xD Tu as pratiquement finis mdr


TaskCluster, Request payload must follow the schema: http://schemas.taskcluster.net/scheduler/v1/task-graph.json# ```{
    "validation": {
        "tasks": {
            "schema": {
                "0": {
                    "schema": {
                        "task": {
                            "schema": {
                                "metadata": {
                                    "schema": {
                                        "name": {
                                            "required": true
                                        }
                                    }
                                }
                            }
                        }
                    }
                }
            }
        }
    }
}```


I added this because some of the SQL scripts were failing badly, but still returning ok return values, so weren't getting trapped by sqitch.  This happened in particular during the upgrade command which failed to connect (thus not creating the releases table), but continued trying to register the release.


thanks :+1:  :)


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

No need to write ```self``` here


Whitespace


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


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


That was supposed to be "folder", not "table", derp.


Make this a Swift array instead of ```NSArray```


"self"


fast-forwarding master to auto = 415b36ba


Gracias!


If we ever add `plugin_idle_add_full()` to provide a deleter function for the container/data, it would be nice here to get rid of the _extremely_ unlikely case where the plugin is somehow unloaded before the idle handler is run and container is leaked.


Then why not make session a $http.get rather than a full $resource?


Getting current assignment first then based on the current assignment, we can propose a new rebalanced plan.


This will be used later


top, aber kannst du mal genau erkläre, wie sich der ERP Code dann zusammen setzt? Ich glaube nämlich, dass man den 2. OERP Aufruf (unten) nicht braucht, und dass du einiges recht kompliziert gemacht hast. Ich push mal einen 2. branch mit meinem Beta merge Vorschlag. Derweil bereite ich nen weiteren Branch zum mergen unserer beiden "merge request branches" vor ;)

```python
last_id = max(oerp.search('product.product'))
```


Yes, HAVE_SYS_UID_H is defined on FreeBSD so at least the check for
__FreeBSD__ is superfluous. I guess the same applies to DragonFly but I
don't know for sure. Some of my earlier tests had indicated that
HAVE_SYS_UIO_H was not defined but if I remove the checks the compile
warnings do not come up and sendfile works correctly.

JP

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

> Why have you added "|| defined(*FreeBSD*) || defined(*DragonFly*)" here?
> Isn't HAVE_SYS_UIO_H set on freebsd even if it is present?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/theom/otp/commit/043c49a6951add5bf7b05cda643c8b4874849e5a#commitcomment-13255334>
> .
>


@cpuguy83 does this look correct?


Shouldn't this be `httpHeaders.toString()` instead?


Fix #12


En la línea 47 sería mejor utilizar el operador OR, ya que sólo en dos casos debe retornar true. Otra sugerencia es usar una variable para el cálculo realizado, ejemplo: int delta


Is there any reason pg_dump wouldn't work? I imagine both primary and secondary master servers would have an identical database. Would it be easier to dump/restore using the pg_dump/psql commands, perhaps via easy buttons on the maintenance GUI?

CSV files might not be difficult to implement, but it would still be time spent that we could spend elsewhere. Besides you mentioned last year it had some issues anyway.


Una sugerencia extra es agregar el test negativo


This works better, but now the textarea displays scroll bars as well as the window itself. Is it possible that the textarea rendering and the data loading are happening in some weird order where extjs doesn't know to grow the textfield with scrollbars on? As soon as I add `preventScrollbars` to the textfield, it goes back to how it was before your fix.

It's also a little weird that the size of the textarea (before scrolling) is different from the size of the window itself, so you first need to scroll the textarea all the way down, then the window, to see the full text. I feel like extjs doesn't know tor resize the textarea after the data loads somehow? I'm admittedly quite new to extjs.


I just updated the chocolatey package and emailed the moderation team about getting this latest update approved. I agree with InfernoZeus that the ping would be great for future versions


⊙﹏⊙b汗


你好，应该在plugins中增加"yahei"而不是"gitbook-plugin-yahei"，否则上传后gitbook会build error：https://registry.npmjs.org/gitbook-plugin-gitbook-plugin-yahei


You need to be careful here with the code running after this. You might get nulls for things you didn't expect.


@andreoav  @shield tags that I can recognize on the local, but on the server can not identify what is the reason


Not sure why you changed that from frozenset(c), but ok.  For reference, I specifically did set(cache) so that it avoided an intermediate list.


感觉,  问一下你是否愿意加入项目的 thanks 列表


...this couldn't possibly work...


能不能改成左右两栏，一边输入一边预览呢？


可以啊。要不你来改。


This makes no sense.


我到现在还不知道你这个怎么启动起来的。。。不过解析markdown 加缪已经教过我了


This is the fix for system logs and application logs now shown on management console. (Refer images in https://wso2.org/jira/browse/ML-194)


spelling typo: prepre => prepare


No, I left one in the last time I went through a review.  Keep forgetting I'm using them.


Gets my +1. I don't recall seeing this option when I was first setting that stuff up :(


I don't know why the current code can make multiple requests queued-up on a single socket. The idea here is that the socket is free (sent and then received the response from server) before it can send new request. The code in line 569 is designed to do that.
The case that you mentioned is an unexpected behavior, the request should be sent immediately without waiting for anything.


Please don't push to amberdb without a pull request.

I don't mind so much with banjo.


lol


haahahaha


pra ser sincero, nao sei se faz alguma diferença quanto à performance, eu só prefiro deixar valores default declarados ja na variavel.


updateMessage should belong in a message class, but :+1:


Where is type defined? I didn't see this parameter in the page: http://maven.apache.org/plugins/maven-dependency-plugin/get-mojo.html


А зачем этот костыль? Чем LRUCache не устраивает?


> [If] we want to assign IDs early and leave them fixed for the duration we could consider it. This would even have advantage in some of the attempts to have some piece parts up early as we could do informal audits of which teams had tested their code or not.

> I think we should allocate these early and have them use even before the competition.

With 4-5 digit team IDs, we have plenty of rounds available before exhausting the pool, so we could do a mix for both. We could assign one set of team IDs beforehand and then assign a new set the morning of the competition. For instance, we can reserve all team IDs beginning with 9 to be "test" team IDs for use prior. So Team Awesome could be allocated team ID 9876-0 starting today, and then on the last day be handed team ID 5432-4.

> I agree a check digit is a nice idea.

I was originally thinking of a simple modulo 10 addition of the digits as in the above examples.

> I already designed an ID code generator for this. It assigns a 4-character code consisting of 2 letters and a 2-digit check.

Two letters means we get 676 possibilities for unique team IDs. If we had only four of the original five digits, we'd get 10,000 possibilities.

How about increasing to five-characters again--three letters and keep the two-digit check? This gives us over 17,500 possibilities.

How are you generating the two-digit check?

(I was thinking of having four letters, but by then we'd have to pass it through some sort of exclusion filter to make sure we don't get certain inappropriate four-letter team IDs. By then, reading at the source code or config for the exclusion filter would be just as amusing as getting the team ID generated anyway.)


Si si, é mais pelo costume/gosto, concordo.


Decide length of dataPayload (e.g. '21.5' is 5 characters {'2', '1', '.', '5', 'NULL'}, so this code won't work for triple digit temperatures recorded.


no comment


comment


@Funmungus I had noticed that inputing without libreadline has a weird issue, so I had compiled scanmem for Android with libreadline. I also have scripts to help the compiling but haven't pushed to github yet. Perhaps I will publish them this weekend. I prefer to do the work on my own fork and I am not going to merge my fork to the upstream. But maybe I can provide some helps for you If you want.


could be more initiative


101280704_lowTempRad (nealkruis) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1340 of 1340 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.25%25-green.svg)


fixed in c7700d6f356d60219342d6c853fb4a7673ed58af


As for rearranging  the logic. the two lines are extra.
the code at line 127 has same effect.


I think you want to use `git cherry-pick -x` so that there's a note referencing the original commit.


fixed in 4b0f1fc


IMO, the sentence could be change to the following one which keeps pace with the description in Azure Portal.
The access type of the container `stemcell` in the default storage account is set to `Blob` to allow public read access for blobs.


fixed in ccc29f6


Hello


@shreyanirmalan: I think you did a ```git checkout -b total-items``` from your ```changeColor``` branch instead of from the ```master``` branch.

I think it's better to start a new feature from the ```master``` branch, unless the new feature is related to one that has not been merged into ```master```.


For things that came from a PR I also do `-e` and add "ref #..." so github cross-links


fixed in dd66b5d


jkjhkjh


@bernardobrezende
Dei uma ajeitada na questão da replicação, não sei se é esta a solução esperada.

Quanto à inventar regras, me empolguei mesmo com o código.


Workarounds for the static icon-paths.


lkjljlkj


didn't know about -x. Doing it.


Since the `REQUIRE` file lists `julia 0.3` as the prerequisite, we can safely drop this code.


Oh, I see, I will definitely remember that for the future!


mmnnm


Thanks 4 replying....any example...how define acl n how to allow that....
On 15 Sep 2015 21:29, "Tully Foote" <notifications@github.com> wrote:

> You can put whatever content you want in the SQUID_DIRECTIVE including
> acls.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jpetazzo/squid-in-a-can/commit/837f744141530391c8fce03944f248ee356790e4#commitcomment-13242000>
> .
>


Нужно перемешать все картинки разом, а не по папке


We should probably have a comment here about why we use 22 as the length. Not sure if we should push some of these numbers to the top as it's very specific to the graph isn't it....


Once these are removed you can delete them - that's fine.


Nice work!


アカウント名が美味しそうだからだめ


The socket has received the response from the server, whether the response is empty or not, we should always set it to free for sending new request. I think we should keep the code unchanged here.
Currently, we use a asynchronous socket as a synchronous HTTP socket. The pending_ flag use here is to keep this socket in synced. pending_ = true means that the socket has sent the data, and is waiting for the server to response, pending_ = false means that the socket has received the response from the server and waiting for new request to send.


Delete extraneous spaces between last line of code and closing curly bracket. Refer to coding convention.


what is this


It's not building. It's missing some dependencies, for instance sample-boolean needs flow/keyboard.mod


just for debugging


mvn should be a command, must must be a executable file.


just copied from stackoverflow, details should be studied in the future.


merged from branch to master


hi, **service node pools** is a xCAT term， the following words are the explanation on it. I will corrected the format for it, see 3bd118e


xCAT doc page may have 4 levels of title at most:

The First Title
===============

The Second Title
----------------

The Third Title
```````````````

The Forth Title
'''''''''''''''


#4402-VRF-outputs-and-multiplied-floor (rraustad) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


this section describes an approach on how to keep some file persistent in diskless mode,it's node on xCAT diskless mode, I think it should be kept.


I'm sort of having a problem understanding how much abstraction this is supposed to provide, and also why any at all. Basically, I end up wanting to do

```lua
function go.get(...)
	exec("go get", ...)
end
```
for pretty much everything.


+On SLES, it is called a pattern.
If we need include different OS, the format is as following:
The valid keyword includes: RHEL, SLES, UBUNTU, CENTOS, X86_64, PPC64, PPC64LE. If the keyword is an OS, it can be postfixed with an OS version e.g. RHEL7.
* **[RHEL7]**

  This part of description is for [rh7] specific.


+    * Add both the directory and the file to the osimage definition: :: =====> this line should not be included in /install/custom/netboot/rh/compute.otherpkgs.pkglist


In all those debug level logging, could we log the principal, the host, the resource and the requested operation.


101280704_lowTempRad (nealkruis) - Win64-Windows-7-VisualStudio-12: OK (1859 of 1874 tests passed)

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


perfect


It's not clear from the logging here whether the operation is allowed or denied. Could we log this in the caller?


Currently, we should not recommend to  execute  "createrepo" under  otherpkgdir /install/post/otherpkgs/rh/<arch>, we should recommend them to create sub-directories under it.


```js
if (!isStream(stream)) {
  throw new TypeError('Expected a stream');
}
```
is the same as
```js
if (!isStream(stream)) {
  reject(new TypeError('Expected a stream'));
}
```
???


If this section is different for rh and sles, this should be divided into different os section.


code.google 서비스 만료라 옮김..


We had been using this behavior for a while, though we implemented a workaround and also by now we're working on converting our build from Maven to Gradle (primarily via a script).

Previously we used profiles to provide more goals than just 'mvn install' and we had a separate script that knew all the details of the dependencies between the various goals in different projects so developers didn't have to remember them all. That custom script would also handle the incrementalism of the builds and therefore could want to build an arbitrary combination of projects. To improve speed by reducing the number of invocations of 'mvn' (with the number of dependencies in our projects it cost us roughly 2 additional seconds per invocation of 'mvn' and it was common to need 30 invocations), these invocations of Maven were batched by creating a temporary reactor project with subdirectories that were symlinks to the actual projects that we wanted to build. This batching started failing when we updated our Maven version (to 3.3.1, I think).

Our workaround was to update our custom script to make the temporary reactor project mirror the entire file structure of the original projects so that finding the parent project by relativePath in the new way without first resolving symlinks would still result in the desired parent project.


The format of the third level title should be the same with our guide: https://xcat-docs.readthedocs.org/en/latest/developers/doc_guidelines.html


This may lead to replica divergence in case of concurrent updates with identical system timestamps. The reason is that concurrent updates may be delivered in any order to replicas. Only causally related updates are guaranteed to be delivered in the same order at all replicas. I propose the following procedure:

- Check if the current value *happened before* the update by comparing vector timestamps. If yes, the value can be overwritten. Even if clocks are not properly synchronized, we have identified the true last writer from causality information in this case.
- Otherwise, the update is concurrent. In this case, use the update with the higher system timestamp. If they are equal, use another value that consistently orders concurrent updates at all replicas, for example the `emitterId`.

I've just [updated](https://github.com/RBMHTechnology/eventuate/commit/969bdd67a70da302c75f6be607a9782d54cd9b6c#diff-89829be350570a9e7e7ba5c89128f182R63) `CRDTServiceOps.update` to have access to all event metadata of `DurableEvent` via the `event` parameter. You can get the `emitterId` via `event.emitterId`.

In the current implementation, `emitterId` is the `serviceId` of the corresponding CRDT service. This may change when #47 is implemented. However, this won't affect the implementation and correct behavior of `LWWRegister`.


This expression isn't correct. Please fix it like ```*(config_.ignore_keth_same_point)```


should be mockbin.org


101162510-SubsurfaceOutwardAngleWarning-5016 (rglazer2) - Win64-Windows-7-VisualStudio-12: OK (1871 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-99.95%25-yellow.svg)


The problem is the loads of Unicode characters this method considers whitespace that are not valid HTML whitespace. Let's be messy and correct rather than clean and wrong.


http://httpbin.org being used everywhere


@josevalim I just ran into this missing import on Phoenix using ecto master. I'm wondering, does this mean I should be putting changeset-related code outside of my models?


ObjexxFCL_Update (DeadParrot) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1871 of 1871 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)


merge yucheng work


:facepunch:


really bad


@indutny This is the solution recommended by @jeisinger: https://github.com/nodejs/node/issues/1451#issuecomment-94529988

I don't understand what you mean by "handle it ourselves".


1354라인에 shift operator #3 << 0 원래것이 맞는것 아닌가요?


#4402-VRF-outputs-and-multiplied-floor (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1310 of 1310 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.16%25-green.svg)


Though I still guess I don't really understand the overall benefit of having the people writing the Rotefiles use `cargo.build()` instead of just letting them call whatever commands they want with `exec` directly, thus giving them full access to those tools' available arguments.

It is nicer to look at, but it kind of just feels like forcing people to learn a new (although very similar) syntax without any real benefit.


ObjexxFCL_Update (DeadParrot) - 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 line is missing two period prefixes that result in an error when sampling in parallel.


This is breaking some patterns. Have you seen issues with this implementation? Have you deployed this anywhere? Specifically, the pickadate pattern is broken due to this.


You're right!  Copy/paste mistake... fixed.


101162510-SubsurfaceOutwardAngleWarning-5016 (rglazer2) - i386-Windows-7-VisualStudio-12: OK (1871 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-99.95%25-yellow.svg)


Failure on circleci is due to current tests not knowing about authentication. Will probably make move to cucumber tests when I can find the time. Will put a story in for it.


CSV means we could take output from an Excel spreadsheet and import it.

The full dump/restore would be with `pg_dump`/`psql` as you said.


@trevnorris we can allocate buffers using "the old way" for small sizes.


where in kong?


Yes, my question is:

 In every checkpoint and assignment I start with the master alcolator. This
means that I start with the most basic alcolator file (wine only) and I
have to add the whiskey controller, then navigation, then... etc.

Is there a way I can use my last file which includes all the changes and
updates so I don't need to do all over again when starting a new checkpoint
or assignment?

On Tue, Sep 15, 2015 at 12:52 PM, dothework <notifications@github.com>
wrote:

> @abnero <https://github.com/abnero> DWMR. Do you still need me to help
> get the storyboard to editable state in this project?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/abnero/Alcolator-1/commit/c8c45fa1ac550e6fc76e1e960e425091ceef1787#commitcomment-13246144>
> .
>


This won't work because later you assign to worlds in onEnable.

Just change in line 53 from:
worlds.contains(e.getWorld().getName()) || all
to:
all || worlds.contains(e.getWorld().getName())

Because all is true when worlds is null, you'll be fine if you check it first.


Should this be `identical(a, b)`?


@douxing, thanks for the PR. I was just wondering, how useful is this without replacing the call to api.github.com? Wouldn't that be blocked by the GFoC of china too?

Perhaps the download mirror option should only work when a version is provided?


Try deleting the `vendor` folder and file `composer.lock`
Then run command `composer install`
After run `artisan cache:clear`


Yeah, there's not much benefit in those examples, but there is more benefit in higher-level abstractions. For example, to build a C++ program, all you need is

```lua
cpp.binary {
    srcs = "src/*.cpp",
    out = "bin/myapp",
    standard = "C++11"
}
```

which should translate to something roughly like this shell script:

```sh
for file in "src/*.cpp"; do
    g++ $file -Wall -g --std=c++11 -c -o "obj/$file.o"
done
g++ -o bin/myapp "obj/*.o"
```

(C++ is a work in progress at the moment -- I should have it out by the end of this week to look at.)


@indutny This can fall short for those who want to "subclass" Buffer. Additionally, unless we scan for that flag, intercept, then store the passed value we can't know for sure that we should only use the old way for 64 byte sizes.


Small detail. You can install the package: https://libraries.io/npm/is-power-of-two
npm install is-power-of-two --save
(--save allows to store it in package.json as a dependency to the project)
There is an amazing trick (check the source code) to check if a value is a power of two.


In general the recipes should describe what you want to do, rather than how to do it. Of course, different types of projects vary a lot how much work is needed for different recipes.


Thanks. We've got this fixed with bc98f46.

On Wed, Sep 16, 2015 at 12:10 AM, Eric N. Brown <notifications@github.com>
wrote:

> This line is missing two period prefixes that result in an error when
> sampling in parallel.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/stan-dev/rstan/commit/8a44f81b5691d9a2b89a3a18c8f06636977ad16c#commitcomment-13252927>
> .
>


We can add more characters if you want.  I think I wrote the class to allow that easily.  The check-digits are generated right now by summing the ASCII values of the characters and taking the low 4 bits (so mod 16).  That is not a very good algorithm because it does not guard against transposing characters.  If we have more than a 4-bit check value it will be difficult to think up more ice cream flavors, so I would have to change that aspect of it slightly.


102966118-HeatRecoveryBypass-4568 (mjwitte) - x86_64-MacOS-10.9-clang: Tests Failed

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


indentation is wrong. Two spaces, like you have at line 289 below


wrong indentation


Je vais étudier la chose  ! ;)
Pour l'instant, bois pour cet hiver, je touche plus pour l'instant...


The assumption that `Widget` subclasses don't override `operator== ` is baked pretty deep into the code already.


investigate the time_zone


@jawaad-ahmad We should probably be having this conversation somewhere else.  It is buried in here, and I don't know how to find new comments easily.  I have to scroll through and look at the age of the individual comments.


insert character_no as primary key for easier indexing from the front end


@thenickcox Does this put us back into a place where we can deploy to production?


change foreign key to character_no


location of the mission?


gfx::AcceleratedWidget as window handle type.


@davidjanghoonlee Part of it is the commitment to having the OS more visible to the user. Part of it is that Linux users tend to be more familiar with the command line. And part of it is that the "old timers" who started Linux are very resistant to making it easier. But at the same time, Android is Linux and is quite user-friendly, so it can be made less "scary" if you want.


add location_no for the table


You can block all rootfs-packages files being copies to the build by commenting out the relevant lines in 3builddstro{,-Z} (check for rootfs-packages). rootfs-packages is ment to hold various "common" or no-arch pets so they can be easier maintained. Currently it has jwm_config and ptheme.


@leandromelo98 Need more detail in your summary.


##
Run ./startqarch at first to edit ari.conf


gfx::AcceleratedWidget as type


No global variables allowed!


@leandromelo98 Yes, quite a bit. Linux and Unix (the base for OS X) are the most similar but not completely compatible. Windows is completely different though, which is why many programs on the command line won't run on it. However, Microsoft is building in some features (like SSH) that will make Windows more compatible with the other two OSes.


Indentation two spaces


@Paul-I Fix your dates to match the template.


### this project is on testing-preview


this is only the source. to use this you need to compile it.

for compiled versions please download priiloader from http://wiibrew.org/wiki/Priiloader


Re: Issue #12


s it because the operations happen on different threads ? Then  what about value of GetData, retrieving that doesn't look safe either.


Which problem does that change fix?


git clone


@look what does this do?


Okay, that makes sense. I forgot how gross gcc/g++/clang are to use.


@jordanmendler This is a shortsighted, very foolish move you should reconsider asap:

* Ads alone will never make the project sustainable, let alone fund developers properly. Bountysource, as recommended by @nullivex, would net you much better and safer funding.
* It does not even comply with AdSense terms of service. In order to serve ads on a site, you need to have a valid claim for it on your account. Best case scenario, funds will get frozen. Worst case scenario, it'll get banned and whatever you have in it now will be lost.
* This will just piss your users off and you'll lose them to a different project. Flyspray isn't exactly unique.
* In a few hours you seem to have pissed a few contributors off already with this move. I admit I'm not a contributor, but If the funding you need is for development, this seems massively counter-productive.

On the bright side, pissing people off is bound to attract attention. If this ends up on some press outlet, you'll end up ridiculed, which may end up bringing attention to the lack of funding on the project. *shrugs*


Yeah. Make & friends make you do that stuff by hand. Some of the recipe ideas come from [Bazel](http://bazel.io), a nice (but very large Java/Python) program for building.


*tails


thx


not clear! Discuss with @sabhareeshr for the requirement


Don't mind me. DERP


unfortunately it needs another rebase (I think only the platform_window  patch). I have add some indentation fixes and a question about data race issue. With those addressed LGTM


I should have been clearer about what I was saying.

I do think docs should be updated, but I think at the very least it should read something like `Valid value is` and not `Valid values are` ... since there's only 1 possibility.  Taking this a little further though, shouldn't we just make these read-only?

There is a precedent for read-only parameters - for instance, see `ctime` as an example:
https://github.com/puppetlabs/puppet/blob/master/lib/puppet/type/file/ctime.rb

There are a number of read-only parameters in the AWS module as well, for instance `instance_id`:
https://github.com/puppetlabs/puppetlabs-aws/blob/master/lib/puppet/type/ec2_instance.rb#L129-L134

I would rather see the description indicate read-only if we never expect a user to change the values.


done


Data entry requires premeditation. Check if it can be removed.


That's strange I only get one scrollbar. I'll try with specifying a fixed height (less than the window) for the testarea and not let it grow. That should prevent scrollbars in the main window object.


I reverted this change initially because adding overflow: auto to post content was clipping post mention previews (i.e. when you hover over a post mention). But I realised that this is something to be fixed in the mentions extension, hence the second reversion :P


The logic may differ based on requirement. So far so good.
TechDebt: Try to generalize


#if defined(USE_OZONE) should be enough


Looks like a mini-typo, that should be `CMAKE_CXX_FLAGS` right?


ObjexxFCL_Update (DeadParrot) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (801 of 801 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)


Maybe it uses the aggro range of L2TrapInstance?


**The function is not supported for NFS-based statelite nodes.** =====> it is no need to mention statelite.


"updatenode cannot be used to sync files to statelite nodes."=====> we do not need statelite in doc any more.


One theory is that removing the FKs ... what? :suspect: Suspense is killing me here


Cr::--D+++

(0) __38=


+** Related To do** ====> Is there any other topics?  If there is no other topics, we can drop this line.


find a way to integrate this update routine into the core files so that every time the script runs, the core files are updated


This check should not be needed. Once we have figured out all the changes we need, we need to find a good way to inject this via Ozone factory classes. This is how we are doing it for WebUI for now, till we figured out the final solution.


These could be put outside the loop, no reason to recalculate the regex for every single command (imagine if you'd have 1000 commands)


Thanks - reworking the sample code as we get ready for publication of the book,


This statement is useless, it doesn't matter if args exists or not, you can pass it anyways. In the end it will be nil in both cases if args didn't exist.


Funding source is now configurable. There's an interface and currently only a single implementation, but we could add alternate implementations like a pretended wallet for running on TestNet or even Main Net.

@dexX7


Have tried or not, this and the PHP version of the relationship


Nope :)  Did that before that was fancy :)


Yes, that's by design.  Each individual test takes 1-3 seconds to run all the combinations, the full set is 90+.
It's not `OuterLoop` because that attribute can still only be placed on methods.  I did make sure to check that again as part of this set of updates.  I'll go ahead and post a build tools PR - should I wait for that to get through and then update this, or do that separately?


+** NOTE **: ===> its format is not right from html looks, please check it.


Test, Test, Allaolevilla arvoilla ei tee mitään


I like this approach (y)


I do not like the duplicated code there.


Ah..haha! Took me longer than I'd like to admit to catch it. :)


My local PHP version is the version of the 5.6.7 server is 5.5.9


Minimal (poorly named) tests of new subclass of BitcoinClient that includes many of the "value-added" methods from BTCTestSupport.

Should be cleaned up and expanded.

@dexX7


This test *does not* extend `BaseRegTestSpec` and shows how the new classes can be used outside of Spock.


My *guess* would be that this relates to InnoDB and space reclamation; `TRUNCATE` appears to reclaim while `DELETE` does not. ([`RENAME` is another quick alternative proposed?)

If truncate reclaims space, then it has to wait for disk activity to do so, which could cause additional delay (time penalty vs disk perhaps, if delete doesn't recover).


Really? I was using Firefox in that case. I'll try Chrome tomorrow.


Groovy traits don't seem to allow final properties, so I added this as a safeguard.


nodeset and updatenode are 2 commands, we can use  format like ``nodeset``


Very little left here after most code moved to FundingSource or BitcoinExtendedClient.


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


Oh wow, that's neat. I hadn't heard of that before. I'm a sucker for Google stuff.


That has been removed a few commits earlier.

On 15 September 2015 at 20:28, Christoph Döberl <notifications@github.com>
wrote:

> this only fixes half the problem. the prefix "Playing " persists..
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/DevLoL/iTunesMQTT/commit/ec4675e6cb81fbae651b9972d9a8c4f93fb83212#commitcomment-13244422>
> .
>

--
http://ananasblau.com


There are many useful functions in here. Now there accessible in an extended RPC client -- and also in pure Java to be useful to more projects.


What you can do is this in ui/base/ui_base.gyp
        ['OS=="win" or use_clipboard_aurax11==1 or use_ozone==1', {  --> use_ozone check is new here
          'sources!': [
            'clipboard/clipboard_aura.cc',
          ],
        }, {
          'sources!': [
            'clipboard/clipboard_aurax11.cc',
          ],
        }],

This will make sure that clipboard_aura is not compiled when use_ozone =1. This should be ok for now as we dont support any other ozone implementation in ozone_wayland and should be easy to maintain patch till we get proper solution in place.


You changed -1 to 1?


Please avoid commiting changes in Gemfile.lock unless you have added a new dependency


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)


** multiple sets of rpms ** that you want to ** keep separate **  ====>Could you check correspond format for **xxx** in RST?


You dont need to add it here, you can have it in platform folder (in ozone_wayland) and add these files to ozone_impl.gyp file. Also you would need to implement factory method here.

// Clipboard factory method.
Clipboard* Clipboard::Create() {
  return new ClipboardWayland;
}


I think my jQuery syntax is incorrect. Other than that, this is correct. I can only test tomorrow evening.


I like the test, probably should add a devDependency for mocha though


@devinrhode2, can you please elaborate a bit to what the current result is and what it would be. This code was merged in. Are you saying so it includes some of the input content?


@VoyagerOne: You are right, it will be corrected (if needed).

On Tue, Sep 15, 2015 at 11:37 PM, VoyagerOne <notifications@github.com>
wrote:

> note (d-1)/4; will be rounded down since it's using integers...
> so if x=(d-1)/4
> d=(1 to 4) x=0
> d=(5 to 8) x=1
> etc..
>
> I don't think that is your intention...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/IIvec/Stockfish/commit/abcc29ec9eaaf660498e5e21af4613771c79e238#commitcomment-13248429>
> .
>

--
IQ World <http://www.ultimaiq.net/>


I made updates but am having a hard time pushing them, will try to tackle in the morning but in the mean time I emailed you my updated files.

I am receiving this error message:
Cindys-MacBook-Air:phomoto CindyTong$ git push origin cindy
To https://github.com/rushindrasinha/phomoto.git
 ! [rejected]        cindy -> cindy (non-fast-forward)
error: failed to push some refs to 'https://github.com/rushindrasinha/phomoto.git'
hint: Updates were rejected because the tip of your current branch is behind
hint: its remote counterpart. Integrate the remote changes (e.g.
hint: 'git pull ...') before pushing again.
hint: See the 'Note about fast-forwards' in 'git push --help' for details.


To PRE.


@evz, in the quarterlies queries, we need to get the pre-election filing if that filing is the most recent filing, no?


@yyfjmb Please don't hand-code variables like this. The outcome would have been every article being attributed to one person (me), which is very confusing. If you don't know how to obtain the authorship information, you can ask me.


fuck u


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


Looks good. The next question is always "how do I edit the docs," of course.


这个 interesting 啊


I did it on a separated commit on purpose, because I don't understand why my `bundle install` changed so many things. Are some dependencies' versions not specified?


Well it seems this fixes the jet tracing. Track jets are recoing pretty well, but there is something wrong with the calorimeter tracing (it worked the last time I looked). I'll look in at that in the morning.


원래 것이 맞습니다. 웹에서 편집하다 오타가 났네요. 수정했습니다.

2015-09-16 13:09 GMT+09:00 Jaewon Choi <notifications@github.com>:

> 1354라인에 shift operator #3 <https://github.com/iamroot12b/kernel/pull/3>
> << 0 원래것이 맞는것 아닌가요?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/iamroot12b/kernel/commit/bdadca286e4bef0ede806096ab60db49eb6cd9a9#commitcomment-13252917>
> .
>


Smashed my head to the wall a hundred times trying to get this to work, so I just gave up xD
Tried to do it using an arraylist but sorting won't work.

If you think of a better solution, let me know


+  * rpm name without version numbers
+  * otherpkgs subdirectory plus rpm name
I think you want to use Bullet Lists, format is not correct  when "*" is in ": ::" from html view.


ok


I don't think this should throw an exception. Instead return bytesRead and propagate and change WriteBytes to return it as well. The caller should decide what to do.


@jordanmendler I am not trying to say I have the right to do anything. However, I have made some very large contributions in the past. I have used Flyspray for many years. I also helped with the move to github and closing numerous issues in the previous internal Flyspray instance. My point is, that even though I have not contributed lately.

That being said, I still care about this project and how it continues.

This is a bad move. A really bad move. Lets clean this up before it goes any further.


+The #INCLUDE ====> we can use ``#INCLUDE``


102966118-HeatRecoveryBypass-4568 (mjwitte) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


original version of von_mapping algorithm (only random), by Chen Zhendong.


#INCLUDE can be in ``#INCLUDE``


s/delet/delete/
I think I have seen this before - dejavu? Maybe it would be worth doing a grep for other occurrences of "delet the".


No reason really, guessed `.keys()` will be less likely to fail randomly.


Card One


#INCLUDE:<file># can be ``#INCLUDE:<file>#``


ObjexxFCL_Update (DeadParrot) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1337 of 1337 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.23%25-green.svg)


Which ruby version are you using?


Creo que no debería ser una clase... pero ya que es una clase -> no necesitamos instanciarla, solo acceder estaticamente a sus campos -> sin getters y mucho menos setters. Voy a dejar otra interfase en mismo lugar


Actually, it's BFS *


Other files use tabs, so why did you change this one to use spaces?

Nice module, by the way :)


man postscripts =====> command should be in ": ::"


```
$ ruby -v
ruby 2.0.0p481 (2014-05-08 revision 45883) [x86_64-linux]

$ cat .ruby-version
2.0.0-p481
```


I think this test doesn't work correctly because config.ignore_kth_same_point isn't set "true"


/install/post/custom/rh/myotherlist    ======>  directory link can be ``/install/post/custom/rh/myotherlist``


/install/post/otherpkgs/<os>/<arch>/ =====> directory link can be ``/install/post/otherpkgs/<os>/<arch>/``


If someone starts a song with a combo over NumberMaxZoomAt, the number will have a zoom of 1 (the default) instead of 1.1 (NumberMaxZoom).  Use "math.min(NumberMaxZoom, scale(...))" instead of the if statement.


#NEW_INSTALL_LIST# =====> can be ``#NEW_INSTALL_LIST#``


The pull request cannot be merged because of merge conflicts; you probably  squashed other commits already on master branch. Can I trouble you with rebasing so the pull request applies with no merge conflict? Thank you!


The "--" syntax is supported in xCAT 2.3 and later.   ===============> we do not need to mention xCAT 2.3, there is no xCAT2.3 in github.


I think the correct thing to do:
find . -exec touch -t 198001010000 '{}'


Title level format should be the same with https://xcat-docs.readthedocs.org/en/latest/developers/doc_guidelines.html


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


'updatenode -F' =====> commands should be in ``updatenode -F``


'xdcp -F', 'xdcp -i -F' and 'updatenode -F' =====> commands format should be like ``xdcp -F``


Not wild on this because it creates extra space in the page. Can we fix it with min-height ?


Feel free to force-undo this commit if you need the buffer


Some questions:

- why do you think this would go wrong? there are no deletions in m_devices, only updates
- why not using a boost::mutex and boost::lock_guard<boost::mutex> l(m_mymutex);
- sometimes you unlock, most times not, this should not be needed when using the above lock?


Your screenshots have shown up an alignment glitch on IE.


update


WTH is this?


シンプルにこれでいいですね。
```ruby
@cart_items = current_user.cart_items
```


already solved the ((dType = pTypeGeneral typo in a previous commit


this resolves #322


+1 for implementing spawn retry!


@schtanislau It was the same thing we saw earlier with the scheduler spamming error logs. We just didn't see it because `drag_create` was disabled. Basically when you clicked on an empty space it tried to run `group_readonly()` but `id` was `null`, so there were errors. Just added check for `id`.


I just tested in both Firefox and Chrome and both worked fine with single scrollbar.