What will we get if a network delay happens here and the "part3" part is loaded first?

The code responsible for tables merging doesn't care about priorities:
https://github.com/dereksantos/angular-translate/commit/561573a16745fabf0eafb00559cd9534aecd424a#diff-607f7e783d326a61fcc59253b7ed447fR302


Should Router also implement Phalcon\DI\InjectionAwareInterface also?


This is a known issue to us. It is related to


TeamCity Yandex QATools :: Html Elements :: pull-request-build [Build 53 [???]](http://teamcity.qatools.ru/viewLog.html?buildId=1940&buildTypeId=htmlelements_pull_request_build) is now running


The case for value SCTP_E_CLOSE is not terminated by a 'break' statement.


TeamCity Yandex QATools :: Html Elements :: pull-request-build [Build 53 [1.14-SNAPSHOT]](http://teamcity.qatools.ru/viewLog.html?buildId=1940&buildTypeId=htmlelements_pull_request_build) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 0:0:0


Hmmm This should be refactored in some way if possible


Mas merge não reenvia tudo, só combina as diferenças.
Agora sobre ficar fazendo merge o tempo todo, acho um saco e pouco produtivo. Nunca usei i esquema do pages para ficar publicando.

Podemos fazer o merge para a branch master somente depois de homologado a page. Que tal?
Ai sim, poderíamos usar o pull request para melhorar a validação e o "merge".


可以合并


Changed to enable app server deployment


This commit has been superseeded. Please see https://github.com/belmiromoreira/nova/commit/1c64273d0a0e3394f27e749ce10f0f997044b9fa


Use the new Ruby 1.9 hash syntax.


Sim. No code review vou ajustar isto. Na hora não quis ajustar o CSS.


@ndeloof: why did you revert this? We had big problems when Jenkins created 2GB temp files on the master for builds so were happy with the fix.


@a11archer, забей на 79-ую.

Мы эту ветку добавили сюда только в виде рабочей папки, для удобства при сравнении с ней и копирования из неё. Просто, раньше нам приходилось каждый раз [лазить к ним в ведро](https://bitbucket.org/ahoyworld/aw-i-a-2), когда хотелось пощупать или посмотреть на этот исходник.


Ok, so nothing I can change here, the code requires a recursive mutex at this point, which I only get with QMutex, not QBasicMutex.


TeamCity CI Experimen :: PHP Symfony2 project :: Symfony Project Test [Build 46](http://192.168.44.2:8111/viewLog.html?buildId=113&buildTypeId=CiExperimen_PhpSymfony2project_SymfonyProjectBuild) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 12 Build time: 00:07:52
### Failed tests
```
Project Test Suite: Test: testDaemonStatus: <no details avaliable>

```


The usage of median instead of mean is also to be coherent with the values calulated during ETL (https://github.com/transmart/transmart-data/blob/master/ddl/oracle/tm_cz/procedures/i2b2_mrna_zscore_calc.sql#L296)


TESTS PASSED
 Mr.Roboto url : http://jenkins.plone.org/roboto/get_info?push=4e084d78682c471a8ac5dd025dd99491
 plone-5.0-python-2.7 [SUCCESS]


Looks like this one does need a container-row, else it goes full width which is undesirable.

Thoughts?

![header_row](https://cloud.githubusercontent.com/assets/61020/5872680/17510122-a2e6-11e4-9a4b-195919687b5b.jpg)


This talk cannot be a journey towards that migration. This talks *explains* the journey of this migration or something like that.


?
There is a workaround suggested [here](http://stackoverflow.com/questions/16332092/spring-mvc-pathvariable-with-dot-is-getting-truncated), but we plan to look over this issue again and come up with a better solution.


Good spot. Thanks.


Sorry - should have tested that.


Hey, it's cool to see somebody take a whack at refactoring this. FYI, we've updated the master branch to use rspec 3.


You're right.
I'm not really familiar with bash...


Can you point me to the jenkins jobs?
I think we should have a long-term solution for ndk bumps, or we'll hit the issue it at each release (as I think we should bump to latest ndk after each release)


comment ---


Ta


@miyaka here is a (very) short description


Block will be tricky without Z3!


Could we do the `Build()` call once at the top? I know it probably is only called once anyway, but it looks fucky... :wink:


Nice.. So I guess the question is should it look like a Assumption error, 'ok, but skipped' or as @Ignored ?


там ещё внизу
```
{$template = './topic.tpl'}
```


Does it make sense to use the constants here?


![checkboxdiff](https://cloud.githubusercontent.com/assets/10582708/5872693/a8725ef0-a2ee-11e4-9139-3eb4278cbf03.png)

On the left side you see the current icon painting of SimpleCheckIcon, on the right side the new painting after the change.


this is in 1.0.2


/*
 Fade

 This example shows how to fade an LED on pin 9
 using the analogWrite() function.

 This example code is in the public domain.
 */

int led = 9;           // the pin that the LED is attached to
int brightness = 5;    // how bright the LED is
int fadeAmount = 7;    // how many points to fade the LED by

// the setup routine runs once when you press reset:
void setup()  {
  // declare pin 9 to be an output:
  pinMode(led, OUTPUT);
}

// the loop routine runs over and over again forever:
void loop()  {
  // set the brightness of pin 9:
  analogWrite(led, brightness);

  // change the brightness for next time through the loop:
  brightness = brightness + fadeAmount;

  // reverse the direction of the fading at the ends of the fade:
  if (brightness == 0 || brightness == 200) {
    fadeAmount = -fadeAmount ;
  }
  // wait for 30 milliseconds to see the dimming effect
  delay(30);
}


> I haven't been following this too closely, but I saw streams and nulls, so just wanted to make sure that binary streams with embedded nulls are still dealt with properly.

Yes, that still works :)  It's just a question of how/when to add a null termination to a string to make it ASCIIZ.


I think it would be cleaner if you pull this mutex protected section into another func (such as `getBackends()`). Then this whole func would be a one liner:

```go
return roundTripHTTP(shuffle(getBackends()), req)
```

Even if you don't make it a single line, it's still easier to make sense of the mutex lock/unlock using the standard `mu.RLock()` followed by `defer mu.RUnlock()` pattern


Please add to .gitignore


Thanks.  Let's leave vstr restructuring to another time :)


Publishing new version to npm.

On January 24, 2015 1:58:35 AM EET, Jonathan Ong <notifications@github.com> wrote:
>what's the reason for these patches?
>
>---
>Reply to this email directly or view it on GitHub:
>https://github.com/rvagg/nan/commit/87c127bba328c8229f2e8e3875b19422888abe5f#commitcomment-9425556


well, my preference is /boards to keep links short...
I mean what else would be in /content or /data apart from board media?


```java
String foo = new String();
```


Correct, the tests can only be run if PhantomJS is installed and there isn't a standard way of enforcing that as a build dependency.  That said, this commit fixes the issue where the create contact form correctly displays the "Create" button versus showing "undefined".  The failing test, "test_0001_should create the record and then show its details", was failing because of that issue and now correctly passes.


```java```


Fixing tag in npm, the hard way


Please remove this line


As for the strings, you should really write your own code instead of adding a dependency. See here: http://stackoverflow.com/questions/3322152/is-there-a-way-to-get-rid-of-accents-and-convert-a-whole-string-to-regular-lette

Is there any reason you can't download the page as html and then strip the elements that you need? That strikes me as better than adding a dependency.


If you're exporting this solely for testing, you could do something like net/http does and just export it from a `_test.go` file: https://github.com/golang/go/blob/master/src/net/http/export_test.go#L20


Of course, duly noted. Thanks, updated. So...

* `5 || 0` evaluates to `5` whereas...
* `NaN || 0` evaluates to `0`

...same for `null || 0` or `undefined || 0` as all of those evaluate to falsy.


yay


STEALER

Kappa


@fatanassow I think the general advice is to`liftF` instead of Suspend now. Will that work for you?


Yeah, now that I know how it works, I can simply call it with an assignment `a = findModule..`, and then `print(a)`


```java

foo = new Stirng();
```


Can we detech the target directory exists first before we want create it?


I'm not sure these two different interfaces, `ServeWebSocket()` and `ServeHTTP()` should be exposed separately on the `Proxy` interface. Why not stick with the `http.Handler` interface and let the implementation of `Proxy` choose to be smart enough to handle conn upgrades?


Should it be `get_int_digits(&p) - 1`? (https://github.com/neovim/neovim/issues/1839#issuecomment-71287023)


Echoing the number of rows regardless of the function used is unnecessary. Would rather echo the number of rows only when there is more than one row.


I've re-factored all to ARTIQ.


effectivement désolé ^^


Plz do not report us to internets police D:

We <3 you guys


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


```java foo = new String();```


```java foo = new string();
bar = new String();```


I like when exploits specify the space available, it's pro, why did you delete it?


Ack


all are now fixed


Still I encourage to avoid enforcing the ```EncoderType``` and do a fine ```BadChars``` analysis. There is a good reason to provide ```EncoderType``` here instead of accurated ```BadChars```?


So are you saying that the game is launching fullscreen as 640x480 even though you have a different valid 4:3 resolution set in your config file?


Did you and Robert agree upon a standard way for controllers to pass status information back to the gui? I'll be happy to use this approach.


just use `http.StatusText(503)` or `http.StatusText(http.StatusServiceUnavailable)` for all of these, rather than inlining the string repeatedly.


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


... автор удалил репозиторий


try: get "/files/:filename" $ do


lgtm!


FilePath and String are aliases, so you don't need to transform one into the other


I apologize the typing error in the message "heloworld" instead of "helloworld"


should probably squash this with the parent so it never gets committed


We need to be careful of not letting "research" extend too far too. In heart bleed, a prominent researcher had to get the remote system to ping back to verify it was vulnerable. That's crossing a line.

How to define "the line"?

// mobile
---
J.Thompson

On Jan 23, 2015, at 6:39 PM, davi <notifications@github.com> wrote:

I'm also on that ABA mailing list and I'll just respond here. The biggest gripe most people seem to have is a big pair of shoes to fill. Can't speak for most people. Will just say my experience is that criminalizing important research is what I hear most often as a complaint. The economics and politics of who is a priority also is a concern, although less than the fact that research should not be criminalized at all.

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


Seeing "FAST" in all caps made me wonder if it was an acronym that I wasn't familiar with.


I found this function: doesFileExist :: FilePath -> IO Bool, which would fit in perfectly (lucky!) So you could get the Bool out of the IO and use an 'if' statement for now.


AFAICT this function is never used


Are these the "type of people who could benefit most"? I'm guessing so, but it isn't clear?


1277852-CommandLineInterface (kbenne) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Tests Failed

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


CD = Continuous Deployment, right? (I would just spell it out)


"stupidly risk changes" should be "stupidly risky changes"?


I think this assertion was lost. Is there a test for the unsuccessful upgrade case? One that ensures that the conn is closed if the upgrade response is not a 101?


agreed. I'm also confused by which liftIO to use. Something to research!


This type of post-mortem is one that I'm not familiar with, I want to read more about this sort of post-mortem.


:beers: this is a github commit-comment :beer:


I'm slightly confused. Why isn't the port relevant for this test? Isn't the idea to have the `instance` ProxyServer connect to an upstream proxy at the specified port?


I wasn't sure how best to handle this. We need some way to delay this until after the cues have loaded but textTracks themselves don't have any 'load' events or anything.
@dmlap @heff thoughts? I guess the main issue here is that if the player gets disposed during the timeout but we could listen to dispose or use the new `component.setTimeout`. Also, 100ms might be too high.


good!


1277852-CommandLineInterface (kbenne) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Good job Bot!!!


Sorry it took so long to respond, I was referring to this[0] portion of the style guide:

"Every function declaration should have comments immediately preceding it that describe what the function does and how to use it. These comments should be descriptive ("Opens the file") rather than imperative ("Open the file"); the comment describes the function, it does not tell the function what to do. In general, these comments do not describe how the function performs its task. Instead, that should be left to comments in the function definition."

[0] http://neovim.org/develop/style-guide.xml#Function_Comments


Wow you're continuously amazing me! Merging and checking it out!


done


1277852-CommandLineInterface (kbenne) - i386-Windows-7-VisualStudio-12: Tests Failed

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


I'm having a lot of trouble tracking down the commit where this was changed from using the helper function:

```go
http.SetCookie(w, cookie.String())
```

Anyway I think it's cleaner and less error-prone that way.


LGTM


Setting the class and style of an element is not `element["class"] = "class"` and `element["style"] = "style"`. To set class: `element["className"] = "class"
To set style: `element["style"]["height" (for example)] = "100px"`


@estesp:
```console
$ vim project/vendor.sh
 update all the things
$ project/vendor.sh
github.com/kr/pty @ 05017fcccf: rm old, clone, rm VCS, done
github.com/gorilla/context @ 14f550f51a: rm old, clone, rm VCS, done
...
$ git add -A vendor project/vendor.sh
$ git status
...
  files show up here
...
```

Sorry I don't have some better output for you, but take this to mean I'm also at a loss for what's going wrong on your box? :innocent:


You could create 2 functions: one for the string generation and one for the html generation. There's probably an elegant way to compose the 2 functions, but I can't think of one. As far as searching, you'd need to look at what your doing with type signatures and search in hoogle. For example, searching on "[m a] -> m [a]" will lead you to the 'sequence' function. Otherwise, experience will help over time.


nit: Why `%v` when `Name` is a `string`?


This does not work. Just prints out the version number but doesn't compile anything.


I think that would be a pretty terrible idea in general, actually (and I've considered it at length previously).

The thing about it that I don't like is pretty simple too.  To populate `/var/lib/apt/lists`, you run `apt-get update`.  Wouldn't it be surprising behavior for you as a user to find that `apt-get update && apt-get install -y some-dep && apt-get install -y --no-install-recommends some-other-dep` fails? :wink:


For the record, this change triggers a regression in libuv on Linux systems < 2.6.19.  The fix is under way, see https://github.com/libuv/libuv/pull/162.  The only affected users are (presumably) people that still run CentOS 5.


I went back and checked -- it was a bug I added when porting UZI-180 to socz80 :(


I'm not sure this interface is useful enough to justify its existence. It's making me have a hard time following all of the pieces and I think it feels overcomplicated. I'd still rather stick with a `RoundTripper` and just have those implementations decide whether to be smart enough to handle upgrades. Otherwise you're offloading the burden of choosing which function to call to the object that's using the `Transporter`, but it's not clear why that decision needs to be made by the caller.


I'm not.  But I'm calling the equivalent of updateAction (specifically the relevant portion of it in this circumstance).  If I don't do this, it doesn't save the value.  It's functionally the equivalent of blurring the field.


Trop stylé ! Envoie le à Étienne et Camille !

Nathan=


4 return values is really pushing it. This is a good place to use an internal struct that holds all 3 of the non-error values together. Maybe a `wsConn` struct?


Thanks!

On Fri, Jan 23, 2015 at 2:02 PM, Alec Thomas <notifications@github.com>
wrote:

> I've bumped the version up to v1.3.5 and gopkg.in now picks up the
> correct commit.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/alecthomas/kingpin/commit/f36a7124e5b80b3fce187828e9bd367468cd2f44#commitcomment-9424016>
> .
>


@Killer1337 Thx for reporting. Please refer to https://github.com/pyload/pyload/issues/1069.


This isn't getting set anywhere when you set up a `ReverseProxy`, therefore there is no periodic flushing going on.


Good questions. In general it would be nice to support title or y axis tick rotations (assuming ggplot does, I never use those options) but first I wanted to tackle the the x axis rotation issue that @mswastik has.

That 9 is a magic number that appears because D3 outputs code like this for the axes

```svg
<line x2="0" y2="6"></line>
<text style="text-anchor: middle;" dy=".71em" x="0" y="9">1970</text>
```


I think it might be better to keep this bash - unless the json file gets more complex. just to reduce dependencies on the builder's system.


Cool, at least it's temporary :)

I like the way this is factored out in `ReverseProxy` and would love to see a similar setup for this other flushing loop. Ideally they'd share the same code.


Also, a 1 second interval feels high. Maybe you planned on changing that anyway. Hopefully we can expose that setting to the code that initializes both servers and use the same interval there.


I have successfully one-upped your avatar.

Get yo' game together.


Hi - I am looking to use binstruct to build a binary modification app by loading a configuration file (xml or json) that defines the format of the binary data. My questions are:

1. Does it support bit manipulation?
2. How about alignments? Do I need to manage alignments myself?
3. Do you know any tools that import binstruct to provide a GUI for it? e.g. a tree structure style kind of layout for user to navigate the whole image?

Thanks for your help in advance!

Aaron


These hard coded values have been addressed in the latest commit.


Whoa, Visual Studio made a secret hidden commit!


True, maybe you can have a pre-hook that run `apt-get update` :)


I've inserted the code from chart.js into the $html variable and organized some of the JS, as well as started to replace some of the deprecated javascript variables with concatenated php variables, which I defined above.


I'm guessing it's going to be quite expensive to pass around the full list of backends every time one changes. But even more importantly, we already have that information in memory on another struct. It'd be better if we didn't have to duplicate the lists of backends on the proxy, and instead could look them up directly on the thing that's already storing them.


Next we need to work through the JS and make sure that the highcharts option variable is receiving everything it needs, as well as obviously putting to the server and testing it out.


IMO this is the wrong fix. This disables warning for calls to `ejs.render*()` (i.e. user-generated calls) in addition to `ejs.__express()`. Plus, *if* we are going on this path, why don't we just set `optInDataWarned` to true in `ejs.__express()`?


'No se puede enviar un email son la cabecera "From"'
El texto correcto en esta linea deberia ser:
No se puede enviar un email sin la cabecera "From"


What I meant is something like this won't throw the warning:

```js
ejs.__express(...)
ejs.renderFile(...) // this one should print the warning
```


fast-forwarding master to borshop-integration-127-bk-gh127-stringify-index-names = f2aba7a3


We need to make sure that the script tags that this function is echoing aren't gonna mess with the browsers creation of the html page from the $html variable.


@tumashu Git commit message 中的第一行（summary）是个标题，常常也会直接作为邮件的主题，所以不需要（也不应该）加句号[1]。

[1]: https://github.com/erlang/otp/wiki/Writing-good-commit-messages#dont


Thank you, this will do. Such logic construction can be found in each second SD2 file, with only difference: within the brackets a new variable is declared, so warning is not issued. As to me, I would rather deny to throw C4706 by a command line flag. The last variant here: https://msdn.microsoft.com/en-us/library/7hw7c1he.aspx, being equivalent one, would render the code completely unreadable.


1. I fix the port to a valid one.
2. This test reproduced the problem that when one would ask for the proxy to
through another proxy it would attempt to set the httpProxy option before it
created the client. Since I am not getting to the execute step in this test,
the port is never used by httpClient, but I do not, once fixed get anymore
NullPointerException.
Thanks,
Omer Azmon

From:  jekh <notifications@github.com>
Reply-To:  oazmon/browsermob-proxy
<reply+002fe11eb192a09340f1ea1568f2c4d78d66015fcbe4881192cf0000000110daaa149
2a163ce008fd3ba@reply.github.com>
Date:  Friday, January 23, 2015 at 16:32
To:  oazmon/browsermob-proxy <browsermob-proxy@noreply.github.com>
Subject:  Re: [browsermob-proxy] fix fault: requesting none existing port.
Though not issue for this (0a616c2)

I'm slightly confused. Why isn't the port relevant for this test? Isn't the
idea to have the instance ProxyServer connect to an upstream proxy at the
specified port?

‹
Reply to this email directly or view it on GitHub
<https://github.com/oazmon/browsermob-proxy/commit/0a616c29625f497d414db59d4
689b98093ad92ee#commitcomment-9425850> .


this could be simply `../media/demo.wav` right?


让我来测试下... :)


Initial commit
 master
commit 00d978c8327b266a98afad4300e29c6a00a3d6ec 0 parents
 saudposts authored 3 minutes ago
Unified Split


What do you mean?
The x axis = 0 and y axis = 1


Time for submodule?


`http.SetCookie` takes a `http.ResponseWriter`, which is out of scope of the transport.


Wow.  These are comprehensive changes to our code base!  Thanks, indeed.

This commit builds with _zero_warnings, and passes all non-python tests, for me.


No, I'm pretty sure that's way worse.

I just always screw up which branch to push to on the subtree.


Initial commit
 master
commit 00d978c8327b266a98afad4300e29c6a00a3d6ec 0 parents
 saudposts authored 3 minutes ago
Unified Split
![image](https://cloud.githubusercontent.com/assets/10678398/5885484/e2c82734-a37e-11e4-976e-2ca7d419c77c.jpg)


good idea :+1:


Yes and I've already reverted that change, actually. One thing to keep in mind, however, is that once we get really going, our builders will themselves be built off the base image, so it's not going to be a dependency problem per-se.

I've reverted the change today because it was breaking packer, which doesn't understand numerical variables, just strings, unfortunately.


I'd be happy to work on that and add type support for unquoted numbers - am I correct in understanding that packer barfs on the following json:

{
  "release": 0,
  "build": 100


Since each of these arguments lead to the same result, you can avoid redundant tests by combining them with `...` OR `...` OR `...`. This will also be faster since the first one that returns true will 'short-circuit' the evaluation. You should read about short-circuit logic; it's an important programming concept.


Yup, just try it and you'll see.


Can you please open a PR to the develop branch with this change?


I've got to say, jq is a pretty cool tool with zero dependencies. And it takes away all the horrible edge casea of trying to parse stuff with awk :-)

Too feature full, if you ask me, but it's available for ununtu and rhel, so it gets my vote.


```python
if not checkURL(urL):
```


That's a different test (as indicated by its presence in a different `it()` block).

In other words, you need to write the test code to match your **behavioral** test descriptions. BDD tests are meant to be read by humans *and* computers.


what if the url is "google.com/some/file/"?


Oh, man. Sorry for that.
Yes, you're right.
Also the following one.


SO MANY PEOPLE STEALING

<3


Aha!


So this is chapters specific. We could potentially assume (require that) chapters start at zero and don't have gaps, and then listen for a cuechange event here, which should be triggered as soon as the cues are loaded and the first cue becomes active.


There's no reason to keep them around if everything works. They don't make the code easier to understand or read.


right, fixed.


You know, I thought about that but went with == false because I felt it was
the more readable version; is there a style guide line about this?
On Jan 23, 2015 5:16 PM, "PTC" <notifications@github.com> wrote:

> if not checkURL(urL):
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kingcritter/The-Wasteland/commit/d7154c81031331c1e3e116a3ebc45d5b3843ca26#commitcomment-9426173>
> .
>


It's not safe to write to `proxy` while it's being read in the path where requests are served unless you protect it with a mutex.


Good point, I'll change that a bit.
On Jan 23, 2015 5:17 PM, "PTC" <notifications@github.com> wrote:

> what if the url is "google.com/some/file/"?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kingcritter/The-Wasteland/commit/d7154c81031331c1e3e116a3ebc45d5b3843ca26#commitcomment-9426183>
> .
>


OK, cool.  Yeah, in general I like totally vanilla, boring, syntax for everything.  That implicit type conversion from !! also made me slightly nervous.

There are some things in the C universe that still make me glitch while reading, like the strcmp returning zero for match.  Then the common idiom being `if (!strcmp(x,y))`
For me, I'm pretty strict that ! means not, and shouldn't be mixed because it makes me more efficient, but I follow Chiri's lead here.

For this specific test, I think this one should actually be:
`return (GetAsyncKeyState(vkey) < 0);`

The docs are not totally clear, but my reading suggests that it's possible that the low order bit might be set in cases where the key is not held down, which would misfire on a strict non-zero test.  And might happen if another client (game thread for example) uses GetAsyncKeyState.  So, least ambiguity in result will be from < 0 test.


You should rename checkURL (which is completely ambiguous and break python naming conventions) to valid_url so that your code becomes
```python
if not valid_url(url):
```
PEP8 says "Don't compare boolean values to True or False using =="


And your comment should reflect that.


ah yeah, missed that. :v:


It said that x = 0, y = 0


No previous cloning to repository.


e2c82734-a37e-11e4-976e-2ca7d419c77c fork:true


variant.getStart().intValue() / binSize
to avoid explicit casting


Thank you!


Ah, it's my browser.  Your code section doesn't have a left-right scroll bar, but if I highlight and drag with the mouse, I can see that.  Ok, very good then!


Do you want: `expect(page).to have_content` ?


Thanks! Are there any other errors?


The `val` here is creating a new local variable named `sc` rather than assigning to the sc field in the LocalSparkContext trait, which will cause this SparkContext to leak and interfere with subsequent tests (see my comment over on the PR).

https://github.com/apache/spark/pull/4136#issuecomment-71293352


consider inlining startBin (to eliminate a name since it is only referenced once)
also, startBin and endBin can be moved into the loop initializer (to reduce the scope that they are visible in)

    for (
        int endBin = VariantUtils.isVariant(variant) ? startBin : getEndBin(options.getBinSize(), variant),
            bin = getStartBin(options.getBinSize(), variant);
        bin <= endBin;
        bin++) {
      context.output(KV.of(KV.of(variant.getReferenceName(), bin), variant));
    }


huehuehuehue


:clap:


@Raydiation I don't understand what this file is for, but this line parses wrong, you meant

`})(window, document, jQuery);`


consider moving startBin and endBin inside the for initializer to reduce the scope that they are visible in

    for (int startBin = getStartBin(options.getBinSize(), variant), endBin =
        VariantUtils.isVariant(variant) ? startBin : getEndBin(options.getBinSize(), variant),
        bin = startBin; bin <= endBin; bin++) {
      context.output(KV.of(KV.of(variant.getReferenceName(), bin), variant));
    }


It turns out this is *painfully* slow when writing/reading screenshots to a USB-stick profile.  I may need to revisit this soon.


hm, actually I guess this entire block needs to go into the above else clause. We get a "none" stemmer for free on init, so if `lang == NULL` then stem will necessarily be the existing "none" stemmer


Oh okay, that makes sense. The test isn't testing the upstream proxy per se, but rather testing that you _can_ set an upstream proxy before calling start(). That should be fine for now; once the 2.1.x interface is defined, we can add a test case that will actually test the upstream proxy. (The mechanism to add the upstream will be different).

Since it's not actually testing the _use_ of the upstream proxy, would the test still be valid if we didn't make the HTTP call? So basically just:
```java
...
instance.setOptions(options);
instance.start();
instance.stop();
```

Thanks for this PR! Glad to have this bugfix make the 2.0 release candidate.


alternately, I could just do a `if lang == NULL, then lang = "none"` block before the hash lookup


nit: in general, i suggest breaking *before* extends rather than after. looks better when you are both extending classes and implementing interfaces simultaneously

    public static class MyClass
        extends SomeSuperClassThatHas<AReally, LongTypeSignature>
        implements SomeAdditional, InterfacesThatArent, PresentInThisCase


@PEMapModder Ok, Ill look for a way to do this so there is only one Thread for most of this stuff


nice work, Joseph. for new code, its nice to use the new style classes (not worth refactoring existing code):

class PsiObject(object):


Ah OK, bummer. I was rather hoping we could just callback the methods too.  Because that's not straightforward, it seems to me that we ought to to go ahead and make this all object based.  Rather than learn a different way to do this, we can just make them full objects and avoid the callbacks.

What if we make a set of subclasses of the VKInputAction that embed their callback/action routines as methods.  That way it's encapsulated without needing to drop back to C code to do the operation. The input initialization will be a little weird because of the G-> globals, but would be OK.

The VKInputAction for something like NextPixelShader would still make sense as an 'action'.  For an override aiming action, it could call an Override Activate method.

I have a medium preference for making it all object-oriented as a better long term strategy, but short term, callbacks are OK.  My reasoning is that long-term I'd like to have objects that we can drop into other wrappers or maybe other tools, and having a nice object API would make that simpler.

What do you think?


I'm not sure this is complete.  It's possible to start an `href` or `src` attribute with only the double slash, letting the browser infer the protocol based on the current page's protocol.  Comparing domains seems like a better idea, if you go that direction with the implementation.


I'm still trying to wrap my head around, well, everything in the class, so I'm probably not understanding something. But could you help me understand how `> 0` is a bug? Isn't an attribute discrete if it maps values to numbers (thus the Domain map is non-empty)?


don't see that as a research issue. that's even broader. simply getting a response from a service would be too low a bar because everyone would be guilty all the time for common acts. perhaps this would be appropriate: "actus reus non facit reum nisi mens sit rea"? (acts do not make someone guilty unless the mind is guilty).


Looks perfect. I'll test asap. Thanks a lot @authenticate :)


Yes. It would be valid.
Thanks,
Omer Azmon

From:  jekh <notifications@github.com>
Reply-To:  oazmon/browsermob-proxy
<reply+002fe11ec70f058b8ce73300a808d55067a5edd9d28f479192cf0000000110dab82f9
2a163ce008fd562@reply.github.com>
Date:  Friday, January 23, 2015 at 17:32
To:  oazmon/browsermob-proxy <browsermob-proxy@noreply.github.com>
Cc:  "Omer Azmon," <omer@azmon.org>
Subject:  Re: [browsermob-proxy] fix fault: requesting none existing port.
Though not issue for this (0a616c2)

Oh okay, that makes sense. The test isn't testing the upstream proxy per se,
but rather testing that you can set an upstream proxy before calling
start(). That should be fine for now; once the 2.1.x interface is defined,
we can add a test case that will actually test the upstream proxy. (The
mechanism to add the upstream will be different).

Since it's not actually testing the use of the upstream proxy, would the
test still be valid if we didn't make the HTTP call? So basically just:

...
instance.setOptions(options);
instance.start();
instance.stop();
Thanks for this PR! Glad to have this bugfix make the 2.0 release candidate.

‹
Reply to this email directly or view it on GitHub
<https://github.com/oazmon/browsermob-proxy/commit/0a616c29625f497d414db59d4
689b98093ad92ee#commitcomment-9426274> .


Метод удален, а ссылка на него в 108 строке осталась.


The patch looks fine, but I'm just wondering about whether we really want documentation-related things to pop up in the API's type. Couldn't we make `docsFor` or `markdown` take additional arguments instead? Intro, outro, etc.


Oh man, too true.  The multiple character sets used by the different Windows API here is a serious pain-point for me.  I'm not sure, but I think if this were using more modern APIs like .net that this would be less of a problem.


@hbeemster - make sure to use URL-safe variant: http://developer.android.com/reference/android/util/Base64.html#URL_SAFE


here!!!!


Thanks for the fix! Did you re-generate from the TTF file, or did you just manual fill in the pixels?


:+1: Awesome, thanks @smarterclayton ... need this in docker, k8s and etcd :-)


You mean add google web,images,videos,news etc.?
是不是这个意思？
你的这个脚本有搜狗，我看另一个脚本只有百度。
搜索在本土化方面，我认为百度处于垄断地位，所有我认为一些百度结果没有搜狗好，所以pretty good.
hahaha...


Doesn't this imply a change to the width of the glyph, does the table at the end of the file need to be updated, or does this render correctly?


my understanding is that iterator.remove() is O(n) on an ArrayList. Maybe a LinkedList would be more appropriate here?


> I guess it’ll print Java: (followed by nothing).

I mean if `java` is no in `PATH`, shouldn't the shell command raises error?  The code should be:

```ruby
java = which 'java'
return 'N/A' if java.nil?
java = `#{java} -version 2>&1`.lines.first.chomp
```

As for second question, I'm not 100% sure either since I have no clean system to test/proof my memory.


i think just l > 0 is enough here?  curbuf->length() is always >0.


consider an anonymous inner class for the comparator, since it looks like it is only used here....


1277852-CommandLineInterface (kbenne) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


consider making the comparator and anonymous inner class since it is only used once to eliminate a name....


dfdsf


This allows for endpoints to provide a CDN hosting path. It's not appropriate for GitHub enterprise, so you can leave it blank in that scenario.


Found BROKEN by codegen.
dded.cmake
Auto-merging casadi/core/runtime/CMakeLists.txt
Auto-merging casadi/core/polynomial.hpp
Auto-merging casadi/core/polynomial.cpp
Auto-merging casadi/core/options_functionality.hpp
Auto-merging casadi/core/options_functionality.cpp
Auto-merging casadi/core/mx/unary_mx.hpp
Auto-merging casadi/core/mx/transpose.hpp
Auto-merging casadi/core/mx/symbolic_mx.hpp
Auto-merging casadi/core/mx/subref.hpp
Auto-merging casadi/core/mx/subref.cpp
Auto-merging casadi/core/mx/subassign.hpp
Auto-merging casadi/core/mx/subassign.cpp
Auto-merging casadi/core/mx/solve.hpp
Auto-merging casadi/core/mx/setnonzeros_impl.hpp
Auto-merging casadi/core/mx/setnonzeros.hpp
Auto-merging casadi/core/mx/set_sparse.hpp
Auto-merging casadi/core/mx/set_sparse.cpp
Auto-merging casadi/core/mx/reshape.hpp
Auto-merging casadi/core/mx/reshape.cpp
Auto-merging casadi/core/mx/norm.hpp
Auto-merging casadi/core/mx/norm.cpp
Auto-merging casadi/core/mx/mx_node.hpp
Auto-merging casadi/core/mx/multiple_output.hpp
Auto-merging casadi/core/mx/inverse.hpp
Auto-merging casadi/core/mx/inverse.cpp
Auto-merging casadi/core/mx/inner_prod.hpp
Auto-merging casadi/core/mx/inner_prod.cpp
Auto-merging casadi/core/mx/getnonzeros.hpp
Auto-merging casadi/core/mx/determinant.hpp
Auto-merging casadi/core/mx/determinant.cpp
Auto-merging casadi/core/mx/call_function.hpp
Auto-merging casadi/core/mx/binary_mx_impl.hpp
Auto-merging casadi/core/mx/binary_mx.hpp
Auto-merging casadi/core/mx/assertion.hpp
Auto-merging casadi/core/mx/assertion.cpp
Auto-merging casadi/core/misc/xml_file_internal.cpp
Auto-merging casadi/core/misc/symbolic_nlp.hpp
Auto-merging casadi/core/misc/symbolic_nlp.cpp
Auto-merging casadi/core/misc/integration_tools.hpp
Auto-merging casadi/core/matrix/slice.hpp
Auto-merging casadi/core/matrix/slice.cpp
Auto-merging casadi/core/matrix/nonzeros.hpp
Auto-merging casadi/core/matrix/matrix_tools.cpp
Auto-merging casadi/core/matrix/matrix.cpp
Auto-merging casadi/core/generic_type_internal.hpp
Auto-merging casadi/core/generic_type.hpp
Auto-merging casadi/core/functor_internal.hpp
Auto-merging casadi/core/functor_internal.cpp
Auto-merging casadi/core/functor.hpp
Auto-merging casadi/core/functor.cpp
Auto-merging casadi/core/function/x_function_internal.hpp
Auto-merging casadi/core/function/sx_function_internal.hpp
Auto-merging casadi/core/function/sx_function_internal.cpp
Auto-merging casadi/core/function/stabilized_qp_solver_internal.hpp
Auto-merging casadi/core/function/socp_solver_internal.hpp
Auto-merging casadi/core/function/simulator.hpp
Auto-merging casadi/core/function/sdqp_solver_internal.hpp
Auto-merging casadi/core/function/sdp_solver_internal.hpp
Auto-merging casadi/core/function/qp_solver_internal.hpp
Auto-merging casadi/core/function/qcqp_solver_internal.hpp
Auto-merging casadi/core/function/parallelizer_internal.hpp
Auto-merging casadi/core/function/parallelizer_internal.cpp
Auto-merging casadi/core/function/parallelizer.hpp
Auto-merging casadi/core/function/parallelizer.cpp
Auto-merging casadi/core/function/nullspace_internal.hpp
Auto-merging casadi/core/function/nullspace_internal.cpp
Auto-merging casadi/core/function/nullspace.hpp
Auto-merging casadi/core/function/nullspace.cpp
Auto-merging casadi/core/function/nlp_solver_internal.hpp
Auto-merging casadi/core/function/nlp_solver.hpp
Auto-merging casadi/core/function/nlp_solver.cpp
Auto-merging casadi/core/function/mx_function_internal.hpp
Auto-merging casadi/core/function/mx_function_internal.cpp
Auto-merging casadi/core/function/mx_function.hpp
Auto-merging casadi/core/function/mx_function.cpp
Auto-merging casadi/core/function/lp_internal.hpp
Auto-merging casadi/core/function/io_scheme_internal.hpp
Auto-merging casadi/core/function/io_scheme_internal.cpp
Auto-merging casadi/core/function/io_scheme.hpp
Auto-merging casadi/core/function/io_scheme.cpp
Auto-merging casadi/core/function/integrator.hpp
Auto-merging casadi/core/function/implicit_function_internal.hpp
Auto-merging casadi/core/function/implicit_function.hpp
Auto-merging casadi/core/function/external_function_internal.hpp
Auto-merging casadi/core/function/external_function.hpp
Auto-merging casadi/core/function/external_function.cpp
Auto-merging casadi/core/function/custom_function_internal.hpp
Auto-merging casadi/core/function/custom_function_internal.cpp
Auto-merging casadi/core/function/custom_function.hpp
Auto-merging casadi/core/function/control_simulator_internal.hpp
Auto-merging casadi/core/function/control_simulator.hpp
Auto-merging casadi/core/function/code_generator.hpp
Auto-merging casadi/core/function/code_generator.cpp
Auto-merging casadi/core/casadi_options.hpp
Auto-merging casadi/core/casadi_options.cpp
Auto-merging casadi/core/casadi_meta.hpp
Auto-merging casadi/core/casadi_meta.cpp.cmake
Auto-merging casadi/core/casadi_exception.hpp
Auto-merging casadi/core/casadi_common.hpp
Auto-merging casadi/core/casadi_calculus.hpp
Auto-merging casadi/casadi.hpp
Automatic merge failed; fix conflicts and then commit the result.

```


Found BROKEN by quick.
bject external_packages/DSDP/CMakeFiles/casadi_dsdp.dir/src/vecmat/vech.c.o
[ 41%] Building C object external_packages/sundials-2.5mod/CMakeFiles/casadi_sundials.dir/kinsol/kinsol_dense.o
[ 41%] Building C object external_packages/sundials-2.5mod/CMakeFiles/casadi_sundials.dir/kinsol/kinsol_direct.o
[ 41%] Building C object external_packages/sundials-2.5mod/CMakeFiles/casadi_sundials.dir/kinsol/kinsol_io.o
[ 42%] Building C object external_packages/DSDP/CMakeFiles/casadi_dsdp.dir/src/vecmat/sdpsymb.c.o
[ 43%] Building C object external_packages/sundials-2.5mod/CMakeFiles/casadi_sundials.dir/kinsol/kinsol_spbcgs.o
[ 43%] Building C object external_packages/sundials-2.5mod/CMakeFiles/casadi_sundials.dir/kinsol/kinsol_spgmr.o
[ 43%] Building C object external_packages/sundials-2.5mod/CMakeFiles/casadi_sundials.dir/kinsol/kinsol_spils.o
[ 43%] Building C object external_packages/sundials-2.5mod/CMakeFiles/casadi_sundials.dir/kinsol/kinsol_sptfqmr.o
Linking C static library ../../lib/libcasadi_sundials.a
[ 43%] Built target casadi_sundials
Scanning dependencies of target casadi_ipopt_interface
[ 44%] [ 44%] [ 44%] Building CXX object interfaces/ipopt/CMakeFiles/casadi_ipopt_interface.dir/ipopt_solver.cpp.o
Building CXX object interfaces/sundials/CMakeFiles/casadi_sundials_interface.dir/cvodes_internal.cpp.o
Building C object external_packages/DSDP/CMakeFiles/casadi_dsdp.dir/src/vecmat/cholmat.c.o
In file included from /usr/include/coin/IpJournalist.hpp:15:0,
                 from /usr/include/coin/IpException.hpp:13,
                 from /usr/include/coin/IpTNLP.hpp:14,
                 from /home/casadibot/slaves/linuxbot/quick/source/interfaces/ipopt/ipopt_nlp.hpp:27,
                 from /home/casadibot/slaves/linuxbot/quick/source/interfaces/ipopt/ipopt_solver.cpp:24:
/usr/include/coin/IpSmartPtr.hpp:18:4: error: #error "don't have header file for stddef"
 #  error "don't have header file for stddef"
    ^
Linking CXX static library ../../lib/libcasadi_lapack_interface.a
[ 44%] Built target casadi_lapack_interface
[ 44%] Building CXX object interfaces/sundials/CMakeFiles/casadi_sundials_interface.dir/cvodes_integrator.cpp.o
[ 44%] Building C object external_packages/DSDP/CMakeFiles/casadi_dsdp.dir/src/vecmat/sdpexit.c.o
[ 44%] Building C object external_packages/DSDP/CMakeFiles/casadi_dsdp.dir/src/vecmat/sdpxlist.c.o
[ 45%] Building C object external_packages/DSDP/CMakeFiles/casadi_dsdp.dir/src/vecmat/sdpvec.c.o
[ 45%] Building C object external_packages/DSDP/CMakeFiles/casadi_dsdp.dir/src/vecmat/sdpmatx.c.o
make[2]: *** [interfaces/ipopt/CMakeFiles/casadi_ipopt_interface.dir/ipopt_solver.cpp.o] Error 1
make[1]: *** [interfaces/ipopt/CMakeFiles/casadi_ipopt_interface.dir/all] Error 2
make[1]: *** Waiting for unfinished jobs....
[ 46%] Building CXX object interfaces/sundials/CMakeFiles/casadi_sundials_interface.dir/idas_internal.cpp.o
[ 46%] Building CXX object interfaces/sundials/CMakeFiles/casadi_sundials_interface.dir/idas_integrator.cpp.o
[ 46%] Building C object external_packages/DSDP/CMakeFiles/casadi_dsdp.dir/src/vecmat/sdpdvct.c.o
[ 46%] Building C object external_packages/DSDP/CMakeFiles/casadi_dsdp.dir/src/vecmat/vechu.c.o
[ 47%] Building C object external_packages/DSDP/CMakeFiles/casadi_dsdp.dir/src/vecmat/diag.c.o
[ 47%] Building C object external_packages/DSDP/CMakeFiles/casadi_dsdp.dir/src/vecmat/sdpalloc.c.o
[ 47%] Building C object external_packages/DSDP/CMakeFiles/casadi_dsdp.dir/src/vecmat/dufull.c.o
[ 47%] Building CXX object interfaces/sundials/CMakeFiles/casadi_sundials_interface.dir/kinsol_internal.cpp.o
[ 47%] Building C object external_packages/DSDP/CMakeFiles/casadi_dsdp.dir/src/vecmat/dtrsm2.c.o
[ 48%] Building C object external_packages/DSDP/CMakeFiles/casadi_dsdp.dir/src/vecmat/rmmat.c.o
[ 48%] Building C object external_packages/DSDP/CMakeFiles/casadi_dsdp.dir/src/vecmat/zeromat.c.o
[ 48%] Building C object external_packages/DSDP/CMakeFiles/casadi_dsdp.dir/src/vecmat/drowcol.c.o
[ 48%] Building C object external_packages/DSDP/CMakeFiles/casadi_dsdp.dir/src/vecmat/onemat.c.o
[ 50%] Building C object external_packages/DSDP/CMakeFiles/casadi_dsdp.dir/src/vecmat/sdporder.c.o
[ 50%] Building C object external_packages/DSDP/CMakeFiles/casadi_dsdp.dir/src/vecmat/spds.c.o
[ 50%] Building C object external_packages/DSDP/CMakeFiles/casadi_dsdp.dir/src/vecmat/dlpack.c.o
[ 50%] Building C object external_packages/DSDP/CMakeFiles/casadi_dsdp.dir/src/vecmat/identity.c.o
[ 51%] Building C object external_packages/DSDP/CMakeFiles/casadi_dsdp.dir/src/vecmat/cholmat2.c.o
[ 51%] Building CXX object interfaces/sundials/CMakeFiles/casadi_sundials_interface.dir/kinsol_solver.cpp.o
[ 51%] Building C object external_packages/DSDP/CMakeFiles/casadi_dsdp.dir/src/vecmat/sdpnfac.c.o
Linking C static library ../../lib/libcasadi_dsdp.a
[ 51%] Built target casadi_dsdp
Linking CXX static library ../../lib/libcasadi_sundials_interface.a
[ 51%] Built target casadi_sundials_interface
make: *** [all] Error 2

```


I think for cases, when we're out of the loop because of offset issue we shouldn't be popping last element because it's not `endobj`.


그럼 ignore에 추가하지 말고 파일을 지우세요!


So basically we never load the Cg stuff from materials/RTShaderLib/Cg no?
I don't remember what was the status about this last time @Bertram25 cleaned the unused stuff, but if we do not use it, I still think we should drop it since Cg is dead upstream and proprietary.


Since /f admin is publicly visible this command should be too. You can just remove this line for now. Better stick to consistency for now. If we some time in the future feel like hiding all moderation commands from those without permissions we should do so all at once for all commands of that kind.


There are many merge conflicts like this one in this commit.


Ah yes... I have a bad tendency to forget that. Thanks for reminding me. :+1:

Cheers
Deadly


Why does it has to be an elif for OSX? does the compiler identify as both Clang and GCC?


Do we really need to look this up for every extended block? Shouldn't this be placed before/outside the for loop?


> It should be visible to other threads if the block is shared. That's the only time.

OK, thanks.


`separateIsX`は初期化されていないので起動時にはゴミが入っています。`true`で初期化するようにします
`UpdatePanes`内の処理は逆になっていました


This is not the correct file. This change should be made in 'spec/database/sample/links.rb'. This file should be removed, of course (in a separate commit).


u can do it ;)


この変更でいかがでしょうか。（長いので2行に分けて、ansという変数を使いました）


This can be simplified with `map()` and `countByValue()` -- I'll tweak that.


The comparison doesn't have to be over `double` here, but yeah you get this annoying message about it being ambiguous unless the right hand size includes something like `.intValue()` I'll tweak that too.


Make a wrapper around `Addressable::URI.escape`. Call it `escape` maybe. And use that.


@meh-uk I agree in general, but in this case, you have included `opts.fromServer` in the release notes already:

https://github.com/openpsa/grid.js/commit/941a3a3b13936557fb91df6f50cdd926f0791797#diff-5e601b5070ec55264211202a7cf0dacdR48

And in the corresponding JS changes (see https://github.com/OlegKi/jqGrid/commit/7d9fc16ac82d5ce2b49226937c8b8849989757fc) are just two lines, so we might as merge them and be done with it.


Same thing again. Should we really calculate this more that once?


Good catch. Fixed.


Done


This commit needs extensive testing. We need to really verify it works as it should.


This commit breaks the test suite. Please investigate and fix.


Consider using addressable in the `absolute?` helper as well.


This seems to break the tests in drools-karaf-itest module. I am not sure why though. @mswiderski do you know if these dependencies are properly OSGified?


## This is a markdown file


This is a good idea as ASN 0 is invalid. In line with [RFC 6996](http://tools.ietf.org/html/rfc6996), you should really use a private ASN in the example. ASN12345 is the ASN of ORG-GSs1-RIPE ( and Italian business ).
Pick any ASN in the range 64512 - 65534 :+1:


Also this commit brings up the question `What minecraft versions should we support?`. If we start using new features in the Spigot API we will drop compativility with 1.7.x version servers and we would be supporting Spigot 1.8 and upwards only.

We could also make use of a "spigot integration" like I did for the armor stands here: https://github.com/MassiveCraft/Factions/tree/master/src/com/massivecraft/factions/spigot

Something to consider and discuss.


guzel kullanim ancak issue'lari kullanmaniz ya da birlikta kullanmaniz daha mantikli


Yes, of course. Good catch.


In the same spirit, it would also be a good idea to use the format [ <private ASN>:<number> ] for the community.

Thank you for updating this repository.


I don't think so. If two fields have the same name but not the same symbol they are most likely types in which case the signatures are the same.


My bad, I did not test this fix. We should find a way to test keyboard interrupts automatically. Will fix that.


great ! :100:


I leave a comment for pdComboBox at here though it is related to this page:
1. RemoveItem in pdComboBox, m_BackupEntries array may need to update apiIndex and entryIndex
2. After AddItem,we want to AddItem again and assign index at 2:
    .AddItem "MotionBlue",2
   We may need update m_BackupEntries's apiIndex,otherwise, two items has the same apiIndex (=2)

I think in new future, you have add Image support for pdComboBox. The AddItem may come with ImageIndex or Size...Though there's no problem on PD at this moment, but if you add certain item or remove certain item, it may bring a problem.

Just my concern.


Would you be able to isolate small test cases that show this behavior so we could report to @lh3? There might be fixes/flags we could pass to bwa-mem to achieve closer to the expected results.

More generally, we have a tumor/normal test case using synthetic data here:

https://bcbio-nextgen.readthedocs.org/en/latest/contents/testing.html#cancer-tumor-normal

and the current validation (with bwa-mem) here:

https://github.com/chapmanb/bcbb/blob/master/posts/cancer_validation.org

I'm not sure if this dataset will trigger the issues but it would be great to have comparions with bowtie2 as well. Thanks much.


You're right that `fcounter` is never used. The problem here is that `fcounter >= array_length - 1` should be `inputf_index >= array_length - 1`. The rest of code is ok.


Which brings us to the next question. To keep individual copyright or assign it over to pki.io as an organisation? This was mentioned in the book Producing Open Source Software (http://producingoss.com/). It made reference to FSF using copyright assignment (http://www.gnu.org/licenses/why-assign.html), but I suspect that most open source software doesn't bother with it and individual contributors retain copyright.


Shouldn't this rather be `ng.filter:filter` ?


please keep the voters


Maybe i don't understand, but blink must be available to cast from land in water(Liquid?) and conversely


And of course this didn't fix the failures either :skull:


tamamdır oraya geçiriyoruz :+1:


저거 내용좀 볼라고 안 지웠는뎅 :(


WIP label kullanin, her konu icin calisanlari etiketleyin. :+1:


Please, don't change the coding style (opening brace of function body should be placed on new line).


ここのwatchは、trueでもfalseでも挙動が変わらないように見えたので、消しました


inline-source-mapは重い、ということがわかったので、一旦消してみました。
（個人的には、あまり必要なかったです）

無いと困る、という場合は言って下さい。gulp-karmaの側で対応したいです。


どうやら、processというモジュールが無いせいで、バグが出ていたっぽいです。

これを追加したら直ったので、バージョンを上げました。


@Subjunk sweet, i'll build and see how i go


The next time I should ask if someone is already working on a topic.
The last day I began working on minetweaker, but your are faster.


Why did you use a pointer ?
QDateTime() does provide a 'null' value if not set, that you can test for ?


改行多くね？


That is true, but we are required to check if the player is still online. While it execute the addEvent.


これで、以下の様なログがでなくなります。
（karma-webpackのバージョンを上げたことで、webpack-dev-serverからwebpack-dev-middlewareを使うように変更されたようですね。）

```chunk    {0} test/pendual/unit/test_music_list.coffee (test/pendual/unit/test_music_list.coffee) 595034 [rendered]
    [0] ./test/pendual/unit/test_music_list.coffee 7281 {0} [built]
    [1] ./test/pendual/get_fixture.coffee 359 {0}
    [2] ./app/scripts/pendual/pendual.coffee 705 {0}
    [3] ./app/scripts/pendual/status.coffee 3046 {0}
    [4] ./app/scripts/pendual/rival.coffee 1803 {0}
    [5] ./app/scripts/pendual/reverse_rival.coffee 1868 {0}
    [6] ./app/scripts/pendual/follow.coffee 1054 {0}
    [7] ./app/scripts/pendual/follower.coffee 1069 {0}
    [8] ./app/scripts/pendual/music_list.coffee 5120 {0}
    [9] ./app/scripts/pendual/music_info.coffee 12121 {0}
   [10] ./app/scripts/pendual/mybest.coffee 708 {0}
   [11] ./app/scripts/pendual/recent.coffee 1017 {0}
   [12] ./app/scripts/pendual/rival_search.coffee 580 {0}
   [13] ./app/scripts/pendual/rival_challenge.coffee 2406 {0}
   [14] ./~/jquery/dist/jquery.js 247387 {0}
   [15] ./~/lodash-node/modern/index.js 11954 {0}```


LGTM


please insert semester name into this message


See, this is simply wrong, the reason I split them apart, was that arcane orb SHOULD be able to be used with essence aura -- Perhaps a different list for Essence Aura? @Jexah


`" >= " << nzBoxCorner_Max << ")";` would be even better.


Why is this necessary ?


Same here.


Greatfull.
But I think there is a bug.
When moving a child in another chassis (in an another cabinet), cabinetid doesn't update.
Isn't it?


名前を変更して、tokenの引数に対応。
tokenではなく、iidx_idで指定すべきところなので、これから修正します。
iidx_idからRivalSearchを使って検索する関数が必要なので作ります。

あと、prefixはgetではなくdlでしたね。それも修正します。


何回も、IIDX.statusって書くのがダルいので、sにしました。


It's licensed. You can legally license somthing. You do realize that. Stay with your public domain and leave my realm if you are trying to piss me off.


I can license code, of course I can.


HTTPのステータスではなく、ページタイプで判定することにしました。


@SubJunk it's compiling, no issues this time :)


多分こっちに入るのは、通信エラーの時とかだけど、
httpBackendでそれをどうテストするのかは分からないですね…


`NzGetNearestPowerOfTwo(16)` will return 16, because 16 is a power of two.
(This is not a square root).


Oh, the fullscreen height wasn't saved because of [this line](https://github.com/IntelOrca/OpenRCT2/commit/7cdcb6402e2457fd216cf858cdfadeadc7b0b2e3#diff-75218e1e64b8201a55c2ac2203e8970cR259).

But even with correct config:

![image](https://cloud.githubusercontent.com/assets/4729533/5887075/645733e6-a3be-11e4-98a1-8fcf0ecac567.png)

It still launches in 640x480 (probably because only window size is considered when creating a window [here](https://github.com/IntelOrca/OpenRCT2/blob/7cdcb6402e2457fd216cf858cdfadeadc7b0b2e3/src/platform/osinterface.c#L200) )


And 32 here (because 32 is the nearest power of two greater than 17).


This is a really great addition, I didn't know catch.hpp until now.
But how can you include it without specifying it's directory ? (i.e. `#include <catch/catch.hpp>`)


Hola,

He dejado los esquemas en la web, Roberto tb me los ha pedido...

Saludos,
Antoni.

2015-01-24 10:40 GMT+01:00 Francisco Cosmo <notifications@github.com>:

> No se ven las imágenes !!!!
> El tema de lecturas no es un problema, si se enfoca bien esta claro.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ihomedomo/ArduinoDomoticExcontrol/commit/2ba9e050f5464daf70591ba8ea25ce3ee5071738#commitcomment-9427402>
> .
>

--
Saludos,
Antoni A.

----------------------------------------------------
Avís Legal: La informació d'aquest missatge és personal i confidencial, i
el seu ús no autoritzat està estrictament prohibit per la llei.
Aviso Legal: La información de este mensaje es personal y confidencial, su
uso no autorizado esta estrictamente prohibido por la ley.


I won't piublic domain stuff if you are trying to do that okay.


can't you add ```disable_if_archived``` to the arguments in https://github.com/karyon/EvaP/commit/e4ab51d44f6d4145852ec6b4c0591b0572510509#diff-4fd5d03c5ca7218a5e9cb2baa73e8d19R47 and use it here?


LOL. That line predates 1.7.9


Medium: Use capital letter for constants


Medium: Remove unnecessary code


Medium: add {


Greatful
But I think there is a bug.
When moving a child in another chassis (in an another cabinet), cabinetid doesn't update.
Isn't it?


how all of the scripts work and how they are connected.


field parameters in XMLs are in correct place (after a lot of effort),

a little more work to add fieldsets,
so that are the same as text field


Seems right.. Don't know why I thought the remove or save affects angular...


In the max-length self-test, cmp_all() is probably not even called right now but we could call it and check that it returns 0. If it does, and cmp_one() returns 1, we have the bug.


以后 Uglify 都有这个 banner 了吗，可以换个更容易阅读的字符画嘛…


Medium: usually I prefer !"0".equals(commaNumber). In that way you avoid a possible null pointer exception


@brendanashworth note your commit message should have `PR-URL:` that includes the _full url_, not just the `#587`, I've fixed this one since it's at the HEAD.


http://www.universalmediaserver.com/forum/viewtopic.php?f=14&t=1656&p=14394#p14394 new issue that i've found after build and test run of java 6 UMS. debug log is at the link


Why do you need to angular.copy? does diffArray change the arrays in any way?


Great! (using generics)


i changed on a later commit to
, strAutoDownloadSubtitlesExclude() ..

is that ok or should i remove the whole line.. ?


I fear that this will have a negative impact on battery life and performance on most devices. Unless those values are non-neutral in most cases there should be a separate shader which supports brightness and contrast.


Никаких абсолютных ссылок. Домен может поменяться.


Medium: this must be final.


Nice.


这还不够易读...

On Sat, Jan 24, 2015 at 7:02 PM, NetPuter <notifications@github.com> wrote:

> 以后 Uglify 都有这个 banner 了吗，可以换个更容易阅读的字符画嘛…
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ivanzhaowy/generator-wdj/commit/91389f0f048183780a65408d5b0cf4f5c800c34a#commitcomment-9427600>
> .
>

--
*Wangye Zhao | 赵望野*
Mobile ＋86 134 6677 0014
E-mail wangye.zhao@wandoujia.com
Weibo @赵望野 <http://www.weibo.com/ivanzhaowy>

*Wandou Labs | Front-end Engineer*
豌豆实验室 | 前端工程师


There's no need for ELSE after a break.


High: use List<E>


ja bym dał 0.85


why not waiting for the tasks to complete?


Those sums & checks should be outside of your main logic


Changed to "The selected subtitles...."
Removed trailing 0.


typo.


That commit was after Jumbo-1.


ddd


Other threads will be waiting. Other solution needed, which will not block while A->B, C->D


Didn't do too much work with this, it might be possible to still have the poster take up the whole of the right panel, 400px from within the series-list directive template without needing the position relative here, it just makes it easier for now to test.


HAHA


This took an awful lot of time. If this were implemented like 2 years ago, many quests would have been simpler to write just because you could avoid converting l2off bypasses to l2j.


Done 1st Pull


GG, I didn't know you could use dropbox params in the Eventbrite API


what exactly is in here? could you add a comment?


Why is it that do we want offer the possibility to suppress generation of time stamps or comments?


it would be nice if you could show this info text only if the semester really is archived :)


If we care about performance this should be
```
#if ARCH_LITTLE_ENDIAN
 			memcpy(key, Key[i], 32);
#else
			for (i = 0; i < 8; ++i)
				((ARCH_WORD_32*)key)[i] = JOHNSWAP(Key[i]);
			i = 0;
#endif
```


[?]

Mehmet Dik Paü Bilgisayar Mühendisliği

24 Ocak 2015 12:36 tarihinde Necati Kartal <notifications@github.com> yazdı:

> WIP label kullanin, her konu icin calisanlari etiketleyin. [image: :+1:]
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/paufsc/journey-to-html/commit/14ab795cba1b07006fdecc7f5216ff1a48e629bb#commitcomment-9427519>
> .
>


same here. what is the key?


Maybe you can use something like http://docs.oracle.com/javase/6/docs/api/index.html?java/util/LinkedHashMap.html


Scrap that ;-)   the memcpy was just for SSE2


Good point to externalize the `VersionInformation`. :+1:


Good job buddy!


Nvm, it's all good :)


Medium: use capital letters for constants


High: Use the constant


Thanks .. I was looking for something like this.


Now just gotta get rid of these methods in QuestState.


непарные апострофы нужно экранировать с помощью \ , исправь вот тут на  ім\'ям


It's for those crazy people who, for whatever reason, need to commit the generated code to source-control - like we need to do in our project. That's why I'm always running for making the generated code stable, as in: the output doesn't change if the input didn't change, even if one guy generated using our maven-based code generation script, and another one generated from within Eclipse with an Ant-script (allows you to use the internal Eclipse compiler, which has a blazing fast turn-around time).


forget it


Good! Thanks!


Экранируй апостроф вот так: \'


m_step/2.0, m_field*m_step and 2*m_step are constants. It might be more efficient to pass them directly to the shader.


Тоже самое здесь и в других местах (везде)


Ustawiam na 0.8.


Has podido ver las imágenes?,

Saludos,
Antoni.

2015-01-24 10:40 GMT+01:00 Francisco Cosmo <notifications@github.com>:

> No se ven las imágenes !!!!
> El tema de lecturas no es un problema, si se enfoca bien esta claro.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ihomedomo/ArduinoDomoticExcontrol/commit/2ba9e050f5464daf70591ba8ea25ce3ee5071738#commitcomment-9427402>
> .
>

--
Saludos,
Antoni A.

----------------------------------------------------
Avís Legal: La informació d'aquest missatge és personal i confidencial, i
el seu ús no autoritzat està estrictament prohibit per la llei.
Aviso Legal: La información de este mensaje es personal y confidencial, su
uso no autorizado esta estrictamente prohibido por la ley.


В целом все верно, но в украинском языке есть такая штука как ' в словах. Этот символ нужно экранировать косой чертой вот этой: \
Если этого не делать, то приложение не соберется, перевода не будет.

Когда поправишь, сделай PULL REQEST чтобы эти изменения отправились в основную ветку.


I was thinking: perhaps in due time we should do something integration test wise (add some test cases) with this work. WDYT?


黑乎乎的，还投影…


@oguzhantasci ya acikladim konuyu size aktarsin. ben ara ara tekrar bakacagim :)


Deal with it.


OffsetU.y = OffsetV.y. Why compute the value twice?

If the shader designed to support NV12 with UV side by side in the same texture? If not UV will most likely have the same coordinates the whole time.


Siehe https://github.com/OpenGovLD/specs/issues/91


Thanks!


OKey

Mehmet Dik Paü Bilgisayar Mühendisliği

2015-01-24 13:24 GMT+02:00 Necati Kartal <notifications@github.com>:

> @oguzhantasci <https://github.com/oguzhantasci> ya acikladim konuyu size
> aktarsin. ben ara ara tekrar bakacagim :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/paufsc/journey-to-html/commit/14ab795cba1b07006fdecc7f5216ff1a48e629bb#commitcomment-9427700>
> .
>


Happy with these amends


```bash
<script>
  (function(i,s,o,g,r,a,m){i['GoogleAnalyticsObject']=r;i[r]=i[r]||function(){
  (i[r].q=i[r].q||[]).push(arguments)},i[r].l=1*new Date();a=s.createElement(o),
  m=s.getElementsByTagName(o)[0];a.async=1;a.src=g;m.parentNode.insertBefore(a,m)
  })(window,document,'script','//www.google-analytics.com/analytics.js','ga');

  ga('create', 'UA-58829288-1', 'auto');
  ga('send', 'pageview');

</script>
```


I'm wondering, wouldn't it be more efficient to use the range [0,0][1,height] of the texture instead of [0,0][1,1]? One could use floor(coord.y) to find the nearest line, operate with integers and finally divide it by the texture height and add 0.5 as offset to sample in the center of the pixel.


@wachterjohannes @thomasgaida @michaelzangerle @jotschgl


Hmm don't know this one..  :smile:


The horse templates can be described too as a 'weapon' template. (with the idea of making them garrison-able by infantry units I believe)


Maybe is more confortable tu you use the format"

wrappedLine.append(str.substring(offset, spaceToWrapAt)).append(newLineStr)


Hmm, how is it different from OC computers using components? Keep in mind this only works for *external* components (e.g. redstone I/O block).

I'm not opposed to adding more customizability, I'm just curious what for (i.e. whether it's worth the effort :P).


This is a question for the ML I guess


could you handle this in a private method, just like `extractJarFileName` e.g. `extractOsgiBundleName`.? Same applies to the `"jrt".equals`.


Low: remove local environment test outputs. This is avoided by TDD.


Do you foresee specialized versions in the future?


High:
- use camel case convention (starts with lower case).
- methods starts usually with "test" to describe what the method do.


tamam her konuyu issue olarak aciyoruz. (açıldı)
sonrada hangi konular kiminse onlari etiketle (etiketlendi.)
o an kim uzerine calisiyorsa WIP label ekleyip tamamlasin (sağ tarafta
label ekle kısmı var ordan yapılacak. issue ya girip bu konuda çalışıyorum
işleme alındı anlamında wip etiketini kullanıyoruz.)
ve bitince #issue numarasina fixed yazarak commitlesin  (örnek olarak "
bicimlendirme tamamlandi. fixed #15 " şeklinde yapılacak.

saygılarımla.

24 Ocak 2015 13:26 tarihinde Mehmet Dik <notifications@github.com> yazdı:

> OKey
>
> Mehmet Dik Paü Bilgisayar Mühendisliği
>
> 2015-01-24 13:24 GMT+02:00 Necati Kartal <notifications@github.com>:
>
> > @oguzhantasci <https://github.com/oguzhantasci> ya acikladim konuyu
> size
> > aktarsin. ben ara ara tekrar bakacagim :)
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/paufsc/journey-to-html/commit/14ab795cba1b07006fdecc7f5216ff1a48e629bb#commitcomment-9427700>
>
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/paufsc/journey-to-html/commit/14ab795cba1b07006fdecc7f5216ff1a48e629bb#commitcomment-9427710>
> .
>


ah. and here's the test :+1:


Hehe should this be static HTML?


I understand well, but it does not cause an error, it returns nothing if the user forgot the file "notice" in the theme folder.

I propose to replace the condition `else if (file_exists (APPPATH .'views / notice.php '))` in this case and add an exception error instead.

When do you think?


Right now, I can't find any particular use case either, it was just an idea.


should be null, why recreating in here if it is done in `mockApplication` ?


wohl eher 3.4.3


@tisto when you revert someone's commits, can you please make a comment with them in it? Otherwise, people don't even realize all their work has been pulled back.


what, i dont even

I

well


Note:

in more complex test, usually I use the following format:

// given
String expected = "Ten dollars";
int TEN = 10;
NumberConverter converter = new NumberConverter();

// when
String actual = converter.transformNumberToDollars(TEN);

// then
assertNotNull(actual);
assertEquals(expected, actual); --> respect the actual and expected position in the assert params, note that JUnit has one order and other fw has another order. I've never remember what is the correct, just see the javadoc.


We could use some sha function from Guava for it: https://code.google.com/p/guava-libraries/wiki/HashingExplained#Hashing -- that should work! :)


High: in ted you must cover success and fails, in this case you must add some test for "abc.d", "123.", etc.


:scream:


Medium: use map.


Because m_corners is private


Don't worry, you and Syd are examples of great hackers who know what they're doing


Ok/


High: Throws an exception like "NumberTooBigException"


What the Composer


A script is horoshiy__))


Medium: name it like "testThrowsExceptionWhenNumberTooBig". exceedtest doesn't use camel case.


I'm running a repository with 1.4T deduplicated data and an ~2.5GB index file. Encountered exactly the same problem reported in the mailing list message quoted above. Glad to report that the error went away after applying this patch.


"Einführung" works but "Einleitung" fits better in my opinion.


You can combine the file creation with manager adding into a for loop


Na takie coś widze 2 rozwiazania:
1. https://google-styleguide.googlecode.com/svn/trunk/javaguide.html
2. usuniecie repo

W każdym pliku masz takie samo coś(nawet nie wiem jak to nazwać). Jeżeli chcesz uczyć innych, to sam musisz umieć ;)


The problem is that distance is calculate with |ax+by+cz+d| / sqrt(a*a + b*b + c*c) but if a, b and c is normalized you don't have the right answer.


So it's 2.0.1 version now? ;)


Genial Iñaki, pocos High. Porfa revisa los High y hacete un pull request asi me llega el email.
El pull request lo tenes que hacer desde un branch en tu fork al master de tu fork.

Falta aun para hacer TDD puro, pero arrancaste muy bien el proceso :D, no te va a tomar mucho dominar tdd.


r+


#Test repo


A tutaj wykazujesz się nie znajomością(może nie zrozumieniem) Javy.

Random może być jeden, nawet na cały projekt:
public static final Random RAND = new Random();


yeah , i have not try to compile on win for now ,so must have many erro like this in win build,
BTW for this error we can try to add
```
#ifndef WIN32
#include <sys/mman.h>
#endif
```
in osdlib_retro.c


Ya me estoy poniendo a hacer las correcciones. Cuando termine de hacer las high te aviso. Tengo algunas dudas de las correcciones que también te las hago después.

Gracias por las palabras de motivación!!! A seguir metiéndole!!


saw approval from Manishearth
at https://github.com/servo/servo/commit/d2f0aac5ce6fc126c750e59c1a9c094ab9dc03e0


merging servo/servo/untrusted = d2f0aac5 into auto


servo/servo/untrusted = d2f0aac5 merged ok, testing candidate = d9c2bfae


Hum, weird, not parsing lead to extra-escaped things in Editor.
Might be due to wrong loginc in save as you pointed out.


[x] done


Indeed, I was thinking to something else while I was writing this. But in the current function, it's while (x <= number), so for 16, the result will be 32. So I change the name for "GetNextPowerOfTwo". Unless, the meaning is different ???


ㅗㅗㅗ


watch your language :-1:
lol


i dont get it, how can these be not equal?
posq[i] != posq[i]


sorry master, i won't do it again =)


* Cause I'm bad and I force my compilator to search in the right folder :) *


No it doesn't update and it is a bug related to an open issue.  The offending code is located here if you'd like to fix it.  https://github.com/samilliken/openDCIM/blob/master/assets.inc.php#L1467  The code you just commented on is 100% correct.


Ignore me, the failures were caused by something else - missing `commons-codec` dependency, https://github.com/droolsjbpm/droolsjbpm-integration/commit/5a9b6c9bb1e45732476c3aa8b246c982731ed588


TESTS FAILED
 Mr.roboto url : http://jenkins.plone.org/roboto/get_info?push=04a065528c614cc49e323d4668474fbf
 plone-5.0-python-2.7 [FAILURE]


Closed Visual Studio Online Task [vso#4345](https://azurecom.visualstudio.com/DefaultCollection/ACOM/_workitems#_a=edit&id=4345): 'This is a test usersnap task submission'

VSO State: Done


[x] Got it!


Is `as_slice()` really required here ? Since you're using slicing syntax, I think just putting a `&` before the expression should do it...


Great. Thanks.


@slothorange what do you think about this?


ShinyAmos: Hey man, can you update the ANE itself for us to 64-bit :)  I already think you're awesome for updating the AirFacebook.ANE


aaaaaa


Please update the ANE to 64-bit for us :) +1


