The firing order of inline event handlers is still incorrect right?:

```javascript
window.addEventListener('click', function() { console.log('first') });
window.onclick = function() { console.log('second') };
window.addEventListener('click', function() { console.log('third') });
```


schön hast du das gemacht !!


local_type_declation 도 그냥 type_declation 처럼 원래대로 두는 것이 좋을 것 같네요. 그런데 제가 보기에는 local_type_declaration과 type_declaration이 동일해 보이는데 이렇게 두 개로 따로 관리하는 이유가 있나요?


ja ich bin ganz deiner meinung!!


@jsrain : I think not. The systems are visible in SCC but I think SCC do not have any report.
In SUSE Manager we are trying to get a better report. The idea is, that somewhen SMT gets some of the infrastructure from SUSE Manager and report all needed data for correct calculation to SCC. Than SCC can generate this report for direct registered systems and SMT systems in combination with the report of a SUSE Manager.
But lets talk about this next year :-)


Alves, with an s.


Fix this.


du bist einfach der beste!!!


Я забыл удалить старые


no.


30/07/2015 18:23:12


Static, if anything, I think.


und schön bist du auch noch!!!


Not very robust.


This method is called `updatePresence` in the java lib.


yes, I can :)
but I had to reset my branch :/
I forgot to check if your code really works, it doesn't.
you should use CMAKE_BINARY_DIR instead of CMAKE_CURRENT_BINARY_DIR


Not very robust. -- joe alvez


사실 assignment_operator로는 거의 모든 binary operator가 사용될 수 있습니다. 예를 들면 "|=", "&=", "^=", "<<=", ">>=" 등이 추가될 수 있겠죠.


Why doesn't `isArrayLike` handle the isObject case?


strings.


whoops, that'll be fixed locall. I'll also do a review of all the docstrings first.


This line has to be a method chaining off of the supertest calls


@ykhodorkovskiy This should take care of the kernel denials


@ntdb This commit breaks timeline generation if no meeting days are selected. The entire timeline is *after* the end of the assignment and/or course. See the travis error, where the entire duration of the course has "no meeting this week", or create a course locally to see the result.

The timeline generation needs to take into account weeks with no meetings, when determining the number of weeks to squeeze a timeline into. So if an assignment is 6 weeks from beginning to end, but one of those weeks is blacked out, then the assignment should be squeezed into 5 weeks.

Also, selecting no meeting days should be a valid option, in which case the system should assume that no weeks are blacked out (unless a week is manually blacked out on the calendar).


Thinking about it, should we relax it to return `Iterable`? Would give implementors a bit more flexibility, but then I don't think batch sizes will be unreasonable high in reality.


@brendo @michael-e I've tested it and yes, it does solve problems... Was it committed ?


Facepalm. There's really no way to do this for all commands at once?


You shouldn't include logs in your PR. Add files one by one to your commit or include files that shouldn't be commited to your .gitignore file


Nice.


I want to send datas from python to the workspace of matlab ?
I'm looking for a command line to do that with mlab ?


there are no strings on me

On Thu, Jul 30, 2015 at 11:25 AM, John-David Dalton <
notifications@github.com> wrote:

> strings.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lodash/lodash/commit/fbe5b7936b3f030509719a37c2daf7d6ddb96896#commitcomment-12446910>
> .
>


test the commit


:+1:

As we discussed, I think there may be additional use-cases to consider when deleting a template interest, but this clearly plugs a hole that currently exists.


>Alright, an example would be maintaining "no censorship beyond legal compliance" and dealing with the (hypothetical) loss of business.

Yep, however I think we already discussed this example enough.

>and once it was deemed to go off the rails, the thread was locked using maintainer controls, as should be. I'm not seeing the problem.

It wasn't supposed to be an example for a problem, just that shitstorm threads happen sometimes and people come other to this thread even if they have nothing to do with it (and therefore GitHub isn't really isolated). Topics like this though have potential to escalate if the repo maintainer isn't moderating (but so far I haven't experienced this yet here).

>That's a risk of doing business.

And they are trying not to trigger risks.

>The exact value of 'ethics' and whether it trumps business is up for personal interpretation.

Though businesses sometimes get pressured to do stuff like this (though most likely not in this case).

>My argument has been about censorship of legal content from the start, not about illegal content. Whether GitHub should be removing illegal content is a much, much more complex topic, and far beyond what I'm discussing right now.
Assuming that illegal content should not be hosted by GitHub for legal reasons, the supervisor should be GitHub. But that wasn't the topic of discussion here.

Okay, that was written poorly from my side. I meant that no supervision at all could derail to illegal content/escalating hate speech. This repo here got locked because some user reported it, not because one GitHub employee found it randomly and locked it for his own personal reasons.


Isn't exporting a noop function needed to avoid errors? I'm sure you've already tested it and I assume it isn't. Weird...


이거 아직 이중 배열은 적용안된 상태인거죠?


이것도 원래 assignment 구현한 커밋에 합쳐주세요


TeamCity xBIM :: XbimExchange :: Build [Build 376](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=2169&buildTypeId=XBIM_XbimExchange_Build) is now running


We don't want to init to 0 the error buffer at each call. What the old back-end do is init it to 0 when we alloc. Then we init it only when we detect an error. That way, when there is no error, we only need to read this value and not zero it everytime.


this will be useless in a few days, when AtkFr and AtkBy are ready :)


Init to 0 at alloc time. See the other comment.


Not sure that this change isn't the reason for backend save & edit errors; maybe will have to revert


Don't use fixed numbers since it will make adaptation to different devices and modes very difficult. I would recommend never using the layout editor since it will always look better and be more adaptable if you use a layout by yourself.


Should be `Bicycle`.


Use meaningful names


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.861.204025c079d11292272c66cdbcfe94863e8af973](http://build.palaso.org/viewLog.html?buildId=60252&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 396, ignored: 3 Build time: 00:03:38


Cheers Mark, I'll add this fix soon.

On Wed, Jul 29, 2015 at 11:53 PM, Mark Streatfield <notifications@github.com
> wrote:

> I think this change has broken the unit tests for test_release as this
> method is now not implemented on the StubReleaseVCS class.
>
> I have lost push access to github.com at the moment (firewall) so can't
> make a pull request. However I resolved this locally by adding the
> following to the StubReleaseVCS class.:
>
> def tag_exists(self, tag_name):
>     data = self._read_stub()
>     if "tags" not in data:
>         return False
>     return tag_name in data["tags"]
>
> And then updating test_release.py so that when create_build_process
> <https://github.com/nerdvegas/rez/blob/resources2/src/rez/tests/test_release.py#L89>
> is called ignore_existing_tag=True is also passed through.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nerdvegas/rez/commit/6a0e49066a554544c5e7f596bed57570d2a2f195#commitcomment-12437241>
> .
>


TeamCity xBIM :: XbimExchange :: Build [Build 376](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=2169&buildTypeId=XBIM_XbimExchange_Build) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:02:08


Whoops wrong user got tagged. @bjeavons :)


I don't see any good reason to use inner classes instead of public classes all in the same package.

eg. very hard to see what are the actual method(s) of `MetricDefinition` (which in addition are pushed to the end of the file)


Don't use @+id in references. Use @id instead


Same


I could also have added it as a test fixture, though this is generally usable, so I opted for not doing that.


This "works" if I change the `this.tweetLength` on line 41 to a number.


Add here your modules that need to be loaded in


Add here the Commands that your are going to use


Don't use @+id in references


Remove this in future?


Why is there a space in the beginning here?


Don't use fixed dimensions


Google play


Эти атрибуты лишние. Видно, код был скопирован из какого-то примера.


eigenlob stinkt.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.862.40912218c31e1293166c894de50513c07bae89bc](http://build.palaso.org/viewLog.html?buildId=60253&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 396, ignored: 3 Build time: 00:05:51


Don't use raw colors


I do not understand this. Why this is useful? Why it is needed?


nö


Because the assets might be located in the theme directory, or in the shared un-themed css directory. Also, for caching reasons, css/theme.css is different from css/theme.css if the theme name has changed. I'd like that to be part of the urls.


Thanks @ajoslin.

The issue is that with `$animate.removeClass`, it's async so there's no way (at least that I know of) to detect when that happens.


Man, already have deprecated methods and not even on v1 release yet. We're a real library now.


außerdem war ich das nicht
da steht ja auch DU bist der tollste


Disable or remove this


don't forget to document that only metrics declared as input can be retrieved (otherwise IllegalAccessException or something)


RelativeLayouts don't have an orientation. Are you sure you don't want to use a LinearLayout?
Please, don't edit the layout with the layout editor, since it's destroying your previous work. You can change it from the code and will get better results ;)


someone was tired


This should be a LinearLayout. Please, don't use the layout editor


Don't use @+id for references


Why 1.6 and not 1.7?


Don't use fixed dimensions


@gomzeegautam the best solution we've found for serving static files is to use compojure's `route/resources`.
You can see an example of that here: https://github.com/invisiblefriend/door/blob/develop/src/door/routes.clj#L89. That will serve anything from the public directory I believe


Use meaningful names


What's the rationale for this `exit()` call here, in the non-crashed branch? I thought the previous behaviour of not exiting was by design. If the user wants to exit, they can do that in their implementation of `websocket_terminate`.


@Alkarex / @aledeg > have you time to read this article? You can see the result on http://dev.freshrss.org/freshrss-supports-pubsubhubbub.html


Same


As long as both arguments are valid GeoJson are valid polygons or multipolygons, this should never fail.  This is done out of an abundance of caution.


@kbenne We got a build of this and tested it.  This fixes the translators; they now write the table objects.  Thanks for the quick fix.


ah - NOW it is reverted. I thought I had done so, but apparently the wrong branch. now I have cherry picked and pushed the reverting commit.


I agree this parameter is some kind of context, but it is really specific to a single component. So, maybe the name of the type should reflect that or the name of the method (eg. `computeForComponent`)


I'd prefer to keep the lineage (since this is just a fork of an upstream project). We only have one dependency on it, so I think it isn't too hard to have a long name here (even though it is ugly).


Thanks everyone! :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: this release fixes _a lot_ of bugs.


Oops


Like this! tons of the changes just have to do with relative paths


It would be helpful if a few words could be added explaining (a) why this check box should be cleared and (b) what are the consequences if you deploy with it selected.


Use a LinearLayout where appropriate


Define in colors.xml


Define in strings.xml


Перехватывает запросы к адресу `/static/**` и перенаправляет их в `/resources/`, где должны лежать js и css файлы.


Don't use fixed dimensions


I had to re-spin the release due to the maven version issue


Subordinate clauses rarely change tense. The entire sentence is a description of the code book that follows. "The example below does A, otherwise doing B:" is not grammatical. "The example below does A, otherwise does B" is.


return type should be an `Iterable`. type `List` puts on the constraint to have all the measures in memory before returning


Don't use @+id for references


@Myoldmopar super. Guess I'll cancel the build i have going then...

