Yes, thank you.  I misrepresented the problem when trying to adapt the language.


Seeing as player is a user defined variable, don't think this should be added.


@alfiehanssen This breaks Vimeo-iOS/dev and also wasn't PR'd.  No major qualms with retroactive review, but lets try to keep dev branches on all submodules building together always.


Can't the server respond with the same information that the TXT record has?  Any turkey can echo a nonce.


Hi @kkiener I see that you have committed your ruby file into the main directory. We would like it to go into the 'ruby' directory.
If you could just drag this file over into the 'ruby' folder of the fork you have locally, then add, commit and push the changes, they will join the others in this pull request.


probably not the right description?


@dudekm Zgodnie z `PSR-2` zawsze nawiasy przy inicjalizowaniu obiektu:

`new Request();` – dobrze

`new Request;` – źle


No, I did not know I was supposed to do that. So to confirm, when I add
some .framework to the Xcode project, I must also add it to the Cartfile?

On Monday, July 6, 2015, wes johnston <notifications@github.com> wrote:

> Did you add this to the Cartfile somewhere?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/allenngn/firefox-ios/commit/a25d10a183c8f971d82c9bf73cc2842c12146220#commitcomment-12021315>
> .
>


Since all this is being run by stagedfunctions, we shouldn't have to wrap things in `Val`s to play tricks with type inference. So I thought it would be clearer to pass `N` as a variable and then instantiate it manually. But if you have an alternative (working) design, I'm happy to see it done differently.


I just see duplicate code where we don't really need it. For now leave it in until you get that second patch done. We can revisit this then


Ops. Forgot this when hastely making the previous fixes.


Suggest naming it something more descriptive than "modules". kmodules, kernelmodules, kernel_modules?


String frozen means no altering any code involving strings? Or something
else?

On Monday, July 6, 2015, wes johnston <notifications@github.com> wrote:

> We're string frozen as of last week, so this patch may have to wait until
> v1.1 of Firefox iOS.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/allenngn/firefox-ios/commit/a25d10a183c8f971d82c9bf73cc2842c12146220#commitcomment-12021321>
> .
>


@dudekm Zbędna pusta linia


@dudekm Spacja między `if` a `(` (`PSR-2`)


To be clear, I think that this is a lost opportunity; there's nothing wrong with what you have here.


this block is empty. if you flip the condition, you won't need it. both the file time and the file name conditions can be put together, and also be extracted into a private method.


I mean it sends to times. Not only.


This documentation should be on `initialize` method, shouldn’t be?


972124-2NewDetailedIceStorage (RKStrand) - Win64-Windows-7-VisualStudio-12: OK (1673 of 1676 tests passed)

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


I'm sure it's not the first misspelling in an open-source project, but you may want to change Grammer to Grammar now before it becomes too pervasive. :sweat_smile:


I imagine this feature (in some form) will be exposed in the testing API... How?


I agree, although i thought it made more sense if it were consistent with the rest of the code.


TeamCity NEventStore :: SQL :: Build [Build 6.0.1](http://build.prxm.it:8811/viewLog.html?buildId=3353&buildTypeId=NEventStore_Sql_Build&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:01:52


Yeah, for parameters sorry. Otherwise the description applies to the class itself.


I actually have a PR that will be posted later today that contains those changes for the whole library


Okay, I’ll fix it.


Ok let the time be our judge and keep a record of the use of that library on new projects. Thanks!


Licencia del proyecto.


Agree to make this change.


Can you clarify this use of ```_.without```?  In your function, you declare an argument ```fieldNames``` which is a non-existent array.

I'm assuming this is functionally a placeholder for the returned array in ```_.without(fieldNames....```, that is, it doesn't actually exist anywhere else?


Please use private fields rather than globals.


Great, so should i change it or will you merge it to yours?


i dont understand your program, i use only geoeditor give me help?


Add this line
CONFIG_ATAGS_PROC=y


profile_name might be better.  Note that all references to users should be stored as the UUID, not the id column, or this field.


I know this function is not new, it is just moved, but it is raising alarm bells with me.  My biggest concern is that we are calling this function in two places with a `std::uint64_t` as an argument for the second parameter.  This puts us into silent overflow mode before we even enter `mulDiv`.

Here is a more robust implementation:
````
std::uint64_t
gcd(std::uint64_t x, std::uint64_t y)
{
    while (y != 0)
    {
        std::uint64_t t = x % y;
        x = y;
        y = t;
    }
    return x;
}

std::uint64_t
mulDiv(std::uint64_t x, std::uint64_t y, std::uint64_t div)
{
    const auto M = std::numeric_limits<std::uint64_t>::max();
    if (x <= M / y)
        return x * y / div;
    auto vd = gcd(x, div);
    x /= vd;
    div /= vd;
    vd = gcd(y, div);
    y /= vd;
    div /= vd;
    if (x <= M / y)
        return x * y / div;
    // Throw or silently degrade here? Or revert to floating point + round?
    if (x >= y)
        return x / div * y;
    return y / div * x;
}
````


Ok, so if I'm following correctly, that would require a customized-multipath.conf template where each section can be generated by attributes.  Should there be a corresponding customized-multipath attribute file where sane defaults can be defined?  Format something like:
```
default['multipath']['defaults']["user_friendly_names"] = "yes"
default['multipath']['defaults']['udev_dir'] =  "/dev"
```
Customized attributes could be applied by a wrapper cookbook or role with a format that separates vendors/products in the following groupings:
```
  default['multipath']['devices']['HP']['OPEN-*']['path_grouping_policy'] = "multibus"
  default['multipath']['devices']['HP'][['OPEN-*']['rr_min_io'] = 1000

  default['multipath']['devices']['HP'][['HSV200']['path_grouping_policy'] = "group_by_prio"
  default['multipath']['devices']['HP'][['HSV200']['rr_min_io'] = 100

  default['multipath']['devices']['EMC'][['Invista']['path_grouping_policy'] = "multibus"
  default['multipath']['devices']['EMC'][['Invista']['no_path_retry'] = 5

```
I could then remove my modifications to the default recipe and default attribute files.


@mohsen1 not exactly, i'm only focusing on the [mock assertions](https://github.com/apigee-127/swagger-test-templates/issues/16) right now. we should open another issue for using model properties in the requests.


@bradwilson Any chance to see build3048 oublished over at NuGet.org?


TeamCity NEventStore :: SQL :: Build [Build 6.1.0-unstable.2+2](http://build.prxm.it:8811/viewLog.html?buildId=3354&buildTypeId=NEventStore_Sql_Build&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:01:56


With the above implementation this last test results in the exact (correct) answer: `0xFFFF000008800000`.


Class names should be in uppercase. Maybe it should also be renamed into something more descriptive, like ZipCreator or ArchiveCreator.


@mohsen1 it is a matter of using join vs. '+', should i open a ticket/branch for that or include it in my current branch?


Agree to make change to use create instead of new to update database.


@mohsen1 this will be handled by my current work, yes


Very curious why Travis passed there :|


/cc @carlosmn @Therzok ^^


it appears to be a left-over field; it was referenced in other places, but that has been cleaned up to refer to status.  I will verify that and then remove it.


Hey Adam,

I know you're busy, but if you could spare one minute, literally, you'd do the Meteor community a [huge favor](https://github.com/acreeger/meteor-moment/pull/23#issuecomment-118940256). Can you please run `meteor admin set-unmigrated mrt:moment`? Thanks!


Shouldn't the argument still be `commander.dbUrl`?


Need to `return` after.


The invite system is a mess.  I would prefer that the invites be issued from a single email account, and a reference to the user who issued the invite (with Invite.sender being updated to refer to User.userid, which in turn needs to be changed to User.uuid).


yes, absolutely. thanks @glima


ops


This worked, users didn't like it too much.


Redundant parens here.


Still have `:ruby_21` listed twice here. :)


Ah ^^


@DarrenN what happened here that we got removed?


Did this change back to `x`?


Singletons would have the same globals but in a wrapper. I would prefer if you used MainFrame approach, or any non-static class, or any class whose destruction we control. Static class destruction may mess up things in the future.


Yay! Thanks guys.


This method mainly creates a file name (and an empty file). You can put the creation of the file name into a private method and have the method that needs the file name access it.


Cool:-)


Requiring `BoundedRange` when it's only needed to ensure that `prev` is O(1) is limiting. It would be nice to have separate overloads: the current `operator()` for `ForwardRange`, and another `operator()` for `BidirectionalRange` that finds the `end` iterator up front before constructing the view (i.e., returns `cycle_view<bounded_view<Rng>>`).


Yeah, in some places the dictionary is instantiated, and then operations are performed on it before it is returned.  Ideally as we progress the codebase the .dict() functions will be replaced by a custom JSONEncode object so that we can simply serialize the object rather than special casing the dict construction.  That will be less error prone and reduce points of failure.


$zip is only needed in creation::zip(). You can instantiate it there.


Might have missed the comment. Why was common require removed?


Why is the left hand side only `NullBinder`? This should be any binder, shouldn't it?


need \`sysctl -n hw.ncpu\`


I'd prefer if this was implemented by removing this block from the elif and instead making it:

  if mime_encoding:
     self.flags |= MAGIC_MIME_ENCODING


Shouldn't the date ranges be set as properties too?


Looks like you haven't added this new file. Can you add and push?


will change it.


Will any of the fields set on the prototype need to be changed on the instance?


wowsers are these tests opaque :)


This is one place we silently convert a `std::uint64_t` to `std::uint32_t`.


Problem: HAVE_SYS_SELECT_H is correctly set in config.log but didn't make it into any Makefile. Therefore it's not active in src/examples/common.h


The problem with this is that its when adding an extension. We auto-generate the password so we don't need to ask the user for it so instead of adding the permission check to the field name I'm removing it from the value. It wasn't needed on the SQL insert only needed on the SQL update.


The scope_id is required because the user may want to use these scripts to copy configuration from one scope to another, even on a different host.

Yeah, we wouldn't necessarily want to delete scopes that had been created since the last backup was taken. Backing up a host is a convenience, but only a single scope should be restored at a time.


Date ranges are part of the overall class. Since those are shared across all export types.


- Actually send the telnet process the kill signal with .terminate(). 'exit' is not guaranteed to kill it.
- Don't call sys.exit() if telnet fails to execute. Rather raise a custom exception and let the caller deal with it. The caller should already be in a try/catch block.
- wait_for_edge is a blocking call so there is no need for the while loop or the return. Just let it block.


wait_for_button_press would be better


But it's reasonable to assume an export wouldn't use them. I think they should be set the same way for consistency.


hmm, leave it as is and if my PR gets merged prior to yours then I'll have you change it


fixed it mourinho


IIRC it wasn't being used.


@jospoortvliet Do you mean for this to go in stable8.1?


че значит деф путс, не гони


well, the simple answer to making `svg` appear in the RStudio Viewer is to do something like this using the `MontyHall` example.

### the example
```r
library(gridSVG)

# R Journal paper

library(grid)
text <- sample(c("goat", "goat", "car"))
cols <- hcl(c(0, 120, 240), 80, 80)
MontyHall <- function() {
	grid.newpage()
	grid.text(text, 1:3/4, gp = gpar(cex = 2), name = "prizes")
	for (i in 1:3) {
		grid.rect(i/4 - .1, width=.2, height=.8, just = "left",
				  gp = gpar(fill = cols[i]), name = paste0("door", i))
	}
}
MontyHall()

grid.ls(fullNames = TRUE)

library(gridSVG)
links <- c("http://www.google.com/search?q=car&tbm=isch",
			 "http://www.google.com/search?q=goat&tbm=isch")
for (i in 1:3) {
	grid.hyperlink(paste0("door", i),
				   href = links[match(text[i], c("car", "goat"))],
				   show = "new")
}
```

### the show in RStudio Viewer part

```r
# htmltools makes it easy for svg to appear in RStudio Viewer
library(htmltools)
# will also use XML for its saveXML to extract the svg in text form
library(XML)

browsable(
  HTML(
		saveXML(
			# name = "" or NULL gives us the output without saving the file; ...$svg is the svg
	  		grid.export(name = NULL)$svg
		)
  )
)
```
Adding the interactivity, especially to polygons, is the trickier part, but I'll be happy to help in this also.


Can drop this comment. Benchmarks should be executed by node for analysis and proof. Also, we trust you. ;-)


Oh, I wasn't aware of this. Cool. Do we need to make sure that configure_requires includes the proper version of EU::MM to support this syntax?


Replacing tr with sed corrected a problem for which the app versions containing some letters were not displayed properly. I had to change the width of the column from 17 to 22 characters to accommodate longer version numbers.
I don't know if this breaks something.


the UUID is the external reference, the id is an internal reference.  We could probably combine them, but for the first kick at the can I left them in place.  The id column should never be referenced in code, other than to manage other database relationships.


Looks good... Wait a few minutes for my next build to completes.  Might lead to another update for this file.


Same as above.


I found an issue here while reviewing my own PR: It's missing the call to this log statement


Ok, I will revert back to the main frame approach.


Thanks for your good work, Jack!


My normal approach is to have one build command that builds everything from source, and to only check in the source. SO let me ask @tmaybe, @migurski what's the pythonic approach to building artifacts?


I prefer terminate as you're guaranteed that the process will be killed. Ctrl+C depends on openocd killing itself (this may fail if the process is somehow hung). Check out: https://pexpect.readthedocs.org/en/latest/api/pexpect.html#controlling-the-child-process
Consider calling terminate(False), and then if the process is still alive after 5 seconds call terminate(True)


Yes. Pretty much all of them. It seems like it shouldn't help to put them on the prototype if they're going to be overwritten on each instance, but for some reason there was an improvement on the benchmarks at the time.

On a semi-related note, I'm currently in the middle of doing some more work on the js parser and am going to be comparing those changes (and this prototype "optimization") against v8 4.4/4.5 to see if things have changed.


Preps for OpenSSL 1.0.2c


@mbarkley Sorry Max for the oversight (I feel the pain when code breaks because usage wasn't checked thoroughly). You shouldn't need to explicitly destroy the bean now as it handles it's own destruction (@Hasys has stated he's tested the destruction - it makes me feel uneasy, but if it's tested, it's erm tested). See https://github.com/droolsjbpm/kie-wb-distributions/commit/aceb599fcf6b6f468ee7a605ee3278d4460c6537 for an equivalent update on another module.


Looks good.  The program runs for me now without modification.   The added comments and code changes are clear; the program is easier to understand.

I am not sure about the exception handler.  I added some debug print statements to the try and except blocks like this:

	for line in contentsList:
		try:
			contentsDict[line[0]] = int(line[1])
			print "Okay "+str(line)
		except ValueError:
			contentsDict[line[0]] = line[1][:-1]
			print "Except "+str(line)

When I run it, I get this output:

Okay ["Arinell's", '4\n']
Okay ['Pancho Villa', '3\n']
Okay ['Andalu', '3\n']
Okay ['Urbun Burger', '1\n']
Okay ['El Toro', '5\n']
Okay ['Casa Thai', '2\n']
Okay ['Taqueria Cancun', '2\n']
Okay ['Little Baobab', '1\n']
Okay ['Charanga', '3\n']
Okay ["Irma's Pampanga", '5\n']
Okay ['Bay Blend Coffee and Tea', '3\n']
Okay ['Giordano Bros', '2\n']

So the except block is never exercised, and the '\n' is successfully ignored by the int() conversion function.   I think it is a good idea to keep the except block, but instead of trying to remove the '\n' I'd just assign an 'impossible' rating value in this case, such as -1.   Then, if any -1 ratings were seen, that would be an indication of a failure to get an integer restaurant rating.

I find it very helpful to place temporary print statements at various parts in a program to confirm the behavior is what I expect.


This is very confusing - it gives the idea that there is an extra byte between the levels, and the data. I went through the code in parquet-mr, and see no evidence for such a byte (nor can I fathom its function).
Is it a typo, or are you using indexing starting on 1 (in which case it should be made clear)?


Again, don't put this in a try/catch block. The caller should be in a try/catch block.
Rather raise custom exception in the event of detected failures or let unexpected exception percolate themselves up.


No, I just forgot to change it back.


```fo.getGalaxySetupData().seed```

is the galaxy seed. I would suggest including both that seed and the empire name so we have differing research routes from game to game and from empire to empire.


**Very** nice! I had been wondering about porting mksh to kLIBC and OS/2 myself, but my OS/2 version is a bit too old, and I haven't installed it anywhere at the moment either.

I may merge a couple of these patches ;-)

Although, with NOPROSPECTOFWORK, this cannot ever be an “official”, working, mksh port – just like Plan 9’s and Syllable Desktop’s currently are (Haiku fixed their kernel signalling bug, so they no longer need it). But due to some differences like OS/2-style pathnames, it differs enough anyway, like mksh/Win32 does, so it can be mksh-os2 of its own, with supporting code shipped in mksh itself. That would make three “inofficial” ports (Win32, EBCDIC, OS/2).


The manual kubectl steps were recently written [here](https://github.com/GoogleCloudPlatform/kubernetes/pull/10595/files) but not merged yet. Can we point to these steps or copy maybe them here? (in the long run its best if everything pointed to a standalone set of kubectl install instructions but for now, pointing what we have is better than nothing)


Do you not need to do some sort of GPIO cleanup on exit?


Looks OK to me.


revise to:
"...which is a module that provides..."


I think this commit is not needed. If you look at the code carefully, a.out and ELF magics are also checked for in this block. The basic idea is that if execve() returns an error, *then* we don’t try to run it with SCRIPTEXEC if it should have worked through the OS – which means you probably broke that scenario.

Please feel free to mail or IRC with me for discussion.


The idiomatic ruby way is to do `changes[type] ||= {}` and `changes[type][title] ||= []`


Nice job!  I have just one more followup comment.

On Mon, Jul 6, 2015 at 10:52 AM, animals4all <notifications@github.com>
wrote:

> I made the revisions! In the "if *name* == '*main*'" condition, I
> accidentally made the *name* variable a string instead of a variable.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/animals4all/Exercise06/commit/a8ee26bd6d9b3ee4fed06e9ed878c9d1eed2f0a7#commitcomment-12018124>
> .
>


The truth is, since drak deleted his fork, we actually are maintaining our own fork. There is actually one backwards-incompatible change in the fork (`whereIn` behavior with an empty array) which we should reverse. It seems that zikula are actually maintaining (in their `master` branch) a fully backwards-compatible fork - it might be worth switching to use that. It would be nice to get some sort of communication from them as to their plans with that repo, so we don't get the rug pulled out from under us like we did with drak.


`2 * sizeof(size_t) == 8` on 32bits?


Interesting. My experience has shown adverse effects in recent V8. The prototype object isn't part of the same object map as the instantiated object. So when changing any of these properties on the instantiated object it still also changes the object map. Instead I'd have to set all possible object properties in the constructor so the object's map never changed during its lifetime.

But, hey. I'm not going to argue with the numbers.


This is my first :blush:
Thank you for the hint :cherries:


Actually, as long as `https://static.silex.me` doesn't work, we can't do anything.
I updated the commit here: https://github.com/indiehosters/Silex/commit/bc3860684bc92f541c0503cf853b6f0d71f01b28


I switched the getByLabel calls to getByToken and also took out the unused variables for looking at tracks , strips, and vertices.


hii, just modified


I got huge delays if one of the local db's did not exist.
In combination with #83 I get delays of 25 seconds!

You could add something like this to see the effect:
```C#
#if DEBUG
    new LocalDbClousotCacheFactory(@"(LocalDb)\NotExistingDB"),
#endif
    new LocalDbClousotCacheFactory(@"(LocalDb)\MSSQLLocalDB"),
    ...
```


Do we really need the -n considering that we're building an hash?


@nulltoken Beta 4's release date is about 2 weeks away (it'll be sync'd up with the DNX Beta 6 release, which I believe is currently slated for July 20th). In the meantime, if you need to use a CI build, you can pull them from https://www.myget.org/F/xunit/


I think the `Ptr{Void}` here is not necessary (but is find to keep).


The common module does checks automatically. For example it checks that no globals have been leaked during the execution of the script when the script is done.


rax retry tests


Were the v0.8 / v0.10 branches of V8 vulnerable to this bug?


Good catch!


This could use more specificity; maybe `user-settings-addons-link` or `osf-user-settings-addons-link`.


Slugs are lists.


Which is why these were all wrapped in brackets


972124-2NewDetailedIceStorage (RKStrand) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1673 of 1676 tests passed)

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


Are we getting rid of the created_at and updated_at?


Why escalate this to an exception?


No, it's all handled on the authoring side (so whoever does the release needs to update their EUM to the latest version).  It's planned for this to be handled better (so the directive isn't expanded in the file when it's parsed) -- but MANIFEST.SKIP only has relevance for authors anyway.


Yes, but for each binder, it only has sense to check against `NullBinder`, `VarBinder` or its same binder structure. Otherwise it's a type error, but we just return the left value as a non-uncovered case.
```haskell
missingCasesSingle _ _ b _ = [b]
```


@LordRalex can you post a release and instructions on how to update? **Thanks and good job!**


tsk tsk


It's such a great idea! Thanks for doing this.


Hi. Thank you for the feedback.
I guess there is no advantage in either of the approaches... And the one I used is already done. I don't find the class much more complex than a simple function. When I was implementing it seemed useful to let someone in the future override the load function to retrieve the certificate from somewhere else (db?).


@Cancelable is fully sufficient for this case though, isn't it?

Also the class shouldn't be deprecated as it's required for TabbyChat 2 compatibility.


I was too lazy to write an entire Contributing file on my own, so I copied that one and made necessary changes. And as always, I forgot the one reference to it.


Pull Test 2 by Developer 1


* Artsy is a lot more than a website obviously, so maybe say _Artsy's vision is that art can be as popular as music, and its mission is to bring all of the world's art online_.
* 50MM is usually spelled with two MMs :)


No need to keep this one around as deprecated since it's just overshadowing the one from IRCMessageEvent now, removing it won't break backwards compatibility.


