Chalupa_batman

On Thu, Apr 16, 2015, 3:01 PM Korvin Szanto <notifications@github.com>
wrote:

> +1 for shaboodley
>
> On Thu, Apr 16, 2015 at 11:14 AM Michael Krasnow <notifications@github.com
> >
> wrote:
>
> > whoops, didn't look at the full file path :P
> >
> > --
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/concrete5/concrete5-5.7.0/commit/7d85d368618f145c52a45a03c88dbaa7bfcc67ac#commitcomment-10763065
> >
> > .
> >
>
> --
> Reply to this email directly or view it on GitHub
> <https://github.com/concrete5/concrete5-5.7.0/commit/7d85d368618f145c52a45a03c88dbaa7bfcc67ac#commitcomment-10763883>
> .
>


Would it make sense to rather create a stub of FirebaseClient for testing?


Não exatamente. O correto seria assim... O valor inicial do link faz parte da pré-condição (given). No when a ação de edição é realizada, devendo ser informado o novo valor para o link. Então no then o resultado é verificado...Como se sabe que a ação foi bem sucedida? A alteração é realizada, alguma mensagem de confirmação é exibida, o sistema segue para outra tela ou permanece na mesma...?

Novamente, cenários não devem ser escritos expressando vontade/desejo/intenção, mas sim ações que partem de uma condição e levam à uma nova condição. As ações, por sua vez, são consequência da vontade/desejo/intenção do usuário, mas isso fica implícito.


We need to be specific about which files they are copying. Example ( index.html, home.html)


More general question: Why move all development dependencies into the Gemfile?


Yes, here is an example:

  https://github.com/InsightSoftwareConsortium/ITK/blob/master/Modules/Filtering/FFT/include/itkHalfHermitianToRealInverseFFTImageFilter.hxx#L30-L78


Blah


`Handlebars.compile(template)(data)`


I wonder if this still fails on recent version of Rubinius these days.


Again, I think we should write out tests to not hit the client at all, otherwise the tests will be quite slow


Hi @marcuswanner , I've imported this repo from Google Code. I see that the 2.2 branch has code unmerged with master.

If I want to start working on this, what is your recommendation? working on master? merging 2.2 to master...? anything else?

Thank you for your time, regards.


Typo:

    M("Progams", f="programme",

rrrrrrrrrrrrrrrrrrrrrr


Having separate img selectors at different points in the page was necessary for the firefox/ie resizing


Other than this nit, LGTM


Have you actually test this? O_o


@smallint: Did you have a chance to test this? I am currently far away from home, which won't be changed until end of week. I am quite sure, I am missing something, in decode itself.


There is no equivalent necessary for mouseup/mousedown events, as far as I know.


can you LTGM on the updated Pull REquest?

On Thu, Apr 16, 2015 at 3:06 PM, Sal <notifications@github.com> wrote:

> Other than this nit, LGTM
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/loosebazooka/gradle-appengine-plugin/commit/75bee25e576953a6e766753f6d39ebe23295937c#commitcomment-10763989>
> .
>


Looks like you have the same problem I've been having with Visual Studio touching the line endings on some files (GZL.ini and VertexShaderManager.cpp).  I always have to check the "number of lines added or deleted" in tortisegit before commiting, revert the file and copy and paste in the changes again in TortiseGIT if it has the problem.  Such a pain.  It only happens on certain files though.


Generally speaking, the Gemfile can express dependencies that can't be
expressed in the gemspec, e.g. git dependencies.
On Apr 16, 2015 12:04, "Aaron Stone" <notifications@github.com> wrote:

> More general question: Why move all development dependencies into the
> Gemfile?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/tamird/mysql2/commit/b64e22cb2d3c5aee2c977f02026f8a105d3b9d6a#commitcomment-10763947>
> .
>


vi aqui rodando e :+1:


`read_timeout` is a FIXNUM here: https://github.com/brianmario/mysql2/blob/master/ext/mysql2/client.c#L549 so it's probably being interpreted as zero.


Also renamed TreeNode.**middle** to TreeNode.**value** for less confusion.


I don't think they hit the client. The stub takes care of that. They seem
to run about the same speed before and after I added this code.
On Thu, Apr 16, 2015 at 8:03 PM Marc Rohloff <notifications@github.com>
wrote:

> Would it make sense to rather create a stub of FirebaseClient for testing?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/openjournals/theoj/commit/348299da48fa4548135a9ed6e07aca6cdead3ebb#commitcomment-10763927>
> .
>


There seems to be something wrong with this change. Installation on OpenShift now fails with:

[Thu Apr 16 14:10:29 2015] [error] [client 127.5.152.129] mod_wsgi (pid=492147): Target WSGI script '/var/lib/openshift/552fb8d5e0b8cd83e5000040/app-root/runtime/repo/wsgi/application' cannot be loaded as Python module.
[Thu Apr 16 14:10:29 2015] [error] [client 127.5.152.129] mod_wsgi (pid=492147): Exception occurred processing WSGI script '/var/lib/openshift/552fb8d5e0b8cd83e5000040/app-root/runtime/repo/wsgi/application'.
[Thu Apr 16 14:10:29 2015] [error] [client 127.5.152.129] Traceback (most recent call last):
[Thu Apr 16 14:10:29 2015] [error] [client 127.5.152.129]   File "/var/lib/openshift/552fb8d5e0b8cd83e5000040/app-root/runtime/repo/wsgi/application", line 25, in <module>
[Thu Apr 16 14:10:29 2015] [error] [client 127.5.152.129]     from .virtualenv import activate
[Thu Apr 16 14:10:29 2015] [error] [client 127.5.152.129] ValueError: Attempted relative import in non-package


Do you have a spec for this? I believe you've introduced an error here, because you're passing the result of `sign_in_and_redirect` to `present?`, which doesn't take arguments.


How do I do  the re-bootstrap? I installed ruby2.0, but rake test reference to ruby1.9.3 yet.


Is this multi-value? If so, what about the multi-validation? Because the following will validate:

```ruby
[''].present? == true
```


You can leave comments on any line


LGTM


this line is empty


done, thanks!


good point


I suggest adding to the comment: "Note that each thread open its own database connection."


This dates back to the initial work done by Vinnie.  The logic here is that the sequence number should be checked first (so an old manifest can be discarded immediately without spending CPU time on verifying a signature), but the original code for `maybe_insert()` didn't check the signature.  Upon reflection, I think `would_accept()`, `maybe_insert()` and `maybe_accept()` should all be combined.


the purpose is to associate to the symbol `T` the expression `sin(pi*Px())`. Could be any Feel++ expression such as finite element fields `idv(T)` or something more fancy. the limit is our own imagination


I need to change the dog's name


line 1 is directions


I vote for `GiovanniAllevi`: after all even the NASA named an asteroid in his honor. I've been so lucky to find two tickets for the Como theater to see him playing the piano... It has been very touching. If you ever have the occasion to attend a concert of him don't let the occasion go :wink:


Yup, understandable.

I'd be interested in having a chat with you about what would need to be done.

Will drop you and email and hopefully we can arrange a chat.


Seems clear to me, and I doubt the cycles are of any consequence.  Anyway thanks for taking the time to review and comment, it really is appreciated.


While it doesn't seem to affect blocking transmission, with --tx-carrier (nonblocking), the audio backends seem to suffer under- or over-runs, which is causing noise on my system (and dropouts if I tell it to use alsa).


I don't mind you using PAC3 the way you are at the moment. I understand the convenience with shipping the code with your addon but if you've made changes (or need custom events) please send me patches.


ancien code à ne pas commit


Just put the `\n` in the original string.


There're also history updates piling up in editOrder() function:

$this->addOrderHistory($order_id, 0);

So when you're on checkout page and doing multiple changes before confirm, even with the fix above you'll get a bunch of identical entries in order_history table with order status ID set to zero. This also should be fixed, but right now I dunno how.

Why do we even need to set order status ID to zero on edit? Weird...


When you amend the commit for the above changes, make sure the add the Signed-off-by into the commit commit. It must be in the commit comment so it is a permanent part of the git repo. Putting it in the pull request is not sufficient. Thanks.


Ohhh nice, i was hoping for someone to re-create(Kinda) the sound system.


Done!

> On Apr 13, 2015, at 6:25 AM, Sam Saccone <notifications@github.com> wrote:
>
> Mind squashing up into the initial commit :)
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/marchant/todomvc/commit/09440b13c66f39df47c49a8d5d52abdf0997f663#commitcomment-10692192>.
>


hey @deborshisaha, this code change makes sure that if there is any custom cell configuration for a form, that the same configuration will be used when displaying nested table views.

i'm not sure how this would be affected by using `PFObject` anywhere, but if you want to post some code here, i can take a closer look and hopefully offer some insight.


regresion


LGTM


I wouldn't do this, because screen sizes and font sizes etc differ. The solution with the spaces seems better.

Feel free to push that one to master, referencing the issue.


shoudn't this be the default case


They’re sorted because otherwise you run into problems where you replace a
‘1,3,5’ value with a ‘1,5,3’ value.

Just copy the code into your own field. Or override the
deflate_value_method.

Gerda

On Thu, Apr 16, 2015 at 1:27 PM, kcryan412 <notifications@github.com> wrote:

> What was the reasoning to sort the values? I am running into issues as my
> select values are custom sorted and my results are now sorted
> alphabetically rather than the order they were on the form. If this sorting
> is needed what would recommend is the best way to override this behavior?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/gshank/html-formhandler/commit/10264cc5fa217b38701d2d8a430c46f013bdc0f6#commitcomment-10762193>
> .
>


Wrong change?


Good.


If I understand correctly, this line seems to indicate that if the triple length is less than 3, you add an fictious <s> </s> sentence?


And this is the way I find out... :)


These cannot be removed. Please add them back in.


Please add rabl back in.


Good call -- it's now available on Maven Central as version 1.0.2.  Thanks!


This could actually be a serializer.


lol


@bebraw Done in cb3e02b65f80c92d862fd31ed822fca873f50af5


This is a minor change and this caused by the fact that the embedded LuaJIT 2.x in 1.7.10.1 doesn't compile on Centos 5 systems: openresty/ngx_openresty#85

It doesn't necessarily mean we need to upgrade, but I see what you mean. I have seen the commit changes between the `1.7.10.1` and `1.7.10.2rc0` and it's minimal.


Too lazy to calculate it myself, but [this site](http://passwordstrengthcalculator.org/index.php) says it's 120 vs 130 bits. Seeing as special characters are more difficult to type, I'd leave this out.


Re the commit message, this isn't "deprecated"... it was removed entirely about 40 releases back, five years ago.


Why ?


Who's trying to call it?

I don't know that we need an unbind, though it "feels" safer. :) As long as all the circular dependency are themselves detached from any reference, GC shouldn't have a problem.


One little extra question,
 I noticed that as of this commit you changed the fat functions from the assembly functions before to this full featured fatfs system that most people use. One disadvantage is that you can't specify the physical disk as easy as the previous functions. Example: before fileOpen(&f, L"nand:/rw/sys/updater.log", FS_READ); // after ???
Is there some way to specify (mount) the actual nand instead of the sdmc with this lib?


By removing these examples, you also remove all examples of filter on multiple fields. Maybe rewrite the examples instead?


We also do have an IRC channel ... ```#thrift``` on freenode ;-)


As I said, can you make this a seperate commit? You can just push it to master.


Not currently. There's a function for reading raw NAND sectors, but they need to be properly decrypted.


If I get this right, the external project's *source_dir* property is pointing to the source folder of another CMake-based project? And the *binary_dir* would literally mean the CMake output folder? The same distinction CMake makes for the project at hand?