<img width="1440" alt="screen shot 2015-07-30 at 10 43 33 am" src="https://cloud.githubusercontent.com/assets/707488/8987683/d96e51ba-36a7-11e5-8b85-08a12a425fc2.png">


@BrandonMathis: @iandonovan and I talked about it and think this kata should be split into at least two parts in the curriculum. The unromanize function took us all of yesterday morning to do, and then this took us almost all of this morning to do.

Also, we found out there's [a way to do roman numeral arithmetic directly](http://scienceblogs.com/goodmath/2006/08/16/roman-numerals-and-arithmetic/). It might be neat to have a kata just for implementing that algorithm.

So, if you agree with all that, the new katas should be:
* unromanize - Convert a roman numeral to decimal
* roman_math - use something like I've done here to do the math (you should note that eval is not allowed, or this would be fairly easy)
* roman_math 2 - Do the roman numeral arithmetic directly via [this method](http://scienceblogs.com/goodmath/2006/08/16/roman-numerals-and-arithmetic/)


I started learning rust yesterday. Criticism is very appreciated.
I wanted nicer error reports here:

    $ hematite world/level.dat
    thread '<main>' panicked at 'Could not read level `world/level.dat/level.dat`: Not a directory (os error 20)', src/main.rs:67

instead of

    thread '<main>' panicked at 'called `Result::unwrap()` on an `Err` value: Error { repr: Os { code: 20, message: "Not a directory" } }', ../src/libcore/result.rs:731


Define in dimensions.xml


Some plugin (like SQALE) needs settings to compute its measure


This form causes issues when err doesn't have .message defined (at least on iOS). Execution just stops when I come across one of these. Timeouts still work though. On a side note, assert doesn't appear to be part of the official Tape API (although it's aliased to .ok in code).

I've tended to replace these with:

      t.ok(!!err2, "Connect to bad peer should throw error");
      t.ok(err2.hasOwnProperty('message'), err2.message);


Better written as `return self.call_api('jobs/analysis', self.URLRequest.POST, args=job_status)`


Without workflow dependency you can't define a new DSL keyword.
The `wrap` syntax is such an ugly hack I don't want to offer this to end users :P


Line commented out


In fact I think that it should compute only one Metric


Better written as return self.call_api('jobs/alm', self.URLRequest.POST, args=job_status)


Make sure you don't include log files in your commit


Not sure, but nothing in the icon is clipped :(


Stray print statement


I didn't want to put too much javadoc as this API will probably evolve a lot, but I could make an effort to do the minimum to understand


`MeasureComputer` misses a method to inform of metrics for which it is capable of providing measures such as `Set<String> getOutputMetricKeys`


Indeed. Relying on Docker-plugin there's no backward compatibility anyway, until this one becomes stable (or we use an alternate slave template class).


is this `connector_type` the same as the `connector_type` in `update_job_status` ? They should match if possible, just so its easier for developers


Great job !


Maybe you could also log when SF is up to date. Like this nothing is printed and the user is left in the dark. Or not?


`@Autowired` использует Reflection API.


@dzafman the change looks good to me. and it passed in my local env.

i am glad that they helped you build your tests =) @guangyy is the original author of them.

but why are you removing `TEST_rados_get_with_subreadall_eio_shard_0` ? the reason is https://github.com/ceph/ceph/pull/5309#issuecomment-125301784 , right ? i am fine with this removal though, as we can always add it back once we want the EIO tolerance feature in future. just asking out of curiosity.

what do you think, @guangyy ?


Since we control the value, might be better to just name the type above having no spaces in it


По-видимому, да. Чтобы не вводить такие проверки в дальнейшем будем использовать тесты.


Okay, đã thấy phần Entity của Long ;)


Is this the commit that added Geopackage support to DB Manager? I have just tested QGIS master and while it is possible to add new geopackage connections (in DB Manager) then all the layers/tables are listed as geometryless and anyway is not possible to add them to the project. Adding geopackage layers using the "add vector" dialog works. Geopackage connections are still not working in the QGIS browser.


Does it sound better just as `filter_connection`? Combo might be too much for users to understand


I might put the logic for generating the JSON for the encode into the Encode file. Also it doesn't appear the generated array conforms to the basic structure laid out here
https://github.com/divmazie/jonathancoulton-theme/blob/dockerDO/docker/encodebot/src/proc2.php#L41

###Notes
1. source_md5 can be generated with md5_file
2. encode_format is from list here https://github.com/divmazie/jonathancoulton-theme/blob/dockerDO/docker/encodebot/src/ffmpeg-output-formats.php (the array keys)
3. use built in functions for urls and make sure they point directly to the attachment FILE, not to the post page
4. metadata should be in the metadata array

Also note that encode CLIFlags should be modified to specifically refer to FFMpeg encode flags, as those are different. All encoding && basic metadata tagging is done through FFMpeg

Also, here are metadata fields we definitely can set
https://github.com/divmazie/jonathancoulton-theme/blob/dockerDO/docker/encodebot/src/ffmpeg-metadata-fields.php (we only attempt to set fields with those keys)

Also, recommend defining a PSR-4 composer autoloader on the classes directory.

https://getcomposer.org/doc/01-basic-usage.md#autoloading

This way, you include the single composer autoload file in your functions.php and, voila, your whole theme has access to whatever is in the classes directory (provided it is named right). Then you don't need to do any require once shit in any of those files.


Was this removed so tests pass? We don't want to remove this


As far as I see (in 2.6.2): Yes, it has been implemented.


I was confused, because I didn't see the revert commit. But why revert it?


On line 91, the method is defined as `self.import_and_return_unsaved_record`. On line 131, it's called as `self.import_and_return_record`. Unless I'm missing something, every call to `self.import` as currently written will be a `NoMethodError`.


(I set the core version to 1.4.1 in the db manually)


@cfare: Don't just blindly copy code -- sometimes it isn't correct.
Understand and use scope!
The egghead.io videos are great on it!
Make sure you understand all parts of this change. Ask, if something is unclear.


did you check for any speed difference?


@neappx As @nixxquality has said multiple times ITT, this repository was locked because it was a _fork_ of the original repository that got reported. This particular repository was not reported.


Just here to say thanks! Sorry if it's the wrong place for it. I'm very thankful for your paragraph "One thing to point out is that we don't set our state with getInitialState but instead use state =. This is because React works with ES6 classes by setting state in the constructor." Owe you one.


> Yep, however I think we already discussed this example enough.

Well, you wanted a realistic option - that is one :)

> It wasn't supposed to be an example for a problem, just that shitstorm threads happen sometimes and people come other to this thread even if they have nothing to do with it (and therefore GitHub isn't really isolated). Topics like this though have potential to escalate if the repo maintainer isn't moderating (but so far I haven't experienced this yet here).

Yes, but they did not come in *through GitHub*, as GitHub has no real discovery features. They came in through other channels. This is no different from a free forum host - should a free forum host disable a forum because somebody used a word on a member's forum that somebody else deemed offensive?

> And they are trying not to trigger risks.

And my point is that these risks exist regardless, thus removing the risk of PR damage 100% is not possible. That means you should be *very* conservative with your risk mitigation strategies, and very carefully consider the tradeoffs. In this case, there's absolutely no indication that non-moderation of things like this repository would harm PR in such a way that it would drive away business.

> Okay, that was written poorly from my side. I meant that no supervision at all could derail to illegal content/escalating hate speech. This repo here got locked because some user reported it, not because one GitHub employee found it randomly and locked it for his own personal reasons.

The decision to lock the repository was still a subjective staff decision by GitHub, regardless of whom the report originated from. They could've just as easily said "sorry, this is outside of our scope of responsibility, go pick it up with the repository maintainer". They didn't.


expect(response.statusCode).to.be.equal(404)


@Keader I saw that little bug I will correct it.

@krizttianmoreno  I will work for that too :)


@cfare: This is a no-no.
From the docs:
`The only appropriate use of ngInit is for aliasing special properties of ngRepeat, as seen in the demo below. Besides this case, you should use controllers rather than ngInit to initialize values on a scope.`
https://docs.angularjs.org/api/ng/directive/ngInit


Definitely (*ITC* being another typo as well), fixed in https://github.com/translate/pootle/commit/f9fd395a35d27fc712c9294c676c6c5a3acf66b3.


Also fixed lantern transformation in tent


> But why revert it?

just to avoid others dealing with the same issue


Thanks!


Is this solution safe?
* What happens, if one flush starts before the previous one has finished (e.g. when interval is 10 ms)?
* What happens if you try reading data before current flush is finished?


Good job, I like coconut oil.


Great, I didn't know about that convention.


:sob:


Do we still need refresh? I though we were removing it as not required any more?


ich weiß dass ich der tollste bin.


klappe


> just to avoid others dealing with the same issue

Well, at some point, we *will* have to deal with the issue anyways :smirk:


das du war auf Mich bezogen!!!


Thinking this further, we should move goOnline to onStart() since it happens before onResume(),


keine ahnung wer meinen account gehackt hat


aufjedenfall hat er ahnung


Do we still need to do this ?Lets just remove it, we will give a really long session timeout


typo: `dataRef` != `patchRef`


Maybe replace this with a list comprehension.


offiziell schizophren


im gegensatz zu dir


Hi Flinny, to make it build here I had to add this commit https://github.com/SlimRoms/kernel_htc_msm8994/commit/70730bca1331fc50c3caacaea00439de1325bd6e

Cause it was erroring on timeconst.pl in line 373, array and omit defiend stuff.

Just wanted to let you know


nö


```sh
echo $'\xF0\x9F\x90\x95'
```


Thanks Stephanie!


It looks like XOP is a soap in MIME with binary sections replaced with
includes. Will suds know how to parse this?

On Tue, Jul 28, 2015, 2:35 AM Alexander D. Kanevskiy <
notifications@github.com> wrote:

> it might be cases, where server returns multipart xop+xml:
>
> HEADERS: {'date': 'Tue, 28 Jul 2015 09:33:41 GMT', 'content-length': '4516', 'content-type': 'multipart/related; type="application/xop+xml"; boundary="uuid:516928f2-fa54-4683-bda4-4e7345e9eba7"; start="<root.message@cxf.apache.org>"; start-info="text/xml"', 'connection': 'close', 'server': 'Apache-Coyote/1.1'}
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/armooo/suds_requests/commit/6af6737aa0b49232e5faf7845aed256114587ab0#commitcomment-12391812>
> .
>


Yes, that's true.


fail :facepunch:


This new test of yours, doesn't work for me very well.
After a short while the watchdog kicks in and resets the device.

pm3 --> hw st
#db# Memory
#db#   BIGBUF_SIZE.............40000
#db#   BigBuf_hi  .............40000
#db# Tracing
#db#   tracing ................1
#db#   traceLen ...............0
#db# Fgpa
#db#   mode.............HF
#db# LF Sampling config:
#db#   [q] divisor:           95
#db#   [b] bps:               8
#db#   [d] decimation:        1
#db#   [a] averaging:         1
#db#   [t] trigger threshold: 0
#db# USB Speed:
#db#   Sending 512000 bytes payload...
Waiting for a response from the proxmark...
Don't forget to cancel its operation first by pressing on the button
Status command failed. USB Speed Test timed out
WARNING: Command buffer about to overwrite command! This needs to be fixed!
WARNING: Command buffer about to overwrite command! This needs to be fixed!
WARNING: Command buffer about to overwrite command! This needs to be fixed!
WARNING: Command buffer about to overwrite command! This needs to be fixed!
WARNING: Command buffer about to overwrite command! This needs to be fixed!
WARNING: Command buffer about to overwrite command! This needs to be fixed!
WARNING: Command buffer about to overwrite command! This needs to be fixed!
WARNING: Command buffer about to overwrite command! This needs to be fixed!
WARNING: Command buffer about to overwrite command! This needs to be fixed!
WARNING: Command buffer about to overwrite command! This needs to be fixed!
pm3 -->


I'm wondering if this 21 version is an Android M+ version.  I would suggest deleting it and sticking with the api16 version.  If you unzip and compare the api21 version with the M version on APKMirror: http://www.apkmirror.com/apk/google-inc/chrome/chrome-44-0-2403-54-android-apk-download/

you will see the same "crazy.xxx.so"...


This fix, as suggested in #324, brakes ODText Writer


das mit dem gegensatz war auf schizophren bezogen !!!


opfa


Not sure, I like the readable default from `ConsolePrinter` where you can see which systems are supported.
I maybe can also add the None case to the ConsolePrinter itself together with the default^^
Yeah that's a good idea, I'll do that :)


hdf


Does this comment really make sense any longer?


no fixy this? @aluttik


This seems to break the current existing files on master:
[INFO] --- license-maven-plugin:2.11:check (default) @ hawkular-end-to-end-tests ---
[INFO] Checking licenses...
[WARNING] Missing header in: /mnt/storage/projects/hawkular/modules/end-to-end-test/src/test/groovy/org/hawkular/inventory/rest/test/InventoryITest.groovy
[WARNING] Missing header in: /mnt/storage/projects/hawkular/modules/end-to-end-test/src/test/groovy/org/hawkular/integration/test/Scenario1ITest.groovy
[WARNING] Missing header in: /mnt/storage/projects/hawkular/modules/end-to-end-test/src/test/groovy/org/hawkular/integration/test/AbstractTestBase.groovy


If you supply a property that does heavy calculations it's your fault imo.


LGTM!


Yeah, about that.  Sorry.  Why aren't you incredibuilding?


Not sure how it would be an M version since I pulled it from the Play Store
using APK Downloader with my device ID (Samsung Galaxy S3 T-Mobile)

On Thu, Jul 30, 2015 at 11:01 AM, Nicholas Buse <notifications@github.com>
wrote:

> I'm wondering if this 21 version is an Android M+ version. I would suggest
> deleting it and sticking with the api16 version. If you unzip and compare
> the api21 version with the M version on APKMirror:
> http://www.apkmirror.com/apk/google-inc/chrome/chrome-44-0-2403-54-android-apk-download/
>
> you will see the same "crazy.xxx.so"...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/opengapps/arm/commit/b8f9df7778896e3310a1100fdca1f73a4166f333#commitcomment-12447935>
> .
>

--
<http://www.facebook.com/djalik>  <http://twitter.com/#!/DJAlik>
<http://www.linkedin.com/in/alexrusinov>
<http://plus.google.com/114580313768137070684>
<http://www.mixcloud.com/djalik/>
<http://psnprofiles.com/DJAlik>


okay I'll call it `obj`


The Hab and Ag Modules are replaced by Mk-V equivalents as of version 0.31.0


@licanulla tested fine for me with sails-mysql, sails-postgresql and sails-mongo--something else might be going on in your code.

@jakutis again, the apps were already broken.  Support for updating a record's primary key varied amongst the adapters (it worked in sails-postgresql, did nothing in sails-mongo and crashed the app in sails-mysql).  And it's just bad practice that no self-respecting ORM allows.


do you really want everyone to know your email???


Git reset --hard!


@thesp0nge your new simpler logger has no method `ok` and it fails to report in some cases like this.

```
lib/dawn/reporter.rb:33:in `write': undefined method `ok' for #<Logger:0x000000036d3988> (NoMethodError)
```


这个 commit 回头不好合并。


no I don't, ConsolePrinter still inherits from ColorPrinter which contains the color functionality^^
LogPrinter only wraps the printer and adds logging functions, existing print behaviour persists.


加えてurlの設計についても追加


I wouldn't recommend using the "promise" off of .find(), because it's not actually a promise. If you were to put this in a Promise.all, for example, or return it from another promise's .then method, this wouldn't work the way you expect.


Instead, use .exec()


This line here was so hard to write.


ユーザー区分の明示化(nomal adminでそれぞれ何ができるかまとめ直す)


>Is this solution safe?
Not really. I guess that the proper solution is to ensure at the log level that flushes are serialized


This commit should be squashed with other commits


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.863.10119e90aae320b0309816f0f9115480a196a4e1](http://build.palaso.org/viewLog.html?buildId=60254&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 396, ignored: 3 Build time: 00:02:52


Use `next(err)` to send this kind of stuff to error handler route.


Same here, this does not have to be separate commit


Thanks; will fix. Note this isn't done yet, though -- I'll bug you to review it when it is if you like!


:sparkles:  @mrjaimisra


Or implement transaction support at the application level, and treat flush as a transaction


findById, also use `.exec`


ログイン・ログアウトには何が必要なのかの明示


Quite cool, the ZActor abstraction looks quite interesting!

+1


Except by the fact, I've copy/pasted, [but ....](https://github.com/search?q=AbilityUnitTargetFlag&type=Code&utf8=%E2%9C%93) T.T


Smart. Thanks. I sensed there must have been a better way to do that, but didn't know how.


This is the error handling route I referred to earlier.


New version coming up - I think it's safer and better.


It's okay. I never check my email anyway.


Missing case for quantifieta. Need  to call walk quantifiers


リスク(不安点)についての洗い出しが必要？


I don't really agree with using getKeyName actually.

`getAuthIdentifier` is defined on the contract, so we should be using that one.

http://laravel.com/api/5.0/Illuminate/Contracts/Auth/Authenticatable.html


That's a lot of spaces lol


Ok...I just did the same, md5sums all match (our repo, apkmirror, playstore). Weird. Back to mfonville's path.


This isn't true.


add `.run()`


no me gusta para nada ese sendPacket


This wouldn't be called if you didn't find the user, you would get `null` in the success function.


Is this fixme now false?


@Inkdpixels The `*` was the culprit^^


Please use loic.lacroix@savoirfairelinux.com


Random trailing newlines


Trying something...


This is what just happened.


Congrats :+1: ;-)


Messing with `_messages_arrived` here feels like spaghetti. Is it possible to just reset the semaphore instead of `acquire()`ing it one at a time?


What you want is to take the parameter error, so you know what went wrong.


Is this fix a cosmetic fix or is our current metadata.yaml in the repos broken?


@tchaikov , right the scope changed a little bit. Wei wrote the first version of the code trying to tolerate errors from a single shard with redundant reads (refer to https://github.com/ceph/ceph/pull/3446), however, that original patch was partial implemented and caused a range of issues, which motivated us to re-write with a new patch (refer to https://github.com/ceph/ceph/pull/5309), the new patch only tries to do fast read, no error tolerance.

For the feature to tolerate errors for reading, I think we'd better have a new patch, I can work on that when having time.

In short, the removal looks good to me :)


The problem with that is you may not be calling flush() on all the queues. I do agree it feels a bit messy.


I would have expected the munge boolean in the type and not the provider.


Any reason this wasn't in the base type?


Nice! You can also combine a lot of this stuff, like:
````
output = []
a.flatten.each_slice(3).map do |e|
  e.push(0) while e.size < 3
  output.push(e)
end
output
````


I'd unindent this so it doesn't look like it's inside the import


Should the state be updated once this is closed, i.e. set `socket = undefined`?


reformat (ctrl + alt + l)


The reviews exist on the game, why not just populate that?


yes. but currently the upgrader is not prepared to handle the situation.


@RevanProdigalKnight
Whoops, edited it, even though the point that this ban was a reaction to a report stays.

>Well, you wanted a realistic option - that is one :)

I just don't want to restart this argumentation since I already mentioned some arguments :stuck_out_tongue_winking_eye:

>Yes, but they did not come in through GitHub, as GitHub has no real discovery features. They came in through other channels. This is no different from a free forum host - should a free forum host disable a forum because somebody used a word on a member's forum that somebody else deemed offensive?

Forum hosts though lock forums if they violate their ToS. I'm not for moderation in general by GitHub, my point was that having no content policy at all isn't a good move from GitHub's perspective, even though freedom of speech is good for the user (as long as people don't take it too far).

>That means you should be very conservative with your risk mitigation strategies, and very carefully consider the tradeoffs. In this case, there's absolutely no indication that non-moderation of things like this repository would drive away business.

I also don't think that this specific repo would cause PR trouble, however still the original repo got reported and then banned and not banned randomly. This has potential to escalate in other cases, not really in this one though. I don't even think this ban was deserved.

Also, just because there is a risk doesn't mean you shouldn't try to reduce the possibility of the risk happening.

>The decision to lock the repository was still a subjective staff decision by GitHub, regardless of whom the report originated from. They could've just as easily said "sorry, this is outside of our scope of responsibility, go pick it up with the repository maintainer". They didn't.
They could've have done that and I think they should've since this repo isn't meant to insult people. My whole argumentation bases around the point that I think having no content policy at all isn't a good idea (from GitHub's perspective), even though I support free speech in general.


rename as "createFileAndAddData"


filemane to fileName


Existiert das Verzeichnis, welches du öffnen möchtest?


Maybe some required/min/max validation on this?


The reviews are on the game documents, but also referenced in the review documents.


I am very sorry!!
I really forgot to test the new code. This shouldn't happen. Sorry for the inconvenience!
Now everything should work. If I ever contribute again I will make sure everything is working.
Again, sorry.
In the future I will also try using rebase. (Not much time atm, just wanted to fix my mistake)


Seems like this would probably be a unique field that was required?


remove empty lines


Yup


either delete the if else block or take an action inside them


@cfare: Use a different class (test case) for different views


move this method to more appropriate place. maybe an utility class


this includes breaks fabtests builds for those of us who use infiniband-free systems.
Please make this conditional as part of some configury option.


Please change logon to something more appropriate


@cfare: Why 4? Also, we must create at least 1 queue that was created before last Monday to adequately test that those are not counted.


Not activated.


Hi Pawel,

I agree. But personally I think it's better to remove it from the html for the use cases when it is not used.


@abadams Oops. Yes that FIXME is now false. I'll force push a commit without this FIXME


@mscoutermarsh @ScottWatson19288 @CodeFlowed @robgraphix1 @hugolmendez
Not sure why but this was not showing up on my local copy. My mentor is stumped as to how when I pulled it down it wasn't showing the merge conflict. We did it on screenshare and it definitely wasn't updating my local copy when I pulled down the master. Long story short. I cloned chess to another directory fixed it and just merged it with master...no need to do a pull request. Just so everyone understands why I did this.


You could move this comment to the right-hand side in place of this right here.


@cfare: Note if the user name is not x['user'], the assertion is skipped! So if it is never x['user'], it will always skip assertion, and the test might pass without ever making an assertion!
Generally avoid more complex logic in tests, make it as declarative as possible, repetition is better than false logic.


Here's an update for the existing tests: https://github.com/django/django/pull/5069


@guegueng I think this is what you want. look good?


I don't think this needs to be tested.


@cfare: Again, no need to create a ton of places, create as many as needed and no more.


I'd add backticks around `.mmdb` so we don't need to add it to the words list


Is this the correct understanding -- one of the reasons we had `serializable` was we didn't want to use `other.store.entries()` here because of the concern of its cost, thus we could not get the key (unless we did the bytes->string conversion) on line 282. Now it's cheaper to traverse the entries with their keys easily available.


Are you testing the original master or your fork?

The command times out after 1,5 seconds:
```
if (!WaitForResponseTimeout(CMD_ACK,&c,1500)) {
		PrintAndLog("Status command failed. USB Speed Test timed out");
	}
