Quoting Kyle VanderBeek (2016-01-22 11:35:04)
> Interesting. It seems that FreeBSD removed clause based on some sort of legal document nearly 12 years ago. The change was [implemented across the entire system](https://svnweb.freebsd.org/base?limit_changes=0&view=revision&revision=216370). The mention of that letter coincides with the notes on the OSD compatible [BSD 3-Clause](https://opensource.org/licenses/BSD-3-Clause).
>
> Is it reasonable to infer that the "quota program" being referred to is quota.c in the usr.bin collection?

What is safe is to not infer anything but ask the developers to kindly
clarify the license by which they are basing their work, and if they
cannot then ask them kindly to rewrite to avoid basing on works of
others for whom they cannot adequately provide proof of free licensing.

 - Jonas

--
 * Jonas Smedegaard - idealist & Internet-arkitekt
 * Tlf.: +45 40843136  Website: http://dr.jones.dk/

 [x] quote me freely  [ ] ask before reusing  [ ] keep private


* Arreglada la manera de reconocer la versión en RHEL y CentOS
* Cambiado el JBOSS_HOME


для этого есть \yii\web\JsExpression


`suite.parent = null`


Could you add a newline? Also, do the same for `model` and `TypeError` as for classification.


@DRedDog i haven't edit Server.php yet, will do it!


:+1:  Thanks!


Because this is the correct way of defining Q_PROPERTYs. At least this is how QtCreator's refactoring creates implementation of Q_PROPERTY. It is necessary for setting this property from QML side. Probably it would be enough to declare setWindowForActivateRequesting as just a Q_INVOKABLE but this is a canonical way from my point of view.


These changes are for last test case in the client_resp_test.go


i have Notepad++ but i can't find the replace..


Please dont revert my changes. There were errors in the Script.


Sorry for the bogus commits. But they also dont do much harm.


These four added spaces in line 17 where the comment is.


need call stopService without excess abstractions.


а что, если сделать такую логику:
```js
this._children = _.without(this._childrent, suite);
suite.parent = null
```

тогда и клонировать ниего не придется


URL is gone: http://hydra.nixos.org/build/30760174/nixlog/5/raw


README.md was not necessarily edited


Have updated the commit message. Travis should not be an issue since it is super easy to see that this code is bug free.


I wonder if there's any commit with more comments than this one.


Changes in List and Download will now not work


url is gone http://hydra.nixos.org/build/30835132/nixlog/5/raw


This seems incorrect, "%*s" is based on bytes and it's useless for multi-byte letters. The "width" is about terminal columns rather than bytes. See how we use mbs_truncate() for scols_column_is_trunc() in print_data().


Actually, this might be a bug, although it's fairly unlikely to bite you.  You shouldn't typically use empty lists as default arguments:  http://effbot.org/zone/default-values.htm


url is gone http://hydra.nixos.org/build/30759350/nixlog/5/raw


This case is from http://redis.io/topics/protocol page. Changes above are made to pass it.
But notice, that you added a space between minus sign and error message, which is not in the protocol description.
But it can be considered a part of the message itself, so there is, probably, no harm in it. So, I just added it in test.


Ah! i understand you now. You're right, Context.resolvePath throws an error if the file isn't found. I didn't know it.

So, i think that you could restore the previous version + try/catch in Context.resolvePath. If it doesn't exist, then it continues to analize the next candidate.


It took me hours to fix Travis - I am blind :(
I can't even see the words "PHP7 support" in the pthreads changelog -_-


>This thread reads like 4chan.

It's 4chan that reads like this thread!


ok a tab, excuse me -.- these comments are a really good use of our time! We have accomplished so much! I could have finished the AntiSpoofAlgorithm but this is clearly more important.


@luiges90 读档你写在部队事件那里了，应该是事件表


Glad it helped!


Uploaded!


还是有些问题==


Do you need to update HivePrivilege to parse GRANT?


Thanks for the reminder. I'd probably rename your wrap fn to `wrap-request-agent` (not sure what you meant by `useragent-peak`?)

Oh, and perhaps instead of "real life use case" I'd say "using as ring middleware".


you can check syntax of json files here http://jsonlint.com/


 for (PersonDialog i : matchedScenBiography) {
                this.Scenario.YearTable.addPersonInGameBiography(i.SpeakingPerson, this.Scenario.Date, i.Text);
            }
这句话有问题，有波浪线，怎么理解？@luiges90


убери это отсюда!
не нужно в каждый шаблон пихать вывод ошибок. Добавь вывод ошибок в layout.html.twig и все.


All other comments on this commit are more important, right. Was just a short note.


我忘了一個地方沒改 = = :sweat:
在YearTable.cs，把 addPersonInGameBiography 前面的private改成public


Well; I guess we can live with this hack living here forever. ;-)


Could we use `blockTd` and `chainTd` instead of `B_td` and `C_td`? It breaks the naming style sooo badly.


https://github.com/krakjoe/pthreads there is PHP7 Support, read issues.


Shouldn't that be #455?


You could also add same for customPlaceholder


Control+h is normal replace, or (more useful) "search in files" under the search tab.. Or edit tab

> Am 22.01.2016 um 13:01 schrieb PocketKiller <notifications@github.com>:
>
> i have Notepad++ but i can't find the replace..
>
> —
> Reply to this email directly or view it on GitHub.
>


Props bro.

On 22 January 2016 at 02:46, mozboz <notifications@github.com> wrote:

> Thanks. Have invalidated this key and sorted this out. Infinite
> embarrassment, esp. after taking care not to commit my local db creds.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mozboz/friendbiz/commit/e9481536fe923422b9323884c8892277fa279e06#commitcomment-15602691>
> .
>


no, i am against get/create/delete method names because the operations classes are not always bound to one table


I don't suppose `addWriteLiteral` is the right method to write contents. What would be the correct method?

I also couldn't figure out how to create recursions here, for when the included templates will have include tags of their own


@Arno-Nymous :+1:


because the identify is an internal representation, you should not be able to set it.


great stuff. I'd love to see this as a branch of https://github.com/imagej/imagej-itk. can you do that?


@jdm thank you very much for the pointer, I know what to do now. It looks like I'm not the first person to assume that the borrow ends after the first if block! - https://github.com/rust-lang/rust/issues/6393


Q_INVOKABLE does not make sense here, as long as you only access the property via property bindings and do not explicitly call the method from QML.

Given the fact that "slots" is a macro that translates to "", "public:" and "public slots:" is the same thing. But you are not intending to use setWindowForActivateRequesting as a signal/slot target.

So I'd place it below `QWindow *windowForActivateRequesting() const;` as you do with all the other properties.


also there are other operations that do not have the short method names. we should achieve consistency, but not with these dubious method names. Also insert is the sql terminology.


Or CTRL-SHIFT-F
Thats find and replace in folders

> Am 22.01.2016 um 13:01 schrieb PocketKiller <notifications@github.com>:
>
> i have Notepad++ but i can't find the replace..
>
> —
> Reply to this email directly or view it on GitHub.
>


hi @Shougo how should I set the pairs for current filetype,
I hava add
```
let g:neosnippet#completed_pairs= get(g:,'neosnippet#completed_pairs',{})
let g:neosnippet#completed_pairs.java = {'(' : ')'}
```
this will miss the default pairs for other filetype. there is no `_` key in g:neosnippet#completed_pairs


if project will be used on server with php >= 5.3 then you write something like
 $title = l(t('Search @backend', array('@backend' => $backend_config['title'])), $path);


anonymize is not delete


Hey @niwinz. response.context is no longer populated after you removed these lines. Can you provide more info on the replication and how to fix this?

See https://github.com/niwinz/django-jinja/commit/3cad5db4ccfff575cba262c9f12ad683ec6813dd


Is the purpose of this method to convert the Presto privilege that a user can specify on the CLI to a privilege that the connector can understand, in this case Hive? If so, there's a mismatch between the privileges Hive supports and the ones that Presto offers.


Puedes mandar el numero de prueba como parametro? Tambien trata de crear una sola responsabilidad para tu clase. Ahora tu clase encuentra el resultado y tambien se encarga de imprimir. Por que retornas 0 siempre? Si yo soy un cliente de tu clase voy a creer que el resultado siempre es 0.


@thebigsmileXD that's what i was looking for.. Thanks


There is test which says the following:
Changed mgcamd config file message from "/var/keys" to "/usr/keys"


@afelisatti please take a look to this. Thank you


Fixing


Keep in mind that the spamassassin rules are height first then width and the new_image function is width first then height


I will try that. Thanks.


Because the code is already here: https://github.com/niwinz/django-jinja/blob/2e1aefd4fa61272881c8c91e3c4164cc6b2cd1ca/django_jinja/base.py#L59


All this documentation is now double. It should be removed from here, since this implementation is no longer in generateZigZagInfill (since it moved to ZigzagConnectorProcessor.cpp). Hence, if the implementation details of the ZigzagConnectorProcessor classes are changed, this functionality changes. What you could say here is that this function calls on ZigZagConnectorProcessor to generate infill, and describe what output is expected.


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


frans has shitty commit messages


Does this fix:

```

======================================================================
FAIL: test_should_use_current_year_for_dates_without_year_0 (tests.test_date.TestParseWithFormatsFunction)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/nix/store/h048qf8s6gicq18jl7vlaxdi5jghwx59-python2.7-nose-parameterized-0.5.0/lib/python2.7/site-packages/nose_parameterized/parameterized.py", line 365, in standalone_func
    return func(*(a + p.args), **p.kwargs)
  File "/tmp/nix-build-python2.7-dateparser-0.3.1.drv-0/dateparser-0.3.1/tests/test_date.py", line 247, in test_should_use_current_year_for_dates_without_year
    self.then_parsed_date_is(datetime(2015, expected_month, expected_day))
  File "/tmp/nix-build-python2.7-dateparser-0.3.1.drv-0/dateparser-0.3.1/tests/test_date.py", line 282, in then_parsed_date_is
    self.assertEquals(date_obj.date(), self.result['date_obj'].date())
AssertionError: datetime.date(2015, 9, 16) != datetime.date(2016, 9, 16)
```


You mean like this? Does that work on your mac still?

https://github.com/ianharrigan/haxeui/commit/bfd0c8bae00a776b84d2df99aa4fbfe061f5583d


But the operation is on one resource. Repeating the name from the class doesn't bring any value.


+1.
Nitpick: You can also change mqtt->data to const and cast at callback time.


for (PersonDialog i : matchedScenBiography)  这一句是不是要改成
foreach(PersonDialog i in matchedScenBiography)  呢？@luiges90


Welcome

> Am 22.01.2016 um 13:19 schrieb PocketKiller <notifications@github.com>:
>
> @thebigsmileXD that's what i was looking for.. Thanks
>
> —
> Reply to this email directly or view it on GitHub.
>


Just like `platform`, there's no difference. Actually I wanted you to query `GET /workers/:platform/identify` and use the returned `identity` in the `PUT /workers` request.


LGTM


done


It deletes the worker, just not his answers / ratings.


Looking at other methods in this class, if you break the line for arguments, you either have all arguments on separate lines or you don't break the line. See `QualifiedTableName` constructor in `getViews()` for an example.


Hi @bradymiller

Finally the ADODB is upgraded :)

