Yes indeed I had assumed that ye would want the solution submitted as a
pull request but never heard back from you. I just noticed it was October
2013! Time flies!

I will delete it now.

On 5 January 2015 at 15:04, neillume <notifications@github.com> wrote:

> Hi Barry
>
> I hope you're well. I've just noticed you forked the repo when you
> submitted your Blagger solution. So the exercise is valid for other
> potential candidates, would you mind deleting the forked repo?
>
> Many thanks
> Neill
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/barrymac/blagger/commit/6cddd1be5777263c7672d173dcce1ae2f3de7902#commitcomment-9154480>
> .
>

--
I must Create a System, or be enslaved by another Man's
Wm. Blake 1757 - 1827


TeamCity Zero-K :: PullRequest test build [Build 33](http://zero-k.info:8080/viewLog.html?buildId=2744&buildTypeId=ZeroK_PullRequestTestBuild&guest=1) is now running


TeamCity Zero-K :: PullRequest test build [Build 32](http://zero-k.info:8080/viewLog.html?buildId=2743&buildTypeId=ZeroK_PullRequestTestBuild&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:20


TeamCity Hyrise :: Unit Tests [Build 55](http://ares.epic.hpi.uni-potsdam.de/hyrise-build/viewLog.html?buildId=63&buildTypeId=Hyrise_UnitTests) outcome was **SUCCESS**
Summary: Running Build time: 00:03:21


TeamCity Zero-K :: PullRequest test build [Build 33](http://zero-k.info:8080/viewLog.html?buildId=2744&buildTypeId=ZeroK_PullRequestTestBuild&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:21


TeamCity Zero-K :: PullRequest test build [Build 34](http://zero-k.info:8080/viewLog.html?buildId=2745&buildTypeId=ZeroK_PullRequestTestBuild&guest=1) is now running


Docs needs to include installation of SDL on all major operating systems.

^^


TeamCity Zero-K :: PullRequest test build [Build 34](http://zero-k.info:8080/viewLog.html?buildId=2745&buildTypeId=ZeroK_PullRequestTestBuild&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:22


TeamCity Zero-K :: PullRequest test build [Build 35](http://zero-k.info:8080/viewLog.html?buildId=2746&buildTypeId=ZeroK_PullRequestTestBuild&guest=1) is now running


Put a ```CHECK_NOTNULL``` here (I guess it'd be optional for the next line, your call)...


can you post a trace so I can see what levels your antenna is giving you?


Thank you for your collaboration. Well, the idea is good but reading the whole thing seemed to me more confusing than helpful. (see my comments).

You should go straight to the commands that helped you, not including what did not help, and specify YOUR Linux distribution, so other people might recognize them.
Apart from that, I would opt for moving your tutorial from the INSTALL file to a Wiki article.


TeamCity Zero-K :: PullRequest test build [Build 36](http://zero-k.info:8080/viewLog.html?buildId=2747&buildTypeId=ZeroK_PullRequestTestBuild&guest=1) is now running


TeamCity Zero-K :: PullRequest test build [Build 35](http://zero-k.info:8080/viewLog.html?buildId=2746&buildTypeId=ZeroK_PullRequestTestBuild&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:21


There'll be a reference to short_desc somewhere in the code. I'll find it and remove it.

Cheers,
L


TeamCity Zero-K :: PullRequest test build [Build 36](http://zero-k.info:8080/viewLog.html?buildId=2747&buildTypeId=ZeroK_PullRequestTestBuild&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:21


TeamCity Zero-K :: PullRequest test build [Build 37](http://zero-k.info:8080/viewLog.html?buildId=2748&buildTypeId=ZeroK_PullRequestTestBuild&guest=1) is now running


I'll add these changes myself since they are small changes, and the build is already failing from unfinished code.


TeamCity Zero-K :: PullRequest test build [Build 37](http://zero-k.info:8080/viewLog.html?buildId=2748&buildTypeId=ZeroK_PullRequestTestBuild&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:21


Serienjunkies.org is not required at all for this hook?


same license prob


@LFDM whenever you have time, can you add a comment here or, elsewhere in the arethusa docs, about when we do and don't need to check the contents of newVal and oldVal? I remember when we walked through coding this that you originally had a check on newVal != oldVal, which prevented the setting of scope.context because the watch was only executed when newVal == oldVal.  I encountered this same problem yesterday in the tei branch on the tei_parser directive (https://github.com/latin-language-toolkit/arethusa/blob/tei2/app/js/arethusa.tei/directives/tei_parser.js#L13) and taking the same check away there fixed it, but I can't remember the exact rationale behind this. Thanks.


Because I don't like having another directory just because of one class :)


Yes, that's correct - that was an oversight.


watch out for a size limit, new lucenes drop BIG fields ...


TESTS FAILED
 Mr.roboto url : http://jenkins.plone.org/roboto/get_info?push=26105a464ccd457fa1fbe1fb95b1a600
 plone-4.3-python-2.6 [FAILURE]
plone-4.3-python-2.7 [SUCCESS]


I usually put it in `libs/App/Http/RouterFactory.php`, and I will probably move it in the future


Hmm.. I'm personally not really a fan of SCons. PolarSSL uses CMake as well as a result (both were evaluated when we decided to add a more complex system than just a single `Makefile`). Granted, PolarSSL is a much different project than what is done in Yotta.

Most of the movements in open source projects have been from SCons to CMake, not the other way around as far as I remember and can find online..


Added exception throwing :)


+1 for https://www.youtube.com/watch?v=5Qn4W4vimwk


Is there a way to pass --no-ansi etc through to the TextWrapper? Also, Is it possible to get it so that the contents such as Phinx 0.4.0 and all the other text doesn't get returned, but important information (Exceptions, Success, Failure) text does?


Considering we're going towards using GregTech as the main way to generate our ores, as well as the fact that we've transitioned from a layered approach to more of a fractal, region-specific approach to generating our ores means that this commit should close #139


@bors: rollup


Constructor is exported.

Example:
var WHMCS = require('whmcs');
//...
var wclient = new WHMCS(config);


thanks, didn't catch that when I converted it from graphbuffer and cmddata.c


revert please, leave it at 2424, this is default for opengrok configs ... (OpenGrok script uses 2424 and most of other customer scripts too)


Just one more thing: The XML file does not have a final newline. This is also highlighted in red in the diff at GitHub. Please add that final newline to the file.


TESTS PASSED
 Mr.Roboto url : http://jenkins.plone.org/roboto/get_info?push=133b6f26290d42eab5d750f89813c93d
 plone-5.0-python-2.7 [SUCCESS]


You're correct - removing .so does the trick. Will revert related CMakeLists changes.


Maybe sensitivity_horizon should be decoupled from rc rate (IMO it is still dependent)?

From user point of view it would make sense to define it as stick position(ratio) where horizon mixing diminishes to zero.


Delete console log


@jasiek does this look like the correct location for JS specs if they are run using Grunt?


r+ rollup, thanks!


You don't need to pass `--no-ansi` to TextWrapper, because it uses `StreamOutput`, which doesn't take decorations. You can use `TextWrapper::getExitCode()` to determine if the command was successful or not.

All that `TextWrapper::executeRun()` does is capture the output that Phinx generates and pass it back as text. You can manipulate it however you want.


молодец


ok

2015-01-06 14:26 GMT+02:00 David Yu <notifications@github.com>:

> We'll make an exception for this one as this had already been included.
> For new functionality, definitely.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kshchepanovskyi/protostuff/commit/362a125df25eb3411abf25367568a8977a0028fa#commitcomment-9167770>
> .
>

--
Regards,

Konstantin


most of them do run on 6767 (see [`test/server.js`](https://github.com/request/request/blob/v2.51.0/tests/server.js#L11)).  I changed it because I thought I had something else running on my machine on port 8080, but I didn't.  This change shouldn't have made it in.


Beyond that, if you want to customize the parameters more specifically, you can create your own version of `TextWrapper`, it's mainly a proof of concept that shows how to use `ArrayInput` and `StreamOutput` with Phinx.


:shipit:


nop use textures in media


or use logger instead ;)


>We use Robo to handle incrementing version numbers and do releases for us.

Can you combine it with githubs release feature? I like the option to describe what is contained in a release ...

>You can do like PHP_CodeSniffer does: increase version number right after making a release, not before making one.

But how do you know what digit to increment? (patch vs. feature, following semantic versioning).


why use twig templating engine on plain html file ?


Needs available version documented.


We have this argument going on whether we should drop the `? super K` completely...
Basically it brings complexity that someone has to deal with on `loadAll`. All comes down that whether this method produces instances of `K` or not. If it doesn't, then we need to map the key instance back to the one provided. I don't  do this below, but effectively we'd need to install the V to the `Map.containsKey` that matched the `loadedKey` in the code below:

```
          Map<? super K, ? extends V> loaded = Collections.emptyMap();
          try {
            loaded = cacheLoader.loadAll(computeResult.keySet());
          } catch (Exception e) {
            for (K key : computeResult.keySet()) {
              failures.put(key, e);
            }
          }

          for (Map.Entry<? super K, ? extends V> loadedEntry : loaded.entrySet()) {
            Object loadedKey = loadedEntry.getKey();
            if (! computeResult.containsKey(loadedKey)) {
              throw newCacheLoaderException(new RuntimeException("Cache loader returned value for key: " + loadedKey));
            }
            final K key = getRuntimeConfiguration().getKeyType().cast(loadedKey);
            V value = loadedEntry.getValue();
```

Now, we can pun on all this and just leave it all up in the air, and force `CacheLoader<K,V>` and do whatever with the "loaded" keys... (i.e. I guess install the one returned by the loader). Or ... What?


I see no reason to set the color this way - see below


ok but when we discuss we discarded at least all that require multi pass. so what should we remove ?


good question: return an ``NonImplementedException`` ?


Does it still need frame events? After all, you removed the function


hmm, :-1:  we won't merge this.


Great!


Tak. Powinno być "na tutorialu" :)


Hi,

I found this tool :+1:

https://github.com/jmcollin/autoindex


Ahh, you nerfed it! :D


just pushed https://github.com/rlei/fastd-android/commit/906deb834a5cd435f470f748474397e0761a3c68
please have a look.

remaining tuntap.c & libancillary fixes will be done in a separate commit.


1: M101 signature is missing


I don't like commented code.


I don't wanted to make sure that we do not forget about it.


Merge remote-tracking branch 'origin/master'


"clreate" -> "create"

:)


$http.get returns a "promise" which has a then method on it.  Essentially, it's saying I promise to try to get the data from "wow.us".  When I do, then your callback (passed in the "then" function) will be invoked.


also define this field name where all the rest of field names are defined (one place, instead of having this around)
QueryBuilder has those fields defined - I planned to fix and reuse them every where, I guess you can start with having this field "scopes" defined only in that one place in QueryBuilder


oh i didn't know that.  That's smart.  I'll add that.


hmm, i'm not sure how i feel.


I'll add a better example, makes sense.


QueryBuilder.REFS
btw. you can  fix all other such occurrences in this file (for full, defs, etc.) to lower the apps footprint and give it a bit more speed
I planned to do it anyways on every file I will touch, so you're touching this one, you can do this


Lol, I'm a bit of a creep and index everything I touch.

Having people reply on Github is an interesting idea!


ping @ndufreche we have to stand and decide what to do: ``console.log()`` or ``loggers`` ?


It seemed a bit too low, the new 20 limit...


same QueryBuilder plea as before ... pls pls pls ;)


Why not just B_MENU_BACKGROUND_COLOR as above?


Louis solved it all!


I don't know. I guess by default it's minor and some feature comes in, then you increment middle number. You can see now that having tag name represented in PHP code doesn't sound right if it's that hard to keep in sync.


``return this.pluginsInstance.hasOwnProperty(puglinName) === true``


I found some minor issues  in demod.c and when I get my pull-requests in order I can start contributing them..


I am actually wondering how much in size are we talking about
(and I was wondering if this cannot be merged with existing Defs data we store already)
use luke - https://java.net/projects/opengrok/downloads/download/lukeall-4.10.3.jar to check index and get some % for the index growth for few repos (if you're patient enough I'd be curious how much onnv will grow)
+ same QueryBuilder plea pls pls ;)


Superfluous curly braces.


hmm, maybe can you use ``foo``, ``bar`` ou ``baz`` instead ?


same QueryBuilder plea


no way ;)


Here is a comment about adding the next file.


@Toilal Shouldn't argparse only be added to the requires list if python is less than 2.7?


size_t is preferred.


\o/ you're such amazing ! Continue like that !


:shipit:


There is fPoseViewHolder, and fPoseContainer. Why are those two needed? The name doesn't really give the difference between the two away.


I don't know if it's best, but it's probably okay.


Are you having a preview from Klaus for VDR 2.1.7? -
otherwise #if defined(APIVERSNUM) && APIVERSNUM >= 20107 will never be true with VDR 2.1.6


please don't comment in the commit, comment in the diff


A cute commit message... again... @Arthur2e5


what is the difference between ``this.isEnabled`` and ``this.enabled`` ?


this is a potential bottleneck - can this be improved with some hashtab? or some other way of lookup ?
traversal of huge amount of scopes WILL be slow


[Top](#Subject)


Hi,

I tried your smtp and could receive two emails on it. I leave them in the inbox ( in french)

Did you change the contacts emails of this form in Customers > Contact ? Because you need to use smtp-test@modulez.ru as receiver.

Regards


Oh interesting. An easy fix would to be to parseInt the value. That way px will be trimmed if present.


'=' is an operator, and goes to the next line.

Also, please make sure not to have any stray whitespace. Any extraneous whitespace should be removed before committing. Pe and other editors can even do this automatically for you.


@butangmucat I just tested, the QtQuickCompiler was only to be used when it exists, when it doesn't, it falls back to a safe choice.


If you use `npm version patch`, it'll update the version and create a commit for you :smile:


Found the topic branch, reviewed and picked it to experimental. Thank you very much. This part of the pull request should be solved.


Damn. I tried it like this but it didn't work. Already assumed it had to be something stupid.
`dpkg -i $(basename $config_xbianrepository) --force-confdef --force-confnew`


commented code === removed from project


My point is just that ``string2T`` is an internal helper for this trait, so I wouldn't want this to be called by an user routine. Ergo the ``protected``
As for the signature, I am on the middle of a commit where the signature will probably change. So, on it :-)


This should also be translated. The specific parts should be translated by segment.


ım not sure but ı think with this change mmcblk0p16 should be mounted via fstab or init
otherwise it can produce some problems when encryption enabled
What do you think Harakhti?
can we add something like that?
/dev/block/mmcblk0p16     /extra     ext4    noatime,nosuid,nodev   wait,check


hmm, this seems useless.

``hide`` => ``empty``


Yes, as I said before - this is just raw functional prototype. I was thinking about some binary tree here (it should be enough to search only by starting line).


`intenstive` typo here!


Hey! As much I'd like to actually maintain this cool project, I won't have the time time for it. So I'll occasionaly may commit some fixes and/or enhancements, but nothing on a regular basis.


wondering if this is optimal to do, if this cannot be merged into defs - see PlainAnalyzer analyze(...)
I'd probably stick this code there instead of this abstract method
I actually just noticed that " doc.add(new TextField("refs", getReader(src.getStream()))); " is duplicated here and might be possible removed ... (can you try pls)


all tests pass:
success: http://buildbot.rust-lang.org/builders/auto-mac-32-opt/builds/3036
success: http://buildbot.rust-lang.org/builders/auto-mac-64-opt/builds/3030
success: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-t/builds/3031
success: http://buildbot.rust-lang.org/builders/auto-linux-32-opt/builds/3022
success: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-t/builds/3022
success: http://buildbot.rust-lang.org/builders/auto-linux-64-opt/builds/3025
success: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-t/builds/3018
success: http://buildbot.rust-lang.org/builders/auto-linux-64-x-android-t/builds/3013
success: http://buildbot.rust-lang.org/builders/auto-win-32-opt/builds/2690
success: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-t/builds/2688
success: http://buildbot.rust-lang.org/builders/auto-win-64-opt/builds/1526
success: http://buildbot.rust-lang.org/builders/auto-win-64-nopt-t/builds/1518


fast-forwarding master to auto = 8efd9901


Also, any particular reason we don't use the original password mail?


kljslfdjskfklsadjfdsa


Be consistent with the way you declare arrays. Either array() or [] (preferably the latter), never both.


kllklkkjlkjhkljk


The next iteration is up. This is what I meant by effort ^^

Somehow RemoveSkeletonsInGroup() is called twice. It removes all skeletons in the group that is reloaded, then the log message "Reloading group.ocd" is printed, and then it removes all skeletons from that group again. Do you have a clue why that is?


part of IQSS/geoconnect#77


that's dpkg parsing.
as soon as first non-option parameter is found (package name), all other 'parameters' are parsed as package names.

(so
```
dpkg -i pkg.deb --xxx
```
should return error pointing to unknown package "---xxx")


other than above this really looks good - good work and good reverse engineering/understanding of opengroks code! (congrats! :) )
fyi - same lexer can be used for java (which also has scope as { } ), so just duplicate it pls


Agreed, proposed as PR in https://github.com/awesomeWM/awesome/pull/74.


part of #77.  urls moved to https://github.com/IQSS/shared-dataverse-information/blob/master/shared_dataverse_information/worldmap_api_helper/url_helper.py


1: M100 needs more reviewers
1: M101 signature is missing


Above HTML will output:

first item
second item
third item


Easier: `delete_skeletons.push_back(skeletonpath);`


And yes again.  Thank you.


Or just `delete_skeletons[i]`, which avoids an unnecessary out-of-range check


saw approval from huon
at https://github.com/nick29581/rust/commit/b2f46c5f2ccec689f2133711e913bd809c28b1c4


merging nick29581/rust/arrays-3 = b2f46c5f into auto


At a minimum we need this and 7b230ab to go in with the merge. The previous three changes are also needed to build userland.


merging nick29581/rust/arrays-3 = b2f46c5f into auto failed


ffg


I guess there should be a space before `&`


I don't immediately see why RemoveSkeletonsInGroup would be called twice. I'm busy with something else at the moment, but it should be fairly easy to find it out, just set a breakpoint in RemoveSkeletonsInGroup in the debugger and see where the calls are coming from.


And space before `&` here


Although without 9f3a8f0 the kernel will fail to boot. It fixes accessing weak symbols in the kernel.


Currently this will break anyone's installation if they haven't yet registered the `FormServiceProvider` and `ValidationServiceProvider`. I will investigate ways to mitigate this. I think we might be able to only render the csrf field if people have updated their app and registered the relevant providers. Either way, such as the major changes required to fix the forms throughout the app, and registering of new dependencies - that it might be worth bumping up a major version (3.0).


Ok that's fine then


newline added.


Huh, weird. That's fine then


Style comment: remove extra space before '>' in all manifests; note: space before '/>' is required.


>I don't know. I guess by default it's minor and some feature comes in, then you increment middle number. You can see now that having tag name represented in PHP code doesn't sound right if it's that hard to keep in sync.

Paratest has way more patches than new features. But sure, it is like a circular dependency ... git knows the code and the code knows gits' current tag...
On the other hand, I've found it very useful in the past to see phpunits version in the output. Maybe someone would say the same thing about paratest^^

So... I'm fine with a YAGNI. Do you agree @brianium ?


Mucho OP newbies otherwise.. :P


Also, see wording from MM on the `external_add_item.pl`, that is, how it was used and a page that was using it actively: <https://pmt.ccnmtl.columbia.edu/item/98655/>


saw approval from nikomatsakis
at https://github.com/huonw/rust/commit/8ba4cfcf7b0ab15fc595bae3bc06004d443d55f5


This commit breaks the Windows build.  curl_multi_wait() does not exist in the provided libcurl.


merging huonw/rust/no-drop-and-copy = 8ba4cfcf into auto


status: {"merge_sha": "345df7a415acd7219d5622890906070ab7619d78"}


huonw/rust/no-drop-and-copy = 8ba4cfcf merged ok, testing candidate = 345df7a4


lgtm
perhaps a comment explaining why it's needed though


Well, that's the buff Ertheia gets


Unfortunately not everyone can just upgrade their version of PHP. Many distros such as Debian-stable and Ubuntu LTS don't update the version of software in their package repositories until a new version of the distro is released. They just backport security fixes into the existing package.

@bweston92 I fail to understand why you'd want to alienate users whose hosting provider does not provide a more up-to-date version of PHP. Especially when the change in to be backwards compatible is so minor and I've done the work for you to boot.


Finally nose 1.3.4 upgrade!


Where do we need Network? It's not even listed in CMake.


lgtm


You should have to separate commits (one for each feature)


Yes, I though it should also work for Java, but didn't tried it yet as I wanted to see it for C/C++ first. This simple solution however won't work for more function definitions on single line (but I think no one is that crazy to do that anyway :-) ). Parser for Python should not be too difficult too. I didn't though much about other languages.


I'd suggest renaming the repo as well (GitHub will add an automatic redirect, so the rename won't cause any problems).


I don't think it's customary to list "recursive" dependencies (dependencies of dependencies). Besides, depending on how our direct deps are compiled, many of these might be statically linked inside them, so they are not required for building Composer.


4558_hpwh_coil_removing_heat_from_tank (nmerket) - i386-Windows-7-VisualStudio-12: OK (1583 of 1584 tests passed)

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


Configure routes


It looks to me like gPidfile is never actually assigned anything.
so a pod file is created when the program starts, but the file won't actually be deleted.


You done the work, you mean changed some `[]` to `array()` and that's not what I'm getting at. PHP 5.3 has reached EOL, therefore security flaws etc propose a risk.

Debian and Ubuntu have repositories with PHP 5.4 or even PHP 5.6 for that matter.

Also I don't think you will find that using a hosting provider will be acceptable due to the PSN terms and conditions, their API is closed, this script just emulates an application they've made.


In this case I think we should use 6767 in all tests as it seems a bit more unusual port to me. In fact I needed to change the port that node-inspector listens to by default to 8000 when I started to work on request, because by default it uses 8080.

Anyway, I think this should be merged in, as it touches a lot of files and might potentially brake other PRs


You need to edit status.c find " for (i = 0; i < EQI_MAX; i++) " some where around line 2959 and edit:

if (i == EQI_AMMO || i == EQI_COSTUME_TOP || i == EQI_COSTUME_MID || i == EQI_COSTUME_LOW)

This should disable bonus stats to effect player from all costume gear.


I added a note to the spreadsheet to keep track of this


:panda_face: this panda loves what it sees :+1: Oh, he also loves bamboo so here is some :bamboo::bamboo::bamboo:


OK


right. i just added a old version cause there was e previous version that didnt work


@akbertram we expect that nation wide locationtype is always inside db (with 20301 id), right?


I don't see the problem, newbies like being OP :D


Having -Xmx in the JAVA_OPTS causes it to be set twice in the running java process.  Not a bit deal but if you remove it from JAVA_OPTS it's only set once.  Whatever is reading it from LS_HEAP_SIZE is setting it.


Maybe something like `problem_expr = Piecewise((Max(x, y), z))` would be better.


comparaison de la liste imprimer avec la liste reel


It's really minor but can you strip the newline?


Good to see someone with good understanding of OOP looks thru Permalinks class. :+1:


Hi Valentin, in our use case,
it seem the library is used only within the server not from the client directly. (Caller of this function exist in the same process/thread) So there seems to be no need to use generator in this case since it got store in the memory already in  set(TC.releases(arch)). Of course, if we are using the library for RESTAPI, generator can be  used for streaming. But seems data size is small anyway, isn't it?


Separated exception handling.
Added a warning log.


Are you REALLY REALLY sure you want to keep that destructive old and smelly GPL license? Actually, no one can use your code now in anything money related, and it also permits you from so.


r=brson


Same here. It allowed the build to proceed with the VOS packages, but I had the error as soon as I tried to open my database. 80000 worked for my database, fyi.


In theory, the macro MONGOC_ERRNO_IS_AGAIN() in mongoc-errno-private.h is the right thing to use here.  Not that I've been fastidious about using it everywhere either.


@Mikely2  is a spammer i tested it and its fucking spammmm!!!!


Good luck bro 😜❤


@mbosner
SierienJunkiesOrg.py comes with pyload and is working.
so no need to have it in my repo


Are methods not allowed to have arguments? or?


合并分支


Included  pt_BR as an optional language


Thanks. I had made a mistake in this commit but then I noticed and fixed.


This is very very interesting! Great job!
What additional things did you add to your SLKTextView subclass?

You should send a PR for this ;)


Psychological Sciences Master's Thesis: "Individual Differences and Basic Logic Ability"


VesselRotationer(VR)이라는 오브젝트를 추가하여 마우스를 따라다니는 것은 VR이고 Vessel은 VR과의 회전각 차이를 좁히는 방향으로 회전시킴


well, this line is a subitem for the channels, not affecting anything else. Without it would look like a global setting for any view, I guess.
Earlier both channel items had an own headline/divider, that went away with that change. I don't like it either, but had no better idea... and hope that it would "fix itself" with one or the other upcoming restructuring. :-D


Probably better to use c.cobrand.moniker == 'fixmystreet' here.


shoudl suspend on an unversioned bucket really make it suspended?


Please do !


Oh, vorsicht.
Du ziehst mit diesem Schritt hier das Photon Backend mit rein, dass wegen der Backend-Scripts, dem Gluon-Builder oder dem IRC-Bot auf aller unserer Hardware installiert ist.

Sollten wir jemals mit dem Gedanken spielen die Änderungen wieder Upstream zu pushen, haben alle andere mangels Photon ein Problem - ja, kann man nachinstallieren, muss aber nicht sein, wegen nur einer Funktion..

Wir haben zwei Optionen - entweder wir belassen beide Projekte getrennt und implementieren zur Not Funktionen doppelt, oder wir ziehen das Photon-Backend so weit in die Karte mit rein, so dass das Backend alle Vorteile ausspielen kann, und der Kartencode insgesamt schlanker wird.


Oh, I really liked these files. https://github.com/metricfu/metric_fu/blob/1adc860c6573086c93b0b99b0fa04bbc9b847c88/spec/quality_spec.rb https://github.com/metricfu/metric_fu/blob/1adc860c6573086c93b0b99b0fa04bbc9b847c88/spec/capture_warnings.rb


Nice.. are you waiting for community contribution to actually write the code? ;)


You are on the good way to become a powerful PHP developer !


I just tried it out, and for me `Removing skeletons in group: /home/armin/devel/openclonk/planet/Tests.ocf/SkeletonInclude.ocs/ClonkIncluder.ocd` appears only once in the log when I type `ReloadDef(ClonkIncluder)`


Ok, changed, see 4dd8c7c..ea57fe3

On  0, ticoann <notifications@github.com> wrote:
> Hi Valentin, in our use case,
> it seem the library is used only within the server not from the client directly. (Caller of this function exist in the same process/thread) So there seems to be no need to use generator in this case since it got store in the memory already in  set(TC.releases(arch)). Of course, if we are using the library for RESTAPI, generator can be  used for streaming. But seems data size is small anyway, isn't it?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/vkuznet/WMCore/commit/b209221521dfea9a511ecf7805c377b3a9055285#commitcomment-9170643


I renamed and did not remove. Does this count as two reloads?


That's what I thought.  Thanks!


:hamster:


Sorry, but I just want to ask a question, is it possible to export those images to WebP directly to save space (without further encoding to PNG)?


Some sources have disableQueryString set? in what cases is that needed?


@sirmordred Referencing to this commit https://github.com/ivanich/android_device_htc_pyramid/commit/9107522219526066919b971a83ebc92221bf6bae mounting it may not be absolutely necessary and reading it directly may yield unwanted results (which is why I left /efs out of the TWRP partition configuration as well)


I don't understand how this receiver even works. If it isn't exported, it shouldn't be able to listen to system broadcasts (according to http://developer.android.com/guide/topics/manifest/receiver-element.html). Please remove this line (default is sufficient enough - presence of intent filter makes the receiver exported).


Hey Christian, why was this behavior added? The 1.4 driver definitely doesn't throw if you don't set the replica set name in the connection string.


Maybe it is detected as two separate events... one file removed and another added? I guess this also depends on the operating system.


If I recall correctly that value was set the same as in native Android firmware. I'll try to bump it to 1.5GHz and see how it will perform. Thanks for pointing that out.


Closes #1


I never saw in practice using arguments... It's very standard to have method without arguments and everything is done inside method e.g:
http://source.roslyn.codeplex.com/#Microsoft.CodeAnalysis/Syntax/SyntaxNode.cs,21
https://bugzilla.xamarin.com/show_bug.cgi?id=20816

I could add support for arguments probably(didn't even try). But didn't feel like good spending of time...


@mitchellh Here's the [MS Connect](https://connect.microsoft.com/PowerShell/feedback/details/750654/powershell-exe-returns-a-passing-0-exit-code-when-a-parsererror-is-thrown) article for this issue. Basically you have to check stderr like you're doing here. :disappointed: I've thought about having the WinRM gem raise an exception for this exact scenario so consumers don't silently fail.

This seems similar to c4422d7c70312de67266e368316da40cbaca6cff, so you may want to cleanup the duplication with that.


good change. any insight into why it fails to decompress?


r=brson


rax retry tests


that's what S3 does (just verified again).


Shafa (First comment)


Unfortunately, no :( Papertrail only provided me the line number failing. Not any real specific information about the actual failure.


Enhancement:

since `ehv.lib`  has now experimental status,
move `primDecsAreEquivalent()` and other useful routines to `primdecUtils.lib`
and include `primdecUtils.lib` in `primdec.lib` and `ehv.lib`?


> It already has wide desktop support, but many feel frustrated when using the API.

This isn't something for html5please, also, why do you think so?


I think Symfony Request has a get method


@noordawod That doesn't fix it, the provider isn't set up correctly. I'll submit a pull request.


Good job on writing a README.
I know we are programmers and not necessarily writers (though we do write a lot!), but if you have some extra time, spend some time correcting the grammar in your document. It does not have to be perfect, but it looks very shoddy IMO.


You need to order the imports by name.


I must admit I have not given it much thought. What license do you suggest?


You can just do `$slack->properties = $properties` here and the model will take care of the magic methods.


Ok


Active will only be one, so perhaps just `where('active', 1)` - maybe even make this a scope?


object


Quiere decir entonces que mejor tengo que hacer merge de la 7.0 a la 8.0 ...


You can do `$properties = $service->properties`


Why did you make this change? the source path isn't `plugins` it's `plugin` are you able to actually get android working?


Sad to see this go... No replacement planned?


@rafales one option would be to move your code out of `node_modules` and into a new folder, e.g. `local_modules` or something. Then you can just set [resolve.modulesDirectory](http://webpack.github.io/docs/configuration.html#resolve-modulesdirectories) in your webpack config to include that folder (e.g. `["local_modules", "web_modules", "node_modules"]`)

You shouldn't have to change any of your code as `require('myapp/someModule')` would look inside `local_modules` just like it does with `node_modules` by default.


`compress-js.php` would be a better name.


Si mejor eso :+1:


You have to leave backward compatibility here. Other wise you break the restarting of the query (Around line 345)


*Insert vague thoughts about defining protocols for things here*


This resolve function is now skipped, the case of not being authoritative is handled before this call point now.


Why not change the function parameter type hint?


I thought it was intentional at first. Then, I noticed the builders downloading libs before building and figured it must be a squeaky clean environment. Also, it runs `./gradlew assemble` before anything else, even though it's not in the config, so cleaning would blow that away. Thanks again for setting this stuff up. I didn't even know it existed.


Just commited this change.


@bweston92  I never claimed it was a lot of work but it still took a non-zero amount of effort.

PHP 5.3 is indeed EOL and I realise there will be no more upstream patches to the 5.3 branch but as far as I can tell Precise Pangolin is supported by Ubuntu until 2017 and they will backport security fixes into their PHP 5.3 package.

I use a Precise Pangolin EC2 image on my server and the latest version of PHP available in the official repo is 5.3.10 but checking the changelog shows patches are included for vulnerabilities found four months after 5.3 reached EOL.

Your point about the PSN terms and conditions is interesting though I'm not sure how anyone would make use of this code without breaking Sony's T&Cs and, in many jurisdictions the law.


yes closure is missing;)


See #57


full stop :monkey:


Thanks, that looks like the one. It felt like it might be even hackier to define it as global... we can leave it this way for now and fix when the jshint issue is closed.


Sieht gut aus, kannst es gerne mal in die Szene in Unity machen.


Montellese:stdstring_removal_part3
This seems to compile fine and runs much better than before. I think we can put that down to the fact that all the Boxee code never used CStdString at all.


Here is an example screenshot of 0.0.1 in CLI mode: ![selection_033](https://cloud.githubusercontent.com/assets/1927298/5631853/265b1028-95ca-11e4-8279-3764a0e236b2.png).


nice code


VDR 2.1.6 is missing a proper API for getting PMT pids. Hopefully, the next version will contain the required patch.


That seems like a reasonable approach. I still prefer the approach where the API itself guarantees the atomicity of the operation rather than implementation convention, but I don't know that I can make a strong argument against it. Thanks.


合并更新


神一样的 patch……


space after bracket


some tests failed:
failure: http://buildbot.rust-lang.org/builders/auto-mac-64-opt/builds/3031
exception: http://buildbot.rust-lang.org/builders/auto-mac-32-opt/builds/3037
exception: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-t/builds/3032
exception: http://buildbot.rust-lang.org/builders/auto-linux-32-opt/builds/3023
exception: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-t/builds/3023
exception: http://buildbot.rust-lang.org/builders/auto-linux-64-opt/builds/3026
exception: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-t/builds/3019
exception: http://buildbot.rust-lang.org/builders/auto-linux-64-x-android-t/builds/3014
exception: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-t/builds/2689
exception: http://buildbot.rust-lang.org/builders/auto-win-64-opt/builds/1527


also, if (keystone_result >= 0)


if (keystone_result) < 0)


Why inconsistent line breaking?

Should it be `if(...) {` or `if(...) \n {`?


y entonces de paso creo que tendré que propagar algunos merge de los que han hecho Uds amigos: me ayudan identificándolos ?


fix comment, it's not related to cephx in any way or shape


@stevebrownlee dat should do it..
Thanks


Thanks, :)


@chrisze just to confirm: is this expected that `screenBounds.size.height` is used for **both** width **and** height of the frame? I'm just looking on the code now, didn't try to reproduce any errors, but thought that maybe this is a typo, maybe you were aiming to rotate the frame for 90 degree instead of making it a square...


This is too narrow, I think a method should be @Cached without being @Crud


Janice TEDOGMO


Well, it surely sucks...


If overloaded, check that this is the @Crud put method


Tightened it to ensure people specify the replica set name and make it
explicit
On Jan 6, 2015 5:16 PM, "Valeri Karpov" <notifications@github.com> wrote:

> Hey Christian, why was this behavior added? The 1.4 driver definitely
> doesn't throw if you don't set the replica set name in the connection
> string.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mongodb/node-mongodb-native/commit/4d4ab9d147dfd75f64dbc3169e546879486a4233#commitcomment-9170853>
> .
>


move this block above the success block, and remove the if statement from the success block


test


Sorry, "safer" as in "we should test both backends so we'll know whether we've broken the frontend or the backend" :)


also, indentation is broken maybe?


sorry @DanRyanIrish, my bad, but you need backticks around the astropy.units.Quantity. This also goes for everytime you reference an object in general.


open scope, close scope even if it's one liner (ceph style)


Yes, I noticed that before Xmas break and realized I had to switch that out. Thanks for pointing that out, I will fix this week.

Sent from my iPhone

> On Jan 6, 2015, at 4:17 AM, Seif Attar <notifications@github.com> wrote:
>
> @rmacdonaldsmith any reason you are installing monodevelop-nunit as opposed to nunit-console? the docker image size is huge now (900mb) assuming that is because monodevelop-nunit has a dependency on monodevelop so you are installing the world when you install this.
>
> —
> Reply to this email directly or view it on GitHub.
>


3 .line having problem


is jagah add karo


it is necessary, remove remark


In docstrings, you mean?


move the else to the line preceding, add a space after 'else'


yeah.


saw approval from brson
at https://github.com/alexcrichton/rust/commit/9d0b3c9fc91dbde74f7cafc85377432447c0d364


merging alexcrichton/rust/less-warn = 9d0b3c9f into auto


status: {"merge_sha": "6539cb417f4a7c2d9d1afce44c196578d2b67f38"}


alexcrichton/rust/less-warn = 9d0b3c9f merged ok, testing candidate = 6539cb41


no, this method is `@required` in the protocol.

`JSQMessagesViewController` correctly implements this method.


I'm assuming there's really no "build" any longer then?


fix comment, there's no cephx here, it's AWS signatures


:black_circle: :heavy_check_mark:


Is this part of an existing connection string spec and/or do other drivers do it? I know this particular change will break some of my code, so I'd like to make sure there's a good reason for it.


I think `dispatch()` is not part of any interface...


Hmm... that's interesting. It (or at least some bitpacked part of it) is probably some kind of "player status" rather than a simple field of flags. I'll try tonight to see which bits are exclusive with/affect one another.


From revision 930db324b08ff155de610fdd8f8f1eb01420882a, 'smrt orthologize' writes distinct files for each orthologous cluster. This will use a little bit more space for alignment files and can cause redundant files for singleton clusters etc, but is in my opinion better for the following reasons:
1) Alignment files generated with 'smrt align' are now not altered at all. If merging fails for some reason, the user can re-run 'smrt orthologize' on the same files as in the first run.
2) If alignments were merged into 'regular' alignment files produced by 'smrt align', merging depended on 'BACKBONE_MAX_DISTANCE' given in the configuration file, which is generally smaller than 'CLADE_MAX_DISTANCE'. This would mean that 'smrt clademerge' could miss sequences that could pass the test for distance but which did not pass in 'smrt orthologize'.


Convert Tests To CoffeeScript


I suggest MIT or BSD, Github itself seems to recommend MIT, and it looks like majority of repositories use it. Github also have very good page describing licenses


Respecto a la rama 7.0, si haces merge en un branch para actualizacion basada en la 8.0 y hacer merge de la 7.
Con eso tendras todos los cambios de la 7 en la 8, posiblemente puedes tener conflictos.


Single backticks?  Like `astropy.units.Quantity` ?


I have reviewed and good with PR


I know this was already there, but would it be possible to use constants here?


Thanks.

To be clear, the link I gave was a separate point - I think you missed a spot in this less file.


Why is it exported? Not sure it is really needed. I though it is a standalone app - isn't it? Or is it also a service for other (3rd party) clients, with convenient API?


So the conditions under which this would be a BC break are:
- User runs composer update
- User has not registered the `FormServiceProvider` & the `ValidationServiceProvider in their `$app` already.


Need a better action name for this intent. It should be similar to android actions in style - i.e. something like ru.mera....action.DO_FOO


Looks good!


Reviewed and good with PR


I think that's fine, though...  not much we can do about it anyway, and if I understand you correctly the reloading itself works fine, it's just that it happens twice. Looks like it's time to wrap this up and get it into master :)


@jgillis Lo and behold;)


Hopefully it also works...


Thanks! I will check this.


Yeah, this is the kind of file that smart people look at and say "Hey, I could probably charge people to do this."


Yes, it actually reloaded twice when I rename. Must have missed the logs in the first version. The last commit is up, I removed some stupid comments.


Otherwise, looks good.


aSortKey not aSortKeys
bSortKey not bSortKeys


Right, it's better thx @gazpachoking


Shame on me, I forgot to use `--author`. See the commit message for who should get the credit.

Vale,
Quintus


bSortKey not bSortKeys


aSortKey not aSortKeys


> I'm not 100% sure what the correct action here is. In principle, when you do Ctrl+Alt+L, the cursor does move into the next line, so it's not outlandish for that line to be affected.

I'm not 100% sure either.  It probably depends on how you define a "line": Ctrl+Alt+L selects the line including its termination character (the \n).  Scintilla displays this as the cursor on the start of the next line (which kind of make sense as it's right *after* the newline character, actually in between the newline and the first character on the next line).
My main point in suggesting use of `editor_select_lines()` was consistency, as we have this function we might as well use it to solve this kind of problems, and consider its behavior the right one (or fix that function).  But it may not be appropriate here.

> Also, with your addition I observe a slight inconsistency. Consider this file:
> […]
> Now, start over but start the cursor after the first tab. Then hold shift and press the right arrow two times (this will select both newlines and put the cursor at the begginning of the third line). Now, removing trailing spaces does remove the final tab.

Wow, indeed.  And it doesn't happen if the last line doesn't have an EOL character.
This is a more or less expected behavior (meaning it's what the code do but I don't know if it's intentional) from `editor_select_lines()`: if the selection is already on "whole lines" (first case), it doesn't do nothing, but when it's not it doesn't consider column 0 to be the end of the previous line.  This should probably be changed in `editor_select_lines()`, as it's the location of the probable inconsistency.

Like this I guess:
```diff
diff --git a/src/editor.c b/src/editor.c
index 3963e59..44c55c4 100644
--- a/src/editor.c
+++ b/src/editor.c
@@ -3687,17 +3687,14 @@ void editor_select_lines(GeanyEditor *editor, gboolean extra_line)
     start = sci_get_selection_start(editor->sci);
     end = sci_get_selection_end(editor->sci);

-    /* check if whole lines are already selected */
-    if (! extra_line && start != end &&
-        sci_get_col_from_position(editor->sci, start) == 0 &&
-        sci_get_col_from_position(editor->sci, end) == 0)
-            return;
-
     line = sci_get_line_from_position(editor->sci, start);
     start = sci_get_position_from_line(editor->sci, line);

-    line = sci_get_line_from_position(editor->sci, end);
-    end = sci_get_position_from_line(editor->sci, line + 1);
+    if (extra_line || start == end || sci_get_col_from_position(editor->sci, end) != 0)
+    {
+        line = sci_get_line_from_position(editor->sci, end);
+        end = sci_get_position_from_line(editor->sci, line + 1);
+    }

     sci_set_selection(editor->sci, start, end);
 }
```


Yes, that is a good point. However the styl file is not using the opacity property as shown in the css-tricks example.
The .btn-cancel could benefit from the RGBA as it works on IE9+ and all other browsers.

- on a side note is there a forum for DemocracyOS?


Not “mnemonic”, unless that’s a meaningless label for the sake of having one.

If the list is actually intended to play any sort of mnemonic role for humans then it needs cleansing of inappropriate entries such as numeral words (what _were_ you thinking?), slang words and words with negative semantic connotations (“rape”), etc. More properly, the entries should use only the stemmed form to avoid associative interference and should be normalised for frequency for native speakers.

Cheers

Graham


@jasondegraw might just be picky but maybe this name should be OS:AirflowNetwork:MultiZoneComponent:DetailedOpening, there seem to be some levels of hierarchy in here


TESTS FAILED
 Mr.roboto url : http://jenkins.plone.org/roboto/get_info?push=2222b16ddb614fa2be77408e0a491563
 plone-5.0-python-2.7 [FAILURE]


:beers: :heart: awesome job!


Good. I'll make 3 commits then, the ones marked with #1180 (the stuff in between squashed), and explain some more in the commit messages. For example that central skeleton loader alone breaks some builds.


80757414-AddAutoSizing (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1469 of 1584 tests passed)

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


It will definitely consume more power.  But I want to see how noticeable it is over the course of a day.  I raised it on the N5 and still get good SOT.


Here is the output from `ember serve  --help` after I manually added the changes in this commit:

    --insecure-proxy (Default: false) Set false to proxy self-signed SSL certificates

This is kind of non-intuitive. It reads like the `--insecure-proxy` is already defaulted to `false`. I understand that the option will be defaulted to `false` i.e. unused but the help message is somewhat unclear.

Should the help message read 'Set true to proxy self-signed SSL certificates' and then you would invoke the option like so: `--insecure-proxy true`?

Just a thought.


Would like a better way to check if this is the first page visited.


Code_Cleanups (DeadParrot) - 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-68.92405063291139%25-red.svg)


Its by choice from the server monitoring and discovery spec. Due to
problems in the past with mixed replicaset seed list issues. I can relax it
if need picking the first replicaset name as the valid one but I rather not
unless there is a very compelling uscase im missing
On Jan 6, 2015 5:44 PM, "Valeri Karpov" <notifications@github.com> wrote:

> Is this part of an existing connection string spec and/or do other drivers
> do it? I know this particular change will break some of my code, so I'd
> like to make sure there's a good reason for it.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mongodb/node-mongodb-native/commit/4d4ab9d147dfd75f64dbc3169e546879486a4233#commitcomment-9171332>
> .
>


caralho.
chega uma notificação, eu penso:
— "porra, me adicionaram no repo do update sem where? WTF?"

venho ver e é um convite.

É ISSO PRODUÇÃO?


Could it be call for help status? but on a player?


I can't make this `.merge` method, because I don't know how to express restriction on `AnyPar` so that the two paths with same inside-out types


@TheCoPa yes there is: [our official mailing list](https://groups.google.com/forum/#!forum/democracyos-app) (it's stated on our README.md)

We'll also be adding gitter shortly, but ATM that's the way to go.


r=alexcrichton p=1 (prioritizing for alpha)


cf. https://review.cloudera.org/r/4834/ comment, not sure it is the best way to do. We could have instead a ini config to turn create home on/off (on by default)


What happens if the executable has been deleted since it was started or the executable's directory is no longer readable by the effective uid/gid?


Makes sense. Its a "may" in the server discovery spec, so sounds reasonable to me. Do we note this in the documentation?


This isn't following the style of the rest of the codebase and should be `char *id`.


p=100 (major breaking change for macro users)


nitpick: `flycheck-get-new-checker-for-buffer` documentation is kind of a lie now, isn't it?

It says:

> If a checker is found set `flycheck-last-checker' to re-use this checker for the next check.


nah, 4.9.1 wasn't released when I pushed this and I haven't got around to change it.
Using 4.9.1 should work fine.


This worked, thanks, Mark.

Now I have a different problem. I have a single item in my index and yet totalCount on the browse result is 2. Here's an example:

/solr/biblio/browse?extras=title&wt=json&json.nl=arrarr&from=A&offset=-3&rows=20&source=lcc&indent=1

```
{
  "responseHeader":{
    "status":0,
    "QTime":9},
  "Browse":{
    "items":[{
        "count":1,
        "useInstead":[],
        "ids":["b3247409"],
        "sort_key":"PR6066.U44",
        "seeAlso":[],
        "note":"",
        "heading":"PR6066.U44",
        "extras":{"title":[["The ruby in the smoke /"]]}}],
    "totalCount":2,
    "startRow":1,
    "matchType":"NONE",
    "offset":-3}}
```
Then if I make 'from' a value that falls after the single entry, I get startRow as 2 as well.

solr/biblio/browse?extras=title&wt=json&json.nl=arrarr&from=Z&offset=-3&rows=20&source=lcc&indent=1

```
{
  "responseHeader":{
    "status":0,
    "QTime":2},
  "Browse":{
    "items":[{
        "count":1,
        "useInstead":[],
        "ids":["b3247409"],
        "sort_key":"PR6066.U44",
        "seeAlso":[],
        "note":"",
        "heading":"PR6066.U44",
        "extras":{"title":[["The ruby in the smoke /"]]}}],
    "totalCount":2,
    "startRow":2,
    "matchType":"NONE",
    "offset":-3}}
```
This doesn't seem right? And now I'm unable to identify the case where the user query falls after the last item in the browse index.