```
I.e. the text `Waiting for a response from the proxmark...  Don't forget to cancel its operation first by pressing on the button` should never appear (this would be displayed after 2 seconds). I suggest that you check your implementation of `WaitForResponseTimeout()`

Here is what I get normally:
```
proxmark3> hw status
#db# Memory
#db#   BIGBUF_SIZE.............40000
#db#   BigBuf_hi  .............40000
#db# Tracing
#db#   tracing ................1
#db#   traceLen ...............0
#db# Fgpa
#db#   mode.............HF
#db# LF Sampling config:
#db#   [q] divisor:           95
#db#   [b] bps:               8
#db#   [d] decimation:        1
#db#   [a] averaging:         1
#db#   [t] trigger threshold: 0
#db# USB Speed:
#db#   Sending 512000 bytes payload...
#db#   Time elapsed: 1183ms, USB Transfer Speed PM3 -> Client = 432797 Bytes/s
#db# Various
#db#   MF_DBGLEVEL......2
#db#   ToSendMax........700057610
#db#   ToSendBit........0
proxmark3>
```
and if a pull the cord during testing:
```
proxmark3> hw status
#db# Memory
#db#   BIGBUF_SIZE.............40000
#db#   BigBuf_hi  .............40000
#db# Tracing
#db#   tracing ................1
#db#   traceLen ...............0
#db# Fgpa
#db#   mode.............HF
#db# LF Sampling config:
#db#   [q] divisor:           95
#db#   [b] bps:               8
#db#   [d] decimation:        1
#db#   [a] averaging:         1
#db#   [t] trigger threshold: 0
#db# USB Speed:
#db#   Sending 512000 bytes payload...
Status command failed. USB Speed Test timed out
proxmark3>
```


@cfare: Only use `self.` for objects accessible throughout the class.


@tdhock I found a bug, and just stuck it in here since it doesn't seem that important.


Clever ... does .toString() not work?


So we don't even need the Xapi_xenops bit because Xenopsd isn't going to look at this. The commit message should make clear that the VDIs would only get wrongly plugged _on newer versions_ without this patch.


This is not robust enough.


`memcpy()` would be better, because `ns` never overlaps with `ptr`.


ditto.


Nice simplification!


You also should check that it put something in the database, or at least called Game.create or whatever.


without the space, the yaml typing mechanism is not invoked. so buggy if not broken.


In Laravel 5, you don't need to do `URL::asset()`. `asset()` just works!.


Ok, there is "transaction support" --- all writes to log are processed via a working queue. I used the same queue for flushes as well: https://github.com/JetBrains/imperative-nodejs/commit/f283662e3e57e14b0dc8be9e91c003278b758477


`/logon` again?.. Please that name is not cool. Change to something more appropriate


ColoredLogPrinter just combined LogPrinter and ColorPrinter while adding no functionality^^


Using table tags for page layout is no longer a good practice. Use CSS for layout instead. Table tags can still be used for tabular data.


Ruby on the stemcell should not be used for anything but the old ruby agent. I don't remember why upgrade did now work but most likely it was some kind of version incompatibility.


The <center> tag is not supported in HTML5. Use CSS instead.


This statement does nothing in real hardware (and qemu), but it signals the Bochs debugger to break.


I don't know the complete history behind these "compatibility" methods (they seem to be from https://github.com/appliedsec/pygeoip/wiki), but maybe we could drop them. They don't seem to be part of the new Python API client.


This is pretty awesome, talk me through it.


Hey - do not look at the database or scheme please - that is the whole
point of the api :)

That VO is all you need.  You can see the types for each field based in
here and look up the codes in the appropriate "code" object.

Call me at 215-500-1233 if you can as I am mobile.

On Thursday, July 30, 2015, Cecilia Donnelly <notifications@github.com>
wrote:

> API: Save SSN, get closer to saving race
>
> SSN now saves correctly for new and edited clients.  Race does not, in
> part because I'm not sure what the API expects.  It looks like I should
> be sending boolean values for each race, but then the db tables I have
> only have room for one "Race Key."  Values for that race key are defined
> in the OpenHMIS tree, as are boolean values like "amIndAKNative."  Which
> should be stored?  If the latter, where do they show up in the db?  This
> is a question for @slifty, probably.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/OpenTechStrategies/openhmis-intake/commit/96c523e1783cbbf634bd12da918a058d720c819a
>

--
Sent from my The New iPad.

Sorry if my smugness causes you to wish you had one.


This isn't valid json. Does this even work?


```ul``` tags should only be used in conjunction with ```li``` tags as children


This doesn't work like this.


I ported shiny to MacOS to see how it worked, and when I add error checking after this line (i.e. err := cookie.Check(); fmt.Println("shm error:", err)) I get this:

shm error: BadBadSeg {NiceName: BadSeg, Sequence: 11, BadValue: 6291458, MinorOpcode: 3, MajorOpcode: 132}

Displaying an X11 window in MacOS works, but the window is all white.

FYI.


@tgummerer - thanks for investigating! 2) sounds like some sort of wonky behavior (or you need to add an explicit dep somewhere). This should "just work".
There are ideas floating around around making build and test rules separate and getting rid of the separate run/get steps methods, so let's go with the separate `_apk`/`_test` approach.


It fails as you are overwriting Path with the object name

I changed the code to:
u.Path = hostSplits[0] + u.Path

and it worked fine. But 'mc cat nutty.s3.amazonaws.com/passwd' failed as it does not handle redirect. We can implement redirection and that will make this work. But ever better thing to do would be not to change u.Host at all and handle it at url2BucketAndObject()

Also minio-go requires fix to handle <bucketname>.s3.amazonaws.com sending a fix for that as well. See if they are fine.


215-400-1233*

On Thursday, July 30, 2015, Dan Schultz <slifty@gmail.com> wrote:

> Hey - do not look at the database or scheme please - that is the whole
> point of the api :)
>
> That VO is all you need.  You can see the types for each field based in
> here and look up the codes in the appropriate "code" object.
>
> Call me at 215-500-1233 if you can as I am mobile.
>
> On Thursday, July 30, 2015, Cecilia Donnelly <notifications@github.com
> <javascript:_e(%7B%7D,'cvml','notifications@github.com');>> wrote:
>
>> API: Save SSN, get closer to saving race
>>
>> SSN now saves correctly for new and edited clients.  Race does not, in
>> part because I'm not sure what the API expects.  It looks like I should
>> be sending boolean values for each race, but then the db tables I have
>> only have room for one "Race Key."  Values for that race key are defined
>> in the OpenHMIS tree, as are boolean values like "amIndAKNative."  Which
>> should be stored?  If the latter, where do they show up in the db?  This
>> is a question for @slifty, probably.
>>
>> ---
>> Reply to this email directly or view it on GitHub:
>>
>> https://github.com/OpenTechStrategies/openhmis-intake/commit/96c523e1783cbbf634bd12da918a058d720c819a
>>
>
>
> --
> Sent from my The New iPad.
>
> Sorry if my smugness causes you to wish you had one.
>

--
Sent from my The New iPad.

Sorry if my smugness causes you to wish you had one.


Interesting comment on the decision not to include comments in json: https://plus.google.com/+DouglasCrockfordEsq/posts/RK8qyGVaGSr


I don't see any reason that this include is necessary.

@patrickmacarthur What was this include for?


I think it works


lets merge


Not really. I've end-up with solution like this:

```python
# Message plugin to handle xom+xml SOAP replies
from suds.plugin import MessagePlugin

class XOMPlugin(MessagePlugin):
    def received(self, context):
        answer = context.reply.decode()
        xml_message = re.search(
            r'(<soap\:Envelope.*<\/soap\:Envelope>)\r?\n', answer, re.M)
        if xml_message:
            reply = xml_message.group(1) + '\n'
            context.reply = reply.encode()

client = suds.client.Client(api_url, plugins=[XOMPlugin()])
```


Ups, das war tatsächlich nicht auf diesem Branch ^^' Allerdings macht das keinen Unterschied, immernoch der selbe Fehler.


@AllanJLuke SIZE_MAX is the C constant denoting maximum number which can be stored in size_t. So by definition it cannot be greater (see 7.18.3 of C spec).


