Commit df13a1d1 authored by Vincent Povirk's avatar Vincent Povirk Committed by Alexandre Julliard

ole32: Change WriteDirEntry return type to HRESULT.

parent 382ffed0
...@@ -978,11 +978,9 @@ static HRESULT WINAPI StorageBaseImpl_SetClass( ...@@ -978,11 +978,9 @@ static HRESULT WINAPI StorageBaseImpl_SetClass(
{ {
currentEntry.clsid = *clsid; currentEntry.clsid = *clsid;
success = StorageImpl_WriteDirEntry(This->ancestorStorage, hRes = StorageImpl_WriteDirEntry(This->ancestorStorage,
This->storageDirEntry, This->storageDirEntry,
&currentEntry); &currentEntry);
if (success)
hRes = S_OK;
} }
return hRes; return hRes;
...@@ -2091,13 +2089,13 @@ static HRESULT removeFromTree( ...@@ -2091,13 +2089,13 @@ static HRESULT removeFromTree(
*/ */
setEntryLink(&parentEntry, typeOfRelation, entryToDelete.leftChild); setEntryLink(&parentEntry, typeOfRelation, entryToDelete.leftChild);
res = StorageImpl_WriteDirEntry( hr = StorageImpl_WriteDirEntry(
This, This,
parentEntryRef, parentEntryRef,
&parentEntry); &parentEntry);
if(!res) if(FAILED(hr))
{ {
return E_FAIL; return hr;
} }
if (entryToDelete.rightChild != DIRENTRY_NULL) if (entryToDelete.rightChild != DIRENTRY_NULL)
...@@ -2127,13 +2125,13 @@ static HRESULT removeFromTree( ...@@ -2127,13 +2125,13 @@ static HRESULT removeFromTree(
newRightChildParentEntry.rightChild = entryToDelete.rightChild; newRightChildParentEntry.rightChild = entryToDelete.rightChild;
res = StorageImpl_WriteDirEntry( hr = StorageImpl_WriteDirEntry(
This, This,
newRightChildParent, newRightChildParent,
&newRightChildParentEntry); &newRightChildParentEntry);
if (!res) if (FAILED(hr))
{ {
return E_FAIL; return hr;
} }
} }
} }
...@@ -2144,13 +2142,13 @@ static HRESULT removeFromTree( ...@@ -2144,13 +2142,13 @@ static HRESULT removeFromTree(
*/ */
setEntryLink(&parentEntry, typeOfRelation, entryToDelete.rightChild); setEntryLink(&parentEntry, typeOfRelation, entryToDelete.rightChild);
res = StorageImpl_WriteDirEntry( hr = StorageImpl_WriteDirEntry(
This, This,
parentEntryRef, parentEntryRef,
&parentEntry); &parentEntry);
if(!res) if(FAILED(hr))
{ {
return E_FAIL; return hr;
} }
} }
...@@ -3334,7 +3332,7 @@ BOOL StorageImpl_ReadDirEntry( ...@@ -3334,7 +3332,7 @@ BOOL StorageImpl_ReadDirEntry(
/********************************************************************* /*********************************************************************
* Write the specified directory entry to the file * Write the specified directory entry to the file
*/ */
BOOL StorageImpl_WriteDirEntry( HRESULT StorageImpl_WriteDirEntry(
StorageImpl* This, StorageImpl* This,
DirRef index, DirRef index,
const DirEntry* buffer) const DirEntry* buffer)
...@@ -3345,7 +3343,7 @@ BOOL StorageImpl_WriteDirEntry( ...@@ -3345,7 +3343,7 @@ BOOL StorageImpl_WriteDirEntry(
UpdateRawDirEntry(currentEntry, buffer); UpdateRawDirEntry(currentEntry, buffer);
writeRes = StorageImpl_WriteRawDirEntry(This, index, currentEntry); writeRes = StorageImpl_WriteRawDirEntry(This, index, currentEntry);
return SUCCEEDED(writeRes) ? TRUE : FALSE; return writeRes;
} }
static BOOL StorageImpl_ReadBigBlock( static BOOL StorageImpl_ReadBigBlock(
......
...@@ -336,7 +336,7 @@ BOOL StorageImpl_ReadDirEntry( ...@@ -336,7 +336,7 @@ BOOL StorageImpl_ReadDirEntry(
DirRef index, DirRef index,
DirEntry* buffer); DirEntry* buffer);
BOOL StorageImpl_WriteDirEntry( HRESULT StorageImpl_WriteDirEntry(
StorageImpl* This, StorageImpl* This,
DirRef index, DirRef index,
const DirEntry* buffer); const DirEntry* buffer);
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment