Nah esse problema está explícito no enunciado.
Ontem decidi-me pela seguinte implementação:
- pipe par-shell-in: por onde a par-shell recebe comandos e os executa.
- pipe par-shell-out: por onde a par-shell devolve a informação do comando "stats".

Problema desta implementação: não podemos enviar SIGTERM aos par-shell-terminal quando se escreve exit-global/CTRL+C porque  não temos o PID deles.


This public static class is completely unnecessary. An earlier experiment that failed and wasn't removed while I was working on this branch.


I like that!    Let us try using those for night mode, too!  (and convert the night mode viewing cone back from orange to something like a lighter blue, too.)


No, the slimerjs tests include the .NET targets as well, see https://github.com/n1k0/casperjs/blob/master/.travis.yml
```yml
    - env: >
        CASPERJS_ENGINE="slimerjs" ENGINE_VERSION="0.8.4" MAKE_TEST_COMMAND="test-dotNET" ENGINE_XVFB="1"
        ENGINE_ARCHIVE_URL="http://download.slimerjs.org/releases/0.8.4/slimerjs-0.8.4-linux-x86_64.tar.bz2"
      os: linux
      addons: {apt: {packages: *apt_cs}}
    - env: >
        CASPERJS_ENGINE="slimerjs" ENGINE_VERSION="0.8.4" MAKE_TEST_COMMAND="test" ENGINE_XVFB="1"
        ENGINE_ARCHIVE_URL="http://download.slimerjs.org/releases/0.8.4/slimerjs-0.8.4-linux-x86_64.tar.bz2"
      os: linux
```


@FadeMind What app use "scissors" icon on tray?


Sweet, thanks. I'll roll that out as soon as I can.


Thanks!

I had literally just done something very similar.... but you pushed first. ;)


Ja, fast ta bort din kommentar om rulesApply x)


Yes, that would be great. :+1:


I see, thanks. Feel free to push further commits against this branch when you do work it out (and assuming you've set up the features build to use a local distro like I described in #1).


please do not use lambda shortcuts. they are not covered in first year lessons :)


it's now ambiguous that 672a happens to be immediately following this part


thanks for information :)


The cclib basic test 'basicMolpro2006/dvb_ir.out' is an example where the len(vibsyms) < len(vibfreqs), using the current cclib/master branch. This is due to the fact that Molpro 2006 includes some extra modes in the 'low-freq' mode section (normally reserved for the 6 trans/rot modes), which are printed without symmetry labels. 'basicMolpro2012/dvb_ir.out' does not suffer from this problem.  GenericIRTest did not fail before my changes, as it did not test the len of vibsyms.

    >>> molpro2006data = cclib.parser.ccread('basicMolpro2006/dvb_ir.out')
    >>> len(molpro2006data.vibfreqs)
    54
    >>> len(molpro2006data.vibsyms)
    52
    >>> molpro2012data = cclib.parser.ccread('basicMolpro2012/dvb_ir.out')
    >>> len(molpro2012data.vibfreqs)
    54
    >>> len(molpro2012data.vibsyms)
    54


Oh, and they have been included only in c# 6


Jose, I think that  a step is missing in the pre-conditions of the scenario because you need that a room is already associated with the location maybe create a new step, something like this:
Given I have a Location "Custom Name" with display name "display name" that the Location has a Room "Room Name" associated


fixed #25


See #731


Did you mean VARCHAR(14)?


Zusätzlich CHAMPS_PARTNER Rabatt Prüfung & Vorbelegung auf 20% eingebaut.


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


:smile_cat:

You're right, in the last comment I got confused and I was referring to another bit of magic: the fact that now installing a `l10n_` module does `try_loading_for_current_company()` so that you don't have a chance (as I was saying) to add an account or tax template before loading the account chart.

Do you thing that is a bug good for OPW, or there is another way to modify the account chart before loading it?

(as for the current commit, I suppose if we leave all our `admin` users in English, we should keep the old behaviour)

Thanks!


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


@PiterEL
https://tvheadend.org/issues/3351


TeamCity Habitat :: Cruise [Build 323](http://build8ua1:8081/viewLog.html?buildId=798&buildTypeId=Habitat_Cruise) outcome was **SUCCESS**
Summary: Tests passed: 106 Build time: 00:01:14


Usage:

    npm run-script screenshot


TeamCity Xbim :: Xbim Geometry :: XbimGeometry [Build 104](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=560&buildTypeId=Xbim_XbimGeometry_XbimGeometry) is now running


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


hah great catch.


emacs proofgeneral provides good looking indentation automatically, which is easy to
use and looks better than this non-indentation.  "Select a region" and press TAB.


Use `setBooleanPossibleValues()` and `setDefaultValue(String.valueOf(false))`


Lol so fast they reverted... Grrrrr


gotta love integer overflows </3


Can anyone tell me why this hasn't made it into the docker/compose project? I really hope you just forgot and will be adding it! ;)


Sorry but on our architecture, we dont need these commits:

uid_cputime: Iterates over all the threads instead of processes.
uid_cputime: fix cputime overflow

We will have bad measurements in the graph of the battery. Since I checked in his day.

It's just a tip, do not attempt to offend. Thank you


proper change


New binder commits except
https://github.com/temasek/android_kernel_samsung_hlte/commit/8dbbad4b1ee1a54d29a644bb426fab246dca82fc
has caused userspace corruption, which lead to data loss, which forced me to do a factory reset on my S6.

Be careful lol


snapshotId is java long, so I think the related jdbcType is BIGINT


Use a guard clause instead of wrapping the code inside a conditional expression.


Ser till att åtgärda det. Ska jag skicka in det?


@sns-seb when is `null` returned ?


Totes ;-)


Alright, my apologies. I'm new to open-source contributing, still learning the etiquette. :)


Not necessary, just habit. removed.


I think a tool like this should only revert information ... maybe someone uses different proxies and used the tool to set an additional one. Especially with the second option ... there can be a lot of substitutions!


test commit


@alvare52 DWMR. Shipping code. Ship it!


Заглянул в хранилище и увидел, что на бордах без поддержки показа своих постов на каждый отправленный пост в `DESU_MyPosts_` всё равно добавляется по `null` - нахрена?


whenever a `CeTaskProcessor` does not implement (yet) returning a `CeTaskResult` (so, currently in Views and DevCockpit)


Well it's possible that you need some custom serializers to serialize the event classes. But I think it makes sense to make it common for all event classes, so that won't be a problem - I'll refactor that


Для чего это поле торчит в интерфейсе? Оно дублирует по смыслу такое же поле из `BaseLoader`. Нужно оставить что-то одно


This also matches the URLs from `ACastIE`, it should be more strict, I think `https?://(?:www\.)?acast\.com/(?P<id>[^/#?]+)/?(|[#?].*)$` can be used.

Note: this is tested by `test_no_duplicates` in [test_all_urls.py](https://github.com/rg3/youtube-dl/blob/c6aa838b51484ad3d036db21ba4e9e834720a693/test/test_all_urls.py#L87). You can manually run it or use `make offlinetest`.


não vi como responde nossa API, mas já tive muito problema por testar com 200 e a API retornar 30x por causa de cache.


Sure, if there's no model updates for an event, no need to deserialize it, so a partial function is fine.

`Class.forName` uses the boot class loader, which especially in app-server environments, may be the wrong one (not the deployed-app-classloader). In general, a rule is to use `Thread.currentThread.getContextClassLoader`, but here we can skip that step entirerly :)


@fsouza the AppDetail will be removed, and I don't know why AppDetailJson has no tests.


@brianbmcintosh DWMR. Ship that code! :articulated_lorry: :trophy:


yep. changed to use `cPickle` and fallback to `pickle`.


I really don't like this automatic indentation mode in proofgeneral, so I have completely disabled it... I prefer not indenting all the tactics in a proof script as this gives me 2 characters more to work with. :)


Can I get `var err error` instead of named a named return value?


Yes, the repeated calls to `context.ApplyChangesOnProperties` looks suspect.  I'm curious as to why we're cloning `visitationHelper` when passing Unchanged and Modified, but not when passing Deleted.  Why do you think that is?

I'm also wondering what might happen if we don't clone visitationHelper for lines 190-192?

I'm also thinking about adding a condition to the second call to processing deletes:
```csharp
if (item.TrackingState != TrackingState.Deleted)
{  // Clone visitor?
    context.ApplyChangesOnProperties(item, visitationHelper, TrackingState.Deleted);
}
```

Any thoughts on these?  TIA!


Больше не используется


test


Этот метод сделан `public` только потому, что нужно его отдельно протестировать. Значит, есть смысл выделить абстракцию. Туда же уедут `_maxFontSize`, `_fontName`, и все такое.


go lint is not very happy :-)


Пора попридумывать новое имя для лоадера. Он не совсем лоадер уже :-)


你好，世界


I have virtual machines for jessie and stretch, so please let me know if you ever want me to test anything.  I have a small amount of experience in creating .deb packages, though I'm definitely not an expert.  Starting from VTK 6.3 and vtk-dicom 7.1, vtk-dicom can build as a vtk "remote module", which means that it can be built during the VTK build and can be part of the same source package as VTK.

VTK's ABI is unstable, most of the VTK development team doesn't care about backwards compatibility of the ABI from one minor release to the next.  Send me an email if there is anything that should be changed about VTK, I'm willing to help to make it work better for package-builders.


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


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


this line is ambiguous to understand


Fint med mig


Поле используется в одном месте, да и следующее тоже. Давай не будем их сохранять, а брать напрямую из статистики?


Thanks, it works. :)


In other works I often see "Apple, Inc."

Maybe this should be the case with this commit as well.


please add whitespace


The scenario login, I think is not necessary, We only use the steps that are already created
Given I log in successfully as "test" with password "Client123"


whitespace


Здорово!


lgtm


Without doing a careful manual inspection, here's what my decoder spits out:

LENGTH out of range [5,255]: 3
User data with no payload. FUNCTION: PRI_UNCONFIRMED_USER_DATA
Unknown SecToPri FUNCTION: INVALID
Unknown SecToPri FUNCTION: INVALID

#2 is definitely poison. Saw quite a few implementations choke on this. Also frames w/ only TPDU header and no segment data.

Anything else in there?


shouldn't this also check for DNAMEs?


Чтобы уж совсем хорошо, можно сделать здесь `IReadOnlyList<Word>` вместо `List<Word>`, а в конструкторе можно принять `IEnumerable<Word>`


All of the files that contain copyright information in this project, the name of the company is just Apple Inc. For example:

https://github.com/apple/swift/blob/master/include/swift/ClangImporter/ClangImporterOptions.h
https://github.com/apple/swift/blob/master/include/swift/IDE/ModuleInterfacePrinting.h

This is the title for most files:
```
// This source file is part of the Swift.org open source project
//
// Copyright (c) 2014 - 2015 Apple Inc. and the Swift project authors
// Licensed under Apache License v2.0 with Runtime Library Exception
//
// See http://swift.org/LICENSE.txt for license information
// See http://swift.org/CONTRIBUTORS.txt for the list of Swift project authors
```


travis Y U fail on this commit???? https://travis-ci.org/HydraChain/hydrachain/builds/94865064 :rage4:


please use the new DNS_ANSWER_FOREACH() macro to iterate through the RRs.


