JusticeRage / Gepetto

IDA plugin which queries uses language models to speed up reverse-engineering
GNU General Public License v3.0
2.87k stars 263 forks source link

Append response to the existing comment (if exist) #16

Closed neryaz closed 1 year ago

JusticeRage commented 1 year ago

Hi! Thanks for this contribution! I don't think I will merge it, because my experience with this plugin is that I ask for a new comment when the previous one is too vague or incorrect. In those cases, I don't think it makes sense to append the new comment, and it may be better to replace whatever used to be there.

neryaz commented 1 year ago

So it is possible to create some mark for the beginning and end of a comment that Gepetto entered and replace only it. The purpose of this PR is in case the user wrote some comment on the function, regardless of Gepetto, I think it's better not to delete the user's original comments.

JusticeRage commented 1 year ago

That makes sense! I'll add this feature when I have a few free cycles :)