[_rename_conflicted_file](https://github.com/tahoe-lafs/tahoe-lafs/blob/2438.magic-folder-stable.4/src/allmydata/frontends/magic_folder.py#L463) specifies the conflicted filename; it shouldn't be specified in two places.


Now we are adding things to `modeSwitchersLife` after we've added these sub-lives to the main-life. That makes me uneasy, since I can't tell what state the mail-life is in. It might start the sub-lives and we are back to cowboy racing.

If the lines 507,508,509 can't be moved to the end of the method without breaking stuff, then I would like a comment about why this is the case. Separating the construction of things from their being added to a LifeSupport might also be a helpful habit.


@markohlebar I delete it and manual install it step by step in the way you offered.  It's OK now. Maybe I did the wrong step.


This looks fine to me.  Pull request?


is Metadata a compound word? So `parseMetaData` -> `parseMetadata `


If necessary, add a ``_get_conflicted_filename`` method to avoid that.


@ereyero revisa please el commit porque te has cepillado algunos fich y creo que no has añadido los que pretendías.


**Awesome** 👍👍


I have 3 servers all on 5.6 and none of them have the referential_constraints table so I would really appreciate the fallback code!

`SQLSTATE[42S02]: Base table or view not found: 1109 Unknown table 'referential_constraints' in information_schema The SQL being executed was: SELECT kcu.constraint_name, kcu.column_name, kcu.referenced_table_name, kcu.referenced_column_name FROM information_schema.referential_constraints AS rc JOIN information_schema.key_column_usage AS kcu ON kcu.constraint_catalog = rc.constraint_catalog AND kcu.constraint_schema = rc.constraint_schema AND kcu.constraint_name = rc.constraint_name WHERE rc.constraint_schema = database() AND kcu.table_schema = database() AND rc.table_name = 'users' AND kcu.table_name = 'users'`


When I added that for supporting [java.sysprops property](https://github.com/kiuby88/brooklyn-tosca/blob/d2f185a6507cd6772430b4ca9cf7d48f8ad9a86c/brooklyn-tosca-transformer/src/test/resources/templates/helloworld-sql.tosca.yaml#L50). Before, when the `unresolvedValue` was a map, the if condition retrieved false, because `resolvedConfigMap.get("dsl")` was not `instanceof BrooklynDslDeferredSupplier`, it was a map, because `java.sysprops` is a map. I have used another commit in a different PR because I think this code worked fine, but this feature is necessary for my use case.


Sorry for my confusion.


@gabrielenava can we have a feedback? (actually @traversaro is writing at the moment)


@pynicolas what makes you sure that this null check can be removed ?


@aulanov I know what you mean, this is the problem https://github.com/docker/docker/issues/16681 had, and this PR was planned to partially fix that.

I understand people want to keep these two behaviors consistent, but I think systemd just doesn't work that way, systemd organizes hierarchy by slice, maybe I didn't understand systemd enough?


Worth lowercasing the string?


Title should have been new `streamline-runtime` API


We're actually going to have the real conflict filename include a timestamp. In any case this wouldn't be the correct way to detect a filename clash, because it has a race condition. That's why ``_rename_conflicted_file `` uses ``rename_no_overwrite``.


Perhaps you could use for now lts-2.22 (uses 7.8 where the floatExpr bug doesn't manifest)?
Also, why not `stack build cabal-install`? That way you can easily switch between 7.8 and 7.10 and always get the right cabal.


Clean up temporary files


@pynicolas Looks like a good check but i don't fully understand its use, could you explain me please?


Metto un commento così ricevo via mail le notifiche.


The whole point of `nyc` it to be able to spawn without doing this. You might have to use `execFile` instead of `exec`, though. Which you should be doing anyways.


The trips are merely compared based on their count? Are you sure that is right?


JODER!

Sabes eso de cerrar el ordenador a la mitad...

Lo miro en la comida.

E.

.........................................
*Eusebio Reyero*
eusebioreyero.com
mail: ereyero@gmail.com
phone: +34 661872775
.........................................

2015-10-26 11:36 GMT+01:00 richhl <notifications@github.com>:

> @ereyero <https://github.com/ereyero> revisa please el commit porque te
> has cepillado algunos fich y creo que no has añadido los que pretendías.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/isiargonauta/openmadness/commit/f02504491b37f8c89cf051608b2cb8fef271e9a7#commitcomment-13980437>
> .
>


"departure" left-over


@IIvec I know that (0,2,3) is probably better and my test will probably fail. Thats why I wanted to use a different formula for low thread counts. But lets see how it goes... Anyway there are lots of backup formulas by others :-) :-) We have no shortage of formulas!

Anyway credit should go to you for coming up with a log formula in the first place. That is the best solution as we scale to higher threads.. As someone pointed out, with original lazy formula we would have had additionalDepth 36(!!) at 72 threads.


This is quite dangerous! It already had Laravel 5.1 support earlier. `~5.0` means everything up to `<6.0.0`. This will also support future versions `5.*` of Laravel even though we don't know if any thing will change in the future.


I think you should try removing line 6 and the OS test of line 7 and only keep line 7


That was minimum possible change for 2.2/2.3 that's why we still having life's at all, we only replace 'paxosLife' with 'modeSwitcherLife', but moving 508-510 to the end of the method looks like a good idea. @lutovich wdyt?


it will then work on all OSs very probably


test


asdf


Possible to abstract so it takes the same args as AES?


why do you have a second jumpMultiplier variable?


and sounds really reasonable to extract 'component add stage' into separate thing in pr for 3.0. do not remember how we did it there @lutovich wdyt?


The OpenSC scripts can't be agnostic about all possible configurations of all systems out there. If you want OpenSC to use your OpenSSL version it's best if you supply the correct compiler/linker flags with `./configure`. This scripts also tells you with which flags it has been configured...


Cool. Thanks.

On October 26, 2015 10:02:21 AM GMT+00:00, aszlig <notifications@github.com> wrote:
>It's documented in the coreutils info page (`info coreutils install`).
>
>---
>Reply to this email directly or view it on GitHub:
>https://github.com/devhell/nixpkgs/commit/6e1ef1352368d1d5773f3e2b64acb0b9393e4bb8#commitcomment-13979769


missing closing bracket


This looks like a good usability improvement. Users will first experiment with onedir and not onefile :-)


Apparently the problem was with 5.0 installations which wanted to use this package. Not sure if that's true, as I updated all my installations to 5.1.
If you have improvements to support all 5.* versions without breaking older versions, make a PR :)


I think the same is valid for [conf-gmp changes](https://github.com/aluuu/opam-repository/commit/3fd46be866efe1469b43543ec97166b502bcb763) I've made, am I right?


This is quite dangerous! It already had Laravel 5.1 support earlier. `~5.0` means everything up to `< 6.0.` This will also support future versions `5.*` of Laravel even though we don't know if anything will change in the future.

Better would be to add `5.0.*|5.1.*` instead. This will lock support to only `5.0` and `5.1`.

Ping @patrickbrouwers


For javascript literals, json_encode is sometimes a better choice than addslashes.

Another option would be to have equivalent versions of xlt,xla and xls for xl_layout_label instead just having the single version.  There would also need to be similar functions for xl_list_label.  That seems like a more consistent approach.

Will probably just go with addslashes for now though.


The problem is that we already have tagged releases with this setup.

Ping @GrahamCampbell


It's hard to see here whether ``is_conflict`` is initialised in all cases. Either:
- set ``is_conflict`` to ``False`` initially and then *only* set it to ``True`` conditionally (not to ``False`` conditionally, because that would be redundant), or
- use unconditional assignments in each top-level branch, e.g. ``is_conflict = (dmd_last_downloaded_uri != local_last_downloaded_uri)`` here.


The common use case for Agda is interactive theorem proving within Emacs, for which GHC is not necessary.  Compiling programs to native code using the MAlonzo backend, for which GHC is required, is an uncommon use case; the backend is even described as [experimental](https://github.com/agda/agda/blob/d7c0e405f17916def44706ede7e32070c4db5cb8/src/full/Agda/Interaction/Options.hs#L565).  Therefore, if the user has no need for GHC, they should be able to save the 800M plus it currently occupies, at 7.10.2.


@aszlig Do you think I should change this to `unix` rather than keep it `linux`?


A better way to express this is `DJANGO_VERSION='>=1.8,<1.9'` (etc) and then use `pip install Django$DJANGO_VERSION`.


Thanks for cleaning up this code.


indeed, see #1541


@ewels There is no "sample prep. kit" info. in statusDB, "reagent_label" could give me a clue, but I think it's more reliable to manually input a fixed number. I set default to 135bp (TruSeq® Stranded RNA HT).


@ajithcj : I would give up any credit, just to see this optimized on 7 threads. But, I understand it's TCEC now. I'll be patient.


this can be moved to Update seeing as it doesn't specifically interact with the rigidbody


OK.


I remember adding that </string-array> after fixing conflicts lol guess it didn't get saved somehow


公开课笔记可以贴到issue里呀


@chrisvest sounds good. I'll try to makes these changes.


I'd rather not lock to any 5.* version, as it would require updating the composer.json file when 5.3 rolls out.
Not sure if the `~5.0|~5.1` does any different than `~5.0` to be honest.


wew


Do you remember in which Zope2 version the templates were unified ?
Is there a reason you never made a pull request ?


whats error


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


@flexiti its basic ws2812 example not about tcp system. if you feed wdt you can use endless loop in init. also your way like sming and its good i think. i will say again its quick fix :) and one more thing. readws2812 not right name. try writews2812


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


This is the repo I was talking about, and my latest commit...

@lukesargent @lauratownsend


`~5.0|~5.1` is the same as `~5.0`. The reason to lock it to `5.0` and `5.1` could make it hard for developers to update their apps to future version of Laravel like `5.2`. Since we don't know if anything is going to change in `5.2` that could break this current setup.


Please count the number of conflicts.


I wouldn't use Impuls for this, impulse is for jumps, but you're telling it to jump forward every fixedUpdate


@markohlebar Thanks very much. Your plugin is very convenient.👍


Create Oracle view in with SYS or SYSTEM or you can create this view in another schema (different user)  granting relative sys/system view


Hey Nick, thank you! That structure is just what I was looking fo.! This is the only bit I'm not sure I understand. What is the purpose of the module / what benefits does it bring? The app seems to run without it..just not sure..
Thanks so much again :)


Thanks! I was also looking to merge the Travis and the tox settings, feel free to jump in :-)


this is a very positive comment. Well done. Good Job. Keep it up.


think some of them might help my device too :P


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


I don't think it's something to worry about now. If 5.2 would break something that 2.0 of this packages uses, we will fix it then. If no fixes are required, I'd rather have support for all 5.* versions in the 2.0 branch. It's a lot easier to maintain. Maintaining 1.0 and 2.0 is already a pain in the ass. So rather not more versions than needed ;)


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


"not to overwrite"


@lindamartin Instead of saving measures to the context and later retrieve them, we keep track of the files for which we save measures. Does that answer your question?


fix #202


What about an option for the return trip? This could be the same overflow menu like in recent trips and  also provide the delete option.


目前对的理解


"departure" left-over. Do you really need this function?


@mikispag commited proper regexp here and recompiled
https://github.com/sshilko/jQuery-AS3-Webcam/commit/cf965c4c9bde74de8bb8909167724a22be28b5cd

http://www.sevenson.com.au/actionscript/testing-regular-expressions/


"departure" left-over


I don't think this would work for the exclude matrix though.


The target framework should be fixed to net45, also check all other packages.config files.


I think it will probably be easier to get meejah's deletion changes merged and tested first, then come back to this.


@lindamartin which one? The check on ncloc is not removed, it's just 2 lines below. The check on the linesToCoverMetric measure can be removed because reaching the current path means that we did not save coverage measures on the file.


Ok, sounds good. How come it's mandatory if `--inner-dist statusDB` is specified then? Can it not use your default of 135?


@pynicolas yes it does! Thanks.


At least this is my hope :-)


Custom TrailerRecord is overriden by default TrailerRecord.


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


`1.8.1` still in comment block


@pynicolas indeed I was talking about the second one. Got it thanks!


It makes more sense to `init()` the effct first and then to start it. This could reset class variables and etc.


Wouldn't it make it harder for developers to update their apps to 5.2 if we lock it 5.0 and 5.1? As it would require me to update the package first.


shouldn't https://github.com/openSUSE/obs-build/pull/210 fix it?


Yes, but I think I sent a request recently that was accepted correcting both conf-gmp and conf-mpfr
so that they are more portable


Could this block be moved below ```ui.search.setOnClickListener()``` so the order in ```onCreateView()``` follows the visual order on the screen?
Also, what is the advantage of calling ```displayFavourites()``` in ```onResume()``` and not here?


Okay, you are right, there seem to be an issue with cbr. I think most of my cbr files have a cbr extension, while they really are cbz. You are probably right about the XADMaster framework. I seem to recall having had some issues with XADMaster when I cloned the project. I'll see if I can figure out what I did to break cbr.

Also you should check out maddTheSane's arc branch. It doesn't seem to have any issues building as long as you remember to run `git submodule sync` and seem to have merged any changes from my master branch. It's probably going to be merged into my master if I don't find any new issues with it. Sadly this one also seem to have issues with cbr files.


k mala es el hambreeeeeeeeee

___________________________

Richard C. Hidalgo Lorite
Director de Sistemas
ISI Argonauta, S.L.
C/Alcalá 196, apto. 13, 28028 Madrid
Telf: +34 91 356 85 08
http://www.argonauta.org

El 26 de octubre de 2015, 11:43, Eusebio <notifications@github.com>
escribió:

> JODER!
>
> Sabes eso de cerrar el ordenador a la mitad...
>
> Lo miro en la comida.
>
> E.
>
> .........................................
> *Eusebio Reyero*
> eusebioreyero.com
> mail: ereyero@gmail.com
> phone: +34 661872775
> .........................................
>
> 2015-10-26 11:36 GMT+01:00 richhl <notifications@github.com>:
>
> > @ereyero <https://github.com/ereyero> revisa please el commit porque te
> > has cepillado algunos fich y creo que no has añadido los que pretendías.
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/isiargonauta/openmadness/commit/f02504491b37f8c89cf051608b2cb8fef271e9a7#commitcomment-13980437
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/isiargonauta/openmadness/commit/f02504491b37f8c89cf051608b2cb8fef271e9a7#commitcomment-13980550>
> .
>


@jvasileff see https://github.com/ceylon/ceylon-ide-eclipse/commit/466df4757bf0d65bb8b7aabd816cbd213637cd8a. Does that fix it?


Mhm, I think this should even work on Darwin, so `unix` should be okay I guess.


vários cuidados:

1. Não utilizar `window.onload`, utilizar `$(function() {});`
2. Não utilizar `document.getElementById`, utilizar `$('#coundDown');`
2. Não utilizar innerHTML, utilizar `html(i)` do jQuery.


@davidfestal apparently `getPhasedUnit()` can return `null`. Is that reasonable?


@UnixJunkie Ok. Should I remove changes related to conf-gmp and pull only zarith's ones?


isso não está correto.


This change suggests that some non-printable characters were added.
In this case, it's a BOM, and that's going to cause some problems to the rather stupid lesson_format code.


May I please ask that how could I add the operand box (the place to input the function expression for BigOperator ) to the other function like integral, limit and log?


could you elaborate more your thought ?


Please check online documentation for references to INSTALL_WITH_RPATH.


I've put this in the tooltip for the search edit filed. BTW, does anybody know why is the search_pattern created not in .ui file, but independently?


two times?


Also i have no idea why github did that.


This was an old code, I've just copied it to .cpp from .h. But OK, it can be formatted.


Which is not a problem, because sizeof(jlong) is apparently always 8. The issue is that sizeof(void*) is only 4 on 32-bit machines, so you are writing only half of the word...

Anyway, I already fixed all these instances in https://github.com/danielkitta/libsigrok/commits/cxx-bindings-work%2Bmartinling


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


Change to 0.5.0


They should be sorted by "most used", right? Then this line is correct.


want to check


thank you. That worked


I don't like the setting of size this causes a really large window on my screen..
Does this work for you?

 build(Ask, {oframe, Tabs, Def, Flags}, Parent, WinSizer, In0)
   when Ask =/= false ->
-    DY  =  wxSystemSettings:getMetric ( ?wxSYS_SCREEN_Y )*3 div 4, %% don't take entire screen.
+    DY = wxSystemSettings:getMetric(?wxSYS_SCREEN_Y)*5 div 6,
     1 =:= Def orelse error({default, 1}),
     buttons =:= proplists:get_value(style, Flags, buttons) orelse error(Flags),
-    NB = wxNotebook:new(Parent, ?wxID_ANY, [ {pos,{-1,-1}},{size,{-1,DY}}  ]),
+    NB = wxNotebook:new(Parent, ?wxID_ANY),
     AddPage = fun({Title, Data}, In) ->
-              Panel = wxScrolledWindow:new(NB,  [ ]),
-              wxScrolledWindow:setScrollbars(Panel,-1,10,-1,10), % no horizontal
+		      Panel = wxScrolledWindow:new(NB, []),
+		      wxWindow:setMaxSize(NB, {-1, DY}),
 		      case os:type() of
 			  {win32,_} ->
 			      BG = wxNotebook:getThemeBackgroundColour(NB),
@@ -827,7 +827,7 @@ build(Ask, {oframe, Tabs, Def, Flags}, Parent, WinSizer, In0)
 		      end,
 		      Sizer  = wxBoxSizer:new(?wxVERTICAL),
 		      Out = build(Ask, Data, Panel, Sizer, In),
-              wxPanel:setSizer(Panel, Sizer),
+		      wxPanel:setSizerAndFit(Panel, Sizer),
 		      wxNotebook:addPage(NB, Panel, Title),
 		      Out
 	      end,


test


Yes, and you should try the edits I suggested too.