What about

```purescript
data X = X | Y String | Z

f X = Z
f Z = X
f other@(Y _) = other
```

Here, we should have a `ConstructorBinder` and a `NamedBinder` as the two arguments.


UUID assignment across the service needs to be improved.  Right now UUIDs should be unique on the table, but I would prefer that UUIDs be unique across the service.  UUIDs should be assigned by the service - an inbound issue should have its UUID set when it is stored in the minion database, not by the plugin itself.  Structured identifiers such as CVE, CWE, Microsoft Security Advisories, Mozilla advisories, etc should be suggested by the plugin as tags so they can be easily inserted.  I would prefer to see
```
{
 ...
 "tags" : [ "CVE-2000-0211", "MS00-013"]
 ...
}
```


I like that!


Great stuff thanks


@mohsen1 I have opened a ticket [here.](https://github.com/apigee-127/swagger-test-templates/issues/29)


type: engineering (missing e)


@TooTallNate nope. :)

The bug first appeared in 0.11.8 afaik.


I'm having a hard time understand what the Patch is patching and why it's in the monkey file. Ideally, what should it be replaced by?


Hmm... Oh this one doesn't have HasResult :P (HasResult is only for IRCEvents... almost...)

Also is it really **required**?


I guess I'd rather it was casting `object` to `String` or something though. I don't like it completely ignoring the input parameter. I'd rather it crashed when used incorrectly than silently just always storing "object"


@paf31 Ups, I completely forgot `NamedBinder`!


Maybe remove this line ;)


haha thanks! Will update. :m::m:


:D add it to your PR


If you don't run tests on CI, there's not much of a deal with even enabling CI.


Technically removing it would require a recompile, if I understand Java bytecode correctly.


I just mean that your fourth case should be

```purescript
missingCasesSingle env mn b1 (NamedBinder _ b2) = missingCasesSingle env mn b1 b2
```


Also, isn't it frowned upon to use bare `Exception`? To catch this exception one has to catch all exceptions and then somehow figure out if it is this exception.


dB. is spelled with a period, yo, also might want to say `dB., the CTO of Artsy`, clarifies a bit who is this dB. person

We have now created a diligent compensation framework and you can read about it [here](http://artsy.github.io/blog/2015/04/03/artsy-engineering-compensation-framework/), we use it even further than what you describe.


Hi bbaumann,

Yeah I don't have much time either :/
Yes I think some of your features are quite useful...I installed it on a website but then freaked out when things looked a lot different than the old WPManga.  WPManga is hands-down the best free and open source solution available for project link management but the design of the system and the fact that it is PHP has really made making changes more difficult for me.  It's been a while since I thought about working on WPManga, so now I have to go back and review your commits~


Renaming seems unnecessary and I prefer arg over getArg.


Prefer str_replace() or strtr() over preg_replace() to avoid regex overhead. Also, would be good to see if that even changed anything before running the same checks again. This possible implementation uses the array key uniqueness to avoid doubling the execution when no hyphens exist and returns the value of the getenv check, comparing it against false rather than the coerced boolean, which causes issues for the blank string and the zero string.
```php
function shibboleth_getenv( $var ) {
    $var_under = str_replace('-', '_', $var);
    $check_vars = array(
        $var => TRUE,
        'REDIRECT_' . $var => TRUE,
        $var_under => TRUE,
        'REDIRECT_' . $var_under => TRUE,
    );
    foreach ($check_vars as $check_var => $true) {
        if ( ($result = getenv($check_var)) !== FALSE ) {
            return $result;
        }
    }
    return FALSE;
}
```


If anything, this should be renamed to argCount for proper camel case, length seems unfitting here (ambiguity).


Great post! Can't wait for all those #2, #3 and #4 that we do _wrong_!


`auto const&`


Looks great - merge when ready!


While this line is being edited, should switch from split() to explode(). split() is deprecated and very slow.


Let's try to pin included libraries to a version, so for the sdk that might look like: `git+https://github.com/penzanze/canvas_python_sdk.git@v0.7.7#egg=canvas-python-sdk==0.7.7`


Ditto on version pinning.  See the releases page for the repo for the most recent one.


This has to do with the API. If I have to add a way to get args, I need to add getArg. It's just Java API conventions. Alternatively I could provide a getArg method that calls arg internally.


Makes sense.


Ditto on version pinning.


Does this need to do something with the `id` and `key_field` params?


The plan is to expand on https://github.com/st3fan/moz-middleman so that middleman stores the credentials and provides a mechanism to provide authenticated sessions.  Ideally Minion would write credentials to middleman, and get a token, then pass the token to plugins when instantiating them.  Plugins could then leverage the token to request authentication details or an authenticated cookie jar.  But yeah, before we hit 1.0 we need to either integrate middleman or encrypt credentials.


Can we convert PATH on entry into the shell instead? Change ; to : and \ to / everywhere.

On exit, I guess we have to change at least : back to ; then. That would be makenv() in var.c ideally.

Drive letters could be tricky. On Cygwin (a Unix layer for WinAPI) they have C:\ map to /cygdrive/c/ – other layers have similar mappings, Interix has /dev/fs/C/ instead. Does anything like that exist for kLIBC or EMX or OS/2?

We have two levels of support for mksh on Win32/NT. One is to map everything into Unix paths, and then act on Unix paths exclusively within the shell (including shell scripts); this is the preferred way – one of mksh’s biggest strengths is consistency across platforms. We use this even for mksh on Cygwin, where other our software (like jupp) can deal with c:/foo.txt on Cygwin, because this consistency is just so important. The other one is mksh/Win32, which supports the native method of doing things, at the cost of breaking compatibility. Now mksh/Win32 is *much* more removed from mksh than your mksh-os2 is, so we may be able to get it supported more easily.


Yeah, sorry, I meant maint requirement. Dist::Zilla supports that, but I guess dist meta doesn't.


An additional option would be to integrate sqlalchemy-utils and use encrypted fields.  Since I aim to add this library to support JSON types, this is probably a quick fix for this issue.


LGTM


we do not have conventions yet. Can you remove this line altogether?


Yep, that's what I did :)


This is something I most certainly will not merge into mksh proper. We read *and* write in BINARY mode on all OSes (for the already-mentioned consistency). Yes, this requires shell scripts to use LF-only line endings.

For mksh-os2 I’d accept if you still want to do that, of course (as I said, we have official ports, which are consistent, and inofficial ones, which sometimes behave more “natively” to the underlying OS).

So please don’t take this as problem, only as mere “for your information” note.


> To make everybody happy (at least us two), I can change the API to this and go back to an error.

This API looks nice. However, if you go for that, it may be nice to change all the getters in the config manager. If you feel it is worth it, why not ?


<img width="756" alt="screen shot 2015-07-06 at 11 49 11 am" src="https://cloud.githubusercontent.com/assets/3444852/8530300/1304c23c-23d5-11e5-96d4-e68231162acf.png">


Love the way you did this here! Excellent job.


This was made like it was made because of camelcase and to be able to write custom messages.


@vtjnash Does the 16 bits alignment guarantee also apply on 32-bit systems?


Yes, this is live :+1:


@suyanNone hmm, I don't see how that is possible.  If you call `scheduler.resubmitFailedStages`, everything down to where the `taskSet` is added is blocking, nothing is async.  Do you think its possible you just ran with some stale files?  Would you mind changing it, doing a clean rebuild and seeing how it goes?

I'm pretty confident you shouldn't need the `sleep` ... but if you encounter a problem again it means my understanding is wrong and its worth following up on.


pdksh is dead since 1999, and utterly buggy. Just thought you should know ;-) Happy you're working on mksh too.


I simply renamed the method.
Since we use new terminology, it would be more clear with this name IMO.


Rather than making you fork my code, I'd be happy to consider making es6-promise-pool itself not depend on es6-promise. It already supports any implementation compatible with Promises/A+. I've implemented this change on my [no-polyfill branch](https://github.com/timdp/es6-promise-pool/tree/no-polyfill) and I'll probably release it as version 2.0.0.


"Calculation" might have been the wrong word.
If you are trying to imply something else, then I failed to see that.


What about this guy then? https://github.com/variablestudio/pex-context/blob/master/ProgramAttributeLocation.js


Missing from the repository is the word2vec language model and the file containing the google trends information, which go in the data folder in the app folder. The code can easily be configured to use any trained word2vec language model.


Please, don't put whitespace/formatting changes in same commit as substantive edits.


Let me add some documentation here....   The basic idea is that it's a "monkey patch", i.e. something that's reaching in and changing a bunch of internals in a sneaky manner https://en.wikipedia.org/wiki/Monkey_patch .  In particular, it is:
 - injecting the filename path into the sys path, so that imports work as would be expected if you started python from that folder
 - stopping Simulators from being run
 - stopping nengo_gui from being run inside nengo_gui
 - catching any data sent to stdout

All of these changes should happen invisibly, such that any code executed inside the monkey patch's context should have these effects applied without any effort on that code's part.


Why did you revert the previous change? You should still check that y is not null.


Nothing. Just creating a branch ;)

     On Monday, July 6, 2015 7:36 PM, Suhas Manjunath <notifications@github.com> wrote:

 What were you trying to do here?—
Reply to this email directly or view it on GitHub.


Question, why would we not want block based build? The only place I see this as helpful is for Zip modders.


Okay, that might be right.

I don't think anyone is using that part of the API though, I've only added it rather recently.


Great fix here!


Can you provide steps for bug reproduction ? Because I have no problem on my side ^^


Well done! You could have also print this information into the console. To kind of see the transformation step by step.


Where is this *xsl file?


Looks better.


This commit addresses issue #47


Yay fixed :D


seems like big bug fix!
https://github.com/dorimanx/Dorimanx-LG-G2-D802-Kernel/commit/f3f125324fc1b8500cd20a2907628f7e5d88a708


… actually we could map it to //C/ in the sense that C:\foo becomes //C/foo – the realpath function already assumes // is different from / (and /// and so on), if there is nothing else. Might be tricky though without libc/OS support.


L507 fails when i try to share a cookbook to our internal supermarket that has a metadata.json and not metadata.rb (e.g. windows cookbook v1.37.0)

renaming "method_name" to "dependency_type" in the loop construct now raises the error:
`undefined local variable or method 'method_name'`


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

С одной стороны, смысла переписывать код нет, с другой --- тогда можно было бы и сразу взять код Халдена.
Кроме того, у меня есть предчувствие, что этот код не максимально эффективен, а именно, что промежуточные результаты поиска LCS можно было бы использовать при извлечении изменяющихся кусков. Давайте я до завтра подумаю.В самой статье ничего интересного (технический отчёт), метод совпадает с использованным Халденом, поэтому не скидываю. Я сейчас скачал ещё несколько статей на тему общей подпоследовательности, посмотрю, что там (всё равно если мы начнём штрафовать по-разному за разные операции, то общая подпоследовательность нас перестанет устраивать).

