Skip to content

Files & Folders Patch Pack

Thomas Debesse requested to merge illwieckz/netradiant:files into master

I made multiple merge requests on different stuff that take places in the same area of code. The problem is: if every MR looks to be mergeable on master, they go insanely conflicting when trying to merge them one after one.

For example the cmdline MR !55 (closed) and lastfolder MR !46 (closed) are adding code at the exact same place, and git can't know by itself what to do in this case:

  • cmdline MR excerpt:
@@ -933,4 +933,6 @@ void Map_LoadFile( const char *filename ){
 	ScopeDisableScreenUpdates disableScreenUpdates( "Processing...", "Loading Map" );
 
+	MRU_AddFile( filename );
  • lastfolder MR excerpt:
@@ -933,4 +933,6 @@ void Map_LoadFile( const char *filename ){
 	ScopeDisableScreenUpdates disableScreenUpdates( "Processing...", "Loading Map" );
 
+	g_strLastFolder = g_path_get_dirname( filename );

This kind of simple things can make merging very annoying.

So, this MR ships all these others MR with all merge issues resolved:

If you're OK with all these changes, you can merge this one to save many headaches, and close all the related MR.

You can also compile that files branch to get an overview of how all these improvements glued together makes the user experience far far more polished, because all these MR are about making user interaction a more polished experience.

I had a special care to make each commit easily readable, so reviewing commit per commit is a bit like reviewing all these MR one at a time.

Edited by Thomas Debesse

Merge request reports