VS-equipment-issues (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


VS-equipment-issues (EnergyArchmage) - 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)


TeamCity Selenium POM :: CI [Build 34](https://teamcity.sponte.uk/viewLog.html?buildId=319&buildTypeId=SeleniumPom_Ci) is now running


VS-equipment-issues (EnergyArchmage) - 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)


* English ;)


Bash has file test operators which might apply here.  http://tldp.org/LDP/abs/html/fto.html


TeamCity Selenium POM :: CI [Build 34](https://teamcity.sponte.uk/viewLog.html?buildId=319&buildTypeId=SeleniumPom_Ci) outcome was **SUCCESS**
Summary: Tests passed: 35 Build time: 00:00:46


at least --enable-libuuid is required in PKG_CONFIGURE_OPTS_HOST (hint: parted:host)


"untranslated", not "untraslated" (also below)


Good, merged


Something is wrong.
I added `ASSERT(nullptr == curTxFrame);` as first line of `EtherMACBase::getNextFrameFromQueue()` function.
When I run fingerprint test, then more than 30 example generates this assert.


is


102941444_CorrectBSDFs (macumber) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.67: Build Failed

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


IIRC str_replace is faster if you don't need regular expressions for anything particular. You also don't have to worry about regular expression escaping that way.


Because I haven't added it yet, so I have to write what works and what doesn't until I have a final version ready :P


This didn't fix the issue
@javieranton1999


@Squareys second problem: your test is wrong. only the pixels inside the processed interval must pass the test ...


Is this dead code?


Typo: "parametter"


`widget_base` is undefined in constraint.lua at line 102


![interpolmk2 easing blend](https://cloud.githubusercontent.com/assets/5783432/9820930/d9b7c81a-58bf-11e5-9d80-9733587c811f.png)
node easing just merged. Sorry


Just curious, is warning and invariant different? I skimmed warning's
implementation and didn't see a calculable difference.

On Fri, Sep 11, 2015 at 10:38 AM Samuel Reed <notifications@github.com>
wrote:

> Thanks. I wanted to keep this as an invariant for now. Could move it to a
> module but it's such a small piece of code, it's just simpler to keep it
> here.
>
> I did recently move another project over to warning.
>
> STRML/react-localstorage@c36e31a#diff-91831cddf2dfe00f42928b79d9c52d97L45
> <https://github.com/STRML/react-localstorage/commit/c36e31a1aa6cca807493a2e15400ab9a224eacf0#diff-91831cddf2dfe00f42928b79d9c52d97L45>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/STRML/react-router-component/commit/e951920c90ba92a7259f6a0ae69759dec7e14d14#commitcomment-13185895>
> .
>


I'm a bigger fan of explanatory comments than anybody else around these parts, but there are some points of confusion here:

- `Backend` is more abstract than is suggested here. Anyway I think the type says it all and it's probably not worth explaining much.
- Doesn't `configChanged` also mutate the Backend? I'm not sure that's right, but where that happens is the trickiest bit of business going on here, and would be worth spelling out.
- Can you articulate the relationship between backend, initialConfig, and configChanged? *Note: the plan is to eventually fold config into Backend, so that "configuration" would be the serialized form and Backed the runtime representation of the same concept. In the meantime, the relationship is a bit indirect.*


Dynamic commands **DUH**


What month is the 48th? ;)


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


Not really - just one hits console.warn, the other throws an error. But I believe, in the future, `warning` will either hit console.error (so we have a stacktrace) or call console.trace().


Looks good overall -- but how do you feel about packaging this plugin as its own git repository? That's our preferred way of managing plugin code.


Commit request is unclear. It doesn't change anything. Is your tree outdated?


Right.. it means "post-hammer".  So if the booting osd is post-hammer, and the cluster features (intersection!) does not include 0_94_4, fail, because some up osd is < 0.94.4


this will appear automatically in the project generator once we include that option and the command line PG already has a --help output that explains how to use them. i could include some comments in the run method but i guess looking at other tests should be explanatory enough.

Why do you think they are not unit tests? they test one unit as opposed to integration tests which would test the interaction of different units.


@crayon34 are you showing history anywhere on UI? I don't see UILabel outlet only for display.


DashboardFixes (macumber) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.67: Build Failed

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


81211464-LinkInfiltrationAndReturnFlows (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1859 of 1860 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)


added space


You should put your copyright here


C'est quoi tous ces fichiers que tu commit ? :-/


With fake DSP data (best I can do)

![screenshot017](https://cloud.githubusercontent.com/assets/6500800/9821059/12ce9046-58b8-11e5-8393-81b2e84fff64.jpg)


oh Doh! I missed that. thanks.


Thanks, @mgoffin Not sure why I didn't do this :-(


Could you elaborate?  All I did was remove the underscore.


@RandomAltThingy xD alright, so, I guess you'll have to fork it and then make a merge request. Do you wanna do half and half (I do some commands, you do the others)?


@notspiff @wsnipex this crashed on my system. I had gif_lib.h (4.1), which had no GIFLIB_MAJOR


The file got added during the fork & pull request process. Probably not 100% necessary, but certainly no harm in having it


We're hardcoding which scale we're talking to, and what password it has.
Not saying we need to fix this now, but do we have a design for parametrizing this?
(One possibility is a BodyTraceScaleDevice class - so you would get the device argument as the second argument to createChannel, after the engine)


@notspiff @wsnipex update, this introduced the crash of cmake
why did you remove if(NOT GIFLIB_MAJOR) ?


`test "brew", "readall", @tap` will read all tap formulae. And I don't think we need brew tests as it doesn't read any formulae files.


Why remove that? :cry:


does it conform to the SPEC? now some includes will  be in includes/openshmem instead of includes/ ?

``` shell
16:04:27  /bin/mkdir -p '/var/lib/jenkins/jobs/gh-ompi-master-pr/workspace/ompi_install1/include'
16:04:27  /bin/mkdir -p '/var/lib/jenkins/jobs/gh-ompi-master-pr/workspace/ompi_install1/include/openshmem'
16:04:27  /bin/mkdir -p '/var/lib/jenkins/jobs/gh-ompi-master-pr/workspace/ompi_install1/include'
16:04:27  /bin/mkdir -p '/var/lib/jenkins/jobs/gh-ompi-master-pr/workspace/ompi_install1/include/openshmem'
16:04:27  /bin/mkdir -p '/var/lib/jenkins/jobs/gh-ompi-master-pr/workspace/ompi_install1/include'
16:04:27  /usr/bin/install -c -m 644 shmem.fh shmemx.h shmem-compat.h pshmem.h pshmemx.h '/var/lib/jenkins/jobs/gh-ompi-master-pr/workspace/ompi_install1/include'
16:04:27  /usr/bin/install -c -m 644 shmem.h '/var/lib/jenkins/jobs/gh-ompi-master-pr/workspace/ompi_install1/include'
16:04:27  /bin/mkdir -p '/var/lib/jenkins/jobs/gh-ompi-master-pr/workspace/ompi_install1/include/openshmem/oshmem'
16:04:27  /bin/mkdir -p '/var/lib/jenkins/jobs/gh-ompi-master-pr/workspace/ompi_install1/include/mpp'
16:04:27  /bin/mkdir -p '/var/lib/jenkins/jobs/gh-ompi-master-pr/workspace/ompi_install1/include/openshmem/oshmem'
16:04:27  /usr/bin/install -c -m 644  oshmem/constants.h oshmem/frameworks.h oshmem/types.h '/var/lib/jenkins/jobs/gh-ompi-master-pr/workspace/ompi_install1/include/openshmem/oshmem'
16:04:27  /usr/bin/install -c -m 644  mpp/shmem.h mpp/shmem.fh '/var/lib/jenkins/jobs/gh-ompi-master-pr/workspace/ompi_install1/include/mpp'
16:04:27  /usr/bin/install -c -m 644  oshmem/version.h '/var/lib/jenkins/jobs/gh-ompi-master-pr/workspace/ompi_install1/include/openshmem/oshmem'
16:04:27  /usr/bin/install -c -m 644  oshmem_config.h '/var/lib/jenkins/jobs/gh-ompi-master-pr/workspace/ompi_install1/include/openshmem/.'

```


First Commit - testing


No worries. Importlib is the proper way to do it in python3 because it has a similar feature, but in 2.7 it doesn't so we need `imp`.


Sorry, I removed it accidentally. Feel free to add it back!


When does people see 9.0? :smiling_imp:


I just tried out yours best splits feature now !!

How MY GOD !!!! This is awesome !! Awesome work !! I'm really impressed of the final result !

So thanks for this !!

Impressive work !! Respect man !!

I can't tell anything else


This was actually meant this was as a default value for running `./travis-ci.sh` from the console.


merged :smile:


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


But what about the other changes? It seems like you forgot those...?


I changed matrix indexing behavior of toRaveledTensor's to same as toRgb. I have no confidence about this.
=>  ret.putScalar(new int[]{k, i, j}, raster.getSample(j, i, k))


- I will remove the doc on `backend`
- Not sure what you mean exactly. My understanding is that `configChanged` is essentially a callback, you can do wtv you want to do with it. Maybe some client mutates the Backend in it, but not the code in the test suite.
- I am not sure I understand this relationship very well. Like I said, I perceive `configChanged` as a kind of callback, not that much more to it. Adding doc on this class is a very secondary concern of this PR, I just bundled it up because I didn't think it was worth opening a another PR for it. If you feel that there is a relationship here that needs to be expanded, maybe you or I could address it in a separate PR.


Thanks; removed.


I guess because previously in Chrome, if you had a million nodes in a subgraph but not connected to the destination, there were no CPU or power costs. Only when the subgraph get connected (directly or indirectly) to the destination was there a CPU and power cost.

Also, the first part of the sentence seems to imply that this processing only happens if you keep a JS reference to the node.  But if you have an AudioBufferSource, schedule it to play, and then drop the reference to it, the source must process, even without a JS reference to it.

I can't currently think of a good way to phrase that.


I tried it this with 2.01 and LORE. Before, 352 items loaded. After, 997. :+1: :+1:

I'm going to try 8.01 too, but I think this will be good to go.


ImprovePackaging (axelstudios) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.67: OK (0 of 0 tests passed)

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


## My comment
# first comment for this repo
I don't know what this is for


Looks good to me but I have not done any testing.  Thanks!

Rod


Is this problem still existing?


languages.yam ? this probably will not work until it's `.yaml` :)


...someone needs to show me how to write test cases for quickcheck. We have ZERO tests for the global and virtual compiler.


Awesome! How do I configure fill styles?

One way would be to have an alist of fill styles with setting global vars like `ess-fill-calls-newlines`. Just like we do with ess-style-alist. That would mean ignoring the default (global) values of config variables.


Cool. I'm installing locally on LORE too, and will report back.


this code here is actually a captured stderr output, which is quite confusing. I'd also suggest displaying that stderr so user knows what went bad. thank you


ringtomes?


woot!


I was thinking of something along these lines but an alist with style parameters is even better.

And the styles would cycle according to the order of appearance in the alist.


@Jedwondle @tyler-travis Please post review.  Here are all of the latest help file changes for "b" and "c" here:  https://jira.di2e.net/browse/STORE-548


BTW, we need to remember the original state of the code and cycle through it at the end. The user might not like the new reformats, so we need a quick way to return to the original state.


https://github.com/ksooo/pvr.mythtv/commit/8d6f3b11368f9e192330c1e48a61ddd3166baa88#diff-4a96ce340435e1efecda6f9744992356R1751
```
if (timer.iClientChannelUid > PVR_TIMER_ANY_CHANNEL)
{
  hasChannel = true;
}
```


Hi Lizhe,

It is strange that some of the charges are so big. If they are small numbers, the program could work properly. It may be good to leave the code as it is due to if there is extremely large charge appeared, there will be error throw out (as you met in your work). You can check the resp1.in and resp2.in file to see whether there are any errors in it. Here is a webpage which has explanation of the RESP input file: http://q4md-forcefieldtools.org/RED/resp/.

Best,
Pengfei


good point


> so we need a quick way to return to the original state.

Another way would be to not alter the undo history, but that's optional.


La inyección de dependencias o DI por sus siglas en inglés, es una herramienta comúnmente utilizada en varios patrones de diseño orientado a objetos, consiste en inyectar comportamientos a componentes.
Esto no es más que extraer responsabilidades a un componente para delegarlas en otro, estableciendo un mecanismo a través del cual el nuevo componente pueda ser cambiado en tiempo de ejecución.

 Es conveniente no confundir Inyección de dependencias (DI) con Inversión de Control (IoC), error muy comúnmente cometido que figura especialmente en la web. IoC es un tema para un próximo artículo.


I'd stick with bForceDelete - but that's just my opinion.


It's technically a direct upgrade if I leave it as is, and it's not like anyone wants mercs to be able to cap faster.


That one tripped me up too. :confounded:


read my comment above. @NoNaMe-Lviv


We should not be calling `os.Exit(0)` from reexec function. It's the responsibility of the caller of `reexec.Init`


No, it should just yield an empty slice (`array[array.length .. array.length]` is a valid empty slice).


Yeah, sorry about that. Tests didn't catch it.


On Sep 11, 2015, at 12:46 PM, Samuel Thibault <notifications@github.com> wrote:
>
> AIUI, X_LIBS is not meant to contain -lX11.

Ah.  What's it supposed to be, then?  I.e., is what Bric^H^H^H^HI did by hard-coding "-lX11" actually the Right Thing to do?

--
Jeff Squyres
jsquyres@cisco.com
For corporate legal information go to: http://www.cisco.com/web/about/doing_business/legal/cri/


101884090--NewVenting-Availability-Schedule (jasondegraw) - x86_64-MacOS-10.9-clang: OK (1851 of 1851 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)


Yes, keeping the reload penalty is intended because it's more powerful than anything else if I don't, because you can more or less instantly get your whole clip back, especially as last player.

I forgot to send you this back when the gunmettle update happened, after I noticed that the reload speed was overpowered for soloers.


Why do we need two go routines here? Why can't we have one go routine for both sending the request and getting the response? In fact why do we even need a go routine? Why can't the reexec process just block on this call? It's not like it needs to do anything else


The problem is that this test misfires so frequently that it does not really provide any useful information and make Travis notifications pretty much useless. Is it possible to turn these into a warning so that they do not imply build failure?

Having a hard coded version number is very inelegant and seems to require updating the test every 3-4 days. Are there cases where a breaking change is made to Racket and not caught by other tests in our test suite? If so, that would seem to be a shortcoming in the test suite for the affected feature.


Okay. I will do the base. As long as you give me some credit on the forums for the rewrite :P
:+1:


Thank you :+1:

I'll let you know in a pull request if I build something nice.


Good then :+1:
It will be released tomorrow then ;)


![inyector de dependencias](https://cloud.githubusercontent.com/assets/11052426/9821547/c54ca242-586f-11e5-96f8-c61e21622085.jpg)

En el diagrama se puede ver el diseño básico del patrón. La clase principal DependencyInyector sirve como interfaz para el conjunto. El usuario interactuará con el conjunto a través de esta clase. Para diseñar esta clase podemos usar el patrón singleton, ya que normalmente nos hará falta una única instancia de la misma.


Create a branch called *rewrite*


@philboeselager I refactored the interfaces of our resource repositories quite a bit and I would like to do some further simplifications. Let's have a closer look together next week!


We are passing a lot of `err` values as is without redefining them with more context from the calling function. This will make troubleshooting user reported issues harder. Can we introduce more context to err values returned in this function and `processReturn`?


Hm OK, and my fix doesn't fix the issue anyway... The problem is that some configurations that were not meant to execute on GDC/LDC seem to get executed now, resulting in the test failures. I'll have to test this on an actual Linux machine, this commit was just a shot in the blue.


Let's not allow anybody other than root (or whoever created the socket ) to have access to it. Which means the access mode value should be `0600`


Same here. `0600`


Add comment
`// The name of the note is determined by X, Y...`


semicolon


Yeah, I try to add unit tests when I have spare cycles.  I'm not really convinced of Quickcheck's value ... it's good when you have a function and its inverse and they share no code whatsoever, but that's a pretty small domain.  Maybe someone else has a different perspective.

If you have policy files that you used to catch this error, you could just encode them as plain ol' pa_ounit unit tests.


Line length on these two.


sinceramente eu preferiria remover o iw.debug do buildout; isso é uma opção de cada um


> Another way would be to not alter the undo history

this doesn't seem to work unfortunately.


not sure if you saw this:

re: lots of junk, thats why i included the include_initial, i am always passing that in as false in the stacker lib to avoid getting the junk: https://github.com/remind101/stacker/pull/90/files#diff-65a106c54767f86dd2517c76ee516287R127.

checking the timestamp works too, but i think then we'd have to parameterize that value and i think include_initial gets you most of the way there.

for stacks that don't exist yet, i just start tailing and handle the boto server error and retry so that i get the first logs, during an update i'll only get the update logs.


By setting this we are saying `geometry` is an officially supported type. We now need polyfills or some answer for adapters that have no idea what a `geometry` type is.


Can we reword the warning message? I don't understand this nor the user will.


I think the correct measurement here is 31mm with a tolerance of ±0.2mm

So why 31.04?


LineEndings (axelstudios) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.67: OK (0 of 0 tests passed)

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


Why do we need to call `Stat` inside the for loop? Doesn't it suffice to be called just once outside the for loop?


Ah, interesting. Well I don't really mind, though since you're `exec`ing `screen`, replacing the shell, sourcing `.bashrc` and setting `PATH` is rather pointless. :)


I'd change
```
most user agents must add 300ms of delay to recognizing tap gestures
```
to
```
most user agents will typically add 300ms of delay before <code>click</code> to allow for double-tap gestures to be handled.
```


For thread-safety you probably want to call `errorMessage.get()` only once and store the result in a local variable


Just out of curiosity, why?


Thanks, fixed in 805db8f4


Removed wildcards


This wasnt ment to be out there to pull. lol
But i hope you dont have any clones of my stolen repo's somewhere.
All others have been removed already.
Else if you do have clones, pls remove them. Hope you will do the right thing
and we dont have to start yet a other drama over this stolen repo's.
Maybe you didnt know what was going on with this repo's, i will not blame you for it.
But enjoy On-The-Go.

Best Regards
BlackDragon


I'm working on a geometry `feature` that I'll also have a PR in for soon. Even though waterline adapters specify the "features" in the package.json, this is the only place that allows me to add new types. Everything not in this list get type-coerced into `string`.


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


I was explicitly trying to avoid mentioning "double tap" since I thought that would make @jacobrossi's lawyer friends happier.  But maybe it's OK in a note?  I certainly prefer your wording.


For the above I would avoid a switch statement unless you enumerate all possible values of this.staff.voice. So perhaps:
```
var octave;
var clef;
switch(this.staff.voice) {
  case 'soprano':
  case 'alto':
    octave = 0;
    clef = 0;
    break;
  case 'tenor':
    octave = 1;
    clef = 2;
    break;
  case 'bass':
    clef = 2;
    octave = 2;
    break;
}
```


plot1.png file


Probably everything is coming from the fact that `analysis-test-plugin` [is not test scoped in `analysis-collector-plugin`](https://github.com/jenkinsci/analysis-collector-plugin/blob/master/pom.xml#L32)


HList (aka Heterogeneous List) ???


With this commit, the line "%!TEX root= ...." will be read from file ".latexer-root" in working folder.
If that fails, it will fallback to the default search.


@tchaikov There was so much wrong with the original code I didn't try to figure out what went wrong in that particular case.  But now the code will do everything it can with whatever information it has.  Previously, a missing SnapSet would skip all the object info logic.  And missing object info would skip the clone checking which would throw the logic off for the next object being checked.


> this doesn't seem to work unfortunately.

Yes. this likely to be tricky because of the multiple commands involved. One way would be to get into pre-command-hook but let's not go that way.


I will push something with all indentation fixed.


It might be easier to handle these using DRF serializer validation ([docs](http://www.django-rest-framework.org/api-guide/serializers/#validation)), which might give more structured error responses.


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


Will this 500 on encoding/JSON syntax errors?


I don't think so.


@rtoy I think the way I rephrased @karlt's statement in this commit does address your issue, since The term "live reference" includes all the above references in this section, including "playing" references.


Consider unbinding events in componentsWillUnmount.


I would move processing of manifests off the main thread, since L40 will lock the thread until the remote server responds. Additionally, errors, etc. in the response will crash the view, which won't be very nice.

I wonder if we should consider having a celery process to queue the manifests for indexing?


I'm not crazy about it either, but this is to be consistent with other prose around setters in the spec.


Car Rental In Varanasi is Varanasi based company which provide instance tour and travel service in Varanasi. We are also provide Rent a Car service across North India specially in cities like Varanasi.  We provide all types of vehicles on


@RandomAltThingy Sure thing :+1:


Unless that check was re-added this week, CDOTAGameConfiguration::AllowDedicatedServers just does "return true;" on Windows.

You can confirm this by starting a server without MM:S and seeing that it stays running after the map command is issued.


The commit log message isn't passing the signed-off-by validation check.  Any chance you can reword it the next time you rebase?


OK


@cam5 I wish I could say it was my own wit. https://github.com/thoughtbot/dotfiles/blob/master/vimrc#L114


The above at least enumerates all the possibilities and helps the reader be more clear on what it does. But really, we're talking about DEFINITIONS of things, and definitions don't really belong in the procedural code when your language supports maps.

```
// Define this somewhere on the class.
var voicePartDefs = {
  soprano: {clef: 0, octave: 0},
  alto: {clef: 0, octave: 0},
  tenor: {clef: 2, octave: 1},
  bass: {clef: 2, octave: 2}
}
```
Then any code can do lookups, like `voicePartDefs['soprano']['clef']`


We probably won't need models, right?

If anything, just a single database table (Manifests?) with URL, created, updated fields.

All the other data can come directly from Solr I think.


rspark-shell has been renamed to run-example


I have these changes and the spinner is the same if not worse than before =/


Is this function removing all the ThirdPartyResourceData of a third-party resource? If so, a function name like removeAllDataOfOneThirdPartyResource makes more sense.


I use Eclipse...
IMO it would be better to configure checkstyle at Maven level, we would get the same benefits without having all that "files noise".


how do you actually download this mod


ok =)


This is really hacky and you'd better report any issue you have with `.destroy` before reaching for such measures.


Richard's milo.createComponentClass is better than Component.createComponentClass... We need to refactor milo-ui to use it everywhere not to spread the old pattern


First off thank you for this, it's one of those features i love and miss.
Secondly I picked this from CM remix
If you think this shouldn't be out or it was stole from you in some way I will gladly remove it, let me know


На сколько я понял, использование prepared statements, позволяет вставлять в поля любые строки, и сервер к ним относится как к данными не пытается как-либо обработать.
Но получается что если мы будем отображать такие строки в браузере, нам нужно при выводе их экранировать. И мой вопрос был про использование JavaScript для экранирования символов прямо на странице.


professor, poderia dar uma olhada agora


:p


Would it be simpler to just call `_$tracker.addClass('fade-out');`? If `_$tracker` already has the class, nothing happens.


I'm not sure what value this provides beyond MenuHeader.


I see. could you switch this to use `iter_docs` and a generator so that at least we don't load every user into memory at once?


develop (DavidGoldwasser) - x86_64-MacOS-10.9-clang: Tests Failed

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


Pushing up a fix, I think you were on the right track as I added the animation to the icon element itself rather than the span that was containing it. derp.


whitelist will be the same with blacklist right?


@AMHOL nothing fancy, probably something that re-uses `Options` API:

``` ruby
class SomeAdapter::Commands::Create < ROM::Commands::Create
  # under-the-hood option :something, reader: true, default: -> command { command.some_container_api_here[:something] }
  inject :something
end
```


@RandomAltThingy Alright, branch created. I meant "sure thing" as in yes, i'll give you credit. B)


Doesn't need to be handed in this PR, but we seem to need this code semi-regularly. Should it be added to `global.js`?


... да, но ведь страница может содержать JavaScript , который будет выполняться в браузере пользователя. Или я что-то неправильно понял?


Is it DOM thing?


**Ultra-minor:** space after colon and uppercase hexa values. (``background: #E7E7E7;``).


This a mistake?


Consider renaming to Library menu.


Something is starting:
https://core.trac.wordpress.org/ticket/25239#comment:15


OK


Yep ok, I'll look at doing something.


### Use '===' to compare with 'null'.

```
if (bla == null) return "Hello, world!"
        ^
```


We can pass `--logging-filter=-PIL.Image` to nose to still suppress them.


Careful with IDs! You should never have more than one ID per page.


It also works well for languages like NetKAT where we have sound and
complete equational axioms.

-N

On Fri, Sep 11, 2015 at 1:30 PM, Craig Riecke <notifications@github.com>
wrote:

> Yeah, I try to add unit tests when I have spare cycles. I'm not really
> convinced of Quickcheck's value ... it's good when you have a function and
> its inverse and they share no code whatsoever, but that's a pretty small
> domain. Maybe someone else has a different perspective.
>
> If you have policy files that you used to catch this error, you could just
> encode them as plain ol' pa_ounit unit tests.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/frenetic-lang/frenetic/commit/2c3eefd14c9fcc3fa2256f4b2d6b1cb8ccc4f20c#commitcomment-13187969>
> .
>


i think you should allow setOperand(0, null)


there is a list of reserved schema attributes - we need to add selectAll to it


Could there be a case where the URL already contains a `?`?


@nicky008  I found there are `"//": "xxxx"` fields in `jp` translation, are they being used or is there a plan of using them? For the quest description, I'm not sure whether I should translate the text literally or explain in detail what to do to complete the quest. (looks like `"//"` fields are for the latter purpose)


Photovoltaics (macumber) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.67: OK (0 of 0 tests passed)

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


Here, the whitelist and blacklist gives a list of regular expressions.

If a method is accepted by a regex on the whitelist, it is logged.

But if a method is also accepted by a regex on the blacklist, it is not
logged.  If it is matched by something on the blacklist *and* the
whitelist, it is not logged (blacklist is preferred).

The whitelist tells things to keep, the blacklist tells things to delete.

Right now this is hardcoded in redexer, but I plan to make this
configurable by a JSON file. Are you currently using this functionality? If
so, I'll implement it right away.

Kris

On Fri, Sep 11, 2015 at 1:46 PM, eldr4d <notifications@github.com> wrote:

> whitelist will be the same with blacklist right?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/plum-umd/redexer/commit/cdc14ce2421def08523d40c49e101aff3b907f11#commitcomment-13188259>
> .
>


Class-based view handlers are vastly preferred over function based.

You can extend rest framework's views.APIView to get a generic non-model based view handler.

See: https://github.com/DDMAL/DjangoDemoApplication/blob/master/codekeeper/views/search.py
And: http://www.django-rest-framework.org/api-guide/generic-views/#api-reference


Linking to my [previous comment](https://github.com/Fer223/startup/commit/1acb358e185f63d9f886a158626d3afa9b6f558d#commitcomment-13188298): avoid using IDs when is not necessary. In this case, ``box`` element is something you're using more than once, so this should be a class instead an ID.


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 589](https://teamcity.labkey.org/viewLog.html?buildId=271560&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 31, passed: 9 Build time: 00:24:14
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyAntigenScatterPlot: <no details avaliable>

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

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

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

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

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

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

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

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

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

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

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


@justin808 I rebased and squashed in the PR.  Thanks!


add note that scope may be null, and that value is null iff this is a 'reset' command


delete this line


Hmmm.... I'm not running Networkb in a batchfile nor do I have  the .1 (rushnet) in my mail.bat which is processing wwivnet, but I seem to be getting rushnet email and I see networkb output scrolling by in my bbs -w window.

Did you implement it and forget? ;-)


Jeff Squyres, le Fri 11 Sep 2015 10:26:36 -0700, a écrit :
> On Sep 11, 2015, at 12:46 PM, Samuel Thibault <notifications@github.com> wrote:
> > AIUI, X_LIBS is not meant to contain -lX11.
>
> Ah. What's it supposed to be, then? I.e., is what Bric^H^H^H^HI did by
> hard-coding "-lX11" actually the Right Thing to do?

AIUI, yes, the order is supposed to be

$(X_PRE_LIBS) $(X_LIBS) -lX11 $(X_EXTRA_LIBS)

because X_PRE_LIBS contains libraries which must be linked before X11,
X_LIBS contains -L or -R flags for -lX11 to work, and X_EXTRA_LIBS
contains additional libs that might be needed for -lX11.

Samuel


well my entire private repo's where stolen. but all have been sorted out now, all stolen copies that i know of and the one who stole them have removed them. Thats why i asked you if you got any clones made. but as you said, you got it this commit from cm remix.
can you link me to them btw?

You can keep on-the-go thats no problem.

Thanks for your response and understanding


you must delete this line


Good call :+1:


is name always a SqlIdentifier? if so make it a SqlIdentifier.


Remove this - unused


What is the meaning of this "Stop" message and why is it still enabled in the test code?


**Minor:** since we're using fixed ``width`` and ``height``, maybe using percentages for margin is not exactly the best way to proceed (your main container size is not in percentages, it'll not change, so margin should be treated the same in this case).


This is not meant to be public API. What is the problem without LIBFREENECT2_API?

`logger.h` is the public header. `logging.h` will be moved to internal directory.


Good catch.


Good suggestion, will save it for another PR :+1:


GroundDomainUpdate (mmatts) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1314 of 1318 tests passed)

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


I think i can squash this commit with the previous one in your PR.


This may be a good opportunity to use ViewSets and Routers:

http://www.django-rest-framework.org/tutorial/6-viewsets-and-routers/


You can define ``margin-left`` value on line 22 like this: ``margin: 5% auto 0 10%;`` and delete this line.


@tchaikov Yes, I'm annoyed that the logic after we fall out of the loop is so similar logic inside of it.  The problem is really that we are comparing 2 lists sort of.  The main loop is on ``objmap.objects``.  But the comparison is with multiple lists called ``snapset.clones``.  However, there are multiple clone lists and in between lists we expect ``head`` or ``snapdir``.

object list               expected
=======               =======
obj1 snap 1           head/snapdir, unexpected clone
obj2 head              head/snapdir SnapSet clones 6 4 2
obj2 snap 7           obj2 snap 6, unexpected clone
obj2 snap 6           obj2 snap 6
obj2 snap 4           obj2 snap 4
obj3 head              obj2 snap 2 (expected), Snapset clones 3 1
obj3 snap 3           obj3 snap 3 ok
obj3 snap 1           obj3 snap 1 ok
obj4 head              head/snapdir Snapset clones 4
<EOL>                  obj4 snap 4 (expected)


I don't think so in this case, but I'll add a check for good measure here anyway.


As a general comment, keep an eye on indentation (also remember adding a space after colons).


ZoneHVACBaseboardRadiantConvectiveElectric-UI (kbenne) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.67: Build Failed

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


I'd like to see a negative test in here, since the problem the tracker story is trying to address is "false positives" where module loading was failing but we weren't detecting it in tests.


*Minor:** ``Helvetica`` is not a web safe font. This should probably be defined as ``font-family: Arial, Helvetica, sans-serif`` (where ``Helvetica`` will be loaded only if the user has that font installed). Further info [here](http://www.w3schools.com/cssref/css_websafe_fonts.asp).


I was thinking about using it in our project in order to log only the calls that we want. I can play with the current version right now so no rush :)


Samuel Thibault, le Fri 11 Sep 2015 19:51:26 +0200, a écrit :
> AIUI, yes, the order is supposed to be
>
> $(X_PRE_LIBS) $(X_LIBS) -lX11 $(X_EXTRA_LIBS)

Or put another way, the right way to detect X11 is to put
$(X_EXTRA_LIBS) in the OTHER-LIBRARIES parameter of AC_CHECK_LIB after
having put X_PRE_LIBS and X_LIBS in LIBS.

Samuel


Okay, I'm going to implement this today: it shouldn't take more than a few
minutes.

My plan is simply to turn a configuration file we have now into a JSON
based config file which specifies a white/blacklist.

Kris

On Fri, Sep 11, 2015 at 1:56 PM, eldr4d <notifications@github.com> wrote:

> I was thinking about using it in our project in order to log only the
> calls that we want. I can play with the current version right now so no
> rush :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/plum-umd/redexer/commit/cdc14ce2421def08523d40c49e101aff3b907f11#commitcomment-13188450>
> .
>


:smile: :+1:


Agreed, waiting on others to agree on publishing the native collectd install documentation that I wrote.


What is this for? Please document.


Agreed.


bày đặt, thầy cho chấm chéo chứ có đọc code của chú đâu mà lo nghĩ nhỉ :v


Where does this value come from? It should probably be the alignment grain size or similar.


This is not needed.

See https://isocpp.org/wiki/faq/freestore-mgmt#delete-handles-null


Grazie per aver tolto le ridondanze! :+1:


I'm glad you like it. It still needs some work to make it public.


dehumidifier-fixup (kbenne) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.67: Build Failed

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


Look good from here.


Why don't you convert this partial into a component that can handle also search results? So in _searchResult you can reuse it instead of doing the same thing in two different places.


J'avais fait cette partie dans l'optique de définir les trucs nécessaire à installer sur la machine pour pouvoir lancer l'application. Du coup tsd il rentre pas dedans si ?


Hello,
I see you pulled a commit from a stolen private repo of mine. I wanted to ask where you pulled this commit from and if you made any clones of these stolen private repo's.
The one who stole my private repo's already removed them.
Also the ones i know of, who made a clone removed them to.
So if you did made any clones then please remove them. i dont want another drama to get this repo's removed again to. hope you can repect that.

Best regards,
BlackDragon


Is there a reason this doesn't need to account for alignment?It is my understanding that on the most special platform (mainframe) all partitions must be start and end aligned.


Number that is divisible by 3 display Fizz, number that is divisible by 5 display Buzz, number that is divisible by both display FizzBuzz else display the number.


:+1: :+1: :+1: :+1: :+1:
beautiful :D


tom pls


@chelm Ok so I guess the the question for me is: should `lib/FeatureServices` be attempting to apply a geometry filter at all? Why have that check there if `req.query.geometry` is being removed?


@Sloria -- did we ever decide where we want to wrap our scripts?


It should still refuse to resize non-leaf partitions that are not extended partitions, since we have no code to handle things like resizing a PV that is already part of a VG.


Significant gaps in testing: update(byte[]), update(long[]), union(HllSketch), toByteArray(), toByteArrayNoPreamble(), asCompact(), getUpdateCallback(), getEstimate() returns not fully tested, getLinearEstimate() returns not fully tested.


In general, try to define reusable names (not tied to something too specific) but also not too vague (can't relate to something quickly). For example, this class could be called: ``.overlay-container``.


Nice catch, I've updated the calculation in a new commit.


Ce `newline` est en trop, non ?


Can you add

>When exit libfreenect2::CpuDepthPacketProcessor::process is called but listener_ pointer is NULL. Adding checking to listener_.
> First time deleting not alloced mem pointer buffer_ will fail.When create Mat buffer_ set NULL and checking when deallocate()

to the commit message?


`newline` en trop ?


`newline` en trop : tu fusionne deux couplets distincts, non ?


Why not using c.ExpDatabaseStorage?


Take out the space here (I'm picky)


da isches kei division?


Base_static and static client need to be removed


What's the output you get from `docker ps --no-trunc`? We expect that some containers have several names and split on the comma for this reason. I didn't test, so I may be mistaken.


Yeah, the check bad, thanks for the report. Here's a fix: #543


develop (evanweaver) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.67: Build Failed

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


return null isch en bad practise, besser ischs en exception zschmeisse. ich gseh aber da nonig so dure, darum machts ja vilicht au sinn da.


Yes, see https://github.com/kubernetes/kubernetes/blob/master/docs/devel/api-conventions.md#lists-of-named-subobjects-preferred-over-maps for some context.

Copying over my irc reply in case you got disconnected:
 i mean maps are verbotem in our api, everything should be a list (see api conventions doc). So instead of a map where we have /foo: serviceRef, we have a list .. something like {pathref{domain: /foo, service: serviceref}}


Samuel Thibault, le Fri 11 Sep 2015 19:57:10 +0200, a écrit :
> Samuel Thibault, le Fri 11 Sep 2015 19:51:26 +0200, a écrit :
> > AIUI, yes, the order is supposed to be
> >
> > $(X_PRE_LIBS) $(X_LIBS) -lX11 $(X_EXTRA_LIBS)
>
> Or put another way, the right way to detect X11 is to put
> $(X_EXTRA_LIBS) in the OTHER-LIBRARIES parameter of AC_CHECK_LIB after
> having put X_PRE_LIBS and X_LIBS in LIBS.

Ah, no, not even, since $(LIBS) will be put after -lX11. What I'm
confident on is the final link order

$(X_PRE_LIBS) $(X_LIBS) -lX11 $(X_EXTRA_LIBS)
Perhaps we just don't need to run AC_CHECK_LIB at all since once
AC_PATH_XTRA is used, no_x is supposed to already tell us whether libX11
is available or not (it already does the link test), and then we can
just do

HWLOC_X11_LIBS="$(X_PRE_LIBS) $(X_LIBS) -lX11 $(X_EXTRA_LIBS)"

accordingly.

Samuel


Consider renaming to AccountMenus (since there is >1 menu).


champion_id needs to be removed from the users model. A user doesn't own a champion so those are separate objects.


Awesome. Thanks Henry and welcome back!


1 observation.

For example, in the `CreateAController`, line 47. You're just creating a new instance of that animal but you are not really adding it anywhere. What's the point? Why do that?


Work related to #1


Hopefully this is now fixed with the change that closed #199.


re PPA: all our depends must be in the PPA, since we need them there to build kodi itself in the ppa.
If there is a package thats trustworthy and works, I can just copy it over. Everything else must be packaged, so I'd prefer to only to this once you decided for a (hopefully 1) lib.

$(CMAKE) gets set by unified depends. For manual building you can test if its empty and set "cmake" in that case. Just like we do with e.g. $(PREFIX) and friends.


Then what happens in case options.resolved is set and there is another conflict?


Duplicate line?


ct une erreur ils on été delete juste apres


Oh FYI this doesn't even have working components in it, don't bother trying :stuck_out_tongue:


These headers break FFmpeg building with dxva2 and d3d11va.
The first patch seems to still be needed.


@luksa : It doesn't look like we can remove the deprecated '*-ds.xml' datasource files without requiring too much effort by the users.

If we want to merge this option, we need to provide instructions in the README file that show how to use this feature. Can you update the file with those instructions?


This looks unrelated.


Does this (massively) improve performance? Is this game dependent?

10x sounds impressive....


Not sure why this one looks off, but added the following:
https://github.com/vbanuelos/nanoGALLERY/blob/6c490d2710c8b60e68b6a51490301efcbde5dbcf/jquery.nanogallery.js#L4144-L4173


Should I see this message logged with this catalog?

```
exec {"\u006E\u0303":
  command => "/usr/bin/uptime"
}

exec {"\u00F1":
  command => "/usr/bin/date"
}
```

not seeing it for some reason.


On 11.9.15 19:04, Michael Boelen wrote:
> Commit request is unclear. It doesn't change anything. Is your tree outdated?

Hmm, I did a pull request like I did before.  The change actually is
just a move of a block of lines and Github shows it to me as such when I
compare to the master branch.

    Ciao, Lobo


Missing javadoc for this class.


For the scenario where $slugs is empty, do we really need to hit Elasticsearch whe that is the only condition in the query? I wonder if you can count the results here and return an empty pager, then check $pager->hasResults() or something like that?


Target blank :stuck_out_tongue:


Some of the errors are fixed, but still autocomplete width is too small :/ I'm getting trimmed text with "..." at the end


Yeah, the check is bad, thanks for the report. Here's a fix: #543


why do those have a dependency to cpluff?


Yes, I'll remove it actually. I needed to try it first with travis. I'll try to isolate the problems and adjust travis soon


@sirLisko there must have been a github bug. thanks :)


Ok pour les chiffres 0 et 1. Mais par contre, `songs` n'autorise pas les nombres : sauf erreur de ma part, il ne sait pas faire la différence entre `1 2` et `12`. Que faire ?


iteration (evanweaver) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.67: Build Failed

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


Are we able to import middleware from `legit-tests`? Maybe something like:

```
import {SetState, Find} from 'legit-tests/middleware';
```


Is there a reason this find has to be in the view? Is there a way to get it out?


4385-CondenserLoop (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Done in 256489036b8f1d4c5a57f3c614a18b6fd7e13430.

I also cleaned up tcpdump to refer to `pcap_wsockinit()` rather than `wsockinit()`.


4385-CondenserLoop (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1857 of 1857 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)


Mk... no ejecutaste los comandos que te mandé ayer en la mañana para ignorar ese archivo de netbeans...


por qué int
se puede cambiar a int32_t?


This needs a javadoc even though it is package private.  How is the developer who wants to extend it use it?


Doesn't this become a range, ever-increasing? /cc: @kenichi


Needs a javadoc.


Not sure if @fiver-watson is going to like this, the user probably wants to see the title of the description as usually. Here you have another place where you could ask ES for a number of descriptions given an array of slugs, like in UserClipboardAction. Maybe you can abstract that away in QubitClipboard or a new QubitClipboardSearch class.


@ngoldman valid question for sure. It is legacy, from before the time we passed filters down to the cache. It would be okay to just remove all support for `req.query.where` & `req.query.geometry` from `lib/Query.js`.

The only issue would be that all caches would be expected to support where based AND geometry based filtering. Every koop cache would have to have geometry capabilities, else FeatureService support would be borked.

Is there ever a time when we want to have FeatureService support w/o having a cache? This goes back to the question of whether or not provider are forced to cache or not. If they are not forced to use a cache then something, somewhere needs to respect `req.query.geometry`... Make sense?


Esto ya lo tenemos


Shouldn't the correctness of the generated code take precedence over how readable the code is?


acá también int32_t


I see the panel pretty much independend from what is happening with the content. The navigation bar icons were never aligned with the content except for the very left side (which is obvious if the content is left aligned).
Anyway, i am thinking more about what is happening when you scroll further down a little or you have a site with no widgets in the left or right aside. Than you see a top bar with 100% width and slightly out of center content below which looks wired to me.
Have a look to /directory /public /calendar or /photos (if there are many) to see what i mean.


Awesome, I think it's working.


Ya lo tenemos


@fplanque this subproject is driving my local branches crazy (well, just a bit :D). I'm not sure if you committed it by mistake or as a temporary thing.

Just to confirm, should we keep this plugin in the main repo?


removegitattributes (kbenne) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.67: Build Failed

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


No hace falta


Please fix the folder spelling on this commit.


The HllSketchBuilder class needs javadocs.  The copy() constructor is not tested.


Y esto??? Quitalo!!


Oui, il rentre dans les pré-requis, j'ai pas trouvé le moyen d'utiliser celui qui vient avec npm. Ça a pas l'air standard comme usage.


Consider renaming to Menu->VerticalMenu and MenuNav->HorizontalMenu or something like that.


New things are in fact not that great


Esto fuera!!


develop (evanweaver) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2308 of 2342 tests passed)

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


Es un notificado, quitalo.


Can we do without a `U getCriteria()` in this interface?


101884090--NewVenting-Availability-Schedule (jasondegraw) - i386-Windows-7-VisualStudio-12: OK (1857 of 1857 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)


Et je pense qu'il en manque.


adding this is a bc break


@chelm I see what you mean. My goal at the internal implementation level is to treat req as static and never modify it (so no deleting properties, which is currently happening all over the place). I'd like to support the case of not using a cache too.

@dmfenton thoughts?


I wonder if you're making too many assumptions here with the custom model?  As shown in the example, this will fail, without passing through the opts to the g5Model constructor as well.  If the intent is for inheritance, at least as illustrated, g5Model will need opts.


Need javadoc on the class.


Olvidate del neon, el lo optimiza para sabermod , yo uso uber toolchain.


Why only these specific generated? index.html, for instance, points to a shitload of other htmls.


Need javadoc on class.


So 2013-2015 instead? Yeah that's probably more accurate.


Needs Javadoc.


I guess it's the doxygen_resources one that point to everything?


@sanbornm I don't see how the correctness of the code is affected by not having this change. Do you have cases where the generated code doesn't compile?


This exception is not tested.


Alright looks good


what does the name of the commit mean ?


I posted a tiny PR. I happened to be assessing `Angelo` and noticed.


For me it only looks correct if I set 	```pos += g.X```
If not the label will overflow the border if it is aligned right.


What about mechanism for koop to central an central flag when a cache is being used to filter data. A provider could override that flag when its not using the cache for filtering and the FeatureService code would be smart enough to respect the current state of said flag. I guess something at the request level would need to be set, but middleware may be good for this.


you already figured out how to solve the .Update() repetition, but for future reference for other projects you can also do this

```
init
{
	vars.watchers = new MemoryWatcherList();
	watchers.Add(new MemoryWatcher<int>(new DeepPointer(0x9001) { Name = "Blah" }));
	watchers.Add(new StringWatcher(new DeepPointer(0x9001) { Name = "Blah2" }, 256));
}

update
{
	vars.watchers.UpdateAll(game);

	print((int)vars.watchers["Blah"].Old + " -> " + (int)vars.watchers["Blah"].Current);
}
```


0.1% chances for Orchestra Platform user to completely swap the foundation class at the moment.


The parameter fields is hiding a field from type OnHeapHash.


What an awesome project!


Needs javadoc.


@HostingItAll This doesn't fix anything. this is the php7 branch, nothing else. Also seems like you did not do `git sibmodule update --init`


@Javran @nicky008
At least currently it's not being used by quest translation overlay.
`$(".name", QuestBox).text( QuestData.meta().name );`
`$(".desc", QuestBox).text( QuestData.meta().desc );`

Not sure why Nicky added it at the first place but adding tooltip to a quest alongside with translation might be a good idea.


they include the generated https://github.com/FernetMenta/xbmc/blob/master/lib/cpluff/libcpluff/cpluffdef.h.in through a multi level include chain.


Eclipse configuration is also provided :-)


I grok you aimed for the simplest solution, but this shouldn't break abstractions.  Additionally, this *will* fail if there is a read only cache in use.

offhand, you likely can get away with moving this into pkgcore.operations.repo.Operations._cmd_api_regen_cache .  We *might* want it as a separate repo operation/method- which regen in turn calls.

Either way, move this code into there, address the read only bit, and it'll be in the "simplest/good enough" bit- we can parallelize it longer term if we care.


Needs javadoc


Oublié de save le fichier avant, donc fichier vide.
La réponse se trouve donc dans le second commit.


@krlmlr yes, it is related, I'll create a PR


Cool, sounds good, been messing about with plugins lately and I think they leave a lot to be desired, so that should be a step in the right direction


I know what you mean, but I find that a lot less out of place as the navigation bar. But this has all to do with how we both see the navigation bar. I see it as part of the page, so expect it to be aligned in the center to. But don't bother, it is not the end of the world for me ;-)


:+1: This is great! I never really got to invest much time into `Base.Test` as I would have liked. Super excited about this!


no point in having contracts if you're going to break them


:dancer:


Just as info, will this be iintroduced in wotlk soon?


Are these automatically generated?  I don't think anonymous users should have access to comments.


Might work, need to test that.


It's just a little function, but too complicate for you, maybe ou'll understand it in 1 or 2 years ;)


:a: :ab: :abc: :abcd: :accept: :b: :baby: :baby_bottle:


it shouldn't matter if 0.0001% are custom, it shouldn't get broken


One small comment: even though `string` is not a reserved word in JavaScript, people tend to not use it as the name of a variable because it represents a js concept. Perhaps `str` or `inputString` would be better here...


develop (evanweaver) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


content admin and event admin should have administration rights over comments.


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 590](https://teamcity.labkey.org/viewLog.html?buildId=271558&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Snapshot dependency failed to start: LabkeyTrunk :: build Build time: 00:00:00


@gpopovic Definitely let me know if this caused issues for the standard autocomplete and I will investigate.


:a: :b: :cactus: :dancer: :e-mail: :facepunch: :game_die: :haircut: :ice_cream: :jack_o_lantern: :key: :large_blue_circle: :m: :nail_care: :o: :package: :question: :rabbit: :sa: :tada: :u5272: :v: :walking: :x: :yellow_heart: :zap:


content admin and event admin should have permission to create event content.


I'd rather focus on [this](https://github.com/balderdashy/waterline/pull/981) as a solution if you have the time. Adding types here one by one and implementing polyfills in all the adapters isn't scaleable.


Awesome, thanks!


:+1:  same comment for the `string` argument name :)


