unfortunately this code leads to a loss of wifi hotspot capabilities, each time the mobile looses mobil data signal, which happens very often at the Deutsche Bahn :-(


I made it a reference so it should be kept. Can have 2 references for free acounts


Ok bu I based all my enum code on the enum of the RenderingMode, and they
are not in capital letters, and there is that = +1 in the end.

2015-11-06 15:49 GMT+01:00 Laurent Rineau <notifications@github.com>:

> Do not change the code or the PR because of those remarks. Just keep them
> in mind for future modifications.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/maxGimeno/cgal/commit/da3edcc51a542a8e8585e943d4255be8771788c5#commitcomment-14243009>
> .
>


2 space 003


Let's log the exception message as well - $e->getMessage()


Bonus comment end


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


this is maybe awkward because it includes a `<script>` tag, so we'd be loading that script for every tweet on the page. It's designed to work that way and will be cached after the first invocation, though, so I think it's fine.


I'm not sure that we want to have a single blank line before namespace declarations, can you disable this in your code style fixer?


Minor: Prefer to use the named constants in `website.util.permissions`

```python
if not node.has_permission(user, ADMIN):
    return False
```


2 space 010


2 space James Bond


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


Ah OK. I hadn't even noticed the green margin control, hah!

Github issues are a good way to collect bugs, but it doesn't look like they're enabled for this fork - I think if you go into your repository settings you can can enable issues quite easily.


I use [this configuration file](https://github.com/concrete5/concrete5/blob/develop/.php_cs)...


I double checked PSR-5, it looks like there won't be a `@type` at all so changing these is smart.


Not sure what these strings mean.


Here is the current output, not all of these are issues but some are.  @jasondegraw and @balbano you guys seemed to use return by reference often, this is safe in C++ but [not with SWIG](http://nickdarnell.com/swig-and-a-miss/).  Let's chat about this so we can clean things up going forward.

[Uploading report.txt…]()


If it should be removed than yes.
The PR was opened before the other was accepted, so maybe I forgot something when I rebased.

> On 06 Nov 2015, at 16:46, Naveen Kuppuswamy <notifications@github.com> wrote:
>
> why does the wbm_setWorldLink still exist? Shall I remove it? @francesco-romano <https://github.com/francesco-romano> @traversaro <https://github.com/traversaro>
> —
> Reply to this email directly or view it on GitHub <https://github.com/robotology/mex-wholebodymodel/commit/efc77a2feb1b8ec13f9693770cf3c1ce07cc2f1a#commitcomment-14244657>.
>


Missing the (c)


Shall I create a PR to update the official `.php_cs`?


in a world where NARS must coexist with other software and protocols, i do not think its helpful to claim a monopoly on "logic".

since meta-NAL is parsed by the Narsese Parser which we've used .nal file extension with, the .nal extension still applies but maybe it should be .meta.nal to indicate rules being separate from other experience files.


I don't think we need to updgrate the version again since none of this code has been merged into develop yet. Nobody but the both of us know about the former TestSiteService and  the new TestCenterService anyway. What do you think?


much wow amazing code


Fixed. Thank! https://github.com/movabletype/movabletype/commit/593e20e56e7f28ef32f3ae33be59a7eccac5cdf6


Was just wondering what the quote was for. It reads weird is all.


For the two debug sections below: $filesource and $fileid would undefined - let's just log the $sourceunpacked array to see what's in there. This can be done with one debug log, something like "File reference array is not valid, with the $sourceunpacked array as debug data. We can then see what's set and what isn't based on the debug data.


Actually, in the Qt5 demos, the coding styles are somehow a mix between the coding style of CGAL and the style of Qt. And in Qt, the enum enumerators are snake-case, and the first letter is a capital letter, like you did.


default.meta.nal


Can we use this instead?


Whoops, default was changed.  Will correct.


Sumimasen.


It seems that from https://github.com/P2Pvalue/swellrt/commit/e48875b9c64eb215dd3bc7c73750785fd8a1e6c4, that ant target does not longer exists.

From a quick view of that commit changes, I guess that the new target to be called is ```dist-swellrt-client-commons``` with the following command: ```ant -f build-swellrt.xml dist-swellrt-client-commons```. But I have not try. @pablojan , am I right or am I missing something?


Thank you for your contribution!


yay! thanks


It's the default `phpdoc_type_to_var` fixer of php-cs-fixer: see https://github.com/FriendsOfPHP/PHP-CS-Fixer#usage


Will this data migration create multiple configurations if run multiple times?


Yeah unless you think we should keep it. It's strange that cs fixer adds that LF by default, neither PSR-1 nor 2 require it.


Let's log the whole $reference array.


Hi,
is there a specific reason to add this maven prerequisite on Maven 3.0.4 in the pom.xml?
+  <prerequisites>
+    <maven>3.0.4</maven>
+  </prerequisites>
I would be interested of the why of this specific version please?

For the moment I did use docker-client 3.1.10 with maven 3.0.3 as it is the maven version used by our CI platform. That maven version will one day evolve for a more recent version but it will probably take time (I'm working for a big organisation where nothing is easy when touching to the existing platforms and tools impacting a lot of projects).
So I'm feared to be stuck to 3.1.10 version of the library for the moment if maven 3.0.4 is definitely a prerequisite.

Thank you for your returns


Let's log the whole $reference array as well as the exception message - $e->getMessage()


Awesome. Thanks!

FYI - my thoughts on this class was to be a lightweight HTTP client based around cURL. The thought was that it could provide similar method names and work style to Guzzle, to make switching over easier when you decide you need something more powerful than what's provided.


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


What is this `total_range`? The number of values inside the interval? If so, these calculations are weird, for instance, adding both when both are positive? Then for [10,20] the total range is 30? Aren't you looking for something like
```C
   if (min < 0 && max > 0)
       total_range = max + min;
   else
       total_range = max - min;
```
?


@macumber I do? Where?


It's the `blankline_after_open_tag` fixer


93653546B-FuelGenShowingInABUPSwhenPV (mjwitte) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


the 4th arg to AC_ARG_WITH is when the user hasn't specified a setting.  if they do pass any webp configure flag, then the 3rd case is run.


Hi @hbtalvi thanks for trying our software.

Current android's build is broken as well as client's implementation. We have to upgrade both to be compatible with latest server improvements.

We can consider to speed up the solution of this issue if you like.

Regards


Blofeld (SP?) strikes again


xmppdotnet
==========

This repository contains source files for the list of
XMPP services hosted at http://xmpp.net/

Post to the operators@xmpp.org list if you have questions.


Re: "update docs, re-add sens as documented (even though it's wrong for non-variables", I really don't like keeping around `sens` just because it's in the docs. Please open a new issue to resolve that.


@AmiraSh
@untilovdmitriy
I modified our database (changed type of Calendar::CalendarColor to string), recreated *.edmx, updated models and viewModel


I will remove this code them.


rdsfd


I just saw a test in `AmendmentTable.test.cpp` that requires <value>. So it is not optional (and is correct above).


Here's one example from the report.txt linked above:

./../../openstudiocore/src/isomodel/WeatherData.hpp
const Vector &mEgh() const {return _mEgh;}

SWIG treats the return as a non-const pointer (so already there is the issue that you can change it from outside the object).  Then there is the issue where you are hanging on to the vector after the weather data object is destroyed, accessing it will either crash or give you garbage


Let's continue this discussion in https://github.com/concrete5/concrete5/issues/3102


Sorry @pablojan and @hbtalvi for the incorrect and quick answer then :sad:


Fixes #1.


The function sol_buffer_steal_or_copy() is causing too much confusion and I'm not even using it anymore. I will remove it.


Hi Agdula, I´d like to talk with you about diff-kit project... Have you been working on it ? I talked with Joe (diffkit creator) and He has said to me that you are keeping this project for now...

Do you have an email we could talk?


it works.


This makes the link broken in django 1.8.


Yes it is. See page 14 of the spec. The field "vendorname" is in bold, which indicates it is required.


Yeah, I get that... but that is not my code :confused:


FYI @kshefchek, the search limit field is disabled when we select "all" species, it only shows up on single species selection. Because Phenogrid takes a limit number for single species, only shows 10 targets (can be configed in Phenogrid) per species in multi-species comparison view. The reason behind this is when the user wants to see 100 targets per species in the multi-species mode, we can't display 100 columns for each species due to screen restrictions. Unless we have some scrolling stuff.

I'll think about a better way to handle the multi-species view so we can let users specify some numbers but still easy to render the view in browser.


It's a useful shorthand right now, with some legitimate uses, and does what it says on the tin. It just doesn't return monomial sensitivities if you pass it a monomial.


93653546B-FuelGenShowingInABUPSwhenPV (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1958 of 1961 tests passed)

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


Check out the link to the top SWIG problems, http://nickdarnell.com/swig-and-a-miss/.  I'm not trying to start a holy flame war that return by reference is wrong, just that we are more often returning by value and that works easier with SWIG


Remove portion of sentence after 'MUST use' per 2015-11-06 call.


I was going to remove it, but its usage in the docs is just as a quick way to get to ["sensitivities"]["variables"]


This looks like a nice way of localizing the hardcoded text! Does `locale` have to be set in `_config.yml` or does it use `en` by default? If you want to prepare a pull request and send it my way I'll look into merging it.


On 06-11-2015 14:04, Guilherme Íscaro wrote:
> The function sol_buffer_steal_or_copy() is causing too much confusion
> and I'm not even using it anymore. I will remove it.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/cabelitos/soletta/commit/ec6a8566c385f4a92d758800586402b318c37607#commitcomment-14245241>.
>
Just remember that `sol_buffer_steal` needs the fix anyway, to not
return invalid pointer.


This shouldn't be added to flag `x` which is used for free-spacing and line comments. It also just wouldn't work... This code defers to the native handling of ES6 flag `u` for astral code points, but if flag `x` is used instead of flag `u`, the deferring can't work (unless you somehow implicitly added flag `u` to the generated regex). And if native flag `u` isn't in place and you also don't transform the match (you could transform it using surrogate pairs, which wouldn't require flag `u` but also wouldn't work as part of character class ranges), then since you're also bypassing the line below this `if` block that throws a `SyntaxError`, a `\x{10000}` token, for example, would in fact match `x` repeated 10,000 times.

But note that you don't need any change here in order to support `\x{...}` with BMP code points (`\x{0}` through `\x{FFFF}`). That would just work due to updating the regex for this syntax token. With astral code points (above `U+FFFF`) it would throw a `SyntaxError` about needing flag `u`, which is appropriate. Perhaps you could add a wrapper that always adds flag `u` to provided regexes if supported natively, if that's the behavior you want.


same place…


It's an experimental serialization format we're experimenting with. It's compact & fast, and if it works well, we'll use it internally instead of the TRI_json_t JSON representation we currently have.
We expect velocypack / VPack to be much more efficient than the current format, because it will do far less memory allocations and needs to copy less data around. Apart from that, it comes with a JSON parser and to-JSON serializer that seems to be faster than what we currently have.


wow


Nice. Can you add an example to `test/lists/map.pn`?

But I'm not sure if filter is a better name than map, and if where is a better name than the perl-like grep.
map returns the result of the method applied to all elements.
grep return all non-empty/non-false results of the method applied to all elements.


This would be great. Meanwhile can you suggest some alternative way? As I have to present a working demo of your api on android to a client on Sunday. They are intending to use it as a part of their bigger project
Date: Fri, 6 Nov 2015 08:01:15 -0800
From: notifications@github.com
To: swellrt@noreply.github.com
CC: hbtalvi@hotmail.com
Subject: Re: [swellrt] Mark new working version (c44aab3)

Hi @hbtalvi thanks for trying our software.

Current android's build is broken as well as client's implementation. We have to upgrade both to be compatible with latest server improvements.

We can consider to speed up the solution of this issue if you like.

Regards

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


so cool


People with more java knowledge, can comment here. Not sure, if this does a copy or a reference.


`FALSE ==`


на втората задача имаше warnings


test


Hear, hear!


Good catch @jessealter


Oh haha, the affinity WeatherData has for @jasondegraw is so strong that it crosses library bounds :-) That was an example from ISO model.


Awesome!


ModifyTimeBinsToUseRounding (JasonGlazer) - i386-Windows-7-VisualStudio-12: OK (1962 of 1962 tests passed)

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


print"hallo"


@edersondisouza I will leave that one :)


Ran into an issue with boost::python and NumPy interop with integer types. It was refusing to convert a numpy.int64 into a C++ int on some platforms (worked on Ubuntu, failed on all others). On Ubuntu, it instead refused to convert numpy.int32's into C++ ints.

NumPy uses non-standard (for Python) integer types in order to be more precise. Boost doesn't seem to like this and will sometimes throw exceptions from extract<int> calls that should be perfectly valid. Whether or not it works appears to depend on platform OS, bit-width and library versions.

Fixed here by using the Python C API directly, instead of boost::python, when extracting integers from Python. This is just a local workaround. For now, this should be enough since usually floating point numbers are used, rather than integers, and those work fine.

Fixing this for the general case would require creating custom boost converters for all NumPy integer types and registering them at some point during startup. We may have to revisit this at some point in the future if it becomes a problem.


You're lucky that GitHub doesn't support :tableflip:


Yes, please remove it from master.


Ok, maybe expand the docstring with all valid types for `p` and corresponding return types?


I have no idea how this could happen. configure is missing because autoreconf -i is needed, I wonder how this got installed locally successfully :/
Sorry for the hassle, I'll submit again.


Do you need to include the `<html> <head> <body>` tags here?  Think about what's already being loaded onto the page via `layout.erb`.


Argh, thanks. I'll fix and test.


Just changes reference - basically nothing except primitives create copies on assignment


Gotta clean up your `<nav>` tags here. Might be causing that error you mentioned.


pico_ipc_create should have 3 args (name, socket_path, mac)...

you should not use the full path as device name
device name should look like "ipc0"


What does this do?


Oh yes it does!

![Oh yes it does](http://pre00.deviantart.net/ee3a/th/pre/i/2012/233/b/6/twilight_is_table_flipping_mad_by_sketchinetch-d5bx3cf.png)


@chisholm, I pushed a fix in 4e4a21484705a3936b0ea5deed26f856e2ad04c4 and now the tests pass :clap:


So with minIdle = -1 maxPoolSize will be also set to -1?


It's just an experimental branch, one of possible solutions for https://github.com/mate-desktop/mate-applets/issues/126. I'm not merging this into master anytime soon :smile:


Wouldn't that be :ponytableflip:?


At a glance maybe, something around characters for DateString?
```
$ cd po
$ python3 update_po.py -k ru.po
```
does not point out what could be the cause!

Note, we could have problem with python_brace (e.g. 'fi.po') but this should not
break compilation.

Anyway, I will try to [run more tests] (https://github.com/translate/translate/raw/master/translate/filters/checks.py).


You can drop the type="desktop", the <id/> tag doesn't have attributes.


That's an excellent question. Because the javascript allows people to load multiple, different entry views (home, people record, directory record, etc) this entry script is given an ID and removed from the DOM so it is not ever executed again when the `#maincontent` is changed via AJAX.


For now, we are only concerned about fluentd being a normalizer - reading raw log messages from the message bus, normalizing them, and writing them to elasticsearch.  That's what this image should provide.  At some point in the future, if we are using fluentd in a different role, we can figure out what to do then, based on our experience with bitscout.


@superbarne Willst du das jetzt/demnächst gepublisht haben, oder soll ich lieber erstmal auf mehr Änderungen warten? Ansonsten, das $.extend war ja nur dafür da, dass ne Kopie erstellt wird, da brauchen wir keinen richtigen Polyfill


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


Thanks!


Also, Tags für bower kannst du ja auch selbst pushen, aber ich hab das gerne 1:1 zum npm-Paket


Yeah, now I can see there is no database field for this.


s/link/device_link ?


many thanks for the explanation. I should have spent more time studying the code. Sorry for the noise.


Why scripts at the end?

@cviral30 @93prabhat93


This translation template is incomplete - you might find the one I wrote for KDE useful, just to be future-proof:
https://websvn.kde.org/trunk/l10n-kf5/scripts/as-metainfo.its?view=markup


@93prabhat93 Did you fix?


You win


No, that's why it is marked as ".future".  The Dockerfile does not put this file in the base image.  I'm only leaving this here because at some point in the future we may want to add support for reading logs from openshift/kubernetes/docker and having the openshift origin fluentd config is a good place to start.


I would check first if there is an actual comment before displaying this  block.
This holds true for the whole template.


@cviral30 What does this mean? @93prabhat93


why not make it an ontological term?


I've added the screenshot caption in a commit of my own, what's the rationale behind translating a developer name?


Actually it was your math with the signs inverted, right?


Maybe I missed it, but is this rendered somewhere in the template?


Would this work with the single regex `(#|\\?)`?


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


One less thing to configure,
```python
CLOUDFRONT_BASE_URL = 'https://%s.s3.amazonaws.com' % S3['bucket']
```


ein


right, my fault, I unintentionally deleted it


ModifyTimeBinsToUseRounding (JasonGlazer) - Win64-Windows-7-VisualStudio-12: OK (1962 of 1962 tests passed)

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


This tag doesn't exist in that scope - refer to http://www.freedesktop.org/software/appstream/docs/chap-Metadata.html#tag-releases for details.
In this particular case, you want
```
<releases>
  <release version="@VERSION@" timestamp="@UNIXTIMESTAMP@" />
</releases>
```


Chenaged the first font in Verdana


Other than that, r+!


It seems that this breaking change had a significant domino effect. Couldn't you have kept the old init method signature, and introduced a new one alongside it to make this non-breaking?


Glad to see such interest on SwellRT. I can start working on this now, I think it could be working tomorrow afternoon (UTC+2) Does it sounds good to you? Also I could give you support/advice for the demo.


Indeed, that was debug code. I've removed it now.


@Synchro thanks for explaining. I read about the zombie processes problem. Docker will hopefully solve that issue natively at some point and this won't be required. For now phusion baseimage does seem like a good workaround. I have one issue with it though, you're not passing the logs to `stdout` but to a file (`exec haraka -c /usr/local/haraka >>/var/log/haraka.log 2>&1`), which is also not a standard practice with Docker. Especially nowadays, since Docker natively supports various ways to log, including `syslog`.


TeamCity Chorus :: chorus-precise64-master Continuous [Build 2.6.517.84061e9aaaaebc4ff0c5328c7b448a99074e4cb5](http://build.palaso.org/viewLog.html?buildId=65698&buildTypeId=bt323) outcome was **SUCCESS**
Summary: Tests passed: 927, ignored: 54 Build time: 00:20:00


Now with just the - path, that should be there since the beginning :)


Повинень бути html4


commit looks ok


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


AIUI, Ardour has functionality in place so that VERSION contains the git-distance to the last "real" release, so I guess I'll forgo on the unix timestamp (which would be a bit awkward to extract from git metadata).


this was starting to get unwieldly


Successful mega-Girl on the train "Fakingit"stars&Katie Stevens are thing question
![avatar_1446512268424](https://cloud.githubusercontent.com/assets/11037470/11002174/452ca2d2-84bc-11e5-8ae1-cb72f994754e.jpg)


Also a test to confirm on multiple instances would be good


Thank you for taking the time to incorporate the previous comments so quickly.

Is there any statement available for your moving away from the former 2-Clause BSD licensing throughout the code?

Cheers :)


I think we should consider a verbosity level between nothing and solution table that prints brief summary info, like solution time and final solution status.


I think we should be printing this (or something similar). Why did it go away?


updates


Looks good :+1:


for all


I would expect the logic to be `video.type === 'video/flash'` - RTMP cannot be anything else.
If this is too problematic I would even consider going as far as: if `rtmp` or `rtmp.url` is configured the video/flash source is delivered via RTMP.


The validator will maybe complain about that, since some distros want a timestamp so they don't need to do expensive version comparisons when generating the data to fetch the latest release. The spec doesn't strictly mandate it though, so go ahead :)


Should these be affected as well?
https://github.com/ev3dev/ev3-systemd/blob/ev3dev-jessie/systemd/ev3-leds.service
https://github.com/ev3dev/ev3devKit/blob/ev3dev-jessie/src/devices/Led.vala#L27


good idea!


Features theme tumbler Disgin
example. Com / helloworld.html Disqus_ur1


"of"


Indeed: see commit 92f5311aafe45c3b6997b507e878790a2ff2eb21


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 70](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=438&buildTypeId=XbimEssentials_XbimEssentials) is now running


It would be good to list them: / _ et al.?


Happy to add an example for the tests.

filter/where are aliased to the same function.
It returns a new tuple with only the elements where the block returns true (equivalent to perl's grep in your comment).

I don't do much perl, so I picked names from javascript/c# that I thought made sense. If you mostly use perl, grep would make more sense.

Currently we don't have `map`, although it can be trivially built using `each` in Potion itself. I can add `map` to this pull request as a native method if you think it's worthwhile doing.

I'm also working on a Sqlite binding for potion, which is not far from completion. I'll send that as a separate pull request though.


Hmm.. true there's clearly some bug in the logic


:+1:  yeah, cannot update :)


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

```


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 70](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=438&buildTypeId=XbimEssentials_XbimEssentials) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:01:17


hai


Test commit


Yeah, sure. I may have gone a little overboard here. If you look closely, the spelling of "Unrecognized" became "Unrecognised" because why not. :)


Yes. I already make an issue for ev3devKit, but I forgot about the systemd service. Thanks.


is it possible the params being encoded ?


But it changes every time you run the tests! D: But yeah, good point.


I have needed this sentinel I don't remember when. I will see what can be done. Do you have a concrete example that show the memory leak?


curved brackets before if statement


Theme Garden fotos
![cropped_header_1446679534799](https://cloud.githubusercontent.com/assets/11037470/11002367/62ca82cc-84bd-11e5-8295-2e1aa9b7a000.jpg)


See: http://stackoverflow.com/questions/18381787/why-am-i-getting-jquery-mobile-1-3-2-min-map-file-not-found-message-in-chrome for an example of why this pops up.


What is new in this files you want to merge with the branch now? Or is it just the code you forgot to sync before our meeting yesterday? I have some fear, that it maybe causes some errors in functions we corrected yesterday together.


By limiting the input length It prevents the regular expression that does the parsing from consuming too much cpu time blocking the event loop.

> On Nov 6, 2015, at 8:16 AM, Ian Macalinao <notifications@github.com> wrote:
>
> What does this do?
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/rauchg/ms.js/commit/48701f029417faf65e6f5e0b61a3cebe5436b07b#commitcomment-14245548>.
>


93653546B-FuelGenShowingInABUPSwhenPV (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (887 of 887 tests passed)

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


93653546B-FuelGenShowingInABUPSwhenPV (mjwitte) - i386-Windows-7-VisualStudio-12: OK (1958 of 1961 tests passed)

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


u spacing in this file is off


It would be nice to have at least one part of the 661 test rerun pmwebd with -X to exercise this part.


`$cur_subscriber['language']` is not inside the `while ($cur_subscriber = $db->fetch_assoc($result))` loop.


Ah right. doh. I take back my suggestion, my fault.


Hi,

> Is it possible to use save-selected-window instead ?

I feel that the wanted behavior is not to disturb the window
configuration. The save-selected-window may work by chance in the
current situation (not tested), but I think saving the window
configuration will be more robust to changes of behavior of the org
commands since it is semantically more correct (IMHO).

> Why do you save-excursion here ?

Mostly by habit. I don't think it is needed here, then I will update the
pull request to get rid of it.


I think `map` and `filter/where` are the best names. perl `grep` is an abonimation.
A native `map` would be better, yes.


Sounds good.


remove space


Why? It was fine in the constructor. And also why register it with "morecommands", new Whatever(); ? Wouldn't the space with "morecommands" be replaced with the command name.


No, I mean what does line 41 do?


too many spaces indented


That will be great. Meanwhile I will try to complete other parts of the demo.Kindly let me know when its fix and I will surely stay in touch for support/advice as I develop it

Thank you for the prompt response and support
Date: Fri, 6 Nov 2015 08:24:03 -0800
From: notifications@github.com
To: swellrt@noreply.github.com
CC: hbtalvi@hotmail.com
Subject: Re: [swellrt] Mark new working version (c44aab3)

Glad to see such interest on SwellRT. I can start working on this now, I think it could be working tomorrow afternoon (UTC+2) Does it sounds good to you? Also I could give you support/advice for the demo.

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


Sorry, bad title here, first time for me : "[*] CORE : Front product search position considering stocks"

My apologies  !


everything inside this describe block is indented too many times


But I still argue we should have a verbosity level between nothing and solution table for printing summary info.


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


`s/openshift/OpenShift/` to be consistent


you can do:

`mdata->path ?: ""`


looking good


Hi Nicholas,

I’ve tested your suggestion and it works as expected but it might hide some data type definition mismatches that otherwise would be caught during testing.  The change I tested is:

row[property.Key] = Convert.ChangeType((property.Value as ScalarValue).Value, _eventsTable.Columns[property.Key].DataType);

Re the PR: I’ve never had to add to, enhance or fix anything in GitHub so I wouldn’t know where to start – best it be left to the experts.

Have a good day,

Glenn

From: Nicholas Blumhardt [mailto:notifications@github.com]
Sent: Thursday, November 05, 2015 4:10 PM
To: serilog/serilog-sinks-mssqlserver
Cc: Glenn Les
Subject: Re: [serilog-sinks-mssqlserver] Support logging properties into user defined columns (5323191)

Hi @glennl122<https://github.com/glennl122> - this seems like a good change to make, though it is possible the value type could be mismatched with the column definition.

Perhaps along with your proposed change, using Convert.ChangeType() on the value and perhaps some further validation, could cover it?

Is this something you're interested in sending a PR for?

Cheers!

—
Reply to this email directly or view it on GitHub<https://github.com/serilog/serilog-sinks-mssqlserver/commit/53231918b6fdc34c37cd7752e56a76b300a548d6#commitcomment-14224682>.


Well, we need to define what we accept here; Only encoded params or params  not encoded. I assume that they are provided encoded. But we can change that;


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


Nice work syncing a leaked patch. Did you do anything on your own so far ?


The developer name could be something like "The KDE Community" or something else which can be theoretically translated, hence we allow it to be translated.
If that isn't the case here, ignore it.


indented to much :)


I'd want to maybe delete Discourse as there isn't much information available about direct careers with the parent company but the work is mainly done as Open Source


Didn't think of that. I guess I'll just use your ITS file then :).


That's incompatible with a verbosity which prints only the table. Which is higher priority?


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


Great @hbtalvi ! we are happy to help you. I will give you feedback frequntly


`$language` should be `$lang_common`.


Is there material being stored to disk as a result of this change?  Is there a bound on how much material can be stored to disk?


Qualcuno qui è moldavo


Force type to string


I just found an easy way to get at the UNIX timestamp from the last git log message (must learn to read man pages):

    git log -1 --pretty=format:%ct

The problem is that many builds (e.g. distro packages) will take place outside of a git repository. :(


Added a display score function


Why hardcoded?

`'wrote'								=>	'wrote:', // For [quote]'s`
Also the string has a colon but you harcoded without a colon. Using the language string results in 2 colons.


This commit doesn't seem to be part of the Nuget [1.42 release] (https://www.nuget.org/packages/Dapper/1.42.0). Calling Get<T> before parameters have been added still throws a NullPointerException.


For tracing exiting with a known error code is useful.
Also, there is nothing preventing split-brain in this condition for a period of (x).


Yes, yes, it almost surely is.


ok


A security hole, that is.


Ooh, nice library.


Thank you :)

Date: Fri, 6 Nov 2015 08:40:26 -0800
From: notifications@github.com
To: swellrt@noreply.github.com
CC: hbtalvi@hotmail.com
Subject: Re: [swellrt] Mark new working version (c44aab3)

Great @hbtalvi ! we are happy to help you. I will give you feedback frequntly

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


Added background.


Right, I didn't yet look into the details of noflo-ui.
I don't think there is a standard way of declaring ES6 usage. Parsing the AST and looking for new features sounds like the safest way. Some people use ```.es6``` or ```.es6.js``` but that doesn't look dominant.
The reason why I'm interested in this is that I tend to use babel at run-time for dev and testing, and this was a missing piece in noflo (e.g. when using mocha I would run ```mocha --compilers js:babel/register```. A global option like that would be sufficient for me as I'd precompile in production anyway.


TeamCity Chorus :: chorus-win32-master Continuous [Build 2.6.98.9aa57669d782491ec4b8069357ee07dae0ea9d81](http://build.palaso.org/viewLog.html?buildId=65700&buildTypeId=bt2) outcome was **SUCCESS**
Summary: Tests passed: 939, ignored: 50 Build time: 00:34:42


argh, it might not be an ordered set...


Is this Eugene-specific?


I was throwing it out there because it's a pattern I pulled from other frameworks/libraries and started to use more.

Any time I start to deviate from a raw boolean return value in an "is_valid" type of function and need to pass back more information when invalid I wonder if I should change it to an assertion method (`assert_valid()` or `validate()`) which raises an exception when found to be in an invalid state.

I guess I feel like an `is_valid()` method should have a boolean return type since I'd expect something like this to work:

```python
if not foo.is_valid():
  print "Something bad happened"
```

But that condition will never be met if we return a tuple (the tuple will always evaluate to `True` as long as its populated). Instead you'd need to do something like:

```python
isvalid, msg = foo.is_valid()

if not isvalid:
    print msg
```

...which isn't necessarily bad, but isn't really what i'd expect at first glance at the method.

Just throwing it out there as a possible alternative!


This one is found when I was working on the bug fix of "LOG ERROR INTO" one.

How this gives error:
When A partition table got exchanged with an external table (may contain the "log error [into]" in its definition).

Backup will first dump an temp external table (due to dependency)
Backup will then write the definition of the partition table.
Backup then write an alter to exchange the partition with that temporary external table.
Backup then drops the new temporary table.

Since the external table become part of the partition table, dropping external table will give error here, therefore, I removed this line.


Code-wise this is correct, since comment is not mandatory, but even if comment is empty this will render an empty paragraph element.


Here a `unused li64StartPos` warning will be raised on release build. Please add a `#ifndef NDEBUG` around it as done on ca8a9f18059fa44804190091a3e72acf68b39c7b.


Why empty lines between imports?


If so, we can create the same functionality by just printing `sol.table()`


Maybe have `suppress_timing` for the tests? Seems like a special case. I think outside the tests, summary info is higher priority than the table.


Right, this will be taken out.


ah, makes sense. I was going to suggest using .toString() for readability
but I realized that null/undefined would throw errors if you did that

On Fri, Nov 6, 2015 at 10:41 AM Adam Baldwin <notifications@github.com>
wrote:

> Force type to string
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/rauchg/ms.js/commit/48701f029417faf65e6f5e0b61a3cebe5436b07b#commitcomment-14246205>
> .
>


As far as I know spark caching to disk is unbounded. This really should be a config. But in our datasets I can't imagine it actually overflowing since this section is responsible for caching the native layers and not the operations.


both params should be user defined.


TeamCity Chorus :: chorus-precise64-master Continuous [Build 2.6.518.9aa57669d782491ec4b8069357ee07dae0ea9d81](http://build.palaso.org/viewLog.html?buildId=65701&buildTypeId=bt323) outcome was **SUCCESS**
Summary: Tests passed: 927, ignored: 54 Build time: 00:17:54


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


I disagree a bit. When you do want timing info and not the table?


Dir.pwd return current path
Dir.home return the home path

By default, aka config file located at $HOME/.aka


Hey Paulo. As you can see I've cloned the original project from http://diffkit.googlecode.com/svn/trunk/. I did not consult it with Joe. The reason I did this is that I wanted to include it as a dependency in other project and it was not mavenized so I've cloned it, moved to maven, made a version and deployed to my teams local repo.
I would not consider myself as a keeper of the project rather someone that borrowed the code :). I'm guessing you need the binaries. If so I could move the project repo bintray.


Also, preference for `print sol.table()` over `suppress_timing`.


Replaced in a later commit with:
```scala
    val config = ConfigFactory.load()
    if (config.hasPath("librato")) {
      LibratoReporter.enable(
        LibratoReporter.builder(metricRegistry,
          config.getString("librato.username"),
          config.getString("librato.token"),
          config.getString("librato.source")),
        5,
        TimeUnit.SECONDS);
    }
}
```


This file/folder isn't in master...


specific test which shows that when dx = nan, matrices are still generated!
Now use (plot, verbose).
Test fail


Won't this realistically enforce a minimum of GCC 5 and Clang 3.4?


You can skip this tag, as it is not mandatory - but if you use it (and add descriptions), you can file in the data manually each release (and you could even translate it that way...).


I think the same change needs to be applied to the kaiwa webclient, not just kaiwa-server.


yup. It should be though. Either I can push it there too or we can wait until you finish working on this and then merge it in.


Can there be 2 types of weather "age"? if the time is set using the GPS time, then use this new calculated age. else, use the short datetime.


I did not know that. thanks.


We suggested the word 'Options' is better than 'Init'. It is more explicit and clear about what this does. Is 'Init' common in other Web APIs?


Oops..missed that one. Thanks for catching it!

Fixed in e2ed68cc70925372681c2fb50b505b110825c8f7


These are subdomain names for preview front-ends. Actually, `government-frontend` does not work in development environment, because dev VM does not run the draft stack by default. It only allows you to "preview" a published version of a guide locally.

I will extract it to a local variable to make it clearer what they are.


In that case you can do something like this:
if self.executed_by:
   executed_by = self.executed_by
else:
   executed_by = ExecutedBy()

executedByFields = ['modelProgramName','modelProgramIdentifier']
self.add_metadata_element_to_xml(container, executed_by, executedByFields)


Most of code in PlanChangesFragment is the same that is in AnnouncementFragment, create abstract superclass, say FeedFragment and have only differing things in subclass (e.g. getFeedUrl())


Yeah, but that seems to involve manual work by the upstream maintainers who provide these metadata files just as a convenience for downstreams. I don't want to put additional burdens on them ;).


Just a comment


add_contact is not worked!


I would say we can have one type of display and it could be "age" if we get something as a good reference. My Stratux Test has a GPS but since the lock has not occurred before the backend code generates its base reference time, then we have a problem.

I know there is a separate thread discussing "time". Once that issue is decided and coded, then we can revisit "age of weather reports".


i think what i did is just klick the "Sync" botton and update my files from the Github server. i think the files in divided-code are still the version which we did together,right?
or i should delete my files  and clone from the server ever time when i want to update my file?


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


webapck.config.js这个文件放在哪个文件夹下啊？


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

```


this is only used on the "unique-stubs-ppc64.h" that is not included on this pull request, only on our next branch. I suggest removing it, or?


Right, should be clarified clearly.


```
cdef z = n
```
is correct but pointless. It's no different than simply `z = n`.


For the cheap features we need here, GCC 4.7 seems to be enough: https://gcc.gnu.org/wiki/C11Status (actually, the feature is there since GCC 4.6, but the C11 switch didn't exist before)
For C11 on Clang I see if the CI fails now (which is running Ubuntu Trusty).


Aha, looks like the CI is happy, so we're good :-)


You can't change traduction in file. You must use transifex.
And your translation does not match the original text


Looks good, but probably should link to http://alaveteli.org/docs/running/requests/#old-requests-6-months-without-activity (which will also need an update as it currently says "6 months".


Don't use ProgressDialog, using ProgressDialog prevents user from opening drawer when he wants to go somewhere else before content is loaded as well as the HTTPConnect may be used in background service, where showing dialog would be pointless and/or annoying.


Great !


Done, sorry for the trouble


add `allChecked` option which will be used in the item template to add a css class


You can change object of notification but you can't delete footer page.
The software is GLPI and we want that the name still be present


Awesome, cheers :D Been keeping a sneaky eye on limba, I like where this is going :)


@alext Updated PR for the above: https://github.com/alphagov/govuk_frontend_toolkit/pull/225


The drop it, or keep it without timestamp - I could use it to create a Limba[1] bundle that way, which demands at least one release tag in the MetaInfo file ;-) (But it's not terribly important, Limba is beta software, so do as you please ^^)

[1]: https://github.com/ximion/limba


You want to delete all information about the name of the software and i'm not agree with that


The Maps 020 returns from the dead :-)


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


will we keep it commented for pr?


:+1:. Thanks


Please send a patch that fixes it.


That's correct, using '%s' for "public" is the right way.
and c_PUBLIC seems to be only used here, can get rid of it.


Okay, that sounds fine from my perspective, but I wonder if @hectcastro would like to comment?


I've changed it to contain one release tag, with only the version attribute.


@racardoso : shouldn't this be removed and the above https://github.com/PPC64/hhvm/commit/90bf32a41b7ae3d0b357d166e0ccf749982c0741#diff-40efcd51abe75970f0a2dcd978fc312bR460 be used?


Принято.


GNOME-Software supports it natively :-)
But I fear that this support will be kicked out in Ubuntu, because otherwise they'd have to put Limba in main, and they also have their own product (Snappy) in that area, which works similar to Limba. So I might need to negotiate a bit ^^
As soon as Limba can build new bundles in a clean chroot using SDK bundles, all the features I consider essential are complete - and the next step will be improving LimbaHub[1].

[1]: https://github.com/ximion/limba-hub


The problem I see with just doing the Convert.ChangeType is in the case that the type cannot be converted, we will fail to write anything to the database thereafter.

```csharp
var columnName = property.Key;
                    var columnType = row.Table.Columns[columnName].DataType;
                    object conversion;
                    var scalarValue = property.Value as ScalarValue;
                    if ( TryChangeType(scalarValue.Value, columnType, out conversion ) )
                    {
                        row[columnName] = conversion;
                    }
```

with the following method

```csharp
   private static bool TryChangeType( object obj, Type type, out object conversion )
        {
            conversion = null;
            try
            {
                conversion = Convert.ChangeType( obj, type );
                return true;
            }
            catch
            {
                return false;
            }
        }
```

This does indeed hide failures of conversion, but allows for other columns to have the data entered successfully.

The now is having a SQL column of XML type since .NET abstracts this layer and the ColumnData type is a string. If some one were to try and add malformed XML, the logger will fail to write anything else to the database thereafter.


I agree in removing the comment and leaving only the `not_implemented();`


There might be a more elegant fix with less checks, but I leave it to LWS authors to decide.


Why is it misleading?


Shame :( Luckily I don't suffer this issue, I'm thinking for Solus[1], which is independent.

Solus has a very simplistic overlayfs based chroot builder you *may* be interested in for this side of things? https://git.solus-project.com/packages/pisi/tree/files/evobuild.py

[1] https://solus-project.com/


It shows me, that you want to merge another branch with the divided-code branch. I have the right to decide if this should be done or not because i am the owner of the project.
If you want to update files in the divided-code branch you can do this without me confirming the changes.

The normal way of working is like this(using a Desktop version of Git-Hub):
1. Sync you Desktop Git-Hub Version with the servers by pressing the sync-Button
2. Work on the code
3. Commit the changes to the Desktop Git-Hub Version
4. Press the sync-Button again to upload your changes to the server


@davidtkelleher hey man, so it would still be necessary to internationalize these EMV commands, that is correct?


if we're ok with this "import contrib from core" than this might be a better solution than merging staticfiles and static.


Do you mean OSVR_REPORT_X?  Or is this X going to be a generic function call method beyond reporting?


Would this wording help clarify?
There is an additional weekly 7.x-2.x Tech Call that occurs each Wednesday at 1:00pm Eastern Daylight Time US (UTC-4).


Does ```ipc``` mean that the library is used in a app that runs by electron framework?


I don't believe so, because the replacement string is conditional.


FYI @opengapps/packagers the 'universal' maps that is nodpi and has all architectures, is not detected as such by the script (it is impossible to distinguish it from arm64- and x86_64-only apps) and as such is manually copied by me into arm and x86, so that we can provide 'nodpi' installs on those platforms too.


@vberruchon Hey thanks for pointing that out. I didn't notice that prereq in the PR. I can remove that part.


Seems important.  Is this coming soon?  Note that the velocity should be reported in the space that the tracker source is in, not the local sensor space.  If that is what this rotation is for, maybe it isn't done.  If this is rotating by the transforms in the config file, then it still needs doing.


I guess this should be

CfgWriteInt(L"DEBUG", L"Delay_Cycles", delay_cycles);

or? otherwise it will always overwrite the ini to 4?


В бинарном же виде


ja ich hab das einfach ohne groß zu gucken warum du das so gemacht hast ersetzt. Wollte nur schnell jquery los werden, wollte es nicht als Abhängigkeit haben. Ist das wichtig das man jetzt hirfür ein neuen tag erstellt?


Тут проблема, ChronoUnit по часам считает, поэтому разница в 1 день может быть уже чрез календарные сутки. Перед сравнением дату нужно привести к началу календарного дня: yourDate.toLocalDate().atStartOfDay();


Why not use `$userID = Gdn::session()->UserID;`? It is a public property that is set in the session class __construct, so I think it should be safe to use it like that.


Same comment here.  If this is required for enabling config-file axis remapping and such, I think we'll need it to make use of the data.


Brawo Pani Izabelo 5!


ja what i did was just klick the sync button, which should pull changes from the server.   i dont know why it comes to like this.


How very odd.


Лучше вынести соответствие команд и котов в отдельный хидер. Чтоб, если надо что-то поправить, не делать этого в трех местах.


I agree about the logging, espcially since syslog is built into baseimage. I suggest you open a ticket for that, or make a PR yourself. I'm only credited on this commit because it was a PR.


TeamCity Gensokyo Lobby :: Lobster Core [Build 0.0.1.62](https://pixelated.design/build/viewLog.html?buildId=194&buildTypeId=GensokyoLobby_LobsterCore&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 274 Build time: 00:00:11


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


Added favorite color


@rafkhan, Are you sure you want use ES6 features here?

![image](https://cloud.githubusercontent.com/assets/4575064/11003302/a1db1bf4-847f-11e5-8a8a-593b7c96aed3.png)


I think this should look like this:
```
#if defined(__GLIBC__)
		snprintf(buf, size, "%s", strerror_r(err, sbuf, sizeof(sbuf)));
#else
		if (0 == strerror_r(err, sbuf, sizeof(sbuf)))
			snprintf(buf, size, "%s", sbuf);
		else
			snprintf(buf, size, "%s", "str_error_r() failed");
#endif
```
Note that the semantics of `strerror_r(3)` for GNU libc differ from that of musl libc, if `_GNU_SOURCE` is defined when `<string.h>` is included.


If you wouldn't mind reverting the rename here I'd appreciate it.  The test does work fine for me and I've verified that it depends on current timezone.  I've fixed that in https://github.com/ahupp/python-magic/commit/3069a4a01a9551d654460214b40123bec2bc6592.


Spam!


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


I wouldn't call it **has**Permission since it is a method of the **discussion**Model. The user has a permission, but not the discussion.


Есть же "case 1-7:"


@dunnbydesign noticed that the vast majority of your statements do not end in a semicolon...  any reason why?


FYI @erichirlemann it's better to rebase your PR before merging it, to avoid these technical "Merge" commits and have a linear history in Git.


Close ticket #13


I'd say remove the `has_recipient()` completely,  isn't that the whole idea behind Python's magic methods? So that we can use beautiful syntax like `x in y` instead of normal methods?


I think we want autoStartup="false" here, correct?


As discussed offline with @jamiecobbett, this is a bit kludgy, but it's fits the pattern of [separating "presentation logic" from the "view logic"](https://trello.com/c/uwFJN1wy/21-refactor-analytics-metadata-component-as-example-of-separation), and avoids the conversion having to be done when calling the component (in each app), keeping it here makes it changeable in the future without having to update all calling apps.


as there is no `PPC64_HAS_PUSH_POP` definition anymore, this include can be removed


@dunnbydesign good use of ES6 arrow functions!


Oops!


Done: d7f9a03e21fff84f57e3ff840d1150b1cb6ec40b


Samuel Loury <notifications@github.com> writes:

> I feel that the wanted behavior is not to disturb the window
> configuration.

It doesn't more than save-window-excursion, please use
save-selected-window if possible.

> The save-selected-window may work by chance

It is not by chance, it work generally well in all situation and is
safer than save-window-excursion.

> in the current situation (not tested), but I think saving the window
> configuration will be more robust to changes of behavior of the org
> commands since it is semantically more correct (IMHO).
>
>> Why do you save-excursion here ?
>
> Mostly by habit. I don't think it is needed here, then I will update the
> pull request to get rid of it.

Ok thanks.

--
Thierry
https://emacs-helm.github.io/helm/


this comment can be removed. `li64` is a stable 5 instruction now.


typo: bc**tr**


Can you confirm that we do not need `in ThisBuild` here?


Gradients are defined as CSS <image> data types; they can be used only where an image data type is required. For this reason, linear-gradient won't work on background-color and other properties requesting <color>.

With that said, why even have this test? (legit question)


typo: bcc**tr**


Hi Agula, it is nice to contact you!

Do you are in the Diff-kit user-group list ?

If so, I have left a message there citing that I was looking for a way to
contact you...  and I have talked with Joe about a possibility to restart
the Diff-kit proejct ...

What do you think about it ? Would you like to do that?

Cheers
Paulo.

On Fri, Nov 6, 2015 at 2:47 PM, agdula <notifications@github.com> wrote:

> Hey Paulo. As you can see I've cloned the original project from
> http://diffkit.googlecode.com/svn/trunk/. I did not consult it with Joe.
> The reason I did this is that I wanted to include it as a dependency in
> other project and it was not mavenized so I've cloned it, moved to maven,
> made a version and deployed to my teams local repo.
> I would not consider myself as a keeper of the project rather someone that
> borrowed the code :). I'm guessing you need the binaries. If so I could
> move the project repo bintray.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/agdula/diffkit/commit/ace0ef667d3697b1f59b243a6be58d5ce9d4c875#commitcomment-14246379>
> .
>


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


if you press the sync button and you have comitted changes on your destop version it comes up like this i think, but i am not sure what this new branch is about, did you created it?


please keep this comment aligned in the same column as the one above


Would prefer this to be raised by the underlying redis command not a method on the lock.


Interesting! Very similar to what Limba does in `limba-build` (minus the upcoming mounting of SDK bundles into the build environment).
I like the Solus approach - and Limba is a pretty nice way of getting a secure PPA-like service, once LimbaHub is completed (I plan to probably use it in Tanglu as well) - problem is that I'm not an expert in writing webapps, I like the low-level C stuff more :D


I guess that it's better to use that condition: https://github.com/atom/electron/issues/3358#issuecomment-154476806


Oops, doesn't fix #77, fixes #76.


Hello @LouTerrailloune,

Can you make a pull request for your changes please?


@bmsatierf hmm, yes I believe so, as the text that I translated for emv.getinfo and emv.adddata does not exist in the en.yml file - just directly on emv.getinfo.erb and emv.adddata.erb files themselves.


@dunnbydesign Lots of "const" declarations.  Some make sense as they will likely never change yet this array and the paths object above may not necessarily need that and a var declaration may be a better option.  While "var" and "const" may seem like they are the same thing, there are definitely a few key differences, which you can read up on and we can discuss, if you have any questions.


I haven't loooked closely at all of the related code, but is it not possible to simply use the GtkApplication signal for this purpose and simply remove this signal altogether?


TeamCity Gensokyo Lobby :: Lobster Server [Build 0.0.1.58](https://pixelated.design/build/viewLog.html?buildId=195&buildTypeId=GensokyoLobby_LobsterServer&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:19


On Fri, Nov 6, 2015 at 1:17 AM, Heikki Linnakangas <notifications@github.com
> wrote:

> Hmm, so the caller that passes a NULL stmt argument is in cdbsreh.c. Why
> are we not seeing crashes because of this? Is the calling function in
> cdbsreh.c not exercised by any regression test?
>
Was trying to find the same yesterday but with so much redness around,
super difficult to spot one. The call relates to ErrorTable something which
is planned to be deprecated for 5.0.
We provided alternative functionality in 4.3 but didn't remove the feature
as some potential customers were using it.
Not fully sure when new functionality was added, tests for old one was
removed or never existed, curious as well will dig-in.

—
> Reply to this email directly or view it on GitHub
> <https://github.com/ashwinstar/gpdb/commit/5792763ded7b6d3c19993f36bbdb885c938577ae#commitcomment-14236166>
> .
>


I don't think it's necessary, but it would be more consistent with the remaining parts of the file. Let me add that.


no i didn't.  maybe we should just ignore the branch.  i hope the divided code branch still works.


yes it should be removed. In fact i removed this i my last commit.


I would not say no to a PPA-like system :) Have the same problem as yourself, love me C, stick a webapp in front of me and the world is suddenly a much larger, more confusing place.

I'm kinda curious if I can support to my native package build format to just emit limba packages off the bat, given the similarities:

https://git.solus-project.com/packages/gedit/tree/package.yml


????

Sent from my iPhone

> On Nov 6, 2015, at 5:12 AM, Gleb Mazovetskiy <notifications@github.com> wrote:
>
> @nick250386 You may have to rake tmp:clear and restart the server.
>
> —
> Reply to this email directly or view it on GitHub.
>


On 11/06/2015 07:21 PM, Ashwin Agrawal wrote:
> On Fri, Nov 6, 2015 at 1:17 AM, Heikki Linnakangas <notifications@github.com
>> wrote:
>
>> Hmm, so the caller that passes a NULL stmt argument is in cdbsreh.c. Why
>> are we not seeing crashes because of this? Is the calling function in
>> cdbsreh.c not exercised by any regression test?
>>
> Was trying to find the same yesterday but with so much redness around,
> super difficult to spot one. The call relates to ErrorTable something which
> is planned to be deprecated for 5.0.

Ah, interesting. By "deprecated", do you mean "removed" ? Or just
documented as deprecated, for removal later?

- Heikki


@cobra could you test this new version please? I think this is ready for a PR.


@dunnbydesign this definitely works and meets the requirements defined in the assignment yet moving forward you may want to try using regex (regular expressions) to match your conditional criteria.  It takes a bit more time (and the learning curve can be rather steep) yet once you get the proper pattern down, you can essentially configure your regex to meet a variety of matching criteria, thus allowing your url check to more dynamic and not be reliant on one specific string, such as "/greet/"


Done


These are not supported on MSVC.


@xu-cheng Did you check this? There were known issues on Mavericks at build with the last update which is why the dep was inserted. Tim signed off on it:

https://github.com/Homebrew/homebrew/pull/44673
https://github.com/Homebrew/homebrew/pull/44235


Ideally ```oldTarget``` should be checked for being ```null```.


I think no. In metadata packs, we're interested in both the identity and content of the constituent parts. I.e. when the agent asks us "do you have my resource types?" we need to be able to tell it that "yes, we have the types with the IDs you expect with the data as you last saw them." This is different from the structural match that we talked about in the past which is concerned with the equality of 2 different resource types. What we could do though is to rename the `ContentHash` class to something better describing the above distinction - `IdentityAndStructureHash` maybe?

Also the javadoc here is slightly inaccurate because the entities are not used to describe the structure - only their blueprints which do not contain the whole path anyway, which is alluded to in the comment.


Hi @fanbookz-owner, Can you make a pull request with your changes please? thx


for all the loads above, I guess a refactor with macros would be a good idea, right?
```
#define LOAD_SIZE(size) do {                                    \
    if (i.s.index.isValid()) a->l##size##zx(i.d, i.s);          \
    else                     a->l##size##z (i.d, i.s);          \
  } while(0)

// and then:
void emit(const loadzlq& i) { LOAD_SIZE(w); }

#undef LOAD_SIZE
```


fix the formatting


remove space


define it on one line please


Понял. Посидел, подумал над таким вариантом. В текущей реализации прикручивать этот код в потроха старой реализации метаинформации будет не хорошо - Илья собрался как раз там делать рефакторинг, чтобы поменять способ сериализации и инициализации.

Плюс он мне не очень нравится - получается, мы в одних случаях будем доставать “сырую” информацию, а в других - обработанную. Тут как раз лучше завести отдельный метод в JNI типа GetWikiLink(). Тем более, что мы можем когда-нибудь решить все-таки отображать тайтлы страниц, особенно когда сделаем поддержку разных языков в генераторе.


same idea here with a `STORE_SIZE` macro as described above for `LOAD_SIZE` macro


Oh, nice idea! Didn't know about that one.


On Fri, Nov 6, 2015 at 9:23 AM, Heikki Linnakangas <notifications@github.com
> wrote:

> Ah, interesting. By "deprecated", do you mean "removed" ? Or just
> documented as deprecated, for removal later?

4.3 documented as deprecated. 5.0 should be removed.


It should not be encoded, otherwise will be decoded something that shouldn't. The libraries consider the data not encoded and then encode themselves.


5255-EMS-dual-Temp-Node-Setpoints (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1962 of 1962 tests passed)

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


typo in the comment: patth.  I'll fix it and merge


That works. Thanks!


removed 'highlight' filter because i did not find a better way to do with the 'span' inside the 'div'.


@e0d Oops.. could you undelete the X?


Delete 'three columns' div


Simple form in order to vizualize how i'll make a good form


They should be independent, but good catch.


5255-EMS-dual-Temp-Node-Setpoints (mjwitte) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


> Did you check this?

No, I haven't. I only check the commit message which said `for all older OS X versions that don't ship with Python >=2.7`.

The major change I need is to make it depends on `:python` not `python`. (Which triggers a bug in test-bot to cause Postgresql test failure on Mavericks only). I will revert it to use `MacOS.version <= :mavericks`


I've created a simple blockquote style for studyquizz


Yeah, that's what I thought as well. I will remove them.


done


这么一直专注技术~ 善 :+1:


Change CSS of pusher widget text area


Done


Yeah, that's wrong. Weird that it did work when I had tested it  O_O

I'll change that parameter to the var tomorrow, way past my bed time right now


ola q que taaas
f
asdfdasf


these lines (352-356) may be removed. They are auxiliaries for X64 code and it was copied here.


This is fantastic. Shame it won't work with two-factor auth though.


I've moved some parts of the main page in differents partials.
And I've resolve some missing or bad link path


Just import what you need


log


sqlite_open() function for SQLite2 only!?


I don't think it worth for only two lines, but we can think about it later, in a new PR related with code reduction. ok?


Change routes and css styles


Is this like :ballmerchairthrow


Aye. That was the case for the last commit, but the one before that introduces the dependency on Mavericks due to CI fatal errors.

I can't link the discussion thread directly because Github doesn't automatically expand outdated diff comments, but the discussion thread is: https://github.com/Homebrew/homebrew/pull/44235

At `DomT4 commented on an outdated diff on 27 Sep` just above the `Just fixed this error, so the next run through should be free of this at least:` comment.


Just import what you need.  If you import everything, you're bringing in a lot of things that could potentially cause namespace issues.


aptoidecrawler?


Should try to avoid global variables.  Also, these are hard coded example values that shouldn't be in 'production' code.


this should be pull as well (sync is for two way replication)


Unnecessary comment if the function name is as clear as it is


New spawns marked with red: http://i.imgur.com/H5VhyMg.jpg


@borshop: retry


nevermind...just noticed you were commenting on my commit!


@andkov - since it is possible there will actually be a "cognitive only" project, these should all be named physical-cognitive (or P-C if you need a shorter name).


should these fields be made private (and be accessed by the getters in ReadCountCollectionUtils)?


Classic python variable/method/anything naming conventions usually uses underlines as opposed to camelcase.


Yes, it should be, but involves more changes in the code, and this was actually my initial idea. I wanted to get this working first, before doing more intrusive patching.


thx


And of course, that message should read "If running on windows ..."


Please add an API called "setDropTimeout(int milliseconds)" that lets you configure these parameters


@DrewSorrels It's looking good. All of the photos I have tested out displayed well so far.


93653546B-FuelGenShowingInABUPSwhenPV (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1425 of 1425 tests passed)

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


Check with 'pofilter' does not light any major issue on "ru.po" file.

'poconflicts' is returning something:
```
# (poconflicts) ru.po
#: ../gramps/plugins/importer/importcsv.py:113
#: ../gramps/plugins/importer/importgedcom.py:124
#: ../gramps/plugins/importer/importgedcom.py:138
#: ../gramps/plugins/importer/importgeneweb.py:153
#: ../gramps/plugins/importer/importgeneweb.py:159
#: ../gramps/plugins/importer/importvcard.py:69
#: ../gramps/plugins/importer/importvcard.py:72
#, python-format
msgid "%s could not be opened\n"
msgstr "%s не может быть открыт\n"

# (poconflicts) ru.po
#: ../gramps/plugins/importer/importgrdb.py:61
#: ../gramps/plugins/importer/importprogen.py:74
#: ../gramps/plugins/importer/importprogen.py:83
#: ../gramps/plugins/importer/importxml.py:467
#: ../gramps/plugins/importer/importxml.py:470
#, python-format
msgid "%s could not be opened"
msgstr "%s не мог быть открыт"
```
but it sounds like false positive!


Another example of an unnecessary comment.  Unless what you're doing is hard to parse, leave these extremely simplistic types of comments out.  This should be an overarching description of what happens / why it's happening / any potential confusions or issues, etc.  Not "and then I set a to equal 5".


The important bit is here. My text editor got rid of all your tab characters.


An example of a good comment.  States something that couldn't easily be inferred from the code.


Oops, forgot to undef `OSVR_X`, good catch.  The idea behind x-macros is here: https://en.wikipedia.org/wiki/X_Macro


@ampiccinin, I started as `physical-cognitive` but then ruled in favor of a simpler name. This is a perfect timing to rename, and I agree with the logic of a cognitive track that explores cognitive-cognitive pairs.


Так, в целом норм, но есть пара важных замечаний и пара дикостей (частично пропущенных мной же в прошлом задании):
1) Во-первых, мелочь, но позволяющая сделать код красивше: вот такое выражение: `films.sort_by{ |f| f.date }` (когда у блока один аргумент, и у этого аргумента вызывается только один метод в блоке) можно сократить до такого: `films.sort_by(&:date)` — конструкция `&:date` состоит из символа `:date` и специального значка, который как раз и создаёт блок с одним аргументом, у которого вызовет один метод — date
2) `sort_by{ |row| row[0] }` — ну, некрасиво же! ты ж выше разобрался, что у этого блока два аргумента и можно писать вот так: `sort_by{|mon, group| mon}`. Но, можно и ещё лучше! Можно здесь использовать конструкцию из (1) чтобы было совсем коротенько :)
3) С месяцами история такая:
  * во-первых, есть стандартная программистская идиома под названием «нулябрь»: если бы твой массив месяцев выглядел вот так: `MONTH_ARR = ["Nullary", "January", "February"...` — то можно было бы писать `MONTH_ARR[mon]` (вместо `mon - 1`)
  * во-вторых, у руби вообще вполне приличная и хорошо продуманная стандартная библиотека; поэтому стоит подумать о том, что такая задача («имя месяца по номеру») встречается очень часто, и что наверное где-то уже есть стандартный массив имён месяцев; причём где-то в логичном месте ;)

4) А вот это — совсем нехорошо, пропустил в третьем задании:
```ruby
films.map{ |ln| ln.author.split(" ") }.
  uniq.sort_by{ |d| d.last }.
  each{ |d|
    # это САМЫЙ сложный способ сказать «напечатать все кусочки через пробел,
    # который мне встречался за всю жизнь. Можно НАМНОГО проще!
    # более того, можно сделать так, чтобы split был только внутри сортирующего блока, тогда
    # сортированные режиссёры вообще будут строками, а не массивами :)
    d.each{ |i| print "#{i} " }
    puts "\n"
  }
```


New spawns marked in red: http://i.imgur.com/59NmM00.jpg


@gogozs 哈哈有机会可以一块儿喝咖啡……


Solves #1.


We should try to match these labels - maybe read them in from the csv?


Hi,
I have updated PHYS14 electron cut-based ID and Isolation selections to Spring15 selections.
Please have a look.


@jwatzman, We need this symlinks because we have them in mcrouter: https://github.com/facebook/mcrouter/blob/master/mcrouter/configure.ac#L17-L20

We should check if this links already exist and not duplicate them.
Or alternatively we could copy these files into hhvm/hhvm-third-party/mcrouter/lib/network/ and hhvm/hhvm-third-party/mcrouter/, but these would require people to make these copies each time they bump mcrouter version.


I think we need to pass `[repo]` here, because `listRepos()` is expecting a list.


On 11/06/2015 07:27 PM, Ashwin Agrawal wrote:
> On Fri, Nov 6, 2015 at 9:23 AM, Heikki Linnakangas <notifications@github.com
>> wrote:
>
>> Ah, interesting. By "deprecated", do you mean "removed" ? Or just
>> documented as deprecated, for removal later?
>
> 4.3 documented as deprecated. 5.0 should be removed.

Great! The less features to maintain the better :-). @ivannovick, can
you confirm that we can remove this error table feature in 5.0? If we do
that, no need work any further on this bug.

- Heikki


I don't think you should need to nest the `runs` as well, but I could be wrong. You should just write this spec in Babel and use async/await and avoid the headaches of the Jasmine queue.


please add a space between `//` and the comment text for the 3 last comments as described on https://github.com/facebook/hhvm/blob/master/hphp/doc/coding-conventions.md#comment-style


I do the first half of these calculations already.  I would read in that value and then do (100 - that value) for the other fraction so you're guaranteed that the total equals 100%


files is now saved to a folder*


I was referencing the updated USBCDC code that apple had, the timer is used for a watchdog/stats gathering that you didn't have implemented and I couldn't figure out how to implement. So it can be removed safely from my branch without issue. See my comment on this line: https://github.com/winsock/HoRNDIS/commit/fc99c5619033950bead6c99008370640300cda67#diff-ca55b4153ff6514687d29fdcf8132b87R947


is this wpa stuff also intended for wpa2?


Thanks for pointing that out, we need to revert this https://github.com/Euphoria-OS/android_frameworks_base/commit/fad6cf5a452486f2b7d285312580195e1fd2e6fe because we merged it manually before.


@PinpointTownes hey I just found out about this project from Marc Müller, what is it all about?


If ~~not~~ running on windows ...


Void было правильно, во-первых потому что в этом методе нигде нет return значенич типа String (поэтому этот код даже не скомпилируется), а во-вторых потому что возвращение строки и не нужно в этом методе в принципе.


Nothing is like Ballmer, Developers!!!


Thanks for the module and the collaboration, I think now everything works out for my use-case.


@evil-at-wow do you feel like adding boost to cmake? O=]


Shouldn't include code that's not in a function.  If you wanna make your life easy, do that trick with the if __name__ == "__main__" stuff so that code only gets run if it's called as a script.


this is awesome, but devs are not ready to accept it ;)


sure it does, i didnt accepted the merge request ;)


saw approval from lukebakken
at https://github.com/basho/riak_kv/commit/6674a28a2f1ac142711296ad35320ed9754d22b5


merging basho/riak_kv/fix/az/check-key-length-in-get-and-delete = 6674a28a into borshop-integration-1260-fix/az/check-key-length-in-get-and-delete


