Both are correct... cache clear is very complex (especially with any level of efficiency, which the current code is okay but not great with), and this code does not handle new records that are joined in a view entity with existing. From the scenarios I'm working with I haven't run into this as a real-world issue, just a theoretical one, so I have no plans right now to add something to handle that (and it would be messy and slow...).


I would be great to use the global location so we could avoid the re-download.


Great comments in `mulRatio()`.  That helps a ton.  Nicely done!


I haven't changed the visibility of these methods.
If you don't specify visibility keyword, it defaults to private in C#.
And generally, the shorter the code is, the easier to read and understand.


But we'll get this for free when we switch to the dotnet-do, so I'll wait.


Right, so _if_ one requires the use of _new functionality_ in the new release of Ceylon x.y.z, then one needs to compile one's module with Ceylon x.y.z. Because, well, it's new functionality! But that's not part of _backward_ compatibility.

What you're essentially arguing is that we can never add new `default` methods to interfaces in new releases of Ceylon. That can't be right.


Ok, good :)


Cool feature


Thank you for this patch, however this warning is required for when people upgrade. I do not want to leave people wondering where healthcheck went :wink:


The idea of that custom field is if they want to copy it to a txt file send to their friends and their friends come back with the final thing and they just paste it there.

Maybe if they are using this the selectors above should be disabled, or otherwise if you change something in the custom one, reflect it on the pickers above. Not sure if I'm explaining myself properly.

Maybe this custom one could be shown only in a modal window if users select "Insert from template".
@geriux what do you think?


I will definitely edit this!


This is based on http://blog.arkency.com/2015/12/stable-circle-ci-builds-with-phantomjs-for-rails-backed-larger-frontend-apps/


Why is the structure type being exported?  I wanted this to be an abstract data type.


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


Функция ничего не меняет в классе, должна быть константной.


Currently they are accessed through GPG. If you manually change the engine back to `kbpgp` then you still need to put the keys at `$HOME/gpgkey`. Secure key storage is hard, unless I keep the keys decrypted.


Ok thank you! here you need a better name for this one, but the first thing to do is to add more context to the PR so we can follow what's happening! and what's happening with the ssh library, plus perhaps the fix should not be here but in the caller ? so the caller of `ForwareAgentAuthentication()` should be smart enough to know if this was already called ? so we don't add this boolean variable ? or perhaps pass a parameter ?

Thank you!


is the comment still relevant?


Dette ser kjempebra ut! Stå på!


This commit seems to have introduced a syntax error that is crashing the futurize checks. Perhaps line 236 or 25?


Change State Type 13 uses players view angle (in world coordinates)
If player is looking away from Diego it plays different taunt than when he is looking at him.


Should that be renamed to `curLine` or even `line`?


Oh .. So it is back :+1:


It might be worth adding a comment what we gain for the increased startup time though?


Why do you feel that comment is obsolete? I think, the usage of `static` might be irritating?


This should not happened...
But if this happen we should make this more visible.
I mean adding a `log.warning` or maybe even raise a runtimeException.


The values/colours that appear in the columns are actually updated with the values pasted in the custom textarea. (Of course you need to save to see them updated.)


change s2ga to s2adc


I think, the `TODO` is not a TODO for the return value, but more for the method implementation itself. Maybe the TODO should be moved inside the implementation? However, a method comment could warn the user :innocent:


errr, this is a guess...


I prefer `ServerIdentity`, what about you ?


How can I *possibly* know if some code I pass a `List` to might call this function!!! It's impossible, unless I code review *every other* module I use, to make sure they don't use new features. And then, what's the point of the new features?

And now you're arguing against static typing in favor of rolling the dice on runtime errors.

> What you're essentially arguing is that we can never add new default methods to interfaces in new releases of Ceylon. That can't be right.

Yes, for frequent patch releases, until a better solution comes along like Java 8 default methods.


Wow ! Congrats!!


We'll soon need to adopt a proper policy for changes. Everything goes through hydra and PRs and for some parts documentation should be required. It's a lot of hassle, but it's better than current state.


TeamCity jonas-requests :: WindowsBuild [Build 28](https://build.neohq.net/viewLog.html?buildId=573911&buildTypeId=JonasHaRequests_WindowsBuild) is now running


Why not left somewhere as comment that parsing of the institution is missing?


Prior comment got wiped out:
use ```std::``` prefix, calling cstdlib functions without the ```std::``` prefix is deprecated in C++.


Maybe we need some kind of process for bigger decisions. Perhaps only partially specified.

Note that the problems are typically with finding consensus, not with anything Hydra can solve.


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


Zéro


Body of HEAD commit contains too long lines.

*GitCommitBear, severity NORMAL, section `commit`.*


Using bash is already in a previous commit.

Coreutils isn't yet, but that can also be added.


это  нужно перенести в модель, и заполнять перед валидации


For size not a multiple of 4, this code is dubious.  This rounds down, then read_command also uses size/4 rounded down, but memcpy uses size.  Some items should be rounded up and/or converted to byte counts, or it should be asserted that size%4 == 0.


> Note that the problems are typically with finding consensus

Yeah, see all of the abandoned PRs that attempted to do this sort of thing the other way.


сообщение переносим сразу в config/locales


Missed removing this pragma?


Okay, you're right: because this is a problem with the group info file rather than the AST, it's worth leaving it with the code that handles the group info file. Thanks for explaining all the relevant info!


1) 아무런 객체화 되어있지 않은 상태로 DB연결, insert, select함.
2) 오라클 연결시 ojdbc14.jar 파일이 있어야하므로 반드시 pom.xml에 추가할것!
(주석처리한 부분 참고-> repository와 dependency)
3) 오라클 연결은
	Class.forName("oracle.jdbc.driver.OracleDriver");
		Connection c= DriverManager.getConnection(
				"jdbc:oracle:thin:@localhost:1521:orcl","scott","tiger");

이며, 참고로 다음과 같다.

> Connection conn = DriverManager.getConnection(
>  "jdbc:oracle:thin:@아이피:리스너포트:SIDl", "오라클 아이디" , "오라클 패스워드");

다른 dbms를 사용한다면 달라지니 그때마다 참고해서 사용할것.
4) db_inform.txt는 테이블 정보 적어둠.


Поменяй комментарий на следующий:

Check settings in WP Admin > General > Reading section.
It's recommended to have WordPress Address and Site Address pointing at the same URL.
Otherwise we can have an issue when WordPress set authentication cookies for WordPress Address only.
In this case users can't edit website via front end as they not logged in as admin there.


DB 연결부분 - 오라클


d'oh! Thank you, @jramstedt , I didn't think of this option; I tested always with the same orientation...
This would also explain to me why the triggers are set up in a star formation in the map; Most likely to give a visual clue.


"Отмена" нужно перенести в config/locales


При создании объекта класса с данным конструктором в поля класса не занесется никакой информации.
В дальнейшем при использовании различных методов результат операций непредсказуем.


ojdbc14.jar 추가 - 메이븐이기때문에 pom.xml에 적어두면 자동으로 추가 된다.


