Skip to content
This repository has been archived by the owner on Jul 13, 2019. It is now read-only.

Audit all code paths for allocations #19

Open
io7m opened this issue Dec 28, 2015 · 0 comments
Open

Audit all code paths for allocations #19

io7m opened this issue Dec 28, 2015 · 0 comments
Assignees

Comments

@io7m
Copy link
Owner

io7m commented Dec 28, 2015

Most of the time, this is going to be down to boxing in log messages. They can be avoided by wrapping log statements in conditionals, so that allocation doesn't occur when the messages won't actually be logged.

if (LOG.isDebugEnabled()) {
  LOG.debug("allocated {}", Long.valueOf(id));
}
@io7m io7m self-assigned this Dec 28, 2015
io7m added a commit that referenced this issue Dec 29, 2015
io7m added a commit that referenced this issue Dec 29, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant