Oops, yeah. It doesn't break anything if left in there but should be commented out just for tidying up.


@jrose-apple  what is HasRawArguments for ? Do not we need only 2 states, 1 for arguments not yet computed and 1 for arguments have been computed ?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 14430](https://teamcity.mesosphere.io/viewLog.html?buildId=123137&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 127 (new) Build time: 00:01:41


@hawkowl @Ashalynd do you have any idea why python says there is no json to decode while it's a perfectly valid json string?
I'm thinking too long maybe?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 14431](https://teamcity.mesosphere.io/viewLog.html?buildId=123138&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 127 Build time: 00:01:50


With my proposal this would become:

```c
n = pwrite(fd, &header.state, sizeof(header.state), offsetof(Header, header));
```
Which I think is easer to grok....


I built a hard disk image (I guess) using the build-filesystem script. Then I did the same createdrives does to put the kernel and the bootblock on the drivea.cpm.

I moved those images to the cpmsim/disk directory and then I tried to run cpmsim. I see the bootblock is fine because it loads the kernel and asks me for the boot device. Here is where I can't proceed any further.

So if I tell it to use 0, it complains because the drivea.cpm disk is empty (no init there). If I give it 4 (I think) it tells me there's a valid filesystem, but there's no init there. I checked drivei.cpm with ucp and I can see the init binary in the root.

Am I doing something wrong or perhaps I am missing something?


SMTube para PiSi


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 14429](https://teamcity.mesosphere.io/viewLog.html?buildId=123140&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 127 (new) Build time: 00:02:24


On return from `select()` there is either an error, handled separately, an FD event, or timeout. The timeout case is simple, we simply reset `timeout.tv_sec=0` and L370 will take care to set a new timeout before the next select.  The FD event case we need to calculate the timeout' value, which should be: `(future timeout) - (current time) == (now.tv_sec + cache_tmo) - (timeout.tv_sec)`.

I believe my code is correct, albeit extremely messy so it needs to be rewritten and simplified just for that reason :smiley:


збігів,(кома) починаючи..


pwrite() returns ssize_t, not int. It should not be cast implicitly to an int, since that is often smaller than ssize_t.... Hence, please add a new variable `ssize_t n` and assign the result to it.


SMTube para PISI


return -errno...


Great stuff! After that we can easily refactor things


Traditionally, I have seen this done as a `trait`, any particular reason why it's an `abstract class`?


(nit) 2016


This has been bugging me. Should this be `State` or `'State'`?


:fireworks: thanks @rickynils!


return 0...


So for the actual template-finding part, the only query I've been able to get working is the following:

```ruby
@template = ApplicationTemplate.find{|a| a.position == position && a.department == dept}
```

These other 4 random attempts all failed to do anything but give me errors

```ruby
@template = ApplicationTemplate.joins(:position).find_by("position = ? AND department = ?", position, dept)
@template = ApplicationTemplate.find_by(position: position, department: dept)
@template = ApplicationTemplate.find_by(position: {department: dept.id})
@template = ApplicationTemplate.find_by(position: position, position.department: dept)
```


I added this in PSU-OIT-ARC/oregoninvasiveshotline@92b7645 with a few tweaks.


Interesting. I basically copied this logic, I guess the analytics calls are async? Does it matter if we don't wait for a response?


- не слідуючого, а **_наступного_**

- **_немає_**


THis call's name should express the weird semantics it has, i.e. `journal_file_set_offline_thread_safe()` or something along those lines...


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 14432](https://teamcity.mesosphere.io/viewLog.html?buildId=123146&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 127 Build time: 00:00:10


The IO is based on your session commit rate. If you keep it low you will use more memory, but performance will be better. For tests just create some random files or read the same file 100000 times.


This breaks the test suite.


controll -> control


usually we check >= 0...


哦，我只有看到blog-admin比较酷，也想搞个类似操作界面的而已。你要是觉得没意义也没关系，我把呆会撤了这个pr就好。
另外，测试页面上有什么问题呢？毕竟我对ego还不熟，也没有实际测试过


Should be fixed in e543b3d


Ownership semantics of the fd are a bit weird... It is closed on success, but leaks on failure...

I think the ownership of the fd should remain at the caller and the caller should close it in all cases...


Say those lines are cited.


thanks

--
Robert Tassy, III

On Wed, Feb 3, 2016 at 4:25 PM, Joseph Ciancio <notifications@github.com>
wrote:

> Great! This file looks mostly organized and there aren't any glaring
> issues, good job.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/RobbyTassy/robert-s-portfolio-site/commit/d42dfa60fe4ae0347098f7c942460f4ebabbd648#commitcomment-15869852>
> .
>


ok I see


Unwrap observable makes sure that it doesn't fail hard on call _id if it's not defined. Though now that it's in the include mapping it should always be defined and makes this now necessary


What is going on with `'elementPrefix' && '@'`? This line is functionally equivalent to:

```
elementPrefix: name === '@'
```


@phillipm I definitely prefer the negative pattern of making it clear that we're not going any further if these conditions are met, as opposed to wrapping the entire method in the positive if statement. What made you switch it?


fd leaked on failure (see above)


Yes, it worked.


рівне кількості повторів рядк**а**


It looks like each message has the same div id of "pretty_text". Please change to a class so we don't have multiple divs with the same id.


The math operations of determining this width should be moved out of the template into a method attached to the scope. The `float` and `height` properties should be defined in actual CSS.


we no longer need this test because ... ?


More reason why this should happen on the scope -- has to be repeated.


Empty class attribute.


Dam Son some great work you did here.


Octree (DeadParrot) - Win64-Windows-7-VisualStudio-12: OK (2067 of 2068 tests passed, 14 test warnings)

 * 549 tests had: AUD diffs.
 * 11 tests had: BND diffs.
 * 8 tests had: MDD diffs.
 * 8 tests had: MTD diffs.
 * 8 tests had: RDD diffs.
 * 4 tests had: EIO diffs.
 * 4 tests had: ESO small diffs.
 * 1 test had: SHD diffs.

![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)


ah, right. thanks.


Need a blank line between functions.  Use:

    $ paver run quality

to see if our lint checks will pass.


I love me some better whitespace :+1:


That's what I suspected, but good to know. The `uniqueSeedComponent` should be enough to ensure unique seeds.


The stuff above appears to duplicate quite some code from journal_file_close().. this should be split out in a new call and be shared with both calls...


Done


I tried out urxvt. It was only able to initialize 88 colors. Anything past that would be randomly chosen from 0-87. Could that be what you were seeing?


This edit is setup for the possible solution listed in errors.


Needs lots of testing.


Yeah, auto formatting with Netbeans style would be nice. I use it often and rarely upload unformatted code.
So maybe we don't need to automate this given that not too many people commit here.

This seems to confirm again that anything coming from eclipse team makes me sad... :laughing:


bincompat?


This needs to have some strong type. If the type can't be easily unified between the implementations then we need to extract separate implementations for the different platforms.


sadf


Use `self.assertIsNone` instead of `self.assertEquals`.


@TrejGun I didn't notice this, but this is very very very bad. There are quite a few places where we rely on `val == null` being a shorthand for `val === null || val === undefined`, and reverting this change will be very difficult. I'll do a quick pass for other instances, but can you do me a favor and also look through in case I miss any?


double check


.._так**и**м_ самим..


'State' of course :) My bad!


We should probably focus on mac and windows users. Ubuntu isn't really known for its good UEx.


To confirm, you don't need sftpChannel.exit(); if session.disconnect() and channel.disconnect() are called?


Exception types are part of the API behavioral contract. We need to revert this and other changes regarding exception type changes or separate the code for XPlat so that the Windows implementation throws the same exception as on Desktop.


Fixed in 4051efa


See #3852


The content of this element may be better suited for a scope method or as a custom filter.


Looks pretty OK conceptually. But I think this could be nicer by allowing the kernel deal with the synchronization for us. Specifically, I'd (mis-)use a pipe as a reference counter for the background threads. When journald initializes it allocates a pipe() pair of fds, and passes the writing side to the JournalFile objects which in turn pass a dup()ed copy of it to their background fsync() threads. That thread closes the copy when it is done with its work. When we need to wait for all bg threads we simply take away the write fd from all JournalFile objects, and then close it. Then, we simply invoke read() on the read side of the pipe. This will block until all write-sides fd copies are closed, at which point we'll get EOF from the pipe.

Using this technique means we don't have to keep track of the pthread_t's. We just need to keep a pair of pipe fds that we can share between all JournalFile objects.

Does this make sense?


I would probably change the comment to something similar to the ffmpeg docs
"this stops" is meaningless for someone who does not know the history. that is everyone but the two of us :)


Shouldn't these be added to [error.json](https://github.com/jhipster/generator-jhipster/blob/master/generators/app/templates/src/main/webapp/i18n/en/error.json)?

I'm getting the following error when trying to edit the Anonymous user
![screen shot 2016-02-03 at 22 44 30](https://cloud.githubusercontent.com/assets/571389/12798091/d15d17c2-cac7-11e5-9c3e-2103dafa6713.png)


So just add a link to this section from the tutorial.


nice find! please pr


@leeoniya, yes, this means all browsers — modern and legacy — apply uniform spacing and padding without this CSS. Nearly a hundred browser and device variations were tested to be absolutely certain. See #506.


It is testing the via the `MarcImprint` class which no longer exists.  Also, the matching of vernacular is tested in the `MarcField` class (which is ultimately what this was intending to test in the first place).


explicitly call out ~20 segments


change ip with your ip


Isn't this already figured out on line 12?


same here, explicitly call out 10th segment


Filenames must match the class name. Please rename to NegoState.cs or NegoState.Common.cs.

I think there's another instance of this pattern I've used in the past. I think we'll need to change that too as it complicates understanding which files form a single class.


This link is a little old and doesn't use this library, but it should demonstrate broadcasting engine data.
http://forum.arduino.cc/index.php?topic=50893.30

Do you need help broadcasting or receiving?


Don't think so... we had gigantic json in optima 1.0


Don't use `print()` here. If there's an error, use standardized logging such as in movie.py or safety.py


This logic of `Math.floor`, `/60`, `%60` is already being repeated -- it should be centralized in a function, possibly (probably) injected as a function.


- **_не в змозі_**

-  конструктор `RegExp`**(коми нетреба)** коли


Yeah, the events get pushed to a queue, so if you are recording an event then immediately navigating to a new page, it's best to use the callback: http://stackoverflow.com/a/12461639/1739725


Great, can also be done with just `_.find(library, { _id: track._id })`


Additionally, all tests passed when I ran in "real mode" (sudo make check PINTOSOPTS='-r')


Sorcery!!


Yeah I don't think it was necessary here but I was thinking about it for gephi but wanted to try it out here first.


Any reason for doing `Object.create` over using `new`? Not incorrect, just curious.


That doesn't actually fix the problem. The problem isn't that you formatted the link incorrectly, the problem is that the link you're using will never work. It's better to use `:ref:` since sphinx will handle link generation for us in that case.


Careful with formatting! This is really, really, really important.


Bah. I just realized this was in the README.rst. I'm not sure what the right link is, but we should hold off on this until we have the right link. I think it might be something like `/en/latest/code-of-conduct`


I am not sure this is the best approach, although not the worst. :)
How about a temporary module on npm?


