aniani/elinks
1
0
Fork 0
mirror of https://github.com/rkd77/elinks.git synced 2025-01-03 14:57:44 -05:00
Code

Re-use @fs in activate_link instead of introducing a new local variable

Browse Source 
This optimisation is made possible by commit
15dce57bc9.
This commit is contained in:
Miciah Dashiel Butler Masters 2006-12-30 11:17:43 +00:00 committed by Miciah Dashiel Butler Masters
parent 7b931a2de0
commit 896cdab771
1 changed files with 4 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
src/viewer/text/link.c
View File

@ -1024,7 +1024,8 @@ activate_link(struct session *ses, struct document_view *doc_view,
* buttons in the group. Do it in this order because * buttons in the group. Do it in this order because
* further @find_form_state calls may reallocate * further @find_form_state calls may reallocate
* @doc_view->vs->form_info[] and thereby make the @fs * @doc_view->vs->form_info[] and thereby make the @fs
* pointer invalid. */ * pointer invalid. This also allows us to re-use
* @fs in the loop. */
fs->state = 1; fs->state = 1;
foreach (form, doc_view->document->forms) { foreach (form, doc_view->document->forms) {
struct form_control *fc; struct form_control *fc;
@ -1036,10 +1037,8 @@ activate_link(struct session *ses, struct document_view *doc_view,
if (fc->type == FC_RADIO if (fc->type == FC_RADIO
&& !xstrcmp(fc->name, link_fc->name) && !xstrcmp(fc->name, link_fc->name)
&& fc != link_fc) { && fc != link_fc) {
struct form_state *frm_st; fs = find_form_state(doc_view, fc);
if (fs) fs->state = 0;
frm_st = find_form_state(doc_view, fc);
if (frm_st) frm_st->state = 0;
} }
} }
} }