Hello! I have a question. When the client send the Login message, it is also encoded using RC4? if true, the key used for encryption that is: `fhsd6f86f67rt8fw78fw789we78r9789wer6re` + `nonce` = `fhsd6f86f67rt8fw78fw789we78r9789wer6renonce` ? I'm using Java RC4 implementation, and the type of the key is `byte[]` and I putting this key as String, the function `String.getBytes()` is valid for this propose?

Currently I have this Login message:
```
27 75 00 00 f5 00 06 02 3e 12 b2 f6 ed 7e 7c 2f
b3 89 f8 f8 bd 34 a6 a2 d4 f9 26 11 d7 d4 82 93
ab 70 12 86 11 72 bc 3b cf e9 32 20 b6 5e 3a 91
3b c4 6a c8 70 cf 4f ea 74 41 44 3e 64 3f 0d fb
3d 4e 5c 7b 5b 12 39 29 6e 26 31 35 a2 bb 40 0f
dd 9e 93 5d 42 55 fe 4c f0 ff 4b d8 7e d6 35 98
96 a8 2b 50 75 3e 3a 3a 9a 99 82 17 64 0e 12 47
c3 af 15 3a fc f2 e6 35 1d e5 5a bd a9 d3 0a a3
9f 55 94 22 df d4 27 96 8d 14 c0 58 24 76 e2 9a
de b9 42 77 55 68 cc 91 a6 18 b9 14 0b e4 43 3f
76 f6 f8 27 ff 22 04 75 53 c2 8f f2 1d 7a 2c dc
0c 35 c4 a6 7c 7d 8b 52 9f 68 b0 50 5c f4 16 76
79 75 16 8f 21 78 5d 91 23 db 47 8b 3a 36 f2 ec
cb ad 7e 55 42 5d 3f 53 28 aa 50 3d f6 10 48 19
52 8b 22 91 31 ec 79 71 63 1d a0 a3 62 24 68 ac
6e 0d 8e 8a 28 51 0d 3c e2 8b 5d 75 37 b6 00 00
1e 00 00 cb 86 a3 4d a2 d8 39 f4 0c 6f 36 31 ab
8a 3a 2a 59 99 3b 73 37 89 65 65 a9 81 9d 33 2f
77 37 16 00 00 0c 00 00 af 1c 99 fd 86 eb 89 6c
de b5 c4 7a 27 7c 00 00 00 00 00 37 16 00 00 0c
00 00 7a 7a 44 d1 ad ad 1a 7b b5 5d 3f 28 27 7c
00 00 00 00 00 37 16 00 00 0c 00 00 93 e9 6a 2b
66 ac f1 14 99 ad 92 86 27 7c 00 00 00 00 00 37
16 00 00 0c 00 00 2e 0f 19 f6 71 be 3f a1 a1 b3
5b 4e 27 7c 00 00 00 00 00 37 16 00 00 0c 00 00
5e 72 88 18 b7 44 ef a1 4e d6 0f 52 27 7c 00 00
00 00 00 37 16 00 00 0c 00 00 06 81 f2 b1 04 83
9b de f6 69 2f 4c 27 7c 00 00 00 00 00 37 16 00
00 0c 00 00 ac cd 96 fb 99 ae ed 06 80 6c 46 44
```

In your documentation you say that `27 75 00 00 f5 00 06` is the header:
- `27 75` message type (0x2775 = 10101)
- `00 00 f5` The payload length
- `00 06` ignored\unknown

Then are 43 (key length) bytes skipped and the rest are decoded using rc4?

Can you submit the decoded message? Than you very much!


That breaks the primary flow: namely describing what TTL means.


map of entries, then return mapToArray ?


This  loop is different because it has two values. The first is `groupId` when the group already exists, and the second is `hashPosition` when the group does not exist.


@dkayiwa this is the person that this problem is set for. But should I change it? In Allergy and ActivityListItem classes it appears like here.


we do we need to do this twice? if it's always done it `before_all` it shouldn't need to be done in `after_all`


wrong phpdoc here. It should be ``Future`` or ``\Cassandra\Future`` (I used the short class name when fixing all the phpdoc previously)


some tests failed:
failure: http://build.servo.org/builders/gonk/builds/444


style: Move this to the top.


why?


Typo, it should be: Problems can be two types: RULE_OUT and HISTORY_OF


Looks good.  Rebase and merge.


80 chars is half of my screen. And the other half is filled by a term. So yes, I'll remove it, again..


Why no reviewable ?


I've added a `defined`.

I think it's beyond the scope of this PR since it would involve both some investigation to find out what the correct values should be for each caller (many are currently relying on this default).  I can see arguments both for and against the change itself.  If consensus is generally in favour I'd be glad to do it!


can you set response = "" in this else clause? I think there are some paths were it doesn't get set


Verdade. Estava vendo isso também, usando callable pode ser passado um metodo estatico etc


Well, @gwendalaze
Remove spaces before punctuation, please.


The order of the regex evaluations just needed to be reversed around the alternator. The former catches the quotation and thus never leads to the latter in a multi-term request.


Why sleep here rather than make the caller handle that?


@lcdss Yes


" False ,numFailedPings" -- can you move the space to after the comma?


saw approval from Manishearth
at https://github.com/Manishearth/servo/commit/5197899e5a33328dbd2820700335072b7dff476b


merging Manishearth/servo/cssfix = 5197899e into auto


woops! fixed


@bryanmtl think `user_id` should be part of `same_as?` check. Otherwise if another account (i.e. family member) is shipping to the same address, it will only persist for the first person to use it.


Cancel button should be right with buttons are on right side


@approce i just missed it for active list item. Can you change it to. The way you have written it, is appears as if you mean the doctor instead of patient. Because it is the doctor who sets this value. Does it make sense?


New error:
```
04-02 17:55:05.196    3410-3410/com.frostwire.android E/AndroidRuntime﹕ FATAL EXCEPTION: main
    Process: com.frostwire.android, PID: 3410
    java.lang.RuntimeException: Unable to create application com.frostwire.android.gui.MainApplication: com.frostwire.android.core.CoreRuntimeException: Unable to initialized main components
            at android.app.ActivityThread.handleBindApplication(ActivityThread.java:4476)
            at android.app.ActivityThread.access$1500(ActivityThread.java:144)
            at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1265)
            at android.os.Handler.dispatchMessage(Handler.java:102)
            at android.os.Looper.loop(Looper.java:136)
            at android.app.ActivityThread.main(ActivityThread.java:5146)
            at java.lang.reflect.Method.invokeNative(Native Method)
            at java.lang.reflect.Method.invoke(Method.java:515)
            at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:732)
            at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:566)
            at dalvik.system.NativeStart.main(Native Method)
     Caused by: com.frostwire.android.core.CoreRuntimeException: Unable to initialized main components
            at com.frostwire.android.gui.MainApplication.onCreate(MainApplication.java:81)
            at android.app.Instrumentation.callApplicationOnCreate(Instrumentation.java:1007)
            at android.app.ActivityThread.handleBindApplication(ActivityThread.java:4473)
            at android.app.ActivityThread.access$1500(ActivityThread.java:144)
            at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1265)
            at android.os.Handler.dispatchMessage(Handler.java:102)
            at android.os.Looper.loop(Looper.java:136)
            at android.app.ActivityThread.main(ActivityThread.java:5146)
            at java.lang.reflect.Method.invokeNative(Native Method)
            at java.lang.reflect.Method.invoke(Method.java:515)
            at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:732)
            at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:566)
            at dalvik.system.NativeStart.main(Native Method)
     Caused by: java.lang.UnsatisfiedLinkError: Native method not found: com.frostwire.jlibtorrent.swig.libtorrent_jni.rss_alert_alert_type_get:()I
            at com.frostwire.jlibtorrent.swig.libtorrent_jni.rss_alert_alert_type_get(Native Method)
            at com.frostwire.jlibtorrent.swig.rss_alert.<clinit>(rss_alert.java:92)
            at com.frostwire.jlibtorrent.alerts.AlertType.<clinit>(AlertType.java:75)
            at com.frostwire.bittorrent.BTEngine.<clinit>(BTEngine.java:49)
            at com.frostwire.android.gui.MainApplication.setupBTEngine(MainApplication.java:125)
            at com.frostwire.android.gui.MainApplication.onCreate(MainApplication.java:64)
            at android.app.Instrumentation.callApplicationOnCreate(Instrumentation.java:1007)
            at android.app.ActivityThread.handleBindApplication(ActivityThread.java:4473)
            at android.app.ActivityThread.access$1500(ActivityThread.java:144)
            at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1265)
            at android.os.Handler.dispatchMessage(Handler.java:102)
            at android.os.Looper.loop(Looper.java:136)
            at android.app.ActivityThread.main(ActivityThread.java:5146)
            at java.lang.reflect.Method.invokeNative(Native Method)
            at java.lang.reflect.Method.invoke(Method.java:515)
            at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:732)
            at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:566)
            at dalvik.system.NativeStart.main(Native Method)
```


Why does this test exclude `fedora-20`?


You're right, it's not pretty but... I like the plan :)

Is there mileage in using something a bit more efficient for the timestamp? mach_absolute_time or similar?


You should also check that these files are executable.


merging Manishearth/servo/cssfix = 5197899e into auto failed


default


:D okay. You should check out the credits, you may find someone you know ;)


You don't need to do escape `/` when using `%r{}`.


Because I want to hide from the caller that this queue is unable to do block-with-timeout.  Some other queue system could and then the time aspect would live in this function again.


Suggest mentioning data type for consistency. `int`?


Hmn, OK. I've tried to wrap my head around contexts but given [the first thing in the section on the docs on it](http://www.w3.org/TR/2014/REC-json-ld-20140116/#the-context) is "_this is non-normative_"... I do wish this would be written in actual language that people use.

Thanks for looking, Tom; I guess Frankie and I will have to decide what to do next Tuesday, because we're going to have to try to make our own home-made attempt at compaction, or something similar.


Doesn't this mean that if you have a gun in your left hand AND your right hand, but your right hand is selected, then the chance will be 20 instead of 40?


@dkayiwa  if so, we also should change it in Allergy class too.
Ok, I can change it to:
"@param person the person that this problem is set for"
Is it OK?


Addresses #233.


This probably should be something like:
```
if(istype(get_active_hand(), /obj/item/weapon/gun))
    W = get_active_hand()
    chance = 40
else if(istype(get_inactive_hand(), /obj/item/weapon/gun))
    W = get_inactive_hand()
    chance = 20
```


Type (wwitch)


write a comment before the class name and put empty ## before every comment @rawan-abdelhalim


Oh sorry :D It was your job to test it.

I kinda suspected you were involved somehow, making those map editors and everything.


Accidentally put the private contents in the example file and example contents in the private file.

Sites are not public and password has been changed.


Should have spaces on either side of `+`


Temporary change while i was doing .tsjs on the managed side.  This got reverted back to .js later on.


