> - 8aa33e3e Rwrite the latency timing/calculation for the OSS backend > There is parctically no explanation why the change is good and why > a new mutex is needed. It looks dangerous. How extensively has the > code been excercised? Why is a change needed at all? I am not aware > of people complaining that the OSS driver does not work. But I > might be missing something.
The OSS latency calculation in 2.1CV is completely nonfunctional on the native OSS, OSS-via-ALSA-emu and OSS-via-ALSA-emu-via-Pulse emu machines here. The OSS driver only 'works' when the driver-based latency calculation is disabled (the 'enable software sync' preferences box, which doesn't enable anything-- it disables driver latency polling and has playback 'fly blind'). People are, in general, used to sync always being broken. :-) > - e0569663 Actually check for error return codes in OSS > This does not adhere to the coding standard that is used in surrounding > code. Fixing that would take too much of my time. Remember that it is > good tone to copy the coding style that you find rather than to force > your own style onto existing code. This includes white-space style. > (IOW, please use tabs, not 8 spaces.) My emacsen here are actually forced to use spaces and two-space indent. Not sure how tabs ended up in it. As for stylistic 'forcing' in most projects here at least, we've long since abandoned the formatting war over here. He who writes the code gets to format it, and for the most part we loosely follow the surrounding style. But insisting on whitespace-perfect style emulation... urk. Or is this about braces? :-) > - 64e8022f Eliminate the YUV file loader's wont > Before I merge this change, I'd like to know why the patch solves a > problem (and what the problem exactly is). "It happens more often > than you think" is a bit too much hand-waving. Can I easily reproduce > the bug? (Oh, and the commant about coding style applies, too.) The YUV4MPEG2 loader was building a from-scratch transient index that required reading every byte in the raw file on every call to open. Open is called *often*, not just when the project is loaded. If the loader already had an index, it threw it away. It was all in memory, nothing was saved. This means that, for example, every undo required every y4m file in the project to rebuild indexes from scratch. On the movie I was working on, that was 20 minutes as it re-read just short of a terabyte of video... ...for no reason. The index was useless, every frame is the same size. The y4m spec doesn't allow frame size deviation (that includes frame headers). I realize this is all still to terse; I'll elaborate in more detail on each point once I've gotten some sleep. Cheers! Monty _______________________________________________ Cinelerra mailing list [email protected] https://init.linpro.no/mailman/skolelinux.no/listinfo/cinelerra