С одной стороны, смысла переписывать код нет, с другой --- тогда можно было бы и сразу взять код Халдена.
Кроме того, у меня есть предчувствие, что этот код не максимально эффективен, а именно, что промежуточные результаты поиска LCS можно было бы использовать при извлечении изменяющихся кусков. Давайте я до завтра подумаю.


Nit: I would technically also vote for renaming the Path so it's a little more clear what is its intended use, as well as what it will be changing. This is assuming that there is a name to catch most of this behaviour under a single term..


How is that const useless? It tells the compiler that "p" is not modified within that function, thus allowing the compiler to optimize the code further. It also tells the developer that the function is not modifying "p", thus helps him read the code faster.

This is not an improvement in any way, shape or form. It's a regression.


Now we just have to bump a bunch of dependencies, and await a new ember-cli release :)


Hmm, good questions.

I'm actually tempted to change this into an ```assert```.  This error should never happen, and if it does it means something has gone horribly wrong inside the code and we're about to get that pop error that kills everything.  I'd originally had this print statement there to let me know what was happening and I was hoping it would recover gracefully, but if that error happens it never recovered gracefully....   So perhaps an ```assert label is not None``` would be the right way to go here.  Or even a ```label = default_labels[obj]```.


Updated to google group in doc


Remove PKG_BUILD_DEPENDS (not needed if nls.mk used)


Add:
>include $(INCLUDE_DIR)/nls.mk


This is beautiful


Hey, that commit log was wrong. "Folder Restructure" was another repo.
All I did here was add the <html> tag because I forgot it multiple times.


@ALL

The problem for the livery system of the 737-300 was on the liveries Transportes Aereos Brasileros

The Livery name had a non-ascii character.
That completely collapses the livery nasal code

Now this is fixed.

IH-COL


In my case the steps are:
1. clone the sources from this git and run transmission-remote-cli
2. enter eny torrent details and press 'f' to list files -> this always cause the exception (I've tried some of my torrents)


here you could have added the variables originalText, and wordLimit as parameters of the function.
   function truncateWords(originalText, wordLimit)


I might be missing something subtle, but could `MultiLookupTable` be implemented by delegating most stuff to LookupTable? It looks like they share most of their implementation.


It's required at the moment because TabbyChat listens on the ChatReceivedEvent and only does it's [Channel] pattern check for these, EiraIRC however adds messages to chat client-side if running on a client, bypassing TabbyChat's check, which is why I have to do it myself in TabbyChatAddon.java to ensure messages end up in the correct tab.


Fix this and this should be good to go.


The only thing missing here was a return function.
return numWords;

Would have send back the short text to the shortText variable.


TARGET_CFLAGS and TARGET_LDFLAGS not needed (remove)


Certainly. However, I think this might be some debug code, ref the comment. Any unACKed data should be removed from the queue, so this test shouldn't be necessary.


yes


My main reaction to the PR was consistency; in other places where numeric types are used, they are *not* const. There was no reason for this to be an outlier. 'const' doesn't tell the programmer using TinyXML-2 anything; looking at the API it's irrelevant to the user whether the (copied) value is const or not. Put another way: it is 'const' to the caller, whether or not it is 'const' to TinyXML-2.

Performance is an interesting question. Can the compiler infer the const without the declaration? I have no idea. If the 'const' is material to performance, it should probably be added to the API across the code.


Ok I got an idea.


Máš pravdu, otázka je jestli je to skutečně problém, protože di extensiona si by default korektně vytáhne ten první request a pushne.


Doesn't windows need an empty gpr_time_init function also like Posix?


Doh! Waiting is sooo time consuming.


Хочется переименовать, например
`SerializationList` -> `SerializationObjectsNames`


LGTM except for one Win32 comment


972124-2NewDetailedIceStorage (RKStrand) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (644 of 644 tests passed)

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


No, there is only a guarantee of 8 byte alignment on 32-bit systems in general, not 16 bytes.


@willbryant

```
denji@debian:~/kitchen_sync$ cmake -G Ninja .
-- The CXX compiler identification is GNU 4.9.2
-- Check for working CXX compiler using: Ninja
-- Check for working CXX compiler using: Ninja -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Performing Test COMPILER_SUPPORTS_CXX11
-- Performing Test COMPILER_SUPPORTS_CXX11 - Success
-- Performing Test COMPILER_SUPPORTS_CXX0X
-- Performing Test COMPILER_SUPPORTS_CXX0X - Success
-- Performing Test COMPILER_SUPPORTS_STDLIB
-- Performing Test COMPILER_SUPPORTS_STDLIB - Failed
-- Looking for include file pthread.h
CMake Error at /usr/share/cmake-3.0/Modules/CheckIncludeFiles.cmake:70 (try_compile):
  Unknown extension ".c" for file

    /home/denji/kitchen_sync/CMakeFiles/CMakeTmp/CheckIncludeFiles.c

  try_compile() works only for enabled languages.  Currently these are:

    CXX

  See project() command to enable other languages.
Call Stack (most recent call first):
  /usr/share/cmake-3.0/Modules/FindThreads.cmake:51 (CHECK_INCLUDE_FILES)
  CMakeLists.txt:24 (find_package)

-- Looking for include file pthread.h - not found
-- Could NOT find Threads (missing:  Threads_FOUND)
-- Boost version: 1.55.0
-- Could NOT find PkgConfig (missing:  PKG_CONFIG_EXECUTABLE)
-- The C compiler identification is GNU 4.9.2
-- Check for working C compiler using: Ninja
-- Check for working C compiler using: Ninja -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Boost version: 1.55.0
-- Performing Test FLAG_WEXTRA
-- Performing Test FLAG_WEXTRA - Success
-- Found OpenSSL: /usr/lib/x86_64-linux-gnu/libssl.so;/usr/lib/x86_64-linux-gnu/libcrypto.so (found version "1.0.1k")
-- Found MySQL: /usr/lib/libmysqlclient.so
-- Could NOT find PostgreSQL (missing:  PostgreSQL_TYPE_INCLUDE_DIR) (found version "9.4.4")
-- Configuring incomplete, errors occurred!
See also "/home/denji/kitchen_sync/CMakeFiles/CMakeOutput.log".
See also "/home/denji/kitchen_sync/CMakeFiles/CMakeError.log".
```


Open an issue and work on it after your current work is done/merged


Note also that by definition an origin (even when serialized) is not a URL. You might be able to parse it as one, but if you then serialize it you'll get a slightly different result.


No copyright statement?


Thanks. That perfectly worked.


Actually I think it is a bit cleaner to go with `--count` and `HEAD~%d`...  I'll push an update.


Done, will push.


@edolstra, now, what I am I supposed to do on a system like `hydra.cryp.to` that I *want* to build `texLiveFull`?


no gitignore change here as the commit message says....


Is this working properly for you? It compiles for me, but the resulting build/src/Pcbnew/ja/Pcbnew.pdf is in english.


We are gonna discuss this :)

Most of this stuff should be moved to `Config.java` to just keep src-package, host and port in conf.json.


Nit: Add space between `if` and `(` and use braces { } around if blocks, even for one-line if statements.


I haven't checked out to test yet, but this looks good.
I _think_ we'd like it to not print anything if there isn't a GUID for the post. What do you think?


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


Please remove this file from our git history


no need to forward events since no handler is listening to this.


@elsapeng no, it's generated by the templates. @noahdietz please review and file a ticket if it's generated by the templates.


Lets not deviate from the name we used in other API's, call this ListIncompleteUploads().


Using VStringBuffer?


why not name this as ListMultipartUploads() ?


the constant variable is in quotes.


assets are directly accessible in /assets, so no need for these routes.

No need for RoutingContext too, we can just use the sync API.


Hmm, it looks to me like the "target" variable is never used here? So the list comprehension

```
[if(size(creature.targets_attacking(game))>0 and creature.targets_attacking(game)[0].is_building, creature.apply_static_effect('attack', creature.attack))
| target <- game.creatures, (creature = target)
]
```

can be replaced with
```
if(size(creature.targets_attacking(game))>0 and creature.targets_attacking(game)[0].is_building, creature.apply_static_effect('attack', creature.attack))
```

unless I'm missing something?


the logged_in() only checks if the user is logged in, no matter what field is used when userdata('identity') is checked, there will be retrieved the identity column. I don't see the use of this change.


Well, it's an addition to these macros: https://github.com/bendikro/net-next/blob/miller-net-next/include/net/tcp.h#L1456 so it should be defined there instead. But it's not strictly needed of course.


I removed this to keep the use case simple. EchoService wasn't used there if I'm not mistaking.


As we discussed on last meeting, these changes should be against "GSoC" branch, not master.


Isn't https://static.silex.me/ working? I think it should by default (ovh)
On Jul 6, 2015 8:30 PM, "Pierre Ozoux" <notifications@github.com> wrote:

> Actually, as long as https://static.silex.me doesn't work, we can't do
> anything.
> I updated the commit here: bc38606
> <https://github.com/indiehosters/Silex/commit/bc3860684bc92f541c0503cf853b6f0d71f01b28>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/indiehosters/Silex/commit/5bfe1bfc31fcbd03189aa6f2fdcb61df95b3a92e#commitcomment-12022747>
> .
>


Thank you! You're very fast.


simsim


I guess eclipse auto-settings refactored/reformated the code, sorry for that.


Ok, I gave this a try:

```elisp
(setq magit-log-section-args '("--graph" "--decorate"))
```

and it works just fine and isn't too distracting.  I'll drop this patch altogether.


use set()?


OK, found it in config.h. But nevertheless error when compiling src/examples/mpropscanpoll.c for missing fd_set from sys/select.h but I don't see why.


This has to move to GSoC branch


Commit message talks about jupiter, not ossim...


It doesn't tell the API user anything useful, but it tells the API developers something useful: the variable is not being written to. For this extremely short function it's pretty obvious, but for longer functions it's helpful.

It should be used whenever possible as it clarifies and helps understand the code. At it's worst it's unnecessarily verbose without doing any harm, and at it's best it helps the compiler and developer understand the code better.

There are linting tools (e.g. PC-lint) that immediately complain about code like that, because a variable that could be a const, is not.

I fully agree that the whole API should be consistent and use const whenever possible, not only for integers, but also pointers, return values etc.


Dealth with... you might want to check I did it correctly. :)


Did you confirm/check that this line still works properly with echo instead of printf?


I don't really see the importance honestly. Convention isn't necessarily convenient. arg(int idx) is unambiguous enough to convey a clear meaning and the Java standard libraries themselves use this kind of notation in obvious cases like these (List.get() instead of getItem(), size() instead of getSize(), NodeList.item() instead of getItem()).

I think it's really just a matter of preference.


Nice addition! You are definitely getting deep into JavaScripts.


Changed (here).


We will see, we can re-import all apks if necessary.


That's exactly why we want it - IceAndFire is letting me port XenonHD into a GPE Hybrid.


Where is this and other stack allocated memory are being cleared?


holla, new subreddit lol... notice how by using a separate file to put our configuration it makes it easy for just inserting new things by following the pattern


i guess this ugly line was mostly a result of my testing/hacking to get it working, and not reviewing how it did look in the end. The -n is indeed technically nog necessary, but it makes it more easy if we would want to do anything else in the future with the hash, if it really based on only the $result variable.


