|
From: Michael J. <mrm...@ho...> - 2008-01-18 14:57:21
|
> I removed some parts of the patch were there were still strings in the fo=
rm TSTRING(BLAH) + xxx
> which should become TF_("Blah %1%") % xxx so that the parameter can be mo=
ved...
Sorted them now.
> By my count, there are now approximately 290 strings left (more or less 1=
45 in each of win32 and
> dcpp =3D) and about the same number already done which means that we're a=
t 50% !
nice, going well then. :)
>> How should CTSTRING(BLAH) end up ?
>CT_ (which is a new macro: "#define CT_(String) T_(String).c_str()")
i didn't realise that this was so new it needed putting in stdafx.h =3D)
And for STRINGS(BLAH) should the T_ macro be used, or a different one ?
Also what macro should be used in the dcpp area ?
Have a few queries for the ones I working on now (still win32):
SearchFrame.cpp:1056
For the search window's title bar which is better ?
setText(T_("Search") + _T(" - ") + T_("Ready to search..."));
setText(T_("Search - Ready to search..."));
Also wasn't sure if the Registry Entries (e.g. for Magnet stuff) in WinUtil=
.cpp should be translatable or not ? Or just some of them.=20
( Especially regarding MAGNET_HANDLER_ROOT which is "DC++" )
This was used in PublicHubFrame, before the cache was introduced. Since no=
thing uses it now, can it be removed, or it want to stay ?=20
#ifdef PORT_ME
double WinUtil::toBytes(TCHAR* aSize) {
blah
Also do the "new"s in SettingsFrame need a corresponding "delete"? Because =
i couldn't see the pages getting deleted anywhere.
There is also a "new" in Util formatTime for non win32 stuff, but i couldn'=
t see any delete for that either.
And last thing, there seems to be a issue with SpyFrame. Open Spyframe and =
just put dc++ in the background for a few minutes and it crashes most the t=
ime. If it doesn't after about 5 mins, clicking the Settings button makes =
it crash. Never get an exceptioninfo or a call stack when ran in debug bui=
ld though. But at least it is reproducible. :)
thanks=20
MikeJJ
_________________________________________________________________
Free games, great prizes - get gaming at Gamesbox.=20
http://www.searchgamesbox.com=
|