authenticated users should not be able to skip comment approval.


:point_right: :point_right: :point_right: LOOK AT THIS :point_left: :point_left: :point_left:


@mobiware I know this was ~2 years ago - do you remember why doKey is wrapped in a 2000MS delay?


The exception is not tested.


Nice! I remember trying this long ago and not working, good to know it works now!


Just watch your whitespace, I'll go ahead and fix this real quick.


The indexing should start at zero.


inorite :) we're gonna get there soon :)


:+1: What happens if the inputs are not numbers?


commit after entry, test time


Are you by any chance using Orchestra Platform for any project right now?


change this message please


What's the revision this is based on to build? Shouldn't we ditch the chrpath dep, now that's not required anymore?

I can do the -bin package to on RPM, but what about making -devel require it? That would make things saner.

I also have to deal with ducktape, my next step.

Will you consider splitting the modules like I did? It's a PITA, but it turned out neat for possible users.


Nice catch!  Fixed in 8e0055b00f3cab71a237ee5bf9cf6b8af725fa5e


Offhand, I feel like we could probably do a with block here and use a temporary named file, which we then tell the rsync_syncer to work on.

Doing so would allow us to automatically clean up the file on the way out without having to do try/finally blocks.


before

```
turf-line-distance#simple x 945,342 ops/sec ±1.93% (93 runs sampled)
turf-line-distance#200 mi x 1,077 ops/sec ±1.20% (94 runs sampled)
turf-line-distance#700 km x 1,212 ops/sec ±3.60% (90 runs sampled)
```
after
```
turf-line-distance#simple x 1,142,493 ops/sec ±0.76% (95 runs sampled)
turf-line-distance#200 mi x 1,255 ops/sec ±1.13% (95 runs sampled)
turf-line-distance#700 km x 1,446 ops/sec ±1.09% (94 runs sampled)
```


