|
Rockbox mail archiveSubject: Re: Recorder experimental new UI, Sound Sliders and larger Tetris source codeRe: Recorder experimental new UI, Sound Sliders and larger Tetris source code
From: Robert Hak <rhak_at_ramapo.edu>
Date: Fri, 21 Feb 2003 14:35:05 -0500 (EST) ---quote--- > Why do you replace proper C comments /* */ with C++isms of //? > There's a good reason for // commenting: ---unquote-- I realize the reason why you did the // what I mean is it is not acceptable for a patch. ---quote--- > Also, use the patch tracker if you want to submist patches. I'm knew to this, didn't know there was a patch tracker. ---unquote--- Welcome to the group :) ---quote--- I mailed the patches to the list because I was asked to do it. The base code is version 1.4, the patches have to be changed to be applied to the current version. I posted the compiled code for evaluation, if the patches are to be applied I'll re-do them for the current version. ---unquote--- I'd go the other way. We can't consider patches for inclusion until we see them based off the current code (well.. more current then 1.4) Try and convert what you have to several small patches and put them in. You are much more likely to get them accepted. "I'll do it once they are accepted" won't gain you much ground. /Adi Received on 2003-02-21 Page template was last modified "Tue Sep 7 00:00:02 2021" The Rockbox Crew -- Privacy Policy |