remove deprecation warning on get_contiguous_memory_format (#37963)

suo · facebook-github-bot · commit 4e93844ab168 · 2020-05-07T02:06:22.000-07:00
Summary: Pull Request resolved: #37963 This function is still widely used in the codebase, so we don't want to add noise to builds with a bunch of warnings. Seems like the comment + macro are already pretty good indications that this functionality is considered legacy Test Plan: Imported from OSS Differential Revision: D21434447 Pulled By: suo fbshipit-source-id: 08162ed6502894ea5d3ccb92dfa0183232cc2ab5
diff --git a/c10/core/MemoryFormat.h b/c10/core/MemoryFormat.h
@@ -31,7 +31,7 @@ enum class MemoryFormat : int8_t { Contiguous, Preserve, ChannelsLast, ChannelsL
 // behaviour of contiguous
 #define LEGACY_CONTIGUOUS_MEMORY_FORMAT c10::get_contiguous_memory_format()
 
-C10_DEPRECATED inline MemoryFormat get_contiguous_memory_format() {
+inline MemoryFormat get_contiguous_memory_format() {
   return MemoryFormat::Contiguous;
 }
 

Original file line number	Diff line number	Diff line change
`@@ -31,7 +31,7 @@ enum class MemoryFormat : int8_t { Contiguous, Preserve, ChannelsLast, ChannelsL`
`31`	`31`	`// behaviour of contiguous`
`32`	`32`	`#define LEGACY_CONTIGUOUS_MEMORY_FORMAT c10::get_contiguous_memory_format()`
`33`	`33`
`34`		`-C10_DEPRECATED inline MemoryFormat get_contiguous_memory_format() {`
	`34`	`+inline MemoryFormat get_contiguous_memory_format() {`
`35`	`35`	`return MemoryFormat::Contiguous;`
`36`	`36`	`}`
`37`	`37`