The simpler, the better. ;)


This modification permit to developer to write his own storage and do the oauth.js have a separate implementation of storage. In this state, the storage is developer charge which can implement custom storage.

exemple: In my example, I use a custom __CrossCookieDomain__ class which implement [docCookie](https://developer.mozilla.org/en-US/docs/Web/API/Document/cookie#Library).
```javascript
OAuth.init(
                'angular', {
                    $provide: $provide,
                    storage: new CrossCookieDomain('.staging.net', false),
                    storageKey: 'platform.tracking',
                    clientId: 'web-app-client',
                    loginFn: () => { window.location.href = '/#/login'; },
                    parseErrorFn: (jqXHR) => {
                        if (jqXHR.status === 401) {return 'refresh';}
                    },
                    tokenEndpoint: `http://localhost:5000/oauth/2.0/token`
                }
            );
```


@UnixJunkie thanks!


I also don't like these ugly prefixes, but I was told to name member variables like this in another review, so....


Since it uses OPT_P_IPWIN32, it could be rejected from server push with route-nopull.


Please I really need help on number of things concerning linphone. Can someone first of all give me the linphone source code for android and secondly
 help me with a detailed explaination of how to run and install it on
 eclipse for windows and if possible give me ideas on what I could work on
 to impove performance of linphone as a telecommunications engineering student. Sincere gratitude in advance.


Good point. Then we can also remove the long click listener.


Future should not contain IO.
Total amount of futures should be known (max: 4 + 1).


We're checking out "master"

---
*G. Ann CAMPBELL* | SonarSource
Product Owner
http://sonarsource.com

On Sat, Oct 24, 2015 at 5:19 AM, Luca Garulli <notifications@github.com>
wrote:

> I guess it was fault of release script that replaced that number. However
> the 2.1.x branch has version "0.7.5.201505241946". Which OrientDB
> branch/tag are you fetching?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/orientechnologies/orientdb/commit/052878252339667d9b7481726bddca1716fced59#commitcomment-13966517>
> .
>


Please also pass `--testsuite-name <something>`, otherwise the NUnit xml of each test has the default name of `TestResults-runtime.xml` and they'll overwrite each other.

I thought about making this a required parameter in https://github.com/mono/mono/commit/04bdf3b9332a4707d196b65deda1fdfba1b88a30, but didn't know where else this is used so I left it as-is.


Is there any way to have this not act as a scrollview, but always be as long as required to show all trips and scroll in the root view instead?
When the directions form includes via locations, there will not be much space to scroll favorite trips.

Also, there's padding at the bottom of this view, so trips don't disappear where the screen ends, but earlier.


Didn't know about this `Array.from`, anyway it looks much much nicer that `Array.prototype.slice.call`. It should be polyfilled by babel [[1](https://github.com/lukehoban/es6features#math--number--string--array--object-apis), [2](https://github.com/zloirock/core-js/blob/master/modules/es6.array.from.js)].


Also when you write patches I would prefer if you used our coding style, i.e. no space around () {} []
otherwise I have rework your patches.

I understand indentation is hard to get rigth in a non emacs editor so I can fix that.


Did you try turning your phone? When I do this I get a crash. I recently made Transportr support turning the screen anywhere. This commit should not break this ;)


I'm sorry I can't remember. It was just quick patch as I needed to debug something so I did not make a proper fix and pull request. I was just trying to get it working in Plone 4.3, not thinking about older versions.


Yes that's right.


Don't you need quickdraw driver for this?
https://github.com/MotorolaMobilityLLC/kernel-msm/commit/48448335385bad14e4002346c39d39c2249c4529


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


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


These lines shouldn't be there obviously. What would be a more appropriate way do to this test @MattiSG  ?


ich würde das zusammenschreiben.
"Attributpunkte"


I understand and merged it. Thanks!


//wwwwwwwwwwwwwwwwww


@alonewolfx2 :
I don't agree with you.
In my opinion ALL examples and libraries should be "Sming event driven".
It is the only/best way to let new users understand what the concept of Sming is.

Maybe it is also good to have an exampe : Basic_Sming_Programming


fughufhgfhgupfug


Eita esse commit ai hein!


"Juwel platzieren" kommt hier denke ich besser.


just curious if this removal doesn't show up that we missed here severity property?


Are you talking about the step 7 in this file here, right? This is wrong, I remove. Check the new version of Readme.md


Could you put here more descriptive name, please? I'd suggest to use corsUrlPattern


Yeah, it was a quick thing I added because individual inserts were very slow, even with synchronous off.

Insert can probably be replaced with a call to BatchInsert with a single element in the slice. Another idea I have is to add a configurable timeout (default 0), and have the TileDB wait that long for multiple insert requests, then batch them all together.


I took a guess and managed to squish the floatExpr bug, so now it works properly with GHC 7.10. I also think you can take a new cabal and drop it on an old install, so this should work even if we use 7.8. The only reason for preferring an already built one is to reduce compilation time and disk space usage on appveyor, so if it turns out to be problematic installing cabal via stack is the other solution.


Can you be more specific?


I think it would be great to change condition to avoid this empty line.


I think many people actually like the white border. I do not prefer it but I know quite some people who do.


Might be easier in the future to split each episode out into its own branch, so that it doesn't smush up with the PR you submitted for the episode before. Github will just keep pulling in the commits from this branch on your end even after you submitted the PR, which is what I think happened here.


I think it would be great to split this method to set of small business related methods.

Thanks :)


Лучше "Пользовательский", как в "Пользовательский CSS".
"Выборочный" имеет несколько другой смысл: https://ru.wiktionary.org/wiki/выборочный


@ElbertoOne : I completely agree with alwey. One or two numbers in a distribution could possibly broke the test.


```lua
local ipad_s = key:gsub(".", function(c) return string.char(c:byte() ~ 0x36) end) .. string.rep("6", 64-#key)
```
就够了吧.

另外, 从性能角度, 把 function(c) ... end 提到外面定义好比较好. 不用每次调用都重新构造一次 clousre 对象.


Так а зачем 50px отступ, что он даёт? Имхо в упор надо делать.


Note - I'm aware this broke duration parsing. Will fix!


Hallo Stephan, and cool work! :-)

There is an Error in

   for (int i = 8; i < count; i+=4) {

count is the Int-count (4 Byte) not the one byte count
   int count = (mapContent.length/4) - 2;

so you need to write
   int count = (mapContent.length) - 8;

or
    for (int i = 8; i < count*4; i+=4) {

OR

Maybe it's a sign and unsigned problem in

   result += mapContent[i] << (i - offset) * 8;

Try using
   result = result | (mapContent[i] << ((i - offset) * 8));

Regards Thomas

Am 25.10.2015 um 14:43 schrieb Stephan Bauer:
> @tomsoftware Could you help here?
> I was not able to get isChecksumValid() to work.
> I load your a.map file you send my by mail. The ```System.out.println(fileChecksum);``` prints out 1781487142 like in your screenshots you send me. So this fits, but the rest is not working (```if (currentChecksum == fileChecksum)```never gets true.
> Do you have some advise?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/jsettlers/settlers-remake/commit/1d9983a6f1894e63edbb9e116e78059c1b55f35b#commitcomment-13971850
>


either Cracow either Kraków


bad id


Could you add space here `((AntiAffinityPolicyNameRef) antiAffinity)`.

Thanks!


Actually, if this test will stand much better after 10000 games, this is the way to go.


Защо не направите един файл с инструкции или информация, които да поставите в папката с домашните?


Also could you add space between bracket and antiAffinity.


Well, then we don't need an adapter and a RecyclerView. We can just use a LinearLayout.


They would have to wait but they will be certain it will work. As of now if they were to update to 5.2 it could break their current setups even though this package says it has support.


Consider using styles


like the white border? please not that this is not a consistent white border around the picture but just some whitespace when the picture does not fit.
So just on one or two sides of the picture.

On 26 Oct 2015, at 13:28, puneetbharill <notifications@github.com> wrote:

> I think many people actually like the white border. I do not prefer it but I know quite some people who do.
>
> �
> Reply to this email directly or view it on GitHub.
>


@alonewolfx2
It's just my suggestion ;)
-WDT should be touch as little as possible, otherwise there is a good chance that your program blocks WDT  when  it really be needed
-against the use long delay warns Espressif

resadws2812 , yes name could be changed to writews2812, it will be nicer


求一 pr


I don't have much time to work on it. You can fork my fork and make a second PR.


Just return `new AsyncResult<>(null)`.


Oh ok....got it! never mind what I said.


If `val == NULL`, we dereference NULL! Was the intention to check for NULL or an empty string (`val == NULL ||  *val == '\0'`)?


That is a good point. Perhaps even better to make SizeUnit typed enum.


It needs to be added before calling `showLess`. So I moved `showLess` and those lines to the bottom now.

It is called in `onResume` so the favourites get refreshed when returning back to the fragment


Similar to `compute_double_colon_len()` but here there is also `FUNC_ATTR_NONNULL_ARG(1)`


No the original was longer and displays OK (this too).
Regarding typos, I'm happy with only one :-o


Слишком много букв.


its more correct, I support it


then I leave the the "Crop" config for now, ok?  we will see how the photos look like.


No need to init non-active effects


@ztizzlegaming do you got any script to update the prices/values from somewhere? or do you do manually?


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


'+1' as some hackers say!


Typo: I think you meant "2.7" in the commit summary.


I think you can replace that whole block with `return Optional.of(desiredType.isPresent()
                ? TypeCoercions.coerce(resolvedConfigMap.get("dsl"), desiredType.get())
                : resolvedConfigMap.get("dsl"));`


On the contrary. Later all you will do is start them


How about separate "Sign out (#{current_user.email})" ?
Like this:
```ruby
- if user_signed_in?
  %li= link_to "Sign out", destroy_user_session_path, method: :delete
  %li= link_to "(#{current_user.email})"
- else
  %li= link_to "Sign in", new_user_session_path
```
IMO, It'll looks much nicely in UI side.


Great. Very good!
Still, you need more text to make this useable for everyone.

Describe step by step with a concrete example what the user has to do.
(e.g. "do this call to get a list of all things available." "then, with the list above, iterate this call for every thing from the above call. the number needs to be replaced by the thing id of the respective thing above".)


Renamed to length and merged


just init here before starting
and remove the init() call from the start() in effect.cpp


I guess this needs ifndef checks.


Where is the quickdraw driver? :P


Sure, no problem.


You can init before start, no need to keep inactive objects might matter when effects are more


:+1: :dancer:


Log the reason ("expected content type to start 'image' but found: ..") rather than throwing and return a null response.


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


My bad, fixed now


Thanks, please send me a pull request when
this patch became actual.


@ewels actually, I'd like to check through all samples, but you think it's better to exit while error occurred?


Всего +6 букв, а предыдущая строка всё равно длиннее.


This is not even checking that the `testName` user actually exists. Is it because user enumeration is not provided?
Also, I see no test that providing `testName` and `testPassword` is a valid login.


Originally the linear and angular velocity of the feet were integrated and then used in the constraints equation to correct numerical integrations errors. Then , the position was no more calculated by integration but using forward kinematics so only the angular velocity is actually integrated and used in constraints correction. Since numerical results shows that this correction is useful to reduce numerical errors, I kept it in the code. A better procedure is to obtain the feet orientation by forward kinematics in quaternion and convert it in euler angles, but since the feet orientation integration error is not critical while a wrong choice of euler angles for some positions is, I didn't do this for now.


`~5.0` ALREADY includes all of `5.x`. Laravel does NOT follow semver, so allowing any `5.x` is a VERY bad idea.


Oh I missed locking part in cpufreq.c driver. Thanks.


Hey just updated my repo with the pbr dx11 stuff. I am getting the bloom thing with DX9 here :/ .... Was this happening before the DX11 merger??

-----Original Message-----
From: Brian Roberts <notifications@github.com>
To: Azaezel/Torque3D <Torque3D@noreply.github.com>
Cc: Tim <rextimmy@aim.com>
Sent: Mon, Oct 26, 2015 4:18 pm
Subject: [Torque3D] from @rextimmy: cubemaps with mips in dx11 operate closer to opengl. need rgb+mip# as a seperate entry. not stored in .w (e8f224f)

from @rextimmy: cubemaps with mips in dx11 operate closer to opengl. need
rgb+mip# as a seperate entry. not stored in .w

---
Reply to this email
directly or view it on
GitHub:
https://github.com/Azaezel/Torque3D/commit/e8f224f7a34f6db8f0c0bb5c2ab32dd6c7a641e6


Only when 5.2.x is released should you allow usage with it. For all you know, Laravel 5.2 will change every class name.


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


Thanks - will check that again. :)


Да и инпутбокс можно сделать поуже. Зачем ты их все такими широкими делаешь, сильно больше их контента?


Doesn't this risk leaking server classpath or file system information to the CF user?


It ensures that other modules which may be called from the event thread should not be blocked due to event thread.


Ich sehe das hier an so manchen Stellen. (Charakter Name, Klassen Name, etc.) Komposita werden im Deutschen zusammengefügt. Deswegen haben wir das Wort Handtasche anstatt Hand Tasche. Bei ungewöhnlichen oder langen Zusammensetzungen kann man einen Bindestrich benutzen, aber ansonsten bitte zusammenschreiben. (siehe http://www.mein-deutschbuch.de/lernen.php?menu_id=54)


a nice thanks for the link


committed successfully


I am thinking to leave this because some targets have system fastboot problem on flash due to small partition ...... like on leo ..... are you having the same issue when compile gapps with ?


is this neccesery?


ça me parait bien mais en fait stickerwriter est un projet abandonné ^^ (je le garde comme backup)


Looks good, please put there also an assert before `self->_argCount`, something like `ASMJIT_ASSERT(varMapping != NULL);` because I really assume that :)


Be careful with indentation. See this line and others.


Я хотел его сделать вообще в 2 раза длиннее, чем сейчас, потому что можно вводить в формате `.rgba`. В остальных случая тоже допустимы длинные выражения.


Forms have:

- FileImageSource - we could add `FromFile` and  `OrFrom("", Assets` method
- StreamImageSource - Shouldn't we add `LoadStream(Stream stream)`  instead `OrFrom("", Stream)`?

Will you add those methods?


213, w poniedzialek


It's a really short race condition, but this is anyway - is it worth taking a temporary copy of `tune`, zeroing tune, then deleting via the temporary? And also making `tune` a volatile pointer to prevent the compiler reorderring stuff.


Hmm, actually I don't think so. It is not possible to enter ex mode in the command-line window so we would never have `cmdwin` set here


Yes, you are right. Thanks for that.


Looks reasonable to me!


:dancer:
:+1:


:ship:


@gavinking : the phasedUnit attached to an `IdeUnit` is stored in a weak reference. So in full generality yes,the `getPhasedUnit()` can return null. However, in the case of a `ProjectSourceFile`, this will only happen when the source `PhasedUnit` has already been garbage collected, for example during a full build, when the `PhasedUnits` instance of a project is not held with a strong reference anymore. However, apart from this case, it might also happen if a reference to a `ProjectSourceFile` is kept (for example in a dialog, or view, ...) after an incremental build has been applied to this `PhasedUnit`.
In fact I'll open an issue about that, to clearly define the use cases when this can happen : it's a question I know I had already planned to answer in the context of the abstraction of the ceylon model management I'm doing currently.


:accept:


you can give me a little things


How to detect remote url images??? Any solution? Because, remote url images can't be detected as an image inside project filesystem.


No I donot have this issue on M2 and Z1C
besides that, I will revert this, as it does not work.

Do you, on your original repo, also have play-services crashing?


It didn't removed the files after the successful compilation? The way it's supposed to work is:
- If compilation succeeds, remove al the generated files
- If compilation fails, keep the files so we can loook at them and see what's wrong


Hi, thanks for your module. I'm using it for a project but I have to make some adjustements.
I made some commits related to what I need.
Do I make PR for each one and you select which one you want to merge?


This confirms what I thought : the rebuild has freed the previous `PhasedUnit` associated to the the `ProjectSourceFile` unit (since Unit object have a *weak* reference to the associated `PhasedUnit`).
@jvasileff : could you open an issue explicitly for your case, and assign it to 1.2 and to me ?


I see -- thanks!

You do raise a good point: the option to exclude drafts/un-published posts and the setting of `published: false` for each seems like worthwhile features to me.


- [ ] The course name not yet appeared below timer.

![image](https://cloud.githubusercontent.com/assets/8113555/10728364/54df1ab8-7bc8-11e5-8df4-d84ba94a59c8.png)


@digitalentity the derivative kick is still present in this commit


one of my friends have them include on castor and seems have gapps working I can't test due to that problem on flash anyway you can find zip flash of new gapps for marshmallow here http://forum.xda-developers.com/slimroms/general/gapps-official-slim-gapps-trds-slimkat-t2792842


We'll now see how much our tests are reliable. I already tested this with -3.4 ELO outcome:
http://tests.stockfishchess.org/tests/view/561807990ebc597e4f23e74d


What was the bug?


QQ


I did this because `chai-as-promised`'s rejectedWith wasn't playing nice with objects:

```js
it('prevents party creation if user is already in party', () => {
  return expect(generateGroup(leader, {
    type: 'party',
  }).then((group) => {
    return api.post('/groups', {
      type: 'party',
    });
  })).to.be.rejectedWith({
    code: 'bar',
    text: 'Already in a party, try refreshing.',
    foo: 'bar',
  });
```

would pass, because all rejectedWith cared about was that it was rejected with an object. Didn't care which object.

However, you're right. This isn't great, but I just realized I can do this:

```js
it('prevents party creation if user is already in party', () => {
  return expect(generateGroup(leader, {
    type: 'party',
  }).then((group) => {
    return api.post('/groups', {
      type: 'party',
    });
  })).to.eventually.be.rejected.and.eql({
    code: 400,
    text: 'Already in a party, try refreshing.',
  });
```

I'll reinstate the old reject code, and update the tests to reflect that. Thanks for pointing it out. I had a gut feeling that there was a better way, just needed someone else to confirm it :)

Also, regarding the message returned, I think it _is_ important to assert on it, in the case of routes that can return different errors under different conditions, but it's very brittle to assert on the text itself.

In the near future, I'm going to write a function that checks the error string using `i18n` and fail the test if:

• the result does not match
• the result and the test equal `error processing string`
• the result and the test equal `string not found`

Any questions?


@rextimmy: http://i.imgur.com/XdYhSC7.jpg 'stock' PBR, http://i.imgur.com/hYx1RRR.jpg deferred dx11, http://i.imgur.com/BUHLkt0.jpg pbr dx11. hrm...


`ibmiotf` is the old version of the client


@jamescun does it matter if the command is still running after the timeout ?


josh


Can't call uniq in sets. Converting groups to trees makes descendent_attrs a set instead of an array / ar_relation.

Sent from my phone. Please excuse my brevity.

> On Oct 26, 2015, at 08:01, Robert Thomas <notifications@github.com> wrote:
>
> What was the bug?
>
> —
> Reply to this email directly or view it on GitHub.
>


that is the source for this update :P


又见大神


ack


fuuuuu, a good one


There may be more bugs of this type (uniq is called 5+ times elsewhere,  but I just fixed the most immediate error, which prevented opening the group page in production )

Sent from my phone. Please excuse my brevity.

> On Oct 26, 2015, at 08:01, Robert Thomas <notifications@github.com> wrote:
>
> What was the bug?
>
> —
> Reply to this email directly or view it on GitHub.
>


I agree, a custom parser written with FParsec should be fair more maintainable than this.


Mmh, why?  I'd prefer to keep the door open for `Prop`.


This is an image for galileo generated by Yocto. The scripts mount it to add an initramfs, with the soletta program as 'init'. If you don't want to deal with Yocto, you can download the image from here: https://software.intel.com/en-us/programming-blank-sd-card-with-yocto-linux-image-linux


"This path of this file." Huh?


After the Update it doesn't work anymore (you said) I wont lose my data in /var/www/html. Must I completely remove it (data lost in var/www/html)?? Or can I press "install with Apache" in the Installation menu and the  data wont get lost.??Thanks for a short reply!


Also, as I worked with the data some more, the word "Item" seemed more intuitive in the scan results than the word attribute, so I changed the name.


Is this because you only want the timeout on the connection initialisation ?

The http.Client has a Timeout field that is used over the whole lifetime of the request. See https://golang.org/pkg/net/http/#Client


@zimbatm checks should not cause side effects, so the command will continue to run (I felt this was safer than trying to clean up) but buddha wont respond to its result.


Also, with `Prop` no longer reserved, the `sort Prop` hacks become even more confusing.  So these should be cleaned up first.


@idella consider using `doicon -s scalable icons/${PN}.svg` to install into modern icon cache locations (`/usr/share/pixmaps` is what is used by default with doicon, but it's rather a fallback, `-s <size>` will cause it to be in `/usr/share/icons/hicolor/<size>x<size>/apps`)... and then update the icon cache via gnome2-utils.eclass:
```sh
inherit gnome2-utils distutils-r1 eutils fdo-mime
[...]
pkg_preinst() {
	gnome2_icon_savelist
}

pkg_postinst() {
	fdo-mime_desktop_database_update
	gnome2_icon_cache_update
}

pkg_postrm() {
	fdo-mime_desktop_database_update
	gnome2_icon_cache_update
}
```


Yes, I see that. But the recovery mode is somehow not working.


@zimbatm Yes, connection initialisation is where the check is most likely to fail (i.e. the service is not available yet) and where you want to know the status before continuing.

You are right however in that `http.Client` should also have its timeout set.


great!


yeah deferred/pbr dx11 has less bloom than 'stock'. hmmm


Вот это не должно быть так - в таком виде оно не работает. Он читает пока не появится "конец файла", а единственный способ послать "конец файла" в сокет - закрыть его.

Как это можно реализовать по-другому? Чем отделять сообщения друг от друга?

Гугл говорит, что можно в первые четыре байта уложить длину сообщения. Тогда можно отказаться от Buffered классов, наверное.


Fixed :)


Та же проблема, что и в клиенте


Aha, ok if it's intentional then that's fine - leave it as it is..


>Альзо, вкладка "Посты" перенаселена. Надо что-то делать.

Можно сделать вкладку "Треды" и вынести часть туда (и не только из "Постов"). Если не хочется расширять окно, можно раскидать "Фильтры" по другим вкладкам и вынести работу со спеллами в отдельный попап, типа как правка настроек и пользовательских стилей. И править спеллы в большом окне будет гораздо удобнее.


TadaAAaaa ! Bug killer was here before the bug !
ah ah aha ...
cool , thx !
--------------------------------------------
En date de : Lun 26.10.15, Elieser Ademir de Jesus <notifications@github.com> a écrit :

 Objet: Re: [JamTaba] removing BUILD TUTORIAL, the text is in README.md now (ae7bd48)
 À: "elieserdejesus/JamTaba" <JamTaba@noreply.github.com>
 Cc: "Ezee" <h2oreggae@yahoo.fr>
 Date: Lundi 26 octobre 2015, 12h25

 Are you talking about the step 7
 in this file here, right? This is wrong, I remove. Check the
 new version of Readme.md

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


Hold command+R duribg the chine to go into internet recovery if the partition is missing.

It will take some time to download the image but it will boot into it.

Pe 26 oct. 2015, la 14:12, togitom <notifications@github.com> a scris:

> Yes, I see that. But the recovery mode is somehow not working.
>
> —
> Reply to this email directly or view it on GitHub.
>


Like.


I just took the first image with python code that was labelled for reuse. I didn't check it actually :)
If you'll have another image to use we could change it easily.


if there could be only one multi enabled at a time (and probably always should, if multi is enabled), it is probably more appropiate to use radiobutton for the different multi types and check against the selected index. what do you think?
i can look into this...


@mher Any timeline on releasing a version with this fix?


Because this is a public constructor, it must first be deprecated and replaced with a call to the new constructor (with the appropriate default value for the new param), and its Javadoc should indicate the new constructor to be used:

http://wiki.pentaho.com/display/PEOpen/Deprecation+Guidelines


Groups have been trees all along. This must have been caused by Matt's PR of turning observations into a set instead of an array.


Мне приходят уведомления на почту о новом комментарии, так что моно не дублировать в почту, а общаться по коду здесь. В почте я тебе ответил на вопрос


horrible


On a second thought, we can still have `Prop` as a built-in later.


А если по-минимуму, то можно просто перенести стиль кнопок в "Общее" к другим настройкам стиля.


@alwey: Ok, I see that I overlooked something. Thanks for the clarification.


thanks @Ealdwulf - I have submiited further additions.


unnecessary changes


@mattyb149, this class is test harness. Do I need to follow the procedure for it?


@bk79 yes, it was not a top priority at that moment. will remove setpoint from d-term soon


Fortunately the only thing that should impact is observations, since that's the only thing that was turned into a set


It does, just copy and paste the same string you used above (`DJANGO_VERSION='>X.Y,X.Y-1'`)

Btw you can respond to line comments below the line comments; no need to continue the conversation at the bottom of the page/commit level.


Good point, I will ask @dkincade and reply here.


@IIvec, Thats interesting... But there was a change that went in between. Main thread no longer updates th->rooDepth now. But I'm not sure if that makes a difference.
====
           for (Thread* th : Threads)
               if (th != this && th->depth <= depth)
-                  th->depth = depth + Depth(3 * log(1 + th->idx));
+                  th->depth = depth + Depth(2 * log(1 + th->idx));

====


153, poniedzialek rano


Namnet borde vara mer deskriptivt än bara stadsnamnet


Veldig dårlig setning. Den kan vi ikke bruke.


check it out pls


You can als set Verbose mode in NVRAM then you can see what is happening while booting ```sudo nvram boot-args="-v"```


I found out a bug so far with reading unsigned/signed but still it is not working yet. I will keep on working on this.

My first target is to finish the checksum check in order to understand unsigned/signed transformation and so on before I go on reading the content.


If using standard in this way is not recommended, you should tell feross that: https://github.com/feross/eslint-config-standard ;)

I don't mind configuring it in a more "standard" way, this just seemed like the most straightforward.


Probably should be "The path of this file".


What does this allow which previously wasn’t allowed before? This already compiles for me in a dist without this commit:
```ceylon
Integer i { switch (0) case (0) { return if (true) then 1 else if (false) then 2 else 3; } else { return 4; } }
```


213, poniedzialek


test


йоб, тебе не кажется, что это уже горе от ума, ліл? Что бы ты некорректное не ввел, стиль просто проигнорируется и в самом плохом случае кнопки просто будут черные.


Salut Teo , ai incercat sa vezi daca iti ruleaza codul ? Nu sunt sigur dar cred ca la comanda  "print" tot timpul trebe sa pui stringul in paranteze : print("teodora");


about ? er dette RDFa -attribut? just curious


Yep, please change it (and in the other files)


searchTerm.length() - 1
(spaces around minus)


Found the french article!
https://fr.wikipedia.org/wiki/Point_de_Lagrange


Thanks for your advice. I didn't think about reading cpython's source.
I learned many things from you.

I found symbolic link implemention from python 3.5.
https://github.com/python/cpython/blob/3.5/Modules/posixmodule.c#L7124-L7192

But, In python 2.7, there is no symbolic link implementation.
https://github.com/python/cpython/blob/2.7/Lib/test/symlink_support.py#L30-L33


Не все будут чёрные (кое-какие станут прозрачными). Я просто не хотел лишний раз вызывать `updateCSS` после каждой введённой буквы.


inline


Android App which shows the time table of College/University.
Android
Fragment
Java


done


i'm glad such a button is here


It might be a bit late for this model, but in the future, when naming properties, don't include the object name. So instead of ObjectNameProperty, just use Property.


Another database? Was this by mistake, or by design?


Perhaps concatenate, coerce to `DNAStringSet` and split them? Would probably be slower though.


I don't have another image.


This is moving to EF 6. Did y'all agree to this?


You should follow Coding Style in new files and rewrite old to follow it if you make significant changes.


App Settings Implementation. Kindly Review The Code @Ashish-developer


You have a bug somewhere in the code...


Denne bør hentes fra pillardata


Perhaps use `as.character` instead of the `@.Data`.


when I run the command this is what I get!
nvram: Error getting variable - 'boot': (iokit/common) data was not found


So let it be till the best times.


@ElbertoOne: Your are welcome. I may have sounded a bit harsh, but I wanted to prevent this test from being put in a bad light. It will be interesting to see this test at 3 threads. Assuming it will pass at 24 threads, at 3 threads it uses the sequence (0,2,2). I am not aware of a test of this sequence.


This commit breaks the build since we now have two android:inputType attributes.


Is it better : https://gist.github.com/6c090ab5a23faa46e8bd ?
Otherwise a warning would already be better as we would know we are missing something.
Thank you for the review! :)


А вообще лучше использовать эту штуковину: https://developer.mozilla.org/en-US/docs/Web/HTML/Element/input/color

Только для сафари оставить просто текстовое поле.


I'm not even sure if there is any convention around VCF that distinguishes `1/0` from `0/1`. Probably fine for now.


Os novos campos adicionados são específicos para o Itaú. Teria que exibir os campos condicionais a cada banco, pois os códigos são diferentes.


感谢大神帮助


Если править циферки в #xxxxxx, то updateCSS все равно будет каждый раз вызываться. Я не думаю, что кто-то всерьез будет туда вводить туда rgba. Эти люди будут использовать стили. А Вводить туда inherit (который у меня чому-то не работает, кнопки черные) или transparent (wut? Белые символы с прозрачным фоном не айс) это вообще экзотика.
Ну, есть и есть. Еще пригодится.


Yeah I already saw your comment on Imoseyon's git ^^


`switch` _statement_, not `switch` _expression_.


If the data are raw originally, could we not just do `newGT[as.integer(GT)]`?


qBittorrent uses m_ prefix for private/protected member variables (this is part of our Coding Style).
These prefixes (ugly on someone's opinion) allow you to quickly distinguish member variables from others.
This, of course, may not be so important for those who do not have to observe a lot of people's code.


Good Job!


Tem como deixar mais genérico e menos específico para o Itaú?


Fantastic! Thanks for the commits and reference. Today I'll do another pass on both [en] and [fr] ( mainly for colors and layout of speech-bubbles ).

Thanks again for the care you put in the english translation; right now as we comment thousands of reader enjoy it ( http://imgur.com/gallery/8ABWs/ , was front page of Imgur, an honnor ) :)


Si CtrlPersistencia és static, no cal instanciar per a invocar storeTable()


I do have a switch *statement*, don’t I? I couldn’t `return` inside a switch expression.


Her har jeg gjort endringene..


Are you sure thous need to be removed?
At least one of them is needed and all but 3083710 are security updates.


blir classene automagisk oversatt til typer i 'byType'? er det et problem å bruke prefix? Hvis du har to like klassenavn, f.eks.?


Changed in 2dbbee46e074b39c60bfa1b1f066f10aaf4f3c60


Yes, that's what I'm thinking too.

I don't think the `sort Prop` hack gets more confusing. Previously `Prop` was syntax for `typeError "Prop is no longer supported"` and not the actual sort `Prop` used by the hack. Removing the syntax for the error should make no difference. Cleaning up the hack would still be nice though.


yeah, that was the thing... I guess it never worked? Not sure, perhaps a php update made it break or something else. Whatever. Works now I hope.


@lucaswerkmeister well it doesn't have an `else if`.


403 error on that page.


I'd say `unordered_map<std::string>` is a better option.


@ewels it's not mandatory anymore, changed to:
parser.add_option('--adapter', action="store", dest="adapter", default="135",
    help="Length of the adapter. Default: 135")


А этот атрибут прям работает? Ни разу не видел эффекта от него


Agreed.


Congrats! Good luck with the launch tomorrow!


It has an `else if` expression… but an `else if` statement also seems to work:
```ceylon
Integer i { switch (0) case (0) { if (true) { return 1; } else if (false) { return 2; } else { return 3; } } else { return 4; } }
```


seriöst namn kalla den nått annat


Yep, that looks fine. I'll make the parameter required tomorrow (it's a holiday here today so I'm technically not working :P)


Many of us use special names for member variables, but not all select this notorious "m_" :)


Why not "return sol_json_is_valid_type(scanner, type)" ?


这个接口的返回数据应该不是这样的 xml 吧，希望参考 test_single_trade_query 那样用真实结构然后替换掉敏感数据。


Looks good.


will fail on node < 4 because of arrow functions


is the structure for my tests


@eherot you’re a champion for managing to get this merged in & shipping this.


Tag «type» is also essential and available at construction time. It's not useful for display purposes, but extremely useful for discovery, i.e. «what metrics does my service emit?»


emang kenapa fungsi relay harus di depan mas?


the correct way is put this inside the describe


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


It's still there; up on line 467.


oops. I wasn't aware of the js workflow then. made the change https://github.com/OpenSourcePolicyCenter/webapp-public/commit/16499b8e951c2f162f9505a84aa0a030d47f131c


Currently the `pdokcal_getPdoMemRegion()` only passes the base address, but for mapping the memory into user space, the size would also be required.

I will update the `pdokcal_getPdoMemRegion()` routine to also pass the `pdoMemRegionSize_l` variable using reference and update the `drv_kernelmod_edrv` implementation to pass NULL for the size parameter.

Please let me know if its okay.


Makes sense. Updated.


Ммм, в `<input type="color">` нельзя выставлять прозрачность. Да и выглядит на PC довольно страшно. Хрен с ним.


@lucaswerkmeister read the description of the commit: a `switch` _statement_ with an `else if` clause! I don't know how else to describe it!


- Please avoid adding db name prefix to sql files (my db name for example in diff)
- code formatting ,....


Да норм ты все сделал, пусть будет как есть. Как щас вполне заебись.


If not difficult could you split this lambda to multiple methods too.


Right, Python 2 support is why your implementation is valuable! The best way to figure out what errors to raise would be to look at what Python 3 does in those circumstances and do exactly that.

Also note how carefully the Python 3 code is written to avoid TOCTTOU race conditions.


true. I'll fix it


@gavinking With a code snippet ;)


ok. in some situation we might end up with a lot of running processes but it shouldn't really be an issue if we are careful


Hmmm, I'll look into this ASAP.


Thanks, thats progress!


No, you will have to add another variable that tracks the IRQ.

As of the Dt2w not working in deep sleep, @vasishath fixed it in mi3, he might have some clue about how to fix it here too.


Perfect :+1:


I didn't want to touch this issue with a km-long stick, but should I worry that it needs backend support?


What, this is now allowed?
```ceylon
Integer i { switch (0) case (0) { return 0; } else if (true) { return 4; } }
```
I hadn’t seen the spec commit (84fa0cdb8ec878cc451b2c6cef4150546a670367), so I assumed this fixed a parser bug, not that it introduced new syntax.

(Also, I think this syntax is crazy – suddenly, the switch is no longer exhaustive in the sense “exactly one of the blocks will be run”. And doing this two days before release?)


We are trying to get rid of sleeps.. not increasing them... this is not a valid fix.

I would rather keep the test failing, set to your ignore list on slower maching...  than adding this fix.


Maybe I just fixed that without noticing... :D At least my local version does not crash


Here's the commit reinstating the error code checks: https://github.com/HabitRPG/habitrpg/commit/04fbdaa10780bfe03c3e4fdcc8cf23337f790ea5


Great !
I will test with your repo.


I'm not sure. One of the players is a random player, so I think he should be generating different moves from game to game.

Jed
________________________________
From: jdove2 [notifications@github.com]
Sent: Sunday, October 25, 2015 11:02 PM
To: kochkov92/Tictactoe
Cc: Chou, Jed H
Subject: Re: [Tictactoe] fixed bugs in perfectPlayer, added reset board function to TicTacToe (a134d1e)

You're sure its no a seeding problem?

Jason

On Sun, Oct 25, 2015 at 10:21 PM, j-chou <notifications@github.com> wrote:

> The code compiles, but there's an issue with playMatch function in
> Referee. When we run judge.playTournament(1000), either player 1 wins all
> 1000 matches or player 2 wins all 1000 matches. We think the board is not
> being reset properly after each match and added a reset function, but still
> only one game is being played.
>
> —
> Reply to this email directly or view it on GitHub
> <https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_kochkov92_Tictactoe_commit_a134d1efa9359519ca3ea6d94725f18981fe19b2-23commitcomment-2D13975571&d=BQMCaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=drJyf_ptmyjtcnhwdvosmeUbQ0JOdEtdCmPOI-BoePM&m=DILN7XvT7YtHTyhbrCEBcpduOXXi8WH7AaZsDHtZdU0&s=4Ua60REwNOFsyFl8jzkOV1ytvzbQxd8zeCQtNdx6O10&e=>
> .
>

—
Reply to this email directly or view it on GitHub<https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_kochkov92_Tictactoe_commit_a134d1efa9359519ca3ea6d94725f18981fe19b2-23commitcomment-2D13975776&d=BQMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=1KfiozGfx5LnNfrTY5zLztV2UAUBX_95zLkKySJMpUw&m=QON4SNvUEcbQRQW5Ur4-eWIlEX8xzB5l0iJboSGb3Tg&s=AKBKjEjqh6zxz6vRNyKb2KShP79vV-EoGztsJEvWp5Q&e=>.


Vi diskuterte det akkurat.
Nå så forsvinner alt av prefiks, så dersom man har to like klassenavn, eller properties med ulike prefix så er det ikke støttet.
Åpen for innspill i ld-graph api


thank you and take care!


thank you


COMMITTING ALL THE THINGS


Would you mind adding a modeline?


Better add this call next to the `strlen(value)` call. Let's not duplicate this iteration.


>Perhaps even better to make SizeUnit typed enum.

+1


这个接口返回的数据太丑了。。。。
我改改


Cool, vous pouviez tout écraser avec `push -f`. Mais ça va le faire.


I know, but it's not the recommended approach;
> This module is for advanced users. You probably want to use standard instead :) -- [source](https://github.com/feross/eslint-config-standard/blob/master/README.md)

Either way we really should use the eslint-babel plugin since it actually fixes some shortcomings with straight-up eslint (specific to generators if I remember correctly). If we add an `.eslintrc` it's likely someone will be tempted to add custom rules at some point, which goes against the whole point of JavaScript Standard Style :bow:


我测试了一下， 觉得你这个太简单粗暴了， 并且没有解决问题。。。 :joy:


Why did you create a native query instead of a JPQL one?


In addition there's precedence already :wink:
https://github.com/atom/line-ending-selector/blob/v0.3.0/package.json#L23

---
**NOTE TO SELF**: Add `eslint-babel` to line-ending-selector.


good idea, but out of scope for this changeset.
i'll move this to a separate ticket.


Yes! Sorry xD


LGTM


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


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


测试的一个作用是知道这个方法会返回什么内容，可以直接用文档上的例子。


Please see https://github.com/cms-sw/cmssw/pull/12095


Try command+option(alt)+P+R to zap the PRAM(NVRAM) maybe that will fix it.

Pe 26 oct. 2015, la 14:32, togitom <notifications@github.com> a scris:

> when I run the command this is what I get!
> nvram: Error getting variable - 'boot': (iokit/common) data was not found
>
> —
> Reply to this email directly or view it on GitHub.
>


@Inrego Could you please contact me at my email address regarding Jackett? It is my username at gmail.com
Thanks!


function is overpower here :)


@pierremtb fixed it :)


Why we do import under cycle?


@FroMage seems to work in the JVM backend.
```ceylon
shared void run() {
    switch (process.arguments.size)
    case (0) { print("no arguments"); }
    else if (process.arguments.first?.equals("foo") else false) { print("first argument is foo"); }
    else if (process.arguments.first?.equals("bar") else false) { print("first argument is bar"); }
    else { print("first argument is neither foo nor bar"); }
}
```


For me it's strange solution for fix EA-74493


and on JS as well.


On a first look I believe `%02hhx` is the correct way to parse each hex digit pair into uint8_t.


I'm not sure this will work. The transformation from file.iso to file.ssif means that we no longer want libbluray to handle the stream. I think once we are inside CDVDInputStreamBluray::Open we can't fallback to CDVDInputStreamFile.


> I didn't want to touch this issue with a km-long stick, but should I worry that it needs backend support?

No.


No worries ;)


WHAT IS ALL THIS!?


... or simply a qcombobox?


test


What do you think about to ignore "ConnectException: Connection refused" for checkCanceled?


Already removed in next commits


Btw, I just pushed version 3.0 of the translator package.


Thanks for the feedback. I would like to learn more about percentages vs pixels. Also, I don't think I'm clear on when to use ems and when to use rems. As I am working through my final project, I need to know these two things especially, well, and a lot of other things as well.

Thanks again.
Terrie


@zertosh `opts.plugin` isn't documented. I don't think it's a good idea to include it here until / unless it is.


I would also suggest to not mess with the existing /huge/ function `IPV4_parser()`, but write a new one for your specific simple need, e.g. `IPV4_parser_hex()`.


> Also, I think this syntax is crazy – suddenly, the switch is no longer exhaustive in the sense “exactly one of the blocks will be run”

So:

```ceylon
switch (foo)
case (is Foo) {}
else {
    if (x) {}
}
```

is exhaustive, but

```ceylon
switch (foo)
case (is Foo) {}
else if (x) {}
```

is not?

Even though they mean _precisely the same thing_.

I used to think that too. But I got over it.


very beautiful way~


Deu bug no meu commit :octocat:


http://map.fichtenfunk-mk.de Scheint aber eine falsche URL zu sein.
Zumindest kann das im moment nicht resolved werden


De rien - fait moi signe si tu as besoin de quoi que ce soit.


I prefer just _. But maintainer chose m_.
This is a collective project. We need to follow conventions if we don't want to turn the code in the trash


hkh


この記述がないと例外処理後に表示されたフォーム画面で投稿する画像を選択することができなかったので加えました
下の画像みたいな感じです
#### 記述しない時
![2015-10-26 22 02 10](https://cloud.githubusercontent.com/assets/14070846/10729601/e1751828-7c2d-11e5-9bbf-dc64a69407ef.png)

#### 記述した時
![2015-10-26 22 05 09](https://cloud.githubusercontent.com/assets/14070846/10729617/f3348dc8-7c2d-11e5-82fb-7cfeb5dbade1.png)


Why letting zgen source the file anyway. Why not just generate the init.zsh file and source this file after the zgen save?


pls check


Intrinsics don't generally validate their arguments. This should be handled by the method system where possible.


@bolshoy fyi, i'll pr upstream soon :)


Tested here a couple of cases and I'm happy with the results. Thanks a lot, +2.


@GrahamCampbell, thank you for the correction. I didn't know about laravel's app->resolve() function.

I added the line to the project's AppServiceProvider register() method. Much cleaner that way.

Thank you.


Actually it looks like `archive_entry.h` is already including `windows.h` since commit e16fd72f8dc3d797fec56d77807b1d1df0131e5c.  I'll drop this patch.


#4399-ZoneSum-vs-VentilationRateProcedure-when-one-zone-has-zero-oa-requirement (rraustad) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


These changes also are in 2.4.2.4 branch (see 76685d7af8e659bc57828127b384264b296985f6).


Okay, I'll drop this for upstream libarchive and keep it in CMake only for now.


oops, been busy.  Fixed.


本当ですね
わかりやすいですし、これから使うようにします！


This should be put in master and c6 as those branches are currently doing the broken thing. It'd be best if fixes for issues which are not specific to 64bit could be placed directly in master.


Leave a comment:
"jKzyou.me"
is a registered domain name


So, I reread what you requested and it actually makes a lot of sense. I must have been particularly tired to not understand it the first time. Sorry for that. Of course "avc" will be mapped to "h264" and "hevc" to "h265" (h264 is the codec, while x264 is the encoder).
Actually you understood it perfectly. The naming pattern tries to map the file name to a scene complaint name. This means it tries to add the video resolution (e.g. 1080p), the video source (e.g. HDTV, DVDRip) and the encoder/codec, depending which one is required (e.g. hevc is not permitted in the name, so it will be mapped to h265). All this parameters together make the "scene quality" preset.

For example, "The.Adventures.of.Star.Fox.2640.AD.sd.dvd.divx.avi", will be mapped to:
```
The.Adventures.of.Star.Fox.2640.AD.DVDRip.DivX.avi
```
For rips I added BDRip, DVDRip and WEBRip now. I hope I didn't miss any. Also, thank you for pointing out that issue, it is fixed now. Hopefully I have been more clear this time. Please let me know if you spot other mistakes or have any other questions.


These changes also are in the 2.4.2.4 branch (see 7f66f715d6f97d3bdb1b19ac1008303b17e8384f).


thanks! IMHO this is the best way to handle errors with critical parts.
We should take a look if there are more possible critical problems in global scope g*** Vars, that should be be catched in main and exit the program...


Should github repo variable be the full repo path? ie `mojotech/squadlocker`


The portability of code it independent of its utility.  We can integrate this change now and still replace the whole file later.


<img width="1364" alt="xcode toolbar" src="https://cloud.githubusercontent.com/assets/3208091/10729777/086d664a-7bc2-11e5-84c3-9b970ca625c4.png">


These changes also are in the maint-2.4.2.4 branch (see f42704dfa87fc0f6beab331cd50f843b5af2186b).


@tvsonar security?


Here you have to assign the return value of realloc to a temporary variable, or else you lose the old value on failure.


New UI Change


how about CDVDFactoryInputStream::CreateInputStream?


in run.sh it's spelled 'celery_broker'. now the link is broken :(


We already have this class at `net.smartcosmos.objects.builder.ObjectBuilder` -- is this a duplication?


the image 'postgresql' doesn't exist. I guess you mean 'postgres' ?


These changes also are in the 2.4.2.4 branch (see 5a92142dab958cee4adcc4b6986ab18162a165fb).


same here: Variables of postgres-image start with 'POSTGRES'. The 'QL' must go :(


ok this var is not needed


perfect


Ideally, a test should use the actual function in question (`get_targets`), rather than duplicating the code separately in the test.

That way, if get_targets changes, your test is still covering the actual codebase. Please make that change here,


I think he intended to point to his recent comment in #14331 (already answered by Aokromes).


Wow, nice one! As for the care, it's normal, most of the other languages depend on a good English translation, so I prefer to get it right early and not change after everyone else starts work!


I dont' think this is needed... Or if it is, I think it should be just <fcntl.h>


Ah, of course. Thanks!


Fixes #1


There are a lot more unchecked strdup calls. E.g. in cfg_dupopt_array()


These changes also are in the 2.4.2.4 branch (see 58ce7669ec5a0654cd21d2f0f7afcbb6e3440dc8).


New line at end of files, please. (There's a pycharm setting to do this automatically) It makes the code work slightly better with certain unix utilities.

http://stackoverflow.com/questions/729692/why-should-files-end-with-a-newline


you can supply extra `func ShouldAutoCommit(ctx context.Context) bool` too.


I now see why you needed the hex parser inside the `IPAddress` structure. You parse a hex string into a `Buffer` structure, the `Buffer` into `IPAddress` struct, the `IPAddress` into a decimal string; but this is a bit overcomplicated I believe, it took me a while to follow...


so that you can use autocommit.ShouldAutocommit(ctx) directly outer.


I have difficulties with Javascript


afsadf


These changes also are in the 2.4.2.4 branch (see a85ed0ef2d57023982fdee7ea623f48921da82f2).


updates/Midsummer Festival Misc Update.sql

Thank you for your answer and explanation. I did some quick research and it appears there is already a SpellFocus GO for SpellFocus 1365 : entry 300068, name TEMP Midsummer Bonfire. Basically, this is a placeholder and that means that we know such a spellfocus GO exists but we don't know its exact entry.
YTDB seems to have made the exact same deductions than you and I would agree with both. Nice. :)


This can't be done by default. Not all drivers support Prime, and they are only needed for certain use cases.
And and... The returned fd needs to be closed somewhere.


Do you have a `reconcile` style command to help users do this reset?


Everything looks good. I pressed "Install with apache" after that it works and it is updated to 6.0.12!!!


Probably nothing, other then virt_use_* was designed for VM's while virt_sandbox* booleans are for containers.

We can just combine them and eliminate the virt_sandbox ones for simplification.


#4399-ZoneSum-vs-VentilationRateProcedure-when-one-zone-has-zero-oa-requirement (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: Build Failed

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


Could be a single Timestamp value


Method has too many lines. [14/10]


Method has too many lines. [12/10]


No functional changes here.


There is not a reconcile BUT I have added new defaulting code into this that @smarterclayton mentioned.  In the api type defaulters if you do not have FSGroup/SupGroup strategies set it will default based on your UID strategies.


As discussed with Saman on flowdock: we may want to look into the use of `with TokuTransaction` in this script. (is it necessary here? Some migration scripts use it, others don't)


Why just not `return false`?


@hasufell Thanks for highlighting this - was missed initially - and for the tip to use `gnome2-utils.eclass` for updating the icon cache. The fix should be committed shortly. Cheers!


How do these settings interact with existing namespaces?  Say the cluster admin updates the SCCs so that I (the project admin) can use these cool new features.  He turns it on, but my namespace hasn't been updated.  Am I screwed?  Project admins can't edit their namespace.


Closes #28, Closes #21, Closes #24


Can we test it? Did You reproduce it before?


I am just curious about this line. What do you mean by method handler implementation and why is that bad?


Thanks!


> There is not a reconcile BUT I have added new defaulting code into this that @smarterclayton mentioned. In the api type defaulters if you do not have FSGroup/SupGroup strategies set it will default based on your UID strategies.

Based on other comments below, it sounds like you may have defaulted in a way that doesn't actually work for existing namespaces.  Is that something that's useful for people?


well done


Also, if this doesn't require an update, why do you tell me that it requires an update?


well done !


hello


凑 @paomian


very beautiful search~


We've added lambda/factory functionality through the config-api, so that we can now subclass (in -swarm proper) the generated classes, in order to add extra functionality.

For instance, our sub-class of JDBCDriver now implicit calls driverName(key) so that the user doesn't have to (but can) explicitly set driverName(...).

We've already been subclassing the root config Foo object from config-api with FooFraction, and this change just allows us to subclass on down the chain, via the factor lambdas.

    new DatasourcesFraction()
      .jdbcDriver( "h2", (driver)->{
        // configure the JDBCDriver here, but you're actually handed our
        // fancy sub-class that has already called driverName("h2") on your
        // behalf
      } )


niubi


good


good job


wonderful


en ,!!!


兄弟们  好啊


o


好


fdsf


哈哈哈哈哈


我是谁


哈哈哈


hhhhh


来个啥


还会刷新呢


Leading capitals in names look weird to me.  Can you give the user facing tag?


hah


夺无可奈何花落去在


牛逼


好神奇呢


hhh


这个好玩！！！！


好神奇，还发了个邮件，给邮箱


The `ServerConfiguration` interface is a -swarm thing, which is responsible for taking a FooFraction and turning it into the List<ModelNode> (or doing other magic that fractions can do), typically using the EntityAdapter/Marshaller stuff from config-api.


good


kkk


well....


This was a functional "useless" import.


略牛  略牛！！！


> The value should be a decimal number between 0 and 1.0.
> All ratio values must add up to less than or equal to 1.0.

These seem like unnecessary conditions. Is there any particular reason to limit this? I could use "20" and "80" the same way you use "0.2" and "0.8" and still the formula for calculating the ratio will work. What might be an useful limitation is to say that every value must be a decimal number greater than zero.


Работает, но тут не нужен. Удаляю.


We've been using the coding style where simple getter functions are without "get", i.e. above would be "height()".


May be you should add spaces


well...


#4399-ZoneSum-vs-VentilationRateProcedure-when-one-zone-has-zero-oa-requirement (rraustad) - x86_64-MacOS-10.9-clang: Build Failed

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


well done


ack


Of course, I mis-read your question.  The `FooConfigurator` is a simple interface to support the new lambda config stuff in config-api.

For every resource type `Foo` generated by config-api, we also generate a `FooConfigurator` interface for lamda-fication.

So you have multiple options now:

    parent.foo( new Foo("key").whatever() )

Or the factory/lambda style:

    parent.foo( "key", (foo)->{
        foo.whatever();
    } )

The lambda in the above is an implicit `FooConfigurator` implementation.


> it sounds like you may have defaulted in a way that doesn't actually work for existing namespaces

What do you mean?  The defaults are not namespace specific.  In the current code (ignoring this doc for now) if your FSGroup/SupGroup strategy is not set then it will default based on the UID strategy.  If you can run as an UID it will default to RunAsAny.  If you cannot, it will default to MustRunAs with no configured range which will look to the namespace for annotations.  If your namespace does not have the supplemental-groups annotation it will fall back to the uid annotation.  If none of these exist the SCC is considered invalid.

> Also, if this doesn't require an update, why do you tell me that it requires an update?

Just haven't updated the UPGRADE.md since the last change.  It's on my checklist for today


Now it should be OK in master. Have you tried?


@nad back-ported this commit into the 2.4.2.4 branch in 22f0f9d5f91f399fbb13456404c68f70ff86a06b.


that's not correct either.


removed backup files and added apache header


package ru.unn.agile.TemperatureConverter.model; or package ru.unn.agile.TemperatureConverter.core;