[PR #310] [CLOSED] Core: Added size_t as a Value supported type. #303

New issue

Closed

opened 2026-05-05 03:45:12 -06:00 by gitea-mirror · 0 comments

gitea-mirror commented

2026-05-05 03:45:12 -06:00

Owner

📋 Pull Request Information

Original PR: https://github.com/ultimatepp/ultimatepp/pull/310
Author: @klugier
Created: 10/16/2025
Status: ❌ Closed

Base: master ← Head: klugier/value-size-t

📝 Commits (3)

c0367e5 Core: Added size_t as a Value supported type.
76e4609 .cosmetics
a576d0f .cosmetics

📊 Changes

4 files changed (+22 additions, -0 deletions)

View changed files

📝 uppsrc/Core/Defs.h (+3 -0)
📝 uppsrc/Core/Value.cpp (+9 -0)
📝 uppsrc/Core/Value.h (+6 -0)
📝 uppsrc/Core/src.tpp/Value_en-us.tpp (+4 -0)

📄 Description

This PR add size_t as a supported type in Value class. Moreover, it added proper documentation and fix missing documentation for float type.

Motivation:

#include <Core/Core.h>

using namespace Upp;

CONSOLE_APP_MAIN
{
	Value a = 20;
	Value b = size_t{20}; // Not possible without change
	Value c = sizeof(int); // Not possible without change
	
	std::vector<int> vec1 = { 1, 2, 3};
	Value d = vec1.size(); // Not possible without change
}

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/ultimatepp/ultimatepp/pull/310 **Author:** [@klugier](https://github.com/klugier) **Created:** 10/16/2025 **Status:** ❌ Closed **Base:** `master` ← **Head:** `klugier/value-size-t` --- ### 📝 Commits (3) - [`c0367e5`](https://github.com/ultimatepp/ultimatepp/commit/c0367e5d567c9d1fb2acc8b06746c1c52337272b) Core: Added size_t as a Value supported type. - [`76e4609`](https://github.com/ultimatepp/ultimatepp/commit/76e4609793f04d2b7fe101b26cd81ab697716023) .cosmetics - [`a576d0f`](https://github.com/ultimatepp/ultimatepp/commit/a576d0fa3a8d5fbb13b2b9b4140aa7773a930695) .cosmetics ### 📊 Changes **4 files changed** (+22 additions, -0 deletions) <details> <summary>View changed files</summary> 📝 `uppsrc/Core/Defs.h` (+3 -0) 📝 `uppsrc/Core/Value.cpp` (+9 -0) 📝 `uppsrc/Core/Value.h` (+6 -0) 📝 `uppsrc/Core/src.tpp/Value_en-us.tpp` (+4 -0) </details> ### 📄 Description This PR add size_t as a supported type in Value class. Moreover, it added proper documentation and fix missing documentation for float type. Motivation: ``` #include <Core/Core.h> using namespace Upp; CONSOLE_APP_MAIN { Value a = 20; Value b = size_t{20}; // Not possible without change Value c = sizeof(int); // Not possible without change std::vector<int> vec1 = { 1, 2, 3}; Value d = vec1.size(); // Not possible without change } ``` --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

gitea-mirror

2026-05-05 03:45:12 -06:00

closed this issue
added the
pull-request
label

No labels

pull-request

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: github-starred/ultimatepp#303

No description provided.

Rows
Columns