DynamoRIO / DynamoRIO/dynamorio

re-organize include dir: app/ subdir?

Open
#1,524 0 comments 0 reactions 0 assignees View on GitHub
Component-API Migrated Priority-Medium Type-Feature Usability
Dominant language
C
Stars
3.2k
Forks
630
Avg merge
2d 13h
Merged PRs (30d)
31

Description

_From [bruen...@google.com](https://code.google.com/u/109494838902877177630/) on August 22, 2014 17:31:46_

With the addition of the annotation headers (3 for app, 1 for client) we may want to add some heirarchy to the include/ dir. One suggestion is to have the 3 app annotation headers (currently in include/annotations/) and dr_app.h together in include/app/.

We may also want to rename dr_annotation.h vs dr_annotations.h?

_Original issue: http://code.google.com/p/dynamorio/issues/detail?id=1524_

Contributor guide

Open the contributing guide

Research direction

Start by reviewing the headers in include/annotations/ and dr_app.h, then search the repository for references to their current paths and to dr_annotation.h and dr_annotations.h. Done means agreeing on the include/app/ hierarchy and any header rename, updating all affected references, and confirming the project still builds.

Written by the indexing model from the issue text.

Assessment

Tech stack
c
Domain
devtools
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.