Thanks, I would have to study the cluster branch but yes, it sounds definitely  something nice to do


If this passes i would retest at 8 threads for real vote checking. But lets see.
good luck!


Entity Layer


Created multidev environment [toys-for-joy#ci-91](https://dashboard.pantheon.io/sites/255bbf5d-2830-4fba-8604-186a8110e36d#ci-91).

[![Visit Site](https://raw.githubusercontent.com/pantheon-systems/ci-drops-8/0.1.0/data/img/visit-site-36.png)](https://ci-91-toys-for-joy.pantheonsite.io/)


@ArranTuna Yep, created an issue for this to be added here: https://github.com/multitheftauto/mtasa-resources/issues/116


به نظرم تا اینجا کاملا مناسب است.


ok


The comment for this commit should instead read:

> Added compatibility with Custom NPCs. #17


روی این بند شک وجود دارد


Thanks!


The comment for this commit should instead read:

> Added compatibility with Custom NPCs.


@puripuri2100 いまTweetに気付きました。申し訳ありません。お待ちしております。Membersに掲載する際のProfileを考えておいて下さい。よろしくお願いいたします。


Technically you could make this a lambda that returns `done` (`() => done`) since it already returns a Promise and you're just wrapping it with another Promise


Why?


Why?!?


Hello ! I am having the trouble copying the bsc_bme680 and a cofig document because I cannot find it. Where do I find them?


also do not forget to run as admin if you have a seperate admin password(
`su`
ENTER PASSWORD FOR ROOT
`cd [DOWNLOAD OF FILE LOCATION]`
`chmod 755 InstallLibsNeeded
./InstallLibsNeeded`
)


Hey @ramybenaroya , the API part in the readme is autogenerated using `documentationjs` (I should probably document that)

Please let me know if you need any clarifications or just want to chat about `pickpick`


опечатка в слове: **п**ервая


No interproscan/5.30-69.0/test-cmds.txt (test file) present, skipping tests


@jnothman Circle fails again : ``You can't use `macro parameter character #' in restricted horizontal mode.``


AA


Good.


Awesome, thanks!


Yeah, okay. Reverting back to the backslashes. At least for now.


@gatorsmile I think this change accidentally went in. (I've done it too) It happens when the result of `jekyll serve` is left in the branch and committed. I'm going to push another update now so will just overwrite it.


It's all advertising from here.


Consider swapping from the bitwise & to the boolean &&.  It looks like the order of operations works and you correctly create truthy/falsy 1s/0s, but `x.length > 0 && x.includes('self')` should be more type-safe and IMO more readable.


Xoring something with 0 does not change the value so I don't see the point of the if


Sigh. Reverted. Let's focus on the release.


When I added asprintf rescue.sh was updated to link asprintf.o into oksh but compilation of asprintf was not added.


I don't know if drive[:] is necessary here. just 'drive' would probably work. what's the data type coming out... list, or array?


Can we remove this, please?
Having to read window numbers and reach for the number keys every time I want to switch windows is a significant hassle.
`s-w` is fine if I want to use ace-window, but in most cases; I don't.


Please run [build]


@zhuweiyou 自动补全一把梭


Файлы для сайта


playing around with themes in settings. did not want to keep.


I update the code.


(also centered everything using a container as everything seemed very messy being across the screen)


S


New Airdrop: Harambee Token!!Leveraging on the blockchain technology, Land
LayBy is building a trusted shared distributed ledger for recording land
buying and selling transactions that can never be altered, corrupted,
forged or replicated in error. Chat with the telegram bot and get 100 HRBE
($70) and get 150 HRBE ($105) for every referral.
http://t.me/HrbeAirDropBot?start=568459030

On Sunday, July 29, 2018, boante <notifications@github.com> wrote:

> good
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> <https://github.com/iride2020/iRide-Token/commit/8f810b3700a24de0ee4f945ef84be7b2f1605610#commitcomment-29874282>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/Am4jXdUIXgFo4MtoqJ_dTIoXQkDO4b1pks5uLbaZgaJpZM4VaEAl>
> .
>


@codabrink Why would we remove this when you can set it to normal behavior with `(global-set-key [remap other-window] 'other-window)`?

(I would also prefer the old behavior to be the default, but since bbatsov prefers the new, and he is the one doing the work, I think it's a bit annoying if we nag him about that when there is a one-liner setting that makes emacs work exactly as we like).


New map for SoF - Sc1 to fix #3383 (removing the fast way to reach the elvish leader, by replacing some regular mountains with snowy mountains).


shliup@microsoft.com


Should be: "Be cautious"


@newtonne Please submit a pull request to fix this soon.

@yan12125 We should have caught this before merging, FYI.


I don't think we allow editing of commit messages.


hi


![](https://github.trello.services/images/mini-trello-icon.png) [040 - Mudar padrão dos Scripts do Liquibase](https://trello.com/c/j1TVjGEF/50-040-mudar-padr%C3%A3o-dos-scripts-do-liquibase)


![](https://github.trello.services/images/mini-trello-icon.png) [041 - Criar Parametrizações de configurações](https://trello.com/c/J2U2cip4/51-041-criar-parametriza%C3%A7%C3%B5es-de-configura%C3%A7%C3%B5es)


Why?


Well, I didn't push it to master, so I could actually do a force push. My motivation for pushing it to this repository rather than my fork was to see if I could get the buildbot to do a build, since Travis timed out. Turns out that's not possible, so I deleted the branch.

I posted #2287 to get some feedback for these changes, but haven't got any so far…


はじめまして。
NNUEEmptyで、テストコマンド「test nnue test_features」でエラーになってしまうようでしたので修正してみました。
評価関数の全計算と差分計算で結果が異なるようでしたので、全計算の方を修正してみました。


It seems that you repaired “side effects” from [surprising code deletions](https://github.com/sohail/csv/commit/9ba7c2a7d7ebbdb379ac12f395c3c2229661166c#r29824681 "Renaming of preprocessor symbols").
Would an other commit message be more appropriate then?


Weird! Let me poke around in the db.


Date needs to be updated to 2018.


You are "missing" this commit because it's in the master branch of the various repositories, not our stable branch.  These changes will be released with the next (admittedly very late) major release of our applications.

Not sure what you mean by it affected ActiveSync, nothing in this commit changes any code related to EAS.


Hi

> On Jul 29, 2018, at 6:18 AM, moodisd <notifications@github.com <mailto:notifications@github.com>> wrote:
>
> hey
>
> —
> You are receiving this because you are subscribed to this thread.
> Reply to this email directly, view it on GitHub <https://github.com/Hack-with-Github/Awesome-Hacking/commit/123ffd5a5a45b5fa732876f7a7dbb164734c10ee#commitcomment-29873830>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AdltfvshqrkOg-OaFm8pA8qA5JxUqkHLks5uLZoLgaJpZM4VlYh5>.
>


I dont understand what is this XD


LOL.


DEFAULT_FROM_EMAIL добавлено  mg в DEFAULT_FROM_EMAIL = 'Django Boards <noreply@mg.kokserek.site>'


It just feels like a weird decision to overwrite such a fundamental key-combination with something inherently more complicated and mentally tasking.
With `other-window` I don't have to think while switching windows. With `ace-window`, I have to stop thinking about the code I'm writing, read the window numbers, match that to a number key, press it, and refocus my thought process back to the track it was on. It just seems like it gets in the way.
Idk, maybe I need to give it a couple weeks for it to become second nature. I might give it a second chance.
It feels like this should be opt-in.


pLeAsE rUn [build]


thanks guys for your done job!


href updated


какой-то очень странный метод, который вызывает сам себя )
если его вызвать произойдет бесконечная рекурсия, переполнение стека памяти и падение программы
мне кажется ты запутался... или просто ошибся?


very very nice!


This presentation includes a high level explanation of version control best practices


тут return не нужен. Моя ошибка


PLEASE run [build]


вот про эту ветвь вычилсений, когда дискриминант болше ноля я при первой проверке написал следующее:

> а еще правильней передать не сам дискриминант, а корень из него, который будет вычислен после  39 строки
> чтобы не считать корень два раза
> сейчас же получается, что дискриминант вычисляется три раза, а корень - два
> таких ситуаций лучше не допускать

номера строк уже поменялись, но суть осталась та же: в 42 строке вызывается вычисление корня и в 43 строке вызывается вычисление другого корня
в обоих методах далее вызывается `squre_root(descrmnt)`, т.е. корень ты вычислешь дважды
а зачем?
еще раз перечитай мой комметарий выше или комментарий Влада на прошлой проверке [тут](https://github.com/megabytus/thinknetica_home_work/commit/281a9f142f10c856bd2c21355f86d05e619e9b95#diff-308d73091b81d6485101ab60e651239e)

мы оба предлагаем тебе сделать одно и то же
сначала вычислить корень из дискриминанта
а потом уже этот корень передавать в методы вычисления корней
чтобы не было вычисления корня дважды


This PR speed ups on mulit-core the build process of stockfish not the speed of stockfish himself.

I see that Michael reports also a speed up of the resulting binaries but that is very probable a fluke because there is no reason the compilers does something different but doing lto in parallel.


pLEASE rUN [build]


But i approve to go sure with half throughput.


PLEASE RUN [build]


Second commit actually. :]


hello,i am kxespal and i write something in github.


first github


```haskell
datadefn = llcc mkEmptyLabel . Definition . Data
```


Yes, please.


The `datadefn` and `reldefn` definitions can be 'shortened' as per my comment on the previous check-in.


So I thought `Adposition.usages` was a list of `Usage`s, but turns out it's a list of `UsageRevision`s. Fixing.


No need for the extra () after `$` (here and in following lines).


I definitely like this change. Though, in some sense, it would be even better if the label was `Table:useCase`.  So I think it should be the `LabelledContent` which should set this label, rather than the printing back-end.


про эту проверку я тебе писал на первой сдаче:

> обрати внимание на "Также" - равнобедренность нужно проверять ТОЛЬКО, ЕСЛИ он прямоугольный, а у тебя она проверяется в любом случае - это нарушение ТЗ, надо исправлять

и в предыдущем варианте ты так и проверял: `is_right_angeled && is_isosceles` - это верно
неверно было дургое, на что Влад и постарался обратить твое внимание: у тебя проверки "прямоугольный" (47 строка старого варианта) и "прямоугольный и равнобедренный" (50 строка старого варианта) шли в двух независимых "иф - блоках", а т.к. эти блоки были не связаны, то выполнялись оба, вот Влад тебе пишет:

> Сейчас если треугольник у нас проямоугольный и равнобедренный у нас выведется
> ```
> This triangle is right angled
> This triangle is isosceles triangle
>```
> А нам бы хотелось видеть только одно из этих 2 сообщений.

далее он тебе объясняет, что нужно использовать один блок и в нем "ифэлсы"
и ты в этом смысле все верно сделал - написал один блок, вместо нескольких независимых и в нем "ифэлсы"
но проверка в 49 строке должна быть как была раньше и на прямоугольность и на равнобедренность: `is_right_angeled && is_isosceles`


> Dear @MiniGameAPI-PM:
> Regarding the release you submitted, named "**MiniGameAPI**" (v1.0.1), for the project [MiniGameAPI](https://poggit.pmmp.io/ci/MiniGameAPI-PM/MiniGameAPI/MiniGameAPI) on 2018-07-29T06:54:19.000Z, there are some problems with the values in the submit form.

I appreciate that this plugin could become useful, but this API plugin lacks documentation. For API plugins, we strictly require good documentation since they are supposed to be used with other plugins. I know there is an example plugin, which would be useful, but I would like to see the following in your documentation, either through Poggit's plugin description or through your GitHub repo's wiki linked from the plugin description:
1. Documentation on how to create a game plugin would be better. Your plugin contains quite a lot of source code, so it is not easy to understand how your whole plugin works. Unless you expect every developer to start off by cloning MGARace, a minimal and elaborated template would be great.
2. The semantics of MiniGameAPI is unclear. How does it manage players? How does the game flow work? Can a player be in multiple games or multiple teams simultaneously? How are ops (or other privileged players who can potentially break the expected semantics of the system by teleporting around) handled? What exactly is a game or a team? To provide a good API, you should define these concepts clearly.
3. It is suggested that you explain why MiniGameAPI should be used, e.g. how it empowers integration of different games, etc. Also, is it possible to have games hosted on multiple servers instead of just one? How well will it work? What if each server only runs one game?

> Your release has been reset to draft. Please [edit the release](https://poggit.pmmp.io/edit/MiniGameAPI-PM/MiniGameAPI/MiniGameAPI/140) to resolve these problems, then click "Submit" on the edit page to have the plugin reviewed again.

> Note: This comment is created here because this is the last commit when the released build was created.

> via Poggit (@poggit-bot)


Fetch


bch


Created multidev environment [wvuonline#ci-441](https://dashboard.pantheon.io/sites/8fb52aa5-5525-4d73-9b84-f2d86faf1da2#ci-441).

[![Visit Site](https://raw.githubusercontent.com/pantheon-systems/ci-drops-8/0.1.0/data/img/visit-site-36.png)](https://ci-441-wvuonline.pantheonsite.io/)


We should make label-free data definitions an error. And fix tiny to give labels.


I also wasn't too happy to search/scroll through 7k lines everytime I want to change something.


Block chain mywallet


если уж ты не стал выносить эту проверку в отдельный блок, как рекомендовал Влад, а присоединил к общему "иф-элсиф-элс" блоку, то "экзит" не нужен
в блоке в любом случае выполнится только одна ветвь
два варианта дальнейших правок:
- либо все же вынести это в отдельный блок с "экзитом" (аналогично блоку в строках 37-40) и поставить этот блок перед 42 строкой (а 43 строку вынести перед ним)
- либо оставить этот код здесь, но тогда убрать "экзит", т.к. он не имеет смысла

первый вариант предпочтительней, потому что не придется считать строки 42 и 44, которые не имеют смысла для равностороннего треугольника


We already have `Int` to represent integers. Why do we also need `Integ`?

The change to printing of `Dbl` here is also note quite "right". If the 'base' of a double is between -3 and 3, it should be displayed 'as is', not in scientific notation.


@mucyomiller yo man WTF is going on? .env  is still there! what i did is pull origin master and commit but the stuff is still there and the .gitignore file seem OK. am confused.


If you don't want to implement KMP, you can use rolling hash as alternative but it needs to watch out overflow issue.


Minor thing, but: why is the content of the item on a new line lined up with the \label? The old formatting of this was 'better' (even if it is inside a .tex file that most should not read).


still left "#educatiuonal-tutorials"


xoa dong nay di


When you expose a new API from a package, you should bump up the version number.


Hi @emukidid after this commit on the Master branch I cannot compile the source anymore.
I'm getting the following error with both the latest as r26 of de devkitPro toolkit, any idea which could cause this?

make -f Makefile.wii
make[1]: Map '/c/devkitpro/examples/snes9xgx-master' wordt binnengegaan
snes9xgx.cpp
C:/devkitpro/examples/snes9xgx-master/source/snes9xgx.cpp: In function 'int main(int, char**)':
C:/devkitpro/examples/snes9xgx-master/source/snes9xgx.cpp:405:23: error: invalid conversion from 'void ()()' to 'resetcallback' {aka 'void ()(long unsigned int, void*)'} [-fpermissive]
SYS_SetResetCallback(ResetCB);
^~~~~~~
In file included from C:/devkitPro/libogc/include/gccore.h:62,
from C:/devkitpro/examples/snes9xgx-master/source/snes9xgx.cpp:14:
C:/devkitPro/libogc/include/ogc/system.h:325:50: note: initializing argument 1 of 'void (* SYS_SetResetCallback(resetcallback))(u32, void*)'
resetcallback SYS_SetResetCallback(resetcallback cb);
^
C:/devkitpro/examples/snes9xgx-master/source/snes9xgx.cpp:493:20: warning: format '%x' expects argument of type 'unsigned int*', but argument 3 has type 'u32*' {aka 'long unsigned int*'} [-Wformat=]
sscanf(argv[4], "%08x", &GCSettings.Exit_Channel[0]);
^~~~~~ ~~~~~~~~~~~~~~~~~~~~~~~~~~~
C:/devkitpro/examples/snes9xgx-master/source/snes9xgx.cpp:494:20: warning: format '%x' expects argument of type 'unsigned int*', but argument 3 has type 'u32*' {aka 'long unsigned int*'} [-Wformat=]
sscanf(argv[5], "%08x", &GCSettings.Exit_Channel[1]);
^~~~~~ ~~~~~~~~~~~~~~~~~~~~~~~~~~~
make[2]: *** [/opt/devkitpro/devkitPPC/base_rules:13: snes9xgx.o] Fout 1
make[1]: *** [Makefile.wii:109: build_wii] Fout 2
make[1]: Map '/c/devkitpro/examples/snes9xgx-master' wordt verlaten
make: *** [Makefile:10: wii] Error 2


Hello Thanks For mod.
I installed it.
When ı open Create Scenario or new game the game gives a error message like this;
"İnvalid tech tree marker(expected NTT, version 1):ran, version 32"

![tome](https://user-images.githubusercontent.com/41870471/43367948-e38c5bc2-935d-11e8-86e9-ec1787afdd6a.JPG)


tet


test


(This will be exhaustive feedback, including nits.)

I can guarantee that someone (probably me 😆) will want this refactored so that it's not a scoped arrow function since it impedes performance and you don't need access to this scope.


Attached are the devkitPro tools I use, I also tried downloading the newer PPC files but it gives me the same error.

Thanks in advance.


You should tell the request whether it should return types or not. That way your code below can be a bit cleaner, right?


\sectionRef isst dazu gedacht, dass man mit dem optionalen Argument, auf das Kapitel im Demtröder verweisen kann, welches dieses Thema behandelt. (Falls so ein Kapitel exisitiert)
Also z.B.: `\sectionRef{Harmonischer Oszillator}[4.2.5]`
Es wäre gut wenn du das noch rgänzen könntest. Falls man in der Klausur noch mehr INfos braucht als in dieser Zusammenfassung auftauchen, oder Beispiele sehen will, kann man diese schnell im Demtröder finden.


n


Reading well what the functions do, they don't parse the messages, they decode it. So we thought it was better to rename it.


You should tell the request whether it should return types or not. That way your code below can be a bit cleaner, right? (You could either do it by setting a property to `req` or by adding an option to `readdir` (which would, I guess, be breaking.)


You can avoid this here by just assigning two diff callbacks to `oncomplete` depending on the `options.types`.


I use this way :


Iterators are confusing, what about doing `val treeSection :: otherSections = sections; ...` ?


In the otrv4 protocol description all the times that ECDH was written, it was in uppercase, we followed that pattern. But we can change it if this is a problem


I think this is slightly faster with `i+=2` and not having to use `lastString`. Also, you can `const newResult = new Array(result.length / 2);`


I'm sure someone will complain about an unrelated space insertion. 😆


good job！


> Dear @SpecterTeam:
> I regret to inform you that the release you submitted, named "**Pinger**" (v1.0), for the project [Pinger](https://poggit.pmmp.io/ci/SpecterTeam/Pinger/Pinger) on 2018-07-23T01:03:57.000Z, has been rejected.

https://forums.pmmp.io/threads/clean-up-our-consoles.5656/

> Please resolve the issues listed above and submit the updated plugin again.
> Note: This comment is created here because this is the last commit when the released build was created.
> via Poggit (@poggit-bot)


### Minuti
Calculating outgoing call length and tell the user how much minutes still remain from the cell - operator plan.


Можно передать day первым аргументом в редьюс. Он будет начальным значением.


a ještě https://github.com/mybb/mybb/commit/aba530b7fa740208db760d9fa649a05dc51568fc


не самое "чистое" решение
хотя и работающее ))
правильно то, что ты не перебираешь все месяцы
не очень правильно, что ты используешь `break`
здесь лучше воспользоваться методом `first` класса `Array`, [доки](https://ruby-doc.org/core-2.5.0/Array.html#method-i-first)
и с его помощью выбрать из массива только нужный тебе подмассив
вот так:
```ruby
array_month.first(month - 1)
```
дальше без всяких `break` пробегаешь по всему подмассиву и все суммируешь
либо еще

(еще в классе `Array` есть метод [`take(n)`](https://ruby-doc.org/core-2.5.1/Array.html#method-i-take), который тоже возвращает первые `n` элементов массива)


I think this is a kind of "cuteness" that can be a bit offputting to beginners.


Could you please also open this ticket: https://github.com/phonegap/phonegap-plugin-push/issues/2141
This ticket is still actual, but nobody unable to continue the conversation

Could you please check this error? It reproduced with blank project


Сейчас код запускается. Но не уверен, что его проверяли
```
:2$ ruby day_of_year.rb
день: 5
месяц: 2
год: 2000
28
```


If we follow some of the feedback from above then we'll be able to just return `result` here and then below we could use the same function as we would use in the callback above, so that we don't have to rewrite the same code twice.


“Unfortunately, the Maximum k -cluster SubDAG problem is NP hard (see problem [GT26] in [2]), and PHANTOM is therefore of less practical use for an ever-growing blockDAG. We thus introduce a greedy algorithm that is more suitable for implementation. We call this greedy variant GHOSTDAG. 4”


Повторюсь еще раз

VladFiliucov 9 hours ago
Повторюсь: Настрой редактор на отступ ровно в 2 пробела, именно так принято в ruby. Причем, именно пробела, а не таба. Т.е. должна быть замена табов на пробелы, т.н. soft tabs. Найди в интернете как сделать soft tabs для твоего редактора


why not make type a well known string so we can get rid of those functions


> shouldn't pmusic_PLUGIN_tray versions be in common32 and common64 ?

At this point everything in common32 and common64 has a petbuild.

I don't know if pmusic_PLUGIN_tray will compile in common32/64 because I really cut down on the number of libraries included in common32/64, but I would be willing to try if I could find the petbuild...

Maybe there is no petbuild, so I also looked for the source code in
http://distro.ibiblio.org/puppylinux/sources/p/

Nothing...  the power of open source depends on being able to find the source code...


You can use LdrEnumerateLoadedModules API instead of tons LDR work.


Лучше использовать новый синтаксис хэшей. Он экономичней по памяти, его легче читать и меньше символов занимает (но последний пункт далеко не всегда преймущество)
```
month = {
  January: 31,
  ...
}
```


@kessler oh, I was just looking for a simple first contribution. Sure, a chat will be nice over a balcony cigarette.


I'm not sure beginners will know what base64 is. Also the length relationship between input and output is a bit complicated. I think

```
stretch('passw0rd').hex()
```

might be better (even if the output is longer).


Maybe use `secrets.token_bytes()`?


чтобы взять последний и предпоследние элементы можно использовать индексы `- 1` и `- 2`, тогда не нужно считать индекс

вот так:
```ruby
element = fibbonacci[-1] + fibbonacci[-2]
```


Fixed in https://github.com/BSidesPDX/bsidespdx.org/commit/bea31391a20a887ac89fd5383250310516f9d5ae


кажется уже писал, повторюсь
массивы в руби рекомендуют записывать, разделяя элементы пробелом:
```ruby
arr = [1, 2, 3] # good

arr = [1,2,3] # bad
```


or should be added to rootfs-packages to live alongside pmusic - there are already items with both a 32-bit and 64-bit version so precedent has been set....


In my case my app (running in emulator) is failing on that line with

```
// check if view has been touched
java.lang.RuntimeException: Unable to start activity ComponentInfo{com.example.android.pets/com.example.android.pets.EditorActivity}: java.lang.NullPointerException: Attempt to invoke virtual method 'void android.widget.EditText.setOnTouchListener(android.view.View$OnTouchListener)' on a null object reference
```


I'd suggest we make everything importable directly from `cryptozero` for simplicity (even if we split it up for maintainability).


CSS Deep Dive Summary: [css-03-deep-dive-summary.pdf](https://github.com/whyang0808/learning-css/files/2238928/css-03-deep-dive-summary.pdf)


AFAIK it doesn't only affect `nix-env -i` usage.  It's used also by `buildEnv`, including `systemPackages`.  I believe especially `.dev` outputs don't belong to `nix-env -i` by default.  Back then when this was created, the main rationale for splitting a package was that the package contained too much by default. People seem often mislead that `nix-env` is good for build environments, but `nix-shell` is a much better fit for that, because it set various other things needed for building.

There's this thread https://github.com/NixOS/nixpkgs/issues/24717 and perhaps others.  I had planned to make it configurable in nixpkgs in the usual way, so that people could do the equivalents of e.g. `programs.man.enable` for `systemPackages`, but IIRC I hit some obstacles with affecting `buildEnv` usage in nixpkgs sources, and I didn't manage to finish the work.


текущий вариант работает верно, но не очень рекомендуемый
он довольно сложный в прочтении, точнее - неочевидный
вообще варианты с "брейк" они не очень рекомендуются к использованию, ввиду их неочевидности
когда я читаю код `for i in 2..100`, у меня в голове уже строится модель кода, который будет перебирать все элементы от 2 до 100... а потом вдруг "брейк"
для подобных случаев рекомендуется использовать конструкцию `while sum < 100` - здесь в первой же строчке обозначено, до выполнения какого условия будет работать петля
попробуй переписать через "вайл"


:2$ ruby fourth.rb
{"a"=>0, "e"=>4, "i"=>8, "o"=>14, "u"=>20}

Ок, он не выполняет тз. Но тут код написан для машины, а не для человека, который будет его поддерживать. Слишком сложное решение, для простой задачи. Когда тебе через месяц нужно будет расширить этот модуль, то ты будешь его читать пару минут, вместо того, что-б понять, что тут происходит с первого взгляда. Нужно просто "тупо" взять массив с буквами и взять массив (или строку) с гласными и пробежать по массиву с помощью each_with_index и проверить, если текущий элемент включен в массив (строку) гласных.  Если включен - то записать его в хэш с index + 1. Научиться писать простой код - это целое искусство. Вот поделюсь статьей на эту тему

https://habr.com/post/347166/


Sources here?
http://murga-linux.com/puppy/viewtopic.php?p=663131
I'll copy them to ibiblio....


Can someone also please update section 7.3.3 substep 3 where it describes the formatting of the maintainers line? https://guide.macports.org/chunked/project.contributing.html I read that over when contributing this patch as I was trying to find documentation on the syntax of the maintainers line. The documentation linked there doesn't describe the `{foo @foo}` syntax so I figured it was out of date, pre-github documentation and copied what everyone else was doing.


название пакетов с маленькой буквы, читать код конвеншен


Названия коммитов должны отражать суть того, что ты комитаешь, а не везде 'Add files via upload'


эта строчка не имеет смысла - уже в следующей ты перезаписываешь значение этой переменной


![](https://github.trello.services/images/mini-trello-icon.png) [042 - Desenvolver Repository - Telefone](https://trello.com/c/Yc62a7Le/48-042-desenvolver-repository-telefone)


код конвеншен ты совсем не читаешь, наверное, CamelCase названия классов, никаких нижних подчеркиваний, и стараться избегать цифр. Давать нормальные имена всему, и классам и функциям и переменным, чтобы они отражали суть того, что ты там хранишь, что делаешь и т.д.


код не отформатирован, перед коммитом делай Ctrl + Alt + L


функция должна возвращать число, а не быть void


test


Используй импорты, а не java.lang.String. Просто String и импорт вверху, это делается в идее с помощью ctrl + пробел


не надо делать отступов между break и кейсами


инструкция default должна быть в break в ней, все написано в код конвеншене


после запятой желателен пробел
лучше давать более осмысленные имена переменым даже при итерациях по коллекциям
здесь можно было назвать `product`, `details` или `item`, `data`... как-то в таком ключе


давать нормальные названия переменным, а не k, n, mas, mas1


если тебе нужен int, то почему не используешь класс Random?


все в функции main - выносить в отдельные функции, в main только тестируем код


@darwinyip

Managed to fix the issue like this:

```java
@Override
        public boolean onTouch(View view, MotionEvent motionEvent) {
            viewHasChanged = true;
            // Add this line to override click
            if (motionEvent.getAction() == MotionEvent.ACTION_UP) {
                view.performClick();
            }
            return false;
        }
```

The linter warning is still there, but the app is no longer crashing on that line.


здесь же у тебя уже есть значение для этого ключа
ты его в 18 строке записываешь в переменную `value`
поэтому запись может быть проще: `value[:price] + value[:quantity]` (ну или переименуй `value` для лучшей читаемости, но обращаться нужно к этой переменной, а не к `cart`)


очень сложно написано, какой-то коэф-т k, который используется везде, название вообще ни о чем не говорит


код написан безобразно, уже 3 недели повторяю, чтобы форматировали код перед коммитом


Ok, I see it is hosted at
http://www.01micko.com/

I see a 64 bit version is already available, were you planning on on just copying the existing pets to common32/64 or would you want me to try to make a petbuild and recompile them?


с названием классов и пакетов то же самое замечание везде


лучше отдельно посчитать, присвоить в переменную , а потом в строке обратиться к переменной
хорошая привычка - отделять логику и рассчеты от аутпута


When choose 2.1.2 version i've got:

```
Linker command failed with code 1

ld: warning: directory not found for option '-L/Users/xxx/Library/Developer/Xcode/DerivedData/XXX/Build/Products/Debug-iphonesimulator/GoogleToolboxForMac'
ld: warning: directory not found for option '-L/Users/xxx/Library/Developer/Xcode/DerivedData/XXX/Build/Products/Debug-iphonesimulator/Protobuf'
ld: warning: directory not found for option '-L/Users/xxx/Library/Developer/Xcode/DerivedData/XXX/Build/Products/Debug-iphonesimulator/nanopb'
ld: library not found for -lGoogleToolboxForMac
clang: error: linker command failed with exit code 1 (use -v to see invocation)
```


зачем ты сортируешь массив?


I agree with you, but, again, I think we should respect the preference of the author of the package, especially when *one line makes things work exactly as we like*?


Вот эта строка не нужна. Зная индекс текущей станции мы всегда можем вычислить ее из массива станций в методе current_station. А запись переменной `@station` - это дупликация данных, что скорее всего приведет к багам в дальнейшем. Хорошо всегда иметь единственный "источник правды"


Found solution here: https://github.com/google/google-toolbox-for-mac/issues/137#issuecomment-314012109


Still getting the hang of commiting on the command line rather than using git gui


написано очень сложно


Hi, I'm also trying to port MK3 fw on ramps. Does your communication over uart to pronterface work? I'm trying to port the latest fw 3.3.1 (#974) but they introduced a second serial port (uart2) and even if i disable it, i can't get it to work. The code for rx and tx in 3.3.1 is different from version you are deriving from (i think 3.1.4?).


Эта проверка не нужна. Если route нет, то метод next_station, который вычисляется из route вернет nil и второе условие не будет удовлетворено


Well, I think I have a solution in 1126fd4—unfortunately it uses functionality that requires Django 2.x (`filter` argument to `Count`).


Please up date. Thanks you.


написано неверно, все поломается при разных размерах массивов


Вот эти сообщения нужно везде убирать. Они у нас будут в клиентсокм коде. Нам важен факт того, отработал ли метод, или вернул nil


до сих пор весь код пишешь в main, все на функции разбивать!


Move left and right must be removed


код вообще нигде не форматируешь


названия переменных с маленькой, код конвеншен


зачем тебе эта функция, если ты ею не пользуешься?


Dave, I agree that there is a missing basecase. The problem is: if you find checkmate, you want to add it to the path. Otherwise the `Bestpath` will never contain the actual checkmate.

I added that, but I also initialized the `Best` value to be the opposite of what we are searching for, instead of 0, otherwise it doesn't work. Not sure why yet.


Вот тело с 107-109 строку сделано идеально. Исправить проверку с elsif на if и убрать все остальное - будет правильный метод


@mucyomiller  fixed it by "git rm .env cached" :)


периметр всегда рассчитать можно по точкам, а что можно рассчитать - нельзя делать полем класса!


Whoops.


очень многого не насчитал из того, что нужно было сделать по требованиям


делай отступ между классом и полями


I thought about doing that (and it would only break the binding layer, which happens in Node.js all the time). The problem I encountered was: How can I pass that option to `AfterScanDir` for the async case? I'd really like to avoid copying the function and doing `AfterScanDirWtihTypes`.


не нашел я тут вывод лучшего ученика, выводишь всех


отступ между классом и методом


не использовать множественный импорт! поправить в настройках идеи, чтобы так не комбинировал, я показывал


отступ между импортами и классом!


до сих пор не форматируешь код!


IBM Cloud [toolchain](https://console.ng.bluemix.net/devops/toolchains/ebb9fcb3-a4f8-46bb-9dc6-2ec5d208f3b1?env_id=ibm%3Ayp%3Aus-south "acqol-deux"): [Delivery Pipeline](https://console.ng.bluemix.net/devops/pipelines/1a74dd51-73bc-4fa3-a136-fcfd99e0daa2?env_id=ibm%3Ayp%3Aus-south "acqol-deux") deployed [acqol-deux](https://github.com/ansleydeakin/acqol-deux/releases/tag/deploy-ibm-yp-us-south-ansley-deakin.edu.au-dev-20180729-163453 "deploy-ibm-yp-us-south-ansley-deakin.edu.au-dev-20180729-163453") to [dev](https://console.bluemix.net/apps/d8c0e304-cee4-4124-a831-708af13fa45c?env_id=ibm:yp:us-south "ibm:yp:us-south:ansley@deakin.edu.au:dev"), including this commit


почему не сортировал с помощью своих компараторов?


Do they all use the same `req` or does `AfterScanDir` receive some other one? If so, you could set `req.withTypes = true` or something.


нет поиска самого лучшего ученика через итератор


создаем утилитный класс для печати коллекций


класс с методом main только для тестирования! весь функционал выносим в отдельные классы


add a move constructor too?


подумай, что в задании может означать вот эта фраза: "Имеет начальную и конечную станцию"


не использовать итераторы, они устарели, изучаем API классов, которые ты используешь


программируем на уровне интерфейсов, а не реализации, поэтому List!


change comment about IReferenceCounted


никаких магических чисел, все выносим в константы


наверное лучше "принт стейшенз" или "лист стейшенз"
"гет стешенз" - как-то начинаешь ожидать, что метод что-то возвращает


no, this is useful only for std::vector containers, not std::unordered_map


ТЗ: "Может удалять промежуточную станцию из списка"
этот метод удалит любую станцию
надо поправить


saame, as the comment for insert!
Also use std::lower_bound for consistency


весь функционал в методе main! только для тестирования он, выносим в отдельные классы


iterative remove_if is slower than a find and then remove/erase at that position


Looking good, I approve.


никакого хардкода и магических чисел, выносим в константы все


ТЗ: "Может возвращать список всех поездов на станции, находящиеся в текущий момент"
"возвращать", а не "выводить на печать"
этот метод работает неверно
ну и он в общем-то не нужен, потому что есть ридер `trains_list` (2 строка), который уже выполняет эту функцию  - "возвращает" список


Программируем на уровне интерфейсов, а не реализации - поэтому List


В данном задании лучше было выбрать LinkedList


Started learning algorithms... so far so difficult


см. выше
"путс" здесь лишний, нужно чтобы метод "возвращал", а не "печатал"


Wow, I just committed way to much. Can't remember the last time that happened.


bonus, is there any need for CConcurentObjectCache::contains?


The project proposes HECMI (Hybrid Ensemble technique for Classification of Multiclass Imbalanced data) to deal with the problem of multiclass imbalance data which has more than one majority and minority class.HECMI is generic in nature and can be used for any domain.It is a hybrid of data based and algorithm based method for imbalanced data.The base classifier of the ensemble is selected
from the existing traditional models with the best cross-validation score. The data is initially partitioned into n parts for n iterations.A data-based approach of oversampling minority class instances is used to balance the data in iterations.The class with least recall is the one misclassified the most and should be paid attention by the classifier.Instances of this class are oversampled and added to the next data part in training.Also, the instances of classes with recall less than threshold are added to the next data part.Final prediction is done by taking the majority of votes of the classifiers.


исправил, в прошлый раз забыл


опять путанница с "возвращать" / "вывдить"
и опять же этот метод не нужен, потому что есть геттер `speed`


IBM Cloud [toolchain](https://console.ng.bluemix.net/devops/toolchains/ebb9fcb3-a4f8-46bb-9dc6-2ec5d208f3b1?env_id=ibm%3Ayp%3Aus-south "acqol-deux"): [Delivery Pipeline](https://console.ng.bluemix.net/devops/pipelines/1a74dd51-73bc-4fa3-a136-fcfd99e0daa2?env_id=ibm%3Ayp%3Aus-south "acqol-deux") deployed [acqol-deux](https://github.com/ansleydeakin/acqol-deux/releases/tag/deploy-ibm-yp-us-south-ansley-deakin.edu.au-dev-20180729-164547 "deploy-ibm-yp-us-south-ansley-deakin.edu.au-dev-20180729-164547") to [dev](https://console.bluemix.net/apps/d8c0e304-cee4-4124-a831-708af13fa45c?env_id=ibm:yp:us-south "ibm:yp:us-south:ansley@deakin.edu.au:dev"), including this commit


никакого вывода на консоль, должен возвращать true-false метод


в задании:
"Может тормозить (сбрасывать скорость до нуля)"

метод реализован в виде стопкрана, т.е. скорость сразу сбрасывается до нуля
(в задании возможно не очень ясная формулировка, но она означает, что поезд может тормозить и это не обязательно должно быть полное торможение, фраза же "сбрасывать скорость до нуля" означает, "обратите внимание! скорость не должна быть меньше нуля" :) )

лучше, чтобы метод принимал значение (аналогично `add_speed`)


Sorry about the commit msg. Basically this commit adds Swashbuckle, Identity, and an Account Controller.
I created a migration for the IdentityDbContext in the web api proj as well


почему shape2? Стараться избегать в названии цифр, и в данном случае подшло бы прекрасно просто shape


I know they are within error margins of each other, but it seems suspicious that this would pass an [0,4] but show less ELO on two different regression tests with this as the only difference (as far as I'm aware). Perhaps I'll run some more tests for verification when I get back.


а если вагонов нет?
пойдем в минус


опять неверный и ненужный метод


утилитный класс final и private конструктор


я хотел сначала так сделать но подуми зачем если нам надо все равно их выводит


странный метод, не находишь?)
принимает тип Triangle и проверяет, является ли он Triangle


то же самое и здесь. Задача была в том, чтобы методы принимали фигуру! т.е. базовый класс и определяли, является ли она труегольником или еще кем-то


It can also be a yad script... i just run pmusic and boy how i regret
it, it's taking forever to scan my hard drive full of repos, code
sources and stuff

I could improve this, but need some generous donations first


Test


По условиям задания ведь не сказано вводить какую либо проверку на отсутствие/присутствие вагонов? Надо такие проверки самому додумывать? )


1. Нет скобочек в последнем else
2. Стараться использовать один return из цикла
3. Есть в классе Double так же как и в Integer статический метод для сравнения, вот его и используем


почему столько пропусков строк? одной достаточно


между классом и полями - пропуск


I don't like the names either.

I'm not sure I really like the way they are used. I'd like to README drive the user level API for defining tasks.


> @yan12125 We should have caught this before merging, FYI.

Absolutely right. On the other hand, I'm curious if it's feasible to add a command to check whether email addresses are valid or not in Travis CI tests. There's a list of current developers in the [Wiki](https://trac.macports.org/wiki/MacPortsDevelopers)


в методе main только проверка написанного, все остальное выносим в отдельные классы (утилитные например)


Updated


так выполняет или нет?


никакого хардкода и магических чисел, все выносим в константы


ты не сделал копирование, ты просто ссылку новую создал на ту же область памяти, поэтому поменяется как твой list, так и listCopy. В данном случае не надо просто ничего возвращать из функции markLength4


зачем в каждой итерации вызывать метод findMax? Один раз и все


Все эти параметры должны собираться из словаря, а не конкатенироваться.


отступ между классом и полями


окружность "является" точкой? Не правильно использовано наследование


код не отформатирован


не нужно использовать скобочек вокруг того, что возвращаешь, ухудшает читаемость кода


first program on c++


A petbuild would be good if it's staying as a compiled binary....the .c contains the compile rune.
http://distro.ibiblio.org/puppylinux/sources/p/pmusic_PLUGIN_trayapp-0.5-source.tar.gz
contains the .c and the /usr/local/pmusic/plugins/pmusic_trayapp.rc
so it's "captured".... ;-)


комментарии надо писать правильно, читаем код конвеншен!


отступ между классом и методом


既存のデータベースを修正する場合は次のSQLを実行する
```sql
UPDATE
  remodel, ship
SET
  remodel.second_ship_id = ship.ship_id
WHERE
  remodel.second_ship_id = ship.id
  AND remodel.member_id  = ship.member_id
```


отступ между классом/интерфейсом и методом


интересно, элипс задается разве двумя радиусами?)


__UPDATE__ moved to __enigma__ branch


зачем переопределять, если ничем не отличается реализация?


дистанцию можно рассчитать, зачем ее хранить как поле класса?


код вообще нигде не отформатирован!


ничего не смущает, что у точки есть площать?


1 отступ


если переопределяешь методы базового класса, то ставь аннотацию Override


<@201745963394531328> test.exe


плохой вариант, лучше использовать отдельно сравнение имени и отчества! по памяти неэффективно


good ideea


Why wouldn't be there a need?


But why? And to what?


Dear Poggit user,

This is an automatic message from Poggit-CI. Poggit-CI was triggered by this commit, but failed to create builds due to the following error:

You are trying to create too many projects. We only allow creating up to 6
new projects per week.

Contact @SOF3 [on Discord](https://discord.gg/NgHf9jt) to request for extra
quota. We will increase your quota **for free** if you are really trying to
build **your own plugins**, i.e. not forks or copied code.

**Do not try to use another account just to overcome this limit**, or you
may be **banned** from Poggit.

As a result, no builds could be created from this commit. More details might be available for repo admins at [the webhook delivery response log](https://github.com/AirCow/MegaShop2/settings/hooks) — see the webhook starting with `https://poggit.pmmp.io/webhooks.gh.repo` and look for the delivery with ID `a815ccec-9351-11e8-9a22-7b79f1d41d7e`.

Shall you need any assistance, you may contact us on [on Discord](https://discord.gg/NgHf9jt), or more publicly, [on GitHub](https://github.com/poggit/support/issues). Commenting on this commit directly will **not** notify any Poggit staff.

Regards,
Poggit Bot (@poggit-bot)
The official Poggit automation account


@RetroSven I had to push several patches to get things to compile again on Android. Try testing if things still work fine on your end.

Also, in future, try to avoid these kinds of scenarios since they are known to cause issues with the Android compilers -

'[exec] jni/../../../../griffin/../menu/cbs/menu_cbs_right.c:288:34: error: format string is not a string literal (potentially insecure) [-Werror,-Wformat-security]
     [exec]       snprintf(msg, sizeof(msg), msg_hash_to_str(MSG_CHEAT_DELETE_ALL_SUCCESS)) ;
     [exec]                                  ^~~~~~~~~~~~~'


Class specialization for not vector is below. This one is only for vector


Dear Poggit user,

This is an automatic message from Poggit-CI. Poggit-CI was triggered by this commit, but failed to create builds due to the following error:

You are trying to create too many projects. We only allow creating up to 6
new projects per week.

Contact @SOF3 [on Discord](https://discord.gg/NgHf9jt) to request for extra
quota. We will increase your quota **for free** if you are really trying to
build **your own plugins**, i.e. not forks or copied code.

**Do not try to use another account just to overcome this limit**, or you
may be **banned** from Poggit.

As a result, no builds could be created from this commit. More details might be available for repo admins at [the webhook delivery response log](https://github.com/AirCow/MegaShop2/settings/hooks) — see the webhook starting with `https://poggit.pmmp.io/webhooks.gh.repo` and look for the delivery with ID `bd57521a-9351-11e8-96e7-009304f158b7`.

Shall you need any assistance, you may contact us on [on Discord](https://discord.gg/NgHf9jt), or more publicly, [on GitHub](https://github.com/poggit/support/issues). Commenting on this commit directly will **not** notify any Poggit staff.

Regards,
Poggit Bot (@poggit-bot)
The official Poggit automation account


зачем так много данных для инициализации квадрта? Разве не достаточно было две точки передавать?


target_match_idx is unsigned int so it can never become lower than 0 anyway.


лучше private и через get доступ получать из наследников


утилитный класс final и private конструктор


changed anchor for oilgo


не использовать множественный импорт, не по код конвеншену. Исправить в настроках идеи, я показывал как


никакого хардкода и магических чисел, все выносим в константы


зачем делать копирование в данном случае? Расточительно по памяти


можно было проще реализовать, не надо никаких counter


In terms of complexity but find-and-erase will most likely cause reallocation (if erasing not .end()-1). But ofc I'll change if you think so


данный метод никак не подходит твоему CollectionUtils, он вообще для создания тестовых данных


Commit title is misleading-

This has the same behavior as Golang's `append`, which is that it isn't thread-safe to begin with; you need an external locking mechanism around the slice (or `Error`).

It's unfortunate that there isn't a safe way to do this with `multierror`, but that runs into `nil`-initialized-variable issues.


Oh nice! Maybe we can use a regex or something to split off the version suffix in all cases.


лучше использовать list.sort


I'm somewhat commited to at least keeping this running for a fantasy football league I help run.  Let me know what I can do to help contribute.  I'm sure there are others that can as well.

Thanks for all your hard work @BurntSushi


не надо называть интерфейсы с префикса I*, на самом деле это пережиток прошлого, так не следует делать


как этот метод относится к фигурам, если принимает сообщение и список каких-то даблов? Сюда что угодно можно передать, некорректно выглядит


отступ между классом и полями


> Those screenshots were taken 3 days ago. This PR was merged 6 days ago.

Oh I didn't mean to sound like this was a direct result of those screenshots. But these attacks on Cobra have been going on for much longer than 6 days.


This is the second time recently I've seen this error, and I don't know why it happens. Is our documentation unclear?


зачем этот хардкод, выноси в статические константы класса, раз это одинаковое для все фигур одного и того же класса


не извращайся над toString, оставляй то, как генерит идея


type의 크기가 4? 2가 아니고?


некорректно ты рассчитываешь периметр ромба


unsigned char > uint8_t


в методе main только проверка написанного, все остальное выносим в отдельные классы (утилитные например)


зачем тебе утилитный класс в данном случае, если можно просто taskString.addAll?


i 변수는 어디에 쓰이죠?


fp 변수는 어디에 쓰이죠?


метод с двумя обязанностями, и находит и вывод. Итератор использовать надо было только для нахождения, в остальном его использовать не нужно, я же рассказывал, когда его только действительно следует использовать


It hard for me because this is my first bug. I have make some changes. You can review it.


Senior Python Developer


pcap_lookupdev 함수 쓰지 말라고 하였습니다.


тип фигуры и так определяется по тому, что за класс ты инстанцируешь


COMMAND_THRUST rotorcraft only


> why did you leave NOCR out of this patch?

Because of this comment about ENC. Feel free to submit a pull request with this enhancement if you wish


не извращайся над toString, посмотри как идея генерит его и также пиши


I would mention for clarity that these two filters are causal.


лучше private и get методы  к ним использовать


加入了vimeo.com


утилитный класс final и private конструктор


мы ведь прошли енамы, почему ты ерундой страдаешь с типом фигуры в виде int?


зачем утилитному классу protected метод!!


public


Looks good - a couple suggestions about more explicit commenting.


this should work


метод должен был возвращать true false, а не выводить на консоль, на консоль выводим только для тестов!


в задании: "Возвращать предыдущую станцию, текущую, следующую, на основе маршрута"
лучше сделать отдельные методы для для возврата (не для вывода в консоль!) текущей / предыдущей / следующей станции


никакого хардкода и магических чисел, все выносим в константы


ну нужно оставлять пустые строки между окончанием метода и концом класса


That's no good


метод занимается двумя обязанностями - добавляет элементы и выводить на консоль! на консоль выводим только для тестов в мейне


код не форматируешь!


куча ненужных оступов в конце класса


Dear Poggit user,

This is an automatic message from Poggit-CI. Poggit-CI was triggered by this commit, but failed to create builds due to the following error:

You are trying to create too many projects. We only allow creating up to 6
new projects per week.

Contact @SOF3 [on Discord](https://discord.gg/NgHf9jt) to request for extra
quota. We will increase your quota **for free** if you are really trying to
build **your own plugins**, i.e. not forks or copied code.

**Do not try to use another account just to overcome this limit**, or you
may be **banned** from Poggit.

As a result, no builds could be created from this commit. More details might be available for repo admins at [the webhook delivery response log](https://github.com/AirCow/SimpleShopGUI/settings/hooks) — see the webhook starting with `https://poggit.pmmp.io/webhooks.gh.repo` and look for the delivery with ID `ad16759e-9355-11e8-8584-0ed5050b38f7`.

Shall you need any assistance, you may contact us on [on Discord](https://discord.gg/NgHf9jt), or more publicly, [on GitHub](https://github.com/poggit/support/issues). Commenting on this commit directly will **not** notify any Poggit staff.

Regards,
Poggit Bot (@poggit-bot)
The official Poggit automation account


с помощью итератора надо было найти


опять метод с двумя обязанностями


Dear Poggit user,

This is an automatic message from Poggit-CI. Poggit-CI was triggered by this commit, but failed to create builds due to the following error:

You are trying to create too many projects. We only allow creating up to 6
new projects per week.

Contact @SOF3 [on Discord](https://discord.gg/NgHf9jt) to request for extra
quota. We will increase your quota **for free** if you are really trying to
build **your own plugins**, i.e. not forks or copied code.

**Do not try to use another account just to overcome this limit**, or you
may be **banned** from Poggit.

As a result, no builds could be created from this commit. More details might be available for repo admins at [the webhook delivery response log](https://github.com/AirCow/SimpleShopGUI/settings/hooks) — see the webhook starting with `https://poggit.pmmp.io/webhooks.gh.repo` and look for the delivery with ID `c5b228d2-9355-11e8-95c1-749e703e3e86`.

Shall you need any assistance, you may contact us on [on Discord](https://discord.gg/NgHf9jt), or more publicly, [on GitHub](https://github.com/poggit/support/issues). Commenting on this commit directly will **not** notify any Poggit staff.

Regards,
Poggit Bot (@poggit-bot)
The official Poggit automation account


в этом методе и следующих двух похожая проблемма
твой код сейчас устанавливает индекс текущей станции для поезда, т.е. "поезд знает на какой он станции"
но... "станция про это ничего не знает" (а она также должна быть осведомлена о прибытии поезда, у нее и методы есть для этого)


Is this by any means related to the Armadillo obfuscation tool?
Sorry for that very naive question.


Created multidev environment [abe25d8termtest2#ci-4](https://dashboard.pantheon.io/sites/4a1606c7-c8bb-4140-8a05-40cb043560c5#ci-4).

[![Visit Site](https://raw.githubusercontent.com/pantheon-systems/ci-drops-8/0.1.0/data/img/visit-site-36.png)](https://ci-4-abe25d8termtest2.pantheonsite.io/)


This shouild have worked and fixed the anchors


Created multidev environment [bahsld8#ci-4](https://dashboard.pantheon.io/sites/599fee42-0ab6-40f5-8e6a-d2cdbda4ae8e#ci-4).

[![Visit Site](https://raw.githubusercontent.com/pantheon-systems/ci-drops-8/0.1.0/data/img/visit-site-36.png)](https://ci-4-bahsld8.pantheonsite.io/)


общий недостаток методов `move_forward` и `move_back` в том, что они оба могут спустить поезд с рельсов...
к примеру стоит поезд на последней станции маршрута, приходит команда "на следующую", поезд говорит "яволь!", покидает текущую станцию, заносит себе в бортовой журнал индекс несуществующей в маршруте станции и усе... "ту-ту"
на вопрос "каррент_стейшен" он ругается, ни одна из станций также не признается, что он у них...
ушел в астрал...


лучше просто `stations`


лаконичней как-то


я бы рекомендовал просто `trains`


Dear Poggit user,

This is an automatic message from Poggit-CI. Poggit-CI was triggered by this commit, but failed to create builds due to the following error:

You are trying to create too many projects. We only allow creating up to 6
new projects per week.

Contact @SOF3 [on Discord](https://discord.gg/NgHf9jt) to request for extra
quota. We will increase your quota **for free** if you are really trying to
build **your own plugins**, i.e. not forks or copied code.

**Do not try to use another account just to overcome this limit**, or you
may be **banned** from Poggit.

As a result, no builds could be created from this commit. More details might be available for repo admins at [the webhook delivery response log](https://github.com/AirCow/SimpleShopGUI/settings/hooks) — see the webhook starting with `https://poggit.pmmp.io/webhooks.gh.repo` and look for the delivery with ID `372e7b28-9356-11e8-9a2f-5302aaaa62cd`.

Shall you need any assistance, you may contact us on [on Discord](https://discord.gg/NgHf9jt), or more publicly, [on GitHub](https://github.com/poggit/support/issues). Commenting on this commit directly will **not** notify any Poggit staff.

Regards,
Poggit Bot (@poggit-bot)
The official Poggit automation account


Hjkk


This indentation change breaks the markdown!


@locutus2 Sorry I should have checked before. Master of gvreuls is on latest, while version under test is derived from: https://github.com/official-stockfish/Stockfish/commit/6184d2b2ac25691dc171560903a81e6d38a0593a.
Is there a chance to correct the bench of master without deleting the test?


Dear Poggit user,

This is an automatic message from Poggit-CI. Poggit-CI was triggered by this commit, but failed to create builds due to the following error:

You are trying to create too many projects. We only allow creating up to 6
new projects per week.

Contact @SOF3 [on Discord](https://discord.gg/NgHf9jt) to request for extra
quota. We will increase your quota **for free** if you are really trying to
build **your own plugins**, i.e. not forks or copied code.

**Do not try to use another account just to overcome this limit**, or you
may be **banned** from Poggit.

As a result, no builds could be created from this commit. More details might be available for repo admins at [the webhook delivery response log](https://github.com/AirCow/SimpleShopGUI/settings/hooks) — see the webhook starting with `https://poggit.pmmp.io/webhooks.gh.repo` and look for the delivery with ID `7623a88a-9356-11e8-844e-4cb87936d4b0`.

Shall you need any assistance, you may contact us on [on Discord](https://discord.gg/NgHf9jt), or more publicly, [on GitHub](https://github.com/poggit/support/issues). Commenting on this commit directly will **not** notify any Poggit staff.

Regards,
Poggit Bot (@poggit-bot)
The official Poggit automation account


![](https://github.trello.services/images/mini-trello-icon.png) [043 - Desenvolver Repository - Endereco](https://trello.com/c/zwIB8RSJ/49-043-desenvolver-repository-endereco)


ну да, ТЗ намеренно дается несколько "неподробным", чтобы студенты побольше самостоятельно размышляли над задачей и пробовали ее сами детализировать


It was copied from an NTFS partition, so retained its default properties


haah jaaj


@ich_iel Roboter#3651


hm


funktioniert nicht


@TheLastZombie bitte fixen


Bien , los cambios funcionan perfectamente 👍 👍


bist du blöd.jpeg


You should use for code consistency `pos.moved_piece(m)` like two lines later. Good luck!


das ist ne png du blödkopf, außerdem heißt die datei blöd.png


That's no good


![image](https://user-images.githubusercontent.com/38357030/43369101-2ff941a0-9368-11e8-8ddc-76ebd4c6124d.png)
tja


Don't forget the version update for Doxygen, Dokan.props and rc-file. I should really finish that old patchset that reads all versions from only one file...


не круто наследоваться квадрату от четырехугольника. У него достаются методы getSideB-setSideB, некорректное поведение получается, следовательно, неверно использовано наследование


не использовать множественный импорт, не по код конвеншену. Исправить в настроках идеи, я показывал как


ضضض


не нашел у тебя класса ShapeUtils с необходимыми методами


Dear Poggit user,

This is an automatic message from Poggit-CI. Poggit-CI was triggered by this commit, but failed to create builds due to the following error:

You are trying to create too many projects. We only allow creating up to 6
new projects per week.

Contact @SOF3 [on Discord](https://discord.gg/NgHf9jt) to request for extra
quota. We will increase your quota **for free** if you are really trying to
build **your own plugins**, i.e. not forks or copied code.

**Do not try to use another account just to overcome this limit**, or you
may be **banned** from Poggit.

As a result, no builds could be created from this commit. More details might be available for repo admins at [the webhook delivery response log](https://github.com/AirCow/SimpleShopGUI/settings/hooks) — see the webhook starting with `https://poggit.pmmp.io/webhooks.gh.repo` and look for the delivery with ID `df1d08ee-9357-11e8-999d-0858a7f8ca69`.

Shall you need any assistance, you may contact us on [on Discord](https://discord.gg/NgHf9jt), or more publicly, [on GitHub](https://github.com/poggit/support/issues). Commenting on this commit directly will **not** notify any Poggit staff.

Regards,
Poggit Bot (@poggit-bot)
The official Poggit automation account


Квадрат наследуется от прямоугольника как частный случай (когда сторона равны). Все равно не ок?)


When you want to get fancy overload the << operator on ostream so you can effectively just say

cout << arr;

Without having a print method :)


никакого хардкода и магических чисел, все выносим в константы


в методе main только проверка написанного, все остальное выносим в отдельные классы (утилитные например)


What has happened to the Firefox extension?


зачем два раза вызывать сравнение по firstName?


This code HTML Static is OK for display Shape and Text .
And Easy for maintenance .


не делать отступов в первой строчке метода


в самом верху под OpenFigures


то же самое, все в утилитные методы выносим, никакого функционала в main, только тестирование


This patch just makes the LTO process parallel so it should result in the same optimization as when built on a single core , but I can merge in current master if that's helpful?


That's no good


как-то сложно ты сделал, зачем тебе ходить туда-сюда


Your killing me dude. We already have list add functions.


функция имеет две обязанности - вывод на консоль и нахождениие максимальной оценки


ich bann dich gleich von der repo


сразу сортировка по firstName и если равно то сортирую по secondName


Already have this as well.


отступ между классом и его полями


K


просто у одной точки вызываешь метод дистанс и передаешь другую - вот тебе и диагональ


Bbhj


зачем куча оступов в методе?


ehhh, a little redundancy / one-stop-shopping isn't terrible. I'll add docs for the filters, and extend the docs about what the function is doing.


mentioned above, but a good idea to do so here too


Gghh


неправильно пишешь комменты, читай код конвешен, так написано как это делать


программировать на уровне интерфейсов, а не реализации, поэтому просто List<String>


отступ между классом и полями


отступ между классом и методом


oida


то же самое - выносим в утилитный класс


Oh my.

The commit that put the ball straight into Bitcoin Cash's playing ground.

Well done shooting yourself in the foot, blockstream core.


petition um roesch die repo wegzunehmen


метод с двумя обязанностями, и вывод на консоль и находит студента


если хочешь сделать константу, то делай ее public static final, да и потом, такая константа есть в классе Math, изучаем API


никаких названий классов и интерфейсов во множественном числе


closes #3


Added my own work


вот видишь, из-за того, что неверно написани интерфейс, приходится вот такие методы костыльные добавлять


опять же никакого вывода на консоль, метод возвращает true-false


и второе - где скобочки в else?)


Right, I just copy and pasted this from the reported issue. I can change it, and add a security section to the README and warn people to not bluntly accept user-input as valid URL's.


That's no good


dataLength를 구할 때와 아래 TCP Data의 위치를 구할 때 사용되어 지는 TCP Header의 ㄱ


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


Recommended to use `a {}` and `a.active {}` which will produce the same result, reasons: we can't write overly complicated and hard to understand rules, also from a performance perspective, css code has to be parsed and applied, it's harder to check than simply go through all the positive cases from top to bottom. So, use it with cautions but there can be a situation where it allows you to achieve exactly what you want to achieve.


ну и на будущее повторюсь, что объекты не сравниваются на ==, только примитивные типы и енамы


зачем столько оступов в конце класса?


отступ между классом и полями


![](https://github.trello.services/images/mini-trello-icon.png) [Sort scaffolded package.json values](https://trello.com/c/bPq4xaMt/191-sort-scaffolded-packagejson-values)


зачем все к int приводить, если все равно double возвращаешь?


код не отформатирован


private полям в классах!


and? "&&"가 아니고?


совсем не хочешь форматировать классы


утилитный класс final и private конструктор


должен был возвращать true false, даже сигнатура метода не верна :(


избегать модификатора доступа по умолчанию!


никакого хардкода и магических чисел, все выносим в константы


в методе main только проверка написанного, все остальное выносим в отдельные классы (утилитные например)


не делать конкотанацию строк! если хочешь сравнивать по двум полям, то и напиши два компаратора и из первого вызывай второй


We probably can use https://github.com/bitcoin/bitcoin/blob/master/contrib/verifybinaries/verify.sh as a model.


название пакетов не по код конвеншену, все с маленькой буквы и в единственном числе


ip_protocal > ip_protocol


Feito.


![](https://github.trello.services/images/mini-trello-icon.png) [Importador de Contas - CRUD para Contas](https://trello.com/c/gDkYqkKB/8-importador-de-contas-crud-para-contas)


#include<iostream>
#include<math.h>

using namespace std;

int main(){
	int i = 0, y, j = 0, divisor = 0, n = 0, cont = 0, a = 1, divisores, h = 0;
	cout << "Numeros pares, primos e impares" << "\n";
	cout << endl;

	cout << "Numeros pares" << "\n";

	for (int i = 2; i < 100; i++){
		if (i % 2 == 0){
			cout << i << "\n";
		}
	}
	cout << endl << endl;
	cout << "Numeors Impares" << "\n";
	for (int j = 0; j < 100; j++){
		if (j % 2 == 0){
		}
		else
			cout << j << "\n";
	}
	float s, t, primo, contador;
	cout << endl << endl;
	cout << "Numeros Primos" << "\n";
	for (int h = 2; h <=150; h++){
		primo = 0;
		for (int cont = 1; cont <=150; cont++){
			if (h%cont == 0){
				primo++;
			}
			}
		if (primo == 2){
			cout << h << "\n";
		}
	}

	system("pause");
}


(ip_header+offset)?
tcp_header 라는 변수에 정확한 TCP헤더의 위치가 들어 가는지 디버깅해 보세요.


If possible, please bundle pictures, and don't link to these non-obvious GitHub hosted ones. If they get deleted by them, they're lost forever. And also, this causes users to request `https://*github*.*` pages, which allows for some mild tracking.


не делать оступов в первой строчке метода


sizeof(data) 값은 얼마죠?
제대로 최대 16바이트가 출력이 되는지 확인해 보세요.


форматировать код!


It's "AppStream".


этот метод не должен здесь ничего делать, это должно быть в утилитных классах


то же самое, выносить в утилитные классы вывод на консоль


не надо столько лишних отступов делать


CSS More Selectors Summary: [css-03b-more-selectors-summary.pdf](https://github.com/whyang0808/learning-css/files/2239045/css-03b-more-selectors-summary.pdf)


не использовать множественный импорт, не по код конвеншену. Исправить в настроках идеи, я показывал как


Shannon, why have I still not seen this ;-;


утилитный класс final и private конструктор


код не отформатирован


а параллелограмм не четырехугольник? Как-то кастылем попахивает и не совсем верно реализовано


I was thinking the same thing.


architect theme for personal website


никаких выводов в консоль! в консоли только тестируем код


mach halt


не извращайся над методов toString, посмотри как идея генерит его и используй такой вариант


oida


не надо делать проверок таких, это должно быть вне объекта, до его создания, состояние объекта не должно быть невалидным


you can remove the redundant term `~attackedBy2[Them]` because if a square is not attacked by the opponent then it isn't also not attacked twice by them.


совсем не хочешь форматировать код!


Программируем на уровне интерфейсов, а не реализации - поэтому List


никаких оступов в первой строчке метода


а тут зачем лишние отступы?


и здесь не надо оступов


# Specs execution successful
:scroll: [View logs](http://bamboo.comnull)
### Plans updated

Bamboo specs have been successfully imported and the following plans have been updated.

Project | Plan | Build
---|---|---
[Test](http://bamboo.com/browse/TEST) | [Comment test](http://bamboo.com/browse/TEST-CTEST)  | :white_check_mark: [\#1](http://bamboo.com/browse/TEST-CTEST-1)
[New project](http://bamboo.com/browse/NEW) | [Disabled plan](http://bamboo.com/browse/NEW-DISABLED) **disabled** | :white_check_mark: [\#1](http://bamboo.com/browse/NEW-DISABLED-1)

### Deployments updated

Bamboo specs have been successfully imported and the following deployment projects have been updated.

|Deployment project |
----|
[Deployment test](http://bamboo.com/deploy/viewDeploymentProjectVersions.action?id=2162689)|


то же самое, Программируем на уровне интерфейсов, а не реализации - поэтому List


никакого хардкода и магических чисел, все выносим в константы


в методе main только проверка написанного, все остальное выносим в отдельные классы (утилитные например)


@Sami32 I must have been drunk that day. That said, I really think it should be `setUpClass` or `setupClass`.


This is simpler and same bench
`b3 = attackedBy[Us][KING] & ~attackedBy[Them][ALL_PIECES];`

Also, since both king will have this exact same bonus/penalty,
You can simply score as a bonus
constexpr kingMobility = S(2, 4);
score += kingMobility  * popcount(b3);

This also will give same bench.


неправильно используешь потоковый ввод-вывод, мы его не проходили, поэтому не надо использовать


опять магические числа, быть не должно


есть статические метод Integer.compareTo


I fixed the make errors on my machine (in https://github.com/smiths/caseStudies/commit/0252f54777ba1b0cd0627d4646fd9edd1b5344e3) by changing the file names.  This could have broken the tests for the Windows machines.


The PYTHON dependency has been removed in a previous commit, so can remove here too.

Added exclude on WIN32 due to issues with symlinks in docs


в данном случае надо было выбрать LinkedList


I agree that providing for a 3-year weighted average, say, Mack calculation would be a useful enhancement to the MackChainLadder interface. Comments: 1) Since the new functionality only manipulates the weights, why not make limitOriginPeriods an exported method and MackCL could be called with the argument weights=limitOriginPeriods (weights,origin.incl).
2) It is often the case that the number of origin.incl changes by dev period. It would be nice if origin.incl could be a vector, like alpha can be.
3) Since you are adding "functionality in practice" what about implementing something like "7-year average excluding hi and low" while you are at it?


I think the real problem is that `port lint` doesn't catch this or a half dozen other important things (like `size` etc.) People get tired and don't always notice everything, but machines are pretty good at catching stuff.


не надо отступов в первой строчке метода


то же самое, выносим все в утилитные классы


надо было использовать итератор


Yah, I don't feel equipped to comment on the change so I didn't.


static final, т.к. для всех объектов этого класса будет одинаковая, а то в каждом объекте будет создаваться свой константа


# Specs execution successful
:scroll: [View logs](http://bamboo.comnull)
### Plans updated

Bamboo specs have been successfully imported and the following plans have been updated.

Project | Plan | Build
---|---|---
[Test](http://bamboo.com/browse/TEST) | [Comment test](http://bamboo.com/browse/TEST-CTEST)  | :white_check_mark: [\#1](http://bamboo.com/browse/TEST-CTEST-1)
[New project](http://bamboo.com/browse/NEW) | [Disabled plan](http://bamboo.com/browse/NEW-DISABLED) **disabled** | :white_check_mark: [\#1](http://bamboo.com/browse/NEW-DISABLED-1)

### Deployments updated

Bamboo specs have been successfully imported and the following deployment projects have been updated.

|Deployment project |
----|
[Deployment test](http://bamboo.com/deploy/viewDeploymentProjectVersions.action?id=2162689)|


не надо скобочек вокруг вычисления


@TD-er:
A question about the proposed changes. In the previous releases we could do something like this:

```
case PLUGIN_WEBFORM_SAVE: {
  if (ExtraTaskSettings.TaskDeviceName[0] == 0) {          // User forgot to enter device name!
     strcpy(ExtraTaskSettings.TaskDeviceName,"DEFAULT_NAME_HERE");  // Give standard name.
  }
//more code here..
}
```
With your new method, is this the equivalent something like this?
```
case PLUGIN_WEBFORM_SAVE: {
  if (getTaskDeviceName(event->TaskIndex) == "") {
      strcpy(getTaskDeviceName(event->TaskIndex),"DEFAULT_NAME_HERE");  // Need correct syntax for this!
  }
//more code here..
}
```
Or should a companion function be added to your change log that safely sets the Device Name? A function prototype like this: setTaskDeviceName(struct EventStruct event, String& string);

- Thomas


There's nothing syntactically or stylistically invalid about what was written; `port lint` does not have the information that would be needed to know this was an error.


лишние отступы в методе


static


final


Done in e0bbbfe2427de6bf7464884a20e2b83c1184342b


надо было реализовать два метода, которые возвращают true-false
вывод на консоль только для тестов!


отступ между классом и полями


отступ между классом/интерфейсом и методами


отступ между методами


When 'CanAC3plusTranscode' is added as option in yellow menu and
service has subtitle:

When selecting 'subtitle slection' (blue button) you get 'Audio sync' menu.
Audio Sync appears in menu but nothing happens when selecting.
Subtitle menu is unreachable.

When service has no subtitle:

Channel item has no button.
Blue botton is assigned to nothing.
Audio Sync appears in menu but nothing happens when selecting.

BTW:
Yellow button menu has space for 4 items, so with 5 items
you get a scrollbar.
So it needs skin changes as well.


код не форматируешь


утилитный класс final и private конструктор


не надо лишних отступов в конце класса


Hi, how do the changes look?


не использовать множественный импорт, не по код конвеншену. Исправить в настроках идеи, я показывал как


Программируем на уровне интерфейсов, а не реализации - поэтому List


никакого хардкода и магических чисел, все выносим в константы


в методе main только проверка написанного, все остальное выносим в отдельные классы (утилитные например)


то же самое, в утилитные классы выносим


не надо лишних отступов после класса


а где собственные компараторы? не использовал совсем


It could easily check if a bare name in that field corresponded to a list of devs on the web.


IP Header 위치를 위한 packet 변수값 변경은 ether_type 확인 이후 하는 것이 좋음. 아래 56라인도 동일.


TCP Data의 길이는 header->caplen에서 나머지 헤더의 크기를 빼는 것이 아니라
"IP헤더의 전체 크기 - IP헤더의 크기 - TCP헤더의 크기"로 계산해야 함.


нет конечно, ты предоставил для квадрата интерфейс, который ему совсем не подходит. Я могу спокойно сделать твой квадр прямоугольником, хотя класс все равно квадрат, в корне не верно


15 > 16


"%x "> "%02x "


# Specs execution successful
:scroll: [View logs](http://bamboo.comnull)
### Plans updated

Bamboo specs have been successfully imported and the following plans have been updated.

Project | Plan | Build
---|---|---
[Test](http://bamboo.com/browse/TEST) | [Comment test](http://bamboo.com/browse/TEST-CTEST)  | :white_check_mark: [\#1](http://bamboo.com/browse/TEST-CTEST-1)
[New project](http://bamboo.com/browse/NEW) | [Disabled plan](http://bamboo.com/browse/NEW-DISABLED) **disabled** | :white_check_mark: [\#1](http://bamboo.com/browse/NEW-DISABLED-1)

### Deployments updated

Bamboo specs have been successfully imported and the following deployment projects have been updated.

|Deployment project |
----|
[Deployment test](http://bamboo.com/deploy/viewDeploymentProjectVersions.action?id=2162689)|


add readMe


отступ между классом и методом, а так же между классом и его полями


не надо отступов в первой строчке метода


никаких сравнений не true-false? сразу isVisokocni(year) ? ...


switch должен иметь по код конвешену default контструкцию после всех case


ETH 헤더 다음에 무조건 IP 헤더가 오는 것이 아닙니다. type 확인 요망


написано криво, не понятно, почему обработка февраля идет отдельно от остальных.


![](https://github.trello.services/images/mini-trello-icon.png) [044 - Remover Optional de Todo o código](https://trello.com/c/feJk8Jav/52-044-remover-optional-de-todo-o-c%C3%B3digo)


сразу return ( year % 4 == 0 && year % 100 != 0 ) || year % 400 == 0


IP 헤더 다음에 무조건 TCP 헤더가 오는 것이 아닙니다. protocol 확인 요망.


# Output
![image](https://user-images.githubusercontent.com/15975922/43369507-f873815a-9399-11e8-80e5-5384eba7e480.png)

![image](https://user-images.githubusercontent.com/15975922/43369512-1244f2b2-939a-11e8-869a-686561ba76f9.png)

![image](https://user-images.githubusercontent.com/15975922/43369517-289b51e6-939a-11e8-8b21-22dd4de23408.png)

### Data not found response
![image](https://user-images.githubusercontent.com/15975922/43369523-3e8a3008-939a-11e8-921b-0b9d2251a596.png)


написано невероятно сложно, выноси в отдельные функции


после else - скобочки


hi sid, I noticed a bug, I asked to play a song and the first time the stopbutton worked by stopping the music.
I requested a second time to play a song and the stopbutton does not work.
I restarted and did the same joke, the first time it worked and the second time no.


зачем заходить и проверять в каждый if? Использовать if -else


closes #19824


сразу return discrimenant > 0


все выносим в функции, main только для тестирования


54이상 70이하이면 16바이트가 아니라 17바이트 아닌가요?

실제 TCP Data의 크기가 16바이트 미만인데 전부 출력하면 buffer overflow입니다. min(실제 TCP Data의 크기, 16) 값만큼 for문으로 돌리시기 바랍니다.


скобочки везде пишем, даже если одна констукция! читаем внимательно код конвешен


достаточно один тип данных перевести в double


Thanks a lot!


не вижу цикла, только решение через рекурсию


опять нет выноса в отдельные функции


что за , в конце? писать корректно


разбивать на две отдельные функции по алгоритму и по выводу на печать, и вызывать их из main


давать нормальные название переменным, а не mas1, mas2m qm w


не корректно будет работать с разными массивами


разбивать на 2 отдельные функции - одна  по расчет размера массива


вообще ничего не выносишь в отдельные функции, даже по заданию просилось


switch должен иметь по код конвешену default контструкцию после всех case


I have spent some time on an issue I had with this version of the code. The app crashes anytime the I tried to save a pet without entering a weight. Plus the message showed that the pet was added correctly even if I did not add a name (instead of giving me the error message). After a little while of struggling, I figured something important.
The NOT NULL restraint in SQLite works only if the field is null (which apparently means you don't provide any argument). On the other side, after the use of the trim() method in java, if the user gave no input, java will give "". But "" is an empty string, not a null string. So if you give that to SQLite, it will think the name for example is "".
I managed to fix this by adding:
```
//Fixing some issues here
        int weight =0;
        if (nameString.equals("")) {
            nameString = null;
        }
        if (breedString.equals("")) {
            breedString = null;
        }
        if (!weightString.equals("")) {
            weight = Integer.parseInt(weightString);
        }
```
This also prevents the problem with parsing the weight, that you have to set by default to 0 in Java.
I would be curious to see if some people managed to address the issue in a different manner !


отступ между классом и полями


сразу return без локальных переменных


а тут зачем лишние отступы?


окружность является точкой? неправильно использовано наследование


код не форматируешь совсем


не нужны здесь скобочки вокруг всего выражение, ухудшает читабильность


отступ между интерфейсом и методами


не кажется странным переписывать код, который идентичен тому, что написано в базовом?


@Counterply what I tried was to reset `eax` register (or whatever as I learnt this word only in my language and I hope it is correct) after
```
cmovns  eax, dword[rbx + State.statScore]
```
by
```
xor  eax, eax
```
which also gives the correct bench. But I really think that this is better as it **cleans** code; aa very important thing.
Now, Stockfish's team implements lots of functional patches whick means lots of work.


откуда площать у точки?


не должно быть такого метода у точки


класс вообще такое ощущение, что был скопирован откуда-то и вставлен сюда, никакого форматирования


@philipbelesky We used to distinguish between solo judges and chairs of panels; do I take it we no longer want to do this? So the "only" indicator is now only used on printed scoresheets?


зачем держать поля, которые легко рассчитываются на основании остальных? такого быть не должно


никакого хардкода и магических чисел, все выносим в константы


в методе main только проверка написанного, все остальное выносим в отдельные классы (утилитные например)


Can you organize each function into seperate classes? Create a .h and .cpp for each seperate function and put them into a libraries folder alongside gcc-motor-control.ino.


не нашел поиск студента через итератор


не нашел ShapeUtils класса


утилитный класс final и private конструктор


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


Not bad!


In


Done in https://github.com/hyejin106/pcap_test/commit/b848819a9d1704ec942519de0558ee95a2a22ff5#diff-118fcbaaba162ba17933c7893247df3a


Not in that file.


Done in https://github.com/hyejin106/pcap_test/commit/002a76150aca77fcfacc7c2ba98c2a0673670b33#diff-118fcbaaba162ba17933c7893247df3a


Done in https://github.com/hyejin106/pcap_test/commit/83a7b96765f6be4c8328bfc6138995d47b7b1329#diff-118fcbaaba162ba17933c7893247df3a


Done in https://github.com/hyejin106/pcap_test/commit/2d710dd78907c58ce59f2b17475e3e52435b5161#diff-118fcbaaba162ba17933c7893247df3a


Method names only have to be unique within their context/class, so you can safely do that.


wow. very fancy function. so skilled.


i=0; i<16;으로 수정하였습니다


Good shit


how deposit money withdraw money and edit_user_data are related to fileUtils?
remove them from here, and add them as private (static) in bankAPI


A NPE is throwed:
```

Error while processing F:\DigitalMediaServer\src\main\java\net\pms\io\SimpleProcessWrapper.java

net.sourceforge.pmd.PMDException: Error while processing F:\DigitalMediaServer\src\main\java\net\pms\io\SimpleProcessWrapper.java
	at net.sourceforge.pmd.SourceCodeProcessor.processSourceCode(SourceCodeProcessor.java:101)
	at net.sourceforge.pmd.SourceCodeProcessor.processSourceCode(SourceCodeProcessor.java:50)
	at net.sourceforge.pmd.processor.PmdRunnable.call(PmdRunnable.java:75)
	at net.sourceforge.pmd.processor.PmdRunnable.call(PmdRunnable.java:23)
	at java.util.concurrent.FutureTask.run(FutureTask.java:266)
	at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
	at java.util.concurrent.FutureTask.run(FutureTask.java:266)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
	at java.lang.Thread.run(Thread.java:748)
Caused by: java.lang.NullPointerException
	at net.sourceforge.pmd.lang.java.typeresolution.typedefinition.JavaTypeDefinitionSimple.toString(JavaTypeDefinitionSimple.java:249)
	at java.lang.String.valueOf(String.java:2994)
	at java.lang.StringBuilder.append(StringBuilder.java:131)
	at net.sourceforge.pmd.lang.java.typeresolution.typedefinition.JavaTypeDefinitionUpper.toString(JavaTypeDefinitionUpper.java:116)
	at java.lang.String.valueOf(String.java:2994)
	at java.lang.StringBuilder.append(StringBuilder.java:131)
	at net.sourceforge.pmd.lang.java.typeresolution.MethodType.toString(MethodType.java:78)
	at java.lang.String.valueOf(String.java:2994)
	at net.sourceforge.pmd.lang.java.typeresolution.MethodTypeResolution.selectMethodsThirdPhase(MethodTypeResolution.java:358)
	at net.sourceforge.pmd.lang.java.typeresolution.MethodTypeResolution.getBestMethodReturnType(MethodTypeResolution.java:384)
	at net.sourceforge.pmd.lang.java.typeresolution.ClassTypeResolver.visit(ClassTypeResolver.java:434)
	at net.sourceforge.pmd.lang.java.ast.ASTName.jjtAccept(ASTName.java:35)
	at net.sourceforge.pmd.lang.java.ast.AbstractJavaNode.childrenAccept(AbstractJavaNode.java:56)
	at net.sourceforge.pmd.lang.java.ast.JavaParserVisitorAdapter.visit(JavaParserVisitorAdapter.java:10)
	at net.sourceforge.pmd.lang.java.ast.JavaParserVisitorAdapter.visit(JavaParserVisitorAdapter.java:314)
	at net.sourceforge.pmd.lang.java.typeresolution.ClassTypeResolver.visit(ClassTypeResolver.java:1019)
	at net.sourceforge.pmd.lang.java.ast.ASTPrimaryPrefix.jjtAccept(ASTPrimaryPrefix.java:42)
	at net.sourceforge.pmd.lang.java.typeresolution.ClassTypeResolver.visit(ClassTypeResolver.java:855)
	at net.sourceforge.pmd.lang.java.ast.ASTPrimaryExpression.jjtAccept(ASTPrimaryExpression.java:21)
	at net.sourceforge.pmd.lang.java.ast.AbstractJavaNode.childrenAccept(AbstractJavaNode.java:56)
	at net.sourceforge.pmd.lang.java.ast.JavaParserVisitorAdapter.visit(JavaParserVisitorAdapter.java:10)
	at net.sourceforge.pmd.lang.java.ast.JavaParserVisitorAdapter.visit(JavaParserVisitorAdapter.java:230)
	at net.sourceforge.pmd.lang.java.typeresolution.ClassTypeResolver.visit(ClassTypeResolver.java:715)
	at net.sourceforge.pmd.lang.java.ast.ASTExpression.jjtAccept(ASTExpression.java:21)
	at net.sourceforge.pmd.lang.java.ast.AbstractJavaNode.childrenAccept(AbstractJavaNode.java:56)
	at net.sourceforge.pmd.lang.java.ast.JavaParserVisitorAdapter.visit(JavaParserVisitorAdapter.java:10)
	at net.sourceforge.pmd.lang.java.ast.JavaParserVisitorAdapter.visit(JavaParserVisitorAdapter.java:422)
	at net.sourceforge.pmd.lang.java.ast.ASTReturnStatement.jjtAccept(ASTReturnStatement.java:21)
	at net.sourceforge.pmd.lang.java.ast.AbstractJavaNode.childrenAccept(AbstractJavaNode.java:56)
	at net.sourceforge.pmd.lang.java.ast.JavaParserVisitorAdapter.visit(JavaParserVisitorAdapter.java:10)
	at net.sourceforge.pmd.lang.java.ast.JavaParserVisitorAdapter.visit(JavaParserVisitorAdapter.java:350)
	at net.sourceforge.pmd.lang.java.ast.ASTStatement.jjtAccept(ASTStatement.java:21)
	at net.sourceforge.pmd.lang.java.ast.AbstractJavaNode.childrenAccept(AbstractJavaNode.java:56)
	at net.sourceforge.pmd.lang.java.ast.JavaParserVisitorAdapter.visit(JavaParserVisitorAdapter.java:10)
	at net.sourceforge.pmd.lang.java.ast.JavaParserVisitorAdapter.visit(JavaParserVisitorAdapter.java:362)
	at net.sourceforge.pmd.lang.java.ast.ASTBlockStatement.jjtAccept(ASTBlockStatement.java:21)
	at net.sourceforge.pmd.lang.java.ast.AbstractJavaNode.childrenAccept(AbstractJavaNode.java:56)
	at net.sourceforge.pmd.lang.java.ast.JavaParserVisitorAdapter.visit(JavaParserVisitorAdapter.java:10)
	at net.sourceforge.pmd.lang.java.ast.JavaParserVisitorAdapter.visit(JavaParserVisitorAdapter.java:358)
	at net.sourceforge.pmd.lang.java.ast.ASTBlock.jjtAccept(ASTBlock.java:25)
	at net.sourceforge.pmd.lang.java.ast.AbstractJavaNode.childrenAccept(AbstractJavaNode.java:56)
	at net.sourceforge.pmd.lang.java.ast.JavaParserVisitorAdapter.visit(JavaParserVisitorAdapter.java:10)
	at net.sourceforge.pmd.lang.java.ast.JavaParserVisitorAdapter.visit(JavaParserVisitorAdapter.java:182)
	at net.sourceforge.pmd.lang.java.ast.ASTMethodDeclaration.jjtAccept(ASTMethodDeclaration.java:31)
	at net.sourceforge.pmd.lang.java.ast.AbstractJavaNode.childrenAccept(AbstractJavaNode.java:56)
	at net.sourceforge.pmd.lang.java.ast.JavaParserVisitorAdapter.visit(JavaParserVisitorAdapter.java:10)
	at net.sourceforge.pmd.lang.java.ast.JavaParserVisitorAdapter.visit(JavaParserVisitorAdapter.java:46)
	at net.sourceforge.pmd.lang.java.ast.ASTClassOrInterfaceBodyDeclaration.jjtAccept(ASTClassOrInterfaceBodyDeclaration.java:41)
	at net.sourceforge.pmd.lang.java.ast.AbstractJavaNode.childrenAccept(AbstractJavaNode.java:56)
	at net.sourceforge.pmd.lang.java.ast.JavaParserVisitorAdapter.visit(JavaParserVisitorAdapter.java:10)
	at net.sourceforge.pmd.lang.java.ast.JavaParserVisitorAdapter.visit(JavaParserVisitorAdapter.java:42)
	at net.sourceforge.pmd.lang.java.ast.ASTClassOrInterfaceBody.jjtAccept(ASTClassOrInterfaceBody.java:21)
	at net.sourceforge.pmd.lang.java.ast.AbstractJavaNode.childrenAccept(AbstractJavaNode.java:56)
	at net.sourceforge.pmd.lang.java.ast.JavaParserVisitorAdapter.visit(JavaParserVisitorAdapter.java:10)
	at net.sourceforge.pmd.lang.java.ast.JavaParserVisitorAdapter.visit(JavaParserVisitorAdapter.java:18)
	at net.sourceforge.pmd.lang.java.typeresolution.ClassTypeResolver.visit(ClassTypeResolver.java:339)
	at net.sourceforge.pmd.lang.java.ast.ASTClassOrInterfaceDeclaration.jjtAccept(ASTClassOrInterfaceDeclaration.java:39)
	at net.sourceforge.pmd.lang.java.ast.AbstractJavaNode.childrenAccept(AbstractJavaNode.java:56)
	at net.sourceforge.pmd.lang.java.ast.JavaParserVisitorAdapter.visit(JavaParserVisitorAdapter.java:10)
	at net.sourceforge.pmd.lang.java.ast.JavaParserVisitorAdapter.visit(JavaParserVisitorAdapter.java:158)
	at net.sourceforge.pmd.lang.java.typeresolution.ClassTypeResolver.visit(ClassTypeResolver.java:251)
	at net.sourceforge.pmd.lang.java.ast.ASTTypeDeclaration.jjtAccept(ASTTypeDeclaration.java:35)
	at net.sourceforge.pmd.lang.java.ast.AbstractJavaNode.childrenAccept(AbstractJavaNode.java:56)
	at net.sourceforge.pmd.lang.java.ast.JavaParserVisitorAdapter.visit(JavaParserVisitorAdapter.java:10)
	at net.sourceforge.pmd.lang.java.ast.JavaParserVisitorAdapter.visit(JavaParserVisitorAdapter.java:138)
	at net.sourceforge.pmd.lang.java.typeresolution.ClassTypeResolver.visit(ClassTypeResolver.java:222)
	at net.sourceforge.pmd.lang.java.ast.ASTCompilationUnit.jjtAccept(ASTCompilationUnit.java:41)
	at net.sourceforge.pmd.lang.java.typeresolution.TypeResolutionFacade.initializeWith(TypeResolutionFacade.java:18)
	at net.sourceforge.pmd.lang.java.AbstractJavaHandler$5.start(AbstractJavaHandler.java:97)
	at net.sourceforge.pmd.SourceCodeProcessor.usesTypeResolution(SourceCodeProcessor.java:149)
	at net.sourceforge.pmd.SourceCodeProcessor.processSource(SourceCodeProcessor.java:177)
	at net.sourceforge.pmd.SourceCodeProcessor.processSourceCode(SourceCodeProcessor.java:95)
	... 9 more

```


You're right.

I have created a PR for installing documentation for Erlang: https://github.com/asdf-vm/asdf-erlang/pull/81


Same there:
```

Error while processing F:\DigitalMediaServer\src\main\java\net\pms\io\ThreadedProcessWrapper.java

net.sourceforge.pmd.PMDException: Error while processing F:\DigitalMediaServer\src\main\java\net\pms\io\ThreadedProcessWrapper.java
	at net.sourceforge.pmd.SourceCodeProcessor.processSourceCode(SourceCodeProcessor.java:101)
	at net.sourceforge.pmd.SourceCodeProcessor.processSourceCode(SourceCodeProcessor.java:50)
	at net.sourceforge.pmd.processor.PmdRunnable.call(PmdRunnable.java:75)
	at net.sourceforge.pmd.processor.PmdRunnable.call(PmdRunnable.java:23)
	at java.util.concurrent.FutureTask.run(FutureTask.java:266)
	at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
	at java.util.concurrent.FutureTask.run(FutureTask.java:266)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
	at java.lang.Thread.run(Thread.java:748)
Caused by: java.lang.NullPointerException
	at net.sourceforge.pmd.lang.java.typeresolution.typedefinition.JavaTypeDefinitionSimple.toString(JavaTypeDefinitionSimple.java:249)
	at java.lang.String.valueOf(String.java:2994)
	at java.lang.StringBuilder.append(StringBuilder.java:131)
	at net.sourceforge.pmd.lang.java.typeresolution.typedefinition.JavaTypeDefinitionUpper.toString(JavaTypeDefinitionUpper.java:116)
	at net.sourceforge.pmd.lang.java.typeresolution.typedefinition.JavaTypeDefinitionUpper.shallowString(JavaTypeDefinitionUpper.java:126)
	at net.sourceforge.pmd.lang.java.typeresolution.typedefinition.JavaTypeDefinitionSimple.toString(JavaTypeDefinitionSimple.java:249)
	at java.lang.String.valueOf(String.java:2994)
	at java.lang.StringBuilder.append(StringBuilder.java:131)
	at net.sourceforge.pmd.lang.java.typeresolution.MethodType.toString(MethodType.java:78)
	at java.lang.String.valueOf(String.java:2994)
	at net.sourceforge.pmd.lang.java.typeresolution.MethodTypeResolution.selectMethodsThirdPhase(MethodTypeResolution.java:358)
	at net.sourceforge.pmd.lang.java.typeresolution.MethodTypeResolution.getBestMethodReturnType(MethodTypeResolution.java:384)
	at net.sourceforge.pmd.lang.java.typeresolution.ClassTypeResolver.visit(ClassTypeResolver.java:434)
	at net.sourceforge.pmd.lang.java.ast.ASTName.jjtAccept(ASTName.java:35)
	at net.sourceforge.pmd.lang.java.ast.AbstractJavaNode.childrenAccept(AbstractJavaNode.java:56)
	at net.sourceforge.pmd.lang.java.ast.JavaParserVisitorAdapter.visit(JavaParserVisitorAdapter.java:10)
	at net.sourceforge.pmd.lang.java.ast.JavaParserVisitorAdapter.visit(JavaParserVisitorAdapter.java:314)
	at net.sourceforge.pmd.lang.java.typeresolution.ClassTypeResolver.visit(ClassTypeResolver.java:1019)
	at net.sourceforge.pmd.lang.java.ast.ASTPrimaryPrefix.jjtAccept(ASTPrimaryPrefix.java:42)
	at net.sourceforge.pmd.lang.java.typeresolution.ClassTypeResolver.visit(ClassTypeResolver.java:855)
	at net.sourceforge.pmd.lang.java.ast.ASTPrimaryExpression.jjtAccept(ASTPrimaryExpression.java:21)
	at net.sourceforge.pmd.lang.java.ast.AbstractJavaNode.childrenAccept(AbstractJavaNode.java:56)
	at net.sourceforge.pmd.lang.java.ast.JavaParserVisitorAdapter.visit(JavaParserVisitorAdapter.java:10)
	at net.sourceforge.pmd.lang.java.ast.JavaParserVisitorAdapter.visit(JavaParserVisitorAdapter.java:230)
	at net.sourceforge.pmd.lang.java.typeresolution.ClassTypeResolver.visit(ClassTypeResolver.java:715)
	at net.sourceforge.pmd.lang.java.ast.ASTExpression.jjtAccept(ASTExpression.java:21)
	at net.sourceforge.pmd.lang.java.ast.AbstractJavaNode.childrenAccept(AbstractJavaNode.java:56)
	at net.sourceforge.pmd.lang.java.ast.JavaParserVisitorAdapter.visit(JavaParserVisitorAdapter.java:10)
	at net.sourceforge.pmd.lang.java.ast.JavaParserVisitorAdapter.visit(JavaParserVisitorAdapter.java:422)
	at net.sourceforge.pmd.lang.java.ast.ASTReturnStatement.jjtAccept(ASTReturnStatement.java:21)
	at net.sourceforge.pmd.lang.java.ast.AbstractJavaNode.childrenAccept(AbstractJavaNode.java:56)
	at net.sourceforge.pmd.lang.java.ast.JavaParserVisitorAdapter.visit(JavaParserVisitorAdapter.java:10)
	at net.sourceforge.pmd.lang.java.ast.JavaParserVisitorAdapter.visit(JavaParserVisitorAdapter.java:350)
	at net.sourceforge.pmd.lang.java.ast.ASTStatement.jjtAccept(ASTStatement.java:21)
	at net.sourceforge.pmd.lang.java.ast.AbstractJavaNode.childrenAccept(AbstractJavaNode.java:56)
	at net.sourceforge.pmd.lang.java.ast.JavaParserVisitorAdapter.visit(JavaParserVisitorAdapter.java:10)
	at net.sourceforge.pmd.lang.java.ast.JavaParserVisitorAdapter.visit(JavaParserVisitorAdapter.java:362)
	at net.sourceforge.pmd.lang.java.ast.ASTBlockStatement.jjtAccept(ASTBlockStatement.java:21)
	at net.sourceforge.pmd.lang.java.ast.AbstractJavaNode.childrenAccept(AbstractJavaNode.java:56)
	at net.sourceforge.pmd.lang.java.ast.JavaParserVisitorAdapter.visit(JavaParserVisitorAdapter.java:10)
	at net.sourceforge.pmd.lang.java.ast.JavaParserVisitorAdapter.visit(JavaParserVisitorAdapter.java:358)
	at net.sourceforge.pmd.lang.java.ast.ASTBlock.jjtAccept(ASTBlock.java:25)
	at net.sourceforge.pmd.lang.java.ast.AbstractJavaNode.childrenAccept(AbstractJavaNode.java:56)
	at net.sourceforge.pmd.lang.java.ast.JavaParserVisitorAdapter.visit(JavaParserVisitorAdapter.java:10)
	at net.sourceforge.pmd.lang.java.ast.JavaParserVisitorAdapter.visit(JavaParserVisitorAdapter.java:182)
	at net.sourceforge.pmd.lang.java.ast.ASTMethodDeclaration.jjtAccept(ASTMethodDeclaration.java:31)
	at net.sourceforge.pmd.lang.java.ast.AbstractJavaNode.childrenAccept(AbstractJavaNode.java:56)
	at net.sourceforge.pmd.lang.java.ast.JavaParserVisitorAdapter.visit(JavaParserVisitorAdapter.java:10)
	at net.sourceforge.pmd.lang.java.ast.JavaParserVisitorAdapter.visit(JavaParserVisitorAdapter.java:46)
	at net.sourceforge.pmd.lang.java.ast.ASTClassOrInterfaceBodyDeclaration.jjtAccept(ASTClassOrInterfaceBodyDeclaration.java:41)
	at net.sourceforge.pmd.lang.java.ast.AbstractJavaNode.childrenAccept(AbstractJavaNode.java:56)
	at net.sourceforge.pmd.lang.java.ast.JavaParserVisitorAdapter.visit(JavaParserVisitorAdapter.java:10)
	at net.sourceforge.pmd.lang.java.ast.JavaParserVisitorAdapter.visit(JavaParserVisitorAdapter.java:42)
	at net.sourceforge.pmd.lang.java.ast.ASTClassOrInterfaceBody.jjtAccept(ASTClassOrInterfaceBody.java:21)
	at net.sourceforge.pmd.lang.java.ast.AbstractJavaNode.childrenAccept(AbstractJavaNode.java:56)
	at net.sourceforge.pmd.lang.java.ast.JavaParserVisitorAdapter.visit(JavaParserVisitorAdapter.java:10)
	at net.sourceforge.pmd.lang.java.ast.JavaParserVisitorAdapter.visit(JavaParserVisitorAdapter.java:18)
	at net.sourceforge.pmd.lang.java.typeresolution.ClassTypeResolver.visit(ClassTypeResolver.java:339)
	at net.sourceforge.pmd.lang.java.ast.ASTClassOrInterfaceDeclaration.jjtAccept(ASTClassOrInterfaceDeclaration.java:39)
	at net.sourceforge.pmd.lang.java.ast.AbstractJavaNode.childrenAccept(AbstractJavaNode.java:56)
	at net.sourceforge.pmd.lang.java.ast.JavaParserVisitorAdapter.visit(JavaParserVisitorAdapter.java:10)
	at net.sourceforge.pmd.lang.java.ast.JavaParserVisitorAdapter.visit(JavaParserVisitorAdapter.java:158)
	at net.sourceforge.pmd.lang.java.typeresolution.ClassTypeResolver.visit(ClassTypeResolver.java:251)
	at net.sourceforge.pmd.lang.java.ast.ASTTypeDeclaration.jjtAccept(ASTTypeDeclaration.java:35)
	at net.sourceforge.pmd.lang.java.ast.AbstractJavaNode.childrenAccept(AbstractJavaNode.java:56)
	at net.sourceforge.pmd.lang.java.ast.JavaParserVisitorAdapter.visit(JavaParserVisitorAdapter.java:10)
	at net.sourceforge.pmd.lang.java.ast.JavaParserVisitorAdapter.visit(JavaParserVisitorAdapter.java:138)
	at net.sourceforge.pmd.lang.java.typeresolution.ClassTypeResolver.visit(ClassTypeResolver.java:222)
	at net.sourceforge.pmd.lang.java.ast.ASTCompilationUnit.jjtAccept(ASTCompilationUnit.java:41)
	at net.sourceforge.pmd.lang.java.typeresolution.TypeResolutionFacade.initializeWith(TypeResolutionFacade.java:18)
	at net.sourceforge.pmd.lang.java.AbstractJavaHandler$5.start(AbstractJavaHandler.java:97)
	at net.sourceforge.pmd.SourceCodeProcessor.usesTypeResolution(SourceCodeProcessor.java:149)
	at net.sourceforge.pmd.SourceCodeProcessor.processSource(SourceCodeProcessor.java:177)
	at net.sourceforge.pmd.SourceCodeProcessor.processSourceCode(SourceCodeProcessor.java:95)
	... 9 more

```

You shouldn't have told to look at the reports ;P


This information is not valid to the total reconstruction and rewriting of all functionality, see the individual file headers for (c) information


WTF? This function is one of the most important functions in fileUtils! Why you declare it as private (static) function? everyone that include fileUtils.h should see it! why you use forward declaration here? put this function in the header!


感谢！
喜欢三个动作条！


This code from line 62 to 76 should be extracted into separate method, eg `setAlertForExecutingTheTask`


Commented code


Empty lines


empty line


empty lines


@Sami32 This commit is from 2016 and has been merged a long time ago. Also, the NPE you're referencing above isn't generated by this code, but by PMD. As far as I know, the PMD plugin isn't even run during normal builds?

I think you should provide some context about what you're trying to do. It also clearly looks like a bug in PMD, not DMS. AFAICU no source code, no matter how crazy, should be able to make PMD crash with a NPE.


maybe extract this line to new method, eg. `ShowPermissionDeniedAlert`


commented code