>What I am trying to say is that you don't have to call emitCallTargetAndAccessor. Instead, you can just call emitCallTarget on node.expression (the same place you call emitCallTargetAndAccessor now), and then get the temp that results from that. Then when you are ready to emit the thisArg, you can just emit the temp that you got from emitCallTarget.
This also means that emitCallTargetAndAccessor is only used in emitCallWithSpread, and it doesn't have to return the accessor anymore.

You still need to emit the accessor? just emitting the temp variable won't work. And you would need the additional node checks to emit the accessor.

I could write as you said just emitting the with emitCallTarget() where the emitCallTargetAndAccessor() is.
and then emit the accessor(if target is defined):
```typescript
if (node.kind === SyntaxKind.PropertyAccessExpression) {
   write(".");
   emit((<PropertyAccessExpression>node).name);
}
else if (node.kind === SyntaxKind.ElementAccessExpression) {
   write("[");
   emit((<ElementAccessExpression>node).argumentExpression);
   write("]");
}
```

hmm. I just figured out that my example was wrong in the comments above should be:
```typescript
new ((_a = object).Array.bind.apply(_a.Array, [void 0].concat(arguments)));
```


should use type and action instead. please refer comment 1


Sure : )


I think you can make this object immutable by making the Line property readonly, and passing all 3 parameters using the constructor


Use an unqualified name here (```ConcreteDirectoryIndexer```) by adding ```using CodeLigh_ConsoleApp``` at the top of the file.


What are you testing here?


Thanks! BitHub has sent payment of  $10.21USD for this commit.


>I could write as you said just emitting the with emitCallTarget() where the emitCallTargetAndAccessor() is.
and then emit the accessor(if target is defined):
if (node.kind === SyntaxKind.PropertyAccessExpression) {
   write(".");
   emit((<PropertyAccessExpression>node).name);
}
else if (node.kind === SyntaxKind.ElementAccessExpression) {
   write("[");
   emit((<ElementAccessExpression>node).argumentExpression);
   write("]");
}

Just figured out that this doesn't work either. emitCallTarget doesn't emit the accessor. I think we need emitCallTargetAndAccessor.


`group === 0` doesn't imply `pos === 1` because the previous group could have been a sequence of non-spread elements.

I think a test of `length === 1` would suffice, but it's somewhat more subtle. I'd prefer to just keep the current test.


Oh I thought your example was correct. That's what my argument was based on. In that case, why is it not
```ts
new ((_a = object.Array).bind.apply(_a, [void 0].concat(arguments)));
```


This commit body is very large. In terms of what reviewers probably need to know it could be reduced to

    This changes serialVersionUID because it adds fields. Downloads have enough state to recover
    this information, but uploads do not and will be lost.

The serialVersionUID change is reverted later in the branch, so I have half a mind to suggest squashing these, but many people here seem to be against that, so I do not propose it.


Hello!  I was linked here from http://www.reddit.com/r/janusVR/comments/35pfwt/bug_broken_portrait_mode_scaling_on_linux/

Try changing something like this:

cfg.OGL.Header.BackBufferSize = Hmd->Resolution;

change to something like

cfg.OGL.Header.BackBufferSize = Hmd->Resolution;
#ifdef __linux__
    qSwap(cfg.OGL.Header.BackBufferSize.w, cfg.OGL.Header.BackBufferSize.h);
#endif

And that will fix it.


Or for that matter, couldn't you alternatively emit the following?
```ts
new (Function.bind.apply(object.Array, [void 0].concat(arguments)));
```
This would reduce the complexity, as it would eliminate the reliance on a temp.


This is not poor code. This is breaking the build now.


The behavior I personally need is the one where I send to a particular address each time I make a packet, while allowing to receive from any remote address.
My comment just points out that we probably should retain the documented behavior of connect(), for those who rely on that.
Also, when a socket is connect()-ed, the beginPacket() function shouldn't need an address/port, because the address/port is already known.
I can't say that I think the WiFiUdp shield library is all that well designed, but it's likely "the standard" that has to be followed.
If I don't like that, I could just use LWIP directly :-)


@demands could I get your help with these bugs/failing tests (with array elements) tomorrow?
thanks


See http://ci.kennethreitz.org/job/requests-pr/656/PYTHON=2.7/console for example


The commit message body is very large here. It could probably be reduced to

    getDescription() is not updated to include these fields because it is mostly used for debugging.


Excellent! Thank you very much, I'll give that a shot.

By the way, does that mean the next version of Janus will have portrait
mode working? That would make me very happy.
On May 12, 2015 7:34 PM, "James McCrae" <notifications@github.com> wrote:

> Hello! I was linked here from
> http://www.reddit.com/r/janusVR/comments/35pfwt/bug_broken_portrait_mode_scaling_on_linux/
>
> Try changing something like this:
>
> cfg.OGL.Header.BackBufferSize = Hmd->Resolution;
>
> change to something like
>
> cfg.OGL.Header.BackBufferSize = Hmd->Resolution;
> #ifdef *linux*
> qSwap(cfg.OGL.Header.BackBufferSize.w, cfg.OGL.Header.BackBufferSize.h);
> #endif
>
> And that will fix it.
>
> --
> Reply to this email directly or view it on GitHub
> <https://github.com/feilen/dolphin/commit/1a0c832dc639a640c19aacf59fb19174d3229177#commitcomment-11166677>
> .
>


I totally understand its history. Thank you all :+1:


apteryx_get_timestamp should also be proxied.

I think proxy_get/_set could be de-duped


I think there is no previous group, because `group === 0`. I prefer the length check because the meaning of length is clear before even looking at the code. group and pos require examining to understand what they are. It's fine though.


I thought if I used the message to provide the message that would be fine, but I get your point. If all the user saw was a stack trace, they aren't necessarily seeing the message. It shall be changed!


`results` here is the contact data. On line 68 is reassigned to be the phone numbers. This is misleading. Please use two variables.


Use of package


Can you refactor this to use a single query? Hint: use `join`


Hi, @KunmyonChoi -- thanks for the history.

I think since cancelHoldPulse was removed quite a while ago and we are not having any problems with the scroll controls, we can assume that the functionality is no longer required. Would you agree?


