The Mana World Forums

I really hate to do this, but I have practically no time at all.

I have some code on o11c/test that redoes the file IO subsystem. It needs verification that reading and writing all files is correct, from all paths.

This is a list of call sites that need to be checked. Additionally, they should be checked for both the case where the file exists and the case where it does not exist.

• login server
  • append ladmin log
  • read ladmin stdin
  • read ladmin cfg
• eathena monitor
  • read monitor cfg (not used)
• char server
  • read party
  • lock party
  • append char log
  • read char
  • lock char
  • write online txt (dangerous, but subject to permission problems otherwise, will be fixed properly later - this is not a new bug)
  • write online html (likewise)
  • read char lancfg
  • read char cfg
  • read storage
  • lock storage
  • read accreg
  • lock accreg
  • read inter cfg
• login server
  • append login log
  • read gm
  • read account
  • lock account
  • append gm
  • append login unknown packets
  • lock gm
  • read gm
  • append login unknown packets
  • append login unknown packets
  • read login lancfg
  • read login cfg
• map server
  • read npc source
  • read resnametable
  • append packet_txt
  • read mob db
  • read mob skill db
  • read battle cfg
  • write atcommand conf
  • append gm log
  • read const
  • read mapreg
  • lock mapreg
  • read skill
  • read item
  • read motd
  • append map log
  • read map cfg

open("npc/functions/barber.txt", O_RDONLY|O_CLOEXEC) = 4
read(4, "// The Barber script has been ar"..., 4096) = 3417
and sits there forever.

Don't you hate it when pushing one segment of code makes something else break?

Workaround pushed ... not sure if I'll investigate the real problem or leave it until my general rewrite of that area.

Appears to work fine for me.