A doc would be nice


@agorrod, did you test which of these cases needed to have this change (or if they all needed it)?

So, there are 3 cases here: page being read, page being considered for eviction, forced eviction failed. Diverting a thread to do eviction work was costing performance in some cases, and I didn't tease apart which specific cases were at issue when I made my change -- did you?


Don't mind the name of this commit....


no, but what makes symfony so attractive is the fact that it really cares about BC, and breaks like this stand out as poor design to me, and is something i look to avoid when selecting open source packages to use - sorry to sound like a dick


hehehehe


A doc would look nice.


Looks good. I'll merge in the morning.


The logic is kind of hard to follow here- in fairness, that's more a complaint for what I originated.  The manipulation of self.basedir and self.uri seems to be the source of the pain- perhaps it would be easier to temporarily instantiate a seperate rsync syncer object w/ the custom basedir/uri values then trying to mangle this instance, then revert the mangling?


If your intent is to empty `_rooms`, I'm not sure setting its length is enough. Maybe it will work, but at the very least it will be inefficient for its previous items would not be garbage collected (wiped from memory). So it's probably better to write `_rooms = []`.


The SSL_verify_mode needs to be an option in the constructor that gets passed through to here, and it needs to default to enabled.  While we may support disabling it or even permit that in the cPanel codebase for legacy reasons, it's generally considered a security vulnerability to turn certificate verification off.


