I'd have separate comment about why the locale env vars should be set for Rbx.


This would be a wonderful place to use the contributed site settings module (see http://docs.wagtail.io/en/v1.3.1/reference/contrib/settings.html)


I still don't follow this? There are no good ways out (return statements) in the above code and therefore this will always be tested on the next iteration, instead of throwing when set. Given your new comment in other code " bool dynamicOffset = false; // Will the offset of the next field vary with each row processed?" I assume this is structured this way so that it doesn't fail if there isn't another iteration when isDynamicOffset is set = true. My only issue now is that is the precedence of the "bad ways out" which are the other throwErrors, i.e. that they will take precedence so shouldn't this test be at the top of the loop?


I'm curious, why did you comment out this code? This means "pythonExtraPaths" no longer works, but the parent's "codeintel_scan_extra_dir" now will. But for some reason, we still have a add_pref_observer(self.extraPathsPrefName...) even though it now does nothing? Should the rest of the extraPathsPrefName code be cleaned up now?


s/for/from


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11020](https://teamcity.mesosphere.io/viewLog.html?buildId=102169&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 212 Build time: 00:04:01


@runcom yes, it should not happen, but it's better to be on the safe side.


This commit is being reviewed as a part of review issue #129.


perfect, I'll update the code


It will be easier for me to comment on the diff in the PR.
But I'd suggest you to test InMethod-Grid 7.2.0-SNAPSHOT with all browsers you have against your application.


Thank you for the reply. I thought that email was suspicious.


..yes updated


thanks


Nice!


Thank you. I will look into this today.


#damn

Ok, I'll take a look after lunch.
On Jan 8, 2016 1:27 PM, "Federico" <notifications@github.com> wrote:

> So, tried with both libraries simultaneously:
> [image: screenshot from 2016-01-08 13 18 38]
> <https://cloud.githubusercontent.com/assets/1950251/12197799/9f3d9ad6-b60a-11e5-97c5-ac9dd9c12442.png>
>
> Cyan is old lib, Purle is new lib (as usual). They both look wrong to me...
>
> Tried with different sphere point density, normalizing and not, tried also
> with discarding Tx Ty and only get N from lib. From that N I then compute a
> new tangent basis with Tx as close as possible to X kinect...
>
> All of them leads to to this.
>
> Of course this is not 100% implying that code library is wrong. I could be
> doing something wrong with picturing markers, but I keep reading that code
> and to me it looks good. Maybe if you can also look at it you may spot some
> bug I keep ignoring...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pacman-project/gaussian-object-modelling/commit/99f6b1d3b05f5e4c049842d06dc4b5c45e78a392#commitcomment-15338411>
> .
>


Give that all this code is being deleted in this commit, I'm not quite sure what you mean?


@sirthias I know this is oooold, but please bear with me: This marks the introduction of
```scala
conn.getInputStream.close()
```
and I wonder what the reasoning was behind this (opening and immediately closing the `InputStream`). Was it to provoke a NPE in case of `getInputStream() == null`? It seems to not have any effect otherwise.

Maybe @jrudolph could chim in as well?


Descriptions of functions and other stuff may be too long so they are not shown in full size. Let's put them in tooltips.


> Hi @awelzel, why do you think that nowait is broken with 1e0b0f3? What specifically did it break? Thx.

I created #688 as it's an independent issue.

> Queue.declare-ok doesn't contain any information that allows the declare-ok response to be matched with a specific Queue.declare request. If an application makes several Queue.declare requests for the same queue, but let's say with different parameters for some reason and different callbacks, how would pika be able to match a specific queue.declare with the corresponding Queue.declare-ok?

Makes sense, Queue.declare-ok only has the queue_name. I'm not sure if it's relevant in practice though. Doing what you describe will result in an error and channel/connection close. It's not allowed to declare the same queue with different parameters.

> Does AMQP guarantee that Queue.declare-ok responses will be returned in the exact same order as Queue.declare requests?

The broker certainly receives the the methods ordered as sent by the client (TCP). As long as the broker is required to process requests and send their response synchronously, it should be ok.

It might need to - if a client is using the no-wait bit, there might be dependencies among the methods. The broker certainly should process the methods in the order received, otherwise the ```bind()``` may fail:

```
Exchange.Declare e1
Queue.Declare q1
Queue.Bind e1 q1
```
Sending the responses in the same order may be expected?

Unfortunately, from the AMQP spec I can't answer your question. I'm cc'ing @michaelklishin, he's very active on the RabbitMQ list and may have enough AMQP knowledge to chime in?

> Also does RabbitMQ guarantee this ordering in a federation configuration?

From what  I understand about federation, it shouldn't be of concern, but I might be wrong.


+- Turn project into a NuGet package


Und da hat wohl jemand Langeweile und spamt andere Leite voll...


ya i think sepolicy mod in ramdisk is only necessary with system-mode supersu flash, and since systemless supersu seems to be unreliable at the time, i also made another commit to force supersu to use system-mode flash. i think it will avoid lockups in the installer, especially if ramdisk is not gzip.


This needs a couple of checks: repo could be null (ISimpleRepositoryModel doesn't require the repository to actually exist locally, it might be a reference to a repository that only exists remotely), and the repository might not have any commits


yes, i also guess it is not worth changing


this can result in an NPE, imho if localeValue is NULL or evaluates to NULL, defaultLocale must be used


![qq 20160108204137](https://cloud.githubusercontent.com/assets/8433954/12198155/51336062-b648-11e5-8ae8-dd456744ce0e.png)


Changes to this file should not be committed, it's an example file for local development.


I don't understand the purpose of changing the value of field JavaPrefixCommandExtractor#PRE_JAVA_CMD_KEY. Why is it required for the test?


Sorry, you want this to be:

```ruby
url "https://github.com/codekitchen/dinghy.git", :tag => "v4.1.0",
  :revision => "f0873d48e432cc061d7feab4068c9c00066e883b"
```


Then you can remove this line.


"a primary graphical diagram editor and two tabular editors for entities and variables"

The wizard for creating new models uses the basic settings and **then** saves repeated input of recurring data.

**When** opening a model it is automatically validated.

**When** saving a model it is automatically validated and the model file is consistently **formatted for readability**.

**When** creating relationships the source and target descriptions are automatically preset to the **appropriate** singular **or** plural names of corresponding entities.


`:branch => :master` is implied automatically so not needed.


`system "#{bin}/dinghy", "version"` will verify you're testing the right one.


`0 changes files` Is it Github's bug?


An upgrade function _must_ call runSql as well as any other tasks.


I think this was the only way to get a hold of the last-modified timestamp.
Would that make sense?


단어들 배열로 받은것들을 String으로 합쳐서 모두 보여주자


You should not overwrite the version, since it's already managed in the parent pom


I think you forgot the real url here? You can just put https://github.com/totetmatt/gephi-plugins/tree/multimode-network


try


No. `isTypeOnleCoercion(varchar(40), varchar(5))` will return false, because we can not coerce `varchar(40)` to `varchar(5)`. So such insert would be rejected.

Is this an issue? It seems like a safer approach.

CC @losipiuk


`nowait` operations must not get a response. Other synchronous protocol methods must be responded to in the same order they were received on a channel. Pipelinening is not possible with RabbitMQ, at least it was never a goal.

All federation operations are entirely asynchronous.


My NetBSD system can no longer build the `ssl` package and, hence, the `LWT` package since `conf-openssl` was introduced. OpenSSL is part of the base NetBSD system, so `pkg-config` knows nothing about it.

If I fork `opam-repository` and patch this, how do I get `opam` to use my modifications?


You can log exceptions with Logger.getLogger(YourClass.class.getName()).log(Level.SEVERE, null, e); and remove all System.out


`summary` sounds like a mutable string :-)


But isn't `getLastModified` read *before* this line?


This is handled by the `canCoerce` call in first line.


We are introducing lots of duplicate code here whereas we only want to tweak the description of the storage feature. We could just follow the same pattern than for the support feature:

```javascript
<PlanFeature
  description={
      isBusiness( this.props.selectedSite.plan )
      ? this.translate( 'You can upload unlimited photos, videos, or music.' )
      : this.translate( 'You can upload up to 10GB of photos, videos, or music.' )
    }
  heading={ this.translate( 'Storage Space' ) }
  willBeRemoved={ this.props.willBeRemoved } />
```

If we want to avoid this approach, then what if we took advantage of components? We could introducing a new component just for this feature:

```javascript
/**
 * External dependencies
 */
import React from 'react';

/**
 * Internal dependencies
 */
import PlanFeature from '../plan-feature';
import { isBusiness } from 'lib/products-values';

const PlanFeaturesStorage = React.createClass( {
  propTypes: {
    selectedSite: React.PropTypes.oneOfType( [
      React.PropTypes.object,
      React.PropTypes.bool
    ] ).isRequired,
    willBeRemoved: React.PropTypes.bool.isRequired
  },

  render() {
    let description = this.translate( 'You can upload up to 10GB of photos, videos, or music.' );

    if ( isBusiness( this.props.selectedSite.plan ) ) {
      description = this.translate( 'You can upload unlimited photos, videos, or music.' );
    }

    return (
      <PlanFeature
        description={ description }
        heading={ this.translate( 'Storage Space' ) }
        willBeRemoved={ this.props.willBeRemoved } />
    );
  }
} );

export default PlanFeaturesStorage;
```

And call it from `<PlanFeatures />` with:

```javascript
...

<PlanFeaturesStorage
  selectedSite={ this.props.selectedSite }
  willBeRemoved={ willBeRemoved } />

...
```

That way our components stay small and are easy to reason about. What do you think?


looks mostly correct apart from that memset. I've tried to restrict my comments to only things that might be harder to find.


Fixed issue #9


The log message is misleading. Setting the pass-through value does not disable saniization, it just changes the return value in case of an exception. Sanitization is still applied.


@profi200 try now.


ここは複数形にしておくと良いです。「threads」

たとえば、クエリが、
`SELECT thread FROM thread;`
とか、
`.. JOIN hoge ON thread.thread ...`（こんな書き方になることは滅多にないと思いますが）
みたいに、テーブル名なのか、カラム名なのか混同してしまいます。

複数形にしておくと、Laravelの便利な機能が手軽に使えるようになります。
下のサイトを読むと、複数形にしておくメリットが序盤で解説されます。
http://readouble.com/laravel/5/0/0/ja/eloquent.html


Yes, but in order to be able to call `getLastModified` we need to call `openConnection`,
And once the connection is open we also need to close it in order to prevent a resource leak.


It breaks my application when using '--no-dev'. Why not working with tags?


I'd like the body of this if statement to be extracted into a separate method: `isArgumentUsed(FormalArgument*)` You should put this new method directly in `OOModel::Method`


Ah, now I understand your question! :)


I will do here a change for the handling of card movement.


Or even, why not put this directly in `FormalArgument`. Then the argument should just find its parent method and do the search.


I think I actually looked through the source of the JDK and realized, that the only way to properly dispose of the resources hogged by `openConnection` was to call `getInputStream.close`.
It does look strange and should have definitely received an explanatory comment.

I also could imagine that the JDK-internal machinery has been improved since JDK6 and this wierd construct here is unnecessary now...


你也可以创建插件告诉ESLint如何处理JavaScript之外的文件


Yeah you wouldn't need the 'p' there, as there would already be styles inside the partial that is being rendered.


I misread the function. When I read "levelTypeCb->getItemIndex" I thought you were in the code called when a level is selected. But you are in the update list so it makes sense to read the levels ^^
Forget my comment, it's my mistake


we typically mix in Slf4jLogging for this, but it doesn't matter.


Ok nice.

On 8 January 2016 at 13:04, Konstantin Schubert <notifications@github.com>
wrote:

> According to the people on Stackoverflow, .gitignore file should usually
> be committed:
> http://stackoverflow.com/a/5765663/1375015
>
> Should you commit .gitignore into the Git repos?
>
> Normally yes, .gitignore is useful for everyone who wants to work with the
> repository. On occasion you'll want to ignore more private things (maybe
> you often create LOG or something. In those cases you probably don't want
> to force that on anyone else.
>
> The "private things" can be mentioned in the $GIT_DIR/info/exclude or
> ~/.gitconfig files as appropriate.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/matthewkenzie/gammacombo/commit/4f7701f2c684523232b81376db58887435d7dac0#commitcomment-15338080>
> .
>


@muhammad-ammar Thanks for pointing this out.


Of course ;)
Il 08/gen/2016 12:39, "Dan" <notifications@github.com> ha scritto:

> Installer update for new feature
>
> May you all please do the other translations? @matcracker @aodzip
> (Italian, chinese), mark others
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/ImagicalCorp/ImagicalMine/commit/4688d1527afc2dac5220dd54a0b82f446cbce0c6
>


Take a look at the latest commit of Genisys and you will get some help.


あと、ここは「id」にしておくと後々便利。
そうしておくことによって、
controllerなどで、`Comment::find(1);`とするだけで、`select * from comments where id = 1;`
を実行したことと同じ結果が得られます。

この辺りが参考になるかと。
http://kore1server.com/171
https://laravel.com/docs/5.0/eloquent#basic-usage


This sucks :(
I'm going for lunch.  Taylor
C.
On 8 Jan 2016 12:36, "Carlos J. Rosales" <notifications@github.com> wrote:

> #damn
>
> Ok, I'll take a look after lunch.
> On Jan 8, 2016 1:27 PM, "Federico" <notifications@github.com> wrote:
>
> > So, tried with both libraries simultaneously:
> > [image: screenshot from 2016-01-08 13 18 38]
> > <
> https://cloud.githubusercontent.com/assets/1950251/12197799/9f3d9ad6-b60a-11e5-97c5-ac9dd9c12442.png
> >
> >
> > Cyan is old lib, Purle is new lib (as usual). They both look wrong to
> me...
> >
> > Tried with different sphere point density, normalizing and not, tried
> also
> > with discarding Tx Ty and only get N from lib. From that N I then
> compute a
> > new tangent basis with Tx as close as possible to X kinect...
> >
> > All of them leads to to this.
> >
> > Of course this is not 100% implying that code library is wrong. I could
> be
> > doing something wrong with picturing markers, but I keep reading that
> code
> > and to me it looks good. Maybe if you can also look at it you may spot
> some
> > bug I keep ignoring...
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/pacman-project/gaussian-object-modelling/commit/99f6b1d3b05f5e4c049842d06dc4b5c45e78a392#commitcomment-15338411
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pacman-project/gaussian-object-modelling/commit/99f6b1d3b05f5e4c049842d06dc4b5c45e78a392#commitcomment-15338548>
> .
>


:+1: please create a pull request to upstream, I was about to implement a similar change :)


develop (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1465 of 1465 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)


> Nevertheless, the `modules` package is totally optional, and it will be up to you to add it to existing apps and/or packages.

Not quite - ecmascript is installed by default with 1.2.x, so any existing projects created at the version, or created with prior versions but with `ecmascript` added will have modules enabled by default.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11021](https://teamcity.mesosphere.io/viewLog.html?buildId=102174&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 212 Build time: 00:09:39


Using...

```
{{{ [wikiparserrules[]] }}}
```

...on tiddlywiki.com yields nothing. How so?


Oh, so that's how you do that. Thanks!


System must be read/write mounted in twrp


#fail


Interesting. Why is this necessary?


Ah, I figured it out:

```
{{{ [wikiparserrules[block]] }}}
```

I think without an operand the filter should return all rules.


This method is unused at the moment. Please put a rather well detailed comment what this does and when we might want to use it. Specifically I want you to describe what are the conditions under which:
- we will start using this
- we will remove this method, since it will not be used later


TeamCity pull requests :: netty [Build 1003](https://garage.netty.io/teamcity/viewLog.html?buildId=4616&buildTypeId=pulls_netty&guest=1) is now running


Put a comment here that points to the other comment which explains the situation.


If name of company is
Allons de l'avant
dolibarr will not work anymore. The ' is a common char in a lot of language so should not be removed or replaced.


why server log its helpful to see what my admins are up to lol


@adrianschroeter You might want to wrap this in an ensure. Otherwise in case of any bug in this codepath global_write_through stays falsy.

Besides, why aren't you doing pull requests?


No..i want to tell to aapav01 is that is any options added on sevsettings or not..??


*SecSettings


Is multiwindow added on this rom or not.??


这里是  exported object，“修改输出xxx的对象"不通顺


@adrianschroeter Why not solve this by fixing the CSS as Chris and I suggested in #1446 ?


rememberTokenって、commentでも必要ですか？
Userモデルだけで良いような気がしたのですが。


My thought was that this is can be the result of any authentication.


Sure. Here's the relevant log output from a [recent build failure](https://travis-ci.org/bbatsov/rubocop/jobs/100805748#L178-L269).

The problem is that Rubinius allows only integers at the beginning or end of an exclusive range (though they're fine with floats for inclusive ranges).


I cloned opam-repository, then edited `packages/conf-openssl/conf-openssl.1/opam`, `opam pin add conf-openssl packages/conf-openssl/conf-openssl.1` to convince opam to use the modified conf-openssl opam file, `opam install conf-openssl`..


@phlax, I found that we call for `get_object()` more than one time for the same object.


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1500 of 1500 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)


ここも、commentsにしておくと良いです。


@adrianschroeter Alternativ here would be to use "release_targets_ng.keys.first". But that didn't seem to be the right approach. What do you say?


Either this comment is useless or this line should say `if name != "":`


It's not clear that they should; this is guesswork based on rubinius/rubinius#3275. I've opened a new issue at rubinius/rubinius#3555, hoping to get some feedback soon.

In any case, it's no skin off my back if this isn't merged. That little red X in Travis bugs me, is all :smile:


@ta2-1 - no ill add an lru_cache on it - its a Django function


> I think without an operand the filter should return all rules.

See #2193.


> Other synchronous protocol methods must be responded to in the same order they were received on a channel.

Thanks @michaelklishin  for your fast reply! Just clarify, to improve latency, it's ok to fire off a number of synchronous protocol methods asynchronously, collecting the responses in a separate step.

> Pipelinening is not possible with RabbitMQ, at least it was never a goal.

I'm not sure I understand correctly - if the above is used by a client, isn't it some sort of pipelining?


Why removing this ?
If you don't want filter on fk_type, just call with fk_type = 0


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

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


selam


@EpicSteve33 You can choose to disable or enable it lol


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1484 of 1484 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)


I believe I am working directly with the master and have not forked at all (there is a book next to the repo on my client). I was entirely synced before I pushed my work. I knew that we would be experimenting with forks, but I thought this (working like we had been) )would be fine because I saw recent files being changed (meaning that I assumed I had the most up-to-date sync, and I was only committing to 1 change when I pushed). Did I delete something? Did something go wrong?


Don't need to add such translation key, it is already available into main file
ErrorFieldRequired


Please try just `{}`


testing /xylem


No all is well and your commit was successful; however, you do indeed have a forked repo. It just isn't where you are working from in your client. We  will discuss this further  before or right after class. Have a lot to share with the group of returning students in regards to forking, branching, cloning, and working command line. Thanks for your continued work on Nelson and your response. See you shortly.


static import?


Log comes from wrong copy/paste ? (from in-memory client).


let's start here


これは質問ですが、このビューを作る目的は、スレッドのコメント数を格納することでしょうか。
コメント数でソートできるようにするためですか？


Not Good


The last modified date still looks kind of provisionally. I will add the german date format for the german version.


iteration (axelstudios) - i386-Windows-7-VisualStudio-12: OK (2347 of 2384 tests passed)

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


>  right balance of technical experts

Some partners are not technical experts (e.g. project manager, or business analysts). Can this be rephrased as `right balance of business intelligence experts`, otherwise `right balance of experts`, or something like that?

Thanks for your suggestions @blueclock.

(code comment)


use `sv check iptables >/dev/null || exit 1`


fk type is not available in this table, it is useless to let him and it makes bug deleting an already approved leave. It does no good to leave code for nothing.


Matt, if it isn't too much of a bother, I think it would be nice to have the generic ignores like .DS_Store, builld, *.root and tutorial/bin/,... at the top and the subpackage-specific ignores targeting the subpackages at the end.


add @lru_cache() here?


I don't think this is how you log to socklog correctly, there are cases where socklog is not installed and it could be a different directory with different log daemons.


应该结合上下文语境吧


Ok, thinks


Why informing the use that cancel that he had canceled ? He know since he is the guy that cancel.
We want to inform the guy who validated the leave that leave is no more valid.


Please put this comment, right above the #include global.h in precompiled.h


> Ah, now I understand your question! :)

Not your fault. Looks like lately I'm having trouble getting my point across.

> I think I actually looked through the source of the JDK and realized, that the only way to properly dispose of the resources hogged by openConnection was to call getInputStream.close.

https://docs.oracle.com/javase/8/docs/api/java/net/URLConnection.html states:
> Invoking the close() methods on the InputStream or OutputStream of an URLConnection after a request may free network resources associated with this instance, unless particular protocol specifications specify different behaviours for it.

although that sounds more network related.

The streams returned by `FileURLConnection` and `JarURLConnection` don't have a reference to der parent connection and therefore I see no way how they could clean up stuff there.


Maybe adjust the text accordingly.


Updated so gadmins can manage users that have grpname located in Extragroup


`autoconf` should be in `hostmakedepends`, in cross compilation you want to run it with hosts architecture, `makedepends` installs them with the targets architecture.


take a look at other templates the "default" is to use `3-clause-BSD`


@dmzaytsev correct format would be `@todo #194:30min We need ...`, you don't have hash `#


Because you don't actually want the root of the git repo, you want the root of the npm package. This file uses this.root to find the package.json.

If your npm root is a sub directory of the git repo it will not be able to find the package.json


мб выше using namespace osm;


Do we really want to run `sshguard` as root, or could we have a separate user?


This field is not necessarly an int. In most cases, it is not an int but a char


Please, use close() in one case: when tab is need to be closed.
http://stackoverflow.com/questions/15067107/difference-between-webdriver-dispose-close-and-quit


@vilchik-elena If the code contains a call before the declaration, then the type of the parameter will not be saved, e.g.:
````
function f1(p1) {
	f2("this is a string");
}

function f2(p2) {}
```


This field is not necessarly an int. It is often a char.  "alpha" would be required to sanitize it.


This field is not necessarly an int. It is often a char. "alpha" would be required to sanitize it.


The mail is sent to the one to ask for leave, so I think he wants to know who he canceled the leave.
Perhaps should we add an email to the one who had validated if it is different from the one that canceled.


It's done because the router need to be deployed on the master node(s)


This commit is being reviewed as a part of review issue #157.


I give a better way lol


Why do you need new instance?
Is it possible to re-use existing one?
return this?


I don't understand this PR :
- in the previous code, the 'continue' statement was making a goto to the } bracket, which is anyway exactly why adding conditional brackets can do, no ?
- in this specific PR : it should be 'if (idealTrackP.nActiveHits(true)>2) {'     instead of   'if (idealTrackP.nActiveHits(true)<=2) {' anyway, cause otherwise it has the opposite sense than expected.


These whitespace changes  does not seems correct (from the PEP point of view)


this no longer works after changes from previous PR


Can you provide an example?


It throws on the next iteration because it's an issue if there is a field AT a dynamic offset, not if there is one that CAUSES the next field to be at a dynamic offset (I assume, Gavin correct me if I am wrong)


Starred on *thrust you* :-)

I hope this will be a great plugin, my best wishes for you.


ditto


In the CI machine run only one example file "UnitarySystem_WaterCoils_wMultiSpeedFan.idf" showed diffs in some of the following unitary system report variables.  Diffs are expected in the following report variables of Unitary Systems and Multi Speed Air To Air Heat Pumps:

Unitary System Total Cooling Rate
Unitary System Sensible Cooling Rate
Unitary System Latent Cooling Rate
Unitary System Total Heating Rate
Unitary System Sensible Heating Rate
Unitary System Latent Heating Rate

Other example files may show diffs if they have more than one Multi Speed Air To Air Heat Pumps, and Unitary Systems where these report variables are included in the IDF.


This break a lot of thing.
If you want to submit a last name like
maison de l'ile
it would fails.


Sorry for my english, Google is my friend.


Must be merged with 3971957


use local variable. it is not used in all tests


Has anyone else tested this set of changes?


This line must be removed.
Why does test need new search page?


`null` should be passed as the first argument to the callback if there is no error and the data should be the second argument.


Regarding precedence of the errors, you could argue either way but I think this is probably the lowest precedence error (i.e. you have written something logically correct but we can't support it, rather than higher priority errors that imply you have written something logically incorrect). So checking at the bottom makes sense.


If the user deselects a technology, the link is supposed to go from `font-weight: bold` to `font-weight: normal` when the `visible` class is removed. These lines result in the `visible` class always being present when the technology cost is non-zero, regardless of whether the user has hidden the technology.

| Beta | Dev |
| ---- | --- |
|  |  |


I get no error on my side. Did you git clean -xdf after applying the patch?


Actually, I've read up on `guard`, looks like a good solution. However, if there is a runtime error because we have assumed a null object is not null, what happens?  Surely the test suite keeps running, or does the entire test suite stop?


No. Credits were added to info


Please, add assertion error message.


This commit is being reviewed as a part of review issue #157.


The callback here does not handle the error passed to it.


Please, make test asserts verbose. Add msgs


develop (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1481 of 1481 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)


I think this should be?

    return (viewWidth - drawableWidth * scale) / 2F;


deep copy ?


@elmarco any thoughts on this?


I had thought about it and had decided it's technically not necessary since is_global should never be True for domain-level backends, but I added them back in just now as CustomDeletions for good measure


Please don't break this unnecessary line.


This path maynot exists or be fordibben  to PHP, so next test file_exists may return an error.
May be we can replace the
if (file_exists($lockfile))
with
if (@file_exists($lockfile))


You might want to use an access token instead of sharing your password with the world:
https://github.com/settings/tokens/new
https://developer.github.com/v3/auth/#basic-authentication


Identical action in both clauses: this.pageNumber.get(pageNumber).click();
move it out of if-else block


>Is it really necessary the if-statement if($mode == 'xhtml') {?
Yes, it is necessary. Indeed MathJax's output is not correctly printed.
I close this PR because I've made too much disorder.


does this take into account https://github.com/recastnavigation/recastnavigation/issues/138#issuecomment-169954739 ?

>We were assuming that the NavMesh would not modify the tiles but as it turns out the links are stored inside the tiles and updated each time you swap them in. Since we were reusing tiles across multiple map instances this ofc would lead to problems.


Heh, good idea!


i fail to see your point. rvalue or not you pass a pointer here. There is
no f (g (x)) here either so no gain as that either. Please be more explicit

/me mumbles a bit about c++11 and passing by const ref not beeing the most
effecient way anymore.

See:
http://www.codesynthesis.com/~boris/blog/2012/06/19/efficient-argument-passing-cxx11-part1/

—
Reply to this email directly or view it on GitHub
<https://github.com/notspiff/xbmc/commit/3c92ce2ebf5238cfdeeda5e8befc715f429513d0#commitcomment-15333116>
.


Should fix issue #153


develop (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1468 of 1468 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 (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1468 of 1468 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)


Theres also this;
https://github.com/Homebrew/homebrew/pull/47841


The word *may*, as a verb, in the most common cases, can indicate permission to do something, as in "you may drive my car," or "May I go to the washroom?"; or indicate possibility, as in "the engine may be flooded," "I may have forgotten something", or "It may rain". In the second meaning, it is a synonym for *might*. In both cases, *may not* is the negation, and indicates lack of permission or impossibility. See also: http://www.merriam-webster.com/dictionary/may

>English is really complicated language.

Totally agree with you there. What other languages do you speak?

>BTW, not closing cursor doesn't cause this problem anymore.

That sounds great!


fillableないけど大丈夫？


s/your/you're/


Today, I will prepare for my comming reexamine, I will work hard to make a good result.


Ah, ok, thanks for exaplnation


ok


Hello Garbrielle,
> I don't understand this PR :
>
>  *  in the previous code, the 'continue' statement was making a goto to the } bracket, which is
>     anyway exactly why adding conditional brackets can do, no ?
You're doing 4 different analyses:

pT, pT ideal, p, p ideal

So, if you skip track for pT with continue command, you would
automatically skip all others pT ideal, p, p ideal. P and Pt with the same
value generate tracks with different radius. That's why.

>  *  in this specific PR : it should be 'if (idealTrackP.nActiveHits(true)>2) {' instead of 'if
>     (idealTrackP.nActiveHits(true)<=2) {' anyway, cause otherwise it has the opposite sense than
>     expected.

That's a bug. You're right. I will correct it. Thank you.
>
> —
> Reply to this email directly or view it on
> GitHub.[ALAA2GoKbVnGSDi2h1KOKd8zw7ndmfhSks5pX7DagaJpZM4HBKKJ.gif]
>
>
>


> I also could imagine that the JDK-internal machinery has been improved since JDK6 and this wierd construct here is unnecessary now...

Not sure about this: http://hg.openjdk.java.net/jdk8/jdk8/jdk/file/687fd7c7986d/src/share/classes/sun/net/www/protocol/file/FileURLConnection.java

Seems that `connect` still opens an `InputStream` that needs to be closed explicitly to release the resources as fast as possible.


I see. I changed it :+1:


I gotta read this book when it comes out.


You should always avoid using object types in names. For example `user_array`, `email_method`, `CalculatorClass`, `ReportModule`. In this case `final_colour_array`.


Don't do this.


thanks, I'll keep in mind.


@ezephir There's no `runAllTests()` call, is this intentional?


You may or may not do this, up to you. But it's not needed.


Thanks for the info! :+1:


@Samira64 replace color names with their hex equivalent
eg: `#ffffff` instead of `white`


    ../video/decode/vd_lavc.c:532:5: error: implicit declaration of function ‘avcodec_profile_name’ [-Werror=implicit-function-declaration]
         const char *profile = avcodec_profile_name(avctx->codec_id, avctx->profile);
         ^
    ../video/decode/vd_lavc.c:532:27: warning: initialization makes pointer from integer without a cast
         const char *profile = avcodec_profile_name(avctx->codec_id, avctx->profile);
                           ^
    cc1: some warnings being treated as errors


/admin можно вынести в @RequestMapping над классом


It looks like your test names will always start with `getMilliSecondsShouldReturnMilliSecond`
Maybe it's because It's its job. You can make it shorter by writing `getMillisSecondsShouldConvertValue`


Think I got it: `FileURLConnection.getLastModified()` opens the stream. But we've got special handling of fles now so that can go I guess.