TeamCity LabkeyTrunk :: CDS postgres 9.4 [Build 231](https://teamcity.labkey.org/viewLog.html?buildId=263630&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Snapshot dependency failed: LabkeyTrunk :: build Build time: 00:00:00


Sure, will do. I was actually about to leave it, but then I realized that while it would increase performance for some cases, it would increase memory usage for *all* cases, so in the end I ditched it because perf difference wasn't worth it.


great, thanks!

On Thu, Jul 30, 2015 at 12:23 PM, Kevin Ferris <notifications@github.com>
wrote:

> @tdhock <https://github.com/tdhock> I found a bug, and just stuck it in
> here since it doesn't seem that important.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/tdhock/animint/commit/b47079515dd417d533f7c218e54126534ebb1909#commitcomment-12448488>
> .
>


Then "uart tx queue full"


Why?


More or less - yes. It's worth it because we also omit some additional calculations, inlining alone isn't that profitable.


Not sure either what is going on here. But "cleaning" the file via Inkscape gets rid of the "clipPath" elements in the svg source, and specifically selecting the linearGradient results in this svg source, and then the correct (?) (dark grey) gradient colours are displayed in Inkscape and also in e.g. Nautilus:

![icon2](https://cloud.githubusercontent.com/assets/7164227/8988976/8148d7e8-36e9-11e5-87ca-07dc437382d1.png)

It seems that in the svg source, now a second linearGradient has been added, which links to the first one (?). I have checked adding a "y=.." value to the first linearGradient, but that did not have any effect.

@palob Do you happen to have an idea what is going on here?


That's what `proxyPort` is for.


In merge, within the only set of changes in this file in this commit (line 230).


Does this support full regular expressions, including .+, .* etc.?


Let's check the return string?


Yeah, I changed my mind in the meantime (see my other comment).


I will.


having an AdbHelper.get(context) sounds better (I think the ExecutionContext has all of the fields AdbHelper needs).
The fact that you have to new up the AdbOptions and TargetDeviceOptions is still somewhat of an issue as you're effectively hardcoding things the user might want to specify.
Might be simpler to add AdbOptions/TargetDeviceOptions to the context. I'm not super happy about plumbing anything through a `global stash of all the things` at all, but this is the easiest way..


Yes.


[E] Could not determine IPv4 address of 'eth0' interface.


I've merged it again,

you shouldn't worry ;), It's just a hobby project,
besides, it's my fault, I should have tested it before merging.

thanks again for the fix!
ps.
if you are interested in merging your "myCharger"  branch, let me know when you are ready
although your charger should differ from the other chargers (defaultCalibration.cpp not included).

Best Regards,
Paweł


Theoretically, if we retry continually, we could end up with an infinite loop.  I'm not sure why that is there, but we have that in a couple places, and we know that retrying like this is bad.  Presumably, we should just kill the addin at this point if it fails to load, and give a failure message.


Das war doch ./data/n mit n als Integer, richtig? Ist auch das richtige Unterverzeichnis für deinen Code vorhanden?


@schleichardt I forced it always `true` because before it was like this for any kind of attribute besides sets. Right?


I'm sorry, I guess it must be a leftover from when I was experimenting with finding a way to get data from the controller that somehow escaped my notice during cleanup! That shouldn't have happened


could put this way:
```c
char buf[len];
```

no need to use `alloca()`


Alternative: s/stop/kill/.

"stop" does a clean shutdown, taking ages (like >30 sec). If we don't want to use the stopped containers, kill is the way to go.


That's why I changed the parsing to load it as a file and pass it through https://github.com/sindresorhus/strip-json-comments.

Then I promptly forgot to add it to package.json so the build failed.


And yes, the name `MESOS_DOCKER_STOP_COMMAND` is bad, `MESOS_DOCKER_DOWN_COMMAND` would be better.


Can you move the run time properties (url, userid etc) to .properties file? The other changes look cool.


your a machine xP


I think we can drop this.  It's needed if we want to support AF_IB/sockaddr_ib.


Very nice, should be able to test and merge soon. Thanks!


I see the point, but I've seen the to many open files issue with Linux, too. Please don't remove it for now.


Cool! I guess you could go even further and check for a newline, comma, or semi-colon. I'm pretty sure I did that somewhere, for some reason..


Genau das hatte gefehlt und ist jetzt da.


Yes, this line was definitely NSFL ;) I just wanted to try to do it in a single line. And yes, parsing code refactor would help – feel free to edit or I'll look at it when I'm free.


this is a bit brute force, and IMO could lead to side-effects. might be OK for now, but i'd prefer to isolate loading the webview if possible. maybe as a later optimization.  isolating the webview might make it easier to reuse it by injecting it into different controllers (e.g. when loading webview for another VC on the stack)


adding new jbjohnson86 bio


Ok, this makes sense. I was trying this originally but was seeing what I thought were thread-related crashers. If I do again, I will post about it.


There are async retries. Also, you can try X amount of times, without
repeating boilerplate code with polly. It seems like 5 times might make
more sense than twice.

On Thu, Jul 30, 2015 at 12:41 PM Hosch250 <notifications@github.com> wrote:

> Theoretically, if we retry continually, we could end up with an infinite
> loop. I'm not sure why that is there, but we have that in a couple places,
> and we know that retrying like this is bad. Presumably, we should just kill
> the addin at this point if it fails to load, and give a failure message.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/rubberduck-vba/Rubberduck/commit/bd474fea2e728ff033c4213067d0c853e4af3b6b#commitcomment-12448825>
> .
>


@adinapoli do you think this is overkill or possibly an improvement?


Whirlwindjon , if you learn how to sniff, you will see it's not hard ...


Adds bio for mothur-westcott

#30


I don't think there's any tangible difference, at least in case of glibc.


I don't think this does the right thing.  Consider:

```
thockin@freakshow:/tmp$ rm -rf a z

thockin@freakshow:/tmp$ mkdir -p a/b/c/d

thockin@freakshow:/tmp$ touch a/b/c/d/file-in-a-b-c-d

thockin@freakshow:/tmp$ touch a/b/c/file-in-a-b-c

thockin@freakshow:/tmp$ mkdir -p z/b/c/d

thockin@freakshow:/tmp$ find a -type f
a/b/c/d/file-in-a-b-c-d
a/b/c/file-in-a-b-c

thockin@freakshow:/tmp$ find z -type f

thockin@freakshow:/tmp$ f=($(find a -type f))

thockin@freakshow:/tmp$ cp -f "${f[@]}" z

thockin@freakshow:/tmp$ find z -type f
z/file-in-a-b-c
z/file-in-a-b-c-d
```


I have a couple of problems with this interpretation of the Interface Segregation Principle:

1. Why redefine the Single Responsibility Principle for a specific feature of one or two specific languages? The SOLID principles, including the Interface Segregation Principle, apply to Ruby and Python as well as they do to Java and C#. If it's just "the Singe Responsibility Principle for `interface`s", why did Uncle Bob bother mentioning it in the first place? [Role Interfaces](http://martinfowler.com/bliki/RoleInterface.html) IMHO are more like "the Single Responsibility Principle for `interface`s".
2. Looking at the definition of the Interface Segregation Principle, we find [it states](https://en.wikipedia.org/wiki/Interface_segregation_principle) "no client should be forced to depend on methods it does not use". No mention of language-level features here at all. A good example from .NET and Java is the `Seek()` method on `Stream`: `FileStream` supports `Seek()`, as does `MemoryStream`, but `NetworkStream` does not. As a "fix" for this, clients of the abstract `Stream` class are required to call `CanSeek` before calling `Seek()` to make sure they can do so. Clients of `NetworkStream` are forced to depend on `Seek()` and `CanSeek` even though anything using a `NetworkStream` specifically knows that it won't be able to `Seek()` on it. (This is also a borderline Liskov Substitution Principle violation: you can't simply call `Stream.Seek()` and have it Just Work; the `if (CanSeek) { Seek(); }` hack keeps it just out of a violation of the Liskov Substitution Principle, because you can run that code on any `Stream` without it blowing up.)


Merging to current upstream state


@megatronir: why can't I see you fix anything if it's so easy?


Looks good!


A `Future` is a stand-in for a value that you'll get at some point in the future. The earlier versions of this project were a future, by doing memoisation by default (so if you ran a future twice, you'd get the cached value).

A `Task` is a stand-in for a computation that may, or may not produce a value when it executing.

So they capture different concepts. `Task` was more general than `Future`, and at some point I removed the memoisation, so the project ceased to be a `Future`, so I decided to just kept working towards making `Task` more useful.


All right, thanks, I need to remember to check the documentation after reading something on stackoverflow!
Will change this for #7 as well.


Ohh here it is: https://web.archive.org/web/20141028080946/http://forum.sa-mp.com/showthread.php?p=3065341


@esteban-uo It's hard for me to understand you, but https://github.com/facebook/react/issues/2912#issuecomment-112215955 may answer your question.


90946736_SHRgt1_4567 (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)


As of 2015-07-30 kalium 0.4.0 is not available as a Maven resource.  The latest version found via search.maven.org is 0.3.0.


Vielleicht muss ich das doch nochmal ausführen. :)


`подключа**е**тся`


This should be `/bin/sh` probably.


Shouldn't it be below with one 'l' and Jumbo is the best option of the two instead of better ?


won't this catch files that may have snuck in but are not part of this commit?  Maybe not a huge deal, but we could pass "${files[@]}" to the script and have it use the commandline if present, else find files itself


Yeah… we need to have a tech discussion on how we will handle the web views… but that will be a separate ticket - out of scope here.

Note: Even when we isolate, this is the only way I know of to force view to be loaded from the nib without adding it to the view hierarchy. Is there another way you know of?


> +StudySwiftForPlayGroundで行う
これ誤字ってますね。
StudySwiftFromPlayGroundです。


Yeah, I'm still working on chunk entities. We won't have any commits for a long time.

> On Jul 29, 2015, at 6:21 PM, Peter Zerangue <notifications@github.com> wrote:
>
> Wow, 22 days ago. That's a long time between commits.
>
> —
> Reply to this email directly or view it on GitHub.
>


Yes. Note that not all libraries can be fused into one big universal library. Since some projects generate different header files for different architectures.


@sgress454 I fully agree with you. By "this breaks apps" I meant that with the current code there is no way to access the primary key value in the beforeUpdate hook, for example. Doing `values[Model.primaryKey] = pk;` solves this need and also prevents the modification of primary key. Should I create a pull request?


Was this made with some tool or manually deciding?


Good call, I didn't check Annex B... follow-up commit incoming.


Since the project is on python, I strongly recommend you to follow pep8 coding syntax because that is the standard python coding syntax followed.

For easiness, you can configure your editor with plugins for automatic detection of pep8 violations. Some best plugins for the same are available for pycharm, Atom and even vim.


This is a bad idea as the ``post.too_short_content`` translation message uses the ``{{ limit }}`` replacement (it misses proper pluralization btw), and ``NotBlankValidator`` does not provide such parameter


The document isn't that complete as I see it. And I was wondering how could I generate a Throttling error or whatever.
If I requested sequentially, I got all ok; or if I requested parallelly, I got all `{socket_error,timeout}` or `{socket_error, {ehostdown, ...}}`. @nalundgaard do you have any better idea?


@Vincent-Michael went through each file using Resharper for C++ which warns about unused headers.


We [just had a problem assuming Unix path separators](https://github.com/dlmanning/gulp-sass/pull/314). Please provide a solution that will work cross-platform.


How do you configure it? Well… you don't, I'm sorry :D Please wait one day or two because we don't have a solution yet, and if you setup something it will be broke at the next update.


I need this file back


You're right that centering shouldn't happen on every tick - that was silly of me. As I understand it, though, the force doesn't take into account the size of the label when assigning positions - it just has single points that repel each other. (If you know a way to get the force to take label length into account, that would be great - I couldn't think of a good one.) So it doesn't really matter whether they're centered as far as whether or not they overlap.  As far as which looks better, though, I'm happy to take it out and leave it to the PAs to ask for it.


Now I see it is redundant, but I also notices SetType does not allow `isRequired` http://dev.sphere.io/http-api-projects-productTypes.html#set-type
So I will replace the `searchable` condition for the `required` condition.


I confess I didn't notice this change when a previous pull request was merged, and the change of behavior was not intended. Originally, the user could opt to exit themselves, or try reconnecting, or something else entirely if they wanted as you've said.

Feel free to submit a PR if you like.


@dcadams: I think we _do_ want this; adding it to a `private.py` file is
_not_ what we want.

devstack doesn't/shouldn't come with SSL configured, so https links will
always, incorrectly, fail. The `HTTPS` Django setting is used here to
construct URLs with the correct transport, along the lines of:

