Skip to content

Commit 824a5ff

Browse files
ContainerFindIncludes: Add some temporary variables
This slightly cleans up a function call. Signed-off-by: Matthijs Kooijman <[email protected]>
1 parent 3a4083c commit 824a5ff

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

Diff for: src/arduino.cc/builder/container_find_includes.go

+3-1
Original file line numberDiff line numberDiff line change
@@ -289,6 +289,8 @@ func writeCache(cache *includeCache, path string) error {
289289

290290
func findIncludesUntilDone(ctx *types.Context, cache *includeCache, sourceFile types.SourceFile) error {
291291
sourcePath := sourceFile.SourcePath(ctx)
292+
depPath := sourceFile.DepfilePath(ctx)
293+
objPath := sourceFile.ObjectPath(ctx)
292294
targetFilePath := utils.NULLFile()
293295

294296
// TODO: This should perhaps also compare against the
@@ -303,7 +305,7 @@ func findIncludesUntilDone(ctx *types.Context, cache *includeCache, sourceFile t
303305
// TODO: This reads the dependency file, but the actual building
304306
// does it again. Should the result be somehow cached? Perhaps
305307
// remove the object file if it is found to be stale?
306-
unchanged, err := builder_utils.ObjFileIsUpToDate(sourcePath, sourceFile.ObjectPath(ctx), sourceFile.DepfilePath(ctx), ctx.DebugLevel, ctx.GetLogger())
308+
unchanged, err := builder_utils.ObjFileIsUpToDate(sourcePath, objPath, depPath, ctx.DebugLevel, ctx.GetLogger())
307309
if err != nil {
308310
return i18n.WrapError(err)
309311
}

0 commit comments

Comments
 (0)