Is it possible you're hitting race conditions in your purchase complete handlers? In my handlers for instance instead of doing any behavior, I simply set data like....
```model.purchaseComplete = true```
And then in my update loop at a time I expect I go
```if(model.purchaseComplete)
var stuff = iap.getstuff();
someMethod(stuff);
```

Here's last 3 months of Pakka Pets android beta errors. No JNI exceptions.

StackFingerprint,Crashes,% of Crashes,
IllegalStateException,36.0,7.627118644067797%,
IllegalStateException,23.0,4.872881355932203%,
IllegalStateException,21.0,4.4491525423728815%,
NullPointerException,19.0,4.02542372881356%,
IllegalStateException,16.0,3.389830508474576%,
IllegalStateException,15.0,3.1779661016949152%,
IllegalStateException,12.0,2.542372881355932%,
IllegalStateException,12.0,2.542372881355932%,
NullPointerException,11.0,2.330508474576271%,
NullPointerException,9.0,1.9067796610169492%,
NullPointerException,8.0,1.694915254237288%,
IllegalStateException,8.0,1.694915254237288%,
NullPointerException,7.0,1.4830508474576272%,
NullPointerException,7.0,1.4830508474576272%,
IllegalStateException,6.0,1.271186440677966%,
IllegalStateException,6.0,1.271186440677966%,
IllegalStateException,6.0,1.271186440677966%,
NullPointerException,6.0,1.271186440677966%,
NullPointerException,6.0,1.271186440677966%,
SQLiteException,6.0,1.271186440677966%,
IllegalStateException,6.0,1.271186440677966%,
NullPointerException,5.0,1.0593220338983051%,
NullPointerException,5.0,1.0593220338983051%,
NullPointerException,5.0,1.0593220338983051%,
IllegalStateException,5.0,1.0593220338983051%,
NullPointerException,5.0,1.0593220338983051%,
NullPointerException,4.0,0.847457627118644%,
NullPointerException,4.0,0.847457627118644%,
NullPointerException,4.0,0.847457627118644%,
NullPointerException,4.0,0.847457627118644%,
NullPointerException,4.0,0.847457627118644%,
NullPointerException,4.0,0.847457627118644%,
NullPointerException,4.0,0.847457627118644%,
IllegalStateException,4.0,0.847457627118644%,
IllegalStateException,4.0,0.847457627118644%,
NullPointerException,4.0,0.847457627118644%,
NullPointerException,3.0,0.635593220338983%,
NullPointerException,3.0,0.635593220338983%,
IllegalStateException,3.0,0.635593220338983%,
NullPointerException,3.0,0.635593220338983%,
IllegalStateException,3.0,0.635593220338983%,
NullPointerException,3.0,0.635593220338983%,
NullPointerException,3.0,0.635593220338983%,
NullPointerException,3.0,0.635593220338983%,
NullPointerException,3.0,0.635593220338983%,
IllegalStateException,3.0,0.635593220338983%,
RuntimeException,3.0,0.635593220338983%,
IllegalStateException,3.0,0.635593220338983%,
NullPointerException,2.0,0.423728813559322%,
NullPointerException,2.0,0.423728813559322%,
IllegalStateException,2.0,0.423728813559322%,
NullPointerException,2.0,0.423728813559322%,
NullPointerException,2.0,0.423728813559322%,
IllegalStateException,2.0,0.423728813559322%,
IllegalStateException,2.0,0.423728813559322%,
RuntimeException,2.0,0.423728813559322%,
NullPointerException,2.0,0.423728813559322%,
IllegalStateException,2.0,0.423728813559322%,
RuntimeException,2.0,0.423728813559322%,
NullPointerException,2.0,0.423728813559322%,
IllegalStateException,2.0,0.423728813559322%,
NullPointerException,2.0,0.423728813559322%,
IllegalStateException,2.0,0.423728813559322%,
NullPointerException,2.0,0.423728813559322%,
IllegalStateException,2.0,0.423728813559322%,
IllegalStateException,2.0,0.423728813559322%,
IllegalStateException,2.0,0.423728813559322%,
IllegalStateException,2.0,0.423728813559322%,
IllegalStateException,2.0,0.423728813559322%,
NullPointerException,2.0,0.423728813559322%,
IllegalStateException,2.0,0.423728813559322%,
RuntimeException,2.0,0.423728813559322%,
IllegalStateException,2.0,0.423728813559322%,
IllegalStateException,2.0,0.423728813559322%,
RuntimeException,2.0,0.423728813559322%,
IllegalStateException,2.0,0.423728813559322%,
IllegalStateException,2.0,0.423728813559322%,
IllegalStateException,1.0,0.211864406779661%,
IllegalStateException,1.0,0.211864406779661%,
NullPointerException,1.0,0.211864406779661%,
IllegalStateException,1.0,0.211864406779661%,
IllegalStateException,1.0,0.211864406779661%,
IllegalStateException,1.0,0.211864406779661%,
IllegalStateException,1.0,0.211864406779661%,
IllegalStateException,1.0,0.211864406779661%,
Resources$NotFoundException,1.0,0.211864406779661%,
NullPointerException,1.0,0.211864406779661%,
NullPointerException,1.0,0.211864406779661%,
IllegalStateException,1.0,0.211864406779661%,
Resources$NotFoundException,1.0,0.211864406779661%,
IllegalStateException,1.0,0.211864406779661%,
NullPointerException,1.0,0.211864406779661%,
IllegalStateException,1.0,0.211864406779661%,
IllegalStateException,1.0,0.211864406779661%,
IllegalStateException,1.0,0.211864406779661%,
IllegalStateException,1.0,0.211864406779661%,
SQLiteException,1.0,0.211864406779661%,
NullPointerException,1.0,0.211864406779661%,
IllegalStateException,1.0,0.211864406779661%,
IllegalStateException,1.0,0.211864406779661%,
RuntimeException,1.0,0.211864406779661%,
IllegalStateException,1.0,0.211864406779661%,
IllegalStateException,1.0,0.211864406779661%,
IllegalStateException,1.0,0.211864406779661%,
IllegalStateException,1.0,0.211864406779661%,
IllegalStateException,1.0,0.211864406779661%,
IllegalStateException,1.0,0.211864406779661%,
IllegalStateException,1.0,0.211864406779661%,
NullPointerException,1.0,0.211864406779661%,
NullPointerException,1.0,0.211864406779661%,
RuntimeException,1.0,0.211864406779661%,
RuntimeException,1.0,0.211864406779661%,
RuntimeException,1.0,0.211864406779661%,
RuntimeException,1.0,0.211864406779661%,
NullPointerException,1.0,0.211864406779661%,
NullPointerException,1.0,0.211864406779661%,
RuntimeException,1.0,0.211864406779661%,
Resources$NotFoundException,1.0,0.211864406779661%,
NullPointerException,1.0,0.211864406779661%,
NullPointerException,1.0,0.211864406779661%,
IllegalStateException,1.0,0.211864406779661%,
IllegalStateException,1.0,0.211864406779661%,
IllegalStateException,1.0,0.211864406779661%,
IllegalStateException,1.0,0.211864406779661%,
IllegalStateException,1.0,0.211864406779661%,
NullPointerException,1.0,0.211864406779661%,
RuntimeException,1.0,0.211864406779661%,
NullPointerException,1.0,0.211864406779661%,
RuntimeException,1.0,0.211864406779661%,
NullPointerException,1.0,0.211864406779661%,
NullPointerException,1.0,0.211864406779661%,
Resources$NotFoundException,1.0,0.211864406779661%,
RuntimeException,1.0,0.211864406779661%,
NullPointerException,1.0,0.211864406779661%,
RuntimeException,1.0,0.211864406779661%,
IllegalStateException,1.0,0.211864406779661%,
NullPointerException,1.0,0.211864406779661%,
IllegalStateException,1.0,0.211864406779661%,
IllegalStateException,1.0,0.211864406779661%,
NullPointerException,1.0,0.211864406779661%,
NullPointerException,1.0,0.211864406779661%,
Resources$NotFoundException,1.0,0.211864406779661%,
RuntimeException,1.0,0.211864406779661%,
IllegalStateException,1.0,0.211864406779661%,


Would "symbolic" be more descriptive/accurate than "modification-freindly"?


Zoo part looks good.


newline


Fixed in 73881ea


aerodrome add looks good


Thanks very much.


On 08/11/2015 03:20 PM, kpj wrote:
> I am wondering whether I should make the 'ltx:..' stuff colored as well.

What do you mean by 'ltx:...' ?


test


@borshop: retry


quedó así de un ejemplo en el que me basé, pero lo modifico


Should `ltx:XMath:` and `ltx:XMArg:` also be colored, or only the corresponding numbers (`0/1`, `6/6`)?


TeamCity Chorus :: chorus-precise64-master Continuous [Build 1.5.426.9a0f7fca55890850de40caf8150c849b4e5258e6](http://build.palaso.org/viewLog.html?buildId=60931&buildTypeId=bt323) outcome was **SUCCESS**
Summary: Tests passed: 859, ignored: 54 Build time: 00:32:11


fyi, I'm not convinced that every model should need a cost (I might simply have a list of constraints), but I don't want this comment to distract us from 0.3.0, and I'm not going to raise an issue unless I find this awkward in practice.


beautiful


On 08/11/2015 03:28 PM, kpj wrote:
> Should |ltx:XMath:| and |ltx:XMArg:| also be colored, or only the
> corresponding numbers (|0/1|, |6/6|)?

Ah, didn't notice the difference initially...
probably just the numbers, I think

> —
> Reply to this email directly or view it on GitHub
> <https://github.com/brucemiller/LaTeXML/commit/eb9bda4e259cdce57c237ddfe6eeb6f50921aace#commitcomment-12648846>.
>


Hi, I think I am having a very similar issue to the memory leak you fixed in this commit. Would you mind letting me know what you did? MapView.mContext is currently leaking my activity.


use/user


ps I like this for now, simple.


Is is supposed to work immediately after deployment? I deployed to staging, but it's not showing any recent revisions. This course, for example, has recent revisions: http://dashboard-testing.wikiedu.org/courses/Michigan_State_University/Growing_Up_Girl-Coming_of_Age_in_Women%27s_Literature_and_Film_%28Summer_2015%29/activity

But it says zero recent revisions: http://dashboard-testing.wikiedu.org/courses?cohort=summer_2015


en realidad no debería ser un driver, me arrepentí luego de hacer el commit, va a estar modificado para el siguiente commit.


Very minor, but does this specifically need to be "is not None" rather than just an "if fw:"?  If there's the chance that something weird that compares False coming back from _get_shared_framework but isn't None, then maybe this should be an isinstance check against against the base-level framework class instead?


Hey @nick22891 try not to do the `.split('T')`.  Use the `moments` library that is already a part of the application to parse the ISO-8601 date.  If you ignore the timezone parts the application will function incorrectly.


So I could write some burley collection logic at some point. I'm sure it could be safely removed


Hmm!


TeamCity Chorus :: chorus-win32-master Continuous [Build 2.5.6.9a0f7fca55890850de40caf8150c849b4e5258e6](http://build.palaso.org/viewLog.html?buildId=60930&buildTypeId=bt2) outcome was **SUCCESS**
Summary: Tests passed: 871, ignored: 50 Build time: 00:36:48


Getting an error on update...
```
=====================================
Running master/update.sh...
2015-08-11T20:26:19 Start master...
2015-08-11T20:26:19 Create search database...
off
2015-08-11T20:26:23 Create master database...
2015-08-11T20:26:27 Create selection database...
Selection database contents:
  interesting_tags count ......... |1268
  frequent_tags count ............ |2508
  interesting_relation_types count |11
2015-08-11T20:26:28 Update history database...
2015-08-11T20:26:28 Done master.
=====================================
2015-08-11T20:26:28 Running bzip2 on all databases...
Done.
=====================================
2015-08-11T20:26:39 Creating extra indexes...
off
Error: near line 17: no such table: main.tags

```


Why do this? remember the schema for report has `de_demand` already in it.
```javascript
var ReportSchema = new Schema({
    re_report_date: {type: Date, default: Date.now()},
    de_demand: {type: Schema.Types.ObjectId, required: true, ref: 'Demand'},
    co_commodities: [CommoditySchema],
    us_user: {type: Schema.Types.ObjectId, required: false, ref: 'User'},
    re_report_name: {type:String, required: true}
});
```


It's a lot simpler if you just concatenate a string...


No longer relevant; got changed back


nice catch! I had a feeling something was still of with aura, but didn't find a way to reproduce.


It's a spelling mistake...


TeamCity LabkeyTrunk :: CDS postgres 9.4 [Build 299](https://teamcity.labkey.org/viewLog.html?buildId=265695&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 28, passed: 2 Build time: 00:08:41
### Failed tests
```
org.labkey.test.tests.CDSMeasureStoreTest.testDemoMeasureStore: <no details avaliable>

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

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

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

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

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

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

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

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

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

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

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


Table hasn't been converted properly.


saw approval from cmeiklejohn
at https://github.com/basho/riak_core/commit/3edf54ade42de4884aba94578adb5b91e5c9cdf3


merging basho/riak_core/bugfix/jrd/join-race = 3edf54ad into borshop-integration-764-bugfix/jrd/join-race


basho/riak_core/bugfix/jrd/join-race = 3edf54ad merged ok, testing candidate = 74f52c59


Table not converted properly.


RUDE


I was thinking that code might need to reopen if the socket dies. Not clear though how that could happen to a send-only UDP socket...


Are you sure that this is safer then a signal?


Same as https://github.com/InfusOnWoW/WeakAuras2/commit/f8b6aa511a882fd6adc549e7a28f20e9675c4fa6


Why did you remove the console.error(err)? until better error handling is in place (displaying it in the ui) this is needed logging...


If this fix is required, then, IMO, it should be done in a separate jira.


What do you think of these generalized templates methods ?
There are a little complicated because all Items must be stored as std::unique_ptr<EventEditorItem> (with EventEditorItem the base class of all XXXXItems classes). So, the functions returning the items need to do a dynamic_cast to T


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (748 of 748 tests passed)

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


Oh ok. Grant was having an issue where memory limit was only being
exhausted in php.dev even though ini_get('memory_limit') was showing '1G'.

Thomas Slade
Tel. 843 324 8568

On Tue, Aug 11, 2015 at 3:31 PM, badoughatcher <notifications@github.com>
wrote:

> So I could write some burley collection logic at some point. I'm sure it
> could be safely removed
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/twslade/Magento-PHP-Console/commit/c60a719a0b2f9cae21781464f33bf08963e643f6#commitcomment-12648900>
> .
>


I did some performance-benchmarks to compare call_user_func_array with a direct call **plus the switch**. I made it 5.3 compatible and did the standard deviation computation in the code, so it can be run on any host (not dependant on PECL stats). I ran it on some cheap shared hosts with Joomla sites (on php 5.3 and 5.4). But I had to adjust the number of iterations to avoid timeouts. Still you can get some indications from it.

I also compared performance with 0 to 5 arguments. I first used a closure for that, like they do in https://gist.github.com/nikic/6390366 , but to be sure that is not biasing the test I wrote it as an object with multiple methods with a varying number of aguments.

I used http://3v4l.org to test on several php-versions. Also there I had to limit the number of iterations to avoid timeouts on their servers.

Here are the results on different php-versions for 0 - 5 arguments:
* with 10x 10000 iterations http://3v4l.org/1Iphd#v568
* with 50x 1000 iterations http://3v4l.org/ArpWM
Move your mouse over the results to see more.

Some tentative conclusions from that:
* on php 5.3 - php 5.6 the switch + direct call is about 2.5 -> 1.2 times faster than call_user_func_array for 0 -> 5 arguments respectively.
* on php7 the difference might even be a bit smaller, and neglectable when the method has more than 2 arguments.
* on hhvm the switch + direct call is about 30 - 50% slower (!) when the function has arguments.

I see 10000 (ten thousand) calls in 1 Joomla run as much, but maybe not totally unrealistic (100 legacy plugins, that are each called 100 times), but I have no example and can not (yet) think of one run with 100000 (hundred thousand) calls. This heavy user number of calls in one run is important to decide whether the gain is worth the trouble. **When I look at the output for 10000 iterations on php5.6.12 in http://3v4l.org/1Iphd#v5612 I see a gain of about 3.5 milliseconds.** On some cheap hosts with php 5.4 I got bigger differences (about 10 milliseconds for those 10000 iterations), but I'll have to look at that more precise.

To compare the absolute time on these virtual servers, here is @nikosdion's test (direct call without a switch), but then with 10000 (ten thousand) iterations via 3v4l.org: http://3v4l.org/26nLr

From php 5.6 onwards we can use array unpacking: that is about twice as fast as those direct calls plus switch. So, in case Joomla 4 would have php 5.6 as minimum (which we don't expect), then we could use that: $func(...$args); in line 37 in https://gist.github.com/nikic/6390366 (with 1 million iterations, done once), which I also ran on 3v4l.org with 100000 (hundred thousand) iterations for comparison. You also see, that on hhvm the call_user_func_array is about as fast as this new array-unpack.


test


I like that you've added a class for RandomValueGenerators, it seems like a useful thing to have around.

I think this should behave differently though. While what we're testing here is a great candidate for random data to exercise possibly unknown cases, we still need the test to be reproducible. The way to do that is to build this class entirely based on a seed value that gets passed in. That way we can produce exactly the same series of values given the same seed. The test should generate that seed value using some input like `currentTimeMillis` and print it out. That way we can just set the seed and get exactly the same test case.


I presume this is wrong - the buffer starts premultiplied (because the renderer is in premultiplied mode) and is therefore always premultiplied. I recognize that this call likely will do nothing if the buffer is already marked as premultiplied but let's please not depend on that hidden behavior when it is only protecting from misusage.


tests


This is in the wrong spot?


Typo. ^_^


Superscripts have been lost here and in others below I think.


No problem. I know that spare time problem. Two of the PRs are cosmetical nature anyway and the other two are one problem and I have solved it locally anyway. So I'm not in hurry and I would be glad if you can take yourself the time to test the patch intensive as I – to be honest – didn't had the time myself to test the other usecases.


Btw, the diff makes it hard to see where this is happening, so disregard if this is actually correct.


Hey, thanks =)


Table hasn't been properly converted.


If there is a failure like that, the lib should handle it transparently. The client of this code has very little info about when it should re-open. Besides, it could simply create a new instance if needed.


I think this was more of a design decision, as the original behaviour is nothing if not cheesily buggy.

Did you encounter some legitimate use that doesn't work in HEAD?


My thinking was that the user can set the coroutine stack to arbitrary size. Therefore, there's no guarantee that the signal handler will fit in. Thread may pose other dangers though. If you have an opinion on the matter, please tell me.


Hmm, I don't believe it is a race condition issue. Our purchase code is very segmented, with each piece requiring the appropriate IAP event to be triggered before going to the next one. Based off the exception and stack traces, the error points to the internals of `IAP.purchase()` so I think this makes sense with the errors we're seeing.


The `<Visible>False</Visible` is so those items don't appear inside of VS


If you have reproducible value generation, then you don't need to have generators that keep values around. To validate that the data was correctly written and read, you can just reset the random value generator to the original seed and test each value as you read them.

You're also keeping values around to generate the expected statistics, which I'll address in a separate comment.


RadianceRefactor (rpg777) - Win64-Windows-7-VisualStudio-12: OK (2191 of 2215 tests passed)

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


awesome!!


Thanks for explaining this.


The desired functionality is to not stick single entries in a list. How about:

```python
    for name, value in lines:
            value = value.split(", ")
            if len(value) == 1:
               settings[name] = value[0]
           else:
              settings[name] = value
```


sd


The `::` indicates that a code block follows, which isn't the case. A list must be separated from surrounding content. I think this is what you mean:

```
        @Steps:

        1. search all relevant logfiles for ERROR/FATAL
```


adsfasdfasd Justin


Ça devrait être beau. L'appel à "igoCoucheContexteSave()" sans $this a été réglé dans un autre commit...


some tests failed:
failure: http://buildbot.bos1/builders/test-riak_core/builds/1682


Rather than calling into the same object that calculates stats to get the expected stats objects, I think you should be testing the stats directly. That is, in your test you should read a page's stats and use those to validate the values in that page. So you would read a the min and max, then check each value by: generating the next from that column's generator and validating that the value read is equal to the expected value, then checking that min <= value <= max. That way you're just testing the meaning of the stats and not sharing code with what you are trying to test.


noooooo


Awesome work Raquel! I did have some trouble running it, and it's only because you forgot two closing `)`'s at the end of your `if` statements right before the `{` in the `getWinner` function :anguished: . Just a demonstration on how annoying code can be sometimes... Keep up the great work, you're doing awesome!


List not printed correctly .


the screen reader user will aware that the dialog has opened and can navigate to


Haha, sure!


Thanks!


This should be correct as-is, can you give counter-examples?


:+1: Thanks.


merging


I think by using @Rule, these are not needed.


I added a `Signomial(cost)` parse so that you can declare your cost as `1`


oh, duh. I get it. I think the original was good. disregard.


You should be able to call `folder.newFile("name")`. See http://junit.org/apidocs/org/junit/rules/TemporaryFolder.html


Indentation above is off.


These should be test cases rather than a randomly-generated property. Both should be tested each time.


The refresh token option is not working under the current settings as it is not an allowed origin


ok nice


Way to take credit for my work, bro :P


Should be fixed now.


I'm not sure I see much value in the IOC way of doing this, but I'm fine including it. Could you keep this in the test class until we decide to use it elsewhere?


ok the SIMD intrinsics is wrong, it was to compile and do not get complain from the compiler. here you will call the intel gather from intel avx2


Forgot to edit this.


The page size should be much smaller than the block size, but this looks like it is the other way around. Default page size is 1MB and the default block size is 128MB.


Some work had been done already by @choules on this - a separate set of AdobeReaderUpdates recipes, which I recently updated and have been using with the latest update.

Just pushed them to the recipes repo here: https://github.com/autopkg/recipes/commit/f4bd153a7d1bc819fd29cc34b40ef3311656a29c


@semperos Why put the Selenium deps in :dev?
If user is expected to include them himself, then this should at least be documented in the README.
Currently, 0.7.1 gives:
```
#<CompilerException java.lang.ClassNotFoundException: org.openqa.selenium.interactions.Actions, compiling:(clj_webdriver/driver.clj:1:1)>
```


I think `c` is undefined, this broke CI


This always creates a new String and a new buffer. For both BINARY and FIXED, I think we should be reusing a single `Binary` object when writing (probably by copying the String into it) to test the case where binary objects are corrupted when reused by the caller. That was the problem in PARQUET-251. On the read side, we should not reuse the buffer to avoid any issues in the test code.


passing a `char *` into a function that takes a `const std::string &` feels questionable to me.  looks like it could be prone to error


Doesn't matter since path.join takes care of it, but I can go change these to just be the filename


Yup will do, some days I wish there was extra hours and others not so much lol.  I have a new local branch which I should probably commit soon, which is a cleaned up version of what I have now but with a bunch of unit tests in it which will help with testing some of these things and preventing errors from popping up again.


Why the complex expression here and absolute values?


Any reason why the Executor would not return `false` immediately instead of misusing a `catch` to define program flow?


Naw, c is defined in `for exp, c`. Works on my computer... :-/


YES FINALLY <3


Should you shut down the app domain when `newFolder == null`? Conceptually I think that a `null` resource folder would mean you don't want any resources loaded right now.


Why not use the BigInteger random constructor? https://docs.oracle.com/javase/7/docs/api/java/math/BigInteger.html#BigInteger(int,%20java.util.Random)


Delete README


TeamCity Ignite :: Ignite Cache Failover Multi JVM [Build 113](http://204.14.53.152/viewLog.html?buildId=59966&buildTypeId=Ignite_IgniteCacheFailoverMultiJvm&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:03:34


TeamCity Ignite :: Ignite Cache Full API Multi JVM [Build 137](http://204.14.53.152/viewLog.html?buildId=59968&buildTypeId=Ignite_IgniteCacheFullApiMultiJvm&guest=1) is now running


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


you need to use nodestime on base and new. test deleted.


On my original fork, I might change this to be configurable.

```
var notificationHeight: CGFloat = 60
```


Yeah, saw it.  Pretty cool


Probably forgot to add to the previous commit.


I am gonna try to find for you some linaro gerrit adding to some modules makefile the ffast-math so you can see


[8/9/2015 12:14:29 PM] emoose: well the fmod changelog at http://www.fmod.org/files/revision_4.44.txt mentioned fixing audio pops in the latest update, i just diffed that update with the previous one and tried the changes out with HO
[8/9/2015 12:14:41 PM] emoose: seemed like that was the only change that was needed, just hope it doesn't break anything else
[8/9/2015 12:14:58 PM] emoose: also the fmod version HO uses is from like 2010
[8/9/2015 12:15:12 PM] emoose: which is weird since none of the 360 halos used fmod iirc
[8/9/2015 12:15:27 PM] emoose: so I guess they've been working on a h3 pc port for a while


reset the buffer


GRACIAS !!! thanks !!!


I've seen tons of graphical glitches with -Ofast.

I'm thinking getting source-codes with exact match with your descriptions
would be very hard.
On Aug 11, 2015 10:59 AM, "Maxime Poulain" <notifications@github.com> wrote:

> From ARM doc:
>
> "NEON technology can accelerate multimedia and signal processing
> algorithms such as video encode/decode, 2D/3D graphics, gaming, audio and
> speech processing, image processing, telephony, and sound synthesis by at
> least 3x the performance of ARMv5 and at least 2x the performance of ARMv6
> SIMD. "
>
> But to force floating computation for floating number to be done on the
> SIMD you need funsafe-math-optimizations which is include in ffast-math.
> But again this may result in not exact result, but for algorithm such
> shadow or graphical tings, you DO NOT need exact result ( see my last link
> about the inverse square)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/temasek/android_build/commit/2b32ac8627030749f47dd228d1bc584d84885a1c#commitcomment-12629780>
> .
>


don't write the buffer everytimes as per the new definition.


Cool idea! I see you're a 4-space guy.  I'm a 2.  I wonder what most people are doing in Swift right now.

I'm reluctant to do 4 :( 2 feels so much cleaner.

But anyway, I love the idea of userInfo on the notification. I just did a basic old String tag (which now feels weak in comparison). Your solution gives so much more.  I may add this to the original fork!

Side thought: I wonder if an extension somewhere that adds userInfo to any NSObject (or nil) would be cool.


But again, if you look at my link, ffast-math is realy dangerous, you need to apply it TO modules you know that the result doesn't matter at all


I should also note that this requires JuliaLang/julia#12551 to not die in type inference


Hm Sorry if I made myself a bit confused, I was not talking about applying Ofast to the whole build, but mainly applying ffast-math (that enables floating operations to be done on the NEON coprecessor of our arm chips.

did you look at my wikipedia link?


I redid a bunch of this stuff this afternoon.  You can revert if you want, but I'm going to end up overwriting, just so you know.


SHDetailViewController doesn't appear to be receiving the character object to populate the view. If this is something you got stuck with I'm happy to work through it during/after class tomorrow.


This is meant to be like a comment... "first I'm looking up the spec, then creating the actual validator object".


It does not. You dont know anybs too.


You know, I can't get it to work at all on my iPad, but it works on my iPhone just fine. I'm not sure why. Maybe I can move on, and we can take a look on Wednesday?


Nice property naming here. Meaningful and gives enough clues as to how they should be used.


Not sure what's bad about this, this is kind of complex and the important part is the `$.map` I think


I'm 100% for no braces in the form `if condition \n return 'foo'`, 95% for when it's `if condition \n someSimpleExpression()`, but 100% against when what is going on is complex enough to warrant more than one line.


Sounds good! We could really benefit from adding more mappings and verifying controller support. Chrome should support the standard mapping for PS4 controllers (though chrome's gamepad support is kind of broken on OS X ... I'm working on that.) Firefox will definitely need a custom mapping for it, though. https://github.com/melonjs/melonJS/issues/34#issuecomment-129052258 describes how mapping are made.


애뮬레이터 처음에 실행하면 오류가 뜨며 정지가 됩니다.;;;


Do move on. And we can explore it Wednesday.

Sent from Outlook

On Mon, Aug 10, 2015 at 7:12 PM -0700, "Levi Cole" <notifications@github.com> wrote:

You know, I can't get it to work at all on my iPad, but it works on my iPhone just fine. I'm not sure why. Maybe I can move on, and we can take a look on Wednesday?

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


For me, this creates an error whenever a mdDialog/mdMenu/mdSelect is shown on Safari 8.0.5. The browser then goes into an infinite loop. Here's the error from the developer console:

[Error] TypeError: undefined is not an object (evaluating 'str.split')
	parseMaxTime (angular-material.js, line 4664)
	computeTimings (angular-material.js, line 4639)
	(anonymous function) (angular-material.js, line 4542)
	(anonymous function) (angular-material.js, line 4694)
	c (angular.min.js, line 116)


typo ^


测试添加输出


修改测试


测试


commit


Nice!


This line isn't necessary, the static CharacterManager instance used for the singleton is actually declared in the sharedCharacterManager method.


Let's verify that the token isn't canceled until after the delay.


"hack" and "for good" don't seem to belong together :D


comment on time unit for both of these


TeamCity Ignite :: Ignite Queries [Build 1667](http://204.14.53.152/viewLog.html?buildId=59957&buildTypeId=Ignite_IgniteQueries&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 709, muted: 11 Build time: 01:00:00


TeamCity Ignite :: Ignite Data Strucutures [Build 1789](http://204.14.53.152/viewLog.html?buildId=59969&buildTypeId=Ignite_IgniteDataStrucutures&guest=1) is now running


TeamCity Ignite :: Ignite Cache Failover2 [Build 125](http://204.14.53.152/viewLog.html?buildId=59967&buildTypeId=Ignite_IgniteCacheFailover2&guest=1) outcome was **FAILURE**
Summary: Tests passed: 53; exit code 1 (new) Build time: 00:26:34


TeamCity Ignite :: Ignite Cache Restarts [Build 1813](http://204.14.53.152/viewLog.html?buildId=59970&buildTypeId=Ignite_IgniteDataGridRestarts&guest=1) is now running


typo?


Looks like we lost most of the providers when doing 'Show All'. e.g. `Microsoft.ClassicCompute` is gone, as out many others. They still show up correctly on staging site. Do you think this is related? It's the only recent change.

/cc @shobak101


@obiot Are you sure this is right? If the tile is flipped, doesn't that break things?


Comment RE running on the main thread already or not


Disregard. I see it now. The mask was moved into the function. :D


### サマリー

オムライスを出力できるようにした

### 実際のコード

```ruby
class Recipe
  def initialize(name)
    puts name
  end
end

Recipe.new('オムライス')
```

### どうしてリーダブルだと考えたかの説明

今後様々な仕様が来ても対応出来るようにするために、Recipeクラスを作ってそこに処理を書いた。

### 見つけた経緯

（「見つけた」場合のみ。ここに見つけた経緯を書く。）


Although this isn't a new test, it would be good to change this to run on a different thread, so we know we don't have a bug where we accidentally continue on the mainThread when we didn't want to.


IMHO, this constant isn't useful. It's only used in one place, and this definition adds a minimum of 7 bytes. :stuck_out_tongue:


I explored that solution before, but I wanted a flexible way to be able to transform on the fly data to a room without a transformer affecting another transformer, a room base transform.

So I can do this:

```javascript
primus.in('room1').transform(transformer1).write(data);
primus.in('room2').transform(transformer2).write(data);
spark.in('room3').transform(transformer3).write(data);
spark.in('room4').transform(transformer4).write(data);
spark.in('room5 room6').transform(transformer5n6).write(data);
```
Is something like this possible directly using `primus.transform`?


This is indeed mimic what we have in `ProjectDependencyResolver`, the _"bizzared"_ logic is already there: https://github.com/aspnet/dnx/blob/dev/src/Microsoft.Dnx.Runtime/Project.cs#L491. However, it shouldn't returns the __default target framework__ if the dependency's nearest target framework can't be found: https://github.com/aspnet/dnx/blob/dev/src/Microsoft.Dnx.Runtime/Project.cs#L506.

@anurse


Tagging #706


When you press a character button, there is no change in the UI to show that you pressed something. This can be confusing for the user (I clicked a few times to be sure). It would be good here to change the appearance of the button that was pressed.


Poke @obiot Any opinions here?


是是是


i use Server::getInstance() not $this->getServer()->boradcastMessage()
On Aug 10, 2015 10:12 PM, "legoboy0215" <notifications@github.com> wrote:

> It does not. You dont know anybs too.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/DarkRealms/HardCorePvP/commit/1aeadb1cf580a74de89cb2ed0af4c7fe8462f604#commitcomment-12630096>
> .
>


the purpose of having `java` installed via `include puppet` is so that you can set specific env for all team members, why not use that instead?

[puppet-java](https://github.com/boxen/puppet-java)

```puppet
java::update_version: '51'
java::base_download_url: 'https://myorg.example/dist/java'
```

no? this defeats the entire purpose of boxen imho


done


@buunguyen
I found [it](https://github.com/51isoft/bnuoj/blob/master/.gitmodules) from #172


Try


Don't do a new commit for that. Amend the original commit to not pollute the history on simple changelog things.


Just squash before merging


Lmao dope


That's more troubled that there need to be


@CodeFlowed This looks great. The only question I have would be - is it necessary to check twice for the `MAX_POSITION` in the diagonal moves? You have a check in the move_ne/nw/se/sw method itself and then it checks again when you call the move_up/down/left/right method.
Just a minor refactor, but I'm not sure the check is needed twice. Your thoughts?


The following is test result with your change, in which server capability "<capability>urn:ietf:params:netconf:base:1.1</capability>" is not shown in netconf-state/capabilities. The reason is that session's capability is not exactly server's capability, it's the merge of capabilities announced by the client and server.

<?xml version="1.0" encoding="UTF-8"?>
<hello xmlns="urn:ietf:params:xml:ns:netconf:base:1.0">
  <capabilities>
    <capability>urn:ietf:params:netconf:base:1.0</capability>
    <capability>urn:ietf:params:netconf:base:1.1</capability>
    <capability>urn:ietf:params:netconf:capability:writable-running:1.0</capability>
    <capability>urn:ietf:params:netconf:capability:candidate:1.0</capability>
    <capability>urn:ietf:params:netconf:capability:startup:1.0</capability>
    <capability>urn:ietf:params:netconf:capability:rollback-on-error:1.0</capability>
    <capability>urn:ietf:params:netconf:capability:interleave:1.0</capability>
    <capability>urn:ietf:params:netconf:capability:notification:1.0</capability>
    <capability>urn:ietf:params:netconf:capability:validate:1.0</capability>
    <capability>urn:ietf:params:netconf:capability:validate:1.1</capability>
    <capability>urn:ietf:params:netconf:capability:with-defaults:1.0?basic-mode=explicit&also-supported=report-all,report-all-tagged,trim,explicit</capability>
  </capabilities>
  <session-id>4</session-id>
</hello>
]]>]]>
<?xml version="1.0" encoding="UTF-8"?>
<hello xmlns="urn:ietf:params:xml:ns:netconf:base:1.0">
<capabilities>
    <capability>urn:ietf:params:netconf:base:1.0</capability>
</capabilities>
</hello>
]]>]]>

<?xml version='1.0' encoding='UTF-8'?>
<rpc message-id="0" xmlns="urn:ietf:params:xml:ns:netconf:base:1.0">
  <get>
    <filter type="subtree">
      <ncm:netconf-state xmlns:ncm="urn:ietf:params:xml:ns:yang:ietf-netconf-monitoring">
        <ncm:capabilities/>
      </ncm:netconf-state>
    </filter>
  </get>
</rpc>
]]>]]>
<?xml version="1.0" encoding="UTF-8"?>
<rpc-reply xmlns="urn:ietf:params:xml:ns:netconf:base:1.0" message-id="0">
  <data>
    <netconf-state xmlns="urn:ietf:params:xml:ns:yang:ietf-netconf-monitoring">
      <capabilities>
        <capability>urn:ietf:params:netconf:base:1.0</capability>
        <capability>urn:ietf:params:netconf:capability:writable-running:1.0</capability>
        <capability>urn:ietf:params:netconf:capability:candidate:1.0</capability>
        <capability>urn:ietf:params:netconf:capability:startup:1.0</capability>
        <capability>urn:ietf:params:netconf:capability:rollback-on-error:1.0</capability>
        <capability>urn:ietf:params:netconf:capability:interleave:1.0</capability>
        <capability>urn:ietf:params:netconf:capability:notification:1.0</capability>
        <capability>urn:ietf:params:netconf:capability:validate:1.0</capability>
        <capability>urn:ietf:params:netconf:capability:validate:1.1</capability>
        <capability>urn:ietf:params:netconf:capability:with-defaults:1.0?basic-mode=explicit&also-supported=report-all,report-all-tagged,trim,explicit</capability>
      </capabilities>
    </netconf-state>
  </data>
</rpc-reply>
]]>]]>


TeamCity Ignite :: Ignite Cache Full API Multi JVM [Build 137](http://204.14.53.152/viewLog.html?buildId=59968&buildTypeId=Ignite_IgniteCacheFullApiMultiJvm&guest=1) outcome was **FAILURE**
Summary: Tests failed: 2, passed: 4885, muted: 278 Build time: 00:39:18
### Failed tests
```
org.apache.ignite.testsuites.IgniteCacheFullApiMultiJvmSelfTestSuite: org.apache.ignite.internal.processors.cache.multijvm.GridCacheAtomicNearEnabledMultiJvmFullApiSelfTest.testPutAllPutAll: junit.framework.AssertionFailedError: expected:<64> but was:<8>
	at junit.framework.Assert.fail(Assert.java:57)
	at junit.framework.Assert.failNotEquals(Assert.java:329)

org.apache.ignite.testsuites.IgniteCacheFullApiMultiJvmSelfTestSuite: org.apache.ignite.internal.processors.cache.multijvm.GridCacheAtomicNearEnabledFairAffinityMultiJvmFullApiSelfTest.testPutAllPutAll: junit.framework.AssertionFailedError: expected:<9> but was:<3>
	at junit.framework.Assert.fail(Assert.java:57)
	at junit.framework.Assert.failNotEquals(Assert.java:329)

```


TeamCity Ignite :: Ignite Compute Grid [Build 1756](http://204.14.53.152/viewLog.html?buildId=59971&buildTypeId=Ignite_IgniteComputeGrid&guest=1) is now running


你在<<_getting_started>>中看到，可以用`git config` 配置 Git。


@pongstr My thoughts exactly. Pemi insisted however that using `brewcask` was easier.


Yeah, I have fallbacks for IE but I think we should do another method.  Plus I think we should apply the same scaling for all header tags.


TeamCity Ignite :: Ignite Data Strucutures [Build 1789](http://204.14.53.152/viewLog.html?buildId=59969&buildTypeId=Ignite_IgniteDataStrucutures&guest=1) outcome was **FAILURE**
Summary: Tests passed: 418, muted: 92; exit code 1 (new) Build time: 00:18:27


TeamCity Ignite :: Ignite Basic [Build 1897](http://204.14.53.152/viewLog.html?buildId=59972&buildTypeId=Ignite_IgniteBasic&guest=1) is now running


Your continued work on this excellent software is much appreciated. Many people regard this as being one of if not the best available. Thank you Nev and all contributors.


首先要做的事情是设置你的名字和邮件地址：


`PlaneBufferGeometry.js` `clone` shouldn't have the copy call...


@sgkim126 이런거 앞으로 웬만하면 rust-lang/rust 스타일 가이드를 따라서 아래와 같은 식으로 써주세여

```rust
use glium::draw_parameters::{DrawParameters, StencilOperation, StencilTest};
```


The change in getConfiguration() only to avoid an error on the webui side when the current DC/DM are undeployed from Studio side. we can remove it for now and try an alternative


现在，你将了解到更多有趣的选项，并用类似的方法来定制Git。


p[i][j]等价于*(*(p+i)+j).想清楚为什么
前者直接定位到元素，后者先取行地址，再取列地址，然后通过地址定位元素


OK, an improvement is always an improvement! :smile:


I had a different setup earlier, that's why I have each field configured instead of the mixin. Fixing it in the next commit.

> On Aug 10, 2015, at 9:19 PM, Simon816 <notifications@github.com> wrote:
>
> set this to remap = false, then you won't need to do it for each of the fields
>
> —
> Reply to this email directly or view it on GitHub.
>


该文件包含有系统里每位用户和他所拥有仓库的配置值。


Nope, this isn't currently in master.


### サマリー

レシピデータを分離した

### 実際のコード

```ruby
class RecipeData
  DATA = [
    'オムライス'
  ].map(&:freeze).freeze
end
```

### どうしてリーダブルだと考えたかの説明

レシピデータ用のクラスを分けた方が今後レシピデータに様々なプロパティが出てきても対応できると思ったため。

### 見つけた経緯

（「見つけた」場合のみ。ここに見つけた経緯を書く。）


你可以传递 `--global` 选项来让 Git 读写该文件。


honestly still not thrilled with the architecture, but programming wise it does seem neat


用 `coffee(bare: true)`


I'm confused about this example then:
```python
from gpkit import Variable, GP
x = Variable('x')
y = Variable('y')
prob = GP(1/x, [x + y <= 4])
prob.genA()
print prob.A
```
output:
```python
y has no lower bound
CootMatrix(row=[2, 0, 1], col=[0, 1, 1], data=[1, -1, 1])
```
The problem has 2 variables and 3 monomials, but A has 3 rows and 2 columns.


T


### サマリー

レシピデータにあるオムライスを出力できるようにした

### 実際のコード

```ruby
require './recipe_data'

class Recipe
  def initialize(name)
    print_data(name)
  end

  private

  def print_data(name)
    puts RecipeData::DATA.find {|d| name == d }
  end
end

```

### どうしてリーダブルだと考えたかの説明

レシピデータを出力する処理を別に分けた方が読む際にわかりやすいと思った。

### 見つけた経緯

（「見つけた」場合のみ。ここに見つけた経緯を書く。）


分组当中, 变量名顺序, 从短到长.


function skynet.yield()
	return skynet.sleep("0")
end
this's needed to modify


二喵


If you want to executes a provided function once per array element, there is another way to write it by Array.prototype/forEach().
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/forEach


This situation actually won't have both before and after the change. I'll remove the guardian statement. It won't happen because duplicate items are not added to `graphItems`.


OK, next time I need to go shopping for difficult to find stuff, you're helping me out.

You found a pretty nice utility inside distutils. Well done!


I realize this isn't the issue here, but I think we should probably be more careful about when we pass arrays (``m.colCat``) rather than copies (``vartypes_without_fixed(m)``) through MPB.


I recommend making the except block explicit:

    except ImportError:


love


I think I prefer the original implementation proposed here:

https://rt.cpan.org/Ticket/Attachment/858832/444494/ProhibitReusedNames.pm.fast

It does everything in one go, which avoids the need for keeping state in `%DocumentTree`. I believe all other policies are stateless (aside from their configuration) so it seems more natural to me.

I haven't tried either one, so I'm assuming they are both equally accurate. Or did you find a problem with the original version?


The logic starts to get confusing. How about defining a local variable?
```c
bool librdkafka_owns_buf = len <= 1024 || (conf.flags & CONF_F_TEE);
...
if (conf.flags & CONF_F_KEY_DELIM) {
    if ((t = memchr(buf, conf.key_delim, len))) {
        ...
        /* Since buf has been forwarded
         * from its initial allocation point
         * we must make sure we dont tell
         * librdkafka to free it (since the
         * address would be wrong). */
        librdkafka_owns_buf = false;
...
msgflags |= (librdkafka_owns_buf) ? RD_KAFKA_MSG_F_FREE : RD_KAFKA_MSG_F_COPY ;
```


Оставлять комментарии – нехорошо.


А вот тут странно – никакого `@changes` нигде нет. Строчкой выше есть `@change`.


Can you call this 'ref'?  I am using reference in my code for what you call 'base'.


This `char buff[]` is going to contain a file name or maybe a full path name.  Better to use a size of 256 to match the size of `char filename[]` in main() of dvbtee.cpp


Why the complex logic here? If you want 1 in N, you can just use `if (randomInt(N) < 1)` right?


a


As a side note, it would probably be nice if a nation with no units of race(1) didn't actually display race(1) and its themes.


the comment doesn't match the actual parameter that the function takes as `str`


I think we need it due to https://github.com/code-newbies/ruby-monday-blog/commit/6d17bf896678a1de898b6cd2dfd68c37a41697de#diff-2f277f649bb643f777504753acacff80R9

Basically, when you call with `tags: post.tags.without(@tag)` it will override the default `post.tags`.


done.


I expect that this line has no use.

However: How can i be sure that in all c++ compilers used around here none might have a possible bug?
Especially as it was not possible to reproduce the crash on my system, and it actually appeared that when debugging..

Also keep in mind that it is pretty strange that something like
std::string someString << OutOfBoundArray
is no trouble, but later on freeing the std::String is


Pretty sure you don't need the `=== true` at the end?


:clap:


Looks good


It looks like I'll have to just estimate, maybe with a binary search, in order to get distance along the ellipse. There is no closed-form solution. See also: http://stackoverflow.com/questions/6070920/measuring-distance-along-ellipse


Lol thanks


This is good (fixes a regression), but it should be before the sysconfig lookup - user stuff precedes system stuff. Can be moved later though.


I like this - very cool idea :-)


Looks good! (thumbsup)


Ref #851 .


I had not pulled in changes from a branch merge into my local master repo


I feel like this conditional belongs in raw_connect...it's already supports service parameter at that method who's purpose is to select other connection types.


Good work on this, looks good.


What is the difference?


for this to work, there must be a call to registerDriver somewhere, but it's not obvious to me where that is.


IMHO, these conveniences aren't adding much, and are _really_ tedious to write in ObjC. my personal preference would be to use the provided API and not try to create separate conveniences for each possible return value. For example:

``` obcj
[[[self.webView wmf_javascriptContext][@"fnName"] callWithArguments:@[@0, @1, ...]] toWhatever]
```

OTOH, it would be kinda neat to parse JS doc comments and come up with "type safe" (sort of) categories for all our global JS functions.  For example:

``` javascript
/**
 @param {String} str
 @return {Number}
*/
function stringToInt(str) { ... }
```

could be:

``` objc
extension JSValue {
  func js_stringToInt(s: String) -> Double { /* call JSContext API stuff here */ }
}
```

Fun side project for another day :-P


oh, it's just down below. n/m


They weren't removed. They were moved into a single location below inside the for loop.

Without this, the ``os.environ['ProgramFiles(x86']`` throws an error due to not existing on 32 bit machines and so the entire process fails with an exception and no optimizer gets built, so it seems pertinent to the fix.


TeamCity Xbim :: Xbim Geometry :: XbimGeometry [Build 23](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=50&buildTypeId=Xbim_XbimGeometry_XbimGeometry) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 02:58:10


Don't commit as the @ncsubot user if at all possible, please. :+1:

Good to keep the commit history useful with actual data.


The design is a little incomplete with regards to multiple app domains.  It permits the creation of several and gives each a unique name, but there's not really a way to switch between them.  But I agree -- conceptually changing the "current" resource folder to none at all means we should tear down the AppDomain associated with current.  We'll have to revisit what it means to have multiple AppDomains.  I'm adding the test for null.


@gbadner We assume PK columns are non-nullable anyway, so I don't see the harm.  But I do not like the name *as  the method is being used*.  As named, it implies to me a validation (pk columns are required to be non-nullable, so throw an error if they are nullable).  I think a better name (again, given how it is actually used) is something more indicative of the fact we will force the pk columns to be non-nullable: forcePrimaryKeyColumnsNonNullable()?


If you could please it actually really irks me because I can't change it myself haha and it doesn't match on my page


R code for survival t cruzi-infected bedbugs analysis


I get "ReferenceError: inputFilename is not defined" here.  inputFilename needs to be defined


an events property


kind of a nitpick/question, but don't we usually export using `module.exports`? this would be the more portable thing to do if we ever wanted to reuse this JS elsewhere. since we're only using it as global JS functions in the app, it's fine for now.


you added trailing whitespace here on line 366.  please remove.


fixed #15


Thank you! This commit fixed the build. It was broken since http://ci.cgeo.org/job/c-geo/3938/


more trailing whitespace on line 369


Should be 'doitdelta' for consistency?  Although I don't really like the 'doit' prefix.  Cannot possibly be left as it was?


please convert this leading spaces into tab to match the rest of this file.


here, too


uncomment this line in add_user.php  if (!$session->isUserLoggedIn(true)) { redirect('index.php', false);}
Then you can access it from browser.


@rajatchopra this is almost certainly the wrong thing to do.  Can you take a look upstream and figure out what should be done here?


it's already under a deprecated header. I'd drop the note; otherwise we'll be repeating it every time we deprecate one.


What do you mean by 'in use' here? It appears that this will just tell you if the compatible driver has been registered or not.


Prefer ```name```, fall back to ```_id```.
Will force-drop latest commit now.


@gbadner No all databases support locking via FOR UPDATE.  I think that is what this change is about.

@drepshas These kind of changes should all be separate Jiras and PRs, as @gbadner  already mentioned in regards to another change.


would be nice to display an error message here so the user isn't left confused


Oh, misinterpreted this. `get-driver-for-url-string` or something?


I didn't know of `dynamic-plot` - that looks great.  When I wrote about "more general solutions," I think I had in mind SVGFonts & the fixpoint / iterative solving plans.  SVGFonts still has some shortcomings, and I don't think we're pursuing the fixpoint changes, so I'd like to find a way to write the equivalent of `textVisualBounded`.

Would a function returning `IO (Diagram Cairo)` work for you?  It will be easier if I don't need to test thread safety with `unsafePerformIO`.  Do you scale the text after computing it's size?  The Pango version will probably only work if you don't scale after calling `textVisualBounded` - though for small scale factors, the inaccuracy may not be obvious.

I should have time to write something by the end of the week.  I think it will be most of the code from `instance Renderable (Text Double) Cairo`, but without the `P.showLayout` near the end, which actually draws the text.


This looks like a fine change, but if you change how AmbFilter turns a Term into a K to compare, you should also change how Terms are changed into K in parseString.


trailing whitespace


Now missing a value


no need for this note.


So 'being used' here means 'the one that handles our connection string'


There's an automatic caveat when emacs lisp files are installed in `HOMEBREW_PREFIX/"share/emacs/site-lisp"` and that's the only information that's Homebrew-specific; as a rule we don't use caveats for general configuration instructions.


it's 'do bit byte shuffle' and that is just there to split the integer.


HEHE i sometimes hate github...


this should probably check to be four?


Hard to say.  This PR is VERY out of date.  These classes are VERY different today.  @drepshas what was the base for these changes?


Seems like calling `(driver-in-use)` would be more correct


"the events are" or "events is"


There's a quotation mark causing trouble here.


Because the order of declarations matters if they are just local functions. In a table it doesn't.


I remember anonymous classes working at one point, @Mumfrey would have to confirm whether they should work or not. If they don't work and they should, then that's a bug with mixins.


small remark: seems like inconsistent tabs/spaces amount


this same object is on line 151. We can probably drop it up there and just have this.


I guess I'll have to read some GCC GC docs. Are you sure that `hash_table` has got GC support? I get this error:
```bash
./gtype-d.h: In function 'void gt_ggc_mx_language_function(void*)':
./gtype-d.h:46:32: error: no matching function for call to 'gt_ggc_mx(hash_table<d_label_hasher>*)'
       gt_ggc_mx (&((*x).labels));
```
And there's not a single `GTY` in hash-table.h


Good, i had to fu** something else then:)


Why is the STATIC_URL prefix necessary?


use Pap.getPluginResource() and Pap.getPortletId()


This might be problematic if the user is writing to more than one file.  I'd rather see this invoked by a single command in the `output` class that would call call `add()` in the `output_stream` class and set the output file size limit on *only* that output stream.


did we verify that removing this is okay?


@blandinw You're right that I forgot to update the README with this information, thanks for catching that. I forgot because I made such a big deal about it in the mailing list.

The dependencies are purposefully in `:dev` because as Selenium-WebDriver releases new versions of their JAR's simply to support new browser versions, clj-webdriver doesn't necessarily need to be changed at all to take advantage of the new versions. If clj-webdriver specifies a slightly older version of the Selenium-WebDriver JAR's in its project.clj, then when people use clj-webdriver they will likely not be able to test against the latest browsers.

I get a lot of Github issues submitted that are directly related to this problem.

So to avoid the issue entirely, consumers of the library need to specify the versions of Selenium-WebDriver's various JAR's that they use for the browsers they target. This also prompted me to make things like the PhantomJS support completely optional, again by detecting whether or not the extra JAR has been included.

I'll update the README. Thanks again for the feedback.


trailing whitespace


For best practice, you should probably quote all of your arguments passed in as variables.


@mdibaiee Meant to delete that comment, sorry.


broken whitespace in `output.h` on every line added


This is not an event bind, it should be in other function


Some how xampp server causing this issue.
replace the include_once in the database.php with the following code require_once(dirname(__FILE__) . DIRECTORY_SEPARATOR . "config.php"); its should work fine.


`xmpp-client` is broken now :(

```
# github.com/twstrike/xmpp-client
./ui.go:416: not enough arguments in call to s.privateKey.PrivateKey.PublicKey.Fingerprint
./ui.go:591: not enough arguments in call to s.privateKey.PrivateKey.PublicKey.Fingerprint
```


Great to hear!


thank you,  Do i replace the pkginfo with what is here?

> On Aug 11, 2015, at 3:55 PM, Timothy Sutton <notifications@github.com> wrote:
>
> f4bd153 <https://github.com/autopkg/recipes/commit/f4bd153a7d1bc819fd29cc34b40ef3311656a29c>


@JourneyJay : SAI (SmartAI) are scripts in the MySQL database (TDB), can be modified directly.
Core scripts = C++ source code, changes have to be recompiled. I hope you knew this, but hey....


why this change?


file_size_limit should be a property of `output_stream`, only.  A function will be needed to pass it through the `output` class into the `output_stream` class, but the user should be able to have multiple `output_streams` tied to a single `output` with different `file_size_limit` values for each


see my comment in output.h for `output::file_size_limit`


Fixed here in PR #9


do we really need to pass `int fd` into this function?  won't this value always be `output_stream::sock` ?


not sure if it's ok, since bs events are made for indication bootstrap plugin activity, instead of initiating this activity.

This needed to be checked, but anyway if works - ok


a lot of broken whitespace in output.h on almost every line...


just check


duplicated arguments in query selections should be on top of the file in const variables, such as NAME.
var TOGGLE_HELP = '#' + instance.namespace + 'toggleHelp';
...
jQuery(TOGGLE_HELP).bind(...


Nice :)


Thanks for add me :+1:


trailing whitespace...


Quite complex, should it be in another separate file/component and used here as a dependency call?


ack


So re-order, or upvalue the variables, then declaration order doesn't matter anymore. :)

ie. this works:

```lua
local foo, bar

function foo()
  bar()
end

function bar()
  print("bar!")
end
```


superflous newline


Thanks Victor - I really appreciate the feedback.


If it is an HHVM specific setting, then it should have a "hhvm." prefix really. I don't want to have to deal with different meanings in Xdebug proper and HHVM's port.

That's also really true for having "xdebug.enable" and "xdebug.default_enable"...


This should just be

```js
let tabTitles = ValidComponentChildren.map(...
```


We may also want to depend on `lwjgl-util`, though I'm not sure if we'll need this or not. @ArcticLight probably knows...


Don't make the argument optional. Just require everyone to explicitly pass it. The tests have your back.


I don't object to this code, but couldn't we just make all freezers be KItem?


what if the user wants to output a single file and cap it's size limit?   perhaps we can make this feature work if the user specifies -ofilename and -lsizelimit but does not specify -r


trailing whotespace... and just skip one line, not two


var lastElem = elem.last();
lastElem.children();
lastelem.trigger()
..

twice is probably ok, but 5 times is too much


use two spaces before #


Possibly... @grosu what do you think?


adding trailing whitespace again


Should be `React.PropTypes.node`


Have you fixed this with Ivan's changes of custom $.fn.show and $.fn.hide?


Logic for error recovery should go here. This is a good candidate for breaking out into another source file - perhaps InterfaceErrors.elm.


Looks good. I am aware of the 2 testcases that fail. I'll get them fixed once dogfood environment is up and running.


Good!


Care to submit a PR? It isn’t a complex fix, and we’re here to help.


What is the point of append if resultWrapper was ```empty()```'ed  before?


Are you sure ``and key.MOD_SHIFT`` is correct?
I think here as well as in the zoom-out case below, we should use ``modifiers & key.MOD_SHIFT`` as [suggested in the pyglet docs](http://pyglet-current.readthedocs.org/en/latest/programming_guide/keyboard.html#modifiers) make sure that the user is not required to press *only* shift, but might also press other modifiers :)


Yeah, if that's the preferred, I can do that.


That will be two jolly ranchers please


Is there a benefit from this test? Could you show an example of where this would break but the other would not?


@MappyMary click `Commit`

![Uploading Screenshot 2015-08-11 13.34.51.png…]()


added trailing whitespace...


On commit message: s/unexpected/undefined/.


98318570_RemoveRadianceCheckbox (macumber) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2179 of 2215 tests passed)

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


wondering why this whitespace was changed here too... you didnt change any code here :-P


$code_types values has being scape on line 56, line 196
 I think that this variable values has being scape all the way in this script. It has being done already.
52 // Generate JavaScript to build the array of diagnoses.
53 function genDiagJS($code_type, $code) {
54   global $code_types;
55   if ($code_types[$code_type]['diag']) {
56     echo "diags.push('" . attr($code_type) . "|" . attr($code) . "');\n";
57   }
196 "value='" . attr($modifier) . "' size='" . attr($code_types[$codetype]['mod']) . "'></td>\n";


trailing whitespace...


I like aliasing ``pyglet.window.key`` to ``key`` in order to get more terse code. Why not use it here as well? :)


RoomAirflowNetwork (lgu1234) - i386-Windows-7-VisualStudio-12: OK (1795 of 1796 tests passed)

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


Hmm, xdebug.default_enable isn't nearly as clear but I don't think anyone would object to a PR changing it to be consistent.


97204612_MultiSpeedFan (Nigusse) - Win64-Windows-7-VisualStudio-12: OK (1803 of 1804 tests passed)

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


Make the tests in kotlin, it should work.


i don't think you exported this function, can we make it static?


Este es un comentaroi


this is probably fine, but I don't really know what it's used for... could you just explain it a bit?  maybe add some more comments


@nelsonab Assuming you have the `recipes` repo cloned already (which you seem to have), then it should be as easy as:

`autopkg repo-update recipes`


is it the only way to handle this css?
is it cleared when the helpBlock is rerendered with new values?
please consider using style attribute instead.


Looks like you spelled "canola" wrong


So, what my intention with this change is (and a lot of others) is too detangle the code a bit. In this case by reducing the number of functions on the WeakAuras table and by documenting what those functions do.


Yes, the problem here is that Soletta is enabling this compile flag but when Contiki (and probably RIOT) will link the final binary it don't have the sanitize flags and don't link with the libubsan, maybe we will generate the soletta static lib with the libubsan symbols.
Making story short, this fix is wrong and I will remove it.


yo why we need this bruh


Lol, incremented the master version for this?


@Dwar3xwar DWMR. Good. Concise. A simple comment about what the code is doing, just one line, will help you when you have to revisit your work after months of not looking at it - which is inevitable.


Personally I think having the utility functions as local/upvalue functions in the file seems cleaner and more logical, but ultimately Stanzilla has to decide what he'll merge.


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1273 of 1273 tests passed)

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


as I mentioned in the previous patch, I would prefer not to have these as properties of the `output` class.  let's only pass them though the `output` class and allow them to be configured differently in each `output_stream` class


sounds an awful lot like
```javascript
if (prevMarker)


Changes look good to me, will merge


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.900.31c69fa8b4003d9a0f8e247a55537c001681fd96](http://build.palaso.org/viewLog.html?buildId=60947&buildTypeId=bt431) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 396, ignored: 4 Build time: 00:03:07
### Failed tests
```
ControlDataIntegrityTests.dll: ControlDataIntegrityTests.CharacterVerseDataTests.DataIntegrity_RequiredFieldsHaveValidFormatAndThereAreNoDuplicateLines: <no details avaliable>

```


please convert the leading spaces to a tab


98318570_RemoveRadianceCheckbox (macumber) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


does this patch enable a new file output to continue where it left off from a previous process?
if that's the case, it sounds cool to me but some documentation & inline comments would help.


overall impression (should not be changed, just a hint):
file is too complex in terms of rendering points. There are just too much possible lines that modify DOM directly. Which overcomplicates the possible maintenance in future. Most likely it will not work properly in not perfect conditions (few other portlets on the page or two of those).

Ideally:
-omit mixing AUI's Component.create, since the portlet is using jquery anyway;
-one simple render function based on a template, so only this function will change the dom using some sort of JS object as a rendering model.
-one simple function for binding events, which should destroy all global events once they not needed.


I don't know if they can be added during build, but as I expect the probe to be use only occasionally it makes sense to retrieve the sensitive vars only when needed.


?????????????????

Enviado desde mi iPhone

> El 11/08/2015, a las 12:55, Kodoku-Shinku <notifications@github.com> escribió:
>
> que onda con esto?
>
> —
> Reply to this email directly or view it on GitHub.
>


develop (axelstudios) - i386-Windows-7-VisualStudio-12: OK (2211 of 2232 tests passed)

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


On 8/11/15 11:54 AM, Ola Bini wrote:
> From your implementation I learned I noticed a few things:
>
>   * You ignore the problem of the secret, just assuming it is there.
>     That's not a solution - just ignoring it.

Yeah, that's my read on the spec. They must have the secret in hand
before starting the SMP - taking it from the user's hand and putting in
SMP's is someone else's problem (given the current spec does not have a
WAITING_SECRET state). That's why it seems to be a fair assumption.

>   * You copy the data we have in the message into the smp state, instead
>     of storing the messages. I'm fine with that, but I don't really see
>     the difference. =)

It is the other way around. The data is generated into each SMP state
and then other data is derived into SMP message (by copying to it in
some cases).

We currently do exactly the same (those method were copied from our
otr3), but in addition to that we also keep each message inside the state.

We don't need to use any previously received message to verify the
current message. The protocol mentions which data should be stored at
each point.

>   * You do a lot of explicit ugly iffing on versions, which is not very
>     nice. Incidentally, the OTR SMP protocol still has version 1 - it is
>     the OTR protocol that is at version 2 and 3.

Got it.

I'll continue on the head-scratching - I believe I understand all the
constraints better now. Thanks.


Awesome ! thank you :)


ah man


A: We don't!


@daniel-beck could you please have a look at the follow up commit?


This strategy sounds great.


@chiggs Had to add this as well.


Please use tabs instead of spaces.


peerDependencies will not duplicate react-native inside react-native-navbar. I think dependencies is a bad idea.


@andresriancho I've included this as each option has a 'get_help()' method, but should say I haven't found a plugin yet that actually implements any help text. Still worthwhile?


Not sure how to write an IT for this though. Any suggestions?


see b74e5ae


Full stop at the end of comments, please.


Since this is being saved in the map state then I need to get it from there unlike viz entity charts which is being saved straight in the state object.


Should we use s" binary classification, but thresholds has length != 2.  thresholds: ${ts.mkString(",")}" for readability?


A missed point from the commit message:

We discovered possible duplicated annotations from our input files. We remove mentions that are coreferent with each other and also have the same span. Such mentions should be treated as identical. Such behavior is also not allowed by our validator.


I thought there was something fishy here :)


Move this up with the rest of the void functions, don't just randomly bundle it otherwise it's untidy.


@tcanabrava please check if it still misbehaves with pr #496
I've tested it here and it seems to be fine now.


From npm `3.x` peerDependencies wouldn't be installed by default. It also introduce "flat" dependency tree, so it'll re-use installed one.


@andresriancho Any problem with using get_quick_instance() that you can see here? I wasn't sure of a way to get the long_desc without creating a plugin instance - not sure if there's a better way to do it.


Please mention the defaults for key and value/values in the header comment


Same here :)


Added note to #152 to track the work to make this behave.


This might be out of left field, but is there a cleaner way to structure `genA` where we iterate through exps instead of through varlocs, so that this add-on magic doesn't need to happen?


This entire function is going to create warnings for obvious reasons. rand returns an integer value, those integer and just so the code is clearer, change your decimal values over to floats (e.g. 1.0f)


I'd rather this could be done without new constructors for `Solver` or `Net`. Could the root solver / net only be assigned when needed by the parallel setup code?


Could you send this to Bob? The formatting looks really weird here...


thank you.  I use autopkgr and have updated the repos and 11.0.10 is still the only one showing up

> On Aug 11, 2015, at 4:46 PM, Timothy Sutton <notifications@github.com> wrote:
>
> @nelsonab <https://github.com/nelsonab> Assuming you have the recipes repo cloned already (which you seem to have), then it should be as easy as:
>
> autopkg repo-update recipes
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/autopkg/recipes/commit/e1b806ffc9ca2bb92cc1292495f8930723d8454f#commitcomment-12650523>.
>


Eg this looks much better here :)


@leggetter All errors and warnings are fixed in https://github.com/pusher/pusher-websocket-java/pull/68.


Sounds like the issue mentioned in [this XDA thread](http://forum.xda-developers.com/showpost.php?s=1c22e3a9d4c004ebbce8d0db3d58a435&p=57761785&postcount=3) as **Fixing internal storage issues**


This resolves #5.


In KORE, freezers are just KLabels.  So K terms with a freezer at top are just KItems.  The implementation can be different if needs to, but does it need to?


once again, Pap.getPluginResource() and Pap.getPortletId()


Is this log here to stay?


@moizjv add more options from [here](https://github.com/appium/appium/blob/master/lib/devices/android/android.js#L398)


Right, I'll take it in again.


yeah but that's npm 3.x. I think it would take a lot of time for 3.x to get adoption. Should we break this now?


Im learning
On Aug 11, 2015 1:49 PM, "Nicholas Egan" <notifications@github.com> wrote:

> sounds an awful lot like
>
> if (prevMarker)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/egansoft/circa/commit/eaa73fcba79a3add1e292b3e5efa144e028c2f4e#commitcomment-12650599>
> .
>


`res.Body` is closed here. I think we have to return ```nil```


s/to use as entry point/to use as the entry point for execution/


this is not an event binding


TeamCity ChameleonForms :: Continuous Integration [Build 250](http://ci.robdmoore.id.au:8010/viewLog.html?buildId=3646&buildTypeId=bt20) is now running


No, we weren't stuck with it, we just ran out of time. We had a lot of git
errors and found ourselves pressed for time. Given a different deadline,
we'd have had it completed. The logic for getting the SHDetailView to
populate the data isn't too bad, we just didn't have time.

On Mon, Aug 10, 2015 at 10:12 PM, Tom Elliott <notifications@github.com>
wrote:

> SHDetailViewController doesn't appear to be receiving the character object
> to populate the view. If this is something you got stuck with I'm happy to
> work through it during/after class tomorrow.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ddistant/character-generator/commit/73e28c8e637fbbe70db136be9170a9cf7365e2f6#commitcomment-12630093>
> .
>


Thanks!


You can have `init` check the sharedInstance var, or return the `[investigate]` value. In objc `new' calls `init`.


Remixed Douglas Tweets.


Extra `https://`?


where did you get `get_help` from? Maybe it should be `get_long_desc`?


HEAD is now at 2e37c58... Merge pull request #2 from andycasey/test
laml65:luminosity-cannon arc$ time python test_cannon.py
2015-08-11 22:02:49,238 [INFO] Training Cannon model using label vector description: 1 + TEFF^4 + TEFF^3 + TEFF^2 + TEFF
Training Cannon model from 671 stars with 7214 pixels each:
[====================================================================================================] 100%

real	1m47.962s
user	1m46.732s
sys	0m0.608s
laml65:luminosity-cannon arc$ git checkout master
M	code/tests/test_something.py
Previous HEAD position was 2e37c58... Merge pull request #2 from andycasey/test
Switched to branch 'master'
Your branch is up-to-date with 'origin/master'.
laml65:luminosity-cannon arc$ time python test_cannon.py
2015-08-11 22:04:48,970 [INFO] Training Cannon model using label vector description: 1 + TEFF^4 + TEFF^3 + TEFF^2 + TEFF
Training Cannon model from 671 stars with 7214 pixels each:
[====================================================================================================] 100%

real	0m11.208s
user	0m10.763s
sys	0m0.336s
laml65:luminosity-cannon arc$


Sure! It'd make it mildly harder to check bounds.


I believe that it should be fine


Perhaps this should be an optional parameter.  It seems reasonable that a caller might not want to bother specifying a `reason`.


jQuery('.mod-2 .panel-title a')?


Do we still need this method if we're not using it on ln#13 anymore?


We could run it always (similar to the promise definition), but with env.code === null


1 comment, otherwise :+1:


oh! probably not...


@moizjv should be sendAction not sendCommand


develop (axelstudios) - Win64-Windows-7-VisualStudio-12: OK (2206 of 2232 tests passed)

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


The AdobeReaderUpdates recipes are _new_. To have a fully updated Adobe Reader 11, you need both recipes now. You can thank Adobe for that wonderful gift. You need to install 11.0.10, then install the 10.0.12 (or whatever) update on top.


From what I got, having them in a table is only useful if you want to use them across files, right? So, if they are not used across files, they don't have to be in a table?


Will this affect artist finder for already-posted images?


One thing here: Move declarations as close as possible to their usages. Otherwise no one knows, what is used where. This was only required in old C ;)


If you need to build the llvmlite packages against GLIBCXX_3.4.20, this would need to go into the installation instructions as well. Trusty has GLIBCXX_3.4.19 max.


Show! :) #26


The policy for the other shapes is to not generate normals since you may want to transform the geometry to create other shapes. For example, create a spheroid with correct normals

addSphere(mesh);
mesh.scale(1,1,0.5);
mesh.generateNormals();


Correct, but you sometimes need to access it using window.numContainers to get the global scope. This should be avoided though.


this if assignment-and-check is not required and those should be avoided anyway as they are hard to read. Can you change that to an assignment and an if please? Even make this the initialization of error (see other comment)


Hmm not sure how spaces ended up there in the  first place, will fix.


will this last aired thing be added to the api running on http://ponyapi.apps.xeserv.us/
if so what url do i use to request the last aired episode?


I think using currentTarget is better. Scope and this can be confusing enough as it is, using something like this makes it clearer I think. But its just personal preference.


starting from here :89 is a duplicate of line :31
duplicate ends on line :119 and corresponding :60


Thanks, will look again


saw approval from mrallen1
at https://github.com/basho/riak_core/commit/9b205cf30c114971089f686d42faf28acc618944


merging basho/riak_core/bugfix/jrd/727 = 9b205cf3 into borshop-integration-765-bugfix/jrd/727


basho/riak_core/bugfix/jrd/727 = 9b205cf3 merged ok, testing candidate = 6278eb6f


saw approval from mrallen1
at https://github.com/basho/riak_core/commit/061c05ca20316c62a8ee2ee3ce66ee4f62101c73


merging basho/riak_core/bugfix/bch/pin-eleveldb = 061c05ca into borshop-integration-766-bugfix/bch/pin-eleveldb


basho/riak_core/bugfix/bch/pin-eleveldb = 061c05ca merged ok, testing candidate = 21f49781


Yeah, they don't have to be in a table. If I use a local variable at the top of the file per function, that should work.

Do you prefer that to the current state of having everything in the WeakAuras table? I do, and I think Nevcairiel prefered that too, right?


$.each is just shorthand for a for loop. think of it as the same thing as:
```
for (var i = 0: i < inputs.length; i++) {
    results.push( $(inputs[i]).val() )
}
```


Ok


Hi Stephanie,

thank you for your answer and for pushing me in the right direction. I really appreciate it.

Best wishes,

Daniel

> On 11.08.2015, at 15:27, Stephanie Weirich <notifications@github.com> wrote:
>
> Hi Daniel,
>
> I have found Matt’s implementation of delete simpler to understand than Kahrs (though see Andrew Appel’s version of Kahrs too (https://www.cs.princeton.edu/~appel/papers/redblack.pdf).) Part of that is due to Matt’s clear write up of the algorithm in his blog: http://matt.might.net/articles/red-black-delete/
>
> However, Matt and his student benchmarked both his version and the Kahrs version and found the latter to be more efficient. See: http://matt.might.net/papers/germane2014deletion.pdf
>
> Best,
> Stephanie
>
>
>
> > On Aug 10, 2015, at 6:50 PM , Daniel Dietrich <notifications@github.com> wrote:
> >
> > @sweirich Hi Stephanie,
> >
> > thank you for sharing the delete algorithms for Red Black Trees. I'm about to implement a Red Black Tree in Java 8. However, the delete method is still missing.
> >
> > Please let me ask one question: Which solution for delete do you recommend, the one from Stefan Kahrs or the one from Matt Might. Both, conciseness and efficiency, are important for me. In Java, we cannot take advantage of GADT's.
> >
> > I'm about to give the algorithm of Matt Might a try but am also curious (without having dived into it, yet) how Stephan Kahrs algorithm does compare to it.
> >
> > A little hint would be great.
> >
> > Thanks in advance,
> >
> > Daniel
> >
> > —
> > Reply to this email directly or view it on GitHub.
> >
>
> —
> Reply to this email directly or view it on GitHub.
>


tester


This line breaks Ansible on devel.

```
michaels-mbp-4:ansible michael$ ansible
Unexpected Exception: cannot import name Templar
the full traceback was:

Traceback (most recent call last):
  File "/Users/michael/clones/ansible/bin/ansible", line 66, in
...
```


Why is this copied from `JRubyPuppetBorrowResult`?  This namespace already has a `require` for `jruby-schemas`, so the existing schema could just be used.  I must be missing something ...


Doh—I'll sort.


Yeah I do prefer that. Also spares us the naming discussion :dash:


TeamCity ChameleonForms :: Continuous Integration [Build 2.0.0-PullRequest113+018](http://ci.robdmoore.id.au:8010/viewLog.html?buildId=3646&buildTypeId=bt20) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 3002, ignored: 1 Build time: 0:0:0
### Failed tests
```
ChameleonForms.AcceptanceTests.dll: ChameleonForms.AcceptanceTests.ModelBinding.ModelBindingShould.Function_correctly_with_checkbox_and_radio_lists: Test(s) failed. TestStack.Seleno.Configuration.Interceptors.SelenoReceivedException : {"errorMessage":"Unable to find element with link text 'Model Binding Example with lists'","request":{"headers":{"Accept":"application/json, image/png","Connection":"Close","Content-Length":"64","Content-Type":"application/json;charset=utf-8","Host":"localhost:52286"},"httpVersion":"1.1","method":"POST","post":"{\"using\":\"link text\",\"value\":\"Model Binding Example with lists\"}","url":"/element","urlParsed":{"anchor":"","query":"","file":"element","directory":"/","path":"/element","relative":"/element","port":"","host":"","password":"","user":"","userInfo":"","authority":"","protocol":"","source":"/element","queryKey":{},"chunks":["element"]},"urlOriginal":"/session/70989950-406d-11e5-ab57-5d5c8f3b784e/element"}}
  ----> OpenQA.Selenium.NoSuchElementException : {"errorMessage":"Unable to find element with link text 'Model Binding Example with lists'","request":{"headers":{"Accept":"application/json, image/png","Connection":"Close","Content-Length":"64","Content-Type":"application/json;charset=utf-8","Host":"localhost:52286"},"httpVersion":"1.1","method":"POST","post":"{\"using\":\"link text\",\"value\":\"Model Binding Example with lists\"}","url":"/element","urlParsed":{"anchor":"","query":"","file":"element","directory":"/","path":"/element","relative":"/element","port":"","host":"","password":"","user":"","userInfo":"","authority":"","protocol":"","source":"/element","queryKey":{},"chunks":["element"]},"urlOriginal":"/session/70989950-406d-11e5-ab57-5d5c8f3b784e/element"}}
   at TestStack.Seleno.Configuration.Interceptors.CameraProxyInterceptor.Intercept(IInvocation invocation) in c:\ConsoleBuildAgent\work\f035b8dd08e41a61\src\TestStack.Seleno\Configuration\Interceptors\CameraProxyInterceptor.cs:line 52

```


test


PAP-12792 is also here


Thanks!


Тут можно просто hypermarkets.put(obj) использовать


Looks awesome! Thanks Craig!


Yes, I can move them into `Net::Init` and `Solver::InitTrainNets`.


Сериализацию (десериализацию) надо вынести в другое место (см. предыдущие комменты)


It's seems like a shockingly weak condition: I have keys and the timestamps agree.  And yet, that's exactly the condition.  See also https://dxr.mozilla.org/mozilla-central/source/mobile/android/base/sync/stage/EnsureCrypto5KeysStage.java#42


It's not a need, it's just that my linux-64 build machine happens to have a recent GLIBCXX (Debian Jessie).
So when using the package on Ubuntu LTS it does not work, unless you add the ppa repository as I did.
Is that an issue?


Hi @alg5,

Why 3.1.1 version it’s a 1.0.* version, 1.0.8 exactly.


Perhaps add a note that all timestamps are extracted from server responses?


thank you both very much!!!!!!!!!!!!!! Its not going to be possible for me to work with this because my environment blocks old versions software.

> On Aug 11, 2015, at 5:08 PM, Greg Neagle <notifications@github.com> wrote:
>
> AdobeReaderUpdates


Same question.


asdfasfdasdf


Any ETA for releasing an update to the NuGet package with this fix ?

I'm currently upgrading my 2Day todo-list app from Windows 8.1 to Windows 10 and I'm facing to sync with Exchange / Office 365 and Outlook.com


94006714-MultipleTowers (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1790 of 1790 tests passed)

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


```console
$  python -i ponyapi.py
>>> last_aired()
{
  u'season': 5,
  u'episode': 13,
  u'is_movie': False,
  u'name': u'Do Princesses Dream of Magic Sheep?',
  u'air_date': 1436628600
}
>>> newest()
{
  u'season': 5,
  u'episode': 14,
  u'is_movie': False,
  u'name': u'Canterlot Boutique',
  u'air_date': 1442071800
}
```

`last_aired()` is defined as an IO action that produces an episode record.

```python
# last_aired :: IO Episode
def last_aired():
    # Get the information on the latest aired episode as variable new
    new = newest()

    # If the new episode is larger than the current time as an integer
    if new[u"air_date"] > int(time.time()):
        # Return information on the _previous_ episode.
        return get_episode(new[u"season"], (new[u"episode"]-1))

    # otherwise, the newest episode is the most recently aired one,
    # so return that.
    return new
```

This function literally queries for the latest episode and naiively selects the second most recent one by subtracting one from the episode number. This doesn't handle season gaps. Adding handling for season gaps would likely be a trivial addition and would make a nice contribution.


s/nto/not?


most likely there is no point of searching those, since the event is propagated, please check it on $el


indeed, thanks


If it runs on every call to highlightElement it is fine I think. But it is out of my knowledge of the code how to do that and where. If someone else can change this commit or create another PR would be great.

Besides I'm realizing that maybe I edited the wrong file. Maybe should be `components/prism-core.js` the file that should had been edited.


`GVAR()`?


А если не будет гипермаркетов?


Indeed. There's no explicit dependence on them. -Joe

On Tue, Aug 11, 2015 at 10:41 AM, justboh <notifications@github.com> wrote:

> Would it be better to remove the imports of matplotlib_rc_chooser and the
> xkcd colors from plot_tools_jgb so there was not that extra dependency?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nistpenning/calc/commit/f18c32b4957b158e5f6c30fdbb847cebee204605#commitcomment-12645205>
> .
>


If a bad actor issued ``DELETE /crypto``, we could have no modified time for ``/crypto`` (I think collections once known but no longer are not present in ``info/collections``).  And now I see what you mean by "fetch and fail".  Change that to "fail to fetch"; I interpreted it differently.


Should probably be `EGVAR(cargo,X)` yes


test


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


it's already nil


Lets keep this but drop the return, i.e.

query_url = __get_data(url)


The 2nd `other` should stay. First should have its name changed to `actual`.


@cohnjesse WTF 3dCART for caching this page


That's the one I meant yeah. That reminds me, why isn't that macro also used in optionals?


Лучше, чтобы такие методы возвращали не void, а то, что они получают. В данном случае, это Bitmap


then we can rename the method to do just one thing, i.e. __filter_data(query_url, data_filter)


Oh, don't worry, most compilers have plenty of bugs. But fortunately it's extremely rare that very common stuff like this is broken.

Seeing your observations, I can imagine that you were quite confused! But the pretty strange behavior you observed is not so strange actually. In the class declaration we can find this:
```
...
uint32 m_auiEncounter[MAX_ENCOUNTER];
std::string m_strInstData;
...
```
Now let's play compiler. When you see code like ```m_auiEncounter[9]``` you're assuming this is perfectly fine. After all, if this would be an out-of-bounds access, it's undefined behavior, which means we're allowed to do anything we like anyway, so we might as well do the same as in the well defined case. This is actually what modern optimizing compilers do constantly: they will assume the code does not trigger undefined behavior and optimize with that in mind, which often leads to surprising results when you do trigger undefined behavior after all! Anyway, we'll be generating code that will simply use ```<address of m_auiEncounter's start> + offset to tenth element```.

But as you've discovered, there is no tenth element, so you'll be pointing to the memory that happens to be just next to the array. Oops, that's where the m_strInstData member is located! So there's bad news here, and worse news. The bad news is that this is actually valid memory too, so you can read and write data here just fine (this is bad because it hides the root of the problem). The worse news is that you're corrupting someone else's memory: writing four bytes here almost always nukes the string member from orbit, because whatever internal structure an std::string has, it is now most likely corrupted.

This explains why you have observed that you can read and write out-of-bounds here just fine, but why a function call on the string member later on causes havoc. In this case it sounds like that operation was running the string's destructor when the instance_temple_of_ahnqiraj object was destructed, but it could have been any other function relying on the first four bytes of the string object.

Ah, the joys of memory corruption and undefined behavior in general! :grinning:


RIP


Delete these old lines of code?


I'll throw out `register-jruby-event-handler` or simply `register-event-handler` as alternative names.  I'd probably prefer the last one but I am okay with any of them (including what's already there).


cc: @SidharthNabar @ericstj @weshaggard

This change only affects the Windows 10 UWP SDK version of System.Net.Http.dll.  It doesn't affect CoreFx apps (i.e. ASP.NET v5, etc.).

The NuGet packages for the Windows 10 UWP SDK are built on a different schedule than the CoreFx packages.  I will let Eric or Wes comment on the schedule.  But we expect to provide updates to the UWP SDKs during the year.


Are you fucking serious?


I think the body here should be empty, just put everything into the initializers (and make sure they're sorted according to declaration).


Unless this is urgent I don't intend to push this to the website


drop the response assignment and take the data as an argument, then the method can do one thing and one thing only and be renamed


Deleted them in a later commit


I was not aware of the previous discussions on the mailing list.
The reasons you state here make perfect sense and fwiw I think it's the right decision.
Thank you for your time, we're happy users of your project :)


Я уже писал про Uri.Builder. Давай я не буду по два раза писать одно и то же.


name this __filter_data(server_response_data, filter_arg)


From what I tested, this is okay for php-fpm and with nginx, but I never test it for any other web server configurations..


this allows us to drop this test as it is already taken care of in the functions that are the entry point to the code


where is `max_validate_attempts` defined? I'm getting `ReferenceError: max_validate_attempts is not defined` when it fails.


Nice


thanks. believe it or not, I think that is significantly more readable.


Nice!


this needs to take the argument now and pass it along, i.e. __parse_response(server_response_data)


Use your judgement on what's the best approach -- including keeping the current. Just wanted to toss the idea around.


@robbiehanson Make sure to commit the .xcworkspace and ensure that all of the schemes are marked as 'Shared' and committed along with the workspace. Also instead of "cd" you can do everything from the root directory, and install cocoapods updates a lot faster:

https://github.com/robbiehanson/CocoaAsyncSocket/blob/master/.travis.yml


The file BackgroundProcessContextExtensions.cs has not been added to Hangfire.Core.csproj, resulting in the dev branch not building.


Хосты не нужно так хардкодить. Для начала, их можно хранить в ресурсах в properties файле.


Fixes #92

A new button is added to show or hide uncovered classes from the coverage results window:
![image](https://cloud.githubusercontent.com/assets/1618378/9211253/2d68240c-4081-11e5-8975-43aff937a0f5.png)


heh, it surprisingly is!


more on #319


That typo was fixed in 80ae61a2.


The way they were are incorrect.  Please look up line endings windows vs *nix.  It will be at lot more clear that way.  Almost all static scanners will complain.  New hosted project uses git attributes which makes library correct.  It's called line normalization.  If there is a problem it can be mitigated by changes the tests to not care about line endings.  In fact the formatter itself cares about line endings so possibly look there.  Regardless though getting line endings from the system makes it correct.  Hard coding
 is always an issue.  That makes it not portable.  If it failed for me them it certainly will fail for others.

Sent by Outlook for Android

On Mon, Aug 10, 2015 at 2:17 PM -0700, "Marvin Froeder" <notifications@github.com> wrote:
So needs a special git install?! Or custom configuration to our tests to pass?!

That doesn't seem right.  Git should not be changing files and making them OS dependent.

The test was originally test independent, but, due git changing the test_file the test become problematic.

or am I missing something?!

Also, if we go that way, everybody who ever decide to contribute to the project would need to reinstall git with some special non-default configuration.

IMO, you should reset this configuration and let files as they were.

---
Reply to this email directly or view it on GitHub:
https://github.com/revelc/formatter-maven-plugin/commit/e0ae79082f4b415ee0fb84844dd3540d07175f7e#commitcomment-12625609


В классе уже 300 строк. Пора разнести его по нескольким классам.


Make one final push to the website, review section has improved a lot

On Tue, Aug 11, 2015 at 5:28 PM, william widmer <notifications@github.com>
wrote:

> Unless this is urgent I don't intend to push this to the website
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/wwwidmer/CAPSTONE2015/commit/458f71329eb79f47c7d526880c4796f37c637e91#commitcomment-12651442>
> .
>


I find this comment a little confusing ... I would say that the callbacks are actually the event handlers.  Perhaps these are "Functions which call the event handlers" or something like that.


![He's doing it again](http://static.flickr.com/3248/3285678255_1905fbfbff.jpg)


Also, I have several other shapes that I haven't committed yet including a cone, disc, prism (https://en.wikipedia.org/wiki/Prism_(geometry)), cylinder, and annulus. The cylinder is basically the same, but centered on z so that it extends -h/2 to h/2. It also has a twist parameter to allow a variety of low-vertex shapes. I'd be happy to add these even if it means kind of replacing your work...


While all these one liners are cute and arguable elegant, and yes the language was written to do just this, it also requires mental loops every time someone looks ta the code. Please expand. Theoretically the one liner is also more efficient, but this is not a performance critical path.


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


Re-add space please


It's either "is dead", or "has died"


It seems easier to use `spec['Name'][1:]`


PLease expand the name of this function "instr" is not intrinsically obvious


#23: Should have been here, but wasn't. Weird.


This is incorrect.  It should be "losing", not "loosing".


I think this part should be landuse=urban and landuse=rural instead.


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


I'm thinking that we maybe want to extend this example and include various networks since that seems to be the more complicate set up these days in the cloud


The bug I'm facing is described [here](https://social.msdn.microsoft.com/Forums/vstudio/en-US/ff66088c-6705-40c0-b839-491b6ed6eb76/uwpwindows-10-bug-in-httpclienthttpwebrequest-when-using-network-credentials?forum=wpdevelop)

> System.Net.Http.HttpRequestException: An error occurred while sending the request. ---> System.NotSupportedException: This IRandomAccessStream does not support the GetInputStreamAt method because it requires cloning and this stream does not support cloning.

I cannot use Windows.Web.Http.HttpClient in the PCL project, so I thought this was related when I saw the commit. Am I wrong ? Any idea for a workaround ?


94006714-MultipleTowers (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1796 of 1796 tests passed)

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


Still understanding this code, but this looks weird. Single pipe is bitwise - the lines below this are casting numbers to remove decimals, but it looks like maybe this is trying to set a default? If so, you should use a double pipe.


If the convention in the commit message is what you're sticking to, then by all means do that consistently. However, it would make more sense to me to include C standard library headers first, then third-party library headers (including GLib), then local metrics headers.


math table 为什么不是“数学表”或“数学表格”。有什么特殊含义吗？


If we see it show up in profiling I'll fix it.