huuu


Tard...


Could you update the ANE to support 64-bit mode?  Thank you.


This line should stay, as it is explicitly talking about transactions.


[x] done


I _think_ this should be `my $response = shift;`.

The after hooks are run *after* the localization of the request/response in the route has gone out of scope. These hooks should use the response returned from the route->execute. The after hook is already being passed the response object, why not use it :)


// Example program
#include <iostream>
#include <string>

using namespace std;

 void bubbleSort(int array[]);

int main()
{

 int arr[]={10,9,8,7,6,5,4,3,2,1};

 bubbleSort(arr);

 return 0;
}

void bubbleSort(int array[])
{
    int temp;
    int swap=0;

    for (int i=0;i<10;i++)
    {

        for (int j=0;j<10-1-i;j++)
        {
            if(array[j]>array[j+1])
            {
                temp=array[j];
                array[j]=array[j+1];
                array[j+1]=temp;

                swap=true;
            }
        }
        if(swap==0)
        break;
    }

    for (int i=0;i<10;i++)
    {
        cout<<array[i]<<"\n";
    }

}


Nope, it's just to avoid package-dependency cycles.


very cool update


@psychon
> Ideas? :-(

Is PR #87 an option now again maybe?


Good point, will do that! :+1:


why this change?


I probably should have. I'll fix that in the next merge.


We have to reset these subproject commits.


Obviously! Doesn't everyone want a link to `axelni` on their profile? =)


[x] done. Maybe you could drop me a line, why TS is better (just want to know)


This function can be cleaned up a lot:

```c
  rettv->v_type = VAR_STRING;
  rettv->vval.v_string = get_tv_string_chk(&argvars[0]);
  if (rettv->vval.v_string == NULL) {
    return;
  }
  rettv->vval.v_string = shorten_dir(vim_strsave(&argvars[0]));
```

The only change that's necessary is that `shorten_dir()` also returns its argument instead of returning `void`. I just recoded this quickly, so another set of eyes seeing if this is the same would be nice.


I removed the end tags intentionally. They are not needed at all at the very end of the file.
I know the part of the documentation about this: "Use Standard PHP Opening and Closing Tags"
But it says use the standard tags and do not use short open tag (<?) and  asp-like open tag (<%).

When a file contains only PHP without output, end tag is not recommended.
* PHPStorm show a warning: "Redundant closing tag"
* You can accidentally send an output if you write a whitespace after closing tag.
* Github shows a warning when a file does not ended with a linebreak.

The autoloaded classes can be loaded before any ob_start() so in a file that contains only class definition, using the closing tag is bad practice. This is true for any case when a file contains only definitions/declarations like constants, variables or classes.


You are an idiot. Educate yourself what Composer is. Or read the readme, the tutorials etc. like everybody else.


Whoops, that's wrong:

```c
  rettv->v_type = VAR_STRING;
  rettv->vval.v_string = get_tv_string_chk(&argvars[0]);
  if (rettv->vval.v_string == NULL) {
    return;
  }
  rettv->vval.v_string = shorten_dir(vim_strsave(rettv->vval.v_string));
```


In the future we could think about replacing these booleans with ad-hoc enums (e.g., IncludingModifiersPolicy { IncludingModifiers, ExcludingModifiers }) because otherwise is difficult to read the code when we have 2-3 boolean parameters. I am not suggesting we need to do this to approve this change.


please change ignoreCase to false bcz most of Assem use right BuffName :( if you use .ToLower() then buff.Name must set into lower too.


all tests pass:
success: http://build.servo.org/builders/linux1/builds/595
success: http://build.servo.org/builders/linux2/builds/594
success: http://build.servo.org/builders/android/builds/630
success: http://build.servo.org/builders/mac1/builds/811
success: http://build.servo.org/builders/mac2/builds/811


Looks good to me!


This is not working


we need to check this, i´ve added the code, but output still looks the same...


Thanks! Let me know if you know an android equivalent. :)


fast-forwarding master to auto = d9c2bfae


TeamCity Hyrise :: Unit Tests [Build 68](http://ares.epic.hpi.uni-potsdam.de/hyrise-build/viewLog.html?buildId=76&buildTypeId=Hyrise_UnitTests) is now running


[x] done


It runs in its own `Thread`. That's why it's **threadsafe**.


TeamCity Hyrise :: Unit Tests [Build 68](http://ares.epic.hpi.uni-potsdam.de/hyrise-build/viewLog.html?buildId=76&buildTypeId=Hyrise_UnitTests) outcome was **SUCCESS**
Summary: Running Build time: 00:02:59


I didn't look at all the invocations, but we might need to be careful here: is the new get_*_digits() family ever called with user-defined input? I'm not sure if it's a good idea to just `assert()` if the user inputs a large number.


Thanks, we are aware of that - this repository was meant to be a playground for docker, but remains useless (others might use the packages and create their own docker images at the docker hub). Will be deleted soon-ish.


[x] jep, done.


And I have NO idea what was being done there.  The code makes no sense. It may have at some point, but I could not figure it out. I left it commented out 'just in case'


TeamCity Hyrise :: Unit Tests [Build 69](http://ares.epic.hpi.uni-potsdam.de/hyrise-build/viewLog.html?buildId=77&buildTypeId=Hyrise_UnitTests) is now running


What’s the point of this? This repo already have 21 forks, all of which can be used to retrieve the original code.


TeamCity Hyrise :: Unit Tests [Build 69](http://ares.epic.hpi.uni-potsdam.de/hyrise-build/viewLog.html?buildId=77&buildTypeId=Hyrise_UnitTests) outcome was **SUCCESS**
Summary: Running Build time: 00:02:57


Manca da definire lo stile del form di aggiunta feedback, in quanto usa radio button e textarea, non ancora definiti nel css


This one may have been the only buggy format.  It is when BOTH cmp_one and cmp_exact both are simply a 'return 1'   if either of them do real work, I think we are fine.  Either of them CAN be used as a cmp_exact.  But if they are BOTH stub functions, then we have problems.

Now for skey, I can not wrap my brain around WHY we needed one of them (cmp_one or cmp_exact), but we did.  skey only has 1 password worked at at a time, and no OMP.  But without the change to cmp_one, the format failed miserably.

Now the only other format I could find that has cmp_one simply being a return 1 was opencl-mscash2. But in that format, it works.  I added debugging code and checked. Yes, there are some false return 1 cmp_one that happen.  BUT they are caught and not treated as a found password because the cmp_exact will fail, thus keeping the wrong password out.  It is ok being done that way (might be a touch faster to have a quick check in cmp_one, but for actual results it does not matter).


Note, I would like to find some way within the TS, to make sure that the password that is FOUND is the right password that actually cracks the hash.  I made some start on this, but it is not complete.  It is hard to do, since the password may be in one of 2 (or 3) fields in the input file (depends upon pass_gen.pl, and also when I created the input files), and there are some that do not even have the passwords there.  The other really hard part of this, is what happens due to prepare()/salt() and our canonization code within Jtr. It makes for external programmatically matching input to the .pot files very hard to do.

In the end, it may be that I can only do this correct password checking on a subset of hashes.  But I am still letting the design stew a bit in my head.


I understand everything)))
It turns out I have not read the instructions))
After proper installation script to work as it should))


