[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #5366 Move war

From: Auto mailings of changes to Lily Issues via Testlilyissues-auto
Subject: [Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #5366 Move warnings out of find/create context functions
Date: Sat, 07 Jul 2018 20:01:34 -0000

commit 51b6513eeeaea69293bd4f554f8021529ae85a49 (HEAD -> pushed/issue-5366-context-warning, origin/staging)
Author: Dan Eble address@hidden
Date: Mon Jul 2 13:36:48 2018 -0400

Issue 5366: Move warnings out of find/create context functions


[issues:#5366] Move warnings out of find/create context functions

Status: Fixed
Labels: Fixed_2_21_0
Created: Mon Jul 02, 2018 11:26 PM UTC by Dan Eble
Last Updated: Sat Jul 07, 2018 07:12 AM UTC
Owner: Dan Eble

Move warnings out of find/create context functions

The motivation for this is that Context::find_create_context () and
find_context_near () should probably be merged for maintainability,
but one of the differences between them that must be dealt with is
that find_create_context () logs when it fails and find_context_near
() does not. Adding warnings to find_context_near () risks being too
noisy, leaving the option taken here.

The new method Context::diagnostic_id (name, id) returns a formatted
string (e.g. "Voice" or "Voice = mel") for use in a log message. It
is used for the warnings that are being moved as well as few other
existing warnings to increase consistency.

Sent from because address@hidden is subscribed to

To unsubscribe from further messages, a project admin can change settings at Or, if this is a mailing list, you can unsubscribe from the mailing list.

Check out the vibrant tech community on one of the world's most
engaging tech sites,!
Testlilyissues-auto mailing list

reply via email to

[Prev in Thread] Current Thread [Next in Thread]