- 1
- 2
- 3
- 4
- 5
- 6
- 7
- 8
- 9
- 10
- 11
- 12
- 13
- 14
- 15
- 16
- 17
- 18
- 19
- 20
- 21
- 22
- 23
- 24
- 25
- 26
- 27
- 28
- 29
- 30
- 31
- 32
- 33
- 34
- 35
- 36
- 37
- 38
static
{
unionSize = Math.max(6, 6);
size = 2 + unionSize + 4;
}
// следующий участок
case 7:
{
int result = (int)((long)getValue(cmd.isByteMode(), mem, op1) & 0L);
if(cmd.isByteMode())
result &= 0xff;
setValue(cmd.isByteMode(), mem, op1, result);
flags = result != 0 ? result & VMFlags.VM_FS.getFlag() : VMFlags.VM_FZ.getFlag();
break;
}
case 49:
{
setValue(true, mem, op1, (int)((long)getValue(true, mem, op1) & 0L));
break;
}
case 50:
{
setValue(false, mem, op1, (int)((long)getValue(false, mem, op1) & 0L));
break;
}
//=========== еще участок ============
long dataSize = (long)ReadData(this) & 0L;
for(int i = 0; inAddr < codeSize && (long)i < dataSize; i++)
{
prg.getStaticData().addElement(new Byte(valueOf((byte)(fgetbits() >> 8))));
faddbits(8);
}
похоже, писал сишник до мозга костей, такие любят писать в низкоуровневом стиле... (к примеру, деление на два обязательно заменят сдвигом)
Прикол тут вовсе не тут, как было отмечено, а в том о чем говорил wvxvw, например.
И если вы не заметили
Вообще не исполниться, так как dataSize всегда ноль (любое число & 0 будет 0), и следственно (long)i < dataSize тут будет false с самого начала.
и
это одно и то же, или какой смысл был во всех этих конверациях?
Кстати еще веселый момент,
На самом деле можно написать так:
{
unionSize = Math.max(6, 6);
size = 2 + unionSize + 4;
}
Ну это нормально, только заменить бы ещё 2,4,6 на sizeof нужных типов ну и какого хуя это делает в java-коде?
Math.max(int1, in2) возвращает то, что больше
Тут то просто
должно быть. Кстати когда я так и сделал, оказалось, что переменная unionSize используеться только в инициализаторе, и на самом деле инициализатор получаеться таким
Аналогично с 2+unionsize+4 - тут видно, что это размер структуры из 3 полей, первое двубайтовое, второе - это тот union, третье - 4-байтовое.
Поэтому отмени свою "оптимизацию" и оставь всё так, как было, потому что это было неспроста.
Тогда можно было бы позвонить в астральную канцелярию и домогаться смысла, но в предоставленном варианте, это просто какое-то недоразумение.
везёт
level - Asian?