http://legacy.python.org/dev/peps/pep-0394/

* python2 will refer to some version of Python 2.x.
* python3 will refer to some version of Python 3.x.
* for the time being, all distributions should ensure that python refers to the same target as python2.
however, end users should be aware that python refers to python3 on at least Arch Linux (that change is what prompted the creation of this PEP), so python should be used in the shebang line only for scripts that are source compatible with both Python 2 and 3.
*  in preparation for an eventual change in the default version of Python, Python 2 only scripts should either be updated to be source compatible with Python 3 or else to use python2 in the shebang line.

I am kind of surprised that osx is trailing behind on this one, reasonable late python installations should provide a python2 symlink as described in the mentioned document.

It still is my opinion that those changes are needed; however: the Makefile could check if there is python2 available and error out, might be more graceful than how it is now.


something


Channeling SPJ, this certainly deserves a comment about why *not* setting up the submodules here is important, as otherwise it can look like an inadvertent omission.


Yes. So if you want to work on some other things and doesn't want to come with those conflicts, you could create a branch at where it still builds, and create a pull for it once done. Merging is.. delicious.


Mir ist's egal, im Moment auch einfach Eclipse Standard bei mir.


novi edit


Hi @raduq, would be nice if you create a pull request with this language fixes to the master repository.
Regards!


Thanks for the review @jdalton! I'll try to fix these as soon as I get back from holiday.


Yeah I'm wanting to merge mid-kid's new CakesROP into my fork of rxTools. but like with his original fork, it compiles a dat that just freezes the browser/mset when I tried to use it. His local build that he got working might not be on github perhaps. I'll ask him and see.


nitpicking: why do you define the var addresses as plural if you don't make it a list?


了承しました。


The relevant code from core is [here](https://github.com/bitcoin/bitcoin/blob/master/src/primitives/transaction.h#L138).  The actual value is 546 satoshis below which the default core client won't accept the transaction.

There have been lots of changes over time, and plenty of bad information to sort through, it seems, but I believe that 546 is still current.  minRelayTxFee can be set by miner's in their configuration, though, so I am trying to find sources to confirm that many/most miners are using higher minrelaytxfee settings, necessitating the higher (~5500) dust limit for usability.  It's a problem of whether to code for defaults, which I think may reduce usability, versus coding to match the majority of miner configs, which is hard to verify and may change as mining economics change.  Again, maybe this is best left to the makers to decide by way of joinmarket.cfg.

Note to self, I'm having a hard time wrapping my head around what exactly [these lines](https://github.com/bitcoin/bitcoin/blob/master/src/primitives/transaction.h#L148-149) do, and why they were [different](https://github.com/bitcoin/bitcoin/blob/6a4c196dd64da2fd33dc7ae77a8cdd3e4cf0eff1/src/core.h#L157) before the dust limit was lowered.


This **monstrosity** is not related to the implementation changes in this PR, but it is a necessary step to make spec pass on Windows.

Tests were failing on Windows because [the check for presence of node_modules in path](https://github.com/atom/deprecation-cop/blob/master/lib/deprecation-cop-view.coffee#L169) is using `path.sep`, but the tests had hardcoded `/` which made the `node_modules` detection failed and `package1` was returneded instead of expected `package2`.

If you want to extract this into separate PR as it is not directly related, let me know.


Should this constructor parameter be supplied in DI? Would look like this in global.php:

```
'Zend_Validate_EmailAddress' => DI\object()->constructor(array('mx' => false, 'deep' => false)); // NOTE: you can supply the class name to DI\object(), but if you don't it is assumed to be the value of the key
```

Maybe this is causing the test failure in travis.


Would it be possible to use try-catch here?


@ApacheThunder Yes. However there are multiply targets that working on.
If you'd like you can create a branch and merge his, and test out those new things.


84691154_AutomateBuildingOfDocs (lefticus) - i386-Windows-7-VisualStudio-12: OK (1682 of 1682 tests passed)

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


Yes indeed, rootSIFT values are quite smalls. Mystery solved !


made the change. I will commit the changes when you have finished the review including the xls() chhanges for the addslashes()


Grad dabei

Randy Schütt <notifications@github.com> schrieb am Fr., 3. Juli 2015 um
18:14 Uhr:

> Lass mal 'nen namespace dafür machen. Prefixe sind so 1980.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/snowiow/SIFT/commit/cd28e2143a75b0ae159efffdf4761001a9510544#commitcomment-11997460>
> .
>


I'll have a look. Probably not an intentional change by the author.


If you would an up to date script, please consider submitting a request to my [fork of this repo](https://github.com/austinkregel/php-login-advanced), thank you.


No i don't, since we don't use the top screen (except for the splash)


You can reproduce this by calling a friend from Antidote. And then have the friend accept the call.

#0 in semaphore_wait_trap ()
#1 in _dispatch_thread_semaphore_wait ()
#2 in _dispatch_barrier_sync_f_slow ()
#3 in -[OCTRealmManager fetchRequestForClass:withPredicate:] at Antidote/Pods/objcTox/Classes/Private/Manager/Database/OCTRealmManager.m:91
#4 in -[OCTSubmanagerObjects fetchRequestForType:withPredicate:] at OCTSubmanagerObjects.m:30
#5 in +[Helper createFetchedResultsControllerForType:predicate:sortDescriptors:delegate:] at Antidote/Helper.m:71
#6 in +[Helper createFetchedResultsControllerForType:predicate:delegate:] at Antidote/Helper.m:58
#7 in -[AbstractCallViewController initWithCall:submanagerCalls:] at Antidote/AbstractCallViewController.m:39
#8 in -[DialingCallViewController pushToActiveCallController] at Antidote/DialingCallViewController.m:94


Covering the entire function isn't necessarily so bad since it will protect us from future changes which might add new allocations.


crap.. should be a -y in there... I had a git fail sorry!


good


We could keep the folder structure and make an exeption in the installer (as in "overwrite files in ART when selected language is different than English").


Minor nit that Underscore's `_.uniq` doesn't treat `NaN` as equiv while `Set` will.


[electrum dust_threshold](https://github.com/spesmilo/electrum/blob/master/lib/bitcoin.py#L33) is set at 546 currently, matching the core client default.


You may also dig https://gist.github.com/bendc/9b05735dfa6966859025


Commented on embarrassingly obvious misspelling of the word embarrassing in commit message.


@bastir85 Erinnerst du dich noch, was du hiermit bezwecken wolltest?


I do as you wish ;-)


Hi Tim, I seem to have encountered another roadblock in the tutorial. At the end of part III Groundwork, it says to write movement(); in the update function. I assume this is the update function belonging to the PlayState.hx source, since I have not written any update function code within the Player.hx itself.

However, once I do that it gives an 'unknown identifier' error towards movement(); . I took a look then at the Player.hx source code to compare with my own and saw this:

override public function update(elapsed:Float):Void
	{
		movement();
		super.update(elapsed);
	}

but that didn't work for me. So I erased the 'elapsed' keywords and that permitted the game to run, however once I press start, it closes and gives me a series of invalids.


This isn't an upgrade...


This actually prevents use with 1.3...


This needs to be immediately reverted.


nope


@m-stein I'm wondering which *error value* is correct - 0.01 (code) or 0.001 (commit message)


@i-trofimtschuk Falls der Commit irgendwas sinvolles macht, könnte man den behalten. Die von Hand hinzugefügen PLZ werden eigentlich nicht gebraucht.


This project will be a potential program for making a work space more efficient


fine to disable. but was it causing problems?


There are two different problems. a) if the changes are needed: I think yes, they are needed. b) whatever those changes are they have to be compatible with OS X and other operating systems.


@winlu can you please add such check?

We can add this as a requirement for builds from source but if it's easy enough, should try to accommodate folks that don't have `python2`.

> On 3/7/2015, at 19:19, winlu <notifications@github.com> wrote:
>
> http://legacy.python.org/dev/peps/pep-0394/
>
> python2 will refer to some version of Python 2.x.
> python3 will refer to some version of Python 3.x.
> for the time being, all distributions should ensure that python refers to the same target as python2. however, end users should be aware that python refers to python3 on at least Arch Linux (that change is what prompted the creation of this PEP), so python should be used in the shebang line only for scripts that are source compatible with both Python 2 and 3.
> in preparation for an eventual change in the default version of Python, Python 2 only scripts should either be updated to be source compatible with Python 3 or else to use python2 in the shebang line.
> I am kind of surprised that osx is trailing behind on this one, reasonable late python installations should provide a python2 symlink as described in the mentioned document.
>
> It still is my opinion that those changes are needed; however: the Makefile could check if there is python2 available and error out, might be more graceful than how it is now.
>
> —
> Reply to this email directly or view it on GitHub.
>


I'd like to suggest the possiblity of showing the source code of each .hx that is changed at the end of each tutorial page. This way, those who are following along with the tutorial can compare their code and see where their mistakes are.


FacilityTabGridView (evanweaver) - Win64-Windows-7-VisualStudio-12: OK (2188 of 2213 tests passed)

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


https://github.com/infirit/mate-sensors-applet/commit/89c9d48b3ba13296f63901a60fb5d24ef003797a and https://sourceforge.net/p/sensors-applet/patches/8/


Ya but I think i misdiagnosed -- thought `ldd/otool` was missing as a result of sudoless switch but in actuality guessing this env var just got dropped.

https://github.com/mapbox/carmen-cache/commit/d9bd90430da252d13f1dd14aa3db1fb429a4239b


What happens if modifiers is empty ?


@rxin, It seems impossible specify decimal as valid input type? currently it just rely on no implicit cast from `Decimal` to other types in `HiveTypeConversion` to make it work, what am I supposed to do?


> should try to accommodate folks that don't have `python2`.
We have to introduce this compatibility since this was merged into stable as well


Just try the code fix provider - remove the params modifier should work.


Hi @gregoire-pointeau: Unfortunately, this won't work. An example `rpc` call is as follows:
`triggerStreaming 1234 (realtime 1) (includeAugmented 1) (speedMultiplier 2.0) (robot iCub)`

Thanks for going ahead and implementing this. I haven't had the time yet, sorry :(


A class that has an "I" prefix like an interface? o_O


thanks nimit!


oops


Добавлены комнаты


> No, because that cog wheel is white, not red, so I'd have to have some massive hacks to override the paint function of the icon just to color it red, otherwise you can't see it because the background is also white.
Use Gimp?


nit: you could use the newly defined "psci_make_powerstate" macro to construct the power state.


This line, and the lines below, have a broken indentation. This seems to be caused by using tabs instead of spaces. The rest of the files is using spaces, so I assume this should use spaces too.


Yeah it is a bad name. I will rename it.


Wrong `@since`


What is it now by the way 1.3RC1?

On Fri, Jul 3, 2015 at 12:46 PM, Sergiu Dumitriu <notifications@github.com>
wrote:

> Wrong @since
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/phenotips/phenotips/commit/977aa486f2eaf16861153ebee682134372bfe772#commitcomment-11997776>
> .
>


Hm, for me qTox doesn't receive incoming call.


I just realized you just took this from the ABMHandler. There, it was not updated yet .. @maxime-petit and I will take care of it.


Will be 1.3M1


It's a shame that there's not a "csvdiff" to show just the one column missing rather than 32K +/- ...


Brilliant @kritikanangia !
Since issue #1 is solved, you can comment there and close it, preferably by mentioning this commit's unique SHA hash, written above. See this https://guides.github.com/features/mastering-markdown/


I suggest you consider adding the following:

# npm packages
node_modules/
# node tools for visual studio
.ntvs_analysis.dat


That's what 0 does. -1 (or any non 0 it seems) will return the full queue but also won't result in an error if called too many times.
The issue here is wonky engine code that was written to make sure people are using the optimized function in all cases. Passing -1 is equivalent to the statement "yes I want the full queue, please don't error on me".


Done in https://github.com/s9y/Serendipity/commit/6588c85d87a0e6a4dc42aa74c2ec87002cec62b9


don't forget that there is a 3rd option, namely making the codegen scripts python2+python3 compatible, removing the need to specify python2 as an environment. I will have a look at it.


Yes please rename to something more generic.


This cacert.pem update is causing issues with the Joomla Updater connecting to some servers. In JCE's case, this is related to the removal of the Thawte Server CA and/or Thawte Premium Server CA certificates.


smtp_port ?


It would be good to keep this sort of thing a config option in the admin screen. This is a specific requirement for the Flora and will probably need to be removed for other uses of the profile editor/system.


@gregory38 Yes it's enabled. That's what I am trying to tell you, before this revision enabling accurate date fixed the shadows but now accurate date option doesn't make any difference. So either it's not getting enabled or not implemented correctly.


I tried this today but it doesn't work :( I figured out that global installed packages can access via command line but not in a script via `require`.

@deepu105 Do you have any other idea to solve this dependency problem?


whoops yes I broke that.


We have no method of enforcing that. Someone should try implementing one.

Not using anything that isn't exported inevitably leads to the no-modularity, export-everything situation we have right now, which isn't ideal either.


That's true, as well. Testing for german words would also be nice, hm...I llike the look of [Entropizer.js](https://github.com/jreesuk/entropizer), it follows the concept of calculating the entropy and leaves the UI to the app, see e.g. the [jQuery Plugin](https://github.com/jreesuk/jquery-entropizer) with the buckets which maps entropy to an actual score.
It clocks in at 1.4 kb minified.

As for zxcvbn-async.js, we could either manually them to the assets folder (they will be copied to the public folder by brunch then) or somehow tell brunch to leave these files alone and manually insert the async script version on the manage account page.


qTox isn't on the new toxav :). I have been testing with objcTox demo


I see, thanks for yours great works. :+1:
I followed your comment in https://github.com/agsh/onvif/pull/13 yesterday when I was debugging but thought that was only relate to getHostname.


Already done. :)
fre. 3. jul. 2015 kl. 18.17 skrev Benedek Fazekas <notifications@github.com
>:

