comparison src/org/tmatesoft/hg/core/HgCallbackTargetException.java @ 394:f52ca9530774 v0.8.0

Resolve FIXMEs: more consistent exceptions
author Artem Tikhomirov <tikhomirov.artem@gmail.com>
date Wed, 22 Feb 2012 03:10:55 +0100
parents a2341e761609
children 2747b0723867
comparison
equal deleted inserted replaced
393:728708de3597 394:f52ca9530774
1 /* 1 /*
2 * Copyright (c) 2011 TMate Software Ltd 2 * Copyright (c) 2011-2012 TMate Software Ltd
3 * 3 *
4 * This program is free software; you can redistribute it and/or modify 4 * This program is free software; you can redistribute it and/or modify
5 * it under the terms of the GNU General Public License as published by 5 * it under the terms of the GNU General Public License as published by
6 * the Free Software Foundation; version 2 of the License. 6 * the Free Software Foundation; version 2 of the License.
7 * 7 *
27 * However, there's no straightforward way to alter throws clause for these methods, and alternatives like generic {@link Exception} or 27 * However, there's no straightforward way to alter throws clause for these methods, and alternatives like generic {@link Exception} or
28 * library's own {@link HgException} are rather obscure. Suggested approach is to wrap whatever exception user code produces with 28 * library's own {@link HgException} are rather obscure. Suggested approach is to wrap whatever exception user code produces with
29 * {@link RuntimeException} subclass, {@link Wrap}. Then, unwrap and re-throw with checked {@link HgCallbackTargetException}. 29 * {@link RuntimeException} subclass, {@link Wrap}. Then, unwrap and re-throw with checked {@link HgCallbackTargetException}.
30 * 30 *
31 * FIXME REVISIT perhaps, shall just throw HgCallbackTargetException from any handler, and do not catch anything in commands at all? 31 * FIXME REVISIT perhaps, shall just throw HgCallbackTargetException from any handler, and do not catch anything in commands at all?
32 * FIXME decide whether shall root at HgException ("throws HgException, HgCallbackTargetException" looks a bit odd now)
32 * 33 *
33 * @author Artem Tikhomirov 34 * @author Artem Tikhomirov
34 * @author TMate Software Ltd. 35 * @author TMate Software Ltd.
35 */ 36 */
36 @SuppressWarnings("serial") 37 @SuppressWarnings("serial")