```python
transport = 'http'
if settings.HTTPS == 'on':
    transport = 'https'
url = "{transport}://example.com/path/to/resource".format(
    transport=transport,
)
```

In fact, this setting is already properly set to `off` inside of
`dev.py`. It appears to have been overlooked in `devstack.py`, where the
functionality should be identical (no SSL configured, so transport must
be http).

Unless I'm overlooking something, this change should even be pushed
upstream.


When joining paths together, use `path.join`


We need to check that exit code is 0.


Actually, it is already false by default. But shouldn't we disable it in the `isRequired` method?
What about nested types?


No need to check that directory already exists before creation.


@robotlolita Thanks for that explanation! Really helpful.


I think I agree with this in principle; the OCP is the hardest of the SOLID principles to grok, it's got the worst name *ever*, and an even worse definition. I feel the "all methods must be virtual" is a strawman, though: it's like the kickback from a cannon when it's fired.

I like to think of the OCP in terms of things like the `Strategy` pattern, and I find it much more useful when viewed in that way. The OCP for me is better described in terms of the question "how can I modify the behaviour of this class without changing its code?". Taken in conjunction with the other SOLID principles, and TDD (or at least unit testing), I get to the point where I've written and tested a class and I know it works. I need to add some new behaviour to that class, but I don't want to risk breaking my nicely-completed and well-tested class by modifying it directly. The OCP is all about that: extending the small, well-tested building blocks you already have with new small, well-tested building blocks. We actually found saw this in action in the last code kata series: we defined an `IRule` interface for snaps that were evaluated against the state of the game, and we could change the game quite considerably by defining and providing different implementations of `IRule`.

I really like the idea of using the OCP as a measure of architectural quality. I've not tried it myself, but it seems to me self-evidently useful.


If we isolate, the webview won't be part of the nib's view hierarchy. Without getting into details now: we have a `WMFWebViewManager` class that:

- preloads webviews
- manages swapping webviews w/ a snapshot inside a container view
- injecting webviews into container view (potentially w/ loading UI)

I also have some thoughts around link handling etc. that could be abstracted away so as to be agnostic to whether link tap comes from native link or web link. We can chat about it on hangout today.


save this file


It this an autotools limitation? Have you try it?


Acho que voce fez besteira


Need to fix a test to adjust for this, I think - https://travis-ci.org/mozilla/fxa-oauth-server/jobs/73410821


be consistent. Do the same everywhere. You don't really want to do the
conditional expression everywhere. So use None. Please.

2015-07-30 18:01 GMT+02:00 Makman2 <notifications@github.com>:

> Not sure, I like the readable default from ConsolePrinter where you can
> see which systems are supported.
> I maybe can also add the None case to the ConsolePrinter itself together
> with the default^^
> Yeah that's a good idea, I'll do that :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/coala-analyzer/coala/commit/0df1de8f967d15599ab8a62e30085fd8cb24ccca#commitcomment-12447946>
> .
>


`lipo` won't create the `lib` dir, if it doesn't exist. Is there any guarantee that it would be created before we run `lipo`?


It looks like we typically use a variable for the special characters that should not be used.  Please see https://github.com/jonathanmccann/liferay-portal/commit/14450e6bb9d78d46a454b8e303ce3e51445530cf#diff-ea207ab10073d125e8f80af5b6342300R5321 as an example.

Can we change this language key to match the one below?


you call every callable member! That isn't necessarily a property!

2015-07-30 18:03 GMT+02:00 Makman2 <notifications@github.com>:

> okay I'll call it obj
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/coala-analyzer/coala/commit/359446741b3bf2eb0e82448774b3a830e4f6bfc7#commitcomment-12447983>
> .
>


The key interface is currently in flux, mostly because HSM integration is tricky because of algorithm support. The interface might change in the future.


I though leaving them would make the upgrade easier if anyone would use them. The methods were documented but I have no idea how often people use them, so I erred on the side of caution.

 If you'd rather just remove them, I'm +1 on having one and only one way to do things.


Maybe "center"? the "normalizer" does things per sample.


and hey, "it's your fault" is still an issue and thats not nice. Please be
nice to your users.

2015-07-30 19:03 GMT+02:00 Lasse Schuirmann <lasse.schuirmann@gmail.com>:

> you call every callable member! That isn't necessarily a property!
>
> 2015-07-30 18:03 GMT+02:00 Makman2 <notifications@github.com>:
>
>> okay I'll call it obj
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/coala-analyzer/coala/commit/359446741b3bf2eb0e82448774b3a830e4f6bfc7#commitcomment-12447983>
>> .
>>
>
>


Prefer import play.api.libs.concurrent.Execution.Implicits._
See: https://www.playframework.com/documentation/2.4.x/ThreadPools


Поправил :+1:


Please see my comments in https://github.com/jonathanmccann/liferay-portal/commit/14450e6bb9d78d46a454b8e303ce3e51445530cf


This should fit on one line.


Doh.  I factored things out into a function and forgot the "else".  Thanks!


still do, step the fuck up!


Center?


Don't know if we can use this, in this case (since Play might not have been bootstrapped yet)

Prefer import play.api.libs.concurrent.Execution.Implicits._
See: https://www.playframework.com/documentation/2.4.x/ThreadPools


@guano creating branches!! Go Taylor!


> ColoredLogPrinter just combined LogPrinter and ColorPrinter while adding no functionality^^

Nope. ColoredLogPrinter added the color to the `[WARNING]` and the other log levels.


If that's the case, should I make 2 PR's?  One on the Stanford fork
reactivating Azim's tool, another on the upstream fork to fix devstack,
whic we can cherry-pick later?

On Thu, Jul 30, 2015 at 9:58 AM, Steven Burch <notifications@github.com>
wrote:

> @dcadams <https://github.com/dcadams>: I think we *do* want this; adding
> it to a private.py file is
> *not* what we want.
>
> devstack doesn't/shouldn't come with SSL configured, so https links will
> always, incorrectly, fail. The HTTPS Django setting is used here to
> construct URLs with the correct transport, along the lines of:
>
> transport = 'http'if settings.HTTPS == 'on':
>     transport = 'https'
> url = "{transport}://example.com/path/to/resource".format(
>     transport=transport,
> )
>
> In fact, this setting is already properly set to off inside of
> dev.py. It appears to have been overlooked in devstack.py, where the
> functionality should be identical (no SSL configured, so transport must
> be http).
>
> Unless I'm overlooking something, this change should even be pushed
> upstream.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Stanford-Online/edx-platform/commit/f488f90df43c38a7b21956e2cff66471cdac63ae#commitcomment-12449280>
> .
>


I am using the icon theme MATE-Faenza which looks like in the screenshot below.
How often do you think the average user makes use of this button?
Because in my opinion the main toolbar should only include the most important buttons.