> could not we do the same with extract-fn?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/clojure-emacs/clj-refactor.el/commit/89ee66923cfc66fcde752651f3b6cd34d0beda5a#commitcomment-11997503>
> .
>


The validation is restricting to a list of well-known constants which does not allow for supplying either a local machine account or more likely a domain service account. This becomes important if the web application being installed will be accessing resources and services on other Windows nodes. The canonical use case would be using Windows integrated security to access a remote database server rather than being forced to use a named user account and password stored in a configuration file.


ja siehe commit msg
durch rundungsfehler kann hier matherrror entstehen. war damals ne recht schnelle lsg. wenn wir distance noch nutzten sollte er drin bleiben oder error handling fuer matherror an alle stellen.


My bad. I updated my assignment on Bloc to reflect the correct commits.


This is pretty much WIP, so here are tons of println which will be converted into tests. However, the last println already outputs:

~~~
Success(List( 123456          987654         ,  ab     cd    ))
~~~


![gsa agile delivery services_mockup v1-01](https://cloud.githubusercontent.com/assets/13055959/8502850/fb68f484-2183-11e5-9f75-a0e83904c3c1.jpg)


Search UI Screen


![gsa agile delivery services_mockup page 2-02](https://cloud.githubusercontent.com/assets/13055959/8502856/111825ca-2184-11e5-8461-81b9c16a63a9.jpg)


PR #1849 fixes the issue, but doesn't have a test case


Since these are object keys now, they don't need quote marks—we could just do `loadJSON: 'p5'` instead of `'loadJSON': 'p5'`


Typo: uppercase L.


Might be valuable to make these function arguments more descriptive: `functionName` and `contextObj`, possibly?


I also missed some of the context for this but I'm not sure I understand why we need to pass the names of the context object vs passing the object itself


Solved, now it only creates 1 instance of each factory ever


The behavior is the same


@cpsievert We could also add tooltips to the legends


listify image


If I do something like this then it works, but it looks ugly :disappointed:
```
- (void)didUpdateCall
{
    [super didUpdateCall];

    if ((self.call.status == OCTCallStatusActive) && (! self.pushed)) {
        self.pushed = YES;

        dispatch_async(dispatch_get_main_queue(), ^{
            [self pushToActiveCallController];
        });
    }
}
```


Yes you're right, looks like I forgot to update the comment when I amended the commit to include `setuid` and `sticky`. The reason I put the comment there was because it's referring to the condition of the `else if` and that was the closes I could put it without placing it on a subsequent line or inside the preceding `if` statement. It's probably best if I just remove that comment.


I guess I should make that function take a userid anyway


Hi @ataravati

To answer your question, no the code is the same for every platform and every Browser.
If you have an issue as above, please open a Issue Ticket, this is the wrong location for the Issues.

Additionally add log files, console output, environment information your testing in such as browser, without that helping you will be simply put impossible

Mat


Yes, there should.


Addressing pootlepress/page-builder-for-canvas#303


Good work.
For help understanding those lines, it's probably easiest to ask in a bitcoin IRC channel.

I have a utxo of value 800 satoshi which confirmed fine when it was made.

An option in the cfg file won't necessarily solve the problem as most people won't change it, and it will add mental effort to them because it's one more thing they have to understand. Notice how it's not configurable in electrum.

Maybe there should be another configurable variable called like *preferred minimum output* where the bots try their best to not create outputs smaller than it, but will if there's no other option. The name is less confusing than *dust* too.


maybe strerror() for error msg ?


Typo: Persistence (assuming you mean that)


Hi Matthias - you edited this but it didn't get merged back in. I don't get any notification about your edit until you do a "PULL REQUEST" so please could you do that? It enables me to just click and approve.

(Actually I'll give you direct edit permission too, but in the mean time could you please do the PR for this edit)


Not because of notifications, but kinda agree with syphurith here: I don't think it's good practice to push buggy/untested changes to master, even less if it's such a huge changeset that breaks other stuff (CakeHax/MSET, for example in this case). Rather do it in a wip branch or something, keep it local, or do it in the pull requests, until tested/finished.
I believe master should always work, but that may be just me.


you removed the 2.0.x constant definition - but it is only defined in master langs.


Hi Dorien - you edited this but it didn't get merged back in. I don't get any notification about your edit until you do a "PULL REQUEST" so please could you do that? It enables me to just click and approve.


Wrong doc.


Hi Dave - you edited this but it didn't get merged back in. I don't get any notification about your edit until you do a "PULL REQUEST" so please could you do that? It enables me to just click and approve.


Can't we have a mainpage that isn't sol-flow.h?
Since we could disable flow build, this .h would be used anyway by doxygen?


@temasek Hi. Why do you revert these commits for the rotation tile icon? After all, this icon is still unthemed. Thanks.


You are checking string "encoding" here. However, as far as reading source code, a feature you added is "multipbyte", isn't it?  So "multibyte" should be checked here.


pad16(), pad20() and pad100() fixed.  b17f0ad


I have similar shadows issue as you when I disable accurate date. http://i.imgur.com/JMWcdnG.jpg

Otherwise it works. Could you send your ini file? Maybe there is another option that break somethings.


oops... will fix. sorry.


Remove this comment


This implementation still exposes the contents of myArrayList to external modification by the user, so I don't think it would be acceptable. But even if that is addressed, can you make a case why JSONArray needs this enhancement?


cosmetic


Changes to
  - source/js/dist/eu/search-object.js
will be overwritten the next time the top-level grunt task is run.  Changes should be made here:
  - source/js/dist/eu/search-object.js

I've commit changes that include this correction.


So this is the "linkat() with override" work-around, right?


Hey @tu1ly,

Just to let you know that I've just pushed a new refactored version of the plugin that allows for more options to be passed through the `_config.yml` file (including the `record_css_selector` that specify which tags should be indexed).


Maybe O_CLOEXEC?


Maybe use mkstemp() to create the temp fd?


Awesome, will give it a try.


Looking OK, besides those things.


Hi Dan,

We did it on Dave Moffat's account, so I've cc'd him in to look after it. Have a good weekend!

Cheers,

Dave

------------------------------------------------------------------

David Ronan

Centre for Digital Music

Queen Mary University of London

School of Electronic Engineering and Computer Science

Mile End Road, London E1 4NS U.K.

Phone: +44 795 864 8937

Email: d.m.ronan@qmul.ac.uk

________________________________
From: danstowell <notifications@github.com>
Sent: 03 July 2015 18:18
To: davidmronan/c4dmpoole
Cc: David Michael Ronan
Subject: Re: [c4dmpoole] Update davidronan.json (3de5efd)

Hi Dave - you edited this but it didn't get merged back in. I don't get any notification about your edit until you do a "PULL REQUEST" so please could you do that? It enables me to just click and approve.

-
Reply to this email directly or view it on GitHub<https://github.com/davidmronan/c4dmpoole/commit/3de5efd09f665521a6efe6919ee4a92a294fc1e8#commitcomment-11998029>.


The first letter in a String must begin with uppercase


@stleary If I'm remembering correctly, this should make a copy of the ArrayList, not a reference. It's been a while since I've worked with Java so I may be wrong.

To answer your question though, for simplicity.


It's not really what you're doing for negative values.  It's rounding away from zero, so it's actually rounding down for fVal < 0. This is the same as what the round() from math.h does

```C
#include <math.h>
#include <stdio.h>
#include <stdlib.h>

static inline int round_half(const double fVal) {
	return fVal >= 0 ? (int)(fVal + 0.5) : (int)(fVal - 0.5);
}

int main(int argc, const char *argv[])
{
	printf("value:      %.2f\n", atof(argv[1]));
	printf("round_half: %d\n", (int)round_half(atof(argv[1])));
	printf("round:      %d\n", (int)round(atof(argv[1])));
	printf("ceil:       %d\n", (int)ceil(atof(argv[1])));

	return 0;
}
```


You specify "." here. So The ALL files in this directory are parsed.
If you specify -R, could make a directory in which input.js and input.java are?
Or if you can remove -R, specify the two files as arguments like: ${CTAGS} .... --encoding-javascript=euc-jp input.js input.java


Okay


Sarcastic git commit message?


I'm doing my best to get the GUI ready quickly. This state will end very soon. Please be patient


@waymondo I use [helm-unicode](https://github.com/shosti/helm-unicode) to insert emojis.  There are probably better input methods, but it's fine for occasional uses of emoji.


You also need to change the first check to

```
| cabalLibVersion >= Version [1,23,0] [] = flags_latest
```


Perhaps an array list of cellTypes?


Since the vendor is the same for the whole repository, this would make a lot more sense as a per-repository setting. We don't have pre-repository settings (yet), but a global setting would work just as well.


`strerror()` should never be used in Soletta. There's a wrapper for that that's always threadsafe, regardless of implementation details.


How is the schema for that going to be? Will it allow duplicated entries?


Use `</>` maybe?


commont tutorials from Ros repositorites contains actionlib_tutorials


Released as v1.0.4


Yeah I am thinking on having on the user profile a list of links and then

```
{% for title, link in content.created_by.links %}
    {{render_link(link)}}
{% endfor %}
```


See, for example, https://docs.oracle.com/javase/8/docs/api/java/util/ArrayList.html#ArrayList-java.util.Collection-
"Constructs a list containing the elements of the specified collection, in the order they are returned by the collection's iterator."

In other words, the new list contains references to the exact same objects as the old list, which exposes them to modification by the user.

Simplicity by itself is not enough to justify a change, I think.


@enzolutions:
I added the `skipCommands` on the next commit and is already fixed this is the code of the listener:
https://github.com/hechoendrupal/DrupalConsole/blob/master/src/EventSubscriber/DefaultValueEventListener.php

Related to the `default` I was meaning to add a new global option as de `generate-chain`


yes good call on more descriptive names :)
i forget why (or if) it needs to be the name, i'll play around and see if this could be simplified by passing the object instead.


yess thanks!


@devemux86 an idea how we could unit test something like this :) ? Just in-memory creation of a largish graph?


да, наверна более развернуто надо писать - там просто отфильтровываются щас те реализации, для которых тип вывести не удалось. Надо для отфильтрованных писать причину в комменте в дереве


you made a typo here, informations should just be information


Thanks. I think I understand.


all corrections from @DokuKaefer


This is what I meant in my previous comment.


This should prevent exposure. If not, I've been away from Java for far too long.


`</>` has some weird behaviour ([link](https://hackage.haskell.org/package/filepath-1.4.0.0/docs/System-FilePath-Posix.html#v:combine)) and [can be very dangerous](https://github.com/scotty-web/scotty/issues/63), so I tend to avoid it.


doing this, it's possible for a request to have __NO__ method, which would be bad.


@he7d3r thanks, fixed in d74d946.


Dear Josh,
                         I could not receive any comment on the following changes. Please do have a look and give some feedback aslo..

Cheers,
Sarmad Masood Shaheen


It would be the preferred option, as it's portable; however, `linkat()` won't work, as it's a hard link and it doesn't work across filesystems (`/tmp` is often a `tmpfs`, `/home` is often on a separate partition, etc).


Please open a new issue and reference the commit. Otherwise this will get lost in space.


> 1) Some URIs require the identifier, so it should be exposed to the module user

Ahhh, good catch! :+1:


Hey, I put the local styles last in  the array in PR https://github.com/FormidableLabs/spectacle/pull/2

This commit seems to have regressed back to the old behaviour. Is this intentional?

If `styles` is first then if you try to set display on the `Image` like
`<Image src={images.github.replace('/','')} margin="0px 5px" height="35px" display="inline" />`
the display gets overridden by the default styles.

Should I submit another PR to switch them around?


I also like this approach by Jean Emer http://jcemer.com/construtores-em-javascript.html


should be
PUB_RATE
or
@@pub_rate


If there is a `data-original` it just returns that, it doesn't actually open a new window. I think you want.

```javascript
return window.open((e.target.getAttribute('data-original') || e.target.src), '_blank')
````


Restoring string value


Die Commit msg. fand ich nicht sehr erhellend. Du meinst einen Domain Error? Dann sollte das IMHO
``` Python
try:
    ...
except ValueError:
  return 0
```
sein. Solange du das nicht über die Maschinengenauigkeit abschätzt, kann das round hier auch nicht garantieren, dass das gut geht.

In der derzeitigen Form würde ich das jedenfalls nicht für den Upstream vorschlagen.


Not sure I understand.


good to hear about.. greping I found sol_util_strerror()...  thanks


Please be more explicit e.g. using a condition like `typeof i18n === 'object'`.


We need attr_reader :group here, and ...


... and:
```
extend Forwardable
def_delegators :"@manager.group", :each
```


this is `manager` not `group` ... same for the instance variables


`each` alone, called on nothing, will trigger the delegater to `@manager.group.each`


@dilumnavanjana answered all roadblocks above


We could provide a textfile in the ZIP with all dpi information of all available packages.


url --> URL


why not forward-declare this in the header and include it directly here?


If the second argument to `</>` is absolute, the result is just the second argument. This was the cause of a security issue in Scotty where you could get `/etc/passwd` by doing `GET //etc/passwd`, for example.

The docs for `</>` make it seem like it might have other gotchas too, but I can't understand them (it looks like they've fallen foul of Haddock markup).

Admittedly the absolute path issue probably isn't a problem here, because Yesod is much better at validating user input, but I still think `</>` opens a huge can of worms for no real gain.


same here


Very good


and here.


Yes


Autor: Marcio. Co-autor: Daniel


test


Autor: Marcio. Co-autor: Daniel.


Hey @krisl This is great. I don't know how I could make this mistake. Can you send me a pull request so I merge it ?

Regards!


8f2f556 fixes I saw in dyna-29. It worked from the start, but I was seeing only 1 test.  I found out why only one test, and fixed that in dynamic_fmt.c


is application/json correct here? My open PR changes rest's server side default to send application/json instead of text/json, and changes the client to check for both


This one could be safely removed


... the client: this line


84691154_AutomateBuildingOfDocs (lefticus) - x86_64-MacOS-10.9-clang: OK (1676 of 1676 tests passed)

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


why isn't that done via `requirements-dev.txt`? because it shall only be forced to use tox in the test-container?


thank you


why is this listed explicitly? as it is a dependency of flake8.


[+] Add read me file


Ah good point, and that's easy to do.
On Jul 3, 2015 9:32 AM, "Luffier" <notifications@github.com> wrote:

> We could keep the folder structure and make an exeption in the installer
> (as in "overwrite files in ART when selected language is different than
> English").
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Sduibek/fixtlang/commit/18fd7cc802c337ec1a5de9cbdef315390bfc5009#commitcomment-11997641>
> .
>


Thank you for your very fast reply.
Now there are two sorting orders.
Maybe it is worth a try? I would vote for it ;)


I am confused. Isn't the method set to `RequestMethods.GET` here?
https://github.com/angular/angular/blob/master/modules/angular2/src/http/base_request_options.ts#L85


so I suppose for consistency we should change all the settings' pages to display a text current setting ...


Hmm, they must have gotten overwritten somewhere along the way. Another PR would be great, thanks!


The encryptor more or less strictly follows the JWE encryption: tools.ietf.org/html/rfc7516. However, there are still many things that need to be discussed on this. Some points:

1. Content Encryption Key, CEK is encrypted by the algorithm given by ```'alg'``` parameter of the header. However, there is a direct encryption we can do according like this http://tools.ietf.org/html/rfc7518#section-4.5. I have done it that way, i.e., keeping the ```'alg'``` value to ```'dir'``` always.
     This is because the 'alg' algorithms suggested here http://tools.ietf.org/html/rfc7518#section-4.1 are not implemented in ```OpenSSL``` yet.
     Yet another thing, according to the spec, CEK (Content Encryption Key) should be 'empty octet sequence', which according to my knowledge is just an empty string ```""```. But the cipher key can't be this short, so I took it  ```'\0' * 32```.

The question is, what is the acceptable level of implementation of JWE in ```MessageEncryptor``` here ?


@adrianovalente tenta refatorar usando `while` e `break`;


@ryandemmer , unfortunately, there's not much we can do about it. It's coming from the official source at http://curl.haxx.se/ca/cacert.pem

That being said, we had had a similar issue on one of our sites. What we did was added an intermediate certificate in the Nginx configuration on our server and that resolved the issue. With the intermediate curl could verify it and it eliminated the update errors.

Try checking for intermediate certificates at https://www.geocerts.com/ssl_checker


The `requirements-dev.txt` is used to install dependencies into the virtualenv created by tox. We don't want to install tox into that virtualenv that tox already created.


@adrianovalente arruma a identação! rs


@TedEwanchyna I would add an id to the button; there are many material-icons that could trigger the clicked event


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

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


Yes, that's better idea


Published


please place a space between "if" and opening bracket.


same here.


Thanks!


actually I got it.


`flake8` and `pyinstaller` aren't needed there neither, right? do make lines 82f in this file sense at all?


missing space...


@oliviertassinari I had to revert the eslint changes for modernizr.custom.js because it broke the docs site.

I think we should add modernizr.custom.js to .eslintignore, but I couldn't get it to work. Maybe you'll have better luck?


factory*


:cool:


it's overridable


Awesome. Thanks.


Entropizer.js looks great, but I don't really have an idea how to rate the password based on the entropy and character classes.
Concerning brunch we maybe could do sth like
```coffee
    javascripts:
      joinTo:
        'js/app.js': /^app\/js/
        'js/compiledList.js': /^bower_components/list.js/
        'js/compiledZxcvbn.js': /^bower_components/zxcvbn/
...
```
?
Alternavely we could add a line to the init script.


The problem seems to exist only with "session" based authentication. OAuth and Token work fine.
The AuthServiceProvider doesn't set a token when using "session" based authentication, therefore  hasValidToken never returns true.

Sry for the inconvenience. Will have a look at the problem. The fix should be fairly easy.


03/07/2015 15:13
Sistema / Aplicativo:
Ambiente: DSI
Equipamento: Desktop
Problema Reportado:Usuário reporta que
Problema Identificado:
Testes Realizados:
Print de Erro: (S/N)Sim Não
Estação Reinicializada: (S/N) Sim
Problema ocorre em outro Desktop: (S/N) Não Sim
HOSTNAME: LACBRA
Telefone:
Localidade:
CTI LA 2T BR OFFICES & BRANCHES SUPPORT
CTI LA ON BR DIST SERV


Note sure I grok the logic here. If UseHostname=yes is set and Hostname= is set as well, then I think the server supplied hostname should be ignored, and the hostname specified in Hostname= should be unconditionally set as transient hostname, as soon as a dhcp lease acquired.


Fixes #35


-- Telefonia

03/07/2015 15:14
Sistema / Aplicativo:
Ambiente: DSI
Equipamento: Desktop
Problema Reportado:Usuário reporta que
Problema Identificado:
Testes Realizados:
Print de Erro: (S/N)Sim Não
Estação Reinicializada: (S/N) Sim
Problema ocorre em outro Desktop: (S/N) Não Sim
HOSTNAME: LACBRA
Telefone:
Localidade:
CTI LA 2T BR OFFICES & BRANCHES SUPPORT
CTI LA ON BR DIST SERV


There's `sol_util_strerrora()` that allocates enough space in the stack for the error message to be filled in by ` sol_util_strerror()`. For error messages, it's the one to use.


This is not OK. The function should not alter *hostname if the host name passed is not valid. hence, please parse this into a temporary variable first, Only if the parsing works fine and the name specified qualifies override *hostname.


-- ISA

Login:
03/07/2015 15:15
Usuário VIP: (Sim/Não)
Sistema:
Problema Ocorrido: Usuário reporta que
Testes Realizados:
Problema Ocorre em outro Desktop:
Numero da solicitação (WS/VSR/BRM/VR):
Nome Completo:
SOEID:
Funcional:
Login da aplicação:
Login espelho:
Print do Questionário da solicitação anexo (SIM/NÃO):
Print do Erro anexo (SIM/NÃO):
Área do Usuário:
Telefone:
Localidade:
COR GL 1T GIDA-FC-NONCSC


There's some code missing that frees the hostname when the Network object is freed.


-- ISA

Login:
03/07/2015 15:16
VIP User: (Yes/No)
System:
Problem Occurred: Usuário reporta que
Tests done:
Does the problem occur in other desktop: Yes No
Request number (WS):
Full name:
SOEID:
Functional:
App login:
Mirror login:
It was taken a printscreen of the requested questionaire? Yes No
Attached Printscreen error: Yes No
User department:
Phone:
Local
It was used any KB for this INC?:
COR GL 1T GIDA-FC-NONCSC


Readme.md should contain requirements and installation steps. It helps other developers easily setup dev environment.


Here is the backtrace on i386:

```
Program received signal SIGSEGV, Segmentation fault.
0x0c9c4dd5 in strlen (str=0x7b3aab8c '\337' <repeats 200 times>...)
at /usr/src/lib/libc/string/strlen.c:39

39              for (s = str; *s; ++s)
(gdb) where
#0  0x0c9c4dd5 in strlen (str=0x7b3aab8c '\337' <repeats 200 times>...)
    at /usr/src/lib/libc/string/strlen.c:39
#1  0x08571b1e in g_strdup () from /usr/local/lib/libglib-2.0.so.4200.1
#2  0x1a9b2d5c in mu_msg_iter_get_msgid (iter=0x7bc0cb80) at mu-msg-iter.cc:371
#3  0x1a9bd5df in get_thread_ids (iter=0x7bc0cb80, orig_set=0xcf7c09a8)
    at mu-query.cc:432
#4  0x1a9bd799 in get_related_query (iter=0x7bc0cb80, orig_set=0xcf7c09a8)
    at mu-query.cc:455
#5  0x1a9bd992 in include_related (self=0x849fc040, iter=0xcf7c0a20,
    maxnum=800, sortfieldid=16 '\020', flags=29) at mu-query.cc:484
#6  0x1a9bec0e in mu_query_run (self=0x849fc040,
    searchexpr=0x871a5100 "date:today..now", sortfieldid=16 '\020',
    maxnum=800, flags=29, err=0xcf7c0bc8) at mu-query.cc:552
#7  0x1a9a862f in cmd_find (ctx=0xcf7c0bc0, args=0x83c268c8, err=0xcf7c0bc8)
    at mu-cmd-server.c:945
#8  0x1a9a99ca in handle_args (ctx=0xcf7c0bc0, args=0x83c268c8, err=0xcf7c0bc8)
    at mu-cmd-server.c:1573
#9  0x1a9a9c1d in mu_cmd_server (store=0x8561d480,
    opts=0x3a9a9420 <MU_CONFIG>, err=0xcf7c0c94) at mu-cmd-server.c:1622
#10 0x1a9ab836 in with_store (func=0x1a9a9a8c <mu_cmd_server>,
    opts=0x3a9a9420 <MU_CONFIG>, read_only=0, err=0xcf7c0c94) at mu-cmd.c:547
#11 0x1a9abb66 in mu_cmd_execute (opts=0x3a9a9420 <MU_CONFIG>, err=0xcf7c0c94)
    at mu-cmd.c:618
#12 0x1a9a0e8d in main (argc=1, argv=0xcf7c0d04) at mu.cc:122
```

Segfault confirmed on amd64 as well but I don't have a backtrace
there.  msgid points at garbage by the time g_strdup() is called.

I think the issue comes down to: the return value from c_str() cannot
be guaranteed valid after the current statement.  This is why
collapsing it into one statement makes a difference.  Maybe it doesn't
happen on all OSes or all architectures but it is repeatable under
OpenBSD-current on i386 and amd64.  My proposed patch fixes the
segfault there.

Note that it only seems to happen with `mu4e-headers-include-related`
set to `t`.


Yes, it is


--DU

Login:
03/07/2015 15:17
Nome da Aplicação:
Descrição do problema: Usuário reporta que
Função e/ou operação indisponível:
Descrever o Impacto ao Negócio: Financeiro Operacional
Impacto Financeiro:
Horário/Data limite para realizar a operação:
Contingência:
Área impactada:
Número de usuários impactados:
Anexar evidência (SCREENSHOT) do erro: Sim Não
Telefone:
Localidade:


memory handling is borked. if you use this like this, then the hostname set in link->network->hostame will be freed by this call, because of the _cleanup_free on the *hostname option. Please intrdouce another pointer "const char *hn" or so, that you either set to link->network->hostname, or to the string you allocate into "hostname". Then use "hn" for the further calls.


-- ATM

Login:
03/07/2015 15:18
Código da ATM (TAG):
Descrição do Problema: Usuário reporta que ATM de sua localidade
Agência / Filial: Agência  /
Localdiade:
Telefone:
Telefone Alternativo:
KB0000001232 - ATM/Problemas Físicos
KB0000001233 - ATM/Problemas Sistêmicos


Yeah, that's fine. I just wanted to see if it was something that was overlooked by accident.

Cheers!


@Exevalon better? The updated version should also be live on haxeflixel.com.


@FroMage this change breaks `testNativeSetterInvalid`, could it be the result of this method subtly different from what it returned before?


The reverse test case is already there. I will add a comment.


-- WINDOWS/ CITRIX LEGADO

Login:
03/07/2015 15:19
Sistema/Aplicativo:
Problema Reportado: Usuário reporta que
Problema Identificado(Detalhado):
Ambiente é Citrix DSI ou Citrix Legado: (DSI / Legado)
Testes Realizados:
Print de Erro(S/N):
Feito logoff da Sessão do Servidor e reiniciado estação (S/N):
Foi identificada e encerrada sessão presa no Citrix : (S/N)
Envolve Impressora:
Servidor Afetado:LACBRA
Falha ocorre com outros usuários: (S/N)
Telefone:
Localidade:
CTI LA EX BR APPLICATIONS-SUPPORT BRAZIL
CTI LA EX BR SERVERS W&S


-- REGIONAL

Login:
07/03/2015 15:19
Impacted applications:
Problem Description: User reports
Request number(if applicable):
Tests Performed:
SSO applications? Yes/No
Account Number / identification:
Impacted environment: production/UAT/SIT
Error message: (attach print screen)
Application type: Web / Desktop
URL (if applicable):
Problem scope: unavailability / slowness / intermittence
Type of impact: financial / image / operational
Number (or percentage) of impacted internal users:
Number (or percentage) of impacted external customers:
Contingency: Yes/No
Phone:
Locality:
CTI LATAM 2T RSS CTI


Does this work with file extensions that aren't 2 chars?


Scripts should be moved to separate file. I'll show you this in future


I don't think `published` should be changeable from the client (if that's what this line is doing).


flake8 would be. I think we need line 82 to build the binaries after the tests run


--MESSAGE

Login:
03/07/2015 15:20
Descrição do Problema: Usuário reporta que
Testes Realizados:
Acessa Webmail(SIM / NÃO): SIM
Quando começou a ocorrer o problema:
Outros usuários estão com o mesmo problema:
Em outra máquina ocorre o mesmo problema(SIM/NÃO):
Usuário está 'active' no Rits e com a senha desbloqueada (SIM/NÃO): SIM
Conta do usuário esta como 'Visible' no User Exchange Lookup (SIM/NÃO): SIM
Qual o sistema operacional: Windows - DSI
Print do erro em anexo (SIM/NÃO):
Foi Anexado NDR e Mensagem Enviada (S/N): Não
Telefone:
Localidade:
CTI LA EX BR MESSAGING
CTI LA 2T BR OFFICES & BRANCHES SUPPORT
KB0000002643 - Recuperação de Emails
KB0000004797 - Problemas no ambiente de Email


Excellent, thank you Gama.


re-adding line 726 ( SafeSymCleanup(fdProcessInfo->hProcess); )
seems to solve issue #323 completely,
tried that after rebuilding with the latest commit 0c8b6c8


--MOBILE

Login:
03/07/2015 15:21
Descrição do Problema: Usuário reporta que
Nome:
SOEID:
Ramal:
Localidade:
Departamento:
Número SIMCARD(se possível):
IMEI: Usuário não soube informar.
PIN: Usuário não soube informar.
Número da linha:
Usuário teve algum outro número de linha?:
Modelo do aparelho: Usuário não soube informar.
Tipo de SINAL (GSM/EDGE/3G): 3G
Tempo que usuário possui Blackberry:
CTI LA EX BR MOBILEWIRELESS


what about a `requirements-test.txt` and a `requirements-build.txt` for the sake of clarity?!


The issue is accurate_blend = 2. This option is mostly a debug option (very big speed hit) and badly interact with others accurate stuff.


Styles will be added in the future, remove now.


Amazing, thank you, and now I know where the repo is to contribute myself :wink:


Ah, I just came across this as well in an older version. Thanks for fixing this!


Lost...


Actually d1sc0tech got it from my but no matter, I see you extended it quite a bit.


Thanks for that, @ploquel !


There is no db dump in vcs, thus there is no way to setup project.


you forgot to add doc to warning-targets on Makefile.vars and it's also missing a PHONY += doc


yes I began to realise this and adding an id seems the right thing to do.
How should we go about this?
Either: reject my PR and add the id that you want
or: accept my PR and then make your change
I am ok either way.


Are you having an issue? If so, what are you loading in place of .js into the browser?


+2

Just a question, have you ran uncrustify in the repository (over all commits) right ?


ack


I was just looking through the changes and wondering how that code would work with non 2 char file extensions.  Wouldn't this same code be used to load .html? I probably should've kept my wondering to myself and left you guys alone, but I didn't. I am guessing you know what you are doing. :-)


Yeah true
On 3 Jul 2015 20:02, "garfield69" <notifications@github.com> wrote:

> so I suppose for consistency we should change all the settings' pages to
> display a text current setting ...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/SchizoDuckie/DuckieTV/commit/ac8ca71597a5e2c4caeadcf38e8d5d2b0849909d#commitcomment-11998321>
> .
>


@ilya-g maybe `@file:suppress("NOTHING_TO_INLINE")`?


By moving the src files are we making it harder to merge upstream changes?


The new version of this uses the new version of SE-Chatbot. KarmaBot can now be found at https://github.com/ArtOfCode-/KarmaBot.


Un commit fixe ça, surement un oubli. https://github.com/minetest/minetest_game/commit/016fa0da40d152d32d50a52b80b71b0e4e2f9d27


I update the code, so it will disable automatically sw blending when using accurate date.


With this final addition I believe we have fixed #12


Could you compile this into plugin please? Downloaded and it's still not in options =(


Oh yes, you are right. I enabled it for this game because it gave kind of accurate lightning (similar to the software mode) on the field. Also it doesn't result in broken shadows in earlier versions. Only started happening from this version.

By the way my fps almost doubled after setting accurate_blend = 1 :)


hell yeah! now it looks way much saner... and makes more sense to have the sol_flow_default_resolver customization.
+1


It shouldn't matter anything, especially since that one was never called when detaching. But sure, I can add it


Let the default capacity be 1. As mentioned by C++ standard, there is no major policy for the default capacity. But for some implementations, the default vector size is 0. We may keep that size.


yeah but effort


I believe this also fixes #14


Never assume I know what I'm doing :laughing: This particular code only affect the loading of plugin modules. It's specifically related to JS.


These are some basic examples of how semantic tags are used in HTML5


Boolean variable should follow the "b" prefix.
And "private" variable should follow the "_" suffix.
Thus, it is better to change "user_destroy" to "bUserDestroy_"


We can keep this dummy clean method.


OK thanks, although in general it would be good if Isengard "followed" master otherwise users on "master" builds may suddenly find themselves downloading Isengard PVR addons which is what happened to OpenELEC bleeding edge testers last night - their latest Milhouse test build shipped with master pvr.demo 1.10.5 last night, while the OpenELEC online repo bumped to Isengard pvr.demo 1.10.6 (and the same for all other pvr.* addons). Now I need these master testers to remove the downloaded Isengard addons so they revert to using the shipped "master" addons... In theory this should never happen.


No prob! I just realized yesterday that I DO have push access on rubygems, so I'll do this now :)


