Skip to content
This repository was archived by the owner on Sep 6, 2021. It is now read-only.

Commit ed0cf7e

Browse files
committed
changes for code review
1 parent 00fca07 commit ed0cf7e

2 files changed

Lines changed: 3 additions & 3 deletions

File tree

src/editor/Editor.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -909,10 +909,10 @@ define(function (require, exports, module) {
909909
* Returns the string-based pos for a given display column (zero-based) in given line. Differs from column
910910
* only when the line contains preceding \t chars. Result depends on the current tab size setting.
911911
* @param {number} lineNum Line number
912-
* @param {number} colum Display column number
912+
* @param {number} column Display column number
913913
* @return {number}
914914
*/
915-
Editor.prototype.getPosOffset = function (lineNum, column) {
915+
Editor.prototype.getCharIndexForColumn = function (lineNum, column) {
916916
var line = this._codeMirror.getLine(lineNum),
917917
tabSize = null,
918918
iCol = 0,

src/editor/EditorCommandHandlers.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -992,7 +992,7 @@ define(function (require, exports, module) {
992992
pos.line += dir;
993993

994994
// translate column to ch in line of new selection
995-
pos.ch = editor.getPosOffset(pos.line, colOffset);
995+
pos.ch = editor.getCharIndexForColumn(pos.line, colOffset);
996996

997997

998998
// If this is the primary selection, we want the new cursor we're adding to become the

0 commit comments

Comments
 (0)