Thanks to the author.
The script demonstrates the structure of MVC engine


See pull #80.


You're certainly welcome to! They're not *my* organizers; they're everyone's! 😜


If past experience keeping custom forks of Clang in sync tells me anything, is that we should try to minimize changes to original source files. So in this case I'd just add a new file `SelectionDAGBuilder_Mono.cpp` and add all of the new code in there, separate from the main file. Most of the times this might not be possible, but when it is, I think we should try to do it.


Show us your feelings sweety.


[x] done


@meh-uk Already done here :

https://github.com/bouks/grid.js/commit/189a67d5598e77e837094181ae606ba495cfe877#diff-2d010088e9cda4a876dbc3e2399c83a0R3137


Nice commit, noticeable improvements!

(1) I definitely vote to move the "Download builds" menu item unter the "Osmand Devenopment" settings, though, they belong below point "Application profiles" there.

Handling of current GPS rack improved somewhat, looks like once the current track is saved, its stats are identical with what prior to that was shown under "Current track statistics".

(2) But number of subtracks under "Current recording track"'s statistics (on All Tracks tab) still seems to be too large.

(3) Also (now mostly correct) statistics there do still not match with what is shown for "Current recording track" statistics on the "Selected tracks" tab.

(4) Also: Once the current recording track has been saved (via the GPX recording widget) and is thus closed, entry "Current recording track" still dos not disappear from the tab "All tracks". (And re-starting another "current recoding track" adds statistics there to the old (closed) file's.)


You can name three developers who care about factions enough to stick around for it, probably your teammates. Please read my comment fully and you'll find the most important reason to keep maven around. You might want to look into Maven's full functionality, see what it's goal is, despite any issues you guys may be having, which I can tell you is most likely caused by your lack of experience with the tool. I've been using maven for over two years and it has never failed me once.

Again, there is an alternative called Gradle, if you ever had any issues with Maven, I really hope Gradle will solve them all. Would you mind sharing some of your issues, maybe we can help you out. :)


TESTS FAILED
 Mr.roboto url : http://jenkins.plone.org/roboto/get_info?push=610d93e1869b41b9bc3c4b9b9e5c268f
 plone-4.3-python-2.6 [FAILURE]
plone-4.3-python-2.7 [SUCCESS]


Ah, well I did wait a few months before even starting so I guess you're very unlucky :') (and it was way easier now that there is an API)


I forgot to delete this line.
Nothing bad is gona happen. I will change it in next commit.


Its funny how you guys remove maven, whilst most of the projects I come across don't mavenize just because they're scared of such a foreign, but once they get help with the setup, they're happier.


Why use find and not something like `$out/lib/ruby/*/rbconfig.rb`? Can't you predict the path?


He did that only because there was hardcoded html bypasses in some NpcStringIds


Imade in r8136. r8135Alwayscrash inSRTP
Error(srtpfilter.cc:706): Failed to init SRTP, err=2
Warning(channel.cc:852): DTLS-SRTP key installation failed
Error(common.cc:59): ../../webrtc/base/thread.cc(347): ASSERT FAILED: !current || current-blocking_calls_allowed_ @ AssertBlockingIsAllowedOnCurrentThread

my environmentXcode 6.1 sdk 8.1
run on iphone6

原始邮件
发件人:Ryan C. Paynenotifications@github.com
收件人:pristineio/webrtc-build-scriptswebrtc-build-scripts@noreply.github.com
抄送:pupuyide@gmail.com
发送时间:2015年1月24日(周六) 02:38
主题:Re: [webrtc-build-scripts] New libWebRTC_objc target gets injectedinto the libjingle_examples.gyp file after doing a gclient sync after callingan update2Revision. Don't ask for code_signing unless you are buildingAppRTCDemo (ipa needs to be code signed) (8d863e4)

