Skip to content

Fix issue #237 - Incorrect usage of return value in flash_file method#263

Merged
hkpeprah merged 2 commits into
square:masterfrom
TheColector:fix/ISSUE-237/fix_flash_file_method_return
Mar 13, 2026
Merged

Fix issue #237 - Incorrect usage of return value in flash_file method#263
hkpeprah merged 2 commits into
square:masterfrom
TheColector:fix/ISSUE-237/fix_flash_file_method_return

Conversation

@TheColector

Copy link
Copy Markdown
Contributor

This PR fixes the issue mentioned by me in #237

JLINK_DownloadFile from JLINK's DLL does not return number of bytes flashed.
It returns error code
@CLAassistant

CLAassistant commented Mar 10, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@TheColector TheColector changed the title Fix/issue 237/fix flash file method return Fix issue #237 - Incorrect usage of return value in flash_file method Mar 10, 2026
@hkpeprah

Copy link
Copy Markdown
Contributor

LGTM. You just have to sign the CLA for us to accept this change though.

@TheColector

Copy link
Copy Markdown
Contributor Author

It's already been signed on my end.
I figure it got stuck somehow as some people described the same issue had happened to them as well.
I'll try to push empty commit to retrigger checks

@TheColector
TheColector force-pushed the fix/ISSUE-237/fix_flash_file_method_return branch from ffa6506 to 8900a85 Compare March 12, 2026 10:12
@hkpeprah hkpeprah closed this Mar 13, 2026
@hkpeprah hkpeprah reopened this Mar 13, 2026
@hkpeprah
hkpeprah self-requested a review March 13, 2026 16:39

@hkpeprah hkpeprah 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.

Approved. Thank you! 👍

@hkpeprah
hkpeprah merged commit a2172aa into square:master Mar 13, 2026
6 checks passed
@hkpeprah

Copy link
Copy Markdown
Contributor

Will release this with the other change for the exec_command when its available.

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.

3 participants