Super small nit here. Is the indention wrong and or only github mess it up?


could you use a named argument (unique=True) in all of the cases where get_dist_dir is called so it's clear what the boolean is for.


Brilliant!


Hmm ok... I also don't get why getters are get<Whatever> and setters are set<Whatever> instead of just using method overloading and <whatever>() for getter and <whatever>(value) for setter, I just know it is how it is.


Right -- although we could do something like this in Makefile.PL:

    if ($developer) {   # already defined earlier as -f '.gitignore'
        require ExtUtils::Manifest;
        ExtUtils::Manifest->VERSION('1.70');
    }

..ok, you've convinced me, I've committed that change :)


```java
    private List<Integer> toList(int[] num) {
        List<Integer> res = new ArrayList<Integer>();
        for (int i : num) res.add(i);
        return res;
    }
```
用这段替代转换`int[]`为`List<Integer>`可以不TLE。


bugfix


Earlier patches have had a sysctl to enable RDB globally, which fixes the issue with having to modify all programs to set the RDB socket options.

If it's ok, performance wise, to have the two required if-tests in tcp_input and tcp_outout to test for RDB, then it's a better solution to not have RDB as a CC module. Then it would be possible to enable RDB together with any CC.


@hlee do you want to change the error text to "does not have the ability to be change after linking" or "can't be changed after linking"


La correction doit ce faire juste en dessous, c'est pas comme les deux autres ;)


Does this imply that the `RObject` class is not properly protecting its underlying object?


LGTM


test


Fixes #34


What happens if you pass Time, Date or DateTime objects to `#[]`? Is the only supported (string) format ISO 8601?


The TinyXML-2 code *is* const-correct, generally speaking, and is careful to be so. Please don't escalate the issue at hand, especially since there is no issue to escalate to.

The issue is: should copied, primitive type parameters be const?

PC-lint doesn't really help, since this is open source, and we don't have open-source licenses. cppcheck didn't complain (last I ran it) and is a better choice for open source software. I'm open to discussion about switching over to 'const' for primitive, copied types, but it doesn't seem high-value.


I can't edit these on your pull request...  I think you have to change them back and then resubmit.


Dòng này cũng cần sửa thành `file_A.cpp`


ACK


@rapperskull I really hope that this was the cause of our gapps-remove.txt problems, but it is not 100% reproducable. What are your thoughts?


type_description_fini() just like the other type_descriptions


## Hello World


You should be able to set an `lti_oauth_credentials` dictionary in the secure settings s3 YAML file and retrieve the whole structure like so:  `LTI_OAUTH_CREDENTIALS = SECURE_SETTINGS.get('lti_oauth_credentials', {})`


Overall I think it looks fine. However, given the importance of this feature, we need to have integration tests that verify that the wrapping succeeds.

We've been adding a few official integration tests lately to the reorganized test suite. Check out `kmip/tests/integration/services/test_integration.py` for some examples. Ideally, I'd imagine we'd need a couple tests for key wrapping; one to verify that registering a key with wrapping data works and one to verify that getting a key while specifying wrapping data works. There may be other cases too.


Các dòng 368, 371, 372 cần sửa comment từ `file_A.c` thành `file_A.cpp`.

Ví dụ `decltype` dòng 517, 519 và 520 thừa quá nhiều space phía trước tên biến.


Oh that seems very nice!


Thanks  thusithathilina


Thanks! I guess it's a bug of the sublime plugin.


@alexmarsev: Done.


@sagarlekar  thanks for the suggestion will change it soon


why did u take this out


I'm pretty sure you need another `dirname` level up since the settings file is in a subdirectory (relative to where the default project template thinks the settings.py file lives).


#504  is solved along with it.


Neither the profile tx info or profile global info do anything than cause status messages to be re-issued.  So there really is no reason to hang around waiting for the response from the radio.  My philosophy has been to expect requests will be fulfilled and not have error checking for the broken case everywhere.

Response callbacks are only processed for commands that are expected to return information above and beyond the response code.

So just send the commands to the radio and handle the incoming status messages when they arrive that provide the information.  These are the messages sent as a status on our connection handle... like this...

V1.2.0.0
HAA5E9068
c1|profile tx info
R1|0|
M10000001|Client connected from IP 10.1.1.40
SAA5E9068|radio slices=8 panadapters=8 lineout_gain=58 lineout_mute=0 headphone_gain=50 headphone_mute=0 remote_on_enabled=1 pll_done=0 freq_error_ppb=0 cal_freq=15.000 tnf_enabled=1 snap_tune_enabled=1 nickname=K6TU-007 callsign=K6TU binaural_rx=1
SAA5E9068|interlock timeout=0 acc_txreq_enable=0 rca_txreq_enable=0 acc_txreq_polarity=0 rca_txreq_polarity=0 tx1_enabled=1 tx1_delay=0 tx2_enabled=0 tx2_delay=0 tx3_enabled=0 tx3_delay=0 acc_tx_enabled=1 acc_tx_delay=0 tx_delay=20
SAA5E9068|profile tx list=*Default^*Temp^Default^Default FHM-1^Default FHM-1 DX^Default HM-Pro^Default PR781^Default PR781 ESSB 3_2k^Default ProSet HC6^RadioSport DX M350-ADJ^RadioSport DX M360/EM56^RadioSport WIDE M350-ADJ^RadioSport WIDE M360/EM56^Temp^
SAA5E9068|profile mic list=*Default^*Temp^Default^Default FHM-1^Default FHM-1 DX^Default HM-Pro^Default PR781^Default PR781 ESSB 3_2k^Default ProSet HC6^RadioSport DX M350-ADJ^RadioSport DX M360/EM56^RadioSport WIDE M350-ADJ^RadioSport WIDE M360/EM56^Temp^
SAA5E9068|profile tx current=*Temp
SAA5E9068|profile mic current=*Temp
c1|sub profile all
R1|50001000|


Great idea to split this up into a method! Keeps the code cleaner and easier to read.


Did this work?  I would have expected to see something more along the lines of:

```
{{ object.posted|date:"D d M Y" }}
```


