mirror of https://github.com/bsnes-emu/bsnes.git
Update to bsnes v038r01? release.
New WIP. Audio panel was revised, it's now the same both in regular and advanced mode. Volume, Frequency and Latency all appear on one row and are all combo boxes with sane defaults. Advanced mode gives them additional options. Frequency adjust remains a slider. Given that 1 tick can mean the difference between frame stuttering once a minute and once an hour, I think it's worth keeping the precision. Code-wise, I merged the ppucounter object into the PPU class (through inheritance). This results in the following code simplification: Before: ppucounter.vcounter() //S-CPU time ppucounter.ppuvcounter() //S-PPU time After: ppu.vcounter() //S-CPU time ivcounter() //S-PPU time (called inside its own class, no need for ppu. prefix) i just stands for "internal". It was that or slow things down with a co_active() check inside the counter read calls. Man, it feels weird editing C++ code after all of that CSS magic. I find myself wanting to write a pattern-matching rule ... uint16 PPUcounter::vcounter() { return cpu_vcounter(); } uint16 PPUcounter::vcounter() [class="PPU"] { return ppu_vcounter(); } > I probably would have stuck it in a carefully-styled <SPAN> rather > than an attribute, but I notice your approach is sanctioned by HTML5 > (or at least it would be if you called it "data-date" instead of > "date"). <h3><span>2008-12-20</span>Title</h3> <blockquote><p>All that is necessary for the triumph of evil is that good men do nothing<span>Edmund Burke</span></p></blockquote> Could work ... looks weird, though. Adding a class type to the spans to state what they are makes it even more bloated, but perhaps worth it ... hmm. HTML5 approach looks cool, too. data- prefix isn't too bad. A good indicator that it's not a real tag. EDIT: oops, looks like I forgot that IE6 can't handle the text-align attribute properly, either. Eh, I'll fix it tomorrow. Tired now. [No archive available]
This commit is contained in:
parent
c13ae98863
commit
9c7ac24ff7