TeamCity pull requests :: netty [Build 1004](https://garage.netty.io/teamcity/viewLog.html?buildId=4621&buildTypeId=pulls_netty&guest=1) is now running


the `FromMilliSecond` part belongs to the When part, for example : `WhenMsecUnit`


The comment you removed below, should stay. Can't you use doxy gen syntax to comment on the declaration? Then it should just be a declaration comment.


So the objectMapper is no more threadsafe.


replace 'white' with `#ffffff`


This works. npm will use git be default and fallback to https :

`git+https://github.com/twbs/bootstrap.git#v4-dev`


when `MsecAmountAsString`


I don't think a builder should take an optional.


you should write `TimeConverter.getMilliSeconds("2 msec");` to make it more readable


Not a big deal, but I noticed that the date here is incorrect.


You make the undeerstanding harder by using variables here.


Any way to leave this as an option? I come from a flat country :P
Love this project!


@nacho noinst_LTLIBRARIES = libsasldb.la, isn't it always static?


Something like [Auth.swift#L911](https://github.com/ably/ably-ios/blob/2bcf5ad29dfa6f246a95a8fd52e51c290b984f4f/ablySpec/Auth.swift#L911)?


記事ではないので、PostsControllerではなく、ThreadControllerの方がよいね。

https://laravel.com/docs/5.1/quickstart-intermediate#the-task-controller

あとは、公式を見るとわかるけど、Controllerは単数名詞の方が良いね。


That'll work too. :smiley:


I am afraid that 200 Nm is a bit low for the Walkman. : D


StoreRequestだと何のrequestかわからないから、ThreadStoreRequestの方がいいね


It's not a given what will happen in 5.1. Just that it'll give JSONAPIRes a
chance to prepare something for us to consider. API mode didn't exist in
4.x, so don't think that's a relevant part of the timeline. Jbuilder will
appear, commented out, in the Gemfile in 5.0 and 5.1 as well.

On Thu, Jan 7, 2016 at 5:19 PM, Ken Ip <notifications@github.com> wrote:

> Fair enough, but that should also be the decision for 5.1. Once you
> "endorse" one in 5.1 this whole argument is invalidated. I just don't agree
> with 4.x -> jbuilder, 5.0 -> nothing, 5.1 -> JSONAPIRes.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/rails/rails/commit/f8edd2043e864aff0bde9289da550709532268a6#commitcomment-15319654>
> .
>


I believe this dependency should also be only for the non-BSD OSes.


there's a unwanted leading space before week


Done.


Is there a reason why `stack install` needs to be used and `stack build; stack exec`don't work?


why


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11022](https://teamcity.mesosphere.io/viewLog.html?buildId=102191&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 212 Build time: 00:08:48


we left those patches alone to ease reviewing process (as I remember Christopher Berner asked for this).

merged with implement `varchar(n)`


This "thoseThat" concept seems to limit genericity (as introduced by the "topic" concept). Unclear if this is an issue, or not (do we need a generic messaging system, or a roboconf-specific one ?)


we left those patches alone to ease reviewing process (as I remember Christopher Berner asked for this).

merged with add mini language


Hi Zbynek,
Ah oki thanks, I get it for the first point.
Yep please update for the second one ;)
Cheers,
Gabrielle


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (946 of 946 tests passed)

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


@awelzel what you describe is pipelining and for synchronous protocol methods it is NOT something that RabbitMQ guarantees is going to work.


Is " +#define EFFECT_CONFIG(author) STE(ReShade/Presets/PRESET/author.cfg)" working correctly?

I couldn't get any effects to load until I renamed the "Default" folder to "Preset"
So instead of .\ReShade\Presets\Default it's .\ReShade\Presets\Preset

Of course I tried to rename the preset in Global.cfg but that didn't seem to do anything either.


Tiens, on mettait à jour le nom ici, avant :P


Indeed, I forgot this one.
The abstract class was built from the initial implementation of the In-Memory messaging.

I fix it on my branch and I will push it later.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11023](https://teamcity.mesosphere.io/viewLog.html?buildId=102205&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 213 Build time: 00:04:30


Hi Martin,
I’m trying to create the pull request, but getting a permissions error when I try to do the git push. Do you need to add me as a collaborator to the project, or is there another way?
My github username is “gerald-curley”.
Thanks,
Gerald.

Date: Fri, 8 Jan 2016 04:35:50 -0800
From: notifications@github.com
To: wicket@noreply.github.com
CC: gerald_curley@hotmail.com
Subject: Re: [wicket] WICKET-5605 Store browser capabilities in local variables in wicket-event-jquery.js (3ba15c9)

It will be easier for me to comment on the diff in the PR.

But I'd suggest you to test InMethod-Grid 7.2.0-SNAPSHOT with all browsers you have against your application.

—
Reply to this email directly or view it on GitHub.


@yoojinkimhong DWMR. Good work here. Keep leaning into the work - post your code to Github and send me messages via GitHub or email marktc@gmail.com


But I don't see how that commit optimizes memory and CPU usage


Thanks. Fixed


More deleted lines than added ones: usually a good thing. ;-)


Oh, nevermind, it was just that stack was catching my +RTS instead of the executable being exec'd.


Modelは、基本的にはDBのtableの1行のデータと捉えておくと良いです。1つのオブジェクトです。

今回、DBにboardsテーブルが有るかというと、無いので、このクラスはModelの本来の目的と異なるような気がします。（コメントに「処理」とあるけど、「処理」はcontrollerにやってもらうほうがいい）

他のModelクラス（ThreadやComment）に、
https://laravel.com/docs/5.0/eloquent#relationships
で書いてあるようなRelationを定義してあげることで、コードがすっきりします。
この辺りの処理も、便利にLaravelがやってくれます。ぜひ、挑戦してみてください。


This way, Boot will use defaults instead of being fixed to a particular version of Boot/Clojure as new versions come out.

These versions can & should be set in a project's Dockerfile or boot.properties file.


Hi,

I'm planning to work on extending your Gist. I will focus on the real
values first and make some plots for fixed `z`, as a function of `a` and
`b`. (Probably for say 5 logarithmically spaced `z` values.) Getting an
implementation that works well for the reals seems like a sensible step:
the current hyp1f1 only works for real `a` and `b` anyway, so this will be
useful even if we never crack the complex case.

But first, I'll compute some reference values. In your script you do it on
the fly using mpmath, but when working on Bessel functions I've discovered
this if not satisfactory. The trouble is that the values mpmath returns
when using default precision (which is only 15 dps) are often wildly off.
At "difficult" points you need hundreds or even thousands of digits for the
values computed to converge to double precision, and these computations
take a long time.

On Fri, Jan 8, 2016, 12:24 AM Josh <notifications@github.com> wrote:

> @tpudlik <https://github.com/tpudlik>, @nmayorov
> <https://github.com/nmayorov>: ya know, we should probably communicate on
> what we're working on so that we don't end up overlapping. Before I fizzled
> out I was working on implementing the rest of the recurrence relations from
> the paper using Olver's algorithm... I can keep working on that if nobody
> objects?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/tpudlik/scipy/commit/9becd51a1d7f7983b54a5e06ddc515574574c7c2#commitcomment-15332492>
> .
>


Heu... This was fixed in a subsequent commit.


Ok, my response is in the next commit.


Fantastic resource, thanks!


They are indeed correct from a PEP8 point of view :-) (item 4 in [there](https://www.python.org/dev/peps/pep-0008/#other-recommendations))


设置 `getWithoutSet` 选项为true时，以下模式被认为是没有问题的：


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


this should not be indented.. its unrelated to the `devise_for` line above


You have to fork the repo into your account.
Then push the changes to your fork.
And finally create a PR from your fork to the original repo.


还是拨号出去的搜索不到网络上的归属地啊～～～只能打进来的才能收到吗？


nice.. good thought on useful information to display


Could be `input_width or self.width` if you wanted


What's this doing?

The purpose of having a one-pass was that we only go through each (x, y) once for every field. This all() turns that into O(n^2). Is there another way?


LGTM!


ok


Thanks for the comment Paul - took a bit of playing but I did manage to
inject some arbitrary script and have the service execute it.

To be honest there are a whole bunch of security concerns that need to be
considered for this, but I've added a github issue for this one.

Sadly I don't think there is any pre-rolled input validation for powershell
script, so I'll probably write something simple initially with some Pester
coverage that can be used to develop it as and when more scenarios are
incovered.

Cheers,
James.

On 4 January 2016 at 15:38, Paul Marston <notifications@github.com> wrote:

> Is this a potential vulnerability? As far as I can see the scriptArguments
> are written directly to the script file from the HTTP request so someone
> could escape out of the current command and run arbitrary commands?
>
> Is there a PowerShell cmdlet to escape/encode special characters?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/perceptile/ScriptServices/commit/7f81863d18a3e9092bab704a490d5dd129a62f80#commitcomment-15245328>
> .
>

--

e:  james@readsource.co.uk
m: +44 (0)7909 990497
t:  @James_Dawson
l:   http://uk.linkedin.com/in/jedawson


This is not at all simplified. You have two definitions going on here, and they're both hard to understand. I propose:

A closure is function that closes over its environment. It has access to the state of the environment, but the variables inside the closure is private.

Example:

    (function (){
      var scopeVar="Hello";

      (function closure(){
        var closureVar=" World";
         console.log(scopeVar+closureVar);
      })()
    })();

scopeVar is accessible inside the outer and the inner function, but closureVar is only accessible inside the inner function.


エラーメッセージがThreadController（PostsController）と逆になっていない？


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


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


Bah, don't comment on old code. In next version of this pull request it is already fixed


ditto


Done,

thank you for noticing the issue! New pull request created.

  Zb.

On Fri, 8 Jan 2016, ghugo83 wrote:

>
> Hi Zbynek,
> Ah oki thanks, I get it for the first point.
> Yep please update for the second one ;)
> Cheers,
> Gabrielle
>
> —
> Reply to this email directly or view it on
> GitHub.[ALAA2L6PkWxXtstEvVHXU8jlo0mZxATAks5pX7Y7gaJpZM4HBKKJ.gif]
>
>
>


timeLimit is always 500ms, so that check is unecessary


It is not generic. It makes sense with the Roboconf usage of the messaging.
A generic messaging system makes no sense for Roboconf. The ThoseThat enumeration helps us to understand whether we target those that import variables or those that export ones. This is really useful to debug messaging when considered from a business angle (in the DM or in an agent).

And it is a convenient method to build a topic name.
But is not exclusive either.


CommentController用のエラーメッセージだと思う。


Personally, I think this is somewhat clearer


@mortonfox whoops, my bad-- thank you!


@pjvohra Something is not quite right here. The commit is showing a subproject - which is a valid technique fro some work - but not for the work in the foundation. Continue with your work and we'll sort this out when we talk next.


Just a few nit-picks:
Would it be possible to call reserve_memory() from
low_mem_init() (or from something else, but mapping_init()
looks like a bad choice)?
And maybe it can handle all cases, including the splitted ones?
Otherwise the DPMI memory init is scattered into 2 places,
which is an asking for problems when the code is to be changed.
You just need to set dpmi_base there, and dpmi_alloc_pool()
will know what to do with it.


(I would agree ;)


@taylorotwell I've come across an issue as a result of this implementation and just wanted to check.

Previously you were able to catch exceptions in Middleware `handle` requests, with an [example here](https://github.com/lucadegasperi/oauth2-server-laravel/blob/master/src/Middleware/OAuthExceptionHandlerMiddleware.php).

This now forces Exceptions to always be handled either at a lower level than the routing pipeline stack or inside of the global Exceptions Handler.

Is it intended in the development of 5.2, to stop Exceptions being captured in the Middleware `handle` methods?

Thank you


Weird class name !


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (930 of 930 tests passed)

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


Because `ARTChannel.history` can throw an error.

Method signature: `ARTChannel.history(query: ARTDataQuery?, callback: ARTErrorCallback?) throws`

The method can throw an `ARTDataQueryError` type:
```
ARTDataQueryError.Limit
ARTDataQueryError.TimestampRange
```

So I need to call the method with the `try` keyword: `try channel.history(query, callback: nil)`


No pb ;)
Cheers,
Gabrielle


isn't timebank updated automatically? I thought it was... :/


那个预编译的都可以呀 发自网易邮箱手机版 在2016年01月08日 21:57 ，秋叶随风ivan写道： 还是拨号出去的搜索不到网络上的归属地啊～～～只能打进来的才能收到吗？ — Reply to this email directly or view it on GitHub.


Yeah it was intended that if something goes wrong all middleware would
still receive an HTTP response. How were you using try / catch in your
middleware?

On Fri, Jan 8, 2016 at 8:01 AM, Matt <notifications@github.com> wrote:

> @taylorotwell <https://github.com/taylorotwell> I've come across an issue
> as a result of this implementation and just wanted to check.
>
> Previously you were able to catch exceptions in Middleware handle
> requests, with an example here
> <https://github.com/lucadegasperi/oauth2-server-laravel/blob/master/src/Middleware/OAuthExceptionHandlerMiddleware.php>
> .
>
> This now forces Exceptions to always be handled either at a lower level
> than the routing pipeline stack or inside of the global Exceptions Handler.
>
> Is it intended in the development of 5.2, to stop Exceptions being
> captured in the Middleware handle methods?
>
> Thank you
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/laravel/framework/commit/f5f843396c2c3cd244334029847fee2c7809123d#commitcomment-15339930>
> .
>


要无线 发自网易邮箱手机版 在2016年01月08日 21:57 ，秋叶随风ivan写道： 还是拨号出去的搜索不到网络上的归属地啊～～～只能打进来的才能收到吗？ — Reply to this email directly or view it on GitHub.


ok


@drod4 DWMR. Good work - you are shipping code. :articulated_lorry: :+1:


Would you care submitting a PR to simplify the entry? :)


Hi, I was wondering if you could tag this release to allow Bower to download and cache it? I'm having problems at the moment using the 0.3.10 tag which is still hard coded to look for the specific version of timezone-js


If this can indeed now mount at any arbitrary path (I've not used Dokan for years), maybe all the `drive` variables should be renamed to `path`, to make the code more readable?
And I guess `self.path = self.drive + ":\\"` now looks a little suspect?


Of course, would be happy to :)


Paris???


確認しました。特にコメントはありません。


Is it safe to uninit lib before the thread is actually stopped? Maybe it's better to uninit after the thread has been stopped


ill test


I also updated the website, so it's live here: http://www.pjrc.com/teensy/gui/


Aside from the confusion that comes from the title of this PR and what it actually does (512 vs 256), this looks solid.

![well_done_sir](https://cloud.githubusercontent.com/assets/12858421/12199573/03220c52-b611-11e5-94df-a84d1c613b7c.gif)


再度書いておくけど、ThreadControllerの方がいいね。
「aab」が何を表現をしているのかがわかりづらいね。

BBSかどうかはドメインもしくは、サブディレクトリで判断できるので、下記が良いと思う。

```
Route::resource('thread', 'ThreadController');
```


Was hat das mit PHP zu tun? Und braucht man das 2016 immer noch?


... or perhaps something more unique like `mountpath`, given all the many other variables named `path` ;-)


this is a new information


Do not export templates


its fix but now this spams console

Warning: fwrite() expects parameter 1 to be resource, integer given in /home/epicsteve33/Desktop/Servers/KingdomCraft/1/src/pocketmine/utils/MainLogger.php on line 262


Indeed. I fixed it and will push it later to my working branch.


But as a test is this not odd.  It should not error, it should pass, and if it fails, then we want to see the error surely?


Hi @taylorotwell As per the example link above.  In my example I had:

```
try {
    return $next($request);
} catch (ExampleException $e) {
    // Json response code
} catch (Exception $e) {
    // Json response code
}
```

This was useful in middle-ware as it allowed you for example to have a "APIMiddleware" which would return exceptions as JSON responses within a API route group, but for other areas of the routing (such as "web" below), the standard HTTP error response system.

```
// api and web being middle-ware groups
Route::group(['middleware' => ['api']], function () {
 // Api routes
});
Route::group(['middleware' => ['web']], function () {
 // Web routes
});
```

Hope that makes sense?

Cheers


The change causing all this work is that empty multichoice fields are now stored as `None` rather than `[]` (so that missing data searches work the same for all field types).

For the code here, since we want the value to be `None` we shouldn't change it to `[]`.

I've been working to find all the places that assume the value of a multichoice field is a list. If you know of others please let me know.


@davidworn DWMR (done with my review). LGTM (looks good to me). KOSC (keep on shipping code). :+1: :articulated_lorry:


Should we merge Heather's PR first? I think it'd be easier for me to merge in her stuff than other way around. Plus she found some invalid HTML.


This does not compile on linux.
`crf.addPairwiseBilateral( 80, 80, 13, 13, 13, im, new MatrixCompatibility( Eigen::MatrixXf::Identity(M,M) ) );`
does


;16/01/08
;introduction

                here is nothing
      thanks!


It is incorrect to remove the block here. `locker` needs to be destroyed before `lock.lockForWrite` is called.


Thanks! Just one issue under linux otherwise all good.


Well, I meant to _make_ it fatal, and redirect
to leavedos(). Do you think we need the current
funny handling?
But I applied your patch as this question is out
of the scope. I just think we don't need that strange
handling of SIGQUIT.


see 4cce9a0


Hey Matt,

I'm making a patch in next Laravel 5.2 release to allow you to do something
similar. Basically we will set the $exception property of the response when
we catch an exception in the pipeline.

This will allow you to do something like:

```
if ($response->status() === 500 && $response->exception) {
     //
}
```

On Fri, Jan 8, 2016 at 8:11 AM, Matt <notifications@github.com> wrote:

> Hi @taylorotwell <https://github.com/taylorotwell> As per the example
> link above. In my example I had:
>
> try {
>     return $next($request);
> } catch (ExampleException $e) {
>     // Json response code
> } catch (Exception $e) {
>     // Json response code
> }
>
> This was useful in middle-ware as it allowed you for example to have a
> "APIMiddleware" which would return exceptions as JSON responses within a
> API route group, but for other areas of the routing (such as "web" below),
> the standard HTTP error response system.
>
> // api and web being middle-ware groups
> Route::group(['middleware' => ['api']], function () {
>  // Api routes
> });
> Route::group(['middleware' => ['web']], function () {
>  // Web routes
> });
>
> Hope that makes sense?
>
> Cheers
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/laravel/framework/commit/f5f843396c2c3cd244334029847fee2c7809123d#commitcomment-15340112>
> .
>


github


How to get stringer using in golang 1.4 after this commit? I'm using
```bash
go get golang.org/x/tools/cmd/stringer
```
It happens on clean environment without stringer previously installed
```bash
docker run --rm golang:1.4 go get golang.org/x/tools/cmd/stringer
package golang.org/x/tools/cmd/stringer
	imports go/constant: unrecognized import path "go/constant"
package golang.org/x/tools/cmd/stringer
	imports go/importer: unrecognized import path "go/importer"
package golang.org/x/tools/cmd/stringer
	imports go/types: unrecognized import path "go/types"
```

```bash
docker run --rm golang:1.5 go get golang.org/x/tools/cmd/stringer
#works without errors
```


I should note the $exceptoin property will contain the full Exception instance.


@kieranjol Yeah!! I really like Font Awesome but figured a native emoji could do better than a full library for just one symbol. If you have a preferred emoji, I welcome customization. :sunglasses: I tried the computer emoji first (and a few others) but it didn't suit the black background.


Please use "const unsigned char*" as it's a read-only param.


Same issue here with the last commit - you just want `with('categories')` instead of `getEager()` here I think. `getEager` is only if you need *every* relation on the object. You also don't need to redeclare $doc, you already have a perfectly workable instance, just add the `->whereHas` clause to the existing object.


good point, this is definitely unfinished, I'll refactor it...


"unshifted": 123


Pass "s" directly instead of casting away the const ...


OK, to get 1=INFO, 2=WARN, 3=ERROR, 4=CRITICAL
Setting to 1 also includes 2, 3 and 4.
Setting to 2 includes also 3 and 4
Setting to 3 also includes 4
would need the following changes to EmonLogger.php (and of course the settings files updated)

    public function info ($message){
        if ($this->log_level = 1) $this->write("INFO",$message);
    }

    public function warn ($message){
        if ($this->log_level <= 2) $this->write("WARN",$message);
    }

    public function error ($message){
        if ($this->log_level <= 3) $this->write("ERROR",$message);
    }

    public function critical ($message){
        if ($this->log_level <= 4) $this->write("CRITICAL",$message);
    }

Just tested this and it works fine, however are we agreed that we don't need a log_level =NONE as it duplicates $log_enabled = false;

I've not committed the above - I think we need a consensus first :)

Paul


Categoryに対してもindex、showが必要になる可能性があるなら、CategoryControllerを作ってそこで管理しても良いと思う。


Ubuntu 14.04 LTS, which has python 2.7.6 (which serializes stuff a bit different in each version, I've heard).


ok


Same here, don't case away the const ... Better update utf8_check to accept a const pointer.


Sometimes I like to put an empty line between options if they have sub-options, like between `roles:` and `tasks:` here.


done


You should adjust also the comment above.


Maybe extract the passphrase to a constant?
Besides, mostly single quotes are used, but here there are double quotes, is there a reason for that?


It might be worth to add in the commit message that the code snippet is free to be used in MIT licensed projects.


You should have adjusted the comment above, but don't do it. Instead bring back the old name, as it is more precise. Use that other name in the .cpp file if you need to have identical names.


This looks great, I would like to see more information about your option ordering scheme like what we talked about yesterday; name, tags, module defn, loop, then options.


I think in order to be more explicit about how we use Ansible like @waded and I were talking about yesterday, we should use long option names, so instead of `-K` we should use `--ask-become-pass`.


Feature req:
Fallback station(s) with time to change back to primary stations


@taylorotwell sounds interesting, although when the new code above fires, it never returns back to the catch block of the middleware, due to it immediately outputting the error, as [shown here](https://github.com/laravel/framework/blob/c61d51a76f41cdbc8c72360b3516ffb7c18821bb/src/Illuminate/Routing/Pipeline.php#L80).


// FIXME: This is a messy way to do it


// FIXME: Catch an error here and rollback model if it fails


Please take another look.


I vote yes


This is handy, and should be included in all the projects/canvas implementations


The according changes in test_parse.expected seem missing, or?


You won't use a try / catch. It will look something like this in your middleware:

```
$response = $next($request);

if ($response->status() === 500 && $response->exception) {
     switch (get_class($response->exception)) {
          case ExceptionOne::class:
                return response();
          case ExceptionTwo::class:
               return response();
     }
}

return $response;
```


Thanks


well caught !


Why is this necessary? Can't we make the export smarter? Alternatively, just put the friend class TreeManager somewhre else, visibility doesn't matter.


There doesn't seem to be a matching iconv_close. Even if this is a test that's going to exit soon anyway it would be nice to not leak resources.


@adrienmo Yeah, we saw tangible improvement (confirmed with `fprof`) when switching to NIF for crc16 hashing.  We had 2 other improvements that made a difference for us:
1. Use a tuple for the slots instead of a list: https://github.com/layerhq/eredis_cluster/commit/c663082b6d22cacb4525ca3498870a042793c497
2. Use a tuple for the slots maps as well: https://github.com/layerhq/eredis_cluster/commit/655e8400e7eff72d4770cf0b98728e5d8a07e090

The combination of all of the above put us in better shape.  /cc @janin


Do you find that funny, or why did not you github to fill in a position yours? How is the the "Arter97-Skyhigh Hacker Mix kernel"? Do not you think that it arter97, UpInTheAir and I deserve to be mentioned by you? What I find funny that you use every day and my UpInTheAir change and we no longer mention? I wish you a lot of fun when copying "Russian Copy Cat" and if you're not so happy can upload your code here ;-)
Oh yes, I have done V11. The codes I ask you to release available so you have to re-new features in your kernel ;-)

Findest du das witzig oder warum bist du nicht in der Lage dein github zu füllen? Wie läuft den der "Arter97-Skyhigh-Hacker-Mix-Kernel"? Denkst du nicht das es arter97, UpInTheAir und ich verdient haben von dir erwähnt zu werden? Was ich auch lustig finde das du jeden Tag UpInTheAir und meine Veränderungen verwendest und uns nicht mehr erwähnst? Ich wünsche dir weiterhin viel Spaß beim kopieren "Russische Copy Cat" und falls du keine bist kannst ja gerne deine Codes hier hoch laden ;-)
Achja ich habe V11 fertig. Die Codes stelle ich dir den zum Release zur Verfügung damit du wieder neue Funktionen in deinen Kernel hast ;-)

