Modify

Opened 4 years ago

Closed 4 years ago

#8138 closed Bugs (fixed)

Conversion warning in sequence_stack.hpp

Reported by: mbradle@… Owned by: eric_niebler
Milestone: To Be Determined Component: xpressive
Version: Boost Development Trunk Severity: Cosmetic
Keywords: Cc:

Description

Hello,

When I compile the example code boost-trunk/libs/graph/example/write_graphviz.cpp with the g++ compiler flag -Wconversion, I get the warning 'conversion to 'double' from 'size_t' may alter its value' (see attached test.txt). Casting this->current_chunk_->size() as a double on line 118 of boost/xpressive/detail/utility/sequence_stack.hpp clears this warning (see attached diff.txt). This is a possible patch. Thanks, and best wishes.

Brad Meyer

Attachments (2)

test.txt (8.5 KB) - added by mbradle@… 4 years ago.
transcript of compilation and warning
diff.txt (774 bytes) - added by mbradle@… 4 years ago.
Change that removed the warning (possible patch)

Download all attachments as: .zip

Change History (4)

Changed 4 years ago by mbradle@…

transcript of compilation and warning

Changed 4 years ago by mbradle@…

Change that removed the warning (possible patch)

comment:1 Changed 4 years ago by eric_niebler

(In [83304]) silence a gcc warning, refs #8138

comment:2 Changed 4 years ago by eric_niebler

  • Resolution set to fixed
  • Status changed from new to closed

(In [83390]) merge [83304] from trunk, fixes #8138

Add Comment

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain eric_niebler.
The resolution will be deleted. Next status will be 'reopened'.
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.