The first reference of an external project for me was [the CMake module for Catch](https://github.com/philsquared/Catch/blob/master/docs/build-systems.md#cmake) and since this was a header only library, the *binary_dir* was not that important there :wink:

To sum this up: `ExternalProject_Get_Property(googlemock source_dir)` gets me the checked out repository, and `ExternalProject_Get_Property(googlemock binary_dir)` gets me the CMake output for the project, correct?


I would go even further.
If the regex is invalid, remove it from settings.

I think this should be done when saving settings, not on every message. Also because try/catch prevent code optimization at runtime.


thanks for the fix


I've updated the branch as per the above suggestions.


I don't like that you removed all the test cases. Otherwise, the HTTPS stuff looks good :+1:


Why this fix is not included in version 2.0.3 ... such a critical bug!
Source code downloaded from http://www.yiiframework.com/download/ - Install from an Archive File - Yii 2 with advanced application template - https://github.com/yiisoft/yii2/releases/download/2.0.3/yii-advanced-app-2.0.3.tgz


ok np.


Who requires that package?


This could be a lot more succinct as ternary:
```
this.label = opts.label === undefined ? this.name : opts.label;
```


I don't remember. Like attacking itself or not being able to navigate or something weird


Wot...


As above, this could be a lot more succinct as ternary:
```
this. requiredMessage = opts. requiredMessage === undefined ? 'Selection required' : opts. requiredMessage;
```


I didn't even select this line.


Need to revert this change, it isn't needed.


this is first comment


No, just plain assert is best.

They also do some magic to print out what is most-likely the code you're interested in (e.g. the bits around where the test failed, and the values involved in the assert) as well as some likely locals.

I didn't bother search/replacing *all* the files, in case your project wanted to keep the unittest-like syntax for some reason :)


Replace with "return resp if buff.to_s.length == 0"


========  Interface  ===========
#import <Parse/Parse.h>
#import "FXForms.h"

@interface Person : PFObject<FXForm, PFSubclassing>
@property (retain) NSString *name;
@end
========  Implementation  ===========
@implementation Person
@dynamic name;
+ (void)load {
    [self registerSubclass];
}
+ (NSString *)parseClassName {
    return @"Person";
}

- (NSArray *)fields {
    return @[@{FXFormFieldKey: @"name", FXFormFieldHeader: @"Required"}];
}
@end

Code breaks saying :

2015-04-16 12:35:21.175 Klippy[546:44694] *** Terminating app due to uncaught exception 'NSInvalidArgumentException', reason: '-[Person field]: unrecognized selector sent to instance 0x174139000'
*** First throw call stack:
(0x183f842d8 0x1957a80e4 0x183f8b3a4 0x183f88154 0x183e8accc 0x1000c7b18 0x1000c5ba8 0x1000d2a54 0x1000d6af4 0x1000ed378 0x1000ecdd4 0x1889c4c84 0x1889c4994 0x188d0154c 0x188ccf610 0x188ccc90c 0x188a48240 0x1889b86ec 0x183f3c2a4 0x183f39230 0x183f39610 0x183e652d4 0x18d67b6fc 0x188a2afac 0x1000eda10 0x195e26a08)
libc++abi.dylib: terminating with uncaught exception of type NSException


```
serveError(w, err)
return
```


Yes, that's right and I add the fictitious sentence at the beginning of the triple. So in a sense, the rest of the triple (the actual dialog) is just conditioned on empty sentences which is OK.

Anyway, it won't affect our models at all on the moviescript dataset, because it only needs to do this for 100-200 triples. I just wanted to keep all the triples so that I can match the exact indices later.


Contact information:
  someone@example.com.


Nice!


FacilityTabGridView (evanweaver) - x86_64-MacOS-10.9-clang: Tests Failed

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


Ah, it's coming from `EventHandler` I think. It may not be necessary. I'll add some 'TODO' logging so I don't forget about it :)


@Maksim415 Please update the following:

* Comments in Javascript should start with `//`
* All variable declaration should end with a semi-colon `;`
* leave a blank line at the end of the file


I've been removing these based on a request from @mhaskel as of last month. I believe it's a recent transition?


Okay thank you. I found it, as I have warnings turned on. Btw you can now do that in the Arduino preferences without needing to modify your platform.txt, but you will have to either build it from the source or just download the nightly build ;)


@Maksim415 :+1:


I believe all of this has been applied in some form of another a while ago (your version appears to be 1.35, 1.37 has a bunch of equivalent fixes for shutdown).


@sueloverso: If `encryption` isn't included in the `wiredtiger_open_basecfg` configuration list, then it won't be written to the base configuration file, so I don't think this merge is necessary.


A quick note: if you need to specify boolean arguments you can use: parser.add_argument('--use-all-triples', action='store_true)', without type nor default value. This will do that if you specify --use-all-triples in the command line args.use_all_triples will be equal to the boolean True.


all javascript files within /assets/javascript are included in application.js by default


Why were those lines messed up? What did I do wrong? Can you tell me so I know what to do next time?


##### Deployment available online:
#### - [app.js](https://s.brightspace.com/apps/ipa/dev/685cee2ecde731e91fac70b360e182de787c3a59/app.js)
#### - [appconfig.json](https://s.brightspace.com/apps/ipa/dev/685cee2ecde731e91fac70b360e182de787c3a59/appconfig.json)


##### Deployment available online:
#### - [app.js](https://s.brightspace.com/apps/ipa/dev/3bc5e8ddd3549f74b87e0091c4d4471bf0aa0d75/app.js)
#### - [appconfig.json](https://s.brightspace.com/apps/ipa/dev/3bc5e8ddd3549f74b87e0091c4d4471bf0aa0d75/appconfig.json)


It is quite strange that on **Mac**, I cannot see the difference of two folders, To-do-List or to-do-list, so I cannot delete the older folders, but under ubuntu it is OK, so I delete it.


ok thanks


Ah , gotcha ;)...Took a long time to see it!


Func name doesn't show what it's doing. Code compares ScheduledPair to DefaultPair, but name says it's comparing two DefaultPairs.


Mai ai de facut inca un tabel in mona.html


it looks great to delete some abundant blank lines.


Smooth :sunglasses:


My previous comment was to name this `HoursOpenID` since the ID is on the `HoursOpen` object rather than `Schedule`, which has no ID attribute (unless that portion of the spec isn't how we wanted it):

```xml
<xs:element name="HoursOpen">
  <xs:complexType>
    <xs:sequence>
      <xs:element name="Schedule" minOccurs="0" maxOccurs="unbounded">
        <xs:complexType>
          ...
        </xs:complexType>
      </xs:element>
    </xs:sequence>
    <xs:attribute name="id" type="xs:ID" use="required" />
  </xs:complexType>
</xs:element>
```


Would be interesting to see what the three different mappings we managed to generate all expanded to, for extra credit this term.


... BrandoSentinel. This ...


##### Deployment available online:
#### - [app.js](https://s.brightspace.com/apps/ipa/0.1.7/app.js)
#### - [appconfig.json](https://s.brightspace.com/apps/ipa/0.1.7/appconfig.json)


And to figure out what was between you, me and travis @guybedford


hola


:smile: ja isch vou gäbig!


Do you mean to have a `PersonID` here?  There is no `OfficeHolder` type.


Hi Matt,

I submitted the pull request right before I wiped out some folders on my computer. Since then, I've reinstalled Matlab and recloned mksort. I realized/remembered that this pull request didn't actually fix the problem, it just made it easier to debug the problem by guaranteeing plotting to the correct axes even when I was clicking around on different axes. The real problem is in oneChannelSorter.fig: pnlTroughHistFRs is in front of axTroughDepthHist, and pnlISIs is in front of axISI1-4.

I don't know if this is unique to Retina MacbookPro running OSX 10.10 and Matlab >= R2014 (I'm on R2015a currently and experienced it in R2014b definitely). rMBPs are terrible devices for creating GUIs and expecting them to work on others' machines so I think it's a good idea for someone on your end to fix this, but I can do a pull request with my modified .fig file if desired.

-Chad


Found the problem here, you need to call setTitle on `R.id.switch_logs`.

There's also an unused variable below, some lines are too long and methods aren't documented. If you fix those, it's good to merge.


This is only a temporary fix, since I had to do it directly via GitHub (since I don't currently have access to my computer).

I will be properly sanitizing input on submission within the next couple of days (unless you want to do it).


Cool, just wanted to let you know in case you didn't already fix it. Will upgrade to new version soon.


Fixed :)


Nit: dont -> don't


Stworzono dwa projekty: właściwą aplikację konsolową oraz projekt zawierający testy do tej aplikacji.
Dodano nieprzechodzący test pierwszej metody.


Correct and correct.


Other places on the code may benefit of something similar.


I wouldn't use ShardManager object but Brando : Brando.withShardsAndSentinel (couldn't find a better name but that's the idea)
The goal is to have a Brando.with{option} api. You are always going to send the same redis requests if a Brando actor is returned or a ShardManager is returned.


We want/need the rest of the `encryption` in the base config file.  We just don't want to write out the `secretkey` contents.  I don't see how I can selectively eliminate a single item from a category from the basecfg definition in the python file `api_data.py`.


Corrected: removed the extra space.


It actually breaks my syntax-highlighting, and "dont" (without an apostrophe) is already widely in use elsewhere in the codebase. Is this really such a big issue? :)


Thank you! :+1:


You're right, HTML special characters are highly unlikely coming from the programmatically defined extensions array elements.   Additionally, the values which are stored in the messages array are passed threw the text() function at time of echo (ie: eRx.php lines 74, 88, 108, & 120).


Dne Thu, 16 Apr 2015 12:10:32 -0700
Daniel Tschan <notifications@github.com> napsal(a):

> There seems to be something wrong with this change. Installation on OpenShift now fails with:
>
> [Thu Apr 16 14:10:29 2015] [error] [client 127.5.152.129] mod_wsgi (pid=492147): Target WSGI script '/var/lib/openshift/552fb8d5e0b8cd83e5000040/app-root/runtime/repo/wsgi/application' cannot be loaded as Python module.
> [Thu Apr 16 14:10:29 2015] [error] [client 127.5.152.129] mod_wsgi (pid=492147): Exception occurred processing WSGI script '/var/lib/openshift/552fb8d5e0b8cd83e5000040/app-root/runtime/repo/wsgi/application'.
> [Thu Apr 16 14:10:29 2015] [error] [client 127.5.152.129] Traceback (most recent call last):
> [Thu Apr 16 14:10:29 2015] [error] [client 127.5.152.129]   File "/var/lib/openshift/552fb8d5e0b8cd83e5000040/app-root/runtime/repo/wsgi/application", line 25, in <module>
> [Thu Apr 16 14:10:29 2015] [error] [client 127.5.152.129]     from .virtualenv import activate
> [Thu Apr 16 14:10:29 2015] [error] [client 127.5.152.129] ValueError: Attempted relative import in non-package

Okay, I've reverted it for now.

--
	Michal Čihař | http://cihar.com | http://blog.cihar.com


I almost always have security at the forefront of my mind when working with code, legacy or new.  The file-name would  have to be in the format defined on line 52 and exists in the correct path, this would only be the log files.


Corrected: along with the erx_default_patient_country options.


:smile: Thank you for reviewing; I've corrected the above notated changes.  Testing has been performed at MI2, against the NewCrop pre-production servers, the Virizon EPCS testing servers, and SureScripts.


cryptominisat and indirectly stp


@fyockm, I generally do, but have been critiqued in our own &-community against long line ternaries, thus have opted for the verbose approach


Thank you for your comments.


Is for learning only.
If you don't like to take some of you work, not make a open soUrce shits
Cheer's


Hmm, some changelog is missing and a tag :O


TeamCity Demo Green :: Develop Build [Build 1.0.5-30](http:/24.246.71.137:8111/viewLog.html?buildId=72&buildTypeId=DemoGreen_Build) is now running


TeamCity Demo Green :: Develop Build [Build 1.0.5-30](http:/24.246.71.137:8111/viewLog.html?buildId=72&buildTypeId=DemoGreen_Build) outcome was **SUCCESS**
Summary: Running Build time: 00:00:08


TeamCity Demo Green :: Develop Build [Build 1.0.5-31](http:/24.246.71.137:8111/viewLog.html?buildId=74&buildTypeId=DemoGreen_Build) is now running


TeamCity Demo Green :: Develop Build [Build 1.0.5-31](http:/24.246.71.137:8111/viewLog.html?buildId=74&buildTypeId=DemoGreen_Build) outcome was **SUCCESS**
Summary: Running Build time: 00:00:05


This should be lowercase to fit in with Ruby conventions. `def create?`


This will throw an error as it doesn't match a method name.

Once you make the change below, it should be `create?`.


@danrigsby It turns out that modern browsers are smart enough to use the variable name for stack traces and don't need the function name.  I've also seen places where people have cut and pasted functions renamed the function variable and neglected to rename the function :angry:


nice


外形材料 笔误, 改一下吧  :)


额外的引擎推进器   引擎提供的额外推力
这样似乎更好一点


@jacekkopecky I would accept a pull request that adds the option to suppress with `autoscroll: false` in the various movement methods. Please ping me on it if you're interested.


Como o Coelho tinha me chamado atenção anteriormente, é bom manter consistência na utilização de tabs ou whitespaces para edentação. O trecho nas linhas 216-220 usa whitespaces enquanto o restante utiliza tabs.


also enabled hw vsync for teh lulz


I just realized that this check is redundant. Checking for context should be enough if / when we support multiple devices per context. I will remove this and make a new commit.


문제에 입력이 중복되지 않는다고 제시되어 있었음.
입력에 중복이 있다면 set 을 쓰면 안돼~~


@jordonias Apologies for being pedantic, but the current line of code actually reads:

`var proc = spawn(process.execPath, args, { stdio: 'inherit' });`

You're seeing a difference when running the above compared to this:

`var proc = spawn(process.argv[0], args, { stdio: 'inherit' });`

...which suggests that the second line is finding a different node executable than the first, no? Do you have multiple versions of node installed? Can you determine which one is being used in the second case?


tested and working


Pra mim, fora os whitespaces, tudo certo e rodando direitinho.


This is a nice ordered list you got here


holaaa


Comenzamos


NSDateFormatter is looking for the Transaction.h file.


Looks good


Doctype?


is it fixed now?


I don't have the printer with me to check, actually.  I'll be able to start looking at this again next week.


I'm curious -- would boost::scoped_array<> solve the problem? Or are we trying to avoid using boost?


You can use the Enumerable#any? to shorten this up... Or perhaps none? if it exists... :)


windows fucks them up (at least old versions of windows used to) not sure about new ones.


It shouldn’t be.  Just completely remove the import statement for transaction.h

> On Apr 15, 2015, at 10:19 PM, Matt Larkin <notifications@github.com> wrote:
>
> NSDateFormatter is looking for the Transaction.h file.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/natemann/PlaidHTTPClient/commit/6cce0348e8b557ee4b7879250df23d7358847bf9#commitcomment-10748934>.
>


same


thi si sok


I'm going to put this back into the code since I use it to do console testing in chrome, I'll take it out before the final release so please don't take it out again.


你好，感谢你的代码。新手求指点。在使用你的代码的时候出现了错误，使用自定义flashview时，报错说无法初始化view，弄了好久也不知道该怎么解决。具体错误为：
Write access not allowed during rendering (\data\data\null\cache\uil-images)
    at com.android.ide.common.rendering.RenderSecurityException.create(RenderSecurityException.java:52)
    at com.android.ide.common.rendering.RenderSecurityManager.checkWrite(RenderSecurityManager.java:544)
    at java.io.File.mkdir(File.java:1305)
    at com.nostra13.universalimageloader.core.DefaultConfigurationFactory.createReserveDiskCacheDir(DefaultConfigurationFactory.java:100)
    at com.nostra13.universalimageloader.core.DefaultConfigurationFactory.createDiskCache(DefaultConfigurationFactory.java:81)
    at com.nostra13.universalimageloader.core.ImageLoaderConfiguration$Builder.initEmptyFieldsWithDefaultValues(ImageLoaderConfiguration.java:580)
    at com.nostra13.universalimageloader.core.ImageLoaderConfiguration$Builder.build(ImageLoaderConfiguration.java:559)
    at com.example.flashview.ImageLoaderTools.initImageLoader(ImageLoaderTools.java:49)
    at com.example.flashview.ImageLoaderTools.<init>(ImageLoaderTools.java:23)
    at com.example.flashview.ImageLoaderTools.getInstance(ImageLoaderTools.java:28)
    at com.example.flashview.FlashView.initUI(FlashView.java:122)
    at com.example.flashview.FlashView.<init>(FlashView.java:101)
    at com.example.flashview.FlashView.<init>(FlashView.java:90)
    at sun.reflect.NativeConstructorAccessorImpl.newInstance0(NativeConstructorAccessorImpl.java:-2)
    at sun.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:57)
    at sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
    at java.lang.reflect.Constructor.newInstance(Constructor.java:526)
    at

谢谢！期待回复。


@tawdle No need to apologize. Thank you for your time.
I'll double check tomorrow if I listed the proper lines I'm not on that machine currenly.
I'll also look to see if it is finding a different version of node.


Since you are looking for an exact match, why not do
UPPER(COALESCE(fname, ' ', lname, ' ', dob)) = UPPER(COALESCE(?, ' ', ?, ' ', ?))
or if you must have highly granular check, at least
UPPER(COALESCE(fname, ' ') =UPPER(COALESCE(?, ' ')) AND ...


Yeah, want to merge.


Now it's working.
Previously every time I click play image it's redirected to root of manga but with some message in Spanish roughly translated as "This series have been been removed by the request of the creator". Something like that. And it's affected to all title.


Typo in commit message: "zerozlipboard" [sic] (too many Zs)


@michaeldouglas E ae cara! Porque seu controller começa com lower case?


SDKJAK


When assigning to `LANG_SCNG(on_stop)` function pointer I get an invalid write:

```
==23424== Invalid write of size 8
==23424==    at 0x850903: zend_restore_lexical_state (zend_language_scanner.l:269)
==23424==    by 0x8526E5: compile_file (zend_language_scanner.l:621)
==23424==    by 0x6CFFF5: phar_compile_file (phar.c:3312)
==23424==    by 0x8C9ABE: zend_execute_scripts (zend.c:1365)
==23424==    by 0x81105C: php_execute_script (main.c:2468)
==23424==    by 0x992E7B: do_cli (php_cli.c:967)
==23424==    by 0x99410B: main (php_cli.c:1334)
==23424==  Address 0x7249a50 is 0 bytes after a block of size 208 alloc'd
==23424==    at 0x4C2745D: malloc (in /usr/lib64/valgrind/vgpreload_memcheck-amd64-linux.so)
==23424==    by 0x808AA8: ts_allocate_id (TSRM.c:257)
==23424==    by 0x8C7250: zend_startup (zend.c:686)
==23424==    by 0x80FD12: php_module_startup (main.c:2058)
==23424==    by 0x991BA4: php_cli_startup (php_cli.c:419)
==23424==    by 0x993F98: main (php_cli.c:1314)
```

Help appreciated.


Is this something related to `ts_allocate_id (TSRM.c:257)`?


close #16


Nice! I'll load these tonight.


Thanks! Will look into it. :-)


So.... this all happens on the USB thread. Cancel's called marking transfers, and then we immediately block on the completion conditions, preventing the finished handler from ever being called.


Is changed, thanks a lot for review.


Thanks.  I suppose "original DOH tests" would be an accurate name.


Dude, selectionName isn't privated. Thats your bug. 100% garuntee it. I told you that in chat. You need to private the damn variable!


What happened to pull requests!? :scream_cat:


If looking to show the latest results, wouldn't it be cleaner to let the database GROUP BY with LIMIT do the work?


I'm not pretty sure about if the issue is going to be fixed anytime soon, but I just saw these comments from HakuNeko's developer.. It seems to me submanga list is going to be pretty dead for a long long time...
![snapshot4](https://cloud.githubusercontent.com/assets/11966520/7173274/85080e14-e3c2-11e4-99f2-2250c9d7b795.png)


It worked for me... Thanks!


TeamCity Demo Green :: Develop Build [Build 1.1.6-32](http:/24.246.71.137:8111/viewLog.html?buildId=78&buildTypeId=DemoGreen_Build) is now running


TeamCity Demo Green :: Develop Build [Build 1.1.6-32](http:/24.246.71.137:8111/viewLog.html?buildId=78&buildTypeId=DemoGreen_Build) outcome was **SUCCESS**
Summary: Running Build time: 00:00:06


TeamCity Demo Green :: Develop Build [Build 1.1.6-33](http:/24.246.71.137:8111/viewLog.html?buildId=80&buildTypeId=DemoGreen_Build) is now running


TeamCity Demo Green :: Develop Build [Build 1.1.6-33](http:/24.246.71.137:8111/viewLog.html?buildId=80&buildTypeId=DemoGreen_Build) outcome was **SUCCESS**
Summary: Running Build time: 00:00:07


Did you just remove this entire branch and remake it?


...line 18 of bl-graphics-pipemenu that is.


eeesh probably, thanks. was dealing with addon hell and impatiently made this fork.


Thanks Taylor (good to see you here!). Let's try it.


TeamCity Demo Green :: Develop Build [Build 1.1.6-34](http:/24.246.71.137:8111/viewLog.html?buildId=82&buildTypeId=DemoGreen_Build) is now running


TeamCity Demo Green :: Develop Build [Build 1.1.6-34](http:/24.246.71.137:8111/viewLog.html?buildId=82&buildTypeId=DemoGreen_Build) outcome was **SUCCESS**
Summary: Running Build time: 00:00:05


@psoloway @jbondpdx we've been disabling these builds in travis-ci to try to ease the load on that service.


Thanks :)


Found it: https://github.com/gruntjs/grunt/blob/7f3c26b06c8039d1cad3a3de9bde9aec3637e768/lib/util/exit.js


Thanks


Didn't realize I was looking at a specific SHA, so the file is probably gone now.


Learning? Seriously? You created a ROM with PACPerformance, added an about section and claimed it as your work. That's not learning that's stealing. And you also removed the copyright header and relicensed the app to GPL v3 which is illegal. And BTW you can't relicense the linux kernel to GPL v3 it's not your work. Learn about licenses first before write/steal any code.

And why you are using my code then if they are so shit?


rax retry tests


