TeamCity libpalaso :: palaso-precise64-master Continuous [Build 2.6.390.23d721bf883c28548cc483d1fb6d742882dfaa4f](http://build.palaso.org/viewLog.html?buildId=58731&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3187, ignored: 112 Build time: 00:06:08


Ok. Well, I just cherry-picked and it came along with that, so I couldn't do nothing about it.


Turns out pep8 does have an opinion on this:
> Be consistent in return statements. Either all return statements in a function should return an expression, or none of them should. If any return statement returns an expression, any return statements where no value is returned should explicitly state this as return None , and an explicit return statement should be present at the end of the function (if reachable).

> Yes:
```
def foo(x):
    if x >= 0:
        return math.sqrt(x)
    else:
        return None
```
```
def bar(x):
    if x < 0:
        return None
    return math.sqrt(x)
```
> No:
```
def foo(x):
    if x >= 0:
        return math.sqrt(x)
```
```
def bar(x):
    if x < 0:
        return
    return math.sqrt(x)
```

Seems pretty clear here `return None` is the right choice.


Yes, please stop doing parenthesized ifs.


d


We could do it this way
    @expert_genres = Hash[ (@profile_expert.genres.pluck(:id)).zip(@profile_expert.genres.pluck(:name))]


晓婷什么相貌，什么身段？坐等描写


This is pretty much the opposite of the xml problem, this complies to the
|[ ]| legacy gtk-doc syntax for code blocks, but doesn't parse github style
code blocks, so I don't think that will be compliant with CommonMark :)

On Wed, Jun 17, 2015 at 11:18 PM, Jasper St. Pierre <
notifications@github.com> wrote:

> I'd like to see some testcases for this, especially with respect to
> CommonMark compliance.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/MathieuDuponchelle/gobject-introspection/commit/a5e14f18778ad40413b2dc5a18aeea03fa948c94#commitcomment-11734581>
> .
>


It's very convenient though :)

On Wed, Jun 17, 2015 at 11:20 PM, Jasper St. Pierre <
notifications@github.com> wrote:

> Nope. Sections files are not XML. You can't do that.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/MathieuDuponchelle/gobject-introspection/commit/a6f2f969d4479d4429a4b89c5443c25c289e24fa#commitcomment-11734605>
> .
>


OK, will do

On Wed, Jun 17, 2015 at 11:23 PM, Jasper St. Pierre <
notifications@github.com> wrote:

> Please do this with each commit that touches things so we don't break
> bisect.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/MathieuDuponchelle/gobject-introspection/commit/17262d3ec3f4c6d6c3f06859dd2642ab5e627fff#commitcomment-11734656>
> .
>


@nhpatt @jmnavarro how is authentication done for microsoft push notification service?


Well if that was a commit hook, I wouldn't have been able to commit these
pep8 violations in the first place :)

On Wed, Jun 17, 2015 at 11:23 PM, Jasper St. Pierre <
notifications@github.com> wrote:

> Meh.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/MathieuDuponchelle/gobject-introspection/commit/723b85684bff9d589130a62d19a9db460bc5d889#commitcomment-11734659>
> .
>


@neuront  酱紫怎么样


Cool

On Wed, Jun 17, 2015 at 11:24 PM, Jasper St. Pierre <
notifications@github.com> wrote:

> Pushed this directly.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/MathieuDuponchelle/gobject-introspection/commit/418940dd1c0d3756af6a4562c88ba97e87a726c4#commitcomment-11734680>
> .
>


No tengo idea, wey. De repente estaba en los files trackeados, pero no sé de dónde salió.


Ack

On Wed, Jun 17, 2015 at 11:24 PM, Jasper St. Pierre <
notifications@github.com> wrote:

> Pushed this direcftly.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/MathieuDuponchelle/gobject-introspection/commit/c27669ca1f6d0eca6420170126675c1db7b7bf12#commitcomment-11734686>
> .
>


How about using BaseTestCase class? The point I want to extend from that class is that I can use data from the typicalDataBundle.json for conveniently building the input for the init() function.


OK, will do

On Wed, Jun 17, 2015 at 11:27 PM, Jasper St. Pierre <
notifications@github.com> wrote:

> Needs a test.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/MathieuDuponchelle/gobject-introspection/commit/c2fa4a56ea4e55adcb093a7d4832c3a2028693fc#commitcomment-11734727>
> .
>


No sé de dónde salieron, wey


No sé


@pwpiwi - latest changes are much more stable/reliable on the raw command side!  works nice with the new "hf search" and "hf 14b reader" commands


Let me guess. you were high when you coded this?


#####Q4我们再给出一个方式，用标括号的方式确定优先级。

结果如下所示：

    resultVar += (var1 + var2) + (var3 % (var4 / var5));

或

    resultVar += (var1 + var2) + ((var3 % var4) / var5);


Yes, I think BaseTestCase should be extended for the page data tests for now. If there are shared functions between the page data tests, then we can make a BasePageDataTest.


Did you get the form persistence working for this?


We could have a previous link yes, my target isn't yelp so I don't really have an opinion here, setting next_section directly, that will just pretty much reverse the awkwardness, as it means we would need to create it in advance, except if we do a second pass but that's not much better


rax retry tests


typicalDataBundle needs the server to run, so I don't think it's a wise choice...


I wasn't aware of that idiom, I'll have a look thanks


Only if BaseComponentTestCase is used and removeAndRestoreTypicalDataInDatastore is run, otherwise the data created from the typicalDataBundle isn't persisted to the database?


Thanks a lot Eric. Sorry for the delay, I was supposed to add this but you did it :-)


`bundle` needs to exist in the `.gitignore`. Notice how laggy this page is even trying to load everything haha.
![screen shot 2015-06-17 at 8 57 40 pm](https://cloud.githubusercontent.com/assets/3409645/8223322/86b50a8a-1533-11e5-804b-5644993dfea6.png)
This is a big problem. I would delete remove `bundle` from your branch, add it to `.gitignore` and re-push.


This kind of customized assertions should use appropriate assertion instead of assert_true.
E.g. assert_greater_than. So we can easily understand what happens there in failure cases.


BaseComponentTestCase sets up the gaeSimulation, but not BaseTestCase


Looks fine for me. I've also repushed https://github.com/mpc-hc/mpc-hc/compare/develop...kasper93:develop


Spawners make bombs spawn? And where are all the spiders gone?


lolz. great commit. :ok_woman:


https://travis-ci.org/XQF/xqf/builds/67424608

The error was "did not find expected alphabetic or numeric character while scanning an alias at line 8 column 180".

it seems it does not like the `\n`


@Grz3s
Please give me the full correct updated line, so I can update.
Thanks


@maya just to clarify, do you mean yellow outline on all the filters? Currently if they have a filter set it has the yellow outline. Are you suggesting that instead of that have a blue outline if they are focused on a filter?


I found the same bug through hours of getting odd results. My only suggestion is that rendered should be set (on line 274) equal to the number of suggestions added through the slice:

rendered += (that.limit - rendered)


hey if u aren't going to put anything inside this or edit yet, just leave it out.


Because I need to pass an int to, for example, `getHostName()`, but a String to `ConnectionInfo.toJson()`. I will change `ConnectionInfo.toJson()` to take an int.


Možná jste své připojení ztratili na poslední pařbě. XD Tahle hláška zní docela divně, spíš bych to přepsal na "Zkontrolujte, zda vám funguje připojení k Internetu."


Leave out the advertisements?

Justin

> On Jun 18, 2015, at 2:02 PM, Bobbilee <notifications@github.com> wrote:
>
> hey if u aren't going to put anything inside this or edit yet, just leave it out.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/jgrant29/bloccit/commit/267f133727a52d4ffe719f41fb2a51a20e973658#commitcomment-11755234>.
>


Ook


@LisaStucky "Amount" would be a better name for this parameters; it's the number of credits being redeemed.


no leave out the new and edit methods because u aren't using them yet


91958608DOASEffectOnZoneSizing (wfbuhl) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1124 of 1131 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.38%25-yellow.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-69.73%25-green.svg)


[Defined here](https://github.com/BranchMetrics/Branch-iOS-SDK/blob/master/Branch-SDK/Branch-SDK/Branch.h#L426)


```sql
-- update
UPDATE creature_ai_scripts SET event_param2 = 20, action1_param2 = 1 WHERE id = 3109801;
```


Konge takk

Olis

> Den 18. jun. 2015 kl. 21.02 skrev Stian Selland <notifications@github.com>:
>
> Noen tegn er reserverte, og kan ikke brukes direkte i html koden din. F.eks større enn og mindre enn kan du ikke skrive direkte inn i koden. Da vil de bli tolket som start og slutt på en html tag. Det samme gjelder 'og' tegnet.
>
> Det du må gjøre isteden for er å bruke en spesiell kode istedenfor. Du finner dem her: http://www.w3schools.com/html/html_entities.asp
>
> Istedenfor 'og' tegnet, så skriver du &
>
> —
> Reply to this email directly or view it on GitHub.
>


Higuita, a asking for this part of code: the strings "C" between the storage class `extern` and type keyword `int` is to denote what? Some type of identifier?


Actually I think this is more our problem than yours... anyone using the full library is using it because they want to play tunes... so there is no cost for you to use play/pauseNote as "init" functions.


Opened a PR to fix the directory creation problem here - https://github.com/galaxyproject/galaxy/pull/371.

I think fetching data dynamically from git does work - it is lazy though so it won't happen until you need it.


Fyi: I reverted this commit on June 13 and re-added it that day with some fixes under the same commit title.


Be careful .the OTA app uses the board name sometimes
For example here i9300 uses board name "m0"
If I don't use m0 then the OTA won't recognize the update
Make sure you check your ro.product.device in build prop or  device makefile


I don't think we want to add the errors.inspect here. The message without that is good enough. The errors will be composed appropriately but he Validation exception, passing it into the validation handler.


Consider to enableOffensiveAbilities default to True instead False if away People get updates and are unaware of change.


I see.  Thank you!  Does everything else look good?

I can't view the posts or advertisements on heroku.


Do you realize that your alignment attempt is a complete waste for anyone who doesn't have their tab width set to the same thing as you?  It's fine if you want lines indented with tabs, but using tabs for alignment after indentation is nonsense.


What @ascriptmaster said. Also, all paralysis-inducing Status moves were affected by immunities in Gen 2 and 3, but it doesn't matter for Stun Spore, and Glare is already implemented properly.


DesignDaysGridView (evanweaver) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

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


oh wait, that's not the handler, that's the exception. nevermind.


This should be `fw.close()` (with the empty parens), since it's a side-effecting method call


@advertisement not plural


Reborn - bug edition :)


this should be not plural


This breaks capybara for me with rspec and Rails 4.2.1 and 4.2.2.


Do you have VS2013 update 4?


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (351 of 351 tests passed)

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


