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

Adding abillity to add and remove xrefs, refactoring xrefs #65

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

oded-el
Copy link

@oded-el oded-el commented Jun 30, 2017

Well, only when I finished this I noticed there's a pull request from January about xrefs.
I don't know what it did not merge, I refactored the xrefs a bit and added the ability to add and remove Xrefs.

@tmr232
Copy link
Owner

tmr232 commented Jul 11, 2017

Hi, sorry for the late response. I am somewhat busy with other things but will try to get to this PR soon.
In any case, I am a bit worried as it seems to be a breaking API change, and I have no measure of the number of people that will be affected by it.
I am not saying "no", but I need to give the idea of breaking changes consideration, and I have not done that yet.

@oded-el
Copy link
Author

oded-el commented Oct 5, 2018

Hi, is there any progression regarding API change?

@tmr232
Copy link
Owner

tmr232 commented Oct 6, 2018

Hi,

I have recently decided to start pushing for a new version of Sark, with a focus on IDA 7.x and allowing for breaking API changes.
As you can see, there are currently 2 PRs for xref-editing. I need to compare them and decide on the preferred method. I hope to get to this soon.

@oded-el
Copy link
Author

oded-el commented Jun 28, 2019

Is Sark still maintained? It feels like it's impossible to contribute

@tmr232
Copy link
Owner

tmr232 commented Sep 26, 2019

Sorry for the huge delay...
I was busy with other things, so didn't have any real attention to maintaining Sark for a while.
This is changing now. The current focus is on testing & migration to IDA 7.4 (dropping backwards compatibility with 6.95).
Once this is done, I'll start considering new features & improvements.

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