Skip to content

Commit

Permalink
[Feature #19757] Add new API rb_data_define
Browse files Browse the repository at this point in the history
  • Loading branch information
@nobu
nobu committed Jul 13, 2023
1 parent d426343 commit 9c1fe90
Show file tree
Hide file tree
Showing 5 changed files with 216 additions and 0 deletions.
13 changes: 13 additions & 0 deletions ext/-test-/struct/data.c
View file
@@ -0,0 +1,13 @@
#include "ruby.h"

static VALUE
bug_data_new(VALUE self, VALUE super)
{
return rb_data_define(super, "mem1", "mem2", NULL);
}

void
Init_data(VALUE klass)
{
rb_define_singleton_method(klass, "data_new", bug_data_new, 1);
}
159 changes: 159 additions & 0 deletions ext/-test-/struct/depend
View file
@@ -1,4 +1,163 @@
# AUTOGENERATED DEPENDENCIES START
data.o: $(RUBY_EXTCONF_H)
data.o: $(arch_hdrdir)/ruby/config.h
data.o: $(hdrdir)/ruby.h
data.o: $(hdrdir)/ruby/assert.h
data.o: $(hdrdir)/ruby/backward.h
data.o: $(hdrdir)/ruby/backward/2/assume.h
data.o: $(hdrdir)/ruby/backward/2/attributes.h
data.o: $(hdrdir)/ruby/backward/2/bool.h
data.o: $(hdrdir)/ruby/backward/2/inttypes.h
data.o: $(hdrdir)/ruby/backward/2/limits.h
data.o: $(hdrdir)/ruby/backward/2/long_long.h
data.o: $(hdrdir)/ruby/backward/2/stdalign.h
data.o: $(hdrdir)/ruby/backward/2/stdarg.h
data.o: $(hdrdir)/ruby/defines.h
data.o: $(hdrdir)/ruby/intern.h
data.o: $(hdrdir)/ruby/internal/abi.h
data.o: $(hdrdir)/ruby/internal/anyargs.h
data.o: $(hdrdir)/ruby/internal/arithmetic.h
data.o: $(hdrdir)/ruby/internal/arithmetic/char.h
data.o: $(hdrdir)/ruby/internal/arithmetic/double.h
data.o: $(hdrdir)/ruby/internal/arithmetic/fixnum.h
data.o: $(hdrdir)/ruby/internal/arithmetic/gid_t.h
data.o: $(hdrdir)/ruby/internal/arithmetic/int.h
data.o: $(hdrdir)/ruby/internal/arithmetic/intptr_t.h
data.o: $(hdrdir)/ruby/internal/arithmetic/long.h
data.o: $(hdrdir)/ruby/internal/arithmetic/long_long.h
data.o: $(hdrdir)/ruby/internal/arithmetic/mode_t.h
data.o: $(hdrdir)/ruby/internal/arithmetic/off_t.h
data.o: $(hdrdir)/ruby/internal/arithmetic/pid_t.h
data.o: $(hdrdir)/ruby/internal/arithmetic/short.h
data.o: $(hdrdir)/ruby/internal/arithmetic/size_t.h
data.o: $(hdrdir)/ruby/internal/arithmetic/st_data_t.h
data.o: $(hdrdir)/ruby/internal/arithmetic/uid_t.h
data.o: $(hdrdir)/ruby/internal/assume.h
data.o: $(hdrdir)/ruby/internal/attr/alloc_size.h
data.o: $(hdrdir)/ruby/internal/attr/artificial.h
data.o: $(hdrdir)/ruby/internal/attr/cold.h
data.o: $(hdrdir)/ruby/internal/attr/const.h
data.o: $(hdrdir)/ruby/internal/attr/constexpr.h
data.o: $(hdrdir)/ruby/internal/attr/deprecated.h
data.o: $(hdrdir)/ruby/internal/attr/diagnose_if.h
data.o: $(hdrdir)/ruby/internal/attr/enum_extensibility.h
data.o: $(hdrdir)/ruby/internal/attr/error.h
data.o: $(hdrdir)/ruby/internal/attr/flag_enum.h
data.o: $(hdrdir)/ruby/internal/attr/forceinline.h
data.o: $(hdrdir)/ruby/internal/attr/format.h
data.o: $(hdrdir)/ruby/internal/attr/maybe_unused.h
data.o: $(hdrdir)/ruby/internal/attr/noalias.h
data.o: $(hdrdir)/ruby/internal/attr/nodiscard.h
data.o: $(hdrdir)/ruby/internal/attr/noexcept.h
data.o: $(hdrdir)/ruby/internal/attr/noinline.h
data.o: $(hdrdir)/ruby/internal/attr/nonnull.h
data.o: $(hdrdir)/ruby/internal/attr/noreturn.h
data.o: $(hdrdir)/ruby/internal/attr/packed_struct.h
data.o: $(hdrdir)/ruby/internal/attr/pure.h
data.o: $(hdrdir)/ruby/internal/attr/restrict.h
data.o: $(hdrdir)/ruby/internal/attr/returns_nonnull.h
data.o: $(hdrdir)/ruby/internal/attr/warning.h
data.o: $(hdrdir)/ruby/internal/attr/weakref.h
data.o: $(hdrdir)/ruby/internal/cast.h
data.o: $(hdrdir)/ruby/internal/compiler_is.h
data.o: $(hdrdir)/ruby/internal/compiler_is/apple.h
data.o: $(hdrdir)/ruby/internal/compiler_is/clang.h
data.o: $(hdrdir)/ruby/internal/compiler_is/gcc.h
data.o: $(hdrdir)/ruby/internal/compiler_is/intel.h
data.o: $(hdrdir)/ruby/internal/compiler_is/msvc.h
data.o: $(hdrdir)/ruby/internal/compiler_is/sunpro.h
data.o: $(hdrdir)/ruby/internal/compiler_since.h
data.o: $(hdrdir)/ruby/internal/config.h
data.o: $(hdrdir)/ruby/internal/constant_p.h
data.o: $(hdrdir)/ruby/internal/core.h
data.o: $(hdrdir)/ruby/internal/core/rarray.h
data.o: $(hdrdir)/ruby/internal/core/rbasic.h
data.o: $(hdrdir)/ruby/internal/core/rbignum.h
data.o: $(hdrdir)/ruby/internal/core/rclass.h
data.o: $(hdrdir)/ruby/internal/core/rdata.h
data.o: $(hdrdir)/ruby/internal/core/rfile.h
data.o: $(hdrdir)/ruby/internal/core/rhash.h
data.o: $(hdrdir)/ruby/internal/core/robject.h
data.o: $(hdrdir)/ruby/internal/core/rregexp.h
data.o: $(hdrdir)/ruby/internal/core/rstring.h
data.o: $(hdrdir)/ruby/internal/core/rstruct.h
data.o: $(hdrdir)/ruby/internal/core/rtypeddata.h
data.o: $(hdrdir)/ruby/internal/ctype.h
data.o: $(hdrdir)/ruby/internal/dllexport.h
data.o: $(hdrdir)/ruby/internal/dosish.h
data.o: $(hdrdir)/ruby/internal/error.h
data.o: $(hdrdir)/ruby/internal/eval.h
data.o: $(hdrdir)/ruby/internal/event.h
data.o: $(hdrdir)/ruby/internal/fl_type.h
data.o: $(hdrdir)/ruby/internal/gc.h
data.o: $(hdrdir)/ruby/internal/glob.h
data.o: $(hdrdir)/ruby/internal/globals.h
data.o: $(hdrdir)/ruby/internal/has/attribute.h
data.o: $(hdrdir)/ruby/internal/has/builtin.h
data.o: $(hdrdir)/ruby/internal/has/c_attribute.h
data.o: $(hdrdir)/ruby/internal/has/cpp_attribute.h
data.o: $(hdrdir)/ruby/internal/has/declspec_attribute.h
data.o: $(hdrdir)/ruby/internal/has/extension.h
data.o: $(hdrdir)/ruby/internal/has/feature.h
data.o: $(hdrdir)/ruby/internal/has/warning.h
data.o: $(hdrdir)/ruby/internal/intern/array.h
data.o: $(hdrdir)/ruby/internal/intern/bignum.h
data.o: $(hdrdir)/ruby/internal/intern/class.h
data.o: $(hdrdir)/ruby/internal/intern/compar.h
data.o: $(hdrdir)/ruby/internal/intern/complex.h
data.o: $(hdrdir)/ruby/internal/intern/cont.h
data.o: $(hdrdir)/ruby/internal/intern/dir.h
data.o: $(hdrdir)/ruby/internal/intern/enum.h
data.o: $(hdrdir)/ruby/internal/intern/enumerator.h
data.o: $(hdrdir)/ruby/internal/intern/error.h
data.o: $(hdrdir)/ruby/internal/intern/eval.h
data.o: $(hdrdir)/ruby/internal/intern/file.h
data.o: $(hdrdir)/ruby/internal/intern/hash.h
data.o: $(hdrdir)/ruby/internal/intern/io.h
data.o: $(hdrdir)/ruby/internal/intern/load.h
data.o: $(hdrdir)/ruby/internal/intern/marshal.h
data.o: $(hdrdir)/ruby/internal/intern/numeric.h
data.o: $(hdrdir)/ruby/internal/intern/object.h
data.o: $(hdrdir)/ruby/internal/intern/parse.h
data.o: $(hdrdir)/ruby/internal/intern/proc.h
data.o: $(hdrdir)/ruby/internal/intern/process.h
data.o: $(hdrdir)/ruby/internal/intern/random.h
data.o: $(hdrdir)/ruby/internal/intern/range.h
data.o: $(hdrdir)/ruby/internal/intern/rational.h
data.o: $(hdrdir)/ruby/internal/intern/re.h
data.o: $(hdrdir)/ruby/internal/intern/ruby.h
data.o: $(hdrdir)/ruby/internal/intern/select.h
data.o: $(hdrdir)/ruby/internal/intern/select/largesize.h
data.o: $(hdrdir)/ruby/internal/intern/signal.h
data.o: $(hdrdir)/ruby/internal/intern/sprintf.h
data.o: $(hdrdir)/ruby/internal/intern/string.h
data.o: $(hdrdir)/ruby/internal/intern/struct.h
data.o: $(hdrdir)/ruby/internal/intern/thread.h
data.o: $(hdrdir)/ruby/internal/intern/time.h
data.o: $(hdrdir)/ruby/internal/intern/variable.h
data.o: $(hdrdir)/ruby/internal/intern/vm.h
data.o: $(hdrdir)/ruby/internal/interpreter.h
data.o: $(hdrdir)/ruby/internal/iterator.h
data.o: $(hdrdir)/ruby/internal/memory.h
data.o: $(hdrdir)/ruby/internal/method.h
data.o: $(hdrdir)/ruby/internal/module.h
data.o: $(hdrdir)/ruby/internal/newobj.h
data.o: $(hdrdir)/ruby/internal/scan_args.h
data.o: $(hdrdir)/ruby/internal/special_consts.h
data.o: $(hdrdir)/ruby/internal/static_assert.h
data.o: $(hdrdir)/ruby/internal/stdalign.h
data.o: $(hdrdir)/ruby/internal/stdbool.h
data.o: $(hdrdir)/ruby/internal/symbol.h
data.o: $(hdrdir)/ruby/internal/value.h
data.o: $(hdrdir)/ruby/internal/value_type.h
data.o: $(hdrdir)/ruby/internal/variable.h
data.o: $(hdrdir)/ruby/internal/warning_push.h
data.o: $(hdrdir)/ruby/internal/xmalloc.h
data.o: $(hdrdir)/ruby/missing.h
data.o: $(hdrdir)/ruby/ruby.h
data.o: $(hdrdir)/ruby/st.h
data.o: $(hdrdir)/ruby/subst.h
data.o: data.c
duplicate.o: $(RUBY_EXTCONF_H)
duplicate.o: $(arch_hdrdir)/ruby/config.h
duplicate.o: $(hdrdir)/ruby.h
Expand Down
14 changes: 14 additions & 0 deletions include/ruby/internal/intern/struct.h
View file
Expand Up @@ -198,6 +198,20 @@ RBIMPL_ATTR_NONNULL((2))
*/
VALUE rb_struct_define_without_accessor_under(VALUE outer, const char *class_name, VALUE super, rb_alloc_func_t alloc, ...);

/**
* Defines an anonymous data class.
*
* @endinternal
*
* @param[in] super Superclass of the defining class. Must be a
* descendant of ::rb_cData, or 0 as ::rb_cData.
* @param[in] ... Arbitrary number of `const char*`, terminated by
* NULL. Each of which are the name of fields.
* @exception rb_eArgError Duplicated field name.
* @return The defined class.
*/
VALUE rb_data_define(VALUE super, ...);

RBIMPL_SYMBOL_EXPORT_END()

#endif /* RBIMPL_INTERN_STRUCT_H */
12 changes: 12 additions & 0 deletions struct.c
View file
Expand Up @@ -1722,6 +1722,18 @@ rb_data_s_def(int argc, VALUE *argv, VALUE klass)
return data_class;
}

VALUE
rb_data_define(VALUE super, ...)
{
va_list ar;
VALUE ary;
va_start(ar, super);
ary = struct_make_members_list(ar);
va_end(ar);
if (!super) super = rb_cData;
return setup_data(anonymous_struct(super), ary);
}

/*
* call-seq:
* DataClass::members -> array_of_symbols
Expand Down
18 changes: 18 additions & 0 deletions test/-ext-/struct/test_data.rb
View file
@@ -0,0 +1,18 @@
# frozen_string_literal: false
require 'test/unit'
require "-test-/struct"

class Bug::Struct::Test_Data < Test::Unit::TestCase
def test_data_new_default
klass = Bug::Struct.data_new(false)
assert_equal Data, klass.superclass
assert_equal %i[mem1 mem2], klass.members
end

def test_data_new_superclass
superclass = Data.define
klass = Bug::Struct.data_new(superclass)
assert_equal superclass, klass.superclass
assert_equal %i[mem1 mem2], klass.members
end
end

0 comments on commit 9c1fe90

Please sign in to comment.