The crash I am seeing is related to the changes they made in r8136. r8135 works...
From what I can tell this approach is working fine for building just the library.
—
Reply to this email directly or view it on GitHub.


Just a little nitpick, but could you get rid of the `else` clause, please? :smile:


Same here


java.util.concurrentmodificationexception java is here in this line .


hrm, should be done.


@blueyed and then what? Thanks to commit 254e50d88, PR#87 no longer fixes this issue (well, technically we are now looking at another issue...). The client now has a tag while we try to focus it (<- previous issue), but it is already focused and thus nothing happens, despire awful.rules having overwriten the border color (<- new issue).


> On 24 Jan 2015, at 00:24, Brent Laabs <notifications@github.com> wrote:
>
> An inode alone is not a unique identifier for the resource at a given path, because multiple devices can be mounted. Each device has its own inode numbers.
>
> Do you plan to make a unique identifier by adding the device number, or add a separate device lookup? Maybe a method called any qw(.v .dev device)?

Ah, good point.  I will take that into account as well.

Liz=


A better solution IMO would be to use the getter and to cast to the right type.


+cmd = program.command('console');


ok, got it in the end! [x] done


In my tests this wasn't a problem, the plane's normal is supposed to be normalized (and is when computed), can you give me a demonstration that the answer isn't correct in the case where normal is normalized?


Does this require any docstring changes?


Actually this is a bug, I'm the one to blame, the function should return the number if it's already a power of two.
I'm fixing this in the master branch, thanks.


Thanks, I do this far too much.  :)


I have no objections to isolating our code; this commit just follows precedent as far as `SelectionDAGBuilder.cpp` goes. Right now, our differences from upstream are not that huge so I'm not personally too concerned.

It's probably up to @vargaz since he's the primary maintainer of this repository.


You could just crack the passwords, `LC_ALL=C sort -u` the pot file, and compute the sha1sum, and store these in your jtrts.dat.

Problems I see:
You can't prove correctness this way, you can only prove consistency.
For hash collisions, especially for formats like crc32, the contents of the pot file will depend on number of forked processes, max keys per crypt, etc.


テスト時間短縮のため＆テストファイルの見通しを良くするため、
それぞれのモジュールにファイルを分割しました。

また、tdd_pirodx_#{module}でTDDができるようにコマンドをつくりました。


こんな書き方できるんですね ( > if hoge in [...] )


array $posts如果都沒有通過is_numeric過濾，$posts會成為空陣列，造成$pno = implode(', ',[]) = ""，產生SQL Error


「angularを何回も読み込んだよ！」というエラーが出るので、
いちいちリセットするようにしてみました。もし何か問題が出たら対策を考えます。


あ、そっか。ごめんなさい、勘違いしていました。


いいですね！ありがとうございます。


Pythonでもできるよー

Coffeeで期待通りの動作をするかは、あんまりまじめに検証してませんが、多分大丈夫でしょう？


use app live


cleanup will result in a segmentation fault in case of an error during expression evaluation


我應該把他放在 354 行之前比較好...


I asked hien about this and he said to add closing tag.


I am going to look at this commit later.


ありがとうございます。いい変更です。


わかりました。結局なくてもなんとかなってますね。


The float versions of Distance and GetLength were requested (and are useful) because of the integer version of those classes. It's meant to let the user get a float distance/length instead of a probably incorrect integer one.


So is this the same as commit 73595441d74e89a1f5e947d0f39ce0008ebb5132 but better?


This is a matter of taste, but I have to admit that this version is more readable.


Can you check if #719 has fixed your issues?


Nice catch!


No it’s not better. It actually reverts nib to a version that is two years old and propel to a version that does not include the fix for the generate-model connection bug.


I’m reverting this.


A plane is defined by a normal vector and a distance from the origin toward that direction, I really can't see why it's a problem.


Hello I am Anshuman


Why release dists in root, when they are built to dist/ folder?


Doesn't matter, this should have been done sooner. L2J's phobia of l2off bypasses is yet another stupid thing.


Now I'm lost. I reverted it exactly to the commit the was changed in 7359544?


TESTS PASSED
 Mr.Roboto url : http://jenkins.plone.org/roboto/get_info?push=710a5c4a21c64ba28878e2615be79565
 plone-5.0-python-2.7 [SUCCESS]


Is this the Processing library generated by your code? If so, we shouldn't be checking its binary into the repo. (But if it's a binary from the LWJGL project, that's fine.)


Ok but as they were not used in the motor, I though this was a relic.


7359544 Is already my fix. The problem was in 1a2393d1f7421f507721f8802e7bccf16a2040dd. And there the previous version for nib was 3f7a284906ce3e402bcb101938270842fdad71fa and this is exactly what you removed in this commit. The same goes for propel: the correct commit is 3f7a284906ce3e402bcb101938270842fdad71fa. In the future, please use `git revert` to revert bad changesets as opposed to verschlimmbessern stuff manually.


This is exactly why we need a documentation: to know why each method is useful :)


I meant saved, not sended.


Please, read the second point again! Do not leave linebreak or anything after closing tag!


Was there a bug ?

It should have worked before, but it is still nice improvement to the code


yeah, thanks, my bad, will fix and upgrade to 1.0.4


Pull request this?


I mean you can remove whole line.


git komutlarını tek komut haline getirilebilir aslında

g='git add .;git commit -m "$1";git push -u origin master'


TeamCity MonoGame :: Develop (Win) [Build 3.3.0.1964](http://teamcity.monogame.net/viewLog.html?buildId=5067&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


Standard bash behavior reads files assuming LF is new line character. Hence CR is considered as part of the line. The idea is to make this file more compatible with other bash versions.


ok removed


So I should skip this patch and incorporate changes in my commit as before?


test


Hello, the additional statemenent `!empty($errors.0)` introduces a situation where errors in Admin pages are not being shown at all.

For example, consider this line `AdminControllerCore::validateRules  Line 3143` : The `$this->errors` variable does't have any values with numeric keys, therefore the errors are not being rendered at all.


Do a proper fetch and rebase from upstream develop; this is already there.


@rimelek , sorry, I understood you wrong.
I didn't leave linebreak, I understood that you tell to leave it. Thanks for correction.


Accessing the Build Server

http://jenkins.cyanogenmod.com/
Authenticating to the Build Server

You must be a member of the CyanogenMod organization, and your membership must be public at https://github.com/CyanogenMod

Jenkins will authorize using OAuth to GitHub.
Using the Build Server

Click the "android" job.
Configure what you want to build.
Build it. Note that regular builds (nightly/snap/whatever) can only be produced by the automated processes. Manually triggered builds will always be labeled as EXPERIMENTAL
Adding or updating a device

This repository is preconfigured to use the git-review plugin. More information can be found at: https://pypi.python.org/pypi/git-review

Edit cm-build-targets and submit a change to gerrit for review. The syntax for that file is documented in its first few lines. You can upload your change to gerrit with commands like these:

git add cm-build-targets
git commit
git review

Once you have submitted your change add the group "Release" to the reviewers.


Hey @HeSe nice!
I think that more than a branch, the GemStone flow should have it's own repo.
What do you think?


Please correct my little mistake ;)


<!-- PMABOT:SPACE -->
This commit contains trailing whitespace, what is prohibited in phpMyAdmin. Please check our [Developer guidelines](http://wiki.phpmyadmin.net/pma/Developer_guidelines#Indentation) for more information.

Offending files: libraries/navigation/NavigationTree.class.php


<!-- PMABOT:SPACE -->
This commit contains trailing whitespace, what is prohibited in phpMyAdmin. Please check our [Developer guidelines](http://wiki.phpmyadmin.net/pma/Developer_guidelines#Indentation) for more information.

Offending files: js/navigation.js, libraries/navigation/Navigation.class.php


<!-- PMABOT:SPACE -->
This commit contains trailing whitespace, what is prohibited in phpMyAdmin. Please check our [Developer guidelines](http://wiki.phpmyadmin.net/pma/Developer_guidelines#Indentation) for more information.

Offending files: js/navigation.js, libraries/navigation/Navigation.class.php, libraries/navigation/NavigationTree.class.php


Yes this is the last thing i will do after the review in order to simplify rebases...
Its already listed pending in the PR opening post.


You guys could simply rename TTI_GAMESERVER and the like to TTF_GAMESERVER, you know...


Oh... I haven't noticed. Really.


I tried to update it...It compiled and showed built successfully in terminal...but still when I use the ane, the app does not compile correctly..


I tried to compile it...It compiled and showed built successfully in terminal...but still when I use the ane, the app does not compile correctly..


OK, cool. If you (or anyone else) does need this, lemme know and I'll look into it.


254e50d could be reverted.


It's usually the first thing people are looking for, so I guess it makes sense to have it on root. It seems like a common practice too.

We use `dist/` for building other things during development as well, so it's really `gitignore`d.


No its not needed, it was a copy paste from download dialog.
I removed now.


Can't believe it took me a week to look at this --- thanks.

On Sun, Jan 18, 2015 at 6:15 PM, Jim Geovedi <notifications@github.com>
wrote:

> there's no thinc-1.50.tag.gz in
> https://pypi.python.org/packages/source/t/thinc/
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/syllog1sm/redshift/commit/ec764abdb2066f649824c76ac3b7d4922b1bc4c3#commitcomment-9329325>
> .
>


Fixed as of 4025e5f508f70a15343bbdb82647523cdc88d5d9 !
I'm not providing an update from the values you have now ([object Object] will not be parsed properly) so reinstall the extension to wipe your database and everything will work as expected.


I guess we should use our UA everytime. What is the reason behind this?


Thanks very much, Jonney.  You are right on all points.  At first, I wasn't planning on adding RemoveItem at all to pdComboBox, to simplify my code, but I ended up needing the function in one place.  (The selection drop-down box, because magic wand doesn't support Outline mode, but the other tools do, so we have to remove one item at run-time...)

So the current RemoveItem code is very limited, because it is only meant for removing the last item from the list.  You are right that a much better system would be needed to handle true removal from any point in the list.

Just as an FYI, I use a separate m_BackupEntries() array for three reasons.

1) I had trouble with Unicode text getting lost if the system managed strings for me.  I think this may be the old "ANSI VB windows breaks things" problem.  However, even though we don't really use them, it is still nice for the system to have ANSI copies of our strings, for accessibility purposes (for example, people who are deaf or blind use special accessibility interfaces, so this way, they can still navigate PD combo boxes, using the system ANSI string copies).

2) As you say, in the future, things like image support would be very nice to have.  Also, at present I have divider lines on some combo boxes.  These kind of special features are easier to use if I have my own special tracking array for custom features.

3) Sometimes we need to add items to a combo box very early in the load process, before we even create the API window.  In this case, I have to have a way to store the strings, so we can add them when the combo box is finally created.  This is also why I use API index, because guys may add many items to m_BackupEntries - some with specific indices, some without - and when those entries finally get added to the API window, it was helpful to know where that window placed them.

At first, only (3) was relevant, so that's why the array was called m_BackupEntries().  I learned about the other problems as time went by, and now, m_BackupEntries does much more...

Anyway, I'm sorry that this sloppy method is causing trouble for you.  I don't have a good solution yet for keeping API index and m_BackupEntries indexes in sync, but if you don't care about accessibility purposes, you could just drop API index completely and only use your own indices.  That might be easiest.

I am working on fixing some high-DPI issues with the combo box right now, but then I will look at improving some of the list management code.


TeamCity MonoGame :: Develop (Win) [Build 3.3.0.1964](http://teamcity.monogame.net/viewLog.html?buildId=5067&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 366, ignored: 6 Build time: 00:16:09


Julian - please delete these files and put the answers into the right file (eg 1.rb/2.rb etc) below everyone else's and make a new pull request, because otherwise I have to spend ages fixing everything by hand!


This one is not needed, isn't it?


I like it.


Right, I think the code changes themselves are fine, and after reviewing all the other custom changes in the file I'm not too concerned either since they don't touch much of the original code. Just something for us to keep in mind in case we start touching up a lot more of the code.