Re: [PATCHES] odd output in restore mode

Bruce Momjian wrote:
> Martin Zaun wrote:
>> 4. Issue: missing break in switch, silent override of '-l' argument?
>>
>> This behaviour has been in there before and is not addresses by the
>> patch: The user-selected Win32 "mklink" command mode is never applied
>> due to a missing 'break' in CustomizableInitialize():
>>
>>      switch (restoreCommandType)
>>      {
>>          case RESTORE_COMMAND_WIN32_MKLINK:
>>              SET_RESTORE_COMMAND("mklink", WALFilePath, xlogFilePath);
>>          case RESTORE_COMMAND_WIN32_COPY:
>>              SET_RESTORE_COMMAND("copy", WALFilePath, xlogFilePath);
>>              break;
> 
> I have added the missing 'break' to CVS HEAD;  thanks.

Why no backpatch to 8.3? Seems like a clear bugfix to me.

//Magnus