Thread found on http://4pda.ru/


добавь пожалуйста перенос строки в конец


Can u add multiwindow with popup view i mean floating multiwindow popup
that every new samsung phones does.
On 8 Jan 2016 7:00 pm, "Luka Galjot" <notifications@github.com> wrote:

> No. Credits were added to info
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/TW-LL-msm8226/ROM/commit/3a8f423a6b6d44422da693b54ae267e8605a6215#commitcomment-15339358>
> .
>


Can I ask why you're not able to put this code in your exception handler? That would stand out as a cleaner solution to me.


should be fixed with f4e4fe0577a3ac90b3f0f666e92647662753e931


@darkestpriest please notice that the networkservice file in spanish will be removed from main repo ( i will keep a local copy just in case) because fermat policy about documentation is to have it only in english. thanks for the update !


This is not supposed to be changed, only the modules list should change in this pom.


Hmmm, in order to not break backwards-compatibility, I wonder if it'd be worth leaving the current `mount` function which just takes a single drive-letter, and then adding a new `mountpath` function which takes an arbitrary full-path? ( with `mount(drive)` then effectively being implemented as `mountpath(drive + ":\\")` )


I guess there went something terribly wrong with this commit -> woulrd point to
fs_join -> fsjoin which is not the same like fs.join which I guess you ment?

Because now my filesystem looks like this

---------------
> root@server:/srv/pyload# ls -1
accounts.conf
files.db
files.db-journal
files.version
logs
plugin.conf
pyload.conf
pyload.pid
scripts
tmp
unrar_passwords.txt
userplugins
('userplugins', u'accounts', u'FilecloudIo.py')
('userplugins', u'accounts', u'OverLoadMe.py')
('userplugins', u'accounts', u'PremiumizeMe.py')
('userplugins', u'accounts', u'ShareonlineBiz.py')
('userplugins', u'captcha', u'AdsCaptcha.py')
('userplugins', u'captcha', u'AdYouLike.py')
('userplugins', u'captcha', u'CircleCaptcha.py')
('userplugins', u'captcha', u'ReCaptcha.py')
('userplugins', u'captcha', u'SolveMedia.py')
('userplugins', u'crypter', u'DlProtectCom.py')
('userplugins', u'crypter', u'Go4UpCom.py')
('userplugins', u'crypter', u'MediafireComFolder.py')
('userplugins', u'hooks', u'AntiStandby.py')
('userplugins', u'hooks', u'AntiVirus.py')
('userplugins', u'hooks', u'ClickNLoad.py')
('userplugins', u'hooks', u'ExternalScripts.py')
('userplugins', u'hooks', u'ExtractArchive.py')
('userplugins', u'hooks', u'HotFolder.py')
('userplugins', u'hooks', u'ImageTyperz.py')
('userplugins', u'hooks', u'LogMarker.py')
('userplugins', u'hooks', u'UnSkipOnFail.py')
('userplugins', u'hooks', u'UpdateManager.py')
('userplugins', u'hoster', u'BitshareCom.py')
('userplugins', u'hoster', u'DailymotionCom.py')
('userplugins', u'hoster', u'ExtabitCom.py')
('userplugins', u'hoster', u'FileserveCom.py')
('userplugins', u'hoster', u'FilesMailRu.py')
('userplugins', u'hoster', u'FreakshareCom.py')
('userplugins', u'hoster', u'Ftp.py')
('userplugins', u'hoster', u'HighWayMe.py')
('userplugins', u'hoster', u'LetitbitNet.py')
('userplugins', u'hoster', u'LinksnappyCom.py')
('userplugins', u'hoster', u'OboomCom.py')
('userplugins', u'hoster', u'OneFichierCom.py')
('userplugins', u'hoster', u'OverLoadMe.py')
('userplugins', u'hoster', u'PremiumizeMe.py')
('userplugins', u'hoster', u'RapidgatorNet.py')
('userplugins', u'hoster', u'RapiduNet.py')
('userplugins', u'hoster', u'RealdebridCom.py')
('userplugins', u'hoster', u'RyushareCom.py')
('userplugins', u'hoster', u'ShareonlineBiz.py')
('userplugins', u'hoster', u'UgouploadNet.py')
('userplugins', u'hoster', u'UpleaCom.py')
('userplugins', u'hoster', u'UploadedTo.py')
('userplugins', u'hoster', u'UpstoreNet.py')
('userplugins', u'hoster', u'WrzucTo.py')
('userplugins', u'hoster', u'XDCC.py')
('userplugins', u'hoster', u'ZippyshareCom.py')
('userplugins', u'internal', u'Account.py')
('userplugins', u'internal', u'Addon.py')
('userplugins', u'internal', u'Base.py')
('userplugins', u'internal', u'Captcha.py')
('userplugins', u'internal', u'CaptchaService.py')
('userplugins', u'internal', u'Container.py')
('userplugins', u'internal', u'Crypter.py')
('userplugins', u'internal', u'Extractor.py')
('userplugins', u'internal', u'Hoster.py')
('userplugins', u'internal', u'misc.py')
('userplugins', u'internal', u'OCR.py')
('userplugins', u'internal', u'Plugin.py')
('userplugins', u'internal', u'SevenZip.py')
('userplugins', u'internal', u'SimpleCrypter.py')
('userplugins', u'internal', u'SimpleHoster.py')
('userplugins', u'internal', u'UnRar.py')

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


You're right. I don't know how I tested that to assume that was possibily replacing a symlink by a file. The only good way to do it is to replace ensure => present by ensure => file. Sorry for that.


Thanks!


https://github.com/GrahamCampbell/Laravel-Exceptions is also clever enough to look at the acceptable content types and can return json responses if the client doesn't understand html but does understand json.


@michaelklishin , ok thanks. I understood wrongly then. I also just came across the following older mailing list post about exactly the same topic.

https://groups.google.com/forum/#!msg/rabbitmq-users/4xLzJxWLz58/2hnp6N4jr38J

Interesting.

Firing off many ```nowait``` operations doesn't cause issues I hope ;)


I was also thinking about this, but was too lazy for now...


For simplicity you could just write:

char *in_buf = "JürgenPC";

This would create a static string on the heap where in_buf points to.


What's the semantics here?
Is it: `docker network connect --link <container on network>:<alias>  <network> <container>`?


The code is already on master. I have fixed the tests by adding the jasypt bundle.


@taylorotwell sounds great, thank you.

@GrahamCampbell, I've done that as a temporary work out. However as said above, being able to separate out handling of errors in middle-ware in a modularised (and large) system is useful.

Also as I've linked there are many third party packages which rely on being able to do this - the OAuth one as an example.


für das hani eigentlich dich agstellt @jheitz !


Hum, I think that was changed with the gephi:migration maven goal.


fixes #40


I find this name to be imprecise and confusing. What does it mean for a protocol to be connected?

`received_connect_frame` would be more to the point.


I think it might be the squared weights which need to sum to 1


Hast du mal versucht das '=' zwischen Cookienamen und Cookiewert encodiert anzugeben also als '%3D'?
PS:
Ich habe mal bei einem OK Link, der einen Wert für 'vdsig' enthielt diesen einfach entfernt und kein Cookie übertragen. Ging trotzdem k.A. ob das immer so ist.


This now fails silently if no key can be found. I think it would be more appropriate to raise in that case. I also think that if multiple keys are found, this should also be considered an error.


Your welcome @gustl-arg :+1:


I seem to have reverted the Ember version from 1.13.11 to 1.13.10. I'm pretty sure I did a `npm install`.


Should this really be in BenchExec? The `[[ ]]` notation is quite ugly and non-standard.

Is there a better solution that could be implemented? For example, it would make more sense to have a separate row in the table for the witness validator options of SV-Comp, wouldn't it?


