[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #5351 Spacing_spanner::prune_loose_columns: prune in-place

[Auto mailings of changes to Lily Issues via Testlilyissues-auto]

• Description has changed:

Diff:

• Needs: -->
• Patch: new --> needs_work
• Comment:

Fails make..

make[1]: *** [out/spacing-determine-loose-columns.o] Error 1
make[1]: *** Waiting for unfinished jobs....
/home/james/lilypond-git/lily/spacing-loose-columns.cc: In function 'void set_loose_columns(System*, const Column_x_positions*)':
/home/james/lilypond-git/lily/spacing-loose-columns.cc:36:50: warning: conversion to 'int' from 'std::vector<Grob*>::size_type {aka long unsigned int}' may alter its value [-Wconversion]
   int loose_col_count = posns->loose_cols_.size ();
                                                  ^

---

[issues:#5351] Spacing_spanner::prune_loose_columns: prune in-place

Status: Started
Created: Tue Jun 19, 2018 10:58 PM UTC by David Kastrup
Last Updated: Tue Jun 19, 2018 10:58 PM UTC
Owner: David Kastrup

Spacing_spanner::prune_loose_columns: prune in-place

This saves a tiny amount of time and memory.

http://codereview.appspot.com/351720043

---

Sent from sourceforge.net because address@hidden is subscribed to https://sourceforge.net/p/testlilyissues/issues/

To unsubscribe from further messages, a project admin can change settings at https://sourceforge.net/p/testlilyissues/admin/issues/options. 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, Slashdot.org! http://sdm.link/slashdot

_______________________________________________
Testlilyissues-auto mailing list
address@hidden
https://lists.sourceforge.net/lists/listinfo/testlilyissues-auto