Bug 57143

Summary: [PATCH] Javadocs and throws clause for WorkbookUtil
Product: POI Reporter: Javen O'Neal <onealj>
Component: XSSFAssignee: POI Developers List <dev>
Status: RESOLVED FIXED    
Severity: normal    
Priority: P2    
Version: 3.10-FINAL   
Target Milestone: ---   
Hardware: PC   
OS: All   
Attachments: WorkbookUtil.java patch

Description Javen O'Neal 2014-10-24 23:53:52 UTC
Created attachment 32146 [details]
WorkbookUtil.java patch

1) org.apache.poi.ss.util.WorkbookUtil.validateSheetName and validateSheetState throw java.lang.IllegalArgumentExceptions but don't declare them in the throws clause.

2) validateSheetState is missing JavaDocs
Comment 1 Andreas Beeker 2014-10-27 00:38:47 UTC
More or less applied with r1634412
I prefer to use javadoc instead of the throws expression to document runtime exceptions.
see http://stackoverflow.com/questions/16757066/javadoc-and-runtimeexception