It will indeed!  (Performance on Linux I'd say is now on par with Windows direct mode)


Usage of package


Should this return a single contact or a bunch of contacts? The name suggests to me that it should just return one contact.


`length === 1` is the only case where we don't emit a call to concat. And slice is effectively acting as a replacement for concat.


Latency too! The point of Portrait Mode is that with Linux, the extra frame
of latency you get on Windows is never added in the first place, as long as
the screen remains unrotated.

(I doublechecked with the Tuscany demo too)

On Tue, May 12, 2015 at 7:41 PM, James McCrae <notifications@github.com>
wrote:

> It will indeed! (Performance on Linux I'd say is now on par with Windows
> direct mode)
>
> --
> Reply to this email directly or view it on GitHub
> <https://github.com/feilen/dolphin/commit/1a0c832dc639a640c19aacf59fb19174d3229177#commitcomment-11166740>
> .
>


Could we make FAS3 return these all lower-cased to make porting easier?


The comment needs to be updated.


Great job!! Code reviews are here to help guide your refactoring efforts, so don't worry we don't expect the code to be perfect.

In fact, code is very rarely perfect.. maybe never perfect. That's the big secret of web development. There is no mastery. No final level: http://t.co/1H5w5H7dUm
`</tangent>`


No, I didn't solve the one from Volga. But you are right, the math is the same, just with numbers. And I didn't understand the solution to cpkc (yet).
But non brute force would be nice, because mine scales really badly.


This leaks memory? Why was this changed? Maybe we want to make the timer's parent the widget?


Stale commit.


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


Sorry, I haven't looked in the task exactly. You're right, of course. Forget everything I said about random. This class is correct now.


get_contact_by_index throws an error and is caught in Application#do_command if an ID isn't in the database :)


@karmi Hi. Thanks for fix this bug. Please publish this changes to rubygems?


On 2015-05-12 6:52 PM, Kyle Warwick-Mathieu wrote:
>
> Hamachi huh? Nice touch. Will we be using Hamachi in our scripting course?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/zonzorp/COMP1071/commit/d327820ac7b9fa0415576cf00922a95771a468d0#commitcomment-11165646>.
>
No, there is no role for hamachi in the scripting course. It solves an
IP conflict problem for the network course now that we are doing the
course entirely on pis.


FacilityTabGridView (evanweaver) - i386-Windows-7-VisualStudio-12: OK (2172 of 2200 tests passed)

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


Testfactories should say TestFactories


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


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


does it mean I can select timestamp column in my table?


Yep! Don't look under the hood - it's a mess.


We don't yet have an explicit "exit" command, so the only way to completely quit any and all scripts running is to force an exception. It's not ideal, but it works given the limitation.


How come you moved to Jest. Just prefer it or ... ?


Can I suggest ~8 lines that would make this more testable and potentially reusable:
```
class RefCountDict(defaultdict):
    def __init__(self):
        super(RefCountDict, self).__init__(self, int)
    def __setitem__(self, key, value):
        if key in self and self[key] > 0 and value == 0:   # could also use <= 0, I guess
            del self[key]
        else:
            super(RefCountDict, self).__setitem__(key, value)
```


The commit message body could be changed to

    This adds the Unix timestamp of the last success and failure to fetch a
    block to the "LastSuccess" and "LastFailure" fields.

The getters don't really belong in this commit because they are not used by it. They are short enough that it makes sense to introduce them along with their usages instead.


1836664-ReturnWaterTempReset (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1144 of 1144 tests passed)

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


>Oh I thought your example was correct. That's what my argument was based on. In that case, why is it not

I think I could write it like that. It was just that you don't share much code with `emitCallWithSpread`. Then I need a new emitCallTarget that assign a temp on the target+accessor and not just the target.

I can investigate if it works with `Function`, because that would be a great solution.


Why make that a local function?


while(


This last commit is just to ensure that the workspace is correct after running the rails server.


Hi @AnotherJoSmith,

I was just mulling on this a bit.  It looks like the only gateway to use this field (so far at least) is USA ePAY.  I'm wondering if passing in the field in the options hash for transactions like `purchase` and `authorize` might suffice.  The question I'm wondering about is how we decide whether a field like this applies to a payment method or if it fits better being passed as an option on the transaction.

For something like `manual_entry`, I could see that applying on the initial transaction but then perhaps not applying later on after the card had been vaulted.

I'd love to hear your thoughts on it!


That being said, I can see the logic of it being a property on the card as well. :)


That worked perfectly, thank you so much!


sorry @cyberden hehe


Would be awesome to include a countdown timer on the individual product's show page. Could implement this with Javascript using the end_date (datetime) in the projects table.


The new repo of pluggable Crosswalk webview is https://github.com/crosswalk-project/cordova-plugin-crosswalk-webview, Could you please try to this?


This changes causes a bug, as the style element `id` is set to an invalid class name. For example,

```
var el = document.querySelector( '#ace_editor.css' );
```

Does not work.


That's a great idea! Would you be okay with me including it as part of this PR, or would you prefer that I leave that for later?


ahhh, I see.  Well then.  I'll end up reversing this latest commit and doing it properly, then.
Thanks for your help, Rev!


This is the best thing working for me so far. The spinner thing doesn't reload and kind of messes my stuff up. The other two ways, hiding and displaying none semi-works but I can't justify the gallery. When I load it at the end either nothing happens or the spinner just breaks it.

Is there a way to fix this or am I doing something wrong?

 $('.btn_photo').on('click', function(event) {
  	//$('.anim, .cinema, .graph, .web').addClass('spinner');
   	//$('.photo').removeClass('spinner');
   	$('.photo').css('display','block');
    $('.anim, .web, .cinema, .graph').css('display','none');
    $('.anim, .web, .cinema, .graph').hide();
    //$("#gallery").justifiedGallery();
  });


- Altered custom.css (removed absolute URL path with a relative one)
- Removed redundant Sign In page
- Added extensive commenting to all PHP files


C/C++ is rock.


None of this commit message body seems necessary.


Cool beans!


Remove outer ()


and Cython is rock too. we can freely use any C/C++ method in python without sacrificing speed.


@JsonFreeman I can confirm that the `Function` dies works :smile: ! I think I can rewrite this to a much simpler commit. I will restore emitCallTargetAndAccessor where it belonged then.


The commit body paragraph beginning "The internal root source" is not necessary.


Whichever is fine with me.


This commit comes too late - the previous commit did not compile because the parts this commit changes were still using the removed `long getLastActivity()`.


This mixes updating the existing column to function, renaming it, and adding a new one.


numの平方根で計算終わっていいと思う


Sprawdzam jak to działa


慣習的にbooleanを返す関数は "is_prime?"みたいな命名をするとわかりやすいです。


This is a bug. If the event is set to increment, it's not longer being tracked!


cとはなにかよくわかりませんでした


@f2prateek Please fix this ASAP


Call me Ishmael0.7.


Looks good, thanks.


xuống dòng cùng hàng của nó , và  sử dùng 2 space


//你过来咬我呀


The other reason I sometimes use it is because not only do I want to exit, but I want to force the system to spit out a full program opcode dump into the output_log as it does so.  At the moment the only way to trigger this is to force a runtime exception to happen, as a normal polite exit doesn't spit out that dump.


問題は10001番目を出力せよ、だから配列に入れる必要ないのでは?


We've been using "update" to refer to switching version numbers, and "upgrade" to going from Basic to Enterprise. This seems to be using "upgrade" for the update. Did we change our mind on the terminology?


Yup. I just re-built all the docs; I'm about to upload them to `pism-docs.org`. Almost there.


Thanks, you should fork the androidarmv6 framework_base then update it. And if you can, try fixing the audio for omniarmv6.


你过来要咬我呀


@negue I added this line because it's possible to buy gems as a gift for yourself. This way, the push notification only gets sent if you are not buying them as a gift for yourself.


Add my edit (name)


Yeah, I saw this at one point in my version I think. It is no longer in master.
Maybe redo this when things settle down.


Shouldn't you declare the element type of `data` as aligned? Otherwise each element of `data` is not guaranteed to be aligned.


unused importです


rubyだと後置if文という文法が使えます

``` ruby

return false if num%i == 0
return true

```

と書けます。参考までに


The commit message body is not necessary.


rax retry tests


素数を順番に求める時は[エラトステネスのふるい](http://ja.wikipedia.org/wiki/%E3%82%A8%E3%83%A9%E3%83%88%E3%82%B9%E3%83%86%E3%83%8D%E3%82%B9%E3%81%AE%E7%AF%A9)を使うといいぞ


> エラトステネスのふるい

:+1


I think there is a bug in GetHashCode when run under Net 4.5.  Kaijin is running win7 pro on net 4.5.  I have seen this bug personally in CGT, but it is also documented here http://www.reanmachine.com/net-4-5-wpf-selector-bug-with-objects-whos-hashcode-are-not-fixed/ and elsewhere.  The bug happens when 4.0 code is run under 4.5 which is what I experience and what Kaijin is reporting.  My work around was to stop using GetHashCode when possible and the problem went away.


Yep


This commit message body could be reduced to

    purge-db4o prereleases have been out for a while, so lost uploads caused
    by changing the serialVersionUID could affect many users. Instead the
    newly added latestSuccess and latestFailure will be initialized to null
    by the deserializer and displayed as "unknown".


This is somewhat of a cheap fix. Ideally the delay would always be saved, but just ignored unless delayed start is selected.


Hi,
Comparing  configure.ac with rawstudio-2.1.pc.in
+RAWSTUDIO_PLUGINS_LIBS_DIR="$libdir/rawstudio/plugins"
and
+plugindir=@prefix@/lib/rawstudio/plugins

I think should be :
plugindir= @libdir@/rawstudio/plugins (not /lib/ )

isn't it ?


Also, i cannot get the client to start, patcher crap...


whitespace


Under linux if you use CMAKE_BUILD_TYPE you can set 'Debug', 'Release', 'RelWithDebInfo' and 'MinSizeRel'.  From the CMAKE doco

CMAKE_BUILD_TYPE=None | Debug | Release | RelWithDebInfo | MinSizeRel
* RelWithDebInfo = optimized but with debug info -- default
* Debug = just debug -- faster compiling and best for debugging
* MinSizeRel = strips the code of any extra symbols

Debug is '-g'
Release is '-O2'
So I use these in either the main CMakeList.txt file for in the settings available from the cache settings in KDevelop (my current IDE).

I have commented out similar changes to the one above and actually removed the -g & -O2 from the definitions as the CMAKE_BUILD_TYPE will add them back in for you.

Jon


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


That works for me, and you even have docs!


This looks like you might have misunderstood something. OpenGL will use all vertices passed to it and then draw triangles (or quads) between 3 or 4 subsequent vertices (see C4FoWLight::Render and the Draw* function it calls). There is no glVertex call for each pixel that is stored in the texture.


In stead of using the word "folder", you can just say "database".


コンストラクタやデストラクタがからの場合は ` = default`をするのもいいんですが、これだと、ただのDefault値を設定されるので、最適化がはかれません。
そのため、もうからの場合は消しちゃいましょう！
そうすれば、コンパイラーが自動的にないことを検知し、無視してくれます。
`= defalut`は、ただの空設定だよ！っていう明示的に示すだけしかないんだよねｗ

なので、ここは、消しちゃいましょう！


I try `alias git='GIT_TERMINAL_PROMPT=1 command git'` and this helps me.
Yeah, it's simple and good solution.


Thanks, I'll take a look. Is there any chance we could meet at 11 or 1.30?
I was planning on going to body pump 12-1, though I'll happily skip it if
12 is the only time you can do.

On Wednesday, 13 May 2015, Simon Farrell <notifications@github.com> wrote:

> implement multiple recalls (initial)
>
> @am8302 let’s meet to talk about this at noon
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/am8302/groupingModel/commit/6b4327ed572ad8364a94f11e4a725e71ff43536c

--
Sent from Mobile


Make a note that this variable is only available in percona server, for oracle mysql, you have to check the `ibuf_size` in `show engine innodb status`.


The second half quote is missing


This says it's not a breaking change but https://github.com/facebook/react/blob/f0bdadf85bb22a590d04d739044e762950a4d5cc/npm-react-codemod/transforms/findDOMNode.js#L90 seemingly works before this change but not after; I get

```
Error: You have a collection of type [MemberExpression]. "closest" is is only defined for "Node".
```

on the line linked above when running under 0.1.4 but 0.1.2 works fine.

cc @cpojer


I think it should be `this._strings.length > 1`


Thank god this is fixed, limit to terminal is incredibly useful for saving dV (I'd been hard limiting acceleration which SUCKS)


Dany must have hard coded it when he was working on it. I'll make it a var and give explanation. It's just a speed where wing damage start to occur.


- 最初の`num = 2`の初期化位置がprime関数のnumと近すぎてまぎらわしい。関数宣言直前で宣言されてる変数って関数内で使われるものと思ってしまう
- 変数`c`が名前から役割を推察できない
- `require "pry"`が不要
- 素数かどうかを判定しboolを返す関数名は`prime?`とかにする。Rubyではメソッド名に?が使える。`Array.empty?`とか


Meeting my students then. Could meet in the afternoon of course

Note that you probably won’t be able to pull this, as it is on a new branch. You can look at it on github though

S

On 13 May 2015, at 9:36 am, Alice Mason <notifications@github.com<mailto:notifications@github.com>> wrote:

Thanks, I'll take a look. Is there any chance we could meet at 11 or 1.30?
I was planning on going to body pump 12-1, though I'll happily skip it if
12 is the only time you can do.


It might be a mess, but that doesn't mean that we should make it worse. ;-)


`squash`したくない場合は`--squash`オプション外してください


`brew install hub`などで`hub`コマンド入れておいてください


Testing Comments


@vinceparedes it shows up in the preview site but just has no written definition or CEDS code linkage for now.


why do we need to cast to list here?


couldn't we simplify this?

```self.ch_names = [epochs.ch_names[p] for p in picks]```


It appears Direct Mode on Linux suffers the exact finnickey issues that it does on Windows (Synchronize GPU Thread and fake-completion neccesary, with support varying from game to game) but the latency improvement is immediately noticable!


Wooo hoooo!!


Thanks.


I'll definitely give it a try when I have some time.


配列からある範囲を取り出すにはRangeの添字を使うか
![image](https://cloud.githubusercontent.com/assets/34204/7602310/f49ad3be-f95d-11e4-8315-c59006db5e17.png)

`Array#slice(開始位置, 個数)`を使うとよいです
![image](https://cloud.githubusercontent.com/assets/34204/7602321/24bc1abc-f95e-11e4-92a0-987e3ae45cc1.png)

配列足す時はreduceで
![image](https://cloud.githubusercontent.com/assets/34204/7602334/59dd6282-f95e-11e4-81d2-a2c1c689bdf6.png)


I was thinking of applying this to `flippedMatrix` instead, since it would happen automatically.  Although I'm super busy this week - thanks for fixing my mistake.

-[Unknown]


ok


sup


This is a moving target - so if we even changed Task size, the default for this kconfig option will change too. To be honest I don't see anyone playing with this option, can we just add #ifdef EZCHIP in the header and be done it.

Also please provide more details why exactly you want this - AFAIKR u were mapping FMT etc here - and they need to be above stack in mem map - why is that - perhaps i need to dig up some old emails on topic !


Thanks Ruben!!


for what is that good for? Vote for delete it again see: https://github.com/openstreetmap/iD/pull/2643


This is being written to the response, loggo would go to a file.


fixed!


Could we use some alias here, e.g. version.latest or smth? I'm afraid we won't keep this in sync with the really released versions


but they could be cached and so we're not checking the blobstore. We know how big the file is so I'll make sure its not too big before getting it.


Yes. 32sp will look pretty good


I think you're missing a space between the npm command and the install argument. Breaking my npm install:
![screen shot 2015-05-13 at 3 02 55 am](https://cloud.githubusercontent.com/assets/96857/7608237/9467c7a4-f91c-11e4-97f9-c7915c01175c.png)


Okay. I'm just noob on git. So, there are many useless commits.


Can u fix changelog to describe the motivation for this change - something along the lines of need for increased vmalloc space for EZChip SoC - hence stealing a chunk of unused kernel/user gutter !


Why this handle here? It should be from controller. Remove this line and handle from controller


<!-- PMABOT:SOB -->
This commit is missing Signed-Off-By line to indicate that you agree with phpMyAdmin Developer's Certificate of Origin. Please check [contributing documentation](https://github.com/phpmyadmin/phpmyadmin/blob/master/CONTRIBUTING.md) for more information.


Use form_tag instead of html tag.


Hi  Harelba,
                     I am trying to run tail2kafka But I am  getting Flush messages and in Server log i got the below  mentioned Error
Error : [2015-05-13 15:28:51,784] ERROR Closing socket for /127.0.0.1 because of error (kafka.network.Processor)
java.lang.OutOfMemoryError: Java heap space
	at kafka.api.ProducerRequest$$anonfun$1$$anonfun$apply$1.apply(ProducerRequest.scala:45)
	at kafka.api.ProducerRequest$$anonfun$1$$anonfun$apply$1.apply(ProducerRequest.scala:42)
	at scala.collection.TraversableLike$$anonfun$map$1.apply(TraversableLike.scala:206)
	at scala.collection.TraversableLike$$anonfun$map$1.apply(TraversableLike.scala:206)
	at scala.collection.immutable.Range$ByOne$class.foreach(Range.scala:282)
	at scala.collection.immutable.Range$$anon$1.foreach(Range.scala:274)
	at scala.collection.TraversableLike$class.map(TraversableLike.scala:206)
	at scala.collection.immutable.Range.map(Range.scala:39)
	at kafka.api.ProducerRequest$$anonfun$1.apply(ProducerRequest.scala:42)
	at kafka.api.ProducerRequest$$anonfun$1.apply(ProducerRequest.scala:38)
	at scala.collection.TraversableLike$$anonfun$flatMap$1.apply(TraversableLike.scala:227)
	at scala.collection.TraversableLike$$anonfun$flatMap$1.apply(TraversableLike.scala:227)
	at scala.collection.immutable.Range$ByOne$class.foreach(Range.scala:282)
	at scala.collection.immutable.Range$$anon$1.foreach(Range.scala:274)
	at scala.collection.TraversableLike$class.flatMap(TraversableLike.scala:227)
	at scala.collection.immutable.Range.flatMap(Range.scala:39)
	at kafka.api.ProducerRequest$.readFrom(ProducerRequest.scala:38)
	at kafka.api.RequestKeys$$anonfun$1.apply(RequestKeys.scala:36)
	at kafka.api.RequestKeys$$anonfun$1.apply(RequestKeys.scala:36)
	at kafka.network.RequestChannel$Request.<init>(RequestChannel.scala:53)
	at kafka.network.Processor.read(SocketServer.scala:353)
	at kafka.network.Processor.run(SocketServer.scala:245)
	at java.lang.Thread.run(Thread.java:745)
![tail2kafka](https://cloud.githubusercontent.com/assets/12342549/7608318/ea5fb806-f985-11e4-8c81-947835129955.png)


Thanks for your fast reply @mgruner - could you please talk some numbers? That would be great.
Don't you think the jQuery DOM manipulation should be avoided as much as possible? Since the DOM has to be re-rendered everytime an element gets changed. This is realy bad if an outer element is changed and will cost a lot performance on the client side. Espacially in cases where some old browsers or low performance machines are present. Additionally it could cause confusions for the client if elements get shown/removed with a delay after the page is loaded. Porting the framework FilterElementPre modules to FilterElementPost is nice! Kudos for that.


what do you mean didn't pass? LTC is still running:
```
LLR: -1.45 (-2.94,2.94) [0.00,4.50]
Total: 197865 W: 32424 L: 31794 D: 133647
```


多发点


try block this line：
hardware/libhardware_legacy/include/hardware_legacy/AudioHardwareInterface.h +126


@stefanceriu that specific tool is two years old.


니냐니뇨


Hey @bigsnarfdude - just a note that you have AWS creds in the open here ^^


Fixing typos.


needs to be moved up as it's in the test dep section


I am aware of that.
I released v3.2 yesterday and a brand new demo project to go with it (It should make understanding what it can do and how to use it a lot easier).


&:hover button on a new line


@qiangxue
this case an error
15:42:34.833 Cross-Origin Request Blocked: The Same Origin Policy disallows reading the remote resource. (Reason: CORS request failed).1 <unknown>


button on a new line


Here are some (rounded) numbers about how much speed penalty there is without caching the TT objects:

```
Module                  Normal (ms)     All TT Caching Disabled (ms)
Dashboard, cached       300             500
Ticket Overview (25t)   360             480
Ticket Zoom (125a)      330             530

Details for Ticket Zoom (125 articles)
Fully Cached:                       330
No cache for AgentTicketZoom.tt     450
No cache for ArticleActionMenu.tt   350
No cache for Header.tt              370
No cache for Footer.tt              370
No cache for FooterJS.tt            350
No cache for AgentNavigationBar.tt  350
```

You can see that every single uncached file adds at least 20-40ms. If
caching is disabled completely the request becomes > 50% slower. This
could happen with an unfortunate combination of pre output filters in a
system, at least for certain parts of the system; and it could have a
heavy impact on overall system performance/scalability.


well! that wouldn't have made much sense


I was expecting more capitals and exclamation marks along with grave words portending doom:
```
DO NOT change the type of index from int64_t or you'll break existing databases!
```
etc. etc.
[Use of biblical imagery and archaic words an optional extra. :)]


comment is missing after IMultiPolygon


@yanboliang  @jkbradley Is it possible to pass  initialModelWeights, initialModelMu, initialModelSigma as a single list to make the method signature more compact?


谈了么？怎么样啊？


@thorsteneckel, you are right, DOM based operations are not always good.


Should the title of this commit use "Bust" instead of "Boost"?


It might be worth saying this for the flattened test too, or not have it here i.e. make it clear that after this PR both styles of test will pass.


@francis22 please report it as an issue, a comment on a year old commit will get lost.


Thanks, should be fixed now.


The pointer-to-slice business is a bit ugly, and this function is only called with respect to the `Entries` field in `Ring`.  And in general, the code in this file seems very entangled with the code in `ring.go`, rather than defining a self-contained data structure.  So I suggest merging the two files, and making the `entries` methods belong to `Ring`.


test


Just like that


Actually, this commit is wrong. Because we are doing a dev release. The issue is that if we are doing dev releases, we need to include things like phoenix_live_reload in the list of applications. :(

@lancehalvorsen, what do you want to do: add :phoenix_live_reload to the list to or already build a release for production? I would prefer the latter but it is not strong feeling/argument, it is completely up to you.


I find it more readable one under another as it was originally, wdyt?


no problem.

First, make a backup copy of your dir (just in case). If you have any unsaved changes, stash them first (git stash)
Then run 'git rebase -i 22902fb'

An editor will open - edit (reorder and change pick to squash) the contents to look like this:

pick d9160f2 Init.d scripts for kernel
squash 6d9fb6e Delete codina-vendor-blobs.mk~
pick b593210 fix
pick d81b0e6 fix
squash 65b8145 fix
pick e615bfd fix
squash 9cf8c82 fix
pick 804be49 Update 00autoload
pick 5ddae82 Update 00autoload

save and quit (you can also delete two last commits since they effectively do nothing).

When you get an error about merge conflict, run 'git mergetool' and select '(d)eleted file', then 'git rebase --continue'. You can edit the messages if you want. When done, do 'git push -f' and that's all ;)


same comment


I'd use @since for all new methods


Why does entries contain pointers rather that `entry` directly?  It would be more efficient, and eliminate the question of whether pointers can be shared (e.g. whether two elements in an `entries` can contain the same pointer). I may have missed some, but the changes required to convert to a `[]entry` seem modest.


It might be worth referencing the commit where the original caching was introduced, which I think is this one: 760bb515630a364b3ad2552430a50001a578972b.


Hi, danke für deinen wertvollen Beitrag, bei diesem so wichtigen Projekt.
Wir werden den Vorschlag evaluieren und ihn gegebenenfalls einbauen.


same, could you please keep it as it was?


Don't use the `%` operator, use the comma. =) Remember that logs will do the formatting lazily for you. =)


I like that @ocharles and @charleso both write Haskell!


Nice commit Edy. we'll need to brainstorm to move the new Cloner interface and the 2 impl to a more generic place  + discuss consequences/relationship with existing XWikiContextProvider and ExecutionContextManager. Thanks!


This is some kind of error, maybe `red` color will be better ?


first commit


```mvn install -Pquality``` fails

`````
[INFO] --- jacoco-maven-plugin:0.7.4.201502262128:check (jacoco-check) @ xwiki-platform-mail-send-default ---
[INFO] Analyzed bundle 'XWiki Platform - Mail - Send - Default Implementation' with 60 classes
[WARNING] Rule violated for bundle XWiki Platform - Mail - Send - Default Implementation: instructions covered ratio is 0.72, but expected minimum is 0.74
``````


Do we need the && !data.field_image.url?


Hi Ralph

Mate could you please tell me where do I put virl-utils zip file after download,means do I put it on somewhere virl location or I keep it on my windows machine from there I run command as you show in one of video to start wireshark capture


The adapter **does not** implement the `AttributeBagInterface`!


Thanks for the number. It's hard to belive that some heavy RegExps and/or hacks won't cause much more impacts on performance that will be needed to deal with this missing feature. In my personal oppinion a strong recommendation would be a more appropriate way to go. I think all of our customers won't be that happy for paying > 4h effort more for 50 ms performance with their customizations. But thanks for making thinks more clear.


Oh well, it seems it does in the latest version ;-)


new


Не к спеху!

On 13 May 2015 at 18:12, Markolm <notifications@github.com> wrote:

> Странно, после школы посмотрю.
> On 13 May 2015 05:23, "Maxim" <notifications@github.com> wrote:
>
> > @Markolm <https://github.com/Markolm> работет!
> > чуть чуть с whitespace неровно только:
> > [image: image]
> > <
> https://cloud.githubusercontent.com/assets/10990645/7603258/0da45c38-f984-11e4-83e4-901ccb80e0c1.png>
>
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/maxim47/C-enums/commit/9a77151170039b68dccc19fdfa983a762e679c19#commitcomment-11168039>
>
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/maxim47/C-enums/commit/9a77151170039b68dccc19fdfa983a762e679c19#commitcomment-11169341>
> .
>


good catch


However, I think the implementation is wrong then. We should not use an adapter, we should create a new child class of `AttributeBag` and add the array access methods.


THAT'S MORE LIKE IT!!!1eleventyone


You still need the mcver in front, the 1.7.10 at the end of the forge version is the branch from the forge repo.


mvn clean install failing with

[ERROR] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Compilation failure

/C:/jitwatch/jitwatch-master/src/main/java/org/adoptopenjdk/jitwatch/ui/MainConfigStage.java:[58,64] C:\jitwatch\jitwatch-master\src\main\java\org\adoptopenjdk\jitwatch\ui\MainConfigStage.java:58: illegal start of type
/C:/jitwatch/jitwatch-master/src/main/java/org/adoptopenjdk/jitwatch/sandbox/runtime/RuntimeJava.java:[46,55] C:\jitwatch\jitwatch-master\src\main\java\org\adoptopenjdk\jitwatch\sandbox\runtime\RuntimeJava.java:46: illegal start of type
/C:/jitwatch/jitwatch-master/src/main/java/org/adoptopenjdk/jitwatch/model/bytecode/BytecodeAnnotationBuilder.java:[56,67] C:\jitwatch\jitwatch-master\src\main\java\org\adoptopenjdk\jitwatch\model\bytecode\BytecodeAnnotationBuilder.java:56: illegal start of type
/C:/jitwatch/jitwatch-master/src/main/java/org/adoptopenjdk/jitwatch/model/bytecode/BytecodeAnnotationBuilder.java:[119,63] C:\jitwatch\jitwatch-master\src\main\java\org\adoptopenjdk\jitwatch\model\bytecode\BytecodeAnnotationBuilder.java:119: illegal start of type
/C:/jitwatch/jitwatch-master/src/main/java/org/adoptopenjdk/jitwatch/model/bytecode/BytecodeAnnotationBuilder.java:[120,61] C:\jitwatch\jitwatch-master\src\main\java\org\adoptopenjdk\jitwatch\model\bytecode\BytecodeAnnotationBuilder.java:120: illegal start of type
/C:/jitwatch/jitwatch-master/src/main/java/org/adoptopenjdk/jitwatch/model/bytecode/MemberBytecode.java:[20,80] C:\jitwatch\jitwatch-master\src\main\java\org\adoptopenjdk\jitwatch\model\bytecode\MemberBytecode.java:20: illegal start of type
/C:/jitwatch/jitwatch-master/src/main/java/org/adoptopenjdk/jitwatch/ui/TextViewerStage.java:[47,52] C:\jitwatch\jitwatch-master\src\main\java\org\adoptopenjdk\jitwatch\ui\TextViewerStage.java:47: illegal start of type
/C:/jitwatch/jitwatch-master/src/main/java/org/adoptopenjdk/jitwatch/chain/CompileChainWalker.java:[77,63] C:\jitwatch\jitwatch-master\src\main\java\org\adoptopenjdk\jitwatch\chain\CompileChainWalker.java:77: illegal start of type
/C:/jitwatch/jitwatch-master/src/main/java/org/adoptopenjdk/jitwatch/chain/CompileChainWalker.java:[78,61] C:\jitwatch\jitwatch-master\src\main\java\org\adoptopenjdk\jitwatch\chain\CompileChainWalker.java:78: illegal start of type
/C:/jitwatch/jitwatch-master/src/main/java/org/adoptopenjdk/jitwatch/ui/triview/assembly/ViewerAssembly.java:[52,52] C:\jitwatch\jitwatch-master\src\main\java\org\adoptopenjdk\jitwatch\ui\triview\assembly\ViewerAssembly.java:52: illegal start of type
/C:/jitwatch/jitwatch-master/src/main/java/org/adoptopenjdk/jitwatch/core/JITWatchConstants.java:[53,72] C:\jitwatch\jitwatch-master\src\main\java\org\adoptopenjdk\jitwatch\core\JITWatchConstants.java:53: illegal start of type
/C:/jitwatch/jitwatch-master/src/main/java/org/adoptopenjdk/jitwatch/core/JITWatchConstants.java:[55,70] C:\jitwatch\jitwatch-master\src\main\java\org\adoptopenjdk\jitwatch\core\JITWatchConstants.java:55: illegal start of type
/C:/jitwatch/jitwatch-master/src/main/java/org/adoptopenjdk/jitwatch/core/JITWatchConstants.java:[146,83] C:\jitwatch\jitwatch-master\src\main\java\org\adoptopenjdk\jitwatch\core\JITWatchConstants.java:146: illegal start of type
/C:/jitwatch/jitwatch-master/src/main/java/org/adoptopenjdk/jitwatch/toplist/AbstractTopListVisitable.java:[45,41] C:\jitwatch\jitwatch-master\src\main\java\org\adoptopenjdk\jitwatch\toplist\AbstractTopListVisitable.java:45: illegal start of type
/C:/jitwatch/jitwatch-master/src/main/java/org/adoptopenjdk/jitwatch/sandbox/compiler/CompilerScala.java:[42,55] C:\jitwatch\jitwatch-master\src\main\java\org\adoptopenjdk\jitwatch\sandbox\compiler\CompilerScala.java:42: illegal start of type
/C:/jitwatch/jitwatch-master/src/main/java/org/adoptopenjdk/jitwatch/ui/JournalViewerStage.java:[52,52] C:\jitwatch\jitwatch-master\src\main\java\org\adoptopenjdk\jitwatch\ui\JournalViewerStage.java:52: illegal start of type
/C:/jitwatch/jitwatch-master/src/main/java/org/adoptopenjdk/jitwatch/model/bytecode/ClassBC.java:[34,73] C:\jitwatch\jitwatch-master\src\main\java\org\adoptopenjdk\jitwatch\model\bytecode\ClassBC.java:34: illegal start of type
/C:/jitwatch/jitwatch-master/src/main/java/org/adoptopenjdk/jitwatch/model/bytecode/ClassBC.java:[36,62] C:\jitwatch\jitwatch-master\src\main\java\org\adoptopenjdk\jitwatch\model\bytecode\ClassBC.java:36: illegal start of type
/C:/jitwatch/jitwatch-master/src/main/java/org/adoptopenjdk/jitwatch/model/bytecode/ClassBC.java:[38,74] C:\jitwatch\jitwatch-master\src\main\java\org\adoptopenjdk\jitwatch\model\bytecode\ClassBC.java:38: illegal start of type
/C:/jitwatch/jitwatch-master/src/main/java/org/adoptopenjdk/jitwatch/sandbox/compiler/CompilerKotlin.java:[43,55] C:\jitwatch\jitwatch-master\src\main\java\org\adoptopenjdk\jitwatch\sandbox\compiler\CompilerKotlin.java:43: illegal start of type
/C:/jitwatch/jitwatch-master/src/main/java/org/adoptopenjdk/jitwatch/sandbox/runtime/RuntimeJavaScript.java:[39,55] C:\jitwatch\jitwatch-master\src\main\java\org\adoptopenjdk\jitwatch\sandbox\runtime\RuntimeJavaScript.java:39: illegal start of type
/C:/jitwatch/jitwatch-master/src/main/java/org/adoptopenjdk/jitwatch/model/bytecode/Opcode.java:[238,66] C:\jitwatch\jitwatch-master\src\main\java\org\adoptopenjdk\jitwatch\model\bytecode\Opcode.java:238: illegal start of type
/C:/jitwatch/jitwatch-master/src/main/java/org/adoptopenjdk/jitwatch/model/bytecode/BytecodeInstruction.java:[28,65] C:\jitwatch\jitwatch-master\src\main\java\org\adoptopenjdk\jitwatch\model\bytecode\BytecodeInstruction.java:28: illegal start of type
/C:/jitwatch/jitwatch-master/src/main/java/org/adoptopenjdk/jitwatch/model/AbstractMetaMember.java:[62,78] C:\jitwatch\jitwatch-master\src\main\java\org\adoptopenjdk\jitwatch\model\AbstractMetaMember.java:62: illegal start of type
/C:/jitwatch/jitwatch-master/src/main/java/org/adoptopenjdk/jitwatch/model/AbstractMetaMember.java:[63,80] C:\jitwatch\jitwatch-master\src\main\java\org\adoptopenjdk\jitwatch\model\AbstractMetaMember.java:63: illegal start of type
/C:/jitwatch/jitwatch-master/src/main/java/org/adoptopenjdk/jitwatch/model/AbstractMetaMember.java:[207,55] C:\jitwatch\jitwatch-master\src\main\java\org\adoptopenjdk\jitwatch\model\AbstractMetaMember.java:207: illegal start of type
/C:/jitwatch/jitwatch-master/src/main/java/org/adoptopenjdk/jitwatch/model/AbstractMetaMember.java:[230,56] C:\jitwatch\jitwatch-master\src\main\java\org\adoptopenjdk\jitwatch\model\AbstractMetaMember.java:230: illegal start of type
/C:/jitwatch/jitwatch-master/src/main/java/org/adoptopenjdk/jitwatch/model/ParseDictionary.java:[13,56] C:\jitwatch\jitwatch-master\src\main\java\org\adoptopenjdk\jitwatch\model\ParseDictionary.java:13: illegal start of type
/C:/jitwatch/jitwatch-master/src/main/java/org/adoptopenjdk/jitwatch/model/ParseDictionary.java:[14,57] C:\jitwatch\jitwatch-master\src\main\java\org\adoptopenjdk\jitwatch\model\ParseDictionary.java:14: illegal start of type
/C:/jitwatch/jitwatch-master/src/main/java/org/adoptopenjdk/jitwatch/model/ParseDictionary.java:[15,58] C:\jitwatch\jitwatch-master\src\main\java\org\adoptopenjdk\jitwatch\model\ParseDictionary.java:15: illegal start of type
/C:/jitwatch/jitwatch-master/src/main/java/org/adoptopenjdk/jitwatch/util/StringUtil.java:[39,48] C:\jitwatch\jitwatch-master\src\main\java\org\adoptopenjdk\jitwatch\util\StringUtil.java:39: ';' expected
/C:/jitwatch/jitwatch-master/src/main/java/org/adoptopenjdk/jitwatch/util/StringUtil.java:[40,43] C:\jitwatch\jitwatch-master\src\main\java\org\adoptopenjdk\jitwatch\util\StringUtil.java:40: ';' expected
/C:/jitwatch/jitwatch-master/src/main/java/org/adoptopenjdk/jitwatch/util/StringUtil.java:[41,39] C:\jitwatch\jitwatch-master\src\main\java\org\adoptopenjdk\jitwatch\util\StringUtil.java:41: ';' expected
/C:/jitwatch/jitwatch-master/src/main/java/org/adoptopenjdk/jitwatch/util/StringUtil.java:[42,38] C:\jitwatch\jitwatch-master\src\main\java\org\adoptopenjdk\jitwatch\util\StringUtil.java:42: ';' expected
/C:/jitwatch/jitwatch-master/src/main/java/org/adoptopenjdk/jitwatch/util/StringUtil.java:[151,53] C:\jitwatch\jitwatch-master\src\main\java\org\adoptopenjdk\jitwatch\util\StringUtil.java:151: illegal start of type
/C:/jitwatch/jitwatch-master/src/main/java/org/adoptopenjdk/jitwatch/util/StringUtil.java:[240,58] C:\jitwatch\jitwatch-master\src\main\java\org\adoptopenjdk\jitwatch\util\StringUtil.java:240: illegal start of type
/C:/jitwatch/jitwatch-master/src/main/java/org/adoptopenjdk/jitwatch/util/StringUtil.java:[306,58] C:\jitwatch\jitwatch-master\src\main\java\org\adoptopenjdk\jitwatch\util\StringUtil.java:306: illegal start of type


szia Zsolt.


Hmm, I'll have to see if we can detect if the Rift has been rotated in the
SDK. Thanks for the heads up.
On May 13, 2015 2:05 AM, "Zachary Cook" <notifications@github.com> wrote:

> Unfortunately the fix for portrait mode messes up landscape mode, maybe
> only swap if portrait mode is detected?
>
> --
> Reply to this email directly or view it on GitHub
> <https://github.com/feilen/dolphin/commit/1a0c832dc639a640c19aacf59fb19174d3229177#commitcomment-11169923>
> .
>


Merge with the similar code in Owner.


Already has been on dev branch


TeamCity xBIM :: XbimWebUI :: Build [Build 53](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1622&buildTypeId=XBIM_XbimWebUI_Build) is now running


fontos fontos a változtatás.


TeamCity xBIM :: XbimWebUI :: Build [Build 53](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1622&buildTypeId=XBIM_XbimWebUI_Build) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:01:02


We could use some git-fu to pretend this commit never happened, because you change the state name in the very next commit so this is just noise.


@vcunat Why would they be needed there? Just link to the shared lib directly.


@lethalman :+1: from me.


SQL-запрос было бы неплохо переписать:
1. в текущем виде много повторов (можно ли как-то снизить их число, в идеале -- убрать полностью?)
2. большой набор зашитых прямо в код цифр. Что это за магические константы? Можно ли от них отказаться?


@DanielJMaher , @lukecampbell , mind if I change this var from streamTableItemView to annotationTableItemView for my commit?  It's not hurting anything as is, so it's ok to leave it for now.


I think go_call_tram should be moved to instance_ulduar.cpp

Because MIMIRON is triggered activation need to attack NPC


Aggiunti personaggi Mago, Strega e Cane. Messi personaggi nel labirinto. Test-cases ancora da fare.


ach ja, tutorial:
https://www.youtube.com/watch?v=13cLUL8VO2A


Player.java med animation


Yes, that is the obvious way, and I initially thought to do it in this way. However, this will mean we have to call multiple times hasClass, addClasses and removeClasses methods.
Since this code is for IE9, the functions won't be inlined and we will have multiple invocations, which means pushing to stack, creating and destroying multiple variables for each invocation. When it is one function, it should be optimized better.
I can do a perf test and see if that is true.


@kundan100 :+1: I feel setting milestones would be more handy, at https://github.com/kundan100/SketchModularJS/milestones


Could be crazy here, but there are no columns in AnnotationTableView.js that are "start" and "end".


policy_scope pracuje tylko na @user i @record (todo_item) i pozwalało na wyświetlanie w dowolnej liście wszystkich itemów usera (a nie tylko tych przypisanych do danej listy). czy doprecyzowanie scope'a w kontrolerze przy pomocy where jest ok? nie za bardzo da się to chyba zrobić w samym TodoItemPolicy


It lets you use a secondary language for translations before returning the default language.


init


なるほど、`to(1)`や`take(2)`外すとこうなっちゃうのね :sweat:

![screen shot 2015-05-13 at 19 51 03](https://cloud.githubusercontent.com/assets/155807/7608981/81f81c58-f9a9-11e4-871c-d39b05cbc3b0.png)


@kundan100 Do we need to push ```src/js/Consolidated.js``` file ? I think this is grunt build generated file which can be included in .gitignore and also i think executing and generating this file would be more interesting then providing it directly through clone :innocent:


The first compile failure is for line:
```
final ComboBox<String> comboBox = new ComboBox<>(profileList);
```
Looks like you are using a compiler (javac) that does not support diamond notation which is a Java 7 feature.

JITWatch requires at least JDK 7 to build and run it.


Ported to 1.6.0 and tested there, when converting molgenis date to date I get an exception in the test script:

12:53:56.206 [qtp992081445-158] ERROR o.m.d.m.a.AlgorithmServiceImpl - error converting result
java.lang.RuntimeException: No Context associated with current Thread
	at org.mozilla.javascript.Context.getContext(Context.java:2302) ~[rhino-1.7R5.jar:1.7R5]
	at org.mozilla.javascript.ScriptableObject.getDefaultValue(ScriptableObject.java:954) ~[rhino-1.7R5.jar:1.7R5]
	at org.mozilla.javascript.ScriptableObject.getDefaultValue(ScriptableObject.java:895) ~[rhino-1.7R5.jar:1.7R5]
	at org.mozilla.javascript.NativeDate.getDefaultValue(NativeDate.java:65) ~[rhino-1.7R5.jar:1.7R5]
	at org.mozilla.javascript.ScriptRuntime.toNumber(ScriptRuntime.java:368) ~[rhino-1.7R5.jar:1.7R5]
	at org.mozilla.javascript.Context.toNumber(Context.java:1592) ~[rhino-1.7R5.jar:1.7R5]
	at org.molgenis.data.mapper.algorithm.AlgorithmServiceImpl.applyAlgorithm(AlgorithmServiceImpl.java:61) ~[classes/:na]
	at org.molgenis.data.mapper.controller.MappingServiceController.testScript(MappingServiceController.java:355) [classes/:na]


Sorry .Yes My Java Home was set to JDK6.Thanks for the Help


No it's not because you are not even getting there according to your logs.


Why do we no longer require swiftmailer?


@shlevy except for static linking... we don't do it, but it may make sense in the future to support it. As debian discussion states, .la is useful for static linking. Maybe we could patch libtool to not use .la when doing dynamic linking and leave the .la files somehow...


Sorry to not have accepted this yet; its just because I don't want to do that until I'm working on the code next so I can test it. (busy with other projects at present). Possibly its because I've been away from the code for a while (and I've not been concerned with reporting uptimes), but I'm not sure I understand your use-case or observed behaviour properly.


@edolstra is this an acceptable solution? If not I can try to patch aterm to get the top of the stack without undefined behavior.


everything good, except: why back to square instead of circle folder icon?


If we do it in the future, we can bring back the .la files. We also have a stdenv adapter for building static versions of libraries, we can just disable the .la removal in that adapter.


Generally, I agree, but I find it very difficult to manage this every time, since the Eclipse code formatter we use has a different setup and it's very easy to apply it by mistake. AFAIK, we are supposed to use the same formatters in order to avoid just this type of problems. We either upate the formatter rules or we update / accept the updates on the formatting of existing code.

See https://github.com/xwiki/xwiki-commons/tree/master/xwiki-commons-tools/xwiki-commons-tool-verification-resources/src/main/resources/


The tab size in your IDE is too long, it screws all my formats and makes merging even harder..


We can reduce this to:

```ruby
def on_sponsor_moderation_threshold?
  supporting_sponsors_count == AppConfig.sponsor_moderation_threshold
end
```

The return value of that single line is already ``true`` or ``false`` so there's no need for the ternary to make it explicit.  There *is* a bit of a debate in the ruby community about predicate methods (those that end with a ?) and if they have to return actual ``true`` and actual ``false`` or if it's fine to return any old object or ``nil`` and rely on ruby's truthiness rules (only ``nil`` and ``false`` are falsey, everything else is truthy).  But if you rely on the value of a predicate method to be anything other than truthy or falsey then I think you're entering a world of pain.  Note that predicate methods are not special, just a social constraint that rubyists use - if it ends in a ? we assume we'll get truthy/falsey values but also that we'll only treat the return as something to use in a conditional, not to grab a leaked value.

Anyway, lecture over.  If you wanted to be super helpful you could use this new method in ``notify_creator_about_sponsor_support`` above and add a similar ``below_sponsor_moderation_threshold?`` method for parity.


Hmm, now that I look closer, it seems we can use `/* jshint phantom: true */` here or in .jshintrc.


What's this for?


Typo in title fixed in 01f61fd


Why is better why?


I can't run examples as using  ```python stocks.py``` on command-line  from examples directory. I think I have trouble with python packaging system.  I got error below.
 ```from kmodes import kprototypes
ImportError: cannot import name kprototypes```  .  Can someone tell me about how to run examples ?


I noticed. I am using default NetBeans settings. What are you using?
We should use a maven pretty-print plugin during build, which will fix this problem.


Because `symfony/swiftmailer-bundle` does that for us. It knows the compatible versions better than we do :)


This won't actually persist the change to the DB; it only changes the state attribute of the in-memory instance.

We have two options:

1. Change this method to do ``update_attribute(:state, SPONSORED_STATE)`` which will also persist the attribute change to the db
2. Put an explicit ``petition.save`` call into the controller.


test


Because the `NamespacedAttributeBag` is also not an adapter on `AttributeBag`. Adapters are to make one class compatible with another interface, not add functionality for something.


If you wanna add tiny things in my code it does't make sense to change it yourself, you can point it out in comments here and I will apply it.


Arguably, the `if` in that function makes it non-constant time. Something like this might work better:
https://github.com/cloudflare/go/commit/e993cbf1af51d74bb76934c9751e0cf9dda4142d#diff-a5d7dce21fe3bd2ed12ca771afdfefe0R74


nothing of importance, was testing something


When we use stubs/mocks (which is fine!) we need to make sure that the method we've stubbed/mocked actually does what we expect.  We usually achieve this with a unit spec for that method, which we're missing in this PR.  The alternative is to treat this as a more black-box test and assert on our expected outcomes (e.g. that the petition in the DB now has the sponsored state.

So either:

1. add a spec for ``update_sponsored_state`` to ``spec/models/petition_spec``

or

2. change this to remove the two stubbing lines and replace with ``expect(petition.reload.state).to eq Petition::SPONSORED_STATE``


Superceded by 78aa371


The problem is that this does not work yet (in 2.0).

That's why I did not include it in the command doc.


I think the whole session class should be deprecated and people should use the Symfony session?


I've had to read this doc and the signature many times to understand it. May I suggest more significant parameter names for the signature? I looked at the man of `memcpy` for inspiration.

~~~
# Copy `n` bytes from `self` starting at `src_offset` into `dest` starting at `dest_offset`
# ...
fun copy_to_native(dest: NativeString, n, src_offset, dest_offset: Int) do
~~~

Maybe `self_offset` would be better than `src_offset`...


You are right, however, your class does both. It adds functionality (array access) and it adds methods to be compatible with the old interface. So what's better now?


The only method added for the old interface **is** the array access…


And the old one was not an interface anyways ;-)


If the 3rd argument is a range and attributes are omitted.


I think you mean "idleServers"  ?


If the 2nd argument is a range and attributes are omitted.


Another reason: If Symfony adds new methods to `AttributeBag`, we would need to add them again to our adapter. Not the case when extending.


I think I wrote in the wrong area. This is to allow NUnit work correctly with Autofixture. This will allow us to generate data within the NUnit framework.


I manually set it in NetBeans:

Number of spaces per indent 2
Tab size 2
Continuation Indention size 4


I wasn't sure about it but I'd vote +1.


Добрый день, это строка отзывов или сам чат?


this function is only shown on Yousemite. So it will clear, the current rootkit is only support Yousemite


> The only method added for the old interface **is** the array access…

Nope, you missed `setData()` and `getData()`.

> If Symfony adds new methods to `AttributeBag`, we would need to add them again to our adapter. Not the case when extending.

That's a good point.


Could we rather always suggest using 'container-usage' explicitly?  Even if we
call the 'container-usage' by default?  Just for consistency and to ease the
user's learning..


Pick the upper-most inline ancestor, for cases like `some text <b>in bold and maybe <i>italic | </i></b><img >`


Сам чат
https://gitter.im/AlexanderTyshchenko/JavaOOP_questions


Please replace `<style>` with `<style type="text/css">` and I'll merge it.


на почту должно прийти приглашение


I will understand if you already want to kill me for the way I using formating...


that looks too complicated for me, to create fake entry under replace point and immediatelly replace it


Of course; it's not quite satisfactory yet though as I haven't figured out how to replace tabs with spaces.  At the moment I still get invalid JSON from many of my pages (I think because of the tabs) - once I figure that out I'll document it in a PR.


so from the examples I take it this is a library to handle configuration values? Similar to what we have in FrostWire for Settings, only that this can handle much more complex configuration scenarios, different file formats (JSON) and what not?


I recommend using standard settings, if other people are going to submit patches. But if you insist, i will generate a clean merge request with one commit with all changes. I am doing the return values part now because it is has to get done, can't wait.


For example, `<p> text <b>in bold</b> and then image: <img> </p>`. It is possible to place a caret right before `img` tag so that `commonAncestorContainer` is `p` (not `text`) and the endOffset (=startOffset) is **3** (0. TextNode, 1. b, 2. TextNode, 3. img).


@pthariensflame Can you try removing this? With the new ghc formula, that issue _should_ go away now.


Related issue: #57


Oh no. I am not the author. many thanks and all credit  to @neobuddy89 for his awesome work.


Can you check if this (and the rest of the "with gcc" stuff) is still necessary?


This requires explicitly set readme in rhscl.yaml, it is missing for PostgreSQL e.g.  Could we
do the inclusion automatically?  Its not blocker, just RFE.


what happens if user click cancel? Does it discard changes?


Can you try `-fno-strict-aliasing` instead?


@vcunat, I'll try to change the format ... it just happens to be awfully convenient for [my script](https://github.com/NixOS/cabal2nix/blob/master/update-nixpkgs.sh) to pass the whole message to git via `-m`.


Ref https://github.com/DanePubliczneGovPl/ckanext-danepubliczne/issues/170


`[NSMutableDictionary dictionary]` returns autoreleased object, but when `dealloc` is called, you're autoreleasing it again by calling `[dictionary_ autorelease]`. Do I miss something?


Thanks for all these comments, this is a big mistake I did there.. I will fix these, and squash my commit to not pollute the commits' logs.


>I think you can forward declare QHostInfo in this header.

Fixed.


>Corner case, but let's follow the general rule. No one liner.

Fixed.


>Same here. And I see above there is one one-liner too. Could you fix that too?

Fixed.


Ah, I see. Sounds fair.


>This should be indented.

Fixed.


related to #12 ??


Same error.


>Dont forget to break the initilization lists.

Fixed.


comment this  in dll mode
we will do it manually from managed side :)


LGTM.


>To the top.

Fixed.


>QStringList can be forward declared.

Fixed.


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


Bug here.


Changed in 643cce98575180a68bf29057fa8f28fc81c7c7eb.


There's an option for the Eclipse formatter not to rejoin lines before formatting, and with that enabled it wouldn't have changed this. On the other hand, that will leave many other things that would fit on a line on separate lines, so it's a trade-off.


`sys` is not needed


I tried with just setting `main`, and it seems to work


```
untar-unzip-download:
      [get] Getting: http://download.igrr.me/win32-xtensa-lx106-elf.tgz
      [get] To: C:\projects\arduino\build\windows\dist\win32-xtensa-lx106-elf.tgz
      [get] ....................................................
      [get] ....................................................
      [get] ....................................................
      [get] ....................................................
      [get] ....................................................
      [get] ....................................................
      [get] ....................................................
      [get] ....................................................
      [get] ....................................................
      [get] ....................................................
      [get] ....................................................
      [get] ....................................................
      [get] ....................................................
      [get] ....................................................
      [get] ....................................................
      [get] ....................................................
      [get] ....................................................
      [get] ....................................................
      [get] ....................................................
      [get] ....................................................
      [get] ....................................................
      [get] ....................................................
      [get] ....................................................
      [get] ....................................................
      [get] ....................................................
      [get] ....................................................
      [get] ..............

untar-unzip-checksum:
     [echo] Testing checksum of "windows/dist/win32-xtensa-lx106-elf.tgz"

BUILD FAILED
C:\projects\arduino\build\build.xml:83: The following error occurred while executing this line:
C:\projects\arduino\build\build.xml:892: The following error occurred while executing this line:
C:\projects\arduino\build\build.xml:703: Checksum failed.

      File windows/dist/win32-xtensa-lx106-elf.tgz failed checksum.
      Please remove "windows/dist/win32-xtensa-lx106-elf.tgz" to download it again.

Total time: 3 minutes 39 seconds
```


+1 on deprecating the whole class :)


:+1:  but this will require a change in the `release.sh` script (https://github.com/killbill/killbill-plugin-framework-ruby/blob/master/generators/active_merchant/templates/release.sh.rb#L21).


metadata.yaml


Sorry but I do not understand your intention here. This change is just a JS mod to not watt 1/2 sec.


Here is the [test](http://jsperf.com/toggle-classes) doing it in this way is ~70% faster


Isn't it a good practice to avoid relying on transitive dependencies (the plugins explicitly invoke [Sinatra](https://github.com/killbill/killbill-plugin-framework-ruby/blob/master/generators/active_merchant/templates/config.ru.rb#L4))?


Then `$this->Session` will be the `Session` class again (right now it is the `ArrayAttributeBag`) and we need to replace every call to `$this->Session` in the core. Then we can drop the alias methods in the `ArrayAttributeBag` class again. Correct?


Ack


I was thinking more to a Java Runtime shutdown hook, but having an extra thread spawned just before quitting seems not ideal.
Also, we likely don't want to provide such an interface to user Ruby code, and these hooks should have known maximum run time.
What about just implementing the flush in `shutdown()` (or `innerShutdown()`)?


This test is more of a daughter test since it is checking the sex of the child.


Missing.


IMHO, `propagatedBuildInputs` is ugly and should be avoided if possible (https://github.com/NixOS/nixpkgs/pull/7681#issuecomment-98644182). So if fixing up *.pc files allows getting rid of them, that's a good thing. But it should be done in a generic setup hook rather than per package (like for headers in https://github.com/NixOS/nixpkgs/pull/7681).

Don't have a strong opinion on removing .la files. They've always seemed kind of superfluous to me, so I guess it would be nice to ditch them.


what was the problem with the previous version ?
I would prefer to not have lines greater than 60 characters


`exit!` does not flush with this patch alone.


Hooray! ^_^


`install_packages_script` and `target_install_script` seem unrelated to this PR.


All the partest options known from scala can now be used with sbt partest. The full list is here: https://github.com/scala/scala-partest/blob/master/src/main/scala/scala/tools/partest/nest/ConsoleRunnerSpec.scala
It is also accessible by running partest --help in sbt. Most interesting:
* partest --grep erasure // test all tests containing erasure in their path, name or source
* partest pos // test only pos tests
* partest tests/partest-generated/pos/A.scala ... // test specific files, path relative to dotty
* partest --verbose --show-diff // get more debugging output

Some caveats:
a) currently, sbt partest runs the regular JUnit test first, which generates all the test files, even if only one file is needed. There is no easy fix to this, because in the current sbt solution only the partestRunner task parses the options, so they're not available to the earlier JUnit runner. But even if they were, the mapping from the tests that JUnit sees to the generated files that partest filters on is not straight-forward.
b) sbt partest also runs the other tests (ScannerTest etc.), to isolate partest could add another alias to Build.scala:
`addCommandAlias("partest-only", ";test:compile;lockPartestFile;test:test-only dotc.tests;runPartestRunner")`
c) option --failed won't work because the log files indicating failure are deleted when the files are regenerated on the next run
d) option --update-check won't work because it only updates the check files in the generated directory, not the original ones


Sorry, I didn't read your commit message. This doesn't sound like an aliasing issue (in the past aterm often broke after GCC updates due to stricter aliasing optimizations).

ATerm hasn't really been maintained for a long time. Maybe it should just be marked as broken.


I didn't get the point. Current volume is a property of kOS "processor". The line between processor "context" and processor state is fuzzy.


Yes, I don't like providing `setData` and `getData` because they sort of define a new API. We don't have to remove the `ArrayAttributeBag` in Contao 5 because actually I find it quite nice to access it with the normal array notation :) So I'd rather deprecate the Session method (but leave the core as it is) and remove everything for Contao 5.


Let's see where this goes.

You guys should give Linux a try. :wink:


It seems that many people already have the native Python installed on their Windows systems, and I want to avoid the duplication of multiple installations of Python, which can confuse them. Also, it seems that the native Python works well with a Lean binary generated with mingw tools (i.e. g++, mpfr, lua, etc). Do you see any problem of using native Python and a Windows binary generated by mingw tools?

BTW, We have a [report](https://github.com/leanprover/lean/issues/593) from a user saying that the system is unstable in Windows. We're still investigating the issue. This commit was a part of the attempts to resolve the issue.


Polish translation.


The compatibility name for timeout is `E_OPERATION_TIMEOUTED`. That's available on old pycurl as well as new.

Also, `E_BAD_CONTENT_ENCODING` seems to be available in el7. Wrong commit message?


Javadoc additions


ack


Utility accessor method added.


HashSet refactored to TreeSet


no! its just a config setting / a tool which formats that:

https://github.com/springlobby/springlobby/blob/master/.clang-format

biggest issue is, that you don't use real tabs...


Did this ever go anywhere? I found your post in the mailing list and it sounds like I'm having a similar issue with a complex deploy that needs to **not** sudo when it clones from github. I'll look at the managed sudoers option though.


Listener coupling for adding/deleting/writing Analysis objects from Analyses pane.


Menu actions for analyses listeners.


Cool thank to u @ Paul ...

Hmm.. now i'm confused xD .. but to make things a bit easier we can take a look at the help Text:

for HZ=100:

	  100 Hz is a typical choice for servers, SMP and NUMA systems
	  with lots of processors that may show reduced performance if
	  too many timer interrupts are occurring.

250:

	 250 Hz is a good compromise choice allowing server performance
	 while also showing good interactive responsiveness even
	 on SMP and NUMA systems. If you are going to be using NTSC video
	 or multimedia, selected 300Hz instead.

300:

	 300 Hz is a good compromise choice allowing server performance
	 while also showing good interactive responsiveness even
	 on SMP and NUMA systems and exactly dividing by both PAL and
	 NTSC frame rates for video and multimedia work.

1000:

	 1000 Hz is the preferred choice for desktop systems and other
	 systems requiring fast interactive responses to events.

So i think 300 Hz is perfect since we don't have higher refresh rate of 60hz .. ;-)
1000 Hz makes probably sense on a desktop PC with 80 Hz+ refresh rates ....
But also on laptops with 60Hz refresh rate for the LCD .. i would choose 300 Hz.


Yes really, according to our coding style analyser.


XML I/O


on line 78 you assume not windows - so this will only run on unix and try and use a windows style escaping (which seems incorrect to me)?


Row element implemented as analyses in model.


Yeah, I saw that the build failed. That's weird:
```
[igrokhotkov@igrokhot-mbp /tmp/1]$ wget http://download.igrr.me/win32-xtensa-lx106-elf.tgz
--2015-05-13 12:39:42--  http://download.igrr.me/win32-xtensa-lx106-elf.tgz
Resolving download.igrr.me (download.igrr.me)... 104.131.82.128
Connecting to download.igrr.me (download.igrr.me)|104.131.82.128|:80... connected.
HTTP request sent, awaiting response... 200 OK
Length: 128052524 (122M) [application/octet-stream]
Saving to: ‘win32-xtensa-lx106-elf.tgz’

win32-xtensa-lx106-elf.tgz    100%[===================================================>] 122.12M  1.49MB/s   in 69s

2015-05-13 12:40:51 (1.76 MB/s) - ‘win32-xtensa-lx106-elf.tgz’ saved [128052524/128052524]

[igrokhotkov@igrokhot-mbp /tmp/1]$ shasum -b win32-xtensa-lx106-elf.tgz
7ec8f2ffdf65dc51262fccfa7a68d9ae885c06c2 *win32-xtensa-lx106-elf.tgz
```
So it is indeed ```7ec8f2ffdf65dc51262fccfa7a68d9ae885c06c2```...


mokee branches are called mkl-mr1 ... :/


Check input args (rudimentary...)


Thank you, I can confirm that your three commits makes DVB-T2 work on the TBS6281 with the open source drivers :-)


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


That would mean we break existing clients when we make incompatible changes
to the plugin. I think we only need to touch the versions in the wizard
when we want to utilize a new feature in the Gradle plugin. Otherwise it's
not a big deal if it uses some older release for a while.

2015-05-13 12:01 GMT+02:00 Sebastian Zarnekow <notifications@github.com>:

> Could we use some alias here, e.g. version.latest or smth? I'm afraid we
> won't keep this in sync with the really released versions
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/eclipse/xtext/commit/e38fe0e6b4731063bb20380cf98f5007d272b632#commitcomment-11172454>
> .
>

--
Stefan Oehme
Consultant

Telefon: +49 (0) 431 990 268  70
Mobil:   +49 (0) 151 561 676  77

oehme@itemis.de
https://www.xing.com/profile/Stefan_Oehme4

http://itemis.de
http://twitter.com/itemis

itemis AG
Niederlassung Kiel
Am Germaniahafen 1
24143 Kiel

Rechtlicher Hinweis:

Amtsgericht Dortmund, HRB 20621

Vorstand: Jens Wagener (Vors.), Wolfgang Neuhaus, Dr. Georg Pietrek,
Jens Trompeter, Sebastian Neus

Aufsichtsrat: Prof. Dr. Burkhard Igel (Vors.), Michael Neuhaus, Jennifer
Fiorentino


ack


Thanks. But this gives errors for the ReStructuredText on the PyPi page. See https://pypi.python.org/pypi/plone.app.theming/1.2.4
I have fixed this and released 1.2.5 where it looks fine again. (Plus a few more unreleased fixes on master.)

No worries. :-) But a suggestion: zest.releaser has the `longtest` command. Run it in a package and it reads its setup.py, parses the long description into html and opens a browser tab with it so you can check if it looks okay. It also prints errors or warnings on the command line. (Looks like PyPi itself can give a few more warnings, but okay.)


This is a daft rule as well, these kind of things are handled by the build step anyway (sass compile step does this for us).


It could be helpful to also print the current version:

```python
print("# %r *!?" % local_version)
```


TeamCity shouldn't care about the `.scss` files anyway.


@Tyrael That should be 14 May 2015, I suppose.


`allocation` settings must be nested in `routing`


I would use '%r' here (without checking the other code conventions)... IDK if it's possible to have a package with a trailing space


Could you call `mexErrMsgIdAndTxt` instead? Giving the error an ID lets you check for specific error types and helps with reporting. IDs look like `"filename:errortype"`.


A great first start! Thanks!


parenthesis token should be around the arguments parameter


Yeah, don't know why we run `scss-lint` as part of the Team City build. Mentioned this in https://github.com/guardian/membership-frontend/pull/522. Once `node-sass` goes in there's an even stronger case to not run it as it means we can loose the Ruby dependancy from TC.


@oscherler -

Hi, I'm terribly sorry about this - I made a really bad call by merging these changes in and not fully understanding the implications. I experienced these same problems tonight when I realized internally we've been using Predis for Resque for the last 12 months (at least) and I switched a php-resque-scheduler instance back to Credis inadvertently.

I want to switch php-resque out to use Predis, rather than Credis because Credis does provide rather naive phpredis/socket functionality without dealing with incompatibilities like this properly. I've raised https://github.com/chrisboulton/php-resque/issues/248 to track this.


Merge


Summary of Traders and diplomats by E. Preeg.Rmd

On the overproduction: in my understanding, new markets won't prevent overproduction, just provide outlets for excess production that would drive down prices in the current markets. Is this the sense of what is meant here?

On the second point, do you mean that they were each fighting to preserve their own domestic production?


Hi,

Thanks for your reply and your explanations.

Sylvain


My 2¢ is that the application should do the work to present the address data without the brackets in the front end.


I can't remember any particular reason so I've chocked it up to churn. I'm not sure we even need ```StringKey``` with this new definition of ```get``` and it would be good to remove an unneeded type. There is no loss of generality: every ```Header``` should have a ```key``` and ```value``` so there shouldn't be a reason to need the ```Raw``` header here unless you really need the type ```Raw``` which is just a ```.toRaw``` away.

There is still a lot of work to be done on the headers (for example, #176) which will also be pretty disruptive.


You are talking about "rev-filter" in the commit message but the file name is "ref-filter".
My vote is for using "ref" everywhere (including the branch name).


@gau1991 What happen if that line already present ?


found it, the file went into the wrong folder... will fix shortly.


You edited some of the gradle file... There is your own gradle home path, you need to let the old one.


f


qeweqwe


LGTM


Here, If the first time it is called with unescape=True, and second time it is called with unescaped=False (without setting `value` in between), it should run __prepare_list again, but it won't as __update_needed will be False from last time.


That was a demo for @aancel
Btw, what about the following policy instead of the 60 characters one: one phrase = one line.
?


This commit is wrong and breaks testing on Chrome on OS X completely. :/ `fs.realpathSync` will always return a truthy value so this plugin always searches for Chrome in `~/Applications/Google Chrome.app` and doesn't even check `/Applications/Google Chrome.app`.


Adding attributes "aria-hidden" set to "true" and "tabindex" set to "-1" when sidebar is hidden. If visible these attributes are removed.


This commit is wrong and breaks testing on Chrome on OS X completely. :/ `fs.realpathSync` will always return a truthy value so this plugin always searches for Chrome in `~/Applications/Google Chrome.app` and doesn't even check `/Applications/Google Chrome.app`.

If anything, `fs.existsSync` should be used, not `fs.realpathSync`

cc @zzo


//when you're on MS SQL server, when updated wordpress but still need update the database and it doesn't work this one may help: By Martijn de Boer//
Not really recommended to change core files but as a quick fix you can change the file wp-includes/db.php on or around line ~2202 from $results = $this->get_results( "SHOW FULL COLUMNS FROM `$table`" ); to: $results = $this->get_results( "SELECT * FROM sys.columns WHERE object_id = OBJECT_ID('dbo." .$table. "')" );


get_spot_price_history() should be modified to handle the case when availability_zone is a list.


btw, why does it even look into `$HOME`? Does anyone on OS X have Chrome there?


:disappointed: I have no idea why I thought this was part of the data...? Fair enough.


Ahhh that was an accident. Can you get it back?
On May 13, 2015 12:22 AM, "Leandra" <notifications@github.com> wrote:

> Boo, you deleted my cat clicking stuff!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ltejedor/space/commit/0fc9f17e612c7472720cac4d3daa30d9995e112d#commitcomment-11168450>
> .
>


Yes, if the definition of constant-time contains the requirement that there are no branches depending on secret data -- and it should -- then something like that or the original code should be used.


@MiteshShah
@gau1991 used
```if not EEFileUtils.grep(self, '/etc/nginx/fastcgi_params',
+                                    'SCRIPT_FILENAME'):```
So if its already present it will not add.


Do you need to add a test to "test/unit/flow_test.rb" as in https://github.com/alphagov/smart-answers/commit/40019853db94f1bed31d03c86bda61858a94591f#diff-d78bd362716c9bb2415cb19c49416d5b?


Closed Visual Studio Online Task [vso#4345](https://azurecom.visualstudio.com/DefaultCollection/ACOM/_workitems#_a=edit&id=4345): 'This is a test usersnap task submission'

VSO State: Done


@mzgol this is the default path with `brew cask`.
See http://caskroom.io
More information provided in #42 too.


this is intended, this comment describes next line


Nit:

As "item" is allocated with xcalloc(), its memory should be full of 0, so there is no need  to set item->name[len] to 0.

But perhaps just changing "xcalloc(1," into "xmalloc(" is the best solution.


@mzgol the correct fix is done in #52


Either one is fine.


Policy doesn't have a tensor_type overload.


By the way, fully compatible with ACE3.


New to github


Unnecessary change for the commit


Why not: #include <cstdint> instead?
Moreover, this is c++11: http://www.cplusplus.com/reference/cstdint/
Older compilers as unfortunately still seen in certain platforms may not support it, AFAIK.


Rebase


Docker ftw :metal:


We want this in place just in case the in-memory idea of a request doesn't match what the store has, so we always want to call to the store.


It is probably too early for me to comment here because I have not seen how settings.valid_until will be used:
change
> settings.valid_until = (Date.now() + settings.expiration * SECOND).format("%FT%TZ")
to
> settings.valid_until = Date.now() + Duration(settings.expiration)

This will allow the settings to have string durations like "60second" or "5minute", or "4week"


Why not this?
```c++
uint32 i = 0;
while(s[i] != 0)
    ++i;
```


Need to check if this is required, since the default request handler for tftp handles the NOENT case.


Can I ask that you reformat the code in keeping with the style of the rest of the code.

Will try to publish a code format guide shortly.

Otherwise happy to accept.


I notice that this has been fixed and completely changed in the next few commits ... So, I don't see much point in keeping this, I think these commits should be squashed with 309ab6d499aadd905fe5801071f4381913593fb2 and 328553a4eebce520b938621195bf26fcbb6b6d56


As the list_delimiters is a list, it would be good to have this as a list too. This would give greater flexibility allowing multiple dict delimiters rather than just 1


Hi Andrew,

I am done with the multiple account functionality and pushed it to development branch please review it and let me know your thoughts and suggestions for the same.


Nit:

Did you consider moving the above 2 lines into alloc_item() and renaming alloc_item() into new_ref_filter_item().

alloc_item() is not just allocating, it's also copying the name field.
And yeah "new_ref_filter_item" is a long name but it's more explicit.


I see, I should've looked at the PR before commenting.

Your new PR fixes issues for me.


I'd prefer 60 to 80 characters too, because sources are more readible when
editing in terminals.
Also, for c++ source codes you can avoid the horizontal scroll bar that can
make the pdf version looks bad.

And for the culture, why 80 characters limit?
http://programmers.stackexchange.com/questions/148677/why-is-80-characters-the-standard-limit-for-code-width

2015-05-13 14:03 GMT+02:00 Vincent Huber <notifications@github.com>:

> That was a demo for @aancel <https://github.com/aancel>
> Btw, what about the following policy instead of the 60 characters one: one
> phrase = one line.
> ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/feelpp/feelpp-book/commit/c85f44f52ad8747280bf3cc366d09dd5326acee4#commitcomment-11174088>
> .
>


はじめのうちは主にmasterブランチで基礎的なところを固めていきましょう．ある程度めどがたったら章ごとに分担をして行くスタイルでどうでしょう


It's also possible to change travis.yml:

```
script: composer test
```

and composer:

```js
    "scripts": {
        "test": "phpunit -v --coverage-clover=coverage.clover"
    }
```


First off is the typename I suppose to be there?

The following comment assumes that typename I is not actually suppose to be there.

This is very dangerous just try and compile the following.
```
Tile<Tensor<double>> tile1;
Tile<Tensor<double>> tile2 = tile1;
```
The issue is that U&& will match anything, so when the compiler is looking for a constructor it sees Tile(const Tile&); But the type of tile1 in the assignment is just Tile so U&& can make a Tile(Tile&) match which is a better match than const Tile& so the wrong constructor gets called.

If you already knew this and the typename I was there to stop this some then please disregard this comment.


sexpress is not supposed to be ensime specific :-P


Thanks!


@hannesvdvreken Interesting! I didn't know about that.


ditto


@cemerick Perhaps I'm confuzzled -- where does the .cljs extension come from now? (I don't know this code, just clicked on recent commit.)


put your server address here fqdn or IP depending on how the cert was generated


nazwa metody przepiękna :d to powinno być prywatne


Maybe reword the commit message to say something like:

Rename the struct refinfo members to be more in line with other similar structures.


Cool.
Hâte de voir l'intégration de cette fonctionnalite dans l'application


@littleguy77 could you test this branch on your devices too see if everything is ok?

I mainly rebased this commit https://github.com/Gillou68310/mupen64plus-ae-gsi/commit/733b3775349d5c4b4891227744b2aad7161ab885 using your GameSurface class ;-)


>1 commit to change 1 line of text
>1 commit to edit 1400 files


If you use Laravel's Container functions here, you can create this class with the dependencies already filled in.

You can use:
```php
$caller = app('Tokenly\XcallerClient\Client');
```


AFAIK, may be slower as using std::find() + std::distance() makes a linear + constant or even linear + linear complexity.


`if ($default == '') {`

Ansonsten kann man "0" nicht als default setzen


What is your reasoning behind squashing all of this into a single actor? Instead of "augmenting" handlers, why not simply have two actors instead and forward message accordingly? I'm thinking about https://github.com/actor-framework/actor-framework/issues/227 for a while now and this "fall though" is something I consider harmful. By the way, this is also against the intuition of *every* developer that ever has used pattern matching in a functional language.

I think the real problem here is that you have two separate entities that you want to fuse together while they really should stay separate. Having one "flow-controller" that hides the worker and transparently forwards or rejects messages is a so much simpler design.


@duff see the full discussion here: https://github.com/Shopify/active_merchant/pull/1680

Sounds like there are at least two gateways that will be using it at this point.


This is completely legit patch from upstream perspective and will be merged right away given that curent 4.1 cycles had the baseline changes ! No point in delaying it


Indeed, thanks for catching [that](https://github.com/jenkinsci/jenkins/commit/f728da2bb349a3cf3e27872e5ba0849d15720767).


@lukecampbell LucasPookus.  This is your bag, baby.


=> "Relative"


Listing repos and issues now works for any username supplied.
In this commit I've the username is obtained from the username text field from the Login page. Assuming the Login is successful, the username is sent to the LogginActivity.

However, one thing worth mentioning is, I've noticed that when you go back from Issues to Repos Activity, the gitUserName variable is becoming null, any ideas how to handle that? You will notice this by logging in with your personal Username and any string of characters as password. Your repos are listed and issues. But when you go back to repos activity, the repos default to gdah192's repos and that's because I specified that if gitUserName is null then use gdah192.


Using Python from mingw packages avoid you to install/use others Python's. Our python just work out the box from any location and don't need to set any environment variables to properly work in different location if you move folder in other place.


Hasn't std::search a undefined behaviour when not finding the desired element?
May also be slightly slower. As we got: linear * linear + constant here vs linear * linear before.


what exactly is the reason for keeping the fmt sections etc above stack ?


Disclaimer: Please don't see any of my comments (which are sometimes wrong, btw) as offences. I'm just discussing changes. :)
Feel free to do the same with me.

As for guidelines, please, make sure to:
- Add spaces before and after equalities a = b and not a=b.
- Same for +, -, /, *
- Add spaces after a comma. '(param1, param2)' and not '(param1,param2)'
- Add spaces when using the 'condition ? true : false' pattern.
- Don't add a space after stating and before ending parenthesis: '(param1, param2)' and not '( param1, param2 )'

Some changes are nice. :) The use of size_t is clever for instance when avoiding a cast.
Care to fix the rest?


@Drupaldise - I have just tried your patches but as a result lost the form fields from the right hand column, see screenshot below of what I have on my test site before applying patches. Are you seeing this?

![project browser - backdrop starter 2015-05-13 13-19-15](https://cloud.githubusercontent.com/assets/10559274/7610258/5820bc22-f973-11e4-90c5-b90723bce3fe.png) The test data is coming from
http://www.moneyscripts.net/project_browser/server (the current default server) and is of course all about Drupal modules.


As a superficial reviewer, I'm wondering whether what you are doing is more like "factor out the filter logic from for_each_ref.c into ref_filter.c".

Otherwise, I would expect 1 commit introducing ref_filter and a follow-up commit making for_each_ref use ref_filter.


そうしましょう。ある程度書き方を共通化した方が良いですね。


Need to test but it's clever. :)


Agora sim, muito bom!! :D


Faltou o retorno no método, pode retornar null.


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


Why not a map indexed by compoundFilter key ?


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

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


It would need to be an OrderedMap because order must be assured.


add_warnings (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (348 of 348 tests passed)

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


Yes, plural.


Showlights.xml is poorly formed.  It has duplicate events at the same time and twice as many events as needed (see my PM).  Probably, causing game to hang.


Shouldn't strings be double quotes ?


Yes it should be plural in the has many.
On May 12, 2015 5:12 PM, "BethFrank" <notifications@github.com> wrote:

> Trying to get depictions to cascade delete - @mjy please take a look at
> this branch
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/SpeciesFileGroup/taxonworks/commit/5ce061c3920fa6c8c1d19ab9f77bca9258309234
>


why uniqueCompoundFilterKey-- ? And why uniqueCompoundFilterKey isnot static ?


That you call "easy on eyes"?


While we are thinking about it: is there a nicer way to accomplish [this](https://github.com/contao/core-bundle/blob/develop/src/Resources/contao/library/Contao/System.php#L234-L237)? Without having to ask for the scope?


Why not 0.1.0? The u-prefix breaks existing code and is not backwards-compatible.


Could you explain this a bit ? I have no clue what this does >_<


Yes, exactly.


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

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


Thank you


ack


Eh eh. Me neither! ;)


@nemequ This rule is not about "access" a value stored in the pointed object, but the conversion itself. There seems no corresponding rules in ISO C++.

However, it violates the strict aliasing rules. These rules do exist long ago (and ISO C++ is even a litter stricter), see ISO C90 6.3 and ISO C++98 3.10/15.

__WG21/N4431__

> 10 If a program attempts to access the stored value of an object through a glvalue of other than one of the following types the behavior is undefined:54
> (10.1) — the dynamic type of the object,
> (10.2) — a cv-qualified version of the dynamic type of the object,
> (10.3) — a type similar (as defined in 4.4) to the dynamic type of the object,
> (10.4) — a type that is the signed or unsigned type corresponding to the dynamic type of the object,
> (10.5) — a type that is the signed or unsigned type corresponding to a cv-qualified version of the dynamic type of the object,
> (10.6) — an aggregate or union type that includes one of the aforementioned types among its elements or nonstatic data members (including, recursively, an element or non-static data member of a subaggregate or contained union),
> (10.7) — a type that is a (possibly cv-qualified) base class type of the dynamic type of the object,
> (10.8) — a `char` or `unsigned char` type.
> 54) The intent of this list is to specify those circumstances in which an object may or may not be aliased.

Since `uint32_t` is unlikely as same as `unsigned char` (this only occurs on platforms with `CHAR_BIT >= 32`) and is just not the case here, it is definitely undefined.


 - __style__: [src/main.cpp:261](../blob/d7c090d29b9809504da1e427a5a201023538fef2/src/main.cpp#L261) Variable 'i' is not assigned a value. (unassignedVariable)
 - __error__: [src/main.cpp:262](../blob/d7c090d29b9809504da1e427a5a201023538fef2/src/main.cpp#L262) Uninitialized variable: i (uninitvar)
 - __style__: [src/chaiscript_stdlib.cpp:18](../blob/d7c090d29b9809504da1e427a5a201023538fef2/src/chaiscript_stdlib.cpp#L18) The function 'create_chaiscript_module_chaiscript_stdlib' is never used. (unusedFunction)
 - __style__: [src/main.cpp:169](../blob/d7c090d29b9809504da1e427a5a201023538fef2/src/main.cpp#L169) The function 'get_eval_error' is never used. (unusedFunction)
 - __style__: [src/main.cpp:140](../blob/d7c090d29b9809504da1e427a5a201023538fef2/src/main.cpp#L140) The function 'help' is never used. (unusedFunction)
 - __style__: [src/main.cpp:216](../blob/d7c090d29b9809504da1e427a5a201023538fef2/src/main.cpp#L216) The function 'myexit' is never used. (unusedFunction)
 - __style__: [src/main.cpp:158](../blob/d7c090d29b9809504da1e427a5a201023538fef2/src/main.cpp#L158) The function 'throws_exception' is never used. (unusedFunction)


I will put together the pull request, although it likely wont be ready for
review until next week.

On Tue, May 12, 2015 at 9:54 PM, Peter Atashian <notifications@github.com>
wrote:

> I'll definitely give it a try when I have some time.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/iliekturtles/winapi-rs/commit/71b2a14ce54ff22886fb4ae9a74e828d3e985512#commitcomment-11167336>
> .
>


If it's working, it's a nice factoring. :)


am besten nach var $employeeWidgetContainer = $('.employees') als widget container suchen und dom suchen bei $employeeWidgetContainer starten mit $employeeWidgetContainer.find(


`uniqueCompoundFilterKey` is more of a generator. I decrement it to generate a new unique id as soon I add a new compound filter with the current id.


What is 4? What is 29? Can you either use c characters ('a') for them somehow or have a #define explaining their meaning?


Yeah, whoops. Fixed on master.


You don't need to put a "-" to separate the name and the description.


It's now bit strange as value prop contain a key and a value ...


I don't object to adding this check, but since COALESCE is not destructive to the underlying data we decided to omit this confirmation.


What does this do?


@rldhont this looks like it needs some unit tests - it's the sort of thing that will easily break in future releases


I get rid of that block in a next commit.


one line = one phrase would look quite awkward I think.
I found the documentation paragraphs that were only one line but displayed over several lines, I personally  prefer when the text is nicely presented without having to configure one's editor to support word wrapping.


Hmm two options:

* Replace the `session` service with one that provides an additional `getScopeBag()` method.
* Add another service that provides this method.


You can add a comment for that


> Bad commit summary line
I only shorten the extremely long lines, I thought it's obvious. Sorry, I couldn't withhold my frustration.


develop (lefticus) - Win64-Windows-8.1-VisualStudio-14: OK (348 of 348 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)


Even if it might not change anything, static props are made to do that isnt ?


Some food for though on this change:

I have worked around this by adding an alias like this:

```
def database?(name)
  ...
end

alias_method :has_database?, :database?
```
Excluding all of spec/**/* seems to exclude too much: I commonly have a lot of other code in my spec directory other than matchers.  Plus, I tend to hoist this kind of code out of the spec directory into a common gem which this exclusion would not cover.


blah


Os form de cadastro vão ficar na mesma página mesmo? Não tinha pensado nisso, mas acho que fica melhor. Se for manter assim eu altero no workshop.


Please, one param per line. (I'm nitpicking but it's just for later changes diffs.)


I'd like to see stdint.h types used wherever possible.

In some hotspots there might be performance issues though, but uint_fast*_t etc kind of types can let us use large types where needed for performance while allowing smaller sizes where needed for portability. This is not a perfect solution though since per-arch optimization might not always agree with uint_fast*_t, but I think we should just special-case the few areas that actually matter.

A rewrite to stdint types would likely need to be done manually to split into several bisectable parts rather than a wholesale search-and-replace. The way the crc and adler code was done is ideal, but those were probably the least intertwined parts of the codebase.


Hmm, it should does the job. But having 3 files for a module seems too much. How about creating `avocado.core.tree` module wich `__init__` containing the functions, `node` with node and `debug` with TreeNode debug?

alternatively how about merging the `TreeNodeDebug` into `tree`. I separated them to save memory, but overall it doesn't consume much...


This only saved me 10% in memory usage. Need a different way to do this.


Indeed, a comment from @pasinskim would be great.


My general advice is to mark new fields' visibility `private` by default. It is always possible to increase the visibility later, but it is much harder to limit it later.

(Not that it matters in the case of a unit test in the default package, of course, but it is a good habit to acquire, I think.)