Or perhaps the whole thing could be replaced with:
```
chance = 0
var/obj/item/weapon/gun/gun = null
var/list/chances = list(get_active_hand() = 40, get_inactive_hand() = 20)
for(var/obj/item/weapon/gun/W in chances)
    chance = chances[W]
    gun = W
if(prob(chance))
    //...


Really? That would be 1st contemporary real-world CPU with such stack ;-).


Maybe we can simply add size check to the Write function, in the beginning, to check that _worldPacket is empty, and if its not - erase it.


is there supposed to be an assert in this method?


@erictse Why not just grab the code out of master?


universal_newlines shouldn't be used, because it converts the input to a string using the default system encoding (and the input might actually be in another encoding).


We really need to prepare a testcase which shows a failure by not scanning *stack*. From that, we can reason in which situations not scanning registers may be dangerous.


it's just build script I adapted from CM12.
Previous offsets were of hammerhead; new are from shamu.


@firm1 pourquoi l'anglais ici au fait ?


Will the last two values always be `int`s? As long as we're wrapping the call, do you think it's worthwhile to do the conversion to nagios format in `run_collectd_nagios`? That way we could write the values a little more readably like

```
run_collectd_nagios(
      HOSTNAME, 'meta-collectd/process_memory-rss',
      'value', _mb_to_bytes(10), _mb_to_bytes(50))
```


Thats only something where im experimenting on, but if you take a look at the [TaskScheduler](https://github.com/Naios/TrinityCore/blob/functionalai/src/server/scripts/Custom/functional_ai_tests.cpp#L86) it seems much more useable for me as the current eventmap, since it supports better scheduling then eventmap.
The main target for me was to make it possible to use lambdas and std::functions to schedule an event inside the context where you need it and proper use of std::chrono. No eventmap with switches is needed anymore.

The other part where i'm working on (listeners for hook functions), well its something experimental lets see where it will lead me. But the main target for me is that you can add and remove several listeners (std::functions/lambdas as well) depending if you need it (in view of phases and difficulty).

If you have questions or suggestions, just ask or inspire me.


Do I need to use the SerialSoftware library debugger for that?
Shift/Control M does not offer any clues what the problem might be...

On Thu, Apr 2, 2015 at 11:32 PM, Alexey Andreyev <notifications@github.com>
wrote:

> Everything looks ok for me (Serial1 port and baudrate, 3.3V, etc).
> I guess if there's no problems with the mbili no.2 jumper
> <http://mbili.sodaq.net/features/> and Wee Work/FW Switch the board and
> wee should work well with this library.
> Can you share the exact serial output while using kick() and restart()
> methods?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/itead/ITEADLIB_Arduino_WeeESP8266/commit/712e1e3a95f0980d4c8765d3ecf04768a89e7d57#commitcomment-10554848>
> .
>

--

This email and any files sent with it are confidential and intended only
for the person it is addressed to. If you have received this message in
error please let us know immediately and delete it. The content of this
message belongs to the sender and the views expressed do not necessarily
reflect the views of United Bible Societies.

United Bible Societies Association, a company limited by guarantee.
Registered in England and Wales No 2264875. Registered Charity No 800058
Registered Office: Stonehill Green, Westlea, Swindon, Wiltshire, SN5 7PJ,
England.


commented out code


TeamCity Azmq :: Build (Windows) [Build 109](https://78.46.71.115/viewLog.html?buildId=445&buildTypeId=Azmq_Build&guest=1) is now running


TeamCity Azmq :: Build (Linux) [Build 83](https://78.46.71.115/viewLog.html?buildId=444&buildTypeId=Azmq_BuildLinux&guest=1) is now running


TeamCity Azmq :: Build (OSX) [Build 67](https://78.46.71.115/viewLog.html?buildId=446&buildTypeId=Azmq_BuildOsx&guest=1) is now running


I'm still trying to think of a compile-time check to ensure that Write() isn't called more than once but I doubt it's possible.


é bom evitar o uso da primeira pessoa, por que esses cenários são pra um grupo de usuários específicos (professor, aluno, etc), pra não ficar muito limitado que só "comigo" que pode ocorrer algum desses scenarios.


90119792FourPipeInd_ErrMsg (wfbuhl) - Win64-Windows-7-VisualStudio-12: OK (1681 of 1682 tests passed)

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


Thank you for the quick reply. Should I continue to use no offsets then? Or should I use these?


I moved these methods because the new `PrePartitionedWindowOperator` uses them


I just pushed a commit that should fix the Firefox keyboard issue.


@norine-badr  delete lines 2,3,20,22,25,27,29,31,34,35 in the applications controller
and make private right above the def in line 37
and add a line between 40 and 41


TeamCity Azmq :: Build (OSX) [Build 67](https://78.46.71.115/viewLog.html?buildId=446&buildTypeId=Azmq_BuildOsx&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 19 Build time: 00:02:00


Shouldn't this read addCustomData?


daí acho que seria bom se pusesse alguns nomes variantes ex: Given "fulaninho" está na página de configurações de assinatura de um curso


This should be converted to use the stream API and inlined.


TeamCity Azmq :: Build (Windows) [Build 109](https://78.46.71.115/viewLog.html?buildId=445&buildTypeId=Azmq_Build&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 19 Build time: 00:02:30


@norine-badr  @maikany


TeamCity Azmq :: Build (Linux) [Build 83](https://78.46.71.115/viewLog.html?buildId=444&buildTypeId=Azmq_BuildLinux&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 19 Build time: 00:02:49


Try converting this to the stream API and maybe inline


Addresses need to be translated into deviceTokens?


@norine-badr  it's preferable to write the comments at the beginning of the class right after the first line of 'class x'
and this should be done in the categories controller and the requests controller


The validate function on Messages required Id to be numeric. The Id set will typically be in hex format, resulting in a send failure.


That one is crazy :/ Good catch @vvo! :+1:


what "type parameter"?


> I do think the deeper nesting and the need to set a default window function makes things more complicated (and there's some code duplication)

IMO: The default window functions make the code in filter_kernels.c prettier (code reuse, clarity (ie. see the sinc/jinc aliases) and no more duplication of the radius/blur/windowing logic), but the code in gl_video.c uglier. Maybe there's a way to make both prettier?

> In general, such specialized things and not-really-commonly-used things are what cause bugs and maintenance burden, though.

ACK. In this case I feel like the improvements to filter_kernels.c make the code easier to maintain, rather than harder. I also think moving the configuration to a single struct that can be passed around in gl_video.c makes that code more robust and easily maintainable as well, but I'm dissatisfied about the options system here. (I would really love to have it directly parse the scaler option as a magic string, directly returning the struct to describe that exact configuration, but that's probably a mixture of EDSL trigger happiness + wishful Haskell thinking)

> Well, what can I say - from my perspective it'd be best if there was a third party, which can overrule me if I make the wrong decision. Instead of waiting and "working" me until I'm convinced. Sorry for being such a shitty maintainer.

I think it's natural for these kinds of decisions to occur whenever there's a conflict of interests - and the interest you seem to be representing most often is that of keeping the code clean and simple, which is something good in and of itself. Part of the reason we've had to bash our heads over this kind of thing is because the features I've been often most interested in are the generally complex “high-end” (placebo) stuff, some of which have conflicted with fundamental assumptions in the existing code (no doubt originally made to keep it clean and simple). It's true that at some point we have to ask ourselves whether or not my feature requests are useful enough to people who are *not* myself to warrant keeping as part of the main mpv codebase, which might not be something either of us can decide by ourselves (without appealing to authority).

> I don't think maintaining two codebases is a good idea. Especially if there's no communication between them, like it was shaping up in the last days. We should stop this.

Agreed.

> There's nothing wrong with that, though it wouldn't be a "real" fork. (Maybe it could even be just a branch on the main git repo. We could make it possible to select the branch with mpv-build and have lachs0r make builds etc.)

This something I also thought about and honestly I think it's the best possible compromise. I will continue to use my repository as a playground for my own personal wants and desires, but I will make it my responsibility to ensure they flow back upstream. As for things that we decide won't go back into upstream (for whatever reasons), I think I will have to accept demoting them to branch status. (Users who really, *really* want crazy feature X like sphinx can still merge from that branch themselves)

(Incidentally, public exposure to my “placebo” fork has already resulted in useful bug reports that I would not necessarily have noticed had it been sitting in a pull request)

> (The rules are: bug fixes and other small changes can be pushed freely, but intrusive changes or things which affect other areas of the player should be reviewed first. Maybe user-visible changes too. If someone wants to require me to put up pull requests for review and reviews them quickly enough, I'd be fine with this too, btw.)

This is indeed something I was not explicitly aware of, and so I apologize for pushing something larger without an explicit pull request - I guess I got carried away by the flow of things and fooled myself into thinking I had “authority” over gl_video.c, which incidentally also contradicts what I originally said when you gave me push rights in the first place (that “I wouldn't push anything major without having it looked at in pull requests”).

So I was wrong to do so on both accounts, and it's not the first time I trigger-happily pushed something that made you uncomfortable, either - so I feel a bit silly on the matter overall. I'm not sure if I'm responsible enough for direct push access.

> While I very much like to have my own opinion about what's broken (such as ASS subtitles trying to match the video colors, which now has to be "emulated" by declaring ASS colors not being sRGB, but some bastardized video-dependent almost-RGB thing, even though they definitely are sRGB when using the colorpicker in Aegisub), and while I'm defending my opinion, I don't always absolutely insist on having my way.

I think in this case we can agree to disagree about what's “correct” and what's “incorrect” as a matter of opinion/goals/preference, and instead stick to the status quo of “just make sure it works like VSFilter” when it comes to ASS compatibility / subtitle rendering in general. That's a “neutral” stance that I think we can both follow, regardless of whether or not we think we're in the right - which is something you've clearly also been doing in the past (with regards to BT.601 compatibility hacks etc.).


@norine-badr  in the charges controller please delete lines 2,8,13,20, 25,28
and add line between 30 and 31


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


Note-to-self: remove listing dep. When dealing with units.


OK

On Fri, Apr 3, 2015 at 12:59 AM, Roman Zayats <notifications@github.com>
wrote:

> @dkayiwa <https://github.com/dkayiwa> if so, we also should change it in
> Allergy class too.
> Ok, I can change it to:
> "@param <https://github.com/param> person the person that this problem is
> set for"
> Is it OK?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/approce/openmrs-core/commit/79459b2732e9df31537bbcf8a33401b8d8bb7a5e#commitcomment-10555334>
> .
>

--
If we keep uppermost in our minds the unkind and unjust acts of others, we
shall find it impossible to love them as Christ has loved us; but if our
thoughts dwell upon the wondrous love and pity of Christ for us, the same
spirit will flow out to others.


isn't this function kinda redundant?


seria bom especificar um pouco mais um ambiente


Update


my favorite!


delete


TeamCity Azmq :: Build (Linux) [Build 84](https://78.46.71.115/viewLog.html?buildId=447&buildTypeId=Azmq_BuildLinux&guest=1) is now running


TeamCity Azmq :: Build (Windows) [Build 110](https://78.46.71.115/viewLog.html?buildId=448&buildTypeId=Azmq_Build&guest=1) is now running


TeamCity Azmq :: Build (OSX) [Build 68](https://78.46.71.115/viewLog.html?buildId=449&buildTypeId=Azmq_BuildOsx&guest=1) is now running


It should do both, since we won't call until something asks for the content, which will happen after we've filtered for the files we care about.


i meant the branch `merge-attempt`. but if the other one works, great!


> backward compatibility?

No, actually, not for bc at all.


require_once 'Tiqr/Message/Exception.php';


the playerX and playerY variables were undeclared


Why don't we want to propagate the preferred properties down through the semi join?


This is needed because we write to the storage folder if the vendor folder is read only. People were complaining about having us force writing to the vendor folder before.


I'm going to merge this in spite of this test, which I will probably modify afterwards.

`inspect` is one of those things that's tricky to test well. I believe in testing against interfaces primarily. Addressable generally has a very well-defined interface that's extremely easy to test against, which makes adhering to that philosophy usually a breeze. The `inspect` method is an exception to that. There's never really been a well-defined specification for what the output of `inspect` should be. Primitives generally have custom formats, while plain old Ruby objects just dump their instance variables. Neither is particularly appropriate for either `URI` or `Template` classes, the latter if only because of the heavy memoization going on. The solution Addressable uses instead is a good compromise, but it's not a well-defined interface. What you really want to test is the following question:

Are the expected values in the map represented by `MatchData` all present in the `inspect` string? The string is meant to be human readable, so the actual format of it isn't important and actually shouldn't be part of the test expectation.


TeamCity Azmq :: Build (OSX) [Build 68](https://78.46.71.115/viewLog.html?buildId=449&buildTypeId=Azmq_BuildOsx&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 19 Build time: 00:01:44


Sweet dude, I'm just playing around with different ideas, like a search bar
to do research on demand.

Do you have anything in mind you want to work on? If not, I definitely
could use some help with the search bar.

On Thu, Apr 2, 2015 at 2:38 PM, Shomari <notifications@github.com> wrote:

> yeah, I'd be down. I don't have a lot of time but I probably could give it
> a few hours on the weekend
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/elyotli/Project-Clue/commit/010d7d3222cd677cba2aed43efa77b0c41f69075#commitcomment-10552896>
> .
>


@maryam-osama confirmed


Might be easier/cleaner to do using a metaclass on the wrapper that has a [`__instancecheck__`](https://docs.python.org/2/reference/datamodel.html#class.__instancecheck__) method.


I think the opposite is actually the case here, right?


See ZendFramework-1.12.11-minimal/library/Zend/Mobile/Push/Message/Apns.php
    public function validate() {
...
        if (null != $this->_id && !is_numeric($this->_id)) {
}


TeamCity Azmq :: Build (Windows) [Build 110](https://78.46.71.115/viewLog.html?buildId=448&buildTypeId=Azmq_Build&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 19 Build time: 00:02:46


r+


rax retry tests


TeamCity Azmq :: Build (Linux) [Build 84](https://78.46.71.115/viewLog.html?buildId=447&buildTypeId=Azmq_BuildLinux&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 19 Build time: 00:02:57


This commit might be avoidable by not having the import in MaKaC/user.py - https://github.com/indico/indico/commit/ec3246d3047d002995ef76c27c1c9807e59dd285#commitcomment-10555633

Being able to import `User` from `indico.modules.users` would be a massive plus since it's going to be used in many places.


saw approval from jdm
at https://github.com/dmarcos/servo/commit/475e696124a51d73fe32c6145aeb05e24eb7ca71


merging dmarcos/servo/issue5290 = 475e6961 into auto


Oh okay. It says you made a ton of changes.


dmarcos/servo/issue5290 = 475e6961 merged ok, testing candidate = a7cf326d


there is a typo but ı dont think its harm


I'm confused by the grouping here. `testunit_sock_sendcmd` and `testunit_sock_connect_WHEN_no_socket_RETURNS_None` seem closely related, but they're in different test classes.

Same with `testunit_assert_collectd_responds_WHEN_sock_sendcmd_fails` and `testunit_assert_collectd_installed_WHEN_bin_missing_RAISES_CriticalError`


@andresgutierrez This breaks the win32 build, probalby only temporary uncommented anyways?


especificar quem é o usuário, no caso, aluno


:D


Remove redundant comment


The gc was recently changed to offset the values it hands out by the size of a jl_taggedtype_t. In some cases, the compiler will assume that a struct is aligned to some larger boundary (up to some platform-defined limit -- on most modern platforms that limit being 16-bytes) and emit vmovsd or similar instructions accordingly


ai poderia ficar:
Given Que o aluno "Rafael" está estudando o assunto "Grandezas e Unidades"


Doh. Yeah, I should have been using reviewable.


#25 @norine-badr


Yeah, preparing a commit to restore it


@maryam-osama  please do one last modification in the bd.migrate/20150322202057_create_notifications.rb
add a line between 15 and 16


I'm getting 500 server errors since this change:
```
FilterError: compass: subprocess had error: stderr=, stdout=    error app.scss (Line 14 of spinner.scss: Undefined mixin 'keyframes'.)

FilterError: compass: subprocess had error: stderr=, stdout=    error app.scss (Line 25 of spinner.scss: Undefined mixin 'animation'.)
```


Still in the middle of reviewing this round, but need to resume later.

Note to self: Resume reviewing at
*  check_collectd_mlab.py: line 310
*  check_collectd_mlab_test.py: line 155


MarkdownDocs (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1674 of 1676 tests passed)

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


acho que o nome do Scenario não tá muito de acordo com o caso acho que seria melhor
Scenario: Modificação de Conteúdo


checked and approved @rawan-abdelhalim


corrigir esse Ques aí


@maikany


@khadijaibrahim


De modo geral, é preciso que tú especifique mais quem seria o usuário. E criar um And especificando mais o ambiente (local da página) que ele está.


This issue is known - to us - issue in chromedriver. I am going to add more information here[1]. I have a question before I commit a change...

Did you use goog.userAgent.WEBKIT and not a more specific check for Chrome for a reason or simply because you felt like that?

(I want to use a more specific check for Chrome here unless that won't work for something that you had in mind)

[1] Guilty of not doing that much earlier, when it first became a thing around August 2014.


Er gjort :)


clear fix


This commit sponsored by Ride.


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


@Dinuuu despues de creacion  y cancelacion de donaciones mando mails correspondientes y redirijo a la pagina en donde estaba.


Merging upstream


+1 Immediately after reading the pyres docs I wanted it to use BLPOP instead


@cyanogen Why not simply include this repo? https://www.codeaurora.org/cgit/quic/la/platform/vendor/qcom-opensource/dpm/

Then, as long as you have the blobs, it should work, right?


Bogus, DISPATCH_QUEUE_SERIAL is equal to NULL.


this is awesome thanks :+1:


Specific to WebRTC use, not gstreamer in general!


this isn't doing what I expect it to do.


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

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


We need a license still :) CC0? :) I let you decide on that :)


Red seems like the best idea for now, maybe make the text a little bigger too.


I'll fix it


confirmed (Y)


dfsd


I think I was going for "doesn't crash when you haven't connected" but I've added an assert to be clear.


@PatrickBM Speaking of filenames, I've heard from some strongly-opinioned scripters that it's bad form to give bash scripts `.sh` extensions. What do you think?


+Import-Module ScriptAnalyzer
I think this should be changed to PSScriptAnalyzer.. or we rename PSScriptAnalyzer.psd1 into ScriptAnalyzer.psd1


*Should be "Finished data model"...


Is this supposed to be `cl-gensym`?


This commit breaks the plugin in redmine 2.x (2.6.3 specifically).


Let's put a comment here explaining your approach, as it may not be obvious to someone looking at the code for the first time.


This really must be fixed as with Symfony 2.6.6 it eventually generates an error extremely hard to track down and fix:
1. ErrorHandler->handleError('2', 'unlink(...): No such file or directory') => incomplete cache
2. FileLoaderImportCircularReferenceException Circular reference detected in "app/config/routing_dev.yml"

@ only changes the error level


IMO Throwable catches too much. OutOfMemory should not be wrapped in an CommandException.
RuntimeException or Exception should be enough.


/*
   Rafal Jopek Poland 31-03-2015
*/

#include "fast.ch"

#define B_TOP        1
#define B_LEFT       2
#define B_BOTTOM     3
#define B_RIGHT      4
#define B_BOOL       5
#define B_BACKGROUND 6
#define B_SECONDS    7
#define B_SAVESCREEN 8

#define B_ELEMENTS   8

/*

*/
   CLASS Hb_Box

      DATA aBox

      METHOD New CONSTRUCTOR
      METHOD BoxOpen
      METHOD Box3D
      METHOD Message
      METHOD BoxPush
      METHOD Delay
      METHOD BoxClose
      METHOD BoxTurn
      METHOD BoxDelete

   ENDCLASS
/*

*/
METHOD New( ) CLASS Hb_Box

RETURN ( Self )
/*

*/
METHOD BoxOpen( nTop, nLeft, nBottom, nRight, cBackground, nSeconds ) CLASS Hb_Box

   ::aBox := ARRAY( B_ELEMENTS )

   ::aBox[ B_TOP        ] := IIF( nTop         == NIL, 0,        nTop )
   ::aBox[ B_LEFT       ] := IIF( nLeft        == NIL, 0,        nLeft )
   ::aBox[ B_BOTTOM     ] := IIF( nBottom      == NIL, MAXROW(), nBottom )
   ::aBox[ B_RIGHT      ] := IIF( nRight       == NIL, MAXCOL(), nRight )
   ::aBox[ B_BOOL       ] := .T.
   ::aBox[ B_BACKGROUND ] := IIF( cBackground  == NIL, "B",      cBackground )
   ::aBox[ B_SECONDS    ] := IIF( nSeconds     == NIL, 0.8, IIF( nSeconds <= 0, 0.8 nSeconds ) )

   ::aBox[ B_SAVESCREEN ] := SAVESCREEN( ::aBox[ B_TOP    ], ;
                                         ::aBox[ B_LEFT   ], ;
                                         ::aBox[ B_BOTTOM ], ;
                                         ::aBox[ B_RIGHT  ] )

   DISPBOX( ::aBox[ B_TOP    ], ::aBox[ B_LEFT  ], ;
            ::aBox[ B_BOTTOM ], ::aBox[ B_RIGHT ], ;
            SPACE( 9 ), ;
            ::aBox[ B_BACKGROUND ] + "/" + ::aBox[ B_BACKGROUND ] )

   ::Box3D( ::aBox[ B_TOP        ], ::aBox[ B_LEFT  ], ;
            ::aBox[ B_BOTTOM     ], ::aBox[ B_RIGHT ], ;
            ::aBox[ B_BOOL       ], ;
            ::aBox[ B_BACKGROUND ] )

RETURN ( { ::aBox[ B_TOP ], ::aBox[ B_LEFT ], ::aBox[ B_BOTTOM ], ::aBox[ B_RIGHT ], ;
           ::aBox[ B_BACKGROUND ], ::aBox[ B_SECONDS ], ::aBox[ B_SAVESCREEN ] })
/*

*/
METHOD Box3D( nTop, nLeft, nBottom, nRight, lBool, cBackground ) CLASS Hb_Box
   LOCAL cBackground_1, cBackground_2
   LOCAL i
   LOCAL nRow := ROW(), nCol := COL()

   ::aBox[ B_TOP        ] := nTop
   ::aBox[ B_LEFT       ] := nLeft
   ::aBox[ B_BOTTOM     ] := nBottom
   ::aBox[ B_RIGHT      ] := nRight
   ::aBox[ B_BOOL       ] := lBool
   ::aBox[ B_BACKGROUND ] := cBackground

   IF ( ::aBox[ B_BACKGROUND ] == NIL )

      ::aBox[ B_BACKGROUND ] := SETCOLOR()
      ::aBox[ B_BACKGROUND ] := SUBSTR( ::aBox[ B_BACKGROUND ], 1, AT( ",", ::aBox[ B_BACKGROUND ] ) - 1 )
      ::aBox[ B_BACKGROUND ] := SUBSTR( ::aBox[ B_BACKGROUND ], 1 + RAT( "/", ::aBox[ B_BACKGROUND ] ) )

   ENDIF

   IF ( ::aBox[ B_BOOL ] )

      IF SUBSTR( ::aBox[ B_BACKGROUND ], LEN( ::aBox[ B_BACKGROUND ] ), 1 ) == "+"

         cBackground_1 := ::aBox[ B_BACKGROUND ] + "/" + ::aBox[ B_BACKGROUND ]

      ELSE

         cBackground_1 := ::aBox[ B_BACKGROUND ] + "+/" + ::aBox[ B_BACKGROUND ]

      ENDIF

      cBackground_2 := "N/" + ::aBox[ B_BACKGROUND ]

   ELSE

      cBackground_2 := ::aBox[ B_BACKGROUND ] + "+/" + ::aBox[ B_BACKGROUND ]
      cBackground_1 := "N/" + ::aBox[ B_BACKGROUND ]

   ENDIF

   DISPBEGIN()

   ::Message( ::aBox[ B_BOTTOM ], ::aBox[ B_LEFT ], CHR( 192 ), cBackground_1 )

   ::Message( ::aBox[ B_TOP ], ::aBox[ B_LEFT ], ;
   CHR( 218 ) + REPLICATE( CHR( 196 ), ::aBox[ B_RIGHT ] -  ::aBox[ B_LEFT ] - 1 ), cBackground_1 )

   FOR i := ::aBox[ B_TOP ] + 1 TO ::aBox[ B_BOTTOM ] - 1
      ::Message( i, ::aBox[ B_LEFT ], CHR( 179 ), cBackground_1 )
   NEXT

   ::Message( ::aBox[ B_TOP ], ::aBox[ B_RIGHT ], CHR( 191 ), cBackground_2 )

   ::Message( ::aBox[ B_BOTTOM ], ::aBox[ B_LEFT ] + 1, ;
   REPLICATE( CHR( 196 ), ::aBox[ B_RIGHT ] - ::aBox[ B_LEFT ] - 1 ) + CHR( 217 ), cBackground_2 )

   FOR i := ::aBox[ B_TOP ] + 1 TO ::aBox[ B_BOTTOM ] - 1
      ::Message( i, ::aBox[ B_RIGHT ], CHR( 179 ), cBackground_2 )
   NEXT

   DISPEND()

   SETPOS( nRow, nCol  )

RETURN ( Self )
/*

*/
METHOD Message( nRow, nCol, cChar, cColorString ) CLASS Hb_Box

   SETPOS( nRow, nCol )

   DISPOUT( cChar, cColorString )

   @ nRow, nCol SAY cChar COLOR cColorString

RETURN ( Self )
/*

*/
METHOD BoxPush( aBox ) CLASS Hb_Box

   ::aBox[ B_SAVESCREEN ] := SAVESCREEN( ::aBox[ B_TOP    ] + 1, ;
                                         ::aBox[ B_LEFT   ] + 1, ;
                                         ::aBox[ B_BOTTOM ] - 1, ;
                                         ::aBox[ B_RIGHT  ] - 1 )

   ::Box3D( ::aBox[ B_TOP        ], ::aBox[ B_LEFT   ], ;
            ::aBox[ B_BOTTOM     ], ::aBox[ B_RIGHT  ], ;
            ::aBox[ B_BOOL       ] := .F., ;
            ::aBox[ B_BACKGROUND ] )

   ::Delay( ::aBox[ B_SECONDS ] )

   RESTSCREEN( ::aBox[ B_TOP        ] + 1, ;
               ::aBox[ B_LEFT       ] + 1, ;
               ::aBox[ B_BOTTOM     ] - 1, ;
               ::aBox[ B_RIGHT      ] - 1, ;
               ::aBox[ B_SAVESCREEN ] )

   ::Box3D( ::aBox[ B_TOP        ], ::aBox[ B_LEFT   ], ;
            ::aBox[ B_BOTTOM     ], ::aBox[ B_RIGHT  ], ;
            ::aBox[ B_BOOL       ] := .T., ;
            ::aBox[ B_BACKGROUND ] )

RETURN ( Self )
/*

*/
METHOD Delay( nDuration ) CLASS Hb_Box
   LOCAL nSecs := SECONDS()

      WHILE( SECONDS() < nSecs + nDuration )
      END

RETURN ( Self )
/*

*/
METHOD BoxClose( aBox ) CLASS Hb_Box

   RESTSCREEN( ::aBox[ B_TOP        ], ;
               ::aBox[ B_LEFT       ], ;
               ::aBox[ B_BOTTOM     ], ;
               ::aBox[ B_RIGHT      ], ;
               ::aBox[ B_SAVESCREEN ] )

RETURN ( Self )
/*

*/
METHOD BoxTurn( aBox, lBool ) CLASS Hb_Box

   ::aBox[ B_BOOL ] := IIF( lBool == NIL, .T., lBool )

   ::Box3D( ::aBox[ B_TOP        ], ::aBox[ B_LEFT   ], ;
            ::aBox[ B_BOTTOM     ], ::aBox[ B_RIGHT  ], ;
            ::aBox[ B_BOOL       ], ;
            ::aBox[ B_BACKGROUND ] )

RETURN ( Self )
/*

*/
METHOD BoxDelete( aBox ) CLASS Hb_Box

   ::aBox[ B_SAVESCREEN  ] := SAVESCREEN( ::aBox[ B_TOP    ] + 1, ;
                                          ::aBox[ B_LEFT   ] + 1, ;
                                          ::aBox[ B_BOTTOM ] - 1, ;
                                          ::aBox[ B_RIGHT  ] - 1 )

   ::Box3D( ::aBox[ B_TOP        ], ::aBox[ B_LEFT   ], ;
            ::aBox[ B_BOTTOM     ], ::aBox[ B_RIGHT  ], ;
            ::aBox[ B_BOOL       ], ;
            ::aBox[ B_BACKGROUND ] )

   ::Delay( ::aBox[ B_SECONDS ] )

   ::New( ::aBox[ B_TOP        ], ::aBox[ B_LEFT  ], ;
          ::aBox[ B_BOTTOM     ], ::aBox[ B_RIGHT ], ;
          ::aBox[ B_BACKGROUND ] )

RETURN ( Self )


setEnabled(boolean isEnabled)


90179004-ConstrNumBackNotInitializedComplexGlazing (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1171 of 1171 tests passed)

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


no need to comment out - the code is always available in github if you want to come back to it - delete safely


Yeah, it should but due to race conditions.....

Shall I leave as is, or put back inside the spinlock?


Great! And sorry about the not being able to log in :3 If I  can help, just ask :P (Like getting DBO staff to help).
> On Apr 2, 2015, at 4:39 PM, Joe <notifications@github.com> wrote:
>
> Once the pull request goes through, I'll add you on the BukkitDev page (besides, can't login nor reset my password right now)
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/20zinnm/AntiBot/commit/96e60eba864e7b7b29631900e66beae912d64dca#commitcomment-10554972>.
>


try to make your code as close to normal English as possible - it is easy to read it this way.. and thus, maintain

boolean isEnabled() {
  return isEnabled;
}


Does it make sense to add more test targets here (assuming there isn't a way to just query/add 'all test targets' in crazyfun)?

ci's SmallTests today has these:
//java/client/test/org/openqa/selenium/support:SmallTests:run //java/client/test/org/openqa/selenium/remote:common-tests:run //java/client/test/org/openqa/selenium/remote:client-tests:run //java/server/test/org/openqa/selenium/remote/server:small-tests:run //java/server/test/org/openqa/selenium/remote/server/log:test:run


Does pylearn2 have a scikit-learn compatible API?


some tests failed:
failure: http://build.servo.org/builders/gonk/builds/445


use camelCase
(IntelliJ IDEA has a nice way to do renamings - it will do the hard job)


This used to cause llvm build/link errors on Win64. I guess this commit means it works now for some versions of llvm?


useless comment :)


@maryam-osama


@maikany


On second that, it wont matter much, cause we're already scoping the `same_as?` check to `user.addresses`, so it wont look at any other `user_id`s ever. All good


Awesome :)


Should be `Gdx.app.debug("NPC:NPC", "instantiating class");`


@norine-badr


I really want to kill this function entirely. (The operation that it implements is explicitly disallowed by the C standard.)


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


right. what you suggested almost works as such. however, it turns out that the actual fft algorithm is borked! just compare it to http://beige.ucs.indiana.edu/B673/node14.html - that is the same algorithm. tried it and our code yields a different answers. got fed up, did https://github.com/notspiff/kodi-cmake/commit/108a0929279383bcada98768a72f9c332b593e17 which works, coefs are properly normalized (try without the window on a single sine or whatever). obviously needs some refactoring, tired and lazy right now.


Probably should be `update()`? Most other things use `update()` functions, and having `Area:update()` delegate on to `Entity:update()` makes the most sense IMO...


@maikany Yaaay Approved!


Try the 6.4.6 version from http://mod-buildcraft.com/pages/download.html that included this changes.


Are there any results to this review?


Sorry, couldn't find it.


name of the method is confusing - ideally is when i don;t need to read code to understand what method does

if I understand the code correctly, it's something like "isAnyDaySelected()" or "isAtLeastOneDaySelected()"?


Ups I mean notifications work now correctly! Too late here in Germany :D


Thomas change 4/2/15


break should be the last statement in case clause, not inside if, otherwise it  falls through to the next case


definitely better. thanks!


Just a warning, with multiplayer/network code coming, we are going to be working very hard to decouple `cs383.team1.model` from `cs383.team1.render` and `cs383.team1.input` almost completely.

Doesn't make any difference for the moment, but in an ideal world `cs383.team1.model` is *completely* ignorant of the other modules. Just a head's up as far as where the code is eventually headed.


Are the ignored tests just the ones related to the donut coconuts?


Just checking that `&^` is subtracting `CompareMtime` from `CompareDefaults`?


@norine-badr  Confirmed wohooo !!!


We're treating positions as `(int, int)` pairs most places. There's even a `Position` object. Probably want to switch over for consistency?


Also removed the 'Results:' from the search page. It's implicit.


Yup, or else it would cause an error in the script.


What is meant by "vendor damage"?


I agree with you, there is no need to use a Double. I think a Integer would be better. I will change it.


See comments on the `GameManager` class.


Do you think it will be a better name?
isFormsSuccessfullyFilteredByMatchingFormFieldsInContainingAnyFormField()


there are libs to format time, look at http://www.joda.org/joda-time/


Oh, perfect! I forgot about that one!


I was trying to improve the if-else-condition meanging and readbility. I agree that creating a variable will makes the code more verbose. But I will acrescent more meaning. Just "true" or "false" doesn't have business meaning associated with it. But the variable isMatched gives me the ideia that something has compatibility. But it is just something I usually do, if you think its better I change it. No problems. Thanks for the review


Maybe this should belong to a `Display` subclass? We're trying to keep `Main` as thin as possible, and let the rest of the classes do most (all) of the heavy lifting. It's just a thought, but a `DialogueBox` does not seem like a top level class nor does it seem like an independent one.


Alexsandro vc salvou e atualizou as modificações no github?


LGTM!


code duplication - try to avoid this as much as you can.. it's a great way to loose time in debugging and overall code maintenance


Again I was trying to improve the if-else-condition meaning and readability. I thought that:

if ( isFormFieldWithinForm(form, formField) ) {
 isWithin = true;
 return isWithin;
}

was more meaningful and readable than, for instance:
for (FormField formField : containingAnyFormField) {
   if( form.getFormFields().contains(formField) )
     return true;
}

But again it is just something I usually do. If you think it is better. I will change it. No problems. Thanks for the review


commit


The same reason I described above. Thanks


Does this record audio internally or does it require the speakerphone to be used (using the mic to record both sides of the conversation)?


I'm not sure what you're trying to do.


List comprehensions are in-memory. Try converting to generator expressions to better fall in-line with `gensim`'s memory philosophy. Also, these three lines could be condensed (though that would lower readability...)


use dependency injection, avoid "new".. why? The great Martin Fowler can answer you better - http://martinfowler.com/articles/injection.html


Add readme file


Are you on Mac or Linux?


SmallTests is already there in the list, but yes, I think populating this list with all the available tests makes sense.  I took the list off Jenkins and didn't make any further changes to it.


You shouldn't be using this repository. Please use the official homebrew-science: https://github.com/Homebrew/homebrew-science
This is my fork for testing.


Shame we can't use finally here.


Typo on the title -- Cleaned*


Hi Allan, if you are having more success with this, would you like to merge this into the master? Please send a pull request :+1:


esta es un prueba gratuita


why use "format" just the half way? :)

String.format("%02d:%02d", hourOfDay, minute)


[![Side-blown horn](http://api.artsmia.org/images/119259/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=119259)


[![Face mask](http://api.artsmia.org/images/119265/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=119265)


[![Snuff horn](http://api.artsmia.org/images/119266/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=119266)


[![Ehrenpforte (The Triumphal Arch of Maximilian I)](http://api.artsmia.org/images/116298/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=116298)


[![The Habsburgs: Rarely Seen Masterpieces from Europe's Greatest Dynasty](http://api.artsmia.org/images/121558/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=121558)


[![Diana as Goddess of the Hunt](http://api.artsmia.org/images/119611/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=119611)


Does AppleDoc not support doc comments on categories?


getter with a side-effect (sets private variable) is very confusing, since outside it's not obvious what's gonna happen


@mkortstiege

Cleanup after yourself :) by deleting **DefaultMusicSearch.png** from skin.confluence as this PR makes this file added last week by ronie completely useless.
OR if you prefer I can remove that file it in https://github.com/xbmc/xbmc/pull/6829 and reference your pull for that cleanup.

that is if I got this code right.


I certainly hope so.


88737258-EPW-Ruby-Access (jasondegraw) - Win64-Windows-7-VisualStudio-12: OK (2183 of 2201 tests passed)

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


ditto


Sorting implies certain grouping sets, but grouping sets cannot be used to infer sub/super grouping sets.


@Optimusprimeums thanks a lot :)


Of the targets currently found in ci's SmallTests, the following are missing here, specifically:

//java/server/test/org/openqa/selenium/remote/server/log:test
//java/server/test/org/openqa/selenium/remote/server:small-tests


Yep. Heh

On Thu, Apr 2, 2015 at 6:00 PM, Graham Campbell <notifications@github.com>
wrote:

> Shame we can't use finally here.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/laravel/framework/commit/a372f6e7d212bc91dc5fe0d54e913428d5a8431a#commitcomment-10556091>
> .
>


get alarm list and.. nothing? i suspect it does smth else besides getting, but need to go inside and read the code.. ufff.. :)


Hi folks, hopefully I'm not speaking out of turn, but I agree with @nathos' sentiment to drop the `.sh` extension.

[This Post on unix.stackexchange](http://unix.stackexchange.com/questions/31760/file-extensions-for-unix-shell-scripts) makes the argument that including the extension is better for portability, which I find to be a reasonable argument to include the file extension.

However, speaking from a usability standpoint, I don't find the `.sh` useful, especially for those who are not as familiar with using command line utilities. In my experience, for the less sophisticated command line user, `./create-issue` is syntactically complicated enough.

I'm interested to hear more opinions though!


Это проверка возможности открыть файл на чтение?


@SubJunk there up mate. I'll get a bit more time soon to help out a bit more where i can :)


Ну так что, крутилки стали работать стабильнее? Оставляем эти изменения?


The order of the buttons is different... Ok is supposed to be on the right
for ICS and above
On 03/04/2015 4:07 am, "jshevek" <notifications@github.com> wrote:

> @timrae <https://github.com/timrae> The layout-v14/ version of
> my_account_register.xml is not materially different from the default
> version of my_account_register.xml.
>
> Any reason I shouldn't just delete
> ./AnkiDroid/src/main/res/layout-v14/my_account_register.xml ?
>
> ~/projects/Anki-Android$ diff
> ./AnkiDroid/src/main/res/layout-v14/my_account_register.xml
> ./AnkiDroid/src/main/res/layout/my_account_register.xml
> 75c75
> < android:id="@+id/cancel_button"
>
>         android:id="@+id/register_button"
>
> 82c82
> < android:text="@string/cancel"
>
>         android:text="@string/sign_up"
>
> 86c86
> < android:id="@+id/register_button"
>
>         android:id="@+id/cancel_button"
>
> 93c93
> < android:text="@string/sign_up"
>
>         android:text="@string/cancel"
>
> 95d94
> <
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jshevek/Anki-Android/commit/4b0aeff679b8b46f83b8b48ba5330dc0cd8986c3#commitcomment-10552379>
> .
>


The build of `haskellngPackages.poppler` is fixed in `6429b1c`.


switch?


If this is not an option, just remove this macro definition.


Не забудь про освобождение памяти


There will be a warning - unused return value


:fire:


MarkdownDocs (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (657 of 658 tests passed)

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


Don't use this PR, tabs on line 158 corrected by PR b9197b3.


ok! 1dbad75


LGTM


Here the value ``l`` is always non-negative due to the check in the previous line. Use ``size_t``.


``l`` should be ``size_t`` in the first place.


O RLY? Who will catch this?


yes, half of the `SpandexResult` transforms (from `SearchResponse`) were dedicated to the donut cocunuts


Relevant to closed issue #28


This should be functional test.
Check anything on screen in assertion.


oops sloppy, removed in b3519b4


I haven't looked in detail, but the commit message should indicate why these are being removed.  Probably worth doing it in a separate commit as well.


Ah, ok well maybe it should go with all the other options in the constructor instead?


Please, be more persistent if you want to refactor warnings and not just silence them.

```
bool IsWMProtocols = event->xclient.message_type == atom_WM_PROTOCOLS;
bool IsWMDeleteWindow = (Atom) event->xclient.data.l[0] == atom_WM_DELETE_WINDOW;

if ( w && IsWMProtocols && IsWMDeleteWindow ) ...
```


Because I called my method *isFormFieldWithinForm(form, formField)*. Its name implies I am looking for a *FormField* inside a *Form*.  If I have done it *Set formFields = form.getFormFields();* here, Maybe I should not created the method at all. (as you suggested below) .


:heart: :+1: :beers:


Who will catch?


@zhang-hua cool. Thanks.


Can be ``-1``?


convert_package_to_weights_array was updated to reflect the refactor of ContentItem: https://github.com/spree/spree/blob/3-0-stable/core/app/models/spree/stock/content_item.rb


"Current throttle value"?


What will happen if we open a 3Gb video file with this?


I mean this.


NAK

How about we find who's pulling qttest into the build phase and eliminate it at the source instead? A quick grep reveals the either `Tools/DumpRenderTree/qt/DumpRenderTree.pro` or `Tools/WebKitTestRunner/Target.pri` (or both) could be at fault. If they're built but not needed at build time nor installed, we should simply avoid building them in the first place.


"Total number of gets"?


"Total size of gets"?


"Number of gets blocked during get_or_fail"?


Thanks, I'll look into it soon.


Is `wrappers` needed?


"Total operations"?


@zhang-hua Cool. Thanks for double checking with Shannon


"Current size of queue"?


It doesn't support out-of-source builds?


But keep in mind this going to be changed again (as of now) to `__DIR__.'/cache/compiled.php'` on 5.1.

If you're looking into a structure that is forward compatible do the following:

`app/Foundation/Application.php`

```php
<?php namespace App\Foundation;

use Orchestra\Foundation\Application as BaseApplication;

class Application extends BaseApplication
{
    /**
     * Get the path to the configuration cache file.
     *
     * @return string
     */
    public function getCachedConfigPath()
    {
        return $this->basePath().'/bootstrap/cache/config.php';
    }

    /**
     * Get the path to the routes cache file.
     *
     * @return string
     */
    public function getCachedRoutesPath()
    {
        return $this->basePath().'/bootstrap/cache/routes.php';
    }

    /**
     * Get the path to the cached "compiled.php" file.
     *
     * @return string
     */
    public function getCachedCompilePath()
    {
        return $this->basePath().'/bootstrap/cache/compiled.php';
    }

    /**
     * Get the path to the cached services.json file.
     *
     * @return string
     */
    public function getCachedServicesPath()
    {
        return $this->basePath().'/bootstrap/cache/services.json';
    }
}
```

`bootstrap/app.php` change the default `Illuminate\Foundation\Application` to above:

```php
$app = new App\Foundation\Application(realpath(__DIR__.'/..'));
```

`bootstrap/autoload.php`

```php
$compiledPath = __DIR__.'/cache/compiled.php';

if (file_exists($compiledPath)) {
    require $compiledPath;
}
```

`bootstrap/cache/.gitignore`:

```
*
!.gitignore
```


I run beutify on everything now, looks great :dancers:


`|| die`


remove the quotes while at it, they're not needed


@oritwas The objv_tracker here was used for a different object type. You can just define a new empty one here, instead of reusing this one.


I think this recursively walks the entire flow tree for every running animation, I think you want to store a map from node -> animation instead.


@mwotton Check this out!


I didn't check all the deps, but remember that you need to ensure a minimum version that is either EAPI=5 or multilib.


This commit is moving you in the wrong directly.


So apparently you didn't make this commit, @mvdstam

Is there a reason you seem to have "vagrant@homestead" attached to your profile?


On Thu, Apr 2, 2015 at 12:41 PM, Fabio Kung <notifications@github.com>
wrote:

> @fdr <https://github.com/fdr> the problem with volumes and bind mounting
> is that hsup may be running a different place (namespace, host) than the
> docker daemon, and volumes being bind mounted with -v src:dst are always
> relative to where the docker daemon runs.
>
> The alternatives (that I can think of) don't look very good either:
>
>    1.
>
>    instead of bind mounting the binary, send its bytes inside the
>    container (stdin, pipes, sockets, etc) so that hsup can self-update. Need
>    to still be careful to send the bytes for linux-amd64, in case the outer
>    hsup is running on a different architecture (darwin)
>
> This is an option if you figure out how to swing it. It's already a
problem to be careful about sending linux-amd64, so I don't understand how
the matter of semantics get worse...rather, wrangling a new transport for
it seems like the part that's net-new and could prove either easy or really
annoying.

>
>    1.
>
>    make the docker image cache optional, and the default path is to
>    always build a new image and bake a new hsup
>
> This may prove better than making a cache that can deliver confusing
results with frequency, inviting frequent blowing of the cache.

>
>    1.
>
>    make the outer hsup somehow discover the right path of the binary to
>    be bind mounted where the docker daemon is. Maybe do it manually first,
>    with a configurable (fixed) location, then try to auto-discover the proper
>    hsup path where the docker daemon is. This still looks fragile, as there's
>    no way to guarantee that an hsup binary will be present where the docker
>    daemon is: just imagine hsup pointing to a remote docker daemon with
>    DOCKER_HOST=tcp://..., or hsup running purely inside containers and
>    not being installed directly on the host alongside the docker daemon.
>
> Sounds like this is too complicated to be useful, as you appear to suggest.

I'll suggest another variant, which is I think worse than the first option
you itemize and related to the second, but potentially may be somewhat
cheaper to implement.  That is to support *slug* caching in a lookaside
directory, perhaps defaulting to tmp.  That's because the main benefit of
the image caching is not so much the image baking itself, which will add
but a few seconds, but slug download over links not inside EC2, which is
highly useful for rapid iteration debugging hsup infelicities on a
problematic app locally.  Sensible defaults using "/tmp" can reduce the
load even further.


Correct. Must be PSScriptAnalyzer. Thanks for catching this.


MarkdownDocs (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1675 of 1676 tests passed)

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


tsk tsk


OK, I'll try it tomorow. Thanks


Uses a variable less and keeps style on other random team generators. Just that.


activation old iphone 5


Nice find. Do DEBs have a similar issue?


That would work I guess. The whole api here is a bit obscure and it's a reminiscent of the earlier structure, where a bucket had only a single object representing it. Take a look at RGWBucketInstanceMetadataHandler::remove(), that does what you need to do here (call bucket_instance_remove_entry() instead).


something


I just upgraded to factor 2.4.1 and noticed that the single quotes are throwing up a usage message for ip link show when the interface is empty. This wasn't a problem before but with the quotes, it is trying to execute `ip link show ''`. I modified ip.rb to get around this by adding the following:

242:  if interface =~ /:/ or interface.length == 0


@akohlsmith Can you explain? Perhaps open a ticket?


I see it: "...treats any Postgres schema other *then* `public` as a namespace. Gonna merge it, becuase I see one of my sentences is a little sucky, too, so Imma change that next ...

"For example, to steal a previous example..." Yikes.


Really? It was probably random spaces and indents; I formatted with
netbeans.
On Apr 2, 2015 6:21 PM, "Derek" <notifications@github.com> wrote:

> Oh okay. It says you made a ton of changes.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Derrreks/SideScroller/commit/f3e46ba39e1b4f352548db30609e89ae7be154fa#commitcomment-10555651>
> .
>


This code appears to be correct.

Can you explain your choice to replace a string with a method that returns a string?

I think the changes would be clearer if they were implemented in separate commits, but that is just a matter of style.


proc pises do komitu anglicky a do kodu cesky ? :D


I guess I have a solution for that which helps here without doing such a hack!


这样会不会有原本继承自父进程的一些默认环境变量都没有了


null cehcks


@navneil nice catch -- thanks!

Do you mind creating a ticket at tickets.puppetlabs.com and putting up a pull-request with that change?


这个replicate_num master那边也会变吧。。 update job以后 这块没变？ 重启的任务会有感知 这块会不一致吧


Thanks Kaushal, this side effect was not intended, but I see why this is happening. I'll have it fixed tomorrow.


if anything, move this to a wiki entry. i didn't like how far up this was in the README, and it took up a decent amount of space

that example Vagrantfile is going to need to be maintained, imo. could also consider adding a `Vagrantfile.windows` or something to the repo.


Note that bucket is unused. Also, maybe change it to syncing_bucket_meta()? or something along these lines, because bucket can still have their data synced.


Fixes the double checking for #22


This is pretty cool to work on!


yes, index join preserves sorting and grouping (in the current implementation)


You'll need to install compass v1.0+, as per our install docs.


Silly me.


@oritwas overall good. See my comments, and a few notes:
 - drop the submodule changes (both commits)
 - add rgw: prefix to the first commit message line
 - add Fixes: #<bug number>
 - add Backport: hammer, firefly

Also, did you verify that the bucket index object is also getting removed (should probably need to put data in the bucket to verify that)? I'm not 100% sure that's the case, in which case we'll need to open another bug, and have a similar fix. Thanks!


No worries. Whatever you have was working with everything else we were using. Animation is one of the newer modules.


use for-each iterator:
for(View view : subViews) {..}


code duplication


try to avoid instanceof, is it possible getTag to return some single type object and have a property to check instead?


This should probably be using the ansible built in line in file module rather than grep. Doing it with grep could cause issues in multiple edge cases.


Closes #23.


I dont really care. He asked for a binary, and i provided. I dont see why people have such a hissy fit over such a small binary in the repo. It looks like justinmeister feels the same as he has not touched that pull request.


r+


MarkdownDocs (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1667 of 1670 tests passed)

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


I like all the cleanups here. It would be easier to use if you could enable/disable more than one feature at once. That way the user won't have to worry about ordering for dependencies between features.


>IMO: The window separation make the code in filter_kernels.c prettier (code reuse, clarity (ie. see the sinc/jinc aliases) and no more duplication of the radius/blur/windowing logic. There's also a clear separation between the mathematical functions themselves and the way they're combined, which means they often end up simpler and significantly more readable than before), but the code in gl_video.c gets somewhat uglier. Maybe there's a way to make both prettier?

It would be nice if there's a way.

Just a random idea: use struct filter_kernel for both real filter and the window, something like:
```C
struct filter_kernel {
    struct filter_kernel *window_filter;
    char *window;
   ....
};
```
It would avoid the nesting. Though it gets a bit icky with memory management.

Anyway, with the possibilities explored and exhausted, I have no strong opinion about this anymore.

>This something I also thought about and honestly I think it's the best possible compromise. I will continue to use my repository as a playground for my own personal wants and desires, but I will make it my responsibility to ensure they flow back upstream. As for things that we decide won't go back into upstream (for whatever reasons), I think I will have to accept demoting them to branch status. (Users who really, really want crazy feature X like sphinx can still merge from that branch themselves)

OK.

>(Incidentally, public exposure to my “placebo” fork has already resulted in useful bug reports that I would not necessarily have noticed had it been sitting in a pull request)

Basically as I suspected in my previous post. Frankly, sometimes I push half-baken features to git master because nobody tests them and further improvements are hard without testing.

>This is indeed something I was not explicitly aware of, and so I apologize for pushing something larger without an explicit pull request - I guess I got carried away by the flow of things and fooled myself into thinking I had “authority” over gl_video.c, which incidentally also contradicts what I originally said when you gave me push rights in the first place (that “I wouldn't push anything major without having it looked at in pull requests”).

OK. Well, when I gave rcombs push access to libass, we ended up with multiple forced pushes to fix mistakes, but I guess it ended up well. So I guess stuff happens. My reaction of removing push access was probably too strong too. (Or not? I don't even know. Not making any explicit rules was a mistake in the first place.)

You wrote or rewrote half of gl_video.c by now, so I think you have some right to control over it.

>So I was wrong to do so on both accounts, and it's not the first time I trigger-happily pushed something that made you uncomfortable, either - so I feel a bit silly on the matter overall. I'm not sure if I'm responsible enough for direct push access.

We can try it, by exercising trigger discipline.

>I think in this case we can agree to disagree about what's “correct” and what's “incorrect” as a matter of opinion/goals/preference, and instead stick to the status quo of “just make sure it works like VSFilter” when it comes to ASS compatibility / subtitle rendering in general. That's a “neutral” stance that I think we can both follow, regardless of whether or not we think we're in the right - which is something you've clearly also been doing in the past (with regards to BT.601 compatibility hacks etc.).

I can't help to get severely annoyed by the absurdity ASS brings with it. But yeah, this works for me.


where is code attribution for hostname magic?


Will you add them?


Thank you, Thomas!


saw approval from jdm
at https://github.com/gilles-leblanc/servo/commit/fe3cffd4666e025192c2d8054ced9901166d9eeb


merging gilles-leblanc/servo/issue-4476-b = fe3cffd4 into auto


gilles-leblanc/servo/issue-4476-b = fe3cffd4 merged ok, testing candidate = d2516a89


OK, so this is actually incorrect... because it only works for constant literals... For actual floating point values in computation ("register" values) we have to store the actual `SmartFloat` variable pointed to by `Val.SmartFloatPtr`. Apparently we need a way to tell whether a `GenericValue` was derived from a constant literal or not...


There's a known bug with putting null in welcome_message.


He also hasn't touched this project in a while too. Regardless, I agree... even though it's bad practice, I've done it before too. In my case were solution binaries I didn't want my students to see the code for, but wanted to provide in the repo.


Why include this as a concern vs. adding the `blockly_options` method directly to the `Blockly` model?


Keepin' it clean.  Nice!


Maybe, but certainly not in this patch-set. Ocsigen_stream is rather subtle and complicated and is still used in other parts of ocsigenserver. Let's first merge this stuff, then we can think about removing ocsigen_stream. :)


about out of source builds: I looked at out of source builds. If it does support them it is not exposed or I simply can't find it.


Yep, will do in a bit.


What does it mean to get multiple inputProperties for PropertyDerivations?


1. The comment is not proper english, and rather impossible to understand.
2. promotion is just a map function, nothing to do with substream here (as shown by the type).
3. It's always used with (fun x -> x).

=> Removing.


I appreciate the concern but the intern bit is incorrect/false and a switch statement actually yields more optimized code than an if/else provides.

Since you asked:
```java
        String f;
        switch(s) {
            case "xyz": case "123": case "def": f = "yes"; break;
            default: f = "no";
        }
        return s;
