all tests pass:
success: http://build.servo.org/builders/linux1/builds/457
success: http://build.servo.org/builders/linux2/builds/456
success: http://build.servo.org/builders/android/builds/492
success: http://build.servo.org/builders/mac1/builds/673
success: http://build.servo.org/builders/mac2/builds/674


All these `hasOption` tests are testing scenarios with `ngOptions`, but the new `ngOptions` does not use `hasOption` so they are not valid any longer.


Why does `TestPost` inherit from `TestPage`, but not `DocumentTestCase`?


Can you please explain, why `self.doc.created` and `self.doc.created_local` should be equal?


I know, it doesn't matter, but created time is newer than modified time.


Same as above - not valid any longer.


Code style requires a space after the "if" and before the "==".


A new helper for setting the select value without "knowing" the implementation of how the surrogate key is generated.


We should prolly make its price relative if pumping duration up to the others, I figured thats why it was so much cheaper but I didn't ask about it.


fast-forwarding master to auto = ba8cf6b0


This new version of this matcher (moved from the select section above) "knows" how to make the surrogate key that is used in the option value. This is the only place in the tests that would need to change if the implementation of this key were to change.


Tracked options do not use surrogate keys but instead a value computed from the option by the `track as` expression.


Not actually sure if this is necessary any more...


i have some problems here and in the following occurrences:

```
/home/phil/Dev/rust/_foreign_projects/linalg.rs/src/cols.rs:8:9: 8:27 error: mismatched types: expected `core::option::Option<Col<'a, T>>`, found `core::option::Option<Col<'_, M>>` (expected type parameter, found a different type parameter)
/home/phil/Dev/rust/_foreign_projects/linalg.rs/src/cols.rs:8         self.0.next_back()
                                                                      ^~~~~~~~~~~~~~~~~~
```

any idea what i did wrong?


It provides assistance migrating code in the old style

```rust
mod foo {
    #![macro_escape]

    ...
}
```

to the new style

```rust
#[macro_use]
mod foo {
    ...
}
```

If people really want to use `#[macro_use]` as an inner attribute that is fine.  I don't think it's worth having a style lint for this, I just wanted to assist in the transition.


Ooh, nice update, really. Thank you.


Right on.  Thanks.


it means we don't need the YS:Z stuff.


@scottgonzalez This isn't something regarding my fork/PR right? The original commit was https://github.com/jquery/learn.jquery.com/commit/36e510f8f5f01c3b949d5ba7bca65a75c3114a1b.


And thanks for the reply!

"Financial ruin is a real risk for most entrepreneurs, and yet I see much more real innovation in business than in academia."

Yes, precisely - your current customers couldn't care less whether your previous business succeeded or failed, they just want to know if the produce you're about to sell them is any good (even for investors, your new idea matters, not just whether your old ideas were successful). I read your point 3 as proposing to change academic funding from this present- and future-oriented model to a past-oriented model - I can see your logic, but I can also see lots of downsides. Isn't the "you can come from any background and make it if you have a great idea" what makes the good parts of capitalism actually work?


Move this above the `public void Run` and rename the first string to `DIRECTORY` for consistency.


Where these somehow missed originally?


I think the line can simply be removed. Or is there a reason to have it there?


Avoid committing this.


P.S. - Sorry, just realized after both my initial comment and my reply to your reply that this all probably belongs in the comments, not in the PR - will try to remember that (again) next time :P.

Justin

On Jan 4, 2015, at 11:27 AM, Greg Wilson <notifications@github.com> wrote:

> Hi Justin; thanks for your mail.
>
> > 1. I think "sixth mass extinction" (no caps) is a more
> > technical/accurate term than the popular science "Sixth Great
> > Extinction", just in case that matters.
>
> I think what's happening is big enough to merit the caps (and even a
> doomful theme song).
>
> > 2. I don't think that I expect ecology to be "fed" that well - the link
> > to human well-being is fairly indirect compared to, say, energy
> > engineering and medicine, both of which I think will receive a huge
> > influx of people and money.
>
> I've tweaked to explain my reasoning.
>
> > 3. I think a downside of your third proposal is that it makes it very
> > hard to recover from a "bad project" that causes a break in your
> > research chain...
> > Ironically, your proposal could lead
> > science to become more conservative, as it increases the cost of
> > failure.
>
> Financial ruin is a real risk for most entrepreneurs, and yet I see much
> more real innovation in business than in academia.
>
> Cheers,
> Greg
>
> --
> Email longa, vita brevis.
> —
> Reply to this email directly or view it on GitHub.
>


> just curious, what problem in levels did this correct?

Massivity of sprites _in_ levels is not affected by this commit. It just saves me from going mad when changing all the clouds to front passive each time I use clouds in a level. Because in 95% of the cases, you want the clouds in front of Alex/ground tiles/enemies/etc. So now they are front-passive by default.

Vale,
Quintus


Ah, so it just explains what's going on. Thanks!


How about stopping the animation of all StdMeshInstances and starting that animation again? Since reloading does not happen during normal play it seems acceptable to me.


Where are the vendor blobs for flounder?


It does work on the department websites, at least for me. However, I did not yet add any widgets for the departments in the demo_data. Will do so now. You are correct that the sites seem to be missing in the fixtures of the widgets though. We should find out why this happens...

I added some info on what I did and how to use it in https://github.com/jonge-democraten/website/issues/41#issuecomment-68646941. That should also answer your questions. You can set the column, multiple elements per column and the order.


@sauer2 is correct, the line is there on purpose. It is intended to look as if the pipe elements are welded together. Not sure why the line is disappearing sometimes in your video, though. Maybe it is too thin for the renderer to properly decide.

Valete,
Quintus


Also, you can use the "blank" variant of the pipe if you want pipe elements without such a line.

Vale,
Quintus


Also someone needs to come on IRC and discuss with @ljfa-ag :3


Yes, we asserted in the wrong result ;-)


Hard drive failed, chamunks is working on getting it back up

On Sat, Jan 03, 2015 at 10:20:05PM -0800, mibby wrote:
> @zml2008 What happened to the jenkins build server? :(
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/PEXPlugins/PermissionsEx/commit/f14e8a85e24fa9b80250627b02bc53f6e3fd383b#commitcomment-9144867


Merged back to "https://translations.launchpad.net/" too.


I hope the `TestPost` class will inherit test cases from the `TestPage`. You know, the Post model behaves similar to Page, so it's good idea to reuse already written test cases.


keep the `table-bordered`


Brian, I updated the file earlier today, so the console.log should no longer be there. There were several other changes as well.


Sure I can :) They should be equal because they represent the same time, but in unequal notation (i.g. they are different in ISO8601 format).


Sorry guys!  I must have not had "update components" active after I pulled from Github, so I overwrote an old Honeybee Honeybee with my own small changes.  Didn't mean to give you more work.


or it's up to you :)


Actually, not. See timestamps :) I just have to swap comments.


It's mostly just fixing the deadlock introduced earlier in the 2.2 series, which could make fio hang itself on random IO. I haven't made any announcements for 2.2.x yet, will do that soon.


Please add commas to make these tuples instead of strings
i.e.:
```python
INSTALLED_APPS += ('pocs',)
MIDDLEWARE_CLASSES += ('pocs.overrides.PocMiddleware',)
```


I'm glad to have made it. :)

Note that it isn't perfect on OS X yet. If you use three-finger drag and you activate disabled cursor mode on button down, the cursor will have time to travel a bit before the system synthesizes the button event and there will be a jump. I'll leave that for the next release, however.


Same here


And here as well


great thanks ! Will test it !


Thanks ti Davide Dellai


First swing at #4


Cool


Thanks, that sounds good to me.  Can extend to other voltages, ala: voltageUpperLimit, voltageLowerLimit...


this is the bindOne..


fix was already in this [patch] (https://github.com/cocos2d/cocos2d-x/commit/8b4953bbfc5752619678e73e62ac857a1403a2d8).
i suggest we should update our lib to the latest stable version.


Thanks a lot! It arrived at the very right time for me!


Again, sounds good.  I had started with Amperage as at one time I had 'currentVoltage'  (as opposed to voltageMeasured as talked about above) and wanted to avoid 'currentCurrent'  :-)


No worries, this is just the feedback i was looking for - will relook at wordage and orders.


Merged back to "https://translations.launchpad.net/php-addressbook"


We have this in out cm-12.0 as well


> If you use three-finger drag

Luckily, that's _the only_ OS X trackpad gesture I have disabled (because it seems to work poorly, and causes more inconvenience, while offering little benefit... it just doesn't seem up to par compared to other gestures if you care about your performance). I have every single other one enabled (because they're all good and useful; aside from "Notification Center" gesture that I have enabled but never actually use).

But yeah, that sounds a lot more minor and can be tackled with a lower priority. :)


(defparameter *r9* '(
						(flanDeHuitres 6)
						(baronDeLapin 1)
						(ganacheAuChocolat 1)))


Ahah, taycon :D


I never said it was better. Just that this project doesn't seem to run into the problems that Paket solves. I could be wrong though, in that case could you please illustrate with a concrete example.


I think is really important to note that idle_add doesn't run in the background, it runs in the same thread, but its added to the queue in a way that will be executed only when there is nothing more important to do.


Your regexp does not cover "application/ogg", is there a reason for that?


OK, I had missed that the Home page has changed type :).