Thanks!


You only want to do this check once per client host.  Add a TODO to this method "Need to only run server version match once per client host creation" and then I'll tackle it during a later refactor


Can you say me that which tree to use for kk based ROMs?
This or https://github.com/armani-dev/android_kernel_xiaomi_armani/tree/cm-11.0?


okay


What in the world is umpa lumpa oil?  Are you sure this is a healthier option?


Чтобы вырезать отверстия для экономии их нужно вставлять здесь?


Please use `logical_px`.


That would make this the only unfrivolously named project for miles around.


Why this change?


> Maybe you should use `editor_select_lines()` to do something like this:
> […]

Hum, if we do we might need to restore the selection afterward as it modifies it.


I don't understand this.  'L' doesn't make a difference here. Maybe you wanted 'U' as in 0xffffffffUL.

FYI (assuming a 32-bit platform):
```
$ cat test.c
#include <stdint.h>
#include <stdio.h>

void show_num(int64_t v, const char *str)
{
    printf("%-15s %jd\n", str, v);
}

#define show(x) show_num(x, #x)

int main(int c, char **v)
{
    show(0xFFFFFFFF);
    show(0xFFFFFFFFU);
    show(0xFFFFFFFFL);
    show(0xFFFFFFFFUL);
    show(-1);
    show(-1U);
    show(-1L);
    show(-1UL);

    return 0;
}

$ ./a.out
0xFFFFFFFF      4294967295
0xFFFFFFFFU     4294967295
0xFFFFFFFFL     4294967295
0xFFFFFFFFUL    4294967295
-1              -1
-1U             4294967295
-1L             -1
-1UL            4294967295
```