Opening a PR beforehand can at least avoid the revert ping-pong. Maybe create a github label for these "bigger" or potentially controversial PRs and issues. (I find it's good to probe for consensus *before* putting too much effort into an implementation.)


Поменяй комментарий на следующий:
Check settings in WP Admin > General > Reading section.
It's recommended to have WordPress Address and Site Address pointing at the same URL.
Otherwise we can have an issue when WordPress set authentication cookies for WordPress Address only.
In this case users can't edit website via front end as they not logged in as admin there.


Why are all of these authors here? They weren't in the original commit: https://github.com/edx/edx-platform/commit/5d8830096a6ca97b0d48cf562430d360ce101cb3


TeamCity jonas-requests :: WindowsBuild [Build 28](https://build.neohq.net/viewLog.html?buildId=573911&buildTypeId=JonasHaRequests_WindowsBuild) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 0:5:10


I think the symlinking of the theme's `public` dir to `public/alavetelitheme` is actually obsolete now - it was a way of making scripts and images accessible before we had the asset pipeline - so we probably don't want to introduce more references to it here. Could pull the theme from `THEME_URLS` in `general.yml instead?


This is important! Even if this exception is not declared explicitly, it is thrown if the encryption is not in place in the JRE.

I think, `src/test/resources/encrypted.pdf` is a PDF raising this exception.

Please do not remove this check (or please make it better) as not dealing with this case results to bug reports. Currently, I can only find https://sourceforge.net/p/jabref/bugs/1257/, which relies on this error message.

A inclusion of org.bouncycastle libraries does not really help as some distributions of JabRef might not include these libraries due to export restrictions.


There's a PR for it https://github.com/aspnet/Mvc/pull/4261...


I understand that it's imperative to maintain quality, and ultimately, someone has to make the call on whether or not something passes muster. And, of course, NixOS is @edolstra's baby. So I'm not terribly upset (although I did commit my weekend to the task). If I made a mistake, I can accept that.

Of course, I'd also like to have a protocol in writing that I can follow so I can make sure that I can make contributions without fear of having "omg, did that actually get committed" reactions.


2 cpus or GTFO.


Time for a new progression test after this amazing contribution!


Поменяй имя функции на
dslc_check_important_wpsettings


We should think about how to simplify the footprint of these methods. We should provide a few extra convention-driven versions that have less arguments and will be used 80% of the time.


`./script/current-theme -h` doesn't show the usage, it gives an error message `Option -h requires an argument`.


Yes @isymchych , if i change the email, also the avatar changes.

My **Jitsi **is **Ubuntu 14.04** - version 2015, the folder "**Modules**" and the file **avatar.js** isn´t in the folder:
/usr/share/jitsi-meet

I have in this folder:
analytics.js   css              google8227355449000794.html  lang                            sounds
app.js         external_api.js  images                       libs                            title.html
chrome         favicon.ico      index.html                   plugin.head.html                unsupported_browser.html
conference.js  firefox          index.html.2                 plugin.header.text.html
connection.js  fonts            interface_config.js          plugin.welcomepage.footer.html

I want to change the image of Avatar, with own mine or another topic.

Do your code do this or another code?

Thanks.


Oops yeah, that should be `":h"`


Nit: remove white space after !


I think you can remove `$count` altogether.


This did come out of a PR though.

Anyway, I'm not saying any particular instance (including) is right or wrong. I'm just saying that revert-centric development with @edolstra saying "I'll check back later to see if this is worth unreverting" doesn't seem like a sustainable or motivating contribution model. It's happened several times recently and is frustrating for all involved.


I'm sure we can find a way, but not as long as everyone just cares about getting some stuff in. Every line that's added we have to maintain and the tech debt can get bigger.

The problem @edolstra has is overload of things to pay attention to.

For this particular issue I think it's possible to solve/conclude, but someone has to step up and prepare documentation for standardized package generation logic.


[This line](https://github.com/wordpress-mobile/WordPress-Android/blob/bd0cce15d81a3c738967cf9034ffd468adaa0188/WordPress/src/main/java/org/wordpress/android/ui/prefs/SiteSettingsFragment.java#L725) also needs to be updated with the zero string.


4.1 Version with appointment function


I think the general assumption is that drawCard is not broken. If it was, it would be very obvious in even playing a simple Dominion game.

Additionally, it only returns -1 on times where the players deck and discard are empty when the player tries to draw a card, which should fail quietly anyway, since there's nothing to be done for the player in this case.


Caps ?


[R/segsites.R:62:1:](https://github.com/statgenlmu/coala/blob/7d9242a08745f455832d785c09d3a283844a135a/R/segsites.R#L62) *style:* **Words within variable and function names should be separated by '_' rather than '.'.**
```r
as.matrix.segsites <- function(x, ...) get_snps(x)
^~~~~~~~~~~~~~~~~~
```


Before:

> ![screen shot 2016-03-10 at 12 07 37 pm](https://cloud.githubusercontent.com/assets/59118/13677631/c085cec8-e6b8-11e5-878e-b594046ef9c7.png)

After:

> ![screen shot 2016-03-10 at 12 06 31 pm](https://cloud.githubusercontent.com/assets/59118/13677630/bc674632-e6b8-11e5-99c1-8a9208a1aa73.png)

https://history.state.gov/beta/historicaldocuments/status-of-the-series


So if I change and paste and save it will change the top values?
Not sure if that will be immediate to the users.

I am thinking of having something more like:

1- "Press here to add custom template";
2- Show modal box, with pre-filled;
3- Click option "Apply"
4- Update colour pickers to match.
5- Save page.

This creates a couple of questions:
0- Does it even make sense?
1- what do you think of this flow?
2- What does Dani think of this flow?
3- How long would something like this take?


把这个加上吧！省的来回修改


Ahh mach das mal in den Konstruktor sinnlos dafür eine neue Methode zu machen


call guards = making sure that the block after a call has a unique predecessor


I'd prefer a normal class with 3 methods


you can use latest from npm.


yes i can see that


@SYSTAP/bigdata-gpu-write-access This change was causing the build failures for cnmem.


Thanks kumajaya


Deprecated?


that's weird that the `dict` constructor is wrapped in a list (I assume that's what `[]` is for) but it wasn't before


It's not always easy to get your hands on such a  `ResolvedJavaType`. That's the reason why `type == null` means `j.l.Object` in `AbstractObjectStamp` for example.


This commit should reference #285 - I'll amend the message to make it more useful.


getting back to the original question (maxime isn't replying, so I've had a look at it) : yes, in this situation, like all other uses, if the enqueue returns <= 0, you need to discard the frame.
If you didn't, you'd be leaking memory. The cpy is allocated, and if you can't enqueue it, it goes out of scope and you've lost all references.
So your refactoring solves a potential memory leak :-)


Yeah, that looks wrong. I'll look in to it.


documentation und ``[XmlIgnore]`` sonst kommt der tag mit in die .xml datei


Actually removing the flag is difficult, because it's the adapter that sets the property. So yes, I could remove the setter, but not the getter.


great it worked with netbeans!
i was able to build and install the plugin, but now i don't see anything in jenkins to configure/use it. Should i do it directly in conf files?
Thanks!


> For this particular issue I think it's possible to solve/conclude, but someone has to step up and prepare documentation for standardized package generation logic.

Ah, my reasoning was that this doesn't _require_ packaging automation at this point - the revs/shas are just as easily edited after the commit as they were before. It was intended to be staged process, where getting things moved out now will make the job of switching to automated packaging much easier. If the go packages quadruple in size, that's going to be much harder to try to do this sort of change.

Also, I think this was clear, but perhaps I should be explicit: the commit didn't change any of the versions used.


Well put current Qt (5.5.1) in any case is not compatible with latest angle


TeamCity jonas-requests :: WindowsBuild [Build 29](https://build.neohq.net/viewLog.html?buildId=573940&buildTypeId=JonasHaRequests_WindowsBuild) is now running


Wäre schön wenn GameConfig nur 'Configuration' heißen würde, nur ein nice to have


In this case the variables can't be extracted because the if helps to prevent a Index out of bounds in the selectablePhases.


@nfeske now having fixed the signalling issues I'm back at the exit and it seems I first have to get the libc task context right to actually support *return from main* is like exit().


Ok, haha, I think I'm gonna wait till you guys have decided what's the supposed behaviour od this feature :D


если у юзера нет gcmid - падает с исключением


Is this correct? It seems to me that moment-timezone returns a string with the timezone name. I'm not sure if that's a change since this was contributed, though.


TeamCity LabKey :: 16.1 Release :: Community :: CDS postgres 9.5 [Build 40](https://teamcity.labkey.org/viewLog.html?buildId=309475&buildTypeId=LabKey_161Release_Community_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 32 (21 new), passed: 18 Build time: 00:59:02
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.cds.CDSHelpCenterTest: <no details avaliable>

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

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

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

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

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

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

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

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

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

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

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


Hi Alex,
I was about to suggest using this to Nate with regards to the "Embedding a syntactic rule inside a lexical rule". I couldn't find a test for it or any other thing we could point him to. Can you add that?


Sorry which jira are you referring to?


I think a modal box or something would be best to avoid like Adrian mentioned, multiple calls etc.

Sort of "Import / Export" buttons and then update if changed, also it would remove stuff from the current view that it's looking pretty packed already.


wenn du in ``deserializeConfig`` GameConfig zurück gibst musst du das hier nicht casten


stop.get() might also hang if docker daemon somehow deadlocks (which happens quite often in the past).
I think what we need to do is that we should issue a stop with a timeout, but perhaps don't discard() the run if we know it's already running.


Another Question, how and how often will changes on http://codecombat.com/i18n become visible to the game?


@vincent-tr excellent!


does the secret start after 7 days or first runs and then waits?


Looks like a bug. I'll fix this.


![success_baby](https://cloud.githubusercontent.com/assets/163977/13677875/f7ec2cc6-e6b9-11e5-9b6a-52f5e22b5a28.png)

getters and setters ftw.


Looks good. Will push to master shortly.


wenn du das deserialisieren auslagerst solltest du das mit dem serialisieren auch machen


> This did come out of a PR though.

I see now – it got no reactions at all in several days. We merged over 500 PRs in the last 30 days and there were hundreds of issue tickets as well... that's why I wrote a label might help people find the "important" ones more easily. (I added "6.topic: policy" some time ago, so maybe that might be reused, at least for some of the cases.)


Поменяй текст на:
**Live Composer**: probably there is a problem in your website settings. Click here to find out more.

Click here to find out more должно быть ссылкой которая ведет пользователя на станицу настроек LC. На этой странице настроек ты должен вывести краный notice выше табов:

Please, check **WP Admin > Settings > Reading page**:  Wordpress Address and Site Address fields should be the same. Otherwise, you will not be able to edit your website in the front end.


I read it first in the article from @rosstuck that @Ocramius mentioned. Indeed an awesome way to handle exceptions.


Leave a comment


YAY! Good plan!


I'm sorry, that was of course non-sense :) But it would work to extract a variable for the whole term, which would make it more readable I believe


Thank you for your contribution here - I think you could make it slightly easier:

```
if (DurationElapsedEvent != null)
    DurationElapsedEvent(_filesPlayed);
```

Rather than the try/catch


Why does it have the object key `website` twice?


nice comment


blank?


Yeah, I think this is wrong as well.

Might be helpful to have macros for the discard positions so that these types of things are easier to find. I'll refactor this.


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


yea, right now I am not sure in anything.

https://dav.saraev.ca/pub/1080i50_h264.ts
https://dav.saraev.ca/pub/1080i50_h264-2.ts

@codesnake those two work for you ? ^^


lgtm


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


Why?


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


Ты забываешь ставить нижние подчеркивания между словами в названиях методов шагов и в методах тестов


What does a context represent? From my quick read it looks like a representation of a layout operation that can be applied by the display node. Since there are the two `apply` methods in this class, I'd recommend coming up with a class name that better describes the responsibility here.


Нет смысла иметь строку такого вида __( '', 'live-composer-page-builder' )
Запомни функция __() используется для перевода строк на другие языки. Как ты можешь перевести пустую строку на другой язык?


The upstream GH repo points to that URL in the "website" block and the bottom part of that SF page also includes a link to it:
<img width="311" alt="screen shot 2016-03-10 at 17 19 04" src="https://cloud.githubusercontent.com/assets/6998367/13677961/357cccce-e6e4-11e5-90a0-1c9e3b1d9aed.png">

My reading of that was "The source code has moved to GitHub" and the homepage has moved to http://librsync.sourcefrog.net/. By all means, feel free to ding upstream to see what they prefer the homepage to be.


I'm avoiding mallocs


Might suggest either dropping `previous` or renaming `pending` to `new`, as the `previous` values still represent the current state of the display node.


It seems this is not the right returnCode, error message and comment ;)


Epic diff!


Should we rename the method for consistency (`WithinTransaction`)?


New commit coming up!


Check important settings in the W3TC plugin
to make sure it doesn't break our page builder
with unnecessary page caching or minimization for logged-in site editors.


The test suite finished running the test files. Below is a summary of the test results:

| Report name | Tests passed |
| ----------- | ------------ |
| _Broken | **11/13** |
| _DVB | 5/5 |
| _DVD | **0/3** |
| _DVR-MS | **0/2** |
| _General | **11/27** |
| _Hauppage | **0/3** |
| _MP4 | 2/2 |
| _NoCC | 10/10 |
| _Teletext | 21/21 |
| _WTV | 13/13 |
| _XDS | **30/34** |
| _Options | **82/87** |
| _DTVCC | **11/14** |
It seems that not all tests were passed completely. This is an indication that the output of some files is not as expected (but might be according to you). Please check the [results](http://ccextractor.canihavesome.coffee/reports/169) page, and verify those files. If you have a question about this report, please contact @canihavesomecoffee.


(Comment 1) This if block could go above `if defer`...


Try https://github.com/kd-11/rpcs3/tree/vulkan-nvidia-tests
There should be no display, just blank screen with color.


please configure your editor to use only spaces, not tabs


(Comment 2) ...and this if block could then be removed.


All bug fixes addressed in commits 261bf79, 9969399, and 1622bce.


bien jugado


should we have any distinction on the nature of the stoppage?

perhaps:
`void onStopped(long token, Reason r)` where `enum Reason { FINISHED, CANCELLED, ERROR }`

or:
`void onCancelled(long token)`
`void onFinished(long token)`
`void onError(long token)`


delete not create


This is a nice cleanup.


Is the concern primarily due to the diff-size of the change? If so, this problem could probably be avoided with some sort of heuristic, like "if the diff touches 'many' package (for some value of 'many'), or includes 500+ish lines, get 3 'LGTM's before merging".

My personal heuristic has been along the lines of:

1. Does it introduce backwards-incompatible changes to existing packaging functions? (no)
2. Does it cause massive rebuilds? (no)
3. Does it introduce a dependency on some sort of automation tool (e.g. `cabal2nix`)? (no)
4. On the corresponding PR, has anyone raised concern in a couple of days? (no)

I'm not saying those points _should_ be the heuristic followed; I merely provide those points as an insight into my reasoning, for whatever that may be worth.


The situation is exactly the same as if the library you're using implements the new method and throws `UnsupportedOperationException`. This does not belong to the class of problems that a static type system is able to prevent.


@soltysh I'm sorry about this.


It seems you forget to remove this ;) (looks like debug code)


Clever! For readability though, what do you think of this?

```
it("returns the initial state", () => {
    let state;
    expect(
      reducer(state, {})
    ).to.equal(null);
  });
```


I don't think the size of the diff is a problem, but I can't really speak for Eelco ;-)


You need to spell out "chili powder", else people will think you are Jessie Pinkman.


Should we make these positions public docs before linking to them? I'm happy to do that. Then we won't have to link to that kind of sad site.


looks great!


@carlesjove I tried this and it didn't seem to work. :+1: waiting on new release


That seems clearer.  Sam, will you do that?


@danielsuter I think you should check your git settings, seems like you are commiting as @adiherzog - maybe that is also why @adiherzog has now become the top contributor of Scenarioo ;-) (I feel a little bit jealous)


> I don't think the size of the diff is a problem, but I can't really speak for Eelco ;-)

Alright, well I'm going to open a new PR, so feedback should probably go there. I'll drop a link here in a moment.


Years of Obj C development and this still gets me.


Why in the fuck is goin on!!


Yup :-)  I've already removed it.  Am slowly refactoring it atm.  I think, however, that the algorithm needs to be rewritten.  The examples were brought up to the point that most of them work in current Perl6, not that they match current Perl6 usage.  Mind you, many of the examples are several years old, so it's not surprising that they're not come up to current standards.


isLwM2mServer() ?
(This is not really a great name but this is the name chosen by the specification ... :'( )


The name is a bit confusing IMHO. How about `setParentInvoice`?


Shouldn't this be {0, 1, 4, 5, 2, 3} ?


Сначала проверь если любая проблемная настройка не активна и выведи след сообщение в виде notice:

**Live Composer:** Looks like you are using W3TC. It's an amazing plugin, but you have a problem in the plugin configuration that can break Live Composer functionality. Click here to find out more.

При клике на **Click here to find out more** пользователь должен перенаправляться на страницу настроек Live Composer где будет более делатьный notice с описанием проблемы:

To make sure Live Composer and W3TC plugins work well together please check the next options in W3TC settings:

- Performance → Page Cache → General ( Don\'t cache pages for logged in users ) – should be selected
-  Performance → Page Cache → General ( Don\'t cache pages for following user roles ) and select user roles: Administrator, Editor, Author, Contributor  – should be selected
- Performance → Minify → General ( Disable minify for logged in users ) – should be selected
- Performance → Database Cache → General ( Don\'t cache queries for logged in users  )  – should be selected

Из вышеперведенных строк показывай только те что нужны.


Yes, I think it's time to start switching to v2.
I'll merge the java client as soon as I finished implementing all the main commands.
Consider that the protocol spec is ready but some of the new commands commands are still not implemented (or partially-implemented) server-side.
I'll add some details in the documentation and update the TODO file.
I hope I'll be able to work those out in the next few weeks.

> On Mar 9, 2016, at 5:59 PM, Damian Gryski <notifications@github.com> wrote:
>
> Should I move my v1 Go client to v2?
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/Shardcached/libshardcache/commit/f63c44cc388e93d5affd9c6754592d81f7f87668#commitcomment-16594803>.
>


This class deserve some javadoc. Particulary about "System call".


please keep such unrelated changes out of the commit


The exit code of the script is 0 regardless if starting everything succeeds or not.

The output though is logged at the agent_0.out.


I don't think this is the right way to determine a dev/ no dev install.


That being looks like it will do it. I will create an index to test that today


what a fuck?


This check isn't necessary `None in FONT_MAP` is False


Please make the `operator bool()`s `explicit`.


This fix #555 ?


2080-delete-terminal-with-coil (kbenne) - i386-Windows-7-VisualStudio-12: Tests Failed (2045 of 2398 tests passed, 0 test warnings)

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


What does an example log message look like here?


should this be `int64_t` instead? The spec describes it as a signed type, rather than unsigned.


Ah - needs check for `-d` as @daserge mentioned


jeje, just for the listeners :)


Se can try to use a relative axis. In stead of number of species in each category, %of species in each category. For `facet_grid()` you can use `axis="free_x"` or something like that.


Just so I make sure I understand, we do not have to check to see if the run engine is running here if we first check to see if we can pause, since the only state we can pause from is running?


No, see comments here: https://github.com/moonlight-stream/moonlight-common-c/blob/master/limelight-common/Limelight.h#L126


Not a fan of short-circuiting `std::isnan` checks everywhere. I think this is likely to cause bugs to be ignored. What's the motivation?


I like to avoid double negations. Instead of `skip-symlinks = false`, I would introduce a configuration option `establish-symlinks = true` or something like that.


I really don't like ExtendedIdentity.
ServerIdentity could be better but as you said System is not really a server.
Personnally, I'm not so disturbed by this as `System` is a kind of HACK, so +1 for ServerIdentity


bug: out of range if limit > #domains


Except for a very limited number of cases (I remember one case with JDBC), the Java universe avoided adding interface methods until default (defender) methods were added.

And, you know the `UnsupportedOperationExceptions` in `java.util` are total crap that Ceylon fixed.


I reverted the commit in order to put the "title" class back, and I kept the name "title-background"

After looking through the changes, it looks like the Titles were initially inconsistent. Some of them, like on edit_globals.php used `<p><b>` from the start, and some used `<span>`. In cases where title-custom was originally added to these span titles, they were converted to `<p><b>` like the other titles.

So you're suggesting that I revert this and use the bootstrap theme to style the spans as `<p><b>`, correct?


Definitively


silent hides errors, so we tried --no-progress instead


A comment inline on the commit.


test test 123 new at pullrequest


But current code  QImage(pixels,w,h,QImage::Format_RGB32) -work.


Pourquoi tout ca alors que tout est dans bower ???
Tu update le fichier bower.json


Pareil !


Testando pull request


fix charnt by charsnt in Sample


Do you mean "always" in the sense that the information is lost for LF eol style or something else?


Yes, I'll change that for you :)


The former.


Single quotation aka literal quotations will not escape \n to a new line. The \n will be considered as part of the string. Use double quotation marks if you want to escape \n to a new line.


The .git folder does not need to be added to gitignore.

The items from the gitignore file should be grouped/ordered so that the file is easier to read, and items are easier to lookup. In your case it would be:
- .idea
- venv

Folders should end with a forward slash so that when you read the file you know which folders are ignored and which files are ignored.

All files should end with an empty line. The main benefit is that later diffs will be much cleaner. http://stackoverflow.com/a/11597911


Here's the PR: https://github.com/NixOS/nixpkgs/pull/13819


I wanted to do the same first, but I decided it's not good for the navigation. I'm wondering what it would be for customers who open docs for finding some particular thing for a store and they go to the Store section and see only a part of the story.

I would expect to see the whole Store's interface in this case without a need to navigate anywhere else. But if you think it would be better to do so, I would proceed with the way you'd like to see.


@sellout Ok, do you think these unit-level tests are valuable, or would it be better to just test this functionality at a higher level?


Ok great, thanks


@NikolajBjorner It does indeed work. Thanks for making the effort to keep it up to date. One minor nit in future could you add source files so that it preserves the alphabetical order? Thanks.


The code does not comply to PEP8.

*PEP8Bear, severity NORMAL, section `autopep8`.*


2080-delete-terminal-with-coil (kbenne) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


I think that this is a genuine bugfix but the case couldn't appear as the only cases where new structured constant can appear during cmmgen are currently only boxed values and atoms, hence never containing a function.

By the way I'd like to clean a few of those things later and drop the dependency from Cmmgen to Compilenv.new_structured_constant such that all structured constants are explicitely received through the argument of Cmmgen.compunit. The fact that only boxing produces new ones means that it shouldn't be too complicated.


@hcanna :ship:


Ah, OK
All components store an attribute self.parentName so that we can track back to which Routine they belong to. Also, I've found a one-liner y to rename the key within a dictionary:
`routines['aNewName'] = routines.pop('someOldname')`

Combining the two things should be roughly this (at line 2039):
```
self.exp.routines[oldName].name = name
self.exp.routines[name] = self.exp.routines.pop(oldName)
for comp in self.exp.routines[name]
    comp.parentName = name
```


You should not commit .pyc files.


Hi @levinson ,

Many thanks for upgrading the pull request, but there are still a number of outstanding issues:

- Tests should be idem-potent, therefore relying on a before-after-all contract is not beneficial as introduces race conditions in parallel testing. If all suites are ran in parallel you cannot rely on the results.

- Assertions are also not done completely right. `foreach` should be `forall` in every instance and and a for comprehension should be used to reduce the amount of code lines and make everything clearer. Have a look at the other phantom tests, see how we generate data in place for every single test and assert just based on that.

Please let me know if you need further assistance with seeing this through and once again many thanks for the effort.


yeah makes sense. :) :+1:


GetRoomMessage
GetRoomLatestMessage
GetRoomMessageByUserID


This was not being used in the project


The overall flow seems reasonable and matches we do for the benchmarking workgroup


Isn't it a default value?


Returning a pointer on auto variable (PickedObjects) will lead to crash when trying to access  those objects from caller. Another question  - do we have any way to determine what types of objects were selected?


ok


OK.  I'll sit until the server and spec are finished.


Yep, I'm fine with this instead of a pr.  Change looks good


Thank you so much this is amazing


C'est super dangereux. Tu laisse la voie pour lancer n'importe quelle méthode php sans paramètre !!


In general, yes, that is the good practice.
In that case in particular, I'm saying that lines 2 and 3 (`from theano.sparse.utils ...` and `from theano.sparse.tests.test_basic...`) will raise an exception when `theano.sparse.enable_sparse` is False.
This does not show up in Travis currently because `enable_sparse` is True, but on a machine where `scipy` is not installed, for instance, it would fail.

So I think the order of these lines has to be:
```python
import theano.sparse
if not theano.sparse.enable_sparse:
    raise SkipTest('Optional package sparse disabled')
from theano.sparse.utils import hash_from_sparse
from theano.sparse.tests.test_basic import as_sparse_format
```
and that `SkipTest` has to be imported before we use it.

If you want to import `SkipTest` and `numpy` before all imports from Theano, that would be even better.


Aim for smaller commits so that changes are easier to track and revert.
Commits should be small enough so that looking through the git history you could easily see the incremental steps that were taken to implement a feature.

Example:
A common thing is that the first commit of a repository is just the gitignore file, and the commit message would be:
```Add .gitignore file```

http://chris.beams.io/posts/git-commit/


repostttttt


What exactly will get loaded here?


It will be nice to add this action in the wps client.


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


Sure - it's just changing the periods of the Python module path name to underscores, so that it is a valid JS variable name.


This logprint is indented too far


This reference will be changed into a reference to the Kolbri global variable (as defined at the 'library' regex you asked about above).


Got it. That makes total sense, and I sort of suspected that was the case after reviewing some of the commits that come after. It might be worth adding a note that explains why this number was chosen.


+                LogPrint("thin", "TX with the same cheap hash was already found\n");
+                if ((existingHash != (*mi).first)) { // Check if it really is a cheap hash collision and not just the same transaction
-                	LogPrint("thin", "TX full hashes are not matching\n");
+                    LogPrint("thin", "TX full hashes are not matching\n");

Indent is too far


There should be two spaces separating the class and the imports. This is a violation of the PEP8 style guide which should be adopted by all Python programmers: http://docs.python-guide.org/en/latest/writing/style/


alkamid.com looked like this:
![pic](http://i67.tinypic.com/bdtnvq.png)

Because there was no default "page.lang" for the main page. This small change fixed it.


Well, it's very complicated to detect that the app has "finished" (no activity or service running).
You can easily interrupt the watchdog as it is a thread with `interrupt`, but the hard part is to find *when*.


what the shit


why would the user use different formatters? If he did that that would break the build, right?


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


I was able to find it by manually inspecting the DOM (as you said, inspect element didn't work.)

It is being dynamically injected by JS. I just hide the CSS class they are using.


Do we really want anything related to internal topics to be client side? This could change in brokers from version to version and the clients should still work. I understand that for now we have no way to get that information, but we will soon (KAFKA-3306). I imagine removing the client side list would be part of the cleanup once thats available. So whatever exists in the mean time should be private so we don't need a deprecation cycle.


`void` argument, it's C!


Yeah, that is something we need to address all over the place...  Alas,  I think your fix does not really solve it completely -- it avoids the early string expansion, but the call to self.slot_status() is done anyway.  This needs a proper guard like [this](https://docs.python.org/2/library/logging.html#logging.Logger.isEnabledFor):

```
if self._log.isEnabledFor(debug):
  slot_status = self.slot_status()
else:
  slot_status = '[unknwown]'
```


Prefer to use `AS_IF`

I'd personally have simplified this to `AS_IF([python2 -c ...], ...)` rather than manipulating `$?`.


@sunsetsystems Yes, this change has been reverted


Its good to see someone take this up and fix it! Its been an issue since #142.
The changes to psychopy/__init__.py probably can all be reverted (unless I missed something). And it would be good to run autopep8 on your new code to bring the whitespace formatting in line with the recently adopted code style for the PsychoPy project. And when the time comes, you can remove the `# OLI ADDED` comments -- git / github keeps track of that information automatically.


`Transform` is the last gate check for making changes in `TransformState`, thus makes sense to safeguard it as much as possible.


commits w08-d03 hw.  Struggled with the delete in the app.js and worked with TA and team to get it to run.  Need more practice.  thanks


What does the `alias` keyword do?


@jensens Jenkins CI reporting about code analysis
See the full report here: http://jenkins.plone.org/job/package-plone.app.z3cform/13/violations
```
plone/app/z3cform/inline_validation.py:15:1: C901 'InlineValidationView.__call__' is too complex (15)
plone/app/z3cform/wysiwyg/__init__.py:0:1: I001 isort found changes, run it on the file
```
Follow [these instructions](https://github.com/plone/jenkins.plone.org/blob/master/docs/source/run-qa-on-package.rst) to reproduce it locally.


Mentioning both ways is probably confusing, just pick one :-)


Much lint!


Nice.


is related to server plugins. Can be renamed to make it more clear


Safeguarding is fine. I think it should be an assert or exception though, not a silent discard.


Please leave testing code (main.cpp) in dedicated  /tests folder along with appropriate project files. It is simply waste of time to create project any time you check out the project tree


I have this name used by isParentInvoice setter. I'll try to modify both.


The second option looks cleaner to me.


I'll do it.


(we don't actually ship only the styles that are on the page, but pssst :wink: )


Because "reverse" is too generic, don't you think ?


I think this way more people can identify with the benefit, maybe?


No passive voice


Slight rewrites and consistent capitalization


Do you want some sort of version-check functionality? Something like

```
HOTDOC_CHECK([0.8], [c dbus gi])
```


Sounds good


ok, protocol v1 will still be supported anyway with the exclusion of the signature things which anyway was never really used.

I'll ping you once I'll be done with the server-side implementation of the new commands.

(the spec is ready btw ... I don't think it's going to change)

> On Mar 10, 2016, at 6:59 PM, Damian Gryski <notifications@github.com> wrote:
>
> OK. I'll sit until the server and spec are finished.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/Shardcached/libshardcache/commit/f63c44cc388e93d5affd9c6754592d81f7f87668#commitcomment-16624255>.
>


hey @fqxp : yep there's an open PR at https://github.com/influitive/apartment/pull/266 which I'm working on. There's an issue with potential high number of database connections with this currently that I'd like to address before we can merge. Also the CI was failing on some builds. I hope to look at it this weekend if I get time.


About the new image files... I would like to see theme-specific images in a separate directory for each theme.  Looks like this commit changes images for existing themes.


Avoid `This is bridge` code comments. It is obvious that it extends from base.html


Should we check to see if EnterpriseLinux is installed first?

Also change message to "EnterpriseLinux ZenPack is not required for the LinuxMonitor Zenpack"


I think you should also add the new variables `prerequisite_command` and `prerequisite_fail_msg`
Reference: https://github.com/coala-analyzer/coala/commit/188a942e92ecc71bcd51d8329fa4fae4e2281f73


Never mind, I see what was discussed already.


don't use multiline string here, that way there'll be lots of spaces in the error message right?


Done


This last commit is GTM, GG for the :fire: of LoadLocaleData :+1:


Not sure if this was or not in the tutorial, but in general you should avoid the `br` tag. Styling should be done using CSS.


@pranavkm you have to stop making changes only for Windows :p you forgot the sh file


It's a different benefit... still valuable though!
I was more thinking of the insight that React Router brought over from Ember: that your routes describe the state of your app and so your router should help compose the components that respond to it


Looks like that exists isn't even used. Thanks!


It's called when someone clicks on Manage Jenkins.


I'd drop the indefinite article:
`Predictable state management: unidirectional data flow allows...`


done


I wasn't implying that we should do what SQL Server does. It was just one that I could find easily.

My point was that we should look at what others are doing for inspiration and then pick something that makes sense.


2080-delete-terminal-with-coil (kbenne) - Win64-Windows-7-VisualStudio-12: OK (2375 of 2398 tests passed, 0 test warnings)

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


ba19a81dd23db6b0aca815697e07d7bb2ec8917a is ok


merged, it looks way better now. I also felt this was weird but I was trying to do things by changing minimal existing code lol


fixed


du kannst direkt im hasPerm ja isAdmin abfragen und wenn es true ist gleich es zurückgeben ;)


Why removing the "meta" here then readding it before sending the request?
Here we can have two approaches:

1. Store meta with the "meta_".
2. Inform user that "meta_" is not required and store meta without the "meta_". I think this is the best approach as the method name is self-explanatory.


You're using XCOM 2 game version which is different from those I have. UModel can't detect that the game is XCOM 2, and uses generic code (see "game=8000" in error message). You should force game to be "XCom2". Also you could provide me a "version scan" report so I could update game detection code so your game version will be detected automatically, like those version I have.

You'd better use forum for messages of such type - I just **occasionally** noticed that I have a message here - I have hundreds of notifications from UE4 pull requests every day (I've subscribed on them), so noticing UModel's comments for commits is a bit hard, especially because they are not listed anywhere later (or at least I don't know about such place on GitHub).


I mention this because the Java class GeographyPointValue equals method uses a default epsilon.  Since doing an exact comparison isn't all that useful, it seems like `operator==` should use a default epsilon too?

If there's a good reason to provide a richer API, we should make Java have it as well.  My preference would be for a simpler and consistent (across Java and C++) API as it will likely make our life easier in the form of fewer support issues, etc.

I think you're right though that most likely users will use these classes to get the data out of their database, and then pass it forward to some other 3rd party tool, so comparing two points or polygons for equality (exact or not) may really only be important for our own unit tests.


It seemed like something I should be using but there was no actual need, removed


cool!


brain fart....

you added the [, so its fine now, I was thinking totally wrong


Where is class title-background defined?  Also won't removing the inline style affect other themes?


ahh, didn't noticed ordering, will do


Thanks! Might be a good idea to use it here too. Even if it's less complicated than the other case.


Don't blame me, blame the original author #copyandpaste


Isn't that from c1ebac8494 ?


oh jaaa....das baumer so ein :D
Thx!


This is nothing a type system is able to prevent, but it is something some library writer discipline is able to prevent.


It actually is called "template strings" – my bad! https://developer.mozilla.org/de/docs/Web/JavaScript/Reference/template_strings


Same options as the original code since they're in the same source file


ich weiß zwar nicht was du von mir willst aber python kann alles besser
ich brauche nur perm einzutippen
und btw:
```bash
# lukas at lukas-XPS-13 in ~ [19:28:01]
→ find . -name "*.py" -exec chmod 744 {}
find: missing argument to `-exec'
```


%d %-5p %t [%c{1}] %m%n


Why not mention the fact that we're using ExtractTextPlugin to build page-specific css bundles?


TeamCity LabKey :: 16.1 Release :: Community :: CDS postgres 9.5 [Build 41](https://teamcity.labkey.org/viewLog.html?buildId=309483&buildTypeId=LabKey_161Release_Community_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 10 (2 new), passed: 42 Build time: 01:31:47
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.cds.CDSVisualizationTest: <no details avaliable>

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

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

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

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

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

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

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

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

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

```


I didn't know it brought that over from Ember, interesting!
Me personally, I identify more with the one I wrote, but that's might just be me. What do you think more people will identify with?


боже что это


Thx


A better commit message would be:
```Add pyc files to .gitignore```


Yes, I like this, it sounds like a clean, straightforward approach.


thank you very much! I will now close the issue ticket :-).


