Skip to content

【igl nanovg part-4】Android : add seperate nanovg sample project#218

Closed
vinsentli wants to merge 12 commits into
facebook:mainfrom
vinsentli:igl_nanovg_part2_android_copy_assert_file_to_sdcard
Closed

【igl nanovg part-4】Android : add seperate nanovg sample project#218
vinsentli wants to merge 12 commits into
facebook:mainfrom
vinsentli:igl_nanovg_part2_android_copy_assert_file_to_sdcard

Conversation

@vinsentli

@vinsentli vinsentli commented Nov 27, 2024

Copy link
Copy Markdown
Contributor

This is a prerequisite pull request for igl nanovg(#213).

@vinsentli vinsentli changed the title android : copy assets file to '/data/data/com.facebook.igl.shell/files/' directory. Nov 27, 2024
@corporateshark corporateshark requested a review from rokuz November 27, 2024 19:28

@corporateshark corporateshark left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There should be a separate NanoVG sample which does this. We cannot change SampleActivity.java like that.

@vinsentli

vinsentli commented Nov 29, 2024

Copy link
Copy Markdown
Contributor Author

This is a general operation,not only for nanovg sample.
So we can use fopen() to access file,instead of only restricted by using Assets_open() on android.

@corporateshark

Copy link
Copy Markdown
Contributor

This is a general operation,not only for nanovg sample. So we can use fopen() to access file,instead of only restricted by using Assets_open() on android.

All existing samples do not require this behavior and we do not want to touch the SD card like that.

@vinsentli vinsentli marked this pull request as draft December 16, 2024 04:05
@vinsentli vinsentli changed the title 【igl nanovg part-2】android : copy assets file to '/data/data/com.facebook.igl.shell/files/' directory. Dec 18, 2024
@vinsentli vinsentli changed the title 【igl nanovg part-4】android : copy assets file to '/data/data/com.facebook.igl.shell/files/' directory. Dec 22, 2024
@vinsentli vinsentli changed the title 【igl nanovg part-1】Android : Add NanovgSample project Dec 22, 2024
@vinsentli vinsentli marked this pull request as ready for review December 22, 2024 13:41
Comment thread build/android/app/src/main/java/com/facebook/igl/sample/SampleActivity.java Outdated
@vinsentli vinsentli changed the title 【igl nanovg part-1】Android : add seperate nanovg sample project Jan 6, 2025
@corporateshark

Copy link
Copy Markdown
Contributor
@vinsentli vinsentli deleted the igl_nanovg_part2_android_copy_assert_file_to_sdcard branch June 15, 2026 01:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

3 participants