Yegappan Lakshmanan wrote: >> Index: src/hardcopy.c >> =================================================================== >> --- src/hardcopy.c (revision 1425) >> +++ src/hardcopy.c (working copy) >> @@ -568,6 +568,8 @@ >> long_u bytes_to_print = 0; >> int page_line; >> int jobsplit; >> + char_u *bname = NULL; >> + int do_return = FALSE; >> >> memset(&settings, 0, sizeof(prt_settings_T)); >> settings.has_color = TRUE; >> @@ -599,11 +601,15 @@ >> */ >> if (mch_print_init(&settings, >> curbuf->b_fname == NULL >> - ? (char_u *)buf_spname(curbuf) >> + ? (bname = (char_u *)buf_spname(curbuf)) > > For maintainability reasons, it is better to assign the bname variable > outside of this > function call.
I'm curious: what's the rationale to have the assignment outside the function call/if-condition? -- Andreas. --~--~---------~--~----~------------~-------~--~----~ You received this message from the "vim_dev" maillist. For more information, visit http://www.vim.org/maillist.php -~----------~----~----~----~------~----~------~--~---