Eu ja tinha ateh escrito no meu blog mano foi mal


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11024](https://teamcity.mesosphere.io/viewLog.html?buildId=102210&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 212 Build time: 00:07:16


My version is in an unstable mode for the moment, some changes will follow ASAP.


This seems strange to have in HostConfig since aliases are supposed to be scoped by network.


When you have main pages with subpages visitors will often nog click on the main page. And webmaster might not even set content for it. What can we do? Some sort of auto filling that main page with a sitemap for that tree? And extra SPM options for filing the content and creating subpage as a repeater for the main page?


Btw. The tests should not need the external databases by default. Maybe it is because of the system you build on. Best use a linux system.


please adjust the comment above


TeamCity pull requests :: netty [Build 1003](https://garage.netty.io/teamcity/viewLog.html?buildId=4616&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 6365, ignored: 23 Build time: 01:24:10


travis does not support windows, and fails due to this line. usr appveyor for windows testing.


If it fails? In this case, the `try! channel.history(nil)` will not fail. The query is created inside the `history`.


![whole](https://cloud.githubusercontent.com/assets/4419138/12200018/060e95a2-b625-11e5-9730-a4964fe6a63d.png)

bu şekilde laravel 5.1 hatası alıyorum. galiba indirdiğim sürüm laravel 5.2, yeniyim. yardımcı olur musun. Teşekkürler


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


The test would pass if `AttributeConverter` was totally ignored. You need something more like a boolean ensuring the converter code is reached.


@joesb, is it ok to publicise this?


adjust the comment above


Adjust the comment above.


Aah. Oops. Doh!


Maybe this could be dynamic so you dont have to update every year?


Thank you for mentioning. Changed....


@smcv

We can both read between the lines, but I still want to say it explicitely: thanks !


good call. i just made a PR to fix. #11090


> Also as I've linked there are many third party packages which rely on being able to do this

Yeh, though this is a documented breaking change in 5.2.


oh, I though it was already used for unions of benchmarks with different options.
What is used in that case? A simple comma?
Then I would agree, that it would be nicer to have a separate row for the witness-options.

PS: non-standard? Do we have a standard for our tables? ;-)


I think you can leave this line out


`clientOptions` is intended to be a wrapper for the cassandra-driver config. see http://docs.datastax.com/en/developer/nodejs-driver/2.2/common/drivers/reference/clientOptions.html . There is no `keyspace` in there.


I think the pull request has been submitted now.

G.

Date: Fri, 8 Jan 2016 05:57:04 -0800
From: notifications@github.com
To: wicket@noreply.github.com
CC: gerald_curley@hotmail.com
Subject: Re: [wicket] WICKET-5605 Store browser capabilities in local variables in wicket-event-jquery.js (3ba15c9)

You have to fork the repo into your account.

Then push the changes to your fork.

And finally create a PR from your fork to the original repo.

—
Reply to this email directly or view it on GitHub.


"Build tools and processes to manage and monitor STAQ's [integrations-platform-as-a-service](https://en.wikipedia.org/wiki/Cloud-based_integration)"


Here you changed the name of the field in the constructor list without changing the name of the field in the declaration.


yes, even more ;)


"...using RDBMS and NoSQL datastores"


KIA


yeah.. maybe I will check again


This is already changed in later commits.


I like it! few tweaks and it'll be ready!


ok


@GrahamCampbell where can I find that message? It's not in the upgrade guide.


QK knitting costs
COGS - Dec 15
COGS - agree matrix to go forward
Budget 1st 1/2 2016
Stock T/o by dept - importance / relevance
Monthly S.J. Invoices - Admin?/Dept?/KD?/LD?
Departmental focus - Pareto of business make up
Prioritise Priorities
DNS - Lansdown / Olorun
Lansdown future development / staff
Ongoing marketing - ? or Guerilla?
Business Overview doc - CW


This variable was removed in the progress of commits.


Debug code.


$post->comments()->count()で件数は取得できるね。
また、この場合だとコメント削除の機能が出来た時に、decrement()もしないといけなくなるね。
もし、このパターンをやるなら、save()した後にincrement()すると、save()が成功してincrement()がDB接続に失敗した場合、データの不整合がおきるからトランザクションを張った方がいいね。

```
DB::transaction(function () use ($comment) {
                $post->comments()->save($comment);
                $post->increment('comment_count');
});
```


My mistake, shouldn't have just copypasta'd from the `getDocs` method above. Fixed in fef1f2e.


Почему-то не нашёл, спасибо.


:laughing:


Lets chat during the standup. My point is that these are tests where we expect the code path to succeed.  So accessing attributes with `?` and calling `try!`, unless I have misunderstood their meaning in Swift, means that we are building in conditions that will cope with failures such as the object being null in the case of `?` or the method raising an exception in the case of `try!`.  However, this is a test, so why are we assuming or expecting this not to work ever?


develop (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1484 of 1484 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)


I agree. My bad for not looking at this properly and merging it. Sorry


// FIXME: This is a messy way to do it


// FIXME: Catch an error here and rollback model if it fails


// FIXME: Regenerate if it expires?


* TODO: Refactor the validator to reduce duplication, maybe move the askWithValidation to an external library.


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


This commit is being reviewed as a part of review issue #158.


TeamCity pull requests :: netty [Build 1004](https://garage.netty.io/teamcity/viewLog.html?buildId=4621&buildTypeId=pulls_netty&guest=1) outcome was **UNKNOWN**
Summary: Canceled (Tests passed: 5954, ignored: 23) Build time: 00:56:16


TeamCity pull requests :: netty [Build 1005](https://garage.netty.io/teamcity/viewLog.html?buildId=4624&buildTypeId=pulls_netty&guest=1) is now running


Yes, unfortunately ObjectWriter has no valueToTree method.
As commented in this method, we could serialize and then parsing as tree, but I don't know if it is more efficient than instantiating ObjectMapper each time.


Todo: use generic comparison function


develop (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1484 of 1484 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)


This code was refactored into the ForkEnvrionmentScriptResultExtractor. That was in the process of committing.


May want to update comment to reflect `:limit => 5000, :period => 1.hour` setting (current threshold per IP).


We wszystkich nowych plikach licencje stosujmy wg wzoru z FreeBSD (oczywiście z nowym rokiem i Marvell zamiast Stormshield).


I noticed an improvement in memory usage of the whole system with this line enabled, and our Xperia V definitely isn't a phone with a lot of ram. :)


This was already removed.


done later


Doh! Sorry, I missed that. Thanks!


> Yeh, though this is a documented breaking change in 5.2.

Where was it documented? It's not in the upgrade guide.

I'm not sure I understand the advantage of checking `$response->exception` normal exception handling?

```php
$response = $next($request);

if ($response->status() === 500 && $response->exception) {
     switch (get_class($response->exception)) {
          case ExceptionOne::class:
                return response();
          case ExceptionTwo::class:
               return response();
     }
}

return $response;
```

vs

```php
try {
    return $next($request);
} catch (ExceptionOne $e) {
    return response();
} catch (ExceptionTwo $e) {
    return response();
}
```


hmm, `parent` is more precise than `item`, why did you change this?


Extra space around `li`.


管理したくないファイルは下記で制御できるよ。

```
laravel_aabChallenge/.git\info/exclude
```

+ 参考サイト
http://tbpgr.hatenablog.com/entry/20130610/1370873259


Koji also supports plain login with username and password. While you're at it, maybe you could also add it there, so we suport all auth methods?


Finde ich gut!
# Klasse


No, for unions of benchmarks the syntax is the same as in the other header cells: `[...; ...]`


If there is no selection, the view.GetSelection() method treats anchorLine<->endLine and anchorCol<->endCol as the same number.  It just begins and ends in the same place.  So this allows me to do the check for single line or selection of lines.


тобі потрібно реалізувати підтримку роботи цього методу.


@kakoni I'm aware. I hope I was able to address your concern in https://github.com/Homebrew/homebrew/pull/47841#issuecomment-170010437.


Do you think there is a way to have backward compatibility for Pyhon2?

You know, There is no distro which comes with Python 3 yet.


+1 damit blinken die roten lampen nicht mehr (ist auch in mein repo)


jaja, erscht mal fotis...

Am 8. Jan. 2016 15:25, um 15:25, Jonas <notifications@github.com> schrieb:
>für das hani eigentlich dich agstellt @jheitz !
>
>---
>Reply to this email directly or view it on GitHub:
>https://github.com/felberj/alpler/commit/e82b823e13ddbc2b2a62de42181a4db3c96f7efa#commitcomment-15340443


Does that work?  Now you're not attaching the relation at all.  I mean, great if it does!


resolved later


>_<  Yes, it absolutely should be.


@mattheworiordan The good thing about `guard let` is precisely that you decide what happens on error, explicitly. (I tend to like explicitness, especially on control flow.)

```swift
guard let result = result else {
    // result is nil; fail test with error
    return // or continue, etc.
}
// happy path here; result is not nil
expect(result.hasNext)...
```


@ricardopereira Right.


What makes this read an `acquire` in your eyes? I would put `memory_order_relaxed` here, because a caller will not write to this location. Since the caller will branch on the result of this function for dependent load/store operations, there's also no write-reorders one might want to prevent.


I see that you made the other change later, but things like this should always be in the same commit.

Generally, unless you have a very very good reason (can't think of one), the code should *always* compile after each commit. Obviously a good way to achieve this is to change code, compile, commit and repeat, instead of change code many times, compile once, manually segment the code in many commits at this point and make each one.


Initial Commitついてアドバイスしますが、今回については、対応は特に必要ないです。
Initial Commitは、開発前のLaravelのソースを最初にコミットしてから、開発分をコミットすると差分が出て何を実装したかがわかりやすいので、今度新規に作る時はそのようにすると良いです。


좋은 리뷰 감사합니다!

MakeEmpty() 함수의 실습 출제 의도는 리스트에 노드가 많이 있을 때 헤더를 남기고 초기화시키는 역할을 하는 것입니다. 명재씨가 구현한 MakeEmpty함수의 경우 처음 초기화를 위해서 만들어지는 것이라면 괜찮지만,
만약 2번 이상 불러올 때 같은 위치에 다시 메모리영역을 할당해주도록 코드가 짜져 있어 아마도 에러가 발생할 수 있을 것입니다.

지적해주신 부분은 조금 더 개선하겠습니다 :-)


Extra space around `a`.


The branches on the result of this load, so why put an acquire here? Atomicity is sufficient. i.e.,  `memory_order_relaxed`.


Again, branch immediately afterwards.


I think you mean:
```php
$databaseNameSql = null === $database ? 'DATABASE()' : "'$database'";
```


아직 client_secret 파일이 들어가 있음. ignore 되는거랑 repo에서 없어지는거랑은 다름.


Would it be possible to align the properties?


创建一个对象，其某个属性只有 setter 而没有对应的 getter，这在 Javascript 中 是个常犯的错误。没有 getter ，你不能读取这个属性，该属性也就不会被用到。（**译者注：** Javascript 中不一定非要同时指定 getter 和 setter ，原文将此表述为"错误"并非是指语法上的错误）


2016 - 2016 xd


Решение о том, добавить ли кандидата в список, принимается на основании бизнес-логики -- "нам эта группа подходит, туда можно закешировать ключ". В методе sort_candidates из этого списка выкидываются группы, для которых в данный момент не удалось узнать дц, машину, свитч и т.д. по причине технической ошибки.
Согласен, что имя не совсем подходящее, поменяю на ```prepare_candidates``` и добавлю комментарии.


Again, `memory_order_relaxed` is sufficient. In particular when considering that there will be no more write on this variable, because the actor is already dead.


Please change the copyright year to 2016


Thank you for writing this, would you please add some more write ups here on SQL Injections. On ringzer it won't allow to see the write ups unless you complete it :)


You could also just call print to get the automatic scrolling

print.sh $1 $2 $3 $4


Multi window is also present on the low end J5, which also has 5.1.1, can you port it? Thanks


@rschenk Thanks!


We just recently bumped into a similar issue, which might be the same as
yours. What is your syslog-ng version? Are you running it in threaded mode?

--
Bazsi

On Fri, Jan 8, 2016 at 11:52 AM, jbfuzier <notifications@github.com> wrote:

> Sorry I made a mistake and did a pull request to your repo's master instead
> of my fork master.
>
> So far I doubt those changes are useful for others, I am in the process of
> investigating problems in the DNS cache. Basically we have around 1000
> hosts logging to the server so we need the cache and what we are seeing is
> that even with cache and expiration set to a high value; the daemon is
> still constantly doing dns reverse lookup every 5-10secondes for the same
> hosts.
>
> So that is why I am adding those quick & dirty debugging log..
>
> --
> Jean-Baptiste Fuzier
> 0670657841
> http://twitter.com/jbfuzier
>
> On Fri, Jan 8, 2016 at 11:25 AM, Balazs Scheidler <
> notifications@github.com>
> wrote:
>
> > A bit more clarity in the messages would be useful, e.g. the fact that it
> > is coming from the DNS cache would be useful.
> >
> > On the code side, proper indentation would be better:
> >
> > msg_debug("DNS Cache full, removing one entry",
> > evt_tag_str("hostname", &cache_first.next->key),
> > NULL);
> >
> > With those, it could even be merged to master.
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/jbfuzier/syslog-ng/commit/6c35a9946e838ca3a5b3d0d54fd582a9dc41ff91#commitcomment-15336583
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jbfuzier/syslog-ng/commit/6c35a9946e838ca3a5b3d0d54fd582a9dc41ff91#commitcomment-15337038>
> .
>


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11025](https://teamcity.mesosphere.io/viewLog.html?buildId=102212&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 212 Build time: 00:04:40


```` creature->IsPet() || creature->IsHunterPet() ```` ?


For better error messages, it might be worth to insert a check here that `len(component_types) >= 1`.


例如：


My vote would be Slider


Thanks @tcard.  I agree `guard` is a good solution, however in tests, what are we guarding against? We don't expect a broken path in the first place.  It's a bit like doing this as far as I can see it:

```
var value = 1
expect(value).to(equal(1))
if (value != 1) { XCTFail("Weird, value was not 1") }
```

`guard` is great in our library code, but I am not so sure in our test code when we expect it to pass.  If it however fails, then why not let it fail "badly"?  That was why I asked in a rather confusing way `However, if there is a runtime error because we have assumed a null object is not null, what happens?`


I am aware of this issue and raised it in the design meeting, we didn't have a solution for that. We are fine for now.
The refactored code allows us fast and easy changes to the current split method.
To address this issue.... I will make this behavior visible, through comments in the user interface.


@willholley pretty sure the right-hand side "name" should be "username." Typo?


sorry, not sure if i didn't read the code or what, but the comment should be updated.  This is not checking that the secrets don't exist, it's checking that the secret files exist and have been truncated.


this would invalidate scripts for hunter pets (when tamed)...

what about checking for PetAI inside creaturescript?


Yeah, I was considering that. I'll look into it.


Atm. it wouldn't pass on CouchDB (which was my reason for changing it). But I agree on more general terms. Best would be to test the persistent format in the store (that'd also help with testing the "converter->grid type" cascade. We could add something as test in core only and test it with the map dialect explicitly.


@bparees I updated that test... so now we check the content in assemble and create marker if the content is "secret" ;-)


As far as render/3 yeah those could have and probably should have been :show and :errors.

Yeah something to think about I don't know at the moment. I see several issues around the lack of session in api. I have been reading and since the token actually carries the user_id with it, there is almost no need for a session and we could store other stuff in the token that we always want back. So for the api I think that is sensible.

Given that the following issues I have pondered about with combining them are
* put_session needs to be for html only, if you using html and rendering pages server side it seems to make more sense
* redirects which either need to happen on the server or possibly like you suggested but only for html.
* the templates to render are different between html and json
  * json -> registration succeeds -> :show
  * json -> registration fails -> :errors
  * html -> registration succeeds -> redirect (no render)
  * html -> registration fails -> :new
* redirect if authenticated only makes sense for html, but this could be pushed up to router and applied only for the html auth/registration pages. So there could be a scope like '/auth/login' and that scope gets a the redirect if authenticated plug.
* and the structure of params is different depending on the content type, because json-api forces the a structure.

Perhaps a first stage is pattern matching on the this difference

```elixir
def create(conn, %{"user" => user_params}) do #<- html

def create(conn, %{"data" => params}) do #<- json-api
```

and check the content type perhaps after... It would be nice if you could pattern match out the content type. Don't know, just where I am at the moment...


Depending on orientation this should return only the horz or vert flags.


Got it!

Thanks, this is working perfectly now.


Technically, other threads might be reading this variable without acquiring `mtx_` first, that's true. However,  `memory_order_release` only changes what stores *this thread* is allowed to perform before it hits this release barrier. And the current thread is already in a critical secion, i.e., between an `acquire` and a `release` barrier. Hence, `memory_order_relaxed` is sufficient, because neither the compiler nor the runtime are allowed to move this write outside of this critical section anyways. Even without the critical section, `exit_reason_` does not depend on any other data (or vice versa). So the order this thread performs its loads/stores in are pretty much irrelevant to other threads.


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1484 of 1484 tests passed)

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


Looks like you misspelled "canola"!


@egor-kalinin good catch! Will do a PR.


Probably just return `MinSizeHint`


```java
zero2 = - zero1
```
would be faster


cast whil stop the movement, I think this is not needed.
If it actually does not, check https://github.com/TrinityCore/TrinityCore/pull/14813


That SamsungSPRDRIL class needs some fix to build properly. You will not see it fail to build. It did failed, but the build system just ignored it and print out file not found message. I am working on fixing it. Hopefully it work :)