Nie w `mov`. Powinieneś użyć `lea`


Because we aren't at the moment, we use ExtractTextPlugin to make one big CSS bundle. This is a good todo though!


Hi Tony,

Looks like they were mostly span, with a h4 and a font and some a p or so in there.

In the places that you modify(ie. add title-background), recommend using the span tag(and no p or b or h4 or font tag).

-brady


`lea` zamiast `mov`


Add new line at the end of the file.


The issue is that once callers rely on LongDecimalType/ShortDecimal type classes, they *see* the class hierarchy. It becomes an API.

I don't think adding a separate class is unnatural. In many ways, it's similar to how Guava defines utility classes to manipulate or operate on collections/etc (e.g., Multimap vs Multimaps)


It looks good. I am concern about the Travis check failing:
ERROR: invocation failed, logfile: /home/travis/build/mozilla/build-cloud-tools/.tox/py27/log/py27-2.log

ERROR: actionid=py27


Nice to see my work commited, when is it available to choose norwegian as language? :+1:


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


masz
```asm
xor BX, BX
...
mov BX, i
```
bez jaj


Nice


bez jaj


`was loaded` is still passive... but fair point!

Suggested replacement:
`availability without a network connection from the instant your users load the app`


bez jaj i jeszcze copy-pasta


such thug
how you do tis ;_;


