This is the mail archive of the archer@sourceware.org mailing list for the Archer 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: [python] Implement gdb.search_memory.


>>>>> "Thiago" == Thiago Jung Bauermann <bauerman@br.ibm.com> writes:

Thiago> This is not the issue here, it seems. But do we want to enforce
Thiago> backward/forward compatibility for each pushed commit? 

No, a best effort is enough.  These sorts of bugs tend to be trivial
to fix; it isn't worth that amount of overhead.  Well, unless perhaps
someone wants to automate it for us...


I do have a couple follow-up suggestions.

A few functions in python-membuf.c are not static but probably should
be.  E.g., get_read_buffer.  (Or, if you intend to export that, give
it a less generic name.)

Also, mbpy_str still casts a CORE_ADDR to a void*, which is bad.
Daniel suggested using paddr or paddr_nz here.

Tom


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