Issue 77131 - Warning in binfilter bf_sch: wrong order of caught exceptions
Summary: Warning in binfilter bf_sch: wrong order of caught exceptions
Status: CLOSED FIXED
Alias: None
Product: General
Classification: Code
Component: chart (show other issues)
Version: 3.3.0 or older (OOo)
Hardware: All All
: P4 Trivial (vote)
Target Milestone: ---
Assignee: bjoern.milcke
QA Contact: issues@graphics
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-05-09 07:21 UTC by bjoern.milcke
Modified: 2013-02-24 21:21 UTC (History)
2 users (show)

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

Attachments
Patch for binfilter to remove warning in bf_sch (2.03 KB, patch)
2007-05-09 07:36 UTC, bjoern.milcke 		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 bjoern.milcke 2007-05-09 07:21:08 UTC 
In binfilter/bf_sch/source/filter/xml/sch_SchXMLWrapper.cxx:225 an
io::IOException is caught. After that a zip::ZipIoException is caught, which is
derived from the former, so that this code is never reached.

In Tinderbox builds this warning shows up as an error in the log, so it is a bit
tedious, though probably not a big problem.

->MBA: If you still have your CWS basemodelrefactoring (including binfilter)
open, please be so kind as to apply the attached patch. Thanks.
Comment 1 bjoern.milcke 2007-05-09 07:36:25 UTC 
Created attachment 44960 [details]
Patch for binfilter to remove warning in bf_sch
Comment 2 Mathias_Bauer 2007-05-10 12:01:43 UTC 
order changed
Comment 3 Mathias_Bauer 2007-06-01 16:20:17 UTC 
please verify
Comment 4 bjoern.milcke 2007-06-06 10:20:31 UTC 
Verified on CWS, patch is applied, the tinderbox build no longer shows the
mentioned warning in the log (there are still some warnings for
sch_SchXMLWrapper.cxx but none that are treated as error in the tinderbox)
Comment 5 bjoern.milcke 2007-07-25 13:21:21 UTC 
Patch is integrated in m222 master. CWSes basing on m222 do not show the warning
in Tinderboxes.