Y u no pick the commit which can toggle it


> Ah, ok. Do you have an additional pointer like a URL / git repo ?

Maybe a few :)

* https://twitter.com/tritum/status/636821413782011904
* https://forger.typo3.org/sprint/cig?boardId=FormsNG
* https://review.typo3.org/#/c/42927/


to help solving issue #976


`&s.VolumeConfigFlags.PersistentVolumeRecyclerMinimumTimeoutHostPath`
should be
`&s.VolumeConfigFlags.PersistentVolumeRecyclerIncrementTimeoutHostPath`

and

`s.VolumeConfigFlags.PersistentVolumeRecyclerMinimumTimeoutHostPath`
should be
`s.VolumeConfigFlags.PersistentVolumeRecyclerIncrementTimeoutHostPath`


@topherfangio one thing i notice is flicker ..

<md-item-template>
         {{::item.text()}}
</md-item-template>

Since i can't use ng-bind because of md-highlight ...i often see on the first serach {{::item.text()}} for a really small time


C'est une question d'interprétation du [fichier original](https://github.com/patacrep/patadata/blob/57e7fdf285f07eac588379bb6b9c267f45b6ee12/songs/3_doors_down/here_without_you.sg#L13).
Il s'agit d'un `verse*` indépendant...donc en théorie ça demande un espace supplémentaire, donc le `{newline}`

Je pense qu'on va se retrouver avec pas mal de petites incohérences de ce style, mais qui sont très complexes à résoudre de manière automatisée...


since this commit, one test case is failing: https://github.com/actano/sternchen_reporter/blob/master/test/test.coffee#L104-L122


103122604-ClimaticDataSummaryBug (JasonGlazer) - x86_64-MacOS-10.9-clang: OK (1851 of 1851 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)


Function was very low of the perf chart. So maybe you will get 1 fps if you're lucky. Well I guess it depends on the game. Worst case it saves some cpu cycles to improve the code in others place.


(And of course, maven uses the same settings)


commit after entry, test time2


I double checked and it seems JS engines actually recognize that intent, so it's actually quite clever as it avoid creating a new Array literal (`[]`). It will even trigger garbage collection of the now-deprecated items. So in the end, I'm not sure what's best :) Always learning


Sure was.


Après vérification, ça m'a l'air d'être correct: pas besoin d'incrémenter quoi que ce soit!

`songs` affiche le `basefret` tel quel, donc c'est une question de convention si le manche commence à la case 0 ou 1 (si `1` est affiché, c'est que l'on est décalé d'une case dans patacrep - et non pas que la case "de base" est la première)

