summaryrefslogtreecommitdiff
path: root/external
diff options
context:
space:
mode:
authorChris Sherlock <chris.sherlock79@gmail.com>2017-04-22 05:58:04 +1000
committerChris Sherlock <chris.sherlock79@gmail.com>2017-04-26 16:21:34 +0200
commit1167df7df59e37fddef0c40c4d27cb2e82e10922 (patch)
treee7f5b6492a9747864874c63a626d70247ff534e9 /external
parent6022a17e79e2cc0a483122bf7f73fcb3e246caf6 (diff)
vcl: refactor ErrorHandler functions
Current, HandleError_Impl is called on by GetErrorString, and HandleError_Impl also creates the error string. However, when you look at what each function does, HandleError_Impl has been imbued with some extraneous parameters: 1. bJustCreateString, and 2. rError bJustCreateString is unnecessary, in fact it just mutates HandleError to create the string, which is not really the core purpose of this function. In fact, the string should be created when we get the string from the error ID (GetErrorString), and in fact the error handler function should call on GetErrorString and not the other way around! This has the happy side effect of vastly simplifying this code, and allows each of the functions only do the thing they are meant to do, and we no longer need these extraneous parameters which just cause problems for code readibility. On to of this, by simplifying the HandleError_Impl function, we just move the code into the public HandleError function because these parameters aren't necessary any more. Change-Id: I4a727b5bbc6d3cdb0519f49b48dc52f8a8976629 Reviewed-on: https://gerrit.libreoffice.org/36849 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Chris Sherlock <chris.sherlock79@gmail.com>
Diffstat (limited to 'external')
0 files changed, 0 insertions, 0 deletions