Issue 119249 - DBG_ASSERT enabled in fpicker (debug=true dbglevel=0)
Summary: DBG_ASSERT enabled in fpicker (debug=true dbglevel=0)
Status: CONFIRMED
Alias: None
Product: Build Tools
Classification: Code
Component: code (show other issues)
Version: 3.4.0 Beta (OOo)
Hardware: Sun Solaris
: P3 Normal (vote)
Target Milestone: ---
Assignee: AOO issues mailing list
QA Contact:
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-04-19 14:54 UTC by Jean-Louis Fuchs
Modified: 2017-05-20 11:29 UTC (History)
2 users (show)

See Also:
Issue Type: DEFECT
Latest Confirmation in: ---
Developer Difficulty: Easy

Attachments
Patch: including lcl_DecoratedFilter on any debug level (702 bytes, application/octet-stream)
2012-04-19 14:54 UTC, Jean-Louis Fuchs 		no flags Details
Patch: main/forms/source/xforms/propertysetbase.cxx (470 bytes, patch)
2012-04-20 10:09 UTC, Jean-Louis Fuchs 		no flags Details | Diff
Patch: main/dbaccess/source/ui/tabledesign/FieldDescGenWin.hxx (547 bytes, patch)
2012-04-20 10:10 UTC, Jean-Louis Fuchs 		no flags Details | Diff
Patch: main/dbaccess/source/ui/tabledesign/FieldDescGenWin.cxx (487 bytes, patch)
2012-04-20 10:14 UTC, Jean-Louis Fuchs 		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 Jean-Louis Fuchs 2012-04-19 14:54:06 UTC 
Created attachment 77462 [details]
Patch: including lcl_DecoratedFilter on any debug level

When building with debug=true dbglevel=0 in the file

main/fpicker/source/office/iodlgimp.cxx

DBG_ASSERT is enabled, but #if OSL_DEBUG_LEVEL > 0 will not include lcl_DecoratedFilter. So the symbol lcl_DecoratedFilter is not found.

I changed define to include lcl_DecoratedFilter (see patch), of course disabling DBG_ASSERT for OSL_DEBUG_LEVEL=0 is also possible.
Comment 1 Jean-Louis Fuchs 2012-04-20 07:27:16 UTC 
The second patch is the same problem in a different file:

main/forms/source/xforms/propertysetbase.cxx
Comment 2 Jean-Louis Fuchs 2012-04-20 09:48:32 UTC 
Sorry for the above message. I was in the wrong issue.
Comment 3 Jean-Louis Fuchs 2012-04-20 09:54:18 UTC 
No is correct. Sorry for the confusion. 

-- All attachments show places where this problem occurs. --
Comment 4 Jean-Louis Fuchs 2012-04-20 10:09:22 UTC 
Created attachment 77471 [details]
Patch: main/forms/source/xforms/propertysetbase.cxx
Comment 5 Jean-Louis Fuchs 2012-04-20 10:10:04 UTC 
Created attachment 77472 [details]
Patch: main/dbaccess/source/ui/tabledesign/FieldDescGenWin.hxx
Comment 6 Jean-Louis Fuchs 2012-04-20 10:14:54 UTC 
Created attachment 77473 [details]
Patch: main/dbaccess/source/ui/tabledesign/FieldDescGenWin.cxx
Comment 7 Rob Weir 2013-02-08 01:38:06 UTC 
Good integration task for a new developer.
Comment 8 Marcus 2017-05-20 11:29:24 UTC 
Reset assigne to the default "issues@openoffice.apache.org".