A minor issue was there but I figured out that in sql.inc wherever we are using "Execute" if the second argument "$binds" is empty then make it false rather null

Also removed "&" from some more classes as well

Please review and let me know

https://github.com/practiceprovider/openemr/commit/df4abc181188300979f3f9046ba935e82cf9e718?diff=unified


The actual issue is the fact that librtmp is not thread safe.  It's not related to SSL support, so adding the  -lpthread into LIB_OPENSSL compilation flags is not the proper way to go.  Let's avoid making a single function thread safe and keeping the rest thread unsafe.  Application developers have to deal with it at the application level.

I propose to revert the commit.


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


"btw, received heal/hot statistic tab may also be interesting and should show hots that was applied out of meterstarter field of view (for example priest buff outside of dungeon in crossserver party)"
=> Should be working in the 0.57 version (just tested it (priest buff => balder (regen hp))

Some interesting thing would be:
=> be able to count constit debuff (and count % of the boss affected by the constit debuff)
=> be able to know what consummable / buff are you using while fighting (and if one of these buff / consummable disapear during the fight)


This is where sharks can hit BJ through windows and viceversa.
![screenshot_doom_20160122_131737](https://cloud.githubusercontent.com/assets/11136108/12510683/01856924-c10c-11e5-82d5-97ab286488b0.png)


是的 (我把Java當C#了 :sweat: )


What is this needed for?


This patch is not right. The idea is that "systemctl start dev-foo.device" is an effective way to wait for some device to appear, as a synchronization point.


哈哈，没事


@chaoos2 Why do you revert Steffen's translation?


Thanks @DeepDiver1975
I was coincidently doing the same fix but when trying to pull it notified me about yours :smile:

I just have seen thast the semicolon at the end is missing.


this patch looks fine to me.


I would include the `if(!sys.FileSystem.exists(c))` because the candidate could have a valid url (Windows) and it doesn't neccesary. If it isn't valid (Mac), it will resolve the path with a new path. If it doesn't find a valid path (your case), then it continues with other candidate (try/catch).

But, with the current version, yes it works.

Thanks ;)


> or ms vs fs primitive: I'm less sure there; would you be fine adding an item in the trello card to figure it out? It'd take some time to rebuild things and try this while I'm working (or at least, trying to work ;-)) on the nova side of things.

Based on some tests, it seems the colocation constraint is not enough there for pacemaker, so it is needed.


the operations has tables and also one does not name getters and setter get and set, its about expressiveness of the code


Using @thebigsmileXD's shortcut? XD


Can you override the `gain` parameter in the subclass?  I honestly don't know the answer to this one...


maybe wrong __gc in lua-openssl cause it, need more time to check and fix


these are not things that are guaranteed. It may not even exists/matter. this is purely a data structure for the crowdplatfom to use, but not me as a user.


TotalSeconds


BTW Travis build failed...


Descrement id


you named the big difference


It's still wrong, it should be "steamCountry=" instead of "steamCounty="


Why? :worried:


Ok, updated as suggested


还有那个PersonIdDialog类是干嘛的？怎么存档那里有波浪线？@luiges90


Year. It makes sense.


As soon as you see 0x260 it basically gaurantees you are dealing with a rct_ride structure. Access of rct_ride fields can be a bit odd I use the technique of looking at the last 3 characters of the address being accessed and remove 0x8f8 to get the field address. 0x1362A56 = 0xA56 = 0xA56 - 0x8F8 = 0x15E = ride->maze_tiles.


Я снова что-то намудрил)) Короче перезалил все файлы по новой, вместе с новой папкой lab4.


Ingen variable til result


When you call sol_coap_unobserve_server() the observation_request_reply() will be called. 100% certainty.


If nToRead > 1 this might completely miss the last block if the data stream isn't a multiple of nToRead.

Shouldn't it be readjusted to read the last partial block?


Please, add a tag to this revision.


A commit message should accurately describe what the commit changes.


It is necessary. I believe there is no other `StyleSet` access that is evaluated as early as this.


@Morwenn can you please add your favourite mingw distribution?


m_frames = 1881
m_sampleRate= 48000

Integer-Division (!) -> 0 always.