"Cleaned up syntax"
There is so much more to do.
MIxed indentation, trailing whitespace... :P


because u need to migrate database changes on heroku env


Thanks. I'll take care of that tomorrow.


There's no safe numbers to choose, except for getpid(), and setpgid(0,0) is the same as setpgid(getpid(), getpid())


FacilityTabGridView (evanweaver) - x86_64-MacOS-10.9-clang: Tests Failed

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


ACK!


Hi Lucas,

I am Rodrigo from UFPE (Recife), Paulo's student.
I am sorry to contact you from here, but your old email no longer works. Could you send me your current e-mail or contact me at: rodrigocaa  at gmail dot com


develop (lefticus) - i386-Windows-8.1-VisualStudio-14: OK (351 of 351 tests passed)

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


ja też nwm


2 new tests and || vs |


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-LibC++: OK (351 of 351 tests passed)

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


@nh13 tested it, it works nicely this way. Where it says program.toString() it prints "CollectGcBiasMetrics".


Nope.  There originally was, but it got refactored out.


Tá show o seu repositório Bianca!! Mandando bem!! Parabéns!!


Would be cleaner to make this the condition of the while loop, rather than saying "while true"


fixed


What I meant was `int[] allWorkers = Ints.toArray(config.getWorkerIds())`. Not sure if I'm missing something.


Fixed this.  Thank you


Tenta utilizar a callback "final" do mapa:

Invés de:

```
$('ul[class=loader]').attr('hidden', 'true');
```

Tenta:

```
google.maps.event.addListener(map, 'idle', function() {
  $('ul[class=loader]').attr('hidden', 'true');
});
```


I ran the rake db:migrate and posts show up but advertisements do not

https://serene-reef-1873.herokuapp.com/advertisement


Fixed.


LGTM


 - __style__: [src/chaiscript_stdlib.cpp:18](../blob/7adbc11869b5a637f3699a1942f316cff41370a8/src/chaiscript_stdlib.cpp#L18) The function 'create_chaiscript_module_chaiscript_stdlib' is never used. (unusedFunction)
 - __style__: [src/main.cpp:169](../blob/7adbc11869b5a637f3699a1942f316cff41370a8/src/main.cpp#L169) The function 'get_eval_error' is never used. (unusedFunction)
 - __style__: [src/main.cpp:140](../blob/7adbc11869b5a637f3699a1942f316cff41370a8/src/main.cpp#L140) The function 'help' is never used. (unusedFunction)
 - __style__: [src/main.cpp:216](../blob/7adbc11869b5a637f3699a1942f316cff41370a8/src/main.cpp#L216) The function 'myexit' is never used. (unusedFunction)
 - __style__: [src/main.cpp:158](../blob/7adbc11869b5a637f3699a1942f316cff41370a8/src/main.cpp#L158) The function 'throws_exception' is never used. (unusedFunction)


Advertisement link works in local environment but not production


## Submission for coursera project (Data Scienctist's Toobox)


They aren't all dead and gone.


Hi mmolhoek,

I am not able to see these changes in the latest version, This functionality is much useful.

Thanks,
Vardhan


This is really, really good!


They are gone from my code, but other code uses them.


I have to rollback Merge broke build.

Results :
Tests in error:
  LoggingStoreControllerTest.testLogRestController:63 » HttpClientError 400 Bad ...
Tests run: 9, Failures: 0, Errors: 1, Skipped: 0
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------


DesignDaysGridView (evanweaver) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Don't remember the number, but it's the currently latest VS 2013 update. Either 4 or 5.


> For gTLDs, you'll be able to update the domain contacts yourself and save the new settings at the bottom of the page.

On what page? Should we add a screenshot here?


develop (lefticus) - Win64-Windows-8.1-VisualStudio-14: OK (351 of 351 tests passed)

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


it seems this code compile `gamesxml2c` before `xqf` but does not generate the `games.c` file before compiling `xqf`.


Tried it out:
Native resolution is fixed :)

Unfortunately this change seems to have broken scaled resolutions - at 2x native and above the right half of the screen is filled with 2 duplicates of the far left side of the screen.
[GS dump](https://www.mediafire.com/?a8wxmx9tjfbbp02)
Occurs in all Champions and Baldur's Gate: Dark Alliance games.


@Hermanverschooten it's probably not this commit, can you give us more information?.


.merge grr .join is in Addressable xD


`if (jvId)` is fine.  No need for the explicit cast to bool in this context.


No need for cast to bool (three places).


Indentation is all wrong in this file.


Won't this brake rails? https://github.com/rails/rails/blob/v4.2.2/actionpack/lib/action_dispatch/http/request.rb#L338


;) yw


Please put a space before and after on the braces. You should read the rails contributing guide which has tips on the style we like.


It does, building worked for me.


Files should have an extra new line at the end.


Screenshot of scanner.


@ErikSArnold @baraabourghli you're correct, it was unintentional. I wanted to move the file, and did so, but I had put the actual creds in the file locally in order to test :disappointed: I'm going to purge this commit from Github now, so that the info is no longer publicly available. I'll also change the account password...


test


Edit it inline and I'll commit it.

On Thu, Jun 18, 2015 at 1:35 PM, Ricky Elrod <notifications@github.com>
wrote:

> Same here, stackage only has 0.12.1.6 right now.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dmjio/snaplet-auth-acid/commit/db348a4ccc626a4dafe7cbb4ced0726ae92fd317#commitcomment-11753590>
> .
>

--
Cell: 1.630.740.8204


@elsapeng delete this branch from remote. You have the secrets in history! Everybody can see this!


this is not rst


It looks like there's also some white space after end here. Please make sure your editor is setup to strip white space.


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-AddressSanitizer: OK (351 of 351 tests passed)

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


How can we get this functionality in the latest  version? If you dont mine can you please checkin the changes one more time..

Thanks in Advance,
Vardhan


Looks good.


```
[illwieckz@ubuntu:/home/illwieckz/dev/xqf-skybon/build] cmake ± cmake -DWITH_QSTAT=/usr/bin/quakestat -DCMAKE_INSTALL_PREFIX=/usr ..
-- The C compiler identification is GNU 4.9.2
-- The CXX compiler identification is GNU 4.9.2
-- Check for working C compiler: /usr/bin/cc
-- Check for working C compiler: /usr/bin/cc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working CXX compiler: /usr/bin/c++
-- Check for working CXX compiler: /usr/bin/c++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Found PkgConfig: /usr/bin/pkg-config (found version "0.28")
-- Found Gettext: /usr/bin/msgmerge (found version "0.19.2")
-- checking for module 'minizip'
--   found minizip, version 1.2.8
-- checking for module 'libxml-2.0'
--   found libxml-2.0, version 2.9.2
-- checking for module 'gtk+-2.0'
--   found gtk+-2.0, version 2.24.27
-- Configuring done
-- Generating done
-- Build files have been written to: /home/illwieckz/dev/xqf-skybon/build
[illwieckz@ubuntu:/home/illwieckz/dev/xqf-skybon/build] cmake ± make
Scanning dependencies of target gamesxml2c
[  2%] Building C object CMakeFiles/gamesxml2c.dir/src/gamesxml2c.c.o
Linking C executable gamesxml2c
[  2%] Built target gamesxml2c
Scanning dependencies of target translation
............................................... terminé.
............................................ terminé.
.................................................. terminé.
................................................ terminé.
.............................................. terminé.
................................................ terminé.
................................................ terminé.
............................................ terminé.
[  2%] Built target translation
Scanning dependencies of target xqf
[  4%] Building C object CMakeFiles/xqf.dir/src/addmaster.c.o
[  7%] Building C object CMakeFiles/xqf.dir/src/addserver.c.o
[  9%] Building C object CMakeFiles/xqf.dir/src/config.c.o
[ 12%] Building C object CMakeFiles/xqf.dir/src/country-filter.c.o
[ 14%] Building C object CMakeFiles/xqf.dir/src/debug.c.o
[ 17%] Building C object CMakeFiles/xqf.dir/src/dialogs.c.o
[ 19%] Building C object CMakeFiles/xqf.dir/src/dns.c.o
[ 21%] Building C object CMakeFiles/xqf.dir/src/filter.c.o
[ 24%] Building C object CMakeFiles/xqf.dir/src/flt-player.c.o
[ 26%] Building C object CMakeFiles/xqf.dir/src/game.c.o
/home/illwieckz/dev/xqf-skybon/src/game.c:186:23: fatal error: /home/illwieckz/dev/xqf-skybon/build/games.c: Aucun fichier ou dossier de ce type
 #include GAMES_INCLUDE
                       ^
compilation terminated.
CMakeFiles/xqf.dir/build.make:261: recipe for target 'CMakeFiles/xqf.dir/src/game.c.o' failed
make[2]: *** [CMakeFiles/xqf.dir/src/game.c.o] Error 1
CMakeFiles/Makefile2:130: recipe for target 'CMakeFiles/xqf.dir/all' failed
make[1]: *** [CMakeFiles/xqf.dir/all] Error 2
Makefile:117: recipe for target 'all' failed
make: *** [all] Error 2
```


New hash syntax


bits like these still need some love though


Inline the message variable.


develop (DavidGoldwasser) - Win64-Windows-7-VisualStudio-12: OK (2190 of 2213 tests passed)

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


what's the reason for? I just wanted to eliminate useless config


Inline the options variable.


We should find a better place to set a secret.


I'm a bit confused. Please let me know what this does. i think I'm merging a change in the main scipy fork into my branch that I'm working on. The purpose for me is to make sure I am working on the actual developments version and not something that's too old.


Hum, I'm not convinced...

In this cas, you have to add also :
```lua
 PLAYER::_0x2975C866E6713290 == PLAYER::_GET_AIMED_ENTITY
```
```lua
 UI::_0x202709F4C58A0424 == UI::_SET_NOTIFICATION_TEXT_ENTRY
```

See commit 9d1429b3afb2dff63d942b6662903c6550f323e0


Actually is here. Rails will brake: https://github.com/rails/rails/blob/v4.2.2/actionpack/actionpack.gemspec#L24

Sure this is not rack's fault, but yeah, I've just run over this.

Will open an issue there.


_0x2975C866E6713290 is still there :)
A few natives still have their old _0x123123 for backwards compatibility:)


I've left some comments :smile: In the future just relink your pr.

Are you sure the claims are sufficiently covered? That file seems meager to me.


BTW: The way you've setup web controllers looks good. I'ld prefer to use Spring services instead of repositories from a controller ... Even your app works today, it may break with future release ... on not


Lol. We are not using this templates with Vue so I think this is fixed already with on the vue branch, thanks James!


Awesome! :+1:


develop (lefticus) - i386-Windows-8.1-VisualStudio-12: OK (350 of 351 tests passed)

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


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.6-ThreadSanitizer: OK (351 of 351 tests passed)

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


Yeah, I suppose. :smiley:


rax retry tests


Help with what?


yeah, and there's a pull request https://github.com/rails/rails/pull/19665/files fixed this and merged into master, but it hasn't been released yet.


deixei tudo em um unico protocolo (https) pq estava aparecendo alertas ao analisar o codigo pelo chrome, pois havia locais referenciando o endereço do blog como http e em outros https


Nice!


Не понял суть


Thanks.  I've been going through the doc on building extensible software today (progress is kinda slow but I think I'm learning it pretty well), so I'll tackle those changes on Monday when I come back in.  Most of them, (fonts, colors, etc) should be really quick as I have them in their own classes anyway.  For the "client request example" thing I'll just mess with it a bit on Monday and see what I can do.  I'll also update those changes on GitHub and make some comments there.  Thanks for the feedback.


It's a harmless warning, at most.


"C" should


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (351 of 351 tests passed)

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


Ciao,

se avete bisogno di una mano io sono disponibile. Seguo spesso hookii anche se non commento, almeno posso aiutare un po' qui.

Tempo fa ho usato brevemente Elasticsearch, anch'esso basato su Lucene.. Solr non dovrebbe essere troppo diverso.
In caso potete contattarmi a questo username sulla mail di google.

Jacopo


I don't think we use them either TBH. We should probably kill 'em. (I just copied from Desktop's old schema)


@SaptakS  can you please verify the working of the code on your system?  I have tested the code unit-wise and it was working fine. Merge with master if it works fine.


You're going to want the issue table instance to have the data for the current issue.  An empty table object isn't going to be of much use to us in the listener :laughing:


@FooBarWidget but can you just make using of that ev-related modifications **optional** (to keep ability to build against system libev anyway)?

As I read your post in libev maillist, you're doing that in debug purposes, but in non-debug cases, I'd bet, it is unneeded to force user to compile against bundled libev every time, isn't it?


tac is not part of the standard base unix commands, the implementation there is also huge and slow as it uses stdio and malloc and allocates big fat buffers.


This should be rust-channels


lgtm


Can you add
// Portions Copyright [2015] [C2B2 Consulting Limited]

after the Oracle license comment at the head of the file. Then I'll merge. i forgot this on the last merge.


Minor simplification:
```
        else
        {
            amount = sle->getFieldAmount (sfBalance);
            if (account > issuer)
            {
                // Put balance in account terms.
                amount.negate();
            }
            amount.setIssuer (issuer);
        }
```


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.6: OK (351 of 351 tests passed)

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


Type consistency would be great, except we don't know the size of the arrays until load_data_set(...) is called. So, we'd be initializing them with a completely arbitrary shape, which seems more confusing than just an empty list.


Why not throw a exception on case c) ?


After this goes in, I think you should submit a new PR with your improvements!


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.67: OK (0 of 0 tests passed)

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


Published to http://ui.edev.d2l.com/85_b77f0f72e44771d7687972d4861b7dbdd48fb307


saw approval from mrallen1
at https://github.com/basho/bitcask/commit/bdfdeac256567c4e7ce596d0886c37f0fd4deb84


merging basho/bitcask/bugfix/file-delete-blocking = bdfdeac2 into borshop-integration-216-bugfix/file-delete-blocking


basho/bitcask/bugfix/file-delete-blocking = bdfdeac2 merged ok, testing candidate = 1b5ad684


develop (lefticus) - Win64-Windows-8.1-VisualStudio-12: OK (350 of 351 tests passed)

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


Published to http://ui.edev.d2l.com/86_b77f0f72e44771d7687972d4861b7dbdd48fb307


No, indeed it wasn't this commit. see issue #896, it is the commit that adds the second parameter to parse_query.


The location header can be a relative URI or a full URI. So maybe it needs to be parsed as an uri first, and then merge only the path attribute?


I try to merge it... it broke the build i rollback and reopen the PR.
Thanks for the feedback. I will add also a Spring service layer to the
logging-sore.

2015-06-18 22:35 GMT+02:00 Heiko Scherrer <notifications@github.com>:

> BTW: The way you've setup web controllers looks good. I'ld prefer to use
> Spring services instead of repositories from a controller ... Even your app
> works today, it may break with future release ... on not
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/marzelwidmer/logging-store/commit/2ea11cca19c28af0793e58edcfe5842c54dabe46#commitcomment-11755819>
> .
>


Ohhh, I didn't now that. Awesome!


That looks very much like comonad's `duplicate: f a -> f (f a)`. Sadly results has type `f a` otherwise it could be `copure` :)


maybe create (yet another?) shell script that's called setup-web-e2e.sh or something, with sections for both Travis and Shippable? then we get most of this out of both .travis.yml and shippable.yml.

You can check whether `$TRAVIS` or `$SHIPPABLE` is set in the environment to choose which to set up.


Do you write safeguarding checks such as these just for more informative failures?


A few comments I neglected in the initial checkin. Naming conventions are really important with QML. I know the name had to start with a capital letter before it would load into the engine properly. id tags cannot start with capital letters either.


Let's try to have default of null so it does not have to be provided. If it is null, sturges will be used. What do you think @RPCMoritz ?


 -- what's the reason for? I just wanted to eliminate useless config --
you mean the annotations on the entity ?  with the col. names ?

2015-06-18 22:35 GMT+02:00 Heiko Scherrer <notifications@github.com>:

> BTW: The way you've setup web controllers looks good. I'ld prefer to use
> Spring services instead of repositories from a controller ... Even your app
> works today, it may break with future release ... on not
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/marzelwidmer/logging-store/commit/2ea11cca19c28af0793e58edcfe5842c54dabe46#commitcomment-11755819>
> .
>


Is this really helpful?


Niezły commit, wyglada dosc podobnie do tego co juz wstawilem - najwidoczniej mielismy podobna koncepcje co do internacjonalizacji LOL co wiecej z racji tego ze ma dosc podobna nazwe do poprzedniego widac ze jest to jego naturalna kontynuacja LOL


See https://github.com/wpscanteam/wpscan/pull/829/files#r31443814


Oh! Right, I was not careful enough. Thanks!


Scary! But OK.


Yuck yuck yuck yuck. Anyone want to write a macro that mangles C++ names? ;)

(Doesn't block this from landing of course)


Hypnosis affects Dark-types...


Is the `?` needed or should this be covered by the `<Loadable>` guard (and an error otherwise)?


`assume*` allows graceful failure of tests that aren't configured. So, this will allow the test suite to pass (with warnings) if the group configuration isn't present. Compared to hitting a hard failure trying to add someone to an invalid group.

Integration tests are something I'm still working on best practices for. They are beneficial, but most require a working external system to execute and that is something that could be really flaky.


Definitely broke our rails 4.2.2. :)


indentation


Still need these. :( They're used as a part of receive_message() to determine what message was received.


Would be nice if we namespaced these names somehow, but that was preexisting so it doesn't have to be done now.


No, sorry, I will not do that. Removing the debugging feature severely restricts our ability to troubleshoot unanticipated issues. Everything is fine when there are no issues, but when there are issues, we can't just tell users "yeah, recompile Passenger against our forked libev so that we can figure out what's going on". I do not consider the ability to build against upstream libev to be a strong enough case to remove the debugging features.


File an issue?


None of these code blocks are correct. You need a blank line before the code begins.


Hello,

I'm not sure it is the right place to report an issue but I couldn't find an issue tracker. The problem I experience is on OS X 10.10 with the current released version: it seems there are some issues when reading some audio files like for instance [this file](http://feeds.soundcloud.com/stream/210794247-reboundcast-episode39.mp3). The times displayed in the control bar are bugging a lot, sometimes displaying a negative value on both sides, sometimes they totally stop incrementing/decrementing.


LGTM


What about these 2,0 entries?


`$issuetable->load(array('issue_number' => $this->hookData->issue->number));`

`$id` is the comment ID, not the issue ID.  Would need the same change in the error message, and the triggerEvent should probably be moved inside the try/catch since it depends on the table object being properly loaded.


can we remove the permutation step for simplicity? (train file could just be permuted before giving it to dissolve, if necessary)


Oh you are absolutely right @xtine! I just thought it's distracting seeing the yellow on first load, but I realize that technically that filter is being set. Here is a screenshot so others can view:
![screen shot 2015-06-18 at 1 56 41 pm](https://cloud.githubusercontent.com/assets/5249443/8242094/06411df0-15c2-11e5-8c69-c0434c465313.png)

But I do think the slider control button should have a focus state as well.


all tests pass:
success: http://buildbot.bos1/builders/test-bitcask/builds/551


Hypnosis is sleep-inducing, not paralysis or poison-inducing.

Basically paralysis-inducing (Gen 3 and below), Thunder Wave, and poison-inducing are the only exceptions to status-inflicting moves ignoring immunity.


Someone on the chat suggested that the script could put everyone on an empty lane at x00 levels so they aren't doing damage to bosses?
Might not be ideal because it exclusively work if people have warps(?), but I thought I might as well pass on the idea.


fixed, thanks...


These shouldn't just be `error` or `notice`. Mind resetting them to the original values in 30990ed5cd69a655a2cf8a892db0ec47d61c4e6c?


move to util class


See openstack/nova@61497bf6c4910eedb64494599f7dde25f2500629 for details why this was removed.


It was [this commit](https://github.com/Nyoho/rack/commit/6af5f928f3a860e9bb74fc0bdd83b30be9d722a9) actually. See PR https://github.com/rack/rack/pull/781. I have 2 PRs open now that resolves this and the tests still pass.


@chewbranca I think I did. Those were errors originally. Check the changed files on the PR, you shouldn't see these lines.


Comented 'code', remove?


Oh, I missed one, sorry


TeamCity NEventStore :: MongoDB :: Build [Build 23](http://build.prxm.it:8811/viewLog.html?buildId=3063&buildTypeId=nes_mongo_build&guest=1) outcome was **FAILURE**
Summary: Error running GitVersion Build time: 00:00:54


TeamCity NEventStore :: MongoDB :: Build [Build 24](http://build.prxm.it:8811/viewLog.html?buildId=3065&buildTypeId=nes_mongo_build&guest=1) outcome was **FAILURE**
Summary: Error running GitVersion Build time: 00:00:05


Thanks for analyzing this issue.

The commit itself is not entirely correct, as the conf_get_bool_value() lines moved are needed several lines above (e.g. the show_tray_icon flag is used in several conditions). Therefore I believe the conf_get_bool() lines must not be moved.

Setting the pane positions at a later time as you did should be ok.


You have only added a single WinSCPnet.dll file. 7plus is distributed both as 32 bit and as 64 bit binary, so we need both versions of this file as far as I know?


Consiglio di creare i seguenti script:

  - uno dedicato solo alla memorizzazione delle immagini;
  - uno per il tracking del soggetto, memorizzando le coordinate (x,y), l'altezza e l'ID in un file di testo;
  - uno per poter popolare il DB partendo dal file di testo.


Nice scenario, but not possible, right? IMHO, sometimes it's good to leave potential NPEs - in future development, it's easier to handle them than being confused with some first-time-displayed pages. This is just my opinion, I agree with the benefit of fixing coverity findings :)


Okay, let’s go with this version! Please publish the new revision whenever you’re ready.


This is required due to the new [container-based infrastructure of Travis CI](http://docs.travis-ci.com/user/workers/container-based-infrastructure/). Both `travis-build.bash` and `travis-run.sh` use the `sudo` command, which is only supported on [Travis' standard infrastructure](http://docs.travis-ci.com/user/workers/standard-infrastructure/).


Except now it won't work on a Mac? Don't they have a different end-of-line marker again?

Maybe we should split on /[\r\n]+/ to cover any combination of linefeeds and carriage returns.


at 2015-06-19 05:05:30, finished ver 0.0.1, deleted a lot of unused information when install keystone+swift.


The formatting of botnames.txt was changed to "\n" from "\r\n" when downloaded from github, causing windows users to parse the names incorrectly (EOL for windows is "\r\n"). So i changed it so that it split "\n" first, then parse out "\r" if its still there.


so cool!


Should we document the benefits of a unit spec timeout here? To ease upstreaming? I did not uptreamed the per-example timeouts yet so it might help me on upstreaming to have a short explanation here.


I haven't removed it. It is in the README.


Here it must be manual replacement though.


Oops! That was a cut & paste error. Sorry about that. I'll fix it.


empty `numpy.array([])`


This doesn't make much sense to me. If this is a struct it picks off the first element type. Surely that's not what is intended.


There's an issue for that already https://github.com/rails/rails/issues/20624 :)


The main reason for this is to allow mutant to mutate code which can potentially loop forever; like that new while loop. I don't know if it should be upstreamed with everything else.


Is it possible that `fileOutputStream` remains null, thus causing NPE here, which is not caught here? Consider adding `if(fileOutputStream != null) { ... }` around close statement.


This actually removed support for the base_url...


This causes the build to fail. https://travis-ci.org/angular-ui/ng-grid/jobs/67236202
The error is
```
line 189  col 32  eval can be harmful.
```


Actually, that's not what I meant. This doesn't seem right if jv is a jl_pvalue_llvmt.


:+1: awesome, thanks


yw :)

> On 18 Jun 2015, at 20:10, Lieuwe Rooijakkers <notifications@github.com> wrote:
>
> fix typo in /me
>
> tysfm @tomsmeding
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/lieuwex/lieuwex.me/commit/89c56a3a88c5a8dc956fc884ae3597fbf2699cdc


Some more nits:

'''bash

And the text above does not fit to the `cat` command to execute.


What is it actually? :) I thought that would be some apple icon, but scripts/lcmc-logo-icon.icns contains some instructions?


hey @agsh, doing this will stop a single-source camera from having a default PTZ profile. Maybe add a check to see if `idx === 0 && this.activeSources.length === 1`?


`Ledger::read` is checking the key type (`k.check(*sle)`) after it makes the SLE, but `CachedView::read` is not.


Actually, I mean the opposite:
you can (just as now) by default compile against bundled libev/libuv. But when user **explicitly** set `USE_VENDORED_LIBEV=no` — then do not call patched debug-only `ev_fd_stuff` in the code.


There is currently an issue open for this. I'll follow up there later. Short answer, it doesn't matter if `scroll` or `auto` is in there, the scroll bar doesn't hide on some browsers.


bxcbcbc


xvcxv


I've seen some files with just \r at the end of lines. I just committed a change which accepts any combination of \r and \n, and doesn't mind whether the file ends with a line terminator or not. So now it will accept pretty much any kind of text format.


that shouldn't happen since if jv is jl_pvalue_llvmt, then ty == jl_pvalue_llvmt


Could be shrank very well if there were a byte-by-byte file IO paradigm in Unix libraries. Sadly only memory-mapped and tape-like paradigms are supported.


this is not working later on with the + client


Nice, thanks.


No, then vt == jl_pvalue_llvmt. `ty` denotes the type you want the resulting value to have.


Fixed on 5c25d40ccba9615e909089ea7985fa7d0002ec4e.


90664130_AlternativeModelPAT (macumber) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

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


The "unhappy" path here is `fid client.authenticated` blowing up with `Excon::Errors::Unauthenticated`, but that's already covered here: https://github.com/heroku/identity/commit/3e57dd628e27f86f8581c1a191cfa4e6b97b6581


Comment is not necessarily true - ie. if the `View` is a `CachedView`


Thanks! Could you update it for the rest of lessons too?


And fixed a bunch of spelling mistakes


fast-forwarding develop to borshop-integration-216-bugfix/file-delete-blocking = 1b5ad684


Oh..


Yeah, I missed extension part and directory path too. Perfect!


Is `assertEquals(user, foundUser)` really a necessary or helpful test?


@zxdgoal, this line is erring on `master`, can you take a look at it when you have a chance?

I wonder if something was missed during any source formatting.

/cc @jonmak08


It should be upstreamed. Since its a PITA to upstream currently where I have to rebase in the mutant setup before actually applying the "to be upstreamed" patch. Than mutation cover the diff after I adapter to upstream code changes (that can be significant), and than rebase the mutant setup away again.

When I *can* upstream the mutant setup, I can upstream all of our other patches faster. //cc @JDUtil Chances you accept the mutant setup? It'll not be active by default on CI, only these timeouts + gem dependency are needed. No other semantic changes to your build will be done.

Context (for @JDUtil), all of our changes are for quality and also "deterministic development flow reasons" mutation covered via [mutant](https://github.com/mbj/mutant). I realize I'm tooting my own horn here, mutant is my project, so naturally I'm biased towards its use. Nevertheless some of the more complex changes we did to spree (both upstreamed and still in our fork) could never have been realized without following the process of:

1. Mutation covering the existing code / functionallity to have a correct baseline specification before a complex change. Sprees specs are in many examples not covering the current semantics very well.
2. Write a spec for the changed semantics, sometimes simplifying the specified semantics from 1. or add new specification
3. Write the code
4. Iterate 2. / 3. guarded by mutant to reduce the code / specification mismatch to near 0% (nothing can prove its 0% but AFAIK you do not get closer to 0% on ruby without mutation testing)


Improved FileEvent Performance


.get doesn't need to be called on a future to trigger callbacks. You can just addCallback like is done below and any error handling will be done as normal error handling for any other error-result is done.


I don't think it really is, I left it there because it was in the original tests, but I'm not seeing much use in it.


This seems to be one of several new files which was added with invalid tags.  I have corrected them.


they aren't supposed to be particularly different though. note too though that a `jl_pvalue_llvmt` is a PointerType, not an Aggregate type, so this test will not be active


@danieleds Oops. Nice catch. I forgot about that there.


I am pretty sure `/tmp` does not exist on Windows. You may want to check how Syntastic does this.


Oh yes, I see the configuration below now. I was thrown off by the configuration you specified in this PR's overall conversation, thinking that's where it was drawing from.


Missing `=` ?


Yes, vt, might be, but if aty is an immutable, then `julia_type_to_llvm(aty)` is a struct.


Good point


@juliaelman It's still showing the two hashes for me. I restarted the server like you suggested. Any ideas?

![screen shot 2015-06-18 at 2 27 46 pm](https://cloud.githubusercontent.com/assets/5249443/8242722/457c4040-15c6-11e5-893b-9c96f367c539.png)


I like when we wrap these in a UX struct, but just a nit.


removed it since there isn't much point


I'd probably move these colors and sizes to the UX struct as well. Again, a bit of a nit.


Have you tested this? I am not sure `javac` will work on an entire project if files it emits are lost during the compilation.


:+1:  Thanks!


Just in case the element's value has a valueDelimiter in it?


I believe that line notes for files which are in both PRs get shared, but general PR comments don't.


It worries me a bit that we'll be reloading more than once when this is shown (I assume we do it somewhere else originally). Is that expensive?


Is there any reason why you created a separate Makefile rule for the directory? Wouldn't it be easier if the `@mkdir -p mtheme` line were also into the `ctan` rule?


Yep ;-) The option was renamed a few hours later because somebody told me on Twitter.


Is _LAST_MONTH used here?  If not, please remove.


lol


Why bother being specific if there are no use cases?


I'd probably call this method SELdidClickSearchButton since its not really "selecting search settings".


yeah, I don't think it would be a valid dn (probably needs to be escaped), but wanted to cover the corner case just in case.


90664130_AlternativeModelPAT (macumber) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Nowhere in CMakeLists the games.c is referenced as a source to compile, so it’s never generated.


What's this offset for? A little comment would be nice.


@bcl  watch out for this one as well https://bugzilla.redhat.com/show_bug.cgi?id=1220890. The --acls could let us into permissions trouble :(


Awesome fix!


Nice catch! :)


Out of curiosity, when do we show this without a presentingViewController?


For example, just for testing purpose, if you do that:

```
set (xqf_SOURCES
	${CMAKE_SOURCE_DIR}/src/xqf.ui

	${CMAKE_BINARY_DIR}/games.c
```

The `games.c` file is generated before compilation. Yes, it’s not the way to compile it so it does not work, but you get the point, `cmake` does not know `games.c` is needed by another file, so this file is never generated.


Вот тут описание проблемы: https://github.com/altocms/altocms/issues/563


Was this committed to the master branch?


Adding slap as a creature effect is well seen. :)


That would have been funny


Why??


I'm a little confused by this change. Before I dig in too deep, right now if I start typing in a search, then hit the search engine button, then come back, we erase everything I typed in. That doesn't seem right to me. i.e. I typed something I was interested in. Why does opening search settings indicate I'm not interested anymore?


Arguments are not separated by `=` but by a space. That said, Neomake will join the args for you, so what you really need is : `l:arg_d = ['-d', expand(g:neomake_java_javac_class_output_folder)]`


+1 Good catch.


Hi @rk700,
I agree with your concerns, that using a Document after it has been closed is too easily possible. The question is how to technically prevent such errors - as opposed to just issuing warnings in the docu.

If you look at conventional IO classes: what is happening there is, that you get an exception if you try to use methods or attributes after close().

I have had a look at fitz.i, and I think something similar is easily achievable in our case - just on the Python level:

1) set the self.name = None in the close() method
2) insert a %pythonprepend in all relevant Document methods that does the following:
      if not self.name:
         raise xxxError("cannot be used for a closed document")
What else can go wrong once this is implemented?


Nit: I like how this is concisely tested.  However, the test calls with many arguments are difficult to read/follow.  Have you considered using `namedtuple`s here?


It would be nice to add a particle effect later. I'll think about it.


Not gonna lie, a little sad to expose these to the frontend :) Is there some reason you need it, or just saving sync and the frontend from having slightly different api's? Being more precise about which login needs updated?


This is a domain level count, not a code level count, so the amount of things that can be held in memory has no relevance.


This change was originally motivated by testing. I was being a bit over paranoid and wanted to distinguish between an expected Illegal groupDn and an IllegalArgumentException that might be thrown by some other method used in decodeStringValue.

I don't feel strongly about this and am fine pulling it out.


What about [my lib](https://github.com/greg0ire/enum) ?


} else {


Looks good. (I think I did this!)


JeepGuy is the login name for Jim Brent.


This new version is breaking my repos. Having to lock down the **rack** version to 1.6.2.


Why?


Hey @modmuss50, you may want to take another look over the LESU. A plain LESU with no extra blocks currently holds a billion EU, when it should be 1 million. The base output of one block is 0 eu/t, when it was originally 5 eu/t (in my opinion it should be 8 eu/t), since there's no point in having a LESU with no expansion blocks hold energy if it can't output any.

Speaking of energy, the "charge" indicator in the UI seems to be working strangely (I haven't looked at the code for this) - it doesn't update to 0 when the LESU is no longer receiving charge, and it seems to always report charge as 1 eu/t less than the actual charge is (tested with Geothermal and Semifluid generators).

Moreover, I can't get the LESU to output energy into a cable or batbox.

It would probably be best if we made the various values (base eu/t out, extra eu/t out per block, charge per block) config-driven as well.

I'm going to try my hand at fixing these issues over the weekend, but I thought I should give you a heads-up so you know what's going on. Also, this gives me a convenient list of what's not working. ;)


you should make sure to remove this


-f, --follow


:facepunch:


Be careful when modifying a list while iterating over it. You should at least "break;" here. I moved this to the main branch and used Iterator#remove()


Yes, we're fixing it.  Should have a new rack release out soon.  I'm really sorry about that/


Better logic in d38f196038a8871f971c1bd7714ab5185d295553 - you are correct.


should this be `dcos marathon version` for consistency?


What happened to lineHeight?


I'll get there eventually, got to get my massive rework PR done, and merge in your PR as well.  Then maybe I'll have some time to look at this :)


This is for Program Assign 2.


nvm


I knew there was another one when I was asking about Mark of Eviction!


Oh, I won't tell you I already planned to port `SaveTokensAsClaims` to the OIDC middleware then :smile:


This is failing because it's a syntax error. Can't have two else statements in an if block.


## this is a markdown file


IMHO that line should depend on environment. For instance I see no sense of using local for test env.


Sorry I could not answer earlier, if you are still around we can push.


LGTM


I really like the improvements you made to this class. I will merge them tomorrow. For the next time, can you please add clean and simple library changes like this one as a separate commit? I can pull them into master in advance and thus reduce the number of merge conflicts, while we still discuss the pull request (or wait for the logic to be implemented).


On LLVM SVN the reflection tests fail after this change.


I think @XhmikosR wanted to push stuff too but I don't really remember what it was exactly.


You probably want an error code different then 0 here.


Ok, push if you want, I will push after you.


@doublejosh I don't really agree with this style. I think operator precedence is pretty obvious here. Parentheses are arguably unnecessary here. For simple expressions using comparison operators (equality, relational), I don't think you should wrap them in parentheses.


Use of `Parameterized` test is a nice Junit feature. It's too bad you aren't needing to run the tests on more than the one data sample set row.


extra space


For code like this the aligned indentation must be updated if you e.g. change the variable name `builder` to something with a different length.

Therefore we usually prefer the following since the indentation is invariant to the changes like that:

    builder = SnappyBuilder(
        package_key, packages, os.path.dirname(ros_path) + "/")


Not sure what to do because I'm sure @XhmikosR had things to push. :/


1.6.4 is out and should contain the fix. I'm really sorry about this. :(


Why was this removed? Doesn't the next line (345) fail then?


I'll check it out @PeyTy, thanks!


@aappling-usgs, try this on for size?


I've known about it for a couple years, this is the first time I've actually used it though. It works great for certain use cases, but not everything ;)


@jdonaldson whoops, it is in Lua!

```
if (length <= 1) return [this];
		if (delimiter.length == 0)
		{
			var r = [];
			for (i in 0...length) r.push(lua.LuaString.sub(this, i + 1, i + 1));
			return r;
		}

		untyped __lua__("local t, ll, d, p
		p = self
		t=setmetatable({}, Array)
		ll=0
		d=delimiter
		while true do
		l=string.find(p,d,ll,true)
		if l~=nil then
		t:push(string.sub(p,ll,l-1))
		ll=l+1
		else
		t:push(string.sub(p,ll))
		break
		end
		end");
		return untyped t;
```


I'm not sure you can change the signature of `server.OnSendingHeaders` without breaking tons of OWIN middleware that expect a `Action<Action<object>, object>`.

http://owin.org/spec/spec/CommonKeys.html


@waneck thanks for that as well!

FWIW, I was trying to do something that leveraged lua's pattern matching capabilities.  They're quite fast, but they behave differently (lua patterns != regex, and many routines return lua-style iterators).


Refs https://github.com/gaearon/redux/issues/136#issuecomment-113166372


@romanmatyus No reason, I just didn't notice it during routine formating.


Thanks so much for the quick fix!


I don't know either. His develop branch has 4 commits which should be fine if submodules were tested. But probably better to wait for him.


Yeah, does that mean OSH callbacks will be async only? :smile:
Is it worth re-opening the ticket in aspnet/Security then?


@doublejosh FYI, I think your description was inaccurate. Since `var` is a declaration and isn't executed at runtime, it doesn't matter how many times you use `var` in front of a variable name. It's still declared at the top of the scope (hoisted) regardless.


remove extra space


I'm pretty sure submodules worked fine, I remember testing this commit for him. I'm not sure if he wanted to push the VS project changes though.


No to taky nezní úplně perfektně. :D Co takhle kompromis "Zkontrolujte, zdali jste připojeni k Internetu." ?


Relevant workaround posted by Phil Calvin on erlang-questions:
http://erlang.org/pipermail/erlang-questions/2015-June/084935.html


what are the good options to 'do the right thing' here for OWIN given that the signature of the new event is going to be async.


@XhmikosR: Push your stuff tomorrow along with https://github.com/mpc-hc/mpc-hc/compare/develop...Underground78:73e5820 since I'm not sure I will be around.


The easy way to fix that is that you just add scroll on hover so it's not so in your face. You can see an example here.

http://www.webhook.com/docs/


I guess we can just wrap the feature in an action that blocks :/  Lou will hate that


I agree with this, we can get rid of VFAT_RESULTS_SPACE.


I will be here tomorrow so I can take care of that if case @XhmikosR will not a able to.


Indeed, and the most important part of it too :p Thanks for noticing!


I think we should continue calculating VFAT_RESULTS_SIZE because 1) it might not be clear to future developers how VFAT_RESULTS_SIZE and VFAT_RESULTS_BLOCKS are related. 2) if we need to change the size of the results partition we will always need to update two values, which can be prone to errors.


Here you could possible add a comment explaining why 8192 blocks lead to a 8MB size in the partition.


I agree, I'll add this section when I get some time (hopefully over the weekend).


If we ever change the partition size, this calculation will still hold. Thus, I think we should keep it.


Also, this version of FunctionMover, isn't quite designed to be used like this. I'll see if I can fix this.


:+1:  Great catch!


@who - I have modified the templates for Java and Android to include all collection formats: csv, tsv, ssv, pipes and multi.


90664130_AlternativeModelPAT (macumber) - i386-Windows-7-VisualStudio-12: OK (2186 of 2213 tests passed)

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


I'd tested this out as well! :)


gah, sorry for the confusion.  Somehow in my brain ```__static__``` == ```__init__```.  I meant ```__init__```.  I'll change up the naming on the next pass.

So, to go back to your point, yes ```__init__``` runs before static initializations.  However, it is generated after class code.  That means any local variable declarations (e.g. the Abstract type definitions for Int, Bool, etc.) are not accessible within the class code, since they are defined in Boot.__init.  This naturally breaks stuff like ```Std.is(4,Int);```.

I think this hoisting trick I've done gives us the best of both worlds.  We can extract variable definitions in ```__init__``` code as dummy variables and let other code reference them.  Then, after all of the normal class code has been defined, we can actually evaluate the ```__init__``` body, using any of the already defined Haxe classes.  Since we need to provide low level workarounds for so many simple types (Strings, Arrays, etc.) this will be very very nice to have, and make externing other lua libraries much less painful.


FacilityTabGridView (evanweaver) - x86_64-MacOS-10.9-clang: Tests Failed

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


PEP8 - Compound statements (multiple statements on the same line) are generally discouraged.


This needs to be a unique name. It's used above which is why it's complaining.


Hmm... I need to think about this more.  The behavior you're describing doesn't sound desirable.  But maybe that's a bug in `DecisionApplier#ignored?`, not in your implementation.

To recap, with your patch, depdendency1-1.js is a 'devDependency', so it will be filtered out by `DecisionApplier#ignored?`, which checks whether **any** of the dependency's groups are ignored.  But, it's also a 'dependency' which **should not** be filtered out so it appears in the reports.  That is, maybe `DecisionApplier#ignored?`, should check whether **all** of the dependency's groups are ignored.  Do you think that would be OK?

I'll experiment with that.  Specifically, I need to research what happens with bundler.  I don't think it will break anything, but unfortunately, I suspect that I still won't be able to remove bundler's knowledge of ignored_groups. (For my reference: need to verify whether recursive dependencies acquire the groups of their parents, or whether they get the 'default' group.  If it's the later, there's no way to distinguish between A) regular dependencies with no group and their recursive dependencies, which we would like to include, and B) recursive dependencies of only ignored groups, which we would like to exclude.  The only way to exclude the recursive dependencies of ignored groups is to never pull them in in the first place, which is to say, continue to use `definition.specs_for(included_groups)` instead of just `definition.specs`.)

Will you confirm one more thing... your tests don't explicitly call out what happens to dependencies that are recursive dependencies of a 'devDependency', but do not appear anywhere else in the dependency graph... that is, they are not primary or recursive 'dependency's, and also not primary 'devDependency's.  It looks like they become only 'devDependency's, but it would be good to see that in action.

It surprises me that there are so many problems here, but I think it's because this code has never been used - we have related code in bundler that bypasses all of this, and have never added another system that depends on the ignored_groups logic.


Lovely!


LGTM :shipit:


Prefer either just `return fCompletion;` or `return bool (fCompletion);`


should it clear on every run?


Yes, new rack version fixed my repos. THANKS!


Is there a reason we have all these booleans instead of using the type variable above, or subclassing?


Yes, that's also my thought


Big update :+1:


I would check on a non-calculated value here. some kind of enum that is always called IMPACT_ASSESSMENT_FELLOW and can automatically be changed by your IDE in case of a refactoring


Consider using:

    for (auto i = mRequests.begin(); i != mRequest.End(); ++i)


please format code to make it more readable

(CMD+ALT+L in IntelliJ)


Consider `auto`


I'm unclear what this algorithm does, please extract it into a method with a telling name


FYI, I think this broke comparisons with partial timestamps as described in [SPARK-8420](https://issues.apache.org/jira/browse/SPARK-8420).  Addressing this in #6888.


enum test here as well instead printName


@bittorf That should be 0.9.__0__.1...?


Passing something like this: (PrimitiveType.INTEGER, true)
will cause a fault, but it will go undetected


World's worst interface!  :-D  Not an action item, I just never saw this.


we are not getting the existing dest_mapping field


you are free to sublicense under apache, but by moving the original license somewhere else you claim copyright. I doubt you can do so (note: I am not speaking for my former employer).

Something along the lines of "Contributions prior to [today] were subject to the following conditions:" with the original license and copyright attribution should be sufficient to avoid this.


r+


Try not use specific classes, in this case you can use a reusable class like margin-top-bottom for example.


The `init_db()` is needed, as the DB is reset during `teardown()`, but the plugin checks `if db.pages_table is None`  to initiate `init_db()` itself (e.g. here: https://github.com/openresearchdata/ckanext-pages/blob/master/ckanext/pages/actions.py#L52). So I guess to call it explicitly in the during `setup()` is not too bad.


A much easier way to do this would be just `return array.reverse();` :)


It looks cleaner with pass on the next line


FacilityTabGridView (evanweaver) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

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


But if this is an old-style pathfind request, why does it have a completion?  It never calls it!


This is never used. Can remove it.


saw approval from graydon
at https://github.com/jedmccaleb/stellar-core/commit/78d5df60a642eb8103de6abc02d972cbc3cd5991


merging jedmccaleb/stellar-core/master = 78d5df60 into auto


jedmccaleb/stellar-core/master = 78d5df60 merged ok, testing candidate = 7f1642e9


This indentation is snafued.


@DennyBravo wtf? it's a ruby project


cool


In my opinion the proper way to do this (and I use proper from the point of view of being a good open source citizen) would be to do this:

replace the LICENSE file with Apache as you are distributing derived code under this

have a NOTICE file that states that prior code was licensed as MIT and include the original copyright *and* license in it - bonus points for including a link to the original project's source repo.

include a section in the README describing the above


cool!


@felipeas

Tem que validar no login, se o pangaré ta bloqueado.

Só o usuário Admin pode setar essa flag.


This line might prevent the return value optimization from happening @HowardHinnant ?

If so, consider:

    if (request)
        jvResult = request->doStatus (context.param);

    return jvResult;


OK by me.


if not, i believe the non-`USE_MCJIT` codepath should work with the MCJIT, just with a reduction in printed information


It only does on the start of the tests. Probably should reset for each test


back trace的时候一定要直接指定s，不能以ss开头，否则会把经过以前s的路线重新计算一遍
[https://github.com/miisme/leetcode/blob/master/wordLadderII.java#L36]


Because it's already initialised in UniversalMain?


Why not push on signatures directly?


Here during handling we will have to check that the last file created is completed or not. In that case we will have to remove the incomplete file. Simply displaying message will not work.


Can you please explain what this code is doing? I do not understand.


It really depends on what you want this to do. I'd really rather have the output from code_llvm be relatively simple and add a separate function for whatever else you need.


But file is being saved in one whole buffer , not chunks so its saved or not saved. Am i right


What is wrong with getSignaturesOfType? It already does resolveClassOrInterfaceMembers.


If lua has an indexOf method, my guess is that it should be even faster
than some kind of pattern matching, though

On Thu, Jun 18, 2015, 18:54 Justin Donaldson <notifications@github.com>
wrote:

> @waneck <https://github.com/waneck> thanks for that as well!
>
> FWIW, I was trying to do something that leveraged lua's pattern matching
> capabilities. They're quite fast, but they behave differently (lua patterns
> != regex, and many routines return lua-style iterators).
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jdonaldson/haxe/commit/43ff2d4509ad40548b30c6c88056c63a06959a67#commitcomment-11757198>
> .
>


Mi punto es traducir el concepto/idea, y no sólo la frase (traducción literal).
"Querer" traduce de forma más precisa "want", pero la idea (ofrecer formalmente la opción o alternativa), se suele traducir más comunmente como "desear".

Para no tener demasiadas discrepancias en lo subsecuente, debe decidirse entre traducir las frases del idioma original (inglés) o traducir la idea que pretenden comunicar.

En lo personal, voto por la segunda. Es más importante que los usuarios hispanos lean un mensaje con vocabulario y redacción que les resulten familiares, a que el mensaje transmitido sea muy similar a su versión en inglés.


exactly. if `aty` is `isbits`, then `vt` is actually now supposed to be a pointer to that struct type rather than the type itself. llvm ends up liking this better.

unrelated (sortof): I was planning to attempt a substantial reworking of codegen in the near future (partially to address this concern more generally, and to be proactive at supporting llvm's gradual change to opaque pointer types). do you have any outstanding code changes that might get impacted, in case it turns into a messy rearrangement of functionality?


@who - Updated Java client. Integration tests looking good for both Java and Android.


De nuevo, tienes razón en cuanto a la precisión de la traducción, pero el objetivo de una traducción es que el usuario del lenguaje destino (español, en este caso) lea una redacción familiar y común. No es tan importante que el enunciado en español tenga una estructura totalmente análoga al enunciado original en inglés. ¿No lo crees?


a


This is a) not right route hook to be using and b) not the right architecture for this particular problem


Coding seems to be fine.
[+]Check


Never saw it! Looks like we ended up following a very similar approach.


It does, at least in C++11.  Might have changed for C++14.  But Tom's rewrite is best for now.


Here we can explain what the didTransition action is and when it is fired.


Dado que 'zip' no es una palabra común en ningún lenguaje, sino un tecnicismo, opté por diferenciarlo cambiando a mayúsculas. Dicho eso, quizá la mejor opción es prefijar un punto: ".zip" o ".ZIP".


@bear the current situation is exactly the same with the exception that the NOTICE and README files are combined. Are we continuing the discussion because two files good, one file bad?

As for the source of the project, it is very prominently displayed at the top of our github page.

Let's not forget that we are not dealing with a classical fork here. We are talking about an original number of ~5KLOC that were *jointly* developed by the original founding company *and* BlueJimp in a project of 80+ KLOC.


And no-standard maps?


for the non-MCJIT case, this added the type definitions you might would want if you are trying to understand the behavior of, or to independently compile the code.

i'm somewhat ambivalent whether to have the default also output the external method declarations (to make it actually usable with llc), or whether that could be an extra named argument to code_llvm


here we should probably explain what the currentHandlerInfos is


I fixed it in the next commit, filter is kept around as recursion gets deeper.


There is a SQSUtils class in Utils now that replace it.
On Jun 18, 2015 4:57 PM, "Tyler Romeo" <notifications@github.com> wrote:

> Still need these. :( They're used as a part of receive_message() to
> determine what message was received.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sportarchive/CloudTranscode-PHP-SDK/commit/abe50118bae8c054e2c50d482fc5e7cd6e716a9d#commitcomment-11756259>
> .
>


I don't have any outstanding code changes, but if you do, please a) try with llvm-svn b) run the Cxx.jl test suite which exercise llvmcall quite a bit. Also note that I do need the ability to pass something by value across the llvmcall interface. I didn't think that this was supposed to break that ABI, is it?

Even if it's supposed to be a pointer there's still something wrong. Right now what happens is that ty is my struct type vt is jl_pvalue_llvmt, so this then ends up trying to load `jv` and then tries to pointer case it in rewrite_type below.


see next commit


Yeah basically UNORM means "I'm giving you an unsigned integer data that the shader should convert to floating point in range 0.f-1.f."


probably need to add an `$(INSTALL_NAME_CMD)` line for this too


Also, FWIW, I verified that adding `vt != jl_pvalue_llvmt` to this if condition fixes my particular test case, so if you're planning to rework it anyway, I'm happy just adding that condition and moving along.


De nuevo, no pretendía traducir palabra por palabra del inglés, sino traducir la idea.
En el español común, enunciados como "Intento de abrir el archivo fallido" no se utiliza. Nadie en el mundo hispano habla así. Adapté el enunciado a algo que el usuario hispano lea e inteprete de forma más natural.

Por ejemplo, otra alternativa que podría retener una estructura similar a la versión en inglés, pero sin sonar como un robot, sería: "El intento de abrir el archivo fue fallido". Aún así, sigue sonando como algo que nadie lee ni escribe comunmente.


Yes it is but then also it may lead to that file is not saved with proper data. I will take care of that part.


DesignDaysGridView (evanweaver) - i386-Windows-7-VisualStudio-12: OK (2193 of 2213 tests passed)

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


I made it constant because it looked nicer ;) - should work with Morpheuz


Es un tema de redacción. Cuando hablamos de comparaciones objetivas (como las que hace una computadora), es redundante e innecesario decir que las cosas 'parecen coincidir'. O coinciden, o no coindicen. Ese enunciado resulta más contundente y claro, y no deja lugar a dudas.

En todo caso, sugeriría corregir la versión en inglés como : "does not match". Pero quise atenerme a un lenguaje por 'pull request'.


Ok, also how will u solve merge conflicts?
On 19 Jun 2015 03:55, "Puneet Agarwal" <notifications@github.com> wrote:

> Yes it is but then also it may lead to that file is not saved with proper
> data. I will take care of that part.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/girishramnani/spoj_acceptor/commit/b8e10e1a46e99d7af5d305a85b8f26aba1c5dd6d#commitcomment-11757673>
> .
>


Is this what I think it is?! :party:


rax retry tests


FacilityTabGridView (evanweaver) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Ok, I see that use case. In that case, I think it should be a separate option to code_llvm (because it does add quite a bit of verbosity. If you don't need the recursive case then just use the non-MCJIT case for both. If you do, we should probably create an option to FunctionMover that does not look at the memory manager to try to find existing functions - because that's not what we're doing though.


The original javascript code was not jointly developed. The joint project consisted of a server (the videobridge; written by bluejimp) and a client (written by me while working at ESTOS).

The NOTICE file (which is what you use if including someone else's code) does not apply.


In any case the assertion failure we get here is still curious - I do not know why it happens, so I'll investigate a bit.


You can create constants for the timeout


hi
kkkk


Awesome!  I'm adding that tonight along with the other homework changes.  Hope you saw my message when the wrong pin is entered.  :D


I have made some changes and merged it into master. Now you move to your master branch and pull the new commits there on master and then merge it with your branch. If there are conflicts, use mergetool. And then you can use `git meld`. After that you will get to know what to do next.


I hope you don't mind that I peeked...? I love this short-cut method... but I think it should be written more DRY... I hope you don't mind if I adopt the concept and write a something that leverages the number_pages method?


Como menciono en casi todos mis comentarios: traducción lingüística en lugar de traducción palabra por palabra.
En concreto, "Esperando por la emulación para iniciar", se puede interpretar fácilmente como: "Esperando a que algo pase (inicio, fin, arribo) de la emulación para iniciar (algún otro suceso o evento)".
El mensaje real es que estamos esperando a que la emulación inicie, o el "inicio de la emulación".


this.modelFor should be updated to

```
this.get('currentModel.name')
```


Ok
On 19 Jun 2015 04:01, "Puneet Agarwal" <notifications@github.com> wrote:

> I have made some changes and merged it into master. Now you move to your
> master branch and pull the new commits there on master and then merge it
> with your branch. If there are conflicts, use mergetool. And then you can
> use git meld. After that you will get to know what to do next.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/girishramnani/spoj_acceptor/commit/b8e10e1a46e99d7af5d305a85b8f26aba1c5dd6d#commitcomment-11757753>
> .
>


Moreover, "Motor de 32 bits" would translate more accurately as "Engine of type 32-bit". It does not imply ownership ("has"), so no worries there.


Might try to write it differently, admit it's a bit funky. Would this be simpler?
```
let resolved =  getResolvedSignaturesOfObjectType(baseType, filter);
callSignatures = addSignatures(callSignatures, resolved.callSignatures);
constructSignatures = addSignatures(constructSignatures, resolved.constructSignatures);
```
The filter.excludeTemp is only needed because of the two function calls and we want to start excluding inherited call signatures we've seen only after those two function are done.


Sabla bra splines!


Name this just PrivateKey please, to stay consistent with the json field name (all the other fields do so). You can add the detail that it's a key in PKCS8 format in the comment if you like to.


Not sure how much this matters, but this doesn't work in old browsers and in WSH+JScript:
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/JSON#Browser_compatibility

May be converted into something like
```js
var Child_options = function() {
	this.eol = '\\n';
};
Child_options.prototype = options;
var child_options = new Child_options();
```

Anyway, I already have some automated patches for things like string[N] vs string.charAt(N):
https://github.com/Infocatcher/AkelPad_scripts/blob/d2dc1306146f108294b7ceb821990fdc51db6b84/jsBeautifier.js#L7756-L7814

So, this doesn't matters for me, if may be easily fixed. And the main question is about officially supported JavaScript versions.
An yes, it's good for me to keep the code more old-school, of course. :)


oye @marceloperez este commit lo que hace es agregar el layout al proyecto. La página principal se ve bien. Sin embargo, hay hartas cosas que aún se ven bien mal. Le quieres pegar una miradita?


I hope you don't mind that I peeked...? Seems that you're on to something important here...

...But, this could become a performance issue as the string's bytes will be duplicated one more time (they're duplicated anyway).... I'm wondering if we can't fix the `#format_string_to_pdf` method instead.

I'm going to see if the `#format_string_to_pdf` can survive without the `force_encoding`. could you test that on your end as well?


CSS3 Transitions not supported by: IE (6,7,8,9)


hello


@casey-collab I updated the subdirectory format


i want to understand why the tests weren't passing for me, looking at this diff from 6e84edf4ac085601f1d03fc7a32e85591ddba2e8, we may need to make changes to some of these lines before things pass


@dkayiwa As you can see on line 2223 in **OpenmrsUtil.java** below, one of the reasons **WeakPasswordException** is thrown is an empty password field so it made more sense to call it empty rather than the earlier *generic* weak password message.


Interesting idea here... but I feel we should keep the `@info[:Producer] = "Ruby CombinePDF #{CombinePDF::VERSION} Library"`.

My name isn't super important, but it would be good for debugging purposes to know the version of the library.


This is not necessary, [module_platform_driver()](http://lxr.free-electrons.com/ident?i=module_platform_driver) sets owner through [platform_driver_register](http://lxr.free-electrons.com/ident?i=platform_driver_register).


I think the type check is redundant because `ledger_->copy` already checks the type.


This conflicts with the check in PM, which at the moment is:
```js
if (!('ppid' in startedRequest)) {
  startedRequest.ppid = process.pid;
}
```

https://github.com/strongloop/strong-pm/blob/f0fce77b40e45d284076775b549abea2071dbbca/lib/server.js#L242-L244


I think the type check is redundant because `ledger_->read` already checks the type.


this do not resolve this issue


Why not simple `/\n/g` instead of `/[\n]/g`?


haha.  i can't read any of this japanese on this page.


I still want to sort out the promises so I can jump the new songs to the top of the list, but, oh, well, good enough for the time being.


Interesting. Maybe if we decompose `Parser` as follows, we can provide a `Comonad` instance for `Automaton`.

~~~haskell
newtype Automaton i o = Automaton
  { result :: o
  , step : i -> Automaton i o
  }
newtype Parser t  f a = Parser (Automaton t (f a))
~~~


pCrcSizeMatch can be nullptr


If you are prepared to disallow it in a pipeline then its trivial to do properly and use tiny amounts of RAM - you either run through the file once counting line starts and then lseek/write/lseek/write them out backwards or you keep a say 1K buffer and work back through it.

If it has to be totally held in RAM you already lost on an 8bit machine and you'd have to spill it to a temporary file and then use the file based algorithms.


all tests pass:
success: https://travis-ci.org/stellar/stellar-core/builds/67443813


Well, you should learn someday. It's going to become mandatory in the US sooner than later ;)


close #3


@gfinak , [here](https://github.com/RGLab/openCyto/blob/master/R/add_pop.R#L22) is how you can add a `polyfunctional` node by one command.


This isn't an issue with his code. @ambrinchaudhary ported the CKEditor to OSGi and forgot to migrate this configuration setting.  I have asked her to take a look at this issue in https://github.com/ambrinchaudhary/liferay-portal/pull/48


Good point


Woah! :) So OF 0.9.0 will replace Poco with Boost before the release?


Yeah, Ill try to distribute the songs based on artist evenly later, like in the spotify article ;)


More `validity` checks are yet to be added in order to make `add_pop` more user-friendly.


fast-forwarding master to auto = 7f1642e9


:heart:


@fippo feel free to plug in the exact number of lines that you believe came from you. I seem to recall that @yanas contributions on the UX were already part of the project in the ESTOS repo but I might be mistaken. I'll let her respond if she cares.

My point was that we are not talking about someone taking a project, adding some makeup and pretending it's theirs. We are talking about eight javascript files that were used as the first building blocks for a much bigger project, after giving ample and prominent acknowledgement to the original contributor: you.

While to this date I continue to appreciate your contribution, there have been many others, who have contributed significantly more to Jitsi and whose name and efforts are far, far less prominently mentioned than yours. So we are going to leave it at that.

One argument that I do accept is that your contributions were funded by your employer at the time so I am happy to reflect the affiliation at the corresponding pages.


an extern "C" code in C++ file is a way to warn the compiler to export the entry for that specific function in order to be called as a regular memory access in C.


What about this? https://github.com/rhinstaller/anaconda/blob/master/pyanaconda/errors.py#L319


es que ahí esta la diferencia en traducir como corresponde a traducir literal, donde la ultima es solo una traducción interpretativa nada mas.

de todos modos ya he corregido esto en la traducción en ingles para que sea compatible con tu versión de la traducción, así que problema resuelto.


la primera parte esta bien, solo me refería a la traducción de "try unzipping ROM manually.".


SpecialCommandName contains OtherName, so it is enough to add.


kok akeh sing dihapus ya ?
navbare kok diubah meneh.... upload file dan daftar pemesanan kan masuk di bag pemesanan
dan juga ga ada keuangan.


thanks


optional nit: suggest a Get/Set pair encapsulating all the time.RFC business in some package the kubelet has access to


bleh, ad hoc boolens. Every one doubles the number of states in the system because someone will surely pipe it through the call stack in a few months. How about using the updateType in the podUpdate? (it's set to SyncPodCreate for new podworkers in an already hacky way, but reusing that feels better)


Suggest making this is a V(3) infof since it's just a failure to record stats right?


claro, en este caso habría que actualizar la frase original en ingles, de otro modo la traducción no es correcta, ya que en la frase original se habrá de una probabilidad, por ello se usa "appear" (parece).


Yep. Placed a fix at the top


This is the C source code for the Water System Multi-objective Genetic Algorithm (WSMGA), which was developed based on NSGA-II. WSMGA is a generic multi-objective genetic algorithm. However, the input format and coding system, and the output function of the program have been specifically designed to suit the requirement of the design and operation optimisation of water distribution systems (WDSs). WSMGA uses both integer and real number coding system. An example of the input and output format and a typical input file for the optimal sizing of WDSs are attached.


Japanese, yes I am interested in continuing.  Spanish, not really.  Looks pretty boring.

"Mandatory" my ass.  Bunch of people down in Texas who can't even speak English--that should be the mandatory language around here.


Could we re-add these exiting/starting logs? It might be helpful to debug #2.


[![Christina Olson](http://api.artsmia.org/images/120579/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=120579)


for the string, the size should be `string.length() * 2`... which is likely bigger than the raw data


el problema es que es una traducción literal interpretada, tal vez habría que cambiar la frase en ingles para dar mas sentido a lo que se quiere decir.

otra forma de traducirlo podría ser: "Esperando que la emulación se inicie".


Cool, thanks for your patience. I guess it could be "!= null &&" or "== null ||" ;)


yeh it probably should have been GSVector4i(real_width/2u, 0, real_width*upscale_multiplier, h));  (obviously using the class function)


@dkayiwa The local variable minLength is just initialized here, with a default value. On line 2213 it gets updated with the value that the end user has set through global properties.


That's expected, so fine with me :+1:


It's fixed in b44bcce :)


Note: reverting because the later commits of openFrameworks started to move to Boost.


We compute the size as the length on disk, which will be UTF-8 (at least that is what the previous code did)


... and that's why Donald Trump should have been allowed to reproduce XD
Don't worry, it will not be boring for your kids (or grandkids?) when they have to learn it in school. ¡Te diviertes, carcamal! :dancer:


I'm getting the message that ssl cert failed, but looking at what you have above, I have those. My question is could they be in the wrong place? Can't get the server to start be cause of this error:
DL is deprecated, please use Fiddle
Fetching gem metadata from https://rubygems.org/............
Fetching additional metadata from https://rubygems.org/..
Resolving dependencies...
Using rake 10.4.2

Gem::RemoteFetcher::FetchError: SSL_connect returned=1 errno=0 state=SSLv3 read
server certificate B: certificate verify failed (https://rubygems.org/gems/i18n
0.7.0.gem)
An error occurred while installing i18n (0.7.0), and Bundler cannot continue.
Make sure that `gem install i18n -v '0.7.0'` succeeds before bundling.
please help


Is not!


I committed a change to satisfy your use cases. https://github.com/Tatsh-ansible/ansible-modules-extras/commit/5e5eec1806e406127484e18492f4c1d6b45a6341

If you think this is good (try it in your `library` directory), then I will issue a pull request.


Looks good, OK to merge.


Parcialmente de acuerdo. La emulación no "se inicia". En todo caso, inicia, o es iniciada.
También, dado que el inicio de la emulación es un evento, no se "espera que"; más bien se "espera a que".

Este enunciando sería más preciso: "Esperando a que la emulación inicie", o "Esperando el inicio de la emulación". Reitero, el objeto principal del enunciado es el evento: el inicio de la emulación.


Hey Drew,

Why do we need to add an eventName param to the URL? It looks like the issue is that the id doesn't have the proper portlet namespace. Adding the `ROLES_ADMIN`  portlet namespace to the id (id: '<%= PortalUtil.getPortletNamespace(PortletKeys.ROLES_ADMIN) %>selectSiteRole') should fix the problem.


Not sure what you mean. There is no SQSUtils class in this repository.


Thank Jon.


python!!


I remember, that this was added for reason. This might not work with other compilers on apple, or something else.


@yjun2 These last two checkpoints are easily the most difficult of all. Only 1 in 5 students ever attempts to complete the assignment, and only 1 in 10 completes any of them without help. So, you have far exceeded the norm, as I knew you would. Your logic and attention to detail are remarkable. I continue to be impressed at every level. That you do not understand it with utter clarity yet, is of trivial matter.


changed ogg default quality to 9 to get rid of "quality too high" warning
(libvorbis seems to only take 0-9 as quality setting with 9 = highest)

set opus default to a bitrate target, as thats their way of vbr coding


Fixed in  6b67bd4, ec7241c.
Looks like we can't trust the github integrated editor for cp-949/EUC-kr files :(


stil lsome bugs, help please


ignore this please. i'm new to this


The Sqsutils clas  is in the cloud transcode repo.

Let's talk about it soon. I'm omw home.
On Jun 18, 2015 7:07 PM, "Tyler Romeo" <notifications@github.com> wrote:

> Not sure what you mean. There is no SQSUtils class in this repository.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sportarchive/CloudTranscode-PHP-SDK/commit/abe50118bae8c054e2c50d482fc5e7cd6e716a9d#commitcomment-11758205>
> .
>


Shouldn't this be `(11, 11.0)`. Technically they're equal but we're expecting a float.


Actually this is probably wrong. If it's an int or long, we want to `float()` it. We don't want to do that for float.


90664130_AlternativeModelPAT (macumber) - Win64-Windows-7-VisualStudio-12: OK (2188 of 2213 tests passed)

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


@Diego-F-Aguirre Nice touch.


By declaring this without a `var`, you're actually making this a global variable (thanks, JavaScript).

I'd just do this:

```javascript
uuid_field.val(_.UUID.v4());
```


Ok, so the issue is that this version of the FunctionMover cannot be used to move function from any Module that is currently known to the ExecutionEngine, because otherwise it may trigger the source module to be compiled by referencing another symbol in it, which can mess with the ValueMap. I'll submit a pull request to make this optional and switch it to using the non-MCJIT code on MCJIT as well. Any further enhancements can be done on top of that. In particular, there's another version of the FunctionMover at https://github.com/Keno/DIDebug.jl/blob/master/src/FunctionMover.cpp that should be better suited for this use case.


Yeah.


FacilityTabGridView (evanweaver) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

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


**FUNKTIONIERT NICHT**
Nur der Drive link funzt


I don´t know how to use it ._.
Please help me.


Looks roughly fine, I'll pull it into my local environment and check it a bit more than a quick glance just to be sure - schema changes can be hairy :grinning:

Also, I want to wait until Labs is stable ( :laughing: :crystal_ball: ) before doing another deploy for non-critical stuff.


It calls it when it completes.


Only the frontend needs to bump visits; the latter is the reason. When we update a login, we need a stable ID to tell the DB which one we updated. No getting away from GUIDs, I'm afraid. As a handy side-effect, it also makes these signatures as minimal as they can be.


This only affects the UI sounds for now although I would hope to play the in-game sound affects using the same api assuming it is stable. Please let me know if the new wavs are too loud or quiet. Ultimately, the sound effects should be determined by the theme and I hope to have something in place before the next release. I do find using the deck editor with audio feedback when adding/removing cards very useful.


Typo in comment: "wallet"


Yeah, this was a straight move, but I'll tidy it up.


`timesUsed` is the name of the field on desktop and in Sync, so like the others we try to match. :/


Thanks.


The `if` and the `else` are doing the exact same thing. (And have inconsistent curlies.)


IIUC it's that we don't allow you to submit a password to a submit URL that isn't the same as the hostname of the stored login. Presumably this is a sanity check. I'm just preserving this logic, not adding it.


"limited applications" je nepravdive tvrdenie, DTrace a Modular Debugger pouzivaju CTF


Nope, look carefully


`usernameField` and `passwordField` are both on their way out, but we need to preserve them for syncing for the time being.


Ping #20612


Acid part:
http://paste2.org/CvanZOey


That makes sense to clarify — as we want to ensure users understand that it's the label, which wraps the input, which is showing/triggering the "not-allowed" cursor. Sorry I didn't include that up front. Thanks for CC'ing me on this though, I'll try to be more specific / thorough in the future :smiley:


ona ta ich zverejnena implementacia je "open", len pod licenciou ktora zabranuje pouzivat tu kniznicu v kontexte napriklad BSD kernelu


The only uniqueness constraint is on GUID. Pretty much anything else (apart from missing values) is valid, if ill-advised, according to MattN and Dolske.


okrem poor/none documentation je problem najma unstable private API


Not sure if this is the best commit to reference, but ping #20612


CRF -> CTF


Yeah, look at the protocols.

`BrowserLogin` has `removeLoginByGUID`, which marks as deleted.
`SyncableLogins` has `deleteByGUID` which applies a server-side deletion to the local store, and `markAs*` which are post-upload 'acknowledgements' that we use when an upload succeeds.

These match the way history sync works.


"for" by som asi vynechal


Prueba


Here we need to replace by HostName


Apparently!


This commit should reference the repo issue: https://github.com/BroadleafCommerce/BroadleafCommerce/issues/1434


It is used but we are asserting after that. We can't use NSLog if don't assert because NSLog doesn't respect specified reporter time. If this particular use case should be treated as error and xctool should stop working then abort here. If this could be treated as a warning - then it should be passed to reporters.


`is_overridden` means there's a local row. This allows us to use `UNION ALL` with a simple `WHERE is_overridden = 0` when querying from the frontend.

We flip this bit when we create the overlay, and flip it back when we delete the overlay.


Excuse me?

Sent from AOL Mobile Mail

-----Original Message-----
From: Lavelle hacked... <notifications@github.com>
To: github/developer.github.com <developer.github.com@noreply.github.com>
Sent: Thu, Jun 18, 2015 11:38 AM
Subject: Re: [developer.github.com] Sync changes from upstream repository (6cb4757)

drop a lawsuit.douch

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


Sometimes. :smile_cat:


I opted not to change the constraints in this patch. I'd be happy to see them tightened up.


This bug fix should be pushed to the ubuntu package: trimage.


This patch is missing from the ubuntu package: trimage


Não rolou adicionar o componente com o npm?


No; we only want this for a count, so this is the syntactic way to specify

```
func factory(row: SDRow) -> Int {
    return 1
}
```

You can't just do `{ 1 }` because the signature is inferred to be `() -> Int`, which doesn't match.


오오 Good job!


Will comment.


FacilityTabGridView (evanweaver) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Augh! Yes, _very_ different. Tired eyes like that means it's time to quit for the night. :)


Después del `#` va un espacio y un mensaje que signifique algo.


修正箇所間違えました、後ほど直します


Limpiar esto.


Good catch!


@siebrand, can you explain why this commit removed several keys from `fr.yml`, including `december` and `dna`? Is it because those keys are not present in `en.yml`, and if so is that just the way translatewiki works or was that a mistake?


Will ReadPFM replace ReadOIIO for future versions of Natron?


This would appear to send current members to the payment page when the member_fee is zero or unset but the non_member_fee is set.  Is that correct?


A este commit le falto los cambios a `db/schema.rb` producto de correr `rake db:migrate`.


navrhoval by som teda nieco ako: CTF has already seen some usage in Solaris and BSDs, but the lack of a highly permissive license, poor documentation and private unstable API prevent it from a wider adaptation.


Esse ```|| []``` é realmente necessário? A gente já não inicializa as activities como um array vazio?


Also removed the alternate error handling that Rob had added in c1aeb750640e82f739ba182f7f4b8a4fc0624d4d


Cambiar el mensaje a texto normal (?)


By doing this on "PLAYER_PORT" the else part (most likely) will never be call.


tu by som mozno viac zdoraznil to, ze sa jedna o 2-clause BSD licenciu, aby bolo jasne ze mame vyhodu: ".. is to provide a 2-clause BSD-licensed implementation ..."


Thank you, that is a nice catch :+1:
Could you please explain the rationale for this fix?


Here: it checks for $prev_interval_max. But to maintain hardlinking, you would only need the .0 folder, right?


Are there any issues if we can't close the connection here? We might want this to be an error if it leaves the agent in a state where it can't continue.


Same issue here, I think that some part is missunderstood when creating this AI.


What makes you think my commit is at fault? Have you looked at the test output?


어제 쀨받아서 코딩하다 새벽에 잤는데,... 엄청 졸립니다 ㅎㅎ


Should we create consts for these values for readability? Not sure what 0.2 / 1.5 correspond to at first glance.


The patch can be removed once busted releases a new version.


Undo this change


Fixes #90


Aquí cambiar a

    ActiveAdmin.register User do


ಠ_ಠ Y U no maven dep


If the primary registrant is a member, all other guest registrants are charged the discounted member fee.  Is this correct?


viem ze aj v mojom texte sa nachadza informacia o C99, snazim sa to vsak postupne konvertovat na C89, cim sa zvysuje prenositelnost. Mozno by bolo rozumnejsie uviest iba "C language" a detail nechat na implementaciu.

Okrem toho, libctf funguje tak na 32bit ako aj na 64bit ELF objektoch, davalo zmysel uviest "... multi-platform C language library with support for both 32 and 64 bit architectures, and properly ..."


Este archivo hace otra cosa que no tiene que ver con el "devise log/reg".