```

Yields:
```
       0: aload_0
       1: astore_2
       2: iconst_m1
       3: istore_3
       4: aload_2
       5: invokevirtual #2                  // Method java/lang/String.hashCode:()I
       8: lookupswitch  { // 3
                 48690: 58
                 99333: 72
                119193: 44
               default: 83
          }
      44: aload_2
      45: ldc           #3                  // String xyz
      47: invokevirtual #4                  // Method java/lang/String.equals:(Ljava/lang/Object;)Z
      50: ifeq          83
      53: iconst_0
      54: istore_3
      55: goto          83
      58: aload_2
      59: ldc           #5                  // String 123
      61: invokevirtual #4                  // Method java/lang/String.equals:(Ljava/lang/Object;)Z
      64: ifeq          83
      67: iconst_1
      68: istore_3
      69: goto          83
      72: aload_2
      73: ldc           #6                  // String def
      75: invokevirtual #4                  // Method java/lang/String.equals:(Ljava/lang/Object;)Z
      78: ifeq          83
      81: iconst_2
      82: istore_3
      83: iload_3
      84: tableswitch   { // 0 to 2
                     0: 112
                     1: 112
                     2: 112
               default: 118
          }
     112: ldc           #7                  // String yes
     114: astore_1
     115: goto          121
     118: ldc           #8                  // String no
     120: astore_1
     121: aload_0
     122: areturn