In the header, the second argument of this function is called `rotation_matrix` instead of `matrix`, which is a better name I think. In any case, they should be synced. This is not specifically for this commit but for some commit in the past on this line (I am assuming you refactored that, but maybe it's not your fault).


0x136294A not 952.


Great! Thanks again.


Damn typos.


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


Replace the bin folder


GCC from https://github.com/arter97/arm-linux-androideabi-5.2/tree/master


Ok, cf3a4e7c09ac1b97a8bcb386d7ca6d862b723b3d fixed it.


From my understanding of the requirements, the predicate function has to respect taints and tolerations with the NoSchedule* type effects. So , I have only considered taints with the NoSchedule, NoScheduleNoAdmit and NoScheduleNoAdmitNoExecute effects. If the effect is PreferNoSchedule , then we simply skip the taint and continue with the next taint. Thoughts ?


If the toleration operation is "Equal" , then its a hard restriction and the triple < Key , Value , Effect > should match.


Moved to #235 for now I suggest using x64 if you can.


I find the use of `size` awkward. How about using collectors to join the privilege names? http://docs.oracle.com/javase/8/docs/api/java/util/stream/Collectors.html


You ca use Matchers.instanceOf (static import though) which is less verbose. I think "is" could also work.


If the toleration operator is "Exists" , then the pod can tolerate all nodes with a particular key. I am not sure whether we should match the effect here as well. Thoughts?


friend actually private


Looks great! Love the color scheme as well!


This method should be renamed.


`PersonIdDialog`是在`Event`裏的子類...

把GameScenario裏，`PersonIdDialog`改成`Event.PersonIdDialog`，在Event.cs裏，`class PersonIdDialog`前加上`public`


I don't think the changes are copyrightable


develop (evanweaver) - i386-Windows-7-VisualStudio-12: OK (2355 of 2390 tests passed, 0 test warnings)

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


This will most of the time be zero, cause of the integer division, right?


@petrvacha , done! This versions is v1.0.1.


Is the client still being used somewhere? If not delete it.


When we reach here , we have tried to match this taint with every possible toleration of the pod and still we didn't find a match. This indicates that we should filter this node out.


nope


using PHPStorm


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


Made it work with openshift, now you can have SEF URLs on openshift


When we reach here , we have processed all of the node's taints and we found a match for every taint . So this pod tolerates all of the taints of this node. So this is an ideal candidate for scheduling the pod. Return true.


that's not really a simplification. it's just a code refactoring with no real functional change...


Ouh! you changed it ...  to your master's version ... grrrrmmmmmlll


CLEAN THIS CRAP


No I feel confused ... on what I commented


Not really, I figured all buttons were mapped that way, but it might not be necessary ?


Why do you suggest the change to sprintf? We don't want to write to a string here.


We can implement new version of client protocol and leave current unchanged. But this is a lot of work.


@luiges90  在PersonIdDialog类加上 public string scenBiography;就好了


https://github.com/FernetMenta/xbmc/commit/8d5e26d56b716095e937402c3711df72b314ac79#diff-b53cfd76db45d86d6c5c1f265451d22bR945 grml :-( no wonder it's not working on my AudioTrack testing - pain of early adaptors.


same version as my master. I changed this yesterday


Nothing to comment in this commit.
BTW, can you reformat the involved XML files (as always: the indentation) in a separate commit?


https://github.com/FernetMenta/xbmc/commit/8d5e26d56b716095e937402c3711df72b314ac79#diff-b53cfd76db45d86d6c5c1f265451d22bR945 <- that was the first version! which had that issue.


@domenkozar yeah, files were moved over to sourceforge, I'll update the package.


look at me comment above!


Date for last version is wrong. It should be 2016.


@MJGaughran @willrogers Good point


@Fasse look at me comment above!


On 01/22/2016 01:28 PM, Lennart Poettering wrote:
> This patch is not right. The idea is that "systemctl start
> dev-foo.device" is an effective way to wait for some device to appear,
> as a synchronization point.
>

Ah ok, I didn't know that was an intended behaviour.


you can just call `= new(request_name)` here as you're calling it within the class


Das führt bei mir zu einer doppelten Codierung: Wenn ich z.B. `<test>` eingebe, steht nachher im Listing `<test>`. Es produziert auch Inkonsistenzen in der Datenbank, weil die Kurzbeschreibung bislang _uncodiert_ gespeichert wurde (anders als der Hint!). Also das muss wieder zurück auf den ursprünglichen Code:

$short_desc = $_POST['short_desc'];

Und dann muss wohl in Zeile 208 das strip_tags() weg, damit solcher Pseudo-HTML-Code beim erneuten Bearbeiten der Beschreibung nicht verschwindet.


the only difference between these two functions was the behavior of the base class. However, since this error was never getting called it didn't seem worth having two functions. If we really wanted to preserve the behavior of failing hard, we could add an additional param to do that, but it didn't seem worth it to me.

@biyeun in case you had a better reason for both of these existing.


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


Ça marche mieux quand on utilise les **bonnes** variables ! Quel con.


Add 'test2'


got it, thanks!


@MartijnKaijser it can easily be added to other addons, it's mostly just a matter of changing the addon name. Travis supports building on OS X with clang I think, I'll look into adding that as a build target. Windows is completely out of the question though, and building with different version of gcc/clang is generally a pain in the ass (it's not as easy as with other languages where you basically just specify which version you want to test against).


Static imports (and same comment as instanceOf).


Ovo si mogao rijesiti puno elegantnije: npr. korisnik odabere vise razlicitih kombinacija (A,C, B, A, A, C, C, D) i ovo se preda u urlu.

Zatim, jednostavno prolazis u petlji po svim filterima i pozivas ih jedan za drugim onako kako dolaze. Npr. image = original image; for (x in y) { f = new filtar(x); image = f.filter(image);}

Ovo je puno elegantnije i bolje rjesenje.


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


请问为什么你工程里找不到图片，但TabBar却能显示图片啊？


TEST_PAYLOAD can replace "test".


See:

https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=206048


That's why about phone kernel version is unavailable, it doesn't like spaces


how about other space characters such as `\t` ?


Merged.


Sorry. Will do.


As far as I can see this fixes the issue only instrument names, but not brackets?


See next commit.


shouldn't this rather be
`return Py_BuildValue("(ddd)", p.x, p.y, p.z);`
?


Review by @nicolasstucki for push to scala-js/master. For reference, the commit for v0.6.5 was https://github.com/scala-js/scala-js/commit/8a6148d2bbe0aa6b1dcb45a373d49d0fd568ad11


@CovertLizard is it still possible to use this as a bundled API? I'm currently looking for a GUI API I can shade into some of my projects such as BanManager and this API seemed like it covered most, if not all of my requirements


ok


Oops,  guess I should check my emails in reverse order ;-)


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


I will add some defines here as well and write a better doc.


done


This seems like it must have been tested elsewhere, is that why you removed it?


I guess this commit breaks the build, isn't it ?


https://github.com/nikhil93uf/kernel_motorola_msm8992/commit/63ac106aceae4c0184372711cd7b726d77f4f701
혹시시간이되신다면 여기에탑재된 아드레노아이더를손봐주실수있으신가요?
state nofilter를연동시키다보니 아직제대로작동을안하는거같스빈다.


Mandatory properties belongs to the Method


@bp-kelley : I believe this is covered by the check at line 455.

This is part of the implementation details, so I'm not sure a ton of testing is required. It might still be worth including a `RANGE_CHECK` to catch bad calls.


@chelmuth: one of the fixups was too defensive that's why Armv7 >= Cortex A9 was now too slow for certain tests. The fixup fixup fixes that ;-)


Should we remove this class too?


you probably want .filter() not .map()


Kind of, but without countless of jar dependencies


`while (left < right)` is sufficient


and then a `collect(Collectors.toImmutableSet())`


LGTM


`toAdd.addAll(ensureContainsMandatoryFields(input))`


不，是我寫錯了
存檔的 `i.scenBiography` 應該是 `i.dialog` :sweat: @kpxp


Yes, left the two blocks for now, not sure if I need to put more code here.


No, it's only to be sure that output is correct when input is.
The case which fails with both *notFound* & *list* was in previous commit.
After some checks, it's a bad behaviour of our integration tests which can't occur in real life.


I probably miss something but I don't see the point of this change


Assignment


Nightly: https://scala-webapps.epfl.ch/jenkins/view/Scala.js/job/scalajs-nightly/656/

I will spawn Weekly later, not to overwhelm Jenkins.


You can't add spaces around `=`, otherwise the shell will interpret this line as a command and `download_dir` as the name of the executable.


Great class 5/7 would use again.


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


There are some missing entries in the `.gitignore` file. After I've built the project, I see `*.js`, `*.js.map`, `bin/*`, `obj/*` files. Adding these entries to `.gitignore` will prevent them displaying as unstaged files in SourceTree and when you run `git status` -> much less noise and chance to make a mistake


The selectHeadersProperties method lower cased the property, so the removal will never find them...
It's part of the point of question i asked in the main PR comments thread.


I don't understand, you write  :
> What do you think of introducing a new class HeaderProterty ?
and you don't do it ...


you should list them here


I'm seeing spaces instead of tabs. Looks like our Visual Studio formatting settings have not been imported. Believe there is a link in the dev environment instructions somewhere. Just import those settings and hit Ctrl-K-D -> should reformat the file correctly.


Hopefully it didnt conflict


Yes, that's a question...


Good job


static


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


no tests ? are you sure it's not just a Guava module that is not imported ?


Optimization: cache the max left/right height. Can safely skip heights below the max:
```cpp
if (height[left] > height[right]) { while (left < right && height[right] <= maxRight) right--; }
else                              { while (left < right && height[left] <= maxLeft) left++; }
```


Make the error code configurable. When reading from the filesystem fails, which is handled here and on L21, `HTTP/1.1 500 Internal Server Error` is the most appropriate error.


`@Each` doesn't quite convey the recursive behaviour as the word is already used in a lot of places as a single level loop construct (`array.each { ... }`, etc).

How do you like `@Descendants`? It reads pretty well to me in the rule definition:

```groovy
    @Mutate @Descendants
    def mutateMyComponents(MyComponentSpec component) { ... }
```


seems unrelated


foreach (PersonIdDialog i in this.scenBiography)
 +            {
 +                result += i.id + " " + i.scenBiography + " ";
 +            }

这一句到底怎么改啊？:sob:


any idea how we can make this less ambiguous? maybe if it starts with a `"`? it's a bit tricky


really ? looks like another test ...


another test


looks like you misspelled "canola"


an asteroid research outpost with engines?


@mostley (:  Now you have me spoiled.  After that raster progress bar hits 100%, and have to sit there waiting for the gcode parser to do its thing (: - i think there needs to be another progress bar (;  (This one could be for importing large gcode files, or for from the svg/raster/dxf/openjscad - so maybe kick it off from the openGCodeFromText in gcode-viewer/ui.js so its universal

I tried but also needs to be asyncronous.  I tried figuring our how you made raster asyncronous but alas my self taught js skills dont understand this and that :(   (Yes i know its good and best practise but i find it hard to understand/read)


again


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 12403](https://teamcity.mesosphere.io/viewLog.html?buildId=110518&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:12


Everything. Totally broken as soon as you used 2 or more transports. Wouldn't pick up, wouldn't drop, dropped at part-way there, more I can't remember...


Reason for this change (@tmack from Slack channel):
```
I noticed some library deprecation warnings when installing portal
Openstack libraries used by the synchronizer
And some python stack traces from  those libraries were showing up in the observer log
Indicating some more of runtime error
```


Hm.. I will look into it and let you know soon.


public ?


I'm not suggesting to change to sprintf. I simply forgot to uncomment "printf" before pushing the changes (I commented them on my computer because otherwise I get an error under Matlab).


messages


... triggering only a `clear` event would be more accurate


A number of reports of lock not locking when it should, locking when it shouldn't. Oh, and it let you load infinite units apparently.


Even though the compiled JavaScript does not enforce it, In TypeScript you can define members as private. I would consider making most of these members private


you keep passing the same arguments over and over, you could define a local UnaryFunction with most arguments already bound


toImmutableList


you could split the line, there's a lot of things here


Estaba mal las iniciales de jmc !


Getters and setters are about variables in a class, for value objects like `Money`, `get` and `set` would be appropriate.

These are operations on a specific resource, not operations on specific tables.


moment didn't use so I deleted it


why do you need a list here ?


The goal of this part is to check the side effects of the setMessages.
It seems more logic to use the associated endpoint than adding a method to serverProbe.
Also, as this is an integration test, we should use the top level mechanism in order to do some checks.


Use goto only for error handling.


seg faults for me with ret=null. Should that be buf->pkt?


if think that firstly we should check which hook was executed, after it write specific logic.


Duplicate empty line.


Alright. I'll go with the `InputMerger`.


it's implemented ?


Hi guys,

what type do I need to feed to Dailyzer  for Dest in this next case: <<CIDKey:2/binary, Dest/binary>>

Thanks in advance.


this one too ?


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


Yeah that's what was there, you're right though that's better.


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


CIDKey:2/binary, Dest/binary


In that case we shouldn't expose it at all. But exposing it makes `PUT /workers` clean without query string parameters.


We could get rid of this content is span and it should still work fine


No need to reserve memory for arrays in JavaScript, rather use the literal syntax:
`this.dataSet = [];`


oops, I will correct it


Tiny comment, otherwise it looks good and works in IE :+1:


Why this name ?


`result += i.id + " " + i.dialog+ " ";` 啊- -


Use goto only for error handling.


Gives the following trace-back with the Error: "ValueError: too many values to unpack"

Traceback (most recent call last):
  File "C:\Users\hitarthdoctor\Documents\GitHub\anarang_HTMLTestRunner\HTMLTestRunner\HTMLTestRunner.py", line 1017, in <module>
    main(module=None)
  File "C:\Python27\lib\unittest\main.py", line 95, in __init__
    self.runTests()
  File "C:\Users\hitarthdoctor\Documents\GitHub\anarang_HTMLTestRunner\HTMLTestRunner\HTMLTestRunner.py", line 1008, in runTests
    unittest.TestProgram.runTests(self)
  File "C:\Python27\lib\unittest\main.py", line 232, in runTests
    self.result = testRunner.run(self.test)
  File "C:\Users\hitarthdoctor\Documents\GitHub\anarang_HTMLTestRunner\HTMLTestRunner\HTMLTestRunner.py", line 789, in run
    self.generateReport(test, result)
  File "C:\Users\hitarthdoctor\Documents\GitHub\anarang_HTMLTestRunner\HTMLTestRunner\HTMLTestRunner.py", line 852, in generateReport
    heading = self._generate_heading(report_attrs)
  File "C:\Users\hitarthdoctor\Documents\GitHub\anarang_HTMLTestRunner\HTMLTestRunner\HTMLTestRunner.py", line 872, in _generate_heading
    for name, value in report_attrs:
ValueError: too many values to unpack


Thanks.


It's in the JMAP specification:
```
notDestroyed: String[SetError] A map of Message id to a SetError object for each Message that failed to be destroyed. The possible errors are defined above.
```


ok but you not only skip the selectHeadersProperties call, you break the design (calling a MessageProperty method instead of writing code outside of it)


I did a few modifications before merging. Instead of an external library I wrote an own resizer. Resizing an image is only three lines of code but we need to keep the aspect ratio, so we need to do a resize and crop in combination depending on the size of the source image.


just write it so we can see what are the consequences


Minor nitpick, but getting the length directly from `columns` member feels less prone to mistakes. Now you'll need to remember to refresh every time the `columns` member changes.


Can we please stop silently correcting things for others features/stories? This change now results in conflicts for all my local modifications, and without a warning.


PR for phobos: https://github.com/D-Programming-Language/phobos/pull/3939


That's also why i asked the question...


Header is missing, could you copy-paste one from another class, otherwise, the test suite won't run.


Rather:
`this.dataSet[i].push([]);`


I'm ok with this, but it's still another test : first is about return value, second is about side effect


ack


I do not have access to `isTypeOnlyCoercion` method here (it is in `presto-main`).

I added explicit check on varchar lengths (yuck).

```java
    // this is hacky implementation
    // proper coercion must be supported in type system
    public static boolean typesMatchForInsert(Iterable<Type> columnTypes, Iterable<Type> valuesTypes)
    {
        if (Iterables.size(columnTypes) != Iterables.size(valuesTypes)) {
            return false;
        }
        Iterator<Type> tableTypesIterator = columnTypes.iterator();
        Iterator<Type> queryTypesIterator = valuesTypes.iterator();
        while (tableTypesIterator.hasNext()) {
            Type tableType = tableTypesIterator.next();
            Type queryType = queryTypesIterator.next();

            if (isStructuralType(tableType)) {
                if (!tableType.getTypeSignature().getBase().equals(queryType.getTypeSignature().getBase())) {
                    return false;
                }
                if (!typesMatchForInsert(tableType.getTypeParameters(), queryType.getTypeParameters())) {
                    return false;
                }
            }
            else if (!Objects.equals(tableType, queryType) && !varcharTypesMatchForInsert(tableType, queryType)) {
                return false;
            }
        }
        return true;
    }

    private static boolean varcharTypesMatchForInsert(Type tableType, Type queryType)
    {
        return tableType instanceof VarcharType
                && queryType instanceof VarcharType
                && ((VarcharType) tableType).getLength() <= ((VarcharType) queryType).getLength();
    }
```


You can initialize members where you declare them:
`indexStart: number = 0;`


Poor naming, but we have to match jmap.io spec, keep it that way


thx. Cherry-picked this commit into Issue-31


AFAIK `render()` never throws an exception, therefore line 482 seems unnecessary.


Apparently Dimo's comment was on Manh's (deleted) comment. Sorry, I misunderstood.


Because a value is assigned where the var is declared, there is no need to specify the type here, it will be inferred by the compiler


> @jmendeth: I wonder if there's any commit with more comments than this one.

Oh yes. [Yes, there is.](https://github.com/MrMEEE/bumblebee-Old-and-abbandoned/commit/a047be85247755cdbe0acce6f1dafc8beb84f2ac)


Rather use `let` for block scoped `var`s or `const` for constant `var`s


We typically don't use braces for single line if statements


yes


Not sure to understand this change. The spec: "The id property is always returned, regardless of whether it is in the list of requested properties. The possible values for properties can be found above in the description of the Message object. In addition to this, the client may request the following special values:"
So why do you need to ensure threadId?


fix #309


wtf is this? why are you refering to exit here?... i'll fix that commit myself


Won't merge 28dc685cf71e0aba1175adf8e5894066919f4832 into Issue-31. Will instead merge these changes from master.


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


lol


Good idea to add return types for this function. Same for `getColumns` and `getDataSet`


Won't merge 23ef7dd7e22b174e6f9ede71e86857e006fcfc8a into Issue-31. Will instead merge these changes from master.


Cool.


Thank you for your sharp eyes!!

How annoying. Older versions require `storage_engine`. It looks like the transition was with the 5.5 version.

(Hopefully) fixed in 3a5721e06ec44304ba531001d3a7b3b495a0f2f7

Thanks again!


haha that is _very_ similar


json_encode can throws exception.


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


In general, using synchronous AJAX requests is a big no-no, but because the ones you use run before you app starts up, it is ok. If this was a really long-running AJAX request, you would, however, have your user looking at a blank screen for some time :)


does this have to be marked as `staticmethod`? or i think we have to add the `self` param


I guess the same change (using EVALUATORS.containsKey) can be done in TransformTranslator as well.


@portey thank you!


(missed this)


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


why not round (or ceil or floor) the value unconditionally?  The resulting code would work for either engine AIUI.
-const size = kStringMaxLength / 200;
+const size = Math.round(kStringMaxLength / 200); // or floor or ceil or what have you


Thank you :+1:


Not that I know of...
```python
>>> class MyClass(object):
...   @staticmethod
...   def foo():
...     print 'foo'
...
>>> MyClass.foo()
foo
>>> MyClass().foo()
foo
```


I smell some duplicate code. You can define a local var as a function that you can re-use inside the callbacks:

    const refresh = function() {
        data.updateData();
        data.clearTable();
        data.drawTable();
    };


I wonder why the two string modifiers at the end have no spaces between them. I see that's how the other exceptions messages are formatted as well ...


Done.


It appears hs_err_pid4700.log is added unintentionally.


je teste en rectifiant qq erreurs, mais je n'ai pas tout compris a l'interface !!!


Still: if you get an error if you leave the `printf` in , we should take it out or change it to something working at least under all our operating systems and Matlab and Octave. Maybe `disp` is the right command?


@joepie91 Oh.

Not sure if feeling jealous or relieved right now.


@kate there is a method to get the property of a build self.build.getProperty("artifactServerPath", None)


Eso me pasa por teclear despues de tomar unas copas de sauvignon blanc, jajaja


the body element already exists on the main page. why the change in index.html?


It doesn't relate to modules exercise, you have
1. Remove thus changes from that branch
1.  Create a separate branch with corresponding name
1.  Create separate PULL request  with "fixed #<NUMBER-OF-ISSUE>"


打印("你好,Swift!")


no, its not on resources, thats rest stuff. i have tables and sql operations, thats the point of the whole operations classes, to have the queries on one place so it's easier to optimise and you don't have to understand the schema to get the rest-package code. Also...no it's not appropriate.
this is the "get" from the upcoming java.money:
https://github.com/JavaMoney/jsr354-api/blob/master/src/main/java/javax/money/NumberSupplier.java
its called `getNumber()`, there are also `getConcurrency`, but the interface defining money has no simple get because its dubious, limits composability etc


I would edit the first line of the commit message to "Add parsing support for GRANT statements"


I've reworked this part in current version of this commit: 8c1c02e


Methods are displayed in two different ways now. `hockeyapp.start` should probably be changed to this new style, too.


This method throws an error because it's not defined in the Grid class


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


Cool, much better ;-)


i have never recommended exposing it


burayi mal ahmet yapmadan benim makinada calismiyor :dancer:


No. Not in the root directory of the system, but in the root(s) of the classpath.

Your fix solves the bug as well, through baking the version into the JAR. That avoids having to point to $scriptDir itself in the classpath in multeval.sh. I think that one should be a no-brainer merge request.


I guess that's for getting rid of wildcards. Should we do the same in https://github.com/cloudera/spark-dataflow/blob/master/src/main/java/com/cloudera/dataflow/spark/EvaluationContext.java#L227 ?


Even with fix, I think there is still a problem with pausing. Both before and after this commit when paused my Yamaha receiver flashes the DTS light. I also get that for a second when seeking. I suspect this indicates invalid data being sent and a more fussy receiver may produce noise or lose signal completely.

Actually - I just hit that - when paused Yamaha switched to PCM and produced loud noise. Doesn't do it every time.

For comparison with omxplayer when paused the DTS light on the receiver doesn't flicker. omxplayer outputs pause burst frames when paused.


Again,

Please, remove and provide thus changes in separate branch ...


I can see in the schema of [DocumentContentUrl][DocumentContentUrl] that the `contentSizeKB` field is nullable. This should be handled somehow. Maybe this field shouldn't be nullable?

[DocumentContentUrl]: https://github.com/openaire/iis/blob/d3d4eee0537cee0ab9c906026e7ca30a2754aeb0/iis-schemas/src/main/resources/eu/dnetlib/iis/importer/auxiliary/DocumentContentUrl.avdl


It's on resources, otherwise it wouldn't accept the proto structures but database records instead, right? ;-)


限制了localhost


It's better to use a relative path for the URL. In that way, if your app is served behind a router of some sort, the URL will still resolve.


done


This should keep ```arc_c``` in range.


We have to submit a PR to jmap.io as the specification is incoherent:
- threadId & mailboxIds are mandatory in Message object; so the client is waiting for such properties
- the example is wrong (missing the id property)


Keep ```arc_c``` >= 2 * maxblocksize.


Testing HTML-Runners


Did you forget to commit 'm4/ax_cxx_compile_stdcxx.m4' file?


If you hover your IntelliJ cursor over this return keyword it will tell you that you can replace this with an expression lambda.


Here a synchronous AJAX request is the wrong choice. The User interface becomes unresponsive during such a request. To the user it would look like the browser hanged. Tweak the delay to 5 second [here](https://github.com/wastrauss/onboard-javascript/blob/master/server/main.go#L18) and see what it looks like


Use phthreads 3.0.7 for now or you will get some problems with the \Collectable class present in the AsyncTask classs... Krakjoe said that Collectable interface is now extended by Threaded, but extending Threaded on the AsyncTask doesn't seems to work... there's need more research for using the 3.1.5 yet... Something that @shoghicp must be done...


It looks good to me


Can you check before or after implementing this solution what percentage of contents is discarded because it is larger than the limit (say, by analyzing logs)?

BTW, ideally we should have some kind of monitoring system that would gather information like this after every run of IIS and notify a human if something seems to be suspicious.


Wat heb je gedaan dan?


Official should be placed before "Contentful" instead of after Jekyll, as it implies right now it's our official plugin.


Done


TypeScript has support for [template strings](https://developer.mozilla.org/en/docs/Web/JavaScript/Reference/template_strings)


No need to specify type, it is inferred


I mean ceylon/ceylon-ide-eclipse#1669.


Correct, thanks. This is fixed in #327 (as well as the formatting)


Type would be inferred if function has return type


yeah i missed the `@staticmethod` decorator :flushed: you're totally right


OK


maybe there's something wrong with the pause burst: https://github.com/FernetMenta/xbmc/blob/565bd3503619a2c79178c58a0c446ec3469e102f/xbmc/cores/AudioEngine/Engines/ActiveAE/ActiveAESink.cpp#L932


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


done


We usually just build up the DOM using string concatenation, but this isn't wrong


Is this documentation still applicable?


Can you expand this out?  It's kinda hard to follow.


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


To check if `myVar` is `null` or `undefined` you can:
`if (myVar) { ... `


I created issue for that.
https://github.com/facebook/presto/issues/4405


Yes, that's correct.

Thanks
--
John J. Rushford
IPCDN Engineering
1400 Wewatta Street, Denver Colorado 80202
John_Rushford@cable.comcast.com

From: James Peach <notifications@github.com<mailto:notifications@github.com>>
Reply-To: jrushf1239k/trafficserver <reply@reply.github.com<mailto:reply@reply.github.com>>
Date: Thursday, January 21, 2016 at 8:04 PM
To: jrushf1239k/trafficserver <trafficserver@noreply.github.com<mailto:trafficserver@noreply.github.com>>
Cc: John Rushford <john_rushford@cable.comcast.com<mailto:john_rushford@cable.comcast.com>>
Subject: Re: [trafficserver] add multi-site origin feature. (89b1314)

The default of true is the current behavior, right?

-
Reply to this email directly or view it on GitHub<https://github.com/jrushf1239k/trafficserver/commit/89b1314489edb4b6cf04c448bac68ceb1fc6db89#commitcomment-15603388>.


In the spec it is coherent that some mandotary fields are not sent to the client if not asked.
There is  typo in the example, messageId instead of id.


look at https://github.com/mapbox/mapnik-vector-tile/blob/v2_spec/test/test_utils.cpp#L28-L57


Yes, you're absolutely right.


This file should not be commited to a public repository as noted in the header.


This is a good idea, I didn't like using JQuery.


Kudos! I couldn't break the app, all edge cases seemed to be handled correctly. "Find" navigation is a nice touch.


체감온도 보정한게 여기 들어가버렸네요 ㅋ


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


Liked that you pre-fetched a larger amount of data than what was displayed on the screen, is good for performance


Thanks :)

Those days I don't really have time to work on MXCuBE but still I can do little things...


@joedemo42 https://github.com/shaarli/Shaarli/pull/448


also the return value is not a list you will get the string with the URL like 	http://server.com/testbuilds/Builder_1818_22_01_2016_12_37_39_+0000


LOL


Is this where you enforce that you can only grant SELECT if you have GRANT and SELECT on a table?


but I still haven't grow up either


@michaelcahill, now that `ret` is explicit, can we quit assigning it as part of the op?

It seems simpler/cleaner to me.

```
diff --git a/src/conn/conn_dhandle.c b/src/conn/conn_dhandle.c
index dedafc2..3869f66 100644
--- a/src/conn/conn_dhandle.c
+++ b/src/conn/conn_dhandle.c
@@ -644,8 +644,8 @@ __wt_conn_dhandle_discard_single(
                F_SET(S2C(session)->cache, WT_CACHE_CLEAR_WALKS);

        /* Try to remove the handle, protected by the data handle lock. */
-       WT_WITH_HANDLE_LIST_LOCK(session, tret,
-           tret = __conn_dhandle_remove(session, final));
+       WT_WITH_HANDLE_LIST_LOCK(
+           session, tret, __conn_dhandle_remove(session, final));
        WT_TRET(tret);

        /*
diff --git a/src/include/schema.h b/src/include/schema.h
index 88a3a39..6a53ee8 100644
--- a/src/include/schema.h
+++ b/src/include/schema.h
@@ -85,18 +85,18 @@ struct __wt_table {
 #define        WT_WITH_LOCK(session, ret, lock, flag, op) do {                 \
        ret = 0;                                                        \
        if (F_ISSET(session, (flag))) {                                 \
-               op;                                                     \
+               ret = op;                                               \
        } else if (F_ISSET(session, WT_SESSION_LOCK_NO_WAIT)) {         \
                if ((ret = __wt_spin_trylock(session, (lock))) == 0) {  \
                        F_SET(session, (flag));                         \
-                       op;                                             \
+                       ret = op;                                       \
                        F_CLR(session, (flag));                         \
                        __wt_spin_unlock(session, (lock));              \
                }                                                       \
        } else {                                                        \
                __wt_spin_lock(session, (lock));                        \
                F_SET(session, (flag));                                 \
-               op;                                                     \
+               ret = op;                                               \
                F_CLR(session, (flag));                                 \
                __wt_spin_unlock(session, (lock));                      \
        }                                                               \
```


Resizing the browser window messes with the layout, play around with the window resize event and see how you can improve it


dude, awesome, I was going to suggest this.

"cleanup, cleanup" -Dora The Explorar.


According to https://github.com/JanTvrdik/nette-tester/blob/c6d81655199d3d9e3a48fe9761c787fa396504eb/src/Runner/ZendPhpDbgInterpreter.php#L49 it requires PHP 7, but here is version check missing. Is it ok?


No, you are right. I'll change that.


I'm not sure. The patch was introduced in issue #120, you'll have to ask @mviikki16.


This can lead to exception `Code coverage functionality requires Xdebug extension.`, although it requires `Xdebug or phpdbg`.


Done.


I would also edit the first line of the commit message to "Add support for GRANT in the Hive connector".


Hi,

Sorry for the delay, I am fairly busy recently.
You can use this project. And it's welcome to improve it.
The NAOqi 2.1 is fine, please refer to the link below for the instruction
of using the project, my work is extended based on it.
https://possiblywrong.wordpress.com/2012/11/04/kinect-skeleton-tracking-with-visual-python/

Hopefully my description answers your question. :)

Br,
Yang

On Mon, Jan 18, 2016 at 8:23 PM, rdixey <notifications@github.com> wrote:

> Hello Niceyang,
>
> I am very interested in your NaoKinect Python Program. With your
> permission I would like to install and use the programs. I have a NAO V5
> with NAOqi 2.1.4.13, and an Xbox Kinect hosted on Ubuntu 15.10 working with
> OpenNI v1.5.2.23.
>
> Will this configuration work with your code?
>
> Also can you share instructions for installing your code?
>
> rdixey@comcast.net
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/niceyang/NaoKinect/commit/37c6cdbeab82bda4f178f1010864d6addc9aa244#commitcomment-15513988>
> .
>


What is a unbundle btw?


done


Did you make any changes to this file? If yes, what changes?


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


или class="current" уже лишнее? это же можно в css сделать через :last-child?


Successfully created the [rsync-optim](http://rsync-optim-static-docs.pantheon.io/docs) environment. New commits to this branch will automatically be deployed.


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


This looks odd. Would doing

local = ...  # type: Any

also work?


Hi,
I'm running VHbbAnalysis/Heppy/test/vhbb_data.py on a 76X data file ([1]) and I get an error that I think it due to this commit (merged with PR #282).

The error is here: [logTestData.txt](https://github.com/veelken/cmssw/files/100858/logTestData.txt)

Are you able to run VHbbAnalysis/Heppy/test/vhbb_data.py ?

Thanks,
Silvio.

[1]: /store/data/Run2015D/MET/MINIAOD/16Dec2015-v1/50000/00EA1DB2-90AA-E511-AEEE-0025905C2CE6.root


In the linked code, doesn't PackAC3 dereference payload[5] which doesn't exist?


Pushed again with "=~" changed to ".include?" and extra spacing in line 39 deleted.
@Fryguy @chessbyte @roliveri please check and merge when possible.  Thanks.


Not reconstracting but rather reconstructing


Same question as above: Could this just be a module attribute of type Any?


Doxygen is complaining about a missing title or group name, maybe:
@defgroup Utils Soletta utility functions

And you be nice to have Utils added to the list in sol-mainloop.h so its easily reachable in the final docs.


Referring to the comment above, what name are we adding to the person?


Complete changelog:

* Allowed selection and displaying (on infopanel) of enemies
* Added `saved` boolean to GameLogic
* Started file writing
* Only one status effect per type per weapon is allowed
* Effects that kill enemies now register kills
* InfoPanel no longer scrolls horizontally
* Fixed health bar disappearing when info panel text overflows (is too wide)
* Changing displayed object now repaints entire InfoPanel
* Weapons with no projectiles now have "None" under projectiles on the InfoPanel
* Increased poison amplifier on the poison dart tower


not sure why this was necessary, but doesn't seem to have broken anything...


That's correct. That's what the method does. The mismatch is because of the difference between the type of privileges you can specify on CLI versus the type of privileges represented internally in Hive. The two privileges present in Hive but missing in Presto are "GRANT" and "OWNERSHIP". You can't specify these privileges on CLI, but internally, the code creates such privileges in Hive to keep track of grant on and ownership of objects.


kl


Should we expect allergies without patients?


No, I did not. I thought the build process updated the swagger documentation automatically. Actually because of this, I did not look into it.


Should these tests be cleaning up after themselves by deleting the tokens they generate? Maybe use a `beforeEach()` or `afterEach()` function?


Yes, it should.


Yes, the performance is basically the same. Well, in fact since some steps are reversed in RAW there is a performance gain (saw it in SSE2).


I think it is about keeping logic of `TypeRegistry#getCommonSuperTypeSignature` as generic as possible.
This test actually tests assertion in this methos. In running presto it should never happen, that data type without implicit literal arguments expanded, is passed as argument to `getCommonSuperTypeSignature`.

@pnowojski Anything to add/correct?


Yes.... will be updated tonight. You can download the file here:
http://git.savannah.gnu.org/gitweb/?p=autoconf-archive.git;a=blob_plain;f=m4/ax_cxx_compile_stdcxx.m4

2016-01-22 14:49 GMT+01:00 Rafał Czaja <notifications@github.com>:

> Did you forget to commit 'm4/ax_cxx_compile_stdcxx.m4' file?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/cruppstahl/upscaledb/commit/22c55bf71e44e63251ae5e5b3b3804f0d962e760#commitcomment-15613175>
> .
>


Hi Feng after rebuild and change many of kernel config options now imq work fine
but in 4.4 have problem with network driver and rcu reset adapter now i go try to fix this problem and test imq and kernel


Maybe "Sum two time values", sum two times sounds like (a + b) * 2 hehehe


Make sure you're indenting consistently.


okay, cool. i can do it, will take a look after `xpath-validate` is merged.


Use spaces, not tabs, for indentation. You can config your text editor to do this.


I guess it should be `disp`, and it's not strictly necessary if everything has been tested and runs smoothly, so it might be a left over from testing?


y u no predicate method?


thx

On 01/22/2016 07:10 AM, CircuitLord wrote:
>
> Good job
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ORCHARD-Minecraft-Mod-Class/RandomMine/commit/4c37408225505154f932cdf0bcf572e064316cdd#commitcomment-15612369>.
>

--
73,

Nick
KG5CKC
-. .. -.-. -.-


Why are you emposing the restriction of having only 3 nodes


Can we remove this change?


I can't see what you are saying, this change should not touch person.


Once you add this folder to doxyfile, our CI won't let this pass undocumented.


Yes, this is the place.


Yes! :+1:


I agree that it is not strictly necessary to have this output. But since the compilation takes overall definitely more than 1 minute on my machine, I thought it's better if the user gets some output during this time...


No, I can't see where in the change you are referring, allergies always goes with patient


What are these exception handlers for?


perhaps add a comment why this is needed? To avoid confusion in the future


Do you need anything more from me on this?


Am referring to newAllergy.getPatient() != null


What is the use of the comment on line 237


#1 Remove C file


ok


#1 Move to C++ and add license header


Yes that's right. Previous discussion:

https://github.com/Teradata/presto/commit/d0beba4c811a1ef7d2abe00cb68c721a2d26057a#diff-e09a0e5523986eef8d2ec9ff6aba8bb3


Good edit!


since from allergy we have to go to allergen then nonCodedAllergen I did not want to check each one separately.


cool


It is just in case there is a data corruption and the allergy passed in does not have a patient properly setup.


The implementation was really only an experiment. It had some several limitations, no framing, max 4kb message size etc.. I am planning on adding it back in, it is just I have had very little time recently. My intention is to build something solid ontop of zeromq maybe. So I'm not starting from scratch.


Can you add the same on Artemis ?


Cant we just check for not null?


not so fast, @ebrehault, you erased my uninstall tests :-)


Here you have to 2 documentation options:
- create a new group and add it under Utils group
- add this api to Utils group


Blocks and hashes should have spaces around them `{ subject.dock(bike) }`


`disp` is not compatible with Octave, `printf` seemingly not with all Matlab versions. The best option is probably `fprintf`, see https://en.wikibooks.org/wiki/MATLAB_Programming/Differences_between_Octave_and_MATLAB#fprintf_and_printf

I will take care of changing it.


No, not really. The privilege that you specify on CLI is represented by two classes: the enum class 'Privilege' in SPI and the 'PrivilegeNode' class in parser module. HivePrivilege internally represents the privileges in Hive.


MIssing doc of this define (I know, it sux)


What do we do when that happens?


:+1: nice test!


right. do you see anything else that is wrong?


I will remove it


Now _this_ shouldn't appear in a commit. Keep them clean: if they're not, put 'WIP' in front of the message.


And add docs for all this functions bellow.


The answer is not directly clear to me. I should have asked maybe in a more
direct way.
What of both cases do you prefer:

1) using iseven() in wfs_fir_prefilter()
2) leaving it as it is at the moment

> trettberg <notifications@github.com> hat am 22. Januar 2016 um 12:47
> geschrieben:
>
>
> You're right, that is probably better.
> Having the true filter order available was the motivation in the first
> place.
> I think that outweights the inconvenience of manual correction.
>
>
> On 01/22/2016 10:19 AM, Hagen Wierstorf wrote:
> >
> > Another possibility would be to add an |iseven()| test directly to
> > |wfs_fir_prefilter()| and raise an error if it is not even.
> > Or do you prefer not to raise an error and simply correct the wrong
> > filter length as it is done at the moment?
> > I'm not sure what is the better option. Normally we raise errors in
> > SFS if the input is not correct, on the other hand it might be
> > convenient to correct such a small error automatically.
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <https://github.com/sfstoolbox/sfs/commit/5eb9c4fd130e97f47bde4ec3f01c27e0363a1425#commitcomment-15607993>.
> >
>
>
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/sfstoolbox/sfs/commit/5eb9c4fd130e97f47bde4ec3f01c27e0363a1425#commitcomment-15610695


Image upload and saving in db part is Done by @scbidua .


I use notepad ++ and for easier replace i use CTRL + F then click on replace tab. Then after entering the strings you can use Replace in all opened documents. I open them all at once.


the logo doesn't show up


Never ever for any reason use pip to install anything on the system… Pip should only be used inside a venv.


Testing sourcemaps


Lanean ditugun aplikazio eta zerbitzuen egoeraz jakitun egon nahi genuen, baina lankide bakoitza aplikazio batzuetaz arduratzen denez egokiago ikusten genuen mezuak erabiltzaileka bidali ahal izatea. Horregatik aldaketa hau.


Some usefull comment)


use eclipse pdt and better it can be done simply


my Octave understands `disp`.


please create a Pull Request for this :)


Employer Signup Designing is done by @Mkhaledfhkiel


Successfully created the [rsync-optim](http://rsync-optim-static-docs.pantheon.io/docs) environment. New commits to this branch will automatically be deployed.


Because of the type hinting am not able to pass an *ArrayObject* as a container.

Passing an array is not possible for me, as i will have to pass it around by reference.
and laravel IOC won't pass it by reference for me.


auto?


need to replace `atomic install` here


Hi Silvio,

the ValueMap egmGsfElectronIDs:mvaEleID-Spring15-25ns-Trig-V1-wp90 is produced in the config file VHbbAnalysis/Heppy/test/combined_cmssw.py , which is the same for data and MC.

I tested the code on MC, with 'python vhbb_combined.py' .
When I execute 'python vhbb_combined_data.py' I get an another error [*].

Cheers,

Christian

--------

[*]

Begin processing the 1st record. Run 251162, Event 13712798, LumiSection 18 at 22-Jan-2016 15:22:58.368 CET
terminate called after throwing an instance of 'edm::Exception'
  what():  An exception of category 'InvalidReference' occurred.
Exception Message:
BadRefCore RefCore: Request to resolve a null or invalid reference to a product of type 'std::vector<reco::Vertex>' has been detected.
Please modify the calling code to test validity before dereferencing.


This might be `trackObserver` instead of `TrackObserver


don't need `--mode=run` here.


Is the `method` propety defined


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


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


Got it, done.


Oh I had done that because I needed it to reload the entire page. We can change it back. It was just for testing. Although, we do need another reload button on the modal. So that the page can be reloaded once the config changes. So that the top nav will update.


The last word `name` indeed has string `me`, if you go through the assert, it will evaluate to true, but your change is asking to check other wise. Hence this test case is invalid here, in this context.


@felliott @chrisseto
This commit may need extra scrutiny. I'm not sure how different interaction with my local file system is compared to osfstorage. There were two issues here.

1. The new async wasn't happy with what was being yielded.
2. The current use of StopIteration exception is not deprecated. See https://docs.python.org/3/whatsnew/3.5.html#pep-479-change-stopiteration-handling-inside-generators


I think now is configuration ok, i test a little and config export is look ok. That edit-form and delete-form link i make just like a see from some tutorial, but this is where i need your help. It confusing for me is that _entity_wizard is also have entity.entity_browser.edit_form.


Yes, if it fixes the issue, I'll add some note (though I'm not sure why this works)


accepts_nested_attributes_for needs allowing mode_attributes: to be permitted!!!


TeamCity pull requests :: netty [Build 1086](https://garage.netty.io/teamcity/viewLog.html?buildId=4983&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 6387, ignored: 23 Build time: 00:41:56


> It is theoretically possible to detect SSE compiler support, but it is quite a lot of work and the commonly used compiler like MSVC/gcc/clang do support them.

Let me elaborate on this: You can do either preprocessor define detection which in case of MSVC with SSSE3 won't work or you can try to compile a little test program consisting only of a simple main function which does dummy calls to every intrinsic used by your code. I'll add a preprocessor based check for AVX2 detection, but as stated this can't portably work for SSSE3.


it is now `fprintf` - it turned out that we already had the `fprintf` in the SMS-EMOA example but it has not been propagated to the `build/matlab/` folder


You don't need to add downloads to addons that only add Lua.


Tested by @apollo-h and Corrected the bug in Structure.


This could be made into a prototype function called `getVisibleChildren`.

Instead of creating a new array every time, maybe store the array as part of `Cesium3DTile`, or even as a scratch array which may be safe now but maybe not later.


@jmendeth Well, to be fair, that one destroyed entire systems.


Or maybe I should have answered more clearly, sorry:

I prefer to check the order in wfs_fir_prefilter and raise an error, if
it's not even.
Because a) good practice and b) hpreFIRorder should reflect the true value.

I pushed a respective change about an hour ago. Will update the pull
request description asap.


`trackObserver` instead of `TrackObserver` because is an object not a class


I see all memory attributes use... "m" in spark... like spark.executor.memory  but why it didnt fail before... weird... never seen this fail... does it omit "b" for some reason...


Uploaded file of the application.


Having a contact is a pre-requisite. I'd put this step in a `setup()` http://mochajs.org/#tdd


You have to also set the texture size in metadata. The polygon loading needs that information.

https://github.com/cocos2d/cocos2d-x/blob/v3/cocos/2d/CCSpriteFrameCache.cpp#L169
https://github.com/cocos2d/cocos2d-x/blob/v3/cocos/2d/CCSpriteFrameCache.cpp#L276


TeamCity Civic Assembly :: Build all [Build 23](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=418&buildTypeId=CivicAssembly_BuildAll&guest=1) is now running


Galdetu badaiteke, non darabilzue plugina?


To avoid duplicate operations here and in `isRefinable`, the code could be reformatted like:

```javascript
var visibleChildren = t.getVisibleChildren();
var tileIsRefinable = t.isRefinable(visibleChildren, cullingVolume);
```

Or something with a similar idea.


TeamCity Civic Assembly :: Build all [Build 23](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=418&buildTypeId=CivicAssembly_BuildAll&guest=1) outcome was **FAILURE**
Summary: Exit code 2; exit code 1; exit code 123 Build time: 00:00:34


TeamCity Civic Assembly :: Build backend + frontend [Build 21](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=420&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) is now running


Already defined in `setUp()`.


TeamCity Civic Assembly :: Build all [Build 24](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=421&buildTypeId=CivicAssembly_BuildAll&guest=1) is now running


Ohh im stuped) thx


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


Check.


Matt,
once you are listing multiple fact tables, this serves as a separator between them.
The current version does not support the correct showing, but in the future it will sort every index (I actually had this code in private beta).
I will remove this part of the code, but it will be back this year, once I will find time for the T-SQL part.

Best regards,
Niko


I fixed it all now


Foloseste mai bine clasa Log pentru a afisa loguri

http://developer.android.com/tools/debugging/debugging-log.html


@nmaier I updated mac os x related check carefully, but not 100% sure I made everything ok.
Could you check this still work on os x?


Usually we create the length variable outside of the loop, like:

```javascript
var length = this.children.length;
for (k = 0; k < length; ++k) {
```

This applies to other areas in the PR.


```javascript
if (!child.hasContent && !child.isRefinable(cullingVolume)) {
```

Should be fine here.


What do you use the fetched allergies for?


the field is called pos, not position, see http://dev.minetest.net/minetest.sound_play#Examples


Need to rewrite this function.


TeamCity LabKey :: Sprint :: Premium :: Git postgres 9.4 [Build 21](https://teamcity.labkey.org/viewLog.html?buildId=298696&buildTypeId=LabKey_Sprint_2_LabkeyPremiumTrunk_GitPostgres94) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 250 Build time: 00:38:12
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.DataFinderTest: <no details avaliable>

```


Can you please fix test failures this has introduced?

https://travis-ci.org/phpmyadmin/phpmyadmin/builds/104069510


Typo in commit message.


Since the if and else blocks do the same thing, you could combine them.


@TomBaxter the `await asyncio.sleep(0)` is a hard requirement here.
Without it this coroutine will never suspend and block the event loop until the file has been fully read.


Doesn't void allergy save the patient?


thats one of the parameters, i also have methods accepting Records or other used Datastructures. As the database guy i am only interested in building an easy to use, simple interface that tries to follow the needs of others while still maintaining good abstractions and a trivial implementation.


Is this a uuid that we are converting to int? :)


Initial commit


I think we want to keep the `$` at the end of this regexp - otherwise it allows arbitrary junk at the end of the URL, and the `[^/]*` portion (which is presumably intended to reject anything with a slash in it) has no effect.


That's done with fff392a5501cb6ef37017aae8125f4dbd3da3d1b


if I do I will have to check for 2 nulls. I thought that will be more messy. Actually this code needs to be rewritten as well, I think I should throw an exception to the caller.


I also split Newton and Picard iteration pre/post solve functions


Do we really need allergyId?


Commit to trigger Shipped?


Bonjour,

Je suis peut-être un peu pointilleux mais...

La solution que j'ai donné dans l'issue associée, d'utiliser une hash table et de vérifier que la clef existe via un isset($this->_memOptions[$elementValue]) est bien plus efficiente.
En effet, l'utilisation d'un tableau classique et de in_array() est moins efficace sachant que, sauf erreur de ma part, de toute façon les options ont une valeur unique.

De façon général, en PHP l'objet Array() ne gère de toute façon que des tables de hashage, du coup si le contenu du tableau ne contient pas de doublons comme dans ce cas, il vaut mieux utiliser la clef si l'on veut seulement vérifier la présence d'un élément dans un Array(), car c'est beaucoup plus rapide.

Après, c'est seulement une suggestion, libre à vous d'implémenter ou non :)


OK, less than a month later so I'm hopeful: Please keep this functionality - I completely rely on pvr support with my Hauppauge card!

Thanks.


@meichstedt Hope it is part of the changelog?
And yes we need to document that - will create a ticket.


Is there any connection between allergies and allergiesList?


Couldn't this be on the plugin initialization? or might be too early?


the oldallergyapi does not support uuid only allergyId which is an int


No, but will check


Should the variable name change to reflect that?


Hi James,

Thanks for taking the time to look it over.  I'll just respond to all your comments here.

  1.  Actually you can have simple retry and dead server retry enabled at the same time.  All the settings are overridable by delivery service (remap).  For some services we may only want to use simple retry while for others we might only use dead server retry and finally for some we might use neither or both.
  2.  The parsing of the response code string is because all the settings are overridable by delivery service (remap), this was a requirement given to me.  They wanted to be able to use header rewrite in remap.config with a configuration file.  A delivery service, remap rule, could then enable simple or dead server retry and override the response codes as deemed suitable for that particular delivery service.  I do make sure that it's only done if were checking the response from an origin server, 'parent_is_proxy == false'.
  3.  I'll move the response_code declaration.
  4.  I'll look at cleaning up around lines 3571 to 3594 to make it clearer.

Let me know what you think about the overridable settings/parsing per transaction.  The global settings in records.config will probably be used but, they want the flexibility of overriding those settings by remap.

Thanks
--
John J. Rushford
IPCDN Engineering
1400 Wewatta Street, Denver Colorado 80202
John_Rushford@cable.comcast.com

From: James Peach <notifications@github.com<mailto:notifications@github.com>>
Reply-To: jrushf1239k/trafficserver <reply@reply.github.com<mailto:reply@reply.github.com>>
Date: Thursday, January 21, 2016 at 8:20 PM
To: jrushf1239k/trafficserver <trafficserver@noreply.github.com<mailto:trafficserver@noreply.github.com>>
Cc: John Rushford <john_rushford@cable.comcast.com<mailto:john_rushford@cable.comcast.com>>
Subject: Re: [trafficserver] add multi-site origin feature. (89b1314)

Since you can't have simple retry and dead server retry enabled at the same time, we should simplify this down to 2 configurations; one for the retry policy and one for the response codes.

proxy.config.http.parent_origin.retry_enabled would accept values 0 (off), 1 (simple) and 2 (dead server).

Initially I didn't like the parent_origin nomenclature but it is consistent with our use of parent_proxy, so not I think it is just fine :)

-
Reply to this email directly or view it on GitHub<https://github.com/jrushf1239k/trafficserver/commit/89b1314489edb4b6cf04c448bac68ceb1fc6db89#commitcomment-15603558>.


Here, you return the function itself, instead of its result. This makes [assertion done there](https://github.com/yglazko/gaia/commit/4479b74afae0fcaed81d32d7cf05bc0468566b4f#diff-c7d78b9f6cc37597e9e16d8a06f85d9dR101) => always pass. That is to say: the test is prone to be a false-positive, which can be harmful.

IMO, we don't need to return anything. `this.waitForFavorited()` will wait until we make sure the icon is there. If the icon never comes, it'll time out with a nice and meaningful stacktrace.

As a consequence, the assertion `assert.notEqual(favorited, false);` is redundant with the wait, so we can get rid of it :)


each allergy is stored in patient.allergies os, we have to get patient.allergies, add the allergy and save patient.allergies


TeamCity Civic Assembly :: Build all [Build 24](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=421&buildTypeId=CivicAssembly_BuildAll&guest=1) outcome was **FAILURE**
Summary: Exit code 2 (new); exit code 1 Build time: 00:12:44


TeamCity Civic Assembly :: Build all [Build 25](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=422&buildTypeId=CivicAssembly_BuildAll&guest=1) is now running


I assume this is meant to overwrite packet->m_type?
Shouldn't it be buf[2] = 3; ?


disclaimer: I am not a lawyer.  This is not legal advice.  If you need legal advice, seek the counsel of a legal professional.  I disclaim all liability for actions taken in light of my comments, offered solely as opinion.

(As Mr. Bumble of *Oliver Twist* said, "the law is a ass": any time I discuss my understanding of the law in any depth, I feel compelled to disclaim my statements for my own protection.)

There are many ways in which (even weakly) copyleft licenses are incompatible with other licenses.  This can get extremely dangerous especially when dealing with a programming language like Ruby where, in the course of perfectly reasonable (okay, some people disagree it's reasonable, but I think most of them are using Java or C++) work, one might need to make alterations to existing, open classes.  The law has not, to my knowledge, been tested on such things, and the supposed "per-file" scope of the MPL may not conceptually excuse such modification of the original work, so that activities like monkeypatching or even subclassing (either pushing changes up the chain or pulling them down) may result in an effective violation if the changes themselves must, for some reason, be used under terms incompatible with the MPL's terms.

I have to ask . . . why did you choose MPL 2.0 as the new license?  What exactly was the licensing need that prompted this change?  Was it the enforced source distribution?


Nit: In Firefox OS, we use `tap()` instead of `click()`.


Not usually a good practice to use exceptions for flow of control. :)


Nit: Unused.


A compromise for archiving the history while keeping a clean repo would be to use `git bundle` You can bundle up the branch into a file you save on a backup/server and then freely cleanup. That way if you ever needed to reference the original feature branch history a year later you just go to the archive and find the bundle named for the feature and add it as a git remote.

```console
git checkout feature/my-feature
git bundle create /path/to/repo-backups/repo-name/my-feature-2016-01-22.bundle master..HEAD
git rebase -i master
git checkout master
git merge feature/my-feature
```

Another alternative would be to have two remotes.

```console
git clone --bare . /path/to/repo-backups/repo-name
git remote add working-backup /path/to/repo-backups/repo-name
git checkout feature/my-feature
git push working-backup feature/my-feature
git rebase -i master
git checkout master
git merge feature/my-feature
```


We should not modify the model in a DerivedValueService, if @Enablement doesn't work on ImportWorkspaceOp model we should implement this logic as a listener


Shouldn't Context.getPatientService().removeAllergy be enough?


you can use `git rev-parse HEAD` to get rid of this stuff. it will give you exactly what you need.


What is the advantage of declaring a temporary variable, here?


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


:warning: Confusing indent


yes I know :)
because I haven't any uninstall profiles for now
I will add back later


TeamCity pull requests :: netty [Build 1088](https://garage.netty.io/teamcity/viewLog.html?buildId=4985&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:00:54


Hehe :trollface:


see earlier comment, old allergyapi does not support uuid only allergyId which is an int


2016.1.21 Angular Validations/form validations/index.html

WOW! Lots of scripts!


Enablement shouldn't really matter as long as visibility for the wizard is working.


No need to setup a new daemon. It's already in the test suite waiting to be started.


@apotheon I suggest reading https://github.com/rubinius/rubinius/issues/3550.


add HSTS support


:tada: :+1:


Where is the DateFormatter?


Does it work now?


erm sorry


This is not necessary when starting daemon from `DockerRegistrySuite`. And if needed, it should be deferred after the daemon is started.


Bernhard, can you please comment out desktop detection stuff?
Autodetect works, and if not, we catch the error.


I was debugging a custom pcb with a chip that supports both spi mode 2 and 0 using beaglebone black. Mode 0 wasn't working properly, so I wanted to try mode 2. Beaglebone black seems to support using different spi modes, so this could also be useful to other people.


Konrad, why not work on gopro 3+ se?


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


This should work, however we really need to come up with a better solution for generalized URL parsing.  Here, we're trying to take parameters out of a REQUEST url which uses ';' as the separator, and create a URL for another system, which uses '&' as the separator.


TeamCity pull requests :: netty [Build 1089](https://garage.netty.io/teamcity/viewLog.html?buildId=4986&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:00:41


game.resources.trimps.realMax() <= game.resources.trimps.owned + 1

Assuming 1mil total trimps, you need 999k trimps before this will trigger. At this point the breed rate is well above 1 trimp per second. At 30 total trimps, you need 29, which might be below the breed rate. (don' know, don' remember)

I suggest replacing the +1 with +(fighting_force/Breed_timer*.2) This should parse out to 2 ticks of the average breed rate over the breed time. Which is below the breed rate at that time, which is why we go with 2 ticks, rather than 1. Should speed up auto fight just slightly(which adds up over hours).


If ```@Enablement``` doesn't work we could try an EnablementService instead.


是否应该改


Yes, but have bug (anchor point or incorrect vertex offset)


If something doesn't work, the [desktop.map] (https://github.com/manjaro/manjaro-tools/blob/master/data/desktop.map) defines

    launcher:binary


corrected and tested again


ok. no problem. I have used xfce as a model and thought it was there for a reason ;)


a revert of this and #235 seems to cover most the issues


right, the comment "Pc" is correct, but I messed it up


use the api:

```python
from plone import api
PLONE_VERSION = api.env.plone_version()
```


Sure, I'll create PR for Artemis on Monday.


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


I reworked autodetect, which is now file based.
Definitions can be changed in /usr/share/manjaro-tools/desktop.map, in case a binary gets renamed or we want to add a DE. ;)


What is the 32 and length code for? Isn't there only one length word?


sounds good


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


I like that you put references for others to learn :D. :+1:


Капчу надо показывать не исходя из настроек, а исходя из того, затребовала страница капчу, или нет.


@hbto costing's segmentation doesn't take journal entries, does it? Is there any need to be written?


ohh..yea..I did forget these.


allergies is  from the database, allergiesList is what we return. We can't return allergies because in one case we want to return an empty list.


Sure, though it reminds me that I somehow don't get Jenkins daemon mail notifications about test failures anymore, probably since we moved the mailing lists. Does that still work?


TeamCity pull requests :: netty [Build 1090](https://garage.netty.io/teamcity/viewLog.html?buildId=4987&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1801, ignored: 4 Build time: 00:02:15


@karimnaaji might be introducing a change with harfbuzz which will move us to gcc5.+ ... I don't know how much older this commit is for?


Sorry, I think I wasn't clear, I just meant to comment it out. Both ways work anyway, and it defaults to autodetect. ;)


TeamCity Civic Assembly :: Build all [Build 25](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=422&buildTypeId=CivicAssembly_BuildAll&guest=1) outcome was **FAILURE**
Summary: Exit code 2; exit code 1 Build time: 00:14:59


But that still doesn't explain why you dropped the location for the VIP.  I guess it's maybe related to `colocation` vs. `group`? but it really warrants an explanation in the commit body.


the convention is to pass a uuid to that method, changing the variable name does not do much if can't change the method name.


of course!!! :)
sometimes, my stupidity tires me...

let me fix that


Error with this commit:
[Fri Jan 22 15:50:16 2016] [alert] [client ::1] /home/yllen/gitsvn/front/.htaccess: Satisfy not allowed here, referer: http://localhost/glpisvn/front/ticket.form.php?id=144

I'm in apache 2.2.15


It is good idea to get rid of `tap` method for better understanding of the code. But rest of the code should stay as it is from my point of view. And good point with `#save!`. So please refactor this commit and it will be merged


Several problems with this:
1) Wyclif does not like removeAllergy, because behind the scenes it is the same as voidAllergy and does not follow the convention of void/purge currently used. (I did not want to use removeAllergy in case it was removed because of this/)
2) Because allergy is an entity removing it requires that it e removed from the allergies list as well.


The indentation is a bit inconsistent across the PR. For a matter of consistency, I'd recommend to use an editor that supports [the .editorconfig file defined](https://github.com/mozilla-b2g/gaia/blob/master/.editorconfig) for the project.


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


It's more specifically for getting rid of the warning (that is suppressed here). It's not an issue in `EvaluationContext`. Yeah, `?` rapidly gets difficult as it can never match another type, so I broke out a method to express the necessary type connection in this piece of code -- the type is not known but it's known to be one same type in this block.


The only problem I see with this approach is that if the connection is successful but the timeout expires then there will exist an open connection but no one will be able to use it. I guess the timeout here should actually be `infinity`, since sooner or later the `shotgun_open_resp` message will be sent.


Like I say, I will rewrite this function to throw an exception to the caller.


TeamCity LabKey :: LabkeyRelease15.3 :: CDS postgres 9.4 [Build 135](https://teamcity.labkey.org/viewLog.html?buildId=298688&buildTypeId=LabkeyRelease153_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 7 (1 new), passed: 42 Build time: 01:27:42
### Failed tests
```
org.labkey.test.tests.CDSTest.verifyFilterDisplays: <no details avaliable>

org.labkey.test.tests.CDSTest.testLearnAboutStudyProducts: <no details avaliable>

org.labkey.test.tests.CDSTest.clickOnLearnAboutStudyProductsItem: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyAntigenScatterPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyLogGutterPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyDensePlotBrushing: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifySubjectCountsWithFiltersAdvancedOptions: <no details avaliable>

```


feel the pain! ;-)