Yes, but this is basically duplicated code that is already at the top of this script.


It's the first commit I believe, so it added every file from the core repo. I believe the files that have been modified have red where there was deletions. So to answer your question, yes I believe that's a fair assumption.


Works fine! Thanks! :+1:


JavaScript code should be written .js files.


raz masz `ax` a raz `AX` weźże jakąś konwencję przyjmij


Just to point out, note that all the code changes in this commit are not in the original codebase(ie. all changes noted in this commit are new changes).


```shell
find . -name "*.py" -exec chmod 744 {} \;
```

Uh ja EOL vergessen.
da gibt es nichts besser zu können. das ist einfach umständlich.
und ich vermute mal dass das für große directories langsamer ist


Oh damn, my bad! :+1: What about adding once: "availability without a network connection from the instant your users load the app **once**"? (Also, do you maybe want to jump on [Gitter](https://gitter.im/mxstbr/react-boilerplate)?)


ten komentarz sugeruje że z neta skopiowałeś


Fixed in:
https://github.com/infosupport/stryker/commit/65a8d963b3cf6e4929e8716e7a58d635eb93aed2
and:
https://github.com/infosupport/stryker/commit/7edd7d824ee974c08642a7bd7f5c0641f2df6a31


Hey Vladimir,

we finally did some further testing upon your suggestion and indeed you are (mostly) right:

`django/contrib/auth/migrations/0003_alter_user_email_max_length.py` contains a single `AlterField` migration. Its' base class (`django.db.migrations.base.Operation`) provides a `allow_migrate_model()` method which checks, among other things, whether the model has been swapped. This is determined (by `django.db.models.options.Options.swapped()` getter method) through comparing the current app label and model name to that pointed to by the setting referenced in the `swappable` Meta-property. If the two match, swapped gets 'app.model', a non-empty string, thus the former evaluates to true.

So this ignored migration is the effect of setting `AUTH_USER_MODEL`. On the other hand, the `swappable` Meta-property in `accounts.models.User` has no effect whatsoever after all, because it points to `AUTH_USER_MODEL` i.e. itself, meaning it is not swapped.

Thanks for your patience!


definiujesz procedury poza segmentem kodu. rebel


Ah! It would be :)


Brevity being the soul of wit, I'm perfectly happy to go with yours


I'd prefer `if (!routingState.equals(prevRoutingState)) {` (both here and in the template)


I don't like this use of multiple classes with names that suggest what the table will look like.  Instead, a single class or ID should be used to identify the context, and the theme's CSS should be about what the table looks like in this context.


przynajmniej jak mnie odpytywał co robi `lea` to na to przystał


Hi guys,

I'm the developer of Royal TSX and just noticed that you are still using our old appcast URLs.
While those still work, we will eventually phase them out.

The new URLs are:
V2 Stable: https://royaltsx-v2.royalapplications.com/updates_stable.php
V2 Beta: https://royaltsx-v2.royalapplications.com/updates_beta.php

V3 Stable: https://royaltsx-v3.royalapplications.com/updates_stable.php
V3 Beta: https://royaltsx-v3.royalapplications.com/updates_beta.php

Also, our official website link now is https://www.royalapplications.com/ts/osx

thx,
felix


please go grab the https://github.com/sexibytes/sexigraf/releases/download/0.99b1/sexigraf-0.99b1.sup new release and apply it, we'll make a full statement soon. Thanks a lot for your support!


а чего вдруг решила переделать?)


@asmecher this was all I needed to change.  Master uses PHPMailer now, and the send method in this class sets the Sender property of the Mailer object, which how you set the Return-Path.


> robie z tego liczba

Tak mi się skojarzyło: http://zapytaj.onet.pl/Category/007,014/2,7572325,Jestem_w_szkole_i_robie_prezentacje_Co_to_liczba_pi_D.html


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18792](https://teamcity.mesosphere.io/viewLog.html?buildId=159940&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:05:09


2081-op-scheme-schedule (kbenne) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2325 of 2397 tests passed, 0 test warnings)

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


challenge accepted
und wieso ist es umstaendlicher wenn ich jetzt einfach nur `perm` eintippe?
klar kann ich den alias auch so setzen aber ich will eine python solution


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


`import` statements should come before `from ... import ...` statements


Closed. Removed it.


The code does not comply to PEP8.

*PEP8Bear, severity NORMAL, section `autopep8`.*


Some blank space is needed to separate logical blocks.


das ist unschön. gefällt mir nicht.


Лучше вместо задания оффсета на всю ширину фигуры (я же правильно понял?) выставить поле `leftToRight` в `false`.


https://github.com/Team694/team694.github.io/wiki/Style-Guide#carousel

> Carousel images must be 960 x 480 JPEGs.


du mir auch nicht und trotzdem akzeptiere ich dich


why not simply `[RCWF]` am I getting something wrong here? not a regex expert...


`v` is not a good variable name. Use descriptive names.


`print('Current database version:', v)` does the same thing, only more pythonic.


We still need to check that the number of sourceCurrencies won't overflow when squared, otherwise someone could pay the min fee when they should have payed more. It's an easy check. Either a lambda with:
```
if (size > maxCost)
            return maxCost;
return clamp(...);
```
or `size > maxCost : maxCost ? clamp(...)` if you want to inline it.


I see he posted a great bug report then ;)


I'd like to see changes that are not specific to this theme put into a separate commit.


not needed since he has a simple executable and not something like `java -S ...` where `...` might not be there


als linuxer musst du doch linux cmd line tools nutzen das geht so doch nicht

```shell
export LANG=C #macht das angeblich noch schneller

find . -name "*.py" -o -name "*.sh" -exec chmod 744 {} \;
```