- `same-value-x-y.js`  and `same-value-x-y-empty.js` are equivalent
- `same-value-x-y-null.js ` has a negative test that, according to this naming scheme, belongs in the as-yet-uncreated `not-same-value-x-y-null.js`
-  Missing file: `not-same-value-x-y-undefined.js` (the choice of what `undefined` is not is a little arbitrary, but taking a cue from the `null` tests, you could at least have `Object.is(undefined, null)`


we'll edit this soon to let the student see advisor info
(I didn't check for that when I edited it last, but I just made it functional with a few edits).
Also, add a  line at the top that says something like "firstName, here are the appointments you have created.", so the student knows what page they're looking at


Gotta love the bundle


Fixed in 931c86d8228026a1e11c8c9960cbf0fc6d556eed, added a tag


It would be great to also modify the CodeGeneration template. @keesd @sfmskywalker


this looks like another prime candidate for rebasing.
I will keep saying things like this until it is second nature :)


What about Redis.withShardsAndSentinel ? I'm not a fan of putting the methods inside Brando since the Brando actor is just a direct connection to Redis, I find that confusing. I suggest this :

Redis.apply(...)
Redis.withShardsAndSentinel .apply(...)
Redis.withShards.apply(...)
Redis.withSentinel .apply(...)


It appears that you missed updating the logparser.js file....


thanks for that. so this appears that there is some misuse of the Parse SDK going on here. your person model object (especially in a subclass of PFObject that implements the PFSubclassing protocol) should not be also implementing the FXForm protocol. these are two separate concerns that are being mashed together and causing this crash.


Glad it helped a bit.  I think the maintainers suggestion to tune this first and then run a sprt test on fishtest makes sense.  If the speedup helps you pass that's ok.  In the end all that matters is that it's stronger overall for whatever reason.


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

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


@yrashk Might be a dumb question (or perhaps just plain ignorance on my part). I tried the code, it runs the init/0 function, but there's something odd when trying to use the ets table (looks like it doesn't get created). Here's a sample run:

```
$ iex -S mix
Erlang/OTP 17 [erts-6.1.2] [source] [64-bit] [smp:4:4] [async-threads:10] [hipe] [kernel-poll:false]

Starting on_load
Finished on_load
Compiled lib/ex_rfc3986.ex
Generated ex_rfc3986.app
Interactive Elixir (1.1.0-dev) - press Ctrl+C to exit (type h() ENTER for help)
iex(1)> RFC3986.parse 'http://www.host.com'
** (ArgumentError) argument error
        (stdlib) :ets.lookup(:rfc3986, :grammar)
    (ex_rfc3986) lib/ex_rfc3986.ex:42: RFC3986.parse/1
iex(1)>
```

Any thoughts?


This is a filler address. Keep it here?


Why the jump from 1771 to 1776?


@dudekm Chyba nie testowałeś czy działa…


@dudekm pełna przestrzeń zbędna


I think it's worth having a test via legacy Test::Builder also, not just Test::Stream?  Even if it's the same thing underneath, having a test that's closer to what legacy code would do in the wild is a good idea.


Sure, as long as we have one "entry point" for the api, I think it's fine


Do you return `:ok` in the init function? Does the table you created has
:named_table and :public attributes?

On Thu, Apr 16, 2015 at 1:06 PM, Marcelo Gornstein <notifications@github.com
> wrote:

> @yrashk <https://github.com/yrashk> Might be a dumb question (or perhaps
> just plain ignorance on my part). I tried the code, it runs the init/0
> function, but there's something odd when trying to use the ets table (looks
> like it doesn't get created). Here's a sample run:
>
> $ iex -S mix
> Erlang/OTP 17 [erts-6.1.2] [source] [64-bit] [smp:4:4] [async-threads:10] [hipe] [kernel-poll:false]
>
> Starting on_load
> Finished on_load
> Compiled lib/ex_rfc3986.ex
> Generated ex_rfc3986.app
> Interactive Elixir (1.1.0-dev) - press Ctrl+C to exit (type h() ENTER for help)
> iex(1)> RFC3986.parse 'http://www.host.com'
> ** (ArgumentError) argument error
>         (stdlib) :ets.lookup(:rfc3986, :grammar)
>     (ex_rfc3986) lib/ex_rfc3986.ex:42: RFC3986.parse/1
> iex(1)>
>
> Any thoughts?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/marcelog/ex_rfc3986/commit/a3ebc17003702c339549da9ca574f1873fee24a1#commitcomment-10765020>
> .
>

--
Y.
My company is hiring: https://trustatom.com/jobs


app doesn't appear to be available in the context, for some reason


yes, working on that already

> On 16 Apr 2015, at 22:05, Zoltán Lehóczky <notifications@github.com> wrote:
>
> It would be great to also modify the CodeGeneration template. @keesd <https://github.com/keesd> @sfmskywalker <https://github.com/sfmskywalker>
> —
> Reply to this email directly or view it on GitHub <https://github.com/OrchardCMS/Orchard/commit/2cfcdfaa8b8db0096294a34abb78e6c869551e84#commitcomment-10765003>.
>


@MacDada wcześniej nie działało już z tymi wyjątkami. Gdy wpiszę błednego usera pokazuje się:

Fatal error: Uncaught exception 'Dudekm\Blog\Entity\EntityNotFound' in C:\xampp\htdocs\blog\src\Dudekm\Blog\Entity\PdoUserRepository.php:62 Stack trace: #0 C:\xampp\htdocs\blog\src\Dudekm\Blog\Controller\UserController.php(26): Dudekm\Blog\Entity\PdoUserRepository->findByUsername('kkkkk') #1 C:\xampp\htdocs\blog\web\admin\index.php(20): Dudekm\Blog\Controller\UserController->singleUser('kkkkk') #2 {main} thrown in C:\xampp\htdocs\blog\src\Dudekm\Blog\Entity\PdoUserRepository.php on line 62


##### Deployment available online:
#### - [app.js](https://s.brightspace.com/apps/ipa/dev/2bcc2b8c073e337929514fe9e909b05d56082766/app.js)
#### - [appconfig.json](https://s.brightspace.com/apps/ipa/dev/2bcc2b8c073e337929514fe9e909b05d56082766/appconfig.json)


> same-value-x-y.js and same-value-x-y-empty.js are equivalent

That's what happens when you "save as", instead of "mv"

> same-value-x-y-null.js has a negative test that, according to this naming scheme, belongs in the as-yet-uncreated not-same-value-x-y-null.js
> Missing file: not-same-value-x-y-undefined.js (the choice of what undefined is not is a little arbitrary, but taking a cue from the null tests, you could at least have Object.is(undefined, null)

Changes inbound


##### Deployment available online:
#### - [app.js](https://s.brightspace.com/apps/ipa/dev/7d24616316edda3a06ae11140f1bb6efbcb1c2ab/app.js)
#### - [appconfig.json](https://s.brightspace.com/apps/ipa/dev/7d24616316edda3a06ae11140f1bb6efbcb1c2ab/appconfig.json)


Wow, some cpp guy is using Java now !


We need to keep it until there is a different way of allocating coarse page tables. (which, granted, should happen soon)


Nit: it appears that the checkbox is selected by default. The descriptions of this and the following test could be clarified for this.


Yeah, same here - so many r's. At first I didn't even see what was wrong, it just looked strange ;)


What about /dev/null? Or the first file in the list?


Great!


It's tricky for stuff like this for public review if you pushed the first commit, asked for review and then later put in the second.  Rebasing after review changes the reviewed history.  For something like this, I'd make a "(rebase later)" comment in the commit message and squash it after review, before merging.

But if you have little commits like this before asking for review, then, yes, please squash them first.


Everything looks in place. Here's the code:

```elixir
  def init() do
    IO.puts "Starting on_load"
    :rfc3986 = :ets.new :rfc3986, [
      {:write_concurrency, false},
      {:read_concurrency, true},
      :public,
      :named_table
    ]
    my_priv = :code.priv_dir :ex_rfc3986
    grammar = ABNF.load_file "#{my_priv}/RFC3986.abnf"
    true = :ets.insert :rfc3986, {:grammar, grammar}
    IO.puts "Finished on_load"
    :ok
  end
```


##### Deployment available online:
#### - [app.js](https://s.brightspace.com/apps/ipa/dev/073faac4004dd43ea1d12faeb1fab89a15da2088/app.js)
#### - [appconfig.json](https://s.brightspace.com/apps/ipa/dev/073faac4004dd43ea1d12faeb1fab89a15da2088/appconfig.json)


##### Deployment available online:
#### - [app.js](https://s.brightspace.com/apps/ipa/dev/7d8efa991aa695fcde9dc88d087fc92571d124ee/app.js)
#### - [appconfig.json](https://s.brightspace.com/apps/ipa/dev/7d8efa991aa695fcde9dc88d087fc92571d124ee/appconfig.json)


thanks!!


Please make a new spec just for this one change instead of reusing the existing ones. I want to make sure we still have coverage for the default case as well.


@dudekm No to komunikat mówi Ci, co jest źle.


Question: why is push_notification_enabled always false in this loop? I haven't seen the UI yet, so I'm not sure how it should behave for existing notifications.


Did you mean `...-messages` (with trailing **s**) ?


As noted in my SO comment, looking at the removed comments above, is it possible this may have broken closeClass? It's not working in my implementation now.


``c.infoAccess = Object.create(null);`` ? in that case we may not to check ``__proto__`` ever.


Actually it's really good point, good job. I've committed additional optimization for this https://github.com/gogotanaka/Rubype/commit/8459ea5f574018988ffde115929c5a8050c642a2

compared by running `bundle exec rake bm`


Check with Morgan (or whoever your resource was) to see if this list is still current.


I just mailed the original author some patches some days ago and he was very responsive. But he actually asked for a test-case that demonstrates that the notice can be triggered. I have since then left that for myself and the weekend, but if you have and more context and how to trigger that, I would greatly appreciate some of such information.


Would it be worth using move semantics here to avoid making a copy of the `AnyPublicKey`?


## This is a markdown file


I don't understand why bother with `final`.


@psoloway I'm done looking at this for now. Ping me after changes get made, but this is a lower priority than apt.


This should be `OpenSSL.crypto`.


##### Deployment available online:
#### - [app.js](https://s.brightspace.com/apps/ipa/dev/4fe47b23a6897e86658b4faeeaf83d7228c3edc8/app.js)
#### - [appconfig.json](https://s.brightspace.com/apps/ipa/dev/4fe47b23a6897e86658b4faeeaf83d7228c3edc8/appconfig.json)


##### Deployment available online:
#### - [app.js](https://s.brightspace.com/apps/ipa/dev/af877bdb5e8ca46a35d8f553afbbc88eadfafb0d/app.js)
#### - [appconfig.json](https://s.brightspace.com/apps/ipa/dev/af877bdb5e8ca46a35d8f553afbbc88eadfafb0d/appconfig.json)


Hi Darius, are you a developer?


This untested patch has been replaced with a new and improved untested patch here: https://github.com/reaperhulk/urllib3/commit/c6d1e7f5aa76c0522d9d84e9b9ad757b7f4d8a22


##### Deployment available online:
#### - [app.js](https://s.brightspace.com/apps/ipa/dev/4adb3b0db2cdbab025bd167410ec2c2ee5f9b512/app.js)
#### - [appconfig.json](https://s.brightspace.com/apps/ipa/dev/4adb3b0db2cdbab025bd167410ec2c2ee5f9b512/appconfig.json)


```
return ephemeralValidatorKeys_.find (pk) != ephemeralValidatorKeys_.end()
  || mUNL.end () != mUNL.find (naNodePublic.humanNodePublic ());
```
Just as clear, IMHO, removes a few lines of code, including the hard-coded return, and has the same effect because of the short-circuiting `||`.


added REDME.md


##### Deployment available online:
#### - [app.js](https://s.brightspace.com/apps/ipa/dev/40ab45268bf735f7e238e076054a152ba97c2d6e/app.js)
#### - [appconfig.json](https://s.brightspace.com/apps/ipa/dev/40ab45268bf735f7e238e076054a152ba97c2d6e/appconfig.json)


This method uses the matchChannel() method to check whether there exists a channel which confirms with the request.


Nice. Whatever name is most consistent is fine by me. As long as the functionality is there ;)


compare -> compares


BTW, I don't think it is worth it to bother with `data-/x-` prefixes, because:

1. There are many more cases where `ngMaterial` does not take prefixes or alternative separators into account anyway.

