@Voyager1 the state needs to be reset on HOP_CHANNEL: https://github.com/FernetMenta/xbmc/commit/b4c0d6b6fe7e37519ccdea3c5d1f99e1821b1913

all the actions that were taken on CELL_CHANGE turned out to be wrong. a cell change can happen in the middle of a data packet.


Looks like this needs to be canola....not canoli...not canols...canola!


interpolation is deprecated, use str.format instead


It uses the actual color if the UIImage rendering mode is set to [UIImageRenderingModeAlwaysOriginal](https://github.com/chadsmith/react-native/blob/5ca6f36b59e0de829131054d351dd9b8b12dcd07/React/Views/RCTTabBarItem.m#L54).


Look,

One hour ago you wrote: But there are others which refuse SPRT test being
stopped on principle, so I think at this point maintainers should decide.

So, I stepped in and stopped the test. Simple as that.

You also wrote: So why this reprehension now and why so late?

There are things like Christmas, family, work. On top of that the incoming
Stockfish 7 release.
Maintainers are not here to babysit people and their tests. It's the
responsibility of individuals to behave sensibly and schedule meaningful
tests.

How to go on from here:
- Design a sensible testing strategy (taking into account available
resources, possible outcomes).
- Stick with what you've designed
- If your tests pass, raise a pull request. At that point, we'll ask for
additional tests if necessary.

On Tue, Dec 29, 2015 at 5:12 PM, pb00068 <notifications@github.com> wrote:

> Hi Joona,
>
> Im fully aware about the fact that SMP tests consume a huge amount of
> resources and for saving resources I already expressed my concerns about TC
> 30+0.3 on these tests 5 days ago (see earlier post in this thread) and
> since you were already mentioned, I guess you was also been notified about.
> So why this reprehension now and why so late?
>
> In the future please think about your expectations BEFORE submitting the
> test.
> How should I understand this?
> For me it is impossible to estimate in advance how strong a patch will
> perform (N.B.: I have only a 8 core machine at my disposal). I guess it is
> impossible for us all.
>
> Just measure its ELO value using a fixed game test
> I did it in the past and was criticied for not using SPRT.
>
> Just measure its ELO value using a fixed game test.
> Also on SPRT-tests the ELO performance can be measured easily and
> continuosly,
> for instance by using http://www.3dkingdoms.com/chess/elo.htm
> so I don't understand the point.
>
> That said I must add that I now feel dissappointed and intimidated.
> You also did't give me an answer about the [0,4] or [0,5] bounds and
> pardon [0, 8] sounds like a provocation to me. I really don't know how to
> go on, please tell me how to stipulate the next test.
>
> Günther
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pb00068/Stockfish/commit/02c9544ca4dd30de5f01cd4ac1cae9b73cae8793#commitcomment-15192724>
> .
>


this should have been detected by codacy!


global state is evil


Yeah, this was a crappy Windows workaround.  This patch is only applied on older VS versions.  see https://github.com/msarahan/conda-recipes/commit/1fabdb2476c8591c715a4024b58a8c3b44235291#diff-a79b984ab74c799bbc6fbb585815195fR9

I will wrap it with an MSC_VER version check, and submit as patch to SMQTK.


@mrshu Shouldn't we rename the main file from `project.py` to `board2slides.py` or something like that. Also what about class name `BoardSearcher` shouldn't that be also renamed to something more appropriate.


Nice use of `Object.create()`


I don't know anything about that, but looking at the readme for godep, what you did seems to be what was needed. However, the doc says that `-r`: "tells save to automatically rewrite package import paths. This allows your code to refer directly to the copied dependencies in `Godeps/_workspace`". It does not look like that happened? I understand from the doc that code importing `"golang.org/x/crypto/bcrypt"` would be changed to point to the local `_workspace` directory? Maybe I got it wrong.


but of course.This is a groovy file that is ignored by Codacy!d-oh!


'1' -> 'verbose'
'1' -> 'validate'


Ok to split clang detection between Apple platform and Linux/Other platform. By the way, can you use the operator -ge instead of -gt, this makes the logic clearer... Also, your patch has no effect on c++0x detection since the 2 tests are equal....


Yeah - me too! I've been pretty haphazard about this until I found this issue :)


loose the whitespace


redondant `std::cerr <<`


+1. same here.


Whoa Whoa Whoa


What are you calling `to_json` on?  My guess is you're calling it on the serializer, which is 'incorrect'. You'll need to use serializableresource now. See docs in master :)


TeamCity LabKey :: LabkeyModules15.3 :: Git sqlserver 2014 [Build 81](https://teamcity.labkey.org/viewLog.html?buildId=293912&buildTypeId=LabkeyModules153_GitSqlserver2014) outcome was **SUCCESS**
Summary: Tests passed: 218 Build time: 00:15:15


ahh, thanks :+1:


This fixes the following issue: https://github.com/biocore/emperor/issues/421


@epogrebnyak Reverted this commit in PR. No changes in sql files. Instead, sql is patched on the fly with username from `DB_INI_DICT`.


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


I'd call "redirect_uri" here "default_redirect_uri" instead. Else, one could easily be confused that it's what the system is supposed to use as opposed to being the fallback.


rename `a` and `b` to something with more meaning


I've been looking into this a bit more and it seems that we're getting a CELL_CHANGE when selecting the language. At that moment we're in a STILL situation with an infinite time (m_dvd.iDVDStillTime = 0). And as you don't put the status back to NORMAL at that time, you're entering this eternal black hole with nothing happening. The NORMAL state brings you back to reading some more data (which puts you back in STILL again, but with a 5 sec timeout this time.)
So what I suggest is to change it as follows:
```
    case DVDNAV_CELL_CHANGE:
      {
        //dvdnav_cell_change_event_t* cell_change_event = (dvdnav_cell_change_event_t*)pData;
        CLog::Log(LOGDEBUG, "DVDNAV_CELL_CHANGE");

        if (m_dvd.state != DVDSTATE_STILL **|| m_dvd.iDVDStillTime == 0**)
          m_dvd.state = DVDSTATE_NORMAL;
      }
      break;
```


Ditto to my comment above in the other function.


make local to `do_checks`


Please look at https://github.com/LubosD/twinkle/issues/26
Comment is wrong. GenericName and Comment must be different. This patch adds translations also.


Another option would be to use a default value with all the possible values, something like `tab|space` but this would make an invalid EditorConfig by default. @sindresorhus What do you think?


Agreed, pushed a change for this.


Wow such nazitude


h


For this to have a chance to work, you would have to increase the first one, and decrease the second one.
Maybe BishopPawnsUs = S(8+2,12+2) and BishopPawnsThem = S(8-4, 12-4) but of course you will have to find out.


The correct commit message is:
adapted learner table to changes in function listLearners (see mlr-org/mlr#580, fi…
…xes #29), restructured learner and measure tables a bit


the commit I linked fixes the issue


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


going to try now


This breaks plugman's support for the --www option.  The comment above is only true if the --www property is not provided.  This also breaks the platform centric development model outlined [here](http://cordova.apache.org/docs/en/latest/plugin_ref/plugman.html).

I logged [an issue](https://issues.apache.org/jira/browse/CB-10274) for this on the cordova jira.

I'll try to submit a patch but I'm new to nodejs and cordova development.


Could you submit a pull request of #26 based on current master?


it does indeed, thanks!!


really? can't I just rename the testcase to `test_check_prerequisites`?


Nice to see this added to the DB. In which way does it help to repeat the condition 3 times after the first?


Hi,
 you have made a great job adding this useful future.
Please  change
 String restore = Preferences.get(Preferences.ApplicationOptions.REMEMBER_WINDOW_POSITION);
To
 String restore = Preferences.get(Preferences.ApplicationOptions.REMEMBER_WINDOW_POSITION,"false");
at MainWindow class.
At that way you will set by default the REMEMBER_WINDOW_POSITION property to be false. This is necessary for the application because at the first run the window paramaters(x,y,width,height) are NULL, so there is no window and the application runs ultimately in the background.

Sincerely yours


I didn't take the step to "godep save", which will change the import path
of dependencies in the Godeps/_workspace files, although running "godep
get" successfully installed bcrypt and blowfish for me into my GOPATH.
I'll update the branch.

On Tue, Dec 29, 2015 at 10:24 AM, Ivan Kozlovic <notifications@github.com>
wrote:

> I don't know anything about that, but looking at the readme for godep,
> what you did seems to be what was needed. However, the doc says that -r:
> "tells save to automatically rewrite package import paths. This allows your
> code to refer directly to the copied dependencies in Godeps/_workspace".
> It does not look like that happened? I understand from the doc that code
> importing "golang.org/x/crypto/bcrypt" would be changed to point to the
> local _workspace directory? Maybe I got it wrong.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nats-io/gnatsd/commit/b8d76bad4cac2dd35b59ef107f46b48cd1df1704#commitcomment-15192903>
> .
>

--
*Colin Sullivan* | Principal Engineer
colin.sullivan@apcera.com <brian@apcera.com>  | @ColinSullivan01


OCD should have the participant's ocd-ids in there too.


Tonto e tonto, tontomon...


I don't know coding standards on this project but would suggest to break this into 2 steps...

It's hard to understand, and at first glance I saw it as a bug (missing ==) ... I know it's valid and works as expected but it's hard to comprehend/understand/read IMHO


Still got a typo here: `malformedStream`.


Последнее изменение не верное. Ссылки должны быть внутри секции, а не на уровне с ней.


thank you for bringing this up. I will pr this fix.


`startUnit` method can be used to show current compiler phase and file on a status bar in Intellij IDEA. But names of compound phases doesn't look very pretty. Probably some simplified name should be passed. Or maybe it should be invoked for each miniphase.


Удалён закрывающий тэг хэд? Зачем?


it's three different cases you test herre and ideally they all have their own function


@mabuaisha Is it possible that we define a field_class variable for agree_to_tos field on 53 (within field definition method for that field) and here instead of checking for field.name, we just set field_class value to whatever value in field.class that is defined in 53?


This method can be used to pass information about compiler crash to IDEA, I'm not sure where it should be called.


You never know how the classes/tests grow but if they grow further you which you had separated them before... look at processing and other modules, we didn't split up there :/


Maybe fix `<stdio.h>` instead like suggested in https://github.com/freebsd/freebsd-ports/commit/002e3b69adcc5a39defe6708d0a090b93cb45d07 to avoid being minority platform.

A quick search shows more similar hacks
https://github.com/DragonFlyBSD/DeltaPorts/search?q=getchar


yum


原文没有提及。。


You refer to 'services' here but you declared it at line 110 as 'array' not as 'services'


Thanks. I'll pull it in.


...and sniffs bicycle seats


An array has a field `length`, you can use that instead of declaring the size in the method yourself.


Looks like you made a typo


I didn't leave a message before commiting


// FIXME: There has to be a way to do this without converting to an array and then back to a collection


why did you remove ws2811 (and others) from keywords?  These seem like valid keywords to me.


Updated to include the import path...  However, I'd like some more input before taking steps that could affect the build.  With current changes, the GOPATH will determine what to use. @derekcollison @krobertson


// FIXME: Move this to another class as there is a lot more we should do here


Strings aren't comparable with the `>` and `<` operators, you'll need to use  `myString.compareTo(otherString)`, the compareTo method returns an integer which you can compare with < or > [compareTo documentation](http://docs.oracle.com/javase/8/docs/api/java/lang/String.html#compareTo-java.lang.String-)


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


actually the operation is called 'move assignment'


Neat. If you PR this I will merge it.


can you just move this to be below totalInner? All var declarations should be at the top of the function.


Sure, my mistake

2015-12-29 18:16 GMT+01:00 Przemysław Biecek <notifications@github.com>:

> Instead of if's like
> if ('Local' %in% class(repository)) {
> }
>
> we can use S3 generics
> searchIn.LocalRepo()
>
> wouldn't it be better?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pbiecek/archivist/commit/e20bc014c00333a22864e2217015a9e405ea8a21#commitcomment-15192780>
> .
>


@UniversalMediaServer/developers this change actually doesn't change anything in the original code. This only add variables to be able to switch off some part of the subtitles support to be tested whether they could have any influence on the rest of the subs support. I think that was bad idea from me to make the subs support not configurable when I wrote it in the past.


I'm working on iOS first. The amount of native code is minimum. We only need the code to show the option on the menu and the one to make the URL include `hot=true` when Hot Loading is enabled. Wanna send a PR for that? :)


What conditions might cause this line to be reached? This occurs occasionally for me and appears to be related to large result sets.


Done.

Note sure how this preference could be true on startup, but it certainly
is always better to explicitly initialize!


I was in holiday. I can't believe you done this.


You can also write `for (a = 0; a < highSubscript; a++)` the compiler will (in this case) generate the same output and the `x++` case is far more common and generally better to read.


Thank you for the contribution Tomas! I just need to do a code review prior merging to ensure that duplicate lines are not desired in certain modes (like raw history).


* TODO: rewrite this as it is doing way too much and is very messy now.


it looks good, bug was fixed; tnx


There is a small typo here (and in lines 31 and 36 as well) that causes those problems with only "urgent" tasks being visible. There's `>` missing just before `</task-line>`! Fix that and everything renders correctly.


That's not what I had in mind. I thought that source files that use to reference bcrypt import would be changed to point to the local "repo" (`_workspace`). Example: plain.go, line 9. Again, I may have misunderstood.


thats why i think you are a genius...keep up..thanks for the patch..


done


Dear @Makuna,

It's your choice which keywords should be in manifest. This is your lib and manifest.

It was a my proposition. From my point of view, the keyword is the common word between the libs. WS8211 - this is specific word to the vendor of CHIP. If user searches using `WS8211` in PlatformIO Library Registry, this lib will be shown because this word is in description.

I mean, that for me these keywords like a "catalog of small categories".

Nevertheless, I can revert back these keywords.

P.S: I see that http://platformio.org/#!/lib/search?query=WS8211 doesn't show libs. This is a bug. I'm working on the search engine... Thanks.


Class UpdateGitMirror is missing a docblock.


@betterwebleon only `##.notification.info` is too broad,
the other two are actually covered by more generic rules
`##.cc-overlay.cc-cookies` -> `##.cc-cookies`
`##.sliderBox > .cookie_policy.full_width` -> `##.cookie_policy`


bug's not reproducible now; tnx


where svg???


// FIXME: Delete tar file if it exists


exit 0 # FIXME: Not sure why I need this, otherwise it fails


unused


this is awesome


Good job)) bug was fixed, tnx


I still have NO idea, why the sha512 hashes published on https://moar.so/blog/qnx-password-hash-formats.html fail.


Файл ридми пустой.


Is there any flag you can just pass to get past whatever prompt you're seeing? Echoing yes is usually pretty hacky


I do not know if the idea of making every object understand 'eval' (or
'value', both of them seem better than 'apply') has consensus or not.

But, if we were to include it, I think we could give one more twist to this
idea, because

method foo() = 6
method foo() = { 6 }

would be semantically (but not syntactically) equivalent, given that both
bodies evaluate to 6

in fact this would be the same as:

const foo = 6
const foo = { 6 }

it do not know what to do with parameters...

method foo(x) = x + 1
method foo = { x => x + 1 }
const foo = { x => x + 1 }

I am aware that this is not fully consistent 'as is', but maybe you could
help me finding the way.

On Tue, Dec 29, 2015 at 1:45 PM, javierfernandes <notifications@github.com>
wrote:

> We have no implicit return anymore.
> Pablo and Nico changed this a while back ago.
> At first I was not sure because I liked to be able to write less amount of
> code.
> Later I found that having the explicit "return" allowed us to detect a lot
> of errors with static checks.
> It will also help the type system inference, since you are explicitly
> telling that the method has a return type.
>
> The only "similar" case is this short method syntax, where you don't need
> the return but you have the '=' symbol instead.
>
> I think that this "short-version" fits all the cases where I wanted to
> avoid writting the return. So it ended up a win-win.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/uqbar-project/wollok/commit/2c36ffccdace1b2dfd7f7ae2836ac29d07637866#commitcomment-15192360>
> .
>


@turadg You have all these references, yet didn't think to look at the spec? Yes, it is valid. It's a new binding for each iteration, just like let.


Same check here?


Why not?

`ivs[dv] = HPdvs[dv] * 2` (IV is DV * 2) makes a lot more sense than `dvs[dv] = HPdvs[dv] * 2` (DV is DV * 2?)


This commit breaks compatibility with PHP version < 7.


Let me check if it's even needed with the debian_frontend env.


bug is not reproducible now; tnx


I'm not sure about buffer values in case of the last c-tor...


If you've made changes yourself, you've obviously broken something. Without errors logs, I have no way of knowing what you broke. http://docs.snipeitapp.com/getting-help.html


Cool, I will take a look how to pass those options and I will get rid of it.
Thanks.


@dtaht can we still have these in an ifdef on the actual driver? It seems this is breaking compile on my end. If not, I can dirty hack a way around it.


This commit adds my solution for session 3 exercise. I have implemented two different solutions.

Solution 1: Simple: I have wrapped pieces of code inside functions and called them in rest of the script wherever required.
Solution 2: OOP: Defined class Pizza and sub classes ChefsChoice and CustomPizza having their own properties and methods (not functions).

@ritu-tanwar @4bhi @SudhaSruthi @japneet-singh @ankur321 : Looking for some valuable feedback for improving the code quality. There are a lot of new concepts used in these solutions. I will be more than happy to answer all/any queries related to the any of the implementation.


Cool, thanks @magsout
May I remind you of: https://github.com/webcompat/webcompat.com/issues/848#issuecomment-163762249 ?


I didn't know about -ge, will use it.
Yes, this is a WIP, I am compiling on all possible compilers on my Linux VM to test it all.


@dacousalr

why does the default need to be changed here?


Avoid unused local variables such as '$tokens'.


* TODO: rewrite this as it is doing way too much and is very messy now.


// FIXME: Move the archive to a seperate command


Whats this whole class for and why are these lines commented out?


forgot psd, will fix it now.


Why commented out?


So this will capitalize the time filter if it happens to be in the list of suggestions? Very inconsistent behaviour and I don't really see how this is desirable.


:) good catch!


Clang in Linux is PITA with c++11 enabled, various conflicts with the gcc headers, which it uses, like [this](http://stackoverflow.com/questions/17775390/clang-3-3-in-c1y-mode-cannot-parse-cstdio-header).

Does someone have experience with Clang on Linux?


This needs to ve I'm gitignore


You seem to have incorrect indentation in many of your files, like these lines


This should make the gnatsd build completely independent.


fyi @dkhamsing maybe it be less fragile about this? I like [docopt](http://docopt.org/) for handling arguments. Ruby: [docopt.rb](https://github.com/docopt/docopt.rb)


I'm learning how to work with github


These lines were commented out, why did you enable them again?


Yes, it's more like I picture the thing to work. Out of curiosity, did you change it manually or is it the result of `godep save -r`?


Why all commented out?


Field declarations should always be first


Why call super constructor?


The whole function. It seems a bit hacky to me to still call the getVideoInfo() function at the crawler site. In my opinion either we only have this function, or we eliminate it completely. Otherwise it's making things unnecessary complicated, I think.


because it errors without it, forget exactly why


It would be a good idea to store x.strip(" ") somewhere and use the same value through out the block. This will remove the extra cost of calculation and improve the performance of the code.


I'm glad that you find it helpful :)


because the addVertexWithUV method no longer exists and I missed fixing these ones


This is not the right fix. The EOF return is correct since Read() in-fact returned for exact amount of bytes requested. Adding a new byte and swallowing 1byte is not necessary.

The correct style is to know that if its lastBlock and return EOF when its valid.

```
@@ -185,8 +184,18 @@ func (c Client) GetObjectPartial(bucketName, objectName string) (ReadAtCloser, O
                                // Get shortest length.
                                // NOTE: Last remaining bytes are usually smaller than
                                // req.Buffer size. Use that as the final length.
-                               length := math.Min(float64(len(req.Buffer)), float64(objectStat.Size-req.Offset))
-                               httpReader, _, err := c.getObject(bucketName, objectName, req.Offset, int64(length))
+                               bufferLength := int64(len(req.Buffer))
+                               remainingLength := objectStat.Size - req.Offset
+                               if bufferLength > remainingLength {
+                                       bufferLength = remainingLength
+                               }
+                               // Verify if its a last block where the buffer length is equal
+                               // to the remaining length of the object.
+                               isLastBlock := false
+                               if bufferLength == remainingLength {
+                                       isLastBlock = true
+                               }
+                               httpReader, _, err := c.getObject(bucketName, objectName, req.Offset, int64(bufferLength))
                                if err != nil {
                                        resCh <- readAtResponse{
                                                Error: err,
@@ -194,6 +203,12 @@ func (c Client) GetObjectPartial(bucketName, objectName string) (ReadAtCloser, O
                                        return
                                }
                                size, err := httpReader.Read(req.Buffer)
+                               // If the read is not for the last block we don't return back io.EOF.
+                               if !isLastBlock {
+                                       if err == io.EOF {
+                                               err = nil
+                                       }
+                               }
                                resCh <- readAtResponse{
                                        Size:  size,
                                        Error: err,
```


Can you incorporate this change and resubmit.


Sure, no problem. You probably already have seen it - just for the archive: I've split the commit and updated the pull request. My name is already included in the AUTHORS file.


this is not real name, alias
nobody prohibits this, right?


-ge is for greater or equal, -gt stands for greater than
have a look here: http://www.tldp.org/LDP/abs/html/comparison-ops.html


amazing work frap! Thank you. you are super!


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


Ah oups, yes..


I don't really mind this for the build process, but I'm not sure about making this the default env when you spin up the image. Let's try https://docs.docker.com/engine/reference/builder/#arg instead. From a 20 second test, it looks like that makes the env available throughout the build, but doesn't persist it.


No it doesn't.


PHP5 users need to install the random_compat package as listed in the suggested dependencies on install: https://github.com/illuminate/support/blob/master/composer.json#L38.


Also, update me when your code becomes usable as a library. Would be interested in seeing that.


Also note that we did not make this breakage on 5.1.


@sfdb Your previous settings were moved to the commented calls


Turns out, unix file names can contain almost any character, Windows filenames are much more limited...


I have asked about this new interface somewhere, but can't find it in github, so sorry to repeat.
This code looks like a first incomplete step to an interface change. As such it is not self-explanatory why it is necessary here but not in other cases. Shouldn't there be a wrapper that expresses intent like ```DefineSimpleUnit``? ```registerUnit``` is what all of them do.


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


Forgot the static settings in miniquora1/urls.py


ElseGroup


The family lines graph is fixed in commit 715277d.

Sorry for the delay in fixing this.  Let me know if you find any more similar bugs.


Hmm! So, thinking of it as a one-element constraint set? Works for me.


Oh, also, then with the ARGS comment below, just make DEBIAN_FRONTEND an ARG, and then it won't mess up people using it interactively.


I can change it back... it doesn't make a difference to me. In the moment I felt they should be renamed.


This does not work to implement `/tomap` & `/maps`. What's the good way, @prokopyl?


By convention, symbolic final constants should have all-caps identifiers.


*functions


I ended up doing it manually - although, I'm new to this.


Well, i suppose you want your real name listed here, feel free to make a pull request for it. Btw i think there is no problem for aliases


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

```


I think I want to leave it up to the user to do a class_exists if needed.

My use-case is that in the case of for instance sabre/dav, I will actually create a full initialized Service class with *all* mappings for *all* the webdav specification. For any given request though only a portion of that will be used. Those that aren't used should not be autoloaded, because autoloading is actually a fairly significant drain on speed. It's literally the #1 factor last time I profiled 'bare' sabre/dav


fesdfsaf


* TODO: Refactor the validator to reduce duplication, maybe move the askWithValidation to an external library.


ok, thx, no, like this on more


My PR's keep removing this old WriteLine and it keeps getting re-inserted into develop again every time you pull someone's PR based on an older develop.


is this gonna add this text to this file?


Yes Matti,
Two digit strings are always permitted as variables, so no point in checking if they are also units.  I should have added this first time around...


Also fixes chaining after a call to Hook.call.


here look like can be division by zero. In float it probably not crash but return infinity


MDB!


love


<!-- PMABOT:SOB -->
This commit is missing Signed-Off-By line to indicate that you agree with phpMyAdmin Developer's Certificate of Origin. Please check [contributing documentation](https://github.com/phpmyadmin/phpmyadmin/blob/master/CONTRIBUTING.md) for more information.


Previously it was `'%s-%s' % (quality, protocol)`.


I was wrong about the cache column of roles column. It is still used in one place which determines which menu items to be displayed based on the role of current user. So I reverted this commit.


@mitsuhiko Awesome thanks!


@Khalimov apologies, I forgot to mention that the filename also needs to match the ID:

i.e. `c-sharp-snippets.json`


This will break the whole extraction if URL turns out not to contain `mp3:`. Moreover this is invalid since top level container is always flv for rtmp.


But it is not semantically equals

```xtend
method foo() = 6
method foo() = { 6 }
```
They both share the idea that return objects which understand "evaluate()". But this won't be transparent for the client code.

Here I want to use the returned number value. It will work for the first solution but not the second.
```xtend
    2 + a.foo()
```
Unless it forces me to do:
```xtend
    2 + a.foo().evaluate()
```

Or unless we have a full typesystem with implicit automatic conversions. Which seems quite far away right now.

Then I guess that
```xtend
method foo = { x => x + 1 }
const foo = { x => x + 1 }
```

Takes us to the path to "slot" OOP like self / javascript.
I'm never seen a statically typed lang based on slots.
I would need to think what this involves.
Usually it leads to a simpler language, but then for more "advanced" developers.


Not sure about this particular change. Why is that necessary?


This changes the hash values, doesn't it? It's fine with me, it's still a correct/good hash, and you save on half of all the calls to `hash_combine`.


I didn't find the spec to be very clear on this topic, and I hoped including these examples would be of use to someone else trying to make sense of problems they encountered.

fwiw, here is the test case:
https://github.com/tc39/test262/blob/1c1a75eead413ab5152e825d3c551908667e405f/test/language/statements/for-in/const-fresh-binding-per-iteration-for-in.js#L10


Yes, it changes the hash values. Saving is due to both reduced calls to `hash_combine` and ordering the dict before hashing


> This will break the whole extraction if URL turns out not to contain mp3:.

i will make it non fatal.
> Moreover this is invalid since top level container is always flv for rtmp.

does this mean that the extension will be flv even if it is an audio file?


I added a **rb_gc()** call to shoes_svg_set_handle() method, to force a garbage collection,
otherwise Shoes memory consumption jumps from 27/35 Mo to 360 Mo in no time ( stabilizing around 360 Mo)
It's very efficient : stabilize around 35 Mo, but might be costly (not noticeable in this demo)


Cool :)


@Xephi I understand that some kinks I mentioned (like the potential for an enum) can be worked out, but I'm trying to find out why using a builder to write `SELECT * FROM table WHERE col = ?` has advantages. So for example, for the diffs I posted in my second comment here, I'd be curious to know how the new code is better than the old one.


What is '1'? Please replace with a define or calculation,  or add a comment to explain. We should avoid magic numbers


может быть переименовать drvier в driver ?


use cors yo


Can you close the PR and push the change you have? it will be easier


Signed-off-by: Binyamin Laukstein <7raivis@inbox.lv>


this should rather go into you global gitignore I think, seems to be from your IDE?


Since I am generating the mapping-code I dont need the fast fail behaviour.


On Tue, Dec 29, 2015 at 3:40 PM, javierfernandes <notifications@github.com>
wrote:

> But it is not semantically equals
>
> method foo() = 6
> method foo() = { 6 }
>
>
> I mean that both evaluate the right hand side before returning, so both
methods return 6.


тРЕБУЕТСЯ УСТАНОВКА ВИНЧЕСТЕРА ПЛАТФОРМА ЛЮБАЯ АНДРОИД ТОЛЬКО


Yes, for rtmp `ext` should always be `flv` since it's media container that actually contains mp3, mp4 or whetever it supports to contain. You can put this metadata (`mp3`) into `acodec` or `vcodec` if you want but `ext` must be `flv`.


#Ruby


They had a reason. By design all resources on the node belong to mesos, but unreserved resources are marked as "oversubscribed" and can be revoked for non-priority tasks. This feature is intended to minimize downtime. CHECK_EQ verifies that all these oversubscribed resources are revokable. But I am "piggybacking" on this api to update the total system resources. A cleaner way would of cause be to add a new type of resources and all the update/sync hooks for it.


WOOHOO I think I did it right!


This will not work unless entire data is read. your fix on https://github.com/minio/minio-go/pull/280 or my fix both will work.


This can break external connectivity for v4 if a driver sets only the v6 gateway. I think the right approach would be to treat v4 and v6 separately for external connectivity.


TeamCity LabKey :: Trunk :: Premium :: Git sqlserver 2014 [Build 143](https://teamcity.labkey.org/viewLog.html?buildId=293951&buildTypeId=LabkeyTrunk_GitSqlserver2014) outcome was **SUCCESS**
Summary: Running Build time: 00:02:18


My eye just fell on a possible typo:
num_responces_we_need
should be
num_responses_we_need


There is a typo here, missing the $ in front of sendgrid.  Should read
$res = $sendgrid->send($email);


#lol.....really?


for example, if size of object is 10 bytes and we do:
length of p is `1`
`n, err := ReadAt(p, 9)`
this will not return EOF but it should.


So the test is being affected by other Describe() blocks?


You can easily check whether the file is flv or not by checking the first 3 bytes of the file that should be equal to flv signature `FLV` in case of flv.


i will just remove this part.


i'm new to all of this , and i hope can get pretty good at this in the future.


Ok, thanks.


I don't think so


and by the way this fixes a few other issues with disappearing highlights in menus too. great stuff :-)


and thanks the informations.


Maybe you can mop up https://github.com/opnsense/core/issues/390 while at it? The JS is wonky.


Do you need that specific version of the kernel, or would `linux-generic` work?  We could merge this, and instead I could just switch to `linux-virtual` in the post phase if the host looks virtual (`virt-what`).


Yes, this change (only the setprop is crucial) and the previous storage change fix eMMC internal SD on M. You need to remove /data/system/storage.xml from your device and reboot for the changes to take effect.


you are right, ignore this comment, I thought king also may contribute to the SEE table


@martinbigio Sure. Will take look at the iOS code to see what I need to do. This is really exciting :)


Oh, wasn't sure if the question was regarding the script reloading/locking. I think this 'pattern' is used throughout the codebase (though I could be wrong at this point). Basically, log the error and optionally include the stacktrace if debug level logging is enabled. I recall a conversation on the dev mailing list awhile back [1] and this resulting ticket [2]. Not sure what the current state is.

[1] http://mail-archives.apache.org/mod_mbox/nifi-dev/201505.mbox/%3Cgit-pr-59-incubator-nifi%40git.apache.org%3E
[2] https://issues.apache.org/jira/browse/NIFI-636


It's being affected by a BeforeEach that occurs at the same level as `Describe("GetExpiredArchives")`


yes, I saw that :) the old stuff is gone :) .. some additional testing to do and then we can close 390 I guess.


We should fix that, but probably not in this PR.


@Thunderbottom
The code is perfect and clean!
See https://www.codacy.com/app/00simy/Cornowser/dashboard


I was almost hoping this line alone would fix it, but there is a much deeper issue that sync is called before publish all by the constraint system generating code...


yes.
It is rare that underpromotions (promotions lower than queen) is the best move, that's why in a move ordering it should go to the end.


No, i just tested this on my machine, it turned out a negative patch, but i will try something else instead.


Really looking forward to this, keep up the good work!


What if I wanted to add bonus for same coloured square enemy pawns which are not supported by adjacent pawns. How about this idea?


Thanks for the tip


:beers:


it should be like this?
```
python2 __main__.py -F 'http://www.srf.ch/play/radio/echo-der-zeit/audio/who-erklaert-guinea-fuer-ebolafrei?id=b5c368b6-4947-4965-b785-3d40495ba6ec'
[SRGSSR] b5c368b6-4947-4965-b785-3d40495ba6ec: Downloading JSON metadata
[SRGSSR] b5c368b6-4947-4965-b785-3d40495ba6ec: Downloading f4m manifest
[SRGSSR] b5c368b6-4947-4965-b785-3d40495ba6ec: Downloading m3u8 information
[info] Available formats for b5c368b6-4947-4965-b785-3d40495ba6ec:
format code       extension  resolution note
HTTP-HLS-MQ-meta  mp4        multiple   Quality selection URL
HTTP-LQ           mp3        unknown
HTTP-HDS-MQ-30    flv        unknown      30k
HTTP-HLS-MQ-30    mp4        unknown      30k , mp4a
HTTP-HDS-MQ-62    flv        unknown      62k
HTTP-HLS-MQ-62    mp4        unknown      62k , mp4a
HTTP-MQ           mp3        unknown
RTMP-HQ           flv        unknown    (best)
```


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


Thank you!


merging file


jhjh


@jgcaaprom Thanks man, This isnt fully ready for use yet, but if you want to test it, make sure you have the commits in the prebuilts_ndk and frameworks_rs repos on this org. 3.7 is the current stable, but make sure you sync it to prebuilts/clang/linux-x86/host/3.7.1 and change the version in build/core/clang/config.mk to 3.7.1


this will not work unless all data has been read. For example if we do:
(for object sized 10, p[] sized 1)
`n, err := obj.ReadAt(p, 9, 1)`
this will not return EOF but it should.

Your fix on https://github.com/minio/minio-go/pull/280 or my fix both will work fine.


Yep, it makes the code more clear as a backtick (\`) can easily be misinterpreted as apostrophe (')


If you want to reference your dependencies from within the `_workspace` directory, then your dependencies also have to import their dependencies that way as well. So this was needed.


This is fixed in PR #974


This one I am most pleased about and it would've saved me lots of (trial and) errors :wink:


Related: https://emacs.stackexchange.com/questions/19149/how-do-i-set-the-upstream-branch-to-push-a-local-branch-to-in-magit


I don't think toList() is a better name, it may be confused with an instance method.  `fromArray()` seems more promising, although still not great.

In java they seem to have gone in the direction of putting these static constructors in their own class, for example `Arrays.asList()`.  I wonder if the `list` method is just in the wrong place.


Sure. What I meant is that gnatsd source files were not modified to point to `_workspace` in that commit. Later, Colin fixed that.


ya..


I think I followed these all. I think it should work. I'll test ;).


@wfenchel : good try. However, I see two weak points of aspiration loop: dependence on rootDepth, and dependence on previousScore. And trust me, more complicated the code is, easier will be to improve it.
So, no need to rush with simplification.


> You can easily check whether the file is flv or not by checking the first 3 bytes of the file that should be equal to flv signature FLV in case of flv.

that's right. i tested this with hexdump. even if it's an mp3 file the first bytes looks like this
```
00000000  46 4c 56 01 04 00 00 00  09 00 00 00 00 08 00 03  |FLV.............|
```


TeamCity node build testing :: PR submitted #3 [Build 1](http://localhost:8111/viewLog.html?buildId=60&buildTypeId=NodeBuildTesting_PrSubmitted3) is now running


TeamCity node build testing :: PR submitted #3 [Build 1](http://localhost:8111/viewLog.html?buildId=60&buildTypeId=NodeBuildTesting_PrSubmitted3) outcome was **SUCCESS**
Summary: Running Build time: 00:00:01


Please undo this change.


Oups, désolé, j'ai oublié !

;)


Why remove it?


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

```


Also, this test is pretty low value, since it depends on you having pasted the same set of hard-coded dates twice. <3


@krobertson - would you mind testing with this branch to ensure it fixes your problem?  I suspect it should, but testing is always good.


Humm...There is already a Threat bonus for attacks on weak pawns (== not pawn protected)

In the middlegame, if the pawn cannot be attacked, it is not a big deal.
But maybe this could work as an endgame bonus S(0, something)


If you call 'make clean' in the build command it will check the whole project again. A simple 'make' should be enough in incremental build. For the first check it is useful to clean the project before the build.


Сломается singletree так как он ожидает одну строку.


I have no clue what this is about :-)


@maxteufel Do we really need a debug message for this?


Hi LunkSnee  :)

I worked a lot today in the CSS part,  I added a code highlighter and a checkbox for printing wp_head optionally.

will be a good if you can test it before I upload it to wp.org repo, thanks.

Best.


Nope, it doesn't work this way anymore.


perfect :)


