When you say none of the themes had any issues, do you mean they displayed alternating lines without the deprecated code?  Or that they never displayed the alternating lines anyway, so removing the code made no difference?  Do you need a theme that displays the shading to test this against?  The Waldorf theme from Crunchbang Linux had them, but I don't know how easy that is to find now.  There is a theme called Ambiance Green which has them, plus some challenging translucence effects, at least in the version I have.


See, I know this change is minimal and the impact is minimal to the API and backward compatibility. However, it does change the API and I really want to avoid that.

I think you can easily address your use case by using a method to dynamically construct instances of IdentityQuery based on those parameters ?


In case of error, you will see ```can't get gid of 'root'``` in the logs and if you are not familiar with the implementation you will be unsure whether ```root``` is user name or group name. The same apply for ```is_user_allowed```.


No, if you want to match `blah/misc/selenium/log` too, you would have to write

    **/misc/selenium/log

The initial `/` is just noise.


Shouldn't this be `return { instance: instance, success: true; }` ?


TeamCity LabKey :: Trunk :: Community :: CDS postgres 9.4 [Build 35](https://teamcity.labkey.org/viewLog.html?buildId=291581&buildTypeId=LabkeyTrunk_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 12, passed: 36 Build time: 01:08:05
### Failed tests
```
org.labkey.test.tests.CDSTest.verifyCounts: <no details avaliable>

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

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

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

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

org.labkey.test.tests.CDSHelpCenterTest.verifyHelpCenter: <no details avaliable>

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

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

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

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

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

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


Surely if this is a test for the ethereal spec, we should be *correct* here, rather than do what synapse happens to be happy with?


Po nazwie klasy mozna podac argumenty agenta. Przekazywane sa w nawiazach klamrowych. Poszczegolne argumenty oddziela sie spacja.


Probably wouldn't hurt to add a little test for the From conversion in `conversion_test.go`


Pretty sure the answer to this is yes, but it's been a long time since this
was bouncing around in my head

On Tue, Dec 15, 2015, 08:38 Burak Can Çelik <notifications@github.com>
wrote:

> Did you use the Freund and Schapire method?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/astromme/AdaBoost/commit/92b351e26be55eca9d8402c06c83d9544512fe8a#commitcomment-14985605>
> .
>


`any | string` :)


You want to avoid committing generated files like this. Perhaps just add them to your global .gitignore.


mds-proxy стоит за thevoid и может не париться о malformed url


Hi @akomakom, Thanks for taking the time to improve this Nagios monitoring plugin. I like the idea of more reliable yaml parsing. I can imagine that other people might like the stats in the output as well. I'm sure quite a few people are not. They probably use an app like https://github.com/puppet-community/puppetboard to have deeper look into the state of their Puppet landscape.
Therefor I suggest you change the default of NOPERF.


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


The operator `new` with argument list has higher precedence than a function call https://developer.mozilla.org/en/docs/Web/JavaScript/Reference/Operators/Operator_Precedence

Without the parenthesis the expression is to perform a `new` on the function `Function.prototype.bind.apply` with parameters `fn, args`. With the parenthesis the expression does a `new` without parameters on the expression `Function.prototype.bind.apply(fn, args)`.


Isn't this going to mean you have to touch this code when you add WinMobile support because the "else" is falling through to Android specific code.  Wouldn't it be better to guard this code with a Android check also?


Sure thing, I just wanted some early feedback before proceeding with writting tests.


This variable can be discarded, unless you plan on using it in someway.


Thanks @volaya and @m-kuhn for your review.
__________________________
Firstly, what about set value to the default one during instantiation ?
__________________________
@volaya for crs parameter, the default value can be None, because if the parameter is optional he can not be set.
@m-kuhn, I think that do a setValue to None is equal to reset value to default. And setValue returns if the setValue done is valid for running the algorithm.
For me the problem with not modifying value when None is passed, is that the parameter will stay at the previous value. It will never arrive in the UI, but can be do in python script.


Wow!


Agreed.


These are :+1:


But that doesn't meet the definition of the bower main?  the minimum
necessary for using the package?

I had the same problem before, bower would install ionic and all of its
dependencies (angular-*) on every update. With the bundle that is not
necessary.
On Dec 15, 2015 7:18 AM, "john-banks" <notifications@github.com> wrote:

> Unfortunately this now results in angular being brought into the project
> twice. To achieve what you wanted, the bundle file should be just
> ionic.js+ionic-angular.js but I suspect that will break things elsewhere.
>
> I made an issue here #4789 <https://github.com/driftyco/ionic/issues/4789>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/driftyco/ionic/commit/bfd9f083b35542a09e655a64b33d5cd130a2ba2a#commitcomment-14983885>
> .
>


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


TeamCity Civic Assembly :: Build backend + frontend [Build 90](http://54.229.176.143/tc/viewLog.html?buildId=199&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:05:51


лол, что это за критерий? у кого id кончается на 706 ? Может это направлено на какого-то человека? у кого у нас id кончается на 706?  Сама статистика, я так понимаю, подсчитывает самый популярный подраздел из Аудио.


`resolve` and `tryResolve` should probably share more of their implementations to avoid having to update both in the future.


                                                                                No PHP nor template code called these functions. Unless there is code doing dynamic function name building and calling, then it wasn't used at all.                                                                                From: Roy R.Sent: Tuesday, December 15, 2015 07:24To: BitLucid/ninjawarsReply To: BitLucid/ninjawarsCc: BeagleSubject: Re: [ninjawars] Remove unused code from lib_player (b011048)did you verify that the api didn't need this?

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


not yet! protobug is required


애초에 `Track`에 있는 `ArrayList`가 public이긴 하지만, `Track`이 가지고 있는 필드를 `SchePagerAdapter`에서 (또는 이를 통해서) 변경시키는건 별로 좋지 않아 보입니다.

차라리 `private Track mTrack;` 으로 `Track` 자체의 레퍼런스를 이 객체에서 가지고 있고,

```java
public ArrayList<Session> getSessionItems() { return mTrack.sessions; }
```
이렇게 getter를 밖으로 빼 주는게 어떨까요? 이러면 `SchePagerAdapter`객체에서 `Track`에 있는 `sessions` 객체에 접근하는 타이밍을 추적할 수 있다고 생각합니다.


Salut je suis Jean Louis j'habite à Saint Ouen (93), Jea suis au chômage.


请贴出github地址，感谢。

原始邮件
发件人:gongzhifengnotifications@github.com
收件人:WayWingsDev/bbambbam@noreply.github.com
发送时间:2015年12月15日(周二) 21:57
主题:Re: [bbam] init repo (afc307d)

Hi WayWingsDev,
 你好,在你的github下发现一份Ambarella SDK的代码, 已经违反了Ambarella的相关License.
 请你看到此信息后及时删除该代码.如有疑问,可邮件联系zfgong@ambarella.com.
 我们保留追究法律责任的权利.
 Thanks
 Zhifeng Gong
—
Reply to this email directly or view it on GitHub.


Hey @inacionery, this change is really needed? DDMFormValues already carries locale information?


Let's also add back in a travis test for 0.10 to make sure it stays as long as we want it to.


I'm confused from all the redirects :)

Could we remove some of them?


Are you suggesting  `\chapter{\lable{a-label}title}`? or that this should work?


Why Period? I would think password expiration intervals would best be described in Durations, e.g. X days.


Suggest Duration instead.


ghc told me at one point that `FlexibleContexts` was necessary to compile this, but now I removed it again and everything still compiles fine. Strange!


К этому сообщению можно добавить message из передаваемого error_code. ОК?


I prefer to not change the attribute name and use the bracket notation to access class property : `html +=   ' class="switch' + (attrs["class"] ? ' ' + attrs["class"] : '') + '"';`


TeamCity Civic Assembly :: Build backend + frontend [Build 91](http://54.229.176.143/tc/viewLog.html?buildId=201&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) is now running


On one hand PropertyMap would be easier, but then in the future if we needed to add some more information it would be harder. With our own class, we have more control.

An example of something we may want to support - deletion of certain properties. Currently we just use a PropertyMap which indicates the properties we can to update, but not delete.


Good point.  I'll catch this this afternoon.


Double quote breaks this file.


                                                                                As we discussed, game cookies are still handled the same way, session cookies are handled by symfony sessions using the default storage configuration. By using the mock storage configuration, symfony sessions don't use cookies.                                                                                From: Roy R.Sent: Tuesday, December 15, 2015 07:46To: BitLucid/ninjawarsReply To: BitLucid/ninjawarsCc: BeagleSubject: Re: [ninjawars] Replace SESSION with Symfony sessions. (8659cac)Does the symfony Session handle cookies internally?  What replaces the initialization of cookies in the ninjawars app now?

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


TeamCity TripleA :: UnitTest [Build 537](http://home.k-moeller.dk/viewLog.html?buildId=1393&buildTypeId=TripleA_UnitTest&guest=1) is now running


`BUNDLE_IS_FAVORITE_MODE` 처럼 단어 사이에 '_' 를 넣는게 낫지 않을까요


Duration is best for capturing hour-scale intervals.


TeamCity TripleA :: UnitTest [Build 537](http://home.k-moeller.dk/viewLog.html?buildId=1393&buildTypeId=TripleA_UnitTest&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 205 Build time: 00:00:45


TeamCity TripleA :: UnitTest [Build 538](http://home.k-moeller.dk/viewLog.html?buildId=1394&buildTypeId=TripleA_UnitTest&guest=1) is now running


Yep, I'm going to modify this code when we release Windows runtime. This is just some time checking code in our test app to verify how much exactly is the `require time` of the app. :)


This is a nasty hack.
Should be implemented properly only within the postgres module as this is purely custom postrgres behaviour.


Btw, here is a nice read - http://chris.beams.io/posts/git-commit/


Please check the code style.


Suggest Duration instead.


Perfect, thanks. Should be a one-liner.


@SteveMarshall Hey look, there is a Ruby URI template implementation!


TeamCity TripleA :: UnitTest [Build 538](http://home.k-moeller.dk/viewLog.html?buildId=1394&buildTypeId=TripleA_UnitTest&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 205 Build time: 00:00:41


Leads to unresolved dependency even after adding jcenterRepo as resolver. Neither 1.1.2 nor 1.1.3 are found.


だからさぁ・・・


I used `present_attribute_option` format for `src/Pim/Bundle/LocalizationBundle/Twig/AttributeOptionExtension.php`, maybe we should unify naming (change present_attribute_option or use present_date)


Udało mi się zlokalizować przypadek w którym możemy tutaj otrzymać ujemne wartości. Przykład:
```
2.1.5 :014 > e = Experiment.first
=> ...
2.1.5 :015 > e.size
 => 621
2.1.5 :016 > e.count_done_simulations
 => 626
```
Warunki dla których otrzymałem taki wynik:
 * eksperyment nadzorowany
 * scalarm w instalacji lokalnej, workery chodzącę na tej samej maszynie
 * symulacje trwają czas bliski zero
 * liczba workerów oraz liczba symulacji w danym rozszerzeniu przestrzeni są takie same

Jak widać taka instalacja powoduje że kolejne symulacje są zlecane i wykonywane bardzo szybko. Problem sprawdza się do tego że size jest polem w eksperymencie, a count_done_simulations jest obliczane dynamicznie poprzez liczenie obiektów SimulationRun  w bazie. W momencie wywołania count_done_simulations wartość size jest już nieaktualna.


Something we do in the `scala-js-dom` library when we are given well typed `dictionary` definitions like this, is create simple constructors for such dictionaries.  In this case, the [specification](http://www.w3.org/TR/push-api/#idl-def-PushSubscriptionOptions) states there is only one property, `userVisibleOnly`.

If you look at `MutationObserverInit` [here](https://github.com/scala-js/scala-js-dom/blob/999bc67d0c83217ffe129fdb782519cbedd4e2bd/src/main/scala/org/scalajs/dom/raw/lib.scala#L4370), you'll see how this is done- we define a trait extending `js.Object`, and then [define](https://github.com/scala-js/scala-js-dom/blob/999bc67d0c83217ffe129fdb782519cbedd4e2bd/src/main/scala/org/scalajs/dom/raw/lib.scala#L4421) a constructor method in a companion object.


Then please drop this commit


これも


I think the choice to reduce to a single buffer should be performed dynamically, not statically like implemented here because if will depend of the types used, and eventually of the behavior the user will specify using hints. So my proposal is the following:

**Step 1**:  for now we keep just the default constructor, with default behavior set to `reduceToSingleBuffer = true`.

**Step 2** (not required to merge this PR) : as soon as [`PromisePublisher`](https://github.com/spring-projects/spring-reactive/issues/50) is available, the converters will convert RxJava `Single`, Reactor `Promise`or other non streamable types (`String`, `List`) to `PromisePublisher`. That will make the decoder able to test if the destination type requires to be reduced or not.


Issue here


コメントアウトで残す必要なし。


Supposedly this class should not be used at all.


Thanks for answering :)


Pensei nisso tudo e depois de ponderar fiquei na dúvida, mas nem é uma coisa nem outra. Vou enviar sugestões B por outro via de comunicação


TeamCity Civic Assembly :: Build backend + frontend [Build 91](http://54.229.176.143/tc/viewLog.html?buildId=201&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:04:03


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


Even a POST could be re-sent? The code only shows the update_account.jsp when the session attribute is present, and this attribute is removed when the user completes the account creation. So I think the login porlet will just keep rendering the update_account.jsp until they answer its rendered question about associating or resetting the existing incomplete user. Is this the behaviour we want? https://github.com/liferay/liferay-portal/blob/master/modules/apps/login/login-web/src/main/java/com/liferay/login/web/portlet/action/CreateAccountMVCActionCommand.java#L459


TIL. Thanks for fixing my faulty approach.


What does `reentry` mean? This term is not explained anywhere.


This might actually read better as:
`if (process.cwd() == '/vagrant') {`

The hard-coding of the /vagrant path feels wrong though.  Is it not be possible to infer that the script has been called by the vagrant setup if the extra params are passed?  This way you could tweak the output also, if you set an `isVagrantInstall` global boolean.


System.out.println("PatientCheckExpiringJob".hashCode())
// -1213835169


Yes!
With the current PR, we'd need to change the `decode` method signature or communicate this through `hints`.


Quite unreadable IMHO


Сейчас я таких не знаю.
И для терминальных ошибок предполагается ```throw http_error{5xx, ...};```


TeamCity TripleA :: UnitTest [Build 539](http://home.k-moeller.dk/viewLog.html?buildId=1402&buildTypeId=TripleA_UnitTest&guest=1) is now running


Could you convert the russian doc into english?


genetic_alteration_type is not used downstream - its only used to select the correct java class to run. At least in the scope of this script.


TeamCity TripleA :: UnitTest [Build 539](http://home.k-moeller.dk/viewLog.html?buildId=1402&buildTypeId=TripleA_UnitTest&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 205 Build time: 00:00:44


These methods in general return different objects: get_concentration_matrix() returns array of concentrations per element which may be different from the solution vector returned by get_solution().


@Juan33650 asi lo agregué, como la ves?


Additionally, no fast-forwarding, and no rewinding, and we never output any data that hasn't been verified, so it's safe to use in a Unix pipeline.


I will copy-paste those functions into already exists in "GlobalFunctions" and will remove this file. Just temporary because I'm a job and busy with a task


刚发现重复提交pull request了 张亮选一个merge吧


The previous logic writes absolute paths instead of relative ones. When generating commonjs code for Node, for example, that means that the code will not be portable given that the require will be based on the absolute path of the build directory (i.e., '/Users/henvic/projects/api.js/build/component/foo.js' instead of '../component/foo.js').

This solves it and works with globals / amd as well.

This conditional is here to allow the user use babel.transform or babel.trasnformAST without breaking up the imports (I don't know what use case would be that, btw). We currently only use babel.transform on a test code and I guess it is there just to avoid creating a fixture file, but I am not sure about that.


다시보니 함수 이름이 조금 비 직관적으로 보입니다.

`orderSessionChronollogically` 따위의 이름이 낫지 않을까요?


Note the specification also [defines[(http://www.w3.org/TR/push-api/#idl-def-PushEvent) a constructor for `PushEvent`


In order to not break backwards compatibility, I'll either field to be used.


OK so don't take account of my previous remark :)


Sorry if I was rude.  I wasn't annoyed with you.  Dealing with the GTK devs always winds me up.


That was my point, more or less. ;)


Well it will bring back the system to the first initial stable state (modulo hysteresis, but there is not a lot), while classical simulation just let the system fall down onto the curve and go unstable right away.

So this might not be needed at all, I say let’s get to this point first and then discuss it with real behaviour to look at.


Why not for f0?


我想再明确下，什么场景下hashcode是负数，谢谢


Needs to be reimplemented properly, which can be a little difficult. Will
try tonight
On Dec 15, 2015 9:25 AM, "Rémi Verschelde" <notifications@github.com> wrote:

> set_paused() will no longer work I guess?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/godotengine/godot/commit/b12a2f456ccccae315426c7dd358bcec87112a0c#commitcomment-14984012>
> .
>


Quick bug fix. In the first version getting the day using NSDateFormatter would result in a different day number for iOS devices and the simulator. New approach is cleaner and works fine for both now. Also just fixed an array out of bounds error that was causing the graph to crash.


Maybe `ByteBufferDecoder` could extend `AbstractRawByteStreamDecoder` as well?


okay, all fine!


Although there's actually no mention of this in the documentation yet, I'm trying to ensure we have a hyperlink to the version of the specification that is implemented (from w3c or whatwg), rather than just MDN (which updates organically over time).  This will make it easier in future to keep track of outdated portions of the library.

In this case, the latest draft was released today! `http://www.w3.org/TR/2015/WD-push-api-20151215/`


@shoreflyer thanks very much for pointing it out. It should be fixed soon (It usually takes jcenter less than 24 hours to include a package).


Having a `nixos` option prefix seems very strange to me, given that *all* NixOS options are about NixOS.


I could do that if this weren't called by signal handler.
I just tried to rely less on Qt functions in signal handler.

doc: http://doc.qt.io/qt-4.8/unix-signals.html


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 9453](https://teamcity.mesosphere.io/viewLog.html?buildId=93104&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 0, ignored: 207 Build time: 00:03:58
### Failed tests
```
null: Kubernetes e2e suite.Kubectl client Guestbook application should create and stop a working application [Conformance]: <no details avaliable>

```


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


Because I failed.


I'm correcting it.


Зачем нам функция сложения в репозитории?


The output is similar, however, the `exons()` method with the filter argument I'm using is much faster than the `exonsByOverlaps` and the `cdsByOverlaps` methods used for `TxDb`. Also, I'm fetching all required data with one single SQL call and process this data to define also the UTR and CDS for protein coding transcripts.


AAAAAAAAAAAAAAAAAAA :100:


## Design

   - All receivers share a symmetric 32-byte key that's used to symmetrically encrypt the message, with authentication.
   - There's still an additional attack that receivers can forge the message among themselves, so therefore, we additionally MAC the message per receiver.  Each receiver gets one "Mac Group", and a MAC key that's shared across all of their devices.  (Give an example here)
   - The message is split up into 2MB chunks, and reordering attacks are prevented by sequential nonces
   - The message ends with an empty payload, to defend against truncation attacks.


@ironcladlou I see that we are testing for [DockerImage](https://github.com/openshift/origin/blob/1904cc304c9fc7b25cfdadc1966b773209714dc5/pkg/deploy/api/v1/conversion_test.go#L18) ICTs while we don't allow them in our validation. Is this probably an artifact?


`NaCl HMAC` might not be a thing.


@fzahle With this commit we do not allow material names containing numbers, e.g. 'YE1600HRC'. Should we allow numbers as of version 1? Then we could put your check into an if condition for version 0.


Thats correct. We will use meta_file_type and ignoring genetic_alteration_type. But, we are not removing genetic_alteration_type from the file.


Лучше пусть функция принимает параметры, которые для нее больше подходят как входящие концептуально. в данном случае это email  и user, подготовить их следует в вызывающем коде.


I suggest adding the width and height as CSS declarations in the header (instead of inline), to avoid the need to use ``!important`` in custom CSS styles.

For instance, if we want to make the map to be responsive and keep an aspect ratio of 16:9 we will need to override the FC styles and inline styling with the following custom CSS:

```
div .fc_addressint_map {
	width:100%;
	height: 0;
	padding-bottom: 56.25%;
	overflow: hidden;
}
div .fc_addressint_map_canvas {
	width:100% !important;
	height: 0 !important;
	padding-bottom: 56.25%;
}
```

Maybe we can move all the inline styles to CSS declarations in the header as a general rule.
Let me know what you think.


That's all so far really, although if you work from the referenced specification then I'm sure you'll be ahead of MDN :).

Thanks for your work so far!


we can combine ListIncompleteUploads, listIncompleteUploads, listIncompleteUploadsRecursive, listIncompleteUploadsNonRecursive, listIncompleteUploadsInRoutine into one function:

*We can have one more function findIncompleteUploadId() that does not do getTotalMultipartSize() so that it can be used in RemoveIncompleteUpload() and putLargeObject()*

```
func (a Client) ListIncompleteUploads(bucketName, objectPrefix string, recursive bool) <-chan ObjectMultipartStat {
	var objectMarker string
	var uploadIDMarker string
	delimiter := "/"
	if recursive {
		delimiter = ""
	}
	ch := make(chan ObjectMultipartStat, 1000)
	go func() {
		defer close(ch)
		// Validate incoming bucket name.
		if err := isValidBucketName(bucketName); err != nil {
			ch <- ObjectMultipartStat{
				Err: err,
			}
			return
		}
		// Validate incoming object prefix.
		if err := isValidObjectPrefix(objectPrefix); err != nil {
			ch <- ObjectMultipartStat{
				Err: err,
			}
			return
		}
		for {
			result, err := a.listMultipartUploads(bucketName, objectMarker, uploadIDMarker, objectPrefix, delimiter, 1000)
			if err != nil {
				ch <- ObjectMultipartStat{
					Err: err,
				}
				return
			}
			objectMarker = result.NextKeyMarker
			uploadIDMarker = result.NextUploadIDMarker
			for _, obj := range result.Uploads {
				obj.Size, err = a.getTotalMultipartSize(bucketName, obj.Key, obj.UploadID)
				if err != nil {
					ch <- ObjectMultipartStat{
						Err: err,
					}
				}
				ch <- obj
			}
			for _, obj := range result.CommonPrefixes {
				object := ObjectMultipartStat{}
				object.Key = obj.Prefix
				object.Size = 0
				ch <- object
			}
			if !result.IsTruncated {
				break
			}
		}
	}()
	return ch
}
```


Indeed, and that would not be easy because converter API is already defined and knows nothing about hints. That why I think we should not spend some time to manage hints and just wait `PromisePublisher`.


`get_fsuid()` and `get_fsgid()` uses the same function `get_proc_fs_id()` which contains sufficient error messages. So, we can use `error_msg_ignore_crash()` instead of `perror_msg_and_die()`.
Also, in the case of `get_fsgid()` there is no `perror_msg_and_die()`.


Это надо вынести в шаблоны и тему и письмо.


Хардкодинг емейла.


TeamCity pull requests :: netty [Build 875](https://garage.netty.io/teamcity/viewLog.html?buildId=4100&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 6229, ignored: 21 Build time: 00:42:47


Yes you're right. I'll fix this.


But `-W058` means something different. It allows you to write
```
return new (Function.prototype.bind.apply(fn, args));
```
instead of
```
return new (Function.prototype.bind.apply(fn, args))();
```


should the same be done in the locked payments and history views? not showing **£**#.##, i mean


В случае не нахождения токена пользователь будет получать страницу 404, не очень информативно, верно?


Is this being merged into 1.1? I have tried running the tests for li3 1.1 branch and it still fails on that section. We'd like to contribute towards PHP 7 compatibility, but don't want to replicate work that has already been done.

Thanks.


TeamCity Civic Assembly :: Build backend + frontend [Build 92](http://54.229.176.143/tc/viewLog.html?buildId=203&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) is now running


Committed 34aaafd.  I'll rebase and request inclusion when my users confirm.  Thanx.


Thank you :)


FWIW: you can see the effect of the problem fixed by this patch by changing the size of hugebuf in BIO_vprintf from "1024 * 2" to "10" and running "make test". You'll get a bunch of errors because the last character of various cipher suites gets overwritten with \0 if you do that (unless this patch has been applied).


Dude, use a ternary condition. :)

`this.drawAngle = this.touched === true ? this.headingAngle : 0;`

If you need details, MDN has you covered.


qwerty


Yeah, you're probably right. Overall what you're going for looks great.


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


I don't think this is a good idea:
* It's expensive: you need to evaluate the config once just to get `nixos.path`, and then again to evaluate the system.
* There is no guarantee that the current Nixpkgs can evaluate the config. E.g. if `<nixpkgs>` is 14.12 and `nixos.path` is 16.12, and the config uses 16.12 features.

I think it would be better to have a more direct way to specify a $NIX_PATH to `nixos-rebuild` (e.g. it could read the desired path from `/etc/nixos/path` or something like that).


How is the version updated related to the WIKIEDITOR-27 issue?


I'll try to use `psetdiff. I used the loop because I wanted to be *fail save*; since the `EnsDb` does contain protein coding and non-coding transcripts and genes CDS and UTR will not be defined for all transcripts. Regarding the more convenient way to extract UTRs, but this would again require an additional query to the database, and this calculation would have to be performed by that function. Since I get all required information by the `exons` call above, I opted to calculate the UTRs and CDS directly here.


Does this now tell me to prep the question when it is open and I am waiting for someone to buzz in?


Please avoid committing these changes since they are not relevant.


We should mention the `| 0x80000000` trick here.


is not None - правильная конструкция


Where is that used?


=> '</para>'.


Would you mind tagging a release for that?


Say it's safe to use 0 nonces because the key is generated randomly per message.


В этой ветке пользователь is None, куда приведет его ссылка на профиль?


Looks like you misspelled "canols"!


Hi goochjj,

I could not found the "dhparams" option in the 2.7 pound I have built from:
* https://github.com/goochjj/pound/archive/upstream/v2.7.tar.gz

Did I use the wrong tar.gz? Or is the "dhparams" missing in 2.7?

Cheers, Georg


How does this interact with `nixpkgs.system`?


You don't need this extra method and it has a wrong name you don't measure anything here you count errors


Понял.


I'd call the last parameter `speed` instead of `tempo`. Not really an issue, though.


hi I'm not understand about this link. if i replace example. mykeylog.hosting.com/log.php it's remotely log the keystrokes? thanks you.
![screen shot 2015-12-15 at 8 59 26 pm](https://cloud.githubusercontent.com/assets/16306913/11813260/ca71f872-a36e-11e5-8fcb-32888f7b38a1.png)


No, that looks bad.  I had only though about regular numbered headings...

How about `\chapter{Some title}\notnumbered`?  (but the meaning of `\notnumbered` isn't clear from the program's point of view, since a command that alters the display of the title was inserted _explicitly_ after the title).

Or we could have command variants for non numbered items, something like `\chapter*{Some title}` or `\nnchapter{Some title}`?  Also, ConTeXt (another TeX based document preparation system) [uses](http://wiki.contextgarden.net/Unnumbered_Sections) `\title` for an unnumbered `\chapter` and `\subject` for an unnumbered `\section`.

Finally, another way to go would be to use optional arguments and options – `\chapter[notnumbered]{Some title}` – but would make parsing commands harder and less homogeneous.

---

Note: on TeX, the `\chapter*` and `\section*` variants do much more than simply suppress numbering.


Makes sense. Just thought that with some simple munging (either here or in the TxDb method), something more could be shared. I really like the features of EnsDb objects (filters, additional columns). Perhaps some could be ported over to TxDb?


I don't think this works: the type must be looked-up first (not always known by the user). See https://github.com/killbill/killbill-cloud/blob/e734e447246701f15e477ecef987106065bfa7b7/kpm/lib/kpm/base_installer.rb#L75.


It is confusing to use consume for a class which does not implements Consumer )


You miss <>


From what I understood of Fortran, dabs should be changed to abs (we don’t use double, but x_precision, and abs auto-detect type while dabs ever use double-precision).


should we really translate those keys - (weight, volume) did you preview it in the UI?


`specified_group_id` and `specified_artifact_id` could now have default values (e.g. `nil`).


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


Can you rename this to `Bump version to 0.2.5` so it would be consistent with other version bumps


Fixed in 2.0.7 :)


this one doesn't look translated


You allocated this consumer again and again it is not needed here


I actually meant something like _"since it makes sense to follow browsers closely and browsers are too lenient..."_. :)
I'll update that too.


Instead of returning `An error occurred` here, you could return `Token Invalid` message.  This is because an error occurred should be used when we want to return a generic message (maybe due to security reasons). However, in this scenario, we need to be as specific as possible to give the API consumer context on what happened.


final


The problem with this commit is that it does not address the heart of the WIKIEDITOR-27 issue. It handles minor keyboard shortcut aspects, but not the fact that you can not search the entire contents of the codemirror's content, using the browser's default search control.

These changes should be handled under a new issue, that is separate from WIKIEDITOR-27 and which targets only these improvements.


Extra `it` in description.


delegate instead of inner


@rmichaelis Point, Label and Circle are saved twice to the MyMap .


Can you add a check that it can overwrite the mapping too?


Thanks I did not see this, this is now refactored :)


TeamCity LabKey :: LabkeyModules15.3 :: Git postgres 9.4 [Build 72](https://teamcity.labkey.org/viewLog.html?buildId=291594&buildTypeId=LabkeyModules153_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 23 (3 new), passed: 268, muted: 2 Build time: 01:27:42
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.ScrumtimeTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.util.JUnitHeader: <no details avaliable>

org.labkey.test.util.JUnitFooter.afterJUnit: <no details avaliable>

org.labkey.test.tests.CDSHelpCenterTest.verifyHelpCenter: <no details avaliable>

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

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

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

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

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

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

org.labkey.test.Runner: org.labkey.test.tests.CDSLoginTest: <no details avaliable>

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


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


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


I updated the PR message for the now-closed PR #1063


TeamCity Civic Assembly :: Build backend + frontend [Build 92](http://54.229.176.143/tc/viewLog.html?buildId=203&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) outcome was **FAILURE**
Summary: Exit code 1; exit code 123 (new) Build time: 00:08:51


Many tools use `babel.transform`, specially tools that handle streams like gulp (including **gulp-babel**), because they don't want to have to fetch the file again, since they already have access to its content. They usually also pass the filename in these cases, but babel doesn't require it, so someone could be using babel without it.

I see now that using relative paths is indeed important though, you're right that the commonjs use case requires it. It might be useful to warn the user about this error when it happens though, otherwise it will be really tricky for someone to debug this. Could you just add a `console.warn` call for this? Also, can you add a test specifically testing this case (I know that the existing test case already covers it, but it doesn't mention the problem, so this case is not clear in the test file), and making sure that `console.warn` is called?


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


This can't be null


Masha


So you’re not really doing a mean_size elements mean, rather a degressive mean where the last element has more weight than the former ones (the n-th former element has a `((mean_size − 1) / mean_size)**n` weight).


Обычно именна внутренних классов тоже с большой буквы называют...


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


Ich hab keinen Plan wieso die jetzt schon wieder dabei war... in der git-console ware das file nicht gestaged aber php-storm hats trotzdem mitgenommen... Naja, jetzt ists wieder die dummy. Mal schauen wie oft das noch passiert -.-


got it.  Thanks


109806134-AbsChlrBlankWaterFlow (JasonGlazer) - 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)


I'm confused: idempotency should be verified [below](https://github.com/killbill/killbill-cloud/commit/bcdd75ef926c7b3f4d10810f0637d6fa6ee30dd5#diff-7fadbb075cc2d9fe93d3480d4d163793R57).


Мне кстати только сейчас дошло, что имелось в виду под переприсваиванием переменной)


