Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add ctkErrorLogAbstractModel to CTKCore library #864

Merged
merged 1 commit into from
Apr 24, 2019

Conversation

jcfr
Copy link
Member

@jcfr jcfr commented Apr 24, 2019

This commit was motivated to support use of the log model from
qSlicerCoreApplication. This is needed to ensure messages are appended
to the log file during the unloading of modules following the call to
"factoryManager()->unloadModules()" from within the qSlicerCoreApplicationPrivate
destructor.

It is a follow up of f49c19a (move Widget dependent classes from Core
to Widgets. See #278)

This commit was motivated to support use of the log model from
qSlicerCoreApplication. This is needed to ensure messages are appended
to the log file during the unloading of modules following the call to
"factoryManager()->unloadModules()" from within the qSlicerCoreApplicationPrivate
destructor.

It is a follow up of f49c19a (move Widget dependent classes from Core
to Widgets. See #278)
@jcfr jcfr force-pushed the add-ErrorLogAbstractModel branch from 666b19d to 4348664 Compare April 24, 2019 05:57
@jcfr jcfr merged commit 8679f32 into master Apr 24, 2019
@jcfr jcfr deleted the add-ErrorLogAbstractModel branch April 24, 2019 06:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant