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

get_class_var_byteの試験関数であるget_class_var_byte_nativeを追加。 #416

Closed
wants to merge 5 commits into from

Conversation

ryun0suke22
Copy link
Collaborator

@ryun0suke22 ryun0suke22 commented Oct 29, 2023

get_class_var_byteの試験関数であるget_class_var_byte_nativeを追加。

試験の流れ... SPVM側でbyte型のクラス変数を設定する。C言語側のget_class_var_byteで正しく値が取れる.

@ryun0suke22 ryun0suke22 changed the title get_class_var_byteの試験関数であるget_class_var_testを追加。 get_class_var_byteの試験関数であるget_class_var_byte_nativeを追加。 Oct 30, 2023
@yuki-kimoto
Copy link
Owner

ほぼ大丈夫です。1点だけ。

get_class_var_byte_nativeの中で、試験コードを完結させてみてください。

以下のコードをget_class_var_byte_nativeに移動させて、

aad9663#diff-ea914896880cd4bd26b253df1a55e8cd934c249a30237b380d94d1f4ae69233fR66-R68

my $success = TestCase::NativeAPI->get_class_var_byte_native;

unless ($success) {
  return 0;
}

となるようにしてみてください。

@ryun0suke22
Copy link
Collaborator Author

@yuki-kimoto
修正を追加いたしました。再度ご確認お願いいたします。

8566386

@yuki-kimoto
Copy link
Owner

ありがとうございます。ばっちりです。マージします。

@yuki-kimoto
Copy link
Owner

Merged

@yuki-kimoto yuki-kimoto closed this Jan 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants