Project

General

Profile

Actions

Architecture #2279

closed

Shoud we use the TreeWalk.release() method after walking the JGit tree ?

Added by Nicolas CHARLES about 12 years ago. Updated over 3 years ago.

Status:
Rejected
Priority:
4
Assignee:
-
Category:
Architecture - Code maintenance
Effort required:
Name check:
Fix check:
Regression:

Description

The documentation is a bit unclear about the subject; it states clearly that RevWalk should be released after use ( http://wiki.eclipse.org/JGit/User_Guide ) but doesn't state the same for the TreeWalk (actually, it says nothing)
However, there is a release method in this class, so it is interesting to see if it should be invoked

Actions #1

Updated by Jonathan CLARKE about 12 years ago

  • Category changed from 13 to Architecture - Code maintenance
  • Assignee changed from Jonathan CLARKE to François ARMAND
Actions #2

Updated by Jonathan CLARKE almost 12 years ago

  • Target version changed from 2.4.0~alpha6 to 2.4.0~alpha7
Actions #3

Updated by François ARMAND almost 12 years ago

  • Assignee deleted (François ARMAND)
  • Target version changed from 2.4.0~alpha7 to 24

Not the kind of code modification we want to handle know, I thing.
I does not seems to leak as it is, so perhaps we shall think to it again in 2.5 ?

Actions #4

Updated by Jonathan CLARKE almost 12 years ago

  • Target version changed from 24 to Ideas (not version specific)
Actions #5

Updated by Nicolas CHARLES over 3 years ago

  • Status changed from New to Rejected

I guess it can be safely rejected: it's been working for 8 years, should still be ok

Actions

Also available in: Atom PDF