So, it seems that the missing window at the first start of the App was due to the missing calibraces to your if statement that makes  setVisible(true); to be not executed as you have already set the property false by default at OptionsDialog class. If you want to reproduce the problem don't forget to delete the created from the App upm.properties fille.
Also, it's better to treat the restore  as a Boolean object as that is the case and you have already define it as Boolean object at OptionsDialog class  on "Store Window position" row. Above is my suggestion:

Boolean restore = new Boolean(Preferences.get(Preferences.ApplicationOptions.REMEMBER_WINDOW_POSITION));
        if (restore.booleanValue())  {
           restoreWindowBounds();
     }
        setVisible(true);


(The branch and revision get passed in via an explicit define in build.h)


Improved performance by caching XName objects.

The following table lists the time it took to construct the Navigator (10,000,000 iterations) using each method:

XDocument.Parse | XDocument | XDocument (caching XNames)
---------------------- | --------------- | ---------------------------------------
2m 54s | 0m 23s | 0m 17s

The benchmarks were run in LINQPad with the following code:

##### XDocument.Parse
```csharp
var isStart = true;
var displayName = "activity";

for (int i = 0; i < 10000000; i++)
{
    var xml = XDocument.Parse(string.Format(@"
         <TraceRecord xmlns='http://schemas.microsoft.com/2004/10/E2ETraceEvent/TraceRecord' Severity='{0}'>
             <TraceIdentifier>http://msdn.microsoft.com/en-US/library/System.ServiceModel.Diagnostics.ActivityBoundary.aspx</TraceIdentifier>
             <Description>Activity boundary.</Description>
             <AppDomain>client.vshost.exe</AppDomain>
             <ExtendedData xmlns='http://schemas.microsoft.com/2006/08/ServiceModel/DictionaryTraceRecord'>
                 <ActivityName>{1}</ActivityName>
                 <ActivityType>ActivityTracing</ActivityType>
             </ExtendedData>
         </TraceRecord>", isStart ? "Start" : "Stop", displayName)).CreateNavigator();
}
```

