Why don't use any scope defination (ex public, protected, private ) . Is there any reason ?
How about this ? ( Public SortTestOfAlgorithm)


It should be good to add the seperate the failure test case.

Ex. TEST_F((PriorityQueueTest,TestSize)


brace 는 for 문 { 를 사용하는 것을 권장합니다


p = shopping_list_.begin(); 가 사용되지 않고 있습니다.
불필요하다면 삭제하는 것이 좋을 것 같습니다.
하지만, 아래 for 문에서 i 대신 사용하면 코드가 간결해질 것 같습니다.


Plz add braket like below.
if (src.size() == 0 || src.length() == 1) {
    return src;
}


!isCharRepeated (src.charAt(index), src.charAt(index-1)) 이 더 이해하기 쉬울것 같습니다.


Best to avoid. If we add it here, then why not in other documents?


Comment는 위에서 사용하던 방식과 일괄되게 //를 사용하시는 것이 보기에 좋을 것 같고,
한라인에 character가 많은 부분은 라인을 바꾸어 주는 것이 좋을 것 같습니다.


다른 부분들과 동일하게 조건문내 공백은 삭제하는게 좋을 것 같습니다.


특정 값으로 정의된 상수는 변수로 정의하여 그 의미가 무엇인지 변수명에 반영이되면, 코드 가독성이 좋아질것 같아요


import와 class  선언사이에 빈라인이 있으면 보기가 편할 것 같습니다.


class의 이름은 Camel style에 따라 대문자로 시작하는게 좋겠습니다.
class templatePriorityQueue -> class TemplatePriorityQueue


brace 는 if 조건문 뒤에 { 를 사용하는 것을 권장합니다.


@fabiangreffrath, worth a suggestion?

![DOOM83](https://user-images.githubusercontent.com/21193394/59325493-ee9d4c00-8ceb-11e9-8a6b-98e753186ab1.png)


Goole coding style에 따라서 if에 해당하는 statement만 존재할 때는 하기와 같이 수정하는 것이 좋겠습니다.
if (count > 1) stringBuilder.append(count);


위의 예와 같이 test 진행 현황에 대한 comment 가 있으면 좋을 것 같습니다.
예) // pop 8, 5, 4 above


A space is necessary before '{'.


data가 nullable이면  annotation을 추가해야 할 것 같습니다.


Give an example usage (as if in UG) so that the reader can visualize the end goal?


함수 정의 후 { 는 라인을 바꾸지 않고 동일한 라인에 위치하면 보기가 편할 것 같습니다.


We are using this Code Style " https://google.github.io/styleguide/javaguide.html".
So Please use this formatting such as "ClassABC {"


data가 nullable이면 data null check가 필요합니다.


현재 작성된 Prince의 Sorting 코드도 Sorting이 잘 되는것으로 보입니다.
그 외에도 STL에서 제공하는 sort를 사용하여 아래와 같이 sorting을 하는것도 가능할 것 같습니다.
<예>

#include <algorithm>
void Shopping::comparePrice(const Goods g1, const Goods g2) {
	return g1.GetPrice > g2.GetPrice;
}

void Shopping::SortByPrice() {
	sort(shopping_list_.begin(), shopping_list_.end(), comparePrice);
}


가격이나 인덱스는 음수가 되면 안되므로 입력단계에서 음수를 체크하고 음수라면 오류를 반환하는것이 좋겠습니다.
또한, 인덱스가 중복될 경우 문제가 될 수 있으므로 중복여부 체크가 필요한지 여부를 전체 로직상 결정하여 반영하는것이 좋을듯 합니다.


Could you make comments about the reason why you want to check if pushQueue function work good?
I think that you can use another method to validata it.


Don't we need the initialized assignment for the var sucn as (int var=0)?


method명은 Camel style로 대문자로 시작하는 형태로 바꾸는 것이 통일성을 유지하는데 용이할 것으로 보입니다.
bool empty() const; -> bool Empty() const;


int 형에 대한 특수화만 구현되어 있고, 일반화된 type T에 대한 구현이 존재하지 않습니다.
아래처럼 일반화된 type T 에 대한 구현이 필요 합니다.
template<typename T> templatePriorityQueue<T>::templatePriorityQueue() {
}


IGN_RPM_COUNT too?


Google code style에 따라서 parameter의 줄바꿈을 할 때는 이전 parameter와 colume을 맞추도록 변경 해주세요.
하기와 같이 변경하면 됩니다.
private void appendLastCharIfNeeded(StringBuilder stringBuilder, int repeatedCharCounter,
                                                              char previousChar) {


This kind of tip needs to be outside of this doc as it could vary based on the module?


shopping_test.cc와 unit_test_main.cc에 중복으로 main()함수가 정의되어 있는데, 향후 한군데만 수정되어 부분 업데이트될 우려가 있기 때문에 중복을 제거해보는건 어떨까요? 의미상으론 unit_test_main.cc 에만 main() 함수 정의해서 사용해도 될것 같습니다.


brace 는 if 조건문은 { } 를 사용하는 것을 권장합니다.


A space is necessary before '{'.


We are using this Code Style " https://google.github.io/styleguide/javaguide.html".
Please use "{" at the previous end of line "


private: 과 public:의 구분을 위해 줄바꿈을 넣으면 더 좋을것 같습니다.


함수명/변수명 잘 적어 주셨습니다. 주석 없이 이것만 가지고도 잘 이해 되므로 주석 없어도 될 것 같아요.


큐에 저장된 엘리먼트가 한개도 없을때에 대한 예외처리가 추가되면 좋을것 같습니다.


Google coding style에 따라서 {를 if문과 같은 line으로 옮기는 것이 좋겠습니다.


pop한 갯수를 저장해놓고 정상적으로 처리했는지 확인하면 어떨까요?
loop를써도될것같구요,

total size와 pop한 갯수를 저장해놓고 비교하는게 필요할것같아요,


We are calling super in a few methods. E. g. in the CS:GO Player class.


Please consider code indent at here


155~157라인에 한칸씩 indentation이 필요해 보입니다.


Plz remove "Char" in variable name.


Please, follow the style guideline (Class Data Members).
Data members of classes, both static and non-static, are named like ordinary nonmember variables, but with a trailing underscore.


It should be good to match the name of class and methods.

Ex. PriorityQueue<int>::PriorityQueue()


member 변수명은 underscore로 끝나도록 변경하는게 좋을 것 같습니다.
std::priority_queue<T> storage; -> std::priority_queue<T> storage_;


이미 size가 3인것이 ASSERT로 확인하고있는 부분이 있어서 empty체크는 필요없지않을까 생각합니다.


stringBuilder.append()
if (count>1)
 stringBuilder.append()
를 묶어서 하나로 빼면 의미가 명확해질 것 같습니다.


list의 null 체크가 빠져있어서, Null Pointer Exception 발생 가능성이 있습니다.
if (list == null || list.getNext() == null) return list; 의 형태가 좋아 보입니다.


else 문에도 { }를 추가해 주시는게 좋습니다.


향후 테스트 확장성을 고려하여 개별 데이터 push 대신 loop를 통해 test pattern을 push 하는 방법이 좋지 않을까 제안드립니다.


해당클래스에서 함수 및 변수명을 아래와 같이 변경하는 방향에 대해서 검토 드립니다.
setNext 보다는 구체적으로 명시하면 가독성이 좋아질 것 같습니다.
data는 구체적으로 nodeDate, getData는 getNodeDate 등과 같이 명명하는 방향에 대해서 제안 드립니다.


It should be good to match the name of class and methods.

Ex. PriorityQueue<int>::~PriorityQueue()


Hi Eliot,
I will remove the translation to mac-roman, it's just that these dirty
corners speed my progression down.
If i understand correctly, either the legacy polling API is not
internationalization friendly, or maybe the wParam does contain someting
else than utf32/ucs4 code? Do we really need to support legacy polling API?
Sharing platform code forever with old VM is not sustainable, it
complexifies everything, IMO we have a license to cut useless code.

Le mer. 12 juin 2019 à 03:39, Eliot Miranda <notifications@github.com> a
écrit :

> Hi Nicolas,
>
> it's put code. Why not just fix it?
>
> On Tue, Jun 11, 2019 at 2:40 PM Nicolas Cellier <notifications@github.com>
> wrote:
>
> > I'm somehow amazed to not see a keymap[wParam & 0xff] buffer overrun
> > protection here!
> > This code is probably never executed since the image side presumably
> > provide a non NULL inputSemaphoreIndex, but it's scary.
> >
> > —
> > You are receiving this because you authored the thread.
> > Reply to this email directly, view it on GitHub
> > <
> https://github.com/OpenSmalltalk/opensmalltalk-vm/commit/a4a7841030a5e4736452282d6e71ffe624a76267?email_source=notifications&email_token=ADY5VUEFXNWVU5PSQB5VXCTP2ALUPA5CNFSM4HXDSVUKYY3PNVWWK3TUL52HS4DFVVBW63LNNF2EG33NNVSW45FKMNXW23LFNZ2F62LEZYBAKSFO#commitcomment-33900718
> >,
> > or mute the thread
> > <
> https://github.com/notifications/unsubscribe-auth/ADY5VUCPLK4B7UBYBTM4ZWLP2ALUPANCNFSM4HXDSVUA
> >
> > .
> >
>
>
> --
> _,,,^..^,,,_
> best, Eliot
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> <https://github.com/OpenSmalltalk/opensmalltalk-vm/commit/a4a7841030a5e4736452282d6e71ffe624a76267?email_source=notifications&email_token=AAFRYIQWUXJJ72IA2SUNXELP2BHU3A5CNFSM4HXDSVUKYY3PNVWWK3TUL52HS4DFVVBW63LNNF2EG33NNVSW45FKMNXW23LFNZ2F62LEZYBAKVEL#commitcomment-33903755>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AAFRYIW3TBYQSMUTZVDSQILP2BHU3ANCNFSM4HXDSVUA>
> .
>


This method can be merged to Constructor to prevent Default Value issue.


:white_check_mark: All Rules (2 / 2) Passed

      :crown: Good job! this pull request is aligned with your organization's best practices!

      :point_right: You can review your rules settings at smart policy management :point_left:

:point_right: click here to view a full list of the events :point_left:


요부분도 마찬가지로, 위에 comment드린거처럼 pop하는 갯수와 현재 남은 size를 체크해서 empty를 체크하면 좋을것같네요,


인덱스 중복을 허용하지 않을 경우 인덱스는 입력받지 않고 자동으로 증가되도록 하는 방법도 고려해 볼만해 보입니다.


Successfully aliased the URL <https://percival-frontend-d7z93se7v.now.sh> to the following aliases.

* https://percival.live
* https://percival-frontend.klauswuestefeld.now.sh
* https://percival-frontend-git-master.klauswuestefeld.now.sh


위 if문에서 return을 함으로 else 문 추가 안해도 될것 같습니다.


Before describing what to do, may be provide some background? e.g., _You'll notice that AddressBook has a separate `*Command` class to represent each command. Each such class contains ... etc._


Is this range right?
Could you make some more comments about the meaning of range?


class의 이름은 대문자로 시작하는 것이 가독성에 도움이 될것 같습니다.
아래 링크 참고 하십시오.
https://google.github.io/styleguide/cppguide.html#Type_Names


iterator 변수 p가 함수내에서 사용되지 않는 것 같습니다. 확인해보고 불필요하면 삭제 하는것을 검토해주세요.


173~176라인의 indentation을 한 칸씩 앞으로 이동해주세요.


179~182라인의 indentation을 한 칸씩 앞으로 이동해주세요.


What is the meanning of the "ch" . Is this "int character from_cli =0 "?


src가 외부에서 전달된 parameter이기 때문에 null check를 해주는 것이 좋겠습니다.
추가로 Goole coding style에 따라서 if에 해당하는 statement만 존재할 때는 하기와 같이 1줄로 수정하는 것이 좋겠습니다.
if (src != null && src.length () <= 1) return src;


for 구문의 경우 { }를 추가해 주세요.


priQueue의 초기화 코드라면, 변수 선언과 함께 초기화 하는 것을 권장합니다.

int custumID[] = {23, 598, 302};
templatePriorityQueue<int> priQueue(customID, customID+3);


let's think about call by referece rather thatn call by value considering that function works well as your intention.


불필요한 line을 줄이기 위해 다른 동작을 수행하지 않는 constructor와 destructor는 헤더파일에서 정의시 같이 정의하는게 좋을 것 같습니다.
public:
      templatePriorityQueue() {};
      ~templatePriorityQueue() {};


가독성을 위해 For명령어내 한줄로 표기도 괜찮을거 같습니다.


Instead of the "br" , How about this "buffer_reader"?


줄 바꿈을 넣어 80 컬럼에 맞춰주시면 감사하겠습니다.


It should be good to match the name of class and methods.

Ex. bool PriorityQueue<int>::Empty() const


주석문은 통일성을 위해서 //로 변경하면 좋을것 같습니다.


Method가 대문자로 시작하고 있습니다. 소문자로 시작하도록 변경이 필요해 보입니다.
BinarySort -> binarySort


Plz remove type in variable name.


현재 작성된 Index의 Sorting 코드도 Sorting이 잘 되는것으로 보입니다.
그 외에도 STL에서 제공하는 sort를 사용하여 아래와 같이 sorting을 하는것도 가능할 것 같습니다.
<예>
#include /<algorithm>
void Shopping::compareIndex(const Goods g1, const Goods g2) {
	return g1.GetIndex > g2.GetIndex;
}

void Shopping::SortByIndex() {
	sort(shopping_list_.begin(), shopping_list_.end(), compareIndex);
}


코드의 인덴테이션을 https://google.github.io/styleguide/javaguide.html  참조하여 수정하면 좋을 것 같습니다.


큐에 저장된 엘리먼트가 한개도 없을때에 대한 예외처리가 추가되면 좋을것 같습니다.


위 케이스와 같이 각 테스트별 예상되는 결과에 대한 comment 추가가 있으면 테스트의 의도를 이해하는데 도움이 될 것 같습니다.


SetGoodsInfo() 에서 index 값과 price 값을 전달받는데요.
항상 설정이 필요한 필드이므로 constructor 에서 설정하는 것이 나을 것 같습니다.

class Goods{
  private:
    final int index_;
    final int price_;
  public:
    Goods(int index, int price);
    int GetIndex();
    int GetPrice();
};


It should be good to match the name of class and methods.

Ex. bool PriorityQueue<int>::Size() const


동작에는 문제가 없으나 아래 Java coding guide brace 관련 내용 참고하셔서 여는 중괄호 앞에 줄바꿈이 없도록 일관성 유지해주시면 좋겠습니다.
https://google.github.io/styleguide/javaguide.html#s4.1.2-blocks-k-r-style

`while (flag)
{

while (flag) {
`


여러 소팅 알고리즘 중에 Selection Sort를 선택하신 이유가 있으신가요?
어플리케이션 요구사항에 최대 Goods 아이템의 개수가 어떻게 되나요?  비기능 요구사항으로 성능 항목이 있나요?
시간, 공간 복잡도를 고려하여 적합한 소팅 알고리즘을 고려해 보세요.
참고) https://hsp1116.tistory.com/33


template 라이브러리의 특성상, 선언과 구현의 분리 컴파일 및 링크가 안될수도 있습니다.
구현부를 별도 파일로 만들고 싶으시다면, priority_queue.hpp 같은 파일에 구현부를 넣고, priority_queue.h 파일의 끝에 #include "priority_queue.hpp" 와 같이 include 하는 것을 권장합니다.


Plz add the braket.
```
if (src.size() == 0 || src.length() == 1) {
    return src;
}
```


Instad of the "n" , How about this  "count_of_number"?


전체적으로 LOC 가 긴데요. 몇개의 method 로 분할하여 각 method 의 역할을 명확히 하는 것이 좋겠습니다.
저희팀에서는 보통 한 method 의 길이는 20 이 넘지 않는것을 권장합니다.


Similarly, explain that our approach here is to start by adding the simplest version of the command first.


templatePriorityQueue를 TemplatePriorityQueue로 변경하면 더 Class의 이름인지 더 명확히 알수 있을것 같습니다.


Plz add the braket like below.
```
if (count > 1) {
    stringBuilder.append(count);
}
```


Please, consider using const in size();


inline comment들 참조 부탁드립니다.


여기도 앞에 나왔던 isNullOrEmpty 를 구현해서 대체 가능할거 같네요.


1)What happen if we update the picklist value from Closed Won to closed Lost?


전체적으로 Block indentation 이 tab으로 분리되어 있습니다.
space 2를 권장합니다.
제안:
tab을 모두 space2로 변경필요합니다.
참고사항: https://google.github.io/styleguide/javaguide.html#s4.2-block-indentation


reversedList와 result가 중복으로 선언되어 있습니다.
ListNode result = reverseLinkedList.reverseIterative(listNode);
위와 같이 한줄로 변경하는것이 좋을것 같습니다.


불필요한 주석은 정리해 주시는게 좋을듯 합니다.


It should be good to match the name of class and methods.

Ex. const int* PriorityQueue<int>::Top() const


Plz remove useless code.


storage를 storage_로 이름을 변경하면 클래스의 멤버 변수인지 더 잘 알수 있을 것 같습니다.


Indentation이 너무 넓으면 한 line이 너무 길어지는 부작용이 있을수 있습니다.
아래 가이드 참고하여, fomatting 조정해 보시길 추천드립니다.
https://google.github.io/styleguide/cppguide.html#Class_Format


if (condition) {
...
}
위와 같은 형태가 회사의 룰이고, 일반적으로 JAVA언어에서의 중괄호 입력 방식 입니다. 참고 부탁드립니다.


한차례의 pop test가 더 진행된 것으로 보이니, 이에 대한 테스트 단계 및 expected 결과에 대한 comment가 추가되는 것이 좋을 것 같습니다.


추후 프로그램이 확장되어 Class Good을 상속한 상세 제품별 Class 생성을 대비하여
접근 지정자를 private 에서 protected로 변경하는 방안도 의견 드립니다.


Update the value 'Closed Win' to 'Closed Lost'


소팅 로직이 중복 사용되고 있습니다. qsort처럼 별도 함수로 만들고 비교 함수를 정의하는 방식으로 정리하고 이를 SortByPrice나 SortByIndex에서 호출하는 방식으로 바꾸면 추후 역순으로 정렬하거나 Goods의 데이터 필드가 늘어날 경우에도 쉽게 대응이 될것 같습니다.


Please consider the above code style as below .
ex) "for(int i=0;i<n;i++) 		{"


line end 에 특수문자가 있는거 같은데요. 제가 잘못 설정한 걸까요?


Wrong translation, and the `c-format` tag should not be removed.


#define guard 가 없습니다. guard 가 없으면, 헤더가 중복 인클루드 되는 경우 컴파일 문제가 발생하는 경우가 있습니다.
아래와 같은 guard를 추가하시기 바랍니다.

#ifndef _PRIORITY_QUEUE_H_
#define _PRIORITY_QUEUE_H_
//소스코드
#endif //_PRIORITY_QUEUE_H_


변수명이 너무 축약되어 pri의 의미가 무엇인지 잘 모를 것 같아요. priorityQueue와 같이 의미를 알 수 있도록 변수명을 정의하는게 좋을것 같습니다.


storage에 대한 제거 처리가 누락 된것으로 보입니다.
관련 부분 처리하는 것이 효과적인 메모리 관리를 위해 도움이 될것 같습니다.


In case where the length of string is needed, src.length() should be used.
size() is used when you need to know the length of collection type according to [this page](https://farcanada.tistory.com/entry/JAVA-length%EC%99%80-length-size-%EC%82%AC%EC%9A%A9%EB%B2%95).


Successfully aliased the URL <https://percival-frontend-58s7ph6xr.now.sh> to the following aliases.

* https://percival.live
* https://percival-frontend.klauswuestefeld.now.sh
* https://percival-frontend-git-master.klauswuestefeld.now.sh


It should be good to consider the empty case.

Ex. if (Empty()) return nullptr;


Please Consider the above code style link  as below
ex) switch(..) {


`c-format`


함수명/변수명 잘 적어 주셨습니다. 주석 없이 이것만 가지고도 잘 이해 되므로 주석 없어도 될 것 같아요.


If queue is not empty, how about make queue empty using LOOP.


Goods 내부의 index 값에 goods_ 라는 prefix 는 불필요해 보입니다.

```
final int index_;
final int price_;
```


1)What happen if we update the picklist value from Closed Won to closed Lost?


Plz add braket like below.
`if (index == (src.length() - 1))`


The auto formatter would have done that, it sorts headers alphabetically. We'd need to put the hdu import before the other casacore imports and then leave a blank line to prevent reordering


Successfully aliased the URL <https://nhentai-bot-pvo41kgcu.now.sh> to the following aliases.

* https://nhentai-bot.stanly61315.now.sh
* https://nhentai-bot-git-master.stanly61315.now.sh


Don't we need to consider default case ?


Update the value 'Closed Win' to 'Closed Lost'


{의 format 이 지켜지지 않았습니다. IDE 의 formatter를 사용하면 팀내 format 을 정형화할 수 있습니다.


함수명의 시작글자만 대문자로 유지되는게 좋을거 같습니다. 또한 함수명이 함수의 기능을 포함 할 수 있는것이 좋을 듯 합니다. 단지 Test의 경우 너무 제너럴한 듯 합니다.


함수 호출을 줄이기 위해서, i 와 min 이 같을 경우에는 swap 을 수행하지 않아도 될 것 같은데요.
if 조건문을 추가해도 동작에 문제가 없는지 검토해 주세요.


A lot of unexpected removed `c-format`.


앞에 빈줄을 추가해 주는게 가독성이 좋을 것 같습니다.


아래 문장은 do 보다 한번더 들여쓰기해야 합니다.
11줄에서 46까지 space 2만큼 들여쓰기해주세요~


class 이름에 굳이 template 이라는 prefix를 넣지 않아도 될 것 같습니다. PriorityQueue 라고 이름을 줄이셔도 좋을것 같습니다.


Please initialize the value such as temp =0;


It should be good to match the name of class and methods.

Ex. void PriorityQueue<int>::Pop()


가격이나 인덱스가 음수로 들어가는 예외상황에 대한 테스트가 이루어 지지 못하고 있습니다. 예외처리 추가와 이에 맞는 테스트케이스 추가가 필요해 보입니다.


변수 p 가 선언 이후 이 함수에서 사용되지 않습니다.
p 변수가 이 함수에서 사용될지를 확인해봐 주세요.


LGTM. 코드 이해하는데 도움 됩니다.


if (result == null) throw new IndexOutOfBoundsException(); 와 같이 한줄로 표현하시던지,
중괄호로 묶을 필요가 있어 보입니다.


i는 외부에서 parameter로 전달받은 값인데, 내부적으로 다시 assignment를 수행 했습니다.
이런 경우 i를 외부 parameter로 전달받을 필요가 없는 상황으로 판단됩니다.
하기와 같이 이 함의 parameter 조정이 필요한 상황으로 보여집니다. 다시 검토 확인 부탁 합니다.
String compressRecursiveInner(String src, StringBuilder sb, char previousChar, int charCounter)


1 tab 이 8 space 인것 같은데, 팀내 룰은 space 4를 사용하도록 하고 있습니다.


All code have to re-format. (Eclipse ctrl+shift+F)

And also change braket policy like below.

```
if (src.size() == 0 || src.length() == 1) {
    return src;
}
```


네 매개변수가 이상하기도 하고 분리했던 메소드가 재호출될 일이 없어서 따로 분리하지 않았습니다.


import와 class 사이는 1 line 띄어주는 것이 좋겠습니다.


수정했습니다.


Google coding style에 따라서 {를 if문과 같은 line으로 옮기는 것이 좋겠습니다.


Please use clean code for the next developer. So please remove unused line


It should be good to match the name of class and methods.

Ex. void PriorityQueue<int>::Push(const int& val)


length는 사용되지 않는 변수입니다.
삭제 부탁드립니다.


= 전후엔 space 를 사용해주세요.


예외 사항에 대한 핸들링 여부를 확인하기 위한 TC로 추가해 보세요.


for문의 ()는 가독성을 위하여 한줄로 유지되는 것이 좋을 듯 합니다.


Google coding style에 따라서 {를 else 문과 같은 line으로 옮기는 것이 좋겠습니다.


need a space after ','.


Google coding style에 따라서 {를 if 문과 같은 line으로 옮기는 것이 좋겠습니다.


clear 함수를 추가하는 것이 좋을 것 같습니다.
예) void clear();


전반적으로 formatting 이 되어있지 않습니다.
툴을 사용해서 일괄 적용하는 것을 추천드립니다.


33 line의
if( argc > 1 ){ ....
47 line의
if(priQueue.empty()){ ...

을 보면 if statement 표현의 일관성 이 없습니다.

if (statement){.. 와 같은 형태로 일관성을 맞춰주는 것이 좋을 것 같습니다.


Plz refactoring three parameter method.
Plz don't exceed two parameter for clean code.


Successfully aliased the URL <https://urbarium-admin-react-j4kex4n9o.now.sh> to the following aliases.

* https://urbarium-admin-react.urbarium.now.sh
* https://urbarium-admin-react-git-master.urbarium.now.sh


return (currentChar == previousChar); 와 같이 하는 것이 더 좋을 것 같습니다.


SetGoodsInfo, GetIndex, GetPrice에 대해서 클래스 선언부에서 inline funtion을 처리 해도 될 수도 있을거 같습니다.
https://google.github.io/styleguide/cppguide.html#Inline_Functions


default에 대한 처리도 필요합니다.


Don't wast the memory for the value in for loop

ex)
Before ->
 for (...) { int x=0;
After ->
int x=0; for (...) {


Class의 method 별로 TEST를 분리하는 것을 검토해주세요. Unit Test 수행 시 function 단위로 하는 것을 권장하고 있습니다. (코드리뷰 심화 p93)


리뷰 내용 추가.


if (condition) {
...
}
위와 같은 형태가 회사의 룰이고, 일반적으로 JAVA언어에서의 중괄호 입력 방식 입니다. 참고 부탁드립니다.


필요한 import문인지 확인 부탁드립니다.
혹, 사용하지 않는 다면 제거 부탁드립니다.


修复缺陷时，需要一同补充单元测试，测试这个缺陷场景。


불필요한 comment 로 보입니다


Is it good to have one PEDAL_TO_TPS_SIZE for both PedalBins and RpmBins?


Plz remove duplicated intialize.
`boolean thereIsNoMoreWordToCompress;`


1라인의 수가 100을 넘지 않도록 해주세요.
이렇게 변경되지않는 텍스트들은 class 에 private static final String TEXT = "xxx" 와 같이 선언해서 사용하는 것을 권장합니다.
혹시 이 텍스트들이 다국어를 지원해야 한다면, xml 등으로 빼서, 언어에 따라 불러 쓰는 방법도 있겠습니다.


Please don't declare the vars in loop condition


comment 표시하실때,
하나의 파일 안에서 일관성을 같도록
// 또는 /* */ 중 하나의 형태로 작성하시는 것이 좋을 듯 합니다.


Successfully aliased the URL <https://nhentai-bot-d3kny5fir.now.sh> to the following aliases.

* https://nhentai-bot.stanly61315.now.sh
* https://nhentai-bot-git-master.stanly61315.now.sh


Plz add braket like below.
```
if (src.length() <= 1) {
    return src;
}
```


Local 변수의 경우, 사용하는 시점에 선언 하는 방향으로 수정이 필요해 보입니다.


0보다 큰 조건문으로 변경하고 return을 없애도 될것 같습니다.


p = shopping_list_.begin(); 가 사용되지 않고 있습니다.
불필요하다면 삭제하는 것이 좋을 것 같습니다.
하지만, 아래 for 문에서 i 대신 사용하면 코드가 간결해질 것 같습니다.


동작에는 문제가 없으나 아래 Java coding guide method naming 관련 내용 참고하셔서 여는 중괄호 앞에 줄바꿈이 없도록 일관성 유지해주시면 좋겠습니다.
https://google.github.io/styleguide/javaguide.html#s5.2.3-method-names

예)
`public void splitSorting(int array[]) { // lowerCamelCase`


함수사이에 한줄또는 두줄로 띄워주세요.
함수 구분이 필요합니다.


if 와 return 사에서 {} 가 없어서 차후에 return 이 오용될 위험이 있습니다.
if (char == 6) {
 return;
}
과 같이 {} 를 추가해주세요


// 로 시작하는 커멘트들이 불필요해 보입니다


Please use the above code style, So we have to use carmel style code formatting.


= 전후에 space 가 들어가야 합니다.


const를 추가하여 define하는게 좋을것 같습니다.
int size(); -> int size() const;


모든 함수사이에 한줄또는 두줄로 띄워주세요.
함수 구분이 필요합니다


Please, add some exceptional code for the case of empty queue.


length는 사용되지 않는 변수입니다.
삭제 부탁드립니다.


Please use "{" for the readability at the for statement


On second thoughts, I feel it is better to move the testing to the end of the tutorial. It takes too much attention away from the main objective of the tutorial.


if (condition) {
...
}
위와 같은 형태가 회사의 룰이고, 일반적으로 JAVA언어에서의 중괄호 입력 방식 입니다. 참고 부탁드립니다.


main(String args[]) 다음 공백을 두고 main(String args[]) throws IOException로 표시하면 좋을 것 같습니다.


br.readLine() 이 꼭 숫자가 아닐 수도 있는데, 그때 Exception 이 발생할 수 있겠네요.
이에 대한 처리를 추가해주세요


a file which can be used to deploy a vm from template via vcenter by ansible


inter_swap 을 사용하기 위해서, #include <algorithm>  문이 포함되어야 하는게 아닌지 확인이 필요로 해보입니다.
http://www.cplusplus.com/reference/algorithm/iter_swap/


Please add description for the functions.


Please revert this one.

At least RHEL / CentOS 7 don't have libpq.pc (postgresql 9.2)


in rails 5 remotipart/rails/rails/engine.rb the file location is not avaliable . plese tell another way to fix in rails 5 plz help


Shouldn't the reader know how to build/run already?


"thereIsNoMoreWordToCompress = (i == src.length())" seems more readable.


전체 코드에 대한 comment 입니다. Tab 사용 없이 2 space 간격으로 style 정리 부탁드립니다.


name 충돌을 예방하기 위해서, namespace 사용을 권장드립니다.


It could be fine to divide into 3 functions in main. (In my thinking)
  - check_empty
  - pushQueueTestData()
  - VerifypushQueue()
  - popQueueTestData()


default case 에 대한 처리가 없습니다.


And sorry for being late on checking all the recent build changes


Google coding style에 따라서 {를 else 문과 같은 line으로 옮기는 것이 좋겠습니다.


Successfully aliased the URL <https://percival-frontend-3f10zul2y.now.sh> to the following aliases.

* https://percival.live
* https://percival-frontend.klauswuestefeld.now.sh
* https://percival-frontend-git-master.klauswuestefeld.now.sh


constructor initializer로 초기화를 해주는것이 좋을것 같습니다.


class 내부 변수에 영향을 주지 않는 함수이므로 const를 추가하여 define하는게 좋을것 같습니다.
int templatePriorityQueue<int>::size() { -> int templatePriorityQueue<int>::size() const {


Please use the above code Style for the others such as "QuickSort" -> "quickSort"


Plz refactoring five parameter method.
Plz don't exceed three parameter for clean code.


Resize images so that the text size in the image matches that of the main body of the document.


Please use code indent at the whole lines


Google coding style에 따라서 {를 if 문과 같은 line으로 옮기는 것이 좋겠습니다.


한 space앞으로 indentation 맞추면 좋을 것 같아요.


Deployment failed with the following error:

~~~
You defined 1 build that did not match any source files (please ensure they are NOT defined in .nowignore):
~~~


6인 경우 종료하도록 되어있는 거 같은데, 앞의 switch 에서 처리하는게 좋지 않을까요?


I think AB3 is their first exposure to Gradle.


imo, do we need to add null check exception handling?


Please delete dead code on souce code


Fail Example은 임시로 만들었고 더 이상 사용하지 않을 것 같습니다. 소스에서 삭제하겠습니다.


</a><p>


> Any particular reason to move to 1.1?
>
> Are there lots of minor features that would be in the next release?

Eager initialization of annotated metrics plus new abstractions needed for Quarkus so that metrics can be registered at build time. So it affects both application developers and the integration code to Thorntail/WildFly/Quarkus.


Successfully aliased the URL <https://nhentai-bot-njl60mweu.now.sh> to the following aliases.

* https://nhentai-bot.stanly61315.now.sh
* https://nhentai-bot-git-master.stanly61315.now.sh


Please use code indent


; 전후로 space 가 있는게 읽기에 좋겠습니다.


> comment 표시하실때,
> 하나의 파일 안에서 일관성을 같도록
> // 또는 /* */ 중 하나의 형태로 작성하시는 것이 좋을 듯 합니다.

Fail Example은 임시로 만들었고 더 이상 사용하지 않을 것 같습니다. 소스에서 삭제하겠습니다.


Google C++ Style Guide 의 "Names and order of Includes" 에 의하면(https://google.github.io/styleguide/cppguide.html#Names_and_Order_of_Includes) Use standard order for readability and to avoid hidden dependencies: Related header, C library, C++ library, other libraries' .h, your project's .h. 라고 가이드 되어 있습니다.
"./shopping.h" 가 먼저 나오고 한 칸 띄운 뒤에 "<iostream>" 순서가 되어야 Readibility를 높이고, Hidden dependency를 제거할 수 있을것 같습니다.


가독성을 위해 For명령어내 한줄로 표기도 괜찮을거 같습니다.


구현을 보면 template을 사용하였으나 실제로는 int형에 대해서만 구현이 되어 있습니다.
혹시, 이후 다른 type에서 사용할 일이 없다면, template을 제외하고 int형으로 고정하는 것은 어떠실지요?
template는 자유도가 높긴 하나, 가독성을 떨어뜨리고, 간혹 그 behavior를 예측하기 어려운 경우가 있습니다.
사용시 신중할 필요가 있습니다.
https://google.github.io/styleguide/cppguide.html#Template_metaprogramming


K & R style braces are recommended here and thoughout this file by the [ Google Java Style Guide.](https://google.github.io/styleguide/javaguide.html#s4.1.2-blocks-k-r-style)


Google C++ Style Guide 의 "Names and order of Includes" 에 의하면(https://google.github.io/styleguide/cppguide.html#Names_and_Order_of_Includes) Use standard order for readability and to avoid hidden dependencies: Related header, C library, C++ library, other libraries' .h, your project's .h. 라고 가이드 되어 있습니다.
"./shopping.h" 가 먼저 나오고 한 칸 띄운 뒤에 "" 순서가 되어야 Readibility를 높이고, Hidden dependency를 제거할 수 있을것 같습니다.


Please consider the package name such as "com.samsung.codereview.xxx"


메뉴를 display를 하는 부분입니다.
displayMenu로 extract method했으면 좋겠습니다.


imin 보다는 의미를 알수 있도록 min 이라든지 iMin 이라든지 하는게 어떨까요?


string을 입력받는 부분입니다.
inputText로 extract method했으면 좋겠습니다.


Google coding style에 따라서 {를 함수 definition line으로 옮기는 것이 좋겠습니다.


Do you have any reason for using size() at the first condition ?

Please use single function as below .
" if (src.size() == 0 || src.length() == 1)" ->  if (src.length() == 0 || src.length() == 1)


1줄짜리 execution 이라도 {} 가 있는게 좋을거 같습니다.


Please check project 1 issues.


All code have to re-format. (Eclipse ctrl+shift+F)

And also change braket policy like below.

```
if (src.size() == 0 || src.length() == 1) {
    return src;
}
```


why don't we change the value name "list" as more detail and straight one?


Sort를 수행하기 위한 소스 코드가 반복적으로 구현되어 있습니다. 반복적인 소스코드는 함수로 만들어서 관리 되면 좋을 듯 합니다.


파라메터들의 , 이후에도 space 를 넣고 구분하는 것이 읽기가 좋을거 같습니다.


리뷰 남겼습니다. 확인 부탁드리겠습니다.


Google coding style에 따라서 하기와 같이 수정하는 것이 좋겠습니다.
if (ch==6) return;


선택된 메뉴를 실행하는 부분입니다.

sort 함수로 extract method했으면 좋겠습니다.


해당 변수가 선언만 되어 있고 사용되지 않는것으로 보입니다.


Please declar the vars at the out of the for loop statement


Google C++ Style Guide 의 "Names and order of Includes" 에 의하면(https://google.github.io/styleguide/cppguide.html#Names_and_Order_of_Includes) Use standard order for readability and to avoid hidden dependencies: Related header, C library, C++ library, other libraries' .h, your project's .h. 라고 가이드 되어 있습니다.
"./goods.h" 가 먼저 나오고 한 칸 띄운 뒤에 "" 순서가 되어야 Readibility를 높이고, Hidden dependency를 제거할 수 있을것 같습니다.

#include "./goods.h"

#include <vector>


Google coding style에 따라서 {를 for 문과 같은 line으로 옮기는 것이 좋겠습니다.


can we change the value name such as headNode, currentNode? what do you think?


Seems like a test code. It'd be better to be removed in the final code.


If execution time is more higher priority, it can use 'multimap' in order to sort element directly whenever push element. Then, SortByPrice() and SortByIndex() will just select which one is 'key' and 'value' between 'Price' and 'Index'.


if (n<=1) {
 return;
}
처럼 {} 를 넣는게 좋을거 같습니다.


동작에는 문제가 없으나 아래 Java coding guide brace 관련 내용 참고하셔서 여는 중괄호 앞에 줄바꿈이 없도록 일관성 유지해주시면 좋겠습니다.
https://google.github.io/styleguide/javaguide.html#s4.1.2-blocks-k-r-style

예)
`for (int j = 0; j < array.length - 1; j++) { `


함수이름은 camel 표기법으로.. binarySort 로 변경하는게 좋겠습니다.


Google coding style에 따라서 {를 switch 문과 같은 line으로 옮기는 것이 좋겠습니다.


Oh have access?
Ok, it looked like request was necessary for me ...

Libc header @ sh4
I've been building for a long time with the 4 header and that works fine.
The 5er header runs well with the necessary patch in the two spark boxes perfectly.
I can not say that about the other box.
I do not know yet if I can do the construction with glibc2.29, but it's my turn. Local adapted quite a bit and especially then changed the gcc.
With glibc2.27 I also had an executable image, but there was not all right.
Well possible that gcc4.94 and glibc2.25 is already the best combination.
Unfortunately again night service, so little time.


int val=a[i];int hole=i; 는  처음 for문 위로 위치를 함수 최초 실행시 한번만 메모리에 할당되기 때문에 메모리를 덜 사용합니다.


Please use packge for this class such as "com.samsung.codereview.xxx"


함수이름은 camel 표기법으로.. selectionSort로 변경하는게 좋겠습니다.


함수 이름은 대문자로 시작하는 것이 일반적인 코딩스타일입니다. 참고 바랍니다.
https://google.github.io/styleguide/cppguide.html#Function_Names


동작에는 문제가 없으나 아래 Java coding guide brace 관련 내용 참고하셔서 여는 중괄호 앞에 줄바꿈이 없도록 일관성 유지해주시면 좋겠습니다.
https://google.github.io/styleguide/javaguide.html#s4.1.2-blocks-k-r-style

예)
`동작에는 문제가 없으나 아래 Java coding guide brace 관련 내용 참고하셔서 여는 중괄호 앞에 줄바꿈이 없도록 일관성 유지해주시면 좋겠습니다.
https://google.github.io/styleguide/javaguide.html#s4.1.2-blocks-k-r-style

예)
`if (array[j] > array[j + 1]) { `


nL, nR 보다는 leftNumber, rightNumber가 나을거 같은데요. 아님 다른 이름이라도 좀더 명확한게 있음 좋겠습니다.


Please remove unused code or comments


주석처리보다는 함수명을 명확하게 바꾸는것이 좋을것 같습니다. shopping 클래스 전체적으로 정리하면 좋을것 같습니다.


< , && 등 사용시에는 전후에 space 를 넣어주세요.


All code have to re-format. (e.g Eclipse Ctrl+Shift+F)
Plz change braket policy.(A->B)
```
A
if(ch==6)
{
    return;
}

```
```
B
if(ch==6) {
    return;
}
```


줄바꿈으로 인해 가독성이 떨어집니다. 줄바꿈을 하지 않거나, p를 사용하거나 하는 것은 어떨까요?


함수이름은 camel 표기법으로.. insertionSort로 변경하는게 좋겠습니다.

각 인자는 의미를 알수 있는 단어/구로 표기했으면 합니다.
ex) a => elements, n=>count


함수와 함수사이엔 line 을 넣어서 구분하기 좋게 해주면 좋겠습니다.


Plz add braket.
```
if(ch==6) {
    return;
}
```


ch를 의미 있는 이름의 변수로 수정하는 것이 readability를 높일 수 있습니다.
ch를 command로 변경하면 좋겠습니다.


{ 후에 indent 가 빠졌습니다.


함수이름은 camel 표기법으로.. mergeSort로 변경하는게 좋겠습니다.

각 인자는 의미를 알수 있는 단어/구로 표기했으면 합니다.
ex) a => elements, n=>coun


입력 받은 값이 범위 이외 값이면 처리시 명확히 출력해주면 좋겠다.
ex) cout << "Input is range over  " << endl;


Google coding style에 따라서 {를 for 문과 같은 line으로 옮기는 것이 좋겠습니다.


함수이름은 camel 표기법으로.. quickSort로 변경하는게 좋겠습니다.

각 인자는 의미를 알수 있는 단어/구로 표기했으면 합니다.
ex) a => elements, n=>coun


주석 자체가 큰 의미가 없어 함수 이름을 명료하게 다듬어 적요하는것이 좋아 보이고 주석을 유지한다면 오탈자 수정이 필요합니다.


#define Guard를 추가하는 것이 좋을 것 같습니다.
#ifndef PRIORITY_QUEUE_H_
#define PRIORITY_QUEUE_H_
...
#endif // PRIORITY_QUEUE_H_


일단, shopping 대신 goods 라야 합니다.
`// Shopping class presents List of goods.`

그리고 naming 을 고민해봐야 할 것 같습니다.
이 클래스가 가진 기능은 Goods 를 리스트에 넣는 기능뿐인데 Shopping 이라는 클래스명이 맞는 것일까요.
Shopping 이라는 이름이 의미가 있으려면 동일한 Goods 여러 개를 넣는 기능이나 가격합계 기능이 있어야 할 것 같습니다.

현재의 구현은 구비한 상품목록에 불과한 것이므로 GoodsList 라는 클래스명을 사용하는 것을 제안합니다.


Plz change variable name.
bufferedReader


예외 처리라는 것을 명확히 출력 해주면 좋겠다.


public static void print(int a[],int n) 에서 인자사이에 한칸 띄워주시면 좋겠네요.
ex) public static void print(int a[], int n)


pIndex 보다는 index 란 이름이 어떨까요? c에서 보던 이름같아서


Please consider package name at here "com.samsung.codereview.xxx"


let's find the value name rather than  "o", such as ListNodeObject


코드의 가독성 측면에서 reversedList를 result로 변경하기 보다는 그대로 사용하는 것을 검토 부탁드립니다.


for(int i=0;i<n;i++) 에서 for와 "(" 사이, 및 다른 인자사이에 한칸 띄워주시면 좋겠네요.
ex) for (int i=0; i<n; i++)


Hi, did you follow the instructions in the readme file (https://github.com/zdealveindy/twinspanR/blob/master/README.md)? You need to first install another package, the riojaExtra from GitHub, and then to install twinspanR library. If not working, please give me more details (R version, error message etc), otherwise I am not able to help you. Cheers, David


header 파일의 경우 #define 가드를 통해 중복 inclulde를 방지할 필요가 있습니다. 참고 하세요.
https://google.github.io/styleguide/cppguide.html#The__define_Guard


xixxxxxxxxxir


특정 범위 값외에는 -1을 push하고 있는데 의미를 알수 있는 매크로가 적당할거 같습니다.
ex) #define CUSTOMNUMER_NUM_OVER -1


Is it right sequence of private and public declaration?
  google style guide
      " The public section should be first, followed by the protected and finally the private section. "


while(i<nL&&j<nR) 에서 while와 "(" 사이, 및 다른 인자사이에 한칸 띄워주시면 좋겠네요.
ex) while (i<nL && j<nR)


코드의 가독성 측면에서 listNode를 result로 변경하기 보다는 그대로 사용하는 것을 검토 부탁드립니다.


리뷰 해주신대로 수정하였습니다.


can we find a good name rather than T, sometimes it is difficult to read the code, imo.


Plz change variable name for readability.


Please consider name of package such as "com.samsung.codereview.xxxx"


There is no default: statemen in switch-case.


n, a 대신. 의미있는 단어/구로 표기했으면 합니다.
ex) n = > inputCount , a=>inputNumbers


Plz remove useless code.


네. 반영하도록 하겠습니다.


Please use the abovce code style (googleStyle) for this.
ex)
  @Override
  public boolean equals(Object o) {


do we need to initialized the value k, what do you think?


if 문장은  { } 으로 해주면 좋겠네요.


for 문에서 종료조건 확인해 주세요.


local-repo


사용하지 않는 변수가 선언되었습니다.


Please use code formatter for the position of annotations


goods를 하나만 정의하고 Set과 Push를 반복적으로 수행하여 중복코드를 줄일 수 있을 듯 합니다.
더 많은 수의 goods를 Set과 Push하는 경우에도 좋구요


удалены лишние библиотеки


brace 는 do keyward 다음에 { 를 사용하는 것을 권장합니다.
for(int i=0;i<n;i++) 는 인자사이에 한칸 띄우면 좋겠습니다.
ex) for (int i=0; i<n; i++)
참조내용: https://google.github.io/styleguide/javaguide.html#s4.1.2-blocks-k-r-style


코드 전체에서 std::vector<Goods>::iterator 가 많이 사용되고 있습니다. 타입정의를 해서 간결하게 표현하면 좋을것 같습니다.
typedef std::vector<Goods> goods_vector;
typedef goods_vector::iterator goods_vector_itr;


Please consider using packagename such as "com.samsung.codereview.xxx"


it is better name resultNode rather than result, what do you think?


It seems difficult to distinguish FindX functions with these naming (Find, Find2, Find3...). It'd be better to name those by what it does or how it's different from others.


Google styleguide에 따르면 A class definition should usually start with a public: section, followed by protected:, then private: 정의되어 있습니다.

이에 따라 private: public: 선언 순서를 변경하는 것은 어떨지요?

* Samsung style guide가 있으면 좋겠네요.. ^^


a[i]=Integer.parseInt(br.readLine()); 는 = 사이에 한칸 띄우면 좋겠네요.
a[i] = Integer.parseInt(br.readLine());


반드시 필요한 주석인지 재 확인 부탁 드립니다.


Google coding style에 따라서 {를 method defition과 같은 line으로 옮기는 것이 좋겠습니다.


Please delete unused code or comment


pop할때 현재 데이터 값을 return하면 좋지않을까요?
const int & templatePriorityQueue<int>::pop(){
       return  storage.pop();


```
class GoodsList {
  private:
    std::vector<Goods> list_;
  public:
    void Add(Goods object);
    void Remove(Goods object);
    void Print();
    void SortByPrice();
    void SortByIndex();
};
```


Google coding style에 따라서 {를 for문과 같은 line으로 옮기는 것이 좋겠습니다.


여는 괄호는 동일 라인에 작성하시는 것을 검토 부탁드립니다.


주석되신,, extract method했으면 합니다. swap 함수로.


Plz add braket.
```
if (n<=1) {
    return;
}
```


flag 지역 변수가 중요한 역할을 하는 것 같은데 flag 라는 변수 이름이 모호한 부분이 있습니다.
Readability를 높일 수 있도록 Naming 고려를 부탁드립니다.


코드 가독성이 떨어져 보이긴 하나 () 추가로도 충분할것 같아서 이렇게 수정하였습니다.


(i == src.length()) 와 같이 ( )괄호를 추가하면 좋을 것 같습니다.


pIndex대신 pivot 처럼 의미있는 단어/구로 표기했으면 합니다.


Google coding style에 따라서 {를 if문과 같은 line으로 옮기는 것이 좋겠습니다.


Plz change parameter name for readability.


I spent a long time with this part of the paper tonight and actually I found a flaw in the bound.  Luckily, it ends up giving us an easier-to-implement (but looser) bound.  Here's my work---it's probably a good idea to try to reproduce it to make sure I haven't gone wrong somewhere.

Our goal is to ensure that the following condition is satisfied by our choice of `m`:

```
z_{β / 2} σ^S / √(m) <= ϵ Φ(q, R) / |R|
```

And our goal is to find the value of m that will, with high probability, cause this inequality to be satisfied.  But we do not have access to the true kernel density estimate `Φ(q, R)` (although you can compute it in the code, and I did while debugging); instead, we can approximate this one of two ways:

  (1) with probability `1 - β`, `Φ(q, R) <= |R| (µ_S - z_{β / 2} σ^S / √(m))`---if the inequality above is satisfied.
  (2) with probability `1 - β`, `Φ(q, R) <= Φ^l(q, R) + (|R| - m) (µ_S - z_{β / 2} σ^S / √(m))`---again this holds if the inequality above is satisfied.

Note this sheds some light on what `Φ^l(q, R)` has to be.  In the paper it's not really clear how it's described, other than the writing `the currently running lower bound on the sum computed using exact methods`.  But the derivation above means that `Φ^l(q, R)` must correspond in the code to just the quantity `arma::accu(sample)`.

So, now that we have a lower bound for `Φ(q, R)`, we can go from the inequality above to Equation (2) in the paper.  However Equation (2) uses an *incorrect* bounding of `Φ(q, R)`: the quantity `µ_S - z_{β / 2} σ^S / √(m)` is the *lower bound* of what we expect the kernel contribution of any reference point to be.  (And, based on our bounds from earlier, a reference point will have kernel contribution *less* than that bound with probability `(1 - β / 2)`.  Although, that applies only if m is chosen such that that inequality is true.)  Thus, if we take `Φ^l(q, R) + |R| µ_S - z_{β / 2} σ^S / √(m)` where `Φ^l(q, R)` is just the sum of the true kernel contributions of `m` points, then we are *double-counting* `m` points.  So we have to instead take `Φ^l(q, R) + (|R| - m) (µ_S - z_{β / 2} σ^S / √(m))` as a bound!  This is an error in the paper, and causes the equation that establishes the bound on `m` to be wrong (unfortunately that equation is not numbered).

So, we could work out that unnumbered equation again simply by starting from the bounded inequality

```
z_{β / 2} σ^S / √(m) <= ϵ (Φ^l(q, R) + (|R| - m) (µ_S - z_{β / 2} σ^S / √(m))) / |R|
```

and solving for m.  However, in the derivation that is done in the paper, there is another error---the upper bound `(µ_S - z_{β / 2} σ^S / √(m)) <= µ_S` is used.  (If you take a look at the result, you can see how this was applied.)  But this is invalid, since we used a lower bound on the right-hand-side `Φ(q, R)` in the first step.  If we now apply an upper bound after a lower bound, the original inequality may no longer hold.

I found it easier to use bound (1) on `Φ(q, R)` instead of bound (2).  Doing that, I rederived the value of `m` and got the following:

```
m >= z_{β / 2}^2 (σ^S)^2 (1 + ϵ)^2 / (ϵ µ_S)^2
```

but you should also rederive it and see if I didn't mess anything up. :)  I also tried deriving it using bound (2) but the algebra became really irritating and it's a little bit late here. :)  Maybe you can do it better than I did.


Google coding style에 따라서 {를 method definition과 같은 line으로 옮기는 것이 좋겠습니다.


if 다음에 한칸 띄우면 좋겠습니다


Sorting이 오름차 순인지, 내림차 순인지 함수 명에 나타나는 것이 향후 혼선을 줄일 것 같습니다.


it can't work still. i think the command of stop have bug ?


Deployment failed with the following error:

~~~
You tried to create a Now 1.0 deployment. Please use Now 2.0 instead: https://zeit.co/upgrade
~~~


Is it right 'push(const int& val)'?


if (min == shopping_list_.end()) { 조건문은 j == i 일 때 항상 참인 조건문인데요.
처음 시작의 min 값은 i 와 같은 값이 되니, 조건문을 제거하고 for 문 전에 min 값을 i 값으로 할당하는게 어떨까요?
아래 처럼 수정이 될 것 같아요. 검토해봐 주세요.

std::vector<Goods>::iterator min = shopping_list_.begin();

    min = i;   // 추가
    for (std::vector<Goods>::iterator j = i+1; j != shopping_list_.end(); j++) { // j = i ==> j = i+1

    min = shopping_list_.end();  // 삭제


문법을 잘모르지만 해당 함수는 true, false 단순 리턴인데 const를 붙이는게 맞는지 확인 부탁드립니다.


변수 선언부와 코드 부분 사이에 한줄 띄우는 것이 읽기가 좋을 것 같아 의견 드립니다.


pointer2 seems not a good name for this variable


Goods 인스턴스가 생성될때 아래와 같이 초기화가 되고 있는데요. validation check가 필요할까요?
  int goods_index_ = 0;
  int goods_price_ = 0;


함수명/변수명 잘 적어 주셨습니다. 주석 없이 이것만 가지고도 잘 이해 되므로 주석 없어도 될 것 같아요.


Yes that was my first choice -> see sh4-bbappend.
Since I was not sure because of patch, I just wanted to upload the appropriate security.
Maybe the patches need to be changed a bit in the libc header kernel I think that's not the same version.
The bbappend can then be joined together.


문법을 잘모르지만 해당 함수는 상수값 리턴인데 const가 누락 되었는지 확인 부탁드립니다.


Google coding style에 따라서 하기와 같이 변경하는 것이 좋겠습니다.
if(a[j]<a[imin])	imin=j;


Plz change variable name

```
int leftLength = left.length;
int rightLength = right.length;
```


I found a bug here---when we take `mThresh` as a `size_t`, this is equivalent to taking the floor of the floating point result.  If you do

```
size_t mThresh = std::ceil(std::pow(z * stddev * (numerator / denominator), 2.0));
```

instead, this should fix it.  The current code can predict a too-small value of `m` as a result. :)


style guide에서
@override
public boolean ... 으로 변경하는 것을 제안드립니다.


![typo1](https://user-images.githubusercontent.com/50777481/59327005-43a88480-8d23-11e9-8233-5210e9b576f3.png)

typo1 ---> Image


제안해 주신 내용 반영토록 하겠습니다.


> #define Guard를 사용하여 헤더 파일이 중복되어 컴파일되지 않도록 하는게 좋을듯 합니다.
>
> #ifndef FOO_BAR_BAZ_H_
> #define FOO_BAR_BAZ_H_
>
> ...
>
> #endif // FOO_BAR_BAZ_H_

네. 반영하겠습니다.


![typo2](https://user-images.githubusercontent.com/50777481/59327033-515e0a00-8d23-11e9-9b2f-a81122fddd22.JPG)

typo2 ---> Image


Google coding style에 따라서 indentation을 space 1로 변경 해주세요.


Plz reformat this code like below.
```
for (int i = 0; i < n; i++) {
    System.out.print(a[i]+"\t");
}
```


수정해서 다시 commit 올리겠습니다.


 std::을 일일히 붙이면 가독성이 많이 떨어지니. using namespace std를 사용하시는 것을 검토해주셨으면 좋겠습니다.
출처: https://koosaga.com/9


동일 sort 가 아래 SortByIndex 에서 나옵니다. 함수로 빼는 것은 어떨까요?
그리고 현재 Sort 방식에서 for 문의 boundary 값 확인해 주세요.


We should use a fixed version to avoid surprises.
https://nvie.com/posts/pin-your-packages/


I have accepted your good comments and will modify it.


중복된 코드가 들어갔네요. 삭제하도록 하겠습니다.


필요없어 보여서 현상태유지 하겠습니다.


Google coding style에 따라서 하기와 같이 변경하는 것이 좋겠습니다.
if(n<=1) return;


Are you sure `GetKernelBandwidth()` should be used here and not `1.0`?  The `z` scores should be for a unit-variance Gaussian as far as I know.  Correct me if I'm wrong.


주석 없이 함수명/변수명 만으로 잘 이해가는 것이 많습니다. 이런것들은 주석 없어도 될 것 같아요.


필수는 아니지만 statement가 return; 한줄이어도 중괄호를 넣어서 해당 if가 어디까지 영향을 주는지 잘 나타났으면 합니다.
나중에 추가 statement를 넣을 필요가 있을 때에도 혼선을 줄일 수 있겠습니다.


전체적으로  Kernighan and Ritchie style 적용하도록 수정하였습니다.


No, thanks. This is only static information which never changes. We have the SKILL cheat in Crispy for the case you forgot which skill level you are on. :grin:


요청하신대로 null check와 함께 수정하였습니다.


Comment에 대해서 전체 검토를 해본결과, pop() & push()  Value에 대한 +- 동작으로 Return에 대한 사용하는 부분은 없습니다.


Already done, and no problem ... we should have waited for you, won't make that mistake again, nor will anyone else ...


수정하였습니다.


1줄로 수정하였습니다.


Так уже не пишут, принято ``return p;``


Right?


templatePriorityQueue<int> expTarget {1,4,2,3,8,5};  이렇게 수행을 이야기 하시는거죠?


main.cc외 다른 파일들도 동일하게 수정하겠습니다.


Please declar final var for the constant stuffs such as

protected static int MAX_COUNT_OF_INTEGER ;


= 연산자와 >= 연산자의 우선순위에 혼선이 있을 수 있으니 괄호를 명시적으로 넣으시면 좋겠습니다.
예)
`boolean shouldChangeLeft = (array[left] >= 0;)
boolean shouldChangeRight = (array[right] < 0);`


Never changes until you remember it and remember about cheat code! Oh well, whatever you say. :)


p는 사용되고 있지 않습니다.


ch 가 char의 약자로 보이긴 해서요. main이라 일단 수정은 안하겠습니다 ^^


동작에는 문제가 없으나 아래 Java coding guide brace 관련 내용 참고하셔서 여는 중괄호 앞에 줄바꿈이 없도록 일관성 유지해주시면 좋겠습니다.
https://google.github.io/styleguide/javaguide.html#s4.1.2-blocks-k-r-style

예)
`if (shouldChangeLeft && shouldChangeRight) {`


点击下载前端代码


동작에는 문제가 없으나 아래 Java coding guide brace 관련 내용 참고하셔서 여는 중괄호 앞에 줄바꿈이 없도록 일관성 유지해주시면 좋겠습니다.
https://google.github.io/styleguide/javaguide.html#s4.1.2-blocks-k-r-style

예)
`}  else {`


@dpetrisko following commit fixed CI failure


'class Goods{'  --->  `class Goods { `

우리 프로젝트는 구글의 C++ 코딩 가이드라인을 따르고 있습니다.
아래 링크의 코딩 가이드라인을 한번 읽어봐 주세요.
https://google.github.io/styleguide/cppguide.html

그리고, 코드 커밋 이전에 cpplint로 소스 코드를 정리해서 커밋 부탁드립니다.
사용법 참고 ) https://blankspace-dev.tistory.com/395


Successfully aliased the URL <https://backend-4go07xtd5.now.sh> to the following aliases.

* https://backend.study-better.now.sh
* https://backend-git-master.study-better.now.sh


Странная конструкция, тут else не нужен.
```
if (...)
    return 1;
return 0;
```

А если еще подумать, то вообще тут правильно написать ``return (stk->top) == 0;`` )
иф тру ретурн тру елс ретурн фолс это пример из анекдотов)


필수는 아니지만 statement가 left++ 한줄이어도 중괄호를 넣어서 해당 while loop가 어디까지 영향을 주는지 잘 나타났으면 합니다.
나중에 추가 statement를 넣을 필요가 있을 때에도 혼선을 줄일 수 있겠습니다.


Now you really like it hot-swappable, right? :sunglasses: Anyway, thanks for the suggestion. But as a rule of thumb, if there is already a switch for it (e.g. in the setup tool) I don't need another one in the menu. And it's not like pitch-shifting is a feature that you regularly toggle back and forth just because - or is it?


how long does it take to finish this test?


Yaaaaaaaaaaaaaaaaaay a new monster ♥


반영했습니다.


I suspect the code is not needed, So please remove.


불필요한 comment 삭제하여 반영하였습니다.


Understood. I will add braces in this code.


google 코딩 스타일 참고하여 일관성을 맞추도록 하겠습니다.


PriorityQueue::Pop() 함수는 return type이 void 입니다.


STL container such as priority_queue, queue and stack does not hande exceptional cases.
Please see https://stackoverflow.com/questions/4892108/c-stl-stack-question-why-does-pop-not-throw-an-exception-if-the-stack-is-em
Because this class is wrapper class for std::priority_queue, it is better to comply with stl contract I think.


예. clean하게 수정하겠습니다.


Done


Не принято как-то сравнивать с 1, обычно сравнивают всегда с 0. В С обычно 0 это false, а все остальное - true, вы тут сильно ограничиваете.


Not sure why this happened lmao.


Successfully aliased the URL <https://urbarium-admin-react-luqm8lcx2.now.sh> to the following aliases.

* https://urbarium-admin-react.urbarium.now.sh
* https://urbarium-admin-react-git-master.urbarium.now.sh


This didn't fix the issue for me since it's still trying to fetch patch.txt which no longer exists on NA servers, i was able to solve the error by removing Dictionary and MabiEnvironment and a few this.env.MabinogiDir +  , which i realize is the wrong way of fixing it and is actually just avoiding the issue, but just thought i would let you know.
http://uotiara.com/img/MabiPack-master.zip


좀더 자세히 설명해주실 수 있을까요?


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


어떻게 하는지 모르겠어요.


나중에 공부해서 할께요


좋은 정보 감사합니다.


GetLastElement는 현재 테스트에만 사용되고 있습니다. 테스트용도만을 위한 것이라면 멤버함수에서 제거하고 다른 테스트 방법을 적용하는것이 보안상 더 좋을것 같습니다.
[https://stackoverflow.com/questions/2396370/how-to-make-google-test-classes-friends-with-my-classes](url)


커멘트 반영하여 새로 commit 하였습니다.


Посимвольно читать - медленно. Хотелось бы буффер и читать в него кусками.


### [Codacy](https://app.codacy.com/app/mehmet.keskin/SpeechToMe/commit?cid=366294311) detected an issue:
#### Message: `Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes.`

#### Occurred on:
+ **Commit**:  665c854fe8881fa45a37097ff684b2f5c41a74a5
+ **File**:    [SpeechToKoctas/Back-End/src/main/java/com/example/demo/ResponseText.java](https://github.com/koctas-as/SpeechToMe/blob/665c854fe8881fa45a37097ff684b2f5c41a74a5/SpeechToKoctas/Back-End/src/main/java/com/example/demo/ResponseText.java)
+ **LineNum**: [13](https://github.com/koctas-as/SpeechToMe/blob/665c854fe8881fa45a37097ff684b2f5c41a74a5/SpeechToKoctas/Back-End/src/main/java/com/example/demo/ResponseText.java#L13)
+ **Code**:    `private String text;`

#### Currently on:
+ **Commit**:  665c854fe8881fa45a37097ff684b2f5c41a74a5
+ **File**:    [SpeechToKoctas/Back-End/src/main/java/com/example/demo/ResponseText.java](https://github.com/koctas-as/SpeechToMe/blob/665c854fe8881fa45a37097ff684b2f5c41a74a5/SpeechToKoctas/Back-End/src/main/java/com/example/demo/ResponseText.java)
+ **LineNum**: [13](https://github.com/koctas-as/SpeechToMe/blob/665c854fe8881fa45a37097ff684b2f5c41a74a5/SpeechToKoctas/Back-End/src/main/java/com/example/demo/ResponseText.java#L13)


u


done.


This is a test to see if this comment shows up on PR #6


Done


@fulv FYI: your commit broke all the Plone 5.2 builds: https://jenkins.plone.org/job/plone-5.2-python-2.7/146/


답변드렸습니다.


Давайте делать ``const`` параметрам функций, где это возможно. Функция проверки на пустоту, которая может модифицировать объект - это плохо.


k 번째를 찾아야하니 값을 알아야 하는데요.
혹시 다른 의미로 하신 말씀일까요?


Will it also show up with the `#6`


très bizarre


Successfully aliased the URL <https://kendraio-app-d8pfr6g2x.now.sh> to the following aliases.

* https://dev.app.kendra.io
* https://kendraio-app.kendraio.now.sh
* https://kendraio-app-git-develop.kendraio.now.sh


Fixes #6486


> Now you really like it hot-swappable, right?

I'd say now I'm in love with all the hot-swappable things! Because I just have to switch something in the game without restarting, that's very, very comfortable.

Well, yeah, pitch-shifting is not the feature which must be changed often, that's for sure. However, my difference is - I'm trying to have everything in game menu, so neither a player, nor me have to close the game, go to setup, change something there, then close setup, then start the game again, and in case of disappointing about the change, make it all way back.

From another side, not everything have to be present in game menu, or it will something like this picture, which was once posted by @jmtd:

![339847](https://user-images.githubusercontent.com/21193394/59327944-bc8fe800-8cf3-11e9-9ab9-66a14f4e2182.jpg)


@fulv I reverted your commit. Please check why the p.a.users test is failing. Feel free to re-add plone.staticresources or revert the commit in plone.staticresources that leads to the test failure.


Fixes #6486 (instead of #6846)


result 로도 의미전달이 충분하다고 생각합니다만. 바꿔도 별로 문제될건 없습니다.
수정하도록 하겠습니다.


That's a yes.


файлы не закрыли


Goole coding guide(https://google.github.io/styleguide/cppguide.html#Spaces_vs._Tabs)에 의하면 Default indentation is 2 spaces.라고 되어있습니다. 들여쓰기를 2 space로 하시면 좀 더 가독성이 높아질 것 같은데 어떻게 생각하시는지요?


아래의 헤더에서 언급한 바와같이 다음과 같이 수정하는 게 어떨까요?

```
void GoodsList::Add(Goods object)
{
  std::vector<Goods>::iterator i = std::find(list_.begin(), list_.end(), object);
  if (i == list_.end())  // object not in the list
    list_.push_back(object);
}

void GoodsList::Remove(Goods object)
{
  std::vector<Goods>::iterator i = std::find(list_.begin(), list_.end(), object);
  if (i != list_.end())  // object in the list
    list_.erase(i);
}

void GoodsList::Print() {
  for (std::vector<Goods>::iterator i = list_.begin(); i != list_.end(); i++) {
    std::cout << "Goods #" << i->GetIndex() << " - Price : " <<  i->GetPrice() << std::endl;
  }
}

void GoodsList::SortByPrice() {
  std::vector<Goods>::iterator p = list_.begin();
  std::vector<Goods>::iterator min = list_.end();
  for (std::vector<Goods>::iterator i = list_.begin(); i != list_.end()-1; i++) {
    for (std::vector<Goods>::iterator j = i; j != list_.end(); j++) {
      if (min == list_.end()) {
        min = j;
        continue;
      }
      if (min->GetPrice() > j->GetPrice()) {
        min = j;
      }
    }
    std::iter_swap(i, min);
    min = list_.end();
  }
}

void GoodsList::SortByIndex() {
  std::vector<Goods>::iterator p = list_.begin();
  std::vector<Goods>::iterator min = list_.end();
  for (std::vector<Goods>::iterator i = list_.begin(); i != list_.end()-1; i++) {
    for (std::vector<Goods>::iterator j = i; j != list_.end(); j++) {
      if (min == list_.end()) {
        min = j;
        continue;
      }
      if (min->GetIndex() > j->GetIndex()) {
        min = j;
      }
    }
    std::iter_swap(i, min);
    min = list_.end();
  }
}

```


memset?


Understood.
Plus, I found out one more. I have added small braces in if conditions.
https://github.com/jaeyoung-shin/codereview/commit/77173639eddd5782e4eb0401d78b86a3743bf6da
Thank you for your review.


Successfully aliased the URL <https://urbarium-admin-react-hj7r99jll.now.sh> to the following aliases.

* https://urbarium-admin-react.urbarium.now.sh
* https://urbarium-admin-react-git-master.urbarium.now.sh


каст не нужен


예.
변경을 해도 무방할 것 같습니다.
속도 개선이 있는걸까요? 아니면 readability개선인지?


final을 사용할때는 초기값 설정 없이 fix된거로 보면 되는건지요?


여기서 T는 관용적으로 사용하는 부분인데요
보통 다음과 같이 명명합니다.
E - Element
K - Key
N - Number
T - Type
V - Value
따라서 제생각엔 그대로 두는게 좋을거 같습니다.


![idea1](https://user-images.githubusercontent.com/50777481/59328139-99caf700-8d26-11e9-89ed-e2d4096c5130.png)

idea page_1 --> Image


![idea2](https://user-images.githubusercontent.com/50777481/59328180-b2d3a800-8d26-11e9-9c15-59bdfce12801.JPG)

idea page_2 --> Image


Got it.
https://github.com/jaeyoung-shin/codereview/commit/250bd55d7395d101bca2b94e221d5ee866c91246


수정하겠습니다.


Or, just have a walk through all of PrBoom+'s menus and submenus...


-1 за утечки и  stack-buffer-underflow
==43639==ERROR: AddressSanitizer: stack-buffer-underflow on address 0x7ffee260135c at pc 0x00010d5bf8cf bp 0x7ffee2601270 sp 0x7ffee2601268
WRITE of size 4 at 0x7ffee260135c thread T0
    #0 0x10d5bf8ce in count_code 5-huffman.c:120
    #1 0x10d5c3950 in main 5-huffman.c:137
    #2 0x7fff724473d4 in start (libdyld.dylib:x86_64+0x163d4)

Address 0x7ffee260135c is located in stack of thread T0 at offset 28 in frame
    #0 0x10d5c30cf in main 5-huffman.c:277

==43642==ERROR: LeakSanitizer: detected memory leaks

Direct leak of 40 byte(s) in 1 object(s) allocated from:
    #0 0x108525de3 in wrap_malloc (libclang_rt.asan_osx_dynamic.dylib:x86_64h+0x60de3)
    #1 0x107b023ec in main 5-huffman.c:307
    #2 0x7fff724473d4 in start (libdyld.dylib:x86_64+0x163d4)

Direct leak of 24 byte(s) in 1 object(s) allocated from:
    #0 0x108525de3 in wrap_malloc (libclang_rt.asan_osx_dynamic.dylib:x86_64h+0x60de3)
    #1 0x107b01e21 in decod_tree 5-huffman.c:17
    #2 0x107b02442 in main 5-huffman.c:309
    #3 0x7fff724473d4 in start (libdyld.dylib:x86_64+0x163d4)

SUMMARY: AddressSanitizer: 64 byte(s) leaked in 2 allocation(s).

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


Successfully aliased the URL <https://urbarium-admin-react-9vguxddl7.now.sh> to the following aliases.

* https://urbarium-admin-react.urbarium.now.sh
* https://urbarium-admin-react-git-master.urbarium.now.sh


Successfully aliased the URL <https://flight-lister-kiilsw5ib.now.sh> to the following aliases.

* https://flight-lister.fatihtelis.now.sh
* https://flight-lister-git-master.fatihtelis.now.sh


Successfully aliased the URL <https://alpaca-ui-7jpwwf62s.now.sh> to the following aliases.

* https://alpaca-ui.now.sh
* https://alpaca-ui.snowdog1.now.sh
* https://alpaca-ui-git-develop.snowdog1.now.sh


한줄의 컬럼수가 조금 늘어나더라도 Code의 Readibility를 높이기 위해서 for(조건문) 내용을 한줄로 작성하는건 어떨까요?
 for (std::vector<Goods>::iterator i = shopping_list_.begin();i != shopping_list_.end(); i++) {


loop, if 문등에서의 indentation, {  파일 전반적으로 방식 통일 부탁드립니다.
 if (isCharRepeated(previousChar, currentChar)) {


Exactly! :fearful:


Thank you for your review.
Could you let me know any reference or official guide about that?
I am not familiar with these policy.


We can change that back, not a huge deal.


아래와 같이 이전 for문의 expression 사이 ;뒤에 시인성을 위해 space 1칸 추가 해주세요.
for(int j=i+1; j<n; j++)


Good catch! Should be fixed into https://github.com/Source-Python-Dev-Team/Source.Python/pull/277/commits/d311073d8059dac6aa2353971fe7f8912589b70e.


Да, уже взялся новую версию переносить - RenderTargetPhaseRainDrops.cpp


indentation을 space 1로 변경 해주세요.


`margin-right` ?


인터넷에서 장바구니에 담은 3개의 상품이 있다고 가정할때
   - "상품 리스트" 가 맞을까요?
   - " 쇼핑 리스트" 가 맞을까요?


Okay done.
https://github.com/jaeyoung-shin/codereview/commit/77173639eddd5782e4eb0401d78b86a3743bf6da


if 문의 indentation을 space 1로 변경 해주세요.


if문 안에 indentation을 space 1로 변경 해주세요.


Google code style에 따라서 indentation을 space 1로 변경 해주세요.


test


단순한 가격,인덱스만으로 이루어진 하나의 개체라고 본다면 상품이 맞을 것 같네여.

Class name 변경
   Shopping --> GoodsList


![인스타그램](https://user-images.githubusercontent.com/50777481/59328530-908e5a00-8d27-11e9-91e3-59a0b6411ce2.JPG)

insta  ---> Image


네. 반영하겠습니다.


class 단위로 UT를 작성하였습니다. 해당 건은 현상태를 유지했으면 합니다.


Deployment failed with the following error:

~~~
The domain somename.com couldn't be found.
~~~


Dias Especiales


Google coding guide(https://google.github.io/styleguide/cppguide.html#Spaces_vs._Tabs)에 의하면 There should be a space between the close parenthesis and the open curly brace. 라고 되어 있습니다. ')'와 '{' 사이에 어떤 경우는 한칸이고 어떤 경우는 빈칸이 없는데 일정하게 한칸으로 들어가는 것이 어떨까요?


if (src.length() <= 1) 로 수정하는게 맞는거 같습니다.
https://github.com/moonny76/codereview/commit/74f33178c5e89349238e466a6bef48f71a7dd03c


brace 규칙에 안맞는 부분 일괄 수정하였습니다
https://github.com/moonny76/codereview/commit/74f33178c5e89349238e466a6bef48f71a7dd03c


이 부분은 기존 logic은 그래도 두고 return을 if문과 같은 line으로 만드는 것으로 수정 했습니다.


직관적 용어가 더 낫네요.
add 수정하겠습니다.


Done. Thank you.
https://github.com/jaeyoung-shin/codereview/commit/ff3abe1ef58758efbdf7295ebe9a8526056b4e5a#diff-8e508b29cf4c003c061bd7f759534410


연산자 우선순위 명확하기위해 필요해보입니다.
https://github.com/moonny76/codereview/commit/74f33178c5e89349238e466a6bef48f71a7dd03c


Yes, I see


brace 를 coding rule에 따라 일괄 정리하였습니다.
https://github.com/moonny76/codereview/commit/7fcfec6495e01c289715df51a883493c05a724b4


함수 동작에 따라 이름을 변경하였습니다.
https://github.com/moonny76/codereview/commit/7fcfec6495e01c289715df51a883493c05a724b4


주석 삭제하였습니다.
https://github.com/moonny76/codereview/commit/7fcfec6495e01c289715df51a883493c05a724b4


pinter2을 의미있는 이름" currentNode"로 변경하였습니다.
https://github.com/moonny76/codereview/commit/7fcfec6495e01c289715df51a883493c05a724b4


@michaelzangl I get `org.openstreetmap.josm.gui.progress.ProgressException: Expected states are [IN_TASK, IN_SUBTASK] but current state is INIT` in this line. How do I handle that?


original list 이긴 한데, 개인적으론 list 만으로도 의미 전달이 충분하다는 생각이 듭니다.
추천해주실 이름이 있을까요?


This is the entire stacktrace:
```
2019-06-12 12:05:59.167 SEVERE: Handled by bug report queue: org.openstreetmap.josm.gui.progress.ProgressException: Expected states are [IN_TASK, IN_SUBTASK] but current state is INIT
org.openstreetmap.josm.gui.progress.ProgressException: Expected states are [IN_TASK, IN_SUBTASK] but current state is INIT
	at org.openstreetmap.josm.gui.progress.AbstractProgressMonitor.checkState(AbstractProgressMonitor.java:84)
	at org.openstreetmap.josm.gui.progress.AbstractProgressMonitor.internalUpdateProgress(AbstractProgressMonitor.java:266)
	at org.openstreetmap.josm.gui.progress.AbstractProgressMonitor.setTicks(AbstractProgressMonitor.java:244)
	at org.openstreetmap.josm.plugins.autobound.DataCollectionAction.actionPerformed(DataCollectionAction.java:64)
	at javax.swing.AbstractButton.fireActionPerformed(AbstractButton.java:2022)
	at javax.swing.AbstractButton$Handler.actionPerformed(AbstractButton.java:2348)
	at javax.swing.DefaultButtonModel.fireActionPerformed(DefaultButtonModel.java:402)
	at javax.swing.DefaultButtonModel.setPressed(DefaultButtonModel.java:259)
	at javax.swing.AbstractButton.doClick(AbstractButton.java:376)
	at javax.swing.plaf.basic.BasicMenuItemUI.doClick(BasicMenuItemUI.java:842)
	at javax.swing.plaf.basic.BasicMenuItemUI$Handler.mouseReleased(BasicMenuItemUI.java:886)
	at java.awt.AWTEventMulticaster.mouseReleased(AWTEventMulticaster.java:289)
	at java.awt.Component.processMouseEvent(Component.java:6539)
	at javax.swing.JComponent.processMouseEvent(JComponent.java:3324)
	at java.awt.Component.processEvent(Component.java:6304)
	at java.awt.Container.processEvent(Container.java:2239)
	at java.awt.Component.dispatchEventImpl(Component.java:4889)
	at java.awt.Container.dispatchEventImpl(Container.java:2297)
	at java.awt.Component.dispatchEvent(Component.java:4711)
	at java.awt.LightweightDispatcher.retargetMouseEvent(Container.java:4904)
	at java.awt.LightweightDispatcher.processMouseEvent(Container.java:4535)
	at java.awt.LightweightDispatcher.dispatchEvent(Container.java:4476)
	at java.awt.Container.dispatchEventImpl(Container.java:2283)
	at java.awt.Window.dispatchEventImpl(Window.java:2746)
	at java.awt.Component.dispatchEvent(Component.java:4711)
	at java.awt.EventQueue.dispatchEventImpl(EventQueue.java:760)
	at java.awt.EventQueue.access$500(EventQueue.java:97)
	at java.awt.EventQueue$3.run(EventQueue.java:709)
	at java.awt.EventQueue$3.run(EventQueue.java:703)
	at java.security.AccessController.doPrivileged(Native Method)
	at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(ProtectionDomain.java:74)
	at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(ProtectionDomain.java:84)
	at java.awt.EventQueue$4.run(EventQueue.java:733)
	at java.awt.EventQueue$4.run(EventQueue.java:731)
	at java.security.AccessController.doPrivileged(Native Method)
	at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(ProtectionDomain.java:74)
	at java.awt.EventQueue.dispatchEvent(EventQueue.java:730)
	at java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:205)
	at java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:116)
	at java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:105)
	at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:101)
	at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:93)
	at java.awt.EventDispatchThread.run(EventDispatchThread.java:82)
```


수정하겠습니다.


중복 처리되는 부분 제거 하였습니다.


Fix #7


룰에 맞게 indentaion조정 하였습니다.


미사용변수제거 하였습니다.


아래에서 ㅗhead로 추가 체크 되는 부분이 있으나, 말씀하신 것처럼, 미리시작 시점에서 처리하는 방향으로 정리 하였습니다.


![button_1](https://user-images.githubusercontent.com/50777481/59328931-9b95ba00-8d28-11e9-963e-335271b3245e.JPG)

button_1 ---> Image


아래 있으니까 괜찮을거 같습니다.


@popovec asked me to comment on this.

It's fine for us at Aventra that support for OsEID is added to the MyEID driver.

I have one suggestion: Could the driver keep the information that it has been initialized with a simulated MyEID card and maybe set "OsEID" to card->name in myeid_get_info, or at least write this information to log? I think it would be good to be able to distinguish in logs, which card has been used.


반영했습니다.


제가 만든 코드가 아니라서요 이 부분 수정은 못할것 같아요.


반영했습니다. 맞는 코드인지는 확인 필요해서 TODO 표시했습니다.


It is hard for me to do refactoring.
It will take some time to do refactoring. :)


Comment 반영했습니다.


헤더에 define guard 추가했습니다. 확인 부탁드립니다.


I think it should be removed because it was already implemented in header file


SplitArray 가 단일한 솔루션을 개발하는게 아닌, 가능한 예제를 보여주기 위한 헬퍼 클래스 정도로 생각하면,
각 솔루션을 나누지 않고, 하나에 두는것도 가능하다고 생각됩니다.


public을 먼저 선언하도록 변경했습니다. 다시 한번 확인해주세요.


Done. Thank you for your review.
https://github.com/jaeyoung-shin/codereview/commit/5eae46db3434a8f33020a871dba80cff14c2aadf#diff-8e508b29cf4c003c061bd7f759534410


파일 전체적인 indentation check가 필요해 보입니다.


Successfully aliased the URL <https://adventure-qip4400cv.now.sh> to the following aliases.

* https://adventure-go.now.sh
* https://adventure.kimhaein.now.sh
* https://adventure-git-master.kimhaein.now.sh


I'm fairly sure this won't work, you're creating an arraylist then treating it like a hash table.  Additionally GBVParams should contain PATH even if it's not specified.  Invoke-Git will assume the PWD if it doesn't get a PATH passed to it and since if no path is specified to this cmdlet, the PWD will be assumed the behavior is probably the same, but it probably makes more sense to stay consistent and be explicit and always pass the path in.


LGTM


It is just for namesake. I haven't used it anywhere. Also, it is an optional field and you can create or join sessions without filling in this field. In the future, I will use it for collaboration.


중괄호 사용 부탁드릴게요.


Goole code style에 따라서 {를 K & R style로 작성 해주세요.


Successfully aliased the URL <https://flight-lister-z9cgb8ggr.now.sh> to the following aliases.

* https://flight-lister.fatihtelis.now.sh
* https://flight-lister-git-master.fatihtelis.now.sh


훨씬 간결하고 보기 좋습니다.


Hi, do you plan to backport this to tcpdump-4.9 branch?


Please **modify** the files, instead of creating different files for different TODO, starting from Meeting 03


thereIsNoMoreWordToCompress는 boolean이고 i, src.length()는 int type으로 type이 맞지 않습니다.
해당 코드의 의도 확인 부탁드립니다.


함수 이름은 소문자로 시작하도록 전체적으로 수정 부탁드립니다.


Google coding guide(https://google.github.io/styleguide/cppguide.html#Spaces_vs._Tabs)에 의하면 There should be a space between the close parenthesis and the open curly brace. 라고 되어 있습니다. 'class Shopping'과 '{'사이에 한칸 들어가는 것이 어떨까요?


Oh, it is the mistake from the spec. This property is applied to all elements. I have to fix it in the spec.


![check_2](https://user-images.githubusercontent.com/50777481/59329238-68075f80-8d29-11e9-8c1c-3d9ef46e974a.JPG)

check_2 --->Image


@MojoCreator Apologies for my ignorance. Thanks for letting me know


At the very least I think to maintain the behavior you're trying here this line should be `$GBBParams = @{}`


예 수정해도 무방할 것 같습니다. 반영하겠습니다.


![select_3](https://user-images.githubusercontent.com/50777481/59329314-8ff6c300-8d29-11e9-9a29-771c0bb26b1e.JPG)

select_3 --->Image


STD를 이용한 Priority Queue의 Wrapper Class라 별도 Fail Test Case는 필요치 않는것으로 파악되었씁니다.


Successfully aliased the URL <https://tryout-sdk-example-8c2kprctq.now.sh> to the following aliases.

* https://tryout-sdk-example.hit2hat.now.sh
* https://tryout-sdk-example-git-master.hit2hat.now.sh


incline


Successfully aliased the URL <https://rc-field-form-7nzkpd4ik.now.sh> to the following aliases.

* https://rc-field-form.react-component.now.sh
* https://rc-field-form-git-master.react-component.now.sh


字符流操作，所以用Reader


文件的路径不好获取


LGTM


I would be happy if you could share the related part of clean code as you mentioned.
Anyway I will try to change it.


Previously UDHR.

Updated with info from https://en.wikipedia.org/wiki/Corsican_language, which says "n older writing, the acute accent is sometimes found on stressed ⟨e⟩, the circumflex on stressed ⟨o⟩, indicating respectively (/e/) and (/o/) phonemes. "

This commit adds É Ô é ô ́ ̂ to aux.


yeah


리뷰 남겼습니다. 확인 부탁드립니다.


对输出流的操作要从内存写入文件并关闭流


Code review passed.


Then, we need to discuss more that spec.
I think, designating spatial-navigation-action attribute to all element is so uncomfortable.


Now could not deploy this pull request since the `now.json` file is empty.
It should be a valid JSON file.

    For more information on now.json visit https://zeit.co/docs/features/configuration#now.json.


A few typos in the commit message:

"Build unstyled of leaderboard" means something (i think you said "Implement unstyled leaderboard")

serviceWroker

Thought there might be another, but can't see it now.


확인 부탁드립니다.


라이센스는 파일 처음에 위치시키는것어 보기 좋을 것 같습니다.
참조 https://google.github.io/styleguide/javaguide.html


Deployment failed with the following error:

~~~
You defined 1 build that did not match any source files (please ensure they are NOT defined in .nowignore):
~~~


Does anyone know a port number that will certainly raise the warning this test expects ? I think the azure vm is particularly forgiving, allowing us to bind to 80/443, I got 427 from docs and it worked locally, but maybe there's a port that will always fail ?


/cc @trowski @bwoebi @kelunik


Successfully aliased the URL <https://operation-code-awm6gtay5.now.sh> to the following aliases.

* https://operation-code.now.sh
* https://operation-code.operation-code.now.sh
* https://operation-code-git-master.operation-code.now.sh


收到了没有呢


oh wow, sorry about that. Good catch.


反正我现在是能到时间后跳转了


코드 인덴테이션을 아래  URL 참조해서 수정하고, 함수이름은  lowerCamelCase 형태로 하는 것이 보기 좋을 것 같습니다.
참조 https://google.github.io/styleguide/javaguide.html


Такого инклюда нет в стандарте


still wondering why the unit test did not anything in first place...


파일 전체적인 indentation check하였습니다.


图片拖过来这里一样的会报错哦


Successfully aliased the URL <https://website-qhjk22nlj.now.sh> to the following aliases.

* https://handmadestudios.now.sh
* https://website.handmadestudios.now.sh
* https://website-git-master.handmadestudios.now.sh


Make sure you were using only 1 commit for 1 TODO. Looking at your (most probably pasted from somewhere else) code and your commit, you only did 1 TODO out of 6. Please fix them immediately.


Done.
https://github.com/jaeyoung-shin/codereview/commit/10ecfd1ea1e86eec272ba2e74115e8c7a1e1caa2#diff-8e508b29cf4c003c061bd7f759534410


함수이름 변경 하였습니다.


修正しました


소스 파일 내 제일 상단에 project copyright 안내를 포함시켜 주세요.
이 내용은 프로젝트 내 다른 소스 파일을 참고하시면 됩니다.


중괄호 추가하였습니다.


Done.
https://github.com/jaeyoung-shin/codereview/commit/de4f8496907d820d458f96c2e2842ef338e5d900#diff-8e508b29cf4c003c061bd7f759534410


Successfully aliased the URL <https://website-k0hxtly4r.now.sh> to the following aliases.

* https://handmadestudios.now.sh
* https://website.handmadestudios.now.sh
* https://website-git-master.handmadestudios.now.sh


О_о вообще непонятно как такие вложенные фор и ифы читать) Лучше расставить фигурные скобочки и поформатировать)


Also regarding this kind of commits. Do not confuse people with multiple commits with the same messages. You may use git bash or Git Kraken to ease your commit process, instead of update one file with one commit.


case文に修正しました


정상 코드로 보입니다.
()만 추가하겠습니다.


#42 #43


수정하겠습니다.


Сильно большая вложенность, нужно уменьшить


원래 push 에는 레퍼런스가 사용됩니다.
그런데 object 가 아닌 primitive 타입이라서 혼란스럽네요.


Class 에서 Public > Protected > Private 순서로 정의하는게 일반적인것 같습니다.


Successfully aliased the URL <https://the-snap-o0gc0rldp.now.sh> to the following aliases.

* https://the-snap.arvigeus.now.sh
* https://the-snap-git-master.arvigeus.now.sh


Notice the naming convention: UPPER_CASE are strictly only for constants


You are supposed to count acceleration, not velocity


Yes, strange. There isn't actually any servicegroups that works with AuthUser in the testdata, but this code should still be executed by the test. I wonder what GetColumn retruns on an invalid Column?

In any case I'll try to improve the testing.


I found this code is useless.
So I will delete it.
Thank you.


branch created in 2019/06/12


The exact location of that file is in your vagrant dir, usually (linux dist.) in
/opt/vagrant/embedded/gems/2.2.4/gems/vagrant-2.2.4/plugins/communicators/ssh


There are two types of state change detection as mentioned in my proposal.
1. URL State : Changes are not very frequent
2. MVP matrix state: Changes are frequent.
The URL state contains the information of which molecule is loaded and what are the styles applied.
Example URL : `server:port/viewer.html?pdb=1UBQ&select=all&style=sphere&select=all&surface=opacity:1&select=all&labelres=`
So if I can detect the URL state change I will get to know which molecule has been loaded, its style .. etc
But URL is not a part of the HTML DOM. so I cannot use some jquery onchange event for URL. So I have two variables: the `old_url` and `new_url` and I check after every 1 second in the initiator's computer if the URL has changed or not. And this function runs only on the initiator's computer and not on those who joined.
But why 1 second? I believe that is the approximate time taken to load the molecule given the query. Also, if the time where given any lesser like say 50ms, then the initiator's computer would have a lot of 'load'. Or if it were any higher then it wouldn't be fast enough.
Once the URL state change detection has been performed the initiator then sends the `new_url` to the server and the server broadcasts it to all the students in the session. On the student's computer, we have functions to generate the molecule given the URL.
Can we do better? (Instead of checking every 1 second, whether the URL has changed or not)
I will look into it. But the current method isn't inefficient since that state change detection is only performed on the initiator's computer and if and only if the URL state change has been changed, the client sends the information to the server. So, there is no load on the server(since this isn't polling it is still web sockets).


Added search, keyboard shortcuts and preference items for pagination of library (Issue #232)


Use 1 commit to update **ONLY 1 TODO**. And also 1 commit can update more than one files. Use git bash or git Kraken if needed.


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


Successfully aliased the URL <https://urbarium-admin-react-ltt3xo5f0.now.sh> to the following aliases.

* https://urbarium-admin-react.urbarium.now.sh
* https://urbarium-admin-react-git-master.urbarium.now.sh


Following article explains binary search with visualised content
https://medium.com/@jeffrey.allen.lewis/javascript-algorithms-explained-binary-search-25064b896470


삭제하도록 하겠습니다


There is a binary file in your commit. Please, remove that. [how?](https://stackoverflow.com/questions/2047465/how-can-i-delete-a-file-from-git-repo)
(Binary files or any user specific data / configs should not be part of commits.)


LGTM


To avoid having to adapt this every year, in RAFCON we now write:
```python
now = datetime.datetime.now()
copyright = u'{year}, DLR'.format(year=now.year)
```


Successfully aliased the URL <https://musiikin-teoria-material-3hvay8eqn.now.sh> to the following aliases.

* https://musiikin-teoria.now.sh
* https://musiikin-teoria-material.mooc.now.sh
* https://musiikin-teoria-material-git-master.mooc.now.sh


초기에 min을 i로 대입하는 방식으로 개선하도록 하겠습니다.


@mstembera
Protospring had set prio to 0 i see. But now i set it back to -1 because its struggles and i will try even 8 times pawn hash here http://tests.stockfishchess.org/tests/view/5d00a3810ebc5925cf09ba16

Your test is also an interresting idea. But as all shared data structures there is probably a problem for NUMA-Architectures because data transfer between nodes is slow. That was the reason all the tables (except TT, that is needed for lazy SMP to work) are now per thread.


we plan to provide compute later?


Deployment failed with the following error:

~~~
The property `engines` is not allowed in now.json when using Now 2.0 – please remove it.
~~~


if(i != min) 조건 추가해 명확히 하도록 하겠습니다.  동작에 문제가 없을것 같으므로 판단됩니다


삭제하겠습니다.


Successfully aliased the URL <https://website-flp7wczzy.now.sh> to the following aliases.

* https://handmadestudios.now.sh
* https://website.handmadestudios.now.sh
* https://website-git-master.handmadestudios.now.sh


> // GetColumn returns a column for given name or nil if not found

There is a `GetColumnWithFallback` which always returns a Column.


closes #50


?? use .css files for that


review하신대로 inline처리하였습니다.


{ 위치가 do/while 에 맞춰지면 좋을 것 같습니다.


제가 이해하기엔 왜 Comment와 List<Comment> 2가지가 있냐고 물어보신 게 맞으신가요? BoardDetail에서 Comment를 여러 개 출력하기 위하여 foreach문을 사용하는데, 그냥 Comment를 가져오면  GetEnumerator에 대한 공용 인스턴스 정의가 포함되어있지 않다고 오류가 납니다. 또한 DisplayNameFor에서 리스트형식(Comments)을 가져와도 오류가 나서 둘 다 넣었습니다.


함수를 합하긴했는데, 깔끔하지가않네요, 결국 합하면서 성능에 영향을 주는게 아닌가싶기도하고, 돌때마다 if문체크가 들어가게되네요,


한 줄이 너무 길어서 분리해서 작성하면 가독성이 좋아질 것 같습니다.


Successfully aliased the URL <https://flight-lister-iy15imjo1.now.sh> to the following aliases.

* https://flight-lister.fatihtelis.now.sh
* https://flight-lister-git-master.fatihtelis.now.sh


추가했습니다.


не треба писати Optional краще. а викликати краще exception. Бо змінювати сигнатуру методів які сказали в домашці не можна


너무길어지는 상황이라, 한줄로, 표기하는거보다 내리는게 좋아보여서 현상태유지했습니다.


줄바꿈 없이는 80 컬럼이 넘어가서 문제가 되는 상황입니다.  그런데 선언을 밖에서 할 경우 일관성을 위해 다른 loop들의 선언도 모두 밖으로 나가야 할 것 같은데, compact 하게 가기 위해 그냥 현재 상태를 유지하는 것과 모두 밖으로 빼는 것과 어떤 것이 좋을지 다시 의견 부탁드립니다.


Successfully aliased the URL <https://alpaca-ui-k00wkk8b4.now.sh> to the following aliases.

* https://alpaca-ui.now.sh
* https://alpaca-ui.snowdog1.now.sh
* https://alpaca-ui-git-develop.snowdog1.now.sh


LGTM


> The 'remotipart' gem seems to be the culprit of this issue.
>
> Using `gem 'remotipart', github: 'mshibuya/remotipart', ref: '3a6acb3'` cleared up the issue.
>
> [JangoSteve/remotipart#139](https://github.com/JangoSteve/remotipart/issues/139)

when i am using this version of gem it showed some error
_**Revision 3a6acb3 does not exist in the repository git://github.com/mshibuya/remotipart.git. Maybe you misspelled it?**_


Successfully aliased the URL <https://operation-code-88jolak8i.now.sh> to the following aliases.

* https://operation-code.now.sh
* https://operation-code.operation-code.now.sh
* https://operation-code-git-master.operation-code.now.sh


커멘트 업데이트가 필요해 보입니다


Please use only 1 commit for 1 TODO. Also, use the files in Github, not the one from Gist


첫번째 Board는 BoardIndex에서 뷰모델처럼 사용하고 있기 때문에 Board에 들어있는 다른 값을 사용하기 위하여 가져왔고, 두번째째 PaginatedList<Board>는 아마도 이것저것 하다 넣었던 것 같습니다. 소스 정리하면서 제외했습니다.


full stack developer du!


Congrats too!


@lochmueller
Thank you, now i use your version.

I looked at the new rule, there are some errors/typos. It should be

There is no "pObj" there, checked it, we do not need that.
        if (!\is_array($frontendController->page)) {
            $skipProcessing = true;
            $explanation[__CLASS__] = 'There is no valid page in the TSFE';
        }


Never expected this to happen to Violet!!!#RIP .... i literally cant believe my eyes..!!!:(


80컬럼 룰로 2번으로 수정하겠습니다.


@NetanelBasal great! This fixed the error; however, the dev tools do not work when running as a shell. No store found. But hey, at least there isn't an error in our builds! Do you think this would ever work using the shell and the Angular Universal?


But in deep theory, thinking more, having a _one_ place, i.e. one executable for all settings could be not as bad, as it may be imagined. Probably the most serious question here is not how to implement it, but how to place and align clearly whole structure: menus, submenus, sub-headers and elements.

Problem №2 is - we are very limited in both vertical and horizontal free space in 320:200 canvas. We actually may have an unscaled text, but it's just god-awful, and probably will not work in non-hires version:

![image](https://user-images.githubusercontent.com/21193394/59330528-65414600-8cfa-11e9-83a6-917ef6ceed3f.png)

Just thinking out loud. We have some kind of national holiday today, so everyone free from work, yay! :stuck_out_tongue_closed_eyes:


Use commit message to specify which TODO is being done here


comment 주신 부분 모두 수정하겠습니다.


저는 일괄 빼는 것을 선호합니다 .^^


Google Style Guide 에 따르면 자기 파일의 헤더가 #include 리스트에서 가장 먼저 나와야 합니다.


to deepspeech2.pytorch


일괄 빼는 것을 선호합니다.
만약 현상태 유지하신다면, 그래도 p는 현재 사용하지 않으니 삭제하는 것이 맞아 보입니다.


> AFAIK yout code will capture the exit status of the gzip rather than the one from curl

That was intended. gzip will fail for broken downloads and corrupt archives.

But yes, your solution achieves the same thing :+1:


if (!\is_array($frontendController->page) {


ths -> the


2 space 간격으로 정리하여 반영하였습니다.


Explicit initialization with constructors (C++ only) 참고하시면
[https://www.ibm.com/support/knowledgecenter/en/SSLTBW_2.3.0/com.ibm.zos.v2r3.cbclx01/cplr387.htm](url)
아래처럼 될것 같습니다.
Goods() : goods_index_(0), goods_price_(0) { }

상황에 따라 copy constructor가 필요할 수도 있습니다.


커멘트 업데이트가 필요해 보입니다.


method내 이름도 동일하게 변경이 필요해보입니다.


Successfully aliased the URL <https://p001-frontend-3f2qglxcb.now.sh> to the following aliases.

* https://p001.now.sh
* https://p001-frontend.screw123.now.sh
* https://p001-frontend-git-master.screw123.now.sh


input data가 random 하여, input data array와 for 를 사용한 push로 변경하였습니다.


수정시 누락했네요. 추가 수정하겠습니다.


수정시 누락했네요. 추가 수정하도록 하겠습니다.


게시글 만들기 페이지 보여주는 메소드라는 것이 무엇을 말씀하시는 지 모르겠습니다. Db Context에서 Board값을 가져온 이유가 무엇인지 생각을 하니 생각없이 return을 페이징으로 해놔서 추가한 것 같습니다.  수정하였습니다.


> `domain=eurosport.*|gamereactor.*`

Wildcards fungerar inte med `domain=`. Adressen måste sluta med en TLD. `domain=eurosport.no|gamereactor.no`


수정하였습니다.


Well that's extremely cool 👍


what's this??


A github repo that's actually a game to help you learn git more inportant

terencesola-index


수정하겠습니다.


Successfully aliased the URL <https://mdgen-s6wx12g0j.now.sh> to the following aliases.

* https://mdgen.shedali.now.sh
* https://mdgen-git-master.shedali.now.sh


unit_test_main.cc 에도 main 함수가 있습니다.
중복함수 인지 체크해 주세요.


@frederic-mahe En fait j' essayais d' afficher les "truncate" names et y ajouter le suffixe comprenant "trimmed" suivi du reste de l'extension de chaque fichier selon les entrées "R1" et "R2" des "paired-end-reads"
Exemple:
BU-TS-09_TGGCCGGT-TAGAGCGC-BH7MK3DSXX_L002_R2.fastq.gz ----> BU-TS-09_trimmed_R1.fastq.gz


Juan wants to change the font import to Roboto


일괄적으로 빼겠습니다.


일괄적으로 빼고 p를 삭제하겠습니다.


Why was this reverted? Please try to include the revert reason in the commit message.


p를 삭제하겠습니다.


Maybe it would be better to move this struct and the enum outside of the class.

Note: it would be nice to replace ONE and ZERO name by something less generic (could add a prefix or a suffix)


@frederic-mahe Je vais revoir ça....


No. That line itself won't be executed in case of `http://example.com/path`. We already filtering it in https://github.com/discourse/discourse/commit/e8fafbc123170dd1f7d2a8adea4e7810585d3e76#diff-df2014f8b8976aed72db66eccb63e298R932.


p를 삭제하겠습니다. 일괄적으로 i,j를 빼겠습니다.


Successfully aliased the URL <https://frontend-v2-7erzbe11f.now.sh> to the following aliases.

* https://frontend-v2.datopian1.now.sh
* https://frontend-v2-git-master.datopian1.now.sh


comment 하겠습니다.


This will hopefully satisfy the requirements from week 1.  I used a YouTube video "#65 Python..."


> If signedness is a requirement, then is this document limited to numeric types?

Why would signedness be requirement? What are your reasons to have unsigned primitive types?

> On 12 Jun 2019, at 04:12, Spencer Killen <notifications@github.com> wrote:
>
> If signedness is a requirement, then is this document limited to numeric types?
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub, or mute the thread.


Why you have such distinguish null/[]? Later logic needs empty array or null ? It should be consolidated somehow.


for문을 돌때마다 if문체크하는 부분이 걸려서 다시 원복하였습니다.


Reason: https://github.com/php/php-src/pull/4235#issuecomment-501130054


This was intended to be Roboto not Poppins


LTGM


Apply formatting and null check


LGTM


Hmm can you check if this break backwards compatibility with uploads on the old scheme? The reason we always hit the database is because the database is the source of truth to determine if an upload is valid. The reason we had the missing uploads nightmare previously was because `Upload.get_from_url` was changed such that it only worked on the new scheme and was not backwards compatible with the old scheme.


## Joker
Éste es mi villano favorito


牛皮


搜索


That is bad practice of path concatenation, use `filepath.Join(...)` instead.


Changed file 1


## Notas
Entiendo porque agregaste a Deadshot, lo necesitaremos


오옷! 블로그 시작하시는건가요!


> Should I remove little endian altogether?

IDK, maybe you shouldn't, because even if endianmess was considered a consequence of past circumstance no longer applying today, that wouldn't change the fact it's a current reality.

My point was to make a statement about if endianmess is an end by itself, a reasonably justified means, or some means that would maybe be considered obsolete, deprecated, or to be extinct. But maybe this is not the document for that, you are going to know.

> On 12 Jun 2019, at 04:29, Alexander Hill <notifications@github.com> wrote:
>
> The only reason I mentioned endianness is specifically because I wanted to be specific. Should I remove little endian altogether?
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub, or mute the thread.


Based on null/[] a node shows either a file or folder in Tree you already knows that. There are conditions between showing different type of nodes between Git, XML and SVN - where based on certain conditions the nodes suppose to be shown between file or folder.


It will be moved to a more appropriate location.


refs#13 : resolve


That's was introduces by me, I use it in stress test.
That's because of pre-made multithreading sync primitives, provided by boost lib, for example "boost::barrier". In standard c++ library they are still marked as "experimental"


# Guason
Es mi villano favorito


I would propose to make the line break between `<b.laky@intrallect.com>` and `on behalf`


Fixed by https://github.com/Homebrew/homebrew-core/commit/6010b3a8d9871e9fca76650209e49e0aa348cf31


Successfully aliased the URL <https://phpserverlessfromnextjs-5v4hfbxsh.now.sh> to the following aliases.

* https://phpserverlessfromnextjs.lukeluklus.now.sh
* https://phpserverlessfromnextjs-git-master.lukeluklus.now.sh


Successfully aliased the URL <https://vw-prices-app-9c6n3npu6.now.sh> to the following aliases.

* https://vw-prices-app.elenakarlsen.now.sh
* https://vw-prices-app-git-master.elenakarlsen.now.sh


Aliasing failed with the following error:

~~~
Error! You should buy the domain before aliasing.
~~~


using namespace std 사용하면 파일 전반에 걸쳐 코드가 간결해 질 것 같습니다.


Successfully aliased the URL <https://frontend-v2-jkqmecrdf.now.sh> to the following aliases.

* https://frontend-v2.datopian1.now.sh
* https://frontend-v2-git-master.datopian1.now.sh


good luck:)


# Notas
Entiendo porque agregaste a DeadShot. Lo necesitaremos.


refs #8 : modify README.md in release branch


typo


refs #8 : modify README.md in release server


那跳转后有没有声音呢


Yes. It's backward compatible. I'm already using this to [find old scheme uploads](https://github.com/discourse/discourse/blob/master/lib/tasks/posts.rake#L416-L424). Anyway, I will add a spec test for it.


현재 Board 테이블에 CommentCount가 있는데 Board 테이블에 CommentCount가 있는 것이 맞지 않으니 뷰모델을 만들어 사용하라 하셔서 따로 만들었습니다.


Ah, takk for informasjonen. Jeg skal få rettet opp i det med en gang.


That's very kind of computer, but unfortunately, it can't know.


- W.r.t. all the _"LoggerFactory.getLogger(Utils.class);"_ : That was a copy paste error. Just before reading your comments I detected it myself as well.

- W.r.t. "_Why \n? Is your output so huge? This may mess with log analytic tools."_
I do indeed sometimes use: "logger.debug("res is\n{}", res);"
the res is indeed a multiline texts. Sometimes 8 lines, sometimes >50 lines. I only(!) use that when having multi-line texts. Some of them are are really "program and build" debug for myself and will later be removed before release.

- W.r.t. "_In all your Edit / Remove Metadata classes is so much duplicated code. It may be good to move all the duplicated code into an abstract class..._"
I fully agree. I already mentioned that earlier in one of our conversations that it is on my Todo. (I also mentioned it in my [InterMediateChangelog](https://github.com/hvdwolf/jExifToolGUI/blob/master/IntermediateChangelog) in the repo).
Now I want to (think about) combining the several exif/xmp/etc. fields in enums as well.
I first quickly copied and paste it all and when finished, which is now after gpano (as I will remove iptc), I want to look it over. I know there is much the same same code, but also small differences. So I wanted to have it complete before building something "universal" for all, otherwise I might have to change my "universal" class/model again after having added the last "more or less the same" class.


Just to be sure as you changed the flow of this function here: Did you verify that these strings are still shown in the plugin after this change?


先解释下这里面的两个driver，分别指的是哪个，说清楚
# 返回当前页面的url
    def get_current_url(self):
        return self.driver.driver.current_url

    # 返回当前页面的title
    def get_current_title(self):
        return self.driver.driver.title

    # 获取当前窗口的handle
    def get_current_window_handle(self):
        return self.driver.driver.current_window_handles

    # 获取所有窗口的handle
    def get_windows_handles(self):
        return self.driver.driver.window_handles

    # 跳转到iframe页面， 可传参数name，id，如果iframe没有这个属性的话，可以用index或者element来定位
    def switch_to_iframe(self):
        self.driver.driver.switch_to.frame()


Demand*


They are set at the bottom, I guess I should move that up


I confirm that this fix works on 32bit ARM microcontrollers! 👍


Woops, sorry about that ...


good job，pkc


don't do that, name is indexed and inserting more than 2000(?) characters in an indexed field will traceback


Thanks for the note: https://github.com/lochmueller/staticfilecache/commit/cca6382c27eaf7bf0ed42b37679ec78d6b864767
Changed the typo in the message and als the variables name. I would keep the page['uid'] check to take care, that the page is valid and found.


That seems to be the case, just confirmed it by putting a value into one of empty ones.


Successfully aliased the URL <https://tryout-sdk-example-pv65u0xwh.now.sh> to the following aliases.

* https://tryout-sdk-example.hit2hat.now.sh
* https://tryout-sdk-example-git-master.hit2hat.now.sh


i would prefer you tu use description_picking


很感谢这么积极的反馈。不过这里有个代码上的建议

```js
href={`https://www.amap.com/search?query=${name}&city=510100`}
```

可以走外面的配置文件。这样这个项目也许能走向全国


Successfully aliased the URL <https://tryout-sdk-example-1qhx0h9g3.now.sh> to the following aliases.

* https://tryout-sdk-example.hit2hat.now.sh
* https://tryout-sdk-example-git-master.hit2hat.now.sh


/cc @tvlooy @Beryllium


Previous was UDHR.

https://en.wikipedia.org/wiki/Chickasaw_language confirms character list.


hello world!


this centered the images in the team section. why'd you remove??


Successfully aliased the URL <https://python-flask-jfoapc4nh.now.sh> to the following aliases.

* https://python-flask.maryosama.now.sh
* https://python-flask-git-master.maryosama.now.sh


Puzzle `1-41b37148` disappeared from [`src/main/java/com/g4s8/ghman/web/TkApp.java`](https://github.com/g4s8/ghman/blob/master/src/main/java/com/g4s8/ghman/web/TkApp.java), that's why I closed #5. Please, remember that the puzzle was not necessarily removed in this particular commit. Maybe it happened earlier, but we discovered this fact only now.


`BU-TS-09_trimmed_R2.fastq.gz` C'est une bonne idée, il faut juste que tu sois sûre que tu ne perdes pas d'information importante en coupant après le premier `_`.

Si tu continue sur cette idée, essaye d'éliminer les variables OUTPUTFILE1 et OUTPUTFILE2, elles ne sont pas très utiles.


python3, you're wild


>  the error is written to the logs anyways,

Ideally, yes. However I remember cases where a `ReferenceError` was thrown but the log was empty and the response didn't provide any clues.

+1 for `ret.errorStack`


yea, me too, guess the issue can be closed


Yes of course I did!


I think this method should be named `buildSsiRegex`.


Hey @trevor-scheer what package version is this? So I can bump my dependency in federation of apollo-graphql


:information_source: IBM Cloud [toolchain](https://console.bluemix.net/devops/toolchains/0d122658-46e5-4981-a505-de8c1bcf2060?env_id=ibm%3Ayp%3Aeu-de 'PA181'): [Delivery Pipeline](https://console.bluemix.net/devops/pipelines/34a6147f-ccb6-4a7e-9046-fa62588bd17f?env_id=ibm%3Ayp%3Aeu-de 'PA181') deployed [MUNI-FI-PA181](https://github.com/europ/MUNI-FI-PA181/releases/tag/deploy-ibm-yp-eu-de-PA181.org-PA181.space-20190419-211402 'deploy-ibm-yp-eu-de-PA181.org-PA181.space-20190419-211402') to [PA181.space](https://console.bluemix.net/apps/e12bc2b1-2120-4f57-b8e6-8351beb45553?env_id=ibm:yp:eu-de 'ibm:yp:eu-de:PA181.org:PA181.space'), including this commit :round_pushpin:


Successfully aliased the URL <https://musiikin-teoria-material-o8z2bky3i.now.sh> to the following aliases.

* https://musiikin-teoria.now.sh
* https://musiikin-teoria-material.mooc.now.sh
* https://musiikin-teoria-material-git-master.mooc.now.sh


これはマージして欲しくないファイルです。


bookmark.htm for ie11 adn bookmarks.html for chrome


jsx 안써도 될껄요?
`import App2 from './layout/App2';` 이렇게 하면 될텐데


これもマージの必要がないと思います。


Nikkie


Successfully aliased the URL <https://html-minifier-eb9u4cl7u.now.sh> to the following aliases.

* https://html-minifier.snehalbhalerao.now.sh
* https://html-minifier-git-master.snehalbhalerao.now.sh


# 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)|


Resolved in ba1d0180602b217e0b6f894581aeb2398f492188


Let me check it.


I believe it is important to distinguish one-time options from options you may change regularly - and keep them apart. For example, Mono-SFX may be selected depending on headphones or speakers. Brightmaps and Smooth Lighting may be selected just to see how it looks. But things like keyboard setup or music backend do not necessarily belong into the game itself, they are selected once for good - at least that's my opinion.


Unnecessary. get_services_state.yml has already the tag.


VLV/CHV microcode is now available in https://github.com/intel/Intel-Linux-Processor-Microcode-Data-Files
For other platforms, https://github.com/platomav/CPUMicrocodes has most of the microcode updates.


security


xóa bớt mấy line trống đi em


수정했습니다.


indent nè


indent :)


![](https://github.trello.services/images/mini-trello-icon.png) [6-3 잃어버린 유기견 등록 UI 구현](https://trello.com/c/E5pVTy2K/51-6-3-%EC%9E%83%EC%96%B4%EB%B2%84%EB%A6%B0-%EC%9C%A0%EA%B8%B0%EA%B2%AC-%EB%93%B1%EB%A1%9D-ui-%EA%B5%AC%ED%98%84)


commented


Deployment failed with the following error:

~~~
The chosen alias "cecil.now.sh" is already in use by another account.
~~~


Okay, done.


Successfully aliased the URL <https://operation-code-oq9glm2k3.now.sh> to the following aliases.

* https://operation-code.now.sh
* https://operation-code.operation-code.now.sh
* https://operation-code-git-master.operation-code.now.sh


Aligned has different data type option which is different from other derived classes. Hence overloaded `OutputAttribute`


@chumer I think this line is still in the latest version, but I don't know where the `#getLanguageReference(Class)` is pointing to. Where is it defined?


Successfully aliased the URL <https://operation-code-3oukektin.now.sh> to the following aliases.

* https://operation-code.now.sh
* https://operation-code.operation-code.now.sh
* https://operation-code-git-master.operation-code.now.sh


Looking into it. Recompiling to enable sockets and try it out locally. Technically speaking, from a UNIX perspective, anything below 1024 is supposed to throw this error - so it's weird that 80 and 443 are being so forgiving. Maybe there's a default setting somewhere that gives users permission to use them.


Hello @cyrille-artho ,
While looking into how choice generators gets initialized starting from the search function, I found out a method, setNextChoiceGenerator() in the class SystemState. I had implemented a new method setChoiceGenerator(), which didn't work out. Maybe we should use this function instead and add a ignore flag to the current CG so that it gets ignored. I'll see if this is possible and if it works.


@hhonkanen thank you for the comment. I pushed the modified change to #1677 adding the information to the log and card->name.