TeamCity Civic Assembly :: Build backend + frontend [Build 21](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=420&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) outcome was **FAILURE**
Summary: Tests failed: 117 (105 new), passed: 1604, ignored: 12 Build time: 00:32:57
### Failed tests
```
Blogs tests: Should delete blog: <no details avaliable>

Document tests: Should add new version of Document: <no details avaliable>

Document tests: Should be able to revert from new version of Document: <no details avaliable>

Document tests: Should be able to propose Document to review: <no details avaliable>

Document tests: Should be able to publish proposed Document: <no details avaliable>

Document tests: Should take a alert when try to upload new version for locked document: <no details avaliable>

Document tests: Should be able to unlock and revert to draft locked proposed Document: <no details avaliable>

Document tests: Should be able to propose reverted Document to review: <no details avaliable>

Document tests: Should be able to decline proposed Document: <no details avaliable>

Document tests: Should be able to propose, publish and expire Document: <no details avaliable>

Media type tests: Should navigate to Media Types page: <no details avaliable>

##### there are 107 more failed tests, see build details
```


Nit: Unused


do you mean I would need to manually select the profile at installation time?


missed `);` at the end of line


这里不是 `C++ 0x11`，就是 `C++ 11`。


Based on this, I thought it should be 1 or 2: http://elementscommunity.org/forum/card-ideas-and-art/insight-enlightenment/msg460542/#msg460542


showhide.js


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


Totally tiny thing but you can also do:

```
from flask.json import jsonify

def myfunc():
    return jsonify(**mydict)
```


@573834746 There isn't any PHP6 at all. PHP6根本不存在。


FYI, on 16.04 the file accesses were added to the network-listener cap in ubuntu-core-security 16.04.13. These accesses are not in ubuntu-core OS snap 16.04.0-7.


Oops, I had made a typo to fixed another typo. -_-


See https://travis-ci.org/mochajs/mocha/builds/104113334, all tests failed.


群信息是 @usernameusernanme:<br/>信息内容
个人信息就直接是内容


done


每次更新synckey，这个好像参考activesync协议来实现客户端的增量更新。have fun！


@nightwing @lennartcl @javruben : new commit id is not pushed yet


this looks super cool to me!


@chrissetto Ok, thanks. I see what you mean. Plugging that line back in is causing the download to fail. I'll work on it.


域名为什么会这么多呢？我一直很奇怪，请指教


It failed on travis - I think we should support gcc 4.9 as long it is not that much effort.

The initialization of the anonymous struct looked also a bit weird :)


The commit message is exactly the opposite of the contents of the commit.


wdf那里用的xdg还是比较。。哦，你是mac党。。。


I can only get it tobuild on nodejs 4, though, not 5.


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