mach das und ich bin froh


Separate blocks with blank spaces


@sunsetsystems
To clarify, table, table-striped, and table-bordered are standard Boostrap classes. That's why they're stacked.
Are you suggesting to make a single-class via SASS that extends these classes?


indentation needs to be corrected


The very same build is OK now. Someone probably restarted it.


File never gets closed.
Prefer using `with` statements so that you don't need to take care to free up the resources you're using.


2081-op-scheme-schedule (kbenne) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


An absolutely amazing bug report in the form of his video, days after he found it (depending on how long he holds videos for)


This should be `{{background-color}}` no?


tests is a class so that it can be further improved upon in future , functionality like checking a particular section of examples can be included for better understanding of  what may be going wrong


Considering what we found while investigating how come swapped models' migrations are ignored (see the last comment at REANNZ@0c8cdb35684f1c5f287b504ca56bd976bcfe8035), we are fairly confident this property does not have any effect whatsoever. Therefore you could skip it, but you don't need to rewrite this commit just for that at this point -- we may revisit it in the future.


This looks like a good start. I'll see about getting some support content in there from the team.


modify readme


no


Should this be in a trap cleanup routine?


@calbrecht thanks for noticing, created [WEB-20736](https://youtrack.jetbrains.com/issue/WEB-20736) for that


Front camera crashed after cherry-picking this commit.

Logcat:
![](http://download.ikke.moe/OnePlusX/Screenshots/20160309170239.png)


I can't see any controversial thing in this commit. It was just a small refactoring... the functionality is exactly the same.


Its already clamped by RPC::Tuning::max_src_cur or RPC::Tuning::max_auto_src_cur.


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


Maybe we should gitignore all generated coverage report folders


Change the name 'helper.rb' to 'coverage_helper.rb'?


Why not move this to before line 954 (say as line 952) and then use port in 954 too?


Squashed.


For some reason it ignored this for me and had me put the admin password of "orca" in.


[Every program attempts to expand until it can read mail. Those programs which cannot so expand are replaced by ones which can.](http://www.catb.org/jargon/html/Z/Zawinskis-Law.html)


Do we want to add a `FloatExtractor` and a `DoubleExtractor` to round this out some?


This looks basically good and necessary (except for the comment on line 992)


I have a fix for that but wasn't quick enough to push it. The problem is `ErrorType` that is now known as `ErrorProtocol`.


Huh

Jesse

> On Mar 10, 2016, at 12:50 PM, Kyle Kelley <notifications@github.com> wrote:
>
> For some reason it ignored this for me and had me put the admin password of "orca" in.
>
> —
> Reply to this email directly or view it on GitHub.
>


When the magit-status buffer is created with LF eol style, a changed file with CRLF eol style shows up like this:

```diff
Staged changes (1)
new file   test
@@ -0,0 +1,3 @@
+line1^M
+line2^M
+line3^M
````


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


I made a couple of inline comments near the top that are general in nature.  I think there should be a separate commit for changes not specific to this theme.  The theme commit should not affect other themes in any way.  Class references should not suggest styling that applies only to one theme.  Sorry I don't have more time to dig through this.


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


Please do
```
try:
    return bool(value)
except ValueError: # <- please catch that explicitly
    return int(value)
```


same like below


This will make the execution jobs run in parallel, multithreaded.
That's different from Reentrant.

http://dotnet.github.io/orleans/Advanced-Concepts/Reentrant-Grains
> Reentrant grain code will never run multiple pieces of grain code in parallel (execution of grain code will **always be single-threaded**), but reentrant grains may see the execution of code for different requests interleaving. That is, the continuation turns from different requests may interleave.


Finisz README


whitespace around template filters


i'm not very practice with SASS, and this is actually taken from en existing `.css`file, so i expect some comments here. My guess is that i can use encapsulation but i need a bit of help :) @FrancescoCioria


It's supposed to be discardCard(handPos, currentPlayer, state, 1), where 1 is the trash flag, and it is set to 1 since we do in fact want to add the Embargo card to the trash pile rather than add it to the discard pile.


you still generate the config file even if `jshint_config` was set, please return `None` in this case to avoid superfluos creation of the temporary configuration file^^


:boom:


hoer auf hier die comment session zu verschmutzen


Don't assume this, we might in the future spin up a new web process for each API request (`webkit_web_view_load_*`)


missing newline


good point, thanks


this one shall not be removed :D


This does nothing.


Is there any difference from the test above not using `settings`?


I see a use of `map-to-hash` in `tests/eval-set.rkt`. Maybe that's why. Is it ok to export only that function for converting to hash?


Added score, fixed spawning, fixed rate of fire


Can you also add the getStatus() enums?


Honestly I think a sync D-Bus call is not a great idea, but no doubt it works....


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


gotcha . makes sense .


Yeah, sorry for being a bit abrupt about reverting this, but I've learned it's better to revert quickly when dealing with commits that have a potentially high impact. (E.g. last year wkennington introduced some new derivation style that IMHO wasn't a good idea, and when I finally decided to revert that, it took me half a day to untangle the web of commits that depended on those changes. In this case there were already two commits depending on this commit. Since `go-packages.nix` contains hundreds of packages, )

Note I'm not necessarily opposed to this commit, but it's kind of a "strategic" change that requires some careful thought, for these reasons:

* It introduces the notion of writing packages in JSON format rather than as Nix expressions. I don't know if that's something we want to promote. In any case it's a new derivation style, and those should always be carefully considered since they make it harder for people to understand / modify a package.

* It introduces yet another auto-update mechanism, and we already have a lot of those, most of them only used by the person who introduced them. (E.g. `gnupdate`, the `.upstream` files, language-specific mechanisms...) This too raises the burden for someone else to modify a package because they have to figure out how the particular auto-updater for that package works. We should really come up with a universal auto-update tool, or at least a standard for what they should do (e.g. update Nix expressions in place or overwrite an external version file?).

* (Slightly tangential) It further entrenches `go-packages.nix`, which I already objected to when it was introduced. We don't have a giant `c-packages.nix` for everything written in C either. Having one big file is suboptimal e.g. due to the risk of conflicts between different PRs, inability to do `git log` to see the history of an individual package, etc.

The meta-problem is of course that we don't have a good process for dealing with such changes. Hopefully #13602 will help a bit with that.


:+1:  Thank you very much!
https://travis-ci.org/Rapporter/pander/builds/115013405


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


Resolves issue #15


+divovi


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18799](https://teamcity.mesosphere.io/viewLog.html?buildId=160012&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:05:13


perhaps a mildly more specific error message? what are the kinds of things that can fail here?


@gianarb this signature update is a BC break here. Also, `$container` is never used in this method, so I'm not clear about the rationale of this change. Could you please elaborate on this?

Can't we make `$second` optional and default to null?

  /cc @weierophinney


I see what you did there


umständliche lösungen sind auch lösungen :anguished:


I'm not sure the best way to handle it.  But it's surely wrong to reference standard bootstrap classes outside the context of your theme.  Those classes could be defined in other themes or contexts using bootstrap.


Yeah.. The one with settings uses the parameters in the dict which will call the `bool_or_str` and `bool_or_int` functions. So the purpose of adding the test is to test those functions


gib auf


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18801](https://teamcity.mesosphere.io/viewLog.html?buildId=160051&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:05:09


Instead of skipping if it's not set, wouldn't it be better to re-index the $components array in the function where it was unset?
Array index is being unset in function add_thumbnail_if_needed( &$components ). A fix would be to do this right after
**unset( $components[ $i ] )** :
**$components = array_values($components);**


действительно так удобнее, сразу видно где модиф


[HZR] 2.2 Developer loads Hot Zone Regions as GeoJSON overlay


файл называется index-logo, а шаблон на просто logo, тут явно какая-то ошибка.


Lol, you can ignore this (and remove the comment). It creates a DrawerLayout. That's what overlays the the app when you swipe left to right.


Those graphs add plenty of concise & valuable info. Thx!


I was talking to David about it yesterday. The alternatives (to make the class [standard-layout][]) seem to be:

- Make everything public.
- Make everything private.
- Make everything public but wrap the private things in another level of class having only private fields.
- Disable the warning because the Itanium ABI guarantees layout anyway. I can't find a reference for MSVC++ struct layout, though; it's probably not guaranteed because they [don't consider C++ a stable ABI format][portability].

[standard-layout]: http://en.cppreference.com/w/cpp/concept/StandardLayoutType
[portability]: https://msdn.microsoft.com/en-us/library/hh438475.aspx


@VoyagerOne: maybe a good idea is to test also 2.05 on LTC. I guess that a function with slightly slower growth than log would be appropriate, but I can't find such simple function.


Instead of skipping if it's not set, wouldn't it be better to re-index the $components array in the function where it was unset?
Array index is being unset in function add_thumbnail_if_needed( &$components ). A fix would be to do this right after
unset( $components[ $i ] ) :
$components = array_values($components);


ah alright :+1:


OK, thanks!


Thanks Ron. This change will be incorporated into the 2.0.2 release.


Little Change


What?


Ok, now I see. The decomposed GlyphComponent.flags attribute in the fonttools' glyf table does not keep all the original flags as they are encoded in the binary TTF, but it clears some of the flags which can be auto generated upon compile based on other data inside the component. The bit 2 is among the ones which can't be guessed and therefore is kept. This is another instance of an optimization which occurs on decompilation instead of compilation...


It seems this should be [`language: cpp`](https://docs.travis-ci.com/user/languages/cpp#Default-Test-Script)


you could write a comment that `bool_or_int` and `bool_or_str` are implicitly tested, imo it's not so obvious :)


Название promo-milestones не нравится, milestones это километровые столбы, немного другое. Предлагаю promo-crossroad


Ewentualnie
const int SIZE = 10;


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


Ewentualnie:
const int EMPTY = -1;


@rogeruiz I wonder if "The minified versions contain a `.min` suffix in the filename before the file's extension. The un-minified versions do not contain the `.min` suffix." is self-explanatory. cc @kategarklavs


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


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


TeamCity libpalaso :: palaso-precise64-libpalaso-2.6 Continuous [Build 2.6.1062.458cfc3a7196cacce23b938c094a09728bb3490f](http://build.palaso.org/viewLog.html?buildId=70212&buildTypeId=PalasoPrecise64v26Cont) outcome was **SUCCESS**
Summary: Tests passed: 3377, ignored: 124 Build time: 00:05:40


The complete can be declared within the try block.


remove


This is an unused import and causes contrib check to fail


You could also just override this function in the DungeonMaster class.


Are you also working on implementing AddDefaultCert as well? Or is this something I could implement?


we are hiding the FAB...right?


I was unaware that the Amazon Fire OS Cordova platform is being deprecated. We have a few old projects that still have separate amazon-fireos targets and we we're looking to update to the latest version of this plugin. We will look into removing that and upgrading to Cordova 6.0.0. Thanks.


It should call isComplete with remove=false.

There is a fetcher job that is polling for the complete status. Current behavior is for that job to remove it. If this one removes it from the queue, then it will make the push job fail.


"Set the DriversitiConfiguration. This needs to be set before using any of the other SDK methods"


gladly. honestly thought the main proj was abandoned


Does CreateAccountModal take 'showRequiredError' as an argument?


Yeah, I think they are. And when our types get overhauled and handled more consistently, they’ll be even moreso.


You can remove this. We really only need comments that relate to our SDK


Can we rename to getUserData()


Did you mean to remove this?


Can you declare a constant for -1, probably , NO_FETCH_IN_PROGRESS


Isn't this supposed to be "gone"?


At some point you might want to look into something that let's you use ES6 syntax.

It makes writing classes *much* easier.

This is what I currently use: https://babeljs.io/

Basically, this is something that will transform ES6 javascript, to a format that most browsers can understand. Not strictly necessary, but it can make writing javascript nicer.


Any progress? I've just started playing again and would love to see a quality package for mudlet.


Should we name it as lastFetchRequestId ?


be be be be


It could be great to have universal reference for non template type.


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


В заголовочных файлах лучше не использовать using namespace ...


Love this MockFetcher and the way you are approaching this. Awesome. Love it.


Yes, I don't know why this is here.


Here: https://github.com/behdad/fonttools/blob/master/Lib/fontTools/ttLib/tables/_g_l_y_f.py#L1047

It means the component "flags" value in a TTX dump should not be taken literally and might be different from the value encoded in the binary. That attribute only encodes a subset of the flags defined in the spec.


No tests for this new implementation with char*?


Good catches, thanks!


Should be fixed in 5d6b47be744


Nice, big elo progress is still posible.


I am learning git hub


Just a suggestion - but...

(1) We have 1024 characters to work with, so we don't really need to worry about length.
(2) The filename needs to be unique, but so do other related attributes.

So...

Why not use something like "<cloud-service-name>_instance_<instance-name>_image_<image-name>" ?


Yes, I agree with your use of spaces, but I would recommend using 2 spaces.

It could be useful to use the style guide someone else has written. This is the one I often use: https://github.com/airbnb/javascript.

You can also hook this up with eslint to make sure that your code meets the style guide (very good as a pre-commit or pre-push hook)


I support this change. Can you do it and verify that it fixes the problem?


Seems like an argument for using for (ResolvedJavaType interface1 : a.getInterfaces()).

Also don't we need to order our search in some canonical way if multiple correct answers are possible?  x.meet(y) should return the same as y.meet(x).


do you mean slash instead? is `'/'` called a slash or a back slash?


It seems like sender would be a good constructor argument.


`conn` can be nil if it failed to connect.


Why not have 'initialize' just be the constructor?


If I don't make any sense, don't worry. I'm writing from my mobile.
Thanks for the clarification!


TROLOLO


An old comment, from when I had everything working and was starting to mess with prototypes... That's now gone.


good point, thanks @maya. I did my best, but I will also defer to @kategarklavs on that one! :dancer:


No problem, thanks for the reply! From what I understand, the Android platform should work fine with Fire OS.


Can this be removed? Also below.


Indentation is kinda weird here.

I might do something like this:

```javascript
console.log("I, " + this.playerName + " am neither a Character nor " +
            "a Dungeon Master, and therefore my greeting should be " +
            "different from a Player.");
```

And align the arguments on subsequent lines with the `(`.


**`git commit -m "Commit de prueba en rama nueva remota mejorandoproceso_masivo"`**

[mejorar_proceso_masivo 2ae1f54] Commit de prueba en rama nueva remota mejorandoproceso_masivo
 1 file changed, 1 insertion(+), 1 deletion(-)

**`git status`**
On branch mejorar_proceso_masivo
Your branch is ahead of 'origin/mejorar_proceso_masivo' by 1 commit.
  (use "git push" to publish your local commits)
nothing to commit, working directory clean

**`git remote -v`**
origin  git@github.com:FernandoTorresL/imss-dspa.git (fetch)
origin  git@github.com:FernandoTorresL/imss-dspa.git (push)

**`git push origin mejorar_proceso_masivo`
`git branch`**

  master
\* mejorar_proceso_masivo
  remotes/origin/mejorar_proceso_masivo


Oleg, I think the code was updated to support the "5 secs" type of syntax, but the admin guide here still calls out time in milliseconds. Probably want to fix that wording.


Could we switch the wording to
```javascript
describe('when theres a Marionette global namespace', function() { // ...`
```
... or something similar, leading with `when`. This follows the consistency of the current test's `describe` blocks?


> Upgraded compiling documentation with description of ELMA H2x

There's a typo (“ELMAH 2x” instead of “ELMA H2x”) in the commit message.


True, I was trying to convey that I understand prototype "inheritance" (not sure if that's the right term) by overriding for Observer but not for DungeonMaster.
Finished product will be more properly set up.


Структура html должна быть такая:

```html

    Методология

    Идея и архитектура БЭМ

    Платформа

    Готовая реализация

    Быстрый старт


An empty list is a literal "evaluable"?


One more ~~self~~ :smile:


I changed it, but I think it doesn't change anything for Travis. IDK, but I can't see any difference.


Does it make sense to combine these methods into one with an error_level parameter?


2081-op-scheme-schedule (kbenne) - i386-Windows-7-VisualStudio-12: OK (2372 of 2397 tests passed, 0 test warnings)

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


Is there a reason we don't remove the tab "Theme"? There's only one nav item, maybe we don't need the nav.


My only other comment here is that style wise, your opening and closing brackets should always start and end at the same indentation (your pep8/ide checker should warn for this).
```
log.info("Create nested type. Native JSON: {0!r}".format(
    native_json
))
```


отлично, не зря. хотя плохо понимаю про Handler


This file needs an apache license to pass contrib check


The editor takes a contenttypeid and a containerid InlineContentControl.EnsureChildControls I suspect these have to tie up together with your changes..somehow


Вместо этих трёх строчек должно быть
matr (int _n, int _m)
: n(_n), m(_m) {


Fair. AFAICT, you are doing the inheritance correctly. I don't really fiddle about with prototypes directly anymore, I use a ES6 precompiler that does a lot of that stuff for me. See my comment  in the previous change.


Thank you for the request. It has been added to the queue (id: 170). To see the progress you can go to the [status](http://ccextractor.canihavesome.coffee/test/170) page. Please note that depending on the current queue, it could take a while before any results will be visible. In each case I will report back here once the tests are done.


Makes sense. :)


NSLayoutFormatOptions are `OptionSetType`s and conform to `ArrayLiteralConvertible`. So you can just do this instead:
```swift
+        let horizontalConstraints = NSLayoutConstraint.constraintsWithVisualFormat("|[tabControllersView]|", options: [], metrics: nil, views: ["tabControllersView": tabControllersView])
```


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


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


Thanks for doing all that!


liczbĘ, nie liczba!


I have been reworking the scripts to be user friendly. They are currently in a mixed state of old and new. I have a lot on my plate these days so progress has been slower than expected.


My guess is the default should be plain `None`, not the string 'None', yes? This will cause issues if someone doesn't supply an endpoint at the command line (which most people won't since they are using the default of S3). If you want this merged upstream, this will probably need to be changed.


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


The changes to `sol_oic_repr_field` could be in a separate commit.


> I didn't want to write the boilerplate to return Err(Unimplemented) for every method other than GET, PUT, and DELETE.

Ah, makes sense.

> Perhaps we could address that concern with some kind of annotation, like #[warn_if_not_overridden].

This would be good, but I don't think that would work on stable?

Perhaps we can add some more options to `compile` so we could opt in to `no_default_implementations`. Taking the idea futher, we could pass a list of items that require implementation e.g. `compile(.., RequireImpls(["foo::Trait::method"]))` (alternatively `DefaultUnimplemented([..])`) but that has the downside of being stringly typed, and so we'd probably have to error if it's not found in the schema.

If this seems sounds I can open an issue to track.


Nice place! :coffee:


button appears but it doesn't start the game


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


This comment doesn't match the constructor


testing


Aber Eier


It looks like www.copart.co.uk is back online. Also, could you add a test for http://forums.lavag.org/Industrial-EtherNet-EtherNet-IP-t9041.html?


@owensd Would you be interested in contributing this code to the swift benchmark suite? We'd like to track the performance of your benchmarks over time and make sure we don't regress.


Thanks for the fix, man :D


From what I understand, what @cmarcelo had in mind was something like this:

```C
    while (!err || err & CborErrorOutOfMemory) {
        orig_encoder = writer->encoder, orig_map = writer->rep_map;

        if (err & CborErrorOutOfMemory) {
            r = enlarge_buffer(writer, orig_encoder, orig_map, size);
            SOL_INT_CHECK_GOTO(r, < 0, end);
        }

        switch (repr->type) {
        case SOL_OIC_REPR_TYPE_UINT:
            size = sizeof(uint64_t);
            err |= cbor_encode_text_string(&writer->rep_map, p, repr->f_slice.len);
            break;
        case SOL_OIC_REPR_TYPE_INT:
            size = sizeof(uint64_t);
            err |= cbor_encode_int(&writer->rep_map, repr->f_int);
            break;
        case SOL_OIC_REPR_TYPE_SIMPLE:
            size = sizeof(uint64_t);
            err |= cbor_encode_simple_value(&writer->rep_map, repr->f_simple);
            break;
        case SOL_OIC_REPR_TYPE_TEXT_STRING:
            size = repr->f_slice.len + sizeof(uint64_t);
            err |= cbor_encode_text_string
                (&writer->rep_map, repr->f_slice.data, repr->f_slice.len);
            break;
        default:
            err |= CborErrorUnknownType;
            break;
        }

        if (!err) {
            r = buffer_used_bump(writer, writer->rep_map.ptr - orig_map.ptr);
            if (r < 0) {
                err = CborErrorUnknownType;
                goto end;
            }
        }
    }
```


This looks great!


This should probably be marked as a private function.


Ah, this is a bug. The trash of the handPos will always happen, so simply moving that to the beginning of the functional block such as between lines 1252 and 1253. Additionally, line 1261 would be removed as well, as it currently might trash 3 cards instead of just two treasure maps.

This change will also necessitate removing the reference to handPos from the current line 1255, such that it reads
`if (state->hand[currentPlayer][i] == treasure_map)`
instead as the treasure map is already gone


the check_for_death() is now running continuously and also playing the "damage" sound continuously :( i think this should stay how it was.


`mov DS:[wsA], AX` itp plox


> In the new version, the results are sent back to the sender once two things happen:
>
>     The FooResults struct is dropped.
>     The returned promise is successfully fulfilled.

I think this covers my concern other than the question the api leaves me with (and would regardless of this change) of "what happens if I return `Promise::ok(..)` and don't mutate FooResults". I guess this is defined somewhere in the capnp docs?


Unrelated question, are all integer values being encoded in 64-bits? That seems wasteful.


Ok you did not have to change those ^^ now we can't override them from environment variables and defaults don't work anymore :D but don't worry about it


I don't think `{first,second}Path` need to be optional on 173-174. As long as you initialize them before use later (which you appear to), you should be fine dropping the optional type..


@HenksGitHub Added beer, in case there's no more wine


@slavchev Question, what do you do with a .so that doesn't have a NSMain, does this mean they can't be used with NativeScript?


I can look into it. It will probably be a few days before I get a chance to though.


IMHO it's easier to understand/debug. I'm not sure that's compelling enough, though. ;-)


@westergu please review


Det Hack du laver med ro rows, kan det ikke løses med Margin?


Nice.


sorry about that


Gah, it was showing as duplicated.

It should still be in your email. Basically I was suggesting using http://babeljs.io to convert ES6 javascript to browser compatible version.


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


we should have new images soon.


Yes, I tend to fix each of the legacy files as I run across them :)


Yes, I modify this. Is there a bug report for this or I reopen this bug ?


did we want to change this "salt" term to "context" or whatever?


good to know then.


I hope this doesn't have any unwanted performance hit going this route. (I don't know)


tested ACK


I will put a comment in the header of the function. Right now only positive integers are used with this function over the code. Also I will take a note to add the logic to support negative numbers.


Ahh yea, this makes sense.


@syurkevi Change the way the functor is implemented. Everything other than `idx`, `idy`, `idz`, `idw` can be passed at construction time.


int main()
{
    int sayi,basamaks;

    printf("Bir sayi giriniz =\n");
    scanf("%d",&sayi);
    basamaks=1;
    do {
        sayi=sayi/10;
        basamaks++;
        }
        while(sayi>=10);
        printf("Basamak sayisi = %d",basamaks);

    system("pause");
}


i know that, but we have already this discussion and always someone come up with using our internal functions to do this kind of think. If I remember one of those times it was you :)

This is far from being a sensitive code that requires optimizations. I do not see problem in using a well known function. But feel free to keep your own implementation.


May be store `scalar<InT>(0.5)` in a temporary variable ?


Ah, no it doesn't! I kept reading that as "showSignupRationale". Fixed and tested: 84cdf1e2ce0cfd9ccbf1870a6090dcccb27f53c0


doesn't tslint pull this in automatically?


Thanks @moiseev !


Same as above/


the reason this didn't matter is that it always returned `None`. whenever we're fetching a case transaction in form processing, we haven't saved the transaction yet. obviously still misleading if someone used the function for something different, but that explains why tests passed


I did something similar some time ago. On the one hand it's clearly nicer to not have to spell out all the lifts, but on the other hand there's also some benefit in being able to look at a piece of code and seeing immediately what the stages are, which is now less obvious. I think the main reason i turned it off again was that I found it hard to debug some issues where functions were lifted accidentally and caused stack overflows that were hard to track down. What do you think?


Why did you remove this? The function has side effects, it's not just a test.


can someone please add some comment to this file?

what to change in this file send emails every five minutes or fifte


 transpose ve iki katı alıp matrix alma
#include <stdio.h>
#include <stdlib.h>
#include <time.h>

int main(int argc, char *argv[]) {
	int matrix[3][3];
	int a[3][3];
	int b[3][3];
	int c[3][3];
	for (int satir = 0; satir<3; satir++)
	{
		for (int sutun = 0; sutun<3; sutun++)
		{
			scanf_s("%d", &matrix[satir][sutun]);

		}
	}
	for (int satir = 0; satir<3; satir++)
	{
		for (int sutun = 0; sutun<3; sutun++)
		{
			a[satir][sutun] = 2*matrix[satir][sutun];

		}
	}
	for (int satir = 0; satir<3; satir++)

		for (int sutun = 0; sutun<3; sutun++)
		{
			c[satir][sutun] = matrix[sutun][satir];

		}
	for (int satir = 0; satir<3; satir++)

		for (int sutun = 0; sutun<3; sutun++)
		{

			b[satir][sutun] = a[satir][sutun] + c[satir][sutun];

		}
	for (int satir = 0; satir < 3; satir++)
	{
		printf("\n");
		for (int sutun = 0; sutun < 3; sutun++)
		{
			printf("%d\t", b[satir][sutun]);
		}
	}
	system("pause");
	return 0;
}


switch-case kullanımı

#include<stdio.h>
#include<time.h>

main()

{
	int a, b, sonuc;
srand(time(NULL));
printf("5x7=");
scanf("%d" ,&a);
scanf("%d", &b);

sonuc=a*b;
printf("%d x %d %d " ,a ,b, sonuc);
a=rand()%4;
if(sonuc==35)

switch(a)
{

case 0: printf("Dogru");
break;
case 1: printf("iyi");
break;
case 2: printf("mikemmel");
break;
case 3: printf("temiz is");
break;
}
else
{

switch(a)
{

case 0: printf("yanlis");
break;
case 1: printf("kotu");
break;
case 2: printf("felaket");
break;
case 3: printf("kotu is");
break;
}
}

	return 0;
	system("pause");

}


Probably a port once upon a time.


Added self here?


update_travis_toolchain (lefticus) - i386-Windows-10-VisualStudio-14: Build Failed

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


Fibonocci
#include <stdio.h>

int main()
{

int degisken, x=0, y=1, i ,sayi;
scanf("%d", &sayi);
printf("%d" ,x);
printf("%d", y);

for(i= 0; i<sayi ; ++i)

{
printf("%d", y+x);
	degisken=x;
             x=y;
            y=degisken+y;
}

return 0;
system("pause");
}


update_travis_toolchain (lefticus) - Win64-Windows-10-VisualStudio-14: Build Failed

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


update_travis_toolchain (lefticus) - Win64-Windows-10-VisualStudio-14-Debug: Build Failed

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


Do we need this function anymore? Can we just move the navbar to the themes template?


this works because whenever we go to fetch a new transaction, we first check if one already exists for that form and case id in `_from_form`.

one assumption is that we never modify case transactions after they've been saved


fixes #2


Store `scalar<InT>(offGrid)` a member variable and use it repeatedly.


Git rm--[directory-PROJECT_DIR-vcs-Git-]


Woo!


Fonksyon kullanımı

#include<stdio.h>
#include<stdlib.h>
int toplama(int x, int y);
int carpma(int x, int y);

int fonk(int x);
int main()
{
	int a, b;
	printf("bir sayi giriniz:\n");
	scanf_s("%d", &a);
	printf("bir sayi daha giriniz:\n");
	scanf_s("%d", &b);
	printf("x+y=%d\n", toplama(a, b));
	printf("x*y=%d\n", carpma(a, b));
	printf("5x^2+8x+12=%d\n", fonk(a));
	system("pause");
	return 0;

}
int toplama(int x, int y)
{
	int sonuc;
	sonuc = x + y;
	return sonuc;
}
int carpma(int x, int y)
{
	int carpma, i, sonucc = 0;
	if (x>0 && y>0)
	{
		for (i = 1; i <= y; i++)
		{
			carpma = toplama(x, 0);
			sonucc = sonucc + carpma;
		}
	}
	if (x<0 && y>0)
	{
		for (i = 1; i <= y; i++)
		{
			carpma = toplama(x, 0);
			sonucc = sonucc + carpma;
		}
	}
	if (x>0 && y<0)
	{
		for (i = 1; i <= x; i++)
		{
			carpma = toplama(y, 0);
			sonucc = sonucc + carpma;
		}
	}
	if (x<0 && y<0)
	{
		x = -x;
		y = -y;
		for (i = 1; i <= y; i++)
		{
			carpma = toplama(x, 0);
			sonucc = sonucc + carpma;
		}
	}
	return sonucc;
}
fonk(int x)
{
	int sonuccc;
	sonuccc = toplama(toplama(carpma(carpma(x, x), 5), carpma(8, x)), 12);
	return sonuccc;
}


I have the same sentiments as Marc.

And admittedly, I don't have a compelling _must have_ use case in mind, but... it just _feels_ like the right thing to do (i.e., maintaining the order).

For me it's not a showstopper either way.

I guess we could always decide to retain the order at later date if a need should arise.