[Libreoffice] Doubt about bogus and commented code

[Xisco Faulí]

Hello there,

I've seen quite often something like this:

-/*
- #i2345#
- if ( nSlotId >= SID_SFX_START && !SfxMenuManager::IsPopupFunction(nSlotId)
)
- {
- // Echte Popups sollen keine SlotIds haben; leider sind
- // da noch Altlasten mit herumzuschleppen ...
- String aTitle = pSVMenu->GetItemText( nSlotId );
- pSVMenu->SetPopupMenu( nSlotId, NULL );
- USHORT nPos = pSVMenu->GetItemPos( nSlotId );
- pSVMenu->RemoveItem( nPos );
- nSlotId = 1;- while ( pSVMenu->GetItemPos(nSlotId) != MENU_ITEM_NOTFOUND )
- nSlotId++; - pSVMenu->InsertItem( nSlotId, aTitle, 0, nPos ); -
pSVMenu->SetPopupMenu( nSlotId, pPopup ); - } -*/ What should I do with it ?
leave it like that ? delete it completery ? or delete the code and leave the
bogus ?

Btw, what about the peaces which say, FIXME ?
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/libreoffice/attachments/20110312/ac9c89a8/attachment.html>