New object exporter not save as right name #95044

New Issue

yonghao lv · 2022-01-19T09:01:03+01:00

yonghao lv commented

2022-01-19 09:01:03 +01:00

System Information
Operating system: Windows-10-10.0.22000-SP0 64 Bits
Graphics card: NVIDIA GeForce RTX 2080 SUPER/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 472.84

Blender Version
Broken: version: 3.1.0 Alpha, branch: master, commit date: 2022-01-18 22:23, hash: 2e5aecf557
Worked: (newest version of Blender that worked as expected)

Short description of error
export an object call 测试.obj
and it save as 娴嬭瘯.obj
export 中文.obj
return 涓枃.obj

I think it cause by the encoding of the win system(China use gbk on win by default)

Exact steps for others to reproduce the error
use the new c++ obj to export with non-English name

**System Information** Operating system: Windows-10-10.0.22000-SP0 64 Bits Graphics card: NVIDIA GeForce RTX 2080 SUPER/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 472.84 **Blender Version** Broken: version: 3.1.0 Alpha, branch: master, commit date: 2022-01-18 22:23, hash: `2e5aecf557` Worked: (newest version of Blender that worked as expected) **Short description of error** export an object call `测试.obj ` and it save as `娴嬭瘯.obj` export `中文.obj` return `涓枃.obj` I think it cause by the encoding of the win system(China use gbk on win by default) **Exact steps for others to reproduce the error** use the new c++ obj to export with non-English name

yonghao lv commented

2022-01-19 09:01:03 +01:00

Added subscriber: @1029910278

yonghao lv commented

2022-01-19 09:33:03 +01:00

Added subscriber: @howardt

Philipp Oeser commented

2022-01-19 09:54:54 +01:00

Added subscriber: @lichtwerk

Philipp Oeser commented

2022-01-19 09:54:55 +01:00

I am getting this on linux with the new exporter (displayed in Gnome Files 41.1) -- which sounds about right:

I am getting this on linux with the new exporter (displayed in Gnome Files 41.1) -- which sounds about right: ![image.png](https://archive.blender.org/developer/F12811213/image.png)

Jesse Yurkovich commented

2022-01-19 10:24:06 +01:00

Added subscriber: @deadpin

Jesse Yurkovich commented

2022-01-19 10:24:06 +01:00

Here on Windows (en-us defaults), I'm seeing something that is probably related but different at the same time. It looks like the filepath used in the exporter is fed directly into the fopen api without regard to Window's UTF16 setup.

Trying to export 测试.obj will yield the following for me:

Here on Windows (en-us defaults), I'm seeing something that is probably related but different at the same time. It looks like the filepath used in the exporter is fed directly into the `fopen` api without regard to Window's UTF16 setup. Trying to export `测试.obj` will yield the following for me: ![unicode-obj-export.png](https://archive.blender.org/developer/F12811255/unicode-obj-export.png)

Philipp Oeser commented

2022-01-19 13:02:26 +01:00

Changed status from 'Needs Triage' to: 'Confirmed'

Philipp Oeser commented

2022-01-19 13:02:26 +01:00

Added subscriber: @ankitm

Philipp Oeser commented

2022-01-19 13:02:26 +01:00

I'll take this confirmed for now.

CC @ankitm
CC @howardt

I'll take this confirmed for now. CC @ankitm CC @howardt

Ray molenkamp commented

2022-01-19 19:06:24 +01:00

Added subscriber: @LazyDodo

Ray molenkamp commented

2022-01-19 19:06:24 +01:00

If someone wants an easy issue, FileHandler's ctor in source\blender\io\wavefront_obj\exporter\obj_export_io.hh needs to call BLI_fopen rather than std::fopen and this should be resolved.

If someone wants an easy issue, `FileHandler`'s ctor in `source\blender\io\wavefront_obj\exporter\obj_export_io.hh` needs to call `BLI_fopen` rather than `std::fopen` and this *should* be resolved.

Aras Pranckevicius commented

2022-01-20 09:28:09 +01:00

Added subscriber: @aras_p

Aras Pranckevicius commented

2022-01-20 09:28:09 +01:00

I can take this as an exercise in how to make a codebase change

Aras Pranckevicius was assigned by Ankit Meel

2022-01-20 09:40:16 +01:00

Ankit Meel commented

2022-01-20 09:40:16 +01:00

In #95044#1291536, @aras_p wrote:
I can take this as an exercise in how to make a codebase change

what a nice crossover
context: https:*github.com/aras-p/ClangBuildAnalyzer/issues/79 https:*github.com/z-aki/buildanalyzeronblender

> In #95044#1291536, @aras_p wrote: > I can take this as an exercise in how to make a codebase change what a nice crossover context: https:*github.com/aras-p/ClangBuildAnalyzer/issues/79 https:*github.com/z-aki/buildanalyzeronblender

blender-admin commented

2022-01-22 02:14:01 +01:00

This issue was referenced by 9350005d8b

This issue was referenced by 9350005d8b56a831f4c592d58fdf190af64efad4

Ankit Meel commented

2022-01-22 08:16:10 +01:00

Changed status from 'Confirmed' to: 'Resolved'

Ankit Meel closed this issue

2022-01-22 08:16:10 +01:00

Ankit Meel commented

2022-01-22 08:16:10 +01:00

9350005d8b