Issue 103714 - sc: new gcc44 && || warning
Summary: sc: new gcc44 && || warning
Status: CLOSED DUPLICATE of issue 102736
Alias: None
Product: Calc
Classification: Application
Component: code (show other issues)
Version: DEV300m52
Hardware: All Linux, all
: P3 Trivial (vote)
Target Milestone: ---
Assignee: dtardon
QA Contact: issues@sc
URL:
Keywords:
Depends on:
Blocks: 96084
  Show dependency tree
 
Reported: 2009-07-22 09:33 UTC by caolanm
Modified: 2013-08-07 15:14 UTC (History)
1 user (show)

See Also:
Issue Type: PATCH
Latest Confirmation in: ---
Developer Difficulty: ---

Attachments
patch that keeps things as they are (828 bytes, patch)
2009-07-22 09:34 UTC, caolanm 		no flags Details | Diff
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this issue.
Description caolanm 2009-07-22 09:33:52 UTC 
gcc warns on mixed && || without brackets these days. So following patch which
doesn't change the logic removes the warning.
Comment 1 caolanm 2009-07-22 09:34:13 UTC 
Created attachment 63682 [details]
patch that keeps things as they are
Comment 2 caolanm 2009-07-22 09:34:58 UTC 
accepting for myself
Comment 3 caolanm 2009-07-22 09:36:21 UTC 
done in cmcfixes61
Comment 4 caolanm 2009-07-22 09:37:03 UTC 
cmc->dtardon: able to verify that the logic is unchanged in cmcfixes61
Comment 5 niklas.nebel 2009-07-22 09:46:58 UTC 
This is the same as issue 102736.
Comment 6 caolanm 2009-07-22 09:53:56 UTC 
groan, turnaround is just so long that I'm forgetting what I've already done
Comment 7 caolanm 2009-07-22 09:54:20 UTC 
mark as duplicate

*** This issue has been marked as a duplicate of 102736 ***
Comment 8 caolanm 2009-07-22 09:56:30 UTC 
close as dup