basho/riak_kv/fix/az/check-key-length-in-get-and-delete = 6674a28a merged ok, testing candidate = eeb57066


@lbianc  Please, also set upper case letter in the first letter after the space


no need for brackets here


I think the libGDX API sucks. We should have a separate ToggleButtonFacade for this functionality. Remember, one of the goals of a facade is to make an API suck less, you don't have to copy the original API


you can remove that. i'll refactor `_addToIndex` to handle duplicates.


Irrespective if we remove ErrorTable code, this needs to be added for
safety in the function.
As coverity will still complain unless we re-write the other parts of
function assuming it will never be NULL.
Plus, 4.3 will need it anyways. So, prefer to push this.


These labels are fixed though, they will be the same for every pie chart


why to keep commented code here?


break under the p statement plz


:+1: bien vu


same here


This should get replaced by custom fonts once I get that working


@jwise I was looking at this from 10.10.5 for some reference:
http://opensource.apple.com/source/AppleUSBCDCDriver/AppleUSBCDCDriver-4303.2.1/AppleUSBCDCECM/DataDriver/Classes/AppleUSBCDCECMData.cpp


Okay I will make adjustments as I integrate and use your code as an input


Yeah, it would be good to push this. Can you please split this patch to only contain this configure.ac change to first add the --enable/disable-webui option since the Makefile.am changes are for a different problem and I think I made some redundant changes in Makefile.am changes which I'll comment on separately.


For WetekPlay users the hbgic_input_mapping function should possibly look like:

*##################################*/
static int hbgic_input_mapping(struct hid_device *hdev,
		struct hid_input *hi, struct hid_field *field, struct hid_usage *usage,
		unsigned long **bit, int *max)
{
	switch (usage->hid) {
	case 0x00070029:	hbgic_map_key(KEY_F3);		break;
	case 0x0007002a:	hbgic_map_key(KEY_F2);		break;
	case 0x0007003a:	hbgic_map_key(KEY_F1);		break;
	case 0x0007003b:	hbgic_map_key(KEY_HOME);		break;
	case 0x0007003c:	hbgic_map_key(KEY_HOMEPAGE);		break;
	case 0x0007003d:	hbgic_map_key(KEY_MENU);		break;
	case 0x0007003e:	hbgic_map_key(KEY_ZOOM);		break;
	case 0x0007003f:	hbgic_map_key(KEY_FAVORITES);		break;
	case 0x00070040:	hbgic_map_key(KEY_MENU);		break;
	case 0x00070041:	hbgic_map_key(KEY_RED);			break;
	case 0x00070042:	hbgic_map_key(KEY_GREEN);		break;
	case 0x00070043:	hbgic_map_key(KEY_YELLOW);		break;
	case 0x00070044:	hbgic_map_key(KEY_TEXT);		break;
	case 0x0007004a:	hbgic_map_key(KEY_HOMEPAGE);		break;
	case 0x000700f1:	hbgic_map_key(KEY_BACK);		break;
	case 0x000c00e2:	hbgic_map_key(KEY_MUTE);		break;
	case 0x000c019e:	hbgic_map_key(KEY_MUTE);		break;
	case 0x000c01c2:	hbgic_map_key(KEY_BLUE);		break;
	case 0x000c0204:	hbgic_map_key(KEY_FAVORITES);		break;
	default:
		return 0;
	}
	return 1;
}
/*##################################*/
This way ensures, that all buttons on Wetek AirMouse remote will not loose their codes.
As a feature, one button at the bottom-right "<--" will be inerpreted as "F2" by kodi.