As written this seems to indicate -a and -t must *both* be specified.


Right.  My understanding from David is that we want to postpone the expensive (ledger accessing) checks for the last possible moment.  Currently there's only one place that verifies a multi-signer is actually part of the allowed list: https://github.com/scottschurr/rippled/blob/m-of-n-entries/src/ripple/app/transactors/Transactor.cpp#L389

My understanding is that the get_signingaccount RPC command is mostly a reference implementation (and I need it for my own development), since smart people won't submit their secret on a network.  The implementation that really matters will be written in javascript.


The empty header view should do the trick, but its always created, even if its not used.  Maybe instead of a trinary operator this could be an if/else block and move the view creation into the proper clause?


There should be at least another check in here to use the block color if it is set and only then fall back to `colors.bar_fg`.


Comment


I don't think we're handling the `err` path here.


Why the `-3`? Either way, this breaks the configurable bar `height`. Please use `bar_height / 2 - block->icon.height / 2` instead.


This violates the `statusline_width` we calculated where we used `5`. Also use `logical_px` here as well.


@Sarcasm Indeed, many thanks for catching this.  Fixed.


New comment


Hm, maybe it should be renamed to full-transponder or so.


I haven't compiled this, but I suspect that this change completely breaks background / border colors. The icon must be drawn after those and background / border should enclose the icon as well.


eh, why cast here? Shouldn't it be done by the function itself.
Documentation strings have 2 things in them:
@params explaining all the params
@return explanation of return value

So the type of return should be fixed in the function itself.


That's just diff messing with you :D


This ./ is may not be true.


Haven't tested this, but I think this is breaking the click detection on status blocks if a block has an icon. The detection needs to be fixed.


This commit would probably be prefixed with ``manpage: dvb: ...`` or so.

Also, minor difference to the usual formatting subject lines are written in mpv, but it doesn't matter.


The `CHARACTER SET utf8 COLLATE utf8_unicode_ci` is repeated 3x in this file and `ENGINE=InnoDB` 2x. Not sure if worth worth effort to turn those into a variable, or not. I just call em out as I see em.


It seemed that the `#if defined` syntax did not work for me before, but maybe I was just imagining things, as of course it should. Should I revert it / are there compatibility advantages of the `#if defined` with other preprocessors?


Please remove this code. It's not existing i3 infrastructure and I'd rather not introduce what I consider dead code.


Not sure open a ticket so we remember to check
On Jan 6, 2015 6:08 PM, "Valeri Karpov" <notifications@github.com> wrote:

> Makes sense. Its a "may" in the server discovery spec, so sounds
> reasonable to me. Do we note this in the documentation?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mongodb/node-mongodb-native/commit/4d4ab9d147dfd75f64dbc3169e546879486a4233#commitcomment-9171741>
> .
>


Why not use the `FREE` macro here?


@LNETeam unneeded comment


Poivron


Since I mis-typed `--dvbin-full-transponder` at least once today, I suppose you are right - will rename it later today.


r+ p=1


Needs a comma after 'mail' here.


Better yet use `os.path.join(directory + "/"+ name + ".out")`


Wordpress, Laravel


Please fix indentation, so 'gplus' and 'link' start on the same line as 'mail'.


It's equivalent.


and btw you may also find use of `os.path.isfile` and `os.path.isdirectory` somewhere in future


Lol
On Tue, 06 Jan 2015 at 6:14 PM ░▒▓█│【Walkman】│█▓▒░ <notifications@github.com>
wrote:

> [image: facepalm]
> <https://camo.githubusercontent.com/805ee37e1f5abcbbadcebbf6e5f1db89fa49efc0/687474703a2f2f6d793335307a2e636f6d2f666f72756d2f696d616765732f736d696c6965732f6661636570616c6d34323535303938372e676966>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Walkman100/Walkman/commit/49469a7e9bbfc7185543a0c572aa5308ddba0e4d#commitcomment-9170822>
> .
>