:tophat: :clap:  :-)


Looks like you mistyped the name of the Canola oil


NegoState has a specific role in the original implementation and it should remain the same. I don't like the architectural change here. Your change is diverging from the SslStream implementation as well: SSPIWrapper should still be used in the Windows context: see the implementation in SslStreamPal.Windows.cs.

The main idea is to keep the implementation for Windows as close as possible to Desktop as we need to be able to maintain both. Changes that are acceptable in a PAL layer should be just to create a very simple layer of indirection that would allow XPlat code to fit easily.


Out of curiosity, why are you clipping another boundary pixel after you have already clipped half the kernel on both sides?


My understanding is it helps to preserve binary compatibility between releases. I believe if it is a trait, and you added a new method, the new version would no longer be binary compatible with the previous one whereas with an abstract class it can be (not sure if it _always_ is). I've seen this used by `scalaz` and `argonaut`. I'm not sure if this is still the case in the lastest version of scala and/or if `sealed` traits provide the same benefits.

On that note, this abstract class should be `sealed` itself.


Exit calls Disconnect.  Also exit is not part of Channel, but is part of ChannelSftp.


Mihir,

Yeah, this doesn't seem to work. If I do:
var names = {
    COND_PASSED: 'passed',
}

then I get an object with "COND_PASSED" as a key, rather than the value of
the COND_PASSED constant as a key.

Did you have something else in mind and I'm misunderstanding? I agree that
the current form is a bit janky.

-Mike

On Wed, Feb 3, 2016 at 12:59 PM, Mike Treaster <treaster@dropbox.com> wrote:

> I thought I had tried this and it had failed, but maybe I was in JS at the
> time... I'll try this again. Thanks!
>
> On Wed, Feb 3, 2016 at 12:55 PM, Mihir Kedia <notifications@github.com>
> wrote:
>
>> I think you can do:
>>
>> var names = {
>>
>> ...
>> }
>>
>> see https://babeljs.io/docs/learn-es2015/#enhanced-object-literals
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/dropbox/changes/commit/58fda65ed92d31a401d3a69f01f8f977177cdca7#commitcomment-15868927>
>> .
>>
>
>