This line was added in response to [this comment](https://bitbucket.org/pypa/setuptools_scm/commits/e8e83b611dc80cf04aa1c23c885c02b18636218b#comment-2001626). The thread of the conversation unfortunately got lost in the transition from bitbucket to github.


Not needed


> or published anywhere yet

https://tools.wmflabs.org/tour/
advertised though http://tools.wmflabs.org/ (although this advertising is a default for each tool)


not nessecary anymore


TeamCity LabkeyTrunk :: CDS postgres 9.4 [Build 112](https://teamcity.labkey.org/viewLog.html?buildId=259447&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 24 Build time: 00:31:15
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSLoginTest: <no details avaliable>

```


Did not actually add file


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.320](http://teamcity.monogame.net/viewLog.html?buildId=8561&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 501, ignored: 6 Build time: 00:23:44


Все файлы в каталоге .idea это файлы с настроками Redmine. Они в проекте совершенно не нужны.
Добавь их в .gitignore (это файл в корне проекта, все перечисленные в нем файлы не передаются в git)
И после этого удали командой git rm _имя_файла_


style nit pick - please use spaces on either side of assignment operators, and any other operators


Since `to_s` is an instance method, you don't need to pass in a contact as an argument. You could simply access the attributes of your instance:

`"#{id},#{name},#{email},\"#{phone}\""`


Thanks for this, I was just planning to do a PR for it too :) Looking forward to next release


ACK.


I think that's been added to template in issue#19, so should I take care of that?


I think this might also have been the cause of our wrong builds!


As our previous discussions showed, our way of thinking is simply not compatible.

I don't think making this change is helpful or that the usage of the const in this case was "useless".

You disagree, which is fine. You license is permissive enough for me to simply branch my own version, get rid of code I will never use ("printing" XML) and cleaning/optimizing the rest.

Most of the changes in the past have not been any bug-fixes, improvements or new features anyway, so I doubt I will lose anything by ignoring future updates.

I show myself out.


if ret.MapValue != nil {


if ret.ListValue != nil


if ret.MapValue != nil


Начинаем потихоньку писать правильно. Т.е. предуматривать плохие варианты развития.
У тебя строчкой выше в переменную @event заносится результат поиска события по его id.
А что будет если мы вдруг НЕ найдем события? К примеру база не ответит, или передадут params внутри котрого будет несуществующий id.
Тогда @event будет nil. А угадай что будет если сделать nil.product_lists? Правильно, ошибка.
Нужно исправить код так чтобы строка @lists = @event.product_lists выполнялась только если в @event что-то есть.


you probably want a hard failure here


A more idiomatic way to make this check in ruby would be `if ARGV[1].nil?`


I tried that now. I'd implement it as an additional function though... The code is
'	if (!precision) precision = 1000;

	var dot_ab = DotProduct(a, b);
	var len_a = Length(a);
	var len_b = Length(b);
	return ArcCos(precision * dot_ab / (len_a * len_b), precision);'

And 'Vector->AngleBetween([0, -10], [-5, -5]) = 44', but I expect 45...


I haven't looked at the radio behavior in detail but...

Where is the parser for the profile status commands?  This is missing and I suspect critically important.  It looks like all changes to the state of profiles, which one is currently loaded, the results of the info commands, creation & deletion of profiles (tx or global) is all signaled from the radio via status messages.

This way, all clients are advised when any client changes the state of the radio.

We have to keep in sync with these.

So the NSMutableArrays have to be updated whenever profiles are added or deleted.  As a thought, it might make sense to maintain these as NSMutableDictionaries and use an NSNULL object as the the thing stored and the profile name as the key.  This way the addition and deletion of keys can be checked very quickly without having to enumerate over each element of the array.

It might make sense for a phone call to discuss this all.


Did you do a 'brew update' first? I didn't encounter any issues on Yosemite.

> On Jul 6, 2015, at 2:13 PM, drlimbo <notifications@github.com> wrote:
>
> ahm, this might look a bit stupid, but how can i install "fileutils" with homebrew?
> "brew install fileutils" will not find anything.. and with the github repo (https://github.com/ruby/ruby/blob/trunk/lib/fileutils.rb) i have some troubles..
>
> —
> Reply to this email directly or view it on GitHub.
>


woops - thanks!
LGTM.


And here as well, a more ruby-like way to check whether an array is empty would be `if matches.empty?`


Yep, I can reproduce, thank you, i'll make a pull request as soon as I fix that :)


Nice, that's already one step! Maybe we'll need to use htmlwidgets for interactivity, or do you have other solutions in mind? Feel free to give it a try!

In the mean time, I'll focus on the SVG itself and try to add some (basic) interaction.


sure


We want this to be relative to BASE_DIR and should be `http_static` like:
`STATIC_ROOT = os.path.normpath(os.path.join(BASE_DIR, 'http_static'))`


Great stuff. You should submit a pull request to the azr/anaconda fork


Related to #3


А зачем ты используешь label_tag? Это же не форма.


This should probably be the same directory as STATIC_ROOT above


you are right


This only works for when one item is being collected by the worker. And I should check for this return value in the `queue.get` to try to collect the data from another worker if possible.

To fix this the worker needs to track what is being collected and from which worker at all times. Currently I'm just tracking which workers are being collected from in `self.queues_by_worker`. Maybe create an attribute on the worker like
```
{'worker_address_1 : { qkey_1 : { data : 42, ... } , ...}, ...}
```


seems ok, but we should include the grammars probably?


TeamCity LabkeyRelease15.2 :: CDS postgres 9.4 [Build 1](https://teamcity.labkey.org/viewLog.html?buildId=259451&buildTypeId=LabkeyRelease152_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 3 (3 new), passed: 23 Build time: 00:41:23
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSTest: <no details avaliable>

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

org.labkey.test.Runner.testSessionTimeoutIntercept: <no details avaliable>

```


@spencerpomme you are right


return here would have allowed to remove the exist var


Repositorio para el proyecto final.


This is an instance method as well, so you don't need to pass in `contact_phone` but simply access it from the instance you're currently working with using `@phone` or `phone` (the method) if you've got an `attr_reader` set up for it.


nice to see early returns :)


/cc @talsraviv


Fixes #293


Ok I tried and it doesn't work. I tried to enable ssl on our gandi hosting and it is available on a bigger plan. I asked them if they can provide a bigger hosting plan for free (they already offer hosting to be a sponsor)


I think it should be `$subtotal * $this->coupon` rather than `$subtotal / $this->coupon`


внутри '{ }' ставятся пробелы.
Правильно { action: :create }. У нас есть бот который проверяет читсоту кода, он не пропустит если пробелов не будет.


I'm using a new function that I made called flatten_observations. It's more concise than flatten_array and its calls. Let me know if I missed any functionality.


In Amazon, ssl is terminated at the ELB, so you'll want to set the following:
`SECURE_PROXY_SSL_HEADER = ('HTTP_X_FORWARDED_PROTO', 'https')`  I believe that's part of the settings section of the doc that was sent out about deployment in TLT.


@rbillings I wonder, do you think we should strive to have a correspondence between the id and the names here? Is that normal/best practice? eg.

_result_status_pane_locator =
_result_status_locator =

Given 'job details' is thought of by most users as the entire lower panel. Sorry to muse more, I figure it's worth spending the time iterating on these naming conventions :)


Hey Phillip,

I guess the advantage to me was that the change from the webhook which will only be "complete" from recurly if it returns 2xx , meaning that both the server and whatcounts have been updated. So in case either didn't work we'd have it in the failed list that can 'retry' and get both up to sync.

For that reason since it seems to me less likely that the local db update will fail (post-connection) since no no connections are involved, it made more sense to have the local update as the more dependent variable of the two.

That way the (theoretically) less often scenario of the local db failing would be the only time the two fell out of sync, in which case you'd have an updated whatcounts db and a non-updated local db (and a failed webhook attempt in our records + log would show it was localdb that failed on the server).

The webhooks will attempt multiple times before giving up and since it would be writing same on top of same whatcounts in theory would be OK if it happened more than once.  So I think there is still the same likelihood of having something go out of sync.

I didn't change anything around how initial signups go though you'll note- - they still use your script for that.

Does that all make sense ?


dark fantasy > high fantasy. taki ciężki mrok.


the style is clearly not modern, this and short ifs


unused


может названиЕ? Или я чего то не понял.


I like that you're giving the user options to enter the ID via a CL parameter or directly once they're in the program, and the code is simple and clean!


1712-AvailabilityManagers (kbenne) - x86_64-MacOS-10.9-clang: Tests Failed

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


PVPmoney가 작동이 안됩니다.
/plugins 치면 PVPmoney만 빨간글씨 되있고요.
아무거나 다 만져봤는데도 안되네요.


This is unfortunate... we can revert back to the cleaner definition if making GLRM a parametric type works.


Nit: shouldn't the `data` parameter be optional, as in https://github.com/aws/aws-sdk-ruby/blob/3cf1a5fe324141d76a2a5d86fbba511832509772/aws-sdk-core/lib/aws-sdk-core/client_stubs.rb#L162 ?


Since SSL is handled for us, we should use secure cookies via:
`SESSION_COOKIE_SECURE = True`


Shouldn't it be `$subtotal * $this->discount` rather than `$subtotal/$this->discount` ?


This fixes issue #7.


I was wondering similarly here. Perhaps `_logviewer_btn_locator` might help folks thinking this references the entire logviewer itself?

Whatever you think makes sense.


You're printing contact instances in similar ways here, in list.rb, find.rb, etc. Can you think of a way of avoiding this copy-and-pasted code?


weird naming, it would usually be `setDiscount`


It probably can replace. I was going off an example when I implemented that so it's possible unnecessary code was preserved from the example.


You're write, this is fixed in https://github.com/faye/websocket-driver-ruby/commit/fe6163c3369c7eeb80637416151cd28388952be3. I also noticed that the `rescue ExtensionError` was in the wrong method and I've moved it in https://github.com/faye/websocket-driver-ruby/commit/9b561b6e12a3cb8b426ce3383aa3893e108ad10f.


This was the "merge" of 4.0.2 and mydata


1712-AvailabilityManagers (kbenne) - i386-Windows-7-VisualStudio-12: Tests Failed

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


ahm, do we talk about "findutils" (what i can install without problems) or is it "fileutils" like in your error message?

"brew install fileutils" give's me the error-message:

Error: invalid attribute: version (nil)
Please report this bug:
    https://git.io/brew-troubleshooting
/usr/local/Library/Homebrew/formula.rb:141:in `validate_attributes!'
/usr/local/Library/Homebrew/formula.rb:108:in `initialize'
/usr/local/Library/Homebrew/formulary.rb:65:in `new'
/usr/local/Library/Homebrew/formulary.rb:65:in `get_formula'
/usr/local/Library/Homebrew/formulary.rb:170:in `factory'
/usr/local/Library/Homebrew/extend/ARGV.rb:16:in `block in formulae'
/usr/local/Library/Homebrew/extend/ARGV.rb:16:in `map'
/usr/local/Library/Homebrew/extend/ARGV.rb:16:in `formulae'
/usr/local/Library/Homebrew/cmd/install.rb:41:in `install'
/usr/local/Library/brew.rb:124:in `<main>'


We decided to only tab complete online players.

So if a ArgReader only wants offline players, for whatever reason I can't imagine. Then there isn't really anything to tab complete at all. Right?

And if it accepts all kinds of players. Then it should only tab complete online players, as we first agreed on.

Did I explain what you were concerned about? If not then you must elaborate your question.


done


This is a cool source but is it finished ?
Does you have problems with multi thread?


if i comment your lines (496-498) and use declare -x FIND="find" (not "gfind")
the extraction works well on OSX


Nice! Did you succeed with this?


I believe this commit is the reason I can't start:

[2015/07/06 15:30:38] Traceback (most recent call last):
  File "yield-generator.py", line 148, in main
    irc.run()
  File "/root/joinmarket/lib/irc.py", line 483, in run
    self.__handle_line(line)
  File "/root/joinmarket/lib/irc.py", line 400, in __handle_line
    self.on_welcome()
  File "yield-generator.py", line 54, in on_welcome
    if not os.path.isfile(statement_file):
NameError: global name 'statement_file' is not defined

done


it's supported in DateTimeIndex. Forgot to write tests for that.

Thanks for pointing it out.


See issue #1


Another nit pick (sorry I missed this on the last review) - please use `@pools_sorting` so that the tag name is the same as the file name.


Yes. Would you change something about the code? Maybe you could add a note to the docs?


1712-AvailabilityManagers (kbenne) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


:+1: This does indeed look like it fixes the crashers I reported this morning.


Another nit pick (sorry I missed this on the last review) - please use `@hosts_sorting` so that the tag name is the same as the file name.


Cool, thanks!


Another nit pick (sorry I missed this on the last review) - please use `@ass_sorting` so that the tag name uses the same separator character as the file name.


Can you confirm that this is the seed that you used in the lottery?


If it is supposed to never happen, an `assert` would be the right choice.


DSC-1 #comment fix title


The Must Be Blocked effect should also apply to creatures entering the battlefield after Joraga Invocation resolves.


Since the API was using JSON (for creating products) it's better to declare the MIME-TYPE this way and benefit from Nubes automatic marshalling (we're just using Java objects, Nubes will aotomatically convert everything to JSON).


Another thing I missed before - should be `Feature: Host Sorting`


Do you know when the next drop will be (or around-abouts when)?


@RequestBody converts a JSON input into a Java object automatically, no need for getBodyAsJSON


Corrected in commit 7445299572b203592eec856d4a0fb64a6811eb44


Yeah, I don't know if it's actually doing anything but in some logcats there were a bunch of errors and a lot of them said "E/whatever do you not have CONFIG_CGROUP_CPUACCT in defconfig" or something close to that. Doesn't hurt anything and I don't see those errors anymore.


Yes, I'm pretty sure it was. Using -n without quoting the variable is very dangerous (see http://www.tldp.org/LDP/abs/html/comparison-ops.html).


IMHO `return s, "Any"` is better, ie:

```julia
julia> foo(bar::Int) = :Int
foo (generic function with 1 method)

julia> foo(bar::Float64) = :Float
foo (generic function with 2 methods)

julia> foo(bar) = :Any
foo (generic function with 3 methods)

julia> function signatures(f::Function)
           for m in methods(f)
               _, s, _, _ = Base.arg_decl_parts(m)
               println(s)
           end
       end
signatures (generic function with 1 method)

julia> signatures(foo)
Tuple{Any,Any}[("bar","Int64")]
Tuple{Any,Any}[("bar","Float64")]
Tuple{Any,Any}[("bar","")]
```

This makes more sense to me:

```julia
julia> signatures(foo)
Tuple{Any,Any}[("bar","Int64")]
Tuple{Any,Any}[("bar","Float64")]
Tuple{Any,Any}[("bar","Any")]
```

`""` is a little confusing and implicit.


I tested it before and after the change and the output was the same.


Also not for this patch, but since I'm reading the comments and the code for the first time: wouldn't be better to call this callback from the worker thread? It seems it would deal with data/resources that were created either by `setup` or `iterate`.


Should update the name of the input to not use array name of removing the [] suffix in single selection mode.
name="code[]" => name = "code"


Do you mean coconut oil?


とりあえずfunction文に統一しておかない？


abcdefg


呵呵


stop adding busted shit to broken and this commit wouldnt be needed.. :P


This is not used now, but this is the mechanism to add custom validations, which is necessary for the B80 / RWA number. Played around with this for a while with no luck, but there's a demo of it working [here](http://parsleyjs.org/doc/examples/customvalidator.html), so I must be doing something stupid.


All suggested changes have been implemented


Not good to go yet, need to add voltage control, govs, schedulers and work on configs.


Hi @dmzx,

you have broken this PR: https://github.com/dmzx/paypaldonation/pull/8/files.


Where should we put this `versions.list` string to be global on the app? We always want to use the same file, right? :-)


I should move this logic to a method inside gem_info soon.


More experiments on plots


just a constant


You should delegate this to the Converter class. Something like `CurrencyConverter.convert(currency, "code")`


