@mauritsvanrees Jenkins CI reporting about code analysis
See the full report here: http://jenkins.plone.org/job/package-plone.app.blob/14/violations
```
src/plone/app/blob/field.py:45:12: P002 found "hasattr", consider replacing it
src/plone/app/blob/field.py:198:1: E305 expected 2 blank lines after class or function definition, found 1
src/plone/app/blob/field.py:278:16: P002 found "hasattr", consider replacing it
src/plone/app/blob/field.py:343:1: E305 expected 2 blank lines after class or function definition, found 1
src/plone/app/blob/field.py:360:1: E305 expected 2 blank lines after class or function definition, found 1
src/plone/app/blob/field.py:387:12: P002 found "hasattr", consider replacing it
src/plone/app/blob/field.py:390:1: E305 expected 2 blank lines after class or function definition, found 1
src/plone/app/blob/download.py:35:1: C901 'handleRequestRange' is too complex (12)
src/plone/app/blob/scale.py:27:5: D001 found adapts( replace it with zope.component.adapter
src/plone/app/blob/scale.py:74:5: D001 found adapts( replace it with zope.component.adapter
src/plone/app/blob/monkey.py:47:1: E305 expected 2 blank lines after class or function definition, found 1
src/plone/app/blob/monkey.py:53:12: P002 found "hasattr", consider replacing it
src/plone/app/blob/monkey.py:56:1: E305 expected 2 blank lines after class or function definition, found 1
src/plone/app/blob/interfaces.py:10:37: Q000 Remove bad quotes.
src/plone/app/blob/interfaces.py:13:17: Q000 Remove bad quotes.
src/plone/app/blob/interfaces.py:14:23: Q000 Remove bad quotes.
src/plone/app/blob/migrations.py:43:1: C901 'makeMigrator' is too complex (15)
src/plone/app/blob/migrations.py:55:80: E501 line too long (115 > 79 characters)
src/plone/app/blob/migrations.py:83:20: P002 found "hasattr", consider replacing it
src/plone/app/blob/iterators.py:13:9: D001 found implements( replace it with zope.interface.implementer
src/plone/app/blob/content.py:254:1: E305 expected 2 blank lines after class or function definition, found 1
src/plone/app/blob/adapters/atimage.py:9:5: D001 found adapts( replace it with zope.component.adapter
src/plone/app/blob/adapters/fileupload.py:17:5: D001 found adapts( replace it with zope.component.adapter
src/plone/app/blob/adapters/atfile.py:12:5: D001 found adapts( replace it with zope.component.adapter
src/plone/app/blob/adapters/file.py:11:5: D001 found adapts( replace it with zope.component.adapter
src/plone/app/blob/adapters/blobwrapper.py:12:5: D001 found adapts( replace it with zope.component.adapter
src/plone/app/blob/adapters/stringio.py:12:5: D001 found adapts( replace it with zope.component.adapter
src/plone/app/blob/adapters/ofsfile.py:11:5: D001 found adapts( replace it with zope.component.adapter
src/plone/app/blob/adapters/xmlrpc.py:12:5: D001 found adapts( replace it with zope.component.adapter
src/plone/app/blob/adapters/pdata.py:13:5: D001 found adapts( replace it with zope.component.adapter
src/plone/app/blob/adapters/webdav.py:12:5: D001 found adapts( replace it with zope.component.adapter
src/plone/app/blob/tests/utils.py:59:9: T001 print statement found.
src/plone/app/blob/tests/utils.py:60:9: T001 print statement found.
src/plone/app/blob/tests/utils.py:61:9: T001 print statement found.
src/plone/app/blob/tests/base.py:29:1: E305 expected 2 blank lines after class or function definition, found 1
src/plone/app/blob/tests/base.py:37:1: E305 expected 2 blank lines after class or function definition, found 1
src/plone/app/blob/tests/test_replacements.py:413:9: E731 do not assign a lambda expression, use a def
src/plone/app/blob/tests/test_base_fields.py:40:1: E305 expected 2 blank lines after class or function definition, found 1
src/plone/app/blob/tests/layer.py:30:1: E305 expected 2 blank lines after class or function definition, found 1
src/plone/app/blob/tests/layer.py:74:1: E305 expected 2 blank lines after class or function definition, found 1
src/plone/app/blob/tests/layer.py:109:1: E305 expected 2 blank lines after class or function definition, found 1
src/plone/app/blob/tests/lingua.py:35:1: E305 expected 2 blank lines after class or function definition, found 1
src/plone/app/blob/subtypes/image.py:25:12: P002 found "hasattr", consider replacing it
src/plone/app/blob/browser/size.py:46:9: E731 do not assign a lambda expression, use a def
```
Follow [these instructions](https://github.com/plone/jenkins.plone.org/blob/master/docs/source/run-qa-on-package.rst) to reproduce it locally.


TYPE:AndroidApp
this App using the Animation in Android Studio


just `Testing IphoneLogo::setWidth`. The `::` signifies it is a property/method belonging to the class. just an fyi


This test shows the mutability of an immutable feature. It should have flagged it in your mind that something was amok here.


I understand. I opened a new ticket https://github.com/Cxbx-Reloaded/Cxbx-Reloaded/issues/177


See how you changed it to work? That doesn't make sense.


Perhaps make this configurable?


Is a `brand` a sequence of `char`? Nope. It should be `Brand brand`.


I am suggesting to delete all IDEA files, they are not necessary to be in github
.idea/*, *.iml etc.


you're missing `Logo logo` and `IphoneSize size`. compile-time error


OK. In general we are not expecting external users other than xapi but in the last few years we have been trying
to follow a reasonable semver.
Should we change it again, we will make sure to bump up the major release number.
Thanks for reporting.


Olet oikeassa, takroittaa että tuosta puuttuu ) ennen ; merkkiä. Korjasin juuri.


Im confused about this. Are we not already creating an iphone? So what would brand have to do with it? We already know it, so this is superfluous


It's too trivials.  If 5 times are all failed,  I think the CA server has some big trouble.  we should stop trying.


This helps people that do not have BTN access due to API migration or downtime.


Perhaps make this configurable?  Possibly @cpu can comment on this.


typically we set our variables to be the object. `IphoneLogo iphoneLogo`. It saves confusion later.


you don't need to check in IDEA folders (.idea/*)


Looks great


вигово


like above, this variable should match the object name, especially with size. The ambiguity of size can make someone think, screen size, camera size, etc, if they are not 100% knowledgable about your code.


Hi, I don' t see your projects in github, could you check in your local changes


If using brand as proper data type, this will never change, so finalize it


В смысле, а код-ревью кому? Я не очень понял


I think this is super clear.


Fair enough.  Sounds reasonable.


I do not think the logo is ever meant to be changed, so finalize it


I do not think size ever changes, so finalize it


Ты случайно зашел, или с coursera?


superfluous since it will never change


Valeu!


@ppaeps  you mean the error message configurable ?


See https://github.com/tildeio/rsvp.js/blob/master/lib/rsvp/promise.js#L128-L130


TeamCity ReOpenLDAP :: devel [Build 354](https://binbug.bigbrother-matrix.ru/teamcity/viewLog.html?buildId=3428&buildTypeId=ReOpenLDAP_Devel&guest=1) outcome was **SUCCESS**
Summary: Tests passed Build time: 02:51:46


Your artifact is `iphone`. You are creating `Phone`. I spent most time in hear commenting thinking you were creating an iphone. This obviously is violating the Singe Use Principle


Não tem nadinha pra um estagiário? :'(


Add new Line


update_sqlite_library (mbadams5) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1820 of 1820 tests passed, 0 test warnings)

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


Ahh, right! Feel free to update. Otherwise I'll change once I'm back on my MB.


see, by your tests you should have realized your logic was wrong. You are packaged as `iphone`, instantiating a `Phone`, and setting a brand of `Iphone`. Your tests should be clues to you that something dont jive


cool code!


Случайно, увидел что ты что-то пушнул, решил чекнуть


update_sqlite_library (mbadams5) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

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


org.springframework.web.bind.annotation.RequestMethod.POST
Even after importing why its here?


Hey, thanks for the updates! Can you follow the contributing guidelines for commits [1] when pushing directly to master? I'm running some stats tools that rely on the format :)

[1]
https://github.com/NixOS/nixpkgs/blob/master/.github/CONTRIBUTING.md#submitting-changes


all along I thought this was something to do with `IphoneSize` and I get here and see its a screensize. If it is screen size, then the class should be named `ScreenSize`.


the screen size never changes. finalize the variable.


superfluous when finalized


If you check out the oxide mod home page there's a sticky post that talks about changes made to the build and deployment systems for oxide.  But the basics of it are that oxide releases won't be getting published to github after this month.  Instead they'll be posted from two different locations which have rest APIs you can call to see if you need to update or not when you start your server.


Initial Commit


with comment!


Also some doc in
https://guides.emberjs.com/v2.11.0/ember-inspector/promises/#toc_labeling-promises

On Mon, Feb 13, 2017, 17:52 Kelly Miyashiro <notifications@github.com>
wrote:

> How is this label passed?
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/damiencaselli/ember-cli-sentry/commit/4768fed9f9f630d726fae9f007956fbe0623993a#commitcomment-20868564>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AF3FjvyvRX0rTzzH1Tg2g-6vNoHHnKwrks5rcN7HgaJpZM4L_1Es>
> .
>


Why not use the C++11 http://en.cppreference.com/w/cpp/chrono rather than clock_t whose precision and range is implementation defined and hence you'd need to deal with overflow when comparing values which is not done in this commit.

Also why comment out the pragma omp? If omp isn't turned on in a compiler then the compiler should just ignore (or issue a warning) the unrecognised pragma.


Droplet IP:139.59.31.164


@Calvarez14, you're getting there. Not quite 100%. Please review my comments. Your tests are looking good but leverage them to help you write code. I made a few comments on them pointing out how you should be "warned" something isn't correct with your code logic when writing tests. Use that tool. Its a great tool. When you write tests and you have to do funky things to see it work, think about how the user of your class has to do the same.

Also, you need two levels of composition, so you are short some classes.


Nope. Not final version. You have 0 packaging.


tabs!


Hey, this breaks RIL on the Z5 dual.


Why are you making the determination what the depth should be? Is this the same across all trumpets? If it is, then its a constant and should be declared as such, formally and style-wise.


Thanks for the heads up Joshua, I somehow missed that info,  sounds great, I'll give it a look, rest APIs is for sure much better :)


tabs! why is the color value set to 4????? and what is a color value of 4?


Congrats!


tabs! what is this set to 27? What if I want 28 etchings?


[Link test](http://adf.ly/1k0Smh)


This will partly address https://github.com/SwiCago/HeatPump/issues/10


why is a constructor taking another etching object as an argument?


Just because the server hasn't removed all references to the player object doesn't mean all code the player has called before destroying the object should be executed. It's actually bad to keep executing the code if the player isn't even online.

There are possible scenarios that can cause the player to execute the command after they've disconnected simply because a plugin is still holding a reference to a player.

Also, just because this has been done before doesn't mean it is right. PocketMine is a mess and spreading the existing mess even further for the sake of consistency is just laziness in my opinion.


If it was even correct to pass an etchign to an etching, I passed you an etching and you still set etching to null


you're passing a color to the color constructor


you're passing a boresize object to a boresize constructor


Yeah I missed it my self the first few times I was on the site today.  It's cool stuff, much nicer for automation purposes to have the APIs to call on.


This will partly address SwiCago#10


사각형의 높이 n과 너비 m을 입력받은 후 사각형 내부에 지그재그 형태로 1부터 n*m번까지 숫자가
차례대로 출력되는 프로그램을 작성하시오.


why? @matheuss


To run this tests use: `rake test:models`.


There could only be one style?


도메인에 www 가 없는 경우엔 문제가 없는건가요?


What is this a style of? I cannot tell it by reading your code.


A Style constructor is taking a Style object.


Update sache.json


Styles have model numbers? So if I invent a new Dance Style, it has a model number? And that model number is 33495?


Strange name for structure that contains 4 members


I meant the timeout: you're hardcoding 5 retries at 5 second intervals.  If servers do get very busy, perhaps this should be configurable or possibly exponentially back off.  As you pointed out in your other comment though: if the servers are in that much trouble, we should just stop retrying.


Man. Commit more often. Branch and commit on features. Its not an "end of the day to do" thing. You wrecked your code completely from where you were. If you are going to change "MouthPiece", branch for that, do the work on `MouthPiece`, commit, merge in. This way in the future if you make a big boo-boo like you did here, you can roll back to where that mistake occurred. You cascaded the same logic error all the way through, but still, work on small features one at a time.


@alviteri 9306 not works for me


Can be hidden inside CallDispatcher. Why to expose to clients?


<문제>
원하는 구구단의 범위를 입력받아 해당 구간의 구구단을 출력하는 프로그램을 작성하시오.
<처리조건>
 (1) 구간의 처음과 끝을 입력받는다.
 (2) 입력된 구간은 반드시 처음 입력 값이 끝의 입력 값보다 작아야 하는 것은 아니다.
 즉 입력된 구간의 범위는 증가하거나 감소하는 순서 그대로 출력되어야 한다.


What is time complexity for vector lookup?


dispatching signal?


OK


BAHHHHH! Make your class singular. Put many `BasicItem` into a collection. You'll thank me later.


Забываешь ; в конце строк


Вокруг оператора присваивания - пробелы


Consider having op== overloaded for Pair. Then, by creating tmp object and passing it into `find_if`, you will get much cleaner code.


OK haha


General idea is correct!


No need to use code font here


А, ну и забей тогда. Это у вышки в голову ударило, надо два онлайн курса закрыть. А этот курс подразумевает review друг у друга. Вот я и подумал, что ты решил отревьюить. Сам-то я бы не полез на фронт :)


Don't forget to change this line ;)


@teh are those tools open source? I'd like to run some stats for the release.


The summary table should also render some kind of tag to show that an API requires Kotlin 1.1


System-Fan-Redux (EnergyArchmage) - Win64-Windows-7-VisualStudio-14: Tests Failed (2104 of 2410 tests passed, 280 test warnings)

 * 584 tests had: AUD diffs.
 * 435 tests had: RDD diffs.
 * 351 tests had: ERR diffs.
 * 304 tests had: Table big diffs.
 * 28 tests had: BND diffs.
 * 29 tests had: EIO diffs.
 * 9 tests had: ESO big diffs.
 * 8 tests had: MTR big diffs.
 * 6 tests had: ESO small diffs.
 * 7 tests had: MTR small diffs.
 * 4 tests had: MDD diffs.
 * 20 tests had: MTD diffs.
 * 3 tests had: Table small diffs.

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


hi @Polymorph-78 wir haben beide leider keine dreambox aber da Dreambox OS enigma2 mitbringt sollte alles auch bei dir funktionieren. DU kannst das gerne mal testen, dann lauft es reibungsloser sobald der skill im store ist.
Hier sind die WebInterface Befehle die Enigma2 mitbringt: `https://dream.reichholf.net/e2web/`
Schau mal ob die bei dir funktionieren. Im zweiten schritt waer es super wenn du uns n ticket aufmachen konntest wo du das resultat von about `https://dream.reichholf.net/e2web/#about` hinterlegst und wir mit dir gemeinsam schauen ob die RemoteControl (Fernbedienung) Codes die selben sind wie bei unseren boxen.


I think i got something error while testing!


wind-pressure-curves (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Tests Failed (1227 of 2459 tests passed, 0 test warnings)

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


........Exception Handling...........


Good one :P


Whoops!

This password has been changed


I think you can just change it in the CMake file. I don't believe I am doing anything too fancy / incompatible. I just leveled up the version to keep it "modern". I am going to be delayed getting in the office today but I will validate when I get in.  If there is a regression I will address it.

Tom Panzarella
Founder & CEO
Love Park Robotics, LLC
2025 Washington Ave, Suite 217
Philadelphia, PA 19146 USA

mobile: +1 215 341 1754
tom@loveparkrobotics.com
http://loveparkrobotics.com

> On Feb 17, 2017, at 6:26 AM, Christian Ege <notifications@github.com> wrote:
>
> Hi @tpanzarella is there a way to do a workarround and provide compatibility with cmake 2.8.12.2 otherwise we will get some trouble when integrating this into our Yocto world.
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub, or mute the thread.
>


TEST


how was this decided?


wind-pressure-curves (jasondegraw) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


Soluciona #13


예전 커밋(780034d) 797줄에서 www 도메인을 특별 처리했던 것을 되돌리는 의미입니다. 이게 안드로이드 웹뷰에서 말썽을 일으키더군요. 쿠키 도메인 속성을 지정해 놓으면 지워도 지워도 계속 살아나는 좀비 쿠키가 됩니다 ㅠㅠ

좀비 쿠키 제거 기능을 2582ef2에서 좀더 철저하게 보완했습니다. 한동안 유지하다가, 지난 일주일간 develop 브랜치를 사용하던 사이트들이 모두 업데이트했을 때쯤 삭제할 예정입니다.


As I expected the `init/update` command deprecation will have some downsides.. 😞

I don't see any of these changes mentioned in changelog:
https://github.com/angular/angular-cli/commit/16bceee
https://github.com/angular/angular-cli/commit/4543be9

I know that they are not breaking changes but still I want to keep in complete sync with @angular/cli...

@hansl @filipesilva
How do you suggest to track such changes? Analyzing commits one by one is not very efficient..


What about voltage regulation on this board?

(dev samples never arrived here unfortunately)


sup


`disableBlend()`?


wind-pressure-curves (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: Tests Failed (624 of 1249 tests passed, 0 test warnings)

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


Refactor - Extract Methods


Hi @Extrawurst,

wenn der skill im store ist werde ich es testen ob es funktioniert.
Wäre toll wenn es mit dem Webinterface der Dreambox auch funktioniert.
Wenn es nicht klappen sollte kann man das Open Webinterface ja auch auf der Dreambox installieren.


@globin I just run a bunch of shell commands so we have stats for our next meetup (n new packages since last meetup). E.g.:

```
git log --oneline --since=2017-02-01  | egrep ".*: init"  | wc
git log --oneline --since=2017-02-01  | egrep ".*:.*->.*"  | wc
```

It's not perfect but pretty close to the actual numbers


@lipebento Could you please tell me about your developer's environment with Mono? OS, IDE/Text editor, build system name. I'll be working on your request powered by custom MSBuild task.


Where is this used?


how was this generated.


hey its just a first set to get hub


Grrr....

I noticed the karma one an hour later because it's in the commit log (and yes, it's far from optimal), but didn't notice that the tslint one was a project change not a CLI change :/

I guess the only real way is to `ng new` in another folder, maybe with `--skip-install` / `-si`, and with the same project name, and then copy everything into your project, call it a manual `ng update`.


how many unique entries did we get in this range?


@Meligy's suggestion is the easiest approach I think.


fffs


ah, yes


Infelizmente por enquanto não @helderdek. Abs.


Agora, temos a opção de dar o seguinte comando:
 lua process.lua driver.png.lua rvgs/pathtriangle1.rvg **-p:x** output.png
onde x é um número qualquer.

O programa renderizará apenas o path de número x.


@Acris if i set in config Monospace font, it will be monospace or not? And in what option is it?


tilde*


Ok that is similar to what I have, might want to add `--no-merges`. ;)


After this commit, if I simulate a fresh install by (on Windows) deleting c:\Users\Jay\AppDada\Roaming\Ride-4.0\ then when I fire up RIDE I get an unusable connection page: there is a Type drop-down, but no matter which type I select (Connect/Start/Listen) no further input fields appear on the page: there is just a GO button.


@chris-belcher I'd just like to point out that the technically correct answer was "**No**, but well spotted" :)

I got the distinction impression that @piratelinux believed this prevented the tumbler from accepting absoffers. Note that his commit is titled "enable support for absoffer in tumbler".

(Also since I'm feeling nit-picky, this commit has nothing to do with the topic, it's just a renaming commit, so the discussion doesn't really belong here.)


@sathyapulse This will be outside class. You can do like `WP_CLI::add_command( 'wsl', 'Wp_Songs_Library_Cli' );` outside class. And remove from __construct method.


Why do you need to use `mem3:local_shards`?  You already have `Db` - a particular shard.  Just check if _local/purge_doc is on this shard or not, and if yes - remove it. This code is already running on every shard, you don't need inside every shard get a list of all shards again.
I think from lines 52-64, you should only keep 56-62, and remove the rest.


it was used here: https://github.com/chocolatey/ChocolateyGUI/blob/develop/Source/ChocolateyGui/ViewModels/SourcesViewModel.cs#L93

I just honestly forgot to remove it from thre resource file when I was requested to change it in the source.


Adding test


@sathyapulse Always add inline documents for command description. Check wp-cli documentation. See this doc. https://make.wordpress.org/cli/handbook/commands-cookbook/#annotating-with-phpdoc


testing


wind-pressure-curves (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: Tests Failed (625 of 1838 tests passed, 0 test warnings)

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


Don't need of this method once you add `WP_CLI::add_command( 'wsl', 'Wp_Songs_Library_Cli' );` in same class file.


yeah they start working on the discord.. thanks!


wind-pressure-curves (jasondegraw) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

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


I don't know why this path is being truncated.. I need to find out why


A Factory should be used here to delegate the construction of the NetworkAccessManager


Why is this an `<object>` and not an `<img>` like below?


On Line 44 of ImageGooglerForm.cs you have "game (box|vover) art".  I believe it should be " game (box|cover) art".

Great job!


ok


develop (kbenne) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

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


Do we really want sticks to be wearable by default? Does it even look good? ^^
I know you probably wanted to show that this is a list, but maybe there is another good example item that can be put on the head.


![](https://github.trello.services/images/mini-trello-icon.png) [/start](https://trello.com/c/RTDp35nA/13-start)


![](https://github.trello.services/images/mini-trello-icon.png) [/registrar](https://trello.com/c/JWiPD5Sh/9-registrar)


OK, I'll remove it from the connect page.


You can use InventoryUtil.isNothing(itemStack) here. It checks is null, material air and amount = 0.

this means you can get rid of six lines and make it into one.


需要写入信息的等级


On Windows the left pane of the connection page is now a little too narrow to show the whole of the new Save button. (Perhaps @abrudz can fix this in the CSS?)


@Acris can u give me config port with your font settings?


It's rly good!

I have just one question and I open an [issue](https://github.com/pipocadigital/grao-de-milho/issues/34) for that.


@pulkitsinghal For validation of Nodejs.


You can shorten this to
``` java
return MConf.get().nonBlockHats.contains(material.name());
```


It needs to support :hover in the SVG.


boiiiiii


ok, this was my fault. everythings ok. DNS works in rescue system with enabled resolvconf package.
good job, thank you very much.


make your commit message meaningful. here I am reading your code wondering why I should look at this commit. Now if you said in your commit message "implements `addRecipe`", that would be a good commit message and I would know why I am here.


execellent


@harshadyeola  Thank you for pointing out this. I need to clean up that.


@cbeust Is there any particular reason why the latest releases aren't present on Maven Central? Last release I see is 1.60.


develop (kbenne) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: Build Failed

![Build Badge](http://img.shields.io/badge/build%20status-failing-red.png)


I see.


Hi, I'm afraid this is not totally correct. FixMyStreet is available under the *Affero* GPL license as given in the LICENSE.txt file in the root of the repository: https://github.com/nditech/fixmycommunity/blob/master/LICENSE.txt


kkkk


The point of an exception set is that the admins can decide whether it is to be used as a blacklist or whitelist.
And in this case, we might run into problems.

When do we care for this exceptionSet? Only when it's a block. What happens if an item is passed into this? If the exceptionset is a blacklist and it doesn't do much. If it's a whitelist, which obviously doesn't contain item materials, we filter too much.

Please adjust the code to accomodate that.


2 conditions for rejection here:

1. There is not recipe by asked for `name`
2. The recipe asked for has a pin that doesn't match the passed `employee`.

For both of those, feel free just to throw a basic exception passing a string literal explaining why.

One trick here, you are receiving a name of a recipe, but you have store the object recipe. If you use `secretRecipe.keySet` it will return a `Set` of `Recipe`. You do not need to worry about collecting that into a variable, just simply use the enhanced for loop to iterate it, collecting the recipe when you find it. You should assume that recipe only occurs once, so once your find it, you're done.


@endJunction Travis [baut gerade die Pakete](https://travis-ci.org/bilke/conan-vtk/builds/202578766).


Seems like inherited fields also gets the "method %s.%s: @constrains parameter %r is not writeable" warning?


Yuck -- I don't like the #ifdefs. Can probably just check to see if 32bit (I.e. sizeof(long) < sizeof(uint64_t) and the size would overflow)


No, problem I already left the office. I will try on Monday.


@jiangphcn Your code looks good to me, well written.

The only thing is that `FoldFun` on the line 204 will be changed.
Its' format should be:

```
FoldFun = fun({PSeq,_U, Id, Revs}, Acc) -> {ok, [{PSeq, Id, Revs} | Acc]} end,
```
You can update `FoldFun` similarly in hastings.


@ivan-nginx The global fonts configured in https://github.com/iissnan/hexo-theme-next/blob/master/_config.yml#L96 will affect subtitle's fonts.


Both line 27 and line 33 updating RubyGems. I think one of them can be removed.


Aquest comentari es per col·locar comentaris en la linia.


Also, as described further down as well, exceptionSets are made to be white- and blacklists.
(Btw, you currently made it a whitelist. The standard is false, so the admin must select the blocks he want to enable)

The admin can choose the way they work. Please change the name to "blockHats" and make it standard=true. Adjust the comment above accordingly please.


Bnccnc


hooking and not hacking? is this some lua coding tutorial xD


TeamCity Platform Services :: Environment Manager 2 :: Package [Build 3.32.0-pre-2-gff9e241](http://infra.teamcity.ci.ttldev/viewLog.html?buildId=51408&buildTypeId=PlatformServices_EnvironmentManager2_Package) outcome was **FAILURE**
Summary: Snapshot dependencies failed to start: 2 Build time: 00:00:01


Blasted search in comments and strings =/

Thanks!


Перебор с отступами


![](https://github.trello.services/images/mini-trello-icon.png) [/status](https://trello.com/c/ud429fTQ/6-status)


I'd like to see the materials support for this be model-agnostic, and not hard-coded to "pants" and "shirts" everywhere, know what I mean? That would allow us to use tint maps on other models (giblets?) easily.

Would it be possible to have channels of the tint map bound to individual meshes of the model? For md3, since they are multi-mesh, that allows us to target upper, lower and head -- perfect. For obj, you get one tint map for the whole thing. I think that makes sense.

Then the material definition would look like:

    tint upper players/qforcer/upper_tint

Or for obj:

    tint default models/gibs/gib1_tint

I dunno, something along those lines. Thoughts?


I think this is what you need to do http://stackoverflow.com/a/23598650/6764571


Where is this used? As far as i can see, [it's only used in one spot.](https://github.com/TheComputerGeek2/MassiveHat/commit/4cdf78f8ea49651755920a735a45492d47dbc311#diff-24cdf2279ce4adc87d9c3863448c53c5L76)

If there is no very good reason to have it as a Requirement, you can move that logic in the class using it and delete the Requirement.

Please apply any other feedback that I give on this class to the class you are refactoring its methods to.


A repeater would have been a much nicer way to do this.


참고: https://www.xetown.com/square/504198


Should I be able to parse something like `url.Parse("127.0.0.1:80")` ?


Can you test http://phhusson.free.fr/superuser-r297.zip ?
This includes https://github.com/phhusson/super-bootimg/commit/c7602e6d2edab17c1b38e02f00f653343c8de064


LOLOLOL... god I was so bad.


This rise another complaint I will just disable `SC2140` for now


You can use ```MUtil.isntPlayer(sender)``` here instead. It'll make the code more readable.


Basically if there were more than half of teams competing with a first round bye then the tournament couldn't progress. There would be matches which would have dependencies met, but wouldn't be notified to make progress.


@dkellner I use a custom serializer for `sqlalchemy.types.Date` because I don't want it to be serialized as datetime. That said, I can still import `sqla_type_mapping` -- my only concern is compatibility.


Hi sorry if I'm writing a comment here, but I couldn't find another way to send you a PM (email, etc).
I've heard a lot about the OPness of poison builds so I tried to recreate in your program some of the many poison builds that I could find on PoE forums.
I've tried 2 build and in both of them the estimated dps was somewhere between 30k and 40k.
I've tried to import my friend's budget cyclone-based character and its estimated dps was 70k.
My questions are: is poison calculated correctly? Is there an option I'm missing while using the program? Like I'm not displaying everything, or maybe I'm not correctly applying auras/curses.
Ty in advance.


Hey @rmisio. Are you planning on implementing the "Typing..." and "Mark as read" features? Those are already implemented in openbazaar-go.


Good catch, we want to update gems before using the `gem` command, so we'd delete the second occurrence. Would like to contribute a patch?


Good catch, we want to update gems before using the gem command, so we'd delete the second occurrence. Would like to contribute a patch?


You can simplify this whole if-statement into one line:

``` java
if (!Perm.USE.has(sender, true)) return null;
```
Please look up what happens behind the scenes of this method and use it from now on :).
(You might have to change the return when refactoring the method. The method you refactor it to is a void return.)


TeamCity Platform Services :: Environment Manager 2 :: Package [Build 3.32.0-pre-8-g22aa8d6](http://infra.teamcity.ci.ttldev/viewLog.html?buildId=51412&buildTypeId=PlatformServices_EnvironmentManager2_Package) outcome was **SUCCESS**
Summary: Running Build time: 00:00:53


wouldn't it better, if you accumulate the samples in Bit32s buffer, and only clam the final result.
It would be more optimized, and potentially better output


Sure.


Hi @Polymorph-78 ,
Da wir die Enigma2 API nutzen und Openwebif nur darauf aufbaut wie jedes andere Webif was auf Enigma2 Receivern läuft bin ich mir sicher dass es läuft. :)


Additionally to the feedback given, please refactor the listener in the main method to an engine:
https://github.com/TheComputerGeek2/MassiveHat/commit/4cdf78f8ea49651755920a735a45492d47dbc311#diff-24cdf2279ce4adc87d9c3863448c53c5L60

Every engine should server one purpose, so only having one listener in there is not bad at all.


> This rise another complaint I will just disable SC2140 for now

What was the complaint / erroneous line? I saw SC2029 (where shellcheck is wrong afaict).

```
In ./tools/provisioning/setup.sh line 316:

    ssh -o UserKnownHostsFile=/dev/null -o StrictHostKeyChecking=no "$@" "$(terraform output username)@$ip"

                                                                          ^-- SC2029: Note that, unescaped, this expands on the client side.
```


Well spoted 👍


fixed in f71869d4376d7e6484b01787efcd1f4baf2c2600


This is my first change


Plus, since material.name() is called twice in here, please make it a field just like material is one.


I am curious as to why you are using BufferLists in these classes instead of modifying these directly via the new higher level AVAudioPCMBuffer? @vgorloff


"Wrong" in the sense that we want it to expand on client side.


No sure why this title is not aligned:
![image](https://cloud.githubusercontent.com/assets/4732915/23066280/d4fc88ea-f500-11e6-8068-922a4eaa42f1.png)


We should version or datasets (as all other datasets), calling a dataset `data_…` is kind of redundant and we could compress CSV into `.xz` as all other datasets.

So why not `YYYY-MM-DD-family_allowance.xz` (where `YYYY-MM-DD` is the day the script ran)?


#133 has been opened.


VS-coil-gaps (EnergyArchmage) - i386-Windows-7-VisualStudio-14: Tests Failed (2122 of 2418 tests passed, 56 test warnings)

 * 350 tests had: AUD diffs.
 * 350 tests had: ERR diffs.
 * 294 tests had: Table big diffs.
 * 2 tests had: ESO big diffs.
 * 3 tests had: Table small diffs.

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


Suggestion:

> 1. `src/fetch_family_allowance.py` creates `YYYY-MM-DD-family_allowance.xz` with data from the Family Allowance database available at  [Portal da Transparência](http://portaldatransparencia.gov.br/downloads/mensal.asp?c=BolsaFamiliaFolhaPagamento).


Precomputing muls should considerably speed up a program that has multiple invocations of FFT.


how? the thing here is that `info.size > ULONG_MAX` is always `false` where `unsigned long` is 64 bit. and it seems that coverity was running on a 64bit machine. if we put
```
if (sizeof(info.size) > sizeof(unsigned long) && info.size > ULONG_MAX) {
 // ...
}
```
it's a dead code in run-time then.


Yay 🎉  Sorry about my last comment — I haven't seen this new commit yet ; )


Many thanks!


фыв


Yes, but I rather ignore this error `SC2140: The double quotes around this do nothing. Remove or escape them.` which just says that the quotes are useless. Also less changes to the code.


Sehr gut. Den Testfall hatte Tom; war vtk-7.0 aus connan und BUILD_SHARED_LIBS=ON... Er wird's in einer Woche testen...


Glad it doesnt corrupt BL. Yeah, actually there were no gpio_config defined and all gpio_config codes were garbage, but after I have seen they're used in Onyx's touchscreen driver I've defined it only for Onyx. You can look to gpio_config_for_onyx function.


I supposed to avoid `pyatspi` dependency because it contains C extensions. Why are you importing this?


@Acris ok, how about `font-weight: initial;` option by default?
As u can see, if we not set any font by default, subtitle appear in bold big font by default:
![image](https://cloud.githubusercontent.com/assets/16944225/23066304/efbb69c4-f52a-11e6-8833-cb34e44c2789.png)
And if we set normal font-weight, we see this:
![image](https://cloud.githubusercontent.com/assets/16944225/23066372/4a1562da-f52b-11e6-8489-f61140c6401f.png)

See on your blog:
![image](https://cloud.githubusercontent.com/assets/16944225/23066383/5d11f6f0-f52b-11e6-8665-b8c96ccb730d.png)
And with `font-weight: initial;`:
![image](https://cloud.githubusercontent.com/assets/16944225/23066398/7f4b7c5a-f52b-11e6-9aa4-7ee236116cb3.png)

What u think?


@madeye see you prefer ss_malloc() over malloc(), which's fine.
But how about the following?

```shell
$ grep -nr " malloc" src/
src/http.c:109:            *value = malloc(len - header_len + 1);
src/tls.c:235:            *hostname = malloc(len + 1);
src/uthash.h:85:#define uthash_malloc(sz) malloc(sz)      /* malloc fcn                      */
src/cache.c:54:    if ((new = malloc(sizeof(*new))) == NULL) {
src/cache.c:275:    if ((entry = malloc(sizeof(*entry))) == NULL) {
src/utils.c:226:    void *tmp = malloc(size);
```


is *used by* print statement


Actually this code taken from Apple sample code.)
During DSP processing, DSP Host might provide arbitrary pointer to buffers (BufferLists).
Not ones which you created during Audio Bus initialisation. Seems there is no way to wrap "foreign pointer" back to AVAudioPCMBuffer.


Hi @sormuras. I feel iffy about having ExtensionContext extend Utilities.

I can see that it would allow `Utilities` to expose non-static methods to `ExtensionContext` implementations in the future.

But despite that, if we (or extension writers) ever do need such non-static utility methods, then I personally think that those would be best served implemented either as `default` methods in the `ExtensionContext` interface; or as `private` methods directly within the classes that need them.


![](https://github.trello.services/images/mini-trello-icon.png) [/desativar](https://trello.com/c/TBHpCBnZ/2-desativar)


Hi @sormuras, I think `Utilities` should be made a `public static final class`; thinking of this from a more theoretical perspective, it doesn't make sense to me personally that `ExtensionContext` would have an "is an" relationship with `Utilities`.


@oerdnj Have you forgotten about this branch maybe?


I think this method should be replaced with a comment in the `TestEngine` class-level javadoc, saying something like:

```text
<b>Note for implementors:</b>
A range of utility methods which may be useful for implementing {@code TestEngine} can be found in {@link org.junit.platform.commons.util.Utilities Utilities}.
```


With `TestEngine.getUtilities` removed, I think this singleton object is no longer needed.


Replying to your question in https://github.com/junit-team/junit5/issues/352#issuecomment-280634372:

> @jbduncan Can you modify/comment the draft https://github.com/junit-team/junit5/compare/issue-246-352-utilities what needs to be changed to accomplish proposol 3? Just replace the default modifier with static?

Yep, make this and all the other methods `static`. :)


1 parent a89418d commit 06ff809da9d8932fb671484fff9511435925d0e3


这个。。。这个事干啥的


Thank you :)

:+1:


You're very welcome! Awesome book, by the way! :)


Closes #146


Well, it's a valid complaint and we can fix it (and should IMHO).


I was focussed on producing quickly. It turns out that modularity in shiny is a little more complicated than I thought and will need a little more research. The colour problem is noted and will be fixed once I know more about how to best implement the apps in a modular way.


Is there a way we can take arguments here to build for various locations?


Hey, just saw your fork. Could you submit a PR so I can fix that?


GlazingEnahncements (lgu1234) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1221 of 1221 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.png) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.png) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-36.27%25-green.png)


@ivan-nginx It's a good idea 👍


Travis build fails for Node 4 & 5.


Can you please create an issue for this and pin-point some of the 2.x features that you think are worth upgrading(*)?

(*) If I understand it correctly we would need to rewrite the configuration file to the new API/syntax which may not be trivial since we do some hacks (from line 73) to support passing multiple files files instead of just single file or directory.

Thank you.


How to use?


Why submit it to packagist again?


`return index_ >= size_` ?


TeamCity Code :: 1. Development :: DDK :: Feature [Build 155](http://mp-ci.centralus.cloudapp.azure.com/viewLog.html?buildId=15755&buildTypeId=Code_Development_Ddk_Feature) is now running


Use git_id tool which makes this automatically as explained here: https://github.com/cmangos/issues/wiki/Backporting-Tutorial#21-backport-commits-with-sql-changes


"maj"


sans `ul` ou `ol` ce n'est pas une liste.


When both even and odds are empty then "Odds win", but there should be a tie.


Nice quicksorter BRI! @brianhlin


Yes, we'd better also change them to ss_malloc.


This commit has been deployed to DEV


Just discussed validFrom & validUntil values with Martin. It should be:
"validFrom": "2017-01-22T**00:00**:00Z",
"validUntil":  "2017-01-**23**T**00:00**:00Z",


status 401
https://en.wikipedia.org/wiki/List_of_HTTP_status_codes


It is a good practice to return only one variable type.


TeamCity Code :: 1. Development :: DDK :: Feature [Build 155](http://mp-ci.centralus.cloudapp.azure.com/viewLog.html?buildId=15755&buildTypeId=Code_Development_Ddk_Feature) outcome was **SUCCESS**
Summary: Running Build time: 00:05:09


405 is the wrong status. just use 200


why an empty catch?


Is there a possibility to return integer instead of array? I can't see this place here in this method.
Maybe we can just pass @return array?


Unhandled Failure


These shotwell updates don't build (```Requested 'gexiv2 >= 0.10.4' but version of GExiv2 is 0.10.3```).

Please test changes before pushing, especially on the stable branch.


Табы vs пробелы, все разъехалось
лучше оставляй отступы в 4 пробела, это дефолт везде
даже гитхаб уверен что у тебя табы как у нормальных людей по 4 пробела а не по 2


Fuzzylite is 3rd party library and we don't modify it in VCMI. Especially if it's not broken.


pizzaroot did that


вопрос на засыпку, как, используя STL, избавиться от этих монструозных свитчей?


Todo - post restservice


document à rendre avant le 24 février sur Ametice


It was misunderstanding, i`ll fix it now.


Hope this password isn't valid anymore. :)


> PocketMine is a mess and spreading the existing mess even further for the sake of consistency is just laziness in my opinion.

First, it is in all right to keep consistency throughout the project.

If you saw PocketMine 1.3, you would know what a mess was. Shoghicp is one the best coders around.

PocketMine is not a mess, it has classes, uses functions, has documentation and the API has a good structure and works well. It's only when shoghicp left it became a mess, after a year of no updates dktapps is cleaning it and fixing what was left broken. You can't expect 0.12 server software (when shoghicp last updated it) to work flawlessly with 1.0.4 server software, can you?

PocketMine was never ment to be server software. It was a test to prove a PHP based server is possible. I recommend you learn the structure of OOP, there are many great library books about it. **The command will execute before the server process leaving, commands are not ran on a separate thread and need no synchronization**

If you want to nit-pick PocketMine go ahead, you can sure find one or two things that are messy. Besides the network processing, it is a well built project.

That is like saying you should run $player->isOnline() when every event is called. That is not optimized and stupid.

Even if the player disconnects, it will not receive any packets sent to it.

/weather set clear -> player online check -> execute
Makes no scene as at all and will always evaluate to true because the server won't process a disconnect while running the command. Unless you can predict the future of when a player disconnects your logic is skewed.

This has now gone off-topic. If you believe PocketMine is a mess go to forums and complain.  This discussion should now end. Since this is the main thread, you don't need to run online checks everywhere which is why we use OOP. You only need online checks if a event like PlayerJoinEvent was cancelled or you use $player->kick().


kiva (nealkruis) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


ich denke das sollten wir morgen Samstag noch etwas diskutieren?


This was done by me. Codenvy is slightly broken for commits


There is a possibility to return only 'even' or 'odd' key (when we'll pass $args variable as a integer).
In this case this line will throw an error (array key does not exist).
We have few options:
- always return both keys
- check if array key exist here
- your proposition ?


When using SQL::Abstract with multiple tables, it is sometimes necessary to disambiguate columns:

    $pg->db->select(['foo', 'bar'], ['foo.id'], {'foo.id', => {-ident => 'bar.id'}});

If SQL::Abstract does not have a name_sep specified, it will attempt to quote the entire table-field name as `"foo.id"` and `"bar.id"` rather than `"foo"."id"` and `"bar"."id"`, producing an error such as:

    DBD::Pg::st execute failed: ERROR:  column "foo.id" does not exist
    LINE 1: SELECT "foo.id" FROM "foo", "bar" WHERE ( "foo.id" = "bar.id...
                   ^ at lib/Test.pm line 50.

The workaround is to create the Mojo::Pg instance with:

        $pg = Mojo::Pg
          ->new($db_uri)
          ->abstract(SQL::Abstract->new(quote_char => '"', name_sep => '.'))
          ;

However, since the name_sep and quote_char values are intrinsic to Postgres, it makes sense for these to be the setting on the default SQL::Abstract object.


![](https://github.trello.services/images/mini-trello-icon.png) [/ativar](https://trello.com/c/dIqOLsnk/4-ativar)


Nice solution to move headers to the config.

You can use `Object.assign` to merge headers objects.


This branch is for test- purpose only, in this particular case clock gives an equivalent precision (~0.01) than the former omp time function. Thanks for the link, I will have a look on chrono and use it instead of clock.
Timers are going to be removed however though.


@leo these break the build 😢. peerDependencies should *not* be locked to one version. Creating a PR to fix it now 👍 🙌


Good idea.  I'd like to have tests for whole toolkit, too.  ;)


Lines 52-59 are similar to lines 64-70. Maybe it'll be a good idea to separate it to another method?
(DRY - Don't Repeat Yourself principle)


kiva (nealkruis) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: Tests Failed (631 of 1249 tests passed, 0 test warnings)

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


I'll push the patch upstream later on.


typo?
```
-    "id": "advanced_twaaks",
+    "id": "advanced_tweaks",
```


I think is missed a place where this escape is needed or this is not the correct place?


@JanMisker @gibbok unless I am misunderstanding, I think there are two different issues here. The latter issue is such that I don't believe you should be able to use getComputedStyle across domains. Perhaps this was failing silently before and now it's failing via an error? For the former issue, I'm thinking we may need to do something to detect win.global.window.parent vs. win.global.window and pick one based on permission/existence perhaps? Regardless, we'll investigate again.


이전에 api app 의 serializer/UserSerializer 와 결국엔 같은 결과값을 내보내는데,
OneToOne mapping 이라 별반 다를게 없나보다.


It doesn't work because of signal/slots connection ideology:

> String-based connections type-check by comparing strings at run-time. There are three limitations with this approach:
>
> - Connection errors can only be detected after the program has started running.
> - Implicit conversions cannot be done between signals and slots.
> - Typedefs and namespaces cannot be resolved.


WHY ?


`pbValidateConf()` is actually a function and not a class.  maybe it should read:
```
\item New helper functions \code{pbValidateConf}, \code{pbGetUser},
  and \code{pbGetChannelInfo} were added (Seth in \ghpr{44} closing
  \ghit{40}).
\item New classes  \code{pbUser} and
  \code{pbChannelInfo} were added (Seth in \ghpr{44}).
```

just a suggestion.


I have backed this hunk out.


@tchaikov I'll bet it actually gets compiled out by the compiler -- and it avoids the ugliness of #ifdefs


Le falta el //


I don't think the issue is CSS, as the divider is movable. It just needs an initial setting more to the right.


Makes no sense since
- no packages are being downloaded there
- most importantly the playbook picks the proxy settings from /etc/apt/apt.conf.d


-

 -
GitHub

 -
jekyll

 -
GitHub Pages

 -
Alan

 -


Methods "getEvenCount" and "getOddCount" are very similar. Maybe it'll be a shorter way to just remove one of them and add second argument to other? Something like:

/**
 * @param string[] $soldiers
 * @param int $soldiersUniform
 * @return int
 */
armyPoints(array $soldiers, int $soldiersUniform)


@csandanov appropriate your work man!

Notes about windows and www-data permission issues and not able to upload anything. Is it possible that we allow users to change the default www-data with root?

I built a custom image that uses root for user/group plus -R argument to the php-fpm

 -R, --allow-to-run-as-root Now i no longer have permission issues


If you want a scheme-less URL that's //127.0.0.1:80. See also #18824.


@skattyadz sorry about that, forgot to uncomment. Fixed in `v0.25.4` on npm.


Should the c taglib be added to styles.jspf?


- For the test to pass in the mooc add this 👉 `import java.util.Scanner;` in `Main.java`

- And change  this👇  in `LyyraCard.java`
 ```
public final double economicalPrice = 2.50;
 public final double gourmetPrice = 4.00;
```

- to this 👇
 ```
private final double economicalPrice = 2.50;
private final double gourmetPrice = 4.00;
```


![](https://github.trello.services/images/mini-trello-icon.png) [/sobre](https://trello.com/c/er2OAGv0/17-sobre)


@dillaman okay, i am just not sure where coverity kicks in to do the check. let's give it a try then!


You can do it in one line : P

return is_array($array) ? array_sum($array) : (int) $array;


Yes.

The snap part is widget-side. This here only affects AI. Of course AI also deserve some lenience but they are good at accurate mex placement - on the other hand, AI could get confused if the building it tried to place ended up somewhere else. So I think the value should be kept low.

For comparison 50 is used by ZK and AI work fine, while 500 is larger than a typical mex triplet.


merge from aenen


thx a lot, great word


Is this a (too) reasonable implementation?
IIR floats have exact representation of 16-bit integers, it would be ok to require ignore to be one of these, if we test this: `ignore_label == float(short(ignore_label))`. I think equality comparison is sufficiently safe because there are no mathematical operations on labels.


System-Fan-Redux (EnergyArchmage) - x86_64-MacOS-10.9-clang: OK (2385 of 2416 tests passed, 553 test warnings)

 * 583 tests had: AUD diffs.
 * 436 tests had: RDD diffs.
 * 29 tests had: BND diffs.
 * 30 tests had: EIO diffs.
 * 9 tests had: ESO big diffs.
 * 8 tests had: MTR big diffs.
 * 30 tests had: Table big diffs.
 * 6 tests had: ESO small diffs.
 * 9 tests had: MTR small diffs.
 * 1 test had: Table small diffs.
 * 4 tests had: MDD diffs.
 * 21 tests had: MTD diffs.
 * 8 tests had: ERR diffs.

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


в этом случае да


base :  base ||| this.name, ?


Q1 - NO
Q2 - Parameter check to filter out unwanted values
Q3 - NO
Q4 - NO


Fixes #1


test


This ring buffer used for 0..2046 and not for 0..2047 (like in `var % 2048`). Is it supposed to be so, or it is just ringed in this way?


Patchfile is missing, but maybe it's never used because the new pre-fetch check errors out on Darwin < 12 (Darwin == 11, should probably be true on < 11 too)?


Typo in the file name


Hey Bruce. Is there any reason you got rid of these automatic platform defines? I dont get why the user has to define these themselves, particularly when working on multiplatform projects.


With a single % it gives:
`Bad local forwarding specification 'H:G'`


- UpperCase them constants 😄
- Nested if looks ugly , separated them 🙏
- Add them ❤ly  **decorater** @Override, so that we don't wonder why we over rode our toString method from baseClass 😄


maybe move this down after that first check?


@danielbarron, @VariableDeclared, @lgallacher Could you guys have a look at the changes and let me know what you think it's not a formal PR as we didn't have discussion on whether we want to push this to upstream or not.


Thanks! I didn't notice that.


Sure, making the change.

If everything else looks good I'll push the baby to CRAN.


These issues should be fixed in commit 953efb82


Not much of an easter egg if it's in the commit log 😉


this line is repeated 3 times, maybe just add it to default statement?


on a second thought, i think the coverity will still complain even we change this way: the same reason: it's dead code in run-time. anyway... better than before.


Huh?


HR ER diagram


this function can be merged with the one above, checking when the first * is found that the next char is \0 or not.


yeah, I think it's good.


Why the whole file is marked as changed?


-[X] add request


fixed.


missing newline.


-[]add request


Sad to remove them. We always wanted to make them display and be synced.


@inferno-chromium @oliverchang should we set up one more machine to run as a Jenkins slave?


this can be done with a static inline.


@tchaikov Coverity was complaining about the "1 > 1"-style argument -- if Coverity complained about dead code it would trip up all over the place since there are tons of examples in Ceph of "if (0)" / "if (true)"


@SirPryderi You can add the album creation here, then use "create_album" as action in upload.php and "json" as the json content


Wawwoow python


TeamCity SOEN341 Team3 :: Build [Build 14](http://localhost:81/viewLog.html?buildId=15&buildTypeId=Soen341Team3_Build) is now running


TeamCity SOEN341 Team3 :: Build [Build 14](http://localhost:81/viewLog.html?buildId=15&buildTypeId=Soen341Team3_Build) outcome was **SUCCESS**
Summary: Running Build time: 00:00:05


Le falta ID
required="required" es lo mismo que `required` solito


Recomiendo cambiar los values a algo mas "manipulable"


Sure, these can be brought back. But it'll have to be a gadget since the engine no longer does anything.


This conflicts with the token storage file that the rapp-api uses for rapp-platform authentication! ([source](https://github.com/rapp-project/rapp-api/tree/master/python#authenticationtokens))
RAPP Platform and RAPP Store token databases are distinct


`\code{pbChannelInfo` -> `\code{pbChannelInfo}`


Yeah, avoiding variable allocation is a good idea.  Cast wouldn't allow the conversion, but I can't remember why that was.  I'll look into changing the cast behavior.


"last star" -> last star currently encountered or last star of the pattern ?


fixes #86


Falta:
- tags
- el texto de la pregunta debería ser un textarea
- considerar que deben aceptar poder pegar código y que se vea "bonito"
- boton de submit cambiar el texto


kiva (nealkruis) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: Tests Failed (632 of 1842 tests passed, 0 test warnings)

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


We could just show it on stats screen ...


kiva (nealkruis) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

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


but `sizeof(info.size) > sizeof(unsigned long)` is also a run-time constant. it's equivalent to `1 > 1` from coverity's point of view, so does `(1 > 1 && expr )`.


OK, what is `gi` then? `PyGObject`? How easy it can be installed? Does it contain C extensions or other dependencies? I thought it has many dependencies.


This commit has been deployed to DEPLOY


Yes, there's still a bit more cleanup that can be done around the upgrade handling.


가장 최근에 입력된 행을 얻어오기 위해서 전체를 소팅하는게 효율적으로 보이지는 않네.. 다른 방법을 찾아보자.


커밋 메시지를 조금 더 구체적으로 작성해보자.


@tchaikov simple fix: "sizeof(uint64_t) != sizeof(unsigned long)" --> "true" or "false"


@lizongzeshunshun 又bug了。。用户名又变成了unknown


Some compilers will complain about assigning unsigned to signed ints, so better use an int32_t here or just ssize_t as in IntervalSet.


:heavy_exclamation_mark: There are 11 visual regressions in this pull request

Please Visit https://github.com to check out these diffs
* screen **back to Rogue on: a star wars story** on **iPhone6** for **en-US** locale
* screen **star wars episode IV: a new hope** on **iPhone5** for **en-US** locale
* screen **back to Rogue on: a star wars story** on **iPhone7** for **en-US** locale
* screen **back to Rogue on: a star wars story** on **iPhone6Plus** for **en-US** locale
* screen **star wars episode IV: a new hope** on **iPhone7** for **en-US** locale
* screen **Rogue One: a star wars story** on **iPhone6** for **en-US** locale
* screen **Rogue One: a star wars story** on **iPhone6Plus** for **en-US** locale
* screen **Rogue One: a star wars story** on **iPhone5** for **en-US** locale
* screen **Rogue One: a star wars story** on **iPhone7** for **en-US** locale
* screen **star wars episode IV: a new hope** on **iPhone6** for **en-US** locale
* screen **star wars episode IV: a new hope** on **iPhone6Plus** for **en-US** locale

| Diff 1 | Diff 2 | Diff 3 | Diff 4 | Diff 5 | Diff 6 | Diff 7 | Diff 8 | Diff 9 | Diff 10 | Diff 11 |
| ------------- | ------------- | ------------- | ------------- | ------------- | ------------- | ------------- | ------------- | ------------- | ------------- | ------------- |
| [![Image of Diff1](https://appsnap.s3-eu-west-1.amazonaws.com/bcojan/movies/4/4.1/diffs/en-US_iPhone6-back%20to%20Rogue%20on%3A%20a%20star%20wars%20story-d41d8cd98f00b204e9800998ecf8427e_thumbnail.png)](http://localhost:3000/diffs/588b0e3db9e9f31f0cb57459/5885f205360140da2e3762ee/588b34f983fa402cd6e9f00a) | [![Image of Diff2](https://appsnap.s3-eu-west-1.amazonaws.com/bcojan/movies/4/4.1/diffs/en-US_iPhone5-star%20wars%20episode%20IV%3A%20a%20new%20hope-d41d8cd98f00b204e9800998ecf8427e_thumbnail.png)](http://localhost:3000/diffs/588b0e3db9e9f31f0cb57459/5885f205360140da2e3762ee/588b34f983fa402cd6e9f00a) | [![Image of Diff3](https://appsnap.s3-eu-west-1.amazonaws.com/bcojan/movies/4/4.1/diffs/en-US_iPhone7-back%20to%20Rogue%20on%3A%20a%20star%20wars%20story-d41d8cd98f00b204e9800998ecf8427e_thumbnail.png)](http://localhost:3000/diffs/588b0e3db9e9f31f0cb57459/5885f205360140da2e3762ee/588b34f983fa402cd6e9f00a) | [![Image of Diff4](https://appsnap.s3-eu-west-1.amazonaws.com/bcojan/movies/4/4.1/diffs/en-US_iPhone6Plus-back%20to%20Rogue%20on%3A%20a%20star%20wars%20story-d41d8cd98f00b204e9800998ecf8427e_thumbnail.png)](http://localhost:3000/diffs/588b0e3db9e9f31f0cb57459/5885f205360140da2e3762ee/588b34f983fa402cd6e9f00a) | [![Image of Diff5](https://appsnap.s3-eu-west-1.amazonaws.com/bcojan/movies/4/4.1/diffs/en-US_iPhone7-star%20wars%20episode%20IV%3A%20a%20new%20hope-d41d8cd98f00b204e9800998ecf8427e_thumbnail.png)](http://localhost:3000/diffs/588b0e3db9e9f31f0cb57459/5885f205360140da2e3762ee/588b34f983fa402cd6e9f00a) | [![Image of Diff6](https://appsnap.s3-eu-west-1.amazonaws.com/bcojan/movies/4/4.1/diffs/en-US_iPhone6-Rogue%20One%3A%20a%20star%20wars%20story-d41d8cd98f00b204e9800998ecf8427e_thumbnail.png)](http://localhost:3000/diffs/588b0e3db9e9f31f0cb57459/5885f205360140da2e3762ee/588b34f983fa402cd6e9f00a) | [![Image of Diff7](https://appsnap.s3-eu-west-1.amazonaws.com/bcojan/movies/4/4.1/diffs/en-US_iPhone6Plus-Rogue%20One%3A%20a%20star%20wars%20story-d41d8cd98f00b204e9800998ecf8427e_thumbnail.png)](http://localhost:3000/diffs/588b0e3db9e9f31f0cb57459/5885f205360140da2e3762ee/588b34f983fa402cd6e9f00a) | [![Image of Diff8](https://appsnap.s3-eu-west-1.amazonaws.com/bcojan/movies/4/4.1/diffs/en-US_iPhone5-Rogue%20One%3A%20a%20star%20wars%20story-d41d8cd98f00b204e9800998ecf8427e_thumbnail.png)](http://localhost:3000/diffs/588b0e3db9e9f31f0cb57459/5885f205360140da2e3762ee/588b34f983fa402cd6e9f00a) | [![Image of Diff9](https://appsnap.s3-eu-west-1.amazonaws.com/bcojan/movies/4/4.1/diffs/en-US_iPhone7-Rogue%20One%3A%20a%20star%20wars%20story-d41d8cd98f00b204e9800998ecf8427e_thumbnail.png)](http://localhost:3000/diffs/588b0e3db9e9f31f0cb57459/5885f205360140da2e3762ee/588b34f983fa402cd6e9f00a) | [![Image of Diff10](https://appsnap.s3-eu-west-1.amazonaws.com/bcojan/movies/4/4.1/diffs/en-US_iPhone6-star%20wars%20episode%20IV%3A%20a%20new%20hope-d41d8cd98f00b204e9800998ecf8427e_thumbnail.png)](http://localhost:3000/diffs/588b0e3db9e9f31f0cb57459/5885f205360140da2e3762ee/588b34f983fa402cd6e9f00a) | [![Image of Diff11](https://appsnap.s3-eu-west-1.amazonaws.com/bcojan/movies/4/4.1/diffs/en-US_iPhone6Plus-star%20wars%20episode%20IV%3A%20a%20new%20hope-d41d8cd98f00b204e9800998ecf8427e_thumbnail.png)](http://localhost:3000/diffs/588b0e3db9e9f31f0cb57459/5885f205360140da2e3762ee/588b34f983fa402cd6e9f00a) |


2AM bugfixes for the win right?  Thank you.


refer by: [http://stackoverflow.com/questions/32621988/electron-jquery-is-not-defined](http://stackoverflow.com/questions/32621988/electron-jquery-is-not-defined)


For some reason, these files might be include'd after rtmidi.h, so they had no effect. Rtmidi falls back to the dummy midi driver instead of the right OS driver, which is useless.
In my project settings on Visual Studio I have to manually put __WINDOWS_MM__ preprocessor variable, which is a pain, but it works for now.
I removed them until I find a solution...


이 방법 보다는 처음부터 서버에서 시간 값을 얻어올때 Timezone 을 같이 보내도록 설정하는게 나을것 같아.


:heavy_exclamation_mark: There are 11 visual regressions in this pull request

Please Visit https://github.com to check out these diffs
* screen **Rogue One: a star wars story** on **iPhone5** for **en-US** locale
* screen **back to Rogue on: a star wars story** on **iPhone6** for **en-US** locale
* screen **Rogue One: a star wars story** on **iPhone7** for **en-US** locale
* screen **back to Rogue on: a star wars story** on **iPhone7** for **en-US** locale
* screen **back to Rogue on: a star wars story** on **iPhone6Plus** for **en-US** locale
* screen **star wars episode IV: a new hope** on **iPhone5** for **en-US** locale
* screen **star wars episode IV: a new hope** on **iPhone6** for **en-US** locale
* screen **Rogue One: a star wars story** on **iPhone6** for **en-US** locale
* screen **star wars episode IV: a new hope** on **iPhone7** for **en-US** locale
* screen **Rogue One: a star wars story** on **iPhone6Plus** for **en-US** locale
* screen **star wars episode IV: a new hope** on **iPhone6Plus** for **en-US** locale

| Diff 1 | Diff 2 | Diff 3 | Diff 4 | Diff 5 | Diff 6 | Diff 7 | Diff 8 | Diff 9 | Diff 10 | Diff 11 |
| ------------- | ------------- | ------------- | ------------- | ------------- | ------------- | ------------- | ------------- | ------------- | ------------- | ------------- |
| [![Image of Diff1](https://appsnap.s3-eu-west-1.amazonaws.com/bcojan/movies/4/4.1/diffs/en-US_iPhone5-Rogue%20One%3A%20a%20star%20wars%20story-d41d8cd98f00b204e9800998ecf8427e_thumbnail.png)](http://localhost:3000/diffs/588b0e3db9e9f31f0cb57459/5885f205360140da2e3762ee/588b34f983fa402cd6e9f00a) | [![Image of Diff2](https://appsnap.s3-eu-west-1.amazonaws.com/bcojan/movies/4/4.1/diffs/en-US_iPhone6-back%20to%20Rogue%20on%3A%20a%20star%20wars%20story-d41d8cd98f00b204e9800998ecf8427e_thumbnail.png)](http://localhost:3000/diffs/588b0e3db9e9f31f0cb57459/5885f205360140da2e3762ee/588b34f983fa402cd6e9f00a) | [![Image of Diff3](https://appsnap.s3-eu-west-1.amazonaws.com/bcojan/movies/4/4.1/diffs/en-US_iPhone7-Rogue%20One%3A%20a%20star%20wars%20story-d41d8cd98f00b204e9800998ecf8427e_thumbnail.png)](http://localhost:3000/diffs/588b0e3db9e9f31f0cb57459/5885f205360140da2e3762ee/588b34f983fa402cd6e9f00a) | [![Image of Diff4](https://appsnap.s3-eu-west-1.amazonaws.com/bcojan/movies/4/4.1/diffs/en-US_iPhone7-back%20to%20Rogue%20on%3A%20a%20star%20wars%20story-d41d8cd98f00b204e9800998ecf8427e_thumbnail.png)](http://localhost:3000/diffs/588b0e3db9e9f31f0cb57459/5885f205360140da2e3762ee/588b34f983fa402cd6e9f00a) | [![Image of Diff5](https://appsnap.s3-eu-west-1.amazonaws.com/bcojan/movies/4/4.1/diffs/en-US_iPhone6Plus-back%20to%20Rogue%20on%3A%20a%20star%20wars%20story-d41d8cd98f00b204e9800998ecf8427e_thumbnail.png)](http://localhost:3000/diffs/588b0e3db9e9f31f0cb57459/5885f205360140da2e3762ee/588b34f983fa402cd6e9f00a) | [![Image of Diff6](https://appsnap.s3-eu-west-1.amazonaws.com/bcojan/movies/4/4.1/diffs/en-US_iPhone5-star%20wars%20episode%20IV%3A%20a%20new%20hope-d41d8cd98f00b204e9800998ecf8427e_thumbnail.png)](http://localhost:3000/diffs/588b0e3db9e9f31f0cb57459/5885f205360140da2e3762ee/588b34f983fa402cd6e9f00a) | [![Image of Diff7](https://appsnap.s3-eu-west-1.amazonaws.com/bcojan/movies/4/4.1/diffs/en-US_iPhone6-star%20wars%20episode%20IV%3A%20a%20new%20hope-d41d8cd98f00b204e9800998ecf8427e_thumbnail.png)](http://localhost:3000/diffs/588b0e3db9e9f31f0cb57459/5885f205360140da2e3762ee/588b34f983fa402cd6e9f00a) | [![Image of Diff8](https://appsnap.s3-eu-west-1.amazonaws.com/bcojan/movies/4/4.1/diffs/en-US_iPhone6-Rogue%20One%3A%20a%20star%20wars%20story-d41d8cd98f00b204e9800998ecf8427e_thumbnail.png)](http://localhost:3000/diffs/588b0e3db9e9f31f0cb57459/5885f205360140da2e3762ee/588b34f983fa402cd6e9f00a) | [![Image of Diff9](https://appsnap.s3-eu-west-1.amazonaws.com/bcojan/movies/4/4.1/diffs/en-US_iPhone7-star%20wars%20episode%20IV%3A%20a%20new%20hope-d41d8cd98f00b204e9800998ecf8427e_thumbnail.png)](http://localhost:3000/diffs/588b0e3db9e9f31f0cb57459/5885f205360140da2e3762ee/588b34f983fa402cd6e9f00a) | [![Image of Diff10](https://appsnap.s3-eu-west-1.amazonaws.com/bcojan/movies/4/4.1/diffs/en-US_iPhone6Plus-Rogue%20One%3A%20a%20star%20wars%20story-d41d8cd98f00b204e9800998ecf8427e_thumbnail.png)](http://localhost:3000/diffs/588b0e3db9e9f31f0cb57459/5885f205360140da2e3762ee/588b34f983fa402cd6e9f00a) | [![Image of Diff11](https://appsnap.s3-eu-west-1.amazonaws.com/bcojan/movies/4/4.1/diffs/en-US_iPhone6Plus-star%20wars%20episode%20IV%3A%20a%20new%20hope-d41d8cd98f00b204e9800998ecf8427e_thumbnail.png)](http://localhost:3000/diffs/588b0e3db9e9f31f0cb57459/5885f205360140da2e3762ee/588b34f983fa402cd6e9f00a) |


Why are there 2 deserializeSets() calls? And there is no path in the case ADDED_UNICODE_SMP is not available.


Nice work dude !!


string and glob string is really invalid ?


really ?


Hm, we are currently running one 10-CPU builder. Looking at the CPU utilization graph (attached), it might be better to add new slaves with 1-2 CPUs, as roughly half of time builder is not busy with compilation and CPUs are idle.
![image](https://cloud.githubusercontent.com/assets/3372458/23068541/61f9a94e-f524-11e6-9dfb-d42e08e6e7be.png)
![image](https://cloud.githubusercontent.com/assets/3372458/23068546/653f2318-f524-11e6-92c2-535143093647.png)


because it was stored with wrong newlines


Yep, hacking may lead to problem eventually.

There are over 40 new fixers.

Search for "feature" in"

- https://github.com/FriendsOfPHP/PHP-CS-Fixer/releases/tag/v2.0.0
- https://github.com/FriendsOfPHP/PHP-CS-Fixer/releases/tag/v2.1.0


You can avoid all this "single-value cases" simply replacing this one value into array on this layer.


:heavy_exclamation_mark: There are 11 visual regressions in this pull request

Please Visit https://github.com to check out these diffs
* screen **back to Rogue on: a star wars story** on **iPhone6** for **en-US** locale
* screen **star wars episode IV: a new hope** on **iPhone5** for **en-US** locale
* screen **Rogue One: a star wars story** on **iPhone6** for **en-US** locale
* screen **star wars episode IV: a new hope** on **iPhone7** for **en-US** locale
* screen **back to Rogue on: a star wars story** on **iPhone7** for **en-US** locale
* screen **star wars episode IV: a new hope** on **iPhone6** for **en-US** locale
* screen **Rogue One: a star wars story** on **iPhone7** for **en-US** locale
* screen **Rogue One: a star wars story** on **iPhone5** for **en-US** locale
* screen **back to Rogue on: a star wars story** on **iPhone6Plus** for **en-US** locale
* screen **star wars episode IV: a new hope** on **iPhone6Plus** for **en-US** locale
* screen **Rogue One: a star wars story** on **iPhone6Plus** for **en-US** locale

| Diff 1 | Diff 2 | Diff 3 | Diff 4 | Diff 5 | Diff 6 | Diff 7 | Diff 8 | Diff 9 | Diff 10 | Diff 11 |
| ------------- | ------------- | ------------- | ------------- | ------------- | ------------- | ------------- | ------------- | ------------- | ------------- | ------------- |
| [![Image of Diff1](https://appsnap.s3-eu-west-1.amazonaws.com/bcojan/movies/4/4.1/diffs/en-US_iPhone6-back%20to%20Rogue%20on%3A%20a%20star%20wars%20story-d41d8cd98f00b204e9800998ecf8427e_thumbnail.png)](http://localhost:3000/diffs/588b0e3db9e9f31f0cb57459/5885f205360140da2e3762ee/588b34f983fa402cd6e9f00a) | [![Image of Diff2](https://appsnap.s3-eu-west-1.amazonaws.com/bcojan/movies/4/4.1/diffs/en-US_iPhone5-star%20wars%20episode%20IV%3A%20a%20new%20hope-d41d8cd98f00b204e9800998ecf8427e_thumbnail.png)](http://localhost:3000/diffs/588b0e3db9e9f31f0cb57459/5885f205360140da2e3762ee/588b34f983fa402cd6e9f00a) | [![Image of Diff3](https://appsnap.s3-eu-west-1.amazonaws.com/bcojan/movies/4/4.1/diffs/en-US_iPhone6-Rogue%20One%3A%20a%20star%20wars%20story-d41d8cd98f00b204e9800998ecf8427e_thumbnail.png)](http://localhost:3000/diffs/588b0e3db9e9f31f0cb57459/5885f205360140da2e3762ee/588b34f983fa402cd6e9f00a) | [![Image of Diff4](https://appsnap.s3-eu-west-1.amazonaws.com/bcojan/movies/4/4.1/diffs/en-US_iPhone7-star%20wars%20episode%20IV%3A%20a%20new%20hope-d41d8cd98f00b204e9800998ecf8427e_thumbnail.png)](http://localhost:3000/diffs/588b0e3db9e9f31f0cb57459/5885f205360140da2e3762ee/588b34f983fa402cd6e9f00a) | [![Image of Diff5](https://appsnap.s3-eu-west-1.amazonaws.com/bcojan/movies/4/4.1/diffs/en-US_iPhone7-back%20to%20Rogue%20on%3A%20a%20star%20wars%20story-d41d8cd98f00b204e9800998ecf8427e_thumbnail.png)](http://localhost:3000/diffs/588b0e3db9e9f31f0cb57459/5885f205360140da2e3762ee/588b34f983fa402cd6e9f00a) | [![Image of Diff6](https://appsnap.s3-eu-west-1.amazonaws.com/bcojan/movies/4/4.1/diffs/en-US_iPhone6-star%20wars%20episode%20IV%3A%20a%20new%20hope-d41d8cd98f00b204e9800998ecf8427e_thumbnail.png)](http://localhost:3000/diffs/588b0e3db9e9f31f0cb57459/5885f205360140da2e3762ee/588b34f983fa402cd6e9f00a) | [![Image of Diff7](https://appsnap.s3-eu-west-1.amazonaws.com/bcojan/movies/4/4.1/diffs/en-US_iPhone7-Rogue%20One%3A%20a%20star%20wars%20story-d41d8cd98f00b204e9800998ecf8427e_thumbnail.png)](http://localhost:3000/diffs/588b0e3db9e9f31f0cb57459/5885f205360140da2e3762ee/588b34f983fa402cd6e9f00a) | [![Image of Diff8](https://appsnap.s3-eu-west-1.amazonaws.com/bcojan/movies/4/4.1/diffs/en-US_iPhone5-Rogue%20One%3A%20a%20star%20wars%20story-d41d8cd98f00b204e9800998ecf8427e_thumbnail.png)](http://localhost:3000/diffs/588b0e3db9e9f31f0cb57459/5885f205360140da2e3762ee/588b34f983fa402cd6e9f00a) | [![Image of Diff9](https://appsnap.s3-eu-west-1.amazonaws.com/bcojan/movies/4/4.1/diffs/en-US_iPhone6Plus-back%20to%20Rogue%20on%3A%20a%20star%20wars%20story-d41d8cd98f00b204e9800998ecf8427e_thumbnail.png)](http://localhost:3000/diffs/588b0e3db9e9f31f0cb57459/5885f205360140da2e3762ee/588b34f983fa402cd6e9f00a) | [![Image of Diff10](https://appsnap.s3-eu-west-1.amazonaws.com/bcojan/movies/4/4.1/diffs/en-US_iPhone6Plus-star%20wars%20episode%20IV%3A%20a%20new%20hope-d41d8cd98f00b204e9800998ecf8427e_thumbnail.png)](http://localhost:3000/diffs/588b0e3db9e9f31f0cb57459/5885f205360140da2e3762ee/588b34f983fa402cd6e9f00a) | [![Image of Diff11](https://appsnap.s3-eu-west-1.amazonaws.com/bcojan/movies/4/4.1/diffs/en-US_iPhone6Plus-Rogue%20One%3A%20a%20star%20wars%20story-d41d8cd98f00b204e9800998ecf8427e_thumbnail.png)](http://localhost:3000/diffs/588b0e3db9e9f31f0cb57459/5885f205360140da2e3762ee/588b34f983fa402cd6e9f00a) |


1. Separar imagenes, media, css y js en sus respectivos folders
2. Recomiendo usar fontawesome
3. Comparar para revisar orden de opciones en menu
4. Validar estructura del home
5. Tabla debería estar en admin


either remove this or move it in the main method. writing to a file has nothing to do with getting the html. please follow proper encapsulation principles.


por que esta la llave en la misma linea que lo demás


A poster sur Ametice avant 24 Février
Ne pas oublier de décrire les CRUD (note de fin du document)


projects_that_show_my_stupidity_in_their_git_history += 1


The `".git"` can be a file instead of a directory (e.g. when working with Git submodules), hence the error code returned is `"ENOTDIR"`.


Falta el figcaption


> cannel

channel

> setted

dafuq is  that


this should be put into two duplicated cases (not a fallthrough) so we don't have this extra branch on the fast path.


para las columnas de precio usar una clase


CR OK!


Myself I [trying to use OpenSL ES](https://github.com/Wohlstand/ADLMIDI-Player-Java/blob/master/app/src/main/cpp/native-lib.cpp) and seems something is wrong here, or I built it without some important optimization flags.


Esto debería ser un nav


I don't like this you have made!!!
Better if you try this `Hello

> @mticona

> [Kesha`](url)


MM ok


@dillaman sorry, i don't follow you.

also i changed this way: https://github.com/ceph/ceph/pull/13473/commits/a36919d87963140e2f26e271a15e32a6fe5bf2bc so we don't encode the type into the code. it's quite the same as `sizeof(uint64_t) != sizeof(unsigned long)`.
 is this what you are suggesting?


If Team A played Team B in Season 1, they will not get matched due to this line in Season 2


extract the constant part into a static constant string outside the method. you can also create a new method that sets up this url.
the content.split() thing will not work for special characters, which is okay, but it should be noted in the method documentation (which is missing).
Also, 0 based page numbering is fine too. Please add the pageNum indexing description (which is the value for the first page? I see it's currently 1, but this should be documented!)


TeamCity Chorus :: chorus-precise64-master Continuous [Build 2.6.735.08d615a412d2a29992932d0fb3bad16df0b1a5f3](http://build.palaso.org/viewLog.html?buildId=84659&buildTypeId=bt323) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:00:40


if both are 0, why do we keep them separate ? Why not just choose either STAR_GLOB or wildcard ?


TeamCity Chorus :: chorus-win32-master Continuous [Build 2.6.369.08d615a412d2a29992932d0fb3bad16df0b1a5f3](http://build.palaso.org/viewLog.html?buildId=84660&buildTypeId=bt2) outcome was **FAILURE**
Summary: Compilation error: src\LibChorus\LibChorus.csproj (new) Build time: 00:00:54


There was no 'draw' option in game


hmmmm.... still getting same error here just on a line 121 rather that line 120:

/bin/sh ../../libtool  --tag=CXX   --mode=compile g++ -DHAVE_CONFIG_H -I. -I../../src  -I../../src   -DPD -I/Applications/Pd.app/Contents/Resources/include  -DMAGICKCORE_HDRI_ENABLE=1 -DMAGICKCORE_QUANTUM_DEPTH=16 -I/usr/local/Cellar/imagemagick/7.0.4-8/include/ImageMagick-7 -g -O2 -Os -falign-loops -falign-functions -falign-jumps -funroll-loops -ffast-math -mmmx -fpascal-strings -MT gem_imageMAGICK_la-imageMAGICK.lo -MD -MP -MF .deps/gem_imageMAGICK_la-imageMAGICK.Tpo -c -o gem_imageMAGICK_la-imageMAGICK.lo `test -f 'imageMAGICK.cpp' || echo './'`imageMAGICK.cpp
libtool: compile:  g++ -DHAVE_CONFIG_H -I. -I../../src -I../../src -DPD -I/Applications/Pd.app/Contents/Resources/include -DMAGICKCORE_HDRI_ENABLE=1 -DMAGICKCORE_QUANTUM_DEPTH=16 -I/usr/local/Cellar/imagemagick/7.0.4-8/include/ImageMagick-7 -g -O2 -Os -falign-loops -falign-functions -falign-jumps -funroll-loops -ffast-math -mmmx -fpascal-strings -MT gem_imageMAGICK_la-imageMAGICK.lo -MD -MP -MF .deps/gem_imageMAGICK_la-imageMAGICK.Tpo -c imageMAGICK.cpp  -fno-common -DPIC -o .libs/gem_imageMAGICK_la-imageMAGICK.o
clang: warning: optimization flag '-falign-loops' is not supported
clang: warning: optimization flag '-falign-functions' is not supported
clang: warning: optimization flag '-falign-jumps' is not supported
In file included from imageMAGICK.cpp:23:
In file included from ./imageMAGICK.h:17:
../../src/plugins/imagesaver.h:71:97: warning: '/*' within block comment [-Wcomment]
       *                                     even though we only expect mimetypes of type 'image/*', the prefix (...
                                                                                ^
imageMAGICK.cpp:121:3: error: use of undeclared identifier 'GetExceptionInfo'
  GetExceptionInfo(&exception);
  ^
1 warning and 1 error generated.
make[2]: *** [gem_imageMAGICK_la-imageMAGICK.lo] Error 1
make[1]: *** [all-recursive] Error 1
make: *** [all-recursive] Error 1


Can we not do `[*value].map { ... }` instead?


Si on met pas cette ligne, il recrée un MainActivity à chaque flip de l'écran...


I wouldn't use OpenSL for a music player.
OpenSL is only useful if you require low-latency (ie: a synthesizer), for a Player the audio latency doesn't matter much.


I'm sure this works great, but I would prefer if instead of implementing this we could just use something that is already done for us, such as [password-generator](https://github.com/bermi/password-generator)

The advantage is that it's code already tested and that we don't have to maintain. Plus, it's already a npm package so we can re-use it anywhere we like.


hash.each __please__ :)


File should have a little bit different format.
First argument should be a single number (matrix size) and also should accept multiple matrix


typo "started first, ne simple way"


Cambiar por input de tipo search


Fixed, will be pushed and published tomorrow.


Prefer `alias`, and underscored methods


Prim via the use of Adjacency Matrix


Maybe exception instead of false return?
Something like FileDoesNotExistException?


@Guillem-db Please, have a look to my corrections. I fixed some wording, typos, etc.

Both notebooks works in Py3.


Hmm, ok fair enough. This block definitely used to successfully detect the OS automatically though, and so far I haven't been able to get the block to use anything other than the dummy midi input even when defining the __MACOSX_CORE__ variable...


We should really move all of this somewhere else. It doesn't make sense that we're caching to a file somewhere automatically.


dgjdyj


Thanks


Is this field $this->file necessary?


yes in the past it worked...


Due to not being sure where te f*ck-up is, changing them all back. This commit actually caused the deathfloodtresholdban-bug!


something is broken since I updated rtmidi to the latest version


need to include fees in pre-check, else they might push total out > max_money

raising max_money +1 gives a bit of room for fees
max_money/2 is not a neat 1billion, but close enough


Ah ok gotcha.


I think I mentioned this in my review as well. I totally agree with this.


Maybe it'll be a good idea to replace it just by radiobutton?


@manulita and @s-mueller Just for your info (e.g. for testing or the study in March): I've made the IsDetailedCollectionEnabled be false by default. Tracking the detailed data (which causes an increased resources and data consumption) should be explicitly enabled for a study only.


Remove this newline please 👍


@tchaikov I guess it will just depend on whether or not Coverity is throwing the issue because on 64bit platforms "UINT64_MAX > UINT64_MAX" is not possible or because it doesn't like ">" / "<" comparisons that will evaluate to a constant. Either way, I think your current change would hit both cases unless you change the first comparison to "!=".


This file is a naive view implementation so I think it will be better idea to check matrix correctness in separated class.


Added


тоби пизда


These methods could probably be moved out of this extensions class somewhere. Question is where?


Yeah. Water ripples would be way more complicated though


libADLMIDI itself is a layer over synthesizer which generating sound from scratch, and in the code I generating chunks taken from libADLMIDI. In first time I have used AudioTrack, but I wish to reduce battery eating and implemented the OpenSL ES.
If I understand, do you running generating process in separated thread and Audio API just using the pre-generated buffer chunk? In my case, generation is going in every Audio API's callback execution (I did same with AudioTrack until I moved to OpenSL ES). I think, yea, I would try to do same parallel generation


Of course I mean #59


@phhusson

+ '[' 4 '==' 0 ']'
+ set -e
+ '[' -f eng ']'
+ scr=/tmp/superuser/scripts/su/changes.sh
+ trap cleanup EXIT
+ readlink -f ../bootimg.sh
+ dirname /tmp/superuser/scripts/bootimg.sh
+ scriptdir=/tmp/superuser/scripts
+ chmod 0755 /tmp/superuser/scripts/bin/sepolicy-inject
+ chmod 0755 /tmp/superuser/scripts/bin/sepolicy-inject-v2
+ chmod 0755 /tmp/superuser/scripts/bin/bootimg-repack
+ chmod 0755 /tmp/superuser/scripts/bin/bootimg-extract
+ chmod 0755 /tmp/superuser/scripts/bin/strip-cpio
+ sh -c '[[ "toto2" =~ "toto" ]]'
sh: syntax error: '=~' unexpected operator/operand
../bootimg.sh[75]: syntax error: '=~' unexpected operator/operand
+ cleanup
+ rm -Rf


Remove the `fr` subdomain from the URL so it redirects to the users Wikipedia language


"MySQL xPaaS image for OpenShift " is incorrect (not part of xPaaS), probably should be just "MySQL image for OpenShift", or "OpenShift MySQL image"


This block of code is similar to code before. I think it is possible to make code a little bit shorter (DRY)


Что то с выравниванием. Нет симметрии


DRY


В одну строку сложно понять. Лучше разбивать.


TeamCity Platform Services :: Environment Manager 2 :: Package [Build 3.32.1](http://infra.teamcity.ci.ttldev/viewLog.html?buildId=51426&buildTypeId=PlatformServices_EnvironmentManager2_Package) outcome was **SUCCESS**
Summary: Running Build time: 00:00:47


"MySQL xPaaS image for OpenShift"   mysql image is not part of xPaaS,  also opened https://issues.jboss.org/browse/OSFUSE-565 as this comes from the readmes where it is also wrong.


Needs preamble


TeamCity SOEN341 Team3 :: Build [Build 15](http://localhost:81/viewLog.html?buildId=16&buildTypeId=Soen341Team3_Build) is now running


TeamCity SOEN341 Team3 :: Build [Build 15](http://localhost:81/viewLog.html?buildId=16&buildTypeId=Soen341Team3_Build) outcome was **FAILURE**
Summary: Cannot start build runner; exit code -42 (new) Build time: 00:00:11


This commit has been deployed to TEST


I guess this part of code can be at least 50% shorter


@janicduplessis This is causing the CI builds to break.

The specific error looks like it's missing a build step for Executor.cpp.

```
make: *** No rule to make target `/home/ubuntu/react-native/ReactAndroid/../ReactCommon/cxxreact/Executor.cpp', needed by `/home/ubuntu/react-native/ReactAndroid/build/tmp/buildReactNdkLib/local/armeabi-v7a/objs/reactnativefb/Executor.o'.  Stop.
make: Leaving directory `/home/ubuntu/react-native/ReactAndroid/src/main/jni/react/jni'
```


Выравние


As said, the specific issue was because one executable script in the project had the line endings. That prevented Linux from properly executing the script.

If we want to be accurate and fair, it was not "Cloud Foundry configuration".


Выравнивание


![](https://github.trello.services/images/mini-trello-icon.png) [Powinna być inna klasa która trzyma widok (ona by miała tez `width` i `height`, miałaby `JButtony`, miałaby metody do tworzenia listenerów i `JButtonów`) etc](https://trello.com/c/dRFCCA0I/51-powinna-byc-inna-klasa-ktora-trzyma-widok-ona-by-mia-a-tez-width-i-height-mia-aby-jbuttony-mia-aby-metody-do-tworzenia-listenero)


Аналогично


Really?  It works for me.  How are you setting the environment variable?


Method returns value, not a path so I guess name is a little bit confusing.
Maybe something like:
voyageCost(); or
tripDistance(); or
distance(); or
your proposition?


Could be good to use a fixture here instead of one of the real frameworks...

More willing to let this one slide though as it was already doing this so you aren't making the behaviour any worse...


This should only happen for tools that are labeled to preserve the python environment. Is that really the case, and should we not rather work on removing that requirement?


This pattern seems to be used in a few places in these tests.


Again, would be nice on the fixture but not the end of the world again


Maybe exception instead of false? It is a nice information to know what's goes wrong.


This commit has been deployed to Deploy app


Nice work, it's good to check all of these


Shorter version:
array_walk_recursive($arr, function(&$value) {
    $value = (int) $value;
});


TeamCity SOEN341 Team3 :: Build [Build 16](http://localhost:81/viewLog.html?buildId=17&buildTypeId=Soen341Team3_Build) is now running


TeamCity SOEN341 Team3 :: Build [Build 16](http://localhost:81/viewLog.html?buildId=17&buildTypeId=Soen341Team3_Build) outcome was **FAILURE**
Summary: Cannot start build runner; exit code -42 (new) Build time: 00:00:03


It's really not true. I use PDQ lib in cpp file and create here static tft-object without problem. But if you try to include it in-to any of header files from project - compilation fail


Great work, much nicer in separate classes


If you start a speculative LTC, then please stop the STC :-)

And, by the way, speculative LTC should use the same amount of ressources of the connected machines, so throughput for LTC = 1000/6 = 166 since the time control is six times longer.


@bwlewis I added some convenience S3 methods for interacting with raw python objects. Any other methods you think would be particularly useful?


@andrejn-ct Fixed platform-dependent split. Mea culpa :) .


I forgot that existed. Some of those messages are crappy though.. like a lot of them even. :[


@vain0 This looks like a very interesting addition to the project.
Do you have sample code, demonstrating how can it be used?
I guess it's related to [this](http://tomasp.net/blog/2014/update-monads/).


okay,haha


Also variable names are a little bit confused, $arr tells nothing about values behind this variable;
I guess in this case there should be something like: $terrainMatrix or $terrainArray


The master for this test is three commits behind


I like this new option. Would you consider opening a Pull Req to the main project?


Not anymore. I decided to keep everything in meters at the moment. Can you had a scale factor to the class with the default being 1e9.


Sally resume has been received


new


when I said synthesizer, I meant a real-time synthesizer. which takes MIDI events from USB or from a keyboard on the screen.

yup, I have thread which ask ADLMIDI to fill a buffer, and I push it to an AudioTrack (which internally also have its own thread and buffers)

That's the problem with OpenSL, you have to generate the buffer in the OpenSL callback, which mean you have a very tight dead-line to generate the buffers. if you miss it, you got choppy sound.

OpenSL works as a pull buffer, where AudioTrack works as a push buffer.


VS-coil-gaps (EnergyArchmage) - Win64-Windows-7-VisualStudio-14: Tests Failed (2121 of 2418 tests passed, 55 test warnings)

 * 350 tests had: AUD diffs.
 * 350 tests had: ERR diffs.
 * 294 tests had: Table big diffs.
 * 2 tests had: ESO big diffs.
 * 3 tests had: Table small diffs.

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


FYI I won't merge this.  If we're copying MPL licenced code, it needs to be in its own file with valid license headers.  We have the "third-party" directory to organize code that is used via some per-file license like MPL.


Awesome, thanks!


good catch


Maybe the term "intensity" is misleading. The Gaussian distribution refers to the spatial distribution of incident electrons. The diameter being the full width at half maximum (FWHM) of this distribution.

A popular Monte Carlo program (Casino) draws electrons in blue. I don't know why they chose that color.


Jon form State of Vermont here...

Typo. Fw2nicName should be FW2nicName at line 624


Edit readme


This commit is causing compile error only in 32-bit environment.  Strangely 64-bit of the same setup is compiling fine.  Windows 32-bit 6.3.0 GCC:

>Compiling src/lib/netlist/nl_base.cpp...
../../../../../src/lib/netlist/nl_base.cpp: In member function 'void netlist::core_device_t::set_default_delegate(netlist::detail::core_terminal_t&)':
../../../../../src/lib/netlist/nl_base.cpp:690:23: error: 'netlist::nldelegate {aka class plib::pmfp<void>}' has no member named 'is_set'
  if (!term.m_delegate.is_set())


Don't commit commented-out code.  If it's not needed any more, remove it.  That's why it's in the git history.


<!doctype html>

<meta charset="utf-8">

<title>标题</title>


@dillaman okay, i think `!=` is different than `>`, but they are equivalent. `>` is semantically more correct in this case. if your theory holds, yeah, my change will hit both cases. because i gave up on appeasing coverity using non-macro approach.


`PerfixKey` -> `PrefixKey`


Hi @jekhor, superb job. Is this firm plug and play to the k40,  I mean, does it need anything else apart wiring the FFC to the endstops etc? (low signal for enable, etc.)
Do you have any documentation available? I've seen other setups need removing fuses, etc.


project_owner とか？


TeamCity Code :: 1. Development :: DDK :: Defect [Build 46](http://mp-ci.centralus.cloudapp.azure.com/viewLog.html?buildId=15764&buildTypeId=Code_Development_Ddk_Defect) is now running


Like.. which one?


単に place でよさそう


Not sure how much of his debug logging you want to leave in.


`walkPerfix` -> `walkPrefix`


`fmt.Errorf` -> `errors.New`


Ah, I see the pattern in this source file is to use `fmt.Errorf`.


hey man, i need a little help. I extracted the npc/gobj phases with the packetparser at this commit ( https://github.com/TrinityCore/WowPacketParser/pull/289 ) and i have more spawns than you. The problem is that I don't know how remove/add phase to A PLAYER.
If we have phase_area table filled with your query, the player at the start will be in all that phases.
When we play scene 1106, the phase don't change, you should be at 4899. Not all those phases.


Finally, I can run the drive.py without problems with this locale:
**LANG=en_US.utf8
LANGUAGE=en_US
LC_CTYPE=en_US.UTF-8
LC_NUMERIC=es_ES.UTF-8
LC_TIME=es_ES.UTF-8
LC_COLLATE="en_US.utf8"
LC_MONETARY=es_ES.UTF-8
LC_MESSAGES="en_US.utf8"
LC_PAPER=es_ES.UTF-8
LC_NAME=es_ES.UTF-8
LC_ADDRESS=es_ES.UTF-8
LC_TELEPHONE=es_ES.UTF-8
LC_MEASUREMENT=es_ES.UTF-8
LC_IDENTIFICATION=es_ES.UTF-8
LC_ALL=**


TeamCity SOEN341 Team3 :: Build [Build 17](http://localhost:81/viewLog.html?buildId=18&buildTypeId=Soen341Team3_Build) is now running


TeamCity SOEN341 Team3 :: Build [Build 17](http://localhost:81/viewLog.html?buildId=18&buildTypeId=Soen341Team3_Build) outcome was **SUCCESS**
Summary: Running Build time: 00:00:05


Some extra text below the SoB line, should that be deleted? It looks like I may have originally authored this? Where did my SoB go?


TeamCity SOEN341 Team3 :: Build [Build 18](http://localhost:81/viewLog.html?buildId=19&buildTypeId=Soen341Team3_Build) is now running


TeamCity SOEN341 Team3 :: Build [Build 18](http://localhost:81/viewLog.html?buildId=19&buildTypeId=Soen341Team3_Build) outcome was **SUCCESS**
Summary: Running Build time: 00:00:04


A full type-switch here seems overkill.  Just use a regular type assertion?


Standard library import come first, then internal imports, then external imports.  This should be after the stdlib imports, separated by a blank line.


TeamCity Code :: 1. Development :: DDK :: Defect [Build 46](http://mp-ci.centralus.cloudapp.azure.com/viewLog.html?buildId=15764&buildTypeId=Code_Development_Ddk_Defect) outcome was **SUCCESS**
Summary: Tests passed: 23 Build time: 00:04:03


`find` は指定IDが存在しないときに例外出すので、 `find_by(member_id: member_id)` かな


Typo in commit message "enbaled" → enabled


1. Please right fully the variable names - no abbreviations
2. List of characters is already a datatype - string :)


Crap. Will do.


TeamCity Code :: 1. Development :: DDK :: Feature [Build 156](http://mp-ci.centralus.cloudapp.azure.com/viewLog.html?buildId=15766&buildTypeId=Code_Development_Ddk_Feature) is now running


Idk, didn't test it after this commit😂


Hy, I've got around the same pb with a module I code. Where can I post it ?


넵 알겠습니다


Looks like Alexander submitted this and it's already in OE-Core/Poky master. 🥇


It is only behind 1 functional patch.
But you are right...next patch I will update to master.


넵넵


TeamCity SOEN341 Team3 :: Build [Build 19](http://localhost:81/viewLog.html?buildId=20&buildTypeId=Soen341Team3_Build) is now running


Понял, надо будет настройки табуляции в нотпаде подкрутить. Спасибо!


budget かなぁ…


1. Do not instantiate any objects in the class files, it is the wrong place. For test purposes, write tests and run python terminal:
```
from Field import *
test = Field()
test.print_field()
```
2. No trailing white spaces, only one empty line at the end


TeamCity SOEN341 Team3 :: Build [Build 19](http://localhost:81/viewLog.html?buildId=20&buildTypeId=Soen341Team3_Build) outcome was **SUCCESS**
Summary: Running Build time: 00:00:12


Please don't do force pushes to `master` or any branch on the main feedstock in the future @arokem. In addition to being bad practice, this messes up our tooling in conda-forge.


department_code ?


Clarified.


TeamCity Code :: 1. Development :: DDK :: Feature [Build 156](http://mp-ci.centralus.cloudapp.azure.com/viewLog.html?buildId=15766&buildTypeId=Code_Development_Ddk_Feature) outcome was **SUCCESS**
Summary: Tests passed: 23 Build time: 00:03:39


Thanks for the free password 💃


아하 네 이것도 수정해볼게요


@yaqianghan 抱歉，现在才看到消息。

通过配置之后，返回的数组会自动转成 Json，如果你想返回你这个格式，你可以在 afterAction 函数里面实现即可。比方说这样返回：

```php
return ['code' => 200, 'message' => '操作成功', 'data' => $data]
```


@mvdbeek You are absolutely correct - sorry for not seeing it sooner. I'll try to find some time today to take another pass at this. Luckily 17.01 still hasn't shipped.


creditor_code ?


We have the lnr script in OE-Core because we couldn't rely on host distros having a new enough ln to include the -r option. This was added to coreutils 8.16 which, iirc, debian stable (or possible centos) at the time didn't include.

I think we'll need to check whether the supported Debian Stable & CentOS ship with new enough coreutils to support this?


1. This is a fast hack, but it should be a nice, readable code.
2. range(24) would make the same


This should be a static inner class (eg `private static class DelayUpdate`) there is no need for it to hold a reference to the current outerclass instance when it is instantiated as it is just a named tuple of data.


Minor typo here "archieve" → archive


Sir, we have decided to work on  "Student Attendance System"  project.
-Padam Rijal


expected_participant_count ?


1. This function should be split into two
- first generates and fills ```self.field``` with indices **and** players (we will eventually pass them through the constructor
- second prints the self.field nicely formatted


Have we verified there's no change when ${libdir} != ${base_libdir}?


Exemple fonctionnel de chiffrement Rijndael Mcrypt php


while not necessary, all of these members can be final as instantiation of this class is always done through the constructor.  While this does not guarantee immutability, it does allow the JIT compiler to perform more optimizations and is generally good practice.


License Mit


duration とか？