##### XDocument
```csharp
var isStart = true;
var displayName = "activity";

var TraceRecordNamespace = XNamespace.Get("http://schemas.microsoft.com/2004/10/E2ETraceEvent/TraceRecord");
var DictionaryTraceRecordNamespace = XNamespace.Get("http://schemas.microsoft.com/2006/08/ServiceModel/DictionaryTraceRecord");

for (int i = 0; i < 10000000; i++)
{
    var xml = (new XDocument(
        new XElement(TraceRecordNamespace + "TraceRecord",
            new XAttribute("xmlns", TraceRecordNamespace),
            new XAttribute("Severity", isStart ? "Start" : "Stop"),
            new XElement(TraceRecordNamespace + "TraceIdentifier", "http://msdn.microsoft.com/en-US/library/System.ServiceModel.Diagnostics.ActivityBoundary.aspx"),
            new XElement(TraceRecordNamespace + "Description", "Activity boundary."),
            new XElement(TraceRecordNamespace + "AppDomain", "client.vshost.exe"),
            new XElement(DictionaryTraceRecordNamespace + "ExtendedData",
                new XElement(DictionaryTraceRecordNamespace + "ActivityName", displayName),
                new XElement(DictionaryTraceRecordNamespace + "ActivityType", "ActivityTracing")
            )
        )
    )).CreateNavigator();
}
```

##### XDocument (caching XNames)
```csharp
var isStart = true;
var displayName = "activity";

XNamespace TraceRecordNamespace = XNamespace.Get("http://schemas.microsoft.com/2004/10/E2ETraceEvent/TraceRecord");
XNamespace DictionaryTraceRecordNamespace = XNamespace.Get("http://schemas.microsoft.com/2006/08/ServiceModel/DictionaryTraceRecord");
var TraceRecord = TraceRecordNamespace + "TraceRecord";
var TraceIdentifier = TraceRecordNamespace + "TraceIdentifier";
var Description = TraceRecordNamespace + "Description";
var AppDomain = TraceRecordNamespace + "AppDomain";
var ExtendedData = DictionaryTraceRecordNamespace + "ExtendedData";
var ActivityName = DictionaryTraceRecordNamespace + "ActivityName";
var ActivityType = DictionaryTraceRecordNamespace + "ActivityType";

for (int i = 0; i < 10000000; i++)
{
    var xml = (new XDocument(
        new XElement(TraceRecord,
            new XAttribute("xmlns", TraceRecordNamespace),
            new XAttribute("Severity", isStart ? "Start" : "Stop"),
            new XElement(TraceIdentifier, "http://msdn.microsoft.com/en-US/library/System.ServiceModel.Diagnostics.ActivityBoundary.aspx"),
            new XElement(Description, "Activity boundary."),
            new XElement(AppDomain, "client.vshost.exe"),
            new XElement(ExtendedData,
                new XElement(ActivityName, displayName),
                new XElement(ActivityType, "ActivityTracing")
            )
        )
    )).CreateNavigator();
}
```


Yeah, kind of unsure how to test this thing. I guess I could just load up the data from the YAML file but then it's kind of just testing itself.


Not worth holding us this PR, but it does seem a little repetitive to now have three `is_$some_status` helper methods with identical implementations. We could probably generate them programmatically and use `Sub::Install` or something similar.

(Untested example)
````perl
my $statuses = Genome::Disk::Allocation->__meta__->property(name => 'status')->valid_values
for my $status (@$statuses) {
    Sub::Install::reinstall_sub(
        into => __PACKAGE__,
        as => "is_$status",
        code => sub { return shift->status eq $status; }
    });
}
````


It means the rake task doesn't need to duplicate the list of ignored files


Is there a reason to keep the space at the end of the line?


Stop doing this! `perspective = (tdims[1] == 3)` should be enough.


Do we need to check to make sure the export directory is not somewhere within our allocation system? Maybe not actually?


Oh okay, missed that somehow. Perfect!


Looks good


just trying to understand the changes; no need to modify this yet.