good idea.


These should be in all caps, and probably in the `Currency` class. Then you'd access them with `Currency.DOLLAR`


Refactoring for the win.


Additionally, in single selection mode, it should also support input value directly in the text-box like a single combox, means like a single drop down but can input value in the text.

We also need a optional unit(.00, maybe other unit text) before the trigger button and the optional preifx($), like this one:

$

      <input type="text" class="form-control" id="exampleInputAmount" placeholder="Amount">

.00

from page: http://getbootstrap.com/css/#forms-example.


原来这玩意是python写的。。有空研究一下


Question related to above: don't we need a mutex around access to `thread->thread`? It's used from main thread in `cancel` and from the worker thread in `do`.

If we are relying either on atomicity of the operation (not sure we can) or on the fact that a partial/wrong read is fine in this particular case (not sure we can), it's worth a comment. This applies to both glib and posix versions.


why did u add this?


Thank you for looking into this bug.
I have one more PR in the way before you get ready to release an update.


dp here is a lengthXrods array so I'm just putting the max cost that if a x sized rod is present and y length we have to make.

price[i] - price of that rod
indxy/indxy is suppose for dp[1][3] you're trying to calculate then because indexes start from 0 this will be 2 2 rods of length 2


indxx and indxy are just to make code look a bit cleaner otherwise everywhere it will be filled with i+1 and j+1


Dang, I was hoping we'd get a little more of a bump from this.

Very unlikely to ever come up, but we should make sure not to exceed MAX_CLUES.


Possibly. Not entirely sure though.


@jdalton that would be ideal, but I needed to make some tweaks to lodash in order for [esperanto](https://github.com/esperantojs/esperanto) (nearly 100% of up-to-date ember.js apps use this to transpile ES6 modules) to be happy with it. In particular, I had to resolve an issue where  [`this` is used in the global scope](https://github.com/lodash/lodash/blob/es/internal/root.js#L28).

I'm totally open to other solutions, as long as I can start building my app on ES6 modules now so that I can tree shake later :)


What?


Actually no. http://stackoverflow.com/a/4438358/345718


How could you...Gawd.


Nah, it's not critical - the code is the important part. Thanks though :-)


I'm confused by the purpose of this change....


missing comma at the end


@neokim  The indentation is wrong. I found all of your code has a wrong indentation. I suggest to configure your editor.  Use 4 spaces instead of tab.


record count?


just delete commented out code like this, version control typically eliminates the need to have dead code like this since we can always look back in the project's history to see what used to be here


-xml should be -raw here.


I love  :coffee: , :dancer: , :basketball:


Code format?


Ok.


Add ` if(_scrollBarEnabled == enabled) return` guard would be better to reduce unnecessary remove and re-adding.


I try to replace all tabs to 4 spaces before creating pull request carefully. Some tabs might be remaining in intermediate commits. Is this latest commit in the PR?
I'm sorry but I prefer to use tab in my other projects.


This commit is useless, I would suggest you to configure the editor behavior to comply with cocos2d-x convention. Thanks.


Why remove these code here?


regularzing constant is not a rate


So, I'm still a little bit confused how the initialization is going to work in Clojure.

```clj
(import 'ddf.minim.Minim)
(def minim (Minim/withDefaultFSH))
```

This is what you mentioned would be the way to instantiate Minim inside of Clojure, but there is no class method named `withDefaultFSH`. Did you ever determine a good way to get around Minim's weird instantiation?


A comment on that answer suggests that `update()` might work, but it's not important.


no need if condition


please add some comments to describe this option


each 10 iteration, we should store a model in tmp folder like NNOutput


sqlite_timeseries (mbadams5) - x86_64-MacOS-10.9-clang: OK (1732 of 1732 tests passed)

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


long type?


LOG.info("regularizedRate: {}",  this.regularizedRate);
is better for slf4j logger


OOM? training and testing memory fraction are all 0.5 by defaul?


Related issue [here](https://github.com/esperantojs/esperanto/issues/151).


oh yes :smile:

thanks for your sharp eyes ! :+1:


The error messages when compiling on Debian seemed to indicate the same:

	/usr/local/src/redis/library.c: In function ‘redis_serialize’:
	/usr/local/src/redis/library.c:2001:28: warning: passing argument 1 of ‘zend_string_init’ from incompatible pointer type
	    *val = zend_string_init(sstr.s,(*val)->len, 0);
	                            ^
	In file included from /usr/local/php70-debug/include/php/Zend/zend.h:35:0,
	                 from /usr/local/php70-debug/include/php/main/php.h:35,
	                 from /usr/local/src/redis/common.h:1,
	                 from /usr/local/src/redis/library.c:4:
	/usr/local/php70-debug/include/php/Zend/zend_string.h:155:40: note: expected ‘const char *’ but argument is of type ‘struct zend_string *’
	 static zend_always_inline zend_string *zend_string_init(const char *str, size_t len, int persistent)
	                                        ^
	/usr/local/src/redis/redis_session.c: In function ‘ps_read_redis’:
	/usr/local/src/redis/redis_session.c:373:48: warning: passing argument 2 of ‘redis_sock_read’ from incompatible pointer type
	     if ((tmp_val = redis_sock_read(redis_sock, &(*val)->len TSRMLS_CC)) == NULL) {
	                                                ^
	In file included from /usr/local/src/redis/redis_session.c:36:0:
	/usr/local/src/redis/library.h:17:22: note: expected ‘int *’ but argument is of type ‘size_t *’
	 PHP_REDIS_API char * redis_sock_read(RedisSock *redis_sock, int *buf_len TSRMLS_DC);
	                      ^
	/usr/local/src/redis/redis_cluster.c: In function ‘get_key_val_ht’:
	/usr/local/src/redis/redis_cluster.c:554:57: warning: passing argument 3 of ‘redis_key_prefix’ makes pointer from integer without a cast
	     kv->key_free = redis_key_prefix(c->flags, &tmp_key, kv->key->len);
	                                                         ^
	In file included from /usr/local/src/redis/redis_commands.h:5:0,
	                 from /usr/local/src/redis/redis_cluster.c:29:
	/usr/local/src/redis/library.h:65:1: note: expected ‘int *’ but argument is of type ‘size_t’
	 redis_key_prefix(RedisSock *redis_sock, char **key, int *key_len);
	 ^
	/usr/local/src/redis/redis_cluster.c: In function ‘get_key_ht’:
	/usr/local/src/redis/redis_cluster.c:590:57: warning: passing argument 3 of ‘redis_key_prefix’ makes pointer from integer without a cast
	     kv->key_free = redis_key_prefix(c->flags, &tmp_key, kv->key->len);
	                                                         ^
	In file included from /usr/local/src/redis/redis_commands.h:5:0,
	                 from /usr/local/src/redis/redis_cluster.c:29:
	/usr/local/src/redis/library.h:65:1: note: expected ‘int *’ but argument is of type ‘size_t’
	 redis_key_prefix(RedisSock *redis_sock, char **key, int *key_len);
	 ^
	/usr/local/src/redis/cluster_library.c: In function ‘cluster_multibulk_resp_recursive’:
	/usr/local/src/redis/cluster_library.c:135:57: warning: passing argument 4 of ‘redis_sock_gets’ makes pointer from integer without a cast
	                 if(redis_sock_gets(sock,buf,sizeof(buf),r->len TSRMLS_CC)<0) {
	                                                         ^
	In file included from /usr/local/src/redis/cluster_library.c:3:0:
	/usr/local/src/redis/library.h:18:19: note: expected ‘size_t *’ but argument is of type ‘long long int’
	 PHP_REDIS_API int redis_sock_gets(RedisSock *redis_sock, char *buf, int buf_size, size_t* line_len TSRMLS_DC);
	                   ^
	/usr/local/src/redis/cluster_library.c: In function ‘cluster_slot_sock’:
	/usr/local/src/redis/cluster_library.c:180:92: warning: comparison between pointer and integer
	     if (!SLOT_SLAVES(c, slot) || (node = zend_hash_index_find(SLOT_SLAVES(c,slot), slaveidx==NULL))) {
	                                                                               ^
	/usr/local/src/redis/cluster_library.c:180:40: warning: assignment from incompatible pointer type
	     if (!SLOT_SLAVES(c, slot) || (node = zend_hash_index_find(SLOT_SLAVES(c,slot), slaveidx==NULL))) {

This was done on a virtualbox with https://github.com/rlerdorf/php7dev


@neokim  Ok, I know. But lots of indentation fix commits in the repository's commit history is not good. Maybe you could use some project wide configuration for cocos2d-x.


Use Java 8 lambda expressions to find the max, then create a product with the max key. This is how SQL auto-increment works (approximately).


I really don't like this kind of prefixing, but I don't have a better idea here either :).


As of #254


@pboyer LGTM


Done.


My assumption was that MSSqlLocalDB would be always available (except for VS2010/VS2008 users).

Is not true for your case?

I'm ok to postpoine this fix and implement proper one... (Or you can do it, and I'll cancel current PR in this case).


It's in commit a40d0afb9888cd1eda635aa0683130c1dab0a301


Oh, I had another thought.  What about ML_IMM (if we move to that?)  - this will need to flush in that case unless `MAP_NOINIT`.

-[Unknown]


@ksimmons thank you for writing this helpful gem. I take it you are using rails 4 and for this you are requiring 'action_controller/metal/strong_parameters'? I am attempting to use with rails 3.2 and the strong_parameters gem. To get it to work i needed to just require 'strong_parameters'


It's better use type that SDL2 returns


can you add `make -j $(nproc)` here?


@lobby417 I don't think the work email address is private information, it's just more convenient to have a Twitter account.


merged with master


It's auto-generated from a snippet I have and used. I usually don't prefix members.


RNMImport can also be installed direct from RForge using the command: install.packages("RNMImport", repos="http://R-Forge.R-project.org").


We have a shared REDIS cache in AWS, so you should set it up here and you can use it to store sessions:
```
REDIS_HOST = SECURE_SETTINGS.get('redis_host', '127.0.0.1')
REDIS_PORT = SECURE_SETTINGS.get('redis_port', 6379)

CACHES = {
    'default': {
        'BACKEND': 'redis_cache.RedisCache',
        'LOCATION': "%s:%s" % (REDIS_HOST, REDIS_PORT),
        'KEY_PREFIX': 'canvas_syllabus_export',  # Provide a unique value for shared cache
        'TIMEOUT': #  default is 5 minutes, but could stand to be longer, so should be thought about and set here
    },
}
```

Note that you can set the cache to be a dummy cache in local settings, if you don't have redis configured via vagrant for example.


I don't know why, it happens only on Travis.


if you want pain you should switch to vim

by your logic, you'll have gains for days


There's a reason I needed a NLP library! ;)


Looks good to me


`curl` is already installed, isn't it? I think the slim dockerfile can be identical to the non slim one


972124-2NewDetailedIceStorage (RKStrand) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1161 of 1161 tests passed)

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


no need for the `--quiet` flag or the `&> /dev/null` redirection


Can you split this in separate lines?


@stevengj the `signatures` function came out at https://gitter.im/JuliaLang/julia by @wookay as an answer to @willwhitney:

> Is there a way to get the names of the arguments to a function?
> I know I can use methods to get the list of methods, but is there a way to read the signatures from them?


awesome, nice left over..
+2


I was stupid, of course the apks stored in the sources do not break if the versionname was shortened in a wrong way, since those are based on the version and not the versionname.


Hey.

Sorry for being overly curious, but is there any reason you needed to introduce this option? Was the msbuild regexes creating issues elsewhere?

If you think there is anything which you think needs to be improved mainline, feel free to create an issue or issue a pull-request. Contributions are welcome :)

Cheers!


Ok, will do that.


Systemd can mount devtmpfs by its own, can you find the right service and make sure it runs? Although I think it should run automatically


If you really need this. You are not obliged to wait the next release. You can use this specific commit with npm.


where tests?


huum, I see.
Any way, I need to update my set since I touch this thing...
+2


Build fails due aufs issues:

fs/aufs/i_op.c: In function 'aufs_follow_link':
fs/aufs/i_op.c:1321:3: error: implicit declaration of function 'nd_set_link' [-Werror=implicit-function-declaration]
   nd_set_link(nd, buf.k);
   ^
fs/aufs/i_op.c: In function 'aufs_put_link':
fs/aufs/i_op.c:1337:6: error: implicit declaration of function 'nd_get_link' [-Werror=implicit-function-declaration]
  p = nd_get_link(nd);
      ^
fs/aufs/i_op.c:1337:4: warning: assignment makes pointer from integer without a cast [-Wint-conversion]
  p = nd_get_link(nd);
    ^
fs/aufs/i_op.c: At top level:
fs/aufs/i_op.c:1391:17: warning: initialization from incompatible pointer type [-Wincompatible-pointer-types]
  .follow_link = aufs_follow_link,
                 ^
fs/aufs/i_op.c:1391:17: note: (near initialization for 'aufs_symlink_iop.follow_link')
fs/aufs/i_op.c:1392:14: warning: initialization from incompatible pointer type [-Wincompatible-pointer-types]
  .put_link = aufs_put_link,
              ^
fs/aufs/i_op.c:1392:14: note: (near initialization for 'aufs_symlink_iop.put_link')
  CC      kernel/hung_task.o
  CC [M]  fs/ceph/file.o
  CC [M]  fs/btrfs/file-item.o
cc1: some warnings being treated as errors
scripts/Makefile.build:258: recipe for target 'fs/aufs/i_op.o' failed
make[2]: *** [fs/aufs/i_op.o] Error 1
scripts/Makefile.build:403: recipe for target 'fs/aufs' failed
make[1]: *** [fs/aufs] Error 2

Also it is to early to start with this. Seems there are many [kernel panics](http://www.phoronix.com/scan.php?page=news_item&px=Early-Linux-4.2-Woes) on several systems beside that fact with aufs.


Theoretically removing the calls to ``configure_job_output(true)`` will prevent output from showing up. If you exit, you will see "Server stopped" from all sub-jobs, but if you only kill the bap2 job, you won't.


Ehm, is this intended?


ok, I don't know the answer to this. When you call `get_post_meta()` can it ever be an empty string? If it can, I think you'd want something different than `isset` here. maybe `! empty()`?


done :)


.bind(this) deleted on purpose ?


Is this UTC? you should mention the timezone.


Yes, so that we can test but it's not meant to be merged of course. ;)


Should this maybe be an opinionated thing controlled by a property? I'd certainly not want to have some security issue because an important filter was just turned off. Going to put in a ticket.


Better be wrong than add the sources again ;)


Ah, ok.


Can this change be included, I test it and it appears to work, I don't know if it solve the issue though... But it better thand hard coded path name...


I think I'd prefer to fix this by still using `REQUIREMENT_MAP`, and changing its values to lowercase.
```ruby
      REQUIREMENT_MAP = {
        gson: "gson",
        yajl: "yajl"
      }
```
That way, `#break_requirements` in `spec_helper.rb` can still do its job.
Let me know if you're ok to update the commits in pull request to reflect this.
Thanks!


@bugobliterator to add more content here.


See line 9 (new) in this diff.


I'd drop aufs, and overlayfs seems to be the way to go.

I'd like to see kernel audit enabled, would solve some issues.


@maxshell, //cc:@AlenPelin I'm working on the @AutoFixture integration and I've got some good list of [working samples](https://github.com/sergeyshushlyapin/Sitecore.FakeDb/blob/57bf3e5e1f0e2288e727a95a0020912de24ae19b/src/Sitecore.FakeDb.AutoFixture.Tests/FunctionalTests.cs). For instance, the code below generates an item and adds it to the default `master` database using the `[Content]` attribute:

```csharp

[Theory, AutoDbData]
public void ShouldCreateContentDbItem([Content]DbItem item)
{
  // act
  var result = Database.GetDatabase("master").GetItem(item.ID);

  // assert
  result.Should().NotBeNull();
}
```
where `AutoDbData` extends the AutoFixture's `AutoData` attribute registering the `AutoDbCustomization`:
```csharp
internal class AutoDbDataAttribute : AutoDataAttribute
{
  public AutoDbDataAttribute()
    : base(new Fixture().Customize(new AutoDbCustomization()))
  {
  }
}
```

Using the custom `OmitOnRecursionFixture` type in tests or in the `AutoDbDataAttribute` class itself looks like a significant limitation to me. Is there any way to replace it with a customization?


Done.


This change should make possible to open files with filepaths like:
    /build/buildd/gtk+2.0-2.24.23/gtk/gtknotebook.c
by using the Ctrl+Shift+o keyboard shortcut.

See:   http://lists.geany.org/pipermail/devel/2015-June/009568.html


I'm pretty sure you still need to decref y, but then you have to incref x.


If "voted" gets added to `base` here, what will `base` look like?  I'm worried it might look like "glyphicon glyphicon-chevron-downvoted".  Will that work?


I'm guessing this is what is breaking things matt. maybe add a require:false here


same as above


Ahh Titanium clients will have to bug a GM to get a point and will display "None" instead of "Triple Attack" in the skill up message. Sadly the skills window won't show the skill.


Not needed?


Thanks for this! I just did a checkout to fix this very problem.


Regarding **kernel audit** we have to check if this doesn't affect **systemd**. Overlayfs seems to be the way to go for the future. Until now we still support **aufs** as it is. Anyway, this kernel series is to early to play with. I'm just early testing to find some issues fast.


good


it look like complex, but presently we do not have better, so
I keep it like that.


Perhaps we should start asking for cover letters and ask folks to indicate what position they are applying for as listed in the "who we are hiring" tab?


Good idea. Done.


Actually I don't think this fixes #118, need to come up with a better test.


Because it was a major pain point in dealing with PRs: every PR was
touching the same file and would have merge conflicts for no good reason.

Now we can just merge from GitHub which has made our lives a lot simpler.
And I just populate the file before each release.
On 07/07/2015 3:22 AM, "Jeremy Simmons" <notifications@github.com> wrote:

> @hazzik <https://github.com/hazzik> why did the release_notes requirement
> get deprecated?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/MehdiK/Humanizer/commit/315cc5083991af5d1c82fefa091b80a6898d8fe6#commitcomment-12021429>
> .
>


Just updated my gentoo kernel to 4.0.5, they have marked it stable.

Regarding kernel audit, iirc, we have to check if the systemd container thingy is still a problem with audit.
That was the only pronlem I found, but its quite old, nothing really new pops up.


Looks good as far as I can tell.


@scribblemaniac I like what you've done with the 32/64-bit logic, but we shouldn't use the trial version. Aside from being personally useless (as a Live 9 Suite owner), according to the documentation;
> Before submitting a trial, please make sure it can be made into a full working version without the need to be redownloaded.

…which isn't the case here. I have to redownload manually from the original url as far as I can tell, and the App version from that URL is called "Ableton Live 9 Trial".

Also I get an `Error: It seems the symlink source is not there: '/usr/local/opt/Caskroom/ableton-live/9.2/Ableton Live 9 Suite.app'`, not sure if that's caused by this change though?

Anyway, I also thought maybe we should have another `ableton-live-suite` formula with the "suite" URLs and this one with the plain "Live" URLs (but not the trail ones, according to the documentation).


I don't think this is necessary since you're already including it in the spec file.


thx ))


