Update class names in DUDhain API dox
ClosedPublic

Authored by kossebau on Jul 13 2017, 2:29 PM.

Diff Detail

Repository
R33 KDevPlatform
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
kossebau created this revision.Jul 13 2017, 2:29 PM
Restricted Application added a subscriber: kdevelop-devel. · View Herald TranscriptJul 13 2017, 2:29 PM

I just found KDevelop::Definition no longer existing, and KDevelop::PersistentSymbolTable seems to be what KDevelop::SymbolTable used to be?

Needs input from someone with DUChain (history) knowledge.

See current docs rendered at https://api.kde.org/extragear-api/kdevelop-apidocs/kdevplatform/language/duchain/html/duchain-design.html

kfunk added a subscriber: kfunk.Jul 13 2017, 7:48 PM

Yes re. the switch from KDevelop::SymbolTable to KDevelop::PersistentSymbolTable:

commit a40d774da71d763cc37cf695c0742a3c746364c9
Author: David Nolden <david.nolden.kde@art-master.de>
Date:   Wed Sep 3 20:29:05 2008 +0000

    Completely remove the runtime symbol-table hash, in favor of the persistent on-disk symbol table.
    This will allow not creating the actual Declaration and DUContext objects while loading, but just creating them on-demand, thus greatly speed up the loading of top-contexts.
    But already now it speeds up loading/unloading, because the symbol-table doesn't need to be updated.
    The downside is that the persistent symbol table contains ALL declarations, and thus more declarations need to be checked while searching.
    
    - Try stopping the too fast growth of the persistent symbol table
    - Add a persistent context table, and rename the previous symbol table to declaration table.
    - Port all the uses of SymbolTable to use PersistentSymbolTable.
kfunk accepted this revision.Jul 13 2017, 7:49 PM

LGTM, thanks for taking care of the docs!

This revision is now accepted and ready to land.Jul 13 2017, 7:49 PM
This revision was automatically updated to reflect the committed changes.