Re: port-sparc64/46652 (sparc ofwboot.net fails to load kernel viatftp)

[Izumi Tsutsui <tsutsui%ceres.dti.ne.jp@localhost>]

I wrote:
> > If the reason why it's int ** is too confusing, a comment should be added.
> > 
> > The code would be less clear to me, not more, if you handed over the
> > pointer to the entire open_file struct when all you want to do is set
> > one member of it (which is a dedicated "I'm void * so you can stick in
> > whatever both ends need", too).
> 
> What kind of comment is reasonable for you?
> Even if you don't want void *, Why not int * but int **?
> I don't see any reason to prefer int **.

Ah, my bad, int * won't work here. I was actually confused...

But I still prefer passing struct open_file * as tftp_open().

---
Izumi Tsutsui