Si on veut changer de convention je peux l'implémenter facilement dans le script (pour éviter de devoir bidouiller `songs`)


since this commit the reaming two test cases are failing (all are failing until the current commit)


Found a much better way to do this.  See 66cf1c2de73e87801aac9bf5fe0f8dc809c44e3c


Symfony also used to have `@api` to mark stable and none-BC code. Yes, you are being a dick for comparing us with such big project, and remember Laravel change contract on patch version too, in 5.1.x if not mistaken. So look into that.


Is this empty intentionally?  If so, a short code comment would help.


Do you need the `prop_transform:` line?


LGTM


@mdmower You're right, I'm gonna revert this, only reason I did change it was because some users on XDA were complaining about having to go back to stock, updating, and re-flashing but I do agree entirely that this commit was not the best way to go about avoiding those complaints.


I think this would be easier to read if we handle the pathInfo.startsWith("/oauth") case before the pathInfo.startsWith("/form") case, then use the if - else if - else if - else pattern for seeing what pathInfo starts with. That way we don't need to check "&& FormAuthHelper.authRedirect().isEmpty()" in the /form case


Yes you can do that :D it's exactly like that!
On Fri, Sep 11, 2015 at 11:11 Jason Kraus <notifications@github.com> wrote:

> Are we able to import middleware from legit-tests? Maybe something like:
>
> import {SetState, Find} from 'legit-tests/middleware';
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Legitcode/tests/commit/a1c61926a49791faffd9c8045425ddd24966238a#commitcomment-13188774>
> .
>