So far we used separate AF_UNIX sockets for the various bits send over. I am pretty sure we should stick to this scheme in this case too. Especially, since the UID shift parameter only makes sense if --private-users is specified.


Should use path_equal() rather than streq().


Where do I put it? Mainloop header?


Ja vermutliche. Ich weiss nicht wieso ich das nicht so gemacht habe. Gab glaub ich nen Grund. Stimmt deine 0?


The comment `The options object will be merged with the values of {@link BaseRequestOptions} before performing the request.` is pretty misleading then.


Thanks a lot, both fixed.


It's may a little annoying. But for 32bit integer variable, the name should follow the "i" prefix.
And we may reduce it to:
```
iCap = (iCap <= 0)? DEFAULT_CAPACITY : iCap.
```


@porcelli  please review.


previous code was a bit broken. And often 2 brokens effects result in a correct behavior :)

Yeah, I noticed the lighting effect. I didn't start to debug it, and I'm glad I didn't do it :) Unfortunately those kind of effects aren't possible to emulate properly yet. I hope future hardware will allow us to do it. It will allow to run all those accurate options without any (from CPU PoV) speed impact.


1) parameters take priority in **private**
2) parameters = _position, _size


The CSS files are being built with compass (which is run locally on my machine) when I edit the SCSS files. Let me know if there is a better way to do this.


Make the trigger **elliptical**


Doesn't matter, fixed it.


Thus it is ``` pData->aItem_ = (Item*)malloc(sizeof(Item) * iCap);  ```


It's accurate, but `BaseRequestOptions` can be bound to any object for customization


oh, right! But it does still fail to parse the frame (that it wrote itself)


Should be:
```
pData->iCapacity_ = iCap;
pData->pDestroy_ = NULL;
pData->bUserDestroy = false;
```


"SafeZone" -> "sos_safe_zone1"


The spec mode scoreboard it`s also bugged, the Blue team is moved.


How do you wrote it?


Unclear commit message


I plan to test the capacity for auto extension.
So it is better to keep the default capacity.
```
CU_ASSERT(VectorInit(&pVec, DEFAULT_CAPACITY) == SUCC);
```


Could you help to split the modification for binary heap to other patch?
Than the code branch line would be cleaner.


Could you please apply it to all instances?
```
$ git grep -n curl
docs/USAGE-GUIDE.md:22:curl http://ftp.openbsd.org/pub/OpenBSD/LibreSSL/libressl-x.y.z.tar.gz > libressl-x.y.z.tar.gz
docs/USAGE-GUIDE.md:83:curl https://www.openssl.org/source/openssl-1.0.2.tar.gz > openssl-1.0.2.tar.gz
```


Could you help to change the naming for "user_destroy".
Thanks!


You gotta run `bundle install` so that the changes are written to `Gemfile.lock` too


TODO: Collect some practice data and create some dashboard graphs with Chartkick / GeoIPData


The method `rotatePositivelyBy90Degrees` is unused. Either delete it, or use it to rotate `vec` and then assign the resulting variables to `triangleVec`  (this would improve readability)


Since I plan to demo the capability for auto extension,
It can be set to "2" or "1" to trigger the behavior.


https://rubygems.org/gems/vagrant-cachier/versions/1.2.1 :+1:


It seems that the only real reason we had to remove this was the segfault on windows.  I would prefer to see that fixed than have the confirmation removed.


is it intendet, that you removed the partialContentMain div?


Can anybody please explain this fix? Thank you


I made some preliminary review.
BTW, it is would be better to keep the policy: One patch for only one structure or configuration.


Dan, is this debugging code for me or for you?


When you port tests from the current authentication mechanism to JASPIC and there are tests that require features you haven't yet ported (e.g. change session ID) then still port the test just mark it with `@Ignore`. This acts as another marker that there is still some work to do.


Queries created this way are not safe to use. This is just a dummy example.


I was looking for consistency here with what is done for the header length in ipv4, but since this and the ipv4 hlen are auto-calculated when properly preparing the packet for transmit, perhaps the answer is to remove the default from the hlen field ipv4 instead of adding a default here.


Nice looks good. So either registry server can be used?


Nice :). We can drop this code from the non-JASPIC FORM auth too.


c


It looks like we could always add the remote_bonus...


oh sry again :+1:


@hai-cea that's weird. It shouldn't have broke the doc site. I'm gonna look at it.


Missing `http://`


привет!
В принципе - хорошо. Но:
задача 15 - можно решить без циклов; кроме того, для числа 98360 она пишет какое-то странное время :)
The current time is 27 hour 19 minutes 20 seconds

задача 16 - можно решить без циклов и условий;

задача 18 - там два задания

А так норм, хорошо


ok, i'll try compiling with and without the SafeSymCleanup() call again
and do more trials to make sure it wasn't just pure luck.
But do i need to rebuild the GUI every time ?


Привет. Тут всё ок.
Маленькое замечание:
`System.out.println(String.format("Miles in total: "+"%8.2f", m));`
тут можно не делать сложение - шаблон надо писать в одной строке
`System.out.println(String.format("Miles in total: %8.2f", m));`


_size = if (count _this > 1) then {_this select 1} else {100.0};


Can you add a Signed-off-by line to the commit message please ?


looks good


It looks very good, but h3.jpg is 10.9 MB. Can you reduce that?


Right. Seems like you're all over this anyway. Not sure what you envision here. Will focus on something else.


@shinnn, yep, done!


@NPadrutt I'd like your opinion on this file. My goal is to make an interface for the connection to the device. I guess what I'm confused on is what stuff goes in the interface and what just goes in the regular model class. I added everything to the interface because I wasn't sure.


Ок, тут всё хорошо, молодец

ТОлько имена переменных меня иногда смущают, но то такое ))


Now you can meld two commits in one with
git rebase -i HEAD~2


<font> is deprecated tag. Should be removed. http://www.w3schools.com/tags/tag_font.asp


I think it would be safer here to declare a `char *shortrefname = NULL` rather than `int is_short = 0`.


background attribute should be removed. It should be styled with CSS


Ping @meadsteve

What do you think about this for a default? I found that once I made this rule only apply to `.ex` files it did more or less exactly what I wanted it to- i.e. not force me to moduledoc my test suite that's already full of descriptive information and dummy modules.

This could be something we do for other rules, and possibly allow to be configured for any rule.


no, this was an error - just add the missing Index Search TAGs - sorry


Привет.

В принципе - всё хорошо, молодец.
Но что это за странные условия в 18 и 19:
` if (x/100>0 && x/100<10){`


Here you could just do:

```
shortrefname = shorten_unambiguous_ref(refname, warn_ambiguous_refs);
```


It's better your code is perfect of course. We(as far as I know) rarely need the candidate to handle the corner cases with exceptions, though...

One more thing worth mentioning is that my code was challenged by my interviewer because the code doesn't compile in the computer..... So, you should have got it....


bgcolor attribute should be removed. It should be styled with CSS


"Updated a few thing to fix compiling errors."


задача 15 так оно так и должно перевести, 98360 секунд это 27 часов 19 минут и 20 секунд :) вроде

задача 18 ага увидел :) делал в 1 ночи, ничего удивительного :)


@cflesner: This will mean dropping support for anything older than iOS 8. We can't do that yet.


wrote this tiny example:
https://github.com/diasdavid/spdy-cross-test/blob/master/node/server.js#L30-L32

And captured with:
https://github.com/diasdavid/spdy-cross-test/blob/master/snif/snif.js

(this is how I've been capturing the other frames)


задача 15 опять же 2 условие я пропустил :( но в целом же правильно считает?


Also, better Windows 10 Cleanup for nVidia.


I generally try and avoid constructs along the lines of
```java
if (!x) {
    foo();
} else {
    bar();
}
```
in favour of:
```java
if (x) {
    bar();
} else {
    foo();
}
```
In most cases, the second form is easier to read.

The names login and nologin don't really help me understand what is going on. I actually have to look at the code to figure out which is which because it isn't clear if 'login' means displaying the login form or processing the provided credentials. Better names might be handleRedirectToLogin and handleLoginForm or something along those lines.


Here after the closing bracket, you could unconditonnaly `free(shortrefname)`.


> задача 15 опять же 2 условие я пропустил
Ты и не мог его не пропустить - я его только что добавил ))) Но в чатике мы это обсуждали.
Да, в целом - правильно


@armadsen good point. I'll fix that.


There is a decompression context. If there was another SYN_ or HEADERS frame before it - it won't be able to decompress it.


Привет.
Спасибо:)
Ну это чтобы определить является ли число трехзначным.
Если целая часть от деления на 100 в пределах от 1 до 10, то число
трехзначное.

пятница, 3 июля 2015 г. пользователь brus-fabrika написал:

> Привет.
>
> В принципе - всё хорошо, молодец.
> Но что это за странные условия в 18 и 19:
> if (x/100>0 && x/100<10){
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/OlgaGrygoryan/homework/commit/33ed489312b1a0171b9c1f3abd76502303a4b757#commitcomment-11998874>
> .
>


а если так:
`if(x > 99 && x < 1000)`
?


Will do. Thanks, Mat!


Ну можно и так:) я не ищу легких путей:)))

пятница, 3 июля 2015 г. пользователь brus-fabrika написал:

> а если так:
> if(x > 99 && x < 1000)
> ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/OlgaGrygoryan/homework/commit/33ed489312b1a0171b9c1f3abd76502303a4b757#commitcomment-11998917>
> .
>


This class is deprecated.


The same change shouldn't be applied here too?  I mean, using ```new InterpretedMutableProjection(streamedKeys, streamedPlan.output)``` instead of ```newMutableProjection(streamedKeys, streamedPlan.output)```?


I definitely see how it can be an issue, but haven't yet adjusted it. Sorry!

Curious though, would you prefer to


Discussion at https://groups.google.com/forum/#!topic/cryptopp-users/LUqkZuCeRX4.


Also, it would be cool to capture chromium's or spdycat frames instead. Because `framer-test` already verifies that it is capable for parsing frames that came out of framer.

The purpose of this test is to verify that the framing scheme is supported by external tools too ;)


On other hand, I can probably verify the frames manually when reviewing this PR. So maybe it is not worth the complications of sniffing the spdycat's traffic.


If you have questions about how to get started "on" developing for VXQuery "," take a look at our Developer


This should really be a message on Basecamp. Can you add it there and also in this rare case notify more people? I don't know all the answers to this one and it would be great to get more eyes on this :)


```javascript
textbox = this.$el.html()  // mi ritorna giustamente il template
```
Esatto, adesso qui dentro non c'e' altro che una stringa... quindi ci puoi fare ben poco.

```javascript
this.$el.html().find("#description").val();
```
Idem, probabilmente la find ti falliva dato che la chiami su una stringa.

```javascript
$(textbox).$("#description").val();
```
Questo proprio non lo capisco!

Non ho capito al 100% cosa vuoi fare (domani me lo spieghi) ma il codice che vedo nel commit mi pare giusto.
```javascript
this.$("#description").val();
```

**this** e' la tua view Backbone. Se guardi nella doc, vedrai che espone un metodo **$** che e' un handler per jQuery chiaramente, ma che lavora esclusivamente dentro la view (piu' tecnicamente, la view e' passata come _scope_ a jQuery). Quindi dentro la tua view cerchi un elemento con id description. Se esiste, **val()** ti ritorna il valore.


third parameter is actually a SCREEN_SIZE defined in draw.h


Bypass the lack of --wildcards in BusyBox's tar by avoiding wildcards and using the complete path.
Unfortunately I cannot test it now.


Do you think it can look better if you bullet these points?


Should it be of or for?


А дело не в лёгкости для ТЕБЯ, а в лёгкости для компьютера )))
зачем делать 2 лишних операции, тем более - таких довольно дорогих в плане производительности?


Finally I was able to check it.

It seems that I had same problem and raised issue in objcTox. You can temporary fix it by using `performSelector:withObject:afterDelay:` method.

The problem with pushing in a loop is different problem. Call just keeps updating and it stays active, so there is logic problem in Dialing controller. I think you can add dispatch_once as you wan't to push controller only once during Dialing controller lifecycle. It would be cleaner than using `pushed`.

So it will be something like that:

```
@property (assign, nonatomic) dispatch_once_t becameActiveToken;
...

- (void)didUpdateCall
{
    [super didUpdateCall];

    if (self.call.status == OCTCallStatusActive) {
        dispatch_once(&_becameActiveToken, ^{
            // workaround for deadlock in objcTox https://github.com/Antidote-for-Tox/objcTox/issues/51
            [self performSelector:@selector(pushToActiveCallController) withObject:nil afterDelay:0];
        });
    }
}
```


$this->getDoctrine()->getRepository('ShortenerBundle:Abbreviation')->findOneByuserId($userId) ??


Do you think it is a better idea to directly address the reader as you, since we know that a developer is already reading these guidelines?
>>"give the developer feedback"


when you say "I would add..." I assumed that you would add, now  I thinking you want me to do it?

So I will make the change then.


@ryandemmer As stated by @nicksavov, this file comes officially from cURL author, which generates it from official Mozilla supported certificates. http://curl.haxx.se/docs/caextract.html
Unfortunately, cURL author regenerates this CA Bundle in cycles of only every 6-7 months.

What we COULD do is :
1. regenerate our own fresh CA Cert bundle version from current Mozilla cert repo more regularly, maybe before any release. It's easy with the Perl script supplied by the same cURL author, "mk-ca-bundle" (But even that wouldn't help for JCE site)
2. add a General Configuration option to enable/disable strict SSL Cert checking (default: On)

I just checked what's wrong with that Thawte certificate on JCE website. The domain certificate is fine, the problem lies in used Thawte Intermediate Cert which is signed with SHA1 algorithm. Mozilla already ignores those certs using SHA1, while other browsers will follow this year. You can confirm that by browsing to JCE site with Firefox - notice the padlock doesn't get green at all. Clicking on padlock and inspecting chained intermediate certs will show they are SHA1-signed. JCE website owner should either exchange intermediate certs for those signed with SHA256, since his domain cert is signed by SHA256.

That eventually means even the freshly generated CA Cert bundle wouldn't help for JCE, it should be fixed by site owner.

Still, I suggest we proceed with coding point (2), and regenerating bundle now and then for point (1)


I believe we do need to end the `response` instead if the `gzipper` fails.


Also, I think you used "you" in the beginning and switched to developer here.


You could remove ${FILESDIR}/tmpfilesd then, too.


Ну если честно, то я не подумала, что все может быть так просто и
искала решение "позаковырестей":)

пятница, 3 июля 2015 г. пользователь brus-fabrika написал:

> А дело не в лёгкости для ТЕБЯ, а в лёгкости для компьютера )))
> зачем делать 2 лишних операции, тем более - таких довольно дорогих в плане
> производительности?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/OlgaGrygoryan/homework/commit/33ed489312b1a0171b9c1f3abd76502303a4b757#commitcomment-11998969>
> .
>


massive :) Thank You


done


I found it on stackoverflow, so fair enough.


Needs a test!


You are right. I've been capturing both go and node.js written frame to make the tests. Best scenario would be to have the de facto example frames given by the spec, that every impl should be able to parse.


s/argOutput/argsOutput/ I assume?


Same here I think: `${FILESDIR}/kdm.service` can be removed, or can it?


Can you add a menu for the encoding, too?


Заковыристей вернее:)

пятница, 3 июля 2015 г. пользователь Olga Grygoryan написал:

> Ну если честно, то я не подумала, что все может быть так просто и
> искала решение "позаковырестей":)
>
> пятница, 3 июля 2015 г. пользователь brus-fabrika написал:
>
>> А дело не в лёгкости для ТЕБЯ, а в лёгкости для компьютера )))
>> зачем делать 2 лишних операции, тем более - таких довольно дорогих в
>> плане производительности?
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/OlgaGrygoryan/homework/commit/33ed489312b1a0171b9c1f3abd76502303a4b757#commitcomment-11998969>
>> .
>>
>


YOLO


It would be a design bug to close the response here. It must be closed (or not closed, or whatever) by the user.


Changed SVN


My fault. I forgot to point this method. Also today I've rewritten setScopes method. Now it accepts array of uri instead of object because onvif command uses only set of uri strings. Is this OK?


This line had been storing the length in a newly created instance variable 'self.off' instead of the pcs.Field named 'offset', which is reachable at self.offset via the inherited __setattribute__ override.


На FreeBSD забиваем?


OK, after the last commit, dynamic_fmt and dynamic_compiler have been fixed, so that dynamic_compiler can handle all possible internal formats that it is going to handle.  There are still some which it is not going to handle at this time (cryptmd5, phpass, PO, HDAA, etc)

Now, I simply need to get these formats to properly 'optimize' so they run as fast as the hand build dynamic scripts.


If we keep in mind that we're solving the issue regarding the source stream not being closed when target errors, then we should end it explicitly.


Remember that what's happening here is an asynchronous caching process. Something like this.
```
[user] --- request() ---> [cached-request] ---> request() ---> [endpoint]
....
[endpoint] ----> response ---> [cached-request] ----> response.pipe(gzip).pipe(cacheFile)
                                                                           |---> response [user] --> //user should be able to do anything with response here, without it closing because of an error in the caching process.
```


A couple of tabs slipped into this patch. I removed them before applying it.


Has this change be tested at all? I don't think so (and if it was tested, then you should have immediately reverted it). There was a reason I disabled these extensions, and I didn't care about how important they were as the goal is to make the wikis work properly (as in that people can view and edit without hundreds of exceptions)

Ansible hasn't run on prod9 for 24 hours now (source: nagios - it ran on prod8 less than 10 minutes ago), and this change was merged in production 19 hours ago. Since MediaWiki shows hostnames in the HTML output, I see that prod9 doesn't throw MWExceptions, but prod8 does. I don't know if prod8 was erroring out before this change got deployed on prod8 as well, but I suspect the reason prod8 is throwing MWExceptions because of this change.

Really, people either should better do testing somewhere else and revert this change immediately, okay? I have no idea why these extensions fail, but I never would re-enable them until I'm 100% sure Orain works perfectly with them enabled.


Hi @suyanNone, I think the original  `scheduler.resubmitFailedStages()` from my test case is actually more appropriate.  Yes, the DAGScheduler will post an async event to do this, so sleeping long enough *should* work.  But, that could just lead to both a slower test and / or flakiness in the test suite.

Instead, `scheduler.resubmitFailedStages()` forces the processing to happen immediately -- eg., it blocks until the failed stages have been resubmitted.  You will see it being used elsewhere in the test suite similarly.  It does mean that at some point, the async `ResubmitFailedStages` will also get processed and `scheduler.resubmitFailedStages()` will get called again, but that doesn't hurt anything.


I think you might have misread this. This is the loading variable


Good call.


No it has been removed


We should use CSS to blur, so we don't have to go into Photoshop every time we want to change it


You can replace the explicit type argument ```new TreeMap<String, Point>()`` with ```<>```.


Ufff... problems with the pycharm commit+push :-1:


Well, actually this should be the behavior for all other examples. If you use a fragment in that pthread example, it will also crash.


Yeah! I learned that the other day. http://jade-lang.com/reference/comments/


Requires you to copy login_opcodes_oldver.conf to server dir from git pull


Check the setprivacymode..
On Jul 3, 2015 6:20 AM, "KilluaFein" <notifications@github.com> wrote:

> If I add my bot to a chat group, the bot stop responding at all. Why?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/yukuku/telebot/commit/a702777b2bec2517cb4fcc6f94d9638cff734a5f#commitcomment-11985448>
> .
>


Para joderle la vida ut


On line 43 ```` if (!parameters[0].ParameterType.IsA(memberType))```` it's asking for a collection element but on this line asks for a collection thus it always fails...


博主能不能发布md格式的，这样的话文档有问题的话，大家可以一起修改


These are the projects I have taken part in and welcome you to join us!


Yes it does. Nice catch.


Good to know. I will take it out until it's necessary. That way there's less to confuse those that are reading the sample.


I'm not quite sure what you mean. I just copied what was in Julia's `.travis.yml`.
But I'm happy to just go back to what was there before! Is this what you're suggesting?


Fixed in https://github.com/777jon/g3_kernel/commit/306d1715697ee03cad45139416fbee031d86e7ca


You don't really need to change anything if you don't want to, I'm just letting you know that the `ARCH` environment variable has no special meaning for Julia packages - it does have special meaning for base Julia's makefiles when you build Julia itself from source though.


Yup, my bad


97648166-FixPIUInducedNode-4926 (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1168 of 1168 tests passed)

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


it changes the variable to live in state as the SSOT rather than renaming school in each function


Done. It worked nicely.
```
var userManager = notification
    .HttpContext
    .RequestServices
    .GetRequiredService<UserManager<IdentityUser>>();
```


It is better practise to use delete, apparently calling the destructor explicitly doesn't release the memory the object was using but only executes the destructor.


You should be able to see from the output of `versioninfo()` that Travis will always install a 64-bit version of Julia, so adding an extra entry to the matrix for `ARCH=i686` is just an extra copy of the Linux worker. It could be possible to manually download and use the 32 bit Linux binaries of Julia, or enhance the Travis script (https://github.com/travis-ci/travis-build/blob/master/lib/travis/build/script/julia.rb) to handle 32 bit Julia in some way. May be best to wait until you're back from New Hampshire before looking into that though.


That commit message is driving me insane


No no, the patch I pointed you out is not a final patch (I haven't even sent a pull request) I just pushed to my repo to show you what I mean, you can even grab this patch and squash into yours, no problem. With this patch I'm just taking care of the "-Dsol_flow_default_resolver=" fragment... all the other issues concerning the resolver and so on are up to your patches.


btw, it look solid hard :dancer:   I like it somehow


git status


:question: Did you mean to remove `Async12` here?


yeah


## yeah


Nope. This function is called only with open and install tags (that's the contract). What we are doing is removing all installs and opens from the queue. We have four options

1. An open comes in, and there are no installs in the queue. The open stays open.
2. An open comes in, and there is install in the queue. The open is now install.
3. An install comes in, and there are no installs in the queue. The install stays install.
4. An install comes in, and there is install in then queue. The install stays install.

You check the queue for the tag to remove installs and opens. You check the queue for the install tag to see if you need to "upgrade" from open to install.


@sojanpr


looks pretty good


you dont need the `,session)` parameter for the definition of `parseResults`, `self.session` gets defined in `__init__` and you get it form the class here.


this is exactly right.


Pas user mau login, bisa login gak?
Pas login, itu hak aksesnya jalan gak?


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

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


Thanks, it works!


`str.replace(/com o a/i, 'com a')`
`str.replace(/estado/i, 'Estado')` => *Estado* sempre maiúsculo.


thank you ^^


Accepted.


Colocar como código: `-logo.png`
Remover parênteses e adicionar `,`


first commit


why do u remove the id? condition check for close tag?


Daylighting_Speed (DeadParrot) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Tests Failed

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


Hah. Nice catch! Can you open an issue please?
(Better yet a PR with the test case would be fantastic)


Yeap!


This is for compatible to existence of `to_json` method.


npoulindupont@gmail.com


Daylighting_Speed (DeadParrot) - 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)


Import method should check md5 and save the data only if it does not match.


@aisamanra Would you mind giving this Haskell a look-see? I'm no Haskeller, but I got it to work. Would love someone who knows the language well to give it a look. Also, feel free to tell me to buzz off - I'm just assuming that because you submitted a PR you don't mind me making you participate more :smile_cat:


Hi, just tried your fix, but I think you need to rewrite the line 117 with:

image_admin_url_name = 'admin:{0}_{1}_changelist'.format(Image._meta.app_label, Image._meta.model_name)

Because in the docs says that you can add your own app_label to the custom Image.

I hope it helps!


i guess this logic can handle the string `</b>`, please add the corresponding test case.


AWS region is hard-coded? if so, it is better to give it as a command line argument.


the above logic still cannot handle the `<!--` as it is not the tag open name state.


wat ?


openenTag is not initialized as [], TypeError may raise if push/pop are called before initialization.


this boolean flag seems no use ....


what crashes? which behavior? which examples? I can't get your point.


hmm ... actually now I am wondering if it should be `runEvent(ResubmitFailedStages)` ... I will think more about this


May be it would be better to use "Form Theming" approach since all your forms are going to be in "bootstrap" style.


note: we don't yet support binomial models specified with NCOL(y)==2, but this should work once we do


0x100000 really looks strange


That stuff should be moved to the init_per_testcase function


That unload to the end_per_testcase function


Same comments for this one


Thanks for the corrections.


Regarding main rule already showing the upcoming, yes it does, but as soon as the upcoming recording starts, it shows the time of the next upcoming. This is confusing as the list 'jumps' as soon as a recording starts, skipping the currently in progress recording. Adding this in made the order more stable and consistent with the upcoming timers which show below the rule.

Yes, if more than one recording is ongoing this will show the times of the most recent recording to start, but this could be considered a 'feature'. I'm happy to change this behaviour if 'check 1st upcoming only for in progress' would be a better solution. I'll run some tests and see what seems most intuitive.

Regarding only showing 'Active/Inactive' for repeating rules, I noticed some discussion on the PR7079 thread as well. I don't agree with this as a philosophy. You need to be able to see quickly which rules are currently doing something, and which have conflicts so you can do something about it (this was also @MikeB2013 's opinion). This change accomplishes that.

Regarding performance, I did re-write this to only get the most recent upcoming, but then @MikeB2013 said he wanted conflicts to appear as well. For that the full upcoming list needs to be searched.
Anyhow, it's only checking an array in memory, and only then if the rule has upcomings. I don't think this will kill an RPi and being able to see if a rule has active conflicts is worth it in my opinion. The timer refresh runs as a background task anyhow.
If you're worried about performance, MikeB2013 says he's got your series_recordings branch running on OpenElec/RPi, so we could ask him to try this branch to see what he thinks?


Please create a new issue for any new feature


test


Thanks :+1:


I think there may be an error with this code in the Angular-Meteor tutorial on Meteor.com. When you you come to the part of the tutorial that has you rebuild the template, the code reads:

```
<!-- replace the existing task template with this code -->
<li ng-class="{'checked': task.checked}">
  <button class="delete" ng-click="tasks.remove(task)">×</button>

  <input type="checkbox" ng-model="task.checked" class="toggle-checked" />

  <span class="text"></span>