4558_hpwh_coil_removing_heat_from_tank (nmerket) - Win64-Windows-7-VisualStudio-12: OK (1583 of 1584 tests passed)

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


So you figured out branches. Great :smile:


@keedi 님과 확인한 결과 같은 `order_id`에 대해서  `status_id`가 한번이상 발생할 수 있습니다. 예를들면 `결제대기` 상태까지 갔던 주문서가 다시 `의류준비`과정을 거처서 다시한번 `결제대기`까지 가는 상황등도 발생할수 있기때문에 `order_id`와 `status_id`에 PK를 걸면 트리거를 통해 insert 되는 시점에 DBD::mysql의 `raise error` 옵션에 의해 500오류가 날수있습니다. 일단 실서버에서는 PK 제약은 먼저 제거했습니다. 코드는 내일 정리하겠습니다.


Лучше отверстия вырезать булевой операцией после строки 42, так же, как вырезается отверстие оси.
Это если хотите добавить улучшения в сам проект. Иначе, проще всего получить модель большой шестерни res = gears.bigGear(20) и вырезать уже из нее. Точно так же, булевыми операциями.


Is there any way to check the site from my fork before I create a pull request?


I can check later what levels I'm getting, however, 170 was far too high, it took a long time before it managed to identify the tag. I'm sure we can set it higher, but not all the way at 170 - I'd rather have the luxury problem of 'ghost tags' when there's no tag there than the real problem of tags being undetected.


This overrides is.date type check. Maybe dateString?


I'm good with the necessity of copies.  What I'm uncomfortable with is naming a method that makes a copy of a vector `static_cast<>`.  When I see `static_cast` written in source code I would be surprised if that call created a copy of a vector.  And today the only places that call `static_cast` may require copies, but as the code base is maintained that may not continue to be the case.


Tony's maybe??


Tabasco maybe??


These scope params would be better implemented as constants that get exported rather than magic strings.


I don't think this works cross platform. You can use `os.EOL` instead to get the right character for the platform.


Done in 660d09a. For now, I just used --comparable, but soon, I plan to overhaul the command-line parameters.


That seems unnecessary, I was fine with immutability by contract. And it's also a BC break.


Yeah I'm sorry about that, I had a file setting defaults that had `App.base = ''`, which of course was wrong.


minor string formatting nit: space at the end here would turn
> The host cant be foundesx02
... into ...
> The host cant be found esx02


Mmmm drinkable diabeetus!


No, it was my bad. I made two of these types of commits, then I discovered that I had set the editor to use tabs for indentation in TEXT mode only, while still using spaces in CODE mode. I commented the other  commit that I had f*ed up, but didn't mention that I'd done the same here.

I intended to do as the HACKING.txt says (and which I agree with 100%), use tabs for indentation. Sorry all.


Use `ol.EOL` here


Simple yet elegant!


This should check against a list of known good scopes instead.


attn @snarfed, this is ok with me; is it ok with you?


Thanks Nik, that answers my question.  There are folks that create plain-old `STBases`.  So the implementation you have here is correct.  Thanks for checking into it.


Strange,  170 is the graph equivalent to 48.  Which is already low.  I can
get static up over 20 graph or 148 uint8
On Jan 6, 2015 12:24 PM, "Martin Holst Swende" <notifications@github.com>
wrote:

> I can check later what levels I'm getting, however, 170 was far too high,
> it took a long time before it managed to identify the tag. I'm sure we can
> set it higher, but not all the way at 170 - I'd rather have the luxury
> problem of 'ghost tags' when there's no tag there than the real problem of
> tags being undetected.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Proxmark/proxmark3/commit/d0d1e245902fa88ef5d092e68b7e62d9d7b297b3#commitcomment-9172074>
> .
>


Thank you Ariel! =)


Yum


Yes, this was the fixed. Fixed in commit 43fbd0e375ea7eca9ed


I am not the author of this patch. I found it online and just ported it for your version of i3. I'm using your version of the i3 patched with this patch. So can do with it what you want. It's just an idea and it is working. I'm just not very familiar with C/C++.


Just curious, but why storing in the FS?


Agreed! will fix in next commit.


Why is this needed?


@snarfed, I don't believe this worked before -- python-instagram media_create_comments' response would have been empty. so this should not work in the same way, creation success but no comment id :stuck_out_tongue:


You mean from the point of view when somebody changes the url?


I'm super hooongray.  Sound awesome.


Might be overcooked and 'Other Stuff' poorly defined


``listFiles`` does not return ``null``


``listFiles`` computed twice (see line 20)


Yeah somewhat :smile:


Shouldn't the name of the setter and getter be updated too?


Yeah, this is not possible. We are trying to decouple components, whereas this is adding more coupling between them. A different solution is needed here.


@yousefcisco This test is failing on master. Mind looking into it?


Peanut butter sushi!


PEANUT BUTTER JELLY TIME!!!! WHERE YA AT, THERE YA GO!!!!!

