Welcome Guest! To enable all features please Login or Register.

Notification

Icon
Error
Unable to parse the configuration setting Text output font (after upgrade to v3)
Options
Previous Topic Next Topic
abelb
#1 Posted : Wednesday, December 14, 2016 4:13:13 PM(UTC)
Rank: Advanced Member

Groups: Registered
Joined: 9/12/2014(UTC)
Posts: 155
Location: Netherlands

Thanks: 19 times
Was thanked: 11 time(s) in 11 post(s)
After I upgraded my 2.x to the current 3.x I first got the "first time" wizard again and after it finished, the NCrunch Tests window shows:

Quote:
The setting 'TextOutputFont' as declared in 'R:\AppData\NCrunch\globalconfig.crunch.v3.xml' could not be read by NCrunch. The default value will be used instead


Nevertheless, it "just runs" the tests after build. I do see a different font in my output window (I remember to have changed it previously because the default cannot display underscores).

It's a minor error and I can probably fix it easily, but I thought I report it anyway as it may be the result of trying to convert the old config files to the new XML format. The relevant line in the XML looks as follows:

Code:
<TextOutputFont>Source Code Pro, 8,25pt, 0</TextOutputFont>


And as an aside: I like the new format a lot! The new configuration system, esp. the Advanced setting, is awesome!!!
Back to top
 |  Edit by user
Remco
#2 Posted : Wednesday, December 14, 2016 11:17:23 PM(UTC)
Rank: NCrunch Developer

Groups: Administrators
Joined: 4/16/2011(UTC)
Posts: 7,145

Thanks: 960 times
Was thanked: 1290 time(s) in 1196 post(s)
Hi,

Thanks for sharing this problem. This actually looks to be a localization issue. Your locale has a comma in the font point size, and NCrunch isn't handling this well. I'll see what I can do about rolling a fix into the next version.

I'm glad you like the new config system!
Back to top
abelb
#3 Posted : Thursday, December 15, 2016 2:58:30 AM(UTC)
Rank: Advanced Member

Groups: Registered
Joined: 9/12/2014(UTC)
Posts: 155
Location: Netherlands

Thanks: 19 times
Was thanked: 11 time(s) in 11 post(s)
Thanks for your quick response (as always!). The issue may be a tad larger than I thought at first (meaning, I noticed a few behaviors that may happen with other errors as well, in particular, the error staying). I tried a few things:

  1. Indeed, the error (re)appears with font-sizes that end up with fractions in the XML (though the font select box doesn't show these)
  2. Clearing the font (hit the red cross), the error stays, though the line in the physical file is gone
  3. Error only disappears after Disabling/Enabling NCrunch, or restart VS2015
  4. Selecting *any* other font, the error stays
  5. Selecting a font with a size that doesn't include the localized comma, still shows the error, but not after restart
  6. Error or not, the font is correctly displayed, both size and face


All of the above appeared to be reproducible multiple times. I'll be happy to try out your next version, if you have found the time to come up with a fix (sure you'll store it with InvariantCulture or something).
Back to top
Remco
#4 Posted : Thursday, December 15, 2016 3:02:15 AM(UTC)
Rank: NCrunch Developer

Groups: Administrators
Joined: 4/16/2011(UTC)
Posts: 7,145

Thanks: 960 times
Was thanked: 1290 time(s) in 1196 post(s)
Thanks for the extra details! My guess is that the conversion/parsing issue is throwing out a few things further up the stack. I'll try to get this fixed.
Back to top
1 user thanked Remco for this useful post.
abelb on 12/15/2016(UTC)
abelb
#5 Posted : Friday, December 16, 2016 10:36:35 PM(UTC)
Rank: Advanced Member

Groups: Registered
Joined: 9/12/2014(UTC)
Posts: 155
Location: Netherlands

Thanks: 19 times
Was thanked: 11 time(s) in 11 post(s)
One more thing I just noticed, this particular font (Source Code Pro) gets stored with a double-value of 8.99999 (with the comma, of course), whereas for instance Arial gets stored as 9 points. Not sure this helps, but just noticed it and thought I'd share it:

Code:
<TextOutputFont>Source Code Pro, 8,999999pt, 0</TextOutputFont>
Back to top
Remco
#6 Posted : Friday, December 16, 2016 10:39:59 PM(UTC)
Rank: NCrunch Developer

Groups: Administrators
Joined: 4/16/2011(UTC)
Posts: 7,145

Thanks: 960 times
Was thanked: 1290 time(s) in 1196 post(s)
abelb;9571 wrote:

Code:
<TextOutputFont>Source Code Pro, 8,999999pt, 0</TextOutputFont>


Thanks again, I'll take a look at this when addressing the localization issue here.
Back to top
Remco
#7 Posted : Wednesday, January 18, 2017 11:45:46 AM(UTC)
Rank: NCrunch Developer

Groups: Administrators
Joined: 4/16/2011(UTC)
Posts: 7,145

Thanks: 960 times
Was thanked: 1290 time(s) in 1196 post(s)
NCrunch v3.3 is now available including a fix for this issue - https://www.ncrunch.net/download.
Back to top
Users browsing this topic
Guest
Forum Jump  
You cannot post new topics in this forum.
You cannot reply to topics in this forum.
You cannot delete your posts in this forum.
You cannot edit your posts in this forum.
You cannot create polls in this forum.
You cannot vote in polls in this forum.

YAF | YAF © 2003-2011, Yet Another Forum.NET
This page was generated in 0.043 seconds.
Trial NCrunch
Take NCrunch for a spin
Do your fingers a favour and supercharge your testing workflow
Free Download