Also, the platform changed from * to Arduino.  This platform is meant to say its a Arduino library then.


OK, I see it now by checking the TC wiki regarding `conditions` and `CONDITION_SOURCE_TYPE_SPELL = 17` :

"conditions with the same value ElseGroup will be used to make logical AND check, so to allow different targets for the same spell effect you have to set ElseGroup respectively."

So with 4 separate `ElseGroup`, this allows for 4 different players to cast their spell on the same target?


**- Added a Stress project to stress test history.**


Is it documented anywhere? Shame on them for fucking with the structure
without bumping the version if that's the case

On Tue, Dec 29, 2015, 11:15 AM Ensiform <notifications@github.com> wrote:

> FYI that BSP structure only applies to Q3. There is differences for ET,
> RTCW, JA, QL etc.
>
> ET and QL share version 47 but there are additional changes in each I
> believe.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jswigart/omni-bot/commit/7d570f6a6414a1c2a79d5c67cc59f0bc6e63c4d1#commitcomment-15192766>
> .
>


TeamCity LabKey :: Trunk :: Premium :: Git postgres 9.4 [Build 883](https://teamcity.labkey.org/viewLog.html?buildId=293953&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 15 (1 new), passed: 248 Build time: 00:35:01
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.RedCapTest: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testDatasetExport: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testSelection: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testSelectingEmptyMeasure: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testStickyFinderFilterOnDataset: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testStickyFinderFilterOnStudyNavigator: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testGroupSaveAndLoad: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testStudyParticipantCounts: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testCounts: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testSearch: <no details avaliable>

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

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


Coverity will complain since it wants the return value checked.


this is a branch with simplified overlays for translations/subdomains. Only the fields that include strings that require translations are included.

The suggestion is to modify the code to steam the master 'en' overlays for all translations but override the fields present in the subdomain overlays. Or the other way around, source the fields omitted from the subdomain overlay from the master 'en', whichever of these options seems best.

... this way, additional 'technical' or 'resource-type' fields can be added without requiring to spread to subdomains, or at least minimise the efforts needed to increment the overlays.


Hm, okay. I just found it a bit odd is all. Thanks for looking into it. :)


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


thanks for digging into this didn't mean to cause this to block. :+1:


does this break chef 11 compatibility by not using the suggest:
```
provides :splunk_app if respond_to?(:provides)
```


So, there's probably a valid question here of whether it makes sense to run tests on the specific dates that are holidays, versus the operation of the classes themselves. But, I'll assume that acceptance tests are valuable in this context for a moment. If I really wanted a valuable test, it depends on the holiday and how its dates are calculated. Overall, though, I'd spot check the holidays I was interested in, plus some weekends.

For set dates, picking this year's April 1 and next year's and making sure that they pass as holidays would be a start. That would also catch the situation where you forget to add April 1 2018 some time in the future. Ditto for Thanksgiving: calculate this year and next's third-Thursday and check those dates.

Weekends are easy enough (loop a few weeks, bump up day number til you get a Saturday. Rinse + repeat), but I'm not sure what type of error or issue they'd actually catch, though. The value might be in the `it "considers every Saturday and Sunday as a weekend"` block as documentation. Right now there's no sense for what the hell these dates are.

If there are holidays that are one-off determined by a national gov't, you're probably fucked.


Conversation for posterity: https://groups.google.com/d/msg/elixir-ecto/B7IbXKsVJx4/Nl50GiuMCwAJ.


OK, I can update that but the upstream project "make" has to be efficient


This was helpful https://github.com/defunctzombie/package-browser-field-spec


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


@pb00068 :
Hi Günther,
work hard to overcome a dissapointment. And be aware that you are not alone. There are more than 7 billon individuals dissapointed from time to time. And everyone that passed through that will tell you the same thing:

The harder it was to overcome a dissapoitment, the more shiny was a life after that.

I think that our maintainers are doing a great job. They are simply not gods.


ignore my comment, just saw: https://github.com/chef-cookbooks/chef-splunk/pull/65/files


This is still incorrect because upcase case first letters mean that this method isn't actually reurning camelCase.


Can you put this condition all on one line.


TeamCity node build testing :: PR submitted #3 [Build 2](http://localhost:8111/viewLog.html?buildId=61&buildTypeId=NodeBuildTesting_PrSubmitted3) is now running


TeamCity node build testing :: PR submitted #3 [Build 2](http://localhost:8111/viewLog.html?buildId=61&buildTypeId=NodeBuildTesting_PrSubmitted3) outcome was **SUCCESS**
Summary: Running Build time: 00:00:01


Heh, sorry, didn't mean to patronize - was in grammar nazi mode.

Thanks again for updating these.

-[Unknown]


`runbenchmarks("arrays", vs = "JuliaLang/julia:master")`


Is this intended to be empty? If so, could you make it like such:
```
BUILD_DEPENDS =  # empty
```


CamelCase or camelCase are both valid camelCase variations.


This should be moved to common and merged with the stuff there. I see some of these look like they might alredy exist under https://github.com/dotnet/corefx/blob/c02d33b18398199f6acc17d375dab154e9a1df66/src/Common/src/Interop/Windows/secur32/Interop.Winnt.cs


The ET Version is 47.  They accidentally used the ET BSP number when making QuakeLive and it was too late to change but the only difference from Q3 to QL is the added adverts lump I believe.  You'd have to check the ET engine source for the ET and RTCW lump differences from Q3.

JKO and JKA use 'RBSP' version 1 with very different structures.


The uppercase first is PascalCase.


Nahhhh, undo. It makes it easier to find out which line mappings have changed by having it show up in git. Otherwise you have to search for it.


This appears to be a copy of `py-elasticsearch/pkg/DESCR` ?


Ok, we need to separate "evaluate":
 * interpreter evaluation
 * wollok code evaluation

The current interpreter doesn't evaluate a closure in that case.
It will just return the closure.

Actually right now { 6 } is not even a closure, is a block statement.

The interpreter DOES evaluate a block statement, but not a closure
(currently  [ .. ] )

Maybe I'm mixing up this with the closure syntax change :S

On Tue, Dec 29, 2015 at 3:53 PM, Nico Passerini <notifications@github.com>
wrote:

> On Tue, Dec 29, 2015 at 3:40 PM, javierfernandes <notifications@github.com
> >
> wrote:
>
> > But it is not semantically equals
> >
> > method foo() = 6
> > method foo() = { 6 }
> >
> >
> > I mean that both evaluate the right hand side before returning, so both
> methods return 6.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/uqbar-project/wollok/commit/2c36ffccdace1b2dfd7f7ae2836ac29d07637866#commitcomment-15194191>
> .
>


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


Please use CommonPath property and add Link metadata, see https://github.com/dotnet/corefx/blob/master/src/System.IO.FileSystem/src/System.IO.FileSystem.csproj#L84 as an example.


Nit: Should be `assertEqual`, I think. This uses the `is` operator but you only need the `==` operator here.


not a problem was worth digging in and understanding it better


So, yeah these are holidays determined by the government. They make the list for the next year in December. So some of the holidays for 2016 that were put in place last year by Przemek are actually now not holidays. But I guess I could just find some holidays which are constant. Seems like January 1st is a holiday every year. April 1st seems like a bad idea. Seems that there's a holiday dancing around the start of April 5 some years and this year it's on April  3. Not sure if it's in a calendar system that would have it fall on the 1st.


conola oil = canola oil


This doesn't work does it? What happens if the content has quotes? Wont the values come out double quoted?


Maybe use `$cwd` instead of `__DIR__` like the others for now? (not sure why we didn't just use `__DIR__`)


These probably should be constants. However @GiriSr should make that call as part of https://github.com/dotnet/corefx/issues/4981


Sorry, the 1st was actually a reference to April Fools as a holiday that has a calendar date that is specified. Versus Thanksgiving which is algorithmic, and more valuable to test. Versus arbitrary, where, yeah, you're probably screwed.

I wonder if there's any useful parameter you could use there (e.g., "Holiday X is always a Wednesday in May, so verify that at least one Wednesday in May is specified as a holiday"), but again we're starting to get into weird and low value tests.


No problem :)


## Description of changes:

* `NoOpPortlet` is an empty implementation of the Portlet interface used for two things:
 1. To mark the new Rest Portlets so the proper href is created when nav menu is rendered (dotAjaxNav.show instead of the big old portlet url)
 2. For the `portlet.xml` Portlet definition not to complain (we still need to include the new Rest Portlets here to be able to list them on the UI, which would be nice to change in the future)

 note: I could have left the BaseRestPortlet to implement Portlet with empty methods but wanted to leave clear that no real portlet implementation is needed, just a dummy for the portlet definition not to complain, that is why using the NoOpPortlet makes it clear)

* `BaseRestPortlet` was renamed to `RestPortlet`
* `RestPortlet` (formerly `BaseRestPortlet`) does not need to implement `Portlet` interface.

* `RulesEnginePortlet` was deleted because is not needed, never was.

* `ESContentResourcePortlet` was renamed to `ESContentResource` and does not need to extend from      `RestPortlet` (formerly `BaseRestPortlet`)

* `PersonasResourcePortlet` was renamed to `PersonasResource` and does not need to extend from      `RestPortlet` (formerly `BaseRestPortlet`)

* `JSPPortlet` which is the one handling the rendering of Rest Portlets now has-a `RestPortlet` instead of extending from it (composition over inheritance)


can we have a more specific keyname here? Per https://github.com/yahoo/coname/blob/master/proto/client.proto#L173-L174, keys are intended to be application specific, so maybe "mail"? and we can get this value from the config file?


Can you please upload a new bin with the last changes? :)
Thanks


why do you need html and json render for create?


I'll leave feedback soon . I'll be first testing clang 3.6 at start


可能后退按钮需要调整成在路由有变化的时候显示，然后点击直接调用浏览器的后退这样


This should not have been an issue


It's still a camelizing function, isn't it? The parameter allows us to adjust the variation we want. Anyway, I have a commit ready if you prefer to drop that option and make it pure PHP.


Can you email me the the image? i'll take a look because i create a image of 18 x 476 and it was working fine. I'll debug it with your image.


>= instead of > here?


Furnaces are not chests. Are you drunk?


This is cool.


Remove the `if` and `pass` and just call `self.listbox.keypress(p_size, p_key)` instead.


> Obviously it will be fold, what's the question?

I wasn't sure how this will get rendered. I've updated the plugin and it works ok for now (aside from looking funny when constructing a few if / else if cases which looks like this: `if true {...} else if true {...} else if true {...} .....`

> I didn't reimplement block folding algorithm, it works for if-blocks in the same way as for other blocks

Other blocks don't fold unless they have at least two lines of code in them which is disables folding for cases like this:
```
if true {
    println("")
}
```

would it make sense to display the line of text like this `{println("")}` instead of `{...}` in the folded view (somewhat like the one for Java does)?


Or is this to satisfy branch coverage?


Love it.


The deck loading performance is very slow on a smaller machine like the  raspberry pi2. Try changing the deck button. The open dialog is on screen until all 53 handles are created.  In practice, a game developer would load them in another thread  and do something visual on screen to distract the user.

However the sample does showcase something and gets people thinking about how to use Svg so it's good for learning and that's the purpose of samples.


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


`else` and `pass` can be removed


Sure! Note, however, that this will affect the raw history view (by
design, I am using it). I think it's more sensible behavior than display
the duplicates, however, that's why I introduced the unique-off option.


In single player, creative players don't die from the void :3


@brett19 I wrote this. I could explain in more detail if you like. Any suggestions?


Look's good.  I just missed it on my first pass and wanted to make sure the differentiation was there.  Reading MD diff's is a pain sometimes :)


@raylim Maybe you can check if this is ok. I'm not really sure what these plots are, but it seems to fix #77


Nit: perhaps `string_to_io_error`?


In feed.php verschoben.


Any reason to drop diffutils from MSys packages ?
Also, no gettext, glew,  gmp, and no headers-git anymore ?
No libiconv, libpng and no winpthreads-git.....?
I'm surprised that so many packages are in fact unneeded, but removing libiconv  libpng and headers-git worries me...
Anyway, I doubt that we can remove all the unneeded packages because of package X depending package Y.


Fixed


Same problem here with OpenSuse and the line missing from the init file! Thanks!


You should add a `$` at the end
`"pattern" : "^eps_sale$"`
othewise not only `"transaction_type":"eps_sale"` but e.g. `"transaction_type":"eps_sale_foo"` would be valid too.


I tested it, of course. Not with every combination of present and missing external libs, just with defaults (all enabled) and mine.


this is a valid improvement and I'm going to bump to the latest RC, so make a PR so I don't have to copy and paste your code ;-)


@aeligature Can we remove live-yoursite.pantheon.io here? So it'd just be `192.123.456.789 yoursite.com` ?


