home *** CD-ROM | disk | FTP | other *** search
/ vim.ftp.fu-berlin.de / 2015-02-03.vim.ftp.fu-berlin.de.tar / vim.ftp.fu-berlin.de / patches / 6.0.163 < prev    next >
Encoding:
Internet Message Format  |  2002-02-02  |  13.9 KB
  1. To: vim-dev@vim.org
  2. Subject: Patch 6.0.163
  3. Fcc: outbox
  4. From: Bram Moolenaar <Bram@moolenaar.net>
  5. MIME-Version: 1.0
  6. Content-Type: text/plain; charset=ISO-8859-1
  7. Content-Transfer-Encoding: 8bit
  8. ------------
  9.  
  10. Patch 6.0.163
  11. Problem:    When using a GUI dialog, a file name is sometimes used like it was
  12.         a directory.
  13. Solution:   Separate path and file name properly.
  14.         For GTK, Motif and Athena concatenate directory and file name for
  15.         the default selection.
  16. Files:        src/diff.c, src/ex_cmds.c, src/ex_cmds2.c, src/ex_docmd.c,
  17.         src/gui_athena.c, src/gui_gtk.c, src/gui_motif.c, src/message.c
  18.  
  19.  
  20. *** ../vim60.162/src/diff.c    Wed Sep 19 18:36:35 2001
  21. --- src/diff.c    Sun Feb  3 13:43:37 2002
  22. ***************
  23. *** 744,751 ****
  24.   #ifdef FEAT_BROWSE
  25.       if (cmdmod.browse)
  26.       {
  27. !     browseFile = do_browse(TRUE, (char_u *)_("Patch file"),
  28. !         NULL, NULL, eap->arg, BROWSE_FILTER_ALL_FILES, curbuf);
  29.       if (browseFile == NULL)
  30.           return;        /* operation cancelled */
  31.       eap->arg = browseFile;
  32. --- 744,751 ----
  33.   #ifdef FEAT_BROWSE
  34.       if (cmdmod.browse)
  35.       {
  36. !     browseFile = do_browse(FALSE, (char_u *)_("Patch file"),
  37. !              eap->arg, NULL, NULL, BROWSE_FILTER_ALL_FILES, NULL);
  38.       if (browseFile == NULL)
  39.           return;        /* operation cancelled */
  40.       eap->arg = browseFile;
  41. *** ../vim60.162/src/ex_cmds.c    Sun Feb  3 12:33:39 2002
  42. --- src/ex_cmds.c    Sun Feb  3 13:29:29 2002
  43. ***************
  44. *** 2050,2057 ****
  45.   #ifdef FEAT_BROWSE
  46.       if (cmdmod.browse)
  47.       {
  48. !     browse_file = do_browse(TRUE, (char_u *)_("Save As"), NULL,
  49. !                           NULL, ffname, NULL, curbuf);
  50.       if (browse_file == NULL)
  51.           goto theend;
  52.       ffname = browse_file;
  53. --- 2050,2057 ----
  54.   #ifdef FEAT_BROWSE
  55.       if (cmdmod.browse)
  56.       {
  57. !     browse_file = do_browse(TRUE, (char_u *)_("Save As"), ffname,
  58. !                             NULL, NULL, NULL, curbuf);
  59.       if (browse_file == NULL)
  60.           goto theend;
  61.       ffname = browse_file;
  62. ***************
  63. *** 2308,2314 ****
  64.           /* ":browse wall": ask for file name if there isn't one */
  65.           if (buf->b_ffname == NULL && cmdmod.browse)
  66.           buf->b_ffname = do_browse(TRUE, (char_u *)_("Save As"), NULL,
  67. !                            NULL, (char_u *)"", NULL, buf);
  68.   #endif
  69.           if (buf->b_ffname == NULL)
  70.           {
  71. --- 2308,2314 ----
  72.           /* ":browse wall": ask for file name if there isn't one */
  73.           if (buf->b_ffname == NULL && cmdmod.browse)
  74.           buf->b_ffname = do_browse(TRUE, (char_u *)_("Save As"), NULL,
  75. !                                NULL, NULL, NULL, buf);
  76.   #endif
  77.           if (buf->b_ffname == NULL)
  78.           {
  79. ***************
  80. *** 2535,2542 ****
  81.   #ifdef FEAT_BROWSE
  82.       if (cmdmod.browse)
  83.       {
  84. !         browse_file = do_browse(FALSE, (char_u *)_("Edit File"), NULL,
  85. !                           NULL, ffname, NULL, curbuf);
  86.           if (browse_file == NULL)
  87.           goto theend;
  88.           ffname = browse_file;
  89. --- 2535,2542 ----
  90.   #ifdef FEAT_BROWSE
  91.       if (cmdmod.browse)
  92.       {
  93. !         browse_file = do_browse(FALSE, (char_u *)_("Edit File"), ffname,
  94. !                             NULL, NULL, NULL, curbuf);
  95.           if (browse_file == NULL)
  96.           goto theend;
  97.           ffname = browse_file;
  98. *** ../vim60.162/src/ex_cmds2.c    Wed Oct 31 10:12:03 2001
  99. --- src/ex_cmds2.c    Sun Feb  3 13:44:58 2002
  100. ***************
  101. *** 1711,1718 ****
  102.       {
  103.       char_u *fname = NULL;
  104.   
  105. !     fname = do_browse(FALSE, (char_u *)_("Run Macro"),
  106. !         NULL, NULL, eap->arg, BROWSE_FILTER_MACROS, curbuf);
  107.       if (fname != NULL)
  108.       {
  109.           cmd_source(fname, eap);
  110. --- 1711,1718 ----
  111.       {
  112.       char_u *fname = NULL;
  113.   
  114. !     fname = do_browse(FALSE, (char_u *)_("Run Macro"), eap->arg,
  115. !                       NULL, NULL, BROWSE_FILTER_MACROS, NULL);
  116.       if (fname != NULL)
  117.       {
  118.           cmd_source(fname, eap);
  119. *** ../vim60.162/src/ex_docmd.c    Wed Jan 30 16:52:44 2002
  120. --- src/ex_docmd.c    Sun Feb  3 13:47:11 2002
  121. ***************
  122. *** 5554,5560 ****
  123.           && eap->cmdidx != CMD_new)
  124.       {
  125.       fname = do_browse(FALSE, (char_u *)_("Edit File in new window"),
  126. !                       NULL, NULL, eap->arg, NULL, curbuf);
  127.       if (fname == NULL)
  128.           goto theend;
  129.       eap->arg = fname;
  130. --- 5554,5560 ----
  131.           && eap->cmdidx != CMD_new)
  132.       {
  133.       fname = do_browse(FALSE, (char_u *)_("Edit File in new window"),
  134. !                       eap->arg, NULL, NULL, NULL, curbuf);
  135.       if (fname == NULL)
  136.           goto theend;
  137.       eap->arg = fname;
  138. ***************
  139. *** 5923,5930 ****
  140.       {
  141.           char_u *browseFile;
  142.   
  143. !         browseFile = do_browse(FALSE, (char_u *)_("Append File"), NULL,
  144. !                         NULL, eap->arg, NULL, curbuf);
  145.           if (browseFile != NULL)
  146.           {
  147.           i = readfile(browseFile, NULL,
  148. --- 5923,5930 ----
  149.       {
  150.           char_u *browseFile;
  151.   
  152. !         browseFile = do_browse(FALSE, (char_u *)_("Append File"), eap->arg,
  153. !                             NULL, NULL, NULL, curbuf);
  154.           if (browseFile != NULL)
  155.           {
  156.           i = readfile(browseFile, NULL,
  157. ***************
  158. *** 6449,6455 ****
  159.           if (cmdmod.browse)
  160.           {
  161.           browseFile = do_browse(TRUE, (char_u *)_("Save Redirection"),
  162. !                NULL, NULL, eap->arg, BROWSE_FILTER_ALL_FILES, curbuf);
  163.           if (browseFile == NULL)
  164.               return;        /* operation cancelled */
  165.           eap->arg = browseFile;
  166. --- 6449,6455 ----
  167.           if (cmdmod.browse)
  168.           {
  169.           browseFile = do_browse(TRUE, (char_u *)_("Save Redirection"),
  170. !                eap->arg, NULL, NULL, BROWSE_FILTER_ALL_FILES, curbuf);
  171.           if (browseFile == NULL)
  172.               return;        /* operation cancelled */
  173.           eap->arg = browseFile;
  174. ***************
  175. *** 6601,6607 ****
  176.           eap->cmdidx == CMD_mksession ? (char_u *)_("Save Session") :
  177.   # endif
  178.           (char_u *)_("Save Setup"),
  179. !         NULL, (char_u *)"vim", fname, BROWSE_FILTER_MACROS, curbuf);
  180.       if (browseFile == NULL)
  181.           goto theend;
  182.       fname = browseFile;
  183. --- 6601,6607 ----
  184.           eap->cmdidx == CMD_mksession ? (char_u *)_("Save Session") :
  185.   # endif
  186.           (char_u *)_("Save Setup"),
  187. !         fname, (char_u *)"vim", NULL, BROWSE_FILTER_MACROS, NULL);
  188.       if (browseFile == NULL)
  189.           goto theend;
  190.       fname = browseFile;
  191. *** ../vim60.162/src/gui_athena.c    Wed Dec 12 20:36:34 2001
  192. --- src/gui_athena.c    Sun Feb  3 15:24:05 2002
  193. ***************
  194. *** 1836,1841 ****
  195. --- 1836,1856 ----
  196.       char_u    *filter;    /* not used (file name filter) */
  197.   {
  198.       Position x, y;
  199. +     char_u    dirbuf[MAXPATHL];
  200. + 
  201. +     /* Concatenate "initdir" and "dflt". */
  202. +     if (initdir == NULL || *initdir == NUL)
  203. +     mch_dirname(dirbuf, MAXPATHL);
  204. +     else if (STRLEN(initdir) + 2 < MAXPATHL)
  205. +     STRCPY(dirbuf, initdir);
  206. +     else
  207. +     dirbuf[0] = NUL;
  208. +     if (dflt != NULL && *dflt != NUL
  209. +                   && STRLEN(dirbuf) + 2 + STRLEN(dflt) < MAXPATHL)
  210. +     {
  211. +     add_pathsep(dirbuf);
  212. +     STRCAT(dirbuf, dflt);
  213. +     }
  214.   
  215.       /* Position the file selector just below the menubar */
  216.       XtTranslateCoords(vimShell, (Position)0, (Position)
  217. ***************
  218. *** 1845,1851 ****
  219.           0
  220.   #endif
  221.           , &x, &y);
  222. !     return (char_u *)vim_SelFile(vimShell, (char *)title, (char *)initdir,
  223.             NULL, (int)x, (int)y, gui.menu_fg_pixel, gui.menu_bg_pixel);
  224.   }
  225.   #endif
  226. --- 1860,1866 ----
  227.           0
  228.   #endif
  229.           , &x, &y);
  230. !     return (char_u *)vim_SelFile(vimShell, (char *)title, (char *)dirbuf,
  231.             NULL, (int)x, (int)y, gui.menu_fg_pixel, gui.menu_bg_pixel);
  232.   }
  233.   #endif
  234. *** ../vim60.162/src/gui_gtk.c    Fri Dec 14 20:19:45 2001
  235. --- src/gui_gtk.c    Sun Feb  3 15:25:14 2002
  236. ***************
  237. *** 875,890 ****
  238.       /* if our pointer is currently hidden, then we should show it. */
  239.       gui_mch_mousehide(FALSE);
  240.   
  241. !     if (dflt == NULL)
  242. !     dflt = (char_u *)"";
  243.       if (initdir == NULL || *initdir == NUL)
  244. -     {
  245.       mch_dirname(dirbuf, MAXPATHL);
  246. !     strcat((char *)dirbuf, "/");    /* make sure this is a directory */
  247. !     initdir = dirbuf;
  248. !     }
  249.       gtk_file_selection_set_filename(GTK_FILE_SELECTION(gui.filedlg),
  250. !                               (const gchar *)initdir);
  251.   
  252.       gui_gtk_position_in_parent(GTK_WIDGET(gui.mainwin),
  253.                          GTK_WIDGET(gui.filedlg), VW_POS_MOUSE);
  254. --- 875,895 ----
  255.       /* if our pointer is currently hidden, then we should show it. */
  256.       gui_mch_mousehide(FALSE);
  257.   
  258. !     /* Concatenate "initdir" and "dflt". */
  259.       if (initdir == NULL || *initdir == NUL)
  260.       mch_dirname(dirbuf, MAXPATHL);
  261. !     else if (STRLEN(initdir) + 2 < MAXPATHL)
  262. !     STRCPY(dirbuf, initdir);
  263. !     else
  264. !     dirbuf[0] = NUL;
  265. !     /* Always need a trailing slash for a directory. */
  266. !     add_pathsep(dirbuf);
  267. !     if (dflt != NULL && *dflt != NUL
  268. !                   && STRLEN(dirbuf) + 2 + STRLEN(dflt) < MAXPATHL)
  269. !     STRCAT(dirbuf, dflt);
  270. ! 
  271.       gtk_file_selection_set_filename(GTK_FILE_SELECTION(gui.filedlg),
  272. !                               (const gchar *)dirbuf);
  273.   
  274.       gui_gtk_position_in_parent(GTK_WIDGET(gui.mainwin),
  275.                          GTK_WIDGET(gui.filedlg), VW_POS_MOUSE);
  276. ***************
  277. *** 900,906 ****
  278.   
  279.   #endif    /* FEAT_BROWSE */
  280.   
  281. ! #ifdef FEAT_GUI_DIALOG
  282.   
  283.   static char_u *dialog_textfield = NULL;
  284.   static GtkWidget *dialog_textentry;
  285. --- 905,911 ----
  286.   
  287.   #endif    /* FEAT_BROWSE */
  288.   
  289. ! #if defined(FEAT_GUI_DIALOG) || defined(PROTO)
  290.   
  291.   static char_u *dialog_textfield = NULL;
  292.   static GtkWidget *dialog_textentry;
  293. *** ../vim60.162/src/gui_motif.c    Wed Sep 26 16:47:48 2001
  294. --- src/gui_motif.c    Sun Feb  3 14:23:53 2002
  295. ***************
  296. *** 1531,1537 ****
  297.   
  298.       str = XmStringCreate(new_label, STRING_TAG);
  299.   
  300. !     if (str) {
  301.       XtVaSetValues(w, XmNlabelString, str, NULL);
  302.       XmStringFree(str);
  303.       }
  304. --- 1531,1538 ----
  305.   
  306.       str = XmStringCreate(new_label, STRING_TAG);
  307.   
  308. !     if (str)
  309. !     {
  310.       XtVaSetValues(w, XmNlabelString, str, NULL);
  311.       XmStringFree(str);
  312.       }
  313. ***************
  314. *** 1552,1568 ****
  315.       char_u    *filter;    /* file name filter */
  316.   {
  317.       char_u    dirbuf[MAXPATHL];
  318.       char_u    *pattern;
  319.       char_u    *tofree = NULL;
  320.   
  321.       dialog_wgt = XmCreateFileSelectionDialog(vimShell, (char *)title, NULL, 0);
  322.   
  323. -     if (dflt == NULL)
  324. -     dflt = (char_u *)"";
  325.       if (initdir == NULL || *initdir == NUL)
  326.       {
  327.       mch_dirname(dirbuf, MAXPATHL);
  328.       initdir = dirbuf;
  329.       }
  330.   
  331.       /* Can only use one pattern for a file name.  Get the first pattern out of
  332. --- 1553,1580 ----
  333.       char_u    *filter;    /* file name filter */
  334.   {
  335.       char_u    dirbuf[MAXPATHL];
  336. +     char_u    dfltbuf[MAXPATHL];
  337.       char_u    *pattern;
  338.       char_u    *tofree = NULL;
  339.   
  340.       dialog_wgt = XmCreateFileSelectionDialog(vimShell, (char *)title, NULL, 0);
  341.   
  342.       if (initdir == NULL || *initdir == NUL)
  343.       {
  344.       mch_dirname(dirbuf, MAXPATHL);
  345.       initdir = dirbuf;
  346. +     }
  347. + 
  348. +     if (dflt == NULL)
  349. +     dflt = (char_u *)"";
  350. +     else if (STRLEN(initdir) + STRLEN(dflt) + 2 < MAXPATHL)
  351. +     {
  352. +     /* The default selection should be the full path, "dflt" is only the
  353. +      * file name. */
  354. +     STRCPY(dfltbuf, initdir);
  355. +     add_pathsep(dfltbuf);
  356. +     STRCAT(dfltbuf, dflt);
  357. +     dflt = dfltbuf;
  358.       }
  359.   
  360.       /* Can only use one pattern for a file name.  Get the first pattern out of
  361. *** ../vim60.162/src/message.c    Fri Sep 28 22:19:57 2001
  362. --- src/message.c    Sun Feb  3 13:42:00 2002
  363. ***************
  364. *** 2475,2483 ****
  365.   do_browse(saving, title, dflt, ext, initdir, filter, buf)
  366.       int        saving;        /* write action */
  367.       char_u    *title;        /* title for the window */
  368. !     char_u    *dflt;        /* default file name */
  369.       char_u    *ext;        /* extension added */
  370. !     char_u    *initdir;    /* initial directory, NULL for current dir */
  371.       char_u    *filter;    /* file name filter */
  372.       buf_T    *buf;        /* buffer to read/write for */
  373.   {
  374. --- 2475,2484 ----
  375.   do_browse(saving, title, dflt, ext, initdir, filter, buf)
  376.       int        saving;        /* write action */
  377.       char_u    *title;        /* title for the window */
  378. !     char_u    *dflt;        /* default file name (may include directory) */
  379.       char_u    *ext;        /* extension added */
  380. !     char_u    *initdir;    /* initial directory, NULL for current dir or
  381. !                    when using path from "dflt" */
  382.       char_u    *filter;    /* file name filter */
  383.       buf_T    *buf;        /* buffer to read/write for */
  384.   {
  385. ***************
  386. *** 2498,2505 ****
  387.           title = (char_u *)_("Open File dialog");
  388.       }
  389.   
  390. !     /* When no directory specified, use default dir, buffer dir, last dir
  391. !      * or current dir */
  392.       if (initdir == NULL || *initdir == NUL)
  393.       {
  394.       /* When 'browsedir' is a directory, use it */
  395. --- 2499,2525 ----
  396.           title = (char_u *)_("Open File dialog");
  397.       }
  398.   
  399. !     /* When no directory specified, use default file name, default dir, buffer
  400. !      * dir, last dir or current dir */
  401. !     if ((initdir == NULL || *initdir == NUL) && dflt != NULL && *dflt != NUL)
  402. !     {
  403. !     if (mch_isdir(dflt))        /* default file name is a directory */
  404. !     {
  405. !         initdir = dflt;
  406. !         dflt = NULL;
  407. !     }
  408. !     else if (gettail(dflt) != dflt)    /* default file name includes a path */
  409. !     {
  410. !         tofree = vim_strsave(dflt);
  411. !         if (tofree != NULL)
  412. !         {
  413. !         initdir = tofree;
  414. !         *gettail(initdir) = NUL;
  415. !         dflt = gettail(dflt);
  416. !         }
  417. !     }
  418. !     }
  419. ! 
  420.       if (initdir == NULL || *initdir == NUL)
  421.       {
  422.       /* When 'browsedir' is a directory, use it */
  423. ***************
  424. *** 2509,2515 ****
  425.       else if ((saving || *p_bsdir == 'b')
  426.           && buf != NULL && buf->b_ffname != NULL)
  427.       {
  428. !         dflt = gettail(curbuf->b_ffname);
  429.           tofree = vim_strsave(curbuf->b_ffname);
  430.           if (tofree != NULL)
  431.           {
  432. --- 2529,2536 ----
  433.       else if ((saving || *p_bsdir == 'b')
  434.           && buf != NULL && buf->b_ffname != NULL)
  435.       {
  436. !         if (dflt == NULL || *dflt == NUL)
  437. !         dflt = gettail(curbuf->b_ffname);
  438.           tofree = vim_strsave(curbuf->b_ffname);
  439.           if (tofree != NULL)
  440.           {
  441. ***************
  442. *** 2560,2566 ****
  443.           *gettail(last_dir) = NUL;
  444.           if (*last_dir == NUL)
  445.           {
  446. !         /* filename only returned, must be in current dir*/
  447.           vim_free(last_dir);
  448.           last_dir = alloc(MAXPATHL);
  449.           if (last_dir != NULL)
  450. --- 2581,2587 ----
  451.           *gettail(last_dir) = NUL;
  452.           if (*last_dir == NUL)
  453.           {
  454. !         /* filename only returned, must be in current dir */
  455.           vim_free(last_dir);
  456.           last_dir = alloc(MAXPATHL);
  457.           if (last_dir != NULL)
  458. *** ../vim60.162/src/version.c    Sun Feb  3 12:42:13 2002
  459. --- src/version.c    Sun Feb  3 15:16:22 2002
  460. ***************
  461. *** 608,609 ****
  462. --- 608,611 ----
  463.   {   /* Add new patch number below this line */
  464. + /**/
  465. +     163,
  466.   /**/
  467.  
  468. -- 
  469. hundred-and-one symptoms of being an internet addict:
  470. 168. You have your own domain name.
  471.  
  472.  ///  Bram Moolenaar -- Bram@moolenaar.net -- http://www.moolenaar.net  \\\
  473. (((   Creator of Vim -- http://vim.sf.net -- ftp://ftp.vim.org/pub/vim   )))
  474.  \\\  Help me helping AIDS orphans in Uganda - http://iccf-holland.org  ///
  475.