TeamCity OsmSharp :: GitHub - OsmSharp [Build 921](http://build.osmsharp.com:8080/viewLog.html?buildId=4065&buildTypeId=bt16) is now running


Beware, ["year 0"](http://en.wikipedia.org/wiki/0_(year)) in Go is actually "0001 BC" in PG. Consequently, `time.Date(-10000, time.February, 3, 4, 5, 6, 0, time.FixedZone("", 0))` is "0011-02-03 04:05:06 BC" (**not "0010-02-03 04:05:06 BC"**)

You can try this is PG:
```sql
 SELECT '0001-02-16 20:38:40'::timestamp; -- ok
 SELECT '0001-02-16 20:38:40 BC'::timestamp; -- ok
 SELECT '0000-02-16 20:38:40 BC'::timestamp; -- parse error
 SELECT '0000-02-16 20:38:40 BC'::timestamp; -- parse error
```


The good thing with that, IMHO, is to not expose the packet. But it would be nice to have somthing similar to get the data...


@seriousamlqz I did not remove the file. In this PR, -agent option works fine. I just removed the `baseaspect` parameter of AgentGenerator. I've inlined the base aspect in the generated .aj file anyway; however, javamop's AgentGenerator will produce another base aspect file will cause error and prevent the generation of agent.jar. That's why I need to make this modification.


That is the desired effect.  As you can see, if a function needs the data before there was a bit of downtime for the idle_add to run, the idle_add is canceled and the function run.


ThX for This Neko


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


4: I102 copyright year is outdated, expected 2015 but got 2014
150: D102 Docstring missing
154: D102 Docstring missing
158: D102 Docstring missing
204: D102 Docstring missing
218: D102 Docstring missing
222: D102 Docstring missing
237: D102 Docstring missing
245: D102 Docstring missing
254: D102 Docstring missing
259: D102 Docstring missing
344: D200 One-line docstring should not occupy 3 lines
358: D102 Docstring missing
499: D204 Expected 1 blank line *after* class docstring, found 0
513: D102 Docstring missing
517: D102 Docstring missing
551: D204 Expected 1 blank line *after* class docstring, found 0
564: D204 Expected 1 blank line *after* class docstring, found 0
585: D204 Expected 1 blank line *after* class docstring, found 0
595: D204 Expected 1 blank line *after* class docstring, found 0
610: D204 Expected 1 blank line *after* class docstring, found 0
620: D401 First line should be imperative: 'Return', not 'Returns'
620: D202 No blank lines allowed *after* function docstring, found 1
620: D400 First line should end with '.', not 'x'
633: D204 Expected 1 blank line *after* class docstring, found 0
654: D204 Expected 1 blank line *after* class docstring, found 0
664: D102 Docstring missing
672: D204 Expected 1 blank line *after* class docstring, found 0
704: D204 Expected 1 blank line *after* class docstring, found 0
724: D102 Docstring missing
735: D102 Docstring missing
766: D102 Docstring missing
776: D204 Expected 1 blank line *after* class docstring, found 0
788: D204 Expected 1 blank line *after* class docstring, found 0
797: D102 Docstring missing
808: D401 First line should be imperative: 'Return', not 'Returns'
813: D204 Expected 1 blank line *after* class docstring, found 0
828: D102 Docstring missing
836: D401 First line should be imperative: 'Return', not 'Returns'
836: D290 White spaces around docstring should be trimmed.
841: D204 Expected 1 blank line *after* class docstring, found 0
870: D102 Docstring missing
925: D102 Docstring missing


Why don't we do the same for linux ?


You mean that I changed the name of the JDHomePage class to HomePage? Is that a problem or everything ok now?


Thanks! Will update when I get back from holiday.


Because the plugins doesn't have the _d suffix. (Separate debug info). They do in the windows SDK.


>A car engine may be 'running' without the vehicle actively moving.

So **Running** is suitable option. This is like me even more than Non-paused.
>That's because "paused" doesn't actually have an alternative state. paused is a temporary cessation of a running state, and running is the state between started and stopped.

This is also logical. It turns out that all the torrents running, excluding really stopped. So "Active" is "Running (Active)", "Paused" is "Running (Paused)" and so on...

>"Unpaused" What? That's not even English in American!

http://en.wiktionary.org/wiki/unpause
But, as I said, this word is not appropriate.

For me, if not delve into technical details **Running** is the best choice.


Haha no, I meant that the actual home pages in the demo data now had the type HomePage and thus were editable.


Everything is more than OK.


nice! :-) :+1:


I'm grabbing this in as such (making a release), but feel free to post another change with the strategy I suggested. I'm actually not using this class at all myself (nor Container.Filterable API, that I find architecturally problematic in many cases).


Where is index.js and what is it for?


This change breaks jQuery a tiny bit:

--- a/jquery.go
+++ b/jquery.go
@@ -546,7 +546,7 @@ func (j JQuery) SetData(key string, value interface{}) JQuery {

 func (j JQuery) Data(key string) interface{} {
        result := j.o.Call("data", key)
-       if result.IsUndefined() {
+       if result == js.Undefined {
                return nil
        }
        return result.Interface()


because I have not checked if boost.simd supports them... we do not represent an ndarray of bool as a vector of bits...


yeah, that makes sense -- 4.9.2

I've definitely gotten calls to that in some cases, just not the current code, so something in the GCC version could make sense. The thing is GCC is only used for linking, not compiling...


That's a sse requirement for aligned load, see http://en.cppreference.com/w/cpp/language/alignas


Instead of changing the adjacent elements having difference at least two, changing them to their mean also works and this leads to slightly more optimized solution.


a bit less, but I fear there is still something funky under the hood...


`args` will be an empty collection.
If it is null we have a bigger problem.


TeamCity OsmSharp :: GitHub - OsmSharp [Build 4.3.0.921](http://build.osmsharp.com:8080/viewLog.html?buildId=4065&buildTypeId=bt16) outcome was **SUCCESS**
Summary: Tests passed: 486 Build time: 0:0:0


Fixes #2.


@iamsmith, can you make a PR that fixes it?


Sorry I not see the branch name lol


I’ve forgotten to state it in the message — this commit is related to #16.


r=steveklabnik rollup


Is it possible that the pipeline is not None at some point, and become None later? During the same session? If not, then it would be better to move self._pipeline = None to the constructor. And simply display the error here.. no need to assign None all the times it will be called..


What's the reason for this when we load all modes a few lines below?


Great :+1:


Why?


r=alexcrichton rollup


Yep, you're absolutely right.


Weird. For some reason the email notification I got was from your fork. Didn't even notice how old this was.


r=Gankro rollup


+1 adding back artisan serve


Actually it is DS18B20


cool, add an update to the readme as well with link to the relevant info


Well, thete's this ``fbegin``.... i'll check.


@plumpudding Because the GPL is long and complicated. I have no knowledge of law, especially US law, and I have never fully read the licence. Thus, relying on something that I haven't even read or can comprehend, seems like a stupid idea to me, especially if I can get into trouble with lawyers for that. In four small sentences, I summarized what I want my code to be: open for everyone to view, learn, modify and redistribute as they wish. I don't know why Richard Stallman needed 339 lines of text to explain these simple statements.

Why, do you think my licence is worse?


Problem is that the first behavior in the chain must return the transport message so that its available on the pipeline. How can we make this more explicit compared to the current magic?

https://github.com/Particular/NServiceBus/pull/2601/files#diff-e024e16eec84fd0c249512d237bb9499R51
Also we want to decision to abort also to be more explicit:
https://github.com/Particular/NServiceBus/pull/2601/files#diff-e024e16eec84fd0c249512d237bb9499R55


TeamCity OsmSharp :: GitHub - OsmSharp [Build 922](http://build.osmsharp.com:8080/viewLog.html?buildId=4067&buildTypeId=bt16) is now running


Just tried to use the current master branch with Python 2, and it looks like Python 2 doesn’t like the UTF-8 apostrophe introduced here


Does RIL work ?
I am in the same situation with moto g 2014 :+1:
I have to fix sensors too


That's because the original name of the city was Gnomereagan. But due to copyright infringement with the name of a former president of the US, it was decided to truncate the name.
Normal races chose to keep the 'a' for Alpha & Omega reasons (that city being more an omega than an alpha, though) but Gnomes chose to keep the 'e' because it reminded them of the word 'engineering'.


Can you add the GitHub releases too from now on? Or even the old ones if it's not much trouble. Thanks!


Sure ...
Whatever scripting language you use, it has an iconv implementation.

BTW: Most probably it would already be sufficient to script the download itself to send utf8 encoding header with highest priority.


You don't need this in public lol. Quite useless for most.


would be more efficient to use yaw gyro signal instead of converting to and from NED


I think you need to remove the space bevor `::`.


thanks! I'll correct it in the next PR (today's breakages are for associated types...)!


I think people can find this within the Bower docs. Also Bower will not be part of INIT in v2.0 anymore.


r+


No. I rather prefer to wander the Great Halls of my Dark Throne Room, laughing at all those people unable to set an entry in the `creature_equip_template_raw` table and keep that Dark and Mighty Power to myself. For ever.


CSS Не закончено! HTML закончено. Отступы и соответствие шаблону пока не проводилось. Скинул, чтобы хотя бы что-то посмотреть. С Новым Годом!


You should delete L2446 as well


Thanks for all the commits.

If you want an interesting tasks look at upgrading the guava version which is used to load step classes dynamically. This is a big update as they have completely changed how things work since the early version. I took this reflection packaged code from another library and integrated because it was changing so often and breaking things. Might be easiest to replace it with another tool, not sure.


TeamCity OsmSharp :: GitHub - OsmSharp [Build 4.3.0.922](http://build.osmsharp.com:8080/viewLog.html?buildId=4067&buildTypeId=bt16) outcome was **SUCCESS**
Summary: Tests passed: 486 Build time: 0:0:0


we cannot have paths hardcoded like this, especially not hardcoding ES paths with version numbers


I put it on github so people could use it as a base for their own stuff, but I can take it down if you'd prefer. Just thought it might be useful for someone at some point.


This is just a secondary repo for plugins I maintain that aren't part of the main project, random stuff like this.


Forgot to say have included a test within the code for this Java file


I get why we don't want any hardecoded paths, but we do need some default location. And since the plugin matches a specific ES version, I don't think it's such a bad idea. But in any case, any default locations you think of are probably better than those


Actually, I thought of that, but as that table is deprecated and I still expect to move its content one day or the other to the `creature_equip_template` table, like TBC-DB or UDB. Thus, I'm afraid starting a wiki entry will just encourage people to use that table and delay the switch.
But I have to admit that this switch is low priority and that some help would be welcome in filling either table.


Gefällt mir gut!


Confirmed, although it doesn't look like the active record callbacks pass any additional arguments, so I don't have a use case to write a regression test for this.  I pushed a commit to fix the problem though.


I see. A non exhaustive list of reasons I think paket is better:

 - faster restore
 - easier to manage dependencies that common amongst projects, such as FSharp.Core in this case
 - you don't have to 'reinstall package' if you change fw target
 - it keeps track of direct dependencies versus indirect transitive ones, so you can remove dependencies that disappear automatically
 - nuget's target files have been buggy for me almost every time I've run into them, including when I tried compiling this project (which is why I changed)
 - paket.bootstrapper.exe is a lesser thing to commit than nuget.exe


![image 2](https://cloud.githubusercontent.com/assets/9560432/5607544/3de85eac-9468-11e4-8169-7c7f421b89f5.png)


sat 2 hours wondering what was wrong with the "ofDrawxxxx" calls , turned out they never existed in 0.8.4 .This should have been done close to the 0.9 release :D


МИР ПОЛОН ЧУДЕС!!!
ДОБРО ПОЖАЛОВАТЬ !!!


@iliastsangaris Right now, each difficulty directory consists only of either JS or Ruby, so a subdivision might not be necessary. But once we decide to add new challenges and mix the two, we should definitely introduce the subdivision you suggested.


Great, understand also, I haven't gone head first into everything you are trying to do here. There can be good reasons for exceptions, my point is try not to use them as a basis of logic.

You are correct about resource management, and that is what a catch clause is for, but again, nothing that cannot be handled/tested for before the critical section. As for performance, see [here]("http://stackoverflow.com/questions/299068/how-slow-are-java-exceptions"). This is just an example and does not take into account situational awareness of the code.

As for security of stack traces, there is none, you can only protect yourself by not returning them, think websites. A good example of this is a sql injection attack, the trace will return hints to an attacker. Now in this particular case, it is a client application and this code is not too concerned about security, but the potential is there.

These issues are not java issues, every language exhibit these problems, some worse than others.

Again, not telling you what or how, take it or leave it.


TeamCity OsmSharp :: GitHub - OsmSharp [Build 923](http://build.osmsharp.com:8080/viewLog.html?buildId=4069&buildTypeId=bt16) is now running


Is it necessary to use `down` here?


@brainstorm I think this modification is a bit out of scope. This is the very first python lesson in the novice directory. Adding too many options can be overwhelming. I am also uncomfortable pointing only to Seaborn as there are many options which improve on the matplotlib color maps.

What about: While there is no "official" plotting library, this package is the de facto standard although other libraries exist with alternate color palettes and plotting options.


Given that the trampoline cache isn't really a cache anymore, why not just generate the code into the far code buffer?


Yes, I'm very much aware of how much ljfa would like to discuss this with me. Taking the time to actually resolve the silly stuff before engaging on the rest of it.


r+


No worries, nothing has ever encouraged me in my entire life!


Add a new option to add randomness in the height of the boards


> "Financial ruin is a real risk for most entrepreneurs, and yet I see
> much more real innovation in business than in academia."
>
> Yes, precisely - your current customers couldn't care less whether your
> previous business succeeded or failed

You and I have clearly had very different experiences in startups...


Great change, thanks.

For the currently disabled sub-projects, I think leaving them broken is fine.  If we ever want to bring those back into current builds, we can do the fixes then.

I want to leave them in the current branch for a little while longer in case Chiri rejoins us.  I'm not certain what he had in mind for those sub-projects.


saw approval from jdm
at https://github.com/saneyuki/servo/commit/b33612b483299dd88817a8bccf18d564c7bc5f92


saneyuki/servo/event = b33612b4 merged ok, testing candidate = 57c086cf


merging saneyuki/servo/event = b33612b4 into auto


all tests pass:
success: http://buildbot.rust-lang.org/builders/auto-mac-32-opt/builds/3009
success: http://buildbot.rust-lang.org/builders/auto-mac-64-opt/builds/3003
success: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-t/builds/3004
success: http://buildbot.rust-lang.org/builders/auto-linux-32-opt/builds/2995
success: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-t/builds/2995
success: http://buildbot.rust-lang.org/builders/auto-linux-64-opt/builds/2998
success: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-t/builds/2991
success: http://buildbot.rust-lang.org/builders/auto-linux-64-x-android-t/builds/2986
success: http://buildbot.rust-lang.org/builders/auto-win-32-opt/builds/2662
success: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-t/builds/2660
success: http://buildbot.rust-lang.org/builders/auto-win-64-opt/builds/1499
success: http://buildbot.rust-lang.org/builders/auto-win-64-nopt-t/builds/1490


fast-forwarding master to auto = b2085d96


now we have to consciously deceive ourselves with `make test_short`


miss spelled canola


Yes because some rules need two traversals (https://github.com/aghosn/Obey/blob/master/model/src/main/scala/scala/obey/rules/VarInsteadOfVal.scala) and so it is not possible to apply .down once and for all after the composition. Fusing should help with that though.


TeamCity OsmSharp :: GitHub - OsmSharp [Build 4.3.0.923](http://build.osmsharp.com:8080/viewLog.html?buildId=4069&buildTypeId=bt16) outcome was **SUCCESS**
Summary: Tests passed: 486 Build time: 0:0:0


Oh. Didn't even notice, was in bed, sorry!


nope

On Sat, Jan 3, 2015 at 5:49 PM, Kase0786 <notifications@github.com> wrote:

> Didn't work?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Kase0786/CaloTown-Server-Scripts/commit/eed7b399434c9e22956ef59aa747e44b0b762e33#commitcomment-9143947>
> .
>


I'm ok without having /d lol

On Sun, Jan 4, 2015 at 4:33 PM, Calo South <shadowsouthskull@gmail.com>
wrote:

> nope
>
> On Sat, Jan 3, 2015 at 5:49 PM, Kase0786 <notifications@github.com> wrote:
>
>> Didn't work?
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/Kase0786/CaloTown-Server-Scripts/commit/eed7b399434c9e22956ef59aa747e44b0b762e33#commitcomment-9143947>
>> .
>>
>
>


These broke the build. Please run `npm test` before pushing, so that we don't have a failing build too often (we don't want to get used to that). Fixed in 459a169


Why does this have two parentheses anyway?


This variable, `includeSource`, is declared as a global here because it was forgotten from the `var` list above.


saw approval from nikomatsakis
at https://github.com/eddyb/rust/commit/9a90d436f64b004a55368751b64b850d710fad34


merging 17 batched pull requests into batch


status: {"merge_sha": "5773bdefff2e47cc007f5cc2af3f80b30303d45a", "rollup_pulls": [[20295, "9a90d436f64b004a55368751b64b850d710fad34"], [19963, "652a9acb7331e0b70457db0a16e20ba576135b98"], [20428, "5de78040a221d614cdb379b62ecd86d8789b5eab"], [20442, "c8fcbe75619bd8177cd3054fb2d4b72f1b2d1e2e"], [20449, "36c63a3abebce58dd550c0ba032148eac541d64d"], [20452, "ebdf331133af89eef06bede54a5fe53ad4e21af3"], [20457, "f031285c3a76b86525695bbb5b46daa9e77df200"], [20464, "28cca28e623c1ac5a2a3e7dcdd31b3a90552c9eb"], [20485, "eebe7360de3253986736a5ab826379ff754da6a3"], [20487, "15dedb22f6c93f1531627ecadcc80a39177c4b61"], [20495, "e8078c3fedcaab143bfdc298ce4fff2ba52953d9"], [20500, "5cc17382d17ff2f9eb335c2e80e28089bb2d5810"], [20505, "21617ad620fde0ece2d98b493b5eff474d97eaf5"], [20510, "d5d6de99b175545029d6cb32874a3f5188ee9229"], [20512, "afdddefc1028625a9a180296ad28ac38df5146dc"], [20515, "f863e82b3225bdc8709f98f302ceca6f8675b89f"], [20520, "36bed41c1fa2f5025563de5a6a5b181b05c4336e"]]}


Testing rollup candidate = 5773bde

**Successful merges:**

- #20295 Allow paths in constants to refer to polymorphic items. (eddyb/poly-const = 9a90d436f64b004a55368751b64b850d710fad34)
- #19963 doc: mailing list is deprecated (tshepang/patch-3 = 652a9acb7331e0b70457db0a16e20ba576135b98)
- #20428 Make all integers in the guide `i32`, implicitely (tbu-/pr_guide_int_to_i32_2nd_take = 5de78040a221d614cdb379b62ecd86d8789b5eab)
- #20442 Fix vim syntax highlighting for `derive` (csouth3/vim-syntax = c8fcbe75619bd8177cd3054fb2d4b72f1b2d1e2e)
- #20449 Put links to discuss.rust-lang.org and #rust-internals in CONTRIBUTING.m... (brson/contributing = 36c63a3abebce58dd550c0ba032148eac541d64d)
- #20452 Move rustup to the combined installer (brson/rustup = ebdf331133af89eef06bede54a5fe53ad4e21af3)
- #20457 Remove graphviz::LabelText::* public reexport (frewsxcv/rm-reexports = f031285c3a76b86525695bbb5b46daa9e77df200)
- #20464 Improve `make_hash` function (ranma42/improve-make-hash = 28cca28e623c1ac5a2a3e7dcdd31b3a90552c9eb)
- #20485 Man page/--help dialog fix (ipetkov/man-fix = eebe7360de3253986736a5ab826379ff754da6a3)
- #20487 Fix typo in documentation. (trapp/doc-namespace-typo = 15dedb22f6c93f1531627ecadcc80a39177c4b61)
- #20495 Update guide for Cargo installation (brson/cargo = e8078c3fedcaab143bfdc298ce4fff2ba52953d9)
- #20500 Fix range sugar (globin/fix/range-sugar = 5cc17382d17ff2f9eb335c2e80e28089bb2d5810)
- #20505 Update guide index to point to the task page (estsauver/doc_20504 = 21617ad620fde0ece2d98b493b5eff474d97eaf5)
- #20510 doc: remove incomplete sentence (tshepang/patch-6 = d5d6de99b175545029d6cb32874a3f5188ee9229)
- #20512 Allow rustdoc to accept vector pattern arguments (bjz/rustdoc = afdddefc1028625a9a180296ad28ac38df5146dc)
- #20515 bench: remove warnings from rt-messaging-ping-pong.rs (tshepang/modernise-ping-pong-benchmark = f863e82b3225bdc8709f98f302ceca6f8675b89f)
- #20520 doc: Add missing `$`s in the Installing Rust guide (nhowell/patch-1 = 36bed41c1fa2f5025563de5a6a5b181b05c4336e)


omg = img


I knew, just a reminder.

On Sun, Jan 4, 2015 at 2:41 PM, HE XIAO <notifications@github.com> wrote:

> @seriousamlqz <https://github.com/seriousamlqz> I did not remove the
> file. In this PR, -agent option works fine. I just removed the baseaspect
> parameter of AgentGenerator. I've inlined the base aspect in the generated
> .aj file anyway; however, javamop's AgentGenerator will produce another
> base aspect file will cause error and prevent the generation of agent.jar.
> That's why I need to make this modification.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/runtimeverification/javamop/commit/3aa042d4fa29ca1d50c251c964c458497a0c4ecf#commitcomment-9147166>
> .
>

--
Qingzhou Luo
http://mir.cs.illinois.edu/~qluo2/

Department of Computer Science
University of Illinois, Urbana Champaign


`fn` is missing from this list.


TeamCity OsmSharp :: GitHub - OsmSharp [Build 924](http://build.osmsharp.com:8080/viewLog.html?buildId=4071&buildTypeId=bt16) is now running


Use img.at <uint16_t>(x,y) probably isn't a continuous data array.


Vincent, are you some sort of a demi-god or what?


![image 4](https://cloud.githubusercontent.com/assets/9560432/5607628/1a02fc2e-946b-11e4-93ce-01fdcffd58de.png)


For me Running is too close to Active so I am not going to change it to that unless we also change Active to something else.
If, when v3.2.0 is released, there are users complaining then we'll have this talk again.


There's plans, but nothing right now. FWIW, exposing this today would not satisfy your use case, I think. The logs for a background context won't go anywhere useful.


Yes, so on Linux systems (the main target) we can default to a path under `/etc/`. If the dictionaries aren't found local to the jar (easy to check), fallback to those locations. We can try and look under Program Data for Windows hosts but I don't believe it worths our time.


initial commit


Yes, I looked at this website and they have an example with sse and alignas(16) but what about avx in this case? And why it doesn't depend on the content type?
I may missunderstand it but I think it should be the size of sse/avx register in byte. Am I wrong?


good call on turning `LLVM_ASSERTIONS` on:
```
reducedim.jl
ordering.jl

This application has requested the Runtime to terminate it in an unusual way.
Please contact the application's support team for more information.
Assertion failed!

Program: C:\cygwin64\home\MPC_CAR\julia32\usr\bin\julia.exe
File: /home/MPC_CAR/julia32/deps/llvm-3.3/lib/IR/Instructions.cpp, Line 1658

Expression: ExtractValueInst::getIndexedType(Agg->getType(), Idxs) == Val->getType() && "Inserted value must match indexed type!"
Makefile:125: recipe for target '/home/MPC_CAR/julia32/usr/lib/julia/sys0.o' failed
make[1]: *** [/home/MPC_CAR/julia32/usr/lib/julia/sys0.o] Error 3
```


And we aren't going to be bound to specific ES version. The latest version can run with any 1.4.x variant.


It's fine, let's do it just Linux tested for now...
Thanks, and enjoy the celeb!!


backported in 0d106db


Made file management the responsibility of the caller, and, as well, made library into a universal portable class library.


some tests failed:
failure: http://build.servo.org/builders/linux2/builds/457


Nitpicking, but normally we wrap filenames in backticks.


This clips the dashboard element if it is too high. Need a solution which:
 * takes the height of each dashboard card and applies it to dashboardItem
 * uses a vertical Flickable item when the total height exceeds 50% of the screen. --> that limit needs to be given by the parent element (i.e. from CardsArea)


no maybe he works at Blizzard :)


This is a hack...
The problem is that enqueue method is called from superclass constructor (before time initialization). Thus, time is initialized after first enqueue to 0 (it was probably incremented in first enqueue).
_ makes there is no initialization after enqueue call.


valgrind is only available on Linux unfortunately. I'll try to have a deeper look into the problem tomorrow. Restarting all animations might be a bit tricky, since scripts might depend on the current animation state (including animation numbers and positions).


.run194470052...job...

...runneraan...job...194470052.dat.run.datbsn...job...job...altgractive...

![email_initiator](https://cloud.githubusercontent.com/assets/8031438/5607661/e7f76e6a-9463-11e4-8ca2-995e3df57fdc.gif)


TeamCity OsmSharp :: GitHub - OsmSharp [Build 4.3.0.924](http://build.osmsharp.com:8080/viewLog.html?buildId=4071&buildTypeId=bt16) outcome was **SUCCESS**
Summary: Tests passed: 486 Build time: 0:0:0


hrm, might be a little too much information in this topic, better keep that in the settings page.


@dashengju
It is better to have buildKeyValuesMap() and use Jedis.hmset().
Then we don't need to use Pipeline and gain performance.


Um...


totally right, read the source wrong when skimming over it.


The only thing this class holds is the label. Do you even need the class ?


Could this go out to crates.io, v0.2.13?


This could be done with a single map.. wherein you get rid of Vertex completely and just use the label as the key for the map.


Ok. Than still add PreviewBG as musictype so the screens still don't know about the setting.
Then check this in CGraphics or so.


Typo, it's `homepage`.


no longer needed. I already fixed this in the main repo


add exception


better here!


These class attributes are init at once in the constructor and then never used after that. Why not make these 'final' ?


I just meant the typo :)


David already fixed this in the main repo. so shouldn't be needed.


use 'final'


Option parser error when no args specified


this is a 'recode' diff. but David fixed this properly in the main repo, so I don't think it is needed now.


Done.


Use this : http://docs.guava-libraries.googlecode.com/git/javadoc/com/google/common/base/Objects.html#equal(java.lang.Object,%20java.lang.Object)


...equals(final Object o) {


Will there be a release published? Or blog update to let others know :)


I think we should just state our expectation that `assert_same_model` are to be used on the same type of estimator and not test for this case explicitly. It will throw an `AttributeError` if necessary, but I think error is more appropriate than test failure in this case (unless you can think of an instance where differing models of the same type would actually have differing interfaces).


@arthurvr Thanks :-) will quickly add that.


Thanks!


hashcode() ? Use this : http://docs.guava-libraries.googlecode.com/git/javadoc/com/google/common/base/Objects.html#hashCode(java.lang.Object...)


hrm, might not be the best idea to review comments to pull requests in a train, at night, after a few Gouden Carolus... will `s/so/of/` in the morning :smile:


@seriousamlqz I've updated the name of agent in that doc. Shall we merge now, or do some more testing on the new agent generator?


"final" :
public Edge(final V startV, final V endV, final double c){


How about using `np.allclose` without the try-except?


maybe align these comments too like the others?


Apologies, and thank you.


Some might dispute the subject of Line 10


prevent time out on connect


auto create history tree and files


better ident


But this is pretty minor, and perhaps from an engineering perspective having an explicit message in this cases is best.


add feeds delay


default mode


Objects.toStringHelper(this)
                  .add("startVertex", getStartVertex())
                  .add("endVertex", getEndVertex())
                  .add("weight", getWeight())
                  .toString();


Ah. I figured that was the case for packages as well. Thanks for explaining. Do we need to fix this elsewhere too, then?


Seems that libnotify doesn't really get picked up, plus it needs `libXss` and `libX11` in order to be able to enable auto-away, see:

```
checking for libnotify... no
configure: libnotify support will be disabled
checking for main in -lXss... no
configure: libXss not found, falling back to profanity auto-away
checking for main in -lX11... no
configure: libX11 not found, falling back to profanity auto-away
```


Thanks!


20001 = 2001*


final


r=aturon


This is property of the graph. Feels appropriate to be an inner enum of class 'Graph'


@dashengju I'm wondering why catch statements should be deleted. It isn't important because it's a test, just I'm curious. :D


use StringBuilder


annotate @Override


Мне кажется, лучше преаллоцировать матрицу `gammas` и заполнять её в цикле, аналогично `beta`. Также, если хочется оптимизаций, то можно также переиспользовать `alpha`, `beta`, `gamma` между итерациями.


Not sure *why* I was an idiot and went from 1 -> 3 without noticing... May the semantic versioning gods strike me down (albeit still valid)!


maybe use the expected exception annotation?


"' Open a browser and copy/paste link in url to  further view/manage/modify the image'"


method arg can be declared as 'final'


md = markdown.  Use [Github flavored markdown](https://help.github.com/articles/github-flavored-markdown/)


This showed up as dead code.


Need to use try with resources to also properly close the reader.


If you want you can have some integration tests which test that
1) When user doesn't provide -baseaspect, we will generate that in the .aj
file
2) When user provides -baseaspect, we will use that file and the generated
.aj file will ignore those specific classes (by running the instrumented
code)

Those are a little tedious so if you've already done the testing manually I
am fine with merging this change.

On Sun, Jan 4, 2015 at 3:59 PM, HE XIAO <notifications@github.com> wrote:

> @seriousamlqz <https://github.com/seriousamlqz> I've updated the name of
> agent in that doc. Shall we merge now, or do some more testing on the new
> agent generator?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/runtimeverification/javamop/commit/06df03d036a300a53f19846d311b068d53be7590#commitcomment-9147400>
> .
>

--
Qingzhou Luo
http://mir.cs.illinois.edu/~qluo2/

Department of Computer Science
University of Illinois, Urbana Champaign


No


this line breaks jot for a reason...


Only single backticks when linked this way. `Base` should be unnecessary.


Dead code.


Dead field.


The {{ }} initialization trick for collections looks very cool, but has bad performance implications (an additional class must be loaded and an additional subclass is in the system).


yeah... actually you are right... it should have raised an `AttributeError`... that would make more sense... ill fix that :)


this get() and then null check is bad practice as 'null' may mean that either that key is not present in the collection or the key's value is set to null. To be on safe side, use contains().
I would rewrite the method like this:

if (vertices.contains(label)) {
  return vertices.get(label);
}

final Vertex v = new Vertex<T>(label);
vertices.put(label, v);
adjacentList.put(v, new HashSet<Edge<Vertex<T>>>());
return v;


IMO лучше избегать таких ad-hoc функций, явные циклы проще и понятней :)


oh... I never knew of that method before.. thanks ;)


So `` :func:`mapfoldl` ``?


Looking up the identifier is dead code. I think we should avoid having commented out code in the main source base.


Being in the "disk" group is almost as bad as being root, since you can write directly to the root disk and MBR (and install a bootkit).


ugly


You should have extracted `$prop->getName()` to `$name`. This is ugly.


Вполне можно обойтись и без этих ребят, так как `HMMIndexError` ничем не отличается от обычного `IndexError` по смыслу; для `HMMValueError` аналогично.


same as above


final


ftffjffjffh


comments


can you suggest a change? I've never had my head wrapped around this as well as you do - I'm especially concerned if this isn't accurate yet :).


You could have embedded this as a summary, but the GPL would be more
helpful to protect your work from being used by proprietary software. In
the case you mentioned ("if I can get into trouble with lawyers") in
another context the GPL would also be more detailed and fix more holes the
lawyers try to find.


thank u me toverrunnerrollerrunallidc.datruunershatemp.runaan

JOBidc.jobcom...runner.


it was late... :)
just committed a fix


Don't insert spaces within parentheses. Should be:

```java
if (!thread.isDaemon()) {
  thread.setDaemon(true);
}
```


.allrunnerroller.job...com.toveraanon


@@ -20,7 +20,7 @@

-    		+


yes


@@ -20,7 +20,7 @@

-    		+

 		 .run.job.toverscripteraan
![bulletpoint](https://cloud.githubusercontent.com/assets/8031438/5607751/4e2511ac-9468-11e4-8586-5bfc8463d856.png)


Don't commit this file.


You should be aligning on sizeof vt or use boost simd aligned_array


What is this for?


r+


I guess this slipped in unintentionally?


I should probably add some tests for `fully_qualified_name`.


Doh! Search and replace fail!


Lgtm so long as send want list to peers is a blocking call.

Go wait close is straight of the text book so probably okay!

<-sendToPeers seems like a bit of an overreaction but it doesn't cause any other problems so probably okay. Curious to see how that'll play out over time. (Probably  Harmless.)


I mean: it will raise an attribute error of its own accord! You don't need to handle it specially!


[SideCI/RailsBestPractices] law of demeter
reference: http://rails-bestpractices.com/posts/15-the-law-of-demeter


[SideCI/RailsBestPractices] remove unused methods (Power#irt_incomp)
reference:


Is this a copy/paste error?


I'll do the HTML extension - no problem!

Sent from my iPhone

> On Jan 4, 2015, at 2:38 PM, Otavio Salvador <notifications@github.com> wrote:
>
> I have included the try-out in a new branch and sent a pull request. About
> the HTML extension I have no idea how this works or is used. So I cannot
> help at this moment.
>
> On Sun, Jan 4, 2015 at 5:15 PM, James Carlson <notifications@github.com>
> wrote:
>
> > Could you add a usage example in the try-out directory for the aquote
> > environnment? We should also add code to module Html5ConverterExtensions
> > (beginning at
> > https://github.com/asciidoctor/asciidoctor-latex/blob/master/lib/asciidoctor/latex/converter.rb#L90)
> > to make this environment available the html backend. I use these extensions
> > in http:/www./noteshare,io .
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <https://github.com/otavio/asciidoctor-latex/commit/17311c13cbe33642f9adca86f80a9462bc2d1b43#commitcomment-9146926>
> > .
> >
>
>
>
> --
> Otavio Salvador O.S. Systems
> http://www.ossystems.com.br http://code.ossystems.com.br
> Mobile: +55 (53) 9981-7854 Mobile: +1 (347) 903-9750
> —
> Reply to this email directly or view it on GitHub.
>


Just a request... Please rename it to hotkeys as this was the genuine name for this feature as created by me for openpli....


OK will do, but first I'll be submitting a PR for review shortly that addresses [this discussion](https://github.com/skryl/qbwc/pull/37#issuecomment-67971270).


My 2 cents, because OS X had same issue, i'd like to save a reference/alternative:
mat3 m3(mat4 m) { return mat3(m4[0].xyz, m4[1].xyz, m4[2].xyz); }
Or with OpenGL >= 2.1:
#version 120


This fucking triangle caused the bug? Amazing.


Have a banana, Flummi: :banana:


what do you mean? why is it not used anymore?


Hi! This is a runnable jar. I've been working on a business and getting ac, so I haven't been working on this. If you want to try it out on linux you've got to unzip it and run the jar through java.


yep, we should :)


alright, I'll look for more info about this


I simplified it instead - the Faker way of doing this is to uses masks.


Why not just do `public let filterName = "CIGaussianBlur"`?


@turanct When I want to use a third party script, or download a package from bower, I dislike having to compile it myself (since often I just want to try it out first), so I like having a compiled version in the repository.

Going to move it to a dist folder and add a minified version though. Extra documentation is for when I know I won't be changing anything core soon!


I think that the change in this hunk introduces a bug.

To determine how many suggestions (results) to append, all that matters is the limit and the number of suggestions currently rendered. This commit changes it so that `rendered` equals the number of suggestions currently rendered PLUS the ones that were returned by the async query.

The `rendered += results.length;` line used to be *after* the call to `_append` (which seems to be correct) but now it is *before*.

I came across this while testing 0.11.0. With a limit of 5, if 3 results were returned, only 2 would display (because `that.limit == 5` and `rendered == results.length == 3`, so `that.limit - rendered == 2). Changing it so that the `rendered += results.length;` line is after the `_append` call fixes it. And FWIW all tests pass with that change and without that change.


"kitabın"


Good job unicod3. Greetings. :)


"kitabı."


huh


不知道现在还能正常爬取数据吗？


Oh yeah, I remember what I was doing! Going to unimport that.


Unbelievably nitpicky, but we should probably use `Agent` everywhere for consistency


I'm very glad to see you implementing Reflect also. The reason I started to implement it for es6-shim was to get it into 6to5, which just recently switched to core.js. I'm surprised that it hasn't been shimmed earlier.


why not setMode ?


if(true) exitWith {};

Debug code?


Indentation in Tagaini is done using tabs, not space. This is to allow people to set the indentation they prefer by changing the tab's width in their text editor.

Also in this function, s is a reference, not a pointer, so testing it for NULL is not needed. It is actually misleading as it ends up calling QString::operator==(const char *), which is probably not what you want to do here.


You are the man!


Looks like you mean 'canola'!


Ditto - jot (the status editor) is currently broken. I have to drive somebody to the airport so would appreciate a close look by somebody - or revert this until we figure it out.


Housekeeping: closing out issues.

This issue was independently addressed and fixed. Apologies for not directly accepting this pull request at the time.


Finally ! How about enabling the many other troubleshooting options in the kernel that have no performance impact ? I never understood why troubleshooting has been made so hard on OpenELEC. As if the OpenELEC team prefers users do not understand and fix problems, and contribute easily.


Que c'est laid un commit gros comme ça !


The else is not really needed here, since the condition returns. I think it looks cleaner to handle errors and special conditions this way:
```
if (special_condition)
    return ....;
<normal condition continues here>
```

The reason is that this allows several conditions to be specified without exceedingly increasing the indentation of the "normal" case. It also ends up being easier to read.

```
if (special_condition_1)
    return ....;
if (special_condition_2)
    return ....;
...
<normal condition continues here>
```


ok. That's what it looked like but I was curious ^^


Thanks to this patch, I have been able to spot what triggered this error: one of the KanjiVG entries (091c1) had an invalid ID that included non-hexadecimal characters, causing isJapaneseChar to be called on an empty string.

I have fixed this, but your patch still stands, as this function should not fail on an empty string but return false instead. If you don't mind fixing the inline comments, and adding the same safeguard to isKanjiChar(const QString &s, int pos) that is also lacking it, I will merge your pull request.


- Why you removed recursive include of scripts dir?
- Duplicated autoinstall_templates recursive include


I would prefer a full name rather than REDIR. Also do you need to differentiate between ERROR and FETCH_ERROR. Maybe the tuple emitted could have the name of the component who emitted it instead if one needs to differentiate between a fetch error and a parsing error


Broken how? Do you get any errors in the error console? Works for me. Have you forced a refresh of the browser cache?


Suggestion: `sizeof(info->FileName[0])`

I find it somewhat mind boggling that info->FileNameLength is apparently expressed in bytes.


I like how you added those paragraph tags.


ok


Thanks, I should evaluate it while keeping your points in mind... The issue you mention with Nuget.targets was just with xbuild, right?


The textarea opens but not the title, categories, and toolbar - so there's no way to submit (button remains hidden). I've refreshed and dumped cache and still persists.


is there a like button for commits?


Nothing relevant in the console.


just type :massage:, @jamesbornholt


Can't an entry be both a directory and a reparse point?


This looks good to me.
btw: Thought about a struct while I was making the class. Any suggestions on when to go the struct route? e.g. when there's only an init() that doesn't do much?


Ah hang on - got a message after the second try.

reference error: channelId is not defined.


Can you check the source html if channelId is defined (just view source)? Should be below the justifiedGalleryActive line.


Using ALL CAPS for a constant is kind of a weird convention, idiomatic C restricts that to macros only.


It's been a long time since I worked on it, but when I was trying to make a bundle for Geany for MacOS, I'm pretty sure I got the keybindings all working without any code changes. I think I had just to ship a config file with the correct bindings (maybe keybindings.conf, or geany.conf, I don't remember). Now with the changes to making Geany actually usable on Mac, I should dig up my work on the bundle and see about getting it merged :)


It doesn't say anything about how the primed function is different from the non-primed function. In Haskell at least there is the convention of using public primed functions for strict versions of normally lazy functions (and with the exact same parameters and types), but none of that applies to F#.
In https://github.com/mausch/Fuchu/commit/86c6614f09cc447b141934bdc7d7740235e0a91e I just shadowed `perfTest` defining the `5` argument, just for the test. The argument `5` also just seems too arbitrary to leave as a "default", I don't think it's worth it having this convenience "overload"... (got burned too many times by convenience methods).


Sorry, have to leave for the airport now.


Make a change .


I would like to make this change #2.


missing break/early return?


Thanks for the feedback. But we decided to go with the well-known underscore naming convention for private fields (as opposed to prefixing them with `this.`).


It's not there for me...


I remember when I was working on better MacOS support one of the big reasons I gave up was because of so many bugs inside Scintilla/Quartz, one of the most crucial (to me) was that when you'd type a C-style comment, as soon as you entered the `/*` it would completely lock up and crash. Did you ever experience that anymore? I was using Lion 10.7 at the time.


Waoww... very cool :D


Right, I see what's happening. When not on a profile page, $channel_id is something falsy, so channelId is not defined. If you got to your wall page it should work, but not network. Should be an easy fix. Just a moment ...


Yep, good idea. Will do that now :)


I meant controllers


Good :+1:


Wonderful :))))


:D


This is not needed:)


Written code is also a social interaction and not just a matter of taste.

I will argue that underscore naming is a well know no-no. Depends on who do you know. See? Social.

You cannot get rid of this. Why not embrace it instead of going for rules created for other languages with shortcomings that C# doesn't have?

This is what you get for open sourcing your code. It's not just clapping. It's also pesky guys like me. :smile:


I'm currently working on your issue. So far I found that intrinsic generate random memory overflow. I have a couple of fix to do.


Note that I also started work redesigning the Item and Transformation systems.


> Also, you can use the "blank" variant of the pipe if you want pipe elements without such a line.

In which case you also lose the screws.

Would it be a good issue to open a tracker issue on the disappearing lines, or is it intended (so as to speak)?  We won't be obligated to put it in realease-2.0.0 (or any given version), but it will allow us to keep track of this.


HI


The object parameter is used to specify the object for which the notification center will register notifications. If it's `self` then only the object who calls this method will register with the center... So the following won't work:

```
class obj1 {
    doSomething() {
        post("aNotification")
    }
}

anotherObject.on("aNotification") {}
```

With `self` instead of `nil` the above registers `anotherObject` to run a block when it hears "aNotification" from itself, and not from any other object.


Bez przesady ;p


Looks like i mispelled canola


Thanks, this is merged.


Hmm... console.log?!


Death to the managers.


@otavio could you send me a short snippet of asciidoc text where you use the attributed quote  (`aquote`) environment?


Should be fixed in latest git


See issue HaxePunk#337. This version eliminates Tilesheet.addTileRect() calls and supplies source rectangles instead of a tilesheet-cached tile index.


Maybe mention that the server should be upgraded?


Yes, it was a great job you did, I made a recommendation!

I'm going to open an issue to change all the `%` by `format` on bottle core.


Question:  Are any of these regex matches that contain '?' actually matching anything?  I think the '?' character after index.php needs to be backslash escaped in lines:  40 49 76 and 85.


Is there some situation in which you think people may need to use that global?


this doesn't belong there


composer.phar should go in people's global .gitignore files


I created that method, because I did not want to increase VerifyFixAsync complexity.


I'm not understanding the purposes of these. Aren't those normally for accessing variables outside of the class? But in our case, they're all encapsulated within the class, so I didn't think that would matter.


How come? Did we lose code during our cleanup or was that not implemented yet?


Accidental push of old NetBeans project


What is that file? Do we really need it? Please consider removing unused stuff.


> Suggestion: sizeof(info->FileName[0])

Done

> I find it somewhat mind boggling that info->FileNameLength is apparently expressed in bytes.

It was surprising for me too - it's not documented clearly anywhere, but empirical evidence clearly suggests bytes not WCHARs.


I'm confused. Why is the class being set to final? That blocks people from extending it (I think), which doesn't seem very helpful since people will need to extend it to make changes to variables and whatnot.


Right, FsCheck is tracked in https://github.com/mausch/Fuchu/issues/36


We need a valid Vagrantfile now, so we make one.


Don't worry, ignore me ;)

You fixed this here ... https://github.com/ronalfy/reorder-posts/commit/b6cc1c2726fd1a82224d37a0cd6c5e3ee22b45aa


It is ok by now - I will merge this PR as-is - but consider the following for future PR's
* prefer camelCase over snake_case for JS.
* prefer top level function definition - ```function x(){}``` over ```var x = function() {}``` when possible
* avoid adding JS code to the application.js file directly. Add it to another file and then include it


Do you remember what in particular you couldn't express with FAKE?


That's really packed code. I'll merge it but I would have split into two lines or even three to make it more readable.


Этот класс можно назвать просто Node (звучит приятнее). Если это имя конфликтует с другим классом, его можно спрятать внутрь класса (private static class Node ....).


Мелочь: Приватные методы я бы поместил в конец класса, чтобы важное (public) было в начале


Крутой тест - переопределение System.out! :)


Neat code. :+1:  Nicely structured. You can delete branch, it's merged.


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


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


True. A junction and a "directory symlink" are marked as a directory. But elsewhere in the code we treat reparse points as symlinks, no matter whether they're a directory or not.

It's actually a bit more complicated - in some parts of the code (stat_handle and readlink_handle) some reparse points are ignored (we resolve them and pretend they're ordinary files), depending on their contents. It bugs me a little bit, but it's out of scope for this patch to fix. Besides I don't know what the right thing to do there is.

The `uv_dirent_type_t` enum doesn't allow the distinction between different symlink types (and neither does `struct stat` for that matter).

As a final remark, I don't think the `d_type` field actually gets used on windows; I couldn't find any place where `HAVE_DIRENT_TYPES` gets defined so the user-facing APIs always report `UV_DT_UNKNOWN`. I don't know why this flag was never enabled on windows, possibly slipped past review.


why not just `return ec.curve.g`?


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


TeamCity Zero-K :: PullRequest test build [Build 7](http://zero-k.info:8080/viewLog.html?buildId=2668&buildTypeId=ZeroK_PullRequestTestBuild) outcome was **FAILURE**
Summary: Compilation error: Shared\PlasmaShared\PlasmaShared.csproj Build time: 00:00:03


So what would you suggest? Use a `#define`, or switch to snake case?

My taste has shifted toward using lesser preprocessor defines and more constants. Maybe I'm being infected with c++-isms. But maybe that isn't the best idea?


Hm, I found `.51` in a service host's configuration.  `.211`'s probably correct, since that's what @verm had allocated.  Neither host is up right now though, so it doesn't really matter!


подивишся останню задачу(8), вона проста. я робив по памяті, скажеш чи правильно.
бля, треба з гітхабом розбиратись


Yeah - this is a left-over just in case you are going to ask. Will remove it.


Interesting service, thanks.


Мелочь по читаемости тесто-кода.
Тут не сразу понятно как тестируется функциональность addFirst и addLast.
Я бы просто заполнил лист, явно вызвал addLast/addFirst с каким-то элементом и проверил бы результат.


I'd use snake_case.


@JohnMcLear The animated GIF looks great, it's long and small size (only 328KB) !
May I ask you which tools did you use ? (to produce the animated GIF /
compress it).
Did you use a macro recorder in order to be able to modify your
animation without having to restart from 0 if you made an error during
recording?


Online demos anywhere?


You can also do `event.player.triggerAchievement(Achievements.dopeAchievement);`


Spelling error


Can this use the provided `name` to a give a hint as to which syntext it was?


@kahdojay

Yes.

Yes, using has_secure_password creates a validation that checks that automatically.


no


Looks good, but want to amend and fix the commit message? Can still do that safely since it's not merged yet. Force push.


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


I'm considering rewriting this line by:

1. adding the `-q` option to the `md5` call (quiet mode; outputs the checksum only and *not* the filename/path alongside it). I figured since we really only should be comparing the contents of the files, and should therefore disregard the paths/filenames.
2. removing the error redirection because the installed config file is practically guaranteed to exist always.

Thoughts, @Tyilo?


Those are some pretty nasty performance numbers - will definitely want to only enable it without hunting if we need it specifically.

Do you happen to know if it had to recalculate the hashes often in AC4, or was it even just the map lookups that killed it? From looking at the code I think it's only supposed to calculate the hash the first time a texture is created or the first time a render target is used, and then look them up in a map from then onwards, but I guess if there were a large number of textures being used even that would be enough to have a significant impact.

Once we support filtering by texture in a shader, we should be sure to only do the hash calculation when the shaders that need it are active - hopefully that way we only pay the performance cost when we need it, e.g. only on a dozen or so UI elements rather than everything in the frame.


Oh well :(


r+

Thanks a ton!


Can't believe I didn't catch this... I checked the code itself carefully, but not the code around it enough... christ


271行目に雲井の名はいらないと思われる。(バイト先からなので本格的にはあとで見ます)


No this commit was it me was it @pranavmathur


Oh thank god, that stuff was leaving me a bit nervous.


it is better to use "inline" keyword here, or not?
I am not sure if it could save some performance lost from additional function call.


I am a nice comment


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


Ok, I'll switch the routines over to use CamelCase. Sorry about that - one of my colleagues hates it with a passion and I've got a little too used to appeasing him (especially since he can just NAK my patches if I do otherwise).


Ok, I'll come up with a different CamelCased name for it


I don't understand this diagram. Or specifically, this subsection of this diagram. What do you mean by the "lca for ranges such as ..."?


Better contract would be to return on subscribe current status, and subsequently status updates.


Cool C++ Lol


all tests pass:
success: http://buildbot.rust-lang.org/builders/auto-mac-32-opt/builds/3000
success: http://buildbot.rust-lang.org/builders/auto-mac-64-opt/builds/2994
success: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-t/builds/2995
success: http://buildbot.rust-lang.org/builders/auto-linux-32-opt/builds/2986
success: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-t/builds/2986
success: http://buildbot.rust-lang.org/builders/auto-linux-64-opt/builds/2989
success: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-t/builds/2982
success: http://buildbot.rust-lang.org/builders/auto-linux-64-x-android-t/builds/2977
success: http://buildbot.rust-lang.org/builders/auto-win-32-opt/builds/2653
success: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-t/builds/2651
success: http://buildbot.rust-lang.org/builders/auto-win-64-opt/builds/1490
success: http://buildbot.rust-lang.org/builders/auto-win-64-nopt-t/builds/1481


fast-forwarding master to auto = c6c78667


number_range = int(raw_input("Pick a Range: "))

for x in range(number_range):
    if x % 3==0 and x%5==0:
        print "FizzBuzz"
    elif x % 5 == 0:
        print"Buzz"
    elif x%3 == 0:
        print "Fizz"
    else:
        print x


You will have to change both, the line <?xml version="1.0" encoding=" ... and the actual encoding.

For the actual encoding:

iconv –f iso-8859-1 -t utf8 -o satellites.utf satellites.xml
mv satellites.utf satellites.xml

The problem is:
As soon as someone changes the file here on github directly, its actual encoding gets converted to utf8 on-the-fly anyway but without the declaration at the beginning.
So NOT using utf8 anywhere is really becoming a nightmare more and more.


@senny when removing this deprecation should I just remove the code or should I raise an ArgumentError in case people is passing a excluding range?


That looks good (as far as I can tell), but it doesn't seem to work: http://share-elm.com/sprout/54a8b4a8e4b05f9c59b24c93

If you can figure out why, and fix the code to make it work in Elm 0.13 (or higher), I will happily merge it in.


Try this one @frank-dittrich :  18e5b57

NOTE, I am seeing no problems at all on my cygwin runs.  Do problems show up each time?   I am setting different seeds, and re-running, to see if problems end up showing up.


saw approval from nikomatsakis
at https://github.com/eddyb/rust/commit/c37e7ae0c7f876b32bfac85dd7089762c2293084


merging 11 batched pull requests into batch


status: {"merge_sha": "7279119e1241e4679e77c6cabbdb7daf20ba3d07", "rollup_pulls": [[20295, "c37e7ae0c7f876b32bfac85dd7089762c2293084"], [19963, "652a9acb7331e0b70457db0a16e20ba576135b98"], [20428, "5de78040a221d614cdb379b62ecd86d8789b5eab"], [20442, "c8fcbe75619bd8177cd3054fb2d4b72f1b2d1e2e"], [20449, "36c63a3abebce58dd550c0ba032148eac541d64d"], [20452, "ebdf331133af89eef06bede54a5fe53ad4e21af3"], [20457, "f031285c3a76b86525695bbb5b46daa9e77df200"], [20459, "77786e8c4f35773d8a08637d9100ef9a457593a9"], [20485, "eebe7360de3253986736a5ab826379ff754da6a3"], [20487, "15dedb22f6c93f1531627ecadcc80a39177c4b61"], [20495, "e8078c3fedcaab143bfdc298ce4fff2ba52953d9"]]}


Testing rollup candidate = 7279119

**Successful merges:**

- #20295 Allow paths in constants to refer to polymorphic items. (eddyb/poly-const = c37e7ae0c7f876b32bfac85dd7089762c2293084)
- #19963 doc: mailing list is deprecated (tshepang/patch-3 = 652a9acb7331e0b70457db0a16e20ba576135b98)
- #20428 Make all integers in the guide `i32`, implicitely (tbu-/pr_guide_int_to_i32_2nd_take = 5de78040a221d614cdb379b62ecd86d8789b5eab)
- #20442 Fix vim syntax highlighting for `derive` (csouth3/vim-syntax = c8fcbe75619bd8177cd3054fb2d4b72f1b2d1e2e)
- #20449 Put links to discuss.rust-lang.org and #rust-internals in CONTRIBUTING.m... (brson/contributing = 36c63a3abebce58dd550c0ba032148eac541d64d)
- #20452 Move rustup to the combined installer (brson/rustup = ebdf331133af89eef06bede54a5fe53ad4e21af3)
- #20457 Remove graphviz::LabelText::* public reexport (frewsxcv/rm-reexports = f031285c3a76b86525695bbb5b46daa9e77df200)
- #20459 Remove unused imports (frewsxcv/rm-unused-imports = 77786e8c4f35773d8a08637d9100ef9a457593a9)
- #20485 Man page/--help dialog fix (ipetkov/man-fix = eebe7360de3253986736a5ab826379ff754da6a3)
- #20487 Fix typo in documentation. (trapp/doc-namespace-typo = 15dedb22f6c93f1531627ecadcc80a39177c4b61)
- #20495 Update guide for Cargo installation (brson/cargo = e8078c3fedcaab143bfdc298ce4fff2ba52953d9)


Oh yeah, also why does the `laters` calculation work.

The existing paths fall into 3 groups, listed above as path conditions 1, 2a, and 2b. These form 3 disjoint sets:
1: {`x` where `x` is the optimal path, and `x.open` can be greater, less than, or equal to `stream.start`.}
2a: {`x` where `x` is _not_ the optimal path, and `x.open <= stream.start`.}
2b: {`x` where `x` is _not_ the optimal path, and `x.open > stream.start`.}
For later streams, we need to consider 1∪2b∪{`x`|stream where `x`∈1}

Instead, `laters` starts with {`x` where `x.open > stream.start`}. If the optimal path’s open coordinate is later, then the optimal path’s `open > stream.start` and 1∪2b = {`x` where `x.open > stream.start`}. If not then this generates only set 2b. That’s why there’s the next `if` statement, to see whether to generate the set 1∪2b. Then just add {`x`|stream where `x`∈1}, and it’s done.


Just coming across this, it needs to be wrapped in a `<p>`.


Addresses #3


I understand that ServerHealthReporter works as a filter, for the report(...)  method.  In theory some other fields might change as well. Does it make sense to expose these as Observable<ChangeNotification<InstanceInfo>> with initial Add, and subsequent Modify operations? In such case we wouldn't need report method, and instead return the latter observable, a bit like forInterest methods in EurekaRegistry.


If I remove the deprecation code the behavior will be the same because the [predicate builder already handle these cases](https://github.com/rails/rails/blob/2d9b9fb5b5f6015e66d3ad5cb96bc1ba117fd626/activerecord/lib/active_record/relation/predicate_builder/base_handler.rb#L9).

Should we deprecate `BaseHandler` before removing these deprecations? cc @tenderlove @sgrif


Great summary! :+1:


Can we decouple self registration from health check, and make SelfRegistrationService service use the healthCheck service?


It sure looks that way. I'll push another change to fix it.


Actually I take that back. I checked uTox, and it filters outgoing audio, as originally merged.

@tux3 @irungentoo


initial commit


84694650_EnableCodeCoverage (lefticus) - x86_64-MacOS-10.9-clang-Debug: OK (1576 of 1576 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)


@jurchiks my english is pretty awful，I dont know how to write JavaDocs:(


Αφου δεν setares ta laws opws eipame, mhn paw gia to years... anupartka
initialized :P xxaxaxa stergio ksupna reeeee :PPPPP

evala compiled version BTW treksto

2015-01-04 3:04 GMT+02:00 Merkaster <notifications@github.com>:

> nomizw upervaleis ligo ? [image: :-1:]
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mtsio/sem/commit/837f75c56547b2060fc52a247ff09813e1baa2a6#commitcomment-9144241>
> .
>


I don't think we deprecated passing them to `where`


updated


Yeah, we don't. But what is the point of deprecating in `find` and `exists?` if it still work on `where`? Also, if we don't deprecate on `where` we still need the code to handle `Base` instances inside `find` and `exists?` or it will just work as before the deprecation.


My assumption is to improve caching, but I'm not sure. I don't the we can remove `BaseHandler` without a deprecation cycle there, though?


To run execute mvn spring-boot:run


Yeah, we will need to have a deprecation cycle. My plan is to keep these deprecations for one more release and deprecate `BaseHandler` too. In the next release we remove all three deprecations together.


r+


r+, p=2


Iterating over [0, 1)?


My strong preference would be to change all the logging, though I don't care if it is to this or one of the other logging facilities you mention (which I'm not familiar with).

The only reason I didn't change it all initially was because I was in the middle of another refactor when I decided I wanted this (which should naturally be in a separate logical commit) and I had enough work in progress that I would have created merge conflicts with myself. I am happy to do this, the effort is minimal (it is barely more than a single line sed expression to make the switch to the whole code base).

At the end of the day this is your call, and I'll go with whatever you decide, but first here's my reasoning:

The reason I don't like the existing method is it uses this syntax everywhere:
if (LogFile && LogDebug) fprintf(LogFile, "finally the only thing we actually care about on this line");

That tends to create a lot of horizontal scrolling to see what is actually important on the line, and is IMO fragile since it has the consequent on the same line as the if without using scope - I don't know how many times I've seen someone append something to a line like that (even just i++;) and been bitten because the thing they added is not dependent on the condition.

In my mind the only sane ways to format if are:

    if (condition)
      single line indented consequent without {} - good for short conditions in the middle of a function;

    if (condition) {
      single line indented consequent with {} - good when ending multiple scopes in a row;
    }

    if (condition) {
      multiple line indented consequent;
      scope required for obvious reasons;
    }

    if (condition) {
     // comment
      single line indented consequent;
    }

    if (really long condition that
        goes over multiple lines) {
      single line indented consequent;
    }

The last two don't strictly need the {} scope, but I've been bitten by something similar when the comment was particularly long, so it looked like a multiline if, but wasn't. In general the first form without {} makes sense for short conditions and consequents where the whole thing is obvious, and for anything longer {} should be used even if not strictly necessary.

The only time I'd consider putting the consequent on the same line as the if, is if all other options make the code less readable for some reason. This is usually restricted to a series of almost identical statements where it's more important to spot the differences in the lines than have the code flow better. In these cases I'd generally use if (condition) { consequent; } even if the scope is not strictly necessary, to avoid getting bitten if I add another statement to each line later.

Coming back to how this relates to logging - the generic condition of "is logging enabled" is never interesting when adding a log message (it may be interesting elsewhere when initially opening the log file, but afterwards we don't care. ever.), only the message itself is interesting, and which log level (info, debug, etc). Explicitly putting these conditions in everywhere we print a log message would make the code horrible as you either put the if on a separate line, which will only emphasise the thing we don't care about, or do it the way it is done now which as I've said is IMO fragile.

There's also an argument to be made that using wrappers may make it easier to switch logging mechanisms later, but given we have to put the effort in to switch the existing code base to use the wrappers that argument doesn't apply to the code base as it stands right now.

At the end of the day though this is your call. If you want to go back to the old style of logging we can do so, but I wanted to present my reasoning before making that decision. I'm not familiar with Log4C or Boost, but I'd certainly be happy to consider moving to them as another option.


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


Where did you pick this commit? I'm asking because the squash commit I made a while ago has much more changes. Have a look here: https://github.com/hellsgod/hells-Core-N5/commit/74a4813d0ae39cf506dfd33b501098e6abbf9852


Why does the outer- or inner-attributeness matter? Why would this check only happen if it's `macro_escape` and not `macro_use`?


Nevermind, I found the cause. You already using some of the commits: https://github.com/engstk/hammerhead-5.0/commit/f47978c19840dcd6fc96bf746794d616d509d49c


@DavidSouther Added the actions convenience method. I like where it is, for now, but feedback is needed (as always).

Also thinking about renaming the repo to song-flux if the branch gets merged, since it would be more than just a dispatcher.


84694650_EnableCodeCoverage (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Tests Failed

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


单词拼错了


The client that creates the publishes is the one who injects the original array of events. Since they have access they can add events to the original events. I use this technique in UI.components. What's your use case?


Marvelous - some day I'll sit down and figure out how to use GIT...


使用 debug   http://npmjs.org/package/debug


Let's chat on Monday about this.


Not sure which of these commits but swoop firing is totally broken now.  Mega rapid fire.  Uses pistol looking projectile instead of the red bolt etc.

Windows.


The render is a bit weird → do some padding and add margin to basic content


例えば Linux 環境tでは、普通空白は除去されないと思います。
ext3 などでは、null と / 以外の全ての文字を使えるので、空白のみのファイル名もありえますから。
ただ、最後に空白を含んだファイル名を通常望まないと思うので、この変更は全ての環境でうれしいと思います。


上面的 html 用 ````html，删除 window.seajs 那一段，参考 calendar/index.md


"hidden"?


84694650_EnableCodeCoverage (lefticus) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


R主加油~


These lifetimes can be elided.


some tests failed:
failure: http://buildbot.rust-lang.org/builders/auto-mac-64-opt/builds/2995
exception: http://buildbot.rust-lang.org/builders/auto-mac-32-opt/builds/3001
exception: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-t/builds/2996
exception: http://buildbot.rust-lang.org/builders/auto-linux-32-opt/builds/2987
exception: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-t/builds/2987
exception: http://buildbot.rust-lang.org/builders/auto-linux-64-opt/builds/2990
exception: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-t/builds/2983
exception: http://buildbot.rust-lang.org/builders/auto-linux-64-x-android-t/builds/2978
exception: http://buildbot.rust-lang.org/builders/auto-win-32-opt/builds/2654
exception: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-t/builds/2652
exception: http://buildbot.rust-lang.org/builders/auto-win-64-opt/builds/1491


maybe not? https://ci.appveyor.com/project/StefanKarpinski/julia/build/1.0.1138/job/pdq0x7r5rp4i7pg7


Well I'll be. I expected SDL to handle this better, oh well, it's not a big deal to work around this.


This is my first time using github as work. The last time mainly to "fork" and download only


onSelect 应该设置在最顶层 Menu 上


This might be clear to you now based on the comment on the other PR.

The use of `--foreground` is intentional with Upstart, since it monitors the process itself. Upstart prefers processes do not run as a daemon (although technically it can handle that scenario).


点击为什么不回改变  active 项？


My bad. Wasn't really thinking on that one. The prior commit was already the correct one. Reverting.


subMenu 不支持左右键展开，收起菜单


This should be indented by a tab!


It is supposed to filter user input after all AFAIK (i.e., outgoing from client)…


Tabs! :wink:


这里建议直接使用fis.util.filter(path, null, settings.notice.exclude)来过滤，保持FIS中对exclude规则处理的一致性。


saw approval from alexcrichton
at https://github.com/japaric/rust/commit/351409a62287c7993bc680d9dfcfa13cba9c9c0c


merging japaric/rust/derive-self = 351409a6 into auto


status: {"merge_sha": "470118f3e915cdc8f936aca0640b28a7a3d8dc6c"}


japaric/rust/derive-self = 351409a6 merged ok, testing candidate = 470118f3


不要使用 a 标签了，使用 span


:bow: Thank you!


I've been stripping the NULL byte from the assembly in Helix mod (my shadertool.py does it automatically) and it hasn't caused any issues so far - the DX9 assemble function produces the same results whether or not it's been given an extra NULL (after all, it doesn't expect the game developers to put it in in the first place, it's purely an artifact from the disassemble).

Really it boils down to the fact that a NULL byte is illegal in an ASCII encoded file, so if something attempts to guess the file encoding it won't be able to determine it. This may be less of an issue in many Windows tools as they tend to assume .txt files are of a given encoding (usually either cp1252 or UTF-8, both of which are supersets of ASCII, but systems in other languages may have different defaults, and applications can vary).

When tools need to deal with the rest of the world (Internet, anything non-Windows, git), they can't rely on the extension or make assumptions about the file type or encoding, so they either use something else (e.g. encoding specified in a HTTP header, or a parameter on the command line) or failing that scan the file and guess it (libmagic is the library almost universally used for this).

In terms of real world why did I make this change right now, it's this:
https://github.com/DarkStarSword/3d-fixes/commit/ac6470011aa2e3876a380b52abe6833f121dd4d6
In that commit a number of changes are not shown in the diff because git saw the NULL byte and assumed they were not text files as they don't match any encoding (2C7BC6C7.txt, B5B5CF02.txt, D0ECABBC.txt & E42806B8.txt - the others I'd already stripped the NULL byte which is why they show).

This doesn't happen as often in files dumped from 3Dmigoto, as it usually stops scanning the file before the end of the HLSL and hasn't seen the NULL byte, but that's really just an optimisation and doesn't change the fact that the file has an illegal character for it's encoding.

I've been using this pre-commit hook to prevent accidentally committing files with these NULL bytes, but it uses the same logic as git, so it misses the NULL byte in longer files:
https://github.com/DarkStarSword/3d-fixes/blob/master/useful_pre-commit_hook


r+ rollup

Thanks!


I think this wants to be indented one less so that it hits both cases of the preceding `if`.


r+

Thanks!


组件只能有一个焦点，就是 menu

<div tabIndex=''0">


Throwing a RuntimeException is typically a bad idea. There are 2 problems, one with exceptions in general and one with RuntimeExceptions. When throwing an exception you have to ask yourself, is this really an exceptional exception? If it is not exceptional, then it is definitely not an exception. But you may say that we want to know about what happened and why, log it. Does the application really need to stop running? Is there anything else that  can happen in which to avoid this situation? So one should ask themselves these types of questions when throwing any type of exception.

Next there is the RuntimeException, there is a security concern and a performance concern here. Both are probably not too terribly high on your list but you should be aware of them none the less. The security concern is when the exception is thrown, java's VM has to stop and gather information that it has. Next because this is a RuntimeException, java asks the OS about things, the OS then must stop what it is doing and deal with java. If the OS does not have the answer then it is has to stop the hardware and ask it what the hell is going on. So not only was that a mouthful, but it is huge performance hit. Now if any of the information that the OS or the hardware gives up is sensitive in any way, well it is now in the open and clear for everyone to see. Things like paths, passwords, etc.

Any of that make sense?


Great. I think you'll need to redo the pull request for it to go through.


Hey Sorry,

I actually don't know what "r+ rollup" means and it wasn't on the contributing.md. Does that mean there's anything else I should be doing to help this along?

~Earl


Me likes!


You should use the Red stack for a temporary slot rather than a static allocation (static allocations are  not thread-safe and they make the resulting binary bigger and).


The noexcept keyword is not supported under Visual Studio 2013. Since this branche is used in bandit, it also prevents the testing framework to compile under VS.


<thffff


Coding style - ok, will change this.

The only reason it's in the DirectInput file at the moment was to change it over to use the new infrastructure (really? the SDK example used these static Action arrays? I'll have to take a look at that, because that is really nasty), and as soon as I write something using GetAsyncKeyState that's going away and this will move out to a separate file.

Regarding the void* on the device - the only reason I did that was because this file is not in the DirectX11 project, so I couldn't go and use DX11 types in it. However, since most of the other projects are not enabled (and only the DX10 project has any indication that it was planning to use the input code), it might be better to move it into the DX11 project and worry about making it generic later if we enable another project that needs it.

Do you have any thoughts on the void* private_data? This is something associated with the key binding when it is initially bound and would be used to find the correct data structure associated with an instance of a particular kind of keybinding (e.g. when I add generic bindings to the ini for const overrides it would point to a struct describing them). This really comes from doing OO in C, so there might be a cleaner way to handle this in C++ since the language is natively OO, but I'm less familiar with C++ so I just went with what I knew.


This is definitely something that needs to be done for the levels.
Since levels are autogenerated by the level editor, though, if the xml structure changes much for a level, these changes will probably be difficult to see through conflicts and just have to be reexecuted after merges.


Macros are still global in Red/System, so try to make their name longer to avoid collisions. You can prefix them with the context name for example. #define QSORT_SWAP(...)


remove semicolon


これは素晴らしいですね！弾幕飛ばし放題の処理に感謝です！


Yeah, after writing this I'm tending to think it might be better to ditch this macro and switch to just using the iterator in the code. Since this needs to get detached from the DirectInput code I might wait until then since this is debugged for now.

These for_each macros can be very clean in C (e.g. take a look at the list_for_each family here: https://github.com/torvalds/linux/blob/master/include/linux/list.h), but the extra iterator object complicates things here and made the macro a wee bit ulgy (seriously, I had to use __LINE__ to ensure the iterators had unique names if multiple of these were used in the same scope - yuck).


Looked for this feature for a long time. Any chance to merge it into 5.0.0? Applied the diff to my private 5.0.0 and it works like a charm.


@64FF00 :+1:


Why? The latest version of https://github.com/passwordmaker/java-passwordmaker-lib is 0.9.9.


Good question.  I actually don't know :-)  Its not even 0.9.10 on my local computer. I was half way expecting me to have forgotten to push some changes up to github.


`use_lzma` should not be a compile flag. Compressed and Uncompressed binaries should be both contained in packaging tool.
For xwalk_core_library, we probably need to create `xwalk_core_library_compressed` besides existing `xwalk_core_library`.
BTW, assets can't be used in Android Library project, you probably need to use `res/raw` instead.


I will revert this change, since I find no proof of anything that 0.9.10 exist.


A couple comments on this one... `math::RandomSeed()` sets the seed properly for Armadillo (via `arma_rng::set_seed()`, which is necessary post-3.930) and also for mlpack random number generators.  The second comment is that `srand(time(NULL))` can cause test results that aren't reproducible between runs; if I never set srand, then it's (effectively) set at compilation time, and I can run the test over and over again to cause a specific failure which I can then debug (if you see `srand(time(NULL))` in any tests, it was probably accidentally left there during a debugging session and should go away...).  The last comment is that setting a specific random seed for a given set of coordinates won't necessarily give the same coordinates on different systems or different Armadillo versions.

Anyway, at the end of the extensively verbose explanation... is setting the random seed to the specific fun-to-say value you've chosen necessary, does any randu-generated set of starting coordinates seem to work, and/or should we extract the 0xDEADBEEF starting coordinates to a file and load them from there?


why is `absolute` positioning used as default instead of `relative`?


afdasfdaefef


Good


Ah no worries! This is an indication to our automation that this PR is ready for merging. You can see the [current queue](http://buildbot.rust-lang.org/bors/bors.html) and this PR will be tested/merged once it reaches the head of the queue.


Yeah, I kinda took a shotgun approach here, but as I was converting things over `from()` didn't really seem that useful in the end.  There's not a huge benefit to avoiding pulling in extra functions, so better to go with simplicity.

Quotes are fine; backslashes are the issue.  It seems like dash processes backslashes in variables when expanding, which makes very little sense...import won't have this issue though, so there's another point for that.

I'll dump from() (but keep it in git history because it's still kinda awesome) and switch over to import for everything.


The variable "subeffect" needs declared as a local before the if/else branch is reached.
Just sticking:
```lua
local subeffect = 0;
```
above the other 3 local vars should be fine.


82d4bfbed117bd27309a2e6a746c044aeee6e4bb
fc1451e16f77b677b24751ea26edd552d87e316c
6418468b7556f36cc56625025fc42be914b3eacc
6418468b7556f36cc56625025fc42be914b3eacc

takes care of 5.0.1 for JFLTE,  Kernel sources are way ahead of Samsung


84694650_EnableCodeCoverage (lefticus) - x86_64-MacOS-10.9-clang: OK (1576 of 1576 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)


just found this, sorry. I thought it was like json, use `json.load` for a stream or `json.loads` for a string


You should add a section in the help file and submit a pull request back to @benmills


Now freqt can accept SubArrays


r=aturon


Fixed a grammatical error


Bad copy/paste here, which results in errant '+' artifacts.


Almost a year later, I'm editing the source code on this, and I still understand why this change was made.

MrMarvin, why did you switch all these to using lambda? My python-fu is not strong enough to understand currently. =\ I would like to become a better programmer, and this is definitely one of my weak spots (lambda functions... eh...)


Thanks, fixed: 19222bfe8962c9987aa96d7bdeb0b5f2ce1505ba


yeah i'm pretty sure this test should still be `@unix_only` for now, we didn't resolve the undefined behavior from msdn right?


84694650_EnableCodeCoverage (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-Debug: Tests Failed

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


As we discussed, map<char, node*> would also be good enough. Because the size of the map is constant, is not varying, the insert/lookup complexity of map, which is O(log n), will be bound to O(log 26) = O(1). Although you reason the same about std::vector, saying its effective asymptotic complexity in this case is O(1), even though it is usually O(n). std::map would be slightly better if you were to benchmark this code.


I think this should read `if io.read():lower() == "y" or options.q or options.quiet then` the script should have returned already if options.h or options.help were enabled.


r+


@Quintus, just curious, what problem in levels did this correct?


I don't think this is needed anymore.


Could you put up a new Download Link for the JAR? The latest link is still 1.4 - Thanks in advance


saw approval from jdm
at https://github.com/saneyuki/servo/commit/bf2a7330558eeb7f4cb7b17dfab08e13b8100df7


merging saneyuki/servo/internal = bf2a7330 into auto


saneyuki/servo/internal = bf2a7330 merged ok, testing candidate = df04b9ad


@zml2008 What happened to the jenkins build server? :(


Found error and fixed it


84694650_EnableCodeCoverage (lefticus) - x86_64-MacOS-10.9-clang-Debug: OK (1576 of 1576 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)


これやめようや


dosnt this boss need weather support
https://github.com/scriptdev2/scriptdev2/blob/master/scripts/eastern_kingdoms/zulaman/boss_akilzon.cpp
not 100% sure so dont flame me if im mistaken 8)


@peti I'm going to assume you didn't mean to come across as you just did, and will talk again later when things have cooled down.  The world is full of all kinds of people, and if you want to collaborate on an open source project, you're going to have to learn to get along with them.  Mistakes I've made can be corrected; but attitudes like this have no place.

If it is not already, please document the cabal2nix collaboration process on the Wiki, for the benefit of others who would like to help maintain Haskell packages.


some tests failed:
failure: http://build.servo.org/builders/mac2/builds/670


i want to see it


Why is this needed?


I think you should accordingly modify `Scratchpad.js` to use these functions as well.


@pranavmathur made the commit, but we were trying to figure out who changed sensors.c. @pranavmathur said it was not his change.


Thank you for thinking about this, I remember when I asked back in 7.x times about portals and claims. :)

That said… can we ask for a preference toggle disabling this? Like I mentioned on dev.bukkit.org page, we’ve been doing our own custom way of dealing with portal travel and now it’s hundreds of lines of code to accomodate for edge cases (as we do direct 1:8 linking as well). I hate to be a whiny contributor but using the highest event without a way to disable this functionlity throws a big monkey wrench into our solution. Would be a shame to have to hack around it if you could help your 3rd party scene using your API (as small as it is!).


strasznie dużo zmian, które już w masterze wyglądają inaczej.
Spróbujcie się domergować mastera, bo potem będzie się ciężko ogarnąć.


I've just published version 1 to the app store. Just waiting for Google's process. Should be a few hours maybe...


The filename needs to be different than the variable name. For example, instead of saying `var PDFObject`, and `pdfobject.js`, try `var ObjectPDF`


You'd better start working on a plugin before creating tons of repositories that you might never have finished.


Why are you requiring this and `FeatureSelectionManager` below?


Add a line space above here.


I suggest breaking this onto multiple lines.


thanx @Harakhti this is causing the issue https://github.com/sirmordred/android_device_samsung_ancora/commit/ecfeab97fab7edbf71916b26b72bfd946568390c#diff-6fe4127564d4ec5c5438d54a7d876c61R48
ım currently tying to fix it


Add some line spaces in this method - it is very dense.


r+ thanks!


A number of unused lines here.


Looks like its up now https://play.google.com/store/apps/details?id=com.aaronjwood.portauthority.free


每次查看都保存到 view_history，这种存储数据的方式，当 view 的量很大的时候是很恐怖的，比如每天有100万次 view，那就要生成 100 万条数据，这对存储还是数据统计都将造成极大的压力。
建议你使用 redis 的数据结构来存储 view，比如用 Redis Hash 来存储。


Hey sorry for the super late reply, I just finally figured out Github notifications! :)

Nice work on your RapidGame. Hope it helps people a lot.

You can communicate with me on here or my email is on my website: http://wizardfu.com

Cheers.


这两个过滤条件有问题，统计热门 topic ，热门 topic 不一定就是 24 小时内生成的，比如几天前发的某个 topic，因为最近浏览量很大，也可能是热门的，同样一周热门 topic  也不一定是在一周内发的 topic。


和 24 小时热门一样，不一定是最近发的帖子。