same here


I think it's the right approach, but the results should be saved: this code will execute on every attribute access if it wasn't set by the data returned from the query. Here's an implementation using `werkzeug.utils.cached_property`, which could be copy/pasted into this code base trivially:

        from werkzeug.utils import cached_property
        class Foo(object):
            def __init__(self):
                self.attr1 = "bar"
            @cached_property
            def attr1(self):
                return 42
            @cached_property
            def attr2(self):
                return 42
        f = Foo()
        print f.attr1  # bar
        print f.attr2  # 42


Initial commit


If so, please use any kernel and give it a test-run. Tell me the outcome and I might change it as needed ...


The field (i.e. database column) used to login with is customizable with: $config['identity'].
If you change its value from the default then the original code does not work. The submitted code fixes this by retrieving the column name used by the user without assuming this is always 'identity'.


`ValidatorsController` is now fixed.


What you mean test run? You mean using systemd conmtainers with audit enabled?
I used a custom mnjaro kernel with audit and openrc, on my devbox, works like charm, but I can't say for any systemd container usage.


Excellent work. Reducing needed config files are always welcome.


Fantastic - thanks Jeff :)
On Mon, Jul 6, 2015 at 11:27 PM, Jeff Yemin <notifications@github.com>
wrote:

> My apologies. It was an oversight. I've marked it for 3.0.3.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mongodb/mongo-java-driver/commit/05ab25b83f0877202584c645555f60934665df3c#commitcomment-12016111>
> .
>


Ha yeah, it's pretty complicated right now. I will try to come up with a way to simplify it if I can, but it works for now


@manisandro Do you think http://hub.qgis.org/issues/13082 could be related to this work?


Verified that cla is signed.


ah, ok. Not familiar with R-forge. will update text.


Yeah, sounds right. I will move to that before merge, as currently we waste instructions sometimes because we resolve immediates immediately so when they are constructed with MIPS arithmetic we load them several times.


Sadly as it is, **systemd** is our default and any changes regarding kernel configs have to been tested on **systemd** aswell. So yes, use any kernel we use currently and test it for any breakage on a systemd installation.


Please don't use implicit subject


Trial program using "Hello World!"


Fixes #1


I think during development this should error


> the App version from that URL is called "Ableton Live 9 Trial".

Most times that is an app you can upgrade to full. If that is not the case here, then we definitely need to revert it.

> not sure if that's caused by this change though?

It likely is. It means the app inside the newly linked packages must have a different name.

> Anyway, I also thought maybe we should have another `ableton-live-suite` formula with the "suite" URLs and this one with the plain "Live" URLs

No. We’re not going to duplicate casks and efforts, and cause confusion. We pick one solution (I don’t care which) and stick with that. Which one do you believe would be best for this case, @danielbayley, and why?

@scribblemaniac Care to submit the fix?