Project

General

Profile

Actions

Bug #628

closed

Warnings in vrscanl.c

Added by Jan Schlamelcher over 10 years ago. Updated over 7 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Jan Schlamelcher
Category:
Library
Target version:
Start date:
2015-04-16
Due date:
% Done:

100%

Estimated time:
Module:
dcmdata
Operating System:
Compiler:

Description

Apply fixes for vrscanl.c to the flex skeleton, so they don't get overwritten the next time the code is newly generated.

Actions #1

Updated by Jan Schlamelcher over 8 years ago

The flex skeleton file for C flex (not flex++) contains GNU Autoconf dependent code that is evaluated when compiling flex for building it into the flex executable. This solution is therefore not viable.

Actions #2

Updated by Jan Schlamelcher over 8 years ago

  • Status changed from Assigned to Closed

While testing whether it is possible to disable the warnings by adding appropriate 'ofdiag' includes to the lexer file (vrscanl.l), I noticed that these warnings no longer exist in newer versions of flex. So the solution is simple: don't use the outdated flex package that ships with Debian 'jessie' if you want the warnings to go away. To be fair: the flex package that ships with current Arch Linux produces code that emits three warnings, of which one has been fixed on 25.01.2017 in the current flex developement version, which is not yet available under Arch Linux.

Actions #3

Updated by Jörg Riesmeier over 7 years ago

  • Category set to Library
  • Status changed from Closed to Reopened
  • Target version changed from 3.6.1+ to 3.6.4

VisualStudio still reports the following warning (8 times, see Dashboard):

warning C4267: '=': conversion from 'size_t' to 'int', possible loss of data
Actions #4

Updated by Jörg Riesmeier over 7 years ago

  • Status changed from Reopened to Closed
  • % Done changed from 0 to 100

Fixed by switching to flex 2.6.4 (see commit da8e84d).

Actions

Also available in: Atom PDF