```

As you can see, this uses `hashCode` to jump to an `equals` virtual method invocation.

So, not only does it _not_ use `intern`, it results in O(log N) complexity versus O(n) complexity for the if-else approach.

To boot, its more readable.


fdsfsfds


I checked they were all multilib. Did I missed something?


happy to walk through, and this stuff is easy to cherry-pick (with the exception of the scheme settings)


@chirag200666 I'm confused: you want this PR to be merged or not?


This should get merged in.


@SubJunk Too easy mate


This is a little confusing to me. Is external ID the ID returned by the single sign-on flow?

(Nit: I'd suggest spelling out acronyms the first time you use them, just to make sure there's no confusion.)


This can't be right. I'm trying to go off of the 2 examples below:

class Comment < ActiveRecord::Base
       belongs_to :post
end

&

class Post < ActiveRecord::Base
	has_many :comments
end


Can you use rebaseAndPropagateChildProperties(...) here?


same as above, can you use rebaseAndPropagateChildProperties here?


This can't be right.

I looked at the different ROUTES. Trying to match HTTP verb with the appropriate controller.

Because I'm getting a ROUTE error. Good thing is I got "Advertisement" to show as a link on the homepage. I believe the assignment wants me to index an advertisement's "title", "copy", and "price" once I click on that hyperlink.


@pavanky Take a look at the whole kernel.


layout with top blox


rax retry tests


thx


update the source


I did. tid goes in steps lile the following.

- tid(0): 0 -> 256 -> 512 -> 1024...
- tid(256): 256 -> 512 -> 1024...
- tid(512): 512 -> 256 -> 1024...


I updated the comment, please check before commenting again.


I suspect this entire folder is not needed for the demo.  I don't know if it's supposed to be committed with Sencha apps or not.


88737258-EPW-Ruby-Access (jasondegraw) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2094 of 2201 tests passed)

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


Also it was failing only for 3d. Which means the original code was correct for 1D and 2D.  The correct loop offset should be like this:

```
dim_type loop_offset = blockDim.x * blocks_x;
```

The earlier offset was wrong because `gridDim.x` was equal to `blocks_x * blocks_z`.


centers_file=sys.argv[1]
raw_centers = ""
with open(centers_file) as file:
    for line in file:
        raw_centers += line.strip()
centerList=eval(raw_centers)


hpwh_residential (nmerket) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


This program will work, but what is the runtime in Big O notation ? This is less efficient than the standard merge. Consider implementing the merge algorithm described below.

Merge algorithm

Assume, that both arrays are sorted in ascending order and we want resulting array to maintain the same order. Algorithm to merge two arrays A[0..m-1] and B[0..n-1] into an array C[0..m+n-1] is as following:

Introduce read-indices i, j to traverse arrays A and B, accordingly. Introduce write-index k to store position of the first free cell in the resulting array. By default i = j = k = 0.
At each step: if both indices are in range (i < m and j < n), choose minimum of (A[i], B[j]) and write it to C[k]. Otherwise go to step 4.
Increase k and index of the array, algorithm located minimal value at, by one. Repeat step 2.
Copy the rest values from the array, which index is still in range, to the resulting array.


91069532-FixUnitTestFailures-Part2 (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1170 of 1170 tests passed)

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


I'm having trouble finding this text: "You specified --require-macs".  Having the newlines part of the translation text can cause problems on the translation site.  Would be better if the whitespace was trimmed on both sides.


Would be better to leave leading / trailing whitespace out of the translation text.  Eg
cout << "\n\n" << _("You specified --require-macs [...]") << "\n\n";


```LUA
str = string_gsub(str, "/.*/(.*lua:%d+:)", "%1")
str = string_gsub(str, "'/.*/(.*lua)'", "'%1'")
```

这个函数的主要功能可以使用上面两行代码实现，代替以上代码。


What if we named it after an animal?


Changes:

-Updated README with proper commands to compile and run
-Added methods doCreateEntry and doGetEntries


tem que testar ae né vei


88737258-EPW-Ruby-Access (jasondegraw) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


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

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


r+


//http://simplesandsamples.com/reverse.java.htmlからコピペ
import java.io.*;
import java.util.*;
class ReverseString{  //TestをReverseStringに改変
    public static void main(String[] args) {
        String src = "stressed";
        StringBuffer sb = new StringBuffer(src);
        String dst = sb.reverse().toString();
        System.out.println(""+dst);
    }
}


rax retry tests


done


MarkdownDocs (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1674 of 1676 tests passed)

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


True. Adjusted


saw approval from jdm
at https://github.com/zerokarmaleft/servo/commit/d00bb057ba33f11c3f08ab0110bed6a07c74c3f4


merging zerokarmaleft/servo/wpt_iframe_tests = d00bb057 into auto


zerokarmaleft/servo/wpt_iframe_tests = d00bb057 merged ok, testing candidate = 54a6e081


@mkdolan sure, emailing you to setup a time for a call.


Where do you put this code??


@Optimusprimeums tags are going up now :)


Why do you delete this var?


Your changes to the makefile don't work with my nodemcu_uploader.py. Do you have the latest version?
I have made the speed configurable in the master branch, however, based on what you did.

Can you verify that you have the latest nodemcu_uploader.py?


@bixycler  sao không gọi đc nhỉ?


90119792FourPipeInd_ErrMsg (mjwitte) - i386-Windows-7-VisualStudio-12: OK (1679 of 1681 tests passed)

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


À, thì ra chỉ dùng @mention đc trong comment thôi: @trphloc, @HCMUS-OSSD/owners


Can you explain why upadeaplist.lua works the way it does?

I would expect the script to simply return the json on the fly rather than creating a file with the information which you need to get later. It's a lot more straightforward that way and serves better as an example, which is what I hope to bundle with the server.

I think something named http/ap_list.lua (see underscore, which follows naming convention of other files) that returns the json on the fly would be better.


The new test needs to include the test harness JS. It timed out on the build machines.


Đáng lẽ phải dùng @mention được trong các file markdown! VD trong danh sách nhóm, danh sách thành viên.


90119792FourPipeInd_ErrMsg (mjwitte) - x86_64-MacOS-10.9-clang: OK (1672 of 1675 tests passed)

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


golang official go's module is now at golang.org/x


Tsk


Was adapted from lean kernel... however it was reverted later and labelled as unstable.


In fact 复杂(complex) and 困难(difficult) are very different here. A difficult issue can be also complex, but not vice versa.


delete blank line for me


Hi @cchristelis, I have fixed it here. The geometry of the impact layer isn't the same with hazard layer (the interpolated layer based on the exposure). I just changed the geometry to `geometry=interpolated_layer.get_geometry()`. If you see in ```vector.get_geometry()```, passing `as_geometry_object=True' will only work for polygon data..


Ya, i noticed it was working for 1D and 2D. But I didn't notice the overlap with the modified offset. I shall fix it.
Thanks for clearly explaining it.


Sorry old commit message, meant to say reversed the personal comments page posts are now from most recent to least


Next time I will. Thank you for the fix.


Possibly remove the not and flip the cases.


@blairanderson Im showing all the sheets in json at this point. I created some demo sheets to see some data


@stresslimit quite rough but its working. It will look even simpler after some basic refactoring. I have a very rough proof of concept working now but need to begin testing to cover the myriad of cases, resize events, combinations, edge-cases, etc. To try the demo yourself checkout the commit before this one and do `npm start`.


Created serializer but not sure if i will need the user serializer


Check out the names in your honor!


I dont know why the jquery here is not loaded


MarkdownDocs (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (657 of 658 tests passed)

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


Notes to self:

* Probably don't need nib/vendor at all, this thing's only targeting Blink, derp
* Remove `-moz` style from `octicons.styl`
* Could use `getElementsByClassName(...)[0]` instead of `querySelector`, might not matter much
* Rename `_playthroughHandle` to `_canplaythroughHandle` (I got lazy over 3 letters after typing 11? Really?)


An animal... seems a little arbitrary but it might also work - did you have any specific animal (or species) in mind?


Complete Assignment! WikiYEAH!


cool; fixed in 322b004


all tests pass:
success: http://build.servo.org/builders/linux1/builds/1255
success: http://build.servo.org/builders/linux2/builds/1253
success: http://build.servo.org/builders/android/builds/1289
success: http://build.servo.org/builders/mac1/builds/1468
success: http://build.servo.org/builders/mac2/builds/1471
success: http://build.servo.org/builders/gonk/builds/434


ok fixed in 7b67483


Just left over.

> On Apr 1, 2015, at 9:41 PM, Cesar Wong <notifications@github.com> wrote:
>
> What's the purpose of the position identifier? %[1]s ...wouldn't %s work just as well?
>
> —
> Reply to this email directly or view it on GitHub.
>


AHHH i JUST DID THAT Damnit I guess i should check here first then


Urk. Sorry. I assumed I should do that with the dictionary.
On Apr 1, 2015 10:10 PM, "Falon" <notifications@github.com> wrote:

> AHHH i JUST DID THAT Damnit I guess i should check here first then
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/falon3/Wordless/commit/beb796c2cca83a3e81ba0200b01b8111eb578c66#commitcomment-10538215>
> .
>


Android with Eclipse still need line 7
<classpathentry kind="src" path="libcocos2d"/>


tex 折行


fast-forwarding master to auto = 54a6e081


is there a way to push this logic down the stack? maybe in the controller we could have @items = Item.where(status: active) and @sold_items = Item.where(status: sold) and loop through them separately? seems like a lot of logic for the view


hwahamoda.com


Best commit! 10/10


hwahamoda.net


could we make this a partial?


Overall, the code also looks good to me. The `isGet` field looked a little specific to me at first, but I actually cannot think of another request type (i.e. `HEAD` or `GET`) that would need special logic here. Perhaps `TRACE` may? In any case, maybe `TRACE` should be offloaded to the application-level. That aside, I think this looks good.

When I rebuilt this branch, though, I could not get the warp server to respond correctly. Perhaps this is just the handling of `HEAD` in wai-app-static? I may be able to spend some time and try to test this out more in the morning. My results:

```bash
$ curl -4 -I -X GET http://localhost:3000/wai/LICENSE
HTTP/1.1 200 OK
Content-Length: 1085
Accept-Ranges: bytes
Date: Thu, 02 Apr 2015 04:12:41 GMT
Server: Warp/3.0.10.1
Content-Type: application/octet-stream
last-modified: Sun, 29 Mar 2015 16:31:39 GMT

$ curl -4 -I -X HEAD http://localhost:3000/wai/LICENSE
HTTP/1.1 200 OK
Transfer-Encoding: chunked
Date: Thu, 02 Apr 2015 04:12:49 GMT
Server: Warp/3.0.10.1
Content-Type: application/octet-stream
last-modified: Sun, 29 Mar 2015 16:31:39 GMT
```


the issue with delaying fastLocalsToBoxedLocals is that we can't do it after the frame has been popped, right?

there are other wrinkles too, apparently calling `locals()` a second time also converts from fast locals to a dict (`f` below), and even returning the locals and frame from a function doesn't populate the locals dict until f_frame is accessed:

```
import sys

def f():
    d = locals()
    a = 0
    print 1, d
    locals()
    print 2, d

f()

print "--------"

def g():
    d = locals()
    a = 0
    print 3, d
    b = sys._getframe(0)
    print 4, d
    return (d, b)

(_d, _b) = g()
print 5, _d
_b.f_locals
print 6, _d
```

```
1 {}
2 {'a': 0, 'd': {...}}
--------
3 {}
4 {}
5 {}
6 {'a': 0, 'b': <frame object at 0x7ffff7e773e0>, 'd': {...}}
```


hwahamoda.websait


1. Add 'Library' option. It should be there even if it is a default option.
2. PHPUnit is for test the development and not require to run the application. It needs to be moved to require_dev
3. There is an option for auto-load library which will help you do load the library from index.php. Please add the same and make the appropriate changes in index.php.
4. Include archive option to avoid to download unnecessary folders (like demos, etc)
5. Authors option is not required and it can be removed.

6. As per the composer documentation we need to commit the composer.lock file along with these 2 files. So that everyone have the same version of software installed.


saw approval from jdm
at https://github.com/mattnenterprise/servo/commit/818f1c57480a8dc6ea61ea7d01973a8bc5c1fcbd


merging mattnenterprise/servo/http_compression = 818f1c57 into auto


mattnenterprise/servo/http_compression = 818f1c57 merged ok, testing candidate = dbd16932


hwahamoda.eg.com


I didn't test the container with the readonly mounts, but I am assuming that Brendan manually verified it.


h w a


1570-EnergyPlus-8-3 (kbenne) - i386-Windows-7-VisualStudio-12: Build Failed

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


hwa


hamoda


Nope! I am all ears if you have an idea.


Yes, I guess `wai-app-static` should be modified.

Here is a modification of my `wai-app-file-cgi`: https://github.com/kazu-yamamoto/wai-app-file-cgi/commit/c3998510b6e3546b370cf90500037889f5a8621f

The point is that HEAD can be treated by the GET code.


hwahamoda.com


Thank you, I'll try thank you.


@gakimball shouldn't this be align-top to match the code below?


Is this so that projects can decide if they want to put resources in separate assemblies?


91362860-Relax-fatal-error-for-DX-coil-zero-capacity-or-flow-rate-add-more-sizing-output-#4826 (rraustad) - x86_64-MacOS-10.9-clang: OK (1669 of 1671 tests passed)

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


just try


nah it's fine, I didn't waste a lot of time. And it wouldn't have been nice and organized yet either


Oh that's where the dangling s originated.  I prefer the single sc21.tex than
many input files.  It's a pain to check on consistency or search for a string
while editing.


- Added support for cooling
- Changed the LogStatements


I was thinking we can make this even more flexible if instead of setting shortCircuit variable to a boolean value, take a number of contiguous ping failures that could be ignored as an argument and set the expect string with that many 'X'. This way for example if a topology has only two hosts test case written for that topology would know what is an acceptable value to pass here.


etyuy


ytry


sorry, for lot of errors,  this was not tested :(


Wow this looks really annoying...

Here's CPython's implementation of f_locals for reference

```
static PyObject *
frame_getlocals(PyFrameObject *f, void *closure)
{
    PyFrame_FastToLocals(f);
    Py_INCREF(f->f_locals);
    return f->f_locals;
}

void
PyFrame_FastToLocals(PyFrameObject *f)
{
    /* Merge fast locals into f->f_locals */
    PyObject *locals, *map;
    PyObject **fast;
    PyObject *error_type, *error_value, *error_traceback;
    PyCodeObject *co;
    Py_ssize_t j;
    int ncells, nfreevars;
    if (f == NULL)
        return;
    locals = f->f_locals;
    if (locals == NULL) {
        locals = f->f_locals = PyDict_New();
        if (locals == NULL) {
            PyErr_Clear(); /* Can't report it :-( */
            return;
        }
    }
    co = f->f_code;
    map = co->co_varnames;
    if (!PyTuple_Check(map))
        return;
    PyErr_Fetch(&error_type, &error_value, &error_traceback);
    fast = f->f_localsplus;
    j = PyTuple_GET_SIZE(map);
    if (j > co->co_nlocals)
        j = co->co_nlocals;
    if (co->co_nlocals)
        map_to_dict(map, j, locals, fast, 0);
    ncells = PyTuple_GET_SIZE(co->co_cellvars);
    nfreevars = PyTuple_GET_SIZE(co->co_freevars);
    if (ncells || nfreevars) {
        map_to_dict(co->co_cellvars, ncells,
                    locals, fast + co->co_nlocals, 1);
        /* If the namespace is unoptimized, then one of the
           following cases applies:
           1. It does not contain free variables, because it
              uses import * or is a top-level namespace.
           2. It is a class namespace.
           We don't want to accidentally copy free variables
           into the locals dict used by the class.
        */
        if (co->co_flags & CO_OPTIMIZED) {
            map_to_dict(co->co_freevars, nfreevars,
                        locals, fast + co->co_nlocals + ncells, 1);
        }
    }
    PyErr_Restore(error_type, error_value, error_traceback);
}
```

Anyway this looks really annoying; CPython can do this because it actually stores its "fast" locals on the frame object, so it still has them around. This seems extremely unpleasant for us to support on our end, since as far as I can tell the only way to support your code would be to snapshot all the local variables at the end of the function call, since otherwise there's no way to support this:

```
def f():
    frame = sys._getframe(0)
    a = 0
    return frame

frame = f()
print frame.f_locals
```


Accept this


忽略eclipse 的项目文件


@iksaku The world seems to want to set aside for each permission.


I'm glad you thought the idea was cool! You're pretty cool, yourself.


This format should now be supported by OpenSlide master.


Where should I put this inside dispatchDisconnectClient or getCharacterList
 if(!g_config.getBoolean(ConfigManager::ENABLE_LIVE_CASTING) &&   !ProtocolGame::getLiveCasts().size()) {
         disconnectClient("There are not casts available atm.", version);
         return;
      }


"Beta testing in complete for" typo.


Thanks! :P


I think it was behaving as a `branch-merge` because `HEAD` got diverted yesterday. I committed a few during my lunch time but didn't have any wifi, so I needed to wait until I got home to push it. By then you had already committed something so git got upset.


Thanks! I seen someone else revert it but I don't seem to have any adverse
effects from it. Still I may revert it as well if those more knowledgeable
than I are doing so. Thanks!

Kind Regards,
Zachariah Kennedy

On Wed, Apr 1, 2015 at 10:45 PM, Pranav Vashi <notifications@github.com>
wrote:

> Was adapted from lean kernel... however it was reverted later and labelled
> as unstable.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/neobuddy89/shamu/commit/98dc16a6d151d2747e9f1825b567d802126fbce0#commitcomment-10538053>
> .
>


support pip and add default credentials


Hey, this doesn't have the "flashes" styling I had added. Could you add that to the LESS file and then compile?


Yes. I've removed the mistakenly inserted text. Thank you.


Would the same change need to be made for line, triangle, circle and image?
context.setGlobalCompositeOperation("destination-out")?


Agreed about the logic. Once we get it all working tomorrow and pushed to heroku, let's do it. I definitely need to get more practice at pushing down the stack.


90119792FourPipeInd_ErrMsg (mjwitte) - Win64-Windows-7-VisualStudio-12: OK (1679 of 1681 tests passed)

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


Probably, I just committed the same for the other methods.


Prefer editing an enum than a file, see below (shortly).


Perhaps we should wait for a few more days and add some photos of our weekend trip?


"Correct! Thanks for not being a bot.." should get it's own line in the language configuration.

My wording can really suck at times.


H


90179004-ConstrNumBackNotInitializedComplexGlazing (jasondegraw) - i386-Windows-7-VisualStudio-12: OK (1680 of 1681 tests passed)

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


hpwh_residential (nmerket) - 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-69.42%25-red.svg)


why "should" on inId and outId and not just must and exists filter on one of them? i think it will be much faster.
also if there are another "should" in our final filter it could not consider it.
also i saw there is missing filter, maybe it will be better on vertex query.


+1

As you (all of you) think of stuff along the way that you want to share, please add it in.


There's a config entry for how long the login delay should be (I believe there is), instead of 10 seconds, you could change it to 5 seconds if you wanted to.  Whenever it gets changed, it should be reflected in this message.


test github


all tests pass:
success: http://build.servo.org/builders/linux1/builds/1256
success: http://build.servo.org/builders/linux2/builds/1254
success: http://build.servo.org/builders/android/builds/1290
success: http://build.servo.org/builders/mac1/builds/1469
success: http://build.servo.org/builders/mac2/builds/1472
success: http://build.servo.org/builders/gonk/builds/435


Sure it'll just have to be done in a week since I've got a crapton of things to do for the next week or so.

Just delete this file and I think you'll be set, since the code you entered in was not designed to drop a file that already exists in the JAR anyways.


fast-forwarding master to auto = dbd16932


hpwh_residential (nmerket) - 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)


Wait whoops guess I was wrong, but either way it shouldn't be hard.

The only reason why I personally think it's bad practice is because it has lots of issues with reloading plugins (classloader stuff), because then eventually it'll lead into unexplainable JVM crashes (crashes that I believe* don't print out exceptions, it only prints out JVM specific information) and causes a nuisance in the long run.

For now I would recommend commenting this code out and leaving the language stuff at default, since this section will have to be tweaked upon when I get a chance at rewriting it.


1. Je hoeft geen cb mee te geven met de makeMessage en makeResult functies, je kan gewoon return gebruiken hier... we gaan geen asynchrone call naar de database doen

2. je ziet dat je zelf [] moet doorgeven aan een functie... dit zou niet mogen, je kan dit oplossen door een config object mee te geven aan de functie en op het einde waar je de array zou gebruiken, messages || [] te gebruiken.


Hier opnieuw, je hoeft geen cb te gebruiken voor deze functie, je gaat niet naar de databank.


Cool ! :100:


build.xml probably has to be updated, too?


If the main configuration and the language one is going to be separated, let's rename this method to "loadMain()" in order to be consistent with what settings we're loading.  I hate the name "SettingsCore" though (that was my crappy way of loading configs back when Bukkit had crappy configuration APIs & wrote my own configuration system).


If you think about this, long and hard, is this efficient? Maybe these could be parsed upon constructor loading instead of toString(), when the translation of the chat colors has to be built every time a message goes out to the player(s).


These variable names are a bit confusing, maybe key & value is sufficient?


I'm pretty sure, it isn't a good idea. Just take a look at http://a.pomf.se/trhnyq.jpg


I actually didn't notice/realise this script was already in the repo when i did this. Whoops. Oh well.


Seems that didn't work. AttributesGridAdapter.java has one.


Oh it isn't. Waiting on some fixes with other plugins


This change removes Maker usability.


hwahamoda.eg.com


Overall I think the implementation is pretty well designed, there just seems to be a bit of things common Bukkit developers do (dropping files, and while it's a convenient thing to do, using getResource() is a pretty intensive call, because you have resources in the JAR that can't be generated on it's own).

But none the less, I think you're doing a marvelous job and praise you for taking up the effort to update this project to 1.8.  Little known secret, I'm thinking of starting a server again so I may even resume development on this as well.


Why this methods is package private ?


@gabbiano I updated my original comment


This would also have been the chance to replace the google code URL here but okay.


91362860-Relax-fatal-error-for-DX-coil-zero-capacity-or-flow-rate-add-more-sizing-output-#4826 (rraustad) - x86_64-MacOS-10.9-clang: OK (1669 of 1671 tests passed)

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


But why is it empty in the code?