`int` should be fine


It doesn't, that's the bug we're trying to fix.


As mentioned in the implementation of `exited`: the function itself already should load with `memory_order_relaxed`.


Why did you not retake the iconUrl ?


No it is not required. But this is not the final state, it was changed during the commits.


Ah, you use a temporary because you re-use it here and don't want a second load, my fault.


do you mean the include comment or the using comment?


`typeof thisMapping` will never be `"undefined"`, however `thisMapping` itself or `thisMapping.mappedName` could be


> However, if there is a runtime error because we have assumed a null object is not null, what happens?
The current test fails with an unexpected failure but the test suite continues.


I made a new version of this, hopefully taking care of all that...


this should check for OOM

If pet goes out of mana, it should chase with "normal" distance, except if its the warlock imp.
(only case of pet that doesnt go "melee form" on out of mana)


ok


I agree it's not as important here as in "actual" code, but I still think it's worth it, to keep tests' error reporting as clear and minimal as simple. That way, we 1. avoid bloated feedback and 2. know that a runtime crash while running the tests (as would happen when calling a method on nil) means an error in the test code (like some state that the test isn't designed to handle), rather than something that is expected to happen. It's a matter of not making a special case out of test code.

(I agree completely though that keeping the `?`s is even worse.)


I'm going to test this using `path_dir` because I think it matches the use case well


are both files required?


The version here only works with ReShade 2.0, which is not yet released.


Thanks for the tip, is changed.


If we get to this point, then there is no spell to cast. This includes cases where the only possible spell didn't cast due to OOM.


Let's be more explicit about the missing required param, can we also display usage?


@Nawa Almost nobody sees comments on commits.  Please open an issue (https://golang.org/issue) or ask a question in a forum (https://golang.org/wiki/Questions).

I think this problem may have been fixed in the last day or two, but I'm not sure.


This PR breaks windows and mac builds: http://buildbot.clementine-player.org/builders/Windows%20Debug/builds/13


This is incomplete. Here is the complete fix: #3444


This one:
`// This alias is deliberitely outside of the namespace to make using this template less verbose`


The refactoring from switch into JSON object was causing all the `xToyObject` functions be called in every call to `attrToJson`, this way we keep the functions in the dictionary and call just the required one.


Sorry, got this commit from blame, but commented on the wrong file inside the commit. Here the code is being moved to `lib/utils/correct-mkdir.js` which is what the blame indicated. The actual cause of the issue is more likely from ebd0b32510f48f5773b9dd2e36ffc0bf0ecf4768 or 2568e40a395b809fb34125b3383f9f2616004f73 ([here](https://github.com/gagern/npm/commit/2568e40a395b809fb34125b3383f9f2616004f73#diff-63fdc6a01d0ec0cbf9772723f2356e46L639)). Which means I git the original author wrong as well. Should I open an issue for this?


I'm wondering about that `(firstOn.x, firstOn.y) == (lastOn.x, lastOn.y)` condition. It means you can't express an open path that ends on its first point. Still better than having no way to make an open path at all I guess?


what happens if eclstatsfile doesn't exist or is not accessible? does Paths.get return null? and then does Files.readAllBytes handle that case?


如果只定义 setter 而没有定义 getter，该规则会发出警告。设置了 `getWithoutSet` 选项，如果只有  getter  而没有 setter，该规则同样会发出警告。


Still I wonder, that German does not properly work in my production environment. I tested with English and French, no issue, but as soon as I change to German, the "submit" (for adding items/custoemrs) does not work anymore.


this catch is prob the answer to my previous question


To make sure only the [correct notification](https://msdn.microsoft.com/de-de/library/windows/desktop/bb760172(v=vs.85).aspx) is handled, you should inspect the lParam of the `WM_NOTIFY` message, as is done e.g. [here](https://github.com/lxn/walk/blob/master/tableview.go#L1238).


fixed


Dang, I killed the `Closes #47786.` by squashing the commits after `brew pull`. Sorry!


That should be all, yes.


@galharth  can pass this to helpers.swift


I think you need to call `registerOpenInISECommand` from `src\main.ts`


Original code compared **each** filename with **each** hash. It is obviously wrong. There are two possibly solution. Firstly, we can compare filenames and hashes in pairs. Secondly, we can consider file to be valid if its hash is in hash list. I decided that the first variant is better.


Done.


nice


Convert string to int. tablica[j]=t[j]-'0'; If we can convert single string to int, we must substract string - '0'


`manœuvres_.back().final_time()` is not a reference.


Already private.


This is for Node, which has supported `Promise` natively for a long while.


Expecting a value only testing the condition with `?` is a bad test so that's why I try always to do:

``` swift
expect(result).toNot(beNil())
expect(result?.hasNext).to(beFalse())
```

If the `result` is `nil` then the first test will tell us and the test will fail with a clean log.
The other way can raise a runtime error or even ignore the error.
I think a runtime error on tests is bad because the log will be difficult to read and the `travis-pretty-parser` will not work as expected.


The issue is we want to guarantee that every middleware *always* receives a
valid HTTP response. If you have a middleware at the top of your stack with
no try / catch in 5.1 and an exception is thrown in your route, that
middleware will *never* have a chance to examine the outgoing HTTP response.

On Fri, Jan 8, 2016 at 8:41 AM, Dan Harper <notifications@github.com> wrote:

> Yeh, though this is a documented breaking change in 5.2.
>
> Where was it documented? It's not in the upgrade guide.
>
> I'm not sure I understand the advantage of checking $response->exception
> normal exception handling?
>
> $response = $next($request);if ($response->status() === 500 && $response->exception) {     switch (get_class($response->exception)) {          case ExceptionOne::class:                return response();          case ExceptionTwo::class:               return response();     }}return $response;
>
> vs
>
> try {    return $next($request);} catch (ExceptionOne $e) {    return response();} catch (ExceptionTwo $e) {    return response();}
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/laravel/framework/commit/f5f843396c2c3cd244334029847fee2c7809123d#commitcomment-15340782>
> .
>


Ok, but to be reverted when we make it movable (we don't want to call `trajectory_->parent()->DeleteFork(&trajectory_)` on an invalidated `not_null`).


@xiexingguo I was just asking if you had a real-world example of how the the current logic produced an incorrect / unexpected result?


Once it's made movable.


@ranacseruet Ultimately I think we should put this into a separate module (not on the fork of their master module). Similar to how they did this in a separate repo:

https://github.com/bobby-brennan/kaltura-spec-converter/blob/master/lib/kaltura-spec.js


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


Juttle Influx Adapter


как это работает? по айди нашел запись. а потом никуда саму пришедшую ентити не передаешь.