Re: Patch: implement new dynamic varobj spec

Tom Tromey
2009-09-14T20:03:16Z

>>>>> "Eli" == Eli Zaretskii <eliz@gnu.org> writes:

Tom> +Note that if Python support has not been compiled into @value{GDBN},
Tom> +this command will still succeed.

Eli> Suggest to add "(and do nothing)" to the end of the sentence
Eli> Btw, does this silent failure have good reasons?

There just didn't seem to be much value in issuing an error.

Tom> +This operation returns attributes of the newly-created varobj. These
Tom> +include, but are not limited to:

Eli> Why don't we have an exhaustive list here?

We may add attributes later.

Tom> +@item thread-id
Tom> +If a fixed variable object is bound to a specific thread, then this is
Tom> +the thread's identifier.

Eli> Is "fixed" used here as opposed to "dynamic"?

I don't know where that came from, I removed the "fixed".

Tom> +For a dynamic varobj, this value cannot reliably be used to form an
Tom> +expression. Also, a dynamic varobj will not report the access
Tom> +qualifying pseudo-children, regardless of the language.

Eli> Do we want to give the reader some hints as to how to achieve these
Eli> with dynamic varobjs?

There is no way to do these.

Tom