Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 350850

Summary: [doc] JSDT doc plugin should include a pre-built help index
Product: [WebTools] JSDT Reporter: Martin Oberhuber <mober.at+eclipse>
Component: GeneralAssignee: Nitin Dahyabhai <thatnitind>
Status: RESOLVED FIXED QA Contact: Nitin Dahyabhai <thatnitind>
Severity: normal    
Priority: P3 CC: cbridgha, kaloyan
Version: 3.3Flags: thatnitind: pmc_approved? (david_williams)
thatnitind: pmc_approved? (raghunathan.srinivasan)
thatnitind: pmc_approved? (naci.dai)
thatnitind: pmc_approved? (deboer)
thatnitind: pmc_approved? (neil.hauge)
kaloyan: pmc_approved+
thatnitind: pmc_approved? (cbridgha)
thatnitind: review+
Target Milestone: 3.3.2   
Hardware: All   
OS: All   
Whiteboard: PMC_Approved
Attachments:
Description Flags
Patch providing a prebuilt doc index thatnitind: iplog+

Description Martin Oberhuber CLA 2011-06-30 10:58:12 EDT 
Created attachment 198917 [details]
Patch providing a prebuilt doc index

+++ This bug was initially created as a clone of Bug #350848 +++

When searching Eclipse Help for the first time, a search index has to be built. The speed of doing so can be drastically improved when the doc plugin ships a prebuilt help index. 

JSDT help should include such an index, such that users can search docs faster.

Attached patch adds the pre-built index to the docs when building them.
Please consider for inclusion.
Comment 1 Martin Oberhuber CLA 2011-09-07 07:15:05 EDT 
Ping, this is planned for 3.3.1 but not yet integrated ?
Could my patch be considered?
Comment 2 Nitin Dahyabhai CLA 2011-09-08 13:10:20 EDT 
Sorry, Martin, it slipped past my notice.
Comment 3 Martin Oberhuber CLA 2011-11-16 15:14:50 EST 
Ping, could my patch be considered please ? 
Indigo SR 2 is approaching fast ...
Comment 4 Martin Oberhuber CLA 2012-01-24 04:31:15 EST 
Ping, 3.3.2 is almost there ... could this be integrated please ?
Comment 5 Nitin Dahyabhai CLA 2012-01-31 09:56:38 EST 
*Explain why you believe this is a stop-ship defect. Or, if it is a "hotbug" (requested by an adopter) please document it as such.
Fix supplied by an adopter, but not reviewed before RC0 cutoff.

*Give a brief technical overview. Who has reviewed this fix?
Updates the build properties to include the "index" folder generated with the doc plug-in.

*What is the risk associated with this fix?
Very low, it's not a code change.
Comment 6 Kaloyan Raev CLA 2012-01-31 11:11:12 EST 
Approved. 

BTW, it is not explicitly noted who has tested this patch. Although it looks pretty safe, please make sure that a committer has tested it before submitting in CVS.
Comment 7 Nitin Dahyabhai CLA 2012-01-31 13:22:34 EST 
Released