From: Dale A. <da...@gr...> - 2010-12-29 20:48:18
|
Personally, I think that this change should be moved to 4.5. The previous revision (19139) was sufficient to fix the original bug (3134215), but it did not address all concerns. I'm not happy with the fix in 19155, I guess there's really nothing wrong with it, but I'd really like to see the split config parsing code centralized and consolidated. Also, I don't have Windows to test with. Alan, do you? Dale On Wed, Dec 29, 2010 at 8:29 AM, Kazutoshi Satoda <k_s...@f2...> wrote: > Hi Dale and Alan, > (Very sorry about duplicate mail, I sent CC to jedit-cvs instead of > jedit-devel. Now sending correct CC to jedit-devel.) > > ez...@us... wrote: >> >> Revision: 19155 >> http://jedit.svn.sourceforge.net/jedit/?rev=19155&view=rev >> Author: ezust >> Date: 2010-12-17 20:51:44 +0000 (Fri, 17 Dec 2010) >> >> Log Message: >> ----------- >> Merge request 3139281, Merge 19144 to 4.4.x. > > After updating 4.4.x to r19158 and restarting jEdit, I noticed that > restoring perspective is broken. > > Before closing jEdit, I had test.txt in C:\home\sandbox\. After > restarting jEdit, I got homesandboxtest.txt in C:\. > > Reverting r19156-19158 did fix this. > > I suspect that '\' in perspective.xml is wrongly treated somewhere in > the new code. > > Could you please test this on a Windows environment? > > Beside this, I'm not happy to see a new option in release branch. > Could you please minimize the change just enough to fix the problem? > > -- > k_satoda > |