worst code ever


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 14434](https://teamcity.mesosphere.io/viewLog.html?buildId=123232&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:00:08


All of the tests in in stats.test.js are wrapped in a try block although the only functions called are the test functions - I cannot quite figure out why. I just went with the current code on this one - personally I wouldn't wrap them in a try catch  if no functions are called - the only exception would be thrown by nodeunit functions ...


+В ECMAScript 5, ви можете дублювати регулярні вирази, переда**_вши_** до конструктор**_а_** `RegExp` таким чином


Formatting! Important!


test a comment


Got it. Yes, I think you're right... FWIW I think:

A) won't be able to truly test this until we merge the branch which adds `source/code-of-conduct.rst`
B) I was copying the PyPA link from [here](https://raw.githubusercontent.com/pypa/pip/develop/README.rst): `.. _PyPA Code of Conduct: https://www.pypa.io/en/latest/code-of-conduct/`


a line note


lodash comes with clone functions (`_.clone` and `_.cloneDeep`)


Si antes se utilizaba una traducción por qué no seguirla usando. Es mejor cambiar los datos de la traducción para que adapte a nuestras necesidades y no poner un nombre fijo. Siempre que sea posible.
Se esta modificando solo una orden de entrada, no es correcto usar el plural "Ordenes de Entrada", debería ser "Orden de entrada".
No era "Modificar Datos de..." sino "Modificar ..."


we should not change this i think. once you complied the binary it stills sits at the same level with etcd binary.


ideally, we should make this a configurable variable not const.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 14435](https://teamcity.mesosphere.io/viewLog.html?buildId=123245&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:00:06


here we are pushing the pv move to the top of the list even if we are not currently in the Principal Variation.. this seems wrong


buf, eres el alma del proyecto nano L


Es solo una orden de entrada, no es correcto usar el plural "Ordenes de Entrada", debería ser "Orden de Entrada".


but but, maybe an addition to the collection, lol


why this sub pkg has a main.go?


mono_error_raise_exception does the same check. No need to do it.


Ok, will change.


Done.


Tests that show mangled type names would be nice, but I guess comprehensive tests can wait until you have the deserialization tool


Good advice. Thanks!
Done.


Seems plausible to me, though I'm not an SELinux expert.


`Path` -> `DirPath`


MacBook-Air:exTVPlayer Leonid$ git submodule init
Submodule 'Submodules/AFNetworking' (https://github.com/AFNetworking/AFNetworking.git) registered for path '../Submodules/AFNetworking'
Submodule 'Submodules/DACircularProgress' (https://github.com/danielamitay/DACircularProgress.git) registered for path '../Submodules/DACircularProgress'
Submodule 'Submodules/MagicalRecord' (https://github.com/magicalpanda/MagicalRecord.git) registered for path '../Submodules/MagicalRecord'
Submodule 'Submodules/Ono' (https://github.com/tviu/Ono.git) registered for path '../Submodules/Ono'
Submodule 'Submodules/SDVersion' (https://github.com/sebyddd/SDVersion.git) registered for path '../Submodules/SDVersion'
Submodule 'Submodules/SDWebImage' (https://github.com/rs/SDWebImage.git) registered for path '../Submodules/SDWebImage'

but doesn't work


Es "Listado de ...".
Ojo con las faltas de ortografía. Debe ser Órdenes de Entrada


I also made another commit after looking at your patch in your own fork, you removed now.
It was a good hint making a clean up around in qtinfo.py and setup.py, which uses qtinfo.


indent for `var`


Same, `Path` -> `DirPath`


Es "Bitácora Almacén".


lorem ipsum


yeah it doesn't...
I forgot to remove the line after testing it...
Here: https://github.com/ArmaLife/Altis/commit/1e42bde03b51b60d45d3f028f40e4276c0f1b9b1


@sgrif Any chance you might have some time to look into this?

https://github.com/rails/rails/issues/20077

This is KILLER for our productivity, as we use tons of AREL. I'm willing to invest some time trying to fix it, but it seems like you have to have a great deal of context before jumping into a domain as esoteric as AREL.

Thanks


Does `basename` warrant its own type?  I think so.


`PathDir`.


If we want to serialize a valid JSON, should we change last_build to lastBuild ?


extra nl


ah sorry, the comment text sent in the email notification was wrong. Its

var names = {
  [ COND_PASSED ]: 'passed',
  [ COND_FAILED ]: 'failed',
  ...
}

On Wed, Feb 3, 2016 at 4:51 PM, treaster <notifications@github.com> wrote:

> Mihir,
>
> Yeah, this doesn't seem to work. If I do:
> var names = {
> COND_PASSED: 'passed',
> }
>
> then I get an object with "COND_PASSED" as a key, rather than the value of
> the COND_PASSED constant as a key.
>
> Did you have something else in mind and I'm misunderstanding? I agree that
> the current form is a bit janky.
>
> -Mike
>
>
> On Wed, Feb 3, 2016 at 12:59 PM, Mike Treaster <treaster@dropbox.com>
> wrote:
>
> > I thought I had tried this and it had failed, but maybe I was in JS at
> the
> > time... I'll try this again. Thanks!
> >
> > On Wed, Feb 3, 2016 at 12:55 PM, Mihir Kedia <notifications@github.com>
> > wrote:
> >
> >> I think you can do:
> >>
> >> var names = {
> >>
> >> ...
> >> }
> >>
> >> see https://babeljs.io/docs/learn-es2015/#enhanced-object-literals
> >>
> >> —
> >> Reply to this email directly or view it on GitHub
> >> <
> https://github.com/dropbox/changes/commit/58fda65ed92d31a401d3a69f01f8f977177cdca7#commitcomment-15868927
> >
> >> .
> >>
> >
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dropbox/changes/commit/58fda65ed92d31a401d3a69f01f8f977177cdca7#commitcomment-15870577>
> .
>


Es "Modificar Bodega"


Also the doc should be changed (it's no longer a prefix).


Es "Listado de Bodegas".


I'm not entirely sure about `--filename` being the best name for this param. Will you look at what other popular tools use?


A few fixups, but this looks great.


Wow this is so cool. You coded this support for CircleCI just in the moment I switched to it from Travis.

thx ! :]]


You're not wrong. Patched.


Ojo con las tildes.


Looks like `-v` and `-h` could use a little capitalization since you are doing that for other options.


I have started crunching on the reflection section dumper, but it actually wouldn't be too hard to add check lines for those.


I think it is better to wrap blocks of code in conditionals instead of breaking out of the method before reaching that block, when possible. The reason for this is that it directly ties constraints that need to be met  for a block to be run with that block.

I'll change it from `!(isFinishing() || isStartingDetailActivity)` to `!isFinishing() && !isStartingDetailActivity` because that is a bit clearer.


Es "Ver Datos de Bodega"


Also `-h` should probably say "print" instead of "prints" for consistency.


I think I'm starting to understand the changes: NegoState became the PAL in this case. We'd prefer things to remain encapsulated as much as possible, including the PAL layer.

Can we instead extract a NegotiateStreamPal class that is similar to SslStreamPal instead of changing NegoState? We can then add all these calls there. For a quick example see how all calls to SSPIWrapper from the original SslStream code were routed to this new SslStreamPal class with .Windows and .Unix versions. The .Windows version basically has the same implementation as the NegoState.Windows.cs functions that you've added.


oh well
and i thought that 10.0.0.x ip addresses was a nice idea :(


they look cleaner yeah


Ah yes, these are some old troubleshooting comments of mine. Good catch.


Es "Modificar Categoría de Producto"


Unfortunately does not seem to work in Windows 10... would have been extremely useful.


Wow. That was really quick! o_O

:+1:


Kinda minor but there's a missing break here. ComplaintType could also use an enum :)


Es "Listado de Categorías de Producto"


Categoría de Producto debe ser con mayúscula


LGTM


Igual que el anterior comentario


Much better. Good thing I checked!


Nops :-1:


This is probably a misunderstanding on my part, by why use two different methods to calculate the gradient.  They aren't actually the same thing.  As I understand it, diff gives the inter pixel difference and gradient gives the central difference.  Should it either use numpy.diff in both places or call numpy.gradient twice to get the second derivative term?

That would also mean you don't have to mess around with the boarder since both first and second derivative terms would have the same number of elements.


It could be, but this is odd because urxvt clearly supports 256 color and
terminfo confirms that as well. And I am pretty sure Python's curses can do
that since Urwid's example has shown if it also uses curses as backend.

Something is going on.

On Thu, Feb 4, 2016 at 5:38 AM, Brandon Dreager <notifications@github.com>
wrote:

> I tried out urxvt. It was only able to initialize 88 colors. Anything past
> that would be randomly chosen from 0-87. Could that be what you were seeing?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Regaerd/PyClock/commit/83f26e5cd4414dc6993612a34a0c05776291092b#commitcomment-15870172>
> .
>


Ah, don't we love Python's unicode support? I remember a few instances where we added explicit `str()` calls, but this certainly the better approach. If you see any, please update at your leisure.

Does this close #148?


AddFirewallContextMenu.reg  Unfortunately does not seem to work in Windows 10... would have been extremely useful.


Seems like this comment should stay here, though.


Yeah, the current tests are... interesting


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 14436](https://teamcity.mesosphere.io/viewLog.html?buildId=123253&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:00:07


Same as above: we can move the `QueryContextAuthenticationPackage` into a new NegotiateStreamPal class and pass exactly the same arguments as before. That implementation should be almost identical to the current NegoState.QueryContextAuthenticationPackage one.


@gribozavr @jrose-apple , can you please review this change ?


Creo que mejor ponemos este header/title con una traducción, pues tal y como está es una parte en inglés y otra en español. Mejor tenerlo en una traducción y luego que el cliente diga con cual se queda mejor.


Good organization, no glaring issues.


What happens if `opts[:filename]` isn't specified?

Are there any security concerns that we should think about here?


This fails. This indicates this is probably an issue with back-tracking.


Why did this change?


Should probably also stay. These were all caused by clang's API docs being impenetrable or absent. I was chasing misreported locations of things involving macro expansions and dared change only the minimum, with the expectation that we'd come back later and do more.


Offtopic:
Isn't the install script outdated (like way outdated..) -> https://github.com/unified-db/Database/blob/master/Tools/Windows_Install_Script.bat


Please remove the space before the colon.


I tried to follow the logic in isSeasonalOverlap, and the if statement is too complex for me to grock in my head.  I'd feel better if you created a test case that explicitly called this utility method with all of the possible scenarios to prove this works!


컬럼 고정 + 셀 병합 + 인라인 에디터 = 헬... X_X


Az adatbázis olyan adatokat is tartalmaz ami nem szükséges a program működéséhez. A következő commitban ezeket törlöm és a felesleges kódot is, ha van. Továbbá belerakok egy funkciót első indításra azoknak, akik év közben kezdenék el használni a progit.
Tervben van a program windows-al való indítása, háttérben való futtatása, és naponta leellenőrzi majd, ha tartozás van és figyelmeztetne.


TeamCity pull requests :: netty [Build 1203](https://garage.netty.io/teamcity/viewLog.html?buildId=5399&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Tests passed: 6409, ignored: 23; resource leak detected (new) Build time: 00:42:17


TeamCity pull requests :: netty [Build 1204](https://garage.netty.io/teamcity/viewLog.html?buildId=5400&buildTypeId=pulls_netty&guest=1) is now running


Please bring back the old comment.


yeah it closes it


Why not just use Optional?


cool


Es "Modificar Movimiento de Bodega"


Nice indentation! Some use of semantic elements. Looks good to me.


What does the above have to do with the pidfile change?


What happens if there are two files with same inode number and same subvolid but on different filesystems?


You are printing error only if added and removed both are non empty? what if you have just one bad added and nothing missing in removed?


Just wanted to try with Python 3 to see if the color issue also in 83f26e5cd4414dc6993612a34a0c05776291092b, but I got:

```python
ImportError: No module named 'ConfigParser'
```

In Python 3, it seems to be named `configparser`, but another thrown at me:

```python
AttributeError: 'dict' object has no attribute 'iteritems'
```

@Regaerd Do you have Python 3 on your system?

I think it might also the time to start tests.


The space is on the wrong side of the colon.  Should be `var result: [String] = []`


should add both sets to message.


fixes #1


- [ ]  move all the contents of  ./data-unshared/  to  ./data/unshared/  and delete it.
Done!


Indentation is off.


Still doesn't work because we need `SITE_URL` set and the `static` folder needs to exist when using `-c`, though we probably just don't need `-c`. But if you're not yet using docker than we can fix this later.


You may not need debug() if error has both sets.


Please remove the semicolon.


I'm a little concerned that this doesn't do anything to update the variance or mask.  The mask should at least indicate that no B/F correction is applied around the edge.  This can be another ticket.


Hello, I think i don't understand how to submit something and how to make the link between my coursera profil and my github profil ....
Is somebody can help me ?

Stéphane


Good catch. Thanks!


Remove the type annotation.


The tests already exist, actually. All test methods that that end with "logic" explicitly call this method.


^^^ Good job! This is really important to having a nice responsive design


It's mostly a sanity check for the -strip* flags, really.


Please also add a comment that explains why this is done like that, or someone will change it back to a lazy let in future.


Is `$parse` used here?


YOLO


HBJH


Estupendo. Sólo una cosa: me resulta poco intuitivo guardar `extend` como `false` cuando está presente. Eso unido al `if not` del bloque hace que tengas que pensar qué es lo que está pasando. Si no hay un motivo especial para hacerlo así creo que quedaría más claro usar `store_true` en vez de `store_false`.


**SUCCESS** - TeamCity HotRod C++ Client :: Pull requests monitor - RHEL [Build 358](http://ci.infinispan.org/viewLog.html?buildId=34947&buildTypeId=HotRodCpp_PullRequestsMonitorRhel6&guest=1) for b17afd4bbe70eaaae375f660561925e5599fd642
Tests: 0, 0 failed (0 new), 0 ignored. Build time: 00:08:24


Es "Crear Nuevo Movimiento de Bodega"


Updated to reflect that https://github.com/lcm-proj/lcm/pull/48 is in LCM upstream repo now.


>urxvt clearly supports 256 color and terminfo confirms

What I saw could be due to the version I installed.

To see how many colors were initialized, you can add...
```self.stdscr.addstr(0,0, str(self.color_range))```
...to clock.update, after the first if/elif


Shouldn't `create :application_template` create a position and department anyway?


Es "Ver Datos de Movimiento de Productos"


Be careful with indentation and spacing, it won't cause an error, but it can get confusing when figuring out which "level" we are in. Proper nesting can make it easy to scan thru the document quickly and find the area you need to edit/fix


Es "Modificar Necesidad de Material"


Nice use of semantic elements


why not
```js
import { createReduxStore } from 'state';
```


Es "Ver Datos de Necesidad de Material"


Easier way to write is: `n1.map(|n| n+1)`


Es "Modificar Registro de Compra"


Just note the signature:
`Player::SetClientControl(Unit* target, uint8 allowMove)`
The `allowMove` value after conversion to uint8 goes directly to the packet. While the present changes about `!apply` do not break any functionality, they are intrinsically wrong until you ensure that the packet handler of the client reads it actually as bool.
As to the substantial change, thanks. I'm glad to see that the bunch introducing this error [against my advice](https://www.getmangos.eu/community-projects/10128-wow-1-12-server-repack-post74290.html#post74290), is active no more.


@Scooletz can you check `queueBindings.SendingAddresses`? Those should be `audit` and `error` queues.


Es "Ver Datos de Registro de Compra"


I just tried with [this snippet][s], it displays right result and I finally saw the issue.

You *overlapped* the index. If you set offset to 128 and loop the index up to also 128, it works fine. The questions is why we can't have more than 256 pairs when terminfo has `max_pairs#32767`, they are the same thing, right?

[s]: http://stackoverflow.com/a/22166613/242583


Great! Media queries are very useful in creating responsive designs.


eeeeeeeeeee


@swift-ci Please test


why the downgrade to moment? something to do with shrinkwrap above? just want to better understand.


Es "Bitácora de Producto"


>Do you have Python 3 on your system?

I do. I guess I should actually start using it.


Es "Modificar Producto"


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 14437](https://teamcity.mesosphere.io/viewLog.html?buildId=123257&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 0, ignored: 218 Build time: 00:11:53
### Failed tests
```
null: Kubernetes e2e suite.Kubectl client Guestbook application should create and stop a working application [Conformance]: <no details avaliable>

```


Can I comment?


needs more pants


Es "Ver Datos de Producto"


Files to be removed must be an input and of data type 'File'. They must also be in the working directory.


Done


Es "Modificar Proveedor"


La palabra "nuevo" debería ir con mayúscula: "Crear Nuevo Proveedor"


La palabra "datos" debería ir con mayúscula: "Ver Datos de Proveedor"


Preco vlastne hadzeme vynimku a nie nil?


this is the same permission has `bootstrap_key_owner` can we factorise this also?


Yeah, improves debuggability.


не определился с фоном при наведении


Es "Modificar Salida de Bodega"


same as `bootstrap_key_owner` again :)


Yay!


"nueva" con mayúscula


Hey sorry I didn't realize this appears to break things and generates invalid go code @oxtoacart. What was the thought behind pulling this into flashlight.go instead of autoupdate.go?


Uz netreba require na `uvo_scraper`


Es "Ver Datos de Salida de Bodega"


this is awkward


Uz sme fakt blizko ;)


Es "Modificar Tercero"


AHHHHHH! It's really hard to tell what's going on here. We have inline styles and inline javascript. We really want to stay away from this. We haven't gone over JavaScript so I don't expect you to know where to put the JavaScript, but we should have a separate .css file for the styles. In HTML files we want only HTML, it's better practice.


Es "Listado de Terceros"


Es "Crear Nuevo Tercero"


Es "Ver Dato de Tercero"


This will likely not happen. I am adding `scroll` event in https://github.com/ampproject/amphtml/pull/1767. I think that should help with this usecase. Till then, please open an issue, ask @rudygalfi to assign the issue to you. Also, change the format to following:

// TODO(nathangoulding, #<issue number>): description.


its not hard to make it working fine again...
But becouse we using 'InstallFullDB.sh' -- and this works brilliant for WIN/linux,
and its very simple ... like double (mouse) click.
what was the point to keep another script?


Yeah, I see now. Why not file an issue? Sounds like a good thing to have in a master. Or you planned make PR for this things?


saw approval from mrallen1
at https://github.com/basho/cuttlefish/commit/8b8151e586f55289fe9cce95364a190e45a26b20


merging basho/cuttlefish/dr/merge_2.0_to_develop = 8b8151e5 into borshop-integration-209-dr/merge_2.0_to_develop


basho/cuttlefish/dr/merge_2.0_to_develop = 8b8151e5 merged ok, testing candidate = a463682b


why'd you remove this?


Ooh, swap_remove. The problem with this though is that it changes ordering in the media list seemingly randomly.

I can think of a way few ways in which it could be done differently but most of them involve leaving in gaps and thus adding lots of complexity in iteration logic. Not sure what's the best way. Probably most efficient would be moving from nettable to custom solution but that's a lot of work.


Es "Modificar Datos de Autobús"


saw approval from mrallen1
at https://github.com/basho/riak_kv/commit/511dda7acfb95d4536d009f43b5fd2583a45767b


merging basho/riak_kv/merge/2.0_to_2.1_for_leak_fixes = 511dda7a into borshop-integration-1340-merge/2.0_to_2.1_for_leak_fixes


basho/riak_kv/merge/2.0_to_2.1_for_leak_fixes = 511dda7a merged ok, testing candidate = d45336cf


Just reading the Nan 2 docs to try and understand why these were all removed. When calling the `capture` method, the following error occurs:

```
FATAL ERROR: v8::HandleScope::CreateHandle() Cannot create a handle without a HandleScope
Aborted
```


@johnktims
```
      much generics

such type inference
                     wow
```


zddddsd


this isn't in the final diff


Es "Ver Datos de Autobús"


Awesome, I'll check that out! I need help broadcasting, we're basically transmitting sensors coming in on the Analog pins on the arduino and broadcasting it to a Lowrance unit.


Es "Modificar Datos de Chofer"


Because the import will be called before
```
require( 'lib/react-test-env-setup' )();
```


Fixed


Es "Ver Datos de Chofer"


curious, would you want to give a default value for this?


I can't find GetContextToken(out ...) anywhere. Can you please point me to where this method got moved?


PackageVersion isn't just used for auto update, it's also used by the config subsystem and other places.  AutoUpdate is specific to the desktop app, so I needed to invert the dependency.

What did this break?


Es "Modificar Grupo Buses"


Use `isnan()` instead of a trick.


Deneme içerik


Good catch ! It's a left over from an earlier version of the PR.


Es "Listado de Grupos Buses"


```
lantern (valencia)]$ VERSION=2.0.11 make darwin
...
OK
Building darwin/amd64...
Build tags: prod
# github.com/getlantern/flashlight
src/github.com/getlantern/flashlight/flashlight-prod.go:39: syntax error: unexpected =, expecting {
src/github.com/getlantern/flashlight/flashlight-prod.go:44: non-declaration statement outside function body
src/github.com/getlantern/flashlight/flashlight-prod.go:48: non-declaration statement outside function body
src/github.com/getlantern/flashlight/flashlight-prod.go:49: syntax error: unexpected }
src/github.com/getlantern/flashlight/flashlight-prod.go:64: syntax error: unexpected =, expecting )
src/github.com/getlantern/flashlight/flashlight-prod.go:126: syntax error: unexpected =, expecting )
make: *** [darwin-amd64] Error 2
```


@fjvallarino Are the assertions the same in both cases?


Es "Crear Nuevo Grupo Buses"


You can safely remove the reinterpret_cast now. The original author agrees.
```cpp
rgb = registered_data[512*r+c];
```


So I think it's only hit for production builds...


It somehow looks like it got merged here. We'd like minimum to no changes in the original design for the first iteration. Please revert the change and move the original methods in the PAL for Windows.


Es " Ver Datos de Grupo Buses"


This return is useless.


double check


Is this safe to do in all cases (file & directory), or should this only happen when `ftype == "link"` and we're following links (so we want to change ensure to the type of thing the link points to)?


This return is useless too.


Shouldn't the new date be 06 JUN 1841?
And also should be applied to app/Census/CensusOfWales1841.php?


Inconsistent indentation.


Es "Modificar Vehículo"


t.Fatalf instead?


Es "Ver Datos de Vehículo"


@ArthurVilaca Ocorreu erro de conflito nos arquivos
 Merge conflict in frontend/scripts/src/wizard_online_order_sale.js
 Merge conflict in backend/application/MilesBench/Controller/OnlineOrder.php
 Merge conflict in backend/application/MilesBench/MilesBench/Model/__CG__Businesspartner.php
 Merge conflict in frontend/app/sale/wizard_online_order_sale.html

Para visualizá-los execute os seguintes comandos:
git checkout issue_1
git pull origin master

Abrir os arquivos e procurar <<<<<<< HEAD
Ele destaca o que está no master e o que está na sua máquina, vc deve deixar o correto e fazer novamente o git add e commit.


From [this](http://stackoverflow.com/a/32511849/242583):

> It includes Python modules that can configure up to 256 curses colors. However curses cannot handle more than 256 color-pairs which means no more than 16 colors if the application will need to handle the 256 color pairs to expose or hide text.

But you can see the correct on your system, then this statement can't be universally true, right?


```
{
  "Major":1,
  "Minor":"0",
  "Patch":1,
  "PreReleaseTag":"",
  "PreReleaseTagWithDash":"",
  "BuildMetaData":1,
  "BuildMetaDataPadded":"0001",
  "FullBuildMetaData":"1.Branch.master.Sha.edb7a14a551d3f31300957e1593bd358eae68bba",
  "MajorMinorPatch":"1.0.1",
  "SemVer":"1.0.1",
  "LegacySemVer":"1.0.1",
  "LegacySemVerPadded":"1.0.1",
  "AssemblySemVer":"1.0.1.0",
  "FullSemVer":"1.0.1+1",
  "InformationalVersion":"1.0.1+1.Branch.master.Sha.edb7a14a551d3f31300957e1593bd358eae68bba",
  "BranchName":"master",
  "Sha":"edb7a14a551d3f31300957e1593bd358eae68bba",
  "NuGetVersionV2":"1.0.1",
  "NuGetVersion":"1.0.1",
  "CommitDate":"2016-02-03"
}
```


You can use
`git rm --cached **/PluginData/MechJeb2/*.cfg`
to get rid of files from your git repository that got needlessly added


oh, you're right! just added it back (now in `EntitySearcher`).
I got confused with the nested for loops, what I should have done is replaced `if (!add) { break; }` with `break;` because `!add` is always true in that case


(comment for linking)


It was already like that. I didn't change anything.


Well it's really up to the user. _blank are discouraged as the user is supposed to be the one choosing to change the current page or open a new one.


@DarkSilencerZA, don't even have to do that. Just look at the sql file.


oops yes.


Ahh.  I'll fix that.


Maybe this is silly but I wanted to add the following utility to test plugins with the bad data parameter added:

  def addBadDataParam(parameterMock : OperatorParametersMock,
                    value : String = HdfsParameterUtils.badDataReportNO) =
    parameterMock.setValue(HdfsParameterUtils.badDataReportParameterID, value)


Fixed


This is a really bizarre check. You're getting the index value of a PC cock that would fit in Penny but then getting the volume of Pennys cock for the index of the PCs cock. Pretty sure you dun fucked up here.


Would be good if you could provide more info about your use cases and things that you think are missing in `offline-plugin`. If you have time for it, of course.


I've been going with the assumption that we want to preserve `source` in the inlined catalog. Did something break if you left `source` in?


I don't plan to PR this yet as I only change it to works for my setup. The ignoreSearch part is incomplete (but good enough for me). But I will file an issue.


@MikaelSmith and I talked about that a bit and couldn't think of a scenario where we shouldn't do it. Do you think it'd be best to only do this is the specific case of followed links for now?


Before commit:
```
{
  "Major":1,
  "Minor":"0",
  "Patch":1,
  "PreReleaseTag":"versiontest.1",
  "PreReleaseTagWithDash":"-versiontest.1",
  "BuildMetaData":1,
  "BuildMetaDataPadded":"0001",
  "FullBuildMetaData":"1.Branch.versiontest.Sha.edb7a14a551d3f31300957e1593bd358eae68bba",
  "MajorMinorPatch":"1.0.1",
  "SemVer":"1.0.1-versiontest.1",
  "LegacySemVer":"1.0.1-versiontest1",
  "LegacySemVerPadded":"1.0.1-versiontest0001",
  "AssemblySemVer":"1.0.1.0",
  "FullSemVer":"1.0.1-versiontest.1+1",
  "InformationalVersion":"1.0.1-versiontest.1+1.Branch.versiontest.Sha.edb7a14a551d3f31300957e1593bd358eae68bba",
  "BranchName":"versiontest",
  "Sha":"edb7a14a551d3f31300957e1593bd358eae68bba",
  "NuGetVersionV2":"1.0.1-versiontest0001",
  "NuGetVersion":"1.0.1-versiontest0001",
  "CommitDate":"2016-02-03"
}
```
After commit:
```
{
  "Major":1,
  "Minor":"0",
  "Patch":1,
  "PreReleaseTag":"versiontest.1",
  "PreReleaseTagWithDash":"-versiontest.1",
  "BuildMetaData":2,
  "BuildMetaDataPadded":"0002",
  "FullBuildMetaData":"2.Branch.versiontest.Sha.fd4a264e0cbecd3f0107e7eda3b09025cb183cf4",
  "MajorMinorPatch":"1.0.1",
  "SemVer":"1.0.1-versiontest.1",
  "LegacySemVer":"1.0.1-versiontest1",
  "LegacySemVerPadded":"1.0.1-versiontest0001",
  "AssemblySemVer":"1.0.1.0",
  "FullSemVer":"1.0.1-versiontest.1+2",
  "InformationalVersion":"1.0.1-versiontest.1+2.Branch.versiontest.Sha.fd4a264e0cbecd3f0107e7eda3b09025cb183cf4",
  "BranchName":"versiontest",
  "Sha":"fd4a264e0cbecd3f0107e7eda3b09025cb183cf4",
  "NuGetVersionV2":"1.0.1-versiontest0001",
  "NuGetVersion":"1.0.1-versiontest0001",
  "CommitDate":"2016-02-03"
}
```


@ashnur I thought about that, and besides being more effort (I haven't published that many npm modules) npm seems more permanent and would require both a git fork and an npm fork. This seemed the fastest and most ephemeral approach.


all tests pass:
success: http://buildbot.bos1/builders/test-cuttlefish/builds/176


saw approval from mrallen1
at https://github.com/basho/riak_core/commit/779d665fb924632beccd18121704568727478503


merging basho/riak_core/dr/update_leveldb_to_2.0.10 = 779d665f into borshop-integration-808-dr/update_leveldb_to_2.0.10


basho/riak_core/dr/update_leveldb_to_2.0.10 = 779d665f merged ok, testing candidate = 1073c253


Thanks!
Done.


Ah, I see. The gotos. Declare `el []float64` in the block above with the todo to clean up later.


@Regaerd Just saw the comment about mine. I commented out the `try` block, it meant no error thrown at me. Also wanted to correct/revise the comment before previous, the overlapping wasn't the cause of this issue also it should be fixed, anyway.


Many thanks!


Good Advice. Thanks!
Done.


all tests pass:
success: http://buildbot.bos1/builders/test-riak_kv/builds/3162


Based on examples in nodejs/node-addon-examples: https://github.com/nodejs/node-addon-examples/blob/master/6_object_wrap/nan/myobject.cc#L11-L13


Test.


```
{
  "Major":1,
  "Minor":1,
  "Patch":"0",
  "PreReleaseTag":"versiontest.1",
  "PreReleaseTagWithDash":"-versiontest.1",
  "BuildMetaData":3,
  "BuildMetaDataPadded":"0003",
  "FullBuildMetaData":"3.Branch.versiontest.Sha.9331628fb45ad4d906f4c21ba844c9ff28b3ea06",
  "MajorMinorPatch":"1.1.0",
  "SemVer":"1.1.0-versiontest.1",
  "LegacySemVer":"1.1.0-versiontest1",
  "LegacySemVerPadded":"1.1.0-versiontest0001",
  "AssemblySemVer":"1.1.0.0",
  "FullSemVer":"1.1.0-versiontest.1+3",
  "InformationalVersion":"1.1.0-versiontest.1+3.Branch.versiontest.Sha.9331628fb45ad4d906f4c21ba844c9ff28b3ea06",
  "BranchName":"versiontest",
  "Sha":"9331628fb45ad4d906f4c21ba844c9ff28b3ea06",
  "NuGetVersionV2":"1.1.0-versiontest0001",
  "NuGetVersion":"1.1.0-versiontest0001",
  "CommitDate":"2016-02-03"
}
```


@joshcooper yeah, Puppet considers `source` and `target` to be mutually exclusive: `You cannot specify more than one of content, source, target`


can we follow what v2 does? we have one stresser per grpcURL?


@tejohnso Yes, the difference is in one case the promise is resolved and in the other it is rejected.


v2 only


- [ ] search for existing stencil files that reference  ./data-unshared/  and update them. Don't change anything in the original repo.
I think, this is done as well. But, I will be mindful and keep correcting if I see this again in the files I am reviewing. @wibeasley


Sure, will fix. Thanks.


let's make this simpler. each stresser only takes one endpoint.


It definitely needs to happen if resource[:ensure] was `link` or `present`. I think it would also override if given `file` or `directory` and the source was the opposite.


So this file was renamed form ImageViewerView.jsx, I created a new ImageViewView.jsx at higher level of the react tree


I don't see how this change improves anything.


we should be able to retry if one connection is down. remember that we do kill the remote machines regularly.


Alright, [this one](http://stackoverflow.com/a/13867170/242583) is more believable. It can't have more than 256 pairs before ABI 6, then how did you have the colors worked correctly?

Are you also using ncurses 6? (ABI 6, I presume meaning the ABI in ncurses 6.0)


We already have `HexToString` and `StringToHex`. If we avoid using streams, we can also avoid duplicating this code.


Looks like you have a typo there. Should be canola oil.


So if you had a ball size of 9 with 2 balls, this would give you a ball size of 81 internally, and when you have 0 balls, the game still acts as if you had two for cum calculations, meaning you would have a massive, exponential gain in cum volume.

Really, ballSizeRaw shouldn't be messed with at all, aside from potentially lowering it if it is above 20 or 30.


This is tightly coupled with the SslStreamPal's declarations and static adapter methods for SecurityStatusPal. Since we're reusing this type, it's best if you could extract a separate class (e.g. SecurityStatusPalAdapter) and move all the adapters and helper methods related to this type there.

My reasoning is that if somehow we would change the order of the enums in the SecurityStatusPal.cs file, this check won't be true anymore.


nice :thumbsup:


> Den 03/02/2016 kl. 20.09 skrev Matt Browne <notifications@github.com>:
>
> So how do we solve this? Without some change to the language or at least
> a prominent addition to the documentation, this would surely lead to
> bugs due to uninitialized objects.

All trygve objects are, by default, initialized to the value null. Most attempts to invoke scripts on objects thus bound will result in a run-time error. There are notable exceptions, such as comparison with null.

We could make arbitrary assignments suitable to the types of the objects. For int s I could just initialize them all to 5280. it is as good a number from the set of integers as any other. If the compiler were prescient and could know that the value was going to be involved only in addition, it could assign the value of 0; if only in multiplication, the value 1; and then, for subtraction or division, it would depend on whether it was being used as the left- or right-hand operand of the operator. Of course, these predictions are in general NP-hard to calculate. The usual convention is to silently initialize integers to zero, without any hint that that is what the developer intended, with the result that the program silently fails by producing the wrong answer unless coincidence offers the happenstance situation that it would have been zero anyhow. The chances of that being true are about 1 in 2 to the power of 64 minus 1.

Do you have a better oracle that I should use for determining the values to which integers, Strings, and doubles should be bound? How about boolean? Is the natural state of a Boolean value true, or false?

This is just silly. If the end user doesn’t specify how things are initialized, they are, and should be, in trouble. The run-time can either produce a fatal error (use of uninitialized value) or just terminate in undefined behavior. Default initialization to other than null results in the latter.


@Eric89GXL is `tsss_dr` being hard coded in this commit?


This config *might* also need to be an Array (getArrayProperty). It sounds like there could be multiple in the list. Strangely, though, I don't even see this `default-roles` configuration listed in the default `authentication-shibboleth.cfg`.


还有第二条方法没走，按照规则构建，比如有人用DP思想直接构建


Done.


PlantCompBaseClassAgain (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (2056 of 2056 tests passed, 329 test warnings)

 * 549 tests had: AUD diffs.
 * 329 tests had: BND diffs.
 * 2 tests had: EIO diffs.
 * 2 tests had: ESO small diffs.
 * 8 tests had: MDD diffs.
 * 8 tests had: MTD diffs.
 * 8 tests had: RDD diffs.

![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)


Are any other code changes necessary here?


Most other Wednesdays should work based on what I've currently got scheduled for the rest of the semester. Switching could work if it'll work for Kyle. Let's bring it up at today's meeting and see if anyone currently on the schedule would be willing to switch?


Gratuitous renaming.


can we also verify that `r[:ensure]` is set to `file`?


Another one.


Nit: we could have kept the old signatures as well.


Is this paragraph still accurate?


if we continue to strip `source` it would be good to verify it's not present here.


TeamCity pull requests :: netty [Build 1204](https://garage.netty.io/teamcity/viewLog.html?buildId=5400&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 6409, ignored: 23 Build time: 00:43:19


TeamCity pull requests :: netty [Build 1205](https://garage.netty.io/teamcity/viewLog.html?buildId=5401&buildTypeId=pulls_netty&guest=1) is now running


Is this list still accurate?


all tests pass:
success: http://buildbot.bos1/builders/test-riak_core/builds/1839


I'm just reading the javadocs you added, if you just update doesn't that mean it a test makes changes to data then changes stick around for ensuing tests, may be am not understanding this


LGTM


Omg I changed the "exhaust" for "exhausting" because I was missing a "to exhaust"... Because the correct form is " in order not to exhaust"... Sorry, I feel bad and stupid u_u


:laughing:


FYI 1.0.39 of the helper and 3.6.23 of the skin just came thru updates...

so far so good only thing I have seen so far is helper error in logs,..

```
14:45:34 T:7160   ERROR: Skin Helper Service --> ERROR in createNFO --> cannot serialize [u'1a425bbd-cca4-4b2c-aeb7-71cb176c828a'] (type list)
14:45:34 T:7160   ERROR: Traceback (most recent call last):
14:45:34 T:7160   ERROR:   File "C:\Users\Mario\AppData\Roaming\Kodi\addons\script.skin.helper.service\resources\lib\ArtworkUtils.py", line 484, in createNFO
14:45:34 T:3232  NOTICE: [StorageServer-2.5.4] '_sqlGet' : 'cacheget_data_new|f609fe529686a733c705f1bb879a7337| - ArtworkDownloader'
14:45:34 T:7160   ERROR:     xmlstring = xmltree.tostring(tree.getroot(), encoding="utf-8")
14:45:34 T:7160   ERROR:   File "C:\Program Files (x86)\Kodi\system\python\Lib\xml\etree\ElementTree.py", line 1126, in tostring
14:45:34 T:3232  NOTICE: [StorageServer-2.5.4] 'run' : 'Done'
14:45:34 T:7160   ERROR:     ElementTree(element).write(file, encoding, method=method)
14:45:34 T:7160   ERROR:   File "C:\Program Files (x86)\Kodi\system\python\Lib\xml\etree\ElementTree.py", line 820, in write
14:45:34 T:7160   ERROR:     serialize(write, self._root, encoding, qnames, namespaces)
14:45:34 T:7160   ERROR:   File "C:\Program Files (x86)\Kodi\system\python\Lib\xml\etree\ElementTree.py", line 939, in _serialize_xml
14:45:34 T:7160   ERROR:     _serialize_xml(write, e, encoding, qnames, None)
14:45:34 T:7160   ERROR:   File "C:\Program Files (x86)\Kodi\system\python\Lib\xml\etree\ElementTree.py", line 937, in _serialize_xml
14:45:34 T:7160   ERROR:     write(_escape_cdata(text, encoding))
14:45:34 T:7160   ERROR:   File "C:\Program Files (x86)\Kodi\system\python\Lib\xml\etree\ElementTree.py", line 1075, in _escape_cdata
14:45:34 T:7160   ERROR:     _raise_serialization_error(text)
14:45:34 T:7160   ERROR:   File "C:\Program Files (x86)\Kodi\system\python\Lib\xml\etree\ElementTree.py", line 1052, in _raise_serialization_error
14:45:34 T:7160   ERROR:     "cannot serialize %r (type %s)" % (text, type(text).__name__)
14:45:34 T:7160   ERROR: TypeError: cannot serialize [u'1a425bbd-cca4-4b2c-aeb7-71cb176c828a'] (type list)
```


Was this the intent?


Same for this function: looks like it was removed.


I looks really strange that self_suspend_internal drops the lock but async_suspend_internal doesn't.

I don't see why both can't have the same behavior


Rotting on my desktop for months now ... with all the others ...


Gratuitous renaming.


That's reasonable too. I think @Raynos will have a say in it anyway.


This needs to be tested well.  I was very surprised at this problem making it this far.

The problem was found on 'input' files for the TS for SXC type.  There was a line that was 40 bytes, and all sorts of multi type warnings were output, even though the 'real' lines have $sxc$ signature, and should not need any --format=sxc overload.

NOTE, I am getting a failure for source in ax-crypt, BUT I was getting that (exact same hash in error) even before this commit, and this commit should have nothing to do with that error.


Сделано +


Gratuitous change.


No problem. Thanks for the great plugin. :+1:


I fail to see the point of this change. It has too much gratuitous changes.


Jordan Knight


I think it'd be preferable to raise if `ensure => file`, but source is a directory or vice-versa. Since we only inline content from a git repo (where both the manifest and module content reside), I think any conflict means something is misconfigured.

Does `present` mean set `ensure` to whatever the source `ftype` is? Or does it mean ensure a file exists, possibly empty.


it's being turned into an option, you should be able to set `p.tsss_dur` and have it take effect


>You overlapped the index

Haha... I saw this and wondered what kind of trouble it would cause.

>But you can see the correct on your system, then this statement can't be universally true, right?

I guess I didn't understand what the issue was. No, it wasn't working right.


Out of curiosity, do you have an estimated time for when DataTables with full support for Foundation 6 will be? I'm currently working on an app with a team that is scheduled to release in about two months and we upgraded to Foundation 6 last month. DataTables is core part of our UI and we're hoping that we don't have to downgrade Foundation. Or, how is 1.10 looking? Would we be see a lot of breakage if we pulled down what you had for 1.10?

Any feedback would be great as I need to pass it on to my team. Thank you!


There are worst things rotting on mine. I poke some them here and there to check if they are still alive.


On these, you don't have to use a print command. the `>>> ` prefix is meant to indicate the Python command prompt, where if you enter a variable name by itself, it responds by printing the value.


This is copy of the code on line 466. What is the reason we're adding the new Debug.Fail here?


This should be the libva api version because that's what libva.pc uses, so you will get a miss match

```
configure:30242: checking for LIBVA
configure:30249: $PKG_CONFIG --exists --print-errors "$libva_pkgs"
Requested 'libva >= 1.3.2' but version of libva is 0.38.1
configure:30252: $? = 1
configure:30266: $PKG_CONFIG --exists --print-errors "$libva_pkgs"
Requested 'libva >= 1.3.2' but version of libva is 0.38.1
configure:30269: $? = 1
configure:30283: result: no
Requested 'libva >= 1.3.2' but version of libva is 0.38.1
configure:30299: result: == Could not find libva. VAAPI support disabled. ==
```


This seems to do the trick:

```diff
diff --git a/pyclock.py b/pyclock.py
index bee88f1..d16defb 100755
--- a/pyclock.py
+++ b/pyclock.py
@@ -31,7 +31,6 @@ class PyClock(object):
     kDEFAULT_WIDTH = 1
     kDEFAULT_HEIGHT = 1
     kDEFAULT_COLOR = 2
-    kFORE_COLOR_OFFSET = 10
     kDEFAULT_FORMAT = '%I%M%S'

     def __init__(self, stdscr, clock_args = None):
@@ -59,8 +58,7 @@ class PyClock(object):
         # some linux terminals throw an exception after 7, but osx supports all 10
         try:
             for i in range(256 if clock_args.color > 10 else 10):
-                curses.init_pair(i, -1, i)
-                curses.init_pair(i + self.kFORE_COLOR_OFFSET, i, -1)
+                curses.init_pair(i + 1, i, -1)
                 self.color_range = i
         except:
             pass
@@ -117,8 +115,7 @@ class PyClock(object):
     def color_index(self, value):
         self._color_index = value if value <= self.color_range else randint(0, self.color_range)

-        self.back_color = curses.color_pair(self._color_index)
-        self.fore_color = curses.color_pair(self._color_index + self.kFORE_COLOR_OFFSET)
+        self.back_color = curses.color_pair(1 + self._color_index)
         self.needs_full_update = True

     @property
@@ -173,7 +170,7 @@ class PyClock(object):

     def draw_number(self, x_origin, y_origin, template_index):
         if self.width * self.height == 0:
-            self.stdscr.addstr(y_origin, x_origin, str(template_index), self.fore_color)
+            self.stdscr.addstr(y_origin, x_origin, str(template_index), self.back_color)
             return

         y = y_origin
@@ -183,20 +180,20 @@ class PyClock(object):
             for h in range(self.height):
                 x = x_origin
                 for c in range(length):
-                    color = self.back_color if line[c] else 0
+                    color = curses.A_REVERSE | self.back_color if line[c] else 0
                     for w in range(self.width):
-                        self.stdscr.addstr(y, x, self.kSQUARE, color)
+                        self.stdscr.addstr(y, x, self.kSQUARE,color)
                         x += 1
                 y += 1

     def draw_punctuation(self, x_origin, y_origin, template_index):
         if self.width * self.height == 0:
-            self.stdscr.addstr(y_origin, x_origin, ':', self.fore_color)
+            self.stdscr.addstr(y_origin, x_origin, ':',  self.back_color)
             return

         y = y_origin
         for r in range(self.char_height):
-            color = self.back_color if self.templates[r][template_index] else 0
+            color = curses.A_REVERSE | self.back_color if self.templates[r][template_index] else 0
             for h in range(self.height):
                 x = x_origin
                 for w in range(self.width):
```


There shouldn't be a comma - it's just `<%= nivo_slider id: 'slider' do %>`


I'm trying to get the get the Assets from the Activity instance using pApplet.getActivity().getAssets() currently, so I can save/retrieve files. Do you have a solution for this?


On Julia 0.4.1 I get:
``` julia
julia> using Lumberjack

julia> error("asd")
WARNING: both Lumberjack and Base export "error"; uses of it in module Main must be qualified
ERROR: UndefVarError: error not defined
```


These three variables, since they now hold jQuery objects, should be named with `$`, like `$successMessage`.


All this hiding and showing seems awfully heavy. Why not just have one element, `draft-message`, and insert whatever content you need from the JS?


@aquamatthias @brndnmtthws Are there any code changes necessary on this page?


[Use Cases.pdf](https://github.com/troyc/Group2/files/116557/Use.Cases.pdf)


Should this constant be moved up to common/Constants? It would be nice to not duplicate this value in SyncConfigurationManager.


**SUCCESS** - TeamCity HotRod C++ Client :: Pull requests monitor - RHEL [Build 363](http://ci.infinispan.org/viewLog.html?buildId=34959&buildTypeId=HotRodCpp_PullRequestsMonitorRhel6&guest=1) for 421b761b600d8240c77280fdceec5c8d2c383b73
Tests: 0, 0 failed (0 new), 0 ignored. Build time: 00:08:11


That works for everything?
Using `curses.A_REVERSE` is clever. Makes the whole thing a lot nicer


Better if you don't use fully qualified reference to the enumerations. It's long already, introduces a long of redundancy, and generally makes it harder to read. You can either do:

```python
from docx.enum.text import WD_COLOR_INDEX
```

at the top of the protocol "session", or do like I do and just leave that detail off altogether in this analysis context.

So this would become:

```python
font.highlight_color = WD_COLOR_INDEX.YELLOW
```


I'm cutting this out, mainly because there's nothing set up for NPCs to retighten over time.


I stand corrected. Not sure why everything I've been doing over past few days is feeling like a mistake.


Thanks for all of your work with Wave! I was wondering if you could explain the difference between CHUNK and sample_width. They both seem to represent the length of each sample. Is this so?


Thank you very much for the quick response! It is so much nicer to look at a table in markdown -- I was having to replace all of the markdown tables with html versions. I will try this ASAP (as soon as I figure out how to update Pandoc).


The comment on this line is no longer needed (and isn't actually accurate anymore)


A comment on a diff.


Good catch.


sddsdsdd


Dude the cone of silence is not good for my anxiety. Seriously your starting to make me depressed.


The 'light' option seems to have disappeared. It's not available in the ui.header-color drop-down anymore.


Do we need this as a bean? Couldn't the dataflowH2Server() bean just call this method after the server is started and then close the connection after the database is created. No need to keep the connection open AFAICT.


Actually, I finally found a database that seems to solve everything it needs to for a ConceptNet index. Check out the `kyotocabinet` branch.


@Olion17 These are the C++ standards, starting with ISO C++03
http://cs.nyu.edu/courses/fall11/CSCI-GA.2110-003/documents/c++2003std.pdf
...
http://www.open-std.org/jtc1/sc22/wg21/docs/papers/2009/n2857.pdf
http://www.open-std.org/jtc1/sc22/wg21/docs/papers/2010/n3035.pdf
http://www.open-std.org/jtc1/sc22/wg21/docs/papers/2011/n3242.pdf
...
Look at the section 4.7 - "Integral conversion", subsection 4 **"If the source type is bool, the value false is converted to zero and the value true is converted to one"**

Unless someone uses an old, old, old compiler, which doesn`t conform to at least C++03 standard, then yes, you are right about bool to int8 conversion :)


@adaur missing the link to
http://localhost:8081/home/manualSelect?show=175001&episode=3&season=5


Gratuitous changes.


From the consensus on the previous comment chain, I think this builder method and the one below can be completely deleted instead of made package scope.


can you comment on the reason we are checking observable for flags and `exec`?


Ah.  I missed the catch vs then :)


The following doc(s) have been deployed to the [1275tokens](http://1275tokens-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/local/cli/machine-tokens](http://1275tokens-static-docs.pantheon.io/docs/articles/local/cli/machine-tokens)
- [source/docs/assets/images/machine-token-ready.png](https://github.com/pantheon-systems/documentation/commit/968d2ee2c9c8a64392dd2b06f6af771b39aa182c/source/docs/assets/images/machine-token-ready.png)
- [source/docs/assets/images/machine-token-ready1.png](https://github.com/pantheon-systems/documentation/commit/968d2ee2c9c8a64392dd2b06f6af771b39aa182c/source/docs/assets/images/machine-token-ready1.png)


Awesome. I wonder about including the variable changes in the main change callback optionally but I can see why you split it out. (Otherwise you end up potentially breaking back compat). I'm already working on adding programs and I have the same dilemma there.

I'll happily update the unit tests to handle the variables. I'll also update my fake-isy project to support variables so it can properly be included in the CI tests.


:clap: :tada:


Message should be in english


This changes orders WRT to locking of *ares*.


@lattner, what do you think about the SIL changes here? Is there a more principled way to do this?


Does this mean external libraries work now? (i.e. use system LIB_X)?


Why now null?


// HelloWorld.cpp : Defines the entry point for the console application.
//

#include   "stdafx.h"

using   namespace  std;

int   _tmain ( int   argc ,  _TCHAR *  argv [])
{
	cout <<  "Hello World! :)" <<endl;
	 return  0;
}


Can confirm, @SN4T14 is in fact a knob.


We're building a new sidenav (new in Projects) and extending that as a "theme" in Phabricator. So it makes more sense for us to support less colors (since it's more than just a header) but have more consistency throughout the UI.


Dear @dorka-lelylan, I know it's a bit techy, but would you love to try to make your first [pull request](https://help.github.com/articles/using-pull-requests/)? :)

In this way I can merge your corrections into the repo. Hugs


Did you also change it when it's called in cleanThreads()?


> `lastInputTypee `

is a typo it should be `lastInputType`


If you're moving to dot notation, why not change this to:

```js
...
data: {
  api_key: mapzen_key,
  point: {
    lat: position[0],
    lng: position[1]
  }
},
...
```


Fixes #1.


Nice. The credit goes to @erikreedstrom.


Also, once you implement this fully, it will be good to have test where getter of flags overrides the `exec` instead of directly overriding of `exec`.


still didn't test it. Probably not.


Not sure whether this matters, but note that the schema for the `plan_product_id` db field defaults to `0` rather than `1` [here](https://github.com/wordpress-mobile/WordPress-Android/blob/feature/plans-add-item/WordPress/src/main/java/org/wordpress/android/WordPressDB.java#L221).


cool


No benchmark regressions (see attachment).

[perf_unicode.txt](https://github.com/suwc/ChakraCore/files/116578/perf_unicode.txt)


stupid bash lol


@hjanetzek I removed all dead code and crossed feature usage between fontstash/alfons in this commit so that we can more easily start restructuring what tangram-es needs for alfons (basically splitting alfonsStyle.cpp).
We will need to add back the sdf header (either by putting the header within alfons or within tangram-es) I guess that it's something a user from alfons would need so a parameter to enable it or not through the API would make sense, so that you directly have a textured distance field back while implementing `alf::TextureCallback`.


Yes, probably.


This looks like a refactoring mistake: as rowRef does not exist in this scope


Actually, as far as I know, it is completely arbitrary. My aversion to such a "magic number" was the impetus for gh-466. No one demonstrated a strong conviction for any value in particular, so I decided to simply modularize a "apparently acceptable" value like this so that I could move forward with implementation. My thinking is that consumers could make local modifications if necessary, although the vagueness of such a contract is pretty dissatisfying to me. I'm all ears for improvements to the documentation or to the organization!


what's that comma for?


you added the battle warscore, then vanilla added this, maybe you don't want both


inputObject doesn't exist in this scope since we got the input from MouseButton1Click


Hmm, I don't think this will work. Marathon-lb is not a drop-in replacement for `haproxy-marathon-bridge`. I would just remove the references to it, and point to the docs for marathon-lb instead.


Same as above, inputObject doesn't exist in this scope since we got the input from MouseButton1Click


oh noes! will fix whitespace asap


To avoid future crashes with different exceptions, I really think it would be better to use:
`except Exception as e:`


Yeah I was alerted to that a minute ago. Can do, and thanks.

—

Suzanne Scala
Technical writer, Mesosphere

> On Feb 3, 2016, at 3:19 PM, Brenden Matthews <notifications@github.com> wrote:
>
> Hmm, I don't think this will work. Marathon-lb is not a drop-in replacement for haproxy-marathon-bridge. I would just remove the references to it, and point to the docs for marathon-lb instead.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/mesosphere/marathon/commit/b013d53fb50b5f3ea6d3473a35acfb0b698e3bd1#commitcomment-15872514>.
>


Trailing space removal could have been done in another commit.
Otherwise clean work. Will be tested and merged asap.
Thanks for the helping hand.


ack


It can take a long while to transpond the sound sample for all the ranges (like, a few minutes on a Pi2). If you're not seeing the "DONE" message then it may still be re-sampling...


oops. my lazily unfinished local changes got pulled in :flushed: will fix


Nit: I'd omit the "Controller" in the name.


hi @jamesreggio Why are the `setTimeout` wrappers added?

Thanks!


It would also probably make sense to add here:

* a distinct counter of all tag keys by issue,
* a (non-distinct) counter of the number of events seen by tag key, so that we can display top tag values as a percentage of all events seen for that tag key.


The runtime warning is usually just a sign that you Ctrl-C'd the program last time, so it didn't finish up and cleanup the GPIO pins - it's save to ignore, usually :)


This only works on non-bare repositories, right?


nothing to see here, just vanilla stripping out a superfluous OR = {}


It'd be nice if in future revisions of the TSDB API we had real composite key support because this is going to get real nasty real quick. :(


Updated the man-page


why are the cxx flags here different from the flags in the FileIO.spec?


Can you use `known_heap_size!` here? Or does `()` interact badly with macros?


Can anyone else validate this commit? I'll be gone for a few hours.
Also the added sample_count() REALLY should be probed by the emulators for proper audio output buffering.


^ my copy paste failure, should be ROOT. kill me


Why does this take a buffer parameter? I thought the idea was for the underlying source author to write to `.view` and then call `.respond(bytesWritten)` when they're done.

Also, maybe default bytesWritten to view.byteLength? Or is that a footgun?


I think this and all the other "Tactical combat system geometry free functions" can go as well.


Hello there!


Note: Open with internet connection (google apis, js, etc.)


It's based on a commit tree, so it shouldn't matter whether it's bare or not. Submodules are per commit state, not global to a repo.


10 * 10 might be too much for the player? same thing w/ nomad_invasion


On Thu, Feb 4, 2016 at 7:01 AM, Brandon Dreager
<notifications@github.com> wrote:
>
> That works for everything?

Seems to be, but only God knows, if there is any curses left. (pun intended)

It needs to be cleaned up, the variable name is opposite, or revert to
before, just `color`, no `fore` or `back`. I will leave that to you. I
am really the true owner of this repo. So, when do I release v1.0.0?
(kidding).

> Using curses.A_REVERSE is clever. Makes the whole thing a lot nicer

Wish that were my idea, got that from the link in my previous comment.


I guess the idea is to provide an empty DB, run the tests and truncate all tables at the end (or drop the whole DB).


Umm dude, you missed some :P


Don't forget to preserve thread safe initialization, say via stdlib_atomic. I'm not sure that will be easy to do with an enum.


fk u harrij


Dont' need it.


This is a "lame" functional programming. Probably it could be written far better. Is this is acceptable? So I can do commit into your master (via separate branch).


LGTM!


Of course RambaJS rules :)


hii


I'll use a updated fork next time. :)


Thanks for the fast reply! Consider this a +1 for a light theme, then :-)

Best,
Daine

> On Feb 3, 2016, at 3:14 PM, Chad Little <notifications@github.com> wrote:
>
> We're building a new sidenav (new in Projects) and extending that as a "theme" in Phabricator. So it makes more sense for us to support less colors (since it's more than just a header) but have more consistency throughout the UI.
>
> —
> Reply to this email directly or view it on GitHub.
>


## Task 1

- [ ] Program is correctly formatted with shebang, description, code (1 pt)
- [ ] Program **does not** include `main()` function or "ifmain" statement (it should not include these per my instructions) (1 pt)
- [ ] Program functions as expected (1 pt)

## Task 2

- [ ] Program is correctly formatted with shebang, description, code (1 pt)
- [ ] Program correctly demonstrates the primary reason to use `main()` function and "ifmain" statement (2 pt)

## Task 3

- [ ] Program is correctly formatted with shebang, description, code (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program includes a function to accomplish 1, 2, & 3 (1 pt)
- [ ] Function returns unmodified `π` and prints it (1 pt)

## Task 4

- [ ] Program is correctly formatted with shebang, description, code (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program includes a function to approximate *e* (1 pt)
- [ ] Program returns the approximate value of *e* and prints it (1 pt)

## Task 5

- [ ] Program is correctly formatted with shebang, description, code (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program is structured and functions as requested (2 pts)


TeamCity pull requests :: netty [Build 1205](https://garage.netty.io/teamcity/viewLog.html?buildId=5401&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 6409, ignored: 23 Build time: 00:43:49


`coment` -> `comment`


Another thing I would comment about would be implementing pagination instead of loading everything. That would make performance much better.


Thanks Davor!!


You might want to check if `windows.globalization.dll` has some better APIs to do this instead of `CharLowerW`?


`source[idx + 1]` would cause `IndexError` if last character in the file is `*`.


spoko
Właśnie robię na szybko wersję mobilną, zrobiłem alerty i biorę się za resztę formularzy. Postaram się zrobić tak, żeby nam się pliki nie nadpisały


I think I got two ideas crossed. There *are* three states:

1. before `argc` and `argv` have been set
2. after `argc` and `argv` have been set
3. after `arguments` has been computed

…but you can distinguish (1) and (2) from whether `argv` is nil. My bad.


It looks like `dataLength` isn't used anymore. Can we ditch it?


Won't need these log messages.


Remove log.


Remove.


FYI I am tired of having to define these "Isxxx" methods; filed https://github.com/tc39/ecma262/issues/354


What for? It's doesn't affect default engine behavior.


Made consistent with Horizon Stable/Liberty


cleanThreads() was already calling threadBackup even before it was named that


Generally we don't need tests that result in `RequestValidationErrors` at the `-OperationsApiTest` level.  Since this is already enforced in `DeleteApplicationRequestTest` this one is redundant and slows down an already slower test suite.


Right, technically i am breaking apis at the client level. Oh well, should not be such a big deal. At least i did not break anything on the server side.


Please check `deflatedFileHandle` for nil, and propagate `handleError` and exit if it is.


If `idx` is zero, `source[idx - 1]` would be the last character in `source`.


This worked out really nicely!!


This still doesn't work as it should. I'll debug tomorrow.


We need to make sure that `idx != 0`.


PlantCompBaseClassAgain (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (2051 of 2051 tests passed, 329 test warnings)

 * 331 tests had: AUD diffs.
 * 329 tests had: BND diffs.

![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)


@atrick /cc @trentxintong Good point!


Is this the same transmission set-up issue?


@KHarshawat
![image](https://cloud.githubusercontent.com/assets/132447/12800792/b4145f30-caa5-11e5-9941-bf07ba1abb55.png)


话说，根据tag整理，具体怎么个整理法？


So if if int is 16 bit, and given that capacity.val is defined to be int32_t, you are passing the address of a 32 bit variable to a function that expects a 16 bits one. It would be more correct to have a temporary variable with the correct type to pass to the function and then assign the result to capacity.val


thanks!


@ncannasse: This seems simple enough, but please take a quick look at it anyway.


Going to remove this since it's always true as dataController:nodeBlockAtIndexPath: is always implemented.


Will remove the commented out lines in my next update


onoez! :flushed:

Well, it was a good run while it lasted. :relieved:


I believe this is the only location it is broken, I just went back and tested Scala SQL contexts and they to, are fixed by this patch.


FIX for page_free_symbol on 4.75DEX/4.76DEX/4.78DEX

#define page_free_symbol                0x637C8

it currently points to 0x637C4


Accidental commit of updated element move functionality


Given that nodeBlockAtIndexPath is always implemented, we no longer have to call nodeAtIndexPath. In addition, we may want to add an assertion if the dataSource returns an object that is nil


Lets make the SYSLOG messages say the same thing for failure states.


@cjwest added necessary media queries for responsiveness


Calling `encodeBriefJson()` from `encodeJson()` might allow you to avoid some duplication.


We will only have one call after i remove _dataSourceImplementsAsyncNodeAtIndexPath
` [nodes addObject:[_dataSource dataController:self nodeBlockAtIndexPath:sortedIndexPaths[i]]];
`


This error is bubbled back to the main, where it's handled. This ideally should have been a specific error type.


I am talking about this kind of situation:

class AClass {
     private int x_;

     public AClass(int x) {
         x_ = x.clone;
     }

     public void test() {
         System.out.println(x_);
     }
}

{
     AClass obj = new AClass();
     ...
     obj.test();
}

Suppose that all instances of AClass should always have x_ initialized
to a value (i.e. that's an invariant). If the call to test() is right
after the AClass a = new AClass() line, then it's obvious that the error
happened because the programmer forgot to properly initialize AClass().
But if there are a lot of other things that happen between those lines
(keeping in mind that obj might be passed around before test() is
called), it would be harder to debug. This is the kind of bug that I'm
concerned about, and which I think Rune is concerned about too. Suppose
I change it to use an init() method instead:

class AClass {
     private int x_;

     public void init(int x) {
         x_ = x.clone;
     }

     public void test() {
         System.out.println(x_);
     }
}

{
     AClass obj = new AClass();
     obj.init();
}

In that case I get a compile error:
line 15: Script `init()' not declared in class `AClass'.

This is inconsistent. The compiler complains if you forget a parameter
to init() but not if you forget a parameter to the constructor.

I understand the reasoning for a default empty constructor being created
automatically if the programmer doesn't write any constructors, but if
the programmer /does/ provide one, then this hidden behavior is quite
surprising. If the compiler finds a call to new AClass() without any
parameters, why not at least give a compiler warning recommending that
the programmer be explicit about this, e.g. if you explicitly write an
empty constructor then there's no ambiguity:

class AClass {
     private int x_;

     public AClass(int x) {
         x_ = x.clone;
     }

     public AClass() {}
}

On 2/3/16 5:46 PM, James O. Coplien wrote:
>
> > Den 03/02/2016 kl. 20.09 skrev Matt Browne <notifications@github.com>:
> >
> > So how do we solve this? Without some change to the language or at
> least
> > a prominent addition to the documentation, this would surely lead to
> > bugs due to uninitialized objects.
>
> All trygve objects are, by default, initialized to the value null.
> Most attempts to invoke scripts on objects thus bound will result in a
> run-time error. There are notable exceptions, such as comparison with
> null.
>
> We could make arbitrary assignments suitable to the types of the
> objects. For int s I could just initialize them all to 5280. it is as
> good a number from the set of integers as any other. If the compiler
> were prescient and could know that the value was going to be involved
> only in addition, it could assign the value of 0; if only in
> multiplication, the value 1; and then, for subtraction or division, it
> would depend on whether it was being used as the left- or right-hand
> operand of the operator. Of course, these predictions are in general
> NP-hard to calculate. The usual convention is to silently initialize
> integers to zero, without any hint that that is what the developer
> intended, with the result that the program silently fails by producing
> the wrong answer unless coincidence offers the happenstance situation
> that it would have been zero anyhow. The chances of that being true
> are about 1 in 2 to the power of 64 minus 1.
>
> Do you have a better oracle that I should use for determining the
> values to which integers, Strings, and doubles should be bound? How
> about boolean? Is the natural state of a Boolean value true, or false?
>
> This is just silly. If the end user doesn’t specify how things are
> initialized, they are, and should be, in trouble. The run-time can
> either produce a fatal error (use of uninitialized value) or just
> terminate in undefined behavior. Default initialization to other than
> null results in the latter.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jcoplien/trygve/commit/8e70df32c622ea9a401e9603df7785740116c4b3#commitcomment-15871815>.
>


Fixed in cc9915ed37faf024d464fff5edf0075fd1e10a83


The changes in this file are unrelated to my change and happened while I was investigating a crash. Reverting


From motorola :+1:


Same here, call `encode()` from `encodeWithAllThreads()`.

Also, I'm not sure if it's worth using streams everywhere like this. There are two reasons why I'm not a big fan of this:
1) it's a bit inconsistent with the rest of the source;
2) you can use strings and still avoid copies pretty easily.


Perhaps this should be SecurityError to clue the coder in as to what kind of problem occurred with createOffer.


Same here.


and here as well.


What's with all the parentheses?  This isn't Python.


Google's style guide prefers single quotes.


ah this was my fault with trying to make optimization. Didn't realize those things needed to load before before the CSS.


Great fix!


Leaky English.  Should be message.


The nice way to write this is the following:
```
return g_string_free (res, FALSE);
```


```swift
if let obj = realm.objects(GroupParent).first {
    return obj
}
```


Can we avoid taking the full page screenshot instead of deleting it after? This can save a lot of time spent scanning long pages.


Agreed; you should try to get this merged.


Na tym polega git, ze jak zrobisz pusha to wskaże różnice.
Umawialiśmy się, że zajmę się responsywnością. Niestety nie da się tego
zrobić i przetestować porządnie kiedy nie są gotowe wszystkie podstrony.

W dniu czwartek, 4 lutego 2016 Andrzej Czarnecki <notifications@github.com>
napisał(a):

> spoko
> Właśnie robię na szybko wersję mobilną, zrobiłem alerty i biorę się za
> resztę formularzy. Postaram się zrobić tak, żeby nam się pliki nie nadpisały
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dkawecka/wolontariusze-fe/commit/09c0bee939854024027af093fce693fddb51b2b0#commitcomment-15872841>
> .
>


Hi! I'm using your character controller (great stuff! Thanks so much for making it and putting it out!) And 5.3, yes, broke it. I've done my own hacks in the code so it's customised, so I can't just grab your changes...

Being a bit noob, I'm not sure exactly which changes are needed to fix it for 5.3 - can you give me a brief explanation? Is it adding the green lines and deleting the red lines? And those other files?

Thanks again!


I think that `OperationError` is correct here.  The model we're operating within aims to provide as little information as possible about error specifics.  `SecurityError` is definitely wrong, since that applies to (attempted) violations of the browser sandbox.


Google doesn't usually use === or !== unless there are issues with null vs 0 vs '' vs undefined involved.  And contrary to sanity, == is inexplicably faster than === (by a negligible amount).


I was just about to post

<img width="380" alt="screen shot 2016-02-03 at 02 59 55" src="https://cloud.githubusercontent.com/assets/810438/12771591/3f2c37de-ca22-11e5-916e-fb69ba0cf91e.png">

Thank you for looking into this!


７行目　追加しました


Its turtle soup. If turtle is illegal shouldn't the soup also?


test


I'm out now, but found this on my way out:
https://help.github.com/articles/redirects-on-github-pages/


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 14291](https://teamcity.mesosphere.io/viewLog.html?buildId=122229&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 0, ignored: 223 Build time: 00:07:14
### Failed tests
```
null: Kubernetes e2e suite.Kubectl client Guestbook application should create and stop a working application [Conformance]: <no details avaliable>

```


@emresar: what does this do?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 14290](https://teamcity.mesosphere.io/viewLog.html?buildId=122228&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 0, ignored: 219 Build time: 00:07:40
### Failed tests
```
null: Kubernetes e2e suite.Kubectl client Guestbook application should create and stop a working application [Conformance]: <no details avaliable>

```


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 14287](https://teamcity.mesosphere.io/viewLog.html?buildId=122221&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 0, ignored: 219 Build time: 00:07:37
### Failed tests
```
null: Kubernetes e2e suite.Kubectl client Guestbook application should create and stop a working application [Conformance]: <no details avaliable>

```


Reverted to singled threaded version because multi threading is not adding any value. A thread <-> db insert rate graph is here:
http://i.imgur.com/fFzZdfr.png


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 14294](https://teamcity.mesosphere.io/viewLog.html?buildId=122232&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 219 Build time: 00:08:21


Oh my god I cant even :laughing:

thIs is GreAT sTufF.


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


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 14292](https://teamcity.mesosphere.io/viewLog.html?buildId=122230&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 216 Build time: 00:08:54


This executes in Node, where we may not have spread arguments.

I'd suggest using `this._super.apply(this, arguments);` instead.


TeamCity pull requests :: netty [Build 1188](https://garage.netty.io/teamcity/viewLog.html?buildId=5359&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 6404, ignored: 23 Build time: 00:42:46


TeamCity pull requests :: netty [Build 1189](https://garage.netty.io/teamcity/viewLog.html?buildId=5360&buildTypeId=pulls_netty&guest=1) is now running


I did fix all the merge issues


cuz turtle soup in theory, could be buy from a licensed seller.
It's the same as if you buy a bird from a petshop, or from someone that came with it in his baggage from amazon.

Turtle soup is allowed in many countries, but you can't just go to the ocean and kill them.


Yes I know that, and that's why I found this weird, In fact I deleted original folder and reinstalled the package, and found the path changed in console.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 14293](https://teamcity.mesosphere.io/viewLog.html?buildId=122231&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 216 Build time: 00:10:41


I see what you mean, I fixed it now


I see now where your coming from. I agree with yall now.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 14296](https://teamcity.mesosphere.io/viewLog.html?buildId=122234&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 174 Build time: 00:05:51


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 14295](https://teamcity.mesosphere.io/viewLog.html?buildId=122233&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 0, ignored: 219 Build time: 00:06:20
### Failed tests
```
null: Kubernetes e2e suite.Kubectl client Guestbook application should create and stop a working application [Conformance]: <no details avaliable>

```


@danielstuart14, good explanation.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 14288](https://teamcity.mesosphere.io/viewLog.html?buildId=122222&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 0, ignored: 219 Build time: 00:06:54
### Failed tests
```
null: Kubernetes e2e suite.Kubectl client Guestbook application should create and stop a working application [Conformance]: <no details avaliable>

```


The "HEX0 <<" is redundant (already included in HEX(N) macro). Also, why do we need to cast to a uint32_t after casting to uint8_t?


Sorry, I set up my .vimrc to be anal about whitespace for me.


Okay, thanks.


dfsfdsd'f's'f'd's


gjgjgj


`npm i ng-img-crop-full-extended`
looks to work ok.


Should we just update string-template templatePath to default value 'templates' and skip this hash?


Good point.


I updated it again (sometimes, it would become ugly until it "Reset" I made that time shorter) that fixed a bug when entering a command


Well it is still kind of blinky..


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 14297](https://teamcity.mesosphere.io/viewLog.html?buildId=122239&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 174 Build time: 00:05:02


Definitely a bug in `grunt-contrib-uglify`. Here's a fix:

```patch
diff --git a/Gruntfile.js b/Gruntfile.js
index e03050f..c235466 100644
--- a/Gruntfile.js
+++ b/Gruntfile.js
@@ -124,7 +124,14 @@ module.exports = function (grunt) {
     uglify : {
       options : {
         report : "min",
-        preserveComments : false,
+        preserveComments : function (node, comment) {
+            var text = comment.value;
+            var type = comment.type;
+            if (type == "comment2") {
+                // multiline comment
+                return /@preserve|@license|@cc_on/i.test(text);
+            }
+        },
         screwIE8 : "true",
         mangle: {
           eval: true
```


Fixes #8.


These changes are great :+1:

Thank you @cursive-ide !


Lors de la réinitualisation du numéro de série, il faut aussi créer un nouveau numéro de génération.


I felt like i was documenting everything under the sun, and often repeating myself.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 14298](https://teamcity.mesosphere.io/viewLog.html?buildId=122240&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 0, ignored: 219 Build time: 00:02:26
### Failed tests
```
null: Kubernetes e2e suite.Kubectl client Guestbook application should create and stop a working application [Conformance]: <no details avaliable>

```


I know. I could altogether get rid of the blink but the problem is the blue backround wouldnt show up proporly


Specific reason to not let opis load? Its very useful to quickly teleport straight to sources of lag in game


il change it so it happens less


man i feel like an idiot


kkkkk


this stops all of the other tests from running, is this just a typo?


Maybe you should just highlight the Ogar^Unlimited


Please crossport this to master. Usually you want to make such changes in master and then port it back to kilo and juno. Will make sure that master always has the latest code.


me gusta


Related Gecko try push: https://treeherder.mozilla.org/#/jobs?repo=try&revision=6c17d36cd716


Printing with PETG


Testing the size impact on the standard library:

`__swift2_reflect`: Field Descriptors
4480 bytes
`__swift2_reflstr`: Field name strings
2544 bytes
`__swift2_typeref`: Mangled type name strings
4272 bytes

This is without using 7-bit encoding for offsets.

@slavapestov - looks like there were no issues mangling the types in the standard library, so using the AST mangler looks promising.


Voici le code:
// Pour resetter le capteur à distance au besoin
int remoteReset(String command) {
  if (command == "device"){
    System.reset();
// ou juste les numéros de série.
  } else if (command == "serialNo") {
    newGenTimestamp = Time.now();
    noSerie = 0;
    savedEventCount = 0;
    Serial.printlnf("Nouvelle génération de no de série maintenant: %lu", newGenTimestamp);
    pushToPublishQueue(evNewGenSN, -1, millis());
    return 0;
  } else {
    return -1;
  }
}


I would suggest to use `SEMMacroBuildCLI` for both static and shared version.

Did you face any issue using `SEMMacroBuildCLI` for static build ?


then it would look ugly. but if you want to remove blink,, delete
```
if (this.resticks > 11) {
        this.firstl = true;
        this.resticks = 0;
    } else {
        this.resticks ++;
    }
```
in line 427 of gameserver.js


Ce code est public, en passant.


it won't blink at all?


yes


it would look fine and will look fine until you type in clear or do a command but it should still revert back to being non ugly but after a longer amount of time


Beauty!


Ok


@dribba
![yep.](http://i.kinja-img.com/gawker-media/image/upload/y5eclcqadgln59a1hwcc.jpg)


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 14302](https://teamcity.mesosphere.io/viewLog.html?buildId=122260&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 0, ignored: 219 Build time: 00:02:38
### Failed tests
```
null: Kubernetes e2e suite.Kubectl client Guestbook application should create and stop a working application [Conformance]: <no details avaliable>

```