After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 691222 - Add "document-[added|removed]" signals to IAnjutaDocumentManager.
Add "document-[added|removed]" signals to IAnjutaDocumentManager.
Status: RESOLVED FIXED
Product: anjuta
Classification: Applications
Component: plugins: document-manager
unspecified
Other All
: Normal normal
: ---
Assigned To: Anjuta maintainers
Anjuta maintainers
Depends on:
Blocks:
 
 
Reported: 2013-01-06 11:26 UTC by Carl-Anton Ingmarsson
Modified: 2013-01-06 18:29 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
libanjuta: Add "document-[added|removed]" signals to IAnjutaDocumentManager. (1.86 KB, patch)
2013-01-06 11:26 UTC, Carl-Anton Ingmarsson 		accepted-commit_now Details | Review
document-manager: Emit "document-[added|removed]" signals. (3.31 KB, patch)
2013-01-06 11:26 UTC, Carl-Anton Ingmarsson 		accepted-commit_now Details | Review
debug-manager: use IAnjutaDocumentManager::document-added to add breakpoints to editor. (3.69 KB, patch)
2013-01-06 11:26 UTC, Carl-Anton Ingmarsson 		accepted-commit_now Details | Review

Description Carl-Anton Ingmarsson 2013-01-06 11:26:32 UTC 
See attached patches.
Comment 1 Carl-Anton Ingmarsson 2013-01-06 11:26:34 UTC 
Created attachment 232854 [details] [review]
libanjuta: Add "document-[added|removed]" signals to IAnjutaDocumentManager.
Comment 2 Carl-Anton Ingmarsson 2013-01-06 11:26:37 UTC 
Created attachment 232855 [details] [review]
document-manager: Emit "document-[added|removed]" signals.
Comment 3 Carl-Anton Ingmarsson 2013-01-06 11:26:40 UTC 
Created attachment 232856 [details] [review]
debug-manager: use IAnjutaDocumentManager::document-added to add breakpoints to editor.

Instead of listening to the currently active document using
IANJUTA_DOCUMENT_MANAGER_CURRENT_DOCUMENT.
Comment 4 Sébastien Granjoux 2013-01-06 16:20:03 UTC 
I think we have already discussed this quite some time ago. It hasn't been done because the use of the current document shell value is working too.

Anyway, I think it could make things a bit easier later. I'm agree with these changes, you can commit them.