[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Lilypond-auto] Issue 3832 in lilypond: Assertion when the version s
From: |
lilypond |
Subject: |
Re: [Lilypond-auto] Issue 3832 in lilypond: Assertion when the version string contains more than dots and numbers. |
Date: |
Wed, 03 Sep 2014 17:35:30 +0000 |
Updates:
Status: Started
Owner: address@hidden
Labels: Patch-new
Comment #1 on issue 3832 by address@hidden: Assertion when the version string
contains more than dots and numbers.
https://code.google.com/p/lilypond/issues/detail?id=3832#c1
Issue 3832: react sanely to ill-formed version strings
Contains commits
Clean up Lilypond_version
Converting to int seems strange, but we want version comparisons.
Conversion to bool for checking validity seems nice.
And bombing out because of invalid data to String_convert::dec2int
seemed definitely non-nice. This version requires specifying at
least major and minor version number explicitly separated by dots.
It accepts a trailing dot, too. A trailing extra_patch_string after
major.minor.patchlevel. is also accepted but not interpreted.
Don't verify result of String_convert::dec2int
The error behavior is not useful and triggers on things like "00".
http://codereview.appspot.com/135380043
--
You received this message because this project is configured to send all
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings
- Re: [Lilypond-auto] Issue 3832 in lilypond: Assertion when the version string contains more than dots and numbers.,
lilypond <=