Modify

Ticket #5250 (closed Patches: fixed)

Opened 3 years ago

Last modified 3 years ago

Tiny warning removal for date_time

Reported by: chrisj Owned by: az_sw_dude
Milestone: To Be Determined Component: date_time
Version: Boost Development Trunk Severity: Problem
Keywords: Cc:

Description

The attached patch changes 'if ((week == 53))' to 'if (week == 53)'. This is because clang++ warns about the double brackets.

It warns because double brackets are the traditional way of stopping the warning in statements like ' if (week = 53) ' (notice single =).

Attachments

date_time.patch Download (466 bytes) - added by chrisj 3 years ago.
Patch to boost/date_time/gregorian_calendar.ipp

Change History

Changed 3 years ago by chrisj

Patch to boost/date_time/gregorian_calendar.ipp

comment:1 Changed 3 years ago by marshall

Applied in [70769]

comment:2 Changed 3 years ago by marshall

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

(In [70985]) Merge date/time fixes to release. Fixes #5250

View

Add a comment

Modify Ticket

Change Properties
<Author field>
Action
as closed
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.