Decl def #26

Merged
merged 4 commits into from May 4, 2012

Conversation

Projects
None yet
2 participants
@garnacho
Contributor

garnacho commented May 4, 2012

This branch fixes declaration/definition switching

  • On the information popup, links to both locations are shown
  • On the left hand sidebar, links invariably point to the definition

Carlos Garnacho added some commits Apr 30, 2012

Carlos Garnacho
dxr: Set extents on definitions
The .csv files will now contain extents information about
function definitions, which allows for a more direct storage
of the declaration/definition map
Carlos Garnacho
dxr: Fill in the decldef table directly from the the csv data
Instead of storing in memory and then iterating through all the
data, store directly in the database and then match the defid at
a later stage.
Carlos Garnacho
www: Fix declaration/definition switching on the information popup
The information popup now offers information about both where functions
are declared and defined, allowing switching between those.
Carlos Garnacho
dxr: make left hand sidebar point to the function declaration
This is most noticeable in header files, where items in the left
hand list will link to the method implementation in the .cpp file,
which is a more intuitive way to read through code.

https://bugzilla.mozilla.org/show_bug.cgi?id=751525

ehsan added a commit that referenced this pull request May 4, 2012

@ehsan ehsan merged commit 3efd33d into mozilla:master May 4, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment