Page MenuHomeSoftware Heritage

Don't count origins len(url) times when calling origin_add_one
ClosedPublic

Authored by olasd on Wed, Mar 18, 6:18 PM.

Details

Summary

I guess the origins variable name was carried over from a refactoring, but it
doesn't match what db.origin_add actually returns. Overall this variable name
made us overcount origins a little.

Test Plan

yes, this should probably be tested

Diff Detail

Repository
rDSTO Storage manager
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

olasd created this revision.Wed, Mar 18, 6:18 PM
ardumont accepted this revision.Wed, Mar 18, 6:20 PM
ardumont added a subscriber: ardumont.

ouch, nice catch!

This revision is now accepted and ready to land.Wed, Mar 18, 6:20 PM
This revision was landed with ongoing or failed builds.Wed, Mar 18, 6:24 PM
This revision was automatically updated to reflect the committed changes.