what about adding a constructor for that struct ?
what about making that struct "global" using the r_ prefixing and proper typedef with camelcase like the other ones?


I really think the project needs to have these standardized! Can I ask why you've picked (y, x) order instead of (x, y)?


The listing id shouldn't be unique. Several people can bookmark the same listing


Gotcha


This breaks GHCi outside of stack projects because we no longer find Prelude which is pretty bad.


Wait, did I just win an argument against the Dan? :fireworks: :facepunch:


Поскольку test1 объявлен с меткой throws, то его вызов либо должен быть окружён конструкцией try {}, либо метод main тоже должен быть помечен throws. Иначе этот код не скомпилируется.


wtf? if you need an array of sequential integers just use a generator or an array of of integers, not a linked list. it will be way faster


I had this script in there because I was initially making really frequent changes to `tree-sitter-compiler` module, and I wanted to always use the head of its master branch, so that I didn't have to keep publishing new versions to `npm.org`. You might just want to :fire: this script though, and use the published version. I'll publish patch releases whenever something important changes.


This won't work on the server side the way you want.  If either `users` or `groups` has a value, it will stomp (supercede) any value set in `subjects`, so this will still be broken.


Yeah, what I'm trying to go for is consistency in data output.  We could have it be some constant in a file, or you could read it in / be passed in as an argument.


if there are many items it is faster to sort once after adding all the elements than adding it sorted


unnecessary else


