#27 CtagsSideKick: Make tag strings intern

closed
None
5
2008-07-06
2008-07-05
Kazutoshi Satoda
No

I encountered huge memory usage with large tags. This
small patch will avoid it. Such tags comes from a
machine generated header file which has 40000+ macro
definitions. In that case, this patch reduces 10M of
java heap.

I found this problem by using JProfiler. It is a great
tool.

Discussion

  • svn diff for plugins/CtagsSideKick/trunk r13006

     
    • assigned_to: nobody --> shlomy
    • status: open --> closed
     
  • Logged In: YES
    user_id=1477607
    Originator: NO

    Committed in SVN rev 13010.
    According to the documentation, using the "intern" method will look for an existing string in the internal string pool. Does this not trade performance for memory? i.e. will this patch not make CtagsSideKick slower?

     
  • Logged In: YES
    user_id=1483238
    Originator: YES

    Thank you for quick reply.

    Yes it does trade performance for memory. But I think it is very small
    part of whole parsing time, especially comparing the time consumed by
    the issue addressed in the patch #2011261.