TeamCity LabKey :: LabkeyRelease15.3 :: CDS postgres 9.4 [Build 112](https://teamcity.labkey.org/viewLog.html?buildId=293949&buildTypeId=LabkeyRelease153_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 12 (5 new), passed: 36 Build time: 01:30:20
### Failed tests
```
org.labkey.test.tests.CDSTest.verifyGrid: <no details avaliable>

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

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

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

org.labkey.test.tests.CDSSecurityTest.verifyFirstTimeUse: <no details avaliable>

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

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

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

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

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

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

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


This is wrong, the case where x = 5 should not return a row. The sample should be an error, thus no value and therefore no row ?

I think?


Might be better to just avoid the extra function, and slip that code straight into the code.


OK seems fair


One extra remark. I just noticed you called `.create` in the `before` block, only to call `.save` in the spec itself. Because `.create` actually runs `.new` (creates a new `Vote` Ruby object in memory) followed by `.save` (persists the data hold by the Ruby object in the database, on the hard-drive), you end up calling `.save` twice for each vote. That's quite a load for simply asserting the validity of votes.

In order for the spec to run faster, you could use `.new` instead, and in the spec itself, use `be_valid` as per the previous comment. That way, you would actually never hit the database, but it's quite fine in that case for you're only concerned about what would happen prior to saving votes into the database (validation of data).


Someone fix this Chinglish mother load...


this doesn't look like code that would disable an extension to me. It seems to be setting a field value (and wonder if this would crash the upgrade if the extension was not present?)

I think the way to disable an extension is with the api.

    civicrm_api('Extension', 'disable', array('key' => "com.klangsoft.flexiblejobs"));

This will just return an error code if the extension doesn't exist, but shouldn't crash the upgrade. The api can also be used to uninstall extensions as well as disabling, not sure which is better.


Nice description :+1:


## "makeCasheMatrix" is a function that creastes a special matrix object
## that cache its invers.

makeCacheMatrix <- function(x = matrix()) {
	m <- NULL
	set <- function(y) {
		x <<- y
		m <<- NULL
	}
	get <- function() x
	cacheSolve <- function(solve) m <<- solve
	getSolve <- function() m
	list(set = set, get = get,
		cacheSolve = casheSolve, getSolve = getSolve)
}

## "cacheSolve" is a function that computes th inverse of the special
## matrix returned by function "makeCasheMatrix". If the inverse has
## already been calculated and the matrix has not changed, then
## "cacheSolve" should retrieve the inverse from the cache.

cacheSolve <- function(x, ...) {
	m <-  x$getSolve()
	if(!is.null(m)){
		message("getting cached inverse of matrix")
		return(m)
	}
	data <- x$get()
	m <- solve(data,...)
	x$cacheSolve(m)
    m
}


Need to add the line

    $this->addTask(ts('Upgrade DB to %1: SQL', array(1 => $rev)), 'runSql', $rev);


Sweet! It looks awesome! And I agree, they should have. =)


ncmlib doesn't build without this patch. To reproduce:

```bash
git clone https://github.com/niklata/ncmlib.git
cd ncmlib
mkdir build
cd build
cmake ../
make
```

Fails with:

```txt
In file included from /tmp/build/ncmlib/exec.c:40:0:
/tmp/build/ncmlib/nk/xstrdup.h:5:20: fatal error: nk/log.h: No such file or directory
compilation terminated.
CMakeFiles/ncmlib.dir/build.make:254: recipe for target 'CMakeFiles/ncmlib.dir/exec.o' failed
make[2]: *** [CMakeFiles/ncmlib.dir/exec.o] Error 1
CMakeFiles/Makefile2:67: recipe for target 'CMakeFiles/ncmlib.dir/all' failed
make[1]: *** [CMakeFiles/ncmlib.dir/all] Error 2
Makefile:83: recipe for target 'all' failed
make: *** [all] Error 2
```


TeamCity pull requests :: netty [Build 914](https://garage.netty.io/teamcity/viewLog.html?buildId=4325&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Tests passed: 6355, ignored: 21; resource leak detected (new) Build time: 01:20:21


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


It seems redundant, but it isn't. It's `ELEID_` followed by the elemental's SpriteName. The second `EL_` is part of the spritename, which we don't have control over, and I don't want to use names we don't have control over, unprefixed (because they might add one that doesn't begin with `EL_`)


Well, I guess all my comments from the previous assignment were unnecessary after all ;)


Considering the last comments in the issue, I guess so.


But you mean that the version of NVENC that ffmpeg has is the "fast" one that allows waiting and not insta-copy?
Would be interesting how QSV fairs, i think i recall there being an ffmpeg with that, will probably try it out if i can get that working.


Excellent. Did you have a hard time figuring out the solution, or was it plain straightforward?


PostDAO에서는 아직 JdbcTemplate 적용이 덜 됐군요!
RuntimeException으로 예외전환이 된다면 [PlaceController의 28번째 줄](https://github.com/NHNNEXT/2015-01-WEB-TOITOITOI/commit/6027f1249694a5182848769be24af8ea5dfb1c57#diff-319313686c8409102d66a56f1e708a42R28)은 [이 전의 `PostDAO.getDearId()`와 같은 문제](https://github.com/NHNNEXT/2015-01-WEB-TOITOITOI/commit/1c39faa2d4655aec6b2cb4d08073296b9f689681#commitcomment-15080235)가 생길것같네요.


@trueadm This was done to avoid an extra if / else check if there is no valid 'object'


@hbstone Can you replace this string / number  in the whole code with `isStringOrNumber()`?


Hello, there!


I don't understand what you're simplifying. What you are doing it:
* breaking the UI update code
* replacing delta += delta/4 + 5, by delta += delta. Already tested, and we know it doesn't work.


TeamCity pull requests :: netty [Build 916](https://garage.netty.io/teamcity/viewLog.html?buildId=4327&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Tests passed: 6355, ignored: 21; resource leak detected (new) Build time: 00:42:31


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


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


TeamCity LabKey :: LabkeyModules15.3 :: Git postgres 9.4 [Build 80](https://teamcity.labkey.org/viewLog.html?buildId=293776&buildTypeId=LabkeyModules153_GitPostgres) outcome was **FAILURE**
Summary: Execution timeout (new); tests failed: 155 (141 new), passed: 31 Build time: 04:01:11
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyColorStudyAndTreatmentVars: <no details avaliable>

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

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

org.labkey.test.tests.DataFinderTest.testDatasetExport: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testImmuneSpaceStudySubset: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testStudySummaryWindow: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testStudyCards: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testSelection: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testNavigationDoesNotRemoveFinderFilter: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testSelectingEmptyMeasure: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testBackDoesNotRemoveFinderFilter: <no details avaliable>

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


## ======
Trình bày lại .
https://picasaweb.google.com/lh/photo/kalK-OYF9hjn4JiOG3NBj26XXLEmDmap0R_1g1pIfzQ
Đặt tên lại
https://picasaweb.google.com/lh/photo/ueFsSZQM8NJgokgnBU8bd26XXLEmDmap0R_1g1pIfzQ

## ======
> return $this->db->select('*')->from('prodouct')-get()->result_array();
viết gọn thành
> $this->db->get("prodouct")->result_array();
khi nào cần các select đặc biệt hoặc có nối bảng thì mới viết đầy đủ ra.

## ======

1. Em làm quen thao tác với object nhé.
2. viết đầy đủ foreach => foreach($array as $key=>$value){}


If you use

    $this->add('date' ...

instead of the deprecated `addDateTime` method then you can skip adding the jcalendar.tpl junk to the template.


@lucasart , Please read the commit message.  I detail how the interval is quite different from before.

The UI code is unchanged, I just reordered some blocks, which was possible since alpha and beta are no treated uniformly (there is no longer a fail-low and fail-high branch for [alpha, beta] computation).


You can't put conditional logic inside {ts} tags. See http://wiki.civicrm.org/confluence/display/CRMDOC/Internationalisation+for+Developers


Have you actually done this? You may simulate that if you use you right speaker as headphones and your left speaker as master.
You can actually hear the sound moving from one ear to the other if you play with the delay. Once the sound is in the middle of you head, the delay is correct.

My original explanations was surly confusing, but the new one is somehow useless, since you can hear a delay (a echo) only if it is big enough > 30 ms, all values below are transformed by your brain into a position.

Maybe you have an idea how to describe that as a nice sentence.


With the above change to the php, this line can be changed to simply

    {$form.next_scheduled_run.html}


You can downl

https://cmake.org/download/


@Rabrennie This doesn't work for a during-game pause menu btw


I agree to merge


slabua:

This is not like draw distance stepping.  Draw distance stepping suddenly reduces your draw distance after a teleport and, over time, increases it in steps.  Once the draw distance has been increased all the way, it doesn't do anything until after the next teleport.  It does not consider the FPS rate.

Automatic draw distance adjustment does consider the FPS rate.  It does this all the time rather than only for some limited time after a teleport.  It gradually adjusts your draw distance to adapt to changing condidtions, optimizing for a decent FPS rate and the highest draw distance you can have, within the limits you configure.  It avoids sudden large reductions of the draw distance unless the FPS rate suddenly goes down, in which case it can use larger adjustments to bring it back up more quickly than otherwise possible.

Those two are entirely different things.  I never liked draw distance stepping because it makes me blind after each teleport by cutting down the draw distance, without giving me any worthwhile advantage.  With automatic draw distance adjustment, I have both a decent FPS rate and a decent draw distance --- within the limits of my hardware and with the parameters configured accordingly, of course.  The draw distance is not reduced below the minimum I specify, so I'm not blinded.  I simply get the best out of my hardware without having to think about it anymore and without having to chance settings all the time.

I consider that an optimization --- or a great improvement.

(The current version also makes use of shadows by enabling and disabling them automatically.  What's missing is an option to make it not use shadows because enabling them is probably a bad idea when insufficient hardware is used.)


ㅁㅁ


need to add a check to see if the game is just paused then


The deleted lines in this commit are not actually in the current ide_methods.bas; I had just forgotten to remove them from my local copy before actually making my current change to the main window title; these lines refer to Steve's fix for the ASCII box, as seen here: http://www.qb64.net/forum/index.php?topic=13314.msg115140#msg115140


shouldn't this be hb_coretext_aat_shaper_face_data_ensure? (note `_aat` bit in ensure name)


likewise, shouldn't this be hb_coretext_aat_shaper_font_data_ensure?


Perfecto!


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


The changes in this method were motivated by two factors:
1) Error handling did not specify the nature of the error, making debugging a bit harder.
2) An almost-infinite loop arose when improperly terminated lines were passed into this method.  This caused a heap overflow error, which was not intended (OutOfBoundsExceptions were fine though) and didn't get properly handled by checkCorruptData(String).


修改样式


Thanks! :+1:


@foxbunny which version of the kernel is this for?
I see v3.18.9+ mentioned here:
http://discuss.outernet.is/t/new-raspbian-images-available-now-with-hdstar-support/1194
But that was awhile ago.
thanks, drew


Blah, thanks for reminding me. No, it wasn't fixed. I was kinda hoping you would know how to.


Does this resolve #6?


It looks like it does. If so, you can also revert f23a1b83147b327686f8ad0fac2341f25bf104c5.


Hm, sorry -- no I haven't tried it. I reverted to your wording about "centering".


Do you think it'd be worth it to leave a comment here, noting that this is only safe to do because this `init` is located in a .go file with `_test` suffix?

If `time.Local = time.UTC` were to happen in normal package code, that would be extremely bad for any package that imports `go-diff`.

Such a note would make it much less likely this potentially dangerous line of code would inadvertently make its way into the normal package.

It shouldn't happen, so it's up to you if you think it's worth it.


It is really hard to maintain the version number of epel-release, so why not just `yum install epel-release` to make life easier :)


what the heck


TeamCity pull requests :: netty [Build 918](https://garage.netty.io/teamcity/viewLog.html?buildId=4329&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Tests passed: 6363, ignored: 21; resource leak detected (new) Build time: 00:42:13


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


never trust visual studio git plugin


It would have been nice of you to respond to emails over the past 3 months.  I ended up giving up on you, secured xhyve.xyz for the project, and setup a github organization.  This move now just leads to confusion =(


Hi, do you think there would be any clean way give the option to revert this to the old behavior? (Other than Monkey-patching or creating a new renderer)
@robin850


LGTM


I've done that in an early version, but this isn't intended behavior in all cases. So I would like you to bind it to a new property `activateApp` or `becomeActiveApp` (or what ever) and set the default to `NO`. So we should let the developer decide (in case his App is using an own main window).


TeamCity node build testing :: PR submitted #3 [Build 3](http://localhost:8111/viewLog.html?buildId=62&buildTypeId=NodeBuildTesting_PrSubmitted3) is now running


TeamCity node build testing :: PR submitted #3 [Build 3](http://localhost:8111/viewLog.html?buildId=62&buildTypeId=NodeBuildTesting_PrSubmitted3) outcome was **SUCCESS**
Summary: Running Build time: 00:00:01


TeamCity node build testing :: PR submitted #3 [Build 4](http://localhost:8111/viewLog.html?buildId=63&buildTypeId=NodeBuildTesting_PrSubmitted3) is now running


TeamCity node build testing :: PR submitted #3 [Build 4](http://localhost:8111/viewLog.html?buildId=63&buildTypeId=NodeBuildTesting_PrSubmitted3) outcome was **SUCCESS**
Summary: Running Build time: 00:00:01


- для начала давай вынесем стили в css файл, код в js файл, в html оставим только html
- зачем использовать форму, если все равно вызываешь preventDefault()?
- по заданию нужно сделать компонент. я рекомендую использовать модуль. про него можно почитать здесь: http://largescalejs.ru/module-pattern/. В твоем примере будет простой public интерфейс этого модуля - метод create() или init(), который на вход получает query главного контейнера (сейчас это форма). вся остальная навигация должна осуществляться относительно этого контейнера.


Could you please send a pull request to us for this command?


Fix it now


переменная form не используется


Actually, don't.


> Other blocks don't fold unless they have at least two lines of code in them which is disables folding for cases like this

That's not true, and it never was. Also it's not true for java

<img width="353" alt="screenshot 2015-12-29 23 01 12" src="https://cloud.githubusercontent.com/assets/140920/12041188/19921cc2-ae80-11e5-8d00-72d6574a4be6.png">


We can't test the function if you do that.


поиск нужного элемента осуществляется по всему DOM дереву. я вспоминал об этом сегодня на лекции. нужно искать относительно рутового контейнера.


Doesn't.


Just incase the person is in play solo, like I was.


Вот такие названия, как идентификаторы, имена классов и другие данные, которые не будут меняться, обычно выносят в константы вверху модуля. Пример константы: var MY_CONST = 123;


Check my last commit then


Hot damn. Good add, Michael Robert Busch.


Does it work with Cordova-ios 4.0.1? On Cordova-ios 3.9.2, I manage to start my app, but on cordova-ios 4.0.1, I still run into an infinite loop when Angular starts. Using ionic 1.1.1 or 1.2.1 does not make any difference. Thanks


TeamCity node build testing :: PR submitted #3 [Build 5](http://localhost:8111/viewLog.html?buildId=64&buildTypeId=NodeBuildTesting_PrSubmitted3) is now running


TeamCity node build testing :: PR submitted #3 [Build 5](http://localhost:8111/viewLog.html?buildId=64&buildTypeId=NodeBuildTesting_PrSubmitted3) outcome was **SUCCESS**
Summary: Running Build time: 00:00:01


Per Kari's previous edits to my docs, punctuation stays outside of styled text. Maybe this is something that should be added to the style guide.


* 1000 : Conversion result : Seconds to milliseconds


@ltw my full name is Michael Lichaels Bichaels. GET IT STRAIGHT.


They have removed bindShared in a minor version upgrade? LOL


@mikelikesbikes My apologies, Mr. Lichaels-Bichaels.


There is an option in the install script to use a different schema name (see https://github.com/gilt/db-journaling/blob/master/install/postgres-journaling#L49) - these two lines will break the dumb find/replace that supports this option. Can you update the install script to account for these two lines? Likely a simple second find/replace for 'journal;' will be sufficient.


@ltw that's Mr. Bichaels to you, sir.


changed to avoid console error: `Warning: React.render is deprecated. Please use ReactDOM.render from require('react-dom') instead.`


@satya164 that's awesome!

I'm planning to land the packager internal transform pipeline to move Hot Loading into the packager instead of the transformer. Hot Loading won't be available until we land that (the PR is already accepted but I to rebase and take another look as it's been sitting there for a month or so). I'm currently rushing on fixing lots of edge cases to make this more usable internally. I'll try do everything we need so that we can use this on open source so that everyone can contribute to it asap :)


Updated the position using Vector2 : (float x, float y) => (Vector2 position).
Setters more secured.


does it know works even if your server restarts


diffutils is a dependancy of autoconf.
gettext is a dependancy of a lot of packages, including yasm.
glew isn't needed.
gmp is installed if needed.
headers-git is a dependancy of gcc.
libiconv is a dependancy of a lot of packages, including gcc.
libpng isn't missing.
winpthreads-git is a dependancy of gcc.

This is easily tested by running `pacman -Qi $MINGW_PACKAGE_PREFIX-<name>` for MinGW packages and `pacman -Qi <name` for Msys packages.


I had to bump this to "^0.24.0" to get it to compile.


Shouldn't the dict that you create have the same map as self->members?

Otherwise it's effectively read-only. i.e. the following won't work:
```
foo.__dict__['a'] = 'A'
```
should do the same thing as:
```
foo.a = 'A'
```


Would be nice (for MIR trans, at least) if we had some function in `base` which would look something like this:

```rust
enum CleanupKind {
    LandingPad,
    CleanupPad,
    None // on -Z no-landing-pads or when cleanup generation should be skippd.
}
fn cleanup_kind() -> CleanupKind;
```

This would not only help sharing “detection” between plain-trans and MIR, but also switching between implementations on other platforms later easily.


what about no-gender name? alf for every gender?


nope:

Parse error: syntax error, unexpected '$this' (T_VARIABLE) in C:\Users\Blenda\Downloads\PocketMine-MP\plugins\SurvivalGame-master\src\Andre\SurvivalGame\Main.php on line 1240

Call Stack:
    0.0298     447520   1. {main}() C:\Users\Blenda\Downloads\PocketMine-MP\PocketMine-MP.phar:0
    0.0338     442424   2. require('phar://C:/Users/Blenda/Downloads/PocketMine-MP/PocketMine-MP.phar/src/pocketmine/PocketMine.php') C:\Users\Blenda\Downloads\PocketMine-MP\PocketMine-MP.phar:1
    1.5527     582976   3. pocketmine\Server->__construct() phar://C:/Users/Blenda/Downloads/PocketMine-MP/PocketMine-MP.phar/src/pocketmine/PocketMine.php:466
    4.1235   16676512   4. pocketmine\Server->enablePlugins() phar://C:/Users/Blenda/Downloads/PocketMine-MP/PocketMine-MP.phar/src/pocketmine/Server__32bit.php:1637
    4.1235   16676568   5. pocketmine\Server->enablePlugin() phar://C:/Users/Blenda/Downloads/PocketMine-MP/PocketMine-MP.phar/src/pocketmine/Server__32bit.php:1881
    4.1236   16676664   6. pocketmine\plugin\PluginManager->enablePlugin() phar://C:/Users/Blenda/Downloads/PocketMine-MP/PocketMine-MP.phar/src/pocketmine/Server__32bit.php:1895
    4.1239   16678328   7. pocketmine\plugin\PharPluginLoader->enablePlugin() phar://C:/Users/Blenda/Downloads/PocketMine-MP/PocketMine-MP.phar/src/pocketmine/plugin/PluginManager.php:562
    4.1243   16678392   8. pocketmine\plugin\PluginBase->setEnabled() phar://C:/Users/Blenda/Downloads/PocketMine-MP/PocketMine-MP.phar/src/pocketmine/plugin/PharPluginLoader.php:123
    4.1243   16678472   9. DevTools\DevTools->onEnable() phar://C:/Users/Blenda/Downloads/PocketMine-MP/PocketMine-MP.phar/src/pocketmine/plugin/PluginBase.php:86
    4.1478   16681880  10. pocketmine\plugin\PluginManager->loadPlugins() phar://C:/Users/Blenda/Downloads/PocketMine-MP/plugins/DevTools_v1.10.0.phar/src/DevTools/DevTools.php:45
    4.1848   16683664  11. pocketmine\plugin\PluginManager->loadPlugin() phar://C:/Users/Blenda/Downloads/PocketMine-MP/PocketMine-MP.phar/src/pocketmine/plugin/PluginManager.php:296
    4.1855   16688896  12. FolderPluginLoader\FolderPluginLoader->loadPlugin() phar://C:/Users/Blenda/Downloads/PocketMine-MP/PocketMine-MP.phar/src/pocketmine/plugin/PluginManager.php:152
    4.1868   16693984  13. class_exists() phar://C:/Users/Blenda/Downloads/PocketMine-MP/plugins/DevTools_v1.10.0.phar/src/FolderPluginLoader/FolderPluginLoader.php:64
    4.1868   16694168  14. spl_autoload_call() phar://C:/Users/Blenda/Downloads/PocketMine-MP/plugins/DevTools_v1.10.0.phar/src/FolderPluginLoader/FolderPluginLoader.php:64
    4.1868   16694200  15. BaseClassLoader->loadClass() phar://C:/Users/Blenda/Downloads/PocketMine-MP/plugins/DevTools_v1.10.0.phar/src/FolderPluginLoader/FolderPluginLoader.php:64


Oh, the problem is actually that the promise that sets `exclude` is not resolved until after this code is executed. So `exclude` has not been defined yet.


I've compiled and tested MPV with "--enable-vapoursynth --disable-vapoursynth-lazy" and it works like a charm.
One thing I don't understand is that you were able to build MPV with vapoursynth using the 2 .pc files I got from you... these don't work at all !!
You should remove every text "Python35.m" from all lib fields, otherwise you'll get errors when MPV wants to link all the libs and searches for the static python lib which doesn't exist (only dynamic, nostatic)


Yes. Thanks for the reminder.


But you used the MSVC .libs and need the .dlls in the $PATH to open mpv, no?


Не, это не дело :( Слишком легко ошибиться, используя такой код, слишком разный функционал выполняет initialize в двух случаях. Если представить, что мы пользователи этого класса, то мы бы, скорее всего, хотели бы хотя бы вот так:

```ruby
list = MovieList.from_file('movies.txt')
list = MovieList.from_imdb
```

То есть у нас есть какой-то обобщённый «конструктор списка из данных», а эти ↑ два метода выполняют доп.функцию — извлекают данные из файла или из интернетов, а потом инициализируют ими MovieList.

Другой подход — вообще отдельный модуль ImdbMovieExtractor с функциональностью вроде `list = ImdbMovieExtractor.run` ← но внутри этого `#run` тебе всё равно понадобится какой-то «обобщённый способ конструировать список»


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


Could also be **IF PreferCurrentCursorSUBFUNC > 0 THEN**, since it's never going to be less than zero anyway, but it doesn't affect the evaluation.


This is somewhat incompatible with MIR. At this stage we will already know which blocks are cleanup blocks and which aren’t, but we cannot exactly tell what kind of landing pads will be used here. I guess we will, in the end, never use this parameter or data stored there, but `needs_invoke` depends on it to decide whether `invoke` should be used or not.


script id not corrected.)
must be replaced 17247... - 842..., 17247... has already.)


