Load and save watch file in 12.11

[euzkoarima]

Hi, the save/load watch file is implemented on 12.11 version ?
I'm trying using it but right click on watches panel do nothing

[oBFusCATed]

No.

[euzkoarima]

Thanks for your answer, is there any plan to include this feature ?

[MortenMacFly]

Thanks for your answer, is there any plan to include this feature ?

I think this would be useful, so yes.

There was a patch in the bug tracker some time ago that did this...

[euzkoarima]

I think this would be useful, so yes.

There was a patch in the bug tracker some time ago that did this...

Great, that's good news !!

[oBFusCATed]

There was a patch in the bug tracker some time ago that did this...

I doubt it will do the job, because there is no serialization API for the watches.

What are the requirements for such a feature?

[MortenMacFly]

What are the requirements for such a feature?

Save what needs to be saved for a BP, replay "adding" BPs' when loading the file. What else?

It was implemented really simple before... saving a few information to a text file... and it was one of my first contributions to C::B, IIRC... ;-)

[oBFusCATed]

Yes, but now there is no single Watch class, but every plugin implements its own version and can add any property it feels necessary.
Do we want to restore the properties? Do we want to load all watches to the current active plugin or do we want to load them as they were.
It is not simple as you can see.

[MortenMacFly]

It is not simple as you can see.

Well... there is one simple interface to GDB how you add watches. Thats all I want to save, nothing more, nothing less. It should be a simple as it was before.

[Jenna]

If every plugin implements it's own version, the plugins should be able to have an own function to save the watches.

[oBFusCATed]

Well... there is one simple interface to GDB how you add watches. Thats all I want to save, nothing more, nothing less. It should be a simple as it was before.

You forget that we now have a generic debugger interface. What about the python dbg plugins or gdb/mi?

If every plugin implements it's own version, the plugins should be able to have an own function to save the watches.

Yes, this is the direction I'll probably take, but not now. Probably you should ping me about this is a month or two

[MortenMacFly]

Yes, this is the direction I'll probably take, but not now. Probably you should ping me about this is a month or two

Yes, thats true and thats also what I meant. Of course I am only talking about the one gdb plugin we have in SVN, nothing else. It had this functionality before and it seems it was useful. Especially with locals gone.

Maybe I wasn't clear enough in the first place - I am not looking for something generic nor an extension to the core, just to the debugger plugin we provide atm. Let me have a look if I find the patch again - maybe its very simple then...

[oBFusCATed]

As you know I don't like non-generic solutions... The patch probably have been provided by Ollydbg.

[MortenMacFly]

As you know I don't like non-generic solutions...

If every plugin implements it's own version, the plugins should be able to have an own function to save the watches.

Yes, this is the direction I'll probably take, but not now.

Hmmm.. I don't get it:

Whats the difference between what I propose and you? The patch does exactly that: Save watches for this very debugger plugin we distribute. ???

[oBFusCATed]

Whats the difference between what I propose and you? The patch does exactly that: Save watches for this very debugger plugin we distribute. ???

What patch? Link?