What should this size be for pause burst? Looks like it will be e.g. AC3 frame size of 6144, but should it just be the size of the pause burst samples? (I'm not 100% sure).


requests.......横空出世


Woops, fixed and re-commited @d5e8a622a19b8a2f4132f595fefe03ce84af6624


yes that's what I planned, but there're might have a clever way (we might use a postinstall step which detects the plone version and import the right sub profile)


This looks great. One thought: Should we offset the timers a bit? E.g.

```
setTimeout(function() {
    setInterval(cardGetter, fullRefreshRate)
}, i * 1000);
```


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


> I just have seen that the semicolon close the end is missing.

it's not only the semicolon which was wrong - OC_Helper::linkTo was in there as well.


DANOBATGROUPen. Zuk idatzitako blog sarrera bat irakurri ostean hasi ginen Cabot erabiltzen.


TeamCity pull requests :: netty [Build 1087](https://garage.netty.io/teamcity/viewLog.html?buildId=4984&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 6387, ignored: 23 Build time: 00:41:54


keeping in mind that `/[a-zA-Z0-9_]/` is the same as `[\w]`
and that there's no need of escaping inside a subset of characters..
the whole regex can be simplified as follows:
`/([\w ,:.#&;-]+)(?=.*\{)/g`


@Mkhaledfhkiel, @apollo-h, @scbidua   covered some part of Designs and Graphical parts.


额，公众号原来这么区分，膜！


i dont think this is ok.
`NUnit.Runners` v3.0 it's exactly `NUnit.Console` below (see https://www.nuget.org/packages/NUnit.Runners/ ), so if 2.6.4 it's not needed, we can remove this reference because can reference a different nunit3-console version
/cc @dsyme @KevinRansom


...has no***w***...


checked and give feed back to @taheem @apollo-h  @scbidua


El slug es el pathname de el URL, usar letras minusculas y "-"  no espacios.


TeamCity Civic Assembly :: Build all [Build 26](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=424&buildTypeId=CivicAssembly_BuildAll&guest=1) is now running


Propose to use `Concat` instead of `+` in such cases. Check not only this place.


should work if you clear out node_modules and npm install with 5.5


Ahh, you're right. Good spot!


I agree, the selector is kinda screwed :smile: . It doesn't really return the list as 1 single element. Instead it returns many elements.

Then, if you wait for a single element, it'll return the first one that matches the selector. In other words, we should either rename the function to `firstContact` or use `this.client.findElements()`


！应该叫长连接更合适些，
我这里看到的都是只要synccheck断开就连接，但服务器会保持连接，只在有推送的时候返回断开。客户立即重连


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


因为我想username 是定长，就直接截取了

发送自 Outlook Mobile

On Fri, Jan 22, 2016 at 7:25 AM -0800, "Liu Yuyang" <notifications@github.com> wrote:

。。。

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


不妨用python3 的future （Promise?）来实现下，我来学习一下Orz


@laem : it might be the same as mode de recouvrement of allegement fillon. We should check that.


Thanks!


@Mkhaledfhkiel , @apollo-h , @scbidua  done parsing of DBData in jade.