2. There are more variations (e.g. using `:` or `_` as separator or using mixed separators).


Function name should match page name. goToCourseSettings


Well, this definitely breaks other things...


> It appears that you missed updating the logparser.js file....

Indeed -_-


You might want to put the values in a set before checking the size:

`a in (1,1,1,1)` => `a = 1`


look into this: http://www.codeproject.com/Articles/16177/Using-AJAX-to-simply-read-from-and-write-to-a-data


Yeah, source_dir equals the CMAKE_CURRENT_SOURCE_DIR of the external and binary_dir equals the CMAKE_CURRENT_BINARY_DIR.


Add task description.


If it bothers you that I have taken your code to study at first you should not release it.
And
I using CM performance inside settings
settings
settings
And do not worry, not coming back to study your code
And I remove this repo cheer's


Nit: I would prefer this be in closeEditor. In general, you shouldn't do this sort of thing in a Backbone view because the view should be re-rendered when it is displayed. But since this code is not behaving in that way and instead hiding stuff in closeEditor, I would prefer that this code also be in closeEditor.


Delete author annotation.


probably best to remove


Remove this empty line.


##### Deployment available online:
#### - [app.js](https://s.brightspace.com/apps/ipa/dev/d8b4849bb561c6322b75a678acc13b47db6023d0/app.js)
#### - [appconfig.json](https://s.brightspace.com/apps/ipa/dev/d8b4849bb561c6322b75a678acc13b47db6023d0/appconfig.json)


##### Deployment available online:
#### - [app.js](https://s.brightspace.com/apps/ipa/dev/ad6f30fb7b0e60f2099c6fcfdc3a85cfdc6cb7d0/app.js)
#### - [appconfig.json](https://s.brightspace.com/apps/ipa/dev/ad6f30fb7b0e60f2099c6fcfdc3a85cfdc6cb7d0/appconfig.json)


Text should be: Remove all content from a specific directory.


Thanks for the help!!!


should have been "R.layout.activity_courses"


i would refactor this into a reduce, instead of map -> filter:

``` objc
[sectionsData bk_reduce:NSMutableArray.new withBlock: ^ (NSMutableDictionary* memo, NSDictionary* section) {
  if ([section isDictionary]) {
   [memo addObject: ... ];
  }
  return memo;
}];
```


why aren't we throwing here anymore?


Must: the button text needs i18n. This is an old file...


sweet


good! I didn't see this before writing my last message. Please also include these lines in the contestadmintool (which is another, smaller portlet used from within the control panel).


Check this once upgraded


please rename to showInTileView()


camelcase


camelcaseify


No you are not studying the source codes. You are stealing them by relicensing and removing authorship.
I don't think you are smart enough to get this. So whatever. You are reported on XDA anyway.


Yay! You're awesome!


good! Don't forget to change the name of the attribute once it's camelcaseified


same here


TeamCity xBIM :: Xbim Geometry :: Release [Build 56](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1414&buildTypeId=XBIM_XbimGeometry_Release) outcome was **SUCCESS**
Summary: Tests passed: 70 Build time: 03:42:19


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


yepp, same here :)


FacilityTabGridView (evanweaver) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2098 of 2196 tests passed)

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


If handled improperly, as many mods do so, this will result in a LOT of worthless entries in the OreDictionary.

I have a reasonable compromise, however.


Update UX per suggestions from @talbs.


we can't add any default values here, right?


good, thank you!


Does this work properly against Substitute?


Check this once upgraded


added AR_lookahead option, which uses the previous value of the target, rather than the output, in the evaluation of a model's output. as such it is suited for lookahead predictions assuming access to the last measured output.


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


TeamCity xBIM :: XbimExchange :: Build [Build 227](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1415&buildTypeId=XBIM_XbimExchange_Build) outcome was **SUCCESS**
Summary: Running Build time: 00:01:20


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


TeamCity xBIM :: XbimExchange :: Build [Build 228](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1416&buildTypeId=XBIM_XbimExchange_Build) outcome was **SUCCESS**
Summary: Running Build time: 00:01:56


Seems like this assumes `checkedLive` was false before that call to `analyzeJunction`? Is that true?


You don't need your IDE files on GitHub. They'll be automagically generated every time you pull the repo.


TeamCity xBIM :: XbimExchange :: Build [Build 229](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1417&buildTypeId=XBIM_XbimExchange_Build) outcome was **SUCCESS**
Summary: Running Build time: 00:01:23


Oh wait, I get it.


The input helper has an action that fires when the user makes an edit so you don't need to use observers
http://guides.emberjs.com/v1.11.0/templates/input-helpers/


I changed these variables to better match the database structure rather than Android's weird m convention.


Ce n'est pas comme ça qu'on met à jour une ressoure. As-tu vu une seul route avec "update" dedans ? regarde ce qui a été fait ailleurs


I will definitely find a way to do it.

The previous implementation took care of the performance hit that you've metnioned, but it was broken with the Neo4j 2.2 API.

My approach was to clean up the code, and start with a naive implementation of the `isDeleted` method.


le `set_user` n'est pas nécessaire vu qu'il s'agit du "current_user"


It looks like you weren't using most of these variables, so I removed the unused ones, and the scope wasn't declared for the Realm, so I went ahead and did that for you.


Should this also check the returned body?


You need a Context object here so that you can create a Realm instance in the UserLoginRequest.

This was also the source of your NullPointerException. The realm variable was null, so when you queried the database, the app would crash.


ditto on checking the body


pourquoi créér une adresse vide par défaut ?


This is just some simplified logic.


If you know the response body then this should check it.  If not make Frank do it.


ok


Etant donné que les commits d'avant ne servent à rien, merci de les supprimer et de mettre ça sur une branche "propre"


You sure it's faster?
Here you can see each_with_index and size.times versions of Rubype compared
````
Calculating -------------------------------------
     each with index     3.124k i/100ms
          size times     3.452k i/100ms
-------------------------------------------------
     each with index     37.506k (± 4.0%) i/s -    187.440k
          size times     41.082k (± 5.0%) i/s -    207.120k

Comparison:
          size times:    41081.7 i/s
     each with index:    37505.9 i/s - 1.10x slower
````
I used following test:
````ruby
require 'benchmark/ips'
require_relative 'Rubype/lib/rubype'

Benchmark.ips do |x|
  class EeachR
    def sum(x, y)
      x + y
    end
    new_typesig :sum, [Numeric, Numeric] => Numeric
  end

  class R
    def sum(x, y)
      x + y
    end
    typesig :sum, [Numeric, Numeric] => Numeric
  end

  a = EachR.new
  b = R.new

  x.report('each with index') { a.sum(4, 2) }

  x.report('size times') { b.sum(4, 2) }

  x.compare!
end
````
`typesig` method uses `size.times` implementation and `new_typesig` - `each_with_index`