This approach does not extract all formats. http://www.trilulilu.ro/big-buck-bunny-1 has both SD and HD, only SD is extracted.


When easyMove updated (in line 518)? It should be always null.
I think EasyMove.get(rootPos.key()) is correct.


https://github.com/AndreiLux/ref-3.10/commit/3f531886dcda4988d358f64e580cac52cf9982c5
이게 더 낫지 않을까요..


Приватные свойства не все поля)


Why change this from AllocateZeroPool() ?


Den her linje skal vi ikke have med!


Or this?


missing new line ;)


:+1: x :100:


@starteleport это в более свежем коммите исправлено, кажется :)


А точно нужно в таких лоадерах читать файл при каждом обращении к свойству?


nice, i was meaning to do that, and still need to do the completions function more generics if I could have how to passe argument to heu _arguments completion functions


maybe it help? -> https://github.com/PlusGit/theta/commit/4ebb04ce4d3b00bf8b6cf3ff993ccdb4d919b90c


All the reformatting makes this much harder to review.


@seb would have say "that deserves a comment in javadoc"


Typo in commit message, should read "Allow Service as `member` of Organization".


Why not just give yourself 2 more on the right hand end instead?

On Fri, Dec 4, 2015 at 8:41 AM, Anders Mörtberg <notifications@github.com>
wrote:

> I really don't like this automatic indentation mode in proofgeneral, so I
> have completely disabled it... I prefer not indenting all the tactics in a
> proof script as this gives me 2 characters more to work with. :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_mortberg_UniMath_commit_08adb7400e6da0883c5d1c5af806c0be3fb2b225-23commitcomment-2D14787162&d=BQMCaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=PAZ_jTPLrSb1btCfAu4RkycWe57N0sGyzbjgeOn2wN0&m=J1KySXpn8lzTY9TJJ6JvBuApaIEq0XoJ_xQkKOSYVUQ&s=pBa3cYawBUQyf-j1mt0jeD6PM_oFhv0w35_eq_-giuA&e=>
> .
>


*Your


Нужны тесты


Sorry, somehow I skipped those two lines from the commit, I just fixed it


Technically, that is an XML "attribute".


Replacing double quotes with single quotes is not a bug, it's something I did in the past. I should have comment the code, because I can't remember why. This require a depth testing on my side, because it could cause regressions on the Linux/OSX sides. Can you please remove this commit from the PR and submit another PR dedicated to this bug ?


why this is necessary? I don't see any usage yet.


cmon man!


그것도 이미 적용되어 있습니다.
시냅스를 띄워 둔 상태에서 화면을 껏다 키면 A57 클러스터가 높은 확률로 화면이 꺼졌을때의 최소클럭으로 떨어진 걸로 인식되는데
사용자들이 무의식중에 Apply 버튼을 눌러서 적용해버리는 일이 많아서 이 방법을 동원한거에요.
그렇게 해두고선 렉 생긴다고 하시는 분들 땜에...;;

2015-12-04 22:52 GMT+09:00 Park Ju Hyung <notifications@github.com>:

> AndreiLux/ref-3.10@3f53188
> <https://github.com/AndreiLux/ref-3.10/commit/3f531886dcda4988d358f64e580cac52cf9982c5>
> 이게 더 낫지 않을까요..
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dwander/N4-StockRider-TW-5.1.1/commit/33a1010a8eeb7cf280c5848a61a7fb8c422fbce2#commitcomment-14787324>
> .
>


Ty :)


@starteleport нужно создавать файлик и проверять, что правильно считались слова?


I think it should be Mongo*Commit*Fields.BucketId


Moreover the downloaded file is not a media at all but a webpage.


Should be ==?


{{ "<br />"|safe if compo.linebreak_between_options and loop.index != compo.options|length else ""}}


Такой же код сейчас в `WordsLoader`-e. Подумай, что можно с этим сделать.


I think the rationale for this was that the Apple KB article one is a vetted, Apple-recommended list, whereas the other ones, while very very similar, were derived from other sources. Ultimately, it's up to the admin to decide which ADF's to use-I would like to add a README or comment to the Apple one, but keep it around (they have changed it periodically too, I think), and retain the extra entries in the individual ADF's.


This checks all the files in the current directory. Even node_modules and all submodules.
We should add a `.coffeelintignore` file or whitelist all directories that we'd like to lint.


TEST


@starteleport выделить в загрузчик списка слов и унаследоваться от него? О_о


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


Oh, you have one in this pr. But it didn't copy it because the other hook is doing that. :smile:


TeamCity Xbim :: Xbim Geometry :: XbimGeometry [Build 104](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=560&buildTypeId=Xbim_XbimGeometry_XbimGeometry) outcome was **SUCCESS**
Summary: Running Build time: 00:30:26


you mean: you have in 8.0 a module that depends on l10n_it (for example) and that is currently adding some account, tax templates before the real objects are generated and you would like to know how to achieve that in 9.0 since the module installation is directly calling try_loading_for_current_company() ?

Is it the use case you want to solve?

Regarding the current commit: in a near future, you'll be able to select your country at the installation of the database, so it will be helpful as well


@jdubois  couldnt we use a better needle naming convention rather than numbers as t\it willbe hard to mange and read as it grows, may be something like jhipster-needle-<moduleName>-<featureName> etc


Варнинги есть? Работает нормально?


in the snapshot table the id is an integer, that's why I kept the integer


See https://github.com/jenkinsci/git-plugin/pull/371, which does not currently attempt to solve _that_ incompatibility.


This breaks python 2.6.


rename 'scrapped' to 'scraped'


Зачем `public`? Вообще, следи, чтобы `public` был только тогда, когда это действительно нужно. Если какой-то метод `public` только ради тестов, что-то тут не так. То же самое про интерфейсы - там не должно быть ничего лишнего.


Shouldn't this be `WorldExts` or something?


Эмм) Так себе тест))


Just cosmetics.


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


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


하긴 쉘 보다는 저걸 응용하는 게 더 낫겠네요.

2015년 12월 4일 오후 10:56, DQ <dqstyle@gmail.com>님이 작성:

> 그것도 이미 적용되어 있습니다.
> 시냅스를 띄워 둔 상태에서 화면을 껏다 키면 A57 클러스터가 높은 확률로 화면이 꺼졌을때의 최소클럭으로 떨어진 걸로 인식되는데
> 사용자들이 무의식중에 Apply 버튼을 눌러서 적용해버리는 일이 많아서 이 방법을 동원한거에요.
> 그렇게 해두고선 렉 생긴다고 하시는 분들 땜에...;;
>
> 2015-12-04 22:52 GMT+09:00 Park Ju Hyung <notifications@github.com>:
>
>> AndreiLux/ref-3.10@3f53188
>> <https://github.com/AndreiLux/ref-3.10/commit/3f531886dcda4988d358f64e580cac52cf9982c5>
>> 이게 더 낫지 않을까요..
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/dwander/N4-StockRider-TW-5.1.1/commit/33a1010a8eeb7cf280c5848a61a7fb8c422fbce2#commitcomment-14787324>
>> .
>>
>
>


Posting in (hopefully) historical thread.


I don't think this will be a good idea, sethostname() (linux-micro port) and systemd's SetStaticHostame
functions might required that the user is an administrator (a.k.a root). This behaviour might cause the test to fail.


The universities are initialized into an array in a numbered way. One can just access the position in the array to access the university by Id


oops.


I don't think so. From the process method standpoint, the annotation is self describing, implementations are allowed to return null.
What javadoc would you expect?


@janvanmansum

Is this better? More but shorter lines. Assembling content and writing even further separated, at the cost of having more content in memory but it are not very large files. Or do you want your constants back and drop the methods-per-file?

See also how I did it for [EasyExportDataset](https://github.com/jo-pol/easy-export-dataset/blob/22cd647679bca43eee40f98b9d487ac7b207f31d/src/main/scala/nl/knaw/dans/easy/export/EasyExportDataset.scala#L60-L68)


Does this `t.Unix()` need to be `t.Unix().UTC()` ?


Ну и `RandomColor` сделан `public` только ради этого теста


This is getting really interesting. The first test of snicolet was with {300, 300} and was yellow.
So there might be room to do a quantum leap also in the eg value !


This will fail if there is no match for regex.


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


Missing space before the opening brace


yeah, it was a quick refactoring because the same string was used in several places


ok


Should be better to make it non fatal as well as f4m.


LGTM.


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


Wrong idnentation


aA


The login validation should be referred to the model, not here


used in rrdb


Wheres Ian :0

On Sun, Nov 29, 2015 at 8:13 PM, ShadowMine <notifications@github.com>
wrote:

> It was a bad idea letting Ian decide...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ianm1647/IaniteReCraftedPE/commit/167be215700011cf6980ecc3b9108414779a40d6#commitcomment-14671503>
> .
>


@lattner Why didn't you use any lexer generator (lex, ragel, ..) for sketch of your compiler?


Do you know if this was ever incorporated upstream?


`if ((state)->is_32bit)`


yes, should be ==


@gforcada is this lyric style intentional? :)

i was in the mood to refactor coredev docs to meet plone documentation guidelines (especially semantic linebreaks) before i fix some of the urls.

but im not sure if i'd destroy the indented rhyme.


Having the macro in `do ... while(0)` may make travis happy


No, that's it. The point is that the second and third frames start within the first one's header - so your decoder scans byte-wise (doesn't consume invalid frames). The fourth frame follows the first and could be masked if the second or third is consumed.

The invalid function codes are due to the control bytes falling values not easily controlled. One might be able to wiggle things around for valid ones.


Doc is nos descriptive enough,
function name should be signUpAction() {


@syl20bnr @TheBB @StreakyCobra  `dotspacemacs-distinguish-gui-ret` is not implemented yet. I'm not sure why it was added here.


LGTM


or should the EG value be lowered?! I am uncertain.


Wrong indentation


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 8067](https://teamcity.mesosphere.io/viewLog.html?buildId=86460&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Execution timeout (new) Build time: 01:00:06


never mind, it looks like https://github.com/sourceryinstitute/opencoarrays/commit/ef5b51103c19446e6a1b23a7aa1acf8496ddd6c7 did the trick


e.g.
```
Not a single commit anymore,
but a series of them,
meaning that all those fears,
concerns,
doubts are taken away!
```


PENETTA


prototype is long, not void...


Would be nice to add some logging in to this for matches and misses.


Code is redundant, please unify with the one above, and place properly
https://en.wikipedia.org/wiki/Don't_repeat_yourself


please, fix return value in case vs->rs is empty


We should revert this back because of the reason as described in communityii/yii2-user#60


Looks good.


Okay Noted


Here, you place the opening brace on the end of the line, not below.


Can we implement this in the refactored version?


hmm, yeah. On checking here I'm getting different values from couchdb than I'm calculating here.
```py
import requests
from base64 import b64encode
from hashlib import md5

URL = "http://localhost:5984/commcarehq/64a6827fc1531ac1287c31df57dd1589"

def md5_digest(data):
    return "md5-" + b64encode(md5(data).digest()[:16])

#print repr(requests.get(URL).text)
doc = requests.get(URL).json()
for name, blob in doc["_attachments"].items():
    data = requests.get(URL + "/" + name)
    digest = md5_digest(data.content)
    print("{} {}".format(name, digest == blob["digest"]))
    print(digest)
    print(blob["digest"])
    print(repr(data.content[:100]))
    print("")
```
output:
```
75d9a4b6ea51566ac3f465bd42a1df41e81dc2f9.xml False
md5-grS+l0raW/69BmIm2S3+uw==
md5-o9pb0nWWaOTn00SPEoTt7g==
'<h:html xmlns:h="http://www.w3.org/1999/xhtml" xmlns:orx="http://openrosa.org/jr/xforms" xmlns="http'

...
```


I don't think that is necessary. By https://tools.ietf.org/html/rfc6672#section-3.1 we are guaranteed to get a CNAME whenever DNAME substitution is employed.


My fault :(, I've recreated the branch with the right commit at address https://github.com/NEventStore/NEventStore.Persistence.MongoDB/commit/f233bd3a2c708d34c0911d35d743f4dab84b982a


Will do.


Missing space before opening brace, please check all functions


Like I asked you before, DO NOT USE MD5!


W karcie node edit pojawia się checkbox "cała sieć", a co z kartą nodeadd ?


a simple starter code with attiny85


Packet in a packet. nice.


We DO accept PR :smile:


전에 커널에다가 여러번 유저모드헬퍼 묶어봤는데 매번 레이스컨디션땜시 엉망되더라구요


Here they are decoded by dissect, in order of appearance, stitched together from separate runs:
```
L> primary frame from master 25605 to 25605: TEST_LINK_STATES
L> primary frame from outstation 25605 to 36521: UNCONFIRMED_USER_DATA
L> secondary frame from master 1438 to 25605: function 14 (obsolete): 00 00 78 91 00 00 00 00 00 00 00 00 00 00 00 00 00 00 B6 77 00 00 00 00 00 00 00 00 00 00 00 00 00 00 95 5D 00 00 00 00 00 00 00 00 00 00 00 00 00 00 FC D6 00 00 00 00 00 00 00 00 00 00 00 00 00 00 5E 71 00 00 00 00 00 00 00 00 00 00 00 00 00 00 1E FE 00 00 00 00 00 00 00 00 00 00 00 00 00 00 CE D7 00 00 00 00 00 00 00 00 00 00 00 00 00 00 5E E8 00 00 00 00 00 00 00 00 00 00 00 00 00 00 7F D1 00 00 00 00 00 00 00 00 00 00 00 00 00 00 55 67 00 00 00 00 00 00 00 00 00 FF FF 00 00 00 00 00
L> secondary frame from outstation 0 to 56944: function 5 (reserved): 00 00 00 00 00 00 00 00 00 00 00 00 17 A7 00 00 00 00 00 00 00 00 00 00 00 00 00 00 E5 A0 00 00 00 00 00 00 00 00 00 00 00 00 00 00 6E 8E 00 00 00 00 00 00 00 00 00 00 00 00 00 00 F5 CB 00 00 00 00 00 00 00 00 00 00 00 00 00 00 E4 8F 00 00 00 00 00 00 00 00 00 00 00 00 00 00 F5 0E 00 00 00 00 00 00 00 00 00 00 00 00 00 00 B3 A0 00 00 00 00 00 00 00 00 00 00 00 00 00 00 6E A5 00 00 00 00 00 00 00 00 00 00 00 00 00 00 1C D1 00 00 00 00 00 00 00 00 00 00 00
```


That have probably fixed #1600


好像这里没有合并功能，必须提交PR才能合并。要不我直接改掉了，谢谢指出。


It should've been fixed a few minutes before in 96fea82e9.


Doc should be something like

/*
 * This function sets the conditions (format and salt)
 * in which the password will be encrypt
 * in which the password will be encrypt.
 *
 * @param $password string
 * @returns string the password hashed
 */


This is a nice idea indeed, but you should use GetMappedFileNameW and then convert the thing to UTF-8


does this also mean we can up PHPUnit from 4.x to 5.x? this means only php 5.6 and higher is supported


Shouldn't we do this directly when extracting the property ?
https://github.com/avTranscoder/avTranscoder/blob/develop/src/AvTranscoder/mediaProperty/FileProperties.cpp#L213


Мне больше нравится такие блоки выносить в отдельные функции.


Trzeba dodać.

Póki co mam prawie skończone - dzisiaj podeślę - możliwość przypisywania klienta do podsieci z poziomu edycji sieci, więc ten temat jeszcze będzie rozwijany.


Check the doc format
The method name is too generic, what is all filled?
$array is NOT a string!!!


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


AMAZE


Yes, that's the usecase.


@pelwell we don't need this subsystem change now that we have dropped ATAGS. @popcornmix has dropped it from the original patch in his 4.3 rebase branch which was applied on 4.4. https://github.com/raspberrypi/linux/commit/0fff4f5aac936bdd6d1fa0bff779a0b7b8e6c9a1


From some experimenting yesterday what I *think* may be happening is that with Firefox at least, the page automatically scrolls to the first element with focus. When the filter panel is closed the “Clear” button at the bottom of the page has focus, so on loading the page scrolls to there. Moving the clear button to the top of the page appears to have worked around the issue.

I need to understand this behavior more fully, but I’ll leave that work-around in place for now. Perhaps there is a way to prevent any element from having initial focus?

Steve
--
Stephen Beaver | +1 (512) 646-4100;239 | sbeaver@netgate.com | www.netgate.com

> On Dec 4, 2015, at 4:10 AM, NOYB <notifications@github.com> wrote:
>
> Same thing happens with print info box messages. If there is one at the bottom of the page it will scroll to it. But only with these section collapse settings applied.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/pfsense/pfsense/commit/d5d5836c4f94e70f5f4ddf1df8ac8bb6e684202c#commitcomment-14782387>.
>


Gute Idee!


Please explain this validation, it does not make sense
What are you trying to validate?
This is closely related to the generic function name


Почему отказался от q.all?


Figured as much - that's why I've been trying to reply promptly during the work day :rocket:


Agreed. I hadn't intended to have anyone review this code until I had removed the reformatting.  When I saw that you were proposing a change near this code, I wanted your input on the idea of toString() for test assertions.  I wouldn't bring this formatting mess into the main line of the plugin.  There is already enough complexity from the inconsistent formatting in the plugin without me introducing more.

I'll try your recommendation of a TestExtension to see if I can learn more about that technique and have a better test as well.


@sirmordred you mean editing libcamera is enough. But yes. And if you want i can try and edit one for you to test, or you can do it yourself either way let me know.


This will never happen, you are just iterating over values, and checking if the values that you iterate exists!


This approach does not work for all videos: http://watch.nba.com/nba/video/channels/playoffs/2015/05/20/0041400301-cle-atl-recap.nba.


Goddamnit. Ignore this, have to do it again.


@mikehaertl Same coin different side I would prefer them completely separated since I knew exactly when I should used each one. However it looks ok to me except you are making the same assumption as I did, and that is that I we're assuming that every model uses the same connection. As I said before it would be better if there is a restriction in the models to use the same connection just in case. Since you are iterating the models in the contsructor you can add a check there effortless.


repeated? does not represent the real use, maybe confirm?


What is this code trying to do?
Change the speed each frame?
Isn't better to use at least an fixed update?


A non static inner test class. Is that intentional? Not important for this test but might give a wrong hint to someone looking at it.


If you are going to leave it empty, add a @todo stating something like

@TODO complete this documentation since Im lazy to do so right now


TeamCity NEventStore :: MongoDB :: Build [Build 5.3.2-beta.1+1](http://build.prxm.it:8813/viewLog.html?buildId=3382&buildTypeId=nes_mongo_build&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 75 Build time: 00:02:41


JA!


That's essentially what this is doing just without a separate factory class, which I felt might be a bit overkill for this little piece of the logic. Also, you could use a function reference instead of a lambda if more complex code is needed. Even this amount of code to support backward compatibility for restoring output connections on converted sops seems overkill imo.

Actually, the impetus for this comes from another pixo request to similarly encode context objects for use in .ini files. Psyop also mentioned using similar logic for storing data in environment variables. If we have several places where we need this type of encoding, maybe it would be cleaner to move this logic into core?

All that said, I'm cool with whatever direction you think is best. :)


There's a typo, the second one should be `MakeUppercase`.


Wrong indentation
Mixed data on the doc,


Please use the ._value property (it is inherited from BaseMLSComponent). See Sensitivity class comparison functions for what I mean.


Mas era suposto enviar-se... Se não dá como é que tencionas fazer?


You only return it when its a string?, and if not?, fail silently?


Same here, please go over all docs, (even the missing ones) and fix them properly


No, not intentional, I just did cut & paste. I will make it static in a moment. Thanks for the suggestion!


Finally, they line up.  I have no idea why double space wasn't working.


Que tal: sempre que a par-shell receber um número, só pode ser um PID, logo guarda esse número
Estou a assumir que a par-shell e o par-shell-terminal são amigos e não há necessidade de protocolos "não confio no outro gajo" estilo TCP/IP.


@mikehaertl Usually I almost hate PHP for allowing that. My approch is either it can be a primitive / object and NULL


Thanks for sorting this.


spacing is off


why the _u postfix?


INDENTATION!!!


Weird.... that commit exactly broke the build for me. Reverting my own change:
```
$ nix-build -A e19.efl
error: attribute ‘dev’ missing, at /home/lethal/nixpkgs/pkgs/desktops/e19/efl.nix:36:31
(use ‘--show-trace’ to show detailed location information)
```


Mixing styles, big no no here!!


well, but the synthetic CNAME cannot be verified, because it won't have precomputed RRSIG RRs... Hence, because the CNAME doesn't have RRSIG we have to ignore it, and can only add the DNAME to the cache, because that#s the RR we can verify...


this needs to implement the `MessageChannelConfigurer` interface


you are returning without releasing memory... there is a leak here.


Propose to log would block error with debug level. this problem should be rare and it could be handy to debug such cases.


Sempre que a par-shell receber um número, ou sempre que a par-shell-terminal receber  um numero?


Ah, @sbrannen already did it: https://github.com/junit-team/junit-lambda/commit/82597c08954d308573ea7a86f27619eef8bdbfa6#diff-c9dce30da70555ec4d31b27abe63cd49R77.


this currently still breaks at a rather importing step: sending mails.

First `twisted.internet.FileDescriptor.write()` really does not want unicode, but than msmtp fails with

```
alot.account.SendingMailFailed: msmtp -a lostpackets -t failed with code None:
A process has ended with a probable error condition: process ended by signal 13.
```

I'll look closer into it next week.


Estou nos labs do técnico, não andas por aqui pois não?


Remove whitespace


Without this fix, in cases of lower height x width values, qr reading isn't working


this should be a `List` since order may be important


Some JSON parsers are OK with comments, but others aren't, so we should delete this line.


Again, indentation


Please check your code is tidy before pushing any changes!


Hmm, if we use ANY like this, then we should really document this here in a big comment:

"Note that we store information about the domain name as a whole (such as NXDOMAIN info) as a pseudo-cache entry on the ANY pseudo-type."

or something like that...


Only two spaces?


EconomizerIntegrationNew (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1971 of 1971 tests passed)

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


@embray - is this commit deliberate?


Please move your CSS to a separate stylesheet


As a developer an implementation, I don't know what's the value to return a CeTaskResult if null is accepted. What's the difference ?


should think about filename `load.zsh` and should be a read-only file


Fair enough.

There's a lot of information we potentially want to display, and I'm trying to sort out the best way to display it all.

Thanks!


Indentation


INDENTATION!


Não, tou em casinha; hoje não havia aulas.

É sempre que  a par-shell receber um número.

Melhor ideia ainda: Quando recebe um número pelo pipe: se não conhecer o número, é um registo e guarda só o PID. Se conhecer o número, é um pedido de "stats", e faz o comando stats que ainda não defini.


Repeated CSS!


Vou fazer pull request do que já fiz, embora não compile


EconomizerIntegrationNew (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1977 of 1977 tests passed)

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


Also, the checks at the beginning of dns_cache_put_negative() that filter out ANY submissions should probably carry a comment about this too, after all we really need to filter them out when we want to use ANY for NXDOMAIN purpose...


**FAILURE** - TeamCity HotRod C++ Client :: Pull requests monitor - WIN_S2012 [Build 244](http://ci.infinispan.org/viewLog.html?buildId=32893&buildTypeId=HotRodCpp_PullRequestsMonitorWinS2012&guest=1) for ee0e4362026bc2842e8b6226c06e868960ff2987
Tests: 0, 0 failed (0 new), 0 ignored. Build time: 00:00:03

Process exited with code 1


not a good reason. Fixing SQ issues must be done on purpose.


**FAILURE** - TeamCity HotRod C++ Client :: Pull requests monitor - WIN_S2012 [Build 245](http://ci.infinispan.org/viewLog.html?buildId=32926&buildTypeId=HotRodCpp_PullRequestsMonitorWinS2012&guest=1) for ee0e4362026bc2842e8b6226c06e868960ff2987
Tests: 0, 0 failed (0 new), 0 ignored. Build time: 00:00:01

Process exited with code 1


**FAILURE** - TeamCity HotRod C++ Client :: Pull requests monitor - WIN_S2012 [Build 246](http://ci.infinispan.org/viewLog.html?buildId=32990&buildTypeId=HotRodCpp_PullRequestsMonitorWinS2012&guest=1) for ee0e4362026bc2842e8b6226c06e868960ff2987
Tests: 0, 0 failed (0 new), 0 ignored. Build time: 00:00:01

Process exited with code 1


@janvanmansum
directory or directly

Now that I saw this example I understand I might have done things wrong in easy-stage-file-item.


Favorite Color: Neon


Sure. I guess this depends on where exactly you filter out untrusted RRs. I'm assuming here that we have the full DnsAnswer from a given packet (not only the subset that is going to be cached). If someone inserted an unverified CNAME that shouldn't have been there all they'll do at this point is to disable caching, so that's fine.

If the intention is to filter out stuff before reaching cache_put() I should rework this indeed.


@dtieri can you configure your text editor so it does line breaks at 80 characters? That makes it easier to review changes on github.

Don't worry about it for this change set, just for future reference.


Re monster weapons (or parts in general) I think it would be better to define monster specific (weapon) parts anyway. That way we avoid the risk of breaking some space monsters every time we change anything about ship parts.


Why would you bake in the version of es6-promise?


I'm coming


Adding comments on images here:
Why the humongous size?
Why use a png file where a JPG file is significantly lighter ?


@spwhitt You should follow the convention of prepending ``$(PREFIX)`` with ``$(DESTDIR)`` here.

See https://www.gnu.org/prep/standards/html_node/DESTDIR.html

e.g.

```
cp $(BIN_DIR)/libHalide.a $(BIN_DIR)/libHalide.so $(DESTDIR)$(PREFIX)/lib
```


No, I do not rely on this functionality.
because systemd it provides.
except for setting ip address of the interface.
and yes I am interested in supporting multiple networks more than backward compatibility


Also, where are the implementation, update and rollback scripts?


`set(map(int, ..))`? that way it'll fail a bit nicer if we send a non-numeric id for some reason


Carry long lines whenever possible.


reg?


registration?


To go into more detail, the first packet claims a length of 3. Depending on the decoder implementation, any of the other three could be hit next:

1. Decoder consumes the full first frame header. The fourth frame is seen next.
2. Decoder consumes the first start marker and scans for the next. It finds the second frame.
3. Decoder consumes start, length, and 3 (length) more bytes. It finds the third frame.

There could be a fourth case where the decoder consumes start, len, len bytes, and a CRC.

Anyway, the point was to demonstrate the ambiguity that we decided to resolve as case 1.
> as long as the start bytes and CRCs check out, the frame is technically a frame and should be skipped as a whole


TeamCity Dirigo Edge :: Pull Requests :: Pull Request Smoke Tests [Build 148](http://teamcity.corp.dirigodev.com/viewLog.html?buildId=1761&buildTypeId=DirigoEdge_PullRequestFullChain) outcome was **SUCCESS**
Summary: Tests passed: 6 Build time: 00:01:32


Eu também não tenho, só vim para aqui porque não me apetecia fazer almoço e para puder trabalhar na versão atual do projeto, já que no meu ubuntu continua sem compilar.

Substituiste as variaveis de condição por semaforos?


registrations? :monkey:


Extraction should not fail when either `media_videourl_hls` or `media_videourl` is missing.


TeamCity Xbim :: Xbim Geometry :: XbimGeometry [Build 105](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=561&buildTypeId=Xbim_XbimGeometry_XbimGeometry) is now running


Out of curiosity, was this a case of you using the operator and then forgetting about it?  I run into similar challenges with Clojure.


Changes as suggested https://codeclimate.com/github/crossroads/api.goodcity/issues/categories/style


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

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


The DNSSEC RFCs suggest to only accept verified RRsets in the cache, so yeah, my intention was to do the validation before adding anything to the cache, and suppress everything that doesn't validate...


TeamCity Xbim :: Xbim Geometry :: XbimGeometry [Build 105](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=561&buildTypeId=Xbim_XbimGeometry_XbimGeometry) outcome was **UNKNOWN**
Summary: Canceled (Exit code 1 (new)) Build time: 00:00:32


`'**/*.js'` instead of `'./*.js', './*/*.js'` should catch all JS files


I see


Where are the paaareeens???? :notes:


das chas doch ned si Martin, so öpis wet ech nöm gse


@mboelen not yet, the main reason is refactored version make an equal test.

There are two ways:
- improve [SSH-7408] to different tests than '=',
- make different tests for:
* `less is better`,
* `more is better`,
* `not equal is better`.


Timestamp can be extracted from `pubDate`.


Sim substitui, permitiu-me encurtar consideravelmente o código do par_sync. Essa parte não faz parte da avaliação portanto não deve importar, a não ser que o prof pertença à Igreja das Condições de Variável.


Oh, I see, I'm sorry. I did some small changes after testing the fix and it broke evaluation.


no data-title ("Delete contributions?" or something like that)?


@robfrank This seems to break the `dserver.sh` script (on Linux Mint and Ubuntu), with the message `Bad substitution` on line 5 ( `${0/%dserver.sh/server.sh} -Ddistributed=true $*` )

When I replace `/bin/sh` back to `/bin/bash`, it works fine.
Is that line POSIX-compliant? (Since bash behaves differently when invoked with `/bin/sh` - source [here](http://stackoverflow.com/questions/26717870/why-does-bin-sh-behave-differently-to-bin-bash-even-if-one-points-to-the-other).)


Yeah, the trouble here is that the DNP3 spec provides no guidance, and different implementations will see different things. bummer.


I did not dare to update the getFilename() method since it is widely used, and this is not the case of allPropertiesAsJson(). And the other pyTest works on Windows, even when the absolute paths to filename contain '\': see how we set the windows environment to test the library: https://github.com/avTranscoder/avTranscoder/blob/develop/tools/appveyor.python.nosetests.bat#L15


Mas ainda lá tens código das "condições de variável", a versão atual do github não está a compilar por causa disso


we live dangerously.

@darokel do you mean like this?
```(/^\/(en||cy)$/).match @slug```

or like this?
```/^\/(en||cy)$/.match(@slug)```


Duration must be in seconds.


It should be noted, that the quotes are replaced in the docstring. Only the swig interface declaration is changed, what for me totally makes sense:

It the header file contained a char 'a', it still should be 'a' in the declaration. If it was a string "string", it should be a string afterwards.

Can you please show an example, were this commit produces the wrong result. When generating the wrapper code with these changes, the source code has changed only at two places:
https://github.com/rainman110/pythonocc-core/commit/f5156784e30498a748ee07f4483e6b8306c9caf9

Both changes seem to be absolutely correct.

I can create another PR to discuss this issue.


`each` is (almost) universally a bad idea anyway.
Additionally, the use of `each` (and friends) on a reference directly is declared "experimental", and is best avoided because it's going away soon anyway.

Since you're not using `$key` here anyway, this wants to be

    foreach my $value ( values %{ $body->{devices} } ) {

Fixed in c107712


fly fucking here: function name is not explicit. netlink_if_link_populate would be more aligned with current naming. in prototype : "struct rtattr *tb[]" should be "struct rtattr **tb"


setting to 1.140 makes ${htsjdk.version} variable useless


same issue here: memory leak


Why??? How about backward compatibility?


What permission will be used if the mode attribute is removed?


can you upgrade to 1.141 or not add pom.xml to your commits ?


Could you add a few more deps in more dirs just to show it find them all. Also a miss would be good to assert.


it didn't work after adding suppot for team mini sites.
i will fix it(don't match nba after `/video/*`).


@tavantzo Well, I see no point in having 2 classes with almost identical code. It's really just an extra feature. The DataProvider will behave as before and thus not break BC.

So if you agree, you could update your PR with my code here (maybe add the connection check, that you mentioned). It would be great if you could test it.


@alexcjohnson could you comment on this?


if you can Avoid using .FindWithTag, it's really high cost and triggers lots og garbage collection. especially if there are a lot of minions on screen it can add up pretty quick. If you just want it to attack a certain thing, set a public game object and just drag the target script in from the editor, from there we can work with oncollisionenter or ontriggerenter to change the target if something else need to be minioned.

Also, avoid Instantiate as much as possible. Better to initialize an array at startup and populate it with a few of your projectiles, then simply set them to inactive and reset their position. That will save us a lot of overhead when things scale up.


did you pull in my changes before committing ?
I removed the .yml files from git and added .yml.template files so we can each use our own config


Obrigado pelo aviso!... Mas já tinha corrigido
Verifica agora, já fiz o pull request.


I think you can just do `arr.map` instead of '[].map.call(arr, `


Any particular reason why you think we might need this? This is not how we want passives to be handled, even if the passive is as simple as "Has low gravity" or "Knife only", it's a good idea to inform the hero buyer of it via a passive skill.


TeamCity Dirigo Edge :: Pull Requests :: Pull Request Smoke Tests [Build 149](http://teamcity.corp.dirigodev.com/viewLog.html?buildId=1774&buildTypeId=DirigoEdge_PullRequestFullChain) outcome was **SUCCESS**
Summary: Tests passed: 6 Build time: 00:01:19


line 25, 26 has problem, needs to be:


@eoghan2t9 .. this fixed it.. lol told u i have missed some silly shit somewhere :p


Isn't it enough to load just the binary files without documentation and source files? I.e. `https\://services.gradle.org/distributions/gradle-2.9-bin.zip`?


You're missing parenthesis


TeamCity Dirigo Edge :: Pull Requests :: Pull Request Smoke Tests [Build 150](http://teamcity.corp.dirigodev.com/viewLog.html?buildId=1785&buildTypeId=DirigoEdge_PullRequestFullChain) outcome was **SUCCESS**
Summary: Tests passed: 6 Build time: 00:01:08


Instead of defining this function here, you could define it at the top-level:
```
function redisDel(hash) {
  return new Promise(function(resolve, reject) {
    client.del(hash, function(err, result) {
      if (err) {
        reject(err);
      } else {
        resolve();
      }
    });
  });
}
```

And then use it always in place of `client.del`.

We can do this in a follow-up.


@mflorea Please check out https://github.com/msmeria/xwiki-platform/commit/32100c4c4b3433973609b9ff695af3010b23a11f


@darokel :( I can't see where my error is


@allcaps It seems that you had this in progress and everything else crashed. I've put it into the fridge to make it work for update the staging site. Sorry for the inconvenience!


Danke! Du hast meinen kleinen Ruf nach Hilfe gehört :)


Just break this line like the previous format ... the previous one looks more readable.


I agree. "/proc/self" is better than "/proc/" + getpid() +...


Will fix


Ok, I made a pull request with this changes but we still have a problem. Apparently the server doesn't know how to to deserialize the body when the quotes from the serialized dictionary are escaped. For example:
{ "variables" : { "variable1" : "value1" } } is  GOOD
{ "variables" : "{ \"variable1\" : \"value1\" }" } is BAD

Now, if you use
```request.AddParameter("application/json", <#= param.SelectToken("name").Value<string>() #>, ParameterType.RequestBody); ```
to generate the template, it will force the user to send you a dictionary which contains only one key (the name of the parameter. For example "variables") and the value of the key will be a serialized dictionary with the variables.

Now I believe that it is better to make this part transparent from someone which uses the API. He should call the method directly with a dictionary which contains the variables/values. The called method should automatically add the name of the parameter and generate the body object of the request.

Replacing that line with
```request.AddParameter("application/json", new { <#= param.SelectToken("name").Value<string>() #> = <#= param.SelectToken("name").Value<string>() #> }, ParameterType.RequestBody);```
will easily fix the problem.

I will send you a pull request. I hope that you agree with the changes.


Thanks, that did it.


This concatenation is too fancy, and Message() could have side effects.
All you need is "/proc/self/stat"


Pois, agora o erro já não é esse


so yes, indeed, you must do one the following:
* create the real objects directly instead of templates (but it doesn't work well with multi-company setups)
* alter the current behavior to avoid calling try_loading_for_current_company() (commenting the yaml file in the __openerp__.py for example) or overwrite this method (to skip its code)...

Anyway, I don't see how it could be seen as a bug that could be solved by an OPW: there is no garantee that a new version doesn't break the custom modules of the community, since we aren't aware of every of them :-s Sorry about that


typo


Orra nem pra mudar os exemplos da documentação???

db products??

NENHUM MOMENTO USAMOS ELA NO CURSO!


**anf** free accounts? lol


@gperciva this works -- hadn't heard of that one, but I can put that line in README in addition to the others perhaps e.g. Try this - but if that doesn't work, here are the manual commands to troubleshoot the configuration?


`request.url.gsub(request.base_url, '')` `/\/(en||cy)\/categories\/*/.match(@slug)`. It's not an error but we use parenthesis `()` after method calls.


I guess this was supposed to be SkyNew**s**ArabiaBaseIE.


Does this work? I had problems with `import Wit` part.

I can although `import wit` and work on it.


is about the iter?


When I saw this, I thought @syl20bnr enabled it while committing :-) I [seems that not](https://github.com/syl20bnr/spacemacs/blob/develop/core/core-keybindings.el#L26-L29)


Yes.


there is an error with code


@varlesh https://github.com/hluk/CopyQ
Copyq

Icon names:
copyq.svg
copyq-busy.svg
copyq-normal.svg

![1](https://i.imgur.com/r2IHYXy.png)
![2](https://i.imgur.com/wP8iYkf.png)


complex method fixed https://codeclimate.com/github/crossroads/api.goodcity/Warden::Strategies::PinStrategy


thanks @bportnoy  any sample code in JS?


> A line must not end with white space.

Both "end" and "trailing" are saying the same thing, so there could be doubt they really do mean the same thing. Best just say it once.

So definitions are:

* BLANK LINE: a line containing no characters; a zero length line. It is still a line, and so needs to be terminated by a LF, which should be covered in the definition of a line.
* WHITE SPACE: in this context, just tabs an spaces?


Opá dread se há erros então faz pull requests.


Vou ver se aprendo a fazer disso então.


bn


using parse_iso8601 return null.


:P Ainda bem. Primeiro tens de fazer merge do meu repositório do github com o teu. Depois alteras no teu repositório, e depois vens ao meu repositório e tens de pressionar o botão verde acho eu.


I am setting this to NaN as discussed. The overshoot is not defined since the condition is never met.


tryhyhyj


Glad to know I'm not losing my mind!

I'm not certain, but at a glance, it looks like they're *reporting* the old chunky border sizes from Aero under Vista/7, but *rendering* using Win 10's thinner borders.  This leaves a good chunk of transparency (or blackness, during screen capture) between the two dimensions, and I have no idea how to retrieve the actual "Win 10" dimensions.


Added a reference to the example


@teryk since you have the hand on the branch, could you change the javadoc of this method to
```
    * Calls the processing code for a specific {@link CeTask} which will optionally return a {@link CeTaskResult} holding information to be persisted in the processing history of the Compute Engine (currently the {@code CE_ACTIVITY} table).
```


I commented out the gui-ret case, because I still don't understand what the problem was with #4036 and I thought it might have had something to do with messing with `<return>`


yes


Merged the commits. The are united. :one:


Ok, fajnie


@darokel thanks. I will work on it


typo! `call` is twice, remove one of them!

you can also remove `-if-exists` as it will always exist.


====================================================================================
def hotel_cost(nights):
    return 140*nights

def plane_ride_cost(city):
    if city == "Charlotte":
        return 183
    elif city == "Tampa":
        return 220
    elif city == "Pittsburgh":
        return 222
    elif city == "Los Angeles":
        return 475
    else:
        return False

def rental_car_cost(days):
       rent = 40 * days
       if days >= 7:
          rent -= 50
       elif days >= 3:
          rent -= 20
       return rent

def  trip_cost(city,days,spending_money):
    return rental_car_cost(days) + plane_ride_cost(city) + hotel_cost(days) + spending_money

""" print trip_cost("trip cost to %d for %d with an extra %d dollars of spanding money") %("Los Angeles",5 days , 600) """

print trip_cost("Los Angeles", 5, 600)


Is the ignore necessary? I guess if you don't ignore it, it is passed along the line of responsibility and ignored somewhere else. This should be preferred.


Tracked as [JENKINS-31903](https://issues.jenkins-ci.org/browse/JENKINS-31903)


Just to put another spanner in the works, does it need to specify that this only concerns lines of *code*? If I escape out of PHP ?> then put some HTML in <?php (putting aside the evils of doing so) then is there any reason why trailing spaces should not be allowed in the HTML, or JS or CSV or whatever is in that section?

How about static strings that span multiple lines (evil again, but often seen) - are they allowed in the first place, and if so, should trailing spaces be disallowed in the string content?


After it's out of WIP, yeah. I think we thought it was all rather repetitive and wondered if it was meant to be that way. :sunny:


Why not pipe it to tar anymore?


Скачиваешь архив, а он называется "panda-master". Креативно :)


@zachpanz88 there's another occurrence of swift.org in the README (line 165), it would be good to linkify that also.


ım a bit confused.. but in my opinion we need to edit all the needed libs..
ı disassembled the several cam libs by using IDA
and here are the loaded libs by libseccamadaptor.so:

.plt:0000C0E8 ; Needed Library 'libui.so'  ******
.plt:0000C0E8 ; Needed Library 'libutils.so' ******
.plt:0000C0E8 ; Needed Library 'liblog.so'  ******
.plt:0000C0E8 ; Needed Library 'libcutils.so' ******
.plt:0000C0E8 ; Needed Library 'libcamera.so'
.plt:0000C0E8 ; Needed Library 'libarccamera.so'
.plt:0000C0E8 ; Needed Library 'libPanoraMax3.so'
.plt:0000C0E8 ; Needed Library 'libActionShot.so'
.plt:0000C0E8 ; Needed Library 'libcaps.so'
.plt:0000C0E8 ; Needed Library 'libbinder.so' ******
.plt:0000C0E8 ; Needed Library 'libmedia.so' ******
.plt:0000C0E8 ; Needed Library 'libjpeg.so'
.plt:0000C0E8 ; Needed Library 'libcamera_client.so'
.plt:0000C0E8 ; Needed Library 'libexifa.so'
.plt:0000C0E8 ; Needed Library 'libjpega.so'
.plt:0000C0E8 ; Needed Library 'libsecjpeginterface.so'
.plt:0000C0E8 ; Needed Library 'libsecjpegboard.so'
.plt:0000C0E8 ; Needed Library 'libsecjpegarcsoft.so'
.plt:0000C0E8 ; Needed Library 'libc.so'
.plt:0000C0E8 ; Needed Library 'libstdc++.so'
.plt:0000C0E8 ; Needed Library 'libm.so'
.plt:0000C0E8 ; Shared Name 'libseccameraadaptor.so'

ı marked the libs that need to be removed with ****
and then we will add string ........linshims.so.....
those removed libs will be loaded through our "libshims.so" because we statically linked these libs https://github.com/RR-msm7x30/android_device_samsung_ancora/commit/50ad00ed0fd92b7b7bd080e797f0733bd6aa6fb1#diff-435a8df441f6274c15d887bc8a051dcbR14


yes so much cooler than `and` :smiley:


i didn't have python 2.6 in the system,but i will see in the docs what is the equivalent for it.


On December 4, 2015 1:42:53 AM EST, "Alton(ius) Blom" <notifications@github.com> wrote:
>The reason to call out SHA-224 is to align with the NIST recomendations
>on the keylength site where it only refers to SHA altorithms, no other
>hashing algorithm is mentioned / endorsed.   This then means that we
>don't require a specific length with password hashing, therfore
>allowing bcrypt at 184 bits.  A bit of a weasel way of doing it, but as
>you identified trying to not contradict our own criteria.  An
>alternative could be to pull hashing fromthat criteria and merge
>hashing and password hashing into the same criterion.  Let me know your
>thoughts.
>
>Looks like the git pull request didnt quite do what i expected (was
>trying to edit locally then push, rather than just editting through the
>github website), ill close this pull request and resubmit without the
>"this is" typo and another error I can see.
>
>---
>Reply to this email directly or view it on GitHub:
>https://github.com/altonius/cii-best-practices-badge/commit/c889ee3b5451af0235a32d8bc65a01f416666900#commitcomment-14780407

I don't want to mandate specific crypto algorithms... one may get broken.  It's probably best to just clearly state that this does not cover password hashes, and handle that with the criterion that focuses on that.
--- David A.Wheeler


Hmm good


I thought it was missing, did not see that the function was commented and when I first review it I remember that it was present, I was tricked by the stealth rebase :-D


:relaxed: nice catch, thanks


I think you should also add the date here.


/**
* @file Human.java
* @author Aled...
* @date 04 Dec 2015
* @see Game.java, Board.java, and Scoreboard.java for related information.
*
* Handles player actions.
*/


You joined 3 different commits here and mixes style with functional changes. If you need to revert functional changes by any reason you end up losing style fixes


yes i will fix it.
sorry i made a lot of errors.


... @@ -0,0 +1,17 @@
  1  +# Auto detect text files and perform LF normalization
  2  +* text=auto
  3  +
  4  +# Custom for Visual Studio
  5  +*.cs     diff=csharp
  6  +
  7  +# Standard to msysgit
  8  +*.doc	 diff=astextplain
  9  +*.DOC	 diff=astextplain
  10  +*.docx diff=astextplain
  11  +*.DOCX diff=astextplain
  12  +*.dot  diff=astextplain
  13  +*.DOT  diff=astextplain
  14  +*.pdf  diff=astextplain
  15  +*.PDF	 diff=astextplain
  16  +*.rtf	 diff=astextplain
  17  +*.RTF	 diff=astextplain


This commit broke node-minify for me, but @NareshBanala 's getPath function has fixed it :+1:


par_sync. linha 77: pthread_mutex_destroy(&exit_called_mutex);

Andei à procura pelos outros ficheiros dum sitio onde esse mutex estivesse a ser usado e não o encontrei, eliminaste-o?


its "find_embed" function


This was fixed in GCC 4.4. Since we only support 4.6 and higher, there is no longer any reason to check for this.


It's ok


changed to 9.3.0


Could also then only unlock such monster parts for player use when they have killed (X number of?) the appropriate monster...


Hi bro , please can you tell me the language used in this code ,thanx :)


нужно добавить папку .idea в  файл .gitignore (т.е. не коммитить ее никогда)
на занятии разберем как это сделать


@MyrionPhoenixmoon The build fails now because of a console.log statement: http://54.88.202.24/jenkins/job/scenarioo-fork-mockups/270/console

Could you please remove it?


Sim eliminei, é um erro.

Nota: O par_sync é  bastante independente, só depende de list.h por causa do tipo list_t e só depende de par_wait.h porque tem de saber que função de wait chamar. Portanto não poderia estar noutro sítio. Qualquer coisa de concorrência tipo mutex ou semáforo só aparece no par_sync.


I'll get a build together here. Just been swamped with my paying day job but being forced to take some time off before the end of the year.


Certo.


Done. Thank you !


This should be:

    Password sent to {email}

since the returned message to the admin flash area shows the new author's email address, not their name.


I don't like this text being server-side


очень просится использование какого-то класса на элементе div


for experiment


html書きたくない


TeamCity Habitat :: Cruise [Build 324](http://build8ua1:8081/viewLog.html?buildId=800&buildTypeId=Habitat_Cruise) outcome was **SUCCESS**
Summary: Tests passed: 106 Build time: 00:01:19


yes. at one point i was toying around with the idea of avoiding changes to certain meta namespaces like `k8s.mesosphere.io`. given the current state of things this func can probably be eliminated in favor of a simple copy.


good catch


hahaha


@nvkelso will you be taking these back to skin and bones?


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

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


What does this mean? :)


Good spot, thanks! fixed in 8eab2b8


The good part is that the cure is actually serializable!!!! :D


`Size` seems not there (correct me if I'm wrong) And, are you suggesting any place which has referenced Image should change to use this DockerImageSpec?


I opened #11 for this.


ok


Can I just personally request that we get a description that makes some sense? :)


closes #328


What's the magic number?


Очень нужная доработка, спасибо.


Enjoy my messy code! :D


@sns-seb ok


:derp:


аналогично с  элементом nav - в проекте такие элементы могут встречаться чаще, чем один раз, а застилены при этом могут быть по-разному


@leninkp this branch also needs rebasing. we have new environments


RTFM, it should supress cmake warning


Thanks :P


Welcome, swift


касаемо форматирования - в одном блоке свойства должны идти на одном уровне, плз, используйте автоформат


Look like it's working but will crash if entering full screen video rendering like in youtube, browser...
![image](https://cloud.githubusercontent.com/assets/8336751/11592864/eed9f0b0-9a55-11e5-8d72-579912f27dc4.png)

this is the crash log I got


--amend -author 'martinusbe' :P


could this go in a cookie ?


wew


@macool making rebasing..


remove this TODO. add a comment along the lines of:
(a) pod update callbacks are racing with potential user updates to the pod, and Mesos the LaunchTask() callback, and that we can't tell if that's what is happening yet;
(b) the scheduler has its own reconciliation mechanisms to deal with apiserver referring to tasks that it doesn't recognize


Why did you remove this method from interface?


lol


"Confirm action is not too bad"...


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


Sorry, both 32 and 64 bits compilations are broken following this commit.
Additionally, 32 is broken due to Lame, since a long time. I miss it.
Thanks.


@mpeeters can I ask if you remember why?  I wonder if this not being done is causing 2.x tests to fail?


Please use the @user notation to be compatible with GitHub's commit message system.
Noob.


опечатка detected :)


Because it is quite useless method.
There is only one places where we have used this metod, so I reimplement this place and then remove method


Lucas Cerri


Huh, I didn't have any problems. Can you post the compile.log somewhere?


what?


Would it be of any use if it wasn't?


silly q: can these authenticated routes be cleaned up with rr's onEnter hook?

https://stackoverflow.com/questions/33794230/how-to-use-react-routers-redirect-function/33796304


This change seems completely unrelated to the purpose of this commit


This doesn't work when elasticsearch is configured with the shield plugin, as it will respond with 401 unauthorized. You should probably just check for any valid http response.


fghf


Must try combien css/ minify css and js.

We have a http://nlacsoft.net/nlsclientscript/merging-js excludePattern system. Then we can surely deactivate merging for some file (less used ) but put it in assets . Great :).


This sounds great, but as you say maybe needs a bit of tweaking. Can sound a bit scratchy compared with :reverb. Tried it round four live_loops each using different synths :tri,:saw,:fm and :prophet on iMac. :saw  and :propeht were very slightly juddery. Could just be lack of resources, but :reverb with room: 0.8 sounds fine.
:saw loop by itself was better.


In IntelliJ, when you load the sources as well, you get code completion and documentation while editing the build script. This is worth the effort, from my point of view.


<a href="images/eagle.jpg"

ссылка здесь должна вести на страницу, а не на картинку. по-умолчанию, во время верстки вешаем  атрибут href="#"


Okay, sorry about that. Here is the last and final version :
Resolves @BinaryBrain 's lies.


either it's public, in which case we shouldn't reference our internal jenkins, or it's private and you can do whatever you want, but this is a bit weird... WDYT @ebuzzing/platform ?


One more change and it is ready!

export BOOTREC_EVENT="/dev/input/event**5**"
export BOOTREC_EVENT_NODE="/dev/input/event**5** c 13 6**9**"


osx too http://buildbot.e.ip.saba.us:8010/builders/build_osx10.9-x64/builds/384/steps/shell_2/logs/stdio
```
WARNING: Caught an exception in the keymap:
ERROR: InterruptException:
WARNING: Caught an exception in the keymap:fatal: error thrown and no exception handler available.
Base.AssertionError(msg="shift!(Workqueue).state == :queued")
rec_backtrace at /Users/osx/buildbot/slave/build_osx10_9-x64/build/src/task.c:651
wait at ./task.jl:359
task_done_hook at task.jl:173
jl_apply at /Users/osx/buildbot/slave/build_osx10_9-x64/build/src/./julia.h:1339
jl_apply at /Users/osx/buildbot/slave/build_osx10_9-x64/build/src/task.c:221
Worker 3 terminated.	From worker 3:	     * repl
ERROR (unhandled task failure): EOFError: read end of file
```


EconomizerIntegrationNew (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1977 of 1977 tests passed)

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


If I use a standard exception, where to store the action ?
Moreover, how to identify my Exception with this action and the classical Exceptional ?


--author* ?


Yep


git commit --amend --author=martinusbe
git push --force orion HEAD:mm6.0

meanwhile sri beech is like, lets keep the kanged commit..
git pull && git push


Actually, I overthought this section a bit. The task-id annotation won't be present on the pod unless the pod has already completed the launch-task-binding phase. There still could be a race between the overall launch-task process and this update, but here we have absolutely **no** way to tell the diff. The best we can do is reschedule the update for later, backing off between attempts, until we exceed `launchGracePeriod` (if it's non-zero, otherwise we re-try forever?! don't like that either)


i don't think the visible condition is needed?
since this view is only used in the videos window it will always return true.


Deriving?


same here.


Some of them:

* Really scrolls to bottom when using chrome!
* Last message no longer hides under input box
* Usercard image is centered. (Not really)
* Usercard name is also centered.


By a miss, do you mean it not flagging when there are no matches?

On the rest: sounds good; I can do that and the logging.


Changes:

* Mobile chat now has message bubbles.
* Mobile chat now looks more like computer chat.
Added a padding for chat under the usercard


Done!
Just for my knowledge, how these parameters value can be found ?


:+1:  @shivanibhanwal


@matthewp
> Does IE8 have qSA?

Yes. But only on the types of CSS selectors it supports. CSS 2.1 selectors should be fine.


Changed:

Basics for mobile chat done, CSS fixes:


Вот это нехорошо. Ты же тестируешь собственно результаты работы метода `MovieList.parse_json` — зачем это замаскировано дополнительным вспомогательным методом? Чтобы много раз не повторять путь к JSON-файлу, его можно было в отдельный let вынести.


Definitely neater, thanks :-)


I will review them and let you know. Thanks


i live for neat :+1:


Request:
Let's please define these as named constants.


why is this a `remote: true`?


Тут, конечно, в сумме довольно слабые тесты.
а) ты не проверил, что он хотя бы title фильмов (не говоря уже о всех полях) из файла правильно считал. А то может твой метод создал 250 пустых Movie? или 250 с одинаковым тайтлом? (вполне реальная ошибка, по крайней мере вторая)
б) а что будет, если файл есть, и в нём корректный JSON, но там совершенно, абсолютно не те данные, которые ожидает этот метод? То есть там вообще посты какого-нибудь блога например, или там ответ какого-нибудь API instagram?


:+1:

Also check out https://github.com/glennr/react-redux-universal-hot-example/commit/908f209793745d00d752b9f6a6f3d52ac13c9a7a for an example of using RR's onEnter hook + location.state.nextPathname.

TL;DR - RR will do the heavy lifting of tracking the auth redirect state in getState().router.location.state.nextPathname. Its a tad cleaner than using a query string. That said, don't ask me how to do SSR + auth redirects on the server without a query string. I don't have a nice answer for that yet.

But I digress, my point is that the new RR auth stuff *might* clean up your auth-related code a bit.


not need


Well it is also compatible with the Seasons and Episode view, the right container is a bit smaller in height. [See this screenshot here](http://i.imgur.com/CoGGsnv.png)


Updated for Bootstrap 3.3.6 (javascript libraries)
Better packaging


LGTM


Thanks a lot!!


Fixed


Reverted on 4.4.


- "create real objects" works for adding a missing tax, but it is less practical for changes / deletions.
- commenting out code in odoo is something we don't do of course... so I guess the way to go is, as I said, a custom module that overrides the method `try_loading_for_current_company` to do nothing.

As for the OPW, it is debatable: we are not talking about custom modules, but about the fact that we always script the steps needed to setup a new instance, including financial setup. Customizations on the account chart are common.

Thanks for your time @qdp-odoo!


Thanks!!


Yeah, good point. I don't think we have a good answer to keeping these in sync, except "be careful when changing the feature spec name/structure".


why did you delete this function?


Possibly related to this: http://serverfault.com/questions/551466/couchdb-attachment-hashes-dont-match-attachment-content

Summary: couch may be calculating the md5 hash on compressed data. The compression algorithm may be different depending on the scenario (snappy for documents??, zlib for attachments, none in some circumstances).

Conclusion: this seems like a bug in couch since it does not appear to be well documented that couchdb is calculating the md5 on compressed data. Also, I believe it is not compliant with the [HTTP spec for calculating Content-MD5 headers](http://www.w3.org/Protocols/rfc2616/rfc2616-sec14.html#sec14.15).

I think I'll stick with the algorithm I'm using here (`"md5-" + b64encode(md5(content).digest())`).


Merging new changes from Richard Yen.


Missing space before `/`


nevermind, just spotted you're using redux-simple-router, so dont have the router state inside your redux store. I see now. :+1:


Jgjf


good catch, I'll fix :)


Tweaking is just a matter of changing default params not the sound - that's all internal inside the ugen. If there are better starting params then do let me know...


Test


Why `450px`? I don't recall this breakpoint.


Two Questions about the fix:

1. Milliseconds makes more sense, but how does it 'fix' the issue?
2. What does the 'and 1 or 0' do?

I'll give it a try with a local build in a moment.


```
var button = $(this),
      img = $(),
      like_count = $();
```
って感じでvarを省略できます。


Do we need to keep a `dbUri` and a `baseUri` or can this `DatabaseURIHelper` just set the `baseUri` on the super to be what it needs?


what for are these brackets?


Can we add a test to confirm that (non-empty) titles / seo_titles have leading / trailing whitespace stripped, as per the Django 1.9 behaviour?


if I understand correctly:
1. rss_alert and rss_item_alert are excluded somewhere on libtorrent.i, so we have N-2 alerts.
2. however `libtorrent.num_alert_types` is a libtorrent constant that does take them into consideration.

is it possible to fix the value of `libtorrent.num_alert_types` instead in C++-land so it reflects the reality of our codebase? or impossible given the structure of the C++ project?


'windows' and 'content' are two different things ;-)
seasons, episodes, movies,.. are all displayed in the videos window.


```
>>> BrokenPipeError({}).args[0] == {}
True
```

I don't think you need to identify your exception. Maybe just check that args[0] is an action


I believe this is because it uses binder fustions and so ida is saying its needed but if you look at the hex only the one lib is loading binder and thats because only the one needs to load it.


This seems like it might be an internal thing, is it used publically?


You don't have any problems dispatching this action at this point? I got turkey slapped by this the other week, and I have an irrational fear of it now.


I understand this is just for testing, but it feels like it could bite us in the butt.


OK, so just to confirm, this then passes an object for JSON serialization which is

```new { varname, value }```

E.g. { variables = variables }
which would serialize down to

"{ \"variables\": \"{ .. encoded object here.. }\" }"

or something like that.


Now that I see what the AuthenticatedComponent is doing, I dig.


This is auto generated and specifically tailored to take advantage of compiler optimization to make the lookup in O(1).


I updated the code snippet for verifying digests above. Notice that the Content-MD5 header is the same as the internal digest stored by couch, and this differs from the python-calculated content md5. I'd be happy to be corrected on how to calculate content md5 if I'm doing it wrong.


What is "I know it?" if yes and there is no answer then there is bag in your questions.


Why is this header needed?


About json: That's great. I just picked the first json module I found and did not check any further. If 'json' can dump more human readable, I would go for it too, accepted! But please remove the commented old line.

About DM dump: accepted!

About quiet/verbose: I like variables to be initialized. NOT ACCEPTED.

About url_download: There shouldn't be a case, where url_download is missing. AFAIK it is only missing for GDoc/GSheet and these are skipped some lines above. Do you have an example of a file that caused that error? What's the 'filetype' attribute of it?

If you fix my issues with your code and recreate a pull request from the, I will accept it. If you have further questions, feel free to ask here or via Hangouts. ;-)


できなかったですか、、、バージョン的な問題かな、おけです


fix


Ah, that solved my previous comment on your last commit.


It's not something we wrote, I believe it's from [here](https://github.com/inexorabletash/polyfill/blob/master/typedarray.js) but not entirely sure.

We recommend feature detection over conditional comments, but as far as I know IE9 is the only browser that needs this (yet supports the other things we need) so it *should* work fine.


Ahh ... I added it because it was used in the checkpoint but when I go back
and review that, it wouldn't be needed because there is no form involved
with the voting portion.  Shouldn't be in there.

On Fri, Dec 4, 2015 at 10:23 AM, bjblock <notifications@github.com> wrote:

> why is this a remote: true?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/aaronkoz74/bloccit/commit/23abcdc022147415a80fc0834e5c32a886f57c33#commitcomment-14789566>
> .
>


Fehlt hier nicht ein **Z** ?


Yeah.

I grabbed that from another blog post on Elm so I understood it on the surface when I wrote that code.

Going back after some time I realized that I didn't really understand it so I want to make sure I really grok what it is doing.

I'm guessing this will be a thing for a while since I'm not using Elm 8 hours a day and only have so much brain space for stuff that I don't use every day. :-)


I discussed yesterday the issue if `num_alert_types` with arvidn (https://github.com/arvidn/libtorrent/pull/306) and I added a comment for the correct semantic of this constant. Btw, `rss_alert` and `rss_item_alert` are not there because all the rss framework inside libtorrent is deprecated.


Daz, could you share what value you see in numbering the directories this way, and what doing so actually means? Implementation priority, importance to user, other?


Why not '&' ?


It is not (for testing). Actually in the process I discovered a couple of missing items in the "manually" created code.


I think I've done this now (2f6a816), but it's not clear to me how the new contributor layout works.  I see a ``profile.html`` in the diazo theme, but no rules in the the diazo configuration that make use of it.


Yes, I realized it after I wrote the email  it was too late yesterday...

Am 04.12.15 15:05 schrieb "webepasc" unter <notifications@github.com>:

> The universities are initialized into an array in a numbered way. One can just
> access the position in the array to access the university by Id
>
>
> Reply to this email directly or view it on GitHub
> <https://github.com/webepasc/Migration-in-Scientific-Communities/commit/4f3053
> a341be9eb9b3eaaa9c40adf6cc4c94f209#commitcomment-14787612> .
>
>
>


TeamCity Habitat :: Cruise [Build 325](http://build8ua1:8081/viewLog.html?buildId=801&buildTypeId=Habitat_Cruise) outcome was **SUCCESS**
Summary: Tests passed: 109 Build time: 00:01:20


I feel like the public methods all ought to return `DatabaseURIHelper` so that the fluent style is preserved.
Otherwise it could be like:
`dbURIHelper.revId("123").encodePath("") //now I can't call build`
I guess that means some things would need to be static utilities, or maybe another class that uses the builder.

I think what I'm saying is there should probably be a distinction between a `DatabaseUriBuilder` that supports the fluent style build chain and a `DatabaseUriHelper` that doesn't.


With these changes it will not send messages to inactive users but it will fail to send them with a success message. :ribbon:


Oh ok, I see. I actually copied that over from one of the existing views. Will remove it.


in setting transAtomList = atomList, a single memory point becomes shared by the two lists. In order to solve this I did a line-by-line copy of atomList to transAtomList, separating their interdependency.


I think that @allcaps is right now in the middle of styling it. Maybe you can set the bases (just basic styling) of the badge display and then allow Coen and the others finish the styling.

Coen, can you confirm it's like I've said or tell Alec how it's supposed to work?

Thanks!


This makes it sound the limit is 250 chars, but it's 200, right?


should probably be protected as we are expecting sub-classes, unless there is a good reason to restrict the sub-classes to the package.


Boost regex will be dead-code after this change. Probably, we may want to remove/delete boost-regex code thn?

Leaving it this way has one potentialy plus that it regex code is still there and can be resurrected easily. But don't think we will return to it if PCRE serves our needs.


wouldn't the compiler substitute the named constant for the number anyway?

if somewhere:
```java
public final static int TORRENT_ALERT = 0;
```

then
```java
case TORRENT_ALERT:
   torrent((TorrentAlert) a);
   break
```

wouldn't the bytecode have the 0 value in there?


All changes to the standard library need to be reviewed by @gribozavr and @dabrahams. Please make sure to add their names to one of the commits so that they are notified.


again should probably be protected


Added support of csgo.tm websockets to process items one by one.


Emil put a prameter in the index method. The parameter says for which month to index the meetings. Any suggestions how to refactor this and make it restful? Because the idea I guess is that since the calendar is monthly the user would choose a month and get all the meetings in that month.

    On Thursday, December 3, 2015 6:45 PM, claudio cardinale <notifications@github.com> wrote:

 @Mihica Why? this is not restful—
Reply to this email directly or view it on GitHub.


$networkSummaries must be $network_summaries (Drupal convention)


have you seen the laravel log?


I guess for the sake of purity we could skip suggesting `mvn` is default value since it will be always overridden by constructor taking `mavenExecutable` parameter.

It would be different if we had non-parameterized constructor.


Avoid injecting `Injector` if possible. If you need this for PermissionOracle, then you should inject PermissionOracle instead.


Ich gehe stark davon aus, dass mit **Nightly** hier idr. eine Daily Entwicklerversion gemeint ist. In dem Fall sollten wir den String bei **Nightly** belassen. Siehe https://de.wikipedia.org/wiki/Nightly_Build


True, but since this is auto generated, and I need to generate the cases anyway to guarantee the sync with libtorrent, how having the constants help?


develop (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1439 of 1439 tests passed)

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


looks fine!


Makes sense, will fix and push
On Dec 4, 2015 3:29 PM, "Lennart Poettering" <notifications@github.com>
wrote:

> The DNSSEC RFCs suggest to only accept verified RRsets in the cache, so
> yeah, my intention was to do the validation before adding anything to the
> cache, and suppress everything that doesn't validate...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/teg/systemd/commit/88db597031ef962272b83c92119ae0e521a23633#commitcomment-14788278>
> .
>


Will do
On Dec 4, 2015 3:26 PM, "Lennart Poettering" <notifications@github.com>
wrote:

> Also, the checks at the beginning of dns_cache_put_negative() that filter
> out ANY submissions should probably carry a comment about this too, after
> all we really need to filter them out when we want to use ANY for NXDOMAIN
> purpose...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/teg/systemd/commit/0ed50dd147ce8807c6f37e5f0b95fad5bd586e4b#commitcomment-14788206>
> .
>


We normally put this on one line.


Injector comes from inherited classes, I've reused it to not adopt all those classes.


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


API czy aplikacji ? Piszemy własne API ?

On Thursday, 3 December 2015, Maciej Jakubowski <notifications@github.com>
wrote:

> Pytanie czy w naszymi API dajemy możliwość dodawania klientów czy samych
> filmów ? A może i tego i tego ? Żeby można było dodać nowy film do bazy i
> nowego klienta. Tak chyba będzie najrozsądniej - bo zakładamy, że z Api
> korzystałby właściciel wypożyczalni, który jednak ma swój lokal - więc jak
> ktoś przyjdzie musi mieć możliwość założenia mu konta na miejscu.
> Więc w mojej ocenie trzeba zrobić 2 zakładki - zarządzaj klientami i
> zarządzaj filmami.
> Dopiszę to do oczekiwań APP.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ISK-2015/projekt-zespolowy/commit/8888cd629b4641c78f24f578eb2b9b20ca2e1227#commitcomment-14771917>
> .
>


Indenting of this method seems wrong.


Dieser String fehlt noch, nur dass er nicht hinten runter fällt ;)


stat comes from the `Environment` instance which I'm trying to not make more things depend on :(

Also it looks like the 3.5.0 mtime implementation doesn't work since we are converting it to a time value ^^

```sh
$ ls -la
-rw-r--r--   1 richardschneeman  staff     3604 Oct 14 10:52 codetriage-logo-0cfe6038dec5f88690be415b08d31413a6351634cce2412b9433f8a8ef2ea307.svg
-rw-r--r--   1 richardschneeman  staff     1466 Dec  3 11:52 codetriage-logo-0cfe6038dec5f88690be415b08d31413a6351634cce2412b9433f8a8ef2ea307.svg.gz
```


@bajtos : Your blessings, please!


byte


Thanks. Working on a fix for this and other cards with the same effect.


came here to read the comments


if you want to do this automaticlaly laravel makes this for you via dot notation  take a look to http://laravel.com/docs/5.1/controllers#restful-resource-controllers (Nested Resources). It is useful test routes via `php artisan route:list`

But in my opinion the months split is just a frontend thing, it's not an our problem.
The meeting should be something with a duration and some timeslots available


А вот тут — по сравнению с предыдущей группой тестов — излишне подробное тестирование. Функциональность-то простая, она либо работает либо нет, так что тестировать отдельно что а) их пять; б) это пять самых длинных; в) они отсортированы так-то; г) они действительно из списка, а не с потолка — это очевидно избыточно (в реальности, тесты такой гранулярности ты писать поленишься).

На самом деле, это обычно решается просто вот так:
1. составляем «образец» (независимым кодом выбираем 5 самых длинных фильмов и правильно их сортируем)
2. сравниваем с реальной выдачей метода


We normally put a space between `)` and `{`


Thanks!


See previous comment.


why not make this `_id`?


the problem is that you use groupId even if you are not able to retrieve this (on the routes you don't specify this)


Einrückung nicht konsistent! Bitte fixen bis 04. Dez. 2015 - 17:00 Uhr!


adb shell getevent

press the volume key for the event, the last number is 64+event


on my commit I specified this as TODO


Welcome swift!

![Swift](https://i.ytimg.com/vi/AydzoleHnho/hqdefault.jpg)


Could you move this above the function body and rename it to `ANIMATION_DIRECTIONS`?


Кстати, вот такая задача делается на самом деле созданием custom rspec matcher (это удивительно несложно), чтобы можно было написать так:

```ruby
expect(five_longest).to be_sorted_by{|m| -m.time}
# а лучше даже так:
expect(five_longest).to be_reverse_sorted_by(&:time)
```


I cloned your repo to have a look at this nice new tool, but:

Preparing index.html

Preparing CHANNELS:
Exporting channel jungschluempfe
Traceback (most recent call last):
  File "../json2html.py", line 204, in <module>
    exportClass(channels, users, 'channel', 'channels')
  File "../json2html.py", line 156, in exportClass
    writeHTML(item, html)
  File "../json2html.py", line 23, in writeHTML
    f.write(data)
UnicodeEncodeError: 'ascii' codec can't encode character u'\xfc' in position 2399: ordinal not in range(128)

Additionally, maybe give the file a chmod +x, ;-)


Check `!output.cancelled` instead of comparing with `false`.


Antipessoal é adjetivo ligado ao substantivo mina. Concorda em gênero, número e grau. O correto é antipessoais.


I would just use `umount --lazy`.


the trailing `-` refers to prereleases and is probably not what you want.


Why would you set the push date yourself?


Why would you set the changesets yourself?


check this out re: localStorage vs cookies war.  https://github.com/erikras/react-redux-universal-hot-example/issues/608


The mp3 issue is being fixed.


use item.value for selection instead of item index


Like previous setter questions, why would you set this yourself?


Missing `jasmine-core`


Link do pliku z przykładową bazą danych (100k wspisów):
http://www.megawrzuta.pl/download/22dd0083f6c05c18877c31135caffed4.html


Touching the screen will already show the controls. With on screen buttons the way it works now is:
No controls/on screen navigation showed.
I tap the screen, both navigation and video controps show up.
No when I press back, it hides the controls again (which also happens when leaving the screen for 2 seconds)
The second press will actually make it go back.

Frankly I think the back button should make it go back to the previous screen. Since the controls hide automatically. That said, it'd be nice to have the controls hide if you press anywhere on the screen like the offical youtube app does.

Basically, have a look at the official youtube app's behavior :)


or something like that https://github.com/rg3/youtube-dl/blob/master/youtube_dl/extractor/udemy.py#L227


I would also like to avoid sending changes if the maps are equivalent.


I think it might be good if the changes to the `CouchDbClient` are isolated to their own commit. If you separate the the new URI building code and helpers from the swap to using them I think it could be easier to review.


make sure slider doesn't capture focus when setting up

goes with framework commit > https://github.com/adaptlearning/adapt_framework/commit/5e809b1201be2c7d540d6c83a50572ca73db1cd3

on framework pr > https://github.com/adaptlearning/adapt_framework/pull/831


This is mistake commit, will be removed later


Ah, btw: you should not imply that the checkout directory is also the data directory. So having a html/style.css might be ok, but only if you copy that file also to the data directory's html-dir when you mkdir it.


this is a function that takes as arguments two numbers and divides first with another


Okay, since this is a three-state enum, it is an unnecessary abstraction to pass it through number. Just use `'none' | 'left' | 'right'` for the state.


Why we have these things set when you instantiate `Push`?


Ok, in this case also downloading the sources is the better option.


I had to put the fix back or zimg could not be build (32bit) !


I don't like this fix. You've hide the problem behind the ifdef.


Strictly implementation order: as defined, each story builds on the last. When browsing, having these things ordered makes it easier to understand. I'd be open to other suggestions.


does this account for +- sign and the decimal dot?


So I got curious and made a test to see if it makes any difference to use named int constants vs int values.

Java code:
```java
public class SwitchTest {

    private static final int A = 0;
    private static final int B = 1;

    public static void doA() {
        int n=100;
        while (n > 0) {
            n--;
        }
    }

    public static void doB() {
        int n=0;
        while (n < 100) {
            n++;
        }
    }

    public static void switchWithConstants(int option) {
        switch (option) {
        case A:
            doA();
            break;
        case B:
            doB();
            break;
        default:
            doA();
        }
    }

    public static void switchWithNumbers(int option) {
        switch (option) {
        case 0:
            doA();
            break;
        case 1:
            doB();
            break;
        default:
            doA();
        }
    }
}
```

Resulting bytecode:
```
public class SwitchTest {
  public SwitchTest();
    Code:
       0: aload_0
       1: invokespecial #1                  // Method java/lang/Object."<init>":()V
       4: return

  public static void doA();
    Code:
       0: bipush        100
       2: istore_0
       3: iload_0
       4: ifle          13
       7: iinc          0, -1
      10: goto          3
      13: return

  public static void doB();
    Code:
       0: iconst_0
       1: istore_0
       2: iload_0
       3: bipush        100
       5: if_icmpge     14
       8: iinc          0, 1
      11: goto          2
      14: return

  public static void switchWithConstants(int);
    Code:
       0: iload_0
       1: lookupswitch  { // 2
                     0: 28 //indeed it replaces the value.
                     1: 34
               default: 40
          }
      28: invokestatic  #2                  // Method doA:()V
      31: goto          43
      34: invokestatic  #3                  // Method doB:()V
      37: goto          43
      40: invokestatic  #2                  // Method doA:()V
      43: return

  public static void switchWithNumbers(int);
    Code:
       0: iload_0
       1: lookupswitch  { // 2
                     0: 28
                     1: 34
               default: 40
          }
      28: invokestatic  #2                  // Method doA:()V
      31: goto          43
      34: invokestatic  #3                  // Method doB:()V
      37: goto          43
      40: invokestatic  #2                  // Method doA:()V
      43: return
}
```


Wouldn't it be better to intercept `SILBasicBlock::erase`? This is what we use most, when eraseFromParent is invoked.

And, BTW, remove is used even if we move instructions between BBs. Do we really want to get notifications even in this case?


We're okay with supporting the `0.4` pre-releases. (Especially since we haven't yet updated our own servers from a `0.4.0` pre-release.)


Its override in most cases because we use it in conjunction with ``autoReisze``. Restored false to make it  Good point, restored the the old value to make it consistent with all other modals.


Hey @bmuenzenmeyer, all the credit stuff still needs to be designed. I'll think about how best to approach it and will run it by you when I come up with something. I definitely want to give you the proper credit for all the hard work you've done, so when I redesign those areas I'll let you know.


I think it probably makes sense to have whoever is working on the new styling and layout get the badges looking correct in the theme.  Until the rules are wired up I can't really know how the styles will impact the view, but at this point the markup in ``contributors.pt`` should be sufficient for whatever styling needs to be applied.


We want to sync this version and the version in the app store. The app store version is still at `1.0`, so we can't just move to `2.1`. Our plan was to tag the old `master` as `2.0`, publish that to the app store, bump this version to `3.0` (or `2.1`) and publish again. The other widgets had a major version bump.


This is testing if ``getDual(x1)`` equals ``getDual(x1)``


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


このくらいの量ならまあ大丈夫だけど
各パート毎にちょこっとコメントが欲しいでござる


Sounds good Brad. I'd be honored to have my mug up there with you and Dave should you deem that appropriate.


That's weird. I compiled fine with a clean /build.
The guy did a git commit --amend so that's probably the problem.
Just remove zimg-git.


I also have another suggestion.
Why a switch statement? what happened to defining an `AlertType[]`?

In the case of the excluded alerts (rss) you can always fill those values with null.


minor indentation issues


Western should be added as well.


`.//` does not work for non-root in python 2.6.


I have to say this test is ugly. Not sure you want to refactor it now, but this test method should be split to a set of smaller focused test methods and the driver plumbing extracted. Anyway this impl is in line with the overall structure of the class and as such is OK, but it hurts.

We should consider this technical debt and come back soon if we want to support this driver for real and keep extending it.


TeamCity Habitat :: Cruise [Build 326](http://build8ua1:8081/viewLog.html?buildId=803&buildTypeId=Habitat_Cruise) outcome was **SUCCESS**
Summary: Tests passed: 109 Build time: 00:01:30


Hi @samdark
When will you release this as 2.0.5?


These 4 lines aren't necessary if you're already testing ``getDual(x2)`` and ``getDual(y2)`` above


Does `<T extends URIBaseMethods>` work? That could be better for ensuring the contract to get some of the fixed method implementations.