poprawiłem issue z bcrypt


@rooneywp I noticed that this commit got merged in and includes a permalink change — are there others that have also been changed? We'll need to add redirects so any old bookmarks, external links, etc. don't break.


@tanberry: I just happen to see this commit and I noticed the link to the 'Issue reporting guidelines' points to a private repository. This means nobody outside the Magento devs can see this. You'll probably need to change this to a more public accessible link?


@thebigsmileXD... now do the ASCII art back -.-


Why aren't we just pushing here?

```js
const plugins = [];
if (inDev) {
  plugins.push([])
// ...
```


We believe this to be fixed with version `0.35.4`.  Can you run a build using that version and see what happens? (I'm a Matterfront maintainer/owner, just checking forks for commits ahead of master).

Thanks!


TeamCity node build testing :: PR submitted [Build 6](http://localhost:8111/viewLog.html?buildId=65&buildTypeId=NodeBuildTesting_PrSubmitted3) is now running


TeamCity node build testing :: PR submitted [Build 6](http://localhost:8111/viewLog.html?buildId=65&buildTypeId=NodeBuildTesting_PrSubmitted3) outcome was **SUCCESS**
Summary: Running Build time: 00:00:01


Agree. Especially when moving sections around. Pushed an update. Noticed a typo.


`runbenchmarks("arrays", vs = "JuliaLang/julia@bb73f3489d837e3339fce2c1aab283d3b2e97a4c")`


Глобальные объекты - это плохая практика. Лучше их избегать. В идеале должен быть только один глобальный объект, который живет в window. Когда ты перепишешь это все на модуль будет только один глобальный объект - Todo.


`runbenchmarks("arrays")`


TeamCity node build testing :: PR submitted [Build 7](http://localhost:8111/viewLog.html?buildId=66&buildTypeId=NodeBuildTesting_PrSubmitted3) is now running


TeamCity node build testing :: PR submitted [Build 7](http://localhost:8111/viewLog.html?buildId=66&buildTypeId=NodeBuildTesting_PrSubmitted3) outcome was **SUCCESS**
Summary: Running Build time: 00:00:01


@Foxiy I believe it should.

Naming is way beyond me as I have no idea where do you want to move with this =) `board2slides.py` looks like a good start though.


```fromArray``` is fine. or maybe the popular ```of``` convention.


man gcc 4.9.3
           c++11
           c++0x
               The 2011 ISO C++ standard plus amendments.  The name c++0x is deprecated.

           gnu++11
           gnu++0x
               GNU dialect of -std=c++11.  The name gnu++0x is deprecated.

           c++1y
               The next revision of the ISO C++ standard, tentatively planned for 2014.  Support is highly experimental, and will almost certainly
               change in incompatible ways in future releases.

man gcc 5.1.1
           c++11
           c++0x
               The 2011 ISO C++ standard plus amendments.  The name c++0x is
               deprecated.

           gnu++11
           gnu++0x
               GNU dialect of -std=c++11.  The name gnu++0x is deprecated.

           c++14
           c++1y
               The 2014 ISO C++ standard plus amendments.  The name c++1y is
               deprecated.


Accidentally made it vSG XD


а зачем listOfTasks? ты заполняешь его данными, но нигде не используешь. думаю его можно удалить;)


Sounds like a very good addition!

I think I would prefer if `log_job_error_with_service_result()` would instead get a new parameter `extra_args` or so, which can be either NULL, or a string like `" --user"` would would be added into the command line. Then, the caller would use `d->bus->is_system ? " --user" : NULL` as parameter, to add this in. I think this way the amount of duplicated code would be much smaller I think!


This new build command works OK but it breaks the normal compile command for c#. Roslyn says that some dependency dll cannot be found. Did this PR remove some feature from dotnet-compile that should have remained there? Is the ability of compiling only the current project to the bin folder being retired?


Let's remove this `h1` completely, I think just having his name below is fine.


Yah that was kinda assumed :)


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


Is this a joke?


GCC 4.8 and Clang 3.3 only recognize the old parameter.
Em 29/12/2015 18:31, "zosrothko" <notifications@github.com> escreveu:

> man gcc 4.9.3
> c++11
> c++0x
> The 2011 ISO C++ standard plus amendments. The name c++0x is deprecated.
>
>        gnu++11
>        gnu++0x
>            GNU dialect of -std=c++11.  The name gnu++0x is deprecated.
>
>        c++1y
>            The next revision of the ISO C++ standard, tentatively planned for 2014.  Support is highly experimental, and will almost certainly
>            change in incompatible ways in future releases.
>
> man gcc 5.1.1
> c++11
> c++0x
> The 2011 ISO C++ standard plus amendments. The name c++0x is
> deprecated.
>
>        gnu++11
>        gnu++0x
>            GNU dialect of -std=c++11.  The name gnu++0x is deprecated.
>
>        c++14
>        c++1y
>            The 2014 ISO C++ standard plus amendments.  The name c++1y is
>            deprecated.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/RangelReale/poco/commit/8acbfad5509d3ecbbbdc33bbaf55f45a0528a87a#commitcomment-15195684>
> .
>


Maybe the promise chaining isn't quite right.


What do you want to accomplish with to force switch ?
It won't update an existing package  because these are already filtered out by the above grep commands.
It would be nice if existing packages would be updated in the process, but if I see it correctly, existing packages aren't touched, even not if there's an update available.
So, only packages not already installed = only new packages come in the var $install[*].
Why would you install a new package with --force ?  It could break another package because the normal check for conflicts is ignored by using --force IIRC.


TeamCity node build testing :: PR merged [Build 1](http://localhost:8111/viewLog.html?buildId=69&buildTypeId=NodeBuildTesting_PrMerged) is now running


TeamCity node build testing :: PR merged [Build 1](http://localhost:8111/viewLog.html?buildId=69&buildTypeId=NodeBuildTesting_PrMerged) outcome was **SUCCESS**
Summary: Running Build time: 00:00:01


if you change `check_prerequisites` it's likely going to change all according tests. Imo it doesn't make sense to split them up, they definitely belong together. Also I would need to copy these `old_binary = IndentBear.BINARY; IndentBear.BINARY = old_binary` two times extra, just more code doing the same.


TeamCity node build testing :: PR submitted [Build 8](http://localhost:8111/viewLog.html?buildId=71&buildTypeId=NodeBuildTesting_PrSubmitted3) outcome was **SUCCESS**
Summary: Running Build time: 00:00:00