## Before
![screen shot 2015-05-11 at 16 22 40](https://cloud.githubusercontent.com/assets/319055/7568315/f5d28c14-f7f9-11e4-9544-769d02598c79.png)

## After
![screen shot 2015-05-11 at 16 04 19](https://cloud.githubusercontent.com/assets/319055/7568312/f2912268-f7f9-11e4-9c2f-5bcdbe4607d2.png)


A diff button may be added later. I have just removed it for now to not confuse the user.


If `NSTimeInterval` conforms to `Comparable`, you don't need to implement this, only `<`.


The package looks great; good work!


big changes...


I'm always apprehensive of bloating code with dependencies unless I need them for the core reason they were made for.


Issue detected: Interpolation in msgid.

I'm going to fix them (more occurrences) and create a PR with regenerated PO files.


I also want tests for `>=` and `<=`.


you do realize that this version (NGECore2) is our code from last year ???


thanks!


You might notice that the code below is repeated 3 times (except for the operation).
You might want to do something like `@operator = ['+', '-', '*'].sample` which will get you a random operation (look up [sample on array](http://ruby-doc.org/core-2.2.0/Array.html#method-i-sample)).
to get your `@answer` you will want to `send` the values to your `@first_num`. (Check out [Object's send method](http://ruby-doc.org/core-2.2.2/Object.html#method-i-send))

Here is the idea:
```
@operator = ['+', '-', '*'].sample
@answer = @first_num.send(@operator, @second_num)
```

This way you can avoid the switch case and also can expand your operations in the future fairly easily.


I agree with that notion.  I keep going back to `sudo make me a pod` and the idea that certain people will be authorized to run particular things in policy.  I think your idea of multiple SCCs that are controlled by policy addresses the issue I had with my design re: knowing exactly how a pod got permission X if the SCC was the one who was identity aware and simply let it pass admission.  This would keep it pointing at an exact SCC with escalated permissions to validate against and even allow a tiered version of 'new admin can creates pods that do Y for customers but they can't give a customer a "rule the world" pod on accident'.


There seems to be a typo here. Also, you are immediately overriding the functions only parameter.


You've already required this in `main.rb`, no need to double require.


be a bit more explanatory about what this Cstruct.t in dh_secret is?


likewise, worth an abstract type over this session id cstruct to avoid them getting mixed up with other buffers?


@Dioul there is a problem when compiling luajit for x64 arches... tells me "pointer size missmatch" so i've compiled it once on a x64 arch and download those files when needed... not really a clean way and depends on the files to be available on my server... do you have any better ideas on how to handle this?


Don't use commit comments like this.
Commit should tell about itself. be careful next time.


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


Yeah, any number followed by a period will render as a numbered list, so doing it this way beats having to update a cascade of line numbers with a weird change log, in the case we insert a new number in the middle.

Do you prefer them to be manually numbered, though?


Yes, for a major version it should be changed. There is no benefit from having result value that is too generic.


Is mbcamera actually a thing? I couldn't find other references to it anywhere...

Basically I think we should avoid as much as possible having different styles of dependencies during the cmake transition, and it seems like getting mw-scons-tools to do the exact same thing as cmake is the best way to accomplish that.  So maybe we should rename:

define_cmake_dependency -> define_standard_dependency
define_library_dependency -> define_legacy_dependency

Also, the only thing left using the old dependency stuff is MBCoreUtils, which I only left alone because it was header only.  It shouldn't take much to get it using define_cmake_dependency, and might even work as is.


Fixes #23


@ruebot not sure that this is invalid per se, but I've never seen a namespace end with "/#". Also, if you want to make the namespace de-referencable (which you should), you'd want a TLD on the hostname. So, I would think you'd want the namespace to be more like `http://islandora.org/definitions/v4#` or `http://islandora.org/definitions/v4/` or `http://islandora.org/definitions/v4/models#`.

Also, the properties/classes typically will have a `rdfs:isDefinedBy` clause.


if __name__ == '__main__':     前面的把这一句语句写错以后至少浪费的我的两天时间，以后必须注意这些粗心，科学上面粗心会让你付出巨大的代价。


Something which is true about all of these mutation methods that notify listeners - they don't pass any arguments to the callback functions regarding the nature of the change. Is that by design? Something that might be improved in a future change? I suppose to a certain extent you can capture some of the pertinent details (such as the name you're observing) in a closure when you register the listener, but other things (such as which IPs got added and removed in this update function) obviously cannot. Thoughts?


Because you already checked in line #64 that either of the player is or isn't alive you know that if `p1.lives` is not less then 1, it must be `p2` who's dead, there is no point in checking it, a simple `if else` would have been enough here.


Is this supposed to be here? Everytime I have DevTools opened the debugger starts for my app, because of this line...


962deb05589490a5b0d057ed8ed11811f596a59b should fix it,
let's see what [Travis](https://travis-ci.org/jruby/jruby/builds/62091969) will say.


:cake: !


Why did you remove the try statement here?

How about 'except Exception' instead of just a bare 'except'. (Presumably the original except: is here for a reason)


yeah, I'll probably just go in and make those changes after I get MBCoreutils switched.


I don't think we really want to show the entire value array here... that could explode the logs


These lines looks like the conversion of `Duration` into `NSTimeInterval`. So, a method for the conversion should be defined at `Duration`. For example, `5.minutes.interval` equals `300`.


As good a case as any to see what this would be in my suggested procedure:
```
if hasattr(other, "__numpy_ufunc__"):  # new style
    return other.__numpy_ufunc__ is not NotImplemented
    # or, if we use setting to None or False, simply:
    # return other.__numpy_ufunc__
    # but this seems a bit less explicit.
elif not isinstance(other, np.ndarray) and hasattr(other, "__array_priority__"):
    return self.__array_priority__ < other.__array_priority__
else:
    return False
```


Heh. I was just about to bug about whether or not I did this right :smile:


Absolutely. Please e-mail me.


This would be a good place to do `p1.lost? || p2.lost?` and define it in the player class to check if `lives <1`.


But this breaks compatibility with Python2.7 ....


Uh, you're just checking "1"? Don't you mean to be checking other?


Looks like we're not checking the item at all!


you can use \$scope.totalLines


This always evaluates to False.


Thanks Brian!

On Mon, May 11, 2015 at 3:33 PM, Brian Underwood <notifications@github.com>
wrote:

> Gah, you're right, thanks! I think when I was testing this I was entering
> ?_escaped_fragment_=/all which is why it wasn't working. I've put it back
> now
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/neo4j-contrib/graphgist-cms/commit/95e855976181acf98dec27b032425e4d773eb64d#commitcomment-11135192>
> .
>

--
ryan boyd
twitter: @ryguyrg
<http://t.signauxneuf.com/e1t/c/5/f18dQhb0S7lC8dDMPbW2n0x6l2B9nMJW7t5XZs2BgXCFN5v0L0PdDL-bW3LyyXR56dGlrf5mkGST02?t=http%3A%2F%2Fwww.twitter.com%2Fryguyrg&si=5299090570608640&pi=cdc1dd2d-5db2-454f-e6d9-bf9454ab0f76>
my book: Getting Started with OAuth 2.0
<http://t.signauxneuf.com/e1t/c/5/f18dQhb0S7lC8dDMPbW2n0x6l2B9nMJW7t5XZs2BgXCFN5v0L0PdDL-bW3LyyXR56dGlrf5mkGST02?t=http%3A%2F%2Fgoo.gl%2FO7K3Y&si=5299090570608640&pi=cdc1dd2d-5db2-454f-e6d9-bf9454ab0f76>
my website: http://www.developeradvocate.com/


Why did you get rid of this line?


No it's not, just made a PR to fix: https://github.com/Polymer/core-menu-button/pull/19


Let me fix a few things


Updated.


Is this affected by vid_displayfrequency?


Can you fix the provided comments?


We can split it, but basic idea about how to split you need to provide me. It would for like if for first line break, split it into two something like that.

Currently single line of address from salesforce is putting into first line of freshbooks address.


great! that stuff has bean missing for a long time !


dive bombing? or kamikaze bombing? because they ought to dive, they are not level bombers, just not crash.


I would rename "manage" to "display" or "print" or anything meaning that the code is going to display the items you found


arregledo sincronizacion de hoja con el codigo del peticionario


why do you import `
+from __future__ import absolute_import, division, print_function, unicode_literals` ?
And i would prefer not to add a new depencies "pytoolbox" for a simple things as a colorize filter, even if i recognize the utility for your personal project ;)


Okay yeah that's right, i'm just going to go ahead and add that back in now..


`PROJECT=SAMSUNG_GOLDEN` => `PROJECT=SAMSUNG_CODINA` ?


Awesome thanks.


Good points. I'll make those changes.

On Sun, May 10, 2015 at 1:46 AM, Alexander Riccio <notifications@github.com>
wrote:

> I think you mean GetModuleHandle(NULL)?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/google/UIforETW/commit/5a56336e57056ba46c76c2996457edfb8545b6d0#commitcomment-11122454>
> .
>

--
Bruce Dawson


careful: varying naming for the last version number: `MICRO` vs `RELEASE` vs `PATCH`.

I think `ADIOS_VERSION_RELEASE` might be undefined here and below.


@oisian fixed as of 39ae6c7


@gpoulin , with this change, `pipe_reader`'s definition is exactly the same as for `pipe_writer`. Can you add some docs to ChainFormat, and also some history why it was introduced? :)


No, it returns all resolutions ignoring the frequency, but on Windows, the resolution settings are independent from the frequency settings. Not completely sure though. Also video modes are requested only once and ignore `vid_restart`s.


Oops, I missed this.


gfjfghj


mainquit() tambien funciona, pero de acuerdo que main_quit es mas estandard.


I do not like "manage" too... But I already have a function ```printItem```, which display the item.
```manageItems``` loops on a list of items, and check if it should be printed, depending on the command line options.
Any idea?


I need to fix my lexer to allow 1..2


You have to check the value of 3 for ppc it could less or more. To understand, look the implementation of NR division specially the initialisation with the reciprocal, find the SIMD IBM version of this reciprocal, and the native precision. Every iteration double the precision.


more reason not to lock the file and directory and finally make them available for all, otherwise removed your open source license !


@blmorris did you find a better fix for this?


Change up/down cursors


FYI this class is linking with simd_vector/trait.hpp try to understand how it is connected.


Why are these volatile? There doesn't appear to be anything special about their use that would require it


Alright


You are right, readlink() doesn't return cannonical path. I replaced it with call to readlink_and_canonicalize.


too aware of my recent eagerness to merge, i checked this out. with that socket_id regex, it's still legal to do 'helloworld12.13'


When working with spring-data-cassandra 1.3.0-BUILD-SNAPSHOT, I am getting an exception on startup with the message "Attribute 'repositoryBaseClass' not found".  I found the method signature change in CassandraRepositoryFactory easily enough to allow the build to complete, but I'm not exactly sure where to add in the repositoryBaseClass attribute.  Any hints to what else might need updating in that project?  Or is there a new annotation I need to add to my @Configuration class to specify an application repositoryBaseClass?


or is that ok?


would `^\d+\.\d+$` be better?


unless, of course, the ruby GC is potentially going to come into our stack frame and break our stuff


And that will probably not work with --root :D


Yes, I think so


Okay, fixed that in next commit.


But that whats trace is supposed to do here considering the context, right? Line 88 already uses this.


Oh yes, you have to type no when they ask you if you want wide charged tees
during the ncurses extension installation
On Mon, 11 May 2015 at 23:23, Cameron Banga <notifications@github.com>
wrote:

> I'm a complete PHP newb, but attempting to follow steps above on latest
> Yosemite (with zero PHP/Pear custom config) and getting the following
> errors when installing ncurses:
>
> configure: error: Wrong ncurses lib version or lib not found
> ERROR: `/private/tmp/pear/install/ncurses/configure --enable-ncursesw=autodetect --with-ncurses' failed
>
> May be something on my end, or maybe a step missing above? Not sure, but
> wanted to toss in my experience here.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/remzr7/Slacker/commit/ac9e6fbfee212aaed825a0858fa826dc81cb8225#commitcomment-11136234>
> .
>


Why not just `fotografde/sms`? whats the xi?


remind me - why is the double call necessary?


Yeah. It will be in dfa.text


you shouldn't rely on the text of the exception to test if a file exists as there are no guarantee that it won't change. What are the other exceptions that sequenceParser.browse can raise ? You can also use python os.path functions to test the file/directory


对


Could you remove all the junk files? Just to make it more presentable and not so messy.


 is in private use range, please remove it.


https://developers.soundcloud.com/blog/offset-pagination-deprecated, this is the new way to fetch the data.


I would put the resetting logic (line 33 till 38) into it's own method so that it's less busy


Really like the early return ;)


ok.. fair enough


Hm, I don't really like this style of dealing with optional features. Why not use the standard idiom, i.e.
```
{ enableGTK ? false, gtk3, ... }:
...
buildInputs = optional enableGTK gtk3 ...;
```
That way, features like GTK support don't get accidentally enabled. (Note that due to the use of `callPackage`, a `null` default for inputs like `gtk3` means nothing.)

Also, `mkEnable`, `shouldUsePkg` etc. now seem to be cut&pasted all over Nixpkgs.


DBC: A return null is enough here, no need to create a new object.


Ruby's GC is designed such that it owns the stack, which is why we have to initialize the VM in main (it records the high address that way).  It needs to mark any VALUEs on the stack before sweeping.


I would like to replace the two line by:
```
filename += os.path.relpath(directory) + '/'
```
But this does not work when I try to display relative paths with recursion (_-R_ option):
```
python2.7 sam-ls.py -R --relative-path
```
Because only the current item knows its relative path in the hierarchy.
Maybe I can split instructions with ```if recursion```?


Oh wait it isn't. eh, so hard to read code without types :p


Example: complete ESP tasks
Using ```title```: Complete E S P Tasks
Using ```properWithAcronyms```: Complete ESP Tasks

The reason I added a new function instead of updating was to maintain backward compatibility.  But yes, this isn't strictly speaking a case type.  On that note, what is referred to as 'title' case is actually, commonly referred to as 'proper' case.

Other potential problems with the 'title' case function are that it will remove underscores and dashes/hyphens and it will not handle lower case acronyms either.  That will lead to undesirable results for any string containing those characters or acronyms.

I would suggest a rewrite of the 'title' function so that these other problems are resolved, in addition to acronyms.


```read bytes < /proc/sys/kernel/random/poolsize || bytes=512```?


Make sure your follow the style guide for spacing. Your code is really hard to read because it's all bunched up (and sometimes really long lines of code).


新浪的大体框架


Sure, but im not 100% whats junk and what isnt, can you point them out?


Agreed... I upgraded heroku-toolbelt this morning to find this same thing... Using rbenv as well...


it's generally more pythonic to write

    if args.inputDirectories:
        inputDirectories = args.inputDirectories
    else:
        inputDirectories.append(os.getcwd())

as it will test if args.inputDirectories is empty


Just on a related note, wasn't there some kind of DSL we discussed at some point? Do you know what happened to this? By no means a need to change it, just wondering in general.


use `\A` and `\Z`.


https://github.com/glowing-bear/glowing-bear/blob/master/js/glowingbear.js#L99-104

Is that what you're referring to?


Yes I agree with you, I hope this is temporary.
The reason is here: https://github.com/mikrosimage/sequenceparser/blob/master/src/common.i#L24


Almost certainly this is me, from either t1000 or santo.
On May 11, 2015 12:24 AM, "Daniel Sank" <notifications@github.com> wrote:

> The changes to ghz_fpga_server weren't done by me and I'm certain I
> wouldn't have written a commit message "Minor fixes". Is someone committing
> from a repo with my username?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/martinisgroup/servers/commit/b9c4527eeaf0e280c7d2885545af6fe050e062a6#commitcomment-11128156>
> .
>


Also, having dozens of these `mkEnable` flags seems very hard to test/validate. Do we really want to support every combination of configure flag?


So, sorry!
On May 11, 2015 8:56 AM, "Peter O'Malley" <ooomalley@gmail.com> wrote:

> Almost certainly this is me, from either t1000 or santo.
> On May 11, 2015 12:24 AM, "Daniel Sank" <notifications@github.com> wrote:
>
>> The changes to ghz_fpga_server weren't done by me and I'm certain I
>> wouldn't have written a commit message "Minor fixes". Is someone committing
>> from a repo with my username?
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/martinisgroup/servers/commit/b9c4527eeaf0e280c7d2885545af6fe050e062a6#commitcomment-11128156>
>> .
>>
>


Instead of `item.getFolder() + '/' + item.getFilename()` it would be better to use os.path.join as it is portable


that is fantastic, thank you! :sparkles:


Ah OK


I don't really know either but a lot of them seems to be debugging files
produced by the compiler, like HPR.ast.hs ect.
Remove most of them, and if you're not sure keep them until someone says
otherwise.

2015-05-11 17:54 GMT+02:00 SoamJaws <notifications@github.com>:

> Sure, but im not 100% whats junk and what isnt, can you point them out?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/DavidLindbom/DATX02-15-29/commit/3c0d3fd8281f8ea853c00d4e773863d93fd5b6e5#commitcomment-11136932>
> .
>


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


Ohh join will be my friend!


Is there a reason why `return new BigNumber(number + '');` isn't used here?


This branch is almost 4 years old, time to get rid of it too ?


Does this capture both the "/" and "/index.html" use cases?

("/" and "/index.html" is a blacklist; is that exhaustive?  Are there other index pages you may wish to prevent from loading here?  index.php or other stuff?)


We might want to check if the id of given record is null at first. This way we can update existing entities instantly


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


This is a comment.


Now everything looks well defined!

For consequent naming, you might want to change the `@VERSION_MICRO@` to `@VERSION_PATCH@` in the (C)Make files and the `configure` script, too.


I am not sure you need the `+=` as for each iteration items is reset to the empty list `[]` (line191). If you want to add each result of the browse function to items, you have to move `items` declaration out of the loop.


Interestingly enough, I can "brew uninstall ruby"... it still works and there's no dependency warning...


@Robinlovelace @nikolai-b
Thanks you two. I now have added a separate dynamic tab for the lines data - see here: https://github.com/npct/pct-shiny/commit/a165b5b59619ffafaa1871dc2628b95ef7ac9fc1


Remove this?


If-branch and else-branch are the same, they should not...


Actually, in PHP, it should be `\z` I think. http://php.net/manual/en/regexp.reference.escape.php


thanks sir we pushed our project simulation and coding,,,,,,


Remove this, too? I don't want JSHint and ESLint in the repo at the same time.


yes, resubmitted.

On Sun, May 10, 2015 at 11:22 PM, Colby Aley <notifications@github.com>
wrote:

> did you mean to remove these?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/stephenwlott/blocitoff/commit/b9379e812130b8c8273b03af9528ac73165bd890#commitcomment-11127488>
> .
>


Sorry in ruby it's again different (probably because of the Oniguruma engine). `\Z` matches the end of string excluding the new-line. `\z` is the one you want.

http://ruby-doc.org/core-2.2.2/Regexp.html#class-Regexp-label-Anchors


> I've updated the commit message with more info about how I did that.

Awesome :-)


No, I haven't found a better fix. I did try to build the unix port once recently and it still didn't find the GNU readline without help; my google searches for a better fix have proven fruitless so far as well - there are quite a few older references to software not finding GNU readline, but no obvious solutions. I do wonder if it is still a problem in newer versions of OSX (I am running 10.7.5, current is 10.9.x) or if it is just a broken configuration on my own very old MacBookPro. I meant to post to the forum to ask if anyone else is having problems running the unix port on OSX; should do that today.
It does seem that there should be a better way to find include headers than adding such a specific line to the search path.


Glad you found it useful!


select is specific and doesnt work on all kind of shells.and some implement it differently than others. So I would prefer to not use it at all.


:+1: That resolved the issue, I would add that small step for the unaware like myself, but seems like I have success.

Again, a total newbie to OS X side PHP like this, but after successful install of ncurses and adding the extension line to the bottom of my php.ini, still seeing this message:

```
Slacker: PHP ncurses is not available
=====================================

This app requires the ncurses PHP extension.

On Ubuntu you can try to automatically install dependencies by running
'sudo make ubuntu-dependencies'

For other platforms, or if that doesn't work, check out the README.md for more
information on installation dependencies.
```

Is there a need to restart PHP/the mac? Again, I admit I'm not good at this stuff at all, but figure if I'm having issues, could probably add this to the docs for idiots like me. :sweat:


How does this fix the brittle test? More reliable slug generation?


レビューしました。
大方問題ないと思います。


Mal la cantidad de espacios


Some (important) observatories have azimuthally dependent pointing limits, like [Keck](http://www2.keck.hawaii.edu/inst/common/TelLimits.html) for example. In my exoplanet scheduling, including these exact pointing limits makes a significant difference in the available targets. Would we consider taking functions as input for `alt_min` and `alt_max` which take one argument – the azimuth?


mal indent


tested and working


The documentation (https://cordova.apache.org/docs/en/4.0.0/plugin_ref_spec.md.html#Plugin%20Specification_config_file_element) suggests using /manifest/application

Would that work?


Yeah true. Oh PHP


Obsolete whitespace. We could also change the name of this method to "submitSubject"


가나다라


「可能としています」でしょうか。


Can we get this tagged as 1.2.1 so it can be labelled as a stable?


:sweat_smile: I have no idea what that is!! Thanks for :scissors:


If you have some time, today, or this week to chat about this, I'd be eternally grateful.

...basically what I'm trying to do is document what we are doing now in the current version of Islandora, and the prepare for what we do in 7.x-2.x.


Yep, sorry. Issue was "organisation-10" came before "organisation-9"


Smart guy :D


I force pushed the branch, new commit is here.
https://github.com/msekletar/systemd/commit/36b3314466b7d19b3d2271f607a6c1ae7505a380

I tested it briefly and it also seems to work with --root. Also, why not change follow_symlink bool? Well, I didn't want to break too many things so rather I patched in this code to handle explicitly the exceptional case.


@asamy45 Ow Fallen ![](http://i3.kym-cdn.com/entries/icons/facebook/000/000/554/facepalm.jpg)


Why not?


How about:
```c++
bool GetPayload(const uint256& txid, std::string& payloadRet)
{
    if (!p_payloaddb->exists(txid)) {
        return false;
    }
    Status status = pdb->Get(readoptions, txid.ToString(), &payloadRet);
    return status.ok();
}
```


OK, that's... I'm not going to say it's reasonable, but at least it makes sense.


Derp


we should clean the `duplicateDialogInput`  on close like we do for the theme creation.


I did not realize: printItem(s) could be a good name.


There is actually a way to solve this without an if/else statement by using:


The only change is that `t.IP` is not evaluated in case of error from `net.ResolveIPAddr`, right?

LGTM.


@iryont why remove the vim line?  I needed it because I use different tabs and spaces.


Any `Host` instance doesn't always have a (at least default) `contacts` attribute present :?


the other solution would be to rely on the Console component. hiddeninput is included in it since years


We don't have all that zones. Look at /sys/class/thermal/ we only have 5.


The spec seems to support a 404 on CRUD actions in my view.


Yes, I had intended to make it "const int" - I will make that change.


^$ would be enough for python. However for ruby lib patch make sure to use \A\z


Except it doesn't include the stream_copy_to_stream hack for some reason.. so I can't really use it.


For a fetch on a relation that does not exist, spec is very specific 404 is requires.


Please look at file changes in pull request. This was removed.


Ideally, I would recommend only using one for loop for code quality reasons. In that case you could either:
1. Check and conditionally swap number and otherNumber
2. Use a high and low watermark variables with MIN/MAX


A 403 is valid on a `DELETE` if all members cannot be deleted.


GBE+ is in flux right now :p


Hmm... `du_dot_du` is a constant in the whole system (something like `1/dt`), so this would not show up as a bug... Then, we may not need `_du_dot_du_neighbor` and `_nodal_du_dot_du_neighbor ` in `MooseVariable`.


Fix for issue #63 will be more complicated now ... you also break all my local changes, I need to merge your code :/


Yup I think so. Relying on "dl=0" somewhere in the URL isn't great, I'd rather have it be GET parameter parsing.


As far as I can see most files do not have it, so what's the point having it within one file only? Anyway, sublime editor can guess indentation based on the contents itself :+1:


Fair enough :) Makes sense.


Thank you @jmchilton


hghh


You really are as dumb as you seem.


Forgot to check in the GenericTypeModelBinder! Will sort that asap ...


our new code the Holocore is fully open source. so whats the problem ?


this throws a C2070 compile error in VS2012. probably related to this issue: http://stackoverflow.com/a/18907221


This is correct in every respect.


The standard way is to use pkg-config, but I don't think it works for readline.


I would change these to something like "The 'name' property cannot contain digits"


Has there been any discussion about this new style of writing packages? Because I'm not sure it's a good idea. For instance, this patch *triples* the size of the Heimdal Nix expression, much of it stuff like
```
+    (mkOther                         "sysconfdir"            "/etc")
+    (mkOther                         "localstatedir"         "/var")
+    (mkEnable true                   "pk-init"               null)
+    (mkEnable true                   "digest"                null)
```
that apparently duplicates all configure options. That seems a maintenance headache, because now every time Heimdal gets updated, you need to go over the `configure --help` output to check whether the Nix expression is still in sync.

Enabling every possible option/dependency is also not necessarily useful, for instance in this case, there is no point in supporting both `readline` and `libedit`.


A few notes:

- I like storing the payloads
- there is no strict need for more than one database at all, and a similar effect would be given, if keys were prefixed, e.g. `pHASH`, `tTRADEKEY`, etc.
- we should probably rather sooner than later insert a "version" flag, which may trigger reparsing, if we update the scheme
- at some point we may think about actual serialization of the data (i.e. something like https://github.com/mastercoin-MSC/mastercore/issues/283#issuecomment-72756268)


Change to "phone number introduced" to "introduced phone number"


Ah yes. Thanks.


by any chance do you think you can help me on a qcom device in case you have time? When i try to compile with GCC 5.1 breaks camera, and i dont have knowledge to fix this. thanks


>
>
> - The only way forward is that the patch needs to prove itself in areas
> where we can measure the impact, let's say depth > 5 * ONE_PLY.
>
>
Test submitted, indeed it is even stricter at depth >= 5 * ONE_PLY


please do `pushedObjects = pushedObjects.concat(change.docs)` instead. as `change.docs` is an array, it implies that it can have more then only one doc. But `pushedObjects.push(change.docs[0])` would only add one doc to the array with which the promise resolves, so docs could get lost.


Okay, I fixed that test. Still seeing the IPv6 issue with TCP BTL, but I believe that is unrelated to these changes. Not sure what happened there, but it looks like the modex URI for IPv6 isn't being correctly formed.

Thanks!


Most files do not have it because I did not maintain the entire tree but specific parts.  And I like vim.


@macbre, please create a ticket for the content team.  It looks like gamestar changed the format of the title again. All they have to do is load the source URL:

https://github.com/Wikia/app/commit/8f9e9126a853bb87b3545ddeca4f393bd201a133#diff-e13cb722bafec6ccb47c24a45718ca0fR44

note the title and update it on the expected line:

https://github.com/Wikia/app/commit/8f9e9126a853bb87b3545ddeca4f393bd201a133#diff-e13cb722bafec6ccb47c24a45718ca0fR71


First, because it was snuck in with [an unrelated PR](https://github.com/balderdashy/sails/pull/2803).  Second, because our score is not great, so if we actually care what Code Climate has to say (which is a separate discussion), we should get our score higher before posting a badge displaying it.  I encouraged the author of the original PR to open a new one specifically for the badge if he thought it was worthwhile.


Starting changover from VMEClient/vmetcpserver to DiagGuiServer message passing


The name of this method is a bit misleading. Change it to something like "isSubjectIdValid"


@Seldaek probably because this has been added in Composer after contributing this logic to Symfony and that the fix was not ported to Symfony too.


Please use more descriptive commit messages.


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


foi só para teste hehe


http://editorconfig.org

Have you tried it? Perhaps it would be good to include such config within OTClient repository.


hmm.. "over use" .. of `except Exception` ;)
you clearly and appropriately mention in the comment that you should only except on .. `KeyError` ;) q

but should'nt you even try & except on KeyError for the `del self.notificationways[notificationway.id]` (that is inside the inner loop) .. ??  I think so actually.. no ?


Maybe the OS? I'm using Centos 6 64bits
Also, Kohana as you know is Case Sensitive in the name of the Controllers.
http://kohanaframework.org/3.3/guide/kohana/mvc/controllers


Yeah and that's why it's now a simple re-usable micro lib that can be
used by Symfony, and the league folks, and composer, and we stop copy
pasting things around. I don't really expect symfony to add a dep on it
though as it doesn't really fit the monolithic model.


Hello,

I'm reaching this from [chromium/issues/detail?id=375297#c15](https://code.google.com/p/chromium/issues/detail?id=375297#c15) and i was wondering under which system and Chrome version was this tested over?

I'm experiencing the same size restriction for Blobs on Incognito windows, and i can't get your workaround to work. Apparently indexedDBs are handled in memory on Incognito mode and therefore this causes the browser to crash due more than the 2GB limit per 32bits application being allocated.

Cheers.


This was intended to normalize the behavior with a full rewrite I have in progress.  I've been reconsidering this and some other changes though, which is why it hasn't been released.  What bug were you running into?


I would rename "filename" to "prefix" because `filename` contains the prefix or the directory, not the name of the file. The name of the file is given later in `item.getFilename()`. It shouldn't impact the performance and give more readability


Not sure `sys` is used


darwin.xml seems to be only file where this entry is called.
https://github.com/xbmc/xbmc/blob/2228e7f209fbf244bb0f9da970903016395043f2/system/settings/darwin.xml#L35

Just above it calls Universal remote also, So Im wondering if the string can simply be named "Harmony remote"


Sorry mistake....for last time
Sorryyyyyy
Aage se I wll note that


I found two issues with this changes:
- if a VM (or SR) is selected, then the Search tab displays everything (like the XenCenter node).
- if two hosts in the same pool are selected, the Search tab displays everything (like the XenCenter node).

The new functionality should only be applied if the only selected node is a live host, i.e. Selection.Count == 1 and the selected object is a live Host. Otherwise, the default functionality (SearchPage.XenObject = pool or standalone server) should be preserved.


@darobin Aren't you missing an Origin header in your request? Or set the crossDomain option to true? There will still be a problem because Varnish also keeps response headers, so if someone tries to use a different Origin header, he will get a wrong Access-Control-Allow-Origin back.


I would make this `["~Next irridiate"]`, to indicate it is approximate, and not exact.


same comment than above for the hosts ;)


revert


Is this going to make things take a super long time now..?


I have added an [issue](https://github.com/httprb/http.rb/issues/224) related to this discussin ;))


Right...


Question, does getFilename return a string ? It is used later on and not converted.


Is this correct? Can't understand why form text color is #fff but in demo page https://bootswatch.com/flatly/ xxx is #e74c3c


This breaks the package when installed with pip.


prb indentation


We should pass `IConnectionSettingsValues` here instead of an index name.


On the right track! But this will overwrite the `data` dictionary you've seeded above. Should probably be changed to something like:

```
data["code"]  = code
data["description"] = description
...
...
```

After that, this should be good to go!


是的…在提交之前忘记了删了……


esse arquivo está sem extensão, por isso o github não reconhece como html. ele interpretou como um simples txt


Once you have an instance of `IConnectionSettingsValues` (passed as a param), then we should check whether there is an index already set on `pathInfo.Index`, and only infer from `T` if there isn't.


Hi there,

We've got a Pull Request open to fix this (#159).  We'll be taking a look and merging it later this week.

Keiht.


Yeah, I thought of doing that, but wasn't sure if that was just me going overboard given my experience over what another student might do. Should just trust my gut.

C

Sent from my iPhone

> On May 11, 2015, at 9:13 AM, Falko Buttler <notifications@github.com> wrote:
>
> Ideally, I would recommend only using one for loop for code quality reasons. In that case you could either:
> 1. Check and conditionally swap number and otherNumber
> 2. Use a high and low watermark variables with MIN/MAX
>
> —
> Reply to this email directly or view it on GitHub.
>


no comment


Thanks @mkustermann


So, none of those methods were mentioned, but I do like the solution! Maybe I should look at the api docs some more.

C

Sent from my iPhone

> On May 11, 2015, at 9:10 AM, Falko Buttler <notifications@github.com> wrote:
>
> There is actually a way to solve this without an if/else statement by using:
>
> —
> Reply to this email directly or view it on GitHub.
>


jQuery sets the cross-domain options automatically. Why not just return `*` for the origin for all requests and call it a day?


First, leading underscores are reserved for compiler/system libraries -- the WiFi shield library seems to me to be a user-level library. However, if you define the WiFi shield library as system-level, then that naming is fine.

Second, I think the word "begun" is crucial, because it distinguis the use of these variables, from other potential uses, such as "what is the IP and port of the local socket." Someone just seeing "_ip" and "_port"  has no idea which IP and which port it is -- local or remote.

So -- is WiFiUDP a system library in this context?


This time it worked out, sorry for the error and thanks for finding and fixing!


It went from 1:42 to 2:31 (including one new test). I think we'll live.


'cause otherwise there could be some notificationways not removed..
if a contact has as notifways `[ nw1, nw2, .. ]` and if `nw1` has already been removed from `self.notificationways` then `nw2, ..` will be missed/skipped and not removed thus..


One comment you probably considered, but seeing the force_encoding on the `encode_component` method gives the feeling that addressable would rather suppress the issue. That being said, I like this fail hard approach.


remove?


Yes: https://github.com/mikrosimage/sequenceparser/blob/master/src/Item.hpp#L66


@wysow See the [PR](https://github.com/wysow/wysow-postfinance/pull/1). I've reversed the deletion commit and picked the last improvements from the marlon-be version ...


Why did you rename the function? The `Code39VINReader` could reuse it simply by calling `Code39Reader.prototype._decode.apply(this)` right?


Please rebase it.


> but should'nt you even try & except on KeyError for the del self.notificationways[notificationway.id]

KeyError or IndexError if it would be a list (don't know by heart and doing free comments ;))


It seems that if you exclude the directories here, the recursive option, `sam-ls -Rd` will not parse the sub directories. Is it intented ?


Looks good! I'll give it a try later tonight.


I've got no idea what style guide you use here but mixed double/single quotes on this line.


Ok great. Also, why is this package not on pypi?


Done here 348fee8aaadf4de66e4b9ff72139671595a37c2a


Pregunta, ¿Este código no está duplicado? Recuerdo que para animar el reorder hacia lo mismo, quizas conviene hacer un metodo o algo mas prolijo para reutilizar


No, I haven't, and including that small line at end of _few_ files is not really something that would annoy you.


Superfluous space at EOL.


Done here 348fee8aaadf4de66e4b9ff72139671595a37c2a
Since I use 'clint' module and call 'colored' method, I need a variable to store the colored string. I named it 'filePath'.


Why ES5?


It's still under development. We plan on doing a full release within a week or two.  As such, please use at your own risk.  Most of it is stable, but we can't promise there aren't bugs that are still hidden.


@tpett whatchu think?


keep trying, this is probably something that is actually stopping me from running from IntelliJ, need to figure out where this languageLevel is set.


Did this work out? Maybe move the `[DEBUG] DB Instance Modification request` inside the `if requestUpdate` block. Could you send this in a PR upstream? :smile:


Just #include <stdbool.h> and use bool instead.


This translated quality is not enough. It need to improve.
This translated is draft.


* **Case 1**: the first element of `args` is a string
* **Case 2&3**: the first element of `args` is a list
  * **Case 3**: the element of `args` might be a list as well


Can you explain this, please?


You are right, and it is done here: 76312e7e059720be8c03dc076d2bf3ab44bb5d36


We use the find command because it searches recursively, whereas the GNU Makefile wildcard does not.


__API Blueprint syntax error on line 1__: expected API name, e.g. '# <API Name>' (Autogenerated message from apiary.io)


Dodgy line ending here...


#include <stdbool.h>


I just remembered that @danicampora uses a Mac (or at least his WiPy Kickstarter videos feature one…)
Daniel, have you run into this problem?


is it here?
(I don't have JDK_1_6 if it's there
![screen shot 2015-05-11 at 12 45 55 pm](https://cloud.githubusercontent.com/assets/163977/7570134/bed908c8-f7db-11e4-86ec-69bdab23f962.png)
)


Another dodgy line ending...


@gmarz should i pass T also? We cannot access T from this internal class. Actually, there is no T i think.


If `filters` and `detectionMethod` are set to a value different than the default one, these new values won't be propagated in the recursive call and the function will behave differently in the subfolders.
Is it what you want ?


Thank you for this! But L1 and L2 Cache sizes really should be integers, not strings.


Thanks for the review, I fixed those issues in the following commit: https://github.com/fallen/artiq/commit/ea831c6052fe89bc4b9880a13068aee261ad2526

If this is now OK I will rebase to squash those two and then send a patch via email


Fixes #9


#9 #8


Current theory: This fails because it needs a GITPERM enviornment variable to be defined. See [Secure-Variables](http://docs.travis-ci.com/user/environment-variables/#Secure-Variables)

e.g., `travis encrypt GITPERM=super_secret --add env.global`


Fixes #8


Yup this works. It's already in a PR - https://github.com/hashicorp/terraform/pull/1907


Hi @taylorotwell,
May I ask why a `/lumen.log` here in the `.gitignore`?


Fixes #8 #9


We couldn't use the official one because it wasn't on the maven repository if I rememeber correctly. I need to have a look if something has changed in the meanwhile.

We have a class called CypherDSL that contains some utility methods but I have to admit that while I was trying to understand the issue I forgot about it.


1: RSA.
2: ElGamal.


http://cog-esgf.esrl.noaa.gov/projects/couplednems/vector_rotation
Vector Rotation

Vector rotation is done for import and export 2D vector fields in MOM5 and CICE. The following assumptions are made

1) The components of the 2D vector fields line up with local lat-lon direction in the mediator, north and east are the local positive directions. The components are defined at cell center for all the grids.

2) The rotation angles defined in CICE and MOM5 are measured counter clock wise (CCW) from local lat-lon directions. This is done in most textbooks and should be a reasonable starting point.

As such, the imported vector field V must be rotated locally as the model component local coordinate system is rotated CCW from well-defined lat-lon directions. The CCW rotation matrix M is

cos \theta         sin \theta

- sin \theta        cos \theta

Rotated vector V' = M V

The transpose of this matrix corresponding to a CW coordinate system rotation is applied to the exported 2D vector fields before they are sent to the mediator.


Should I change it to..

```html
<!DOCTYPE html>
<?php

header("Location: /privacy/index.php");

?>
```

?


@roccolord

> Added Ubuntu in detection:
`UNAME_KERNEL_DEBIAN := $(shell if uname -v | egrep -q "Debian|Ubuntu" ; then echo true; else echo false; fi)`

I can definitely add that to the PR, thanks for the information regarding support on Ubuntu.

> and on line 73 in init script I had to add & at the end:
`nohup $DAEMON $DAEMON_ARGS > /dev/null 2>&1 &`

Hmm, now I'm not 100% sure I tested the init script, but I feel like I did and it didn't work on Debian any other way (though I tested systemd *.service file more so than the init script.

I'll do another build test with your changes, and as long as everything is good I'll add to my PR for pcs.


sure, but there's ambiguity from linking to the homepage- is it included to advertise the freelance work, or to work for Toptal?


@acmenucms We might have to create an array for Multiselect (for the different languages)


The second release is the only one that is relevant here, because the first one is not really open source. That said, I'm fairly certain that the two are the exact same code: they're DOS Hexen 1.1 source code. The Hexen 1.0 source has never been released.

You can find a bit about the Raven source licensing issue on the wiki: http://doomwiki.org/wiki/Raven_source_code_licensing

The edits in your comment reflect what I've said - in Hexen v1.0 the intermission text was in the executable, and in v1.1 it was moved into the WAD. So you can't play with the v1.0. WAD because the intermission text lump is missing.

The best solution would be to release a PWAD containing the missing lumps and upload it to idgames. If someone wants to do that I'm happy to include a link to it in the Chocolate Hexen error message. But I see no other good acceptable solution that will allow Chocolate Hexen to play with the v1.0 IWAD out of the box. I'd prefer to just keep the error message and instruct people to upgrade to v1.1.


r+


Yeah i know but the compiler threw an error at me so i decided to test
string.
Am 11.05.2015 18:49 schrieb "Joe Maples" <notifications@github.com>:

> Thank you for this! But L1 and L2 Cache sizes really should be integers,
> not strings.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/DerRomtester/one_plus_one/commit/045023723bbed2aaf3606445ae6a7395f2acf2f1#commitcomment-11138041>
> .
>


Забыл грохнуть


Are you absolutely certain that you can't have `args.absolutePath` and `args.relativePath` at the same time ? If it is possible filename will be the concatenation of both.


The requirement to adapt was announced [here](https://groups.google.com/d/msg/spring-data-dev/iSUX9CujpLA/iIqVt-yYH0UJ). The announcement also includes all necessary steps to get the codebase compile and work again. Pull request for Spring Data Couchbase highly appreciated! :)


Why not just use an ILanguageService for this?


поправлю


:smile: I can see why that might have caused some confusion though


this can be simplified to `tex := render.GetTexture(logoTexture)`


Great, so I'm the English expert now... To me "to perform" sounds a bit better. How about:

Avocado is set of tools and libraries which performs automated testing altogether with the possibility of manual interaction. One can call it a test framework with benefits. Native tests are written in Python and they follow the unittest pattern but every executable can serve as a test.

You can also mention that there are APIs for other languages too.


yep


but wait.. in fact you remove all `self.notificationways` (and all contacts too) which are referenced by all hosts or services which are removed.. so ?

a notifways can't be referenced by more than a contact ? I think yes, no ??  'cause if yes then there would be a problem if a contact is referenced by both a host or service removed and one which isn't..
same problem for/with the `self.contacts`  ..

I'm not wrong, doesn't it ?


Yeah, you might have a point here.

I left it as a warning because I figured there could be people who might want to play using the v1.0 IWAD, buggy as it is. For example the levels in the v1.0 IWAD are different, so maybe people would want to check out the differences.

You're right that a lot of people (Mac or Windows users) are likely to completely miss this warning because they won't even see the warning.

A better solution might be bomb out with an error but to include a command line argument to allow the check to be overridden.


@jennybc 1c0ecf70716c0b17de3858f238f40dea00c8a766 should remove the warning, but it was harmless in any case.


The purpose of the -noindex addition is to prevent having the "ToC" page in the root of the Webseclab site. It covers only "/" and "/index.html" entrypoint - you should not be able to get the TOC page otherwise due to this routing:
https://github.com/yahoo/webseclab/blob/master/utils.go#L107
(in the hindsight, utils.go is a poor location for those functions, should move them to handler.go - noticed that I myself look into that file first for the functions code).
Since /index.php is not defined in the router, it will get 404 as all other unknown / unmapped entrypoints.

Normally you want the index page - as starting point for human browsers, or to feed it to a scanner and have the links to all the cases in one place.  "-noindex" is needed for a special case when a scanner (in particular, OWASP ZAP) is very diligent in crawling from the root and therefore scans all of the Webseclab even when I feed it just a single test URL with no outgoing links.  In such cases, having the test isolation would be useful (in particular, I want to share some findings with the zaproxy team and focusing the scanner's output for a single test makes for cleaner demo/PoC).  I though about the whitelisting approach where you would give something like "pathlimit" parameter with just a single entrypoint prefix - but I think it's an overkill and actually less desirable.  With -noindex I can turn off the TOC page but still be able to feed all the individual tests to the scanner as needed without restarting the Webseclab - which would be required with -pathlimit everytime you want to try a different entrypoint.  Since individual tests are not cross-linked and open-source automated scanners usually don't have psycho powers :smile: , they should not be able to, given a testA, discover and jump to testB or testC!


In theory the compiler has a copy of this for the purposes of our new implementation of the build task. Is there a way we can unify this? (cc @agocke).


Ah ok I see. Then how does your library compare to [kafka-python](github.com/mumrah/kafka-python), why did you develop your own instead of joining that effort?


Nope, you're not missing anything.  This was just a line that should have been deleted.


My bad. Code is OK.


I've already got the grade. Thanks!


LOL. Anyway, thanks for the input!

OK, so we now have even more to choose from. @ruda feel free to mix and match among the suggestions, and I'll take as it the best of breed! :)

Thank you guys.


These is_x methods should be called assert_x.


We prob should have a test for this.


Good god, someone had a change of mind ;)


Are these actually shared between C# and VB? I wouldn't have expected so...


@thockin -- we should have a chat about this change and how it feeds in to the Kube-Proxy rewrite.

The REDIRECT rule that I've removed here didn't work in our case, since for traffic coming from a pod, the calico source interface doesn't have an IP (the host-end of the veth pair doesn't get an IP, but the pod-end does). This causes the REDIRECT rule to fail, since it apparently tries to use the IP of the source interface as the rewritten destination.

I tried replacing this REDIRECT with the DNAT rule, and it worked; in the Vagrant case, the DNAT chooses the IP 10.0.2.15 (the vagrant-host link IP) to DNAT the service traffic to, and everything works fine. This will need more testing in other environments before we can be sure it's working in all cases.

The comment seems to suggest that doing all DNAT would be desirable if possible. Can you give a bit more detail on the environments/testing that you did to empirically determine that the REDIRECT rule was required? And what testing would you want to see before you'd accept this patch?

Obviously there's a big comment rewrite and code tidy up if this new approach floats, but I wanted to check in with you before starting down that road.


Ok, I will, Daniel put some thoughts into that, I am capturing it under DM-2733


It kind of does :)


This used to be a library called samsa, for the 0.7.x branch of Kafka and had a full balanced consumer implementation.  We started to upgrade to 0.8.x before finding kafka-python and decided to keep with that since the maintainer expressed that he didn't want a balanced consumer implementation in that library.  We need a balancing consumer for our internal uses, so not having that wasn't an option.  He's also since indicated that he's not interested in maintaining the project, so we decided to finish our own upgrade and rename it to pykafka.

At some point, we'll run some benchmarks between the two libraries, but that'll be after we do a full release.  I'm pretty confident our implementation will be more performant, and we're also trying to integrate a wrapper for the C library librdkafka for even greater performance.  Those speedups would be optional, obviously, as the pure-python implementation will be easier to install for some people.


**You really should read the documentation rather than searching solutions on the web**. Searching is almost the **worst** way to solve a programming problem when you are **learning something** instead of **fighting a deadline**.
Read through the docs of `subprocess` and you will find much much better and safer implementations for `run` and `run_i`. They can be implemented within about 50 lines of code in total.

And, like I said, decorator `target` here is mainly for *registration* (mapping target names to target functions), rather than *decoration*. Your `target` is still incorrect. Read my comment [here](https://github.com/Zing22/yake/commit/20f014dfa579cfd4b9695e461cdfc82f1a5f7f11#commitcomment-11122370) carefully.


There is soo much poop


Actually I guess it has an implementation that calls into the host object, but maybe there's some shared parsing or something that can be unified.


You are correct.  These had more than one statement in them originally and when the extra statements got deleted I just failed to remove the braces.  Fixed.


Oh. There is true mistake. I wanted to write:
```response.toStream.takeWhile(_ => token.nonCancelled).foreach(exchange.write)```


Ok buena, ma non ho capito che intendi con posizionarla alla posizione del cursore


Тимофей, в двух словах, что такое private и public attributes? Константы и переменные?
А то пишут, что в питоне их нет.


#2 Add day/night classes done.


And hello to all the broken scripts out there :smile:
But yea, sounds better


@kbourgoin thank you very much for the thorough comparison. This helps me a lot.


On pourra en discuter ce soir :)


For setup-scripts we enforced bash since there are still recipes expecting it, can we do the same here/


Ok ma tanto questo lo cambieremo, i numeri non c'entrano più una cippa ahah


Why add this to the query string when you go through all the metadata properties later anyway. This is unnecessary duplication in the INSERT query


no wait - @diederich knows for sure ping ping ping


Will add, but I want to watch this after updating Bittrex API to v2.0 with their new release. I'm not sure if v1.1 was deprecated.


Priority one is following the style used in the code you are changing. This file and all the files near it use the standard of underscore first for member variables.  What coding guidelines state underscore after? Where is the one you are referencing?

The size 512 is a judgment call.  I pointed at document that states the value of a good standard minimum that is only slightly above 512 that would make this "judgment call" reasonably future proof.  Exposing this value for the application to change would the optimal solution.


Same as previous comment


Wow just saw the new site - looks great! Good to have a changelog section, and I guess we are adding the 'try out/demo' section back later?


I personally prefer the configurability and find it exposes buggy builds or
build inputs since it requires dependencies and certain features to be
present. Autotools also builds in warnings for nonexistent configure flags
so it's easily to tell if old flags are still around. I also don't mind the
workflow of doing a diff on configure / configure.ac between versions as it
gives me a nice indication of what dependencies might have changed and
doesn't take much effort. Supporting every possible configuration through
flags is nice for users with more complex use cases than what we envision
the default to be. It could give the user flexibility similar to gentoo,
which if implemented correctly is a selling point for certain users.

As far as the way optional flags are structured, I'm not strongly in favor
of one or the other, as I think you add a lot of extra typing / assert when
using a separate flag from the dependency. However, that style of flag is
more descriptive of what you are enabling or disabling. I think we need to
have a discussion about what is preferred and unify our current flag system.

On Mon, May 11, 2015, 08:56 Eelco Dolstra <notifications@github.com> wrote:

> Also, having dozens of these mkEnable flags seems very hard to
> test/validate. Do we really want to support every combination of configure
> flag?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/NixOS/nixpkgs/commit/a2a35082ccd64b4d00d88778d9633fe18c0bb215#commitcomment-11136974>
> .
>


Приватные, они же закрытые -- это условное обозначение полей класса, начинающихся с `_` или с `__`. Публичные, соответственно, все остальные.

Константами также только условно называются переменные, изменение которых не предусмотрено архитектурой приложения.

Это, действительно, лишь соглашения. Можно изменить и константу, и получить доступ к приватному полю.


OK, I saw the commit.

But my question was about `write_space`, I don't see any changes to `__repr__()` regarding `write_space` ...?

See https://github.com/spatialaudio/jackclient-python/pull/3#discussion_r23136819 and https://github.com/amstan/jackclient-python/commit/9002ac903a96e5a7292be6cefef9987320394a4c#commitcomment-9096324


Trying to make a hardlink before copying the backup tar file to prevent restore faile due to lack of space. If hardlinking fails then make a copy.


Should be "in sync exchanging" or "in sync by exchanging"?


I don't think this code is gonna work outside of the Linux domain as /proc/net/if_inet6 is Linux only. Please make it OS independent


@zilav @Hlp Wasn't this the fix for VMAD and empty properties?  It's sever commits after the 3.1 tag set in commit 6e615161898c851162cbd165249f8993a0938b2e.  So that leads me to believe that only 3.1.1 has the VMAD fix.  Isn't that correct, or am I missing something?


and  for completeness - this is where it is used:

https://github.com/xbmc/xbmc/blob/master/tools/EventClients/Clients/OSXRemote/xbmcclientwrapper.mm#L360


nit: typo


True, but it's a cache and player's UUID will get pushed out of it over time. The size is set in the spigot configuration (user-cache-size Default: 1000), so the issue is what to do when the player's name is not in that cache (even if it is increased). The previous method to find any player was too file system intensive for large file sets (the admin had > 115,000 files) and was timing-out the admin's server. The new approach uses a disk-based hash table that is designed to keep the data out of memory until its needed.

One alternative could be to use getOfflinePlayer, and then if the player is not in there, just say "unknown player". That would probably work for regular player commands, like ban, but for an admin trying to clean up, it would be a bit odd.


Can you return a more descriptive error - I'd prefer to say "Missing X-GitHub-Event or X-Gogs-Event"


Of the options given I believe `Right(undefined)` is best. This function could be curried, which would remove the zero-arguments edge case. If we make this change we should do so in a separate pull request: `Either.of` is based on `Maybe.of`, so both would need to be updated.


Do you plan to create a pull request? It would be nice if we could get this into the 3.0 release!


Any reason for the extra spaces?


Does GCS not support utf8?


CL: https://codereview.chromium.org/1138063003/


If there is an error that stops the installer completed flag from being switched, this could go for.ev.er.


Почему она мешает?
Как я понял это поле используется в модулях, чтобы показываться при вызове -help. Это как metadata.


Seems like your commit explanation got cut off on this one...


Спасибо, ясно!


Undercova it's the same with your holo project, there are many other functions that are not activated because you lock the access to the source file, is just stupid, keep one eyes in your .gitignore file, and you know very well what I mean.

Zigeh in your place I would stay courteous, the wheel turns faster than you think ;)


Documentation says use free(...) to deallocate memory.
If you look into kiss_fft_alloc(...) there is only one malloc for the whole struct.
https://github.com/itdaniher/kissfft/blob/master/kiss_fft.c
https://github.com/itdaniher/kissfft/blob/master/README


The commit is pretty old but I went through a period of culling everything I deemed "extra" to keep the script extremely lean and readable.
I wanted it to be an exmaple on how to download from HypeMachine so others could use it in their apps etc..


Unresolved conflicts in your pull request.  Running 'git status' before committing should tell you what's up. Also, consider installing 'ACK' on your command line.  You can use it to install '<<<<' before submitting.


Should there be any separation between `EnvironmentVariableSubstitutions`, `ScriptVariableSubstitutions`, or `ContextVariableSubstitutions`? I'm not sure theres any way to tell the 3 apart, but if there is, do you think it's worth considering? I can't think of a use case to distinguish the last two, but it might be desirable to leave env vars unexpanded I suppose.


WHYYYYYYYYYYYYYYY?


Suppression de tous les fichiers inutiles du site afin d'améliorer la lisibilité de l'arborescence


```p.tryComplete```  :)


Она мешает, потому что на первой строке должна стоять инструкция `#!`, если требуется запускать приложение не через `bash`.

`bash` читает первую строку файла и, если находит на ней конструкцию вида `#! prog`, запускает `prog`, которой передаёт на исполнение оставшуюся часть файла.

Теперь про эту часть вопроса:
> Как я понял это поле используется в модулях, чтобы показываться при вызове -help. Это как metadata.

Во-первых, в `__author__` сейчас записано `Admin`, что не даёт никакой дополнительной информации, во-вторых, информация о создателе есть в истории git, и она не склонна устаревать. Поэтому я считаю это поле устаревшим и не нужным.


This is basically output of 2to3. Indeed it made quite a mess in some spots. I dont really care about python2.7 support. It should die, the sooner the better :) As for  other fixes - PRs are welcome i guess ;) I dont really maintain this generator, its just a fork from somewhere else.


This check isn't safe - you can't know this, because portal IP can change at any time.  In fact, if you need to check the internal registry this check is flawed totally (because clients will not know that).

The behavior I was expecting:

1. If you select an image in a registry we would create an image stream for you that points to that image in the registry
2. If you select an image in an image stream, we would always just point to that image stream.

That is sufficient so users get what they expect - we don't need to detect cases of 1. that happen to be 2, because a user can just provide 2 instead of 1.

The builder/generator code should always call ImageRef.ImageStream() - but the ImageRef should know prior to that call (at construction time probably) that the input to the ImageRef is an existing image stream or an image in a Docker registry.  The call to ImageStream() should either return the existing stream or a new stream that has DockerImageRepository set.

I don't think NeedsDockerRepo is correct - either the imageRef has that info or it doesn't.  We want a boolean on ImageRef that is "ExistingImageStream" that if false, causes a new image stream to be created, but when true uses the name/namespace on the image ref as the name/namespace of the stream.


The standard I reference is the C++ programming language standard. As in, the current style used by the WiFiUDP library breaks the C++ language specification. However, given that these are private members, I doubt it will actually cause near-term harm, so I'd be happy to change this.

I still don't understand how 576 bytes is any more "future proof" than 512 or 256. I don't think this value has anything to do with MTU (unless there is some particular implementation issue like fragmentation.) The reason I'm asking about this is that I believe that 576 is a red herring in the context of this particular allocation. You seem very insistent on this value, and I presume for some good reason. I'm asking that you educate me and point at the part of the implementation code that I'm missing where the size of outgoing buffer blocks is tied to the MTU used for incoming packet reassembly!


Don't ask my how, but somehow the adc_ecmd is brocken after this change. See ML.


Cette page a vocation d'être un résumé des objectifs primaires et secondaires de l'outil que nous cherchons/développons


Do `$frame` and `${frame}` count as `FrameSubstitutions` or `VariableSubstitutions`?


Sorry I'm not keen on github.. How can I change the commit accordingly to
my language? Must I do it all again?

2015-05-11 16:45 GMT+02:00 William <notifications@github.com>:

> This commit changes the strings for English not for Italian.
> Look at your commit and change accordingly based on your language because
> this is totally wrong.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/zephiK/android_packages_apps_Settings/commit/a315d2220d0ab6164ccbb7afe55592863a13c34a#commitcomment-11135415>
> .
>


Permettre à tous les utilisateurs de débattre de ces propositions et d'apporter leur(s) contribution(s) afin de les améliorer


In pratica Lo script attuale quando clicchi su un emoticon della tastiera
aggiunge quella emoticon alla fine del testo.
ci starebbe fare che la inserisce  el punto in cui hai messo il cursore
(come quando scrivi ). Però Sì può aspettare a implementare perché magari
togliamo la casella di testo e mettiamo dei rettangolino uno per ogni
emoticon da inserire
Il 11/Mag/2015 19:10, "ghia16" <notifications@github.com> ha scritto:

> Ok buena, ma non ho capito che intendi con posizionarla alla posizione del
> cursore
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/igorravasi/Memo/commit/b4994c8c9ae8bb58a7d6598c891f012a59cf2fe6#commitcomment-11138458>
> .
>


We have a letter in the jscs-dev@ titled "New JSCS Website design" there is more info in there, long story short, yeah, it would be cool, although @mdevils is busy with other stuff though (


good catch, fixed


Ah già è vero ahah poi bisogna anche cambiare la favicon che fa schifo, l
avevo fatta con paint per prova
Il 11/Mag/2015 19:11, "ghia16" <notifications@github.com> ha scritto:

> Ok ma tanto questo lo cambieremo, i numeri non c'entrano più una cippa ahah
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/igorravasi/Memo/commit/96980226ea61e94c1e642bcf4697eecf5ee3578d#commitcomment-11138499>
> .
>


Queda pendiente la carga de episodios


Does this work? I thought a parenthesis was needed around the entire condition like:
```sqf
if (GVAR(zeusAscension)) then {
```

I must be wrong though


Constructing Observable from scratch is useful skill.
In this particular case it could be one-liner using `map` and `onErrorReturn`.


@AchimTuran mhhh we use kiss_fftr and it seems to alloc a bit more - see:

https://github.com/Memphiz/xbmc/commit/981eae8417396338009b2e87391c3ba7e833e362#diff-f858bb63b49d32522a42e16bb74711ebR27 - or am i tricked by macros here?


sure, fixed


there should be a return statement after the callback is invoked


Would prefer we call this `.dockercfg` instead - then it would work for regular pods without change as well.  I think there was agreement we could loosen these key names, so I would recommend submitting that change first before this and then changing to `.dockercfg`.


trailing whitespace :(


Why are we trying to do this?  Do we care this much?


@mariofusco since we are moving this back, could we rename the modules to better reflect the actual content. `drools-osgi` suggests this is Drools integration to OSGi. But it also contains jBPM (and possibly OptaPlanner?).

I would propose something like `kie-osgi` and `kie-karaf-features`. WDYT?


What happens when there is an API version change to DockerConfig?  How do we manage that?


No need for usermod here, right? `useradd -g ceph` should take care of it


No - just # as in the comment above. Shall I further clarify with a second comment about $frame?


`take(1)` looks wrong.


Maybe, leave the id attribute and mark it as obsolete?


there is nothing locked in Holocore. it is completely open
you do realize what .gitignore files are for ?? lol ... seems like no


How does sally's bundle-secrets command fit into this structure?


How about we just define the following here:

  basestring = (str, bytes)


Since the command is "docker" i'm not sure the `docker-`prefix is required


Funziona, però sarei più indirizzAto a implementarlo, molto prossimamente (magari domani) nel modo che ti avevo detto, lato server.  Questo funziona ma non è il miglior modo (andava di moda agli inizi del web 2.0 ma adesso ci sta di più fare le cose server Side)


Я до сих пор путаюсь с именами. Непонятно, стоит ли в функциях менять input_path на input_file. Вроде это все-таки путь до файла, все логично


> What happens when there is an API version change to DockerConfig? How do we manage that?

The pull code already has to interpret this value, so if we won't be able to interpret it on the other end, validation should fail.  When there is change, we must update the puller to understand it and add the new allowed format to validation.


Mixing case all over the place


The commit message needs to  be changed RE http://tbaggery.com/2008/04/19/a-note-about-git-commit-messages.html


Но может быть имелось какое-то другое разбиение на более мелкие функции. Я пока придумала только это


ctypesgen is now on github: http://github.com/davidjamesca/ctypesgen

Despite this being the raw output of 2to3, this patch could be fixed relatively easily. Does this actually work, at least for some cases you tested?


Oops.. comment has cutted out when editing.. I'll explain it


It actually does: https://cloud.google.com/storage/docs/concepts-techniques (scroll down to Object Names section). I guess this was just a way to simplify filenames. It don't think it'd be too difficult to change to UTF-8.


Use `recovered` implemented above.


Its just that the enum name seems to imply they might be. What about `HashSubstitutions` instead? That's a bit less clear as to what a hash does maybe, but more inline with `TildeSubstitutions` (rather than `HomeDirectorySubstitutions`) and doesn't confuse with `$frame`.

Or if you really don't like that, then yeah, perhaps clarify here and add to the tests showing whats expected of $frame.


The `%{rhel}` macro is defined on CentOS, so you can simplify `0%{?rhel} || 0%{?centos}` to just `0%{?rhel}`. (Unless there is some difference related to OBS that I don't know about?)


Thank you :)


I think the semi-colon here is a typo (and repeated throughout the other commands)


success: _success, error: _error instead


```header("Location: /privacy/");```.


Add `timedOut()`.


issues (Which are common in android apps) fixed:
1)keycode issue refer https://code.google.com/p/chromium/issues/detail?id=118639


I'm C-64 fan so thought it would be do nicely.. yes this needs little bit explanation and actually I'll make it 32 just because I can ;)


> How does sally's bundle-secrets command fit into this structure?

They are complementary, but not a replacement to this.  If you already have a `.dockercfg` file, you should use `bundle-secret`.  If you want to make one with specific credentials, you can use this command.

I made this because I needed it for easy testing.


In v6 you use Settings to lock apps in memory, see in: Settings/res/values/arrays.xml


There's no distinction between script and context variables - script variables are just the source of a base set of context variables. I think we could tease apart environment substitutions from variable substitutions if needed, and have `VariableSubstitutions = ContextVariableSubstitutions | EnvironmentVariableSubstitutions`. I'm a bit inclined to leave that until we have a use case though, unless you have one in mind already.


I felt kinda bad for doing this, knowing how it's a pretty breaking change. But it felt necessary =).


Hmm well yep, looks like it came a week later after the release. Ok, wait
maybe a week or two, I'll need to make some updates to lodgen soon.

2015-05-11 20:15 GMT+03:00 Sharlikran <notifications@github.com>:

> @zilav <https://github.com/zilav> @Hlp <https://github.com/Hlp> Wasn't
> this the fix for VMAD and empty properties? It's sever commits after the
> 3.1 tag set in commit 6e61516
> <https://github.com/TES5Edit/TES5Edit-GoogleCode/commit/6e615161898c851162cbd165249f8993a0938b2e>.
> So that leads me to believe that only 3.1.1 has the VMAD fix. Isn't that
> correct, or am I missing something?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/TES5Edit/TES5Edit-GoogleCode/commit/1c5b520447f2004a08367cfab4cf9fc9302da8df#commitcomment-11138581>
> .
>


Just noticed a misspelling in the comment "goind up"


Лучше объявлять как `List<Node>`.


Проблему с незакрашиванием решения в риал-тайме это не решает?


Si aspettiamo e valutiamo, la priorità adesso è mettere in piedi il multiplayer e i relativi script, domani ne parliamo


Sorry for that. I had to implement Axis to Button mapping (and reverse) and could not pass opportunity to add D-Pad Buttons to the list, since people had issues mapping them.


I know very well, the function of this file, here is one exemple in your .gitignore file, /odb , with this fonction you are unable to acces in the source file in this directory and unable to build a server with a fonctional chat ingame because the source file for the chat is in /odb , fo from the moment, or you hinder the access to the source file, Your project cannot not be flagged open source end of story


`items` has `nonEmpty`.
What happens if `items` become empty after `filter()` is processed?


(from the uneducated) Is there no need for the a==b case?


Domani lo facciamo in lab allora


Add `timedOut()`.


Agree they're not pretty, but none of these were actually *wrong*. If self were nil, then they'd fall through in all cases.


I don't understand your questions :)
Demo video: http://www.youtube.com/watch?v=MZNoUMRXaME
There is in preferences of the Fluid app possible to select web rendering engine? I don't found.
All works on default settings. I think that fluid uses webkit (aka Safari).


@jasonmalinowski to my understanding language services are exported using `ExportAttribute` which is not portable.


Not understanding the comment here, could you explain?


I can see why the confusion, but I think FrameSubstitutions is clearer than HashSubstitutions since we have another meaning for hash throughout Gaffer, and # seems to have a bunch of names anyway - pound and number being another two that spring to mind. Since this is a slightly esoteric corner anyway, I think an extra comment and a test should do the trick.


Can you back on this?
I think that '||' should be removed.
In the gallery options we say that option is valid only if the global fancybox is active.


@jasonmalinowski I haven't done the exploratory work yet for Visual Basic.  I had hoped that I could create one interface they could both share `CompilerHost` with a small number of unimplemented methods.  Starting to think though it may end up being language specific which isn't a big loss.


Indeed - since fixing that will mean rebasing, how about I wait for all your comments and then push a rebased fix for everything?


sre


Looks like a spurious console.log here - writing to the console in our implementation.


Just make this an interface? It's internal so changing it's totally allowed.


Ah, I think its maybe a type? `s/2 off 8/2 of 8`?


Then we'll want to remove this notice in the gallery option...
would there be a side effect ?


Oh yeah, you're right, fixed.


what address have you send something to?? I don't seam to have anything in my forum mail box!!


Nah, for some reason people started that parenthesis thing as a myth.

You can even have more complex conditions without parenthesis, but the interpreter will only go from left to right until it reaches the first command that returns something outside of a previous command (so comparisons won't work).

As in:
```
if 5 > 2 then {};
```
Will throw an error because it immediately returns 5. However this will work:
```
if isNull driver vehicle player then {};
```


x


@snopoke Better? There's still a little bit of duplication between get_by_name and get_by_alias, but much less.


why not std::array? :)


You're right, given that they were not accessing *ivars* directly! Oops. :)


@snopoke This actually was deliberate: users no longer enter their own names for domains, so the domain is just created with a blank string, which tells the constructor to use the id for a name


Aha, ok. Well, the parenthesis is not a myth then as you only can skip them in corner cases. For readibility and consistency with the rest of the code base I'd say that we should keep parenthesis from now on.


your name+@reply.github.com :)


 Jess, for the following:
def updateHours(self,newHours):
        self.__numHours += newHours
When i test this function, it doesn't update hours when I use self.__hours but it updates it when I simply remove the __ and use self.numHours += new


I guess the only one I can think of in practice was the fiddling we had to do with `IE_FARM_CORES`, which we now have set to 0 all the time (except on the farm where its set to a specific value). I'm actually not sure this extra substitution behaviour would have helped us there, but maybe it'd allow us to keep it unset in the user env, and have the `LocalDispatcher` create/override env vars during the dispatch?

Or perhaps some future tool could dispatch the same render across multiple shots, and we'd want to prevent the $IEENV_WORKING_PATH substitutions in the main process?


哈哈哈果然不依靠框架自己处理的地方是最容易出错的地方。
另外现在还在频繁更新的beta阶段吧，不开个dev分支真的好吗。。。


Add class description in comment.


Hi @DavidUM, in the case of `moon.SimplePicker`, if `wrap` is not set to `true`, the sending of the `holdpulse` event (sent at a specified frequency, defaulted to every 200ms, when a user presses and holds on a user interface element) is cancelled when we reach the last item in the picker, in order to respect the value of `wrap`. So, I don't think that situation applies here as there is no concept of wrapping. We *might* consider cancelling the sending of the `holdpulse` event once we reach either end, because even though the call to `set` in `next` or `previous` will effectively do nothing as the value will not be different, we can prevent some unnecessary (minor) ops here. But I'd want to make sure there are no side effects of doing this, with respect to how the `moon.Slider` behaves.


I don't think we need this JIRA link here, but it's not a huge deal for me.


Dunno, I though about the lightbox option in global settings as a real global option, if you don't want the lightbox effect at all disable that. Anyway of course having two separates options will give the user more combination, having it in the gallery but not in for normal page/posts, makes sense. In this case remove the notice and also adjust this:
https://github.com/Nikeo/customizr/blob/dev/inc/parts/class-content-gallery.php#L53 (-55)

Is that always been so? There's a typo, I wanted to do tc_fancybox to store the global fancybox setting :D . Sorry, so no side effects, just a stupid line of code I wrote, am I a side effect? I will think about that :P


Fist atualization


Primeira Atualização


tested and working


vind ik leuk


OK guys, can we agree with the follow:
```
Avocado is a set of tools and libraries which performs automated testing, altogether
with the possibility of manual interaction. One can call it a test framework with benefits.
Native tests are written in Python and they follow the :mod:`unittest` pattern,
but any executable can serve as a test.```

If so, I will rebase it in this PR and we can merge it.


Great change!


Should be `git pull origin v2`.


If you are reading this as a script, this would be `cd ..`


Why?


Looks good with the comments I made.


We're not depending on STL


Maybe change the test to
```jl
@test strwidth("\0w") == charwidth('\0') + charwidth('w')
@test strwidth("\0α") == charwidth('\0') + charwidth('α')
```
in order to work around buggy `wcwidth`s?


dogs


@jriguera: pyflakes says both logging and configobj is not used...


มมทมมทมทมทมทมทม


https://bitbucket.org/H3C4T3/holocore/commits/d3a0f4a53eb410e83dd6526dd8c3c2d2d2264896?at=development

lulz


Oh yes it works fine. I used this in my [other app](https://github.com/r-ku/toxicpie) and observed no problems whatsoever.


@jriguera: PEP lint correction


Perhaps this should be the other way around (`Currency currency, BigDecimal amount`) to match other methods?


Ah ah ah
done!


@jriguera: PEP style guidelines, "is None" is the preferred way to check if something is a None value.


thx for the file Obi i work on it :)


@jriguera: PEP lint correction -- lines needs to be under 79 characters per line.


Right. That's what happens when you push right before you sleep :) - I'll close those two new issues first and then update.


Only one thing that we agreed upon but is not reflected there:

"... tools and libraries **to** perform ..."

Other than that, it LVGTM (looks very good to me).


:D


don't use <br> tags for layout - use margin and/or padding instead


You are correct and this should not have been merged. If you have the ability to revert the commit, please do. Or if you prefer, I can open another PR reverting the change as I'm the one that opened the PR with the bad documentation to begin with.


OK


Use the preferred constructor format: `function Game([param1,[param2, ... ]]){ ... }`


don't use spaces after attribute names (see http://www.w3schools.com/htmL/html5_syntax.asp - Spaces and Equal Signs)


adding the __ in front shows that that piece of information is private. As a result we would not be able to directly change it, but in a method we can


indentation


Oh, sorry for forgetting to reply, but it is reopened now.


May I ask what was the command you ran with what options?

-James Richard Jr
jamesrichard91@gmail.com
(203) 814-8774

On Sun, May 10, 2015 at 3:40 AM, Thomas <notifications@github.com> wrote:

> Doesn't work for me ...
> [INFO]
> ------------------------------------------------------------------------
> [ERROR] Failed to execute goal
> org.codehaus.mojo:exec-maven-plugin:1.2:java (default-cli) on project
> update-center2: The parameters 'mainClass' for goal
> org.codehaus.mojo:exec-maven-plugin:1.2:java are missing or invalid ->
> [Help 1]
> [ERROR]
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/JamesRichard/backend-update-center2/commit/c2dc7a4fbbc92adb190fd2886b9c4ff17b11cb89#commitcomment-11122283>
> .
>


None of this really matters other than education, as I had seen the response in the issue, he had the changes already, they were not merged in yet, so it doesn't sound like this pull will go in.

Please point me to this standard?  C++ language truly doesn't have a standard for this in the language definition.  There is some lower level "decoration" that happens by the compiler that you can see in the debug symbols; but even those change from compiler to compiler.
Its a convention that many developers use single _ before for member variables just like they use camel case, but conventions change over time and by the team/code base you work with.  For a while m_ was common but Hungarian notation has dropped out fashion.  Most books show a style that allows for as many lines of code in the smallest space, and by no means were these styles meant to be standards unless specifically called out as the topic.  It sounds like you are referencing a book that did call out this as a topic, please include a link so I may read it.

On the subject of the size, I think the right solution is to expose the value, defaulting to 512 is fine as the value is a judgment call, my judgment would be slightly higher.  BUT,  larger values are possible.  What are the effects of someone sending a larger packet?  In code and in the network.

Well, even if it was left alone at its 1024 value, that single packet could be split up by routers that only have a 576 MTU.  The idea is if you keep it below the minimum, the single packet will stay a single packet; while still giving you the maximum size for the app.  Splitting may cause packet delays and increased chance of loss.  How significant of delay or loss depends greatly on the routers that we have little control over.

I have already seen IoT libraries that use UDP have their own buffers that use 576.  This is why I am "interested" in this specific value and led me down this rabbit hole.


This runs when the document is ready - how could you trigger it based on a custom event?


Actually there should probably be a check at the end too right?


@juh2 @gforcada I added a script in the locales folder and resynced the translations.


Ah, the smell of dead code roasting on an open fire always makes me think of home.


`const` is an `ES6` feature, and should be avoided for now. Try collecting most global variables into an `App` object and using `Object.freeze` or `Object.seal` to achieve similar results.


I might be able to use these to speed up the front-end rendering

Refs #168


唉晚了晚了，我做了几次大改动（换编辑器）。
改完才想到开个分支。之后会注意的~

XSS也是悲剧，有点被惯性思维控制了，感觉有这样的正则/@([a-zA-Z0-9_\-\u4e00-\u9fa5]+)/g就没事。结果没想到是不匹配他的也输出了。


Avoid function expressions for constructors.


Can't you say `className in this.allClasses` ?


> Mixing case all over the place

Camel case are internal (only get hit during delegated calls).  Lowercase matches flags.


Большое спасибо, бекпортировал измения в [основной сайт](http://zenway.ru/) ))
Всё работает, как ожидалось )

с нетерпением жду 0,90


This addresses Issue #8


Type: it should be "regardless where they are stored"


> Since the command is "docker" i'm not sure the docker-prefix is required

`username`, `password`, and `server` are global flags for factories, so they have to be different or we will get collisions.


I thought this was importing external Python pulp libraries.  Can we change this to `from .pulp import PulpServer` so it's clear it's an import specific to this package?


I'd expect to hit a python error here because line 19 uses different indentation than the lines that follow.


(Note the period)


@jriguera: If the library doesn't exist, we really shouldn't do anything.


"in a specific"


Should we have ```./build.sh``` here or you removed it on purpose?


|| die


@jriguera: argparse is the preferred library now.  I'll fix this in a later commit.


"in a single project"


The Portable MEF (i.e. "MEF v2") version is portable. This is used widely in the workspaces API -- there's nothing different you're doing here.


Done.


Thanks for your contributions! It could be nice if you do a PR and then I can review and accept your changes.


Zou je niet een abstracte class maken, die een functie voorsteld, zodat we daar dingen mee kunnen doen?


Then why we aren't using an extra PATH variable for the whole faked compliers..

Btw we can ask the wrappers to read CC and CXX..


@jriguera: Generic function to change the NetApp API to json styled Python dictionary objects.  This will allow someone to write the output from an API call as json to a file.


Same comment here as above.


oh ermm I had no idea I could get messages like that, I'm not a github wiz at all
where is tha inbox? I'd have though it would have forwarded to my address I'm registered with


Dan heb je onbepaald gedrag en mag je zelf kiezen.


I mean, most words that start with `lean` are derivatives or `lean` so I think it'll be good for the joke


Sigh.

----- Original Message -----
> > Since the command is "docker" i'm not sure the docker-prefix is required
>
> `username`, `password`, and `server` are global flags for factories, so they
> have to be different or we will get collisions.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/deads2k/kubernetes/commit/b0ce650db31e05632227214c3165f4e991c43d74#commitcomment-11139566


I think we would put bundle in here as create:

  kubectl secrets create --directory=...
  kubectl secrets create-dockercfg ...

I'm ok with that then.

----- Original Message -----
> > How does sally's bundle-secrets command fit into this structure?
>
> They are complementary, but not a replacement to this.  If you already have a
> `.dockercfg` file, you should use `bundle-secret`.  If you want to make one
> with specific credentials, you can use this command.
>
> I made this because I needed it for easy testing.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/deads2k/kubernetes/commit/b0ce650db31e05632227214c3165f4e991c43d74#commitcomment-11138960


Also, are you testing the plugin with the latest diamond version?
BTW, I have in the todo list, create a docker container with plugin (and diamond), so we could easily spin up monitoring instances ....


tried to make readme


why dynamic cast? can't you just use static_pointer_cast ? I don't think we use rtti anywhere right now


Should we use golang.org/x/net/context as the import and adjust the GOPATH for the build?


"in the following way" is the correct form IMO


here


"on a project"


The client wants to use the title field for title and caption for caption (or in this case for the photographer data that was in the Clique Tools.) I think we should include the `caption` field but the only problem with that is the imported photos for all the sites we've done, the caption field is being used for the title.


I don't understand this change: previous code seems right (and it has been like that for a while).
MAX_SCP_TIMEOUT is the upper bound for the timer value (240 seconds).


Dang it. Those two lines were indented with a tab, because I didn't have expandtab turned on for kickstart and, well, oops. It does actually work but it's gross and I'll fix it.


I agree that it would be nice (and more correct), but since this is in use by fuse-bxms I am not sure I can change this now. I'll coordinate with Hiram Chirino.


if you look in my file the gitignore is empty so use you brain next time :)


How can define a require field like following image im my cart page
![11128780_968098606564109_1628926636_o](https://cloud.githubusercontent.com/assets/12399263/7571768/960fa81a-f83b-11e4-89b0-a38ba24534b2.jpg)
![screenshot_3](https://cloud.githubusercontent.com/assets/12399263/7571775/a26d7844-f83b-11e4-9c36-bd678617059b.png)

plz give me the solution as soon as possible.Thanks


Ah.  I think in this case we could add a generic `Update<T>()` overload.


I don't know if you can change the PR,  but in case you can't just make another one please (and thanks).


Good catch, these won't ever be used. Make sure you aren't missing a semicolon.


the last version v2.6.7


There's stray whitespace at the end of this line.


Путь до файла - это, по факту, имя файла)


Stray whitespace at the end of this line. Sorry, I get OCD over stuff like that.


Two things:
1. `<br>` doesn't need the self-closing `/` because we're not in XHTML
2. We don't need the `<br>` at the end of this paragraph. In fact, it's best to avoid them altogether, since text can wrap in all sorts of ways on different screen sizes.


@mitchellh done in a672d9c


this.allClasses is an Immutable.Set<String>, not a javascript hash/dictionary, so `className in this.allClasses` won't work. But thanks for calling me on this, as in a closer look at the documentation, I see that I can instead use `this.allClasses.has(className)`.


The first level of the subgrid is expanded but we need some configuration at the dataset level that allow us to know when we have to start expanded

Related to https://github.com/GriddleGriddle/Griddle/issues/156


I am guessing that this flag is temporary? Otherwise an explanation would be nice in the code that explains the reason why we don't want this behavior in production code.


Prefer `std::uint32_t`


Hey Gareth, you mind updating the settings.yaml in piksi_tools file as to explain how this setting (and any related settitngs) work?

I'd like to generate the .pdf documentation of our settings and am not sure how this works and it's intention.


GetNuspec can throw a PackagingException, which is the base type of all the other packaging related exceptions. That should be caught also here.


Only problem is that you defeated the purpose of having an abstract virtual function by not properly overriding the function in the sub classes :P


:+1: thanks @shawnbot !


I don't develop much in my mac anymore because I never got the debugger working with the CC3200 in OSX (I cry a little every time I remember that...). I just tried to build it and I get this compilation error (I followed all the instructions here BTW: https://github.com/micropython/micropython/wiki/Micro-Python-on-Mac-OSX).

```c
CC ../py/modstruct.c
CC ../py/modsys.c
../py/modsys.c:181:23: error: use of undeclared identifier 'MP_QSTR_stdin'
    { MP_OBJ_NEW_QSTR(MP_QSTR_stdin), (mp_obj_t)&mp_sys_stdin_obj },
                      ^
../py/obj.h:84:56: note: expanded from macro 'MP_OBJ_NEW_QSTR'
#define MP_OBJ_NEW_QSTR(qst) ((mp_obj_t)((((mp_uint_t)(qst)) << 2) | 2))
                                                       ^
../py/modsys.c:182:23: error: use of undeclared identifier 'MP_QSTR_stdout'
    { MP_OBJ_NEW_QSTR(MP_QSTR_stdout), (mp_obj_t)&mp_sys_stdout_obj },
                      ^
../py/obj.h:84:56: note: expanded from macro 'MP_OBJ_NEW_QSTR'
#define MP_OBJ_NEW_QSTR(qst) ((mp_obj_t)((((mp_uint_t)(qst)) << 2) | 2))
                                                       ^
../py/modsys.c:183:23: error: use of undeclared identifier 'MP_QSTR_stderr'
    { MP_OBJ_NEW_QSTR(MP_QSTR_stderr), (mp_obj_t)&mp_sys_stderr_obj },
                      ^
../py/obj.h:84:56: note: expanded from macro 'MP_OBJ_NEW_QSTR'
#define MP_OBJ_NEW_QSTR(qst) ((mp_obj_t)((((mp_uint_t)(qst)) << 2) | 2))
                                                       ^
../py/modsys.c:197:8: error: invalid application of 'sizeof' to an incomplete
      type 'const mp_map_elem_t []'
STATIC MP_DEFINE_CONST_DICT(mp_module_sys_globals, mp_module_sys_globals_table);
       ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
../py/obj.h:175:21: note: expanded from macro 'MP_DEFINE_CONST_DICT'
            .used = MP_ARRAY_SIZE(table_name), \
                    ^~~~~~~~~~~~~~~~~~~~~~~~~
../py/misc.h:101:33: note: expanded from macro 'MP_ARRAY_SIZE'
#define MP_ARRAY_SIZE(a) (sizeof(a) / sizeof((a)[0]))
                                ^~~
../py/modsys.c:197:8: error: invalid application of 'sizeof' to an incomplete
      type 'const mp_map_elem_t []'
STATIC MP_DEFINE_CONST_DICT(mp_module_sys_globals, mp_module_sys_globals_table);
       ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
../py/obj.h:176:22: note: expanded from macro 'MP_DEFINE_CONST_DICT'
            .alloc = MP_ARRAY_SIZE(table_name), \
                     ^~~~~~~~~~~~~~~~~~~~~~~~~
../py/misc.h:101:33: note: expanded from macro 'MP_ARRAY_SIZE'
#define MP_ARRAY_SIZE(a) (sizeof(a) / sizeof((a)[0]))
```

Any clues? This is my clang version:
```
Daniels-Macbook-Air:unix danicampora$ gcc -v
Configured with: --prefix=/Applications/Xcode.app/Contents/Developer/usr --with-gxx-include-dir=/usr/include/c++/4.2.1
Apple LLVM version 6.1.0 (clang-602.0.49) (based on LLVM 3.6.0svn)
Target: x86_64-apple-darwin14.3.0
Thread model: posix
```


Even I thought so but when I'm trying on IDLE shell, the updateHour method does not change the _numHours value for some reason. But it works when I remove the __


resolved, sorry about that.


#10662,#10663 => #10662, #10663 (white space)


Good catch!


#11036,10971 => #11036, #10971


class RenderManager {
public:
    virtual void render() = 0;
};

class RenderTable : public RenderManager {
public:
    void render(); // wtf???
};


i'm sure whole Northrend needs a lot of work... you will find plenty stuff to work on Grizzlyhills ;)


s/Or/or

LGTM


s/9seconds/Sergey Arkhipov/ (will be in the upcoming mailmap update)


Woop :smile:


@poppywis3 Are you rocking python3?


`ioloop` should be optional.  Default to [`tornado.ioloop.IOLoop.current()`](http://www.tornadoweb.org/en/stable/ioloop.html#tornado.ioloop.IOLoop.current).  This is the standard pattern in tornado land.


Fix problem ExecJs::RuntimeError
Hello Rails tutotial
4.3 Setting the Application Home Page


It's not important, but "{}".format() (ie without {0}) is supported in 3.1+. Just before you went crazy trying to fix all the empty ones.

https://docs.python.org/3.1/library/string.html


This might be cleaner as:
```
if res.nil?
  return
end
```

This saves you an indent level for the following code.


:wink:


Idents got mangled here


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


Is this intended to match the upstream behavior?


@JakeWharton @jisung 1.2.0 Is not available on Maven Central, is that intentional?


All of the ```"No webpage body from #{target_host}:#{rport}"``` should either use an api that automtically prepends host:port, or prepend host:port to the message to be consistent with the normal output.


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


Thank you


This might be better off returning an object capable of representing currency. Might want to have a separate interface for that, or assume that it would always be an ItemStack and return an optional of the former.