Issue 63159 - Patch to use d_type field in dirent structure if possible
Summary: Patch to use d_type field in dirent structure if possible
Status: CLOSED FIXED
Alias: None
Product: porting
Classification: Code
Component: www (show other issues)
Version: OOo 2.0.2
Hardware: All All
: P3 Trivial (vote)
Target Milestone: OOo 3.2
Assignee: radekdoulik
QA Contact: issues@porting
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-03-14 11:41 UTC by radekdoulik
Modified: 2009-08-13 12:00 UTC (History)
3 users (show)

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

Attachments
dtype patch (7.42 KB, text/plain)
2006-03-14 11:42 UTC, radekdoulik 		no flags Details
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 radekdoulik 2006-03-14 11:41:58 UTC 
Attached patch tries to use d_type to avoid unnecessary calls to stat. Not sure
if it belongs in framework component, didn't find sal one.
Comment 1 radekdoulik 2006-03-14 11:42:21 UTC 
Created attachment 34861 [details]
dtype patch
Comment 2 Olaf Felka 2006-03-14 12:22:26 UTC 
of @ hro: Please have a look.
Comment 3 caolanm 2006-03-14 18:40:57 UTC 
reassign to sal
Comment 4 Stephan Bergmann 2006-03-15 08:04:27 UTC 
...still for hro.
Comment 5 hennes.rohling 2006-05-19 15:38:57 UTC 
Looks good, I will test this before applying...
Comment 6 hennes.rohling 2006-08-07 10:32:49 UTC 
.
Comment 7 kai.sommerfeld 2006-11-01 16:07:24 UTC 
kso->all: hro is currently sick and will not be available before January 2007.
That's why there is currenly no actual activity here.
Comment 8 hennes.rohling 2007-04-10 12:34:03 UTC 
Target set
Comment 9 hennes.rohling 2007-05-22 12:34:18 UTC 
Planned for 2.3.
Comment 10 hennes.rohling 2007-08-01 10:20:05 UTC 
Changed target.
Comment 11 caolanm 2007-08-22 13:08:31 UTC 
18 months :-)
Comment 12 hennes.rohling 2007-11-05 10:17:37 UTC 
Added to CWS hro10
Comment 13 kai.sommerfeld 2008-01-14 11:19:15 UTC 
radekdoulik: sorry, no time left to integrate the patch in 2.4 codeline.
Comment 14 hennes.rohling 2008-02-13 14:16:57 UTC 
Fixed on CWS hro30
Comment 15 hennes.rohling 2008-02-19 10:31:42 UTC 
@radekdoulik: Please verify commited changes on CWS hro30
Comment 16 radekdoulik 2008-04-22 12:38:40 UTC 
Sorry for the delay. I have verified that the fix is committed to hro30 cws.

Thanks
Radek
Comment 17 kai.sommerfeld 2008-07-17 08:54:32 UTC 
I do very sorry, but there is no time left to integrate the fix into 3.0 code
line. Last CWS integration deadline has passed already. CWS containing the fixed
needds to be resynced to latest milestone and the fix needs to be re-verified
after CWS is Ready for QA again. => 3.1, back to hro
Comment 18 kai.sommerfeld 2008-07-17 08:55:18 UTC 
.
Comment 19 kai.sommerfeld 2008-07-17 08:55:53 UTC 
.
Comment 20 caolanm 2009-01-07 12:40:25 UTC 
What's the status of this ? The bug target is 3.1, the workspace target is 3.2,
the estimated due date is set to "2008-02-22" and the workspace description is
"Patches for OOo 3.0" and it was submitted in 2006 :-)
Comment 21 kai.sommerfeld 2009-02-19 08:34:50 UTC 
Unfortunately we missed the 3.1 code freeze deadline. Bug is fixed, but
respective CWS is not approved by QA. Really don't like this, but need to 
retarget the task to 3.2. Sorry.
Comment 22 kai.sommerfeld 2009-07-07 15:45:31 UTC 
Taking over... patch applied to CWS kso32fixes.
Comment 23 kai.sommerfeld 2009-07-14 08:55:45 UTC 
radekdoulik: Please verify.
Comment 24 kai.sommerfeld 2009-07-16 14:24:59 UTC 
radekdoulik, cmc: Can one of you guys please verify the issue. CWS due date is
July 21. Thanks.
Comment 25 radekdoulik 2009-07-17 09:28:27 UTC 
I have verified that the patch is indeed applied to kso32fixes cws.
Comment 26 thorsten.ziehm 2009-07-20 14:53:22 UTC 
This issue is closed automatically and wasn't rechecked in a current version of
OOo. The fixed issue should be integrated in OOo since more than half a year. If
you think this issue isn't fixed in a current version (OOo 3.1), please reopen
it and change the field 'Target Milestone' accordingly.

If you want to download a current version of OOo =>
http://download.openoffice.org/index.html
If you want to know more about the handling of fixed/verified issues =>
http://wiki.services.openoffice.org/wiki/Handle_fixed_verified_issues
Comment 27 thorsten.ziehm 2009-07-20 15:35:48 UTC 
Sorry this issue was wrongly closed. This issue will be reopened automatically.
And will be set after that back to fixed/verified.
Comment 28 thorsten.ziehm 2009-07-20 15:40:15 UTC 
Set to state 'fixed'.
Comment 29 thorsten.ziehm 2009-07-20 15:44:28 UTC 
Set back to state 'verified/fixed'.

Again. Sorry for the mass of mails.
Comment 30 caolanm 2009-08-13 12:00:09 UTC 
closed, integrated m54.