Page MenuHomePhabricator

Clean up lldb-types.h

Authored by labath on Jul 7 2017, 2:35 AM.



It defined a couple of types (condition_t) which we don't use anymore,
as we have c++11 goodies now. I remove these definitions.

Also it unnecessarily included a couple of headers which weren't
necessary for it's operation. I remove these, and place the includes in
the relevant files (usually .cpp, usually in Host code) which use them.
This allows us to reduce namespace pollution in most of the lldb files
which don't need the OS-specific definitions.

Diff Detail


Event Timeline

labath created this revision.Jul 7 2017, 2:35 AM
krytarowski added inline comments.
18 ↗(On Diff #105603)

Is this needed here?

labath added inline comments.Jul 7 2017, 3:29 AM
18 ↗(On Diff #105603)

good point. I'm pretty sure it isn't.

labath updated this revision to Diff 105613.Jul 7 2017, 3:29 AM

Remove unused include from auxvector.h

Eugene.Zelenko added inline comments.
16 ↗(On Diff #105613)

I think will be good idea to include csignal instead. Same in other files. See

16 ↗(On Diff #105613)

Good idea. I wasn't aware <csignal> existed.

labath updated this revision to Diff 105980.Jul 11 2017, 2:39 AM

Use csignal instead of signal.h

This revision was automatically updated to reflect the committed changes.