</li>
```
So if the code is copied exactly you won't be templating any values into the span!


To summarize: Closing the response is the user's responsability. Closing cache files is ours.


Got it. I understand now. Thanks.


Why so heavy framework has been chosen for so simple task?


(y)


Note for testing changes before deploying stuff:
1) You always should first disable the Ansible cron before testing, and in the ideal production environment you let know others you did that (although I never really did that, but there were reasons)
2) You do what you want
3) You test stuff with `curl -v -H 'Host: meta.orain.org' 'http://127.0.0.1/wiki/Main_Page'`
4) Confirmed the change works? Then you apply the change to the playbook, and enable Ansible on the server again.

This should really help.


That works and looks a lot nicer! Thanks :smile:


Hi @oligriffiths, we have some patches we applied this week to joomlatools/joomla-composer to backport first. I'll tag a new release together with those changes.


When the way you split the code into smaller methods starts to result in duplicate code then that is the time to re-think. I really, really do not like duplicate code. It always causes problems. I have spent literally months cleaning up duplicate code in the Connectors and I don't want to see this code starting to head down the same path the Connectors went down.


:+1:

I will do the same in the NuclearJS docs


yes


Awesome. Thanks!

> On 3 Jul 2015, at 16:07, Steven Rombauts <notifications@github.com> wrote:
>
> Hi @oligriffiths, we have some patches we applied this week to joomlatools/joomla-composer to backport first. I'll tag a new release together with those changes.
>
> —
> Reply to this email directly or view it on GitHub.
>


Fixed


The buffer is never "." or ".." because these don't match the mask, so the first two strcmps were not needed


lol i am semi-retarded at least


@ophian - Please do not commit to 2.0.x! This is too problematic and I wouldn't classify this as a bugfix that is meant for a 2.0.x series release, especially if untested.

Also, I think this will not work in most cases. You cannot simply replace mysql_* with myslqi_* because parameter order changes: mysqli_query() uses:

mixed mysqli_query ( mysqli $link , string $query [, int $resultmode = MYSQLI_STORE_RESULT ] )

but mysql_query previously used:

mixed mysql_query ( string $query [, resource $link_identifier = NULL ] )

this means that with the patch above, no query will work at all. So it would be good to either remove the commit in master and make a branch/pull request for others to test, or fixup the current situation with an importer that has been tested at least once successfully ;)

Thanks for the effort and trying to get this sorted out!


I believe this will cause an error since `pulp::consumer` hasn't been evaluated yet.


Glad you like it :)

Do you want to update the code and do a pr, or would you prefer I do it? (I won't touch your code/feature without your say so - though I may have a couple of questions about what shortcuts exactly should be covered by this code if you would prefer I do it, as I'm not sure what the use cases are for this exactly...)


Okay. I think I'm going to add dictionary-like methods to those list models so that you can access them either by index or by token. I'll send you a pull request for comment when I've got the change ready.


Need to fix : toString() does not accept any parameter


This should have a version specifier. `>=2.7` would work.


I think you meant canolz oil...


I completely agree with you!


Would you move this to the top under the base64 import?


I think there are many gaps in hostel application. Whoever is looking after this part may come and discuss to me. I think this part needs a more attention.


once is enough ;)


LGTM


This `_orderRequestQueue` should live in the transaction handler. We can pass the transaction handler requests via the `IOrderProcessor.Process` method


This whole property will be removed, users should use the Add/Update/Cancel functions on this class instead.


sadas


Fixes #40


Thanks, will keep in mind for next time. However, it did lead me to another theory by doing that. I noticed most of the errors happened on pages that brought up ParserFunctions related material, so I'm wondering if switching to another branch of PF would fix things.


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (352 of 352 tests passed)

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


It seems you are now setting the expire date to a date in the history?
In https://github.com/LitusProject/Litus/blob/master/module/CudiBundle/Resources/views/cudi/admin/sale/booking/partials/navigation.twig#L133 you have to enter a date to query all bookings with an expire date lower that that. And now you are setting the expire date to this date.

So if you want to extend all bookings expiring before 3/6/2015, the expire date will be set to 3/6/2015. What is the goal?


This appears to cause problems on our Moodle 2.8.6+ (build: 20150618). When . is used to concatenate the token strings, a number of questions do not render their input fields and only display 'qtypemultianswerXmarker' token text  - see the following post on the Moodle quiz forum: https://moodle.org/mod/forum/discuss.php?d=316235#p1267127

Reverting to the + operator resolves the problem.


 - __style__: [src/chaiscript_stdlib.cpp:18](../blob/496f5aff7aa743afea6e6c5ba9d88a9b8929515c/src/chaiscript_stdlib.cpp#L18) The function 'create_chaiscript_module_chaiscript_stdlib' is never used. (unusedFunction)
 - __style__: [src/main.cpp:169](../blob/496f5aff7aa743afea6e6c5ba9d88a9b8929515c/src/main.cpp#L169) The function 'get_eval_error' is never used. (unusedFunction)
 - __style__: [src/main.cpp:140](../blob/496f5aff7aa743afea6e6c5ba9d88a9b8929515c/src/main.cpp#L140) The function 'help' is never used. (unusedFunction)
 - __style__: [src/main.cpp:216](../blob/496f5aff7aa743afea6e6c5ba9d88a9b8929515c/src/main.cpp#L216) The function 'myexit' is never used. (unusedFunction)
 - __style__: [src/main.cpp:158](../blob/496f5aff7aa743afea6e6c5ba9d88a9b8929515c/src/main.cpp#L158) The function 'throws_exception' is never used. (unusedFunction)


develop (lefticus) - i386-Windows-8.1-VisualStudio-14: OK (352 of 352 tests passed)

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


Do we really need this isSendEmail variable? It seems it is always set to 'false'?


MediaWiki runs the 1.24 ParserFunctions branch. You could try to switch - I don't think it works, perhaps it does, perhaps it doesn't.


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-LibC++: OK (352 of 352 tests passed)

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


random versión 1


I see now, I was thinking about [this](http://forums.soartex.net/threads/mod-contributions-texture-submissions.888/page-3#post-12463) post from the forum.
It has some 1-block flowers, manasteel armor, mana diamond and pearl, and rune0-15 included.

I haven't taken a look to see how they fit in with Soartex Fanver, but I'm sure they will fit just fine.

I will PR them if that would help?


What's this file?
Did you created a branch called david?


@zappoman- I think you may have thought I was asking about executing the batch in the overlay's render method, whereas I was referring to executing it at the end of renderHUD method, based on @huffman's comment. If I misunderstood your misunderstanding, I apologize, and I would ask for further clarification on how to handle the batching here.


develop (lefticus) - Win64-Windows-8.1-VisualStudio-14: OK (352 of 352 tests passed)

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


@paulolieuthier This one was removed.


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.6-AddressSanitizer: OK (352 of 352 tests passed)

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


Are you going to test this *now*, or revert this now?


@hastalafiesta Did you try this commit for green band issue?
https://github.com/sudosurootdev/frameworks_av/commit/1dc8b0911918a0ad19dcd82d43f3cfe029016ad3


Also includes NetLogo version of transmission model


Will rebase next week.


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.6-ThreadSanitizer: OK (352 of 352 tests passed)

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


Man


You aint got an idea of how GitHub sites works right?


This introduces few other improvements for monitor:

* removed rat-race between child-killing loop and child-respawning loop
* old_pid / respawn_id is not passed to child by argv any more (uses process.env instead)
* not leaving dummy monitored apps when stopping the monitor after child was respawned and before it subscribed back.
* plesk compatibility tests performed


You need to have the username "Mod-a-thon" to register mod-a-thon.github.io


You could use single qoutes ' instead of double qoutes " here. Are you aware of the difference?


Once working, yes

On Friday, July 3, 2015, Vaughan Hilts <notifications@github.com> wrote:

> Would this address #1881
> <https://github.com/photonstorm/phaser/issues/1881>?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/photonstorm/phaser/commit/8919e4e128c927cdeb8fb6aef67933815b5f3713#commitcomment-11996521>
> .
>

--
Photon Storm Ltd.
http://www.photonstorm.com

Skype: richard.davey
Twitter: @photonstorm

Registered in England and Wales.
Company ID: 8036404
VAT Number: 136 4333 27


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (352 of 352 tests passed)

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


develop (lefticus) - i386-Windows-8.1-VisualStudio-12: OK (351 of 352 tests passed)

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


Was going to test this now, changing ParserFunctions should not break things globally, given it mostly provides added functions to template code, so if it doesn't work, the situation will remain the same in the worst case.


![1097682_491570557594304_828261944_o](https://cloud.githubusercontent.com/assets/13166524/8504595/335c7008-21cc-11e5-99df-0d1094a29c0e.jpg)
55-e;'\


You could call swap single character here. I think that's more readle compared to add 32


@InnerFire @W4TCH0UT @akhilnarang


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.6: OK (352 of 352 tests passed)

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


You can check out (as in, `git clone`...) the jquery plugin and open index.html for a nice demo on how to use it: https://github.com/jreesuk/jquery-entropizer


Can you provide some examples of usage?


maybe you want to intorduce a constanste for the 32 as well and explain the significance


You shouldn't need an `id` on the `body` element -- there's only one in the DOM, after all -- and the background color should probably go in external CSS.


Can we get this JS into an external file, or does Facebook make that difficult?


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.67: OK (0 of 0 tests passed)

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


You are almost there


Hi HaCk3Dq - I tested your changes in this commit and I think that this is not correct - I'm getting
```nimrod
lib/pure/os.nim(1025, 19) Error: type mismatch: got (typedesc[OSError])
but expected one of:
os.raiseOSError(errorCode: OSErrorCode, additionalInfo: string)
os.raiseOSError(msg: string)
```

looking at `raiseOSError` I think that this is accurate - OSError is a type, and the error code needs to be retrieved using `osLastError()` or equivalent to get a concrete OSError.

(If a custom error is needed, I'm not sure we can accommodate it.  In the past it looks like we got the error message with `osErrorMsg()` on the error code, but since `raiseOSError` only accepts an int / OSError type, there's no way of passing a custom message along in the exception).

I'll submit a PR to your fix-2573 branch.


This issue can only happen if the web server was upgraded to require stronger crypto at some time.  That's when this script stopped working for users of e.g. IcedTea < 7-2.5.3 .  Since Freenet contains code for Java 5 and 6 this must have been a new server change which brought this issue out.


Move JS to external file if possible.


Added newline to seperate core modules, from npm modules, from local modules.


Why is this here if it's commented out? Can it simply be removed?


The responsible for our CuDi thought current implementation "felt weird".
Best way to explain is by using the easter vacation period, that is how he explained it to me.

He wants to select a date (the day after the vacation) and every booking before that date (these will be before the vacation) will be set to this current date.


develop (lefticus) - Win64-Windows-8.1-VisualStudio-12: OK (351 of 352 tests passed)

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


So is Morph a more advanced implementation of the old stock mode setting that we had in vanir kk44 under developer options?


The cwd should actually already be resolved, but I suppose it might have happened for a reason.
Instead of doing it all over the place I'm now doing once though.


Maybe update the text in the modal for this, so it explains exactly what will happen


Thanks for doing it 👍
On Jul 4, 2015 2:18 AM, "TJSteveMX" <notifications@github.com> wrote:

> @akhilnarang
>
> 5.4.9 for the i9505
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/ResurrectionRemix-OTA-Server/resurrectionremix-ota-server.github.io/commit/74d5d616f790256d18fe74011f101237c44d6ce5
>


Too sleepy to reply now, waiting 6hours
On Jul 4, 2015 2:15 AM, "Swapnil Solanki" <notifications@github.com> wrote:

> @InnerFire <https://github.com/InnerFire> @W4TCH0UT
> <https://github.com/W4TCH0UT> @akhilnarang
> <https://github.com/akhilnarang>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/W4TCH0UT/zz_sprout/commit/bc6f521f48c2c38a2c2155f4613a61094f2f3185#commitcomment-11999341>
> .
>


The commit message says remove Gemfile.lock, but I don't see it removed here...


Not sure that's technically true.


La validazione della mail è fatta dalla funzione is_email, che è fra le funzioni core di WordPress (wp-includes/formatting.php).
Non prevede il formato user@localhost perchè manca l'ultimo punto (vorrebbe qualcosa come user@localhost.it, e inserendolo effettivamente viene accettato).
https://core.trac.wordpress.org/browser/tags/4.2.2/src/wp-includes/formatting.php#L2179


Any time Akhil


Agreed. Bored.


The refactored version is more explicit and also handles having the same port twice.


-1

There is a missing TODO here. Now it's only used inside sol-oic-server.c because we only support the default static server info, but support dynamic info will still be added. In that scenario, this structure will come from the user so the structure needs to be public.


This check was changed from `instance of State` to this line. Would like feedback on this...


Owkay, first thing I'll do tomorrow


@paulolieuthier maybe running the sript again will produce better results, than rebasing, in this case.


Geth, c'mon, just revert your commit so the affected wikis come back and then you can test...


Moved all unnecessary `hasOwnProperty` calls up and inlined where possible.


Tom, what's our coding style policy on column alignment here?

I would _prefer_ we keep the equal signs aligned for readability; if we don't that's fine I guess.


thanks so much


Perhaps `isRunningOnYosemiteOrLater` should just be:

    return NSAppKitVersionNumber >= NSAppKitVersionNumber10_10;

Apple's documentation uses `floor` because they're checking if the version is lower and they want 10.10.1 to compare equal to 10.10. The use of `floor` here isn't accomplishing much and could break in surprising ways in the future. If someone ever needed to adapt this code to test for e.g. OS X 10.11.2 or greater, the current form with `floor` would return false unless you were running 10.12.


Did we want to remove all the logging functionality?


The path got resolved (i.e. made absolute) below in L47. The differentiation doesn't make any difference that way.


Inlined this, b/c now it's only used once.


thanks!


@ingwinlu yes, that would be great. We did it for `rabbitmqadmin` earlier this year.


Refactored this so there is no more need for temp vars. As hapi allows to pass a string rather than a function here, I could remove that entirely.


Good point. Will do.


fix your git config :smile: it has the wrong email so your profile pic is missing


Are these completions still being used? I think we need to remove them so we can do the async stuff as a separate issue.  I imagine the transaction handler will be the one in charge of this type of behavior


also, if you're looking for a collaborator - i can help ? there are a lot of PRs that should be responded to.


Here you just need to check if client is NULL, all the others parameters are safe be NULL ?


@nateirwin I changed the `package.json` repository URL in https://github.com/ryanjhodge/npmap-symbol-library/commit/2bec78cc016de77c5b50306a0392d0bcbd96cb8e

After your Reorg, should it be changed back to how it was?
```
"repository": {
     "type": "git",
     "url": "http://github.com/nationalparkservice/npmap-symbol-library.git"
},
```


Do we need this 'font-family' twice?


Can you please clarify why did you decide to remove this block?


Could this be merged to master for 1.59 please?


You're right.


This project uses [FFmpegWrapper](https://github.com/OpenWatch/FFmpegWrapper-Android). Within that, see [FFmpegWrapper.c](https://github.com/OpenWatch/FFmpegWrapper-Android/blob/master/FFmpegWrapper/jni/FFmpegWrapper.c)


I want to store the current brewing recipes in the pointer array. The server crashes here at the check, what do I making wrong...


I have lost track. Was this in the pull request that I merged?