diff options
author | Philipp Wollermann <philwo@google.com> | 2018-03-23 07:39:27 -0700 |
---|---|---|
committer | Copybara-Service <copybara-piper@google.com> | 2018-03-23 07:40:39 -0700 |
commit | 23e1c5d8d267e5825552ce5b05ddfb8ae8972688 (patch) | |
tree | c8427eb39a03d0c586f24aaed2cce6cc58065423 /WORKSPACE | |
parent | d58bd26ea754a189c87ef0af795998acd2ad2874 (diff) |
Refactor and cleanup the sandboxing code.
- Remove Optional<> where it's not needed. It's nice for return values, but IMHO it was overused in this code (e.g. Optional<List<X>> is an anti-pattern, as the list itself can already signal that it is empty).
- Use Bazel's own Path class when dealing with paths, not String or java.io.File.
- Move LinuxSandboxUtil into the "sandbox" package.
- Remove dead code and unused fields.
- Migrate deprecated VFS method calls to their replacements.
- Fix a bug in ExecutionStatistics where a FileInputStream was not closed.
Closes #4868.
PiperOrigin-RevId: 190217476
Diffstat (limited to 'WORKSPACE')
0 files changed, 0 insertions, 0 deletions