Re: [dcplusplus-devel] Some more translations again

[Michael J. <mrm...@ho...>]

Just a few more queries to finish off most off win32 (excluding the stuff w=
hich may need Keyboard Accelerators)=20

The F_ macro doesn't work in win32.  Somethings in HubFrame.cpp and WinUtil=
.cpp could do with it. e.g.=20
	speak(ADD_STATUS_LINE, STRING(CONNECTING_TO) + client->getHubUrl() + "..."=
);
	LogManager::getInstance()->message(STRING(MAGNET_DLG_TITLE) + ": " + Text:=
:fromT(aUrl));

Should something like this be added to stdafx.h ?

#define F_(String) boost::format(gettext(String))
or
#define F_(String) boost::format(dgettext(PACKAGE, String))

and if so, inside or outside the ifdef unicode thingy ?


I really have no idea of what to do to the two functions which do this
	if (ResourceManager::getInstance()->isRTL())


The Toolbar stuff in MainWindow.cpp, e.g.
	toolbar->appendItem(IDC_RECONNECT, image++, TSTRING(MENU_RECONNECT));

i assume this is for the Tooltips.  But the tooltips don't display accelera=
tors or shortcut keys, so should it keep them ?=20
(not all of them use the right stringdef for the accelerators & shortcuts k=
eys to be displayed, so if keeping, should these be changed to the ones whi=
ch have them ?)


Also is the AWAY string in caps for a reason ? =20
The two places which use it are MainWindow.cpp:294
	statusSizes[STATUS_AWAY] =3D status->getTextSize(TSTRING(AWAY)).x + 12;

and MainWindow.cpp:550
	setStatus(STATUS_AWAY, Util::getAway() ? TSTRING(AWAY) : _T(""));

thanks
MikeJJ
_________________________________________________________________
Get Hotmail on your mobile, text MSN to 63463!
http://mobile.uk.msn.com/pc/mail.aspx=