New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 645541 link

Starred by 1 user

Issue metadata

Status: Archived
Owner: ----
Closed: Oct 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Feature

Blocked on:
issue 644372



Sign in to add a comment

[Findit] Make FlattenDepTree non-recursive?

Project Member Reported by wrengr@chromium.org, Sep 9 2016

Issue description

Where: infra/appengine/findit/common/chromium_deps.py

In GetChromeDependency we locally define a recursive function FlattenDepTree. It looks like DEPS never gets very deep, so we don't have to worry about stack overflow issues; but, there might still be performance benefits to refactoring this function to be non-recursive (i.e., to build up a work queue as needed).

If this ever looks to be a bottleneck, or if someone's just looking for something to do, this might be worth exploring.
 
Blockedon: 615262 644372
Summary: [Findit] Make FlattenDepTree non-recursive? (was: Make FlattenDepTree non-recursive?)
Status: Archived (was: Untriaged)
Blockedon: -615262

Sign in to add a comment