This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [patch] only update dcache after write succeeds


Pedro Alves wrote:
On Monday 14 September 2009 21:20:38, Michael Snyder wrote:

IOW, if some target method does return > 0, then the write succeeded, right?
Are there different kinds of "success" in effect here?
Well, maybe only in our case. ;-)

If nobody else has any worries about it, I'm OK with it.

----
* In our case (process record), it's a bad thing for the target
beneath to be called after the user has said "no".

The user is saying "no" to the whole high level operation, not to a single partial transfer. In that case, you shouldn't even attempt to partial xfer in the target beneath, I would say. A thrown error for those cases looks like the way to go. Perhaps even better would be to be able to foretell if the transfer is going to be problematic and warn/query upfront, but it's hard in some cases.

Agree, "error()" is the way to go here.





Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]