The "findPersistent" command, which opens a dialog that stays after the first match, exists only since version 5.5 (http://codemirror.net/doc/releases.html).


Yes, because doing a "real" mean is very costly and slows down the
simulation.

On 12/15/2015 03:35 PM, Bruno Pagani wrote:
> So you’re not really doing a mean_size elements mean, rather a
> degressive mean where the last element has more weight than the former
> ones (the n-th former element has a |((mean_size − 1) / mean_size)**n|
> weight).
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/M2-AAIS/BAD/commit/3b0fef04a8be8414a6b93f9ce69151c65459712e#commitcomment-14987032>.
>


Isn't now a good moment to write an idempotent `script/bootstrap` script and a `script/test` script (that calls `script/bootstrap`) and use `script/test` inside CircleCI? As suggested by https://github.com/github/scripts-to-rule-them-all


TeamCity Civic Assembly :: Build backend + frontend [Build 93](http://54.229.176.143/tc/viewLog.html?buildId=205&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) is now running


`uri` мы не должны передавать в конструктор, так как он гененрируется автоматически при первом сохранении объекта (за генерацию отвечает org.ayfaar.app.utils.UriGenerator)


Thanks! Will take a look at latest master.


no real changes


Yes, that is much better for NPM newbies like me ;)


I'm not sure if we should require a namespace if the official `group_id` is used (it just means the entry is missing in our directory, which can happen for test and demo plugins).


the `end` parameter could also be calculated


After this commit, I'm getting many proguard-related messages when building the TestApp (https://gist.github.com/zugaldia/1692a85f601ab5a02956). Is this as expected?

/cc: @tobrun @bleege


Does this mean a new and improved Rawkets is coming? I remember seeing a screenshot of some larger ship looking things...


Sure. Perfect.
On Dec 15, 2015 11:34, "Maira Bello" <notifications@github.com> wrote:

> Many tools use babel.transform, specially tools that handle streams like
> gulp (including *gulp-babel*), because they don't want to have to fetch
> the file again, since they already have access to its content. They usually
> also pass the filename in these cases, but babel doesn't require it, so
> someone could be using babel without it.
>
> I see now that using relative paths is indeed important though, you're
> right that the commonjs use case requires it. It might be useful to warn
> the user about this error when it happens though, otherwise it will be
> really tricky for someone to debug this. Could you just add a console.warn
> call for this? Also, can you add a test specifically testing this case (I
> know that the existing test case already covers it, but it doesn't mention
> the problem, so this case is not clear in the test file), and making sure
> that console.warn is called?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/henvic/babel-preset-metal/commit/92293f5840653767f56ac87d16786694407452a4#commitcomment-14987026>
> .
>


Whoa... incredibly fast... Thank you!


@cgfrost This class does not use this dependency and therefore it should not be passed in.


OMG the CVC binary is 243MB!! Considering that Z3 binary is 18MB, do you think there was something misconfigured during the build process? Debug symbols?
Are they statically linking all their dependencies??


You need to separate commits in which you do code formatting from the ones in which you actually do code changes, otherwise your changes might get lost in translation and reviewing such commits is almost impossible (highly unreliable).


I would instead write to a temporary directory and replace it at the very last moment (see base_artifact.rb for instance).


gn


This commit is being reviewed as a part of review issue #149.


looks like here and above you use wrong error message


Так было, потому что больше ничего не было (request_id) и никак нельзя было понять, что за запрос обрабатывается.
Сейчас это не решает никакой задачи, поэтому убрал.


Nitpicking but missing "." at the end (and other points have that ;) )


Can't we use HTTPS endpoint ?


обычно переменные называются с маленькой буквы...


Сейчас – да.


похоже эта функция здесь лишняя


Ну да, сейчас в словарике окажется последнее присваивание некоторой переменной. Но с другой стороны, если эта переменная где-то была определена, то значит в следующих выражениях будет использована, а затем когда нужно переприсвоена.

Короче, скажи как тебе лучше - я попробую переделать, если это необходимо.


Warum ändern?
"15.12.2015, 15:42:05" passt doch...


похоже лишний импорт


Hi bro, I'm the first one to visit you.


you're wellcome


What's the type here?


Who is @djipey? It looks like that usename is the author of this commit.


Isn't the Exception $code an integer? Now you're using if for strings, right?


최종


я бы назвал просто `parse`, так как по названию класса понятен контекст


Isn't `TcpServerConnectionFactory` interface enough here?


k.  sounds good


Not anymore.


does it make sense?


@ELouisYoung
`ignore: [/submodules\/.*/]`
https://github.com/johnbintz/rack-livereload#tweaking-the-options


How are you building for ppc? I pass this to configure without issue on "CC='gcc -m32' CXX='g++ -m32' --host=powerpc-linux-gnu --with-cpu=power8", and have no issues.

The --host parameter is necessary because the build system will guess wrong and choose 64 bit headers, resulting in very strange build failures.


Missing validation?


Can we at least make this download optional? E.g. by requiring it only in certain configurations.


5 issues solved under: https://codeclimate.com/github/WebDevJL/EasyMvc/issues/categories/security


ha, it's me. Problem with my user name in git. Sorry.


Разумеется. Но я не люблю тратить время на переписывание старого кода просто, чтоб было красивее.


TeamCity Civic Assembly :: Build backend + frontend [Build 93](http://54.229.176.143/tc/viewLog.html?buildId=205&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:08:51


Remove `lookup_arg && specified_arg` check (`nil.to_s` == `''`), as the check would break with blank strings.


Context is the one place where performance matters enough that you actually notice a speed drop from using 'new' here. The context function has a lot of things like this for performance reasons, things I would not do anywhere else.


We can rename:
```
authHeader        = "AWS4-HMAC-SHA256"
```
`authHeader` to `SignV4Algorithm`


хорошо бы проверить на `null`, ` `, `""`, `"dsf234"`, `xI i`, `,`  и т.п.


Sorry you are right course, my example has a dumb mistake.

It should be `return Math.floor((1-jitter) * interval)`. The intuition is that each step has a range like `X <= interval < 2X` where X starts at min and doubles each time until 2X is > max. So you might get `random() = 0.999999...` on step N and then `random() = 0` on step N+1 which means you'll have almost same backoff time twice, but on average they will always increase and with exponential trend.

So the bug was that jitter was between 0 and 0.5 and we want it to be between 0.5 and 1 so it scales the output between full and half.

You new version looks like it works too, but I find it easier to reason about jitter just being a scaling factor between X and 2X so up to you :)

Inf on the power is also a good spot.


@marcellustavares, I think it's necessary, because i need the values using the site's locale.


I was copying the way Test::Builder did it, I was nto sure why Test::Builder did it this way, so I didn't mess with it. Thats said I much prefer our, so I will switch it.


Переприсваивание временных нам не важно, а ID для нормальных переменных я уже сделал, там пару строк всего надо было добавить.


Needed for die-on-fail and similar behaviors. Easy to have it now and rmeove it if it becomes unnecessary. It would be very hard to make it necessary later.


:bowtie:

:heart: URI RFC 6570


I realize that it's not new code, but `.format()` should be preferred over `%`

Quote:

> Why would you not use it [`.format()`]?
 * not knowing about it (me before reading this)
 * having to be compatible with Python 2.5

http://stackoverflow.com/a/5082482/2319028


`iit`


тоже интересна реакция на `null`, `0`, `-1`, `Integer.max` и т.п.


I'm glad you find it helpful! I realized that I was used to this happen, but obviously it's distressing to people looking at the console log.


@thatoddmailbox Why bootbox. Swal is sooooo much better.


I try to use qw{} for multi-line, and I use qw// for quick/short


BTW, shouldn't we have automated checks for itt / ddescribe & friends?


(not to mention that it's not obvious at all, and google results don't really help)


感觉略黑科技，cgo的一些感觉


Any news on that? It will be extra cool to have it back!


@sjparkinson Any news on that? It will be extra cool to have it back!


I want to make sure I understand how the linkers are playing with symbols. The versions contained in the version file have no bearing on how the binary is named by autotools? If so, I will revert all but the change in the Version file.


LEgacy Test::Builder behavior, and it has to be done here to preserve back-compat.


I'm not sure I understand the purpose of that validation. `specified_group_id`, `specified_packaging` and `specified_classifier` can never be `nil` (see above).

Also it makes the code a bit hard to read (maybe a unit test for this would help?).


I forgot to ask to check the lib:* and devel:* versions. These should not be $portVersion in general.


Legacy Test::Builder behavior, without this here I can;t support back-compat.


I actually didn't check the size of the binary, and it definietly shouldn't be that big, the pre-compiled binary which can be downloaded is maximally 15MB big. I will check what went wrong there...


`*.suo` and `*.user` files are machine specific and should not be added to the repo. Please remove them.


I am so happy that you come here.Do you have forked my project and deal with problems?


Remember the benefit of API v3 we were talking about yesterday?


I cannot directly use eval here, would have to use try. See my comment on your later comment about why try is what it is.