![toolbar](https://cloud.githubusercontent.com/assets/1301412/8989378/1dde38e4-36ec-11e5-8431-ebe4b914d6b8.png)


... and replace
 by


Ahh, I see.  Good call.  Yes--a PR would be great.  Leave in the warning, though.  Thanks!


You should avoid using the system random number generator here.  We want PCG for this.


@sgress454 But sometimes, we will ignore `80` proxyPort,  so `port` will fetch `sails.config.port`.
And I do not like `80` in the url.


@neappx

Why do they need a content policy? The only reason they are doing is they are caving to SJWs
I think that any valid code should be allowed to remain, regardless of offensive language.


I'm not exactly sure what will happen, I've never tried it and the chance of it happening is very slim. Worst case scenario is that the server panics and quits. Since I wrote this for just personal use (me and possibly a few other people, less than 5 people) I wasn't concerned about client ID conflicts as they would be so rare. But if it's an issue for you, you could always setup the server to assign client IDs, which takes a bit more code to write.


Everything but reading list at this point. If we switch to SQLite (the framework) we can force it to use our sqlcipher as well.


Thanks @larissasimplyhired !


@frantisheq See https://github.com/CyanogenMod/android_kernel_sony_msm8974/blob/cm-12.1/sound/soc/codecs/tfa98xx.c#L1863
We pushed the stock HAL and the eq files, and then made this print the data.

Also what @BuzzBumbleBee said, not sure if I still have the original code / patch around, but the basic idea is to have a printk there with the data.


Ahh so isolate == create web view programmatically?


Not needed.


Let's remove them and add a note to the release notes to assist with the upgrade.


If we can actually fetch this from a DB, we might do so once and cache the result for a given JSON. This way, we always have the correct result, but do not have to fetch it every time.


Cool, so this basically means that if used as a decorator, the `use_cassette` decorator must be applied before `gen.coroutine`.


Yeah, I can't think of many valid use cases where you'd want to patch only one coroutine. When you do have multiple coroutines running, they're still part of the same test context. For example,

    with vcr.use_cassette(..):
        responses = yield [AsyncHTTPClient().fetch(...) for x in xrange(10]

You want all those responses to be recorded to the same cassette. For the case where you want responses recorded to separate cassettes, you'd want until you exited the context for that cassette.

If a valid use case comes up, it should be possible to limit the scope of patching to a single coroutine by using a StackContext (it will automatically patch and unpatch as Tornado switches between coroutines).


For some reason, GitHub isn't seeing introscreen.py, if some of the commits seem strange it's because I'm trying to fix this.


How are you testing in parallel? still all from the same erlang node? If so, it might be better to write an escript and run lots of separate Erlang processes. Another option might be to play with the lhttpc connection pool size, but I am not sure how easy that is within erlcloud.


tested and working


`80` will never be in the URL you get from `getBaseurl`.  That's what this line does--ignores values of `80` and `443`.  If you set `proxyPort` to `80`, it won't show up in the URL.


Isn't the fact that `R.initFrozen` returs the FrozenGen causes `SysR.random` work with PCG's RandomGen implementaion?


I'm also finding it not doing what I expect when I use 'update'. This is a bit strange given what update is supposed to do. So locally, I've put the writes back in. I'd quite like to understand why this is actually necessary, since surely 'update' ought to work...

I'm beginning to suspect the problem is with Idris' handling of ibc files, rather than idris-vim. Something very strange is definitely going on...


#Currently saving codecademy Python notes in Evernote, and MVA Python notes in Visual Studio 2015.


I would still keep the `open` classmethod. Seems very convenient and it's
used in the code example in the doc.

On Thu, Jul 30, 2015 at 12:08 PM, Tim Graham <notifications@github.com>
wrote:

> Let's remove them and add a note to the release notes to assist with the
> upgrade.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/fcurella/django/commit/f276c25c1b31ded8435412478b2ca761f87b0d80#commitcomment-12449521>
> .
>


If session is private shouldn't this return const if at all?


@luis-pereira
This commit as well as c02fc4b don't compile any longer, apparently due to some faulty interaction with lxqt-globalkeys:
```
[ 95%] Building CXX object CMakeFiles/lxqt-runner.dir/configuredialog/configuredialog.cpp.o
In file included from /usr/include/lxqt-globalkeys-ui/LXQtGlobalKeysUi/ShortcutSelector:1:0,
                 from /usr/local/peter/lxqt-runner-git/src/build/ui_configuredialog.h:12,
                 from /usr/local/peter/lxqt-runner-git/src/lxqt-runner/configuredialog/configuredialog.cpp:30:
/usr/include/lxqt-globalkeys-ui/shortcutselector.h:43:38: error: expected initializer before ':' token
 class SHARED_EXPORT ShortcutSelector : public QToolButton
                                      ^
/usr/include/lxqt-globalkeys-ui/shortcutselector.h:45:20: error: expected declaration before end of line
CMakeFiles/lxqt-runner.dir/build.make:382: recipe for target 'CMakeFiles/lxqt-runner.dir/configuredialog/configuredialog.cpp.o' failed
make[2]: *** [CMakeFiles/lxqt-runner.dir/configuredialog/configuredialog.cpp.o] Error 1
CMakeFiles/Makefile2:60: recipe for target 'CMakeFiles/lxqt-runner.dir/all' failed
make[1]: *** [CMakeFiles/lxqt-runner.dir/all] Error 2
Makefile:136: recipe for target 'all' failed
make: *** [all] Error 2
```


Yeah, I think that might be the best way. Might experiment with this in my native article content patch adding links


Oh, I think I just thought it was a strange to let outsiders pass in a date here. i.e. either you're storing a Favicon and Storage can set the stored date for you, or you're retrieving them and storage also can get it. So the internal _date lets the Storage framework change the date without exposing its setter to outside code. We'd probably be better with

internal var _date = NSDate.now()
public var date: NSDate { return self._date }

or an internal init() method that takes a date parameter and public one that just sets the date to NSDate.now()


yes


I find this name a little unwieldy..how about something like CustomSSLTrustContextCreator?


@megatronir you are the guy that doesn't share any work with tc no? very poor.


This is  a great name.


Single quotes, also `assert short_uuid == '3cvd8mz9'


I don't call every callable member, the `generate_repr` supports also getter like methods. If you supply a function that does heavy calculations I can't do anything against it. We are all adults in python :P


Line length?


Thanks, fixed in 1f04873517e41d2f54fc4e65c46c08ba85b23010.


This makes the assumption that the URLs used by our client modules should be hard-coded into the client lib at deploy time. I would argue they should not be. I would generally argue that hard-coded URLs should not be inserted into the clients at all, but rather set from configuration at run time (from client configs - as they are in our current CLI system). Setting hard-coded URLs is begging for trouble.

We already have to set user tokens from configuration at run time. Since this is the case, why not set URLs the same way?

If we decide not to set URLs (or any other dynamic configurable variable) at deploy time, then this argument for running type compiler at deploy time no longer applies.

On Jul 30, 2015, at 8:37 AM, olsonanl <notifications@github.com> wrote:

> I disagree; the deployment process in general requires invoking the type compiler in order that the proper default URLs be embedded into the client modules so that clean installs work properly (particularly in the standalone releases). It should be invoked at deployment time.
>
> �
> Reply to this email directly or view it on GitHub.
>


It's not unused...?


Wake up :)


what about some choices?


Ok je savais pas et je l'ai vu dans le JS, alors je l'ai remis dans mon commit d'après.


Роман, добрый день!

«Напишите консольное приложение, эмулирующее взаимодействие покупателя с торговым автоматом с печеньками.

Автомат предлагает меню: кексы по 50р 4 шт., печенье по 10р 3 шт. и вафли по 30р 10шт.

Автомат принимает монеты номиналом 1, 2, 5, 10 рублей.

Покупатель имеет кошелёк, в нём 150р монетами данных номиналов.

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

Автомат выдаёт все не потраченные покупателем деньги, если у него запросили сдачу.

Автомат выдаёт сдачу минимальным количеством монет.

Обратите внимание на обработку ошибок, таких как:

   + у автомата нет монет на сдачу,

   + у автомата нет выбранного пункта меню

   + внесённой суммы не хватает для покупки

   + у человека кончились деньги в кошельке при попытке внести деньги в автомат

Технические требования:

Язык программирования: C#

Решение должно быть оформлено в виде репозитория на github

Критерии оценки:

Выполнение условий задачи

Следование ООП

Читаемость кода

»


what about stuff with stuff?


Actually I have not tried, it is just that before iOS 8.x, only static linking was allowed


TeamCity LabkeyTrunk :: CDS postgres 9.4 [Build 232](https://teamcity.labkey.org/viewLog.html?buildId=263640&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Snapshot dependency failed: LabkeyTrunk :: build Build time: 00:00:00


I found my problem. I don't set the `proxyPort` to `80` and it's just empty, so it will get `sails.config.port` in the URL.


is lcr2 now the official tool? if so, we can indeed do that


please rename it to castling_allowed


Thank you! Good catch.


@sgress454 thanks  :)


If you pulled the latest prima driver from victara then it is not
(necessarily) the same as the one plus one.  I am tracking caf's prima repo
(LA.BF.1.1.2_rb*, or something like that).
On Jul 30, 2015 12:57 PM, "RenderBroken" <notifications@github.com> wrote:

> WLAN: Import Prima driver from OnePlus One
>
> Pulled from CM - Victara
>
> Thanks @crpalmer
>
> Signed-off-by: Zachariah <zkennedy87@gmail.com>
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/RenderBroken/msm8974_Victara-Stock_render_kernel/commit/f9198db41a50e2c917ffc306870a2813b21cc49c
>


Just curious - what's the reasoning behind using a temporary table instead of doing it in memory?


Oh, I'm sorry, I looked wrong. Two packages use it.


What language to you love


consider using itoa to convert int to string


looks good!


You can even just use `Assert.assertEquals(trimmed, expectedTrimmed)` now :)


Any reason this wasn't made a constructor option?


Do you set _setupAdded to be true anywhere in this codepath? Do you need to? Seems to only be set if the reflector is rewritten.


It's like doing /ignore on all players essentially. They aren't notified that the messages aren't sent like with tptoggle.


rather than a bunch of separate steps here, nesting should be able to achieve the same effect while still preserving the order expectations:
```
t.click('>>nx-coreui-security-anonymous-settings checkbox[name=enabled]',  function() {
    t.click('save', function(){
        t.waitForEvent(..., next)
    }
}

```


I think the main reason I tend to think of ISP as being a poor man's SRP is because of the [Clean Coders video](https://cleancoders.com/episode/clean-code-episode-12/show) I first came across it in. The main example was the Xerox problem from the "Origin" section on the wikipedia article with the problematic `Job` class. (Amusingly, the real problem wasn't the design being bad, but the fact that this being C++ meant that linking took forever with a big god class in the middle).

Anyway, the main point of ISP (at the time) *seemed* to be "if you can't have SRP in a class because it's a stupidly big god class and you don't have time to split it up, at least you can have SRP on the interfaces that access it, which is nearly as good." It's probably more nuanced than I give it credit - I always assumed Role Interfaces were how you attained ISP :)

I think there's a similar argument for ISP as "no client should be forced to depend on methods it does not use," though. If a class or interface has some methods that some clients won't use, that smells like an SRP violation. In your example you could define "streaming from a stream" and "seeking through a stream" as two separate responsibilities, I think. (What is a client supposed to do when `CanSeek` returns false, anyway? I bet most just end up throwing `NotSupportedException` ;)


At home hobby?


I'm testing them without the `libpng = libpng15` overrides now. Slooowly.


:(  I am so sad.


tested and working


add this back with link to wh blog


Work on syntax.  Noob is invalid when used with "@cjlems".  Fail.


Nice)


Katrina Essex commited this?

Me thinks you forgot to set your email in the git config :stuck_out_tongue_winking_eye:


Yeah, I considered that, but I prefer not having the extra prototype method that doesn't work as expected. `handleEvent` is really a data property that happens to be function-valued IMO.


Keep coding!!


Got it!


Да там конечно setFlowersDao


wait this is sort of strange what about just calling lookup once?


```
var proj = projCodes.lookup(sr.latestWkid || sr.wkid || options.wkid)
```


Can you create this token in the test so it's clear where it came from?


When in DataProbe I call VolumeNode.GetReferenceSpace(...)
GetParentTransformNode return always a NULL. Is this a wrong usage of it?


because it might be missing from the lib


So am I. Neverball has an update.


LGTM


That's the reason I didn't remove `serializable` earlier. If you look at Metadata's introduction in e4bd1c7 you can see it was using `entrySet()` originally.

I think the reason was more that Louis needed/wanted the Metadata.Key, which we don't have if constructed from the transport. With the Key, we are able to avoid constructing the key's byte[], and I think there was the perception that it made getParsed()/getSerialized() in MetadataEntry simpler.


if latestWkid is not null, but is not in the library then we're going to end up with nil


No I mean that code:
```cmake
if(NOT EXISTS "${dir}")
  file(MAKE_DIRECTORY "${dir}")
endif()
```

is equivalent to
```cmake
file(MAKE_DIRECTORY "${dir}")
```


Moreover, the only way to obtain a Float via PCG is to use `data Gen` [1] which in turn may be constructed from `FrozenGen` only by allocating new `MutableByteArray` [2]. We don't really want to do that each time we ask for next random number.
[1] https://github.com/cchalmers/pcg-random/blob/pure/src/System/Random/PCG/Fast/Pure.hs#L84
[2] https://github.com/cchalmers/pcg-random/blob/pure/src/System/Random/PCG/Fast/Pure.hs#L183


tested and working


This is factually incorrect.


I think not every window should have Audacious included in its title,
like a message dialog or the equalizer. Other opinions?


Nope:

```
share/base_image.c: In function 'image_load_png':
share/base_image.c:97:9: warning: implicit declaration of function 'png_sizeof' [-Wimplicit-function-declaration]
         if (!(bytep = png_malloc(readp, h * png_sizeof(png_bytep))))
         ^
share/base_image.c:97:56: error: expected expression before 'png_bytep'
         if (!(bytep = png_malloc(readp, h * png_sizeof(png_bytep))))
                                                        ^
Makefile:298: recipe for target 'share/base_image.o' failed
make: *** [share/base_image.o] Error 1
builder for ‘/nix/store/pscrxrgalnds3q587il3i604s1gfrszi-neverball-1.5.4.drv’ failed with exit code 2
```

Revert?


Perhaps. There is a more serious issue here for which I am preparing a patch.


Default URLs to the production services absolutely should be in the client code. Otherwise a bare client install requires user intervention to get started. We can have user-config overrides in the way you do it with dot files, but those should not be required to operate.

You only have to set user tokens for services requiring authentication.

On Jul 30, 2015, at 12:13 PM, cshenry <notifications@github.com<mailto:notifications@github.com>> wrote:

This makes the assumption that the URLs used by our client modules should be hard-coded into the client lib at deploy time. I would argue they should not be. I would generally argue that hard-coded URLs should not be inserted into the clients at all, but rather set from configuration at run time (from client configs - as they are in our current CLI system). Setting hard-coded URLs is begging for trouble.

We already have to set user tokens from configuration at run time. Since this is the case, why not set URLs the same way?

If we decide not to set URLs (or any other dynamic configurable variable) at deploy time, then this argument for running type compiler at deploy time no longer applies.

On Jul 30, 2015, at 8:37 AM, olsonanl <notifications@github.com<mailto:notifications@github.com>> wrote:

> I disagree; the deployment process in general requires invoking the type compiler in order that the proper default URLs be embedded into the client modules so that clean installs work properly (particularly in the standalone releases). It should be invoked at deployment time.
>
> �
> Reply to this email directly or view it on GitHub.
>

—
Reply to this email directly or view it on GitHub<https://github.com/cshenry/Workspace/commit/22f4d466b0b212e3e1a0ba71f84e98563a6e0a99#commitcomment-12449624>.


good to go!


:shipit:


Would this be cleaner?
```dart
if (_setupAdded) {
  buf.write('bootstrapStatic${node.argumentList}');
  return;
}
var args = node.argumentList.arguments;
buf.write('bootstrapStatic('${args[0], ${args.length == 2 ? args[1] : null}, () { ${_getStaticReflectorInitBlock()} })');
```


I like it more when identicon is hidden until user starts typing. Having it originally hidden requires to put it under account name input - this way when identicon appears it doesn't affect user's input.


Thanks for a reminder. I am trying to get the script that will do automatic brewing done (I have some problems with `libpng` on my mac) and after that I will also update `README.brew`


I'm using a module from the css-modules people. https://github.com/css-modules/css-selector-tokenizer I didn't look into it in details, but they seem smart and it might be handy for other use-cases.


@megatronir do this! JUST DO THIS! and see how you screwed up


Only if the function decorator needs to have gen.coroutine applied to itself. That seems like a strange test construction to me, but I suppose that you are correct.


When in DataProbe I call VolumeNode.GetReferenceSpace(...) (where VolumeNode is a ScalarVolumeNode)
GetParentTransformNode return always a NULL. Is this a wrong usage of it?


yup okay. But still it'd be cool to call once. I think a cleaner way to do it would be change the esri-proj-code lookup to accept an "SR" object and be smarter. Then koop can be smarter about using the `options.wkid` in that object.


nit' consider putting on one line


Yeah, looks really legit. I already added it to [get-css-classes](https://github.com/johnotander/get-css-classes/commit/1f30afe274b942f9872d33490aca4715735fabc7), too.


Isn't it going to halt as soon as one returns true-ish?


apparently brilcalc (http://cern.ch/cmslumisw/brilwsdoc.html) is supposed to replace lcr2.py


yup it'll work fine, im okay with it


"Semantic versioning (semver) is applied as much as possible."
well, this change is a big BC break added in patch version 1.2.3


this should also be removed


The main reason why a started looking at this is why the planned statusicon-qt plugin (based on QSystemTrayIcon) draws its name starting with a small letter 'audacious'. The best way I could find to fix this is to set application display name.

On the other hand, I see no problem with the title being suffixed with application name. Well, probably except for tiny windows, where title is drawn with ellipsis in the middle (WM problem).


Can you submit a pull request for this one? It seems pretty useful! Would address this issue https://github.com/cboscolo/elb2loggly/issues/7


You allocate buffer in virtual memory (in memory of virtual ps3). Why?


@smeezekitty
With Content Policy I mean the classic ToS stuff you find on almost any site like G7 & 8 here on GitHub:

>7. We may, but have no obligation to, remove Content and Accounts containing Content that we determine in our sole discretion are unlawful, offensive, threatening, libelous, defamatory, pornographic, obscene or otherwise objectionable or violates any party's intellectual property or these Terms of Service.

>8.Verbal, physical, written or other abuse (including threats of abuse or retribution) of any GitHub customer, employee, member, or officer will result in immediate account termination.

It's fine as it is I guess, they just should only enforce it if it REALLY is necessary (from both used & business perspective). I also would like that GitHub also provides access to the source (at least to the owner if they decide to ban a user.


Forgot to comment.
Add
- some alt actions (show the commandline with the command)

Update
- guideletebuttons (they are centered now)
- descriptions


And try to free it with *free(..)*? How?


I think accidentally hitting enter in intelliJ on things that weren't what I was trying to insert also automatically inserted the import statements to go along with them.


Its best to just ignore the people who dont appreciate what you do


Sorry, I had forgotten the tests :(


This will make it so that you can't catch exceptions in your tests.

    try:
        response = yield get(get_client(), 'http://httpbin.org/status/500')
    except HTTPError as e:
        assert e.code == 500

Will always raise because the exception never goes back to the context where it's caught.

Fix in #180.


We are using version 3.3, which was added through b74c05b:

https://github.com/glyptodon/guacamole-client/blob/b0e7caf8452df5097717e3e13a3413b5e49abf6c/extensions/guacamole-auth-ldap/pom.xml#L23


What's wrong with 4.3?


90946736_SHRgt1_4567 (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (745 of 745 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-7.11%25-green.svg)


> @neappx I also would like that GitHub also provides access to the source (at least to the owner if they decide to ban a user.

:+1: to that.


Yep. The order didn't change; that'd be a separate issue.


@maethor: I sent an email to @Simage to explain ...


Ok...GitHub had a major problem and while trying to restore my files it went too far back and reverted two days of work. This is me trying to restore it.


Sure, but `SysR.random` is at least as bad (probably worse).  Then fix then is to make line 64 work on integer math: `min h $ 1 + truncate (log (gen v i) / logHalf)`.


Thanks, I really need to change the commit message then. Thanks bro! Still
I appreciate the help. I had been asked to work on a Stock kernel and the
biggest complaint was that there was not any Wifi.

Kind Regards,
Zachariah Kennedy

On Thu, Jul 30, 2015 at 12:17 PM, Christopher R. Palmer <
notifications@github.com> wrote:

> If you pulled the latest prima driver from victara then it is not
> (necessarily) the same as the one plus one. I am tracking caf's prima repo
> (LA.BF.1.1.2_rb*, or something like that).
> On Jul 30, 2015 12:57 PM, "RenderBroken" <notifications@github.com> wrote:
>
> > WLAN: Import Prima driver from OnePlus One
> >
> > Pulled from CM - Victara
> >
> > Thanks @crpalmer
> >
> > Signed-off-by: Zachariah <zkennedy87@gmail.com>
> >
> > ---
> > Reply to this email directly or view it on GitHub:
> >
> >
> https://github.com/RenderBroken/msm8974_Victara-Stock_render_kernel/commit/f9198db41a50e2c917ffc306870a2813b21cc49c
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/RenderBroken/msm8974_Victara-Stock_render_kernel/commit/f9198db41a50e2c917ffc306870a2813b21cc49c#commitcomment-12449677>
> .
>


good point.


Removed all error messages instead of having constant messages with no variables


done.


I think we will want to use `===` in these cases.


Might as well change `inEvent` to `ev` while you're at it. :smile:


@neappx

Fair enough. But Github shouldn't have to bend to every complaint or report. Losing the whiners wouldn't be losing that much business and would make Github a less hostile environment overall.


added more explanation.


А как это работает?

Или имелось в виду `BEM =BEM || { helpers: {} };`?


looks good


It uses an empty mock.


TeamCity LabkeyTrunk :: CDS postgres 9.4 [Build 233](https://teamcity.labkey.org/viewLog.html?buildId=263644&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 4, passed: 0 Build time: 00:09:45
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSVisualizationTest: <no details avaliable>

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

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

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

```


Before I said the issue was that you cannot have two pages with the same title in the navigation. This has been tested to be incorrect. The determining factor is the filename of the page. You cannot have two files with the same name. The moment you do it will always take the last parent element with the same page name.


@tombatossals Seems like ```controlType``` will never === 'custom' because of the previous ```if``` statement.  As a result, "custom" controls no long work.


have you already setup clang build under linux?
if so, can give a short description in the readme (maybe new readme for osx, if steps for osx / linux are the same)
i tried to use clang under qtcreator / linux but ended up with tons of build problems...


test comment


This layout doesn't look correct. It doesn't correspond to the TransitionBlock struct (see callingconvention.h@73). The Transition block should overlap the stack area right above the float registers. So from the TransitionBlock structure, I would expect the CalleeSavedRegisters above the float registers and the ArgumentRegisters above the CalleeSavedRegisters. Also, the order of the registers should be reverse. So I think the layout should look like this:
```c
+// Stack layout:
+//
// (stack parameters)
// ...
// fp
// lr
// CalleeSavedRegisters::x28
// CalleeSavedRegisters::x27
// CalleeSavedRegisters::x26
// CalleeSavedRegisters::x25
// CalleeSavedRegisters::x24
// CalleeSavedRegisters::x23
// CalleeSavedRegisters::x22
// CalleeSavedRegisters::x21
// CalleeSavedRegisters::x20
// CalleeSavedRegisters::x19
// CalleeSavedRegisters::x30 (Lr)
// CalleeSavedRegisters::x29 (Fp)
// ArgumentRegisters::x7
// ArgumentRegisters::x6
// ArgumentRegisters::x5
// ArgumentRegisters::x4
// ArgumentRegisters::x3
// ArgumentRegisters::x2
// ArgumentRegisters::x1
// ArgumentRegisters::x0
// FloatRegisters::d7
// FloatRegisters::d6
// FloatRegisters::d5
// FloatRegisters::d4
// FloatRegisters::d3
// FloatRegisters::d2
// FloatRegisters::d1
// FloatRegisters::d0
```


tested and working


Docs Introdution


Add Introduction


I used `spawn/1` to test that, but I forgot to init the config for the newly spawned process.
I get all `ok` now even I make 500+ request at the same time again and again, and I also crash my local network router once. I gave it up! I'll use `ThrottlingException` as the error type like `ddb_impl` did for now, since the ddb [common errors](http://docs.aws.amazon.com/amazondynamodb/latest/APIReference/CommonErrors.html) is `Throttling` too.


> u.Path = "/" + hostSplits[0] + u.Path

You are right, the hostSplits[0] is the bucket name that we want and the actual path would look like this.


Correction-- it doesn't need the mock. There is no request to the comments service because Commentables don't actually live there.


Might I suggest a conditional load based on an option to change the default output from `json` to `xml`? The majority of users of `node.js` don't use `xml`.


Well what should you replace this with?!


@Rikk You are right!
https://github.com/notepad-plus-plus/notepad-plus-plus/commit/efbea233d22b0e49fd2424db6bc95c5bf55edaff


Good point that force doesn't take into account the sizes of the labels. This was definitely a misconception I had when I made my comment.

For simulating the label length with the force layout, one idea would be to associate multiple points to each label in the force layout, and fix the positions of these points relative to each other, kind of like this for each label:

.          .          .          .          .          .

.          .          .          .          .          .

.          .          .          .          .          .

There could also be other ideas. However, I think that we should definitely wait for some PA feedback before fine-tuning the labeling too much. I wouldn't feel compelled to tackle this problem unless the PAs express dissatisfaction with the labeling.


Seems broken here. May be ``` *port ``` fix this.


This really should be a conditional such as `report` with a type of `xml` or `json`.


I only put it here b/c I am using it in 2 different tests. Will copy to both tests if you prefer.