Support and General Use > Recording
Recording Enhancements Pack
vinylivo:
I just updated the AGC patch: http://forums.rockbox.org/index.php?topic=2841.msg20404#msg20404
paulheu:
--- Quote from: Mmmm on March 05, 2006, 02:40:48 PM ---I have finally, with much gnashing of teeth and pulling of hair, managed to work out how the gui system works and have managed to implement the WPS peakmeter into the remote....Hooorraaaaaaaaaaaaaaaaahhhh.... :o
--- End quote ---
But.. but I have had the recording screen on my remote from the start..
Mmmm:
--- Quote from: paulheu on March 06, 2006, 06:08:23 AM ---
--- Quote from: Mmmm on March 05, 2006, 02:40:48 PM ---I have finally, with much gnashing of teeth and pulling of hair, managed to work out how the gui system works and have managed to implement the WPS peakmeter into the remote....Hooorraaaaaaaaaaaaaaaaahhhh.... :o
--- End quote ---
But.. but I have had the recording screen on my remote from the start..
--- End quote ---
You mean with the patch in post#1 right?
The recording screen isn't implemented on the remote with the normal build yet!
What I meant is...
The patches in post#1 are hacks that won't ever get implemented into CVS, so you will always need to use a special build or a patch to record from the remote screen. They work fine but they don't mesh with the rest of the rockbox code very well (meaning that a peakmeter for the recording screen is all very nice but what if you want to use it in the rwps or for the radio? You just can't get it to work with my code...well, not without a total rewrite and downgrading of the current WPS code anyway..... So therefore not an acceptable solution).
I am working on a version that will work for any target, (without using any preprocessing(#if/#ifdef)) and blends in with the already present methods of using the display...much harder to do but I'm nearly there now!
It will be more flexible, robust and just generally easier to maintain/alter. Of course any non programmers won't notice the difference, it's just that more will be possible with the new code! It is so flexible that even if a player is introduced that has 3 screens it would automatically support the 3rd screen no matter what size it is! Without changing any code at all!!!!! [/advert] :D
When I've finished, with a bit of luck someone will decide to commit it so that you no longer need a patch! (I get the feeling this will be the most difficult bit!)
I hope that explains what I'm up to a bit more clearly! :)
When I do get it finished i'll post the patch/build here and it would really great it if everyone could test it, (if it works properly you should notice no difference whatsoever (apart from %pm working in rWPS). Then I can fine tune it just like I did with the hack patch to get it really perfect.
petur:
--- Quote from: Mmmm on March 06, 2006, 06:53:46 AM ---When I've finished, with a bit of luck someone will decide to commit it so that you no longer need a patch! (I get the feeling this will be the most difficult bit!)
--- End quote ---
Nice to see you're writing code that is committable :P
The problem is I don't have a remote (actually the bigger problem I don't have time), so this will be hard for me to test before commit - I may have to trust you :P :P
Mmmm:
Damn...there I go modifying your posts again.... :D
I think I put it all back in place....
--- Quote ---
--- Quote from: Mmmm on March 06, 2006, 06:53:46 AM ---When I've finished, with a bit of luck someone will decide to commit it so that you no longer need a patch! (I get the feeling this will be the most difficult bit!)
--- End quote ---
Nice to see you're writing code that is committable :P
--- End quote ---
:P
--- Quote ---The problem is I don't have a remote (actually the bigger problem I don't have time), so this will be hard for me to test before commit - I may have to trust you :P :P
--- End quote ---
I promise I will test till I'm blue in the face..... :D
should I compile and test for all targets or are there just a select few that will do the trick?
Navigation
[0] Message Index
[#] Next page
[*] Previous page
Go to full version