![image](https://cloud.githubusercontent.com/assets/2319066/5632882/f3abf052-9597-11e4-9dbb-2f8c368de289.png)


What temperature do we set it to?


Ok well it's only array syntax anyhow.

Also if you host it on your home server then you pretty much are breaking only Sony's terms and conditions rather then a hosting companies as well.

—
Sent from Mailbox

On Tue, Jan 6, 2015 at 4:30 PM, Christopher Wyatt
<notifications@github.com> wrote:

> @bweston92  I never claimed it was a lot of work but it still took a non-zero amount of effort.
> PHP 5.3 is indeed EOL and I realise there will be no more upstream patches to the 5.3 branch but as far as I can tell Precise Pangolin is supported by Ubuntu until 2017 and they will backport security fixes into their PHP 5.3 package.
> I use a Precise Pangolin EC2 image on my server and the latest version of PHP available in the official repo is 5.3.10 but checking the changelog shows patches are included for vulnerabilities found four months after 5.3 reached EOL.
> Your point about the PSN terms and conditions is interesting though I'm not sure how anyone would make use of this code without breaking Sony's T&Cs and, in many jurisdictions the law.
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/ilendemli/gumer-psn-php/commit/531b5e61ed0aca8cb06b42794c834012ef0210ab#commitcomment-9171076


Spec


Nice - how do you pick good green onions?


I'm getting an error due to this commit.

$ arc
PHP Fatal error:  Call to undefined function phutil_console_format() in /home/becorder/src/arcanist/scripts/arcanist.php on line 513

Fatal error: Call to undefined function phutil_console_format() in /home/becorder/src/arcanist/scripts/arcanist.php on line 513


ehh... that would break all the tests if we ran it on a different OS


Ну типа да, возможно.


shouldn't you rethrown the NotFoundHttpException you cached when the legacy kernel also returns a 404 ?


Do you need to add some water for this to work?


Add some Filé before serving


What's the best store to buy cheesecake from?


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


From the perspective of the user of `gatt`, these things aren't `Adapter`s, they're just the device. The fact that they're `Adapter`s is an internal device. How about calling this `Device`?


Hm. Actually, looking at it again, it may be that testmax only checks the first 65 samples, while the rest of the implementation (correctly) looks at the entire buffer to find a match


I have never seen anyone doing `$httpRequest->getUrl()->setSth(...)`. This also kills performance optimization of `StaticRouter` ([proposed](https://github.com/nette/application/pull/35) for Route as well)) when using nextras/link-factory, because `$this->httpRequest->getUrl()` now always returns different instance.


All these methods need to be separated by a newline.


imho better use trigger instead of this https://github.com/kostya/eye/issues/88#issuecomment-62616586


We said this on IRC, but just leaving a comment for the record: This seems wrong. Adjacent non-terminals are not necessarily safe even though parsing is already greedy, because changes might cause us to consume more than we used to.


fixes #39


80757414-AddAutoSizing (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1475 of 1590 tests passed)

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


Ah, misunderstood the word "keep" here :)


Maybe we could make this a config option, but I don't even think that the compilers respect `os.EOL`... and we certainly don't want to mix line endings


So in this commit the issue resolved is when ``file`` is ``NULL``?


Rename this file config.go


I am trying to follow that as well.  (don't shoot me if I've missed something - I'll adjust as I figure it out)


@malukenho this is about what you spoke to insert title and text dependencies in construct method I will fix this tonight.


likewise this seems wrong. I think we want a "follow" value here that is never considered safe, not one that is always considered safe.


I think so you meant os.path.join(directory, name+".out") because concatenating and passing to os.path.join won't do anything different.


Also, you don't _need_ to use the correct line ending for a platform... modern editors can handle whatever line ending you throw at them.


Thanks, see https://secure.phabricator.com/D11254


I like raw salmon


Or maybe this should be a `Peripheral`. What is the plan for peripherals and centrals?


Lets use 'wikitext' instead of 'wt'.


ClientConfig or just Config


This seems close but not quite good enough. In particular, we don't require a trailing separator, so using the separator as the follow set is too strong. We really want the follow to be some kind of union of the separator and whatever comes is next after the sequence to account for the final element, no?


What is `generator_of_xml'` for? Why does `uri` ignore `xmlbase` here?


OverrideConfig


The meaning of `Start` depends on whether this is peripheral-only or a device in general. If it is a device in general, then we may need to be clearer here.


develop (axelstudios) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2064 of 2188 tests passed)

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


this seems correct, though.


@rafales I'd like to make a breaking change that asks people to configure `exclude` on their own (normally put `node_modules` in there). Before that, maybe we need to go the ugly way and explicitly test both `node_modules/react` and `node_modules/webpack` with Unix and Windows separators. Would you file an issue for this?


Same for `Scan` -- peripheral vs central changes the meaning a lot.


r+ rollup

Thanks!


Hmm. Oops.  You are correct...  That is not good
On Jan 6, 2015 12:36 PM, "Martin Holst Swende" <notifications@github.com>
wrote:

> Hm. Actually, looking at it again, it may be that testmax only checks the
> first 65 samples, while the rest of the implementation (correctly) looks at
> the entire buffer to find a match
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Proxmark/proxmark3/commit/d0d1e245902fa88ef5d092e68b7e62d9d7b297b3#commitcomment-9172301>
> .
>


i literally only made that change so that i would have a commit message


and idgaf about autowhitespace


ConfigLoadingRules


Why does a `Descriptor` need to be an interface instead of a concrete type? Seems like it'd be the same across all platforms, and a concrete type will be more efficient.

Also, why is it exported?


Where would I buy this?


RecommendedConfigPathFlag, etc


Consider instead of:
```Python
for h in allhosts
    for a in acthosts:
```

... doing something like this...

```Python
    host_names = [ h.name for h in allhosts ]
    for a in acthosts:
        if not a in host_names:
            print "The host cant be found " + a

    for h in allhosts:
        if h.name in acthosts:
            enable_autorestart(h, defstartdelay)

```

Instead of looping through both lists looking for matches ... first we validate that acthosts has all valid host names, then we act on the hosts in the list of hosts to be acted upon.


"$HOME/SDL" not "/Users/n0iz/SDL", mistakes...


Same question as for `Descriptor` -- why change this to an interface?

I have the same question for all the low level `gatt` type changes.


How about this:
https://github.com/Proxmark/proxmark3/commit/e47e52a2c8f2d925cefe2f5f1a55ceceb7ca0813


It's bad practice to implement Write on an object in golang that doesn't match io.Writer's signature.  WriteFile would be better.


Ideal but more complicated is immutable Url.


Тогда лучше добавить к полигону набор точек (до строки 40), описывающий отверстия и задать пути. Вот здесь можно глянуть, как это делается: http://en.wikibooks.org/wiki/OpenSCAD_User_Manual/Using_the_2D_Subsystem#polygon


Uggh. This commit still has some remnants of my local debug commit that I didn't separate out properly. I'll push a commit later that sets this back to zero, and removes the dump_pkt variable.


Though in theory we don't need to check the entire bitstream, just enough
to make sure we got a high bar reading...  The more times we loop through
the entire sample set the slower our arm demods go...
On Jan 6, 2015 12:42 PM, "marshmellow marshmellow" <marshmellowrf@gmail.com>
wrote:

> Hmm. Oops.  You are correct...  That is not good
> On Jan 6, 2015 12:36 PM, "Martin Holst Swende" <notifications@github.com>
> wrote:
>
>> Hm. Actually, looking at it again, it may be that testmax only checks the
>> first 65 samples, while the rest of the implementation (correctly) looks at
>> the entire buffer to find a match
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/Proxmark/proxmark3/commit/d0d1e245902fa88ef5d092e68b7e62d9d7b297b3#commitcomment-9172301>
>> .
>>
>


The change is small, however, I like to check error conditions early and exit, rather than placing large blocks within if-clauses if possible. I find it makes following the flow more simple.


I'll accept ClientConfig since you have a config type.  This file should be called clientconfig.go


OverrideClientConfig


80757414-AddAutoSizing (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Yes, my patch does that, it checks only as long as it has to


Do you mind reverting this commit? Everything's good, except 1.7 does not  run on Yosemite, and thus I require 1.8. Anyways, downgrading to Java 7 will also invalidate some of the code that uses Java 8 conventions. I have 1.8 running on Mac and Windows, so at the very least using 1.8 should not cause problems.


That is awesome! I am glad I can contribute. Thanks for all your work man!
On Jan 3, 2015 12:37 AM, "friedrich420" <notifications@github.com> wrote:

> People are loving it my man!!
>
> Sent from Yahoo Mail on Android
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/friedrich420/N4-AEL-Kernel-N910F_EUR-Source-/commit/54babdf0a8290b4926dde4d3a60b223f9ec865c8#commitcomment-9141710>
> .
>


what about other headers and cookies ?


What is the contract of this interface? Does resolve return always one item and completes or streams a sequence of items?


I think for certain sequentials it's beneficial from a performance perspective to save the result of calling `seq` on them instead of calling it three times: `seq`, `first` and `rest`.

```
(when-let [s (seq s)] (cons ....))
```


@tomjakubowski I suppose this entire `defrecord` form could be put inside an if form that uses the new `-setup` arity only if the the compiler is `0.0-2665` or later, thus making it not be a breaking change. Let me know if you'd prefer a PR with that instead.


GoDoc


We had that in the past with `FreezableObject` and it was removed. I don't understand why you are trying to solve this because I don't consider current situation problematic.


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


See NODE-345

On Tue, Jan 6, 2015 at 12:19 PM, Christian Amor Kvalheim <
notifications@github.com> wrote:

> Not sure open a ticket so we remember to check
> On Jan 6, 2015 6:08 PM, "Valeri Karpov" <notifications@github.com> wrote:
>
> > Makes sense. Its a "may" in the server discovery spec, so sounds
> > reasonable to me. Do we note this in the documentation?
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/mongodb/node-mongodb-native/commit/4d4ab9d147dfd75f64dbc3169e546879486a4233#commitcomment-9171741>
>
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mongodb/node-mongodb-native/commit/4d4ab9d147dfd75f64dbc3169e546879486a4233#commitcomment-9171955>
> .
>


_allowedKey and _correctSignature are short enough, and only used from _checkKey, that I think the logic might be easier to follow if they were inlined into _checkKey.


Having `->` in the ty follow set just doesn't seem right. For example, `FnMut()` is a type, and so `FnMut() -> int`, so I think it doesn't belong there.


Looks good.  I usually try to do similar, just must have been rushed.


Ok, thanks for comments. Always learning :-)


Ok


It is kind of surprising that this works without `-XTypeSynonymInstances`... Perhaps I do not understand the semantics of that extension well enough though.


Make this a typed error (export var ErrContextNotFound) so that someone can handle this specifically.


I changed a bit the comment and added an inline comment about the comparator for sort() as darktrojan suggested.


export ErrNoContext


ErrContextNotFound


drop this comment?


Yes that's correct.


Looks good to me.


Okay then...


All errors should start with a lower case `fmt.Errorf("unable to read ...")


+1 apart from the line comment.


Also, `auth-path` is going to depend on the flag the user chose.  So you have to decide whether to validate using the flags the caller provided (probably the right thing to do) or be more generic about the flags.  I'd prefer the former unless you can make an argument about the latter.


Validate loading files feels wrong to me.  Can validate be called more than once?


Immutable Url sounds better to me.

@JanTvrdik It is very simple. Let's take Url from http request in one service, pass in to another as argument to some method and then it can be changed very easily. The service accepting Url as argument in its method should not care about the source of the Url. It can be passed from another place as custom instance a there could not be a problem with mutable Url. However with Url from HttpRequest it could create some weird bugs hardly to discover and test.


pls @pranavmathur use issues if you only want to start dialogue.


s/authTechniquesFound/methods/g


I believe this causes an attribute order dependence between `xml:base` and other attributes that use relative URIs. If I'm reading the w3.org spec correctly, this should not occur and the `xml:base` for an element should be used on all of its relative reference attributes. I: a) haven't tested this, b) may be reading the spec wrong, c) don't know if you've somehow handled this elsewhere.


@freakboy3742 -- what do you think of these changes? I am nervous they may cause weird and unintended/unforeseen sideeffects if people implement the flags on custom fields. Can we leave them out of this PR for now or do you think they are correct?


What do you use randomGenerator for?


I just checked, and yes, it does seem to mean that this is error message is less meaningful. It also seems like it is used in https://github.com/puppetlabs/puppet/blob/master/lib/puppet/indirector/indirection.rb#L308. I can add it back but rename it something like `to_uri`.


Instead of making Jacobians a separate part, please just make it a section in the Low Mach Navier Stokes chapter.


This change request removes HealthCheckService. Do you want to implement it separately from self-registration?


though we probably could have lowered 170 to 145-155 or so..   not sure how other antennas react.


This should be fixed in HEAD (2035330780a4745b1b3b92644819c405aa7698b4). Thanks for the report!


(if they stay in, the backslashes used for line continuation should be replaced with parentheses)


Complete feels like the wrong word, but I don't have anything better.


Hm. I tested the patch before commit, and it worked. However, I did it wrong, so it continues if there are low values (should be justnoise = dest[idx] < THRESHOLD). So, changing it to the correct way, I don't get stable readings. Same problem as before...


@flavour , Sorry, i will change it to interval. But this is not implicit as it is initially set to blank and only sets the interval if it still remains blank when selected.


ClientConfig


Yeah
On 6 Jan 2015 16:48, "DanRyanIrish" <notifications@github.com> wrote:

> Single backticks? Like astropy.units.Quantity ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/DanRyanIrish/sunpy/commit/ae1d9d4c385440b39e8b16543a17af3b0cecc263#commitcomment-9171397>
> .
>


Why don't these use the `_of_xml` functions on lines 187-191 above?


Why do you keep these two as class level variables?


We should make sure slugs are slugs....
so beer-is-nice

If we can figure out a way to pass the template engine swig helpers we can create a slug filer (as we did with kala.com)

https://github.com/segmentio/metalsmith-templates
which uses
https://github.com/tj/consolidate.js


what graph levels do you have?


khkhkh


kbb.kb.kjb.jkb.kb.k


This is not a build error thanks to @friedrich420. This is copying changes instead of cherry-picking them and not getting the portion that removes dependancies resulting from porting Note 3 changes to the Note 4.


I'm trying to find that... how do I see that ? There are no scales on the graph...


Thanks :)


@stekycz I understand the problem in theory I just don't think we have the problem in the real-world. Or at least not often enough to warrant this change which has clear disadvantages.

We had immutable `Url` in Nette 2.0 and it was removed in Nette 2.1.


[Dockerfile]
- adding the -discovery flag to the startup script
- adding the volume to the docker file

[Docker/startup.sh]
- adding the startup.sh run file
- adding the external docker file

[stage/Dockerfile]
- fixing up the Dockerfile to use busybox

[gofmt]
- applying the go formatting to the code

[README]
- updating the README

[store]
- adding the extra logging for changes in dynamic files
- removing the reference to resource var, not required
- deleting any dynamic configs whom are within a deleted directory
- fixing up the code for the above changes
- moved the channels for k/v,templates and fs into the struct
- using fs.Update() to push changes to the dynamic config
- adding the readonly flag, allowing for config store read/write
- separating out the node updates into specific methods to better handle their respective changes
- a few code changes to the store; name changes, code movements etc
- spliting out the node and timer events into seperate methods
- adding the node{update/delete} methods
- got the initial sync working
- now the structure
- performing a gofmt on the source code
- finishing off methods in the filefs

[dynamic/resource]
- no longer require the event handler methods
- adding the getlist method
- fixing the list method

[fs/fs]
- adding the Update() method, ensuring we check the content hash before updating
- working in the watcher
- will probably have to move this into a seperate struct
- added the touch() methods

[tests]
- adding a test haproxy dynamic config
- fixing the template for haproxy. Golang text templating is HORRID!
- adding a script to sync a directory of configuration files to etcd
- fixing the script to synn
- fixing the test template code

[gofmt]
- applying a gofmt on the code

[config-fs]
- working to clean up the dynamic configs, the watching of services and keys and release of resources afterwards

[etcd]
- cleaning up the code

[kv]
- moving the NewKVStore() here, as it makes more sense
- moving the command line flag into the package

[dynamic]
- mostly changes to the comments to reflect the name changes

[config-fs]
- moving elements around
- change templates to dynamic config
- moved the file system methods into fs package
- remove the utils in discovery
- fixed up the packages to reflect the changes

[discovery]
- removing the middleware is discovery and using the agent direct

[watcher]
- adding a channel type
[tests]

[template]
- added the event handles for changes in services and k/v

[discovery]

[discovery]
- removing the tags from the service
- adding consul as default agent
- todo: need to now listen for events and send upstream
- fixed various elements in the consul agent and discovery as whole
- updated to use a type channel
- fixing up the command line flag

[general]
- removed the resource manager and placed into the store instead

[template]
- adding the TemplatedConfig struct, which *should* be the entry point for templated config
- need to consolidate keys with the kvstore - perhaps??
- adding elements for the templated resources
- moving the file notifications in to a separate service
- removing from a package and placing in the base
- working in the resource - need to establish a struture tho

[Godep]
- adding the fsnotify to the dependencies

[cache]
- removed the package for now, not really required


Any chance of uploading this fix to the Chef Supermarket?


max=109,. min -85, mean -5... but I don't can't see quite what the 'normal' value is


develop (axelstudios) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


It may not be doable within metalmisth.json though.


@flavour Oh ok, should i change the Todo to "change implicit default to explicit default with interval of 12 months (or according to deployment settings)" ??


So the LMNS chapter would go under the FE Formulations part?


@myleshorton This fixes the non-blocking close behavior, hopefully makes it a little clearer, and adds a unit test for Close.

There are some other things fixed as well (see inline comments).


Neat


Make sure that the listener is closed by putting this in a defer.


that should mean in that function your maximum dest[idx] == 237 which should be plenty...  maybe there is another issue.


>See also: igloo - my last posts from about 2 or 3 days ago.
Yep, that's why I assumed you'd rolled them all into one option.


Make sure that if idletiming.Conn.Close was called, we actually close the underlying connection.


Less or compass, but I think less is best.
On 6 Jan 2015 14:02, "Gerben" <notifications@github.com> wrote:

> Yep. I thought perhaps placeholders for each item would make sense to make
> subsequent commits clearer. Let's clean up the css soon anyway. Using LESS
> or Sass?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Treora/cothink/commit/6688f17220276c8833b8a402bb7112fc707b9172#commitcomment-9168111>
> .
>


I think it'd work too, right. Do you see any advantages of (q) beside being more elegant?


@flavour , should i use it as an attr that was passed to S3DateWidget?
i.e interval = self.interval_default ???


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


What is the win here?  Are you planning to monkey-patch the source module, rather than the MUT?


brednie :-)
kupiłem ten kod na giełdzie :-)


You should remove the
```
// + 32 below for some safety against bad input (for good data
```
comments.
And you might try to inform upstream.


Hm, may have been problem with other tags lying to close to the antennna... I adjusted it down to 140, it works now, and is pushed.


Move author and date to the header (it should state copyright, year, optionally author and date.. also it should explicitly state the license - so other people out there don't need to contact you about license of this published code)


Isn't this call redundant since the only way you can get a value from c.closedCh is from a prior Close call?


Write javadoc for each class, method and field, according to Java standards.


Doh, that's supposed to be c.conn.Close().


80757414-AddAutoSizing (Myoldmopar) - x86_64-MacOS-10.9-clang-Debug: OK (1469 of 1584 tests passed)

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


It's unclear to me why I changed it.

The rename part was a sacrifice to the lint gods since the global `connection` gets overwritten in a method signature as a `local`.


I 'think', we probably could remove all the + stuff, and simply allocate rar_malloc(length), but I have not tested that yet.

I also removed the printf. I left it here to 'show' in testing that the fix is working.  I have another commit after this one that gets it into 'proper' quiet mode.


if you try it with no tag do you get junk?  140 is just 12 on the graphbuffer


I don't know if this makes sense as the domain is a DNS name and not a URI. ???


If it makes sense for `domain`, should it also be used for `path`???


Comment for actualizing date commit


Is ``avatarId`` a ``Deferred``?


Gotcha. I think that makes the above comment incorrect though right, since then it would close the underlying net.Conn?


Probably not as the GUID is supposed to be opaque?


The comment is wrong, fixing.


I don't get any junk, no ...


I am testing rar_malloc(length) right now.


Reviewed the PR and looks good.


No need for deployment setting. This is a username thing so for credentials it can be 12 months


OMG the hard tabs! Fixed in 450a6ffa14b35873fba63c9e105ad8c1f13b20bb


You have two changes from @xiaolu mixed in with https://github.com/StarKissed/starkissed-kernel-trlte/commit/90189bce2a2aaf265215640f966c0bb6dbac31fb but the title claims it is a KNOX modification, while the undocumented half is for building the kernel in the source instead of to an out folder.


and as you look at the comments, So many people do not like this guy XD


>The .btn-cancel could benefit from the RGBA as it works on IE9+ and all other browsers.

Real browsers and IE10+ [1]

[1] https://github.com/DemocracyOS/app#browser-support


Yes


This should be # CONFIG_SEC_RESTRICT_ROOTING is not set


I realized that we are already calling c.conn.Close() inside of c.Close(), so I took this out.


Is this a reimplementation of `resolve`?


cool


^ @myleshorton


@myleshorton Updated comment to reflect that we do actually return the error from calling Close().


(sorry I don't have the pm3 in front of me so I can't test that value right now)  :)


Yup makes sense.


This commit is just workaround. It is not "BC break" in contract sense, it just prevents unwanted change. But it (unlike immutable Url in 2.0) throws no exception.

FreezableObject was wrong way, because immutability was mutable state. I removed it.

Better is real immutable object without setters. Or all setters can return clone. Nette mutates Url only in few places, so it is problematic only due to compatibility.


It seems like this has introduced a bug where `/tmp/ocpu-temp/` gets deleted. I think it is caused by the first line where you removed the `*` but did not add `-mindepth 1`.


This should be ../../drivers/video/msm/mdss/ <- it is relative to define_trace.h


@myleshorton Updated comment.  We have a strange bifurcation here:

1. If the conn idles, it does not close the underlying connection.  That's up to the client to handle in onClose().

2. If conn.Close() is called, it does close the underlying connection, in compliance with the expectations of the net.Conn.Close() interface which this implements.

I don't love this, but we need the "don't close on idle" behavior because enproxy detects the idle and, instead of closing immediately, does some cleanup on any outstanding network activity before closing.  If it didn't do this, we would get a bunch of "blah blah on closed connection" type errors.


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


Done


:+1: thanks! i might even vote for putting the comment details (everything except the first line) in the commit message instead of the body; i generally don't put too much history in comments, just current state. i don't feel strongly though.


Fair enough.


This was what depwalker came up with, those are all DLL's required for
getting it to run on windows. Many of them are dependencies of libav e.g.
libx264, libogg, libvorbis and all audio/video related
2015/01/06 16:56 "Tapio Vierros" <notifications@github.com>:

> I don't think it's customary to list "recursive" dependencies
> (dependencies of dependencies). Besides, depending on how our direct deps
> are compiled, many of these might be statically linked inside them, so they
> are not required for building Composer.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/performous/composer/commit/cb34360d69ff59aa0deb261e05ca5fdea52aa4e5#commitcomment-9170560>
> .
>


Code_Cleanups (DeadParrot) - 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-68.92405063291139%25-red.svg)


Make this plugin better and best


can you drop the ~ from these deps


The contract is a stream of InstanceInfo items. The stream can onComplete anytime with just one item, with a sequence or never onComplete, it doesn't matter.


could just be:

    from gcloud.datastore.connection import get_entities


can you fix the whitespace?  space between ) and {, space between } and else and {


space after the commas


best to make these absolute paths with a leading /


ditto


we need alt tags to validate


alt


I think it is a good idea creating a canonical Marlin repository.

By the way @boelle , I have just try to submit a fix and have lost my write access. Do you know what is going on??


same as above


I've reverted the change, as it clearly isn't a good idea at the moment.


nice comment, bro.... LMAO


r+


yup, looking at http://instagram.com/developer/endpoints/comments/#post_media_comments and expanding the _Response_ box confirms. sgtm.

more importantly though, did you apply and get comment permissions? if so, we can't actually test or use this, right?


Would you mind explaining your rationale for using void 0 in the first place and why you changed to undefined?


Oh sorry! I'm not a Linux user, so I thought your second statements deletes directories while the first one deletes only files.
I thought because you added a second line with `type -d` because directories weren't included in `find -delete`. But [they are](http://explainshell.com/explain/1/find). And they are also processed after their contents. So a working solution would only need the second line in each case with a an added mindepth argument.


Now that I've said it out loud, I actually think this bifurcation makes sense.  IdleTimingConn is being polite and not just closing the conn out from under you, rather it just notifies you when the connection has idled.


r=nick29581 p=1


Agree. But potentially not necessary if we put `get_entities` in `__init__` instead. See discussion on previous PR.


Thought so


TeamCity MonoGame :: Develop (Win) [Build 3.3.0.1792](http://teamcity.monogame.net/viewLog.html?buildId=4203&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 293, ignored: 6 Build time: 00:16:17


This seems to work across shells, but i'd be more comfortable (and it'd be more consistent) if you used "if test x$use_boost...".

Same goes for the ones below.


Yes, LGTM!


This three-prong check is done because we don't currently tokenize `:::`, `::` separately from `:` (ie, Ace just sees sequential `:` tokens). This will be fixed when we switch to new Ace.


Short notice: I don't think it's this commit that's causing it (as it didn't remove the processing of packet listeners in a executor service). Anyway, I re-designed how smack handles incoming stanzas, and I think that re-design is also fixing your issue. I currently testing the new code. Stay tuned for code pushes. (Feel free to still make a forum post).


That make sense but maybe call the method onIdle then?


I'd like to see this be a bit more robust. We should check for "http://" as well as "https://"


I, too, have the feeling that strange things are happening here. I spent a couple of hours to come up with this extremely simple patch. Mostly because I trying to find the best way to avoid overlapping instances. Along the way a run into weird stuff. For example I made the module compile by adding `ScopedTypeVariables` even though there was no `forall` or nested signature in the affected function. I was wondering about `TypeSynonymInstances`, too.

Do you understand why the `m ~ n` approach fails and using the same type variable succeeds?


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


ad performance in router: the `constructUrl(..., $refUrl)` is not ideal, it would be better to pass $refUrl to constructor (and yes, it has another diadvantages).

$refUrl micro-optimization are rather insignificant, because there are bigger challenges. Router should be initialized only once, to avoid using tricks like

```php
	function afterCompile(Nette\PhpGenerator\ClassType $class)
	{
		$method = $class->methods['createServiceRouter'];
		$method->setBody('return unserialize(?);', [serialize(eval($method->body))]);
	}
```

etc.


Should this be dm-devel or dm-users?  The latter would seem more comparable to the previous list.


keep in mind client updates will force us to occasionally update these ID's. That only takes looking at what they are in npc_list.sql and altering this file to match.