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

Refactor VisualNavbar to use API instead of p-j #2953

Merged
merged 1 commit into from
May 30, 2022
Merged

Conversation

thestr4ng3r
Copy link
Member

@thestr4ng3r thestr4ng3r commented May 29, 2022

Your checklist for this pull request

Detailed description

With the updated rizin, this also fixes any error prints saying
"Cannot alloc for this range" and json errors when no valid range is
available.

Partially addresses #2666

@XVilka XVilka requested a review from imbillow May 30, 2022 00:54
@thestr4ng3r thestr4ng3r marked this pull request as ready for review May 30, 2022 05:16
@thestr4ng3r thestr4ng3r marked this pull request as draft May 30, 2022 05:17
With the updated rizin, this also fixes any error prints saying
"Cannot alloc for this range" and json errors when no valid range is
available.
@thestr4ng3r thestr4ng3r marked this pull request as ready for review May 30, 2022 05:23
@thestr4ng3r thestr4ng3r merged commit 1914b34 into dev May 30, 2022
@thestr4ng3r thestr4ng3r deleted the navbar-api branch May 30, 2022 11:01
@ITAYC0HEN ITAYC0HEN mentioned this pull request May 31, 2022
3 tasks
wargio pushed a commit that referenced this pull request Jun 8, 2022
With the updated rizin, this also fixes any error prints saying
"Cannot alloc for this range" and json errors when no valid range is
available.
XVilka pushed a commit that referenced this pull request Jun 8, 2022
With the updated rizin, this also fixes any error prints saying
"Cannot alloc for this range" and json errors when no valid range is
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.

2 participants