Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

불필요한 async 삭제 #2665

Closed
Navi-Labs opened this issue Feb 26, 2024 · 1 comment
Closed

불필요한 async 삭제 #2665

Navi-Labs opened this issue Feb 26, 2024 · 1 comment
Assignees
Labels
CS 외부에서의 처리 요청 반영을 위한 PR enhancement 현행상 문제는 없으나 코드 개선을 위한 PR

Comments

@Navi-Labs
Copy link

async func(sprite, script) {
const red = script.getNumberValue('RED', script);
const greeb = script.getNumberValue('GREEN', script);
const blue = script.getNumberValue('BLUE', script);
return Entry.rgb2hex(red, greeb, blue);
},

async func(sprite, script) {
const color = script.getField('COLOR', script);
const value = script.getValue('HEX', script);
return Entry.hex2rgb(value)[color];
},

두 블록 change_rgb_to_hex, change_hex_to_rgb에서
await 구문을 쓰지 않음에도 함수가 async로 정의됨
promise가 리턴되어 executor에서 1틱씩 딜레이가 발생함

@kimorkim kimorkim self-assigned this Mar 19, 2024
@kimorkim kimorkim added enhancement 현행상 문제는 없으나 코드 개선을 위한 PR CS 외부에서의 처리 요청 반영을 위한 PR labels Mar 19, 2024
@kimorkim
Copy link
Collaborator

@Navi-Labs
제보 감사합니다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CS 외부에서의 처리 요청 반영을 위한 PR enhancement 현행상 문제는 없으나 코드 개선을 위한 PR
Projects
None yet
Development

No branches or pull requests

2 participants