Skip to content

Fix automatic addition of TEfficiency to current directory#9265

Merged
lmoneta merged 1 commit intoroot-project:masterfrom
lmoneta:tefficiency_fix_ctor
Nov 12, 2021
Merged

Fix automatic addition of TEfficiency to current directory#9265
lmoneta merged 1 commit intoroot-project:masterfrom
lmoneta:tefficiency_fix_ctor

Conversation

@lmoneta
Copy link
Copy Markdown
Member

@lmoneta lmoneta commented Nov 9, 2021

Fix using TDirectory::TContext in TEfficiency constructor

This PR fixes #9189

@phsft-bot
Copy link
Copy Markdown

Starting build on ROOT-debian10-i386/cxx14, ROOT-performance-centos8-multicore/default, ROOT-ubuntu16/nortcxxmod, ROOT-ubuntu2004/soversion, mac1014/python3, mac11.0/cxx17, windows10/cxx14
How to customize builds

@lmoneta lmoneta requested a review from pcanal November 9, 2021 17:10
@lmoneta lmoneta self-assigned this Nov 9, 2021
@phsft-bot
Copy link
Copy Markdown

Build failed on mac1014/python3.
See console output.

}
Build(name,title);
}
}
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
}
}

…al histograms to the current directory use a separate scope to avoid side-effects

such as removing addition of TEfficiency to current directory.
See issue root-project#9189
@lmoneta lmoneta force-pushed the tefficiency_fix_ctor branch from 6a93cd9 to b80b5ec Compare November 9, 2021 17:40
@phsft-bot
Copy link
Copy Markdown

Starting build on ROOT-debian10-i386/cxx14, ROOT-performance-centos8-multicore/default, ROOT-ubuntu16/nortcxxmod, ROOT-ubuntu2004/soversion, mac1014/python3, mac11.0/cxx17, windows10/cxx14
How to customize builds

@phsft-bot
Copy link
Copy Markdown

Build failed on mac1014/python3.
See console output.

@lmoneta lmoneta merged commit 004b6a7 into root-project:master Nov 12, 2021
@lmoneta lmoneta deleted the tefficiency_fix_ctor branch November 12, 2021 08:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

TEfficiency constructors not appended to current directory

3 participants