git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[PATCH] Style changes per suggestions from Junio in #git

From
RBR. Tyler Ballance <tyler@slide.com>
Date
Dec 21, 2008, 21:37 UTC
Message-ID
<1229895454-19498-3-git-send-email-tyler@slide.com>
In-Reply-To
<1229895454-19498-2-git-send-email-tyler@slide.com>
Moving includes into git-compat-util.h, move away from malloc(2)
Signed-off-by: R. Tyler Ballance <tyler@slide.com>
---
 environment.c     |    9 +++------
 git-compat-util.h |    2 ++
 2 files changed, 5 insertions(+), 6 deletions(-)
diff --git a/environment.c b/environment.c
index a3b6bab..aa36360 100644
--- a/environment.c
+++ b/environment.c
@@ -7,8 +7,6 @@
  * even if you might want to know where the git directory etc
  * are.
  */
-#include <sys/time.h>
-#include <sys/resource.h>
 
 #include "cache.h"
 
@@ -80,10 +78,9 @@ static void setup_git_env(void)
 		git_graft_file = git_pathdup("info/grafts");
 	
 	if (DYNAMIC_WINDOW_SIZE) {
-		struct rlimit *as = malloc(sizeof(struct rlimit));
-		if ( (getrlimit(RLIMIT_AS, as) == 0) && ((int)(as->rlim_cur) > 0) ) 
-			packed_git_window_size = (unsigned int)(as->rlim_cur * DYNAMIC_WINDOW_SIZE_PERCENTAGE);
-		free(as);
+		struct rlimit as;
+		if (getrlimit(RLIMIT_AS, &as) == 0 && (int)as.rlim_cur > 0)
+			packed_git_window_size = (unsigned int)(as.rlim_cur * DYNAMIC_WINDOW_SIZE_PERCENTAGE);
 	}
 }
 
diff --git a/git-compat-util.h b/git-compat-util.h
index 9603ca6..dad2dc8 100644
--- a/git-compat-util.h
+++ b/git-compat-util.h
@@ -188,6 +188,8 @@ extern int git_munmap(void *start, size_t length);
 #else /* NO_MMAP */
 
 #include <sys/mman.h>
+#include <sys/time.h>
+#include <sys/resource.h>
 
 /* This value must be multiple of (pagesize * 2) */
 #define DEFAULT_PACKED_GIT_WINDOW_SIZE \
Previous: R. Tyler BallanceNext: Matthieu Moy
Message 3 of 8 in “Dynamically adjusting packed_git_window_size”
  1. R. Tyler BallanceDec 21, 2008
  2. Add support for changing packed_git_window_size at process start timeR. Tyler Ballance, Dec 21, 2008
  3. Style changes per suggestions from Junio in #gitR. Tyler Ballance, Dec 21, 2008
  4. Matthieu MoyDec 21, 2008
  5. Shawn O. PearceDec 21, 2008
  6. Shawn O. PearceDec 21, 2008
  7. R. Tyler BallanceDec 22, 2008
  8. Shawn O. PearceDec 26, 2008

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.