TeamCity DNN JavaScript Libraries :: CI [Build 124](http://teamcity.engagesoftware.com/viewLog.html?buildId=402&buildTypeId=DnnJavaScriptLibraries_Ci) is now running


Nice find here. I was relying on the upper if to control that there were at least some windows but then removing them does have that risk of an outofrange exception.  I wonder if the overall logic of removeoldwindows needs to be adjusted.


TeamCity DNN JavaScript Libraries :: CI [Build 124](http://teamcity.engagesoftware.com/viewLog.html?buildId=402&buildTypeId=DnnJavaScriptLibraries_Ci) outcome was **SUCCESS**
Summary: Running Build time: 0:0:11


Clean slate?


shouldn't this be overrides too?


A webservice (non-upgrade) and websocket (upgrade) have nothing to do with each other.

The code you have highlighted is used to determine if an upgrade exists.

If it doesn't satisfy the upgrade rules, then the request is passed along into the next steps (which could be a Filter, a Servlet, another Handler, etc...) as-is, no change to the request.

eg:

 * You can have a Servlet on "/foo" and a websocket on "/foo"
 * If your FooServlet has doPost() enabled, and a POST is sent to "/foo" then the doPost() is called.
 * If an Upgrade request is sent to "/foo" then the upgrade occurs.

I'm beginning to suspect your problem is somewhere else, and has nothing to do with this line of code.

This check on valid WebSocket upgrade has existed on all stable releases of  Jetty 9, with minor updates for case insensitive checks and multi-value checks since its first release.

The Websocket implementation in prior versions of Jetty (such as Jetty 8) didn't even have this class, and the websocket layers were completely different.  So an upgrade would have required you to completely rewrite your websocket usage of Jetty.


My guess is that it statically links all dependencies.


I have a local branch that splits these already in the works :-)


Possibly... My thinking was that if the device is never ready to set up, control points may just wait forever for that to change and never actually interact with the device.


Hmm.. Perhaps a reasonable point; given as SYN-507 already exists, that keeps track of what we're not implementing/testing yet. I'll change this one.


While the mapping `plugin_key` -> `plugin_name` is stable for official plugins, it may not be for custom ones. Should we check first if a mapping already exists and points to the same `plugin_name`?


По поводу лишних логов согласен.
По поводу ERROR не согласен. Здесь нет причин нам беспокоиться что что-то пошло не так. Типо "клиент пришёл с какой-то фигнёй, ну ок".
Выводить значение ETag нет смысла – тебе всё равно что-там. Ровно так же тебя не оч волнует какой ETag корректный.


OK. In that case setter will be added.


It appears that Durations and Periods overlap at days. I thought password expirations would most likely be expressed in days or months and deployers would likely want DST taken into account.


This login method return very specific message when credential are not valid. This should be avoided  since a malicious user can use the returned message to easily hack your system. Take for example, a malicious user using a random email and password to login to your system, as soon as he receives `invalid password` as a response, he will know that that email account exist and he will only need to find the correct password for that email account.


We have dropped support for ProrotypeJS. Use jQuery instead.

http://platform.xwiki.org/xwiki/bin/view/DevGuide/JavaScriptAPI#HBridgingcustomXWikieventsbetweenPrototypeandjQuery


Questa in origine era indicata come commit n. 49
https://code.google.com/p/kelly/source/browse/trunk/src/Kelly/?r=49


note, in the case where __excepts != 0, and popcnt(__excepts) == 1 (guaranteed by the macro), __builtin_ffs = __builtin_clz + 1 = __builtin_ctz.


They are for one of the more obscure platforms. Without it it fails, I think it was for VMS.


Looks like this: https://www.desmos.com/calculator/9t1gu1g8zr

With blue line being the lower bound on retry time and green being upper for a given number of retries.


Or `nixos-rebuild` could evaluate `nix.nixPath` and add it $NIX_PATH for the final system evaluation, like like it already evaluates `nix.package` to get the Nix package. This does suffer from problem 2 though.

Another solution that avoids all these problems is to have `nixos-rebuild` build a file named `/etc/nixos/system.nix` (or something like that) if it exists, which would be responsible for returning the same result as `<nixpkgs/nixos>` (i.e. the evaluated NixOS system). E.g.
```
import "${fetchTarball https://github.com/NixOS/nixpkgs/<hash>.tar.gz}/nixos"
  { configuration =
      { config, pkgs, ... }:
      { ... };
  }
```
It could also use `scopedImport` to set a Nix search path declaratively.


Extra blanks.


ok


Посмотри код на предмет соответствия стандарту PSR, прочитай про стандарт PSR.


Макрос использовался, потому что в gcc 4.4 не было лямбд.


I don't like loading things (especially heavy things!) in `static {}` --- the problem it gets done on import, even when CVC4Environment is not used. It's much better to do it in static method, cf `Z3FormulaManager`


Работай с сессиями средствами Yii


So, it looks like it was all dependencies needing updated?


@nywillb that's why this commit deletes bootbox and says swal is better than bootbox...


Well we don’t need the mean very often I think (or do we), don’t know what you have in mind when talking of implementation, but I see two possibilities (given that we need to store the `256*mean_size` latest elements whatever we do to calculate the real mean I think):

1. Store `mean_size` previous element in `mean_elements`, each divided by `mean_size` and sum them when needed for `mean`. To add an element, `mean_elements[2:mean_size]=mean_elements[1:mean_size-1]` and then add the new element divided by mean_size in case 1. Calculating mean cost 256*mean_size sum operations each loop (plus 1 division…), the memory needed is 256*mean_size x_precision values. Not sure about the exact cost of array transposition.

2. Store the calculated mean too, only add/remove the new/oldest element while also adding/removing it to/from the `mean_elements` array. This reduces the number of operations needed to only from `256*mean_size` sums to `256 sums+subtractions`, which is not negligible, at the only cost of a supplemental array of 256 `x_precision` values (while we already have 256*mean_size in both case).

So, though I’m not sure about actual cost for `mean_elements` changes and the cost of the rest of the code (but compute_variables is probably really heavy), I think this is doable.

However, as long as it works, I wouldn’t matter a lot about it.


If you`re making this an AMD module, then this enabled check does not make much sense any more, does it? It is already used in the EnableSyntaxHighlighting document's JSX.


Yes, you are right.


Shouldn't this work `scope :active_games, -> { where(winner: nil) }` - seems to be putting out the right SQL in the console and the variable has been set correctly in the controller, but still picking up forfeited games for some reason.


Test::Builder always localized __DIE__ when it needed to eval things like this. It also localized $@ and $!. The reason for all these is that Test::Builder more or less guarantees that running ok(...) or any other tool will never squash your $! or $@. It is debatable weather that is a worthwhile thing to do, but if I don't make Test2 do it as well then I break backcompat.


Yes, the trick of moving the import lower-- it is just working working around the circular dependency. Given that these are new packages, I'd prefer we figure out how we want to structure them to avoid circular dependencies.


Once again would prefer to load in the static method. Also two calls to `loadLibrary` should probably be together.


@sebmck Why version of this empty package is changed? And then, why sometimes only?


+1

This deployer would like DST taken into account :-)


Virgin Mobile LTE apns-conf


Actually the important is not need at all. Only I rare case some bad template CSS may be  needed.

About the in-linew idth height  statements maybe we can remove them if someone sets 0 to the parameters. We can change description on the parameters to say about zero value

So let's just remove important and if width and height is zero then we skip in-line style too. What do you think?


I tried to do it, but it didn't work.
I'll try it again.


Une application Java Swing qui permettra aux utilisateurs de gérer leur liste de morceaux de musique.
La première partie permet d'importer les morceaux de musique dans une base de données à
partir des données stockées dans un fichier CSV qui vous est fourni. La durée des morceaux, en
minutes:secondes dans le fichier, devra être convertie en secondes dans la table.
La deuxième partie permet de gérer les morceaux de musique stockés dans la BD, c'est-à-dire
les visualiser sous forme de grille avec la durée en minutes:secondes, les modifier, les
supprimer et en ajouter à partir de l'écran d'affichage.


I’m not sure about this condition. Could you explain it a bit and tell in which conditions this happens?


Работай с $_GET средствами yii


the ones on user.delete are still really irritating but I don't see an easy fix for them


I tried what you suggested, the thing is that is is quite hard to know
when to start recalculating the mean. The easiest one was a worst mean,
but it still gives an indication of to what extent we're converging and
I think it'll be enough.

On 12/15/2015 03:53 PM, Bruno Pagani wrote:
> Well we don’t need the mean very often I think (or do we), don’t know
> what you have in mind when talking of implementation, but I see two
> possibilities (given that we need to store the |256*mean_size| latest
> elements whatever we do to calculate the real mean I think):
>
>  1.
>
>     Store |mean_size| previous element in |mean_elements|, each divided
>     by |mean_size| and sum them when needed for |mean|. To add an
>     element, |mean_elements[2:mean_size]=mean_elements[1:mean_size-1]|
>     and then add the new element divided by mean_size in case 1.
>     Calculating mean cost 256/mean_size sum operations each loop (plus 1
>     division…), the memory needed is 256/mean_size x_precision values.
>     Not sure about the exact cost of array transposition.
>
>  2.
>
>     Store the calculated mean too, only add/remove the new/oldest
>     element while also adding/removing it to/from the |mean_elements|
>     array. This reduces the number of operations needed to only from
>     |256*mean_size| sums to |256 sums+subtractions|, which is not
>     negligible, at the only cost of a supplemental array of 256
>     |x_precision| values (while we already have 256*mean_size in both case).
>
> So, though I’m not sure about actual cost for |mean_elements| changes
> and the cost of the rest of the code (but compute_variables is probably
> really heavy), I think this is doable.
>
> However, as long as it works, I wouldn’t matter a lot about it.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/M2-AAIS/BAD/commit/3b0fef04a8be8414a6b93f9ce69151c65459712e#commitcomment-14987528>.
>


Marco, I always suggest to use json.dumps instead of str when you convert dict to a string representation. This allows to avoid tiny mistakes with quotes and unicode. The str does not produce correct string representation suitable for json.loads. For example:
```
rec={'bla': u'foo', "foo":1}
print rec
{'bla': u'foo', 'foo': 1}
import json
json.loads(str(rec))
Traceback
....
ValueError: Expecting property name: line 1 column 2 (char 1)
```

while if you do
```
json.loads(json.dumps(rec))
```

everything will work fine. The point is that using str your client may not be able to parse string representation back to json.


Bug here, I assume you want to add the `term` to `converted`. Amusing we don't have a test case covering this.


But $_mpu_irq in dosemu2 is 5, because it is so in sb16.


Is it worth considering having either a boolean like `should_retry` in disconnect as well as reason string such that client doesn't have to parse strings?

The other benefit even if we think number of unique strings in small is that old client code that is still running will still behave correctly this way even if a new type of error is introduced on server side that should be considered fatal.


Incorrect, atom is everything without logical operators inside, see `Z3FormulaManager` for an implementation.


Localizing $@ is reasonable.  I was just noting that inside the try, $@ will be initially reset to undef.  Other implementations of this idea preserve $@'s current state inside the try.

I'll have to look into how Test::Builder messes with `__DIE__`.  In general, I think it's a pretty terrible idea to localize it.


If it's not implemented IMO we should throw an exception.


This is just a condition so that we don't have to compute the mean for each iterations, but only when we're getting close to the convergence point.


Shouldn't this reduce the dmg for every ally hero and not only for himself?
This unit deals 15% less damage for 2.5 seconds.


109806134-AbsChlrBlankWaterFlow (JasonGlazer) - x86_64-MacOS-10.9-clang: OK (1979 of 1979 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 apply this, hardware midi will stop working.
I really think you should see into .asoundrc to
map "default" to "virtual".


why not simply use the folly URI since proxygen already depends on it

https://github.com/facebook/folly/blob/master/folly/Uri.h


@Greyvend There may be other cases I didn't catch but I fixe this in https://github.com/MongoEngine/mongoengine/pull/1117


Ни одного комментария в коде. Также подписывай методы. Посмотри PHPDoc


will this work fine on windows? because filepath will use `\`


Hi @MadLittleMods,

The difference between your code and the original example is that in your example `subsciption` is still a promise to a subscription. In other words, you can't invoke `.unsubscribe()` on it directly.

```js
var subscribe = client.subscribe('/channel', onMessage);

/** As an example, wait 10 seconds and cancel the subscription */
Promise.delay(10000)
  .then(function() {
    if (subscribe.isCancellable()) {
      subscribe.cancel();
    } else {
      return subscribe.then(function(subscription) {
        subscription.unsubscribe();
      });
    }
  });

```

However I'm not sure if that would help the reader or confuse them more.

I'm still not totally happy with `.subscribe` being a promise to a subscription. However, when I make it return a `subscription` instead of a `promise` it also doesn't feel right. For now I've settled on the promise route, mostly for consistency.


Probably incorrect, would have to look more closely at CVC4 code.
Almost _everything_ is a function: arithmetic operators, logical connectives, etc.
Uninterpreted function is a special subset of functions...


Are there actually notices being generated here? It shouldn't be necessary to pre-declare variables being passed by reference for multi-return purposes like this, and as far as I know it's still the standard way to access, say, $matches from preg_match.


Дублирование кода


ok, I see that the check is too restrictive, it should allow for names like you suggest. I just intended to make a check to separate the actual material name from the counter e.g. splitting `triax00` into `triax`and `00`. I'll take a look at fixing this.


You need to give the explanation as a second argument, e.g. `option "with-apache-22", "Do something"`.


Прочитай как правильно именовать переменные


why did you change makeBucket request from `https://bucketName.s3.amazonaws.com` to `https://s3.amazonaws.com/bucketName` ? did you see timeouts as well?


Nope, Kind.IFF is also an equivalence.
I don't know what is the difference vs. EQUAL is TBH, I think we have to check for both.


Story: gradle/langos#126


IMO, the SSX should be left here, since this is where the module is defined and customized. The EnableSyntaxHighlighting SSX should only contain the "isEnabled" check and an `@import` to this SSX.


saw approval from bkerley
at https://github.com/basho/riak-ruby-client/commit/48b34a5fac95bca659860cc7e41cb27871ffe0e7


merging basho/riak-ruby-client/bk-disable-ts-buildbot = 48b34a5f into borshop-integration-255-bk-disable-ts-buildbot


basho/riak-ruby-client/bk-disable-ts-buildbot = 48b34a5f merged ok, testing candidate = 63decf5f


IFF is equivalence, not `ifThenElse`.
I haven't found an operator for `ifThenElse` though.


Oops, thanks for the fix!


OK, so the use of `mean_size` here is because you expect tiny variations to be divided by `mean_size` as the effect of averaging. Fair enough.


Beware, this release is broken (`%{with ...}` etc.).


This check should be removed from this location, as explained above.


Yeah, that's why I didn't say flat out, "you should change this".  I was just raising it as a potential issue.  What happens is that the compiler downgrades the object to a Map representation -- but that's effectively what this is anyway.  I'd say we should use a real map, but then that means we need to start worrying about compatibility with old JS versions.

So I agree, not worth fixing right now.  But thanks for everyone taking the time to think about the issue with me.


Hey @samdroid-apps, can you explain how to produce tihs?


you could use `status, payload = V1::Authenticate.decode_token(token)` since package makes it less obvious what is been returned. Also assigning package.last to user variable makes your program very confusing to read since package.last should be the payload returned from decoded token.


Bonjour,
Pour cette semaine 2 je vous donne le lien vers mon profil codecademy. Cela vous permettra de suivre mon évolution sur différents exercice sur divers language : html,css, ruby...
https://www.codecademy.com/users/gainde64/achievements


No problem! I stared at the code for a couple minutes thinking "this looks exactly right, what the hell?", haha.


@StrykerSKS I don't think you'll get an answer, in large part due to this quote:
> I think that I'm entitled to be rewarded for the work I've been doing for the past 18 months.

That isn't an unreasonable sentiment, but it also says something very strong about the chances for future development vs payment for past work..


https://sprnt.slack.com/archives/iguana/p1450075413000099
https://sprnt.slack.com/archives/iguana/p1450076585000100
https://sprnt.slack.com/archives/iguana/p1450076652000101


А если написать `-93` что будет?


Nope. I'm going to walk around the funny place.
If I work out the problem, message you. See you~


@floatdrop What do you think about instead fixing it in `unzip-response`?  Could do a `if (res.req.method !== 'HEAD') {}` in https://github.com/sindresorhus/unzip-response/blob/99e7e7789ac9656e016cecf4b66fe508cf998309/index.js#L5


It can be combined with lines 10-11


Hi Dennis, I'd prefer not to do this check for the existing files by default, since a user might legitimately want to overwrite the existing files. Could you make this optional for now? In the longer term I think we'll put the existing file checks into the job submission code in CombineToolBase.py so that all statistical methods can benefit from it.


Hi,

Seems there are issue without ar(php. I just put a if false in 2.06.

With ar-php :
![ar_php](https://cloud.githubusercontent.com/assets/1439428/11814174/a7419f8e-a345-11e5-9d6d-baf9e4eb8215.png)

Without : look at screenshot

![without-ar_php](https://cloud.githubusercontent.com/assets/1439428/11814179/b0b5e304-a345-11e5-9ada-aab523c886d1.png)


hi !

it seems that you have found a bug, and that you have the correction :)

could you make a pull request ? it would be grate !

thanks,

Laurent


The reason this should work is that the toggle is a boolean, and reports, "N" or "Y" to sysfs, but the app isn't looking for that.


I am far from sure on this. but it maybe better just to add a plugin to rebar 3 that takes the version from the command line. Again - far from sure.


somewhat the same here. The natural dependency directory in Erlang is whats pointed to by ERLLIB. Rebar should just support setting that, or perhaps we set it on the command line. That seems safer to me.


arquivo atualizado. porem nao consigo dar update no merge.
alguem sabe por que?


forgot to consume energy and ammo


same


this should be `on-change=onChange(newColor)`


I do not think this is the correct syntax here


I'll keep it because this silly message bothered me not even in this context.


@Shnoulle Can you put these in the bugtracker in for LS3? I think these bugs won't happen when the whole system properly uses utf8 everywhere.


good job!
待ち受けポートを自動で選ぶことを考えてましたが、このほうがいいですね！


Typo in commit subject. "These NIE areas don't have GSS codes because we don't know them.” - I don’t think they have any because they are legally identical to the Parliamentary boundaries.


> Any news on that? It will be extra cool to have it back!

Will do, I'll do a quick comparison with phpcbf first though which I've used on a few things since this.


This ignores the `.babelrc` file but it's currently not working, see babel/babel#3168.


Nitpick: this should really be `http.StatusNotFound` or possibly `http.StatusUnauthorized`.

`ServiceUnavailable` (503) is meant to signal that the service is under maintenance or high load and is a temporary error condition (like most other 5xx) - correct client behavior is to try again later, but this is really indicating that the requested resource doesn't exist at all or is blocked form access.

From RFC2616:

> 10.5.4 503 Service Unavailable
> The server is currently unable to handle the request due to a temporary overloading or maintenance of the server. The implication is that this is a temporary condition which will be alleviated after some delay.


Yes, that's clearer :)


http://codetunnel.io/isolate-scopes-explained/
Look for "Okay, I get all that. What about "&"?"


Done in c580feb


커밋코멘트#1: 좀 더 상세히 기술해주세요


i know this is outdated and all but i keep a tap on your stuff ;) this screams for a template!


@sindresorhus thought about it, but there is no mentioning about accessing `request` from `response` object (like I'm never used internal fields for this).


OK, but i think we already use UTF8.

* Number not "translated"
* English string in RTL

Updated screenshot
![without-ar_php](https://cloud.githubusercontent.com/assets/1439428/11814351/8dbf753a-a346-11e5-9b20-53b80d4f6199.png)


Then you should change the message, since I've added angle selection with muliple video streams


This can be safer too I think. For an erlang app. It will always be in _bulid/<flavor>/<name>-<version>. Where flavor (in this case) is going to be 'default'. We should just assume that is there, test for it and copy it to the the correct location. That correct location is something like '$out/lib/erlang/lib'.

An erlang releases are a whole other thing. The should be treated like any other application. That is, copied to someplace like '$out/var/..' or '$out/share' or '$out/opt' then the release start script that would be in '$out/opt/<name>-<version>/bin' could be linked into '$out/bin'.


Maybe File::Spec->catfile($tdir, "HUB-$hid") would be more appropriate?


確かに仰るとおりです。
頂いたアドバイスを元に、ユーザーネームは確認出来る状態に戻そうとおもいます。


While the tutorial works, it doesn't follow the path we setup on Saturday. So, I'm going to try to move these pieces into place. Wish me luck :smile:


Very similar comments to the above I think.


109806134-AbsChlrBlankWaterFlow (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1985 of 1985 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)


커밋코멘트#2: 전반적으로 만족합니다.
내일 봅시다.


109806134-AbsChlrBlankWaterFlow (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (911 of 911 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.88%25-green.svg)


no wtf pls


Nice, thanks!


Don't forget that files should always end in a newline.


@ironcladlou here's the test you asked for.


Partly yes - get_par_info() could be removed. But TransportDG currently does not use row_4_el from mesh, so get_row_4_el() has to be provided.


just awful


Will this hal work for oneplus 2? It has 1150 though.
Also where did you find this code?


Gotcha.  I'll keep under models but switch `Runner` back to `Use Case`...


TeamCity YaccConstructor :: YC.Core General Build [Build 512](http://teamcity.codebetter.com/viewLog.html?buildId=204256&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) is now running


There may or may not be a prototype of Rawkets in the pipeline with larger Battlestar Galactica-style ships…

![](http://cl.ly/1b1l2b3f1Q2o/Screen%20Shot%202015-12-15%20at%2015.13.53.png)


Overall this looks awesome!

Thanks @FZambia, I'll try it out soon - from quick read through it seems more or less complete. Are you waiting to finish any part of it?


hm. is there an ssl error if we try to use apiv2.indico.io


This is legacy code which was moved to a new page.


This one works properly: https://github.com/domchristie/humps


rebar update really makes me nervous.


TeamCity YaccConstructor :: YC.Core General Build [Build 512](http://teamcity.codebetter.com/viewLog.html?buildId=204256&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:01:17


OK, but then shouldn’t we reset the mean after Mdot kicks? It would be cleaner this way, right?


do you mean span?


Perfect, thanks


Whether it will work on other devices depends on the TZ API implemented by the vendor.
I didn't find this code anywhere, I wrote it from scratch.


Looks good.


> did you forget a space between Tags and First/LastReference ?

no, i want to refrence the TagsFirstReference  and TagsLastReference


TeamCity pull requests :: netty [Build 877](https://garage.netty.io/teamcity/viewLog.html?buildId=4102&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 6229, ignored: 21 Build time: 00:43:01


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


What do you mean by lately?


@cgfrost I'm not sure why this name has been changed, but lease change it back.  We will not be calling **ANYTHING** `Impl` in our code base.  First, it's non-descriptive (everything is an implementation), and second it's an abbreviation which doesn't need to exist since characters on a computer are free.


Can we check if we can omit the `<div class="product-list-pagination">` altogether?


> How are you building for ppc? I pass this to configure without issue on "CC='gcc -m32' CXX='g++ -m32' --host=powerpc-linux-gnu --with-cpu=power8", and have no issues.

These flags does fix the issue.
My bad.  I didn't notice my ppc server was using a ppc64 kernel.  That does mess up with autotools.

So, the only thing missing in this commit is the signed-off-by from Steve as he's a co-author.


Isn't the pc plugin specified in the rebar config of something that depends on it already? I would say that the eventual `rebar3 nix ...` command should eventually just parse that information and add it to the nix dependencies if required. I dislike updating the rebar config for every hex package.


seee above


Thx!


(I first though this was in case you’ve started computing the mean but finaly this became true before, but since it’s not linked to the fact the mean exists or not…)


or just figure out how to modify rebar3 to not need that if some flag is present :)


You could have mock_diskdrive = mock_wmi.WMI.return_value


Will do, I just did this at the beginning to make it look different to the Classes also beginning `Default`, helped me understand the code.


Also may be use ioutil.TempDir() function to create a temp dir on the fly.


I know this file shouldn't be here, I've already removed it locally.


I see what you did here)


Thx ! Thx !!


Looks good.


Not sure why this is in your code.  You're not working on routes.


you're absolutely right!! omg.. i'm going to fix this today.


oi


I like your solution more because in your case `maxRetry` option has more sense - value of interval never exceeds it. So am I right this is the final variant:
```javacript
function backoff(step, min, max) {
    var jitter = 0.5 * Math.random();
    var interval = min * Math.pow(2, step+1);
    if (interval > max) {
        interval = max
    }
    return Math.floor((1-jitter) * interval);
}
```
I tested for big step numbers - it does not affect as `Infinity` always `>` than `max` so function just works.


Yeah you're right. `bool`, `int` etc are all handled the same. `string`, `object` and `array` need special handling however. I'll see if I can improve it. Thanks for the hint.


ok, will do


@jfinucane to tag btw, @antonioschaffert


hum, that's awfully odd… we probably don't need exporting this if it's currently private, as it means current plugins don't use that, and it was never officially part of the API (although we do have a [documented GeanyData member pointing to it](http://www.geany.org/manual/reference/structGeanyData.html#acb6821501ac9049607d2ebef7d322300), but never had doc for the members).

So maybe rather than exporting this here we should rather fix the API before trying to convert it.  And I guess that would be removing the public pointer to this as it doesn't seem used (oddly enough).


Thx!


Teste de aplicação web.


Oohk. Thanks for the reply! :)
On Dec 15, 2015 8:47 PM, "maniac103" <notifications@github.com> wrote:

> Whether it will work on other devices depends on the TZ API implemented by
> the vendor.
> I didn't find this code anywhere, I wrote it from scratch.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/CyanogenMod/android_device_oppo_r7plus/commit/275e3c916a57ce909a87d09bf52db9dd4729e35e#commitcomment-14988149>
> .
>


Please inline this.


This one is fixed later in the branch -  as part of making things more
hermetic. Please also note that I've rewritten few commits in the pull
request.
On Dec 15, 2015 16:15, "Eric Merritt" <notifications@github.com> wrote:

> rebar update really makes me nervous.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/gleber/nixpkgs/commit/7cb1736a2afd3fcf11f7744bd66655fd84cfec8d#commitcomment-14988108>
> .
>


@erikh Is it still your e-mail?


А вот для каждой переменной получить список переменных, от которых эта переменная зависит, нам бы не помешало. Даже нет. Так лучше ->

Нужно получить набор интервалов (строка-начало, строка-конец), которые ограничивают каждое выражение, левая часть которого не является временной переменной.
Пример:
```
1. b = 5;
2. @t1 = 1;
3. @t2 = @t1 + 2;
4. a = @t2 + 6;
5. if <..> then ...
```
Здесь явно выделяется интервал 2-4, который определяет переменную `a`. Т.е. я в процессе обхода трехадресного кода хочу проверять: если начался подобный интервал, то я просто перехожу к его выходу, пропуская все содержимое, и для переменной (в данном случае - а) беру его свернутое представление, которое получено функцией `RollExpressionsToNormalForm`.

Сделать отдельной функцией, оформление неважно (все равно это потом выпилится из проекта). Считать, что у нас каждая не-временная переменная определяется непрерывной последовательностью (т.е. в середине не будет какого-нибудь `с = 5;`, не относящегося к этой переменной)


:feelsgood:


Characters are cheap. Please ensure that they name of the argument is descriptive.  `response` or `getSpaceSummaryResponse` are candidates.  In fact, once you inline the `getSummary()` call above, you should have access to the `response` argument.


Hey Joe,

Yep, it's required exactly for the reason you mentioned. FALSE is passed because it's unknown which process is initial owner, other callers will simply get an existing mutex handle. But the one who happened to get the ownership will have to release, because the purpose of this function is not taking ownership but just create or open. Otherwise the mutex will stay locked by the owner process.

The whole construct here is so primitive that i'm really scratching my head :) I will have to start more research because it seems that it might be something incompatible in the actual usage code. Say it is compatible with the pthreads stuff, but some detail i don't yet see how it makes compatible with a custom implementation.

Maybe you could answer two questions yet

- there aren't recursive locks, are there some? I was looking for that as this implementation does not support recursion, but haven't found any.
- what is the reason putting the actual lock item into the shared memory? Talking explicitly about _apc_cache_header_t.

Thanks.


Need to write something like

"Warning: umount failed: %s, retrying" % result_stderr


Ex app have the option for control camera. :+1:


seems to make sense as the type is used in public API, odd we don't doc this.  Same for `GeanyBuildGroup` and `GeanyBuildSource`.


These libs get loaded automatically when importing the project, here's the error message:
![sans titre](https://cloud.githubusercontent.com/assets/2852068/11814652/3739f7f6-a348-11e5-8a5e-c1c8e1cc9ea8.png)
Also I have to update to android 23.


Good luck. Send along a link when you have it moved and I can take a look.


That is fine, but don't do it in /tmp.  Just do it in /run.


teste


Actually, the reverse...


Your variable naming here is not very clear. The variable named `lists` and `set` does something different from what their names denote. You could use something like instead.
```   records_to_skip = limit.to_i * (page.to_i - 1)
```
which makes it clear what the values stored the variable are supposed to be used for.
Check out this [link](http://www.techrepublic.com/blog/software-engineer/variable-names-should-usually-be-descriptive/)


Oh and I've removed that user so don't bother xD


I'm still having issues, even after this.


all tests pass:
success: http://buildbot.bos1/builders/ruby-client-test/builds/424


this should maybe be `if keyserver_port`?


Sure


I don't think it's all that clumsy really, but I do think the default value for `keyserver_port` should be 80 to be backward compatible and respect environments with filtered outbound connections.


But I think the bare fixes to current doc should be done separately from adding various specifics to your gtkdoc stuff


This would be easier to follow if this would be called mock.sentinel.planned_vm_path


Not new, but none of the members of CCloneWorkflowItem are initialized in constructor, which would be cleaner.


Looks good except for the last section on mobile.


It's still good to support both and allow the end-users to decide which one they will use ;)


I am being a bit pedantic and I am not a maintainer, but I would put this into a different commit. since it has very little to do with adding the rebar3 package.


I'm afraid this null check was not entirely unnecessary. If the meter response data is not found, there is an exception, but if the type is not the right one (not sure if this ever happens) null is returned.


TeamCity pull requests :: netty [Build 876](https://garage.netty.io/teamcity/viewLog.html?buildId=4101&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 6229, ignored: 21 Build time: 01:21:35


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


you removed firewall_driver here and put it in the ml2 config file...but you didn't do the same for linuxbridge config file. Let's be consistent, either remove it from linuxbridge and ovs config file and keep it in the general ml2 config file or remove it from the ml2 config file and keep it in ovs and linuxbridge config files.


This commit is being reviewed as a part of review issue #150.


Yep!


Who is that?


This is fixing the bug.


this is a git learn


this probably shouldn't be public.  it currently isn't, and only here so the "private" `auto_indent_mode` member of `GeanyIndentPrefs` can have its type.


Nit: Drop the `,` here.


@Sodman @Butterwell Just so everyone knows what everyone is doing. We have a factory responsible for getting and storing basic user data (userId and type of user). However, we get those values on successful login or signup. Chris has a service which is doing similar things, but reads and writes with local storage, instead of depending on the login response.

These are both valid, but we may consolidate in the future.


I am adding myself to list of maintainers along with the first package I add. For me it looks like fine-grained enough :) If a maintainer of nixpkgs thinks it's not fine-grained enough, I'll split.


this is git learn


We need to use `(err error)` so that `defer action.RollbackOnError(&err)` binds to the right variable that is actually set to the current error being returned when returning from the function.


Just making the code simpler when I am at it...


Switch mock.sentinel.element_name with mock.sentinel.disk_path as that's what it will be there.


Some more class names produced by Scala running with tinylog master @ b0a5feffd1f7de3820b15efe044b7d0a2947ff14
Below: Scala language construct, resulting class name, desired class name
```
Package object:     package$ => package
Object:             Launcher$ => Launcher
Trait:              CoreServices$class => CoreServices
```
The rule that should work in all cases would still be _"strip everything after $"_ (2nd $, anonymous functions/classes (all lowercase)), except if it starts with an uppercase letter, then scan till next $ and repeat. That would strip all cases I've encountered, but keep inner classes. WDYT?


:+1: :smile:


1. "Use this option to for each file" - what?
2. Don't you see that line length is limited with '\n' here?
3. You need to indent "Use" better. Look around here indentation is a little different.


Linking in deps seems problematic to me. It would be safer to get rebar to take deps from `$OTPROOT/lib` or  in a slightly more difficult case '$ERL_LIBS'.


This would be easier to read if you'd put this dict in a variable such as 'expected_disk_paths'


Why have we changed `coefname` to `name` ? what about compatibility with the previous archivist versions?


probably shouldn't be doc'd, and the reference to it labeled as "internal" should be dropped, as it's both useless and incomplete (as it misses this).
BTW this enum is dangerous, as its names like NONE are likely to conflict…


The reason here is just for logging purpose on client side - client always stops reconnecting after receiving this message at moment. But I think boolean advice here is a good idea - will add it.


In the case of a compiled erlang Application. The built app will always be in './_build/default/<name>-<version>'. You should just assume that is there and copy it to '$out/lib/erlang/lib'. That should do everything you need. Though there *might* be some need to add that directory to `$ERL_LIBS`


We have removed such tag ( I mean `date` tag)?


Might be automatic emacs file processing or something like that... didn't do it on purpose ;)


IMO the quickstart should be all positives. Just the simplest shortest way to get going. It's important to keep the initial experience positive.

This information you added is great ... I love it ... but I suggest we push this to another doc, like a FAQ for getting started. And we instead provide a link to it.

cc / @wardbell for thoughts


yep!


@antoinemarchal?


`name` perhaps is the tag of artifact's name.
`coefname` should be a tag of explanatory variables names


This section explains how to add one or multiple images to a table subreport.<br/>


What's with that date? Why it is removed?


> This was actually for testing whether submitting non-full-date dates work. I believed they should but looking at the schema again it doesn't seem like it.

Well, the tests can only test what is valid. You can always monitor the Travis status after opening a PR or pushing to one, by the way. And you are right, providing only year and month was not. Fixed this in https://github.com/hbz/oerworldmap/commit/046ecb9eb996a836c7f9d200a3dd53a8d7b10863.


Another one, inside:
- https://github.com/gramps-project/gramps/blob/master/gramps/plugins/graph/gvfamilylines.py

There are still issues in the Gramps code with calls to the following code
- get_person_from_handle (2 occurrences raise an exception)
- get_place_from_handle (3 occurrences raise an exception)


[uploader] [Tandem] ensure jellyfish doesn't crash uploading during an extended bolus


@mnewnham lines 263-267 are now dead code since you're returning unconditionally.

Considering http://dev.mysql.com/doc/refman/5.7/en/mysql-insert-id.html, is it worth trying to use mysql_insert_id() first, and only fall back on SELECT LAST_INSERT_ID() if it fails ?


Why have we removed `alternative`? What with the compatibility with the previous archivist versions?


Addressing @justcfx2u's comments on this doc. https://github.com/mbruzek/docs/issues/41#issue-121137912


type: `QueConf`


I think `indexOf` can return `0` and still be a match (`Twenty Ten` is at index `0`). We can probably use either `( ~ this.getDefaultThemes().indexOf( name ) )` or `( -1 !== this.getDefaultThemes().indexOf( name ) )`.


mir


in seconds


Well, rather than lower it afterwards, we should probably just increase it a bit less in the first place.


In a first step a table needs to be configured. For this, the process !preparTable is used. The column containing the image needs to be defined which is done using


sfsdf


iiir


Uhm, this is now fetching all ufront modules when cloning haxe repo. not a good idea I think.


Since this string is an enum, please document the states here. i.e. no, yes, no-toilets.


Avoid replacing this class with a mock object, use a mock decorator instead.


`In the example below all columns from the original table are included in the printed table without formatting.<br/>`


:+1: nice


This is due to the new website merge. It has ufront submodules in it for some reason, I think they are not needed nowadays when there was ufront lib releases, but we have to be sure. @jasononeil could help with that I think :)


Anda ya ni friegas "Repocitorio"???? Repositorio analfabeta


This does not look like a good idea. You will most likely end up with multiple IDs with the same value. What are you using these IDs for anyway?


please use unsigned long here.


`To add a subreport in form of a jasperreport template containing all of the columns that are to be included in the printed report. This requires the dynamic flag in the process !prepareTabel to be false.


Haxelib does not seem to run anymore after "make install"


Was trying to build openchange against latest stable samba 4.3 and got stuck with error: ‘struct dcerpc_pipe’ has no member named ‘assoc_group_id’    dce_call->context->assoc_group->id = private->c_pipe->assoc_group_id

Thanks to your fix my build was finally sucessful.


```apcu_entry``` is the only place that "requires" a recursive lock, but can just as well function without one, native recursion is used if available.

The cache header has the lock because it's per cache, not per process; if there is two caches in a process (which is well supported in non-windows OS) we need each cache to have a lock, rather than share a lock between them.


I just happen to be updating haxe on OSX, bad timing :)


@tresf As such? 26e43c7


What do you want to achieve with this? With the current code, you are setting this preference per user's browser. However, you are enabling/disabling it below per-element, but using the same key, so the effect is that each enable/disable you do does not target the particular element the checkbox is attached to, but the entire page (browser preference actually).


Good job!


Why? The return value of a mock, if not specified otherwise, will be a mock object anyway


Full access to the time governor is needed - I will modify EquationBase::time() to return non-constant reference.


間違えるほど上達するので、どんどん間違えていきましょう！


`paginate` and `check_result` methods are methods that does not belong to this controller. You could handle pagination at the model level or you could create a PORO class to handle pagination and search.


Read the note above, what do you want to achieve?


Oh, this is ES6, so we can even do something like `this.getDefaultThemes().find( name )` or maybe `this.getDefaultThemes().includes( name )` (haven't tested either, but they should work).


Remember to use implicit `return`!


As I've mentioned before, this does not make sense. Use different mock objects for the conn objects to ensure that those are passed properly. Otherwise, those may get swapped in the future by mistake.


In a good C code I would make char ** static array with all the args[0]-[6] and use its number of items in all the calculations around here. But ok let's check what CRIU guys would say,


Looks like this isn't needed, is it?


You've missed one method "_update_planned_vm_disk_resources". Please add assertions on all the called methods. Also, add one for "_get_ip_address_list".


Well, apparently, this was the first commit after the initial commit!


/cast summon @deadsquirrel


if (!img) then ret = 0; cleanup and silently fail?


We don't have time to do a cleaner solution than that. If we do this, we're pretty sure we'll go back to the initial point, since there is no evidence of hysteresis in our system.


Title of commit is a bit off : this a sensor dedicated to pom.xml


usually you don't push logfiles to git repos; instead, you are expected to mention it in .gitignore file


I think parsing dm.devicesize in generic layer code is a problem. Name suggests that it is a graph driver specific option and if that's the case then parsing should happen in graph driver.

I think there are two ways to handle it.

- Either create a generic option for size which is valid for all graph drivers. For example, say --rootfs-size. Parse this option in generic layer and pass size to graph
drivers.

- Or Pass all the storage option strings to graph driver and parse it in graph driver during actual Create() call.


> note, in the case where __excepts != 0, and popcnt(__excepts) == 1 (guaranteed by the macro), __builtin_ffs = __builtin_clz + 1 = __builtin_ctz.

I expected something exactly like this if you exchanged clz and ctz.
I may be missing something, but this is what I get:
    __builtin_ffs(8) = 4
    %s(__builtin_ffs(8)) = 28
    __builtin_ctz(8) = 3
    __builtin_clz(8) = 28


That should be exactly identical to the current implementation. I would check to see if the winner is actually nil on the specific game affected. Also, just a nitpick but Game.active_games seems a bit redundant to me. I would call it Game.active.


Cool! I'll fix your notes and then will merge this into master. There is the last part of #44 left - do not show login screen in case of insecure web.


Why do you have this line here?


Good idea, I leave that to you.


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1448 of 1448 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)


This should be named after pom.xml it does not concerns xml files but only pom files. If it will change, then we'll rename.


What does it say? :)


Given that rootfs size per container seems pretty generic and valid for all containers, how about introducing --rootfs-size option for docker-run. @rhatdan WDYT.


I don't understand the comment about "connection over a proxy to a proxy". I thought that port 443 was required, but I could be wrong about that.

I felt it was better to have a full example of how to setup Let's Encrypt support so there wasn't any confusion. Why is it confusing to have the `-v` section?


Goto to skip 2 lines of code?


Hmm. Do you have any recommendations? My thinking was that if we ever do decide to use HTML with languages other than JavaScript in browsers, we would change this sentence. Maybe just prefix this sentence with "At this time"?


While this makes the test pass, will this solve the problem in mongodb?  The mongod process is not going to be closing and reopening the connection.  Would a force checkpoint accomplish the same thing (and I thought compact actually did checkpoint every iteration of its loop)?


Keith should we assert that the size is still greater than fullsize here, after the reopen?


OK


To make this better, you could have an argument such as 'valid_device_name' and
fake_phys_dev_name = r'\\.\PhysicalDrive1' if valid_device_name else ''


TeamCity TripleA :: UnitTest [Build 540](http://home.k-moeller.dk/viewLog.html?buildId=1406&buildTypeId=TripleA_UnitTest&guest=1) is now running


@sirthias Is this the way to nest unmarshallers?


You should really consider sticking with using the singleton to retrieve the appropriate IDs.  While very unlikely, it is technically possible for the IDs to vary between Magento installations


Oh, I think I know the problem. Hold on


rename this to "test_get_device_number_from_invalid_device_name"


TeamCity TripleA :: UnitTest [Build 540](http://home.k-moeller.dk/viewLog.html?buildId=1406&buildTypeId=TripleA_UnitTest&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 205 Build time: 00:00:45


Could it be 3053e1a38f0c2bcaf56743eec97af6c028dcdf46?


@lazyguru Can you make a pull request ? We will be happy to merge it


Не вижу дублирования.