There are about a million places in the code where [y, x] was already in use. Only a couple places were using [x, y], though.
The image-output works fine when using [y, x] (i.e. the images aren't rotated) and I think I noticed some more advantages that I've forgotten by now.


positive conditionals first (if (is_middle) instead of (!is_middle)


1. Please issue a WARNING level log when this happens & determine how many messages were not received.

2. Also, what happens if this code is executing simultaneous with the receipt of the last message needed?  Crash.  You need to protect this and the receive code with a mutex.

3. Finally, you need to delete the entry in the trk hash table or the table will still fill up.


2 lines? no, you're going to reduce 5 blocks of instructions. We can replace from line 203-237 (34 lines) to only 12 lines, being also less repetitive.


Параметры тесту не нужны.


@brentschmaltz hello and welcome! :smile:

This project is an OIDC server middleware (well, rather a framework actually) that is heavily inspired from the `OAuthAuthorizationServerMiddleware` that came with Katana (same approach, similar primitives).

You can see it in action by cloning the repository and playing with the MVC sample.
It's also used in the whole new `OpenIddict` OIDC server I'm currently developing: https://github.com/openiddict/core


i think there was something in r_util for that


Hm, valoarea lui ```x.done? == true``` nu e cumva ```true``` daca ```x.done?``` e ```true``` si ```false``` daca ```x.done?```e ```false``` ?


Leak?


Thanks for the heads-up. During the next minor update I can remove the old version numbers from the README file so it only shows the ones available for download.


Вместо input нужно указывать непосредственно само строчное выражение .


More a question, you will get orientation events and resolution changes event won't you?


I'm afraid your master branch is somewhat outdated ...
Hint: never ever commit test patches to your master branch!


Aș fi lăsat-o drept ```done!``` sincer. Așa eu aș interpreta metoda aia drept interogare de valoare, nu un assignment.


I haven't actually tried that yet. Maybe I should backup my stuff and try it.


@borshop: merge


> What about this wording?

I don't like `valid`, because to me, the `none` state does not reflect an invalid state. Semantically, it means often "empty" or "nothing."

Overall, I'd prefer simply three adjectives. For the same reason we omit `get`/`set` prefixes in our style guide, I also find it consistent if we avoid `is`/`has` prefixes and use an adjective for boolean condition instead.


Strange, just merged file with last template and tests on Russian date handler failed!

@vassilii or @vantu5z might have a look at file for the Russian translation?


Nu sunt sigur, cine e ```@items_done``` ?


I think we'll want to use the more idiomatic way of constructing many-to-many relationships. Instead of creating an independent Bookmarks model, we'll want to create something like this:
```
association_table = Table('association', Base.metadata,
    Column('left_id', Integer, ForeignKey('left.id')),
    Column('right_id', Integer, ForeignKey('right.id'))
)

class Parent(Base):
    __tablename__ = 'left'
    id = Column(Integer, primary_key=True)
    children = relationship("Child",
                    secondary=association_table)

class Child(Base):
    __tablename__ = 'right'
    id = Column(Integer, primary_key=True)
```


Yes, indeed, isn't that genius? ;-)
This is what NetGuard differentiate from other solutions.

So, all traffic is routed into the sinkhole, except applications which are allowed (so disallowed from "using" the sinkhole).


FYI, I added the `_hash` stuff as an optimization, as `get` and `set` should generally be faster using an object key lookup than iterating an array. This update removes all those optimizations.


saw approval from lukebakken
at https://github.com/basho/riak_kv/commit/e1b2acd0f35466e4c13efff668123676a0b0f518


merging basho/riak_kv/fix/az/split-ts-bits-off-riak_kv.proto = e1b2acd0 into borshop-integration-1259-fix/az/split-ts-bits-off-riak_kv.proto


basho/riak_kv/fix/az/split-ts-bits-off-riak_kv.proto = e1b2acd0 merged ok, testing candidate = d2d476dd


saw approval from lukebakken
at https://github.com/basho/riak_kv/commit/6674a28a2f1ac142711296ad35320ed9754d22b5


merging basho/riak_kv/fix/az/check-key-length-in-get-and-delete = 6674a28a into borshop-integration-1260-fix/az/check-key-length-in-get-and-delete


basho/riak_kv/fix/az/check-key-length-in-get-and-delete = 6674a28a merged ok, testing candidate = 7e04500d


added tests for missing ALU operations => added missing tests for ALU operations


saw approval from lukebakken
at https://github.com/basho/riak_pb/commit/388c5fddff980ec68b608442fce7d66b0db9560b


merging basho/riak_pb/fix/az/split-ts-bits-off-riak_kv.proto = 388c5fdd into borshop-integration-157-fix/az/split-ts-bits-off-riak_kv.proto


basho/riak_pb/fix/az/split-ts-bits-off-riak_kv.proto = 388c5fdd merged ok, testing candidate = bdac78c8


I'm ok with making a constructor to simplify the code and on using camelcase, but  i think we don't need to make it global because it's used only in this file (for this same reason i didn't use the 'r_' prefix)


I"ll change it on our **next** branch. Leave it for this PR then


Please can we get out of the habit of making already horribly overgrown functions even longer than they already are?  It's extremely hard to understand/maintain/debug functions when they are this long.  This should have been an opportunity for removing the code smell by refactoring the existing function into smaller pieces, not making the smell even stronger :-(

Hmm, maybe it's time to [build a Code Climate engine](http://docs.codeclimate.com/article/300-the-codeclimate-platform#build) for shell ...


implementing it on **next** branch


Also, should have a way to be just run as a script.  Currently you have to import your script and then run the function manually.  You should instead be able to run it as a script (_ _ name _  " _ _ main _ _ " {spaces so that markdown doesn't go crazy}) with the appropriate arguments being passed along to the method.


In Alloy is usual we declare us variables at the top of function. So, could you pull the variable declaration to the top of the `clearFieldError` function?


ok
On Nov 6, 2015 7:02 PM, "Riccardo Schirone" <notifications@github.com>
wrote:

> I'm ok with making a constructor to simplify the code and on using
> camelcase, but i think we don't need to make it global because it's used
> only in this file (for this same reason i didn't use the 'r_' prefix)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ret2libc/radare2/commit/0f3d689d2221b0936183a8b3aadf51b87ab2291d#commitcomment-14248188>
> .
>


sorry, this is needed for the ABI registers on the cpp file. The comment was wrong, simply remove that.


Sorry. Didn't realise the README listed the available versions. I've updated it.


develop (macumber) - i386-Windows-7-VisualStudio-12: OK (2335 of 2370 tests passed)

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


Sorry, I gotta say I keep confusing `ID` and `DiffID`. Wouldn't it better for `Layer.ID()` to be what is today `DiffID` (because that is the ID of the _layer_), and the other one be the `ChainID()` (the ID of the chain _including_ this layer)? I know I'm bikeshedding, but it just doesn't stick in my mind.


Very happy this has its own type :+1:


I need to remove elements from this list. I want to always take the lowest element, but i can reintroduce elements in the list. Using an array of integers would require to traverse every time the array from the start to check whether the relative element is available or not.


Works for me — thank you!


I think this can be removed, since I was initially confused about which component was meant to be linked against libgputop.so.

The gputop program itself is meant to just be a launcher that saves you from setting the environment variables needed to interpose OpenGL so shouldn't need to be linked with anything interesting.


Same for lines 895, 898, 980 and 983.


5255-EMS-dual-Temp-Node-Setpoints (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (888 of 888 tests passed)

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


@ke-sun , we do this so that all the imu messages and related params are under the namespace /node/imu/


What's wrong with http://downloads.sourceforge.net/project/boost/boost/1.59.0/boost_1_59_0.tar.gz ? That seems more stable than fossies.


if its always sequential you only need a “min” and “max” values, are those values removed randomly? because it looks like its always taking the first which means min++;

> On 06 Nov 2015, at 19:05, Riccardo Schirone <notifications@github.com> wrote:
>
> I need to remove elements from this list. I want to always take the lowest element, but i can reintroduce elements in the list. Using an array of integers would require to traverse every time the array from the start to check whether the relative element is available or not.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/ret2libc/radare2/commit/0f3d689d2221b0936183a8b3aadf51b87ab2291d#commitcomment-14248239>.
>


i can't to that here, because inside the loop(at line 168) i'm using r_list_pop to always take the lowest element.


ahh ok


There's a reference to #projectOrganizerScope left over in `website/static/js/pages/dashboard-page.js`.  I couldn't find it anywhere, so I'm guessing it should be deleted?   That file also has some commented-out request code that I think can be removed unless you were saving it for future reference.


Example:
def piechart(...):
    pass
def read_in_args():
    pass
def main():
    args = read_in_args()
    piechart(args)
if __name__ == "__main__":
    main()


@jameskoster does this really need to be applied to all `td`s in the shop table? With the recent update a lot of shop tables are looking a bit wonky in full width templates.

Example 1: https://cldup.com/dPOKtG7gmJ.png
Example 2: https://cldup.com/TnBUWUej1Z.png

I know it can be overridden obviously, just feels a bit heavy-handed for every table cell to have this as the default.


@davidtkelleher would you mind to move the text to the respective `yml` files?


Isn't this just going to call itself? Wouldn't you want something more like this:

```js
const old = request[method];
Request.prototype[method] = function () {
  const r = old.apply(this, arguments);

  // ...
};
```


See
  https://github.com/ArangoDB/velocypack
and the readme there.
In 2 sentences: a new binary JSON like format which allows fast access to subvalues without parsing or unpacking. We are going to use it in ArangoDB internally.
  Best,
    Max

Am 6. November 2015 16:47:06 MEZ, schrieb CoDEmanX <notifications@github.com>:
>If you don't mind: In two sentences, what is VPack?
>
>---
>Reply to this email directly or view it on GitHub:
>https://github.com/arangodb/arangodb/commit/861661774a2f692e6bf173441e7c1e452a38e6df#commitcomment-14244684


Чуть позже заметил, когда в стайлгайд полез )


Не знаю, ошибку  понял )


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


This version is breaking the Seeding command.


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


pop is min++...

> On 06 Nov 2015, at 19:07, Riccardo Schirone <notifications@github.com> wrote:
>
> i can't to that here, because inside the loop(at line 168) i'm using r_list_pop to always take the lowest element.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/ret2libc/radare2/commit/0f3d689d2221b0936183a8b3aadf51b87ab2291d#commitcomment-14248283>.
>


Нужно экранировать апострофы: [здесь](https://github.com/sawka6630/unchecky-translate/commit/8c955b2d3be5c0fc659522add4df83ffffecf703#diff-73d09f053f382874997527b4ccf65f64R14) и [здесь](https://github.com/sawka6630/unchecky-translate/commit/8c955b2d3be5c0fc659522add4df83ffffecf703#diff-73d09f053f382874997527b4ccf65f64R134). Я недосмотрел, а исправить сейчас не получается.


Вроде контент, ссылка на главную страницу.


mit em neue build prozess chasch jetzt de build level injecte, und denn je nachdem en bedingig ifüege. chöntsch das na mache? BuildLevel isch es enum ;)
```
@Value("${reimbursement.buildLevel}")
private BuildLevel buildLevel
```


some tests failed:
failure: http://buildbot.bos1/builders/test-riak_kv/builds/2709


saw approval from lukebakken
at https://github.com/basho/riak_pb/commit/388c5fddff980ec68b608442fce7d66b0db9560b


merging basho/riak_pb/fix/az/split-ts-bits-off-riak_kv.proto = 388c5fdd into borshop-integration-157-fix/az/split-ts-bits-off-riak_kv.proto


basho/riak_pb/fix/az/split-ts-bits-off-riak_kv.proto = 388c5fdd merged ok, testing candidate = c3ce1806


panaic?


Are all of these warnings really obsolete?


Great! Your comments are extremely helpful. The result might be slow to come by but I AM keeping them in mind. Thank you!


Could you just remove this line from your commit? You can edit your commit using `git commit --amend` then `git push --force` to push it again. Thanks!


good!


with a few changes to this function it could go into the `interfaces` folder


:+1: on the move to an event based workflow.


There's a comment in `website/routes.py` referring to projectCreator.js:

    # Create project, used by projectCreator.js

Please update the comment with projectCreator's successor, or delete it if there isn't one.


yeah, i started a new PR with a fresh implementation and JS tests that only has the bits that are flaky here, in an attempt to figure out what's going on. I just added a passing test using waitsFor, and am gonna make it pass with async/await next. It seems so much more palatable than messing around with jasmine.


could use `is_deprecation`


We did read your previous comment on this, this is the fix to quote it correctly.

errtblname (not the errtablename one) is not quoted, and it concats the prefix later in the PQExpBuffer.

then tmpStr = fmtId(but->data)  does the final quotation properly.


I haven't checked Doug (since it requires Javascript to be on
and I very rarely allow that), but I am assuming that since you didn't
say anything that my original commit of Ivan's updated Russian
translation did /not/ break any Travis tests.  You are saying that it
was only when Jerome updated the template and then merged it
with ru.po that something happened.  Very strange.  Here's my commit:

https://github.com/gramps-project/gramps/commit/c42d4cf3f8b5a10b027928b2ceb1e8422ba507d3


I had to revert that, because without the splash, there were no footstep sounds anymore at al.


If not iF


add code from
Main::onCommand() - case 'love'.


This allows the SAM to extend across non-Syriac letters, which it arguably shouldn't do.... silly example:

  echo 070F 072B 0718 0061 0062 0063 0712 002e | hb-unicode-encode | hb-view seguihis.ttf

shows an abbreviation mark that spans the embedded "cba" as well as the Syriac letters.

Unlikely to affect most clients, though, if we assume that text will normally be partitioned into script runs before being passed to harfbuzz. So maybe not worth trying to do anything smarter here.


i know this is not the right commit where you added it, but i feel like `casedb_base.py` could go into` interfaces`. i feel like it'd be nice if we had one folder where we could see all the base classes that get used in both SQL and Couch


LGTM, but please remove "r=cpeterso" from your commit message because I don't have the authority to r+ Gaia code. <:-)

It seems like `getSignalBars()` should be a function or getter on the `network` objects instead of a standalone function. But that is beyond the scope of this bug fix. And, someday, someone should refactor other Gaia code to use your new `getSignalBars()` function.


It seems a bit surprising to see the *_SYMBOL types included here; do we know whether Uniscribe lets the SAM span things like a dollar sign?


Mh, bower richtet sich nach den Tags auf Github, deswegen mein ich nur


Am uitat sa sterg metoda , nu se apeleaza niciodata oricum