I wrote [a small test program](http://feos.mtheall.com/csndtest.zip) that plays a looping stereo sample (with panning) and displays the value of the unknownZero field. The sample plays correctly for me, and the field is always zero. Could you both test this program under your system?


A set has an order, but it is based on the key. Inserting won't necessarily add it to the "end", so popping from the end doesn't really matter. Could have grabbed the next from the start, I think. However, I don't see a reason to change this, as it does look nice this way in terms of clarity.


Je ne vois pas trop où tu veux en venir, je change la syntaxe du set_user en current_user ou la fonctionnalite du set_user n'est pas bonne ?


The only question is whether it *is* important for us to have it at the end of that list. @rfk ?


updated.


D'accord.


done.


I accidentally added a duplicate ubsan entry during the merge: https://github.com/cpi-llvm/compiler-rt/commit/4e06a85397891b6a94493c09b016b6df76c630e4#diff-5b579ba0237255aa2c66982834e4fa5eR37

This commit removed it.


Tu as une variable disponible dans tous les controllers qui défini l'utilisateur actuellement connecté : `current_user`


Wow.  I will review and update tonight.


It doesn't -- this test was re-enabled in another commit 7e444003992ec1ccd91ee48ea9e58ac8e4e326f8


Quand on crée un compte on met toute les informations dès le départ ?


Ok merci


This is unnecessary. Optional.get() will fail with NoSuchElementException and a message.


Oui j'ai voulu distingue mes routes, j'ai corrigé directement en ;
 - PATCH 	/users/:user_id/:address_id(.:format)
 - DELETE 	/users/:user_id/:address_id(.:format)


If this is browser specific add a rule only targeting the browser and version! Changing this item to dashed creates an issue when zooming on a page leaving open space in between the caret. It should remain solid


Target firefox only for the version you experience the issue! See www.browserhacks.com


You can also use the flexible array member:
const wchar_t           chars[];
clang doesn't have problem with it and it seems to work in MSVC as well, altough MSVC issues the warning 4200. So you could just remove the 0 from the square brackets.


Inline this variable


Fix for Issue #2


probabaly is not how you spell probably.  You've got that in two spots.


Huh, where did you dig that out? Indeed the changes were not what we wanted and after discussing I rejected the patch and some parts were ported by hand by Micha.
But this commit never made it into the pxarCore-2.0 (or now master) branch.

Am 16. April 2015 15:17:03 MESZ, schrieb Urs Langenegger <notifications@github.com>:
>"myscript" does not look like a good name?
>
>---
>Reply to this email directly or view it on GitHub:
>https://github.com/psi46/pxar/commit/e77b2b2ad0a7ff81210b35a69b3160a4a73708aa#commitcomment-10756829


@GlobalGISteam Please open an issue with a test case if you're seeing rendering problems on page zooming.


typo?


Autant pour moi je l'enlève, j'ai pensé bien faire.


Fixes #2


@vcunat: That was related to the discussion in #7349, so the "problem" referred here was documentation.


TeamCity MonoGame :: Develop (Win) [Build 3.4.0.210](http://teamcity.monogame.net/viewLog.html?buildId=6166&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


Typo on line 10, property name should be endLocation_lon


Ctrl-H becomes Command-H on Mac, which is a system shortcut for 'hide window'.  This is probably why we were using meta-h in the first place.

I suggest we use either 'Ctrl-Shift-H' on all platforms or put an ifdef here to use Meta-H on mac and Ctrl-H everywhere else.


null check


extra paren?


@carlosmunoz yes. I've been using that `JndiBackedConfig` and `applicationConfiguration`. Perhaps we should just go ahead with system properties implementation since I'm not aware of anyone that uses jndi config to override the value.


c'est pas bon non plus


Makes sense, done in https://github.com/cpi-llvm/llvm/commit/5c5322aa882f8792e1b3e3a20f0b45b50fdccef7


Yep, I consistently misspell consistent when writing docs


also, verify that partitioningColumns contains all hashingColumns


This should be `hasPartitioningRequirements && partitioningColumns.get().isEmpty()`


Editing sqlite shouldn't be hard, no?

Right now I have an instance of it running on Heroku with Firebase as the config storage since Heroku doesn't persist local file writes. Firebase lets you edit the datastore directly over a web interface. I've even used it to store trigger definitions with the limitation that callbacks can't be serialized and stored (e.g. BotCommandTrigger).


Ah, you're right. Turns out zero is fine, though, so I've updated it to be zero.


Good call. I've gone ahead and clarified this further.


Good stuff.. Since _filters_ do not appear to be used here, might as well comment it out for the sake of compatibility.
Thanks @dominikborkowski, please submit a _Pull Request_


Did that ever work? or should we remove that?


Not to nitpick, but perhaps this is better suited for an `md` file? It can go in `overlay/README.md`


Tiny comment - either put the `end` call at the start or the end of both subexpressions, as it's just a bit easier to read.


@caindy Should this line be commented out too?


@danielmcormond yes


Are you becoming active on the gridster project again?  I just merged many of the community pull requests into my fork, I'd love to work toward unifcation


Style note: this should be CSRWhitelist.


For posterity: this didn't blow up because of the conversion to int: https://github.com/brianmario/mysql2/blob/master/lib/mysql2/client.rb#L35


Done: https://github.com/cpi-llvm/llvm/commit/499cda88bce89bb5700bfe0f6de315a4b485b902


saw approval from javajolt
at https://github.com/nicorevin/riak-python-client/commit/4dca64c8539bb1e2f2ac90f619b7e75810927bd6


merging nicorevin/riak-python-client/CLIENTS-349 = 4dca64c8 into borshop-integration-402-CLIENTS-349


nicorevin/riak-python-client/CLIENTS-349 = 4dca64c8 merged ok, testing candidate = b2e4574a


Thanks :+1: it works......


test


Great catch, thanks!


Please remove the `**` from here. The styling will be built-in to the pattern rendering them.


- Can you use a [trait](https://github.com/thoughtbot/factory_girl/blob/master/GETTING_STARTED.md#traits) for the Factory?
- Can you rename the method something like `is_a_graduate?` or `is_graduate?`
- Assert using the `be_true` matcher. You can read about it [here](https://www.relishapp.com/rspec/rspec-expectations/v/2-2/docs/matchers/be-matchers#be-true-matcher). It is a bit faster.

@louiedp3


Closes issue #44


thanks @zendawg !


Resolves #111 (falsly stated it resolves #103)


replace there ~~font-family~~ and ~~font-size~~ with **@include font-regular(15px)**


TeamCity MonoGame :: Develop (Win) [Build 3.4.0.210](http://teamcity.monogame.net/viewLog.html?buildId=6166&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 481, ignored: 6 Build time: 00:14:48


replace there ~~font-family~~ and ~~font-size~~ with **@include font-bold(7px)** at the begin of *&__text*


Maybe rename to `ResponseProcessingComplete` or `ResponseCallbackNowComplete`, or etc...


This is wrong. It only suppresses the warning and keeps the code as incorrect as it is. The correct way to call `isdigit()` or another `<ctype.h>` function on a `char` is by casting to `unsigned char`.


@Mnkras :+1:


I guess the trait is preference, maybe talk to others to hear their thoughts.


@aembler SOMEONE GOT MY REFERENCE


life changing stuff


nice cleanup and very good improvements to the code.
I miss an example how to use the new logging functionality.


@sedouard Making some adjustments to handle `gui.App.crashRenderer` (see my commit message). If something looks awry to you, feel free to chime in.


I think adding redundant casts like these is poor style. This is C, not C++, and I don't see any point in cluttering up code to make it compile as C++.


:-1: It was better as `void const*`, `std::size_t`. Rationale: the pointer/size does not impose any strict requirements on how the data must be stored.


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


@Ombridride asked it explicitly on the TodoList, he wanted to remove those chuncks from the mapgen.


TeamCity xBIM :: XbimExchange :: Build [Build 230](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1418&buildTypeId=XBIM_XbimExchange_Build) outcome was **SUCCESS**
Summary: Running Build time: 00:01:06


@danielmcormond I fixed this in github's ui, good catch, and thank you for looking


@amar59 done with my review. checkpoint complete.


>ВНИМАНИЕ ИЗМЕНЕНИЕ В ИНТЕРФЕЙСЕ >Добавление методов в интерфейсах повышают минорную цифру релиза (вторую слева); изменение поведения уже существующих интерфейсов повышают мажорную цифру релиза (первую), т.к. изменения обратно не совместимые со старым кодом проектов; исправление работы функционала, ошибок, поднимают последнею цифру релиза (третью), эти изменения не влияют совместимость с кодом проектов.


Fixing the width of this button is realy accurate?

We may expect some layout bugs with translations.


remove ~~*@font-face*~~s below, its ambiguous! Therefore use mixins **font-regular(size)**, **font-bold(size)**, **font-semi-bold(size)**.


There is some more commented debug code here.


Initialize pointer


Thank you very much.


Updated.  thanks.


1570-EnergyPlus-8-3 (kbenne) - x86_64-MacOS-10.9-clang: Build Failed

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


updated.  thanks.


Done.


I think I had it like that because I needed nwjs to return a non-zero and this was the only api I could find. However that was before I did TAP output. I'm assuming that testem fails still right?


This is not a formatting fix, but a typo/bug, should we bring this in with a separate patch?


ceci peu être remplacer par

`has_many :product_categories, dependent: destroy` cherche `dependant` sur [cette page](http://guides.rubyonrails.org/association_basics.html)


I don't understand why this is necessary? This file should be kept in the source tree. Github pages branch contains the generated html.


1570-EnergyPlus-8-3 (kbenne) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

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


it is unsemantic place ~~**p**~~ elements in label, therefore use **span**


Probably it changes color and underline props of the `headline__title` :(


Long story short, this change should be attributed to @mtodat. See https://github.com/facebook/folly/pull/132 for details.


Oh man. We're using both camel case and underscores?


I can't see it without magic around brunches, but I guess


for readibility and future maintenance of code, the internal loop in
echo -n "" > $origin/$supernova;for i in $origin/$andLight;do grep '\.' $i >> $origin/$supernova;done
may also be removed
echo -n "" > $origin/$supernova | grep '\.' $origin/$andLight >> $origin/$supernova


I see. The thing is that it's much more difficult to document stuff after some time passes, because one forgets too much.


There are no such block ~~**week-calendar-radioform**~~, do you mean **week-calendar** ??


1570-EnergyPlus-8-3 (kbenne) - i386-Windows-7-VisualStudio-12: Build Failed

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


Compiler warning: comparison between signed and unsigned integer expressions


Merge has obviously failed. nbdiff went wrong. I will do a rollback and retry the merge again.


I think these two casts can go away entirely.


1570-EnergyPlus-8-3 (kbenne) - Win64-Windows-7-VisualStudio-12: Build Failed

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


@sodre I'm surprised the syntax works https "github.com:joyent" syntax works.
I expect
```git+https://github.com:joyent/node-docker-registry-client.git```
should be "github.com/joyent":
```git+https://github.com/joyent/node-docker-registry-client.git```

Are you will to fix throughout, test and resubmit?


package hohman;

import java.util.ArrayList;
import java.util.Random;
import was.GameBoard;
import was.GameLocation;
import was.Move;

/**
 * An example sheep
 * @author dr
 */
public class Sheep  extends was.SheepPlayer{

    private static Random rand = new Random();
    GameBoard board = null;
    Move direction = null; // direction we're taking
    //private GameLocation getPasturePosition;//gets pasture position??

    @Override
    public void initialize() {
        // you cannot call "getGameBoard" in the constructor, as the
        // game board is created after all the players.
        board = getGameBoard();
    }

    @Override
    public Move move() {
        // if direction is not yet set, choose a random one
        GameLocation myLocation = this.getLocation();
        ArrayList<GameLocation> PastureLocations = board.getPasturePositions();
        ArrayList<GameLocation> ObstacleLocation = board.getObstaclePositions();
        int i = 0;
        GameLocation nearestPasture = PastureLocations.get(0);
        GameLocation nearestObstacle = ObstacleLocation.get(0);

        float x = 0;
        float y = 0;

            direction = new Move(nearestPasture.x - myLocation.x, nearestPasture.y - myLocation.y);//rand.nextFloat() * 2.0 - 1.0, rand.nextFloat() * 2.0 - 1.0);
            // scale direction to lenght and quantize it to maximize distance covered.
            direction = direction.scaledToLength(getMaxAllowedDistance()).quantized();

    ArrayList<GameLocation> trk = new ArrayList();
            trk.add(getLocation());
            //trk.add(new GameLocation(0,0));
            trk.add(nearestPasture);
            removeVisualizations(); // remove all previously set tracks
            visualizeTrack(trk);

        return direction;

 }
     private boolean isPathDouble(GameLocation a,GameLocation b){
        move();

         GameLocation mySpot = this.getLocation();
         ArrayList<GameLocation> PastureLocations = board.getPasturePositions();
         GameLocation closePasture = PastureLocations.get(0);

         direction = new Move(closePasture.x - mySpot.x, closePasture.y - mySpot.y);

         return isPathDouble(mySpot, );

        }

        }


We didn't measure it, but we have the exactly same logic in old `kpm restore` and the perf was good in our previous use of `kpm restore`


"Crate.IO is a bronze sponsor of"


`Also update nw-runner to not swallow diagnostic lines (even though they don't appear in the final output--something to investigate)`

Yeah so this is something I noticed -- nw-runner isn't swallowing the output, its testem that isn't displaying it. I haven't been able to figure out why. You can see if you run just node nw-runner it shows up just fine.


Looks like it brakes Chef 11.


Change this into

#ifdef WITH_ECC
#include <openssl/ec.h>
#endif


See, I told I was zeroed out!


hi


1570-EnergyPlus-8-3 (kbenne) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


These two should probably be in AMDatamanAppController, rather than BioXAS specific due to their prefixes.


@leahpadgett this line is what loads the google query (or webpage) in the webview browser. To find the reason your code will not load the query add a breakpoint here and see if it is ever called (executed). If if is not, then either use NSLog statements or breakpoints within this method to see what steps are really happening. Does that make sense? I can also point you to code that does work - would you like that?


copied directly from cal's example on the cpan module.

=3OE=


Should we bother using StreamAdapter here as well?
i.e. we should probably be more careful about what `CEF` thread we read the stream from (esp. if it isn't a memorystream)

Bill


figures


This should be ```length+1``` as ```GetEnvironmentVariableW()``` returns the length in characters WITHOUT zero terminator. Also, the second call below will probably not work correctly because if your buffer is too small the contents on return are unspecified.
Note that this is all info from [MSDN](https://msdn.microsoft.com/en-us/library/windows/desktop/ms683188%28v=vs.85%29.aspx), which could be wrong and it might actually work as-is...

>If the function succeeds, the return value is the number of characters stored in the buffer pointed to by lpBuffer, _not including the terminating null character_.

and
> ```nSize [in]```: The size of the buffer pointed to by the lpBuffer parameter, _including the null-terminating character_, in characters.
>If lpBuffer is not large enough to hold the data, the return value is the buffer size, in characters, required to hold the string and its terminating null character _and the contents of lpBuffer are undefined_.

(emphasis mine)


copied directly from cal's example on the cpan module

=3OE=


@leahpadgett done with my review. checkpoint complete. press on!


Are we going to remember to manually update this every time we create a release branch? I think we can later improve this by extracting the tag from the cloned git repo.


This is what I am thinking. What do you think @ChrisChinchilla

In addition to attending dotScale 2015 (The European Tech Conference on Scalability) the Crate.IO team is also an [bronze sponsor](http://www.dotscale.io/partner/crate). Join us for a live demo of Crate at our booth, ask the Crate team questions, or just come by and say hello.


I haven't tested this, but I think it is better to change naddr's type to `struct rb_sockaddr_storage *`. This will also reduce casts in callers (since the pointed-to object must actually be a `struct rb_sockaddr_storage`).


The above two casts are not redundant and are an acceptable way to silence `-Wcast-qual`.


Yes, testem does report the failures. I'll have to check to make sure builds with failed tests do error out on Travis as well.

I'll have to figure something for testem's output. Without those lines, it's impossible to know the cause of any failed tests. `ember test` does it correctly. Surely, there's a way =).


Add the partial name option to "Available fields to search" in search for debtors.


The relevant part is this here:

> If lpBuffer is not large enough to hold the data, the return value is the buffer size, in characters, required to hold the string **and its terminating null character**

Look at what I'm doing:
I give it 0 (and a 0-pointer too, I must add), to test how big the buffer must be (which includes the space for the 0-terminator), then I allocate the space, then I call the function again to get the actual string.

This all *should* work, no?


I'll make that change..


To be precise, clang does have a problem with this when -Wpedantic is passed (which it is for LLILC).


can_create only affects what happens if the object does not exist on disk.  That is, can_create = true causes get_object_context to return an empty ObjectContext instead of a null result.  It doesn't tell you anything about the cache.


I'll test that change, thanks.  I thought about that, but wasn't entirely sure about it...


Class CommandTemplate does not have a Docblock comment.


waiting_for_unreadable || waiting_for_degraded?


Hmm, maybe just here and then we (or I) can do it separately? That'd keep your patch self-contained.


And I can't speak for a real Windows, but this works exactly as planned with Wine.


##### Deployment available online:
#### - [app.js](https://s.brightspace.com/apps/ipa/dev/f2636795a67649879d06a14924a008c260108894/app.js)
#### - [appconfig.json](https://s.brightspace.com/apps/ipa/dev/f2636795a67649879d06a14924a008c260108894/appconfig.json)


##### Deployment available online:
#### - [app.js](https://s.brightspace.com/apps/ipa/dev/6ceb5ab99965b0bb294bbe0fc447ef23de028d89/app.js)
#### - [appconfig.json](https://s.brightspace.com/apps/ipa/dev/6ceb5ab99965b0bb294bbe0fc447ef23de028d89/appconfig.json)


Fixed, please check the updated patch


Did this belong in an earlier patch?


Would BitArray work instead?


Oops sorry!


Oh (sorry for all that noise here), and if it *didn't* return the space needed for the \0 in the case that the string doesn't fit, there would be one case where you couldn't differentiate between success and failure: if the string fits exactly into the buffer.


``createSecurePair`` have only 4 arguments. fifth argument is ignored (!)

Yes, you mean ``options`` in ``new SecurePair()``, but ``createSecurePair`` do not pass it. It will be nice if you fix ``createSecurePair``.

P.S. Do you know how I can fix MY code that use SecurePair obsolete API? what is equivalent?


Interesting. I have somehow assumed that C++11 has this as a standard feature when the C99 standard supports it.
Ok, leave it as it was, LGTM


ah yes, it's late. MSDN says one thing above (it returns the size *without* 0-terminator) and then a few lines later it says it returns the length *with* 0-terminator. Sometimes WIN32 is strange ;-)
I should go to bed now :)


I see what you did here… :smile: I now get why you wanted to do this fix yourself :laughing:

![commitswar](https://cloud.githubusercontent.com/assets/216089/7191914/82aee6e0-e491-11e4-804d-e4725ef32e59.png)


Also, generally, blocked is not a good name for this.  ObjectContexts *can* be blocked, but it means something else.


Unused import. Use `com.google.common.base.Charsets` for charsets.


PHPCS: Expected 1 space after closing parenthesis; found "\n        "


It says two things because there's three separate cases:
- The variables doesn't exist: return 0
- The variable exists, but the buffer is too small: return number of characters required (including the \0)
- The variable exists and fits the buffer: return number of characters required (without the \0)

So you can basically check that the call succeeded if != 0 && return value < sizeof(buffer).


I wish there were a mechanism to limit the maximum time of the pending operations. If it waits for longer than a pre-configured time, it should either fail or allocate a new connection (i.e. similar to ThreadPool's RejectedExecutionHandler)


Hannibal node attributes only please


Flatten dat pyramid doe


Consider using GetCustomTypeInfo() from the tests instead.


What do you think of #141 then?


Just seeing this commit now. This will only work when you're creating a new record. `create_with` will be ignored if person already exists. You can leave `person = Person.create_or_update(person_params)` and still use `if person.valid?` code below.


I had

#export DYLD_LIBRARY_PATH=$DYLD_LIBRARY_PATH:/Users/jenkins/mosek/7/tools/platform/osx64x86/bin

in the .bash_prfile, which is what was part of the failure. Regardless, I have to actually add the export to the jenkins script. If you make another pull request it should hopefully work.


You missed `file:reload` :p


(Please write it in the commit message if it's a special case like that, otherwise it's confusing)


This should probably be in Hannibal since we now default to [60300[(https://github.com/bloomberg/chef-bach/blob/hdp2.2/cookbooks/bcpc-hadoop/templates/default/hdp_hbase-regionserver-initd.erb#L90) at this time.


@socketpair what kind of use case do you need? io.js does work on regular `stream.Duplex()` instances, when they are passed as a `socket` option to the `TLSSocket`.


Ich könnte auf ein fehelndes Komma hinweisen, aber ich bin ja für Sprachwandel und weniger Kommas gehört auf jeden Fall zu den Sachen, die die deutsche Sprache sich vom Englischen abschauen sollte ;)

Na gut. Im ersten commit.


Hi there,

This plugin was developed for a talk I gave a while ago demonstrating how to develop a plugin. I have not worked on it since! In your required version you have '2013051409d'  - there shouldn't be a 'd' there. What is the problem you are experiencing?


`your`


@bublik42 hmm.. maybe you'er right. I'm gonna revert it, thank you for letting me know.

Aside from that, Rubype become 8x faster after this commit https://github.com/gogotanaka/Rubype/commit/6be724653b37ee11e434abd1aa7fe5cef564c81e


@kfranquenouille ce commit ajoute mes répertoires pour compiler, en gardant la compatibilité avec ce qu'il y avait déjà. Normalement tu ne vera pas la différence, au pire, ``git revert 1f7e96d480812249efc11f9e6137a6ac1848b5e2``


In principle there's a separation but the two got mixed up sometimes. In case of scalac, the separation works fine and the sourcepath option is working correctly. I can't think of a reason why scaladoc would be different but apparently is.


Thanks! Care to provide an update for the directvnc man page as well?

Cheers


+2, exactly what I would have done


Do we need a struct for { Type, DkmClrCustomTypeInfo }?


So, the whole reason I use a set is *because* of the ordering by key - it actually is helpful to grab from the largest key. Reasoning as follows:

 - First note that (in terms of the final result) it doesn't actually matter which order we look at junctions. It's a bit like bubble sort, in that we keep looping until there are no more changes, i.e. we hit stability. Intuitively (though I haven't proved it) that's independent of the examination order - there's only one minimal set of variables, and if we loop until there are no more changes then we've hit that minimal set.
 - Because we propagate changes *backwards*, we want to start at the end - in theory, this will mean that we begin with the junctions least likely to change once we've scanned them. If we started at the beginning, any junctions in the middle of the function with altered liveness could flow changes backwards all the way to the start. While junctions at the end can change, they're less likely to (I've empirically verified this by taking things from the beginning - it gives a 25% slowdown...which is still faster than the previous implementation).
 - This is where the set is valuable - we are able to *always* choose the last-most junction, because junction ids (seem to) correspond directly to the order of their location. This contrasts to the vector-as-a-stack approach, which could end up choosing a bad order to reduce junctions in.

One note about EXIT_JUNCTION after looking at this again, I don't think we actually need to add it to the junctions to be considered at all (as long as we start by adding all inblocks to the block workset) - it has no live vars and is not the entry point for anything so it will only be considered once.
(I'll do this in a separate PR)


Hey Andy. I don't. I don't think there's a way around that, and I've ended up disabling SMTP message saving.


actually the way I got around this was to use stderr for all of the diagnostic output. testem doesn't swallow stderr. That's what their [doc says](https://github.com/airportyh/testem#processes-with-tap-output) anyways. Feels like a hack but thats what I've been doing.


ooo nice catch :D


Hey, I hope this is helpful for you:

You don't need the comparison with `T`, as `clean` itself is a boolean expression. So you could write:
```R
if (clean) {
  MWX_data <- MWX_data[MWX_data$Sample != "Reference",]
  MWX_data$X <- NULL
  rownames(MWX_data) <- NULL
}
```
Is there a reason why you leave the extra else if and else there?

Ah, and also, `T` is a global constant and can be redefined, so when writing scripts I usually try to use `TRUE` and `FALSE`. As this is more explicit, I tend to notice that me writing the constant is redundant altogether (`if (booleanVariable == BOOL_CONSTANT) ...`). This is usually the case, when I write them not as a parameter, but in general control flow. When the variable has a good name, this lends to more natural sounding statements, because the "weird" equality expression is not there anymore.


Okay!

I will remove the piece of code as it serves my purpose. Thank you.


So some logic as simple as 'line starts with '#' -> stderr' should display.


Ah ouais j'avais pas vu le message d'erreur à propos de la licence, autant pour moi. Pourtant le module compilais sur un autre arbre de sources.


new commit


The class ProjectController has a coupling between objects value of 14. Consider to reduce the number of dependencies under 13.


Avoid unused parameters such as '$template_id'.


Well, yeah, :)


Class TemplateController does not have a Docblock comment.


Where is the capability for volume_raid_create_cap?


@ExSport thanks :)


lets change it back to 500?


I just noted another change:

 diag();

used to be a no-op (at least to the user: it did print nothing)
now it emits a single line with just a #

#


Just saw the comments, I'll be here all week, lol


Is type ever null? If not, move assert to top of method.


does the `, {}` in `fetch()` do anything here?


ImmutableArray<bool>?


Should this return ImmutableArray<bool> instead?


Do the other arithmetic operators need to be added too I wonder?


I don't believe that's available in the ResultProvider.


I'm all for better naming :+1:


Oh, and powerups obey the Target property


agreed


:heart: you @tianon


Consider extracting common helper method from this property and the Type property. Then this could be get { return GetType(_member.GetOriginalDefinition()); }.


Ooohhhh baby. Welcome to my world.


back to 500? i cant seem to edit your game without cloud9.io why?


Nice


You could use a DataProvider (like is one in SchemaChangesTest) here to provide true/false


The `CefResourceHandler` doco says `Class used to implement a custom request handler interface. The methods of this class will always be called on the IO thread.`

I'm not quite clear at the moment on how `StreamAdapter` would fit here?


In preparation for another future patch.


@hamiltont  Actually, **THIS** is nice.


/app.js


Yes. The target is `self` in gen 5.


FYI, it's a good time to close #1338 - I think it might have helped avoid a lot of this debugging time by printing out a warning that it needs to own the directory. It's probably valuable to test the directory ownership *after* calling chown, because the call will fail quietly in some installations.


Weird, I thought I had already done this. But looks good.


Corrigir


Received an error here (action_settings was unknown) - needs to be resolved.


penis


``adpcmS=0 adpcmI=0 unk=201#####``  ( ``#####`` somewhere between ``80000`` and ``9FFFF``)
so I guess the problem is system dependent ? this is bad since I rely heavily on that field for audio playback...


这个以前是咋工作的？


ACID part:
http://paste2.org/fBXEDfX0


Revert


we don't need this


Not at the current time, but that is something we would like to do in the future (cc @trevorrowe).


some tests failed:
failure: http://buildbot.bos1/builders/python-client-test/builds/335


we don't need the `"[]"`


Ну и говно. Иди лучше Биллу Гейтсу яйки лижи


this is correct, however, to check for undefined you can do: `datatype === undefined`
and to set a default you can do: `datatype = datatype || 'base64';`


saw approval from fadushin
at https://github.com/basho/yokozuna/commit/59106f7207c1ba61caf4937afe30780ba211423d


merging basho/yokozuna/bugfix/stop-index-creation-loop-on-bad-data = 59106f72 into borshop-integration-487-bugfix/stop-index-creation-loop-on-bad-data


basho/yokozuna/bugfix/stop-index-creation-loop-on-bad-data = 59106f72 merged ok, testing candidate = eddfc81c


TeamCity MonoGame :: Develop (Win) [Build 3.4.0.211](http://teamcity.monogame.net/viewLog.html?buildId=6168&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


Quick question - if the try fails, is it going to restart the import and fail again each time the app starts?


@rhizome ,  if closeclass uses a target that is provided in the ajax response -- then it won't ever attach. I updated the SO answer w/ an example


dfssfdsf


Looks like we can remove `tile__foot--4`. This class is meant to specify how many lines we want in the footer. Seeing as we always have, only 1 line, we can remove that class because the default assume 1 line.


I would expect here:
```
return zipUtil.encode(false, mem_io, temp_dir, dataType);
```


Seeing as we're changing the footer below to a single line, this can actually remain as `has-foot` -- this lets the tile know there's a footer.


Just inline this


Bug fix triggered by rightscale/praxis/issues/167


is the unknown type nullable?


It looks like at most, this will be a 2 line footer? If so we should change instances of `tile__foot--4` to `tile__foot--2` for this part of the template (i.e. the front of the tile)


TEst


Again, we can change this to `tile__foot--2` because we'll, at most, have a 2-line footer.


I've moved registry of the Action/ActionInfo to AMAppController (with other actions) and the AB to AMDatamanAppController. 35a2e51


Some variant on this problem seems to have returned in the latest builds. However now I can't reproduce it as easily (because it seems to happen much more slowly than in the previous report).
The symptom is again that after either some number of songs, playlist reloads, or station changes, the volume creeps upward. Calling BarUiActCallback(BarUiActVolReset) puts it back to normal.


I have no idea why https://github.com/JordanSparkes/SEP-Group-Assignment/network is showing this push as it is but meh, it's only a readme so it's nothing important


@stustison does the claims order truly matter here? In JDK8, the assertion will expect claim4, but get claim7 instead.


last argument, add "buffer":
var zipped = this.getZippedInstance(masterName, instanceName, precision, 'buffer');


Thanks!


看不出来这里都修改了什么，github的diff有问题？


damn, its "nodebuffer"
http://stuk.github.io/jszip/documentation/api_jszip/generate.html


Maybe we should add `fallback_write` functions for all the UARTs, to make sure this message makes it out to the host?


I think we actually want to force a 1-line title here so there's space for the address and phone number. Let's change this to `tile__title--1`


all tests pass:
success: http://buildbot.bos1/builders/test-yokozuna/builds/1088


Lets be sure not to enable this right now. I think there are plenty of unaligned accesses going on when we read SBP messages unfortunately.


I suggest taking the code and separate it into a boolean returning method called isAnotherSessionScheduleEdited() or something similar and just calling this method inside populate()


Actually, I don't.
Resolved.


Hi Michael,

May I ask you why did you change the line "command = gunicorn app:app -b localhost:8000" in flask_project.conf?

It used to work properly the old way. Now, it fails because it can´t find anything in that path.
Maybe I missed something.

Thanks.


@Dream-Master Don't worry about it, I can handle it. :)


done.


TeamCity MonoGame :: Develop (Win) [Build 3.4.0.211](http://teamcity.monogame.net/viewLog.html?buildId=6168&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 483, ignored: 6 Build time: 00:14:32


SPaG zero


Looking very good my friend ! Cheers !


Duplicate line with the one below?


FacilityTabGridView (evanweaver) - Win64-Windows-7-VisualStudio-12: OK (2175 of 2196 tests passed)

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


[style] The above two if's should just be an if-else block.


Efficiency suggestion: change the order in which conditions are evaluated: check first if it's not read only and then go into isValidToSave()


if does it mater, I use simply <p> element or <span> one with "display: block" property?


Its built with gcc, not VS like the other submodules, and I would need to somehow put that into the build system and build it before ffmpeg. Not really worth the trouble.


SPaG should be


Currently, the lock isn't always held here.  This is what caused the buildbot failures.  See.

```
[ 1264.508131]  [<ffffffffa020e31f>] spl_panic+0xbf/0xf0 [spl]
[ 1264.508179]  [<ffffffffa03928e2>] zfsdev_getminor+0xb2/0xf0 [zfs]
[ 1264.508210]  [<ffffffffa0330a96>] zfs_zevent_fd_hold+0x36/0x80 [zfs]
[ 1264.508241]  [<ffffffffa038b9d6>] zfs_ioc_events_next+0x36/0xc0 [zfs]
[ 1264.508308]  [<ffffffffa03926cf>] zfsdev_ioctl+0x4af/0x5a0 [zfs]
[ 1264.508316]  [<ffffffff811ba555>] do_vfs_ioctl+0x2e5/0x4d0
[ 1264.508321]  [<ffffffff811ba7c1>] SyS_ioctl+0x81/0xa0

```


Agree, this is a better interface and the general approach for this patch is good.  We've just got some corner cases to run down.


remove


Thanks for getting git sorted out. (that’s sounds odd).

I’ll review it shortly. Press on!

—
Sent from Mailbox

On Thu, Apr 16, 2015 at 11:30 AM, Diego-F-Aguirre
<notifications@github.com> wrote:

> So I guess because I was on a different branch other than the master it
> didn't push the changes I did on that branch. Here is the commit for
> checkpoint 27 called "Refactored user name and captions"
> https://github.com/Diego-F-Aguirre/Blocstagram_V2
> On Tue, Apr 14, 2015 at 5:30 PM, Diego Aguirre <diego3884@gmail.com> wrote:
>> Weird, I'll check tonight when I get home from work.
>>
>> --
>> :::::::::::::::::::::::::
>> Diego F. Aguirre
>> 973-885-0724
>>
>> http://www.linkedin.com/in/diegoaguirre3d
>> http://diegoa3d.com/
>>
>>
>> On Tue, Apr 14, 2015 at 3:58 PM, dothework <notifications@github.com>
>> wrote:
>>
>>> @Diego-F-Aguirre <https://github.com/Diego-F-Aguirre> not much here to
>>> look at. did all the files get committed?
>>>
>>> —
>>> Reply to this email directly or view it on GitHub
>>> <https://github.com/Diego-F-Aguirre/Blocstagram_V2/commit/47afd8ff26a35cfff85aa62e33433ecde1c8b076#commitcomment-10722273>
>>> .
>>>
>>
>>
> --
> :::::::::::::::::::::::::
> Diego F. Aguirre
> 973-885-0724
> http://www.linkedin.com/in/diegoaguirre3d
> http://diegoa3d.com/
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/Diego-F-Aguirre/Blocstagram_V2/commit/47afd8ff26a35cfff85aa62e33433ecde1c8b076#commitcomment-10759967


We've talked about that earlier. You are free to give any URL here. (I think it must be https://)


@KamranMackey I dont worry. Its good for me to keeping forge version 1291 and update forestry. I can wait for the weekend np.


Sorry, late day.


@edmistj @azturner This helps explain why we're not seeing Rock impact statistics for a bunch of systems (that have less than 100 people). Obviously this filter was put here on purpose -- if so, lets also remove all the older impact records where that count is greater than 0 but less than 100.


Sure, makes sense


fast-forwarding develop to borshop-integration-487-bugfix/stop-index-creation-loop-on-bad-data = eddfc81c


// code @Francois
public String genererData (Tree t, SymbolTable st) {

		return null;
	}
	public String genererInstruction(Tree t, SymbolTable st) {
	//StringBuffer rep = new StringBuffer();
//cas ou c'est une affectation
	 // TODO
	 return null;
	}
	public String genererfonction(Tree t, SymbolTable st){
		StringBuffer rep = new StringBuffer();
		//Symbol symbol = st.getSymbol(t.getNoeud().getIdentifierTDS());
		//StringBuffer rep = null;
		    rep.append("return:");
		    rep.append("DEALLOCATE()");
		    rep.append("POP(BP");
		    rep.append("POP(LP");
		    rep.append("RTN()");
	  // cas c'est label est --main-- se referent a la fonction d'origine----
		    rep.append("main:");
		    rep.append("PUSH(LP)");
		    rep.append("PUSH(BP)");
		    rep.append("MOVE(SP,BP)");
		    genererInstruction(t,st);
		    // TODO
		    genererAffectation(t,st);
		    //TODO
		    genererExpression(t,st);
		    //TODO
	// appel de la fonction -------
		    rep.append("CALL()");
		     //TODO
		    /*-----
		     *
		     */

	    //TODO
	return null;
	}
	}


Here's another way:

    colors = ['Green', 'Red', 'Black']
    for color, customer in zip(colours, customers):
        my_shop.sell(color, customer)

It scales more easily.


wrap params


@scusi You're right, should have created an explicit example. But, you have it there in the comments for the New function.
  client, err := govt.New(govt.SetUrl("http://some.url.com:port"), govt.SetErrorLog(log.New(os.Stderr, "VT: ", log.Lshortfile))


mon commit ne passe pas why ???


ah ok. noticed that, thought it was just a stylistic choice. ill change


Hey, thanks :)


Solves issue https://github.com/luvit/luv/issues/138


Well, yes, but... that doesn't mean you want to necessarily have ReadResponse() block other IO thread processing.

If you set bytes_read to 0, you can CefPostTask(TID_FILE, <actually read more data>) or Task.StartNew( ... => { readMoreData...; callback->Continue(); } ) or something similar.

Bill


Good job mate!


Use string.length


This is an example of a minor change that probably doesn't need to be reflected in the changelog. It is a docs only change for proper english. Downstream users would not need to be aware of this change when upgrading react-bootstrap as one of their dependencies.


Maybe use Markdown links instead of just putting in url.


Use the default `CMAKE_INSTALL_PREFIX` variable to set the installation directory.


`Ember.observer` over `.observes` remember, we don't want to use prototype extensions in addons


Yeah. I think you're right. It should work since it's been added to the virtualenv. But now that I think of it, the old command should work either way. I'll update. Thanks!


after this commit is submitted we should bump to 0.2.3


2 small things, the LGTM


I haven't tested it yet, but it looks good to me :smile:


Just check a-z


Looks good.


Plugin provided by views should not be redefined here. This should be removed.


Should this method really take the cl parameter? It is only valid to pass the class the method is declared in, but that class is already available using decl.symbol.classSymbol.


This seems to break the use of integer custom values, as described in the documentation? Fixnum does not respond to empty?.


Marshalling onto the `File` thread does make sense from a performance point of view, just adds a whole nother level to the complexity.


No! cregCount is signed32 bits! DAMN YOU!


There's a stray `i` here.


Where does that come from actually? Never seen it in a Python project.


looks good


Should we keep the example/test separate at least after a __name__ == __main__ statement? Anyway this is too hardcoded for general use.


test


Sadly, it still exists. Try: http://www.ratebeer.com/beer/new-belgium-tour-de-fall/2791


what if the instance has no service keys or just one service key...  we should have a test that verifies that all is good. No?


This seems valid in hardened/current/master branch too.


Uhm, but the icon that what previously used by ".mdi-maps-local-airport" is now showing what was the icon of ".mdi-maps-local-library". There was some kind of random rotation, I guess. Feature or bug? :)


Heck yeah. :frowning: async stuff is LOADS of fun.


did you intend to overwrite anything that was there previously, each time?


This too, would get overwritten between runs/tests.


Get rid of this


[![Nocturne: Palaces](http://api.artsmia.org/images/7399/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=7399)


Need the write lock acquired around this.


Need the read lock around this.


Possible, I dunno. Change it back if you feel like it; I only made the change to get some extra symmetry.


```
2015/04/16 13:33:32 [WARNING] keke142: illegal argument while handling click: java.lang.IllegalArgumentException: Slot out of range [0,45): -1
2015/04/16 13:33:32 [INFO] keke142: [rejected] WindowClickMessage(id=0, slot=-1, button=0, transaction=96, mode=1, item=null)
```
this is the error