:+1:

Tiny comment: even though `string` is not a reserved word in JS, we tend to shy away from it as a variable name because it could be confusing. Perhaps simply `str` or `inputString` would do here.


Nosotros tenemos la v2, no hace falta, además lo he añadido hace poco.


Unhandled exception has occurred in your application. If you click Continue, the application will ignore this error and attempt to continue. If you click Quit, the application will close immediately.

startIndex cannot be larger than length of string.
Parameter name: startIndex.

![chrysanthemum](https://cloud.githubusercontent.com/assets/11824179/9823324/d195e452-58ef-11e5-8f0e-428d945969f8.jpg)


This will be further extended to inlcude the detection of multiple simultaneous key presses and in the end to replace the Control Cluster from the example VIs.


This function does not return anything. It simply logs to the console.


interesting. why and where do they include this? AE and player code have no dependency to cpluff


Will implement.


So I just got around to looking into this and I'm unable to reproduce on my forked repo or on master. I cleaned up a little and I'm pushing a re-enabled version. Maybe something else fixed it.


`\d{2}` implementé: ae50c7bc283d106bbdee3db7bc36f4fc576a5886


The local variable cb is hiding a field from type OnHeapHashFieldsTest


Work related to #1


I took this out as it's giving away too much.


I think it would be OK (and simpler) to just build the arm64 modules unconditionally by adding arm64 to GRUB_PLATFORMS in coreos-overlay/profiles/coreos/base/make.defaults.

Would there be any problem in doing it that way?


We shall remember this day for the rest of our lives.
Hail Raptorch!


You can try to give the panel (icons only of course) the same width as content width by adding
.container-fluid to
main {
	margin-left: auto;
	margin-right: auto;
	max-width: $main_widthpx;
}
i am not sure if i like that better though...


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


2012?!? pre-swift?


81211464-LinkInfiltrationAndReturnFlows (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1865 of 1866 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)


We tend to shy away from using `string` as variable or argument name because, even though it's not a reserved word, it's confusing. Choose `str` or `inputStr` or something like that to avoid any confusion.


The scripts in xivo-utils have not been designed to work in a distributed environment. If we want to use this particular script in a distributed environment, then we should rework xivo-utils from the ground up.


Glad I wasn't the only one wondering that.


haha!


best reviewed as https://github.com/JuliaLang/julia/commit/b44700ba85c58dc9adf0f9b149e658e28c195a49?w=1


This is exactly what this model is meant to be.

Are you saying:

1. Remove the 'title' field and add an 'updated' field.
2. Do this, but without models (modify database directly).


Good job on this.  Was just going to write it myself when I saw you had saved me the trouble.  Thanks!


For readability:

```
amethyst (req_root)    17874.15 (± 150.41)  24.81× slower
 artanis (req_root)   443421.31 (± 5738.80)        fastest
   beryl (req_root)   128771.06 (± 9765.26)  3.44× slower
amethyst (req_id)      15667.19 (± 364.56)  16.22× slower
 artanis (req_id)     254103.97 (± 3104.89)        fastest
   beryl (req_id)      69536.91 (± 1022.40)  3.65× slower
amethyst (req_middle)  13556.67 (± 107.90)   4.52× slower
 artanis (req_middle)  61224.30 (± 411.86)         fastest
   beryl (req_middle)   4180.92 (± 32.31)   14.64× slower
amethyst (req_bottom)   9925.64 (± 88.15)    6.90× slower
 artanis (req_bottom)  68454.69 (± 659.62)         fastest
   beryl (req_bottom)   2847.93 (± 19.47)   24.04× slower
```


4385-CondenserLoop (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (789 of 789 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.12%25-green.svg)


Yep!


:+1: for the function, but adding strings should not return `NaN`... Can you please verify?


@topherfangio i think i found a weird bug. When "item" in ng-repeat is not called item..search items are not displayed
http://codepen.io/anon/pen/gaajNM


![boton_me_gusta svg](https://cloud.githubusercontent.com/assets/1192958/9823450/f3968456-58ce-11e5-91bd-a8792ed3a769.png)


Looks OK


https://github.com/Legitcode/tests/wiki/Middleware @kenwheeler is making one right now for simulating events


In general I think having these available could be useful, even if only for development use.


develop (kbenne) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


No, we do not require 2.27.1, the change in .1 is only optimization (2.27 checks for mtab in libmount monitor as well as in systemd,  it's redundant, but not mistake).


I would re-add this solid collision and add another with the same texture for a non collision light, because you can do some pretty awesome stuff with solid light blocks when it comes to building.


done


:+1: for the use of `switch` and the re-use of previous functions! Please note that your `break` statements are not necessary here, because using `return` already breaks out of the `switch` :)


:cry:


I know that `_tryToFillStatusBar ` invoked on post stage of linking with really epic hack. I think my hack should be marked as a hack of the same problem – finding a right place for filling a status bar.


commit after entry, test time3


@c4milo Maybe correctness is not the right terminology as it does compile.  go fmt just can't align the struct types producing what I would consider non idiomatic go code, something gowsdl strives to achieve.  If you were to write the code manually that gowsdl generates you wouldn't have those spaces.


That's a pretty good idea. It would make this view less cluttered as well. :+1


It'll be shipped in puppet-agent version 1.2.4.


commit after entry, test time4


I see... but without getopts none of the vars are checked...


Links are in https://tickets.puppetlabs.com/browse/FACT-1205.


We can update when we push packages.


That's an excellent point Andrew. I'll move everything past 'making sure there's a remote_url in the POST' into an asynchronous job.


I thought it was the default? isn't it?


Since this is not (yet) released to pyPi, do we need some additional text for the version, like the `-odl` used elsewhere?


:+1: for the code. I'd like to give you a warning concerning your `for` loop. After the closing parenthesis of the `for`, you are allowed to pass one and only one thing called a **statement**. In this case your statement is:

```javascript
o += s[i];
```
If you needed to have more than one stateament execute in your `for`, you would have to surround those statements with a block `{...}`. If you write one-statement `for` loops without blocks, then you leave yourself open to future errors. Say you need to add a second statement to your `for` loop: you might add it without using a block, and then the statement will not be part of the for loop! My suggestion to you would be to **always** use blocks after a `for` statement, even if you only have one command in there.


Sorry that your work was taken, thanks for the awesome feature.


This broke the serial usb com port detection on windows 10 !
That was the reason for the change, i will combine all, and do a compare for found ports,
this will be slower


Please always use the `===` operator for testing equality. The `==` is called lenient equality and has some weird rules about it. I would suggest never using it, even though it is part of the language.


Required for Windows 10


Lawyers notwithstanding, we need to find some wording that developers will actually understand. Wonder if we can hone in on something that's clear but avoid the one combination that's objectionable. "two taps in quick succession", "two consecutive taps", "double-taps" (without "gesture" afterwards), other?


see next commit for fix


@TheGreyGhost Heya! Could'nt really find a way to contact you so hope you read this.

I was wondering (and really really hoping) if you might have a version of MinecraftByExample for 1.7.10?


Hey this is a really change


yes, looks like a merge error


@clois take a look to this new version


![Uploading QQ截图20150912013934.png…]()


@zhenzhu1234567 take a look to this new version


hahahaha ok


yes, I agree it is not idiomatic. I actually had it more idiomatic before but changing the templates turned out to be more difficult so I backed up. I also wanted to point out that this change will comment properties if they come with comments.

If you still believe that the tradeoff is fair I will merge it. However, we need to address first the issue with comments and create another issue to remind us to correct the template once https://github.com/golang/go/commit/e6ee26a03b79d0e8b658463bdb29349ca68e1460 is released on February next year.


Kannst du nem Laien erklären, wie das funktioniert?


Nice.


@HannahMitt  @gabrielfalcao Indeed awesome project.


Once again, nice. Wording is important.


Cool.


Legit :)


Why did you add that?


Since we're just tracking Manifests (users will be providing manifests) we really only need a URL to the manifest (so we can re-index) and minimal metadata to know when it was added or updated (just because that might be useful sometime). That's why I suggested it be called "Manifest."

"Document.title" might be confused with the actual document title which will be held in Solr.


@caw13 another test


There is no dedicated function in standard library to achieve this. You can always write your own function **GenerateRange(begin, end, step)** that will generate required vector.

In cases when you know how many elements you want to generate you can use **std::generate**. It allows you to fill the array using any function (including those that have internal state).
Suppose you want a modified version of **std::iota** with specified step:
```cpp
 std::vector<int> v(10);
 int A = -2;
 int step = 3;
 std::generate(v.begin(), v.end(), [&A, step] {
     A += step;
     return A;
 });
 // will fill v with [1, 4, 7, 10, ..., 28]
```


Thanks. I had been longing for this.


logging site -> log in page


space after if


Please put the full comment from the patch here.  It tells why we are doing this.


handle_system_error() doesn't have a return value, I don't think you want to return its result here (and in similar statements below).


@graysky2,
what you mean? i can't understand what you mean, what check must do?
How getopts check var?

As i understand, your getopts:
1. Create string = argv
2. Parse it
3. Set value, what placed after specified key, to specified variable
4. Do something

My hack:
1. Make string with var_name=value
2. Init it and check it, by declare (you can't run this with: sudo psd-helper rm -Rfv /, you got a error)
3. Do something

What kind of check you mean, and where they must be placed?

Any way, you can just use only this patch, if you don't like hacks, you solutions also work good, and i don't see something "wrong" in it.
You can just merge/use only this patch, because it's drop if-elif
https://github.com/Nefelim4ag/profile-sync-daemon/commit/9be21acd6b0e8702ecaf2ef21bd6c46a5832a01b


:+1: Please use something other than `string` as your argument name because it could be confusing.


Indentation.


Before doing this, could you please check if my PR https://github.com/leipert/step-git-push/pull/8 solves that issue, too?


Let's check for stereo and either set the number of channels or fail. Also make libsndfile optional and skip the new code if it's missing.


To add a command, copy this file and change everything to do with the command you wanna add.


I will add the sender instaceof Player


In general I think having these available could be useful.


Typo here.


Minor miscapitalization here.


FACT-910 is part of FACT-927, can probably leave it out of the release notes.


Du hast schon richtig die Navbar mit der Klasse navbar-collapse angelegt. Es fehlte eben nur noch die Interaktion, um die dann fehlenden Menüpunkte wieder anzuzeigen. Grundsätzlich läuft das über den Javascript-Teil http://getbootstrap.com/javascript/#collapse
Ich gehe davon aus, dass am Anfang der DOM (nachdem er geladen wurde) auf data-toggles (gibt da ja bei bootstrap mehrere Kategorien) geparsed und dann der passende Javascript drangehangen wird.


Ahh, I didn't see the comment issue, will fix.


why testing


These 4 spaces indentation aren't consistent with 8 spaces elsewhere.


That was easy.. just a bug from Darius, I guess. Thanks!


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

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

org.labkey.test.Runner: org.labkey.test.tests.CDSVisualizationTest: <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>

```


I'm running into a problem with this patch. The DPDK vhost_user implementation resets all the memory associated with the device when it receives VHOST_RESET_OWNER. It looks like this is meant to mimic what the kernel vhost_net code does. So, I believe there needs to be another message for queue reset instead of overloading VHOST_RESET_OWNER for this behavior.


Hi


It more closely resembles the style how Node.js is written


Please `return` the value instead of logging it to the console.

One general comment here: function arguments are scoped to the function itself. Therefore you are not required to go out of your way to give different argument names to different functions. The only requirement is that a single function have different argument names.


Also, as solution, file can be used, to transfer data, from psd, to the function, something like
echo TMP=$TMP ... > $XDG_RUNTIME_DIR/.psd.$browser
sudo psd-helper $XDG_RUNTIME_DIR/.psd.$browser

Where helper just source file with vars, and after delete it.


Mat rspark-submit was renamed to run-example...
rspark-shell  - is a valid script to run spark shell


I suggest sending this comment to the qemu-devel mailing list and CC the patch author... github is just used as a git mirror for qemu, not many people actually watch the infrastructure


logging site -> login page


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

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

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

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

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

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

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

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

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

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

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

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


Thanks for the information, I will do that.


plutôt que changement, "modification"


Where did you get this from? This is the header-guard name in Microsoft's headers. It doesn't do any good on e.g. MingW. The *normal way (TM)* is to define `WIN32_LEAN_AND_MEAN`.


I don't believe `ntohl()` is expensive much, but since this is on a hot path it would be neat to only call it once, putting it into a `const int message_length` or something.


Futher investigation for trips that overlap is needed


I do not understand this sentence. Also, linking to the docs for signature inheritance would be nice.


Yeh, I know we do, but very rarely. Most contract additions do go to the master.


Cela tient en une ligne...


Why `lexically`?


I totally forgot to get back to you! Lemme try and clarify some: This brush needs to hold two 3D points because at any time it could be told to be (for example) 3/4 of the way closed. (There is only a `distance` keyvalue; the second position must be calculated.)
The `GetLocalOrigin()` point is a reference point similar to `GetAbsOrigin()`, but is computed with respect to the object's parent.
See entity description: https://developer.valvesoftware.com/wiki/Func_movelinear (the fix it links to is in an earlier commit and this change fixes a bug mentioned therein.)
See also: This file


Okay - Should I keep the /documents branch of the URLs then? I ended up calling them documents because in an earlier email you gave ``backend.com/document/[id]`` as an examples at some point. I agree that Manifests is a better name. I'm asking because it sounds like having a ``/documents`` branch might be some super-conventional web design thing that I don't know about!


Much better


Change the sentence structure to not start with `although`, and just state the facts directly.


wildcard, not your fault but it still could be fixed


that's only a codename for now, only toying with it, nothing to announce until I'm much more advanced :)


Good idea.


"dont l'un est correct pour un commit Git et dont un autre est complètement invalide.


So is this. But I goes you don't know that since you haven't use Orchestra Platform.


consiste à .... et à ....


81211464-LinkInfiltrationAndReturnFlows (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1865 of 1866 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)


I got it from the w32_fzs.h file; I just copied and pasted it.  If there's a better way to do it, go ahead.


I also don't agree that small projects can't follow the BC rules symfony do (or simelar conceptys). Infact, I think it's more acceptable for larger projects not to that than smaller projects. I don't really want to take any more of you're time though. You do a great job, and I'm very grateful for all your open source work, even if I don't always sound it. I complain too much. :)


I think it's easier to see by comparing this to the `AccountMenu` in the next commit. This could just be a naming problem.


@CamoSplatty14 Its late, I will be back tommorow.


An idea (on the hoof), this could also be something like
bool ignore;
var tasksToRecordImmatureSilos = from silo in siloStatusOracle.GetApproximateSiloStatuses(true).Keys.Where(s => !s.Equals(siloStatusOracle.SiloAddress)))
where !immatureSilos.TryGetValue(silo, out ignore)
select RecordImmatureSilo(silo));

A point to note that without materializing the query (``.ToList()`` or somesuch) this would be really "run" after ``if(!isStarting)`` and hence after possibly notifying the listeners. That is, it would alter the order of calls without materializing the task collection.


Great use of `switch` here, however you do not need to `break` from inside a `case` if you are doing a `return`, as it is redundant. `return` will immediately break out of both the `switch` AND the function.


On n'utilise pas "drapeau", mais "option"


`return localVariable1` would work just as well here, since you already did the computation :)


Check out [Boost.Endian](http://www.boost.org/doc/libs/release/libs/endian/doc/index.html), which was added in Boost 1.58. It uses highly-optimized compiler-specific endian swapping, where available. However, you might want your users to be able to compile Bonefish with earlier versions of Boost.


could we do this backwards compatibly - seems like no biggie ... won't exists on **1.7.4** ?


@RandomAltThingy It's alright, heh. I actually thought of telling you to take a break xD It's daylight over here but gnight


Got message: "Cannot read input of undefined". I'm using ``postcss ^5.0.3``.


 "les convertit en un script"

peut-être garder "fast-import", c'est la commande git.


Привет.
Я не сомневаюсь в твоих аргументах))

Возможно, есть какие-то отличия в коде или особенности Windows 10.
Давай я лучше тебе скину архив html + css.
Возможно у тебя мой вариант будет отображаться корректно.
Я и сам новичок. Нашёл статью на Хабре и гитхабе поправил ошибку.
Автор статьи на Хабре не я. Я автор одного исправления в коде.
11.09.2015 16:11 пользователь "iserjik" <notifications@github.com> написал:

> У меня в браузерах Chrome и Firefox. Crome от программы OpenServer.
> Firefox стандартный. Всё крутится на Windows 10. Если дашь почтовый ящик,
> могу кинуть видео как это происходит с стандартными настройками в стиле и с
> теми что я написал. Если это поможет в понимании. :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Mirantus/whitesquare-wordpress/commit/40a9d76d2c29ce83d0ad56800f230f200cd91b34#commitcomment-13182547>
> .
>


@CamoSplatty14 I lied, its 20:16 (8:16 PM) but I'm gonna watch TV.


You're right--sorry, my mistake.


serveur d'hébergement


Jacob and I agree using `click` is better - do you like the current PR a little better?  Perhaps it's OK not to explain any further why - we can talk about the impact on the click event without mentioning double tap at all.  It's not like people can't google 300ms click and find a million explanations ;-)


@ncouraud I just saw that you've registered in bing webmaster tools for the site on Wednesday, did you find out why bing would flag the site?


deux façons de "faire" parler


Canonical names for this iterators are **begin** and **end**. That's how it is in STL. So for the sake of consistency people started using it everywhere.


```javascript
var word;
```
would suffice here. Usually `null` is reserved to represent the absence of an **object** rather than that of a primitive value. If you just say `var word;` then `word` will be `undefined` rather than `null`.


I have no idea, I just tried to fix parallel builds.


Added info about favorite path


Many times it is useful to declare a class once and reuse it throughout a piece of code (e.g. plugins).

    var CUSTOM_CLASS = '.class-name',
        $el = $(CUSTOM_CLASS);

    ...

    if ($someEl.hasClass(CUSTOM_CLASS)) {
        $someEl.removeClass(CUSTOM_CLASS);
    }

    ...

    $someOtherEl.find(CUSTOM_CLASS);

    // etc

However, this is not possible currently since `.addClass`, `.removeClass`, `.toggleClass`, and `.hasClass` do not accept class names which include the dot, while `$()` and most tree traversing methods either accept or require it.

I think it would be beneficial to allow these other methods to accept class names which include the dot.


To PRO


Even though this is logically correct, I would have used:
```
if (str[i].length > longest)
```
here. Why? Since we are looking for the longest word, it seems more intuitive to do a "larger than" comparison. What do you think?


↑ *submodule

Yep, I got that working few minutes after making my comment, I just forget to run the source instead of the phar...

Two things that I noted on the server after some tests:

- Many people already said that but creative items can't be picked-up from inventory, an basic issue

- MCPE Light updates issue still there, some levels heavy modified by world editors haven't a properly light population, and due to the high speed of chunk loading now, the client freezes constantly while updating lights... Can be solved setting a low view distance but that isn't the point, maybe a pocketmine function for auto-correction of level's light population can solve that, but this is an issue for another branch.

- And some blocks of old levels are bugged or being interpreted bad from mcpe client, for example trapdors that aren't on the right position

Just a question, I've made a plugin that uses an interface for inheriting with some classes, should now I use scalar type hints on that classes or just on interface or on both for a good performance? Just a small doubt...


I think of this more like "menu" and "menu with navbar". I don't think the menu & nav are completely orthogonal. I'll play around with the naming more.


To PRE.


With your getters all called `get_*()`, do you really need the `typename` in here? There's no other symbol called `rawsocket_connection::fail_handler` (or any of the ones below), is there?


Аааа, понятно. Да, кинь пожалуйста. Вот моя почта msi.mail@yandex.ru
Код я сначала с Хабра брал, потом с Гитхаба копировал. Но кинь, я просмотрю и сравню на досуге.


@RandomAltThingy xP It's fine


example:
Scanning dependencies of target audioengine
autoreconf: configure.ac: tracing
[  2%] Building CXX object build/cores/audioengine/CMakeFiles/audioengine.dir/AEFactory.cpp.o
In file included from /data/src/xbmc/project/cmake/../../xbmc/addons/AddonManager.h:33:0,
                 from /data/src/xbmc/project/cmake/../../xbmc/addons/AddonDll.h:26,
                 from /data/src/xbmc/project/cmake/../../xbmc/cores/AudioEngine/DSPAddons/ActiveAEDSPAddon.h:23,
                 from /data/src/xbmc/project/cmake/../../xbmc/cores/AudioEngine/DSPAddons/ActiveAEDSP.h:30,
                 from /data/src/xbmc/project/cmake/../../xbmc/cores/AudioEngine/Engines/ActiveAE/ActiveAEBuffer.h:24,
                 from /data/src/xbmc/xbmc/cores/AudioEngine/Engines/ActiveAE/ActiveAESink.h:28,
                 from /data/src/xbmc/xbmc/cores/AudioEngine/Engines/ActiveAE/ActiveAE.h:25,
                 from /data/src/xbmc/xbmc/cores/AudioEngine/AEFactory.cpp:24:
/data/src/xbmc/project/cmake/../../lib/cpluff/libcpluff/cpluff.h:44:23: fatal error: cpluffdef.h: No such file or directory
 #include "cpluffdef.h"
                       ^
compilation terminated.


Any idea how this change might impact TCK deployment via the cli?  I think that we are using the below options:
org.jboss.as.cli --controller=${jboss.admin.host}:${jboss.admin.port} --connect deploy --force --unmanaged @{archive}


prêt à être poussé...


@iVertx As I said, this doesn't introduce bugfixes, it's just PHP7 support.

I'll be working on scalar type hints through all the core later and do tests with them :)


I was also thinking about moving this to `core` and renaming this file to `menus.js`.


Please use {} for nested "if" blocks!


We do consider BC heavily,  especially on component that are commonly used, you should know as I been amending/reviewing your testbench PR.

But this is an internal feature that was never advertise/documented. As important to keep BC I need to weight in the technical debt if more user come in, this is what small projects often struggle with.

Fixed it later on minor release, just like Laravel? That's as/far worst. People using it on 3.1 should be able to upgrade to 3.2 without any BC. If we really want to talk about SemVer.


WildFly doesn't deploy the archive, when this commit is used, if I revert this change, deployment works.


cet identifiant-là


Alright, thank you!


103122604-ClimaticDataSummaryBug (JasonGlazer) - Win64-Windows-7-VisualStudio-12: OK (1824 of 1851 tests passed)

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


Another nice trick that allows you to greatly simplify and generalize this function but involves usage of templates:
```cpp
template <typename It>
Node* CreateTree(It begin, It end) { ... }
```
This way you get the following advantages:
1. This function will auto-magically work with std::vector, std::set, std::list, etc. iterators. This allows you to create Tree from std::set.
2. You don't need to typedef Iterator type.

The syntax is pretty straightforward but if you want to get more insight - don't hesitate asking me.


also blocks form authentication


@amberwebb Thanks for your input. I think I've addressed all the issues you noted.


> However, this is not possible currently since `.addClass`, `.removeClass`, `.toggleClass`, and `.hasClass` do not accept class names which include the dot

Classes starting with a dot work fine for me. If they don't for you, please create a test case at https://jsfiddle.net  or https://jsbin.com and report a bug with a link to the test case.


:open_mouth: wow! I like that. Can you please explain to me in your own words how `apply` works and what it does?


Reference #1507


j'ai pris l'habitude de mettre le mot commit en forme avec un underscore : _commit_, _commits_


Fixed.


Here you do not need to put `letters` between parentheses.


Actually, you're confusing CSS selectors with class names. `$('.class-name')` will find elements with a class `class-name`, not `.class-name`.

It's not a good idea to start class names with a dot because of issues like that.


I wonder whether you've done this because you like evergreen trees or you just simply forgot to delete this object :)


@gpopovic Can you possibly open a new issue with a Codepen so that I can see exactly what you're doing?


:+1: for the logic, but I would not use `string` as an argument to my function. Even though it is not a reserved word in JS, it sounds confusing.


I'm personally more for clarity, or at least making it read well. See https://github.com/w3c/pointerevents/pull/20#issuecomment-139638023 though for a slight expansion to current Pr


Si vous avez lu...

ensuite, vous pouvez ....


when I try to use these refs (for workspace name and namespace) , there's a null pointer exception! huh?


Ok here is the fix for this. Instead of using $find['time'] as the variable, use $vote['time']
this means that the following needs to be added around line 31 to defune the variable

$votes[$vote['id']]['time'] = $vote['time'];

$vote['time'];

now make sure that in the database `time` field, you use hours and not minutes or seconds.


Is this the test harness? How would someone know how to run tests coming to your repository for the first time?


An empty string would be simply `""` rather than `" "`. Try it and see what result you get :)


fix-window-frame-calcs-5095 (nealkruis) - x86_64-MacOS-10.9-clang: OK (1818 of 1846 tests passed)

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


these two labels/fields are added when-not (:workspace-id props)


Does it make sense to check in NetBeans configuration files into the repository? Why or why not?


You're welcome! Let me know when you need help with Transifex...


Do unit tests belong here?


J'ai banni le "verbe" committer : soit j'utilise valider ou créer un commit


if I don't call this here, then the placement isn't quite right.
Still, the placement of the second new label (WSNamepace) isn't right ....
I try to call it *inside* the for loop, but then stuff disappears altogether.


:+1: for the logic :) Try giving your argument another name than `string`. Even though it is not a reserved or special word in JS, it is confusing because it corresonds to a JavaScript value type. Maybe just